Loko Air BOM

Will the BOM for Air be made available?

@tyagodani   It is added now, forgotten

@tomipiriyev thanks a lot!
By the way, been looking at all the parts and ti seems that the PA1010D GPS module has been discontinued. Really can't seem to find it anywhere. Is there another module that came to replace this one? Or where do you get it from? Cheers

This one has Lora and GPS integrated: https://www.seeedstudio.com/Wio-WM1110-Module-LR1110-and-nRF52840-p-5676.html
Seems like the project could be revised to use that, no?

@tyagodani   gps you can find from https://www.mirifica.de/en/cd-top/mediatek-mt3333/cdtop-technology-pa1010dv2-cd-pa1010d2_100011_1327.
LR1110 can't be used, it does processing on LoraCLound , not in the mcu to save power

Cool, thanks @tomipiriyev! One more question: on the link you sent there are two version. "Version 1" and "GPS+GLONASS". Which one should I wonder, if there is even a difference?
Thank you

@tyagodani  yes it has differences,   it is better to use GPS+Galileo+Glonass,  to have faster search . but I haven't made hands on comparison of these two versions . we use v1

@tomipiriyev thanks for the briefing. Already ordered!
One last thing, if you wouldn't mind. I'm thinking of ordering the PCB with the smaller SMD components already assembled and I'll solder things like the GPS and Lora modules and the port myself (I still don't have the proper equipment to solder the small SMD stuff nor the practice). Could you please upload the CPL Pick&Place file?
Thank you once again!

@tyagodani  uploaded

Thank you and thanks for the tips!

Hello, I wanted to ask about GPS module if it's possible to use the Quectel L86 since is easier to find and also cheaper. From technical parameters, seems to have the same ones. Maybe for the programming part, firmware need to be adapted, but I think it's not a big problem. What you think?

hi @marksdemellin  Quectel L86  is 4 times bigger than the current module we use.
We are also looking for low price Chinese alternatives to use , to lower the production cost.
incase if I find something suitable , I will notify you .  currently the size , weight , power consumption's are the main criteria for such a small product.


Bump min rust version to 1.30.0

Some dependiencies rely on cc, which after
https://github.com/alexcrichton/cc-rs/commit/228513449f3e2674cdce0f53206c9449af9d3778
requires 1.30.0.

Codecov Report

Merging #297 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #297      +/-   ##
==========================================
+ Coverage   87.17%   87.17%   +<.01%
==========================================
  Files         101      101
  Lines       14414    14411       -3
==========================================
- Hits        12565    12563       -2
+ Misses       1849     1848       -1

Impacted Files
Coverage Δ

juniper/src/validation/rules/no_fragment_cycles.rs
89.74% <0%> (-0.07%)
:arrow_down:

juniper/src/tests/model.rs
91.24% <0%> (-0.07%)
:arrow_down:

juniper/src/schema/meta.rs
87.97% <0%> (+0.25%)
:arrow_up:

Continue to review full report at Codecov.

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


Feature request: add DEPRECATED selector

ruff check --isolated --select=ALL selects ANN101 which is deprecated. I would love to be able to do ruff check --isolated --select=ALL --ignore=DEPRECATED. (I know I could do ruff check --isolated --select=ALL --preview, but I don't want to add all preview rules.)

Another great selector would be INCOMPATIBLE_WITH_FORMATTER, including (for example) COM812.

Sounds reasonable to me. I rename this back to DEPRECATED and track the formatter incompatible rules in https://github.com/astral-sh/ruff/issues/10225

I think we should do https://github.com/astral-sh/ruff/issues/10342 instead

By the way, if you don't want to enable all preview rules, you can use explicit-preview-rules.


chore: Add Multi Pane Layout feature flag for specific emails

Add features flag for Multi Pane Layouts for only a specific email ids

/ok-to-test


Added supporting SSO logon for consoles

If VM has SSO method 'guest_agent', then it will try to logon, if there will be an error, then user get modal window:

Fixes: https://github.com/oVirt/ovirt-web-ui/issues/65

@sjd78 Yes, logon request work for users (tested on brq-dev).

ci build please

ci build please

https://jenkins.ovirt.org/job/oVirt_ovirt-web-ui_standard-check-pr/188/artifact/build-artifacts.fc28.x86_64/ovirt-web-ui-1.5.0-0.20181109git19cd4f2.fc28.noarch.rpm

https://jenkins.ovirt.org/job/oVirt_ovirt-web-ui_standard-check-pr/188/artifact/build-artifacts.el7.x86_64/ovirt-web-ui-1.5.0-0.20181109git19cd4f2.el7.noarch.rpm

ci build please

ci build please

[need to fix the build]

ci build please

[need to fix the build]

fixed!

LGTM


Saving robot player code

Would users request the ability to save their projects? Would robot players then require players to have their own login?

I don't think this is required. We want it to be easy for players to join and we have previously decided that login is only for teachers.
In the current implementation, the player code is saved locally in the browser's localStorage when you press the "auto-play" button. Instead of the database + login solution, a simple solution would be to add a "Don't forget to save your code"-warning on the player page after a robot-game has ended + perhaps a "download your code"-button


MapReduce test fixes - second attempt

Made community detection algorithm deterministic in case of ties (implementations on other platforms already used this deterministic tie breaker).
Altered forest fire model implementation to ignore the newly created vertices when selecting links to follow. Not all platforms support alteration of the graph structure during algorithm execution, so to introduce consistency all platforms should "pretend" that the new vertices do not exist until after the algorithm has finished.

Closes #48

Refer to this link for build results (access rights to CI server needed):
http://jenkins.tribler.org//job/Graphalytics_pull_request_tester/23/
Test PASSed.


PlotWidget.pickItems returns objects

This PR makes PlotWidget.pickItems and items' pick method returns a PickingResult object rather than a tuple (item, indices) or indices. This enables to provide richer picking information in the future without breaking the API and is consistent with the picking API of plot3d.
The PickingResult class of plot3d inherits from the new one of plot and extends it as more features are available for now in plot3d picking.

LGTM


feat(jetbrains-gateway): add rider support

Add Jetbrains Rider as a supported IDE
NOTE: This will download https://download.jetbrains.com/rider/JetBrains.Rider-2023.3.2.tar.gz, which is not the latest.
closes #187

Thank you @michaelbrewer.
Are you planning on updating IDE versions? I can wait to push a new release till then.


[BUG] Intermittent issue with longhorn manager pods having tolerations on upgrade

Describe the bug
The longhorn manager pods initially fail with error crashloop and then recover after sometime on upgrade longhorn from v1.0.2 to v1.1.0 if there are some tolerations.
To Reproduce
Steps to reproduce the behavior:

Install Longhorn v1.0.2.
Set some tolerations like key1=value1:NoSchedule; key2:NoExecute
Wait for the longhorn pods to be redeployed.
Upgrade Longhorn to master.
The longhorn pods goes to crashloop initially and then come up healthy.

Expected behavior
There shouldn't be any error in the longhorn pods.
Log
time="2020-11-28T07:39:24Z" level=info msg="Start overwriting built-in settings with customized values"
W1128 07:39:24.948114       1 client_config.go:541] Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This might not work.
time="2020-11-28T07:39:24Z" level=info msg="cannot list the content of the src directory /var/lib/rancher/longhorn/engine-binaries for the copy, will do nothing: Failed to execute: nsenter [--mount=/host/proc/19196/ns/mnt --net=/host/proc/19196/ns/net bash -c ls /var/lib/rancher/longhorn/engine-binaries/*], output , stderr, ls: cannot access '/var/lib/rancher/longhorn/engine-binaries/*': No such file or directory\n, error exit status 2"
I1128 07:39:24.964600       1 leaderelection.go:241] attempting to acquire leader lease  longhorn-system/longhorn-manager-upgrade-lock...
I1128 07:39:24.992407       1 leaderelection.go:251] successfully acquired lease longhorn-system/longhorn-manager-upgrade-lock
time="2020-11-28T07:39:24Z" level=info msg="Start upgrading"
time="2020-11-28T07:39:25Z" level=info msg="No API version upgrade is needed"
time="2020-11-28T07:39:25Z" level=error msg="Upgrade failed: upgrade Pods failed: upgrade from v1.0.2 to v1.1.0: upgrade instance manager pods failed: failed to update toleration annotation for instance manager pod instance-manager-r-f0a4823d: Operation cannot be fulfilled on pods \"instance-manager-r-f0a4823d\": the object has been modified; please apply your changes to the latest version and try again"
time="2020-11-28T07:39:25Z" level=info msg="Upgrade leader lost: khush-test-lh-wk2"
time="2020-11-28T07:39:25Z" level=fatal msg="Error starting manager: upgrade Pods failed: upgrade from v1.0.2 to v1.1.0: upgrade instance manager pods failed: failed to update toleration annotation for instance manager pod instance-manager-r-f0a4823d: Operation cannot be fulfilled on pods \"instance-manager-r-f0a4823d\": the object has been modified; please apply your changes to the latest version and try again"

Environment:

Longhorn version: Longhorn master
Kubernetes version: v19.2
Node config

OS type and version - Ubuntu 18.04
CPU per node: 2 vcpus
Memory per node: 4 GB

Verified with longhorn master - 12/01/2020
Validation - Pass
No intermittent issue is observed on upgrading a longhorn set up having tolerations.


Fixing package.json for windows

Added fix for windows as core package json would give issues due to the cp command not existing on windows. Along with this change added new scripts to packages/mirrorful and removed removeFileOrDirectory.ts as it is no longer needed.

Oh very interesting, have not heard of https://www.npmjs.com/package/run-script-os before.
I think run-script-os should be a dev dependency tho, and installed with npm install --save-dev run-script-os.
Looks like you have it installed a regular dependency. Could you fix that @karanvirsb  ? Thanks for opening this!

Oh very interesting, have not heard of https://www.npmjs.com/package/run-script-os before.
I think run-script-os should be a dev dependency tho, and installed with yard add -D run-script-os.
Looks like you have it installed a regular dependency. Could you fix that @karanvirsb ? Thanks for opening this!

I have fixed it and moved it into the dev dependencies!

Great, we'll get this merged in today!


Security test case: Fake authentication headers

Added security test case in which a client sends fake authentication headers.
Prerequisites:
CountryA has access to the services
CountryC does NOT have access to the services but has a client cert from a trusted CA
Test case:
CountryC somehow gets hold of the public part of CountryA's auth cert and fakes the ingress headers

PR points to old test suite, must be included in new one instead


Inaccurate sync percentage

Problem details

The sync percentage is inaccurate. Currently it uses the current and estimated highest block from this API: https://eth.wiki/json-rpc/API#eth_syncing

It doesn't look like there is a solution for this. The best we can get is a rough, inaccurate estimate an just let the user know the sync percentage is an estimate. I briefly confirmed  this with a nethermind dev as well.


Require PROJ.4 compatible horizontal and vertical projection declarations.

For any software to accurately interoperate with a geospatial dataset it must be given or make an assumption about the datum and projection used for the geospatial content. It is unacceptable to omit this information regardless of the scale or intended use of the data. Specification of the reference datum (horizontal and vertical) and projection (as applicable to the dimensionality of the data) should be a requirement akin to inclusion of units for coordinate variables.  If the requirement for a dataset to include such metadata is considered too onerous for data producers who are unfamiliar with the datum their data uses, the CF community should adopt a default lat/lon/elevation datum and encourage software producers to standardize on that datum to foster consistency across the community. What default to use should be determined in consultation with the National Geodetic Survey and their counterparts internationally.
Proj.4 has been the de facto implementation of coordinate transformations, more or less, since the beginning of digital geospatial data. The ability to integrate CF-described geospatially referenced data with tools that implement the Proj.4 projection libraries is important.
Conversion of geospatial data into CF-described files requires CF support for the prevailing set of projections and reference datums.
Use of identifiers from the EPSG naming authority and conventions consistent with OGC-WKT should be supported. The issue that forces this assertion is the need for 'shift grids' to convert to/from non-parametric datums. This is of particular importance for vertical datums but is also important for the common NADCON conversion to/from the NAD27 datum.
In practice, codes defined by the EPSG naming authority, encoded either alone or as part of a WKT datum/projection declaration, are necessary for integration of data with web services and for conversion to and from other formats. Geospatial applications that desire to interoperate with CF should not be forced to construct utilities like this one.. This leads to the conclusion that proj.4 strings, EPSG codes, or WKT projections should be allowed for specification of projections.

Moving this here for posterity. I think it was fixed in: http://cfconventions.org/cf-conventions/cf-conventions.html#use-of-the-crs-well-known-text-format


Error compiling deps since last update, not sure....

Environment

Elixir version (elixir -v): 1.5.1
Database and version (PostgreSQL 9.4, MongoDB 3.2, etc.): PG 9.6
Ecto version (mix deps): 2.2.6
Database adapter and version (mix deps): postgrex, "0.13.3
Operating system: OSX sierra

Current behavior
Hi,
I was going to update a personal project that was built when phoenix 1.3 was just released, so when I run:
mix deps.get && mix deps.compile I get a error from a Hex package called apartmentex with the following output:
== Compilation error in file lib/apartmentex/migrations_path_builder.ex ==
** (CompileError) lib/apartmentex/migrations_path_builder.ex:2: cannot import Mix.Ecto.build_repo_priv/1 because it is undefined or private
    (elixir) src/elixir_import.erl:77: :elixir_import.calculate/6
    (elixir) src/elixir_import.erl:18: :elixir_import.import/4
could not compile dependency :apartmentex, "mix compile" failed. You can recompile this dependency with "mix deps.compile apartmentex", update it with "mix deps.update apartmentex" or clean it with "mix deps.clean apartmentex"

Since the lib didn't changed from there to here I must assume that this maybe have something to do with the recent changes in Ecto, specially about this line:
 (CompileError) lib/apartmentex/migrations_path_builder.ex:2: cannot import Mix.Ecto.build_repo_priv/1 because it is undefined or private
This lib was fine since then, so I tried to look for changes in Ecto and Phoenix Ecto to see if Mix.Ecto.build_repo_priv/1 exists but couldn't find it so I could use some help here.
I tried to built a new mix phx.new my_project and add {:apartmentex, "~> 0.2.3"} to mix.exs but the same error goes in the mix deps.compile but no success.
The lib block that origins the error:
defmodule Apartmentex.MigrationsPathBuilder do
  import Mix.Ecto, only: [build_repo_priv: 1]
  @migrations_folder Application.get_env(:apartmentex, :migrations_folder) || "tenant_migrations"

  def tenant_migrations_path(repo) do
    Path.join(build_repo_priv(repo), @migrations_folder)
  end
end

So any help here would be appreciated.
Thanks.

Apartmentex was using private modules of Ecto. We changed our private modules, as we are allowed to do, and that broke Apartmentex. The solution is to copy the private functions Apartmentex was using to their repo (or at least file a report there). Thanks!

Thanks
Filipe Veloso
Diretor de Tecnologia e Operações
Tel. 55 11 2386-2001
Cel. 55 11 99195-3155
www.umanni.com.br
Responda a nossa pesquisa de satisfação:
https://goo.gl/forms/kYP9pK0mShvDgW7F2
2017-10-13 17:39 GMT-03:00 José Valim<EMAIL_ADDRESS>
Closed #2278 https://github.com/elixir-ecto/ecto/issues/2278.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/elixir-ecto/ecto/issues/2278#event-1292994515, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACWufPBswVrhBUQNA5oOmEWe1rWdn2ymks5sr8qdgaJpZM4P5ArY
.


APIconnection Error Error communicating with OpenAI

Hi buddy
My server and client have already up. My API-key is correct. Otherwise the server won't start.
I received a error Retrying langchain.chat_models.openai.acompletion_with_retry.._completion_with_retry in 16.0 seconds as it raised APIConnectionError: Error communicating with OpenAI.
I used proxy clash because I am in china, I have the access for chatgpt, Please help me how to resolve this issue

seems like you need to raise a ticket with the library that is used openai-whisper ?
https://github.com/openai/whisper

If seems related to the network situation with proxy/IP block thing. Would suggest to test if you can directly connect to whisper API, e.g. like this
curl https://api.openai.com/v1/audio/transcriptions \
  -H "Authorization: Bearer $OPENAI_API_KEY" \
  -H "Content-Type: multipart/form-data" \
  -F model="whisper-1" \
  -F<EMAIL_ADDRESS>

Closing for now, feel free to reopen.

If seems related to the network situation with proxy/IP block thing. Would suggest to test if you can directly connect to whisper API, e.g. like this
curl https://api.openai.com/v1/audio/transcriptions \
  -H "Authorization: Bearer $OPENAI_API_KEY" \
  -H "Content-Type: multipart/form-data" \
  -F model="whisper-1" \
  -F<EMAIL_ADDRESS>
If this doesn't work, it means you network has issue and can't connect to OpenAI

I also encountered the same problem, executing this command returned: curl: (26) Failed to open/read local data from file/application.
Does<EMAIL_ADDRESS>need to be modified? Or how to do it? Thanks!


Add support for user defined type guards

function isNumber(value: any): value is number {
    return typeof value === "number";
}

Typedoc renders as:
function isNumber(value: any): boolean {
    return typeof value === "number";
}

Would be nice to keep the type guard in the documentation instead of falling back on the normal type.

This is available as of 0.15.2


Source function is not able to get 'XML' elements and it is hitting the timeout of 30 seconds and crashing if a page contains lot of 'XML' elements.

Hello,
I am using your library. One of the page in the application I am using has a lot of 'XML' elements and I was trying to use the source method in BaseClient class to retrieve xml elements. I am getting timeout error since the connection timeout in the class is set to 30.0 seconds. Can you please provide an API/customizable timeout method to customize the timeout according to user needs.
I was able to write a function and change timeout for now. Thanks.

Change timeout also is crashing the app most times. It works only sometimes. To give you a scale of XML element characters I am trying to read through 'source' method. Characters: 153300, Line breaks : 1000.
Traceback (most recent call last):
File "C:\Python37\lib\site-packages\urllib3\connectionpool.py", line 600, in urlopen
chunked=chunked)
File "C:\Python37\lib\site-packages\urllib3\connectionpool.py", line 384, in _make_request
six.raise_from(e, None)
File "", line 2, in raise_from
File "C:\Python37\lib\site-packages\urllib3\connectionpool.py", line 380, in _make_request
httplib_response = conn.getresponse()
File "C:\Python37\lib\http\client.py", line 1344, in getresponse
response.begin()
File "C:\Python37\lib\http\client.py", line 306, in begin
version, status, reason = self._read_status()
File "C:\Python37\lib\http\client.py", line 275, in _read_status
raise RemoteDisconnected("Remote end closed connection without"
http.client.RemoteDisconnected: Remote end closed connection without response
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Python37\lib\site-packages\requests\adapters.py", line 449, in send
timeout=timeout
File "C:\Python37\lib\site-packages\urllib3\connectionpool.py", line 638, in urlopen
_stacktrace=sys.exc_info()[2])
File "C:\Python37\lib\site-packages\urllib3\util\retry.py", line 368, in increment
raise six.reraise(type(error), error, _stacktrace)
File "C:\Python37\lib\site-packages\urllib3\packages\six.py", line 685, in reraise
raise value.with_traceback(tb)
File "C:\Python37\lib\site-packages\urllib3\connectionpool.py", line 600, in urlopen
chunked=chunked)
File "C:\Python37\lib\site-packages\urllib3\connectionpool.py", line 384, in _make_request
six.raise_from(e, None)
File "", line 2, in raise_from
File "C:\Python37\lib\site-packages\urllib3\connectionpool.py", line 380, in _make_request
httplib_response = conn.getresponse()
File "C:\Python37\lib\http\client.py", line 1344, in getresponse
response.begin()
File "C:\Python37\lib\http\client.py", line 306, in begin
version, status, reason = self._read_status()
File "C:\Python37\lib\http\client.py", line 275, in _read_status
raise RemoteDisconnected("Remote end closed connection without"
urllib3.exceptions.ProtocolError: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response'))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Python37\lib\site-packages\wda\utils.py", line 47, in inner
return fn(*args, **kwargs)
File "C:\Python37\lib\site-packages\wda_init.py", line 460, in fetch
response = httpdo(url, method, data, timeout)
File "C:\Python37\lib\site-packages\wda_init.py", line 130, in httpdo
return unsafe_httpdo(url, method, data, timeout)
File "C:\Python37\lib\site-packages\wda_init.py", line 160, in _unsafe_httpdo
timeout=timeout)
File "C:\Python37\lib\site-packages\requests\sessions.py", line 530, in request
resp = self.send(prep, **send_kwargs)
File "C:\Python37\lib\site-packages\requests\sessions.py", line 643, in send
r = adapter.send(request, **kwargs)
File "C:\Python37\lib\site-packages\requests\adapters.py", line 498, in send
raise ConnectionError(err, request=request)
requests.exceptions.ConnectionError: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response'))


does not start.

hello I start by telling you that I really like your module, but I can't get it started. now I'll explain what I want to do. First I tell you that I use as modules "PageSelector" and time module "MMM-Openmapweather". I did as it said and I did all the things. but the best and that makes me see the normal screen, without background.how can I do? what should I do? what should i change? thank you for your kind reply. best regards mister hepler

I don't think MMM-Openmapweather is supported as a source for this module. This module supports  currentweather, MMM-NOAA3, MMM-DarkSkyForecast. Have you tried any of those sources?
You can put in a request for me to add MMM-OpenMapWeather support. That could be a fun project.

Hello. how are you?
I have installed the NOOA3 module. but i can't get it to work. as provider of noaa3 is openweather. I would like to pin your form on a page and I downloaded DUMMY. how can I do? thank you.

Okay, let's start with the basics. Please copy & paste the two parts of your config.js that control NOAA3 and MMM-WeatherBackground. Be sure to delete your API keys when you post. I don't need that part. Let's make sure that things are configured properly.
Just to be sure you see the NOAA3 module when your mirror is running, correct?

this post can be closed. I managed to solve it I don't know how ahahahahah. I would like to ask you if in my form I can modify the image file? I'll explain, my idea is to put a GIF or a video with the sound (when possible) of the weather. example: strong wind, wind sound. storm with lightning? sound of lightning and so on. obviously it cannot be done for all types of weather, I mean the sound. but it would be nice. thanks for the availability mister

Ok, I'll close it. I started on the openmapweather and I'm like 75% sure that it doesn't broadcast events like the documentation says it does. So I'm glad that you got it working.
Regarding the image file: you can specify your own image collection. If you look at the code in MMM-WeatherBackground.js you can see the format for the collection. Those numbers represent image IDs are collection numbers at the Unsplash website. You can provide your own collection information via the config.js configuration.
As for sound... that's beyond the scope of this module. If you want to develop the capability, I'll link to it. If you create a pull request, I'll consider it. But I'm not going to add sound to a background image module.

Hello, how are you?. can I create it in my personal form? that is to
configure for me and still make a pull request of course.
my intention is to create in addition to the image also the sound of the
weather.
example:
rain with thunder: https://www.youtube.com/watch?v=HsQlpaYTE0s
snowstorm wind: https://www.youtube.com/watch?v=vplX-qr4AIE
summer: https://www.youtube.com/watch?v=HTc_2XmKK-M
summer night: https://www.youtube.com/watch?v=2Xzfy7zyt0A
winter: https://www.youtube.com/watch?v=I0oLjBDB2NQ
are examples of sound that can link with an image, gif or video. what do
you think? thank you for your time.
BEST REGARDS sir
Il giorno ven 2 lug 2021 alle ore 05:13 Brian Hepler <
@.***> ha scritto:

Ok, I'll close it. I started on the openmapweather and I'm like 75% sure
that it doesn't broadcast events like the documentation says it does. So
I'm glad that you got it working.
Regarding the image file: you can specify your own image collection. If
you look at the code in MMM-WeatherBackground.js you can see the format
for the collection. Those numbers represent image IDs are collection
numbers at the Unsplash website. You can provide your own collection
information via the config.js configuration.
As for sound... that's beyond the scope of this module. If you want to
develop the capability, I'll link to it. If you create a pull request, I'll
consider it. But I'm not going to add sound to a background image module.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/BrianHepler/MMM-WeatherBackground/issues/9#issuecomment-872680937,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMHHK4SB6LA2N7K6M2YVX4TTVUVFPANCNFSM46FYAYQQ
.

You are more than welcome to create your own branch. In fact, I encourage it.


Getopt_long returns a -1 when it's done parsing the command line.

Changed to a signed char so that this works when run on a raspberry pi.

Thanks for the patch but these sort of fixes need to go to the upstream repository, not one of my personal development branches.


Character requires more effort to move up slopes

How to reproduce: Line up your crosshair/middle of the screen with the top corner of a slope, then simply move forward. You will notice that for some reason the play seems to get pushed along the slope rather than climb up it as if it was a flat surface, so your crosshair/middle of the screen will end up more to the side of the slope. In most games, X and Z should not change when climbing slopes.
Suggested fix: I have no idea, I looked at this template because I myself was also desperate for a solution but all 3d kinematic examples seem to have a similar problem.

@Gamepro5 Do you still have this problem?

I've found a little fix about this maybe try this out: https://godotengine.org/qa/25732/slow-speed-up-slopes


Ran npm audit fix

Is there any reason not to do this?

As a complete beginner, I asked myself and Google the same question, and I found at least one source that tells me there could be a reason: https://overreacted.io/npm-audit-broken-by-design/

There is no real reason either way. None of the vulns audit dumps are real.

Closing as all deps were updated


Proposal: NestJS sample application

Hey everybody,
I saw that there are not many Typescript framework on this repository, it could be really valuable to have sample with Nest.js.
I'm able to make some code sample to contribute for this project.
Note: I can sign my commit but i'll never put my LastName on github :)

Yes please propose a sample with nest.js 🙏
And we just require that your commits are sign-off no more

Done :)
https://github.com/docker/awesome-compose/pull/292


[XXS - CVSS Score 6.5] Update ckeditor to 4.11.4

CKEditor 4.11 fixes an XSS vulnerability in the HTML parser reported by maxarr. The vulnerability stemmed from the fact that it was possible to execute XSS inside the CKEditor source area after persuading the victim to: (i) switch CKEditor to source mode, then (ii) paste a specially crafted HTML code, prepared by the attacker, into the opened CKEditor source area, and (iii) switch back to WYSIWYG mode. Although this is an unlikely scenario, we recommend to upgrade to the latest editor version.
https://ckeditor.com/blog/CKEditor-4.11-with-emoji-dropdown-and-auto-link-on-typing-released/

How to find vulnerabilities
https://retirejs.github.io/retire.js/
More information on Snyk
https://snyk.io/vuln/SNYK-JS-CKEDITOR-72618
Also solves
https://github.com/django-ckeditor/django-ckeditor/issues/540

Fixed in 3d8879d4d6d7b42048eedda8aaf8b


Fixed a typo

Just a small typo fix

Once you sign the CLA then we can merge this

@kentcdodds cla is all taken care of 👍


issue with netvars

When injecting you will get this issue with the netvars I am fixing it right now just wanted to inform you guys about it

0x10000000018 most of time means nullpointers or when you hit something you wasn't supposed to (smth out of bounds from an array).
And what is causing that is probably because the element gets set to the next element even if the element is the last in the array... which will cause an out of bounds issue. (translating the loop: copy unallocated_data to element var, if the element is not null then we will set the element to the next one. See the problem, once we arrive at the last element the element still isn't null so we will set it to the next element which will cause out of bounds)
if (element == nullptr)
    break;

Add the code above and you should be fine^^


ID and Demo application are using the SDK directly

As a developer building Tonomy ID
If I update the SDK code, then the Tonomy ID and demo applications That are running in my developer environment use the changed code
So that I can test the change code immediately
The Tonomy ID and demo app in the integration library should be using the SDK library as a dependency. This is for the development environment so that if the developer changes the SDK code then that will also be updated in the applications.
Done:

[ ] When the SDK code is changed, The running Tonomy ID app updates to use this code
[ ] When the SDK code is changed, The running demo app updates to use this code

wait till https://github.com/Tonomy-Foundation/Tonomy-ID-Integration/issues/20 is done to continue this

is there a PR for this @Rebal67


Cannot declare a parameter named 'b'  in strict mode

Which version of PhantomJS are you using? Tip: run phantomjs --version.
x86_64
2.1.1

What steps will reproduce the problem?
phantomjs> (function (){'use strict';console.log('hi');var a = function b (b) {};})();

Expected result
hi

Actual result
Cannot declare a parameter named 'b'  in strict mode

Which operating system are you using?

$ lsb_release -a
Distributor ID:	Debian
Description:	Debian GNU/Linux unstable (sid)
Release:	unstable
Codename:	sid

AND
$ lsb_release -a
Distributor ID: Debian
Description:    Debian GNU/Linux 8.6 (jessie)
Release:        8.6
Codename:       jessie

Did you use binary PhantomJS or did you compile it from source?
Binary release

Please provide any additional information below.
For some reason, phantom JS fails to scope the parameters or something to a named function which have a parameter with the same name. This issue is not present in any other JS-engine I've encountered (Chrome, Node, Firefox). This issue is also not present if I do not assign the named function to a variable. I.e use function b(b){} rather than var a = function b(b){}. It works as intended in non-strict mode.

phantomjs> (function (){'use strict';console.log('hi');function b (b) {};})();
hi
undefined
phantomjs> (function (){'use strict';console.log('hi');var a = function b (b) {};})();
Cannot declare a parameter named 'b'  in strict mode

  phantomjs://repl-input:1 in global code

I should add, that this is for any arbitrary name on b, as long as the function name and argument is the same, this bug is present.

+1
Same issue happens for me.

This was a bug in Webkit that was fixed about a year ago (and also affects any versions of Safari more than a year old) -  PhantomJS 2.1.1 uses a much older version of Webkit than that and therefore has the bug.  Not much PhantomJS can do about it, until the 2.5 release.


Allow no iteration.id and put logic on iteration.name

Input to discussion in #299

Use iteration.name to create iteration.uuid
Allow iteration.id to be None
Allow arbitraty iteration.name

Context: Iterations with arbitrary name may or may not be equivalent to iter-0. Therefore, the current implementation of assuming iteration.id = 0 when arbitrary name should be changed. However, with arbitrary names, we have (currently) no way of deriving the iteration ID.
Can we live without it until further notice??

Propose to merge when dependencies are mapped and accounted for:

[ ] Sumo FMU frontend
[ ] fmu.sumo.explorer
[ ] Webviz-4D
[ ] DynaGeo
[ ] Webviz
[ ] SSDL/REP

[ ] Update to synch with changes coming in from #314


[Feature request] Hide yazi and show terminal

Is your feature request related to a problem? Please describe
In the shell block I can't see the output of the previous command, run the previous command e.g. work in terminal.
Will you be willing to contribute this feature?

[ ] Yes, I will try to implement it

Describe the solution you'd like
I prefer the Midnight Commander-like solution of toggle on/off the whole interface by hotkey, or the Ranger-like solution where the 'exit' command exits from the terminal.
Describe alternatives you've considered
No response
Additional context
No response

How do I return to Yazi in the terminal? I haven't used this before, could you describe the process? It would be great if you could provide a screen recording.

Midnight Commander style:
https://github.com/sxyazi/yazi/assets/5439042/c0b0bf1f-7c3a-4a89-92d1-045712e354ea
Ranger style:
https://github.com/sxyazi/yazi/assets/5439042/8918864c-07b2-4df9-9299-de10a515dd1d

Upd. It seems that it's easily done with  exec = "shell --block zsh" but how to put Enter key after zsh command?

See https://yazi-rs.github.io/docs/tips#dropping-to-the-shell

Oh, thank you, I missed this.


No direct link for listed tokens

Tokens listed on the exchange are supposed to have a direct link. e.g https://example.com/trade/WETH-DAI but the sdk only has the default homepage https://example.com for all tokens. Please can this be fixed? It's it's available do tell how i can fix it. Thank you

This can be done by adding a router library in web project. We don't want to do too many decisions for users, and we want keep this repo easy to customize. I think you can take a look at https://github.com/supasate/connected-react-router.


Upgrade to Next 13: TypeError: (0 , _graphql.getArgumentValues) is not a function

Hey, there – thank you for doing an amazin job with this library.
I upgraded to Next.js 13 an got this error:
I use Pothos in conjunction with Recoil where I store the data and the fetches are handled through urql.
Maybe this information helps you a bit. I can provide more if needed.
{
   "errors":[
      {
         "message":"(0 , _graphql.getArgumentValues) is not a function",
         "locations":[
            {
               "line":2,
               "column":3
            }
         ],
         "path":[
            "currentUser"
         ],
         "extensions":{
            "code":"INTERNAL_SERVER_ERROR",
            "exception":{
               "stacktrace":[
                  "TypeError: (0 , _graphql.getArgumentValues) is not a function",
                  "    at addFieldSelection (/Users/felixhaberle/Desktop/git/backstage/node_modules/@pothos/plugin-prisma/lib/util/map-query.js:123:53)",
                  "    at addNestedSelections (/Users/felixhaberle/Desktop/git/backstage/node_modules/@pothos/plugin-prisma/lib/util/map-query.js:91:17)",
                  "    at addTypeSelectionsForField (/Users/felixhaberle/Desktop/git/backstage/node_modules/@pothos/plugin-prisma/lib/util/map-query.js:47:9)",
                  "    at queryFromInfo (/Users/felixhaberle/Desktop/git/backstage/node_modules/@pothos/plugin-prisma/lib/util/map-query.js:197:9)",
                  "    at resolve (/Users/felixhaberle/Desktop/git/backstage/node_modules/@pothos/plugin-prisma/lib/field-builder.js:33:55)",
                  "    at field.resolve (/Users/felixhaberle/Desktop/git/backstage/node_modules/apollo-server-core/dist/utils/schemaInstrumentation.js:56:26)",
                  "    at resolveField (/Users/felixhaberle/Desktop/git/backstage/node_modules/graphql/execution/execute.js:464:18)",
                  "    at executeFields (/Users/felixhaberle/Desktop/git/backstage/node_modules/graphql/execution/execute.js:292:18)",
                  "    at executeOperation (/Users/felixhaberle/Desktop/git/backstage/node_modules/graphql/execution/execute.js:236:122)",
                  "    at executeImpl (/Users/felixhaberle/Desktop/git/backstage/node_modules/graphql/execution/execute.js:116:14)",
                  "    at execute (/Users/felixhaberle/Desktop/git/backstage/node_modules/graphql/execution/execute.js:60:35)",
                  "    at execute (/Users/felixhaberle/Desktop/git/backstage/node_modules/apollo-server-core/dist/requestPipeline.js:205:48)",
                  "    at processGraphQLRequest (/Users/felixhaberle/Desktop/git/backstage/node_modules/apollo-server-core/dist/requestPipeline.js:148:34)",
                  "    at processTicksAndRejections (node:internal/process/task_queues:96:5)",
                  "    at async processHTTPRequest (/Users/felixhaberle/Desktop/git/backstage/node_modules/apollo-server-core/dist/runHttpQuery.js:221:30)",
                  "    at async graphqlHandler (/Users/felixhaberle/Desktop/git/backstage/node_modules/apollo-server-micro/dist/microApollo.js:31:55)",
                  "    at async ApolloServer.handleGraphqlRequestsWithServer (/Users/felixhaberle/Desktop/git/backstage/node_modules/apollo-server-micro/dist/ApolloServer.js:87:34)",
                  "    at async /Users/felixhaberle/Desktop/git/backstage/node_modules/apollo-server-micro/dist/ApolloServer.js:30:21"
               ]
            }
         }
      }
   ],
   "data":{
      "currentUser":null
   }
}

For example, this is the fetching part:
import { atom, selector } from "recoil";

import { client } from "../graphql/client";
import { GetCurrentUserDocument } from "../graphql/getCurrentUser.generated";
import { User } from "../graphql/types.generated";

// fetch user data from the server
export const currentUserFetch = selector<User | null>({
  key: "currentUserFetch",
  get: async ({ get }) => {
    try {
      const response = await client.query(GetCurrentUserDocument).toPromise();
      return (response.data.currentUser as User) || null;
    } catch (error) {
      console.error(`currentUserFetch -> client.query() ERROR: \n${error}`);
      return null;
    }
  },
});

// set user data in the store
export const currentUserState = atom<User | null>({
  key: "currentUserState",
  default: currentUserFetch,
});

can you check to make sure your version of graphql is also up to date?

@felixhaeberle going to close this out, and assume it was an issue with the graphql version, if that wasn't the issue or you are still running into this problem let me know and I can re-open it

Sorry for not answering, it was indeed the graphql version!! 😊 Thanks 🙏


Create parser

added refactored lexer;
added start of parser with:

re-identification of tokens
treatment of single and duplicate metacharacters

Approved


Docker container for LIS

Hi, I am interested in using LIS. I was wondering if LIS is available as a Docker image. I found this project from 2017, but I couldn't find information on how to run LIS in a Docker container/where to get the image.
https://www.nas.nasa.gov/SC17/demos/demo29.html

Hi @jschap1, thanks for reaching out. I believe that was an experimental demo of Docker+LIS so the image is not publicly available. That said, we are currently circling back to this idea and working to containerize LIS with Singularity.


Provide methods to query added controls on a panel

Panels should keep track of added control and expose the required methods to query them:

hasControl(Control c)
getControl(String id)
getControls()

Resolved through https://github.com/DwcJava/engine/pull/466


Landmarks in cavas are not showing up

When we do a landmark search, headers in canvas are not showing up.
Here is the configuration

Hi, @ShravanSunder :)
I think that feature has not been implemented because, in canvas mode, it is assumed that getting information about the files in the node would be impossible. Someday, when that becomes possible, I will consider implementing it.

@tadashi-aikawa that makes sense.  I just noticed there's no hash for canvas files in fileCache.  And i guess no metadata 😢


Only raise EmptyKeyAttributeException when the value node is empty

This fixes a bug where all updates to keys (including GSIs) would raise
EmptyKeyAttributeException, even when the new value was not an empty string.

This fixes #3760. It's also a duplicate of #3763 - serves me right for not looking first. Leaving it open because the test case is different from the one in the other PR.


DBZ-4491 add support for Google PubSub emulator

https://issues.redhat.com/browse/DBZ-4491
Hi,
Let me give you a disclaimer and say, that I am not a java developer. So whatever bad stuff you see... please bear with me. 🙏
There are still a couple of things to do, but I just want to get your feedback before advancing any further.
What I still need to do

[ ] spin up a docker image running the pubsub emulator as part of the test setup
[ ] write a little bit of docs

PS:
I do have some issues running mvn clean install. So formatting might be off.
The PostgreSQL connector ITs fail for some reason, but I will take this in the chat.

@calinilie Hi, thanks a lot. This is definitely great start. I left few comments related mostly to organization of the code. Functionally it looks good so far.

Hey @calinilie, any news here?

@calinilie, hey there, could you indicate whether you're still planning to wrap up this one?

@calinilie Cool, we are looking forward!

spin up a docker image running the pubsub emulator as part of the test setup

Testcontainers should be used for that (see the tests for the Pulsar or Pravega sinks for an example).

Thanks @gunnarmorling , I'll look into this. Unfortunately, there is no official docker image for the pub sub emulator, although it is not too complex to create one.
Haven't worked with GenericContainer before so I'll see if i can spin up a container using that interface.
For reference, this is the dockerfile for the pubsub container i used to get the tests passing locally.
# Version. Can change in build progress
ARG GCLOUD_SDK_VERSION=alpine

# Use google cloud sdk
FROM google/cloud-sdk:$GCLOUD_SDK_VERSION

# Install Java 8 for Pub/Sub emulator
RUN apk --update add openjdk8-jre
RUN gcloud components install pubsub-emulator beta --quiet

COPY start-pubsub.sh .

EXPOSE 8432

RUN ["chmod", "+x", "/start-pubsub.sh"]

ENTRYPOINT ["sh", "./start-pubsub.sh"]

start-pubsub.sh

#!/usr/bin/env bash

# Check user environment variable
if [[ -z "${PUBSUB_PROJECT_ID}" ]]; then
  echo "Missing PUBSUB_PROJECT_ID environment variable" >&2
  exit 1
fi

if [[ -z "${PUBSUB_LISTEN_ADDRESS}" ]]; then
  echo "Missing PUBSUB_LISTEN_ADDRESS environment variable" >&2
  exit 1
fi

# Config gcloud project
gcloud config set project ${PUBSUB_PROJECT_ID}

# Start emulator
gcloud beta emulators pubsub start \
  --data-dir=/opt/data \
  --host-port=${PUBSUB_LISTEN_ADDRESS}

@calinilie Thanks alot, I fixed formatting to get he build running. This PR is good to be merged next week. Could you please jsut extend https://github.com/debezium/debezium/blob/main/documentation/modules/ROOT/pages/operations/debezium-server.adoc vit the new parameter and maybe a note about the emulator usage?

@jpechane thanks!
I had run mvn process-sources to apply the desired formatting according to the code style in this commit https://github.com/debezium/debezium/pull/3250/commits/502eb5f9178cdf054b768aeed08c02b23cd6eabd
Thought that would have been enough. 🤷♂
I'll write some docs in the PR. Assume that is ok.

@calinilie Applied, thanks a lot for the effort and patience!

Thank you all for making it happen 🙏


net: TestLookupHostCancel failures with "i/o timeout" on netbsd-arm*-bsiegert builders

--- FAIL: TestLookupHostCancel (22.63s)
    lookup_test.go:954: lookup www.google.com on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:61734-><IP_ADDRESS>:53: i/o timeout
FAIL
FAIL	net	55.015s

2020-12-22T18:36:29-c06a354/netbsd-arm-bsiegert
2020-12-02T16:44:10-5984ea7/netbsd-arm64-bsiegert
2020-11-30T20:15:10-848dff6/netbsd-arm-bsiegert
2020-11-24T07:07:06-6965b01/netbsd-arm-bsiegert
2020-11-19T20:37:38-59f5fda/netbsd-arm64-bsiegert
2020-11-19T19:31:07-498d8d5/netbsd-arm64-bsiegert
2020-11-10T16:25:27-c68745b/netbsd-arm-bsiegert
2020-11-04T06:14:02-5ca43ac/netbsd-arm-bsiegert
2020-10-06T22:54:58-3923460/netbsd-arm64-bsiegert
2020-09-16T03:02:13-eaa97fb/netbsd-arm-bsiegert
May be related to #43140, or perhaps the builder's DNS server has occasional connectivity issues?
CC @bsiegert @golang/osp-team

Yes, the builder has occasional packet loss unfortunately. I will move it to a location with better internet in a month.

Yes, the builder has occasional packet loss unfortunately. I will move it to a location with better internet in a month.

Looks like the connection upgrade did the trick! No failures on -bsiegert builders since January. 😃
greplogs --dashboard -md -l -e 'lookup www\.google\.com .*: i/o timeout' --since=2020-12-01
2021-11-20T01:00:16-be18cd5/darwin-amd64-nocgo
2021-03-25T04:06:37-c69515c/darwin-amd64-nocgo
2021-01-26T00:02:15-ce8b318/netbsd-arm64-bsiegert
2020-12-22T18:36:29-c06a354/netbsd-arm-bsiegert
2020-12-02T16:44:10-5984ea7/netbsd-arm64-bsiegert
2020-12-01T23:39:15-cf7aa58/solaris-amd64-oraclerel


Update Posts “juish-nature”

Automatically generated by Netlify CMS

👷 Deploy Preview for benevolent-pika-f5fbe0 processing.

Name
Link

🔨 Latest commit
7eaed50e30571627e39d30f62175b49ea5d10ada

🔍 Latest deploy log
https://app.netlify.com/sites/benevolent-pika-f5fbe0/deploys/62e3f66a4925790008eebf70


inline comment

fix #50

can you please make linter happy and add some tests for the case you added?

can you please make linter happy and add some tests for the case you added?

ok. I'll finish it this week


Please improve feedback on az upgrade for better experience

Related command
az upgrade
Is your feature request related to a problem? Please describe.

I want to see progress update when running az upgrade, right now it just looks stuck for 2 min

Also another thing, and quite annoying, is that it pops up an "install window" where it asks me to click next.. I didn't expect this Window and I only saw it by mistake. Can't it install an MSI in a silent mode without UI so it stays in the terminal, I don't want to be yanked out of the terminal experience when upgrading az cli.

Updating Azure CLI with MSI from https://aka.ms/installazurecliwindows
Downloading MSI to C:\<user>\AppData\Local\Temp\tmp6ac0qfjo\azure-cli-2.62.0.msi
Installing MSI

Describe the solution you'd like

An improved experience would be:
Updating Azure CLI with MSI from https://aka.ms/installazurecliwindows
This may take 2-3 min depending on the speed of your Internet connection

Step [1/5] Downloading MSI to C:\<user>\AppData\Local\Temp\tmp6ac0qfjo\azure-cli-2.62.0.msi
Step [2/5] Installing MSI
[.xxxx....]  ~2 min left

Also, please run MSI install in terminal only, don't pop open an install Window

Thank you for considering
Describe alternatives you've considered

Additional context

Thank you for opening this issue, we will look into it.

For progress update, we'll add it to feature request list.
For silent mode, it's not possible.

For non-admin user, the UAC window always pops up. See https://github.com/Azure/azure-cli/issues/16617
az exits when installing MSI; otherwise, some files are still in use and the installation fails. See https://github.com/Azure/azure-cli/issues/22741. If we use silent mode, the user won't be able to know whether the installation has finished.

az upgrade also support update extensions by using --all option.
When I use az upgrade --all to upgrade extesions. It take a lot of time to perform version checking one-by-one.
Since we do have installed version list and available version list. Should can metadata rather than check online for each extensions.
I don't use az upgrade --all now. Use script is faster
  remote_versions=$(az extension list-available --query "[?installed].{name:name, version:version}")
  local_versions=$(az extension list --query "[].{name:name, installed_version:version}")

  versions=$(jq -n --argjson local "$local_versions" --argjson remote "$remote_versions" '
    [$local, $remote] |
    transpose |
    map({
      name: .[0].name,
      installed_version: .[0].installed_version,
      version: .[1].version
    })
  ')

  outdated=$(echo "$versions" | jq -r '.[] | select(.installed_version != .version) | .name')

  if [ "$outdated" == "" ];
  then
    echo "No outdated azcli extensions found."
  else
    for name in $outdated; do
      echo az extension update --name "$name" --allow-preview true --verbose
      az extension update --name "$name" --allow-preview true --verbose
    done
  fi


Revert Removal of Senior roles

About the PR
I reverted the removal of Senior Roles because I thought it was a bad idea and that they had a lot of value for teaching new players.
Why / Balance

For many departments senior roles were useful for teaching new players as well as indicating to new players who should be trusted to help them. This was especially apparent in Security where the complexity of the job as well as the need for the Head of Security and Warden to deal with other concerns during a round made it difficult for them to effectively teach Cadets. Relying on other Security Officers was not always possible as the job takes a long time to learn and Security Officers are frequently not up to the challenge of teaching new Cadets as they are not always completely competent themselves. They were also useful for Security with the current meta of placing Cadets in squads lead by Senior Officers to help teach Cadets.
Senior roles were useful because they provided a way for players to self designate as instructors as well as allow new players to quickly identify who they could expect to help them to learn their job.
Concerns about people playing those roles and refusing to assist new players are better resolved either in character (demotion etc) or out of character through Moderator intervention.
Technical details

This is just a reversion of the commit that removed Senior Roles. I have not tested it at time of writing so it may cause issues.
Media

[X ] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog
:cl:

add: Senior Roles.

it begins

changelog typo

cl has a typo, close pr now

Please add a "." to the end of the cl entry

and remove s

Failed to build because Seniors don't have spawnpoints on Aspid.

can we add and remove seniors every week it would be funny I think

It'd be far better to reimplement them as alt titles that're automatically applied if a player meets the conditions. Senior roles as a separate job carry maintenance overhead for not much benefit over that alternative.

an idea that may be better is experienced players can opt in to being a teacher
if an intern role joins it tells them either in greeting or as a popup in center for visibility, saying talk to XYZ to learn stuff

Look, don't start a back and fourth. That makes this whole thing look really bad.
Try to actually address concerns brought by #22303 and on the original #18181 PR, otherwise the opinion isn't going to change.
I've given up on this whole ordeal because I'm not knowledgeable enough to do a lot of this and am busy with my own stuff, hell the original thing was just a middle ground because we had no other way to do it at the time.
Currently the general idea from maintainers is to make it into some kind of opt-in title system based on your playtime, but that doesn't have any work going into it as far as I'm aware.
However for other ideas, a good place to start is proper communication on what the role does, both to the player playing it and new players. Whole slew of ways to do this, to name a few;

Some kind of text pop-up for people taking the role, in chat, in it's own box, who knows.
Text pop-up for new players pointing out what a Senior role is and does.
Job icons that point out Seniors to Interns and Interns to Seniors.

For more balance and design choices;

Removal of their extended access so people stop using them just to multi-job. (I am not a big fan of this as it restricts what a Senior can teach, but if it helps in teaching new players I'll live with it)
A system that treats Senior role slots the same as the basic job slot, so if all the Engineer slots are taken, so is the Senior role, as to prevent people from just picking it to divert full departments.
Make a new whitelist system and make it whitelist only, so admins allow select people they know would use the role properly. (Once again, I don't like this idea as it creates more admin work, but I could see it being very good.)

There are very few ways to actually "mechanically enforce" or "encourage" teaching, so I do kind of disagree with those stances, but it stands there are ways to improve the system from what it is, and it's not going to get re-added if you don't do that.

We're not looking to readd them at this time.
If you can implement a system that can prefix a title depending on hours played that would be a preferable solution. And even then the "Senior" title should also be optional.

If you can implement a system that can prefix a title depending on hours played that would be a preferable solution. And even then the "Senior" title should also be optional.

I'm going to leave my collection of thoughts about this here, as it's the best place to be seen for any soul who's going to try to add this system, and so I don't have to mention them after it's made.
(I'm not a maintainer, but I feel like these gripes are reasonable.)

The title needs to be opt-in, as stated above.
The fact it'd take a basic job slot is poor to me, as a Senior shouldn't be expected to drop their teaching to help a department.
I don't think the possibility of having a department full of all Seniors is sane or a good idea, I personally think the title should be limited to one person per department.
Having it not be it's own role restricts their access, meaning it restricts what they can teach without extra hurdles to jump through.
Preferably admins would have a way to ban players from using the title if they aggressively misuse it, just like the role.
The lack of communication between new players and what a Senior does still needs to be fixed some how, as it'd be even worse now because a new player isn't going to go around checking people's ID cards like that. The clothing was made to make them stand out at least a little bit.

How would that be implemented? We could make it clearer that it's a teaching role but that is just a recommendation to players.  Leaving it to admins to enforce it will put a big burden on administration because we'll have to look over seniors to make sure they are teaching people along with everything else. I guess you could make it a whitelist thing but that may be a bit too much.

How would that be implemented? We could make it clearer that it's a teaching role but that is just a recommendation to players. Leaving it to admins to enforce it will put a big burden on administration because we'll have to look over seniors to make sure they are teaching people along with everything else. I guess you could make it a whitelist thing but that may be a bit too much.

The game is castrated - and rejoices
and when they ask us to return what was good and everyone supports it, we get a refusal...
 Please refrain from being toxic in future.

How would that be implemented? We could make it clearer that it's a teaching role but that is just a recommendation to players...

Is the job title and description of Scientist just a recommendation too? Can scientists just go do cargo work if they want to without changing their title? The fact that it requires 60 or however many hours to even unlock (and should also be optional) is in my opinion enough for them to be blamed if they didn't figure it out yet.

Leaving it to admins to enforce it will put a big burden on administration because we'll have to look over seniors to make sure they are teaching people along with everything else. I guess you could make it a whitelist thing but that may be a bit too much.

Admins don't have to put more effort than any other rule breaker in the game. If someone is ahelped for refusing to teach with a senior prefix then they can be checked out. Not all seniors need to be tracked constantly. There might even be plenty of rounds where a senior exists and there are no players that require teaching after all.

There might even be plenty of rounds where a senior exists and there are no players that require teaching after all.

is there any other role in the game where your purpose is irrelevant for "plenty of rounds"? Half the reason seniors dick off so hard is because you might not have any new players to teach or the new players might not want you to instruct them or even know that you are supposed to.
roles are meant to just be default IC representations of tasks that someone is meant to be doing. They're not set in stone, they're not unchangeable, but they represent a concrete concept.
Seniors and mentor roles in general fail because you're approaching an IC problem with some kind of RP solution that just isn't going to work. New players just need the information presented as clearly as possible so they can learn and do their stuff. Having to RP out an encounter to learn things is just going to lead to OOC talk and be arguably a harder barrier of entry than just reading the wiki.
that aside, my issue isn't with teaching new players info, but what roles do to game balance as a whole. Roles shouldn't be overlapping and doing one another's jobs. If a role is marked by experience and is going to be twiddling their thumbs for possibly entire rounds, it's just going to lead to situations where they do other roles jobs because teaching new people isn't enough content to remotely justify a job.
needing admins to enforce manage players and doing their jobs isn't the issue, but it becomes an issue with the role as a whole when you can go multiple rounds in a row with literally nothing to do, especially on MRP servers where players are going to be more skilled, on average. A player with nothing to do is going to inevitably step on the toes of other roles, which isn't ideal.

Is the job title and description of Scientist just a recommendation too? Can scientists just go do cargo work if they want to without changing their title?

On LRP, yes actually.
I think you'll generally find that what drives people to "do their job" is more the fact that they started with access and equipment for their job, and the fact that if you selected a job you probably don't hate playing it.

If someone is ahelped for refusing to teach with a senior prefix then they can be checked out.

New players, who would be the "victims" under such a policy, generally are not going to know what rules are being violated and start slinging ahelps on the first day of the job.
Ahem.
Seriously this entire thing seems so mechanically poor. Tying a charitable intent (teaching) to an exclusive prestigious job slot just creates a conflict of interest. Now admins have to clean up the mess.
Having it as an alternate job title is really the only good choice here. It doesn't convey prestige, it doesn't require extra playtime. People enable it if they want to teach and there's no other reason to enable it otherwise.
Also I'm sure I've seen some arguments like "how would new players know who to ask for getting taught without this?". Uhm:

:m hey I'm kinda new, does anybody mind mentoring me?

Like seriously even if I want to be charitable I do not see these as necessary in the first place, let alone worth kicking up drama over.

is there any other role in the game where your purpose is irrelevant for "plenty of rounds"? Half the reason seniors dick off so hard is because you might not have any new players to teach or the new players might not want you to instruct them or even know that you are supposed to.
...
but it becomes an issue with the role as a whole when you can go multiple rounds in a row with literally nothing to do, especially on MRP servers where players are going to be more skilled, on average. A player with nothing to do is going to inevitably step on the toes of other roles, which isn't ideal.

So is the general consensus that senior roles are only there to specifically teach and completely ignore their job otherwise? Because I was not personally aware of that. I think senior roles should be something of a marker for new players to know they are people that have designated themselves as willing to teach the job.
Also RP roles already exist that have the same "problems" so you might as well remove them too. Isn't half the point of the game the roleplay aspect?

Seniors and mentor roles in general fail because you're approaching an IC problem with some kind of RP solution that just isn't going to work. New players just need the information presented as clearly as possible so they can learn and do their stuff. Having to RP out an encounter to learn things is just going to lead to OOC talk and be arguably a harder barrier of entry than just reading the wiki.

So why do intern roles exist? As far as I know, intern roles have 100% overlap with regular roles, uselessly occupy job slots similarly to senior roles and fail to advertise degree of knowledge in that specific job (since hours fail as a metric for knowledge, experienced people can still use intern roles if regulars are filled up and the amount of time required to unlock the regular role is pitiful). The only thing that is left is the RP component of playing as a character that doesn't know the job.
I don't know what makes you think that talking to people in the game would "arguably (be) a harder barrier of entry than just reading the wiki", but there are people who learn better from examples than from being loredumped by the wiki. Let's not forget how out of date/incomplete it is and just generally is a pain to use during the game. If "information presented as clearly as possible" is the goal here then the wiki is a really bad replacement over a living breathing human being that can actively explain the game to you with the thing that is being explained right in front of the new player.
As things stand right now, new players won't just suddenly stop asking other players for help with learning their job, and people who know their job will still have to take time out of running the station to teach the new player, and this will also again lead to the same exact issues of 'rp solution for ic problem' and 'is probably going to lead to ooc chat', so I really fail to see how removing senior roles is in any way related to this.
The only thing it accomplishes is to make it harder for new players to find people that are willing to teach them. I don't know how much harder that is or how big of a problem it is but frankly considering the only benefits are (as far as I can tell) to stop people from using the role as a playtime flex, well, I just wonder why we didn't wait for a replacement before removing it.
And not to get off topic but

New players, who would be the "victims" under such a policy, generally are not going to know what rules are being violated and start slinging ahelps on the first day of the job.

Is this seriously necessary? Because the only thing even thinking to write this in the first place manages to do is taint this whole debacle as thinly veiled hatred towards new players.
Just to clarify my position on this so people don't just assume I want this reverted or whatever:

removing senior role = good (poor implementation in hindsight plus role bloat)
removing senior role without replacement = bad (why now? no seriously I'm not trying to be mean or anything I assume it was this bad for a long time so why specifically now and not when a replacement is available?)
intern roles still existing = highly confusing considering removal of senior roles (what is even the point of these roles? none of the ss13 servers I'm aware of have this)
prefix thingy = good I hope this one happens
learning by reading the wiki = good although wiki could be better but alas
learning by asking others = also good
arguing that only one of these is necessary/is sufficient/should exist = bad (more options are always better imo)

Well that was closed pretty quick.

New players, who would be the "victims" under such a policy, generally are not going to know what rules are being violated and start slinging ahelps on the first day of the job.

Is this seriously necessary? Because the only thing even thinking to write this in the first place manages to do is taint this whole debacle as thinly veiled hatred towards new players.

I genuinely cannot fathom what could possibly make you read my comment like this.

I genuinely cannot fathom what could possibly make you read my comment like this.

No other rule break would justify using victim with quotation marks like this. The only reason that I can think of for someone to put quotation marks in that context is that they do not agree that new players would be victims in that circumstance, that they are actually posers and have not suffered from being affected by a rule being broken like everyone else does. In reality if that rule were to exist new players wouldn't be "victims", they would just be plain victims like every other person who is affected by a rule break.
Now I don't know if this was your intent but I genuinely do not see any other intent to adding quotation marks like that. You're not quoting anyone as far as I can tell.

I put "victim" in quotation marks because it's such a petty and minor transgression (that isn't even in the rules, and absolutely should not be either).
I'm going to encourage you to be a little more open minded (and also, like, engage with the actual argument?) instead of saying stuff like "you hate new players!!".

changelog typo

Cl fuckery, doesn't say "Added"

I put "victim" in quotation marks because it's such a petty and minor transgression (that isn't even in the rules, and absolutely should not be either) that it barely even seems severe enough to use the word. If I wanted to be less brief I would've used "the people who are affected" or something.

If you don't think it's severe enough to use the word then don't use the word. You're the one who picked the word. If the word didn't reflect what you meant to say then don't use that word. It's literally that easy.
Either than that it appears we are in agreement. You think it would be a petty and minor transgression, and I agree that you think that. Considering advertising that part of your belief is completely unrelated to the rest of your post I fail to see the reason behind adding it aside from, well, advertising that you think lesser of new players.
 As for whether or not taking a role whose only purpose is to be a teacher for new players (and do your job) and refusing to teach new players counts as being a dick to new players, well, personally I think it counts but what do I know. It's not like I think these people should be permabanned. But if a person is misusing a role to that degree I don't see why they shouldn't be rolebanned at least.

I'm going to encourage you to be a little more open minded (and also, like, engage with the actual argument?) instead of saying stuff like "you hate new players!!".

Do you really think the reason that I ignored the rest of your comment was because I am too closed minded?  There is nothing in your initial post that I disagree with aside from maybe the first paragraph but that part is also completely off topic so I didn't bring it up.
The only reason I mentioned you at all is because of that unnecessary comment of yours.

 Yes, Senior roles are important and discussion is needed, but they're not major to the game unless you're new.
If you're going to discuss be respectful and forward discussion on how to reimplement the system in a new fashion, if you have gripes outside of that, there are other places to voice those without making everyone who collectively like Senior roles look bad.


doctl: 1.102.0 -> 1.104.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/digitalocean/doctl/releases.
meta.description for doctl is: A command line tool for DigitalOcean services
meta.homepage for doctl is: https://github.com/digitalocean/doctl
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.103.0 in filename of file in /nix/store/bab0nc856xkcxshz4zrdav3ndk8lbqv1-doctl-1.103.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
doctl

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/bab0nc856xkcxshz4zrdav3ndk8lbqv1-doctl-1.103.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A doctl https://github.com/r-ryantm/nixpkgs/archive/0728a7ec4171cec1e59eaf3f985f2d36e240e105.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/0728a7ec4171cec1e59eaf3f985f2d36e240e105#doctl

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/bab0nc856xkcxshz4zrdav3ndk8lbqv1-doctl-1.103.0
ls -la /nix/store/bab0nc856xkcxshz4zrdav3ndk8lbqv1-doctl-1.103.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

doctl

Maintainer pings
cc @langston-barrett for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/digitalocean/doctl/releases.
meta.description for doctl is: A command line tool for DigitalOcean services
meta.homepage for doctl is: https://github.com/digitalocean/doctl
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.104.0 in filename of file in /nix/store/p82kjyk3xh5r6xjz1w527haglzrgkicz-doctl-1.104.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
doctl

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/p82kjyk3xh5r6xjz1w527haglzrgkicz-doctl-1.104.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A doctl https://github.com/r-ryantm/nixpkgs/archive/699c7d2efaf59ae76dc4c3629081ac28aed5c73b.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/699c7d2efaf59ae76dc4c3629081ac28aed5c73b#doctl

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/p82kjyk3xh5r6xjz1w527haglzrgkicz-doctl-1.104.0
ls -la /nix/store/p82kjyk3xh5r6xjz1w527haglzrgkicz-doctl-1.104.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

doctl

Maintainer pings
cc @langston-barrett for testing.


Build has rusted

The build process for CircuitPython has been updated.  Need to upgrade the Build to the newest version, and fix the name in PyPi to match the published name "sparkfun-circuitpython-qwiictwist" not "adafruit-circuitpython-qwiictwist" since this a community CircuitPython library and not an official Adafruit library.

Need to update the build following the process detailed in Creating and Sharing CircuitPython Library documented on Adafruit's Learning website.
https://learn.adafruit.com/creating-and-sharing-a-circuitpython-library/creating-a-library


Using DisplayLink to emulate vsync on OSX

This should fix #24478
Implemented a simple version of the fix SDL did. We create a DisplayLink object which will start a background performance thread. This thread will do a callback right before the display is about the update.
We set a flag in this callback and release a conditional lock that was started right before our buffer swap.
Do note that the DisplayLink sends out its callback based on the main display, in a multiple display setup the sync will be based on the main display only.

Cherry-picked for 3.0.7.


Add maven-clean-plugin for delombok removal in the modules where it's…

… used
This plugin is used for the removal of the delomboked sources that generate the duplicate classes errors in Intellij.
Signed-off-by: Alexandra Bucur<EMAIL_ADDRESS>

Now I see what you said about javadoc. I'll put it in draft until I can get back to it. The issues appear in my local environment even though I do have the plugin installed, but I'll check again soon all details.

Doing a little more reading ...

https://intellij-support.jetbrains.com/hc/en-us/community/posts/115000763864-Can-t-permanently-exclude-delombok-from-generated-source seems to suggest that as long as delombok is outside generated sources, you should be ok (which is the case with your proposal), implying the config change isn't needed - which isn't what we appear to see
https://stackoverflow.com/questions/37801851/how-to-get-rid-of-duplicate-class-errors-in-intellij-for-a-mavenized-project-usi reports the above doesn't work, and suggests using clean, or the workaround (same as you found)
https://github.com/awhitford/lombok.maven/issues/7 (older article) talk through a few options - nothing particularly new here
In terms of where to put the source - https://github.com/awhitford/lombok.maven/issues/7 talks about moving lombok'd code to src/main/lombok or similar but indicates it causes issues in any case

In terms of maven I don't think we can do any better than you have proposed. I agree the scope is limited in terms of delombok is ONLY being used here to support javadoc .
I will try and check

source jars - I suspect the source jars are unaffected, ie they contain lombok code. I'm not sure what intellij will make of these when trying to build a project that is consuming one of our artifacts.
built javadoc - try javadoc.io with one of our lomboked & non-lomboked modules
gradle - check in intellij

further checks/discussions complete. Supporting the proposal as the best option to enable lombok.


Issue with tickMarkFormatter and timezone

Lightweight Charts™ Version: latest
I'm implementing timezone support with timeFormatter and tickMarkFormatter, but the tickMarkFormatter at tickMarkType 2 (DayOfMonth) still starts the day as UTC.

Thoughts?

The tickMarkFormatter is called with three arguments: https://tradingview.github.io/lightweight-charts/docs/api#tickmarkformatter
The second argument is TickMarkType which lets you know the type of tick mark. You can check if that is equal to TickMarkType.DayOfMonth, and then rather return the day only instead of the full date string.
If the formatter function returns null then the default tick mark formatter will be used as a fallback. So you can use the fallback for all cases except Time and TimeWithSeconds. This is most likely what you would like to do if you only want to change the time displayed.
Note that the returned string should be the shortest possible value and should have no more than 8 characters. Otherwise, the tick marks will overlap each other.
The other alternative is to change the date values for the data before giving them to the library as mentioned here: https://tradingview.github.io/lightweight-charts/docs/time-zones#how-to-add-time-zone-support-to-your-chart

This is how I'm approaching the tickMarkFormatter.

The only problem is the mismatch with the timeFormatter.
When the tickMarkFormatter with TickMarkType.DayOfMonth fires, it isn't midnight in the timeFormatter with the timezone, the tickMarkFormatter still starts the day as UTC.
Where the tickMarkFormatter starts the day...

The timeFormatter correctly points out that it is not midnight in the timezone, but it is midnight in UTC (I'm in GMT+2).

The timeFormatter code is like to tickMarkFormatter without the TickMarkType.

Right, so the issue is that the crosshair label doesn't display the same time as the tick mark.
It's not possible to control the placement of the tick marks, so in this case the simpler solution is to correct the timestamps in the data before it supplied to the library. https://tradingview.github.io/lightweight-charts/docs/time-zones#how-to-add-time-zone-support-to-your-chart
Then you wouldn't need to adjust the tick mark and time formatters.

Got it, so literally adding the offset in seconds to the timestamp? I guess this only applies to hourly or similar intervals, right? For timestamps where time is not necessary like daily/monthly that can change the date entirely.


Fix logarithmic uniform ranges

= Description =

Make uniform ranges usable in interval ]0;1].

= Context =

Uniform ranges are defined over an interval, and can be either linear or logarithmic. Currently, the generation of a logarithmic uniform range on interval ]0;1] will fail.

= Success criteria =

Logarithmic uniform ranges can be used on ]0;1].

= Expected issues =

Mathematical issues around 0.

Imported from trac issue 38.  Created by bmoreau on 2014-06-10T14:46:00, last modified: 2014-06-10T14:46:00


Compilation produces errors and fails

error[E0658]: non-reference pattern used to match a reference (see issue #42640)
  --> src/bitcoin/address.rs:28:13
   |
28 |             Type::P2PKH => Address::p2pkh(private_key),
   |             ^^^^^^^^^^^ help: consider using a reference: `&Type::P2PKH`

error[E0658]: non-reference pattern used to match a reference (see issue #42640)
  --> src/bitcoin/address.rs:29:13
   |
29 |             Type::P2WPKH_P2SH => Address::p2wpkh_p2sh(private_key),
   |             ^^^^^^^^^^^^^^^^^ help: consider using a reference: `&Type::P2WPKH_P2SH`

error[E0658]: non-reference pattern used to match a reference (see issue #42640)
  --> src/bitcoin/address.rs:75:13
   |
75 |             Network::Testnet => TESTNET_ADDRESS_BYTE,
   |             ^^^^^^^^^^^^^^^^ help: consider using a reference: `&Network::Testnet`

error[E0658]: non-reference pattern used to match a reference (see issue #42640)
  --> src/bitcoin/privatekey.rs:90:13
   |
90 |             Network::Testnet => TESTNET_BYTE,
   |             ^^^^^^^^^^^^^^^^ help: consider using a reference: `&Network::Testnet`

error[E0658]: non-reference pattern used to match a reference (see issue #42640)
  --> src/monero/network.rs:18:9
   |
18 |         Network::Testnet => Some(&[0x35]),
   |         ^^^^^^^^^^^^^^^^ help: consider using a reference: `&Network::Testnet`

error[E0658]: non-reference pattern used to match a reference (see issue #42640)
  --> src/monero/network.rs:19:9
   |
19 |         Network::Mainnet => Some(&[0x12]),
   |         ^^^^^^^^^^^^^^^^ help: consider using a reference: `&Network::Mainnet`

error[E0658]: non-reference pattern used to match a reference (see issue #42640)
  --> src/monero/network.rs:20:9
   |
20 |         Network::Error => Some(&[0x00]), // fix
   |         ^^^^^^^^^^^^^^ help: consider using a reference: `&Network::Error`

error[E0658]: non-reference pattern used to match a reference (see issue #42640)
  --> src/zcash/address.rs:24:13
   |
24 |             true => private_key.to_public_key().serialize().to_vec(),
   |             ^^^^ help: consider using a reference: `&true`

error[E0658]: non-reference pattern used to match a reference (see issue #42640)
  --> src/zcash/address.rs:25:13
   |
25 |             false => private_key
   |             ^^^^^ help: consider using a reference: `&false`

error[E0658]: non-reference pattern used to match a reference (see issue #42640)
  --> src/zcash/address.rs:36:13
   |
36 |             Network::Testnet => TESTNET_ADDRESS_BYTES,
   |             ^^^^^^^^^^^^^^^^ help: consider using a reference: `&Network::Testnet`

error[E0658]: non-reference pattern used to match a reference (see issue #42640)
  --> src/zcash/privatekey.rs:90:13
   |
90 |             Network::Testnet => TESTNET_BYTE,
   |             ^^^^^^^^^^^^^^^^ help: consider using a reference: `&Network::Testnet`

which version of rust are you using?

Here's what I did to try to reproduce your problem:

Installed Rust 1.28
Build with --release flag and code the master branch
Verify the release executable works properly
cargo install wagu
Verify the cargo installed version works properly

Which operating system are you on? also are you building from source or using cargo install wagu?

I'm using cargo install wagu. Running it in a linux vagrant VM - this one: https://app.vagrantup.com/laravel/boxes/homestead
Should I try a manual build?

@Swader I was able to an installation issue by installing openssl on this Vagrant vm.
To do this I ran sudo apt install libssl-dev pkg-config.

Oddly enough now it works - but the update didn't do anything, it was already at newest version. Thanks!


Support to receive direct data messages

The method handler was already there but no action was done. I've added the required code line to forward the received data such that the JavaScript code receives the notification.
I've also tested it and it works perfectly.
Furthermore, I've fixed two typos.

@salim7 please fix the merge conflicts


code references k8s.io/spark-on-k8s-operator which does not resolve to an accessible repo

i'm trying to make use of the generated client code, but can't even fetch the dependency because the client refers to a repo that doesn't exist...
import (
	sparkClientset "github.com/GoogleCloudPlatform/spark-on-k8s-operator/pkg/client/clientset/versioned"
	sparkV1aplpha1 "github.com/GoogleCloudPlatform/spark-on-k8s-operator/pkg/apis/sparkoperator.k8s.io/v1alpha1"
)

$ go get
# cd .; git clone https://github.com/kubernetes/spark-on-k8s-operator /Users/scott/go/src/k8s.io/spark-on-k8s-operator
Cloning into '/Users/scott/go/src/k8s.io/spark-on-k8s-operator'...
ERROR: Repository not found.
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
package k8s.io/spark-on-k8s-operator/pkg/apis/sparkoperator.k8s.io: exit status 128
package k8s.io/spark-on-k8s-operator/pkg/client/clientset/versioned/typed/sparkoperator.k8s.io/v1alpha1: cannot find package "k8s.io/spark-on-k8s-operator/pkg/client/clientset/versioned/typed/sparkoperator.k8s.io/v1alpha1" in any of:
	/usr/local/Cellar/go/1.10/libexec/src/k8s.io/spark-on-k8s-operator/pkg/client/clientset/versioned/typed/sparkoperator.k8s.io/v1alpha1 (from $GOROOT)
	/Users/scott/go/src/k8s.io/spark-on-k8s-operator/pkg/client/clientset/versioned/typed/sparkoperator.k8s.io/v1alpha1 (from $GOPATH)

In your import block, can you try using k8s.io/spark-on-k8s-operator/pkg/client/clientset/versioned and k8s.io/spark-on-k8s-operator/pkg/apis/sparkoperator.k8s.io/v1alpha1?

Yes, currently k8s.io/spark-on-k8s-operator will not resolve. I think what you need to do is:

git clone github.com/GoogleCloudPlatform/spark-on-k8s-operator $GOPATH/k8s.io/spark-on-k8s-operator.
Use k8s.io/spark-on-k8s-operator/pkg/client/clientset/versioned and k8s.io/spark-on-k8s-operator/pkg/apis/sparkoperator.k8s.io/v1alpha1 in imports.

git clone github.com/GoogleCloudPlatform/spark-on-k8s-operator $GOPATH/src/k8s.io/spark-on-k8s-operator

gets the job done (notably for anyone else reading this you left out src above)
tho it would be nice if this weren't necessary.. either by having k8s.io/spark-on-k8s-operator resolve properly or regen the code to reference github.com/GoogleCloudPlatform/spark-on-k8s-operator

I will update the code to reference github.com/GoogleCloudPlatform/spark-on-k8s-operator since this is not under the kubernetes project anyway.

Opened #154. Closing this.


NodePoppler error

I have a docker container based on debian:jessie image with the necessary libraries installed as in your instructions. npm install fails with the following errors:

<EMAIL_ADDRESS>install /var/www/app/current/node_modules/pdf-fill-form
node-gyp rebuild

make: Entering directory '/var/www/app/current/node_modules/pdf-fill-form/build'
  CXX(target) Release/obj.target/pdf_fill_form/src/pdf-fill-form.o
  CXX(target) Release/obj.target/pdf_fill_form/src/NodePopplerAsync.o
  CXX(target) Release/obj.target/pdf_fill_form/src/NodePoppler.o
../src/NodePoppler.cc: In function 'QBuffer* writePdfFields(WriteFieldsParams)':
../src/NodePoppler.cc:265:16: error: 'formFields' was not declared in this scope
     qDeleteAll(formFields);
                ^
../src/NodePoppler.cc:266:12: error: 'page' was not declared in this scope
     delete page;
            ^
../src/NodePoppler.cc: At global scope:
../src/NodePoppler.cc:271:3: error: 'bufferDevice' does not name a type
   bufferDevice->open(QIODevice::ReadWrite);
   ^
../src/NodePoppler.cc:274:3: error: expected unqualified-id before 'if'
   if (params.saveFormat == "imgpdf") {
   ^
../src/NodePoppler.cc:277:3: error: expected unqualified-id before 'else'
   else {
   ^
../src/NodePoppler.cc:281:3: error: expected unqualified-id before 'delete'
   delete document;
   ^
../src/NodePoppler.cc:283:3: error: expected unqualified-id before 'return'
   return bufferDevice;
   ^
../src/NodePoppler.cc:284:1: error: expected declaration before '}' token
 }
 ^
../src/NodePoppler.cc: In function 'QBuffer* writePdfFields(WriteFieldsParams)':
../src/NodePoppler.cc:267:3: warning: control reaches end of non-void function [-Wreturn-type]
   }
   ^
pdf_fill_form.target.mk:118: recipe for target 'Release/obj.target/pdf_fill_form/src/NodePoppler.o' failed
make: Leaving directory '/var/www/app/current/node_modules/pdf-fill-form/build'
make: *** [Release/obj.target/pdf_fill_form/src/NodePoppler.o] Error 1
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/local/nvm/versions/node/v6.10.2/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:276:23)
gyp ERR! stack     at emitTwo (events.js:106:13)
gyp ERR! stack     at ChildProcess.emit (events.js:191:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:215:12)
gyp ERR! System Linux 4.9.27-moby
gyp ERR! command "/usr/local/nvm/versions/node/v6.10.2/bin/node" "/usr/local/nvm/versions/node/v6.10.2/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /var/www/app/current/node_modules/pdf-fill-form
gyp ERR! node -v v6.10.2
gyp ERR! node-gyp -v v3.4.0
gyp ERR! not ok
<EMAIL_ADDRESS>/var/ww

Also on mac, the install fails with your new version...I had it previously working, but installing the new version from npm I get the following error.
not sure if both cases are linked but I would guess so. Could you please update the install docs?
> node-gyp rebuild

2017-05-24 17:32:17.260 xcodebuild[1942:225563] [MT] PluginLoading: Required plug-in compatibility UUID E0A62D1F-3C18-4D74-BFE5-A4167D643966 for plug-in at path '~/Library/Application Support/Developer/Shared/Xcode/Plug-ins/Unity4XC.xcplugin' not present in DVTPlugInCompatibilityUUIDs
2017-05-24 17:32:17.261 xcodebuild[1942:225563] Failed to load plugin at: /Users/floriannorbertbepunkt/Library/Application Support/Developer/Shared/Xcode/Plug-ins/Unity4XC.xcplugin, skipping.  Reason for failure: *** -[__NSPlaceholderDictionary initWithObjects:forKeys:count:]: attempt to insert nil object from objects[0]
  CXX(target) Release/obj.target/pdf_fill_form/src/pdf-fill-form.o
In file included from ../src/pdf-fill-form.cc:2:
../src/NodePoppler.h:5:10: fatal error: 'QtCore/QBuffer' file not found
#include <QtCore/QBuffer>

@florianbepunkt I think that the latest merge on master has an error
I don't see tags versions :(
for now, you can use a specific commit
It works for me.
https://github.com/tpisto/pdf-fill-form/commit/bacbf8d1ed31fe8dc805c933e92aebb6fc223c00
Change your package file with:
"pdf-fill-form": "git://github.com/tpisto/pdf-fill-form.git#bacbf8d1ed31fe8dc805c933e92aebb6fc223c00"

@gatodeveloper Thank you. I can confirm... the latest merge has a bug.
But I have another interesting error: Filling the pdf on my mac works fine. Running the same app inside my docker container works except that € signs in my form field become fi ligatures.

I have now published 3.2.1 that should fix this. Please report if not.

Thanks @tpisto, Works for me.

@tpisto Confirmed. It's fixed. Thank you.


Tweek api health point should be monitored better in DockerFile

Current implementation check localhost/health, but ignore the contents

We're currently satisfied with what we get from /health


why you stopped using exampleSite?

Hi,
I'm trying to use your hugo theme, and I'm wondering why you stopped using the exampleSite feature of Hugo (which is a pretty cool feature) in the commit e2837e4ae002c8b31475f2333bcd2a53360ed2d4
Thanks

@netgala Mainly because of the following cons.

Much harder to install the theme without having to copy and paste files etc.
harder to deploy to Netlify. Yes you can write a clever build command but then the user has to replace this once they have installed the exampleSite, so again makes it harder to install.
harder to deploy to Stackbit. Stackbit previously did not support install from exampleSite, but now it does.

Can you tell me the benefits of using exampleSite? I'm open to reverting back to using it, but whats the killer reason?

We have changed back to the exampleSite structure


Update migration activities support in migration-api-overview.md

Category

[ ] Content fix
[ ] New article
[x] Example checked item (delete this line)

DELETE THIS LINE BEFORE SUBMITTING | For the above list, an empty checkbox is [ ] as in [SPACE]. A checked checkbox is [x] with no space between the brackets. Use the PREVIEW tab at the top right to preview the rendering before submitting your issue.

Related issues

fixes #issuenumber
partially #issuenumber
mentioned in #issuenumber

DELETE THIS LINE BEFORE SUBMITTING | If this fixes (aka: closes) or references an issue, please reference it here. This helps maintaining the issue list as it will (1) link the PR to the issue & (2) automatically close the issue when this PR is merged in.

What's in this Pull Request?

DELETE THIS LINE BEFORE SUBMITTING | Please describe the changes in this PR. Sample description or details around bugs which are being fixed.

Submission guidelines

!!IMPORTANT!! - All submissions must complete the baseline sections included in this template. Ignoring or deleting this template may result in closing the issue with the label type:incomplete-submission.
Follow our guidance on How To Create Good Pull Requests.
Target the main branch of this repo.
When changing a page, ensure you update the ms.date front matter wih the current date in the format MM/DD/YYYY.
Review all build checks and address the automated errors, warnings, and suggestions.
NOTE: The live site is based on the live branch. Site owners periodically refresh live branch from the main branch so merged PRs won't immediately appear on the live site. Please be patient to see your changes appear on the live site.

DELETE THIS SECTION BEFORE SUBMITTING

@paya54 Please complete the instructions in the PR description as required for new PRs. Once this is done, we'll review the PR submission.


SDK references are lost when a Specific Platform has a dependency

XRTK - Mixed Reality Toolkit Bug Report
Describe the bug
Currently, SDK assets are copied to the user's project when installed, this causes the GUID's for those assets to be regenerated to avoid conflicts.
However, if a platform's configuration has a dependency on SDK assets, the references they had stored will no longer be correct, resulting in bad configuration.
To Reproduce

Create a new XRTK project
Register the Scoped Registry and Install the XRTK.SDK package (which automatically registers the Core)
Wait for package install, copy and guid regen to complete
Add another platform, e.g. Oculus
When prompted, say "Yes, Absolutely" to register the service configuration automatically
Wait for platform to install
Run "Mixed Reality Toolkit -> Configure" to setup the scene and initial configuration
Select the MixedRealityToolkit Gameobject and observe the configuration screen in the inspector
Navigate to "XRTkRoot -> Input System Profile -> Oculus Controller Data Provider profile -> Advanced -> Lect Oculus Touch profile -> Spatial Pointer Profile"
Observe Registered Pointer Profiles are null (SDK references lost)

This occurs anywhere there is a reference to an SDK component in a platform
Expected behavior
The SDK reference should be preserved and the configuration should be valid
Actual behavior
Broken "Null" references
(also observed in Platform Hand Pointer configuration, with a reference to the SDK Hand Pointer Default Pointer prefab)
Your Setup

Operation system Windows Standalone / Android
Unity Version 2019.4.15
XRTK Version 0.2.0-preview

Additional context
Discussed in #Troubleshooting channel on Discord

I suppose the best we can do is create a map from the known guids from the sdk to the new regenerated ones, and so a search/replace on them.
Does this still happen if we install the platforms first then install the SDK?

Happens either way, simple fact, the GUID's in the UPM package are different to the GUID's when copied to the asset folder, so yet, they will always be broken

Tested with UPM, sadly the issue failed testing

How did it fail?

I had forgotten to update the sdk with the breaking changes. The PR is open for that now.

I think we also still need to add the sdk as a dependency to all the platforms for this to work correctly


Migrate inactive users to Alumni group.

@johngian can you please r this?
The purpose is to migrate all users with no activity since the end of October to the Alumni group.
The migration is going to take place the first days of 2016.

Tests are failing. Please take a look at:
https://travis-ci.org/mozilla/remo/builds/98305898#L632

Code looks OK in terms of functionality. r+wc
I suggest we snapshot the DB before the migration just in case we need to rollback.


Use Conjur CLI v8.0

Desired Outcome
Update helm chart notices to use CLI v8.0.
Implemented Changes
Updated README.md and conjur-oss/templates/NOTES.txt to refer to the v8 Conjur CLI tag and use the updated command syntax.
Updated CHANGELOG.md in preparation for new release of the helm chart, v2.0.6.
Connected Issue/Story
Resolves #178
CyberArk internal issue ID: ONYX-33379
Changelog

[x] The CHANGELOG has been updated, or
[ ] This PR does not include user-facing changes and doesn't require a
CHANGELOG update

Test coverage

[ ] This PR includes new unit and integration tests to go with the code
changes, or
[x] The changes in this PR do not require tests

Documentation

[x] Docs (e.g. READMEs) were updated in this PR
[ ] A follow-up issue to update official docs has been filed here: [insert issue ID]
[ ] This PR does not require updating any documentation

Behavior

[ ] This PR changes product behavior and has been reviewed by a PO, or
[ ] These changes are part of a larger initiative that will be reviewed later, or
[x] No behavior was changed with this PR

Security

[ ] Security architect has reviewed the changes in this PR,
[ ] These changes are part of a larger initiative with a separate security review, or
[x] There are no security aspects to these changes

Yes, I'm working on that already :)