Feature request - Customisable and floating Roll 3d6 button

The Roll 3d6 button is the most GURPSy thing that GURPS for Foundry has - I am fully ready to admit that.
However, sometimes, when you use a lot of modules like I do, the button becomes a bit cumbersome.
It would be great if the button could be resized, moved, or maybe (Steve Jackson forgive me for what I am saying) hide it altogether.
Less crucial, but since we're going down this road why not also give the option to customise the button image? This way we could maybe adapt it to the setting and the tone of the campaign?

Current solution:
Settings:

Positions are:

Left, neat user toolbar
Right, near chatbox

Bucket at left (default position, custom 3d6 dice):

Bucket at right, defaut 3d6 icon

@chrismaille Hi! Thanks for implementing the idea, but there is one small bug. If you use a user interface module that centers the macro panel, then with these changes applied, this does not happen, and the macro panel remains in its normal position

@chrismaille for example RPG Styled UI

I see. RPG Styled UI is a v11 module that mainly hardcode the hotbar CSS in the new position. I think it's risky to try to support this changing foundry interface CSS like they do. But I might look at this module again if they port it to v12.
As a workaround, my suggestion is to use the right option. Please let me know if this helps.

@chrismaille I'm using a fork of this module, I don't know how much suitable it is for use in V12 - https://github.com/Oxy949/rpg-styled-ui . Unfortunately, using different positions does not change anything ; ( . But anyway, thanks for the new feature!


conda installation is broke

approxposterior v0.2rc0 depends on emcee version > 2.2.1, their rc02.  Currently, that version is not available on conda-forge, hampering the conda installation.  Installation using pip and from source still work, but I need to figure out the conda issue.

emcee has officially made v3 the default option (see here), so I should be able to fix the feedstock!

I fixed the feedstock!


Query param values missing

I'm running into an odd problem where query params seem to almost work in the output, but not quite. Instead of seeing the params used in the actual test request, I get something like https://api.example.com/v1/foo?group_by=&timeframe=. The params used do show up, just as empty values.
I'll try to dig into this on my end, figured I'd go ahead and open an issue in the meantime in case someone else has seen this.
Thanks for taking this project over!

Hi @mmcc,
I'm not entirely sure what you mean. Where exactly do you expect the parameter values to appear?

Do your parameters have descriptions in the controller annotation? I found
that unless they do the placeholders don't show.
On 27 Jun 2017 8:56 a.m., "Mathias Polligkeit"<EMAIL_ADDRESS>wrote:

Hi @mmcc https://github.com/mmcc,
I'm not entirely sure what you mean. Where exactly do you expect the
parameter values to appear?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/KittyHeaven/blue_bird/issues/20#issuecomment-311269966,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALxVeQzA_eV36jGoTP4Dm3gA3wCjnwEks5sIKeGgaJpZM4OEG3_
.

I would expect the parameters to include the values used in the response, or not be there at all, but the current feels confusing at best. It could be that I'm missing something entirely, but I think I'd rather the query params not be in the example at all.
Here's an example:
In the controller:
api :GET, "/v1/foo" do
    title "foo example"
    description "This is an example"
    parameter :group_by, :string, [optional: true, description: "Field to group the data by"]
    parameter :timeframe, :array, [optional: true, description: "Timeframe array", default: ["24:hours"]]
    parameter :page, :integer, [optional: true, description: "Current page"]
    parameter :filters, :object, [optional: true, description: "Filter key pairs"]
  end

Algolia output

So, I think part of the confusion here is that only the query params used in the test are passed through to either of the examples, but their values are ignored. I would expect the URL on the right to have the values that were used in the test, i.e.,
GET https://api.example.com/v1/foo?group_by=os&timeframe="24:hours"

Hope this helps!

Now I understand mean. If you want to display some values in the url, you have to define the example key in the parameter options. For example:
parameter :group_by, :string, [optional: true, description: "Field to group the data by", example: "something"]

The url displayed in the right column is generated by aglio, and aglio uses the example values of the parameter definitions for this. The api blueprint specification does not allow to specify different example values per request. If we were to extract the example values from the test requests automatically, we would have to arbitrarily choose a single query parameter value out of all the tests for the route. I'd rather not do that.

Got it. Yeah, that's fair enough, I think it's just a little confusing that the API request example URL could be largely unrelated from the example request below it. Sounds like this is more an API Blueprint / Algolia limitation than BlueBird, though.
On the parameter front, I'm currently generating the apib file and going back and manually adding in the request body parameters manually. Unless I'm mistaken, BlueBird currently only supports documenting URI parameters. Is this on your radar or should we consider jumping on that ourselves? Not a big deal in the short term, but long term it'd be nice to have everything documented in the api DSL.

You are right, currently it is only possible to describe path and query parameters.
The API Blueprint specification handles body parameters separately. You can describe body parameters in the attributes section.
In Swagger, you can describe the body parameters  by defining a single body parameter with a schema object.
I'd like to see better handling of request bodies and body parameters in the future, but I'm not working on that right now. So if you want to have a go at it, go ahead! Before implementing anything, we should first think about how to solve all the steps in a nice way:

Extracting the request bodies / body parameters. (Currently, we only read the body_params from the Plug.Conn. Can/should we extract the original body somehow in the tests?)
Providing a dsl to describe those body parameters. Should the body parameters be described the same way as the path and query parameters, or should they be handled with an additional macro parameter (by taking a hint from api blueprint and adding attributes to the api macro)?
Creating an internal representation from the logged test requests and the api macro descriptions.
Converting the internal representation to both api blueprint and swagger format.


put v-if in place of v-show to destroy element on hide

#225

@brunokunace
Did you try it? because having v-if you can not clone the element and should give error

@luisDanielRoviraContreras yes, i try, v-if remove the div from v-dom, putting  on the place.


fix(specs): make languages an enum

🧭 What and Why
Restrict allowed languages for settings like queryLanguages and indexLanguages to the list of supported languages, as currently the clients suggest that all string values are accepted, which would lead to errors when using.
Changes included:

Introduce a new enum supportedLanguage
Restrict items for indexLanguages, queryLanguages, removeStopWords, ignorePlurals, and the dictionary response to be of supportedLanguage type.

🧪 Test

Kotlin client fails to build.
Swift tests fail with some type error I don't quite understand.

Inre-run the Kotlin one to check it wasn't just a one-off error
The Swift error is interesting, let me take a look in the generated code

It seems kotlin doesn't understand the oneOf anymore, now that there is a $ref in one of the element...
I can also take a look

I found the issue for Swift, I'm gonna push a fix

I found the issue for Kotlin, I'm gonna try to push a fix 😄

@aallam This commit fixes the issue, but I would love to have you to take a look at it, to confirm

@kai687 All good, everything is fixed for now, you can merge!
@aallam will take a look next week, and we will push a fix in another PR if needed.


Bug third and fourth player trying to play crashes everything and freezes the third and fourth

good night friends, I spent all day obfuscating the codes, hours without result and continuing to test until things work, I am managing to play with the server hosting on my machine, I had problems connecting a third player and a fourth, because it crashed, it was slow, then reading other errors i had the idea of restart the game, and that solves the problem, i will be testing playing 5 players to play bombeman and other games
restart

the bug is this

I'd recommend waiting for the next version of netplay

Good afternoon, I admire your dedication here in Brazil, I went to check the js code, impossible to see, I used a website to format it and it was visible to obfuscate, debugging and I ended up finding the netplay error and I was able to play, and debugging I managed to circumvent the limit of 2 players per room
https://beautifier.io/

Why are you editing the minified files

I'm using version 31.6 that takes netplay, I tried to make it work without the reduced and it gave an error on the console and it didn't work, if we happen to talk on discord or in the group, I understand javascript and other languages and it could help

Yeah,  just ping me in the emulatorjs discord. The invite link is in the emulatorjs readme

in fact, the emulator is directing to this reduced one, I tried to rename it to the complete one and it gave the error, I will start to understand the codes more and see the difference of each one

in fact, the emulator is directing to this reduced one, I tried to rename it to the complete one and it gave the error, I will start to understand the codes more and see the difference of each one

Enable EJS_DEBUG_XX


For the possibility of building florist with drake

Before starting I do think this may end up as a "WONTFIX" because http://www.ada-auth.org/standards/rm12_w_tc1/html/RM-13-7.html does not define Interrupt_Management, however if there is a potential for a system.interrupt_management implementation it might be the step needed to get florist built against drake.
Using florist-gpl-2017-src.tar.gz I am able to modify Makefile.in to include the necessary --RTS option like so:
- GPRBUILD_FLAGS = $(GCCFLAGS) $(LIBRARY_TYPE_OPTION) $(PROJECT_FLAGS)
+ GPRBUILD_FLAGS = $(GCCFLAGS) $(LIBRARY_TYPE_OPTION) $(PROJECT_FLAGS) --RTS=/usr/lib/drake/x86_64-pc-linux-gnu/7.2.0

After the usual ./configure I get the following errors from make (gprbuild):
gprbuild -p -O2  -XTHREADS=True   --RTS=/usr/lib/drake/x86_64-pc-linux-gnu/7.2.0 -Pflorist -cargs -O1
Setup
   [mkdir]        object directory for project Florist
   [mkdir]        library directory for project Florist
Compile
   [Ada]          posix-timers-extensions.adb
posix-timers-extensions.ads:37:21: "SYSTEM.INTERRUPT_MANAGEMENT" is not a predefined library unit
posix-timers-extensions.ads:37:21: "POSIX.TIMERS.EXTENSIONS (body)" depends on "POSIX.TIMERS.EXTENSIONS (spec)"
posix-timers-extensions.ads:37:21: "POSIX.TIMERS.EXTENSIONS (spec)" depends on "POSIX.TIMERS (spec)"
posix-timers-extensions.ads:37:21: "POSIX.TIMERS (spec)" depends on "POSIX.SIGNALS (spec)"
posix-timers-extensions.ads:37:21: "POSIX.SIGNALS (spec)" depends on "SYSTEM.INTERRUPT_MANAGEMENT (spec)"
gprbuild: *** compilation phase failed

Florist is just an implementation of POSIX.5(Ada interface to the POSIX system calls), specialized to GNAT.
Based on that, on GNAT, System.Interrupt_Management is not declared in System, but a child package, s-intman.ads. It's not marked as GNAT library. It belongs to details of implementation. If I had to say, it should not be used from others.
Also, s-intman.ads is not last one. Even supposing it's emulated in drake, in next, you will see GNAT.Task_Lock and Interfaces.C_Streams are missing.
If we grapple squarely with this issue, making new version of POSIX.5 for drake is legitimate way. However, the translate headers C.* have already been existing, so I don't feel it is necessary. The functions of those are same.
OK, Indeed, the most part of existing Ada libraries not only florist are specialized to GNAT. A compatibility layer is necessary in practical use.
I had thought so too a few years ago and tried to make it, started from https://gist.github.com/ytomino/4649592 that is the minimum for AUnit.
Then, I gave up after 10 days.
Even either "POSIX.5 for drake" or "compatibility layer", those are large works and beyond my capacity...
When focusing only on a minimum compatibility layer only for florist, it may be satisfied with making 3 or 4 packages, perhaps.

Thank you for clarifying the situation.  I suppose it does make sense that florist is an implementation based on GNAT's runtime.  The amount of work is even more significant than I expected.
I'm not pressing for this to be added so feel free to ignore this issue since it is a tad out of scope.

I have pushed the previous and halfway minimum compatibility layer.
Roughly I have succeeded to build Florist with drake in Mac OS X.
However, there are many unimplemented. (I faked with raise Program_Error)
I don't think it's able to withstand practical use.
If you try, here is an experimental procedure:
cd drake
patch -p0 ../gnat4drake/florist.diff
make ... # install
cd gnat4drake
./configure # generating gnat.gpr and c.gpr
cd ../florist
./configure ...
vi Makefile # same as your change
vi florist.gpr # insert 'with "gnat";' to the first line
export GPR_PROJECT_PATH=$PWD/../gnat4drake
make

I had unexpected that the patch for drake's adainclude is needed. Honestly, I don't want to apply this change into the repository of drake. Probably this patch will be left.


failure on initial contact

Stand-alone use.
Ping works just fine, but:
DEBUG:pytradfri.util:JSON file not found: tradfri_standalone_psk.conf
Please provide the 'Security Code' on the back of your Tradfri gateway:  xxxxx
DEBUG:pytradfri.util:JSON file not found: tradfri_standalone_psk.conf
DEBUG:pytradfri.api.libcoap_api:Executing nnn.nnn.nnn.nnn post ['15011', '9063']: {'9090': 'xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx'}
DEBUG:pytradfri.api.libcoap_api:Received: v:1 t:CON c:POST i:24f3 {} [ ]
{"9091":"xxxxxxxxxxxxxxxxxxxxxx","9029":"1.4.0015"}
Traceback (most recent call last):
  File "/usr/lib/python3.5/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
File "/usr/lib/python3.5/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/usr/local/lib/python3.5/dist-packages/pytradfri/__main__.py", line 48, in <module>
    psk = api_factory.generate_psk(args.key)
  File "/usr/local/lib/python3.5/dist-packages/pytradfri/api/libcoap_api.py", line 164, in generate_psk
    self._psk = self.request(Gateway().generate_psk(existing_psk_id))
  File "/usr/local/lib/python3.5/dist-packages/pytradfri/api/libcoap_api.py", line 96, in request
    return self._execute(api_commands, timeout=timeout)
  File "/usr/local/lib/python3.5/dist-packages/pytradfri/api/libcoap_api.py", line 90, in _execute
    api_command.result = _process_output(return_value, parse_json)
  File "/usr/local/lib/python3.5/dist-packages/pytradfri/api/libcoap_api.py", line 195, in _process_output
    return json.loads(output)
  File "/usr/lib/python3.5/json/__init__.py", line 319, in loads
    return _default_decoder.decode(s)
  File "/usr/lib/python3.5/json/decoder.py", line 339, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python3.5/json/decoder.py", line 357, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Does it work if you run the lib from docker?

Haven't tried yet. Is there a docker image somewhere? Seems the download tar file doesn't contain the "./script/dev_docker" script.

Yes, there are docker images in the root folder that are run using the mentioned script. The tar ball only contains files needed to run the lib from within Python. so you need to clone this repo to get the docker files.

Also, what's the content of the tradfri_standalone_psk.conf file? I suggest redacting the key before posting here.

Docker on raspbian seems to be a bit of an issue. Where is tradfri_standalone_psk.conf supposed to live - I don't seem to be able to find it?

In the same folder as where you clone the repo or run pytradfri from. Make sure you have read/write access in that folder as a file is created.

Sounds like an error in your environment. Is the directory also writable by your user? What happens if you run as root?

Yes, ran it from (writeable) home directory, as root. No file created.

I would also try running from Docker and make sure the key on the gateway is correctly written in. Other than that, I’m afraid I’m out of ideas.

Sorry, I’m out of clues. It’s worked on my rpi3 in the past, at least in the docker.

Could it simply be the Trådfri base station is broken?

After some discussion, we've come to the conclusion that there's not much we can do here but to keep the project up to date with the firmware: https://github.com/ggravlingen/pytradfri#known-issues

On 10-04-19 19:54, Patrik wrote:

After some discussion, we've come to the conclusion that there's not
much we can do here but to keep the project up to date with the
firmware: https://github.com/ggravlingen/pytradfri#known-issues

OK, thanks for looking into it!
Julf


Update language to reflect current menu names and options in VSCode

The names of various menus and options in VSCode has changed since this tutorial was written, making it somewhat confusing.  This PR changes that language to match the current version of VSCode.

Docs Build status updates of commit 40aa162:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-debugging.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

@mscharlock, can you please take a look at these changes?

Docs Build status updates of commit 23ca33c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-debugging.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 894dbda:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-debugging.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Thanks for the contribution! Appreciate it! :)


Duping a prop doesn't show a ghost of it.

I was recently played Garry's Mod, and everything was fine regarding Adv Dupe 2. Next time I logged on it stopped working correctly though. When you dupe a Garry's Mod prop a ghost of it won't show up and when you dupe a prop which is from a addon like Sprops it will show up as errors for a ghost. When I load a saved dupe it shows the ghost fine though. Here is the error I get when I dupe something:
[ERROR] addons/adv dupe2/lua/weapons/gmod_tool/stools/advdupe2.lua:1738: attempt to index local 'Phys' (a nil value)
  1. MakeGhostsFromTable - addons/adv dupe2/lua/weapons/gmod_tool/stools/advdupe2.lua:1738
   2. fn - addons/adv dupe2/lua/weapons/gmod_tool/stools/advdupe2.lua:1770
    3. unknown - addons/ulib/lua/ulib/shared/hook.lua:183

If anyone knows a fix to this problem it would be highly appreciated. (I have tried reinstalling the addon already)

wait so was this fixed?
On Tue, Sep 15, 2015 at 11:16 AM, Jstone2540<EMAIL_ADDRESS>wrote:

Closed #58 https://github.com/wiremod/advdupe2/issues/58.
—
Reply to this email directly or view it on GitHub
https://github.com/wiremod/advdupe2/issues/58#event-409579796.

I've never seen it happen.


Add cupy func generate

This is re-submit of #1018, which had been merged, but was reverted as #1998. I and @hisa0507-san made the following changes from #1018:

Explicitly write that the code is partially borrow codes from corresponding NumPy's objects.
Move part of document of CClass and RClass to c_ and r_, respectively.

Thank you for your comment. I fixed the code.

Can you check flake8 and merge master branch?

OK, LGTM

Thank you.


Base slave v.3.7 and 3.7.0 images have newer oc

$ sudo docker run --rm --entrypoint=sh -ti docker.io/openshift/jenkins-slave-base-centos7:v3.7
sh-4.2# oc version
oc v3.9.0-alpha.0+c863312-37
kubernetes v1.8.1+0d5291c
features: Basic-Auth GSSAPI Kerberos SPNEGO
$ sudo docker run --rm --entrypoint=sh -ti docker.io/openshift/jenkins-slave-base-centos7:3.7.0
sh-4.2# oc version
oc v3.8.0-alpha.0+dcfc49c-236
kubernetes v1.8.1+0d5291c
features: Basic-Auth GSSAPI Kerberos SPNEGO

This breaks some oc operations on 3.7 clusters - e.g. creating jenkins pipelines

this is being fixed in https://github.com/openshift/jenkins/pull/454

docker.io/openshift/jenkins-2-centos7:v3.7 has the oc v3.7 client in it.
the v3.9/latest images will continue to have oc client's from origin master.


malloc(): largebin double linked list corrupted (nextsize)

I am getting this error malloc(): largebin double linked list corrupted (nextsize) intermittently and can't run this for more than 5 registration pairs.

Haven't seen this before. How large is your RAM?

It is 32G. It seems that FCGF gives too many inliers for TEASER and allocating too much memory.

For reference: error msg is from here: https://github.com/kraj/glibc/blob/master/malloc/malloc.c#L3868

I tried on Intel CPU : OK,
but on AMD CPU: corrupted (malloc).
Anyone have an idea?

For AMD CPU users,
Code:
import os
os.environ["OMP_NUM_THREADS"] = "4"
Terminal:
export OMP_NUM_THREADS=4  #~ Max thread
#echo $OMP_NUM_THREADS     #show 4
python ./filename.py


submitting labs 1-3

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Submitting Labs 1-3
What did you learn from the changes you have made?
Learned the following:

Creating my first deep learning model
how to adjust learning rate to influence the outcomes of the model
using different models to test for outcomes

Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
N/A
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
N/A
How were these changes tested?
Tested the code by running it couple of times
A reference to a related issue in your repository (if applicable)
Checklist

[x] I can confirm that my changes are working as intended

I have added labs 4,5,6 to the same branch.


set Solana dependency ranges

Due to an issue in Anchor, installing the CLI from crates.io or running cargo install locally is failing. This can be fixed by pinning Solana dependencies so they don't go higher than 1.14. This PR also pins borsh to 0.9 patches as there are breaking changes in borsh 0.10.0 that don't always play well with Solana crates.

Thanks for the patch!


ACCEPTANCE: Post Failure

@geetika-jain the acceptance tests are failing during the build because we didn't put a local db connection string in the appSettings.json file.
Put a local db connection string like we did here please:
https://github.com/microsoft/mixedreality.dmx.core/blob/main/DMX.Core.Api/appsettings.json


Display Rotation Crashes Chromebook

Please paste the output of the following command here: sudo edit-chroot -all
Password:
name: cinnamon
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/cinnamon...
crouton: version 1-20200724165050~master:08dac970
release: bionic
architecture: arm64
xmethod: xorg
targets: xiwi,x11,xorg,core,cli-extra,keyboard,touch,audio,extension,gtk-extra,gnome
host: version 13560.0.0 (Official Build) canary-channel kukui
kernel: Linux localhost 4.19.152-10453-gdb6d2cd8e743 #1 SMP PREEMPT Wed Oct 28 00:52:55 PDT 2020 aarch64 aarch64 aarch64 GNU/Linux
freon: yes
Unmounting /mnt/stateful_partition/crouton/chroots/cinnamon...

Please describe your issue:
Using the command xrandr -o left to rotate the screen crashes my Chromebook and restarts it. Display isn't fixed on startup.
If known, describe the steps to reproduce the issue:
sudo startcinnamon to enter the chroot
Once inside chroot, Ctrl+Alt+C to bring up the terminal
Inside the terminal, xrandr -o left, which leads to a restart.
Alternatively, going to display settings and manually rotating the display leads to a restart.
This is also a problem I occasionally have with chroots in general.
[Note: I used Tenn1518's repo https://github.com/Tenn1518/crouton-cinnamon to use the Cinnamon desktop environment]

Have you tried other targets (not form other repos)? If so does this happen with them?

I have. It occasionally happens with KDE, LXDE, and used to happen every time with Gnome and XFCE.
Cinnamon seems to be working now, so I'll close and reopen if necessary.
Thanks!


Handle snow objects with no namespace in api calls

Description of changes:
Without this, the new kubectl implementation fails since it doesn't allow an empty string for namespace. Most of the time, users pass the eks-a objects with no namespace, which means they get created in the default namespace.
Before it was miraculously working since kubectl works even if you pass the --namespace flag with no value, it just uses default.
This makes sure all API calls that try to extract the namespace from the eks-a objects pass a namespace, even when the objects don't have it set.
It also ignores errors when trying to delete an object that doesn't exist.
Testing (if applicable):
@taneyland ran manual tests with snow for both upgrade and delete 🙏🏻
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/cherry-pick release-0.15

/lgtm

/approve


Added common in ising documentation

Summary
Fixes #1255
The fix for documentation of 'common' not being autogenerated.
Details and comments

@manoelmarques Does it look good?

@manoelmarques Could you please review the PR?


Customizing source line break behavior

I don't know if this is possible and if there is demand but:
Current line breaking behavior

source
compiled

\n
[ ]

\
linebreak()

>1 times \n
parbreak()

Want to be able to customize it to

source
compiled

\n
[ ]

\n\n
linebreak()

>2 times \n
parbreak()

\
linebreak()

or

source
compiled

\n
linebreak()

> 1 times \n
parbreak()

\
continue line

Bonus if this behavior can be changed depending on the context (e.g. in a paragraph vs table cell) and if rules for trimming line breaks in certain contexts can be defined.
Doing this in Typst code right now is almost impossible, mostly because

show rules like #show regex("\\s"): white_char => ... never capture line breaks in the source
\n gets compiled to [ ] (making it indistinguishable from many other things that generate [ ])
the content tree is currently not well traverseable and mutable.

Regarding the demand: one major use case would be typesetting poems. Those have manual line breaks by definition, and putting '' at the end of every line quickly gets annoying. I'd gladly do something like
#let poem(
  text
) = {
  set par(join-lines: false)

  text
}

I can imagine adding a set rule to respect line breaks in source and turn them into linebreak() instead of [ ].
Configurable \ for line continuation, in contrast, would in my opinion do more harm than good. It's really the opposite of what it does now and it would be hard to judge at a glance what a piece of code does. This problem already comes up a bit with the hard linebreaks, but there I see more of a use case.

I really dislike overriding symbol behaviour if not necessary.
Could a field be added to the Space node in the ast or just a new node be added? The source already has the comment
/// Whitespace in markup or math. Has at most one newline in markup, as more
/// indicate a paragraph break.

Maybe it would be best to keep the information about the type of whitespace in the node?

AST nodes already contain their complete text, so it's easy to find out the number of newlines in it.

I would also be interested in finding a solution to this. I expected that with the recent changes to the realisation subsystem, I could simply #show "\n": linebreak() but that doesn't work.

The show rule doesn't operate on the source text but on the realized text. In the case of your show rule, there are no "\n" in the realized text, so it doesn't match. (If it did match, it would replace a linebreak with a linebreak, to no real effect.)

Regarding the demand: one major use case would be typesetting poems. Those have manual line breaks by definition, and putting '' at the end of every line quickly gets annoying. I'd gladly do something like
#let poem(
  text
) = {
  set par(join-lines: false)

  text
}

I wonder if any work could be done in this direction? I really enjoy using Typst and, in particular, would like to expand its usage to poetry. Such a feature would be of huge help, because so far editing a poem requires manually add a lot of line breaks.


Add env = "CARGO_MANIFEST_DIR" to clap_cargo::Manifest

This would allow this value to be injected via the environment variable CARGO_MANIFEST_DIR

Upon further investigation, this ENV variable is not provided to custom scripts


Adjust Graphics pane size so that text chars do not distort grid at 100%

Duplicated (and now fixed) in #733


RATIS-1118. Remove DataStreamServerRpc.start()

https://issues.apache.org/jira/browse/RATIS-1118

R @runzhiwang @szetszwo
Do you think wether this change make senses?
The drawback of the start() is people must remember to call it, which seems too error prone when there are many start(), e.g. it might be easy to forget start one of the rpc.
Unless there is something must be done before the data stream server start, otherwise I think we can remove the start() as well.

For servers, they usually has a start() method.  I guess servers may involve more resources so that the initialisation often takes a long time.  It easy to manage to separate the initialisation and the start methods.  e.g. io.grpc.Server has a start method.
In contrast, clients are usually lightweight and connecting to a single server/service.
It is a bigger change to remove the DataStreamServerRpc.start() method since RaftServer also has a start() method.  We must make them consistent.

@szetszwo  makes sense.
Will close this PR now and if there is a need, we can continue discussing in the future.


add a workaround in workflow for unit test py3.6 bug

resolves #28

Seems the latest ubuntu version for Python 3.6 is 20.04, currently ubuntu-latest stands for 22.04
      {
        "filename": "python-3.6.13-linux-20.04-x64.tar.gz",
        "arch": "x64",
        "platform": "linux",
        "platform_version": "20.04",
        "download_url": "https://github.com/actions/python-versions/releases/download/3.6.13-99136/python-3.6.13-linux-20.04-x64.tar.gz"
      }
    ]


0.0.59 Release

0.0.59 Release.

Codecov Report

Merging #948 (fe8e613) into master (9c278b8) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #948      +/-   ##
==========================================
+ Coverage   82.56%   82.60%   +0.03%
==========================================
  Files         190      191       +1
  Lines        4802     4812      +10
==========================================
+ Hits         3965     3975      +10
  Misses        837      837

Impacted Files
Coverage Δ

zhaquirks/xiaomi/mija/sensor_switch.py
58.69% <ø> (ø)

zhaquirks/philips/ltd011.py
100.00% <100.00%> (ø)

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 9c278b8...fe8e613. Read the comment docs.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

10 of 10 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 82.606%

Totals

Change from base Build 988203476:
0.04%

Covered Lines:
3975

Relevant Lines:
4812

💛  - Coveralls


Doesn't work with Open Composite

something about IVROverlay_25 being unkown/unsupported.

Do you have a game and the open composite version you are using to reproduce the issue you have encountered?

Skyrim VR, and open composite hasn't updated in a while, just grab the newest system wide launcher

Btw this seems to be because open composite does not support overlays, so would it be possible to have a toggle for the overlay, so it still works on open composite without trying to render something it can't?

Sorry I did not have the time to test open composite, especially as i have the Valve Index which needs StreamVR.
Did you test the latest Reshade version 5.3.0 if the issue is still present? (Just replace the DLL from the Vrtoolkit release backage)

No the latest official reshade also has the issue, the only dll i've found that doesn't have the issue is one I got from an early build of reshade for VR, forgot exactly from where, but you can find a copy of it in the mod downloadable from here https://www.nexusmods.com/skyrimspecialedition/mods/47989

Yes thats the first reshade release fholger did on implementing VR. It does not have the streamVR overlays and due to that it does not crash. Guess the ImGui overlay must be optional when steamVR is not loaded avoiding a crash. No time to look into this but hopefully the reshade dev crosire will do it one day for open composite users.


Output minification step breaks SourceMap chain with inlined sourceContents

Problem:
When I enable output minification and have enabled inlined sources (called sourceContents and enabled by setting either the builder option sourceMapContents to true or setting sourceMaps to 'inline') the final SourceMap generated for my build does no longer contain them.
Reason:
When minify: true the final minification step in lib/output#writeOutputs uses uglify-js for source minifcation. But in there neither the uglify option sourceMapIncludeSources is set (don't know if this is mandatory but it is told in the official uglifyjs docs) nor are the sourcesContent items passed to the uglify.SourceMap instance by calling setSourceContent for them.

👍

Released in 0.15.16.


DateClock key violation on nonconsuming Advance and Rewind interface choices.

Reopening of #631.
This PR proposes making Advance and Rewind of the Time interface consuming. Alternatively, if choices are required to be nonconsuming at the interface level, then archival can be push to the implementation level in DateClock instead.

@matteolimberto-da should I still reorder the imports?

@haoyang-da we discussed internally and would prefer to rather add the "archive self" in the implementation, as it leaves the interface untouched and gives more flexibility to the implementation. Could you change the PR to this end?

Sure let me make the change.

@matteolimberto-da should I still reorder the imports?

Yes please, we have them ordered throughout the repo (I incorrectly assumed CI would enforce that)


chore: release 4.0.0

:robot: I have created a release *beep* *boop*
4.0.0 (2019-06-07)
⚠ BREAKING CHANGES

deps: this will ship async/await in the generated code
upgrade engines field to >=8.10.0 (#587)

Bug Fixes

deps: update dependency @google-cloud/common-grpc to v1 (#607) (084dc8c)
deps: update dependency @google-cloud/paginator to ^0.2.0 (#560) (8fe33a1)
deps: update dependency @google-cloud/paginator to v1 (#593) (bfb2255)
deps: update dependency @google-cloud/precise-date to v1 (#600) (f52494f)
deps: update dependency @google-cloud/projectify to v1 (#591) (22713c1)
deps: update dependency @google-cloud/promisify to v1 (#592) (cb76922)
deps: update dependency arrify to v2 (#577) (6e0ddc8)
deps: update dependency google-auth-library to v4 (#599) (21b9995)
deps: update dependency google-gax to ^0.26.0 (#586) (0f88be2)
deps: update dependency merge-stream to v2 (#624) (3aa676d)
deps: update dependency p-queue to v4 (#558) (7547e21)
DEADLINE_EXCEEDED is no longer retried (#598) (1cac4fc)
include 'x-goog-request-params' header in requests (#573) (e0cb9dc)
treat deadline errors as idempotent (#602) (b3d494a)
deps: update dependency p-queue to v5 (#578) (7827fb4)
docs: add google.type namespace (#605) (5cc6dc1)

Build System

upgrade engines field to >=8.10.0 (#587) (970d335)

Features

add .repo-metadata.json and move to new README template (#636) (11007cf)
support apiEndpoint override (#634) (6a5eb36)
support apiEndpoint override in client constructor (#639) (f6ebb27)

Miscellaneous Chores

deps: update dependency gts to v1 (#584) (135ac6d)

@JustinBeckwith #622 seems through?

Yup!  Right now failing tests look to be a problem though 🙃

Kind of funny these can go out of SLO, not sure if that means we should apply that label or if we should be using it as an indicator to cut a release.

That means we should be cutting a release :) We have fixes and features that have been sitting here for at least 2 weeks.  That having been said - this one is hella complicated, and we need to make sure we rationalize the gax update, grpc vs grpc-js, grpc-gcp, etc.

Blocked by #644

Release is at https://github.com/googleapis/nodejs-spanner/releases/tag/v4.0.0

The release build has started, the log can be viewed here. :sunflower:

:egg: You hatched a release! The release build finished successfully! :purple_heart:


[FEATURE REQUEST]: Adding an AI Chatbot to the Website

Integrate an AI-powered chatbot into the website to enhance user interaction, provide instant support, and improve overall user experience.
Chatbot Functionality:
Answer common user queries
Provide information about our services/products
Assist users with navigation on the website
Collect user feedback and provide initial support
Expected Outcomes:
Improved user engagement and satisfaction
Reduced response time for user inquiries
Enhanced ability to gather user data for future improvements

@akshitagupta15june please assign me this issue under gssoc.


Update storsimple-configure-mpio-windows-server.md

Some edits to the general step-by-step instructions

@sadrat
This article is no longer in the StorSimple TOC or in the repo. Not sure if the contributor is using an old version of the classic portal that was retired and deleted a while back. Thanks!

@alkohli Thanks! But in that case what do you want to do with this pr?

@SBudo  I will port your changes (if applicable) to the new version of this article! Thanks very much for your feedback!
@sadrat
Could you please delete this PR by the end of this day as the changes cannot be merged into the file in this PR? These will need to be done in a different PR. I am opening another PR and will send you the link shortly.

@sadrat
Here is the PR that contains the changes from this PR #36291 I will now close this PR. Thanks!

@alkohli sorry but I cannot close PRs. Since you are the owner of the content, you would need to close the PR.


fix(modal): Fix focus when the dialog is close or cancelled

This can be reproduced on the sample page on http://angular-ui.github.io/bootstrap/#/modal

In the Dialog section click 'Open me!' button
Close the Dialog
Hit tab key

Result: focus is lost in the page
According to http://www.w3.org/TR/wai-aria-practices/#dialog_modal
When the dialog is closed or cancelled focus should return to the element in the application which had focus before the dialog is invoked. This is usually the control which opened the dialog.

Thanks for this - this is good work.


Fix makefile for AZP

The AZP machine doesn't support override command :(

Should be fixed on master


Change the httpClient timeout

Is there a way to manage the internal httpClient Timeout? The default 100sec timeout is to quick for my usecase. thank you.

At the moment there is no way to specify that. It might make sense to extend the GroupShareClient class with a settings structure that will then be passed across the implementation. Happy to accept PRs that would enable this.

Thank you. Sure i'll setup the project on my workstation!


Explain common middleware config

For details see issue #13.
Should probably also be merged into the dev branch.

Thanks @chefe - I was also quite confused for a while.

Thanks :)


Add and modify shaders used for fullscreen quads

General changes:

Adding knob --fullscreen-quads-type to toggle between shader type (noise / solid color / texture)
Refactoring shaders related to quads
Updated UI visible/invisible logic for knobs related to quads
Instead of rebuilding pipelines for fullscreen quads during Render(), all 3 are built during setup and just switched between

Solid color shader

Now visibly different hues for consecutive quads
Reduced options: Red, Blue, Green, White

Texture loading shader

Same texture loaded for every quad, 1:1 pixel:texel
Added new large texture, texture loading shader

Minor changes

Moved skybox texture to the benchmarks asset folder

Note: I added a separate knob for the type since there will be a texture option in the future

Note: I am not sure about if there is still a validation error on Linux relating to push constant size now that there are three separate pipelines, please let me know if that shows up.

I thought it made more sense to include the texture shader implementation in this PR as well so I merged the two.

I thought it made more sense to include the texture shader implementation in this PR as well so I merged the two.

make sense, thanks!

Should be ready for merge now


Question - Push notification & security system

@Fleker Is there any way to send push notification using gcm?
I added a virtual security system, searched for motion-door windows sensors but they are not available...
Is there any plan for adding security system to touch control?
Arm - Disarm like on off maybe... (becuase i have to arm-disarm by talking to asistant which has a language limit)

No, there is no push notification functionality. If you want to notify a change in the device state, you would use report state.
I can't provide a timeline on adding touch controls for specific traits.

Thank you, it is good to know about push notifications.
I will go with this
when exports.reportstate = functions.database.ref("washer").onWrite(async (change, context) => {...finished; simple "subscripe to topic" mobile app will be a solution.
const functions = require('firebase-functions');
const admin = require('firebase-admin');

admin.initializeApp(functions.config().firebase);

//Now we're going to create a function that listens to when a 'Notifications' node changes and send a notificcation
//to all devices subscribed to a topic

exports.sendNotification = functions.database.ref("Notifications/{uid}")
.onWrite(event => {
    //This will be the notification model that we push to firebase
    var request = event.data.val();

    var payload = {
        data:{
          username: request.username,
          imageUrl: request.imageUrl,
          email: request.email,
          uid: request.uid,
          text: request.text
        }
    };

    //The topic variable can be anything from a username, to a uid
    //I find this approach much better than using the refresh token
    //as you can subscribe to someone's phone number, username, or some other unique identifier
    //to communicate between

    //Now let's move onto the code, but before that, let's push this to firebase

    admin.messaging().sendToTopic(request.topic, payload)
    .then((response) => {
        console.log("Successfully sent message: ", response);
        return true;
    })
    .catch((error) => {
        console.log("Error sending message: ", error);
        return false;
    })
})
//And this is it for building notifications to multiple devices from or to one.

There is huge blank for security system, i hope you fill it fast so no need 3.party solutions and manage everything from Google Home.
We need more touchable devices.
Thanks again.


Refactor client lib separating client retrieval logic

we need to implement the DispatcherProvider interface and pass it in as an parameter, like this:
https://github.com/uber/cadence/pull/1291/files#diff-4c51d95b26718af4a2ac3bec33e54e49R126


Logit shape change in euclideanDistance()

Hello, while using your code with my dataset I encountered a dimension error which traced to the line below:
https://github.com/dimitymiller/openset_detection/blob/657bbaede8b3eab78cfa8eab35bfcdd1c84fa6e2/mmdetection/mmdet/models/losses/anchor_cross_entropy_loss.py#L113
In the euclideanDistance method, the input logit is reshaped (_, num_classes+1) to account for the background class.
This creates a dimension error in anchor_cross_entropy since the first shape of mask and distances do not match.
https://github.com/dimitymiller/openset_detection/blob/657bbaede8b3eab78cfa8eab35bfcdd1c84fa6e2/mmdetection/mmdet/models/losses/anchor_cross_entropy_loss.py#L51
I'm confused if just naively reshaping the logit to fit the additional background class is correct.
Can you please provide some reasoning behind this code?
Thank you in advance!

The object detectors used in the paper (Faster R-CNN and RetinaNet) produce a logit for each training class (self.num_classes) and an extra logit for the background class. Other object detector architectures don't predict a logit for the 'background' class explicitly and will not produce the same dimension logit vector.
If you're only changing the dataset and not changing the architecture, a dimension error seems strange. Are you creating AnchorwCrossEntropyLoss with the num_classes your object detector was trained for (rather than the num_classes in the test dataset)? For example, a COCO-trained detector should have num_classes = 80, even if you test on a different dataset.

Thanks for the timely reply!
As you said RetinaNet is supposed to produce an additional class prediction for background.
Turns out there's a mistake in my custom config that prevents that, which resulted in the aforementioned dimension error.
Thank you for the clarification!


There is no way to download the CU50 project as a whole.

I am trying to follow the tutorial for the Contoso University project. The project download link takes me to the github page but there is no way to actually download the full project. While I am not a github expert, I do know how to use the basic features. The download project button dose not show on any page.
Thanks
Gary Hustis

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

ID: 7a54c97f-ddd4-b041-5e5f-a2df009ace50
Version Independent ID: fa488196-1f98-08ea-adcc-17d9dac86727
Content: Razor Pages with Entity Framework Core in ASP.NET Core - Tutorial 1 of 8
Content Source: aspnetcore/data/ef-rp/intro.md
Product: aspnet-core
Technology: aspnetcore-data
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

See

Rick,
Thanks for the quick response.
I followed that link multiple times over the past week ensuring myself that I was missing something. But each time I hit the circled statement which is vague and ambiguous. I cannot find any link in the/a sample directory.
On GitHub I do not find the traditional DownLoad Project button. See below.
I have downloaded many projects from Git using that button in the past. I must be simply missing something.
@.***D72FBB.A4FCA6D0]
@.***D72FBB.A4FCA6D0]
Gary Hustis
<EMAIL_ADDRESS>585-739-9781 (m)
From: Rick Anderson @.>
Sent: Monday, April 12, 2021 2:17 PM
To: dotnet/AspNetCore.Docs @.>
Cc: ghustis alphaparagon.com @.>; Author @.>
Subject: Re: [dotnet/AspNetCore.Docs] There is no way to download the CU50 project as a whole. (#22042)
See
[image]https://user-images.githubusercontent.com/3605364/114441670-61391600-9b67-11eb-9e17-14f56b3e4dfc.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/AspNetCore.Docs/issues/22042#issuecomment-818023567, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASPOJAY7VP7VAOMIDD4VNRTTIM2I3ANCNFSM42ZUB7TA.

Do you see this in your browser?

Rick,
Thank you for sharing that link. Every other link I found sent me deeper into the directory structure. However I had already found my way to that zip file and have downloaded it a few times as I had gotten to it a few different ways thinking it might be a different file each time. The bottom line is that I had extracted and scoured the contents of that zip and cannot find the Contoso University project in it. That zip file seems to have a collection of small projects that demonstrate particular features, which I’m glad to have and will certainly come in handy later.
BTW – I have been following the CU tutorial and it is nicely detailed with adequate explanations.  So if I don’t get the full project I’m sure I will by the time I finish the series. If I have a need I can navigate to the cu50 github page and copy the code I need.
Thanks for your help.
Gary Hustis
<EMAIL_ADDRESS>585-739-9781 (m)
From: Rick Anderson @.>
Sent: Monday, April 12, 2021 6:48 PM
To: dotnet/AspNetCore.Docs @.>
Cc: ghustis alphaparagon.com @.>; Author @.>
Subject: Re: [dotnet/AspNetCore.Docs] There is no way to download the CU50 project as a whole. (#22042)
[image]https://user-images.githubusercontent.com/3605364/114472068-52655a00-9b8d-11eb-888d-5b2965ed3cc0.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/AspNetCore.Docs/issues/22042#issuecomment-818292474, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASPOJAZNRD6RGANKUVGNWGTTIN2DHANCNFSM42ZUB7TA.


Don't store complex data in PDFDocument.formInfo, and replace the fields object with a hasFields boolean instead

This patch is based on a couple of smaller things that I noticed when working on PR #12479.

Don't store the /Fields on the formInfo getter, since that feels like overloading it with unintended (and too complex) data, and utilize a hasFields boolean instead.
This functionality was originally added in PR #12271, to help determine what kind of form data a PDF document contains, and I think that we should ensure that the return value of formInfo only consists of "simple" data.
With these changes the fieldObjects getter instead has to look-up the /Fields manually, however that shouldn't be a problem since the access is guarded by a formInfo.hasFields check which ensures that the data both exists and is valid. Furthermore, most documents doesn't even have any /AcroForm data anyway.

Determine the hasFields property first, to ensure that it's always correct even if there's errors when checking e.g. the /XFA or /SigFlags entires, since the fieldObjects getter depends on it.

Simplify a loop in fieldObjects, since the object being accessed is a Map and those have built-in iteration support.

Use a higher logging level for errors in the formInfo getter, and include the actual error message, since that'd have helped with fixing PR #12479 a lot quicker.

Update the JSDoc comment in src/display/api.js to list the return values correctly, and also slightly extend/improve the description.

/botio unittest

Thank you for finetuning this!


How to implement search( or filter) functionality

Hi Sir,
How to implement search( or filter) functionality for both header and child item?
Deeply appreciated.

Hi @wormboss. The library don't support such functionality, so implementation is up to you.


Show the value of a range field

Hello,
I am using your range field, but i want to display the value on an other input/div and change the value when we slide. How can i do that?
Thank you.

Hi, you would need Javascript for this.
Justin
On 17/02/2012, at 1:23 AM<EMAIL_ADDRESS>wrote:

Hello,
I am using your range field, but i want to display the value on an other input/div and change the value when we slide. How can i do that?
Thank you.

Reply to this email directly or view it on GitHub:
https://github.com/justinfrench/formtastic/issues/803

@Ditchou what have you tried?  For general questions, please use the Google group or Stack Overflow. Closing this one for now!

I tried that, but my html was escaped. Thank you for help !

@Ditchou did you try it with the .html_safe on the end? That really should work. Can you show your code please so I don't have to keep guessing?


Validation nation

@dougbu - added fixes for failing tests

:watch:

@dougbu - updated, squashed and rebased

:shipit: don't want to block you on https://github.com/aspnet/Mvc/pull/4010#discussion_r51300495 question. That would be a change in behaviour while you're mostly making FormFileModelBinder more consistent here.


Benchmark passphrase API of Secure Cell

Some benchmarks for passphrase API added in #577.
Right now we don't have support for this API in RustThemis so we call C code directly in decryption benchmark to prepare the input data.
Note that due to the nature of the passphrase API – KDF makes repeated invocations purposefully slow – the benchmarks are abysmally slow too and we have to cut down the sample sizes to execute them in a reasonable time frame. And we don't need to have much variety in the message size since execution time is completely dominated by KDF costs until we get to gigabyte-sized messages.
There is quite a few “inheritance” in the code from master key benchmarks but I'll leave it around for now until we are sure what utilities can be shared. Plus, Criterion has issues with reusing code so we might need to introduce additional helper crates for that which I would like to avoid doing.

🚧 This branch depends on #577 and #580 so it has to wait before they are merged. To get better diffs the PR is targeted to a temporary branch which has to be removed before this PR is merged.
Once dependencies are merged, do the following:

[ ] rebase ilammy:kdf/benchmarks onto cossacklabs:master
[ ] retarget this PR to master
[ ] delete ilammy/kdf+benchmarks

Follow the order. If the base branch is deleted early then this PR might get autoclosed.
Checklist

[X] Change is covered by automated tests
[X] Benchmark results are attached
[X] The coding guidelines are followed

Dependencies have been merged. Rebased and retargeted onto master with no changes.


cyclops-crush should check that  is before  in index.html

some silly mistake (moving comments around for no good reason) resulted in a broken build with no errors reported

I think that this should go into lint-html instead. htmlHint supports adding custom rules and gulp-htmlhint exposes a method to it.
Looks like a pretty simple (not necessarily easy) task.
At this point in time, there isn't any validation done in cyclops-crush. If we do this, it opens the gate to other validation only done here as well. If you want to perform validation before a build they should be consistently enforced. I would suggest adding an npm script. This will allow one to do npm run dist.
{
   "scripts": {
     "dist": "gulp manticore-sting && gulp cyclops-crush"
  }
}


disciplined includes

Summary
I got Include What You Use (IWYU) working on our repo in CI. It still has too many positives (both true and false) left to enable, so this is a (large) first round of fixes. IWYU ensures that every source file includes exactly what it uses, nothing more and nothing less. This gives you more freedom to move things around/forward declare things in other files as you know nothing is transitively including things.
Test plan

make clobber; make configure-devel && make test
CI
Will import this PR and check the internal build.

Thanks for the explanations. I think the disconnect for me was that I was considering something like the return value of a function as an implementation detail, where the header would be free to change the exact type as long as it was compatible with the code that was using it.
e.g. For a function:
header:
  std::optional<int> foo();

consumer:
  auto x = *foo();

The return type could be changed to something else without needing any changes in the caller:
header:
  std::unique_ptr<int> foo();

consumer:
  auto x = *foo();

So I wouldn't have personally included the optional or memory headers in either case, unless the caller is explicit about the type, e.g.:
std::optional<int> f = foo();
auto x = *f;

Maybe I'm just wrong about this though...

Might take another approach to this at some point in the future, but dropping it in its current form.


Parmount Network - Collected 0 video ids

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2019.11.05
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

youtube-dl.exe https://www.paramountnetwork.com/episodes/iabobz/mom-poodle-fuzz-and-a-twinge-of-jealousy-season-5-ep-5 -v
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['https://www.paramountnetwork.com/episodes/iabobz/mom-poodle-fuzz-and-a-twinge-of-jealousy-season-5-ep-5', '-v']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2019.11.05
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.18362
[debug] exe versions: ffmpeg N-95111-g87ddf9f1ef, ffprobe N-95111-g87ddf9f1ef
[debug] Proxy map: {}
[debug] Using fake IP <IP_ADDRESS> (US) as X-Forwarded-For.
[ParamountNetwork] mom-poodle-fuzz-and-a-twinge-of-jealousy-season-5-ep-5: Downloading webpage
[ParamountNetwork] f856686a-b7e2-11e8-8b3b-70df2f866ace: Downloading info
[download] Downloading playlist: Poodle Fuzz and a Twinge of Jealousy
[ParamountNetwork] playlist Poodle Fuzz and a Twinge of Jealousy: Collected 0 video ids (downloading 0 of them)
[download] Finished downloading playlist: Poodle Fuzz and a Twinge of Jealousy

Description

Trying to download doesn't actually process any videos. Usually there are 4 or 5 segments when getting these.

You must provide account credentials.


Doesn't work using Ionic serve

What do you think using Ionic utilities? https://ionicframework.com/docs/api/service/$ionicPosition/


[Relay] Odd's 'n ends changes to help Collage.

Complete the implementation of WithFields.
(Unfortunately they appear to be without unit tests and I continue this tradition...)
InferTypeExpr for InferTypeLocal but return the expression rather than the type.
Remove python binding of InlineComposites since C++ impl was removed some time ago.
Make IndexedGraph<Expr/DFPattern> more robust as stand-alone datastructure, and avoid unnecessary copies.
This will become a fundamental datastructure in Collage rather than just a helper for DFPatternMatcher.
Extend IndexedGraph with a notion of 'basic block' on every dataflow node. Needed by Collage to
avoid impossible partition.
(Again, unfortunately not unit tested).

Are the properties in WithFields documented somewhere? That probably shouldn't be too difficult to unit-test.

I'd forgotten I'd adjusted the visit order in IndexedGraph to respect idx(x) < idx(y) => y does not depend on x with let-bound vars. That deserves proper unit testing. Stand by.

This is now ready for review.

Thanks @mbaret , PTAL.
Unfortunately Matthew is out for a while so it's up to us to build up expertise ourselves.

Shoot I goofed and will need #11616 to fix it.


[MeshModel] Add examples for different Kubernetes resources

Current Situation
We do not have an adequate amount of Kubernetes examples published in MeshModel.
Proposed Change
See the examples and CRDs from this repository and

Create a dedicated folder or section for examples and CRDs.
Update and enhance documentation for better clarity and usability.

Contributor Guides and Handbook

MeshModel Repository
🖥 MeshModel Overview
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

How to make updates to MeshModel
Instructions for Models

Forthcoming

Instructions for Components
While the default shape for new components is a circle, each component should be considered for its best-fit shape.

Review and familiarize with the available set of predefined relationship types. Refer the Cytoscape node types for a list of possible shapes.
Propose a specific shape, best-suited to visually represent the Component. Example - Deployment as a pentagon.
Proposee a specific icon, best-suited to visually represent the Component. Example - DaemonSet as a skull icon.

Instructions for Relationships

Identify the relationship and any specific constraints to be enforced between the two specific components, their models or potentially other components, models, or environmental considerations.
Propose a specific visual representation for the relationship. Visual representation examples:

Hierarchical
Sibling
Binding

Prospose the appropriate relationship type, using one of the predefined set of relationship types or suggest a new relationship where an existing type does not fit.
Create a Relationship Definition (yaml). See the Relationship Schema in MeshModel and examples.
Create a policy for evaluation of the relationship (rego). See examples.

Instructions for Policies

Forthcoming

hey @Boombag0607, Can I work on this

Hey, I'm new to open-source community and would like to kick-start my journey with this
It would be great if assigned to me

Hey @Boombag0607, I'm new to open-source community and would like to kick-start my journey with this
It would be great if assigned to me

Hey @Boombag0607, could you assign this issue to me?

Hi @Boombag0607 , I was working on a front-end issue & stumbled upon here. I'd like to take a look at this issue, but I'm a bit new to kubernetes, it would be great to  have some starting resources.

Hi @leecalcote, can I discuss about this issue in tomorrows meeting? I have some doubts abut how we are gonna show the examples under the title. What else will be there including a brief idea about the example..!


Rdm plot

implements a basic plot, which shows an RDMs object as the individual rdms + the average

Hi all,
could someone else approve this pull request. Although this is not the publication ready version RDM plot I believe this one is very handy for quickly looking at a RDM and should be available in general

Is someone already working on the merge conflict? If not I'd be happy to give it a go

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- pyrsa/vis/rdm_plot.py  3

See the complete overview on Codacy

Codecov Report

Merging #56 into master will decrease coverage by 10.25%.
The diff coverage is 10.52%.

@@             Coverage Diff             @@
##           master      #56       +/-   ##
===========================================
- Coverage   81.76%   71.51%   -10.26%
===========================================
  Files          53       38       -15
  Lines        4717     3079     -1638
===========================================
- Hits         3857     2202     -1655
- Misses        860      877       +17

Impacted Files
Coverage Δ

pyrsa/vis/rdm_plot.py
12.06% <0.00%> (-2.22%)
:arrow_down:

pyrsa/rdm/rdms.py
77.66% <16.00%> (-8.67%)
:arrow_down:

tests/test_data_utils.py

tests/test_simulation.py

tests/test_something.py

tests/test_inference.py

tests/test_data.py

tests/test_descriptor_utils.py

tests/test_util_matrix.py

tests/test_model.py

... and 6 more

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 2f29b7f...c0ed307. Read the comment docs.

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- pyrsa/vis/model_plot.py  1
- pyrsa/vis/rdm_plot.py  3

See the complete overview on Codacy


finnhub code portion not active

Hi ,
Need help with Finnhub portion of code. Everything works and compiles and my arduino shows crypto stocks
perfectly but does not advance past the crypto coins. Another words it wont post or show the stocks even though I generated an api key, Adjusted the true/false tag and it still doesn't show those stocks. Is there and additional step I'm missing to get stocks to show?

vs

Any help appreciated. Thanks!

Does the standard code with your API key work as provided? I would start there.
The true/false value tells the program if the asset is a crypto (true) or a stock (false), so it must be false for finnhub.
I haven't run this project in a while and don't currently have the HW to run it ATM

Thank you for replying, you indirectly solved it. I assumed your flags were "on or off" instead of true or false ;) That corrected it!!
QQ, is there code that can be run from a pc or mac that you are aware of that can easily update the api key and or the wifi settings? This would be really handy to know. Thank you again!!

No problem!
I can see how that could be confusing, I just updated the code to use CRYPTO and STOCK flags instead of true and false.
I dont think so you will have to re-flash the ESP when those values change.

ok, no worries, I will also update my code to reflect the crypto/stock adjustment. ;)


TECH TALKS AND PROJECTS LINKS OPEN IN NEW TAB

Added target="_blank" in Tech talks and Projects sections.
Thanks!

Hi @itsaakashpatel ,
Wrong section, the issue was targeting towards the sections on the main page src/pages/index.js, should only need to modify the ProjectCard and the TechTalkCard components. See https://github.com/ericjiang97/ericjiang.dev/blob/master/src/pages/index.js#L198

This relates to #15


fix #1130

In this PR:

Moved the update code from sample_template.py to base_metadata_template.py & generalized for prep template updates
Fixed an issue with sanbdox -> sandbox
Fix #1130 (add update to the prep template)

Coverage decreased (-0.21%) to 78.47% when pulling 045f89b8ef85c6ed408ef4eff7e524d6ede5eed2 on antgonza:issue-1130 into 906a55653ae8f8abd4f13e96ac52f177ba9a5470 on biocore:master.

Thinking about this a bit more, is there any metadata column that we don't want the user to update? One of the examples that pops in my mind is barcode. Although it is useful to update the barcode in case that the user messed it up, if there is a PreprocessedData generated from it, we shouldn't allow to update the PrepTemplate. The main reason is that one of the PreprocessedData will be generated with an old version of it, and the information in the PrepTemplate is critical for the contents of the PreprocessedData. Thus, I think a bit more thought should be imposed when updating a prep template.
An option to prevent this is: create a function called can_be_updated. The SampleTemplate implementation always returns True, while the PrepTemplate one checks if there is any PreprocessedData generated. If a PreprocessedData is generated, it then checks which values are being updated. If it is one of the columns controlled by PREP_TEMPLATE_COLUMNS_TARGET_GENE it doesn't allow the update, while if it is any other column, this changed is allowed. The reason why I want to allow this other changes is because some of those updates may be needed for EBI submission, and those changes will not impact the contents of the PreprocessedData.
Sorry for the long comment, but I think is better to be explicit than implicit.

Coverage decreased (-0.17%) to 78.52% when pulling 74a0697e6a519127614345db33115abc6ef2896d on antgonza:issue-1130 into 1d864aaf4766e288adcd7d7b80d36ab20a4f597f on biocore:master.

This is ready for another review ...

I was trying to update a template and some error happened. However, all that is shown is this, so it's hard to debug...

ready for another pass ...

:+1:

@squirrelo a reminder about reviewing this one.

few comments.

ping @wasade, @josenavas and @squirrelo

A couple of non-blocking comments and a question. Otherwise this looks good!

Just one comment about consistency. Thanks @antgonza
:+1: once that is solved!

Done.

:+1: @squirrelo you also reviewed this PR. Can you review, or merge if there are no more comments?

ping @squirrelo

Couple comments and questions. I'll test the interface shortly.

@squirrelo fixed the texts that you requested ...

Text is much clearer, thanks.


fix: use same external rule as rollup for esbuild (fix #9549)

Description

Fix #9549
If a-module is external, do not treat a-module/file as external during optimizeDep.
Additional context

What is the purpose of this pull request?

[X] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[X] Read the Contributing Guidelines.
[X] Read the Pull Request Guidelines and follow the Commit Convention.
[X] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[X] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[X] Ideally, include relevant tests that fail without this PR but pass with it.

This would break a fair amount of apps so we probably couldn't do this in Vite 4. Since this is related to esbuild optimization in build which is still experimental, we probably want to find a non-breaking way to support this if possible.


Failure of .outrigger.yml to generate

Note: This needs confirmation, as it could have been a file copy failure.

False alarm.


empty stdout

Hi @colindembovsky , thanks a lot for your blog tutorial, it's really great content.
I was wondering if you've come across the fact that stdout from the plan and apply comes out empty. I'm trying to search the github documentation but I think they might've deprecated that function. Do you have a workaround?
Cheers, Thanks!

Ignore me eheh, I had the wrapper script flag disabled. again, thanks !


Progress bar

Progress bar is sofar showing progress for each solver object, and starts over for each timestep. Would be nice to see the current timestep also (maybe overlaid or below). E.g.      "Current Filename"  t=2/20

added f2423a2a021bea6a57062f55cf628fa860a653f2


[BUG] LSP Client is not fully initialized when it starts listening, results in a bug with the progressPart

Issue: LSP Client is not fully initialized when it starts listening, results in a bug with the progressPart
Description:
I've encountered an issue in the vscode-languageclient/node module where some initial LSP messages get dropped during communication under certain conditions. Specifically, in the following code:
connection.listen();
await this.initialize(connection);
resolve();

The problem occurs because the listen function is called before the initialize function, which opens the LSP connection. If the LSP server starts and immediately begins sending messages, some of the earliest messages might be dropped, as the client is not yet fully initialized to handle incoming communication. This leads to unexpected behavior and a lack of proper message handling.
Steps to Reproduce:

Set up a project with the vscode-languageclient/node module.
Start an LSP server that sends messages very quickly after initialization.
Observe that some of the first messages are dropped and not processed by the client.

Details:

We can see from the image above that right after the LSP Client starts and begins listening, it receives a ‘window/workDoneProgress/create’ request from the Server. However, at this point, the LSP Client has not yet been initialized, so there is no appropriate handler for this request. Additionally, due to the nature of the ProgressPart, the progress of this ongoing operation cannot be reported correctly afterward.
Expected Behavior:
The client should be able to handle all incoming messages from the LSP server without dropping any, even if the messages arrive immediately after the connection is established.
Suggested Solution:
One possible fix would be to ensure that the client is fully initialized before it starts listening to the connection. Reversing the order of operations—initializing first, then starting the listen process—may prevent messages from being dropped.

Actually this is by design. See here https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#initialize
Until the initialize handshake is complete only a limited number of message types can be exchanged. This is because some of the client behavior in terms of messages depend on the result of the initialize request.
If you want to report progress during initialize you can use the progress token on the initialize request. You need to let the VS Code LSP client know to insert such a token. See https://github.com/microsoft/vscode-languageserver-node/blob/main/client/src/common/client.ts#L362


Unblock concurrent-extra, threads, threads-extra and snap-server

Checklist:

[X] Meaningful commit message - please not Update build-constraints.yml

[X] At least 30 minutes have passed since Hackage upload

[ ] On your own machine, in a new directory, you have successfully run the following set of commands (replace $package with the name of the package that is submitted, $version is the version of the package you want to get into Stackage):
stack unpack $package
cd $package-$version
stack init --resolver nightly
stack build --resolver nightly --haddock --test --bench --no-run-benchmarks

As these packages depend on each other, the above commands fail. But running the tests with cabal works. Except for concurrent-extra which has a failing test.

Thanks! :)

I've marked concurrent-extra as an expected test failure.


nncp: init at 5.3.3

Motivation for this change
use NNCP, a UUCP-like store-and-forward toolset with crypto, written in go
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Thanks @doronbehar for reviewing! I've just pulled the small corrections and rebased to master.

(rebased to current master)

rebased to current master, still compiles and runs.

The eval error has nothing to do with you.

Result of nixpkgs-review pr 75772 1

1 package built:

nncp

Thanks for your contribution.


jsonrpc: error objects

looks like we don't return proper errors yet. e.g. when not finding a filter.
http://www.jsonrpc.org/specification#error_object
@debris wrote:
[11.06.15 15:08:58] there is also some proposal, regarding errors:
https://github.com/ethereum/wiki/wiki/JSON-RPC-Error-Codes-Improvement-Proposal
[11.06.15 15:09:23] it's not yet implemented in cpp, but I think that go supports most of this errors

eth_call is missing compliance:

the eth_call uses data_encoder [ https://github.com/ethereum/pyethapp/blob/develop/pyethapp/jsonrpc.py#L1091 ]
Upon transaction failure the function call that implements the JSON RPC method returns False [ https://github.com/ethereum/pyethapp/blob/develop/pyethapp/jsonrpc.py#L1156 ]
And the data_encoder cannot handle bool only str and unicode [ https://github.com/ethereum/pyethapp/blob/develop/pyethapp/jsonrpc.py#L391 ]

The proper return should be roughly:
{
    code: 3,
    message: 'Execution error',
    data: [{
        code: 104,
        message: 'Rejected',
        reason: 'Execution error'
    }]
}

Addressed here https://github.com/ethereum/pyethapp/pull/161


Any example script available for multiple permissions with the reason of permission?

Actually I want to take 4 permissions in one go.
The following are the permissions

audio
external storage
record video
exposing sensitive information during recording

I tried it using Permission[] RequestPermissions( params string[] permissions ) but didn't get how exactly use it, I even tried to go through the unity forum of this repo and the closed issues as well, but didn't get any help.
@yasirkula Please help?

Hope this helps:
// These permissions must be declared in AndroidManifest.xml file
string storage = "android.permission.WRITE_EXTERNAL_STORAGE";
string audio = ...;
string video = ...;
string sensitiveInfo = ...;

// Request all permissions at once
AndroidRuntimePermissions.Permission[] result = AndroidRuntimePermissions.RequestPermissions( storage, audio, video, sensitiveInfo );

// Check if all permissions are granted
Debug.Log( "All permissions granted: " + System.Array.TrueForAll( result, ( permission ) => permission == AndroidRuntimePermissions.Permission.Granted ) );

<?xml version="1.0" encoding="utf-8"?>
<manifest
  xmlns:android="http://schemas.android.com/apk/res/android" package="com.setik.androidutils">
  <application android:icon="@drawable/app_icon" android:label="@string/app_name">
    <activity android:name="com.setik.kampertee.AndroidUtils" android:label="@string/app_name" android:configChanges="fontScale|keyboard|keyboardHidden|locale|mnc|mcc|navigation|orientation|screenLayout|screenSize|smallestScreenSize|uiMode|touchscreen">
      <intent-filter>
        <action android:name="android.intent.action.MAIN" />
        <category android:name="android.intent.category.LAUNCHER" />
      </intent-filter>
    </activity>
    <meta-data android:name="unityplayer.SkipPermissionsDialog" android:value="true" />
  </application>
  <uses-permission android:name="android.permission.ACCESS_FINE_LOCATION" />
</manifest>

Above is my exact AndroidManifest.xml. Where exactly to declare the permissions in this script. I know it may sound a bit frustrating for you, but I'm a noob when it comes to XML. @yasirkula Please help. 🙏

You just duplicate the uses-permission line for each permission.

Sorry, I forgot to thank you for this repo and quick responses on this repository.
Also, I'm hereby giving you 5 star rating on the asset store.

Thank you, I appreciate it 😃

Hi @yasirkula ,  I am asking for 4 permissions at the very start of the app. That has been done.
But I don't want user to move forward if any of the permission is denied.
For that I used AndroidRuntimePermissions.Permission result3 = AndroidRuntimePermissions.CheckPermission(storage);
and keep doing it for every permission (I guess array wasn't working for that, but leave it that's not a concern)
Now my question is how do I debug this info in my unity simulator that the permission is denied or granted?
If not my unity simulator, even if I'm doing this in my android phone, still how can I debug information?

Permissions will always return Granted in Unity editor. On an actual device, you can Debug.Log the permission result(s) and see those logs via "adb logcat", Unity's "Android Logcat" package or an in-game debug console like this.

Thanks, It helped me!!

Happy to help 😺


No Model Saved when Training without FSDP or PEFT

System Info
Python 3.9. Issue is with code in utils.train_utils.
Information

[X] The official example scripts
[ ] My own modified scripts

🐛 Describe the bug
The train function in utils.train_utils does not currently have a way to save a model when not using PEFT or FSDP.
I use the default train_config in configs/training.py and I've added these keyword args at the top of main which update train_config:
    kwargs["run_validation"] = True
    kwargs["model_name"] = "/path/to/models/Llama-2-7b-hf"
    kwargs["quantization"] = True
    kwargs["batch_size_training"] = 16
    kwargs["val_batch_size"] = 16
    kwargs["num_workers_dataloader"] = 10
    kwargs["lr"] = 1e-5

For enable_fsdp=False, train() gets passed fsdp_config=None leading to errors at here.
A quick check can be added for fsdp_config=None but how should models be saved when not using either PEFT or FSDP?
The save_model_checkpoint() and save_optimizer_checkpoint() functions seem to rely on FSDP.
Error logs
Model does not get saved when training without PEFT or FSDP.
Expected behavior
Control flow to save models when not using PEFT or FSDP would be expected.

@jpmcd thats is by design as we are providing only these two options, the reason is to train even 7b on single GPU without PEFT you will run into memory issues and going over 1 gpu, you would need a mechanism to shard your model which we use FSDP here. Can you pls share some numbers from your setting where you can achieve 1 gpu training without PEFT, that would help us to add the option.

Sure, I'd be glad to share. I've actually been able to successfully run training on the 7b model without PEFT or FSDP, but with quantization=True, on our cluster's system which is comprised of mildly dated Xeon G6 CPUs + 2 V100 Voltas per node. Like you mention, I am only using one GPU for training. It isn't fast but it seems to work. Though I'll admit I've been reading into the specifics of quantization to see if there's something I'm misunderstanding about this.
What specifics would be helpful to share? E.g. device/memory specs, batch size, model length, and any particular observations like seconds per batch?

I see, I think our assumption here for checkpoint saving logic has been if someone use PEFT or FSDP. I am not sure if we would change that yet, but pls feel free to fix the logic here as it works for you.

sorry didn't mean to close it.

Okay, I'll give it a shot and see if it works on a large enough machine.


Copy Offload requests are failing as we scale the number of using processes and compute nodes

We are seeing the following failure from the nnf-dm API when running a 4-node/16-process allocation:
(Further detailed discussion was created here: https://llnl.slack.com/archives/C020U81E05U/p1722894207606109, and here: https://llnl.slack.com/archives/C020U81E05U/p1722967213939429
flux run '--requires=-host:rzadams[1033-1048]' -q pdev -t 1h -N4 -n16 --setattr=dw="#DW jobdw type=lustre capacity=1440GiB name=scrcache requires=copy-offload" ./test_api -t 5 -s 9GB
flux-job: fcJbo2NAK3m

Shortly after starting the test, we see the following failure when the copy offload has started:
2024-08-05 14:37:22:937 AXL ERROR:1971777 rzadams1068: @ nnfdm_stat:96 NNFDM Offload Status UNSUCCESSFUL: 0
Offload Command Status: {3/STATE_COMPLETED}, {0/STATUS_INVALID}
    Offload Command Status:
      Command:
      Progress: 0%
      ElapsedTime:
      LastMessage:
      LastMessageTime:
    Offload StartTime:
    Offload EndTime:
    Offload Message: internal error: could not determine source type: could not stat path ('/mnt/nnf/df5e1995-c0f6-4b56-8e8b-a00d7a70807d-0/martymcf/scr.defjobid/scr.dataset.1/rank_9.ckpt'): command: mpirun --allow-run-as-root -n 1 --hostfile /tmp/nnf-dm-42p79/hostfile -- setpriv --euid 54987 --egid 54987 --clear-groups stat --cached never -c '%F' /mnt/nnf/df5e1995-c0f6-4b56-8e8b-a00d7a70807d-0/martymcf/scr.defjobid/scr.dataset.1/rank_9.ckpt - stderr: kex_exchange_identification: Connection closed by remote host
Connection closed by <IP_ADDRESS> port 2222
--------------------------------------------------------------------------
ORTE was unable to reliably start one or more daemons.
This usually is caused by:
* not finding the required libraries and/or binaries on
  one or more nodes. Please check your PATH and LD_LIBRARY_PATH
  settings, or configure OMPI with --enable-orterun-prefix-by-default
* lack of authority to execute on one or more specified nodes.
  Please verify your allocation and authorities.
* the inability to write startup files into /tmp (--tmpdir/orte_tmpdir_base).
  Please check with your sys admin to determine the correct location to use.
*  compilation of the orted with dynamic libraries when static are required
  (e.g., on Cray). Please check your configure cmd line and consider using
  one of the contrib/platform definitions for your system type.
* an inability to create a connection back to mpirun due to a
  lack of common network interfaces and/or no route found between
  them. Please check network connectivity (including firewalls
  and network routing requirements).
--------------------------------------------------------------------------

From @bdevcich on 8/7:
Good news. I can replicate this on our end.

We've been able to reproduce this on our end just using YAML to create the data movement requests. So it has nothing to do with the nnf-dm daemon itself.
Going one step further, we can also reproduce this by skipping data movement altogether and using mpirun to run sleep commands from the nnf-dm-controller pod (running on a k8s control plane node) on a nnf-dm-worker pod (running on a rabbit node).
for i in {1..512}; do
	mpirun --allow-run-as-root -n 1 --hostfile /tmp/blake sleep 10&
done

$ cat /tmp/blake
10-42-3-193.dm.nnf-dm-system slots=2 max_slots=20

If we run this using a hostfile that contains localhost (such is the case for gfs2/xfs data movement), we don't see this problem. Presumably because ssh is not involved.
mpirun using ssh under the hood. The ssh configuration on the nnf-dm-worker pods running on the rabbit nodes have MaxStartups set to the default of 10:30:100.
From what I understand, this means that after 10 unsecured connections, it will start dropping 30% of the new connections. After 100 unsecured connections, it will drop every new one. Each ssh connection is unsecured until authentication is complete, so even though we're using ssh keys, this still applies. Bumping the value up to 512:-1:512 makes this problem go away so far in my testing.
To fix this, we need to change the nnf-mfu image used by the nnf-dm-worker pods. The images used for the nnf-dm-workers should be able to be changed using gitops, so this fix might not require any code change.
That being said, I do have some improvements coming (like being able to change the command used for stat) in the NnfDataMovementProfile.

The fix to increase the number of ssh connections is here:

https://github.com/NearNodeFlash/nnf-mfu/pull/21

The ability to alter the stat command via NnfDataMovementProfiles is being added here:

https://github.com/NearNodeFlash/nnf-sos/pull/363

I've updated the nnf-mfu image to use 4096 for MaxStartups. This image can be used by updating the gitops configuration to pull the new image by updating the image tag to <IP_ADDRESS>-8221. This change should look similar to this:
diff --git a/environments/htx-1/nnf-dm/nnf-dm.yaml b/environments/htx-1/nnf-dm/nnf-dm.yaml
index 0d184e4..09ca3df 100644
--- a/environments/htx-1/nnf-dm/nnf-dm.yaml
+++ b/environments/htx-1/nnf-dm/nnf-dm.yaml
@@ -560,7 +560,7 @@ spec:
         - -De
         command:
         - /usr/sbin/sshd
-        image: ghcr.io/nearnodeflash/nnf-mfu:0.1.1
+        image: ghcr.io/nearnodeflash/nnf-mfu:<IP_ADDRESS>-8221
         name: worker
         securityContext:
           capabilities:

Once changed, this will cause the nnf-dm-worker-* pods to restart, but no upgrade is necessary. I've tested this on top of the latest NNF release (v0.1.6) and it should resolve the problem short term. I believe more tweaking will be necessary long time.

Additionally, the entire sshd_config is now configurable by editing the nnf-dm-worker-config ConfigMap in kubernetes. Users can increase the value of MaxStartups by editing the ConfigMap: https://nearnodeflash.github.io/dev/guides/data-movement/readme/#sshd-configuration-for-data-movement-workers
https://github.com/NearNodeFlash/nnf-dm/pull/208


Opensearch Image Improvements

Added log stash options as environment variables
Added buffering options as environment variables

Let me know if you require any adjustments.

I'll rework the commits tomorrow. Let me know if you want to keep version 1.0.6 of fluent-plugin-opensearch in the template/gemfile.erb or bump it up to 1.0.8. That part is a bit nebulous for me at the moment.
On our end, we have validated 1.0.8 of fluent-plugin-opensearch against the previous 1.15.2 release using self-published images (reading logs from 4 kubernetes clusters and sending them to an on-prem opensearch cluster running on version 2.2.1 of opensearch). We can try it against the updated 1.15.3 release if you like.

I'll rework the commits tomorrow. Let me know if you want to keep version 1.0.6 of fluent-plugin-opensearch in the template/gemfile.erb or bump it up to 1.0.8. That part is a bit nebulous for me at the moment.
On our end, we have validated 1.0.8 of fluent-plugin-opensearch against the previous 1.15.2 release using self-published images (reading logs from 4 kubernetes clusters and sending them to an on-prem opensearch cluster running on version 2.2.1 of opensearch). We can try it against the updated 1.15.3 release if you like.

The comment above can be ignored. I hadn't read your reply. It is clear now.

I made the adjustments. Let me know if you see something else.

@cosmo0920 Could you check whether adding these parameters is reasonable?

Thanks!


run composer config to set allow-plugins

Fixes https://github.com/bluehorndigital/setup-drupal/issues/13
Replaces https://github.com/bluehorndigital/setup-drupal/pull/14

GitLab for Drupal is undergoing maintenance


Question regarding media-breakpoint-between

I was trying out @media-breakpoint-between but it doesn't behave the way I expected.
Given:
@include media-breakpoint-between(sm, md) {
  div {
    background: tomato;
  }
}

Actual:
@media (min-width: 576px) and (max-width: 991px) {
  div {
    background: tomato;
  }
}

Expected:
@media (min-width: 576px) and (max-width: 768px) {
  div {
    background: tomato;
  }
}

Example with extracted code in sassmeister.
Respective files:

https://github.com/twbs/bootstrap/blob/78fc4d23fcefe31397444b75f43728c38e13c590/scss/mixins/_breakpoints.scss
https://github.com/twbs/bootstrap/blob/78fc4d23fcefe31397444b75f43728c38e13c590/scss/_variables.scss#L181-L187

FYI is breakpoint-max is changed from:
@function breakpoint-max($name, $breakpoints: $grid-breakpoints) {
  $next: breakpoint-next($name, $breakpoints);
  @return if($next, breakpoint-min($next, $breakpoints) - 1px, null);
}

To:
@function breakpoint-max($name, $breakpoints: $grid-breakpoints) {
  @return if($name, breakpoint-min($name, $breakpoints) - 1px, null);
}

It works as expected.

That's not the right fix I think @ksrb; that doesn't snag the next breakpoint at all. I think what we need to do is clear up the nested mixins on the between mixin and then not reuse that one for the only mixin. Doesn't make sense to map those to one another.
Change I'm suggesting is explicit min and max mapping:
// source
@mixin media-breakpoint-between($lower, $upper, $breakpoints: $grid-breakpoints) {
  $min: breakpoint-max($lower, $breakpoints);
  $max: breakpoint-max($upper, $breakpoints);

  @media (min-width: $min) and (max-width: $max) {
    @content;
  }
}

@mixin media-breakpoint-only($name, $breakpoints: $grid-breakpoints) {
  $min: breakpoint-min($name, $breakpoints);
  $max: breakpoint-max($name, $breakpoints);

  @media (min-width: $min) and (max-width: $max) {
    @content;
  }
}

// examples
.test {
  @include media-breakpoint-between(sm, md) {
    background: red;
  }
}
.test2 {
  @include media-breakpoint-only(sm) {
    background: blue;
  }
}

Which outputs this:
@media (min-width: 767px) and (max-width: 991px) {
  .test {
    background: red;
  }
}

@media (min-width: 576px) and (max-width: 767px) {
  .test2 {
    background: blue;
  }
}

@mdo sorry to come back to this after so many months, so it seems like there was actually a bug.
But I was mostly confused about the semantics of the breakpoint functions.
$grid-breakpoints: (
  xs: 0,
  sm: 576px,
  md: 768px,
  lg: 992px,
  xl: 1200px
) !default;

To me @media-breakpoint-between(sm, md) should target sm: 576px and md: 768px.
But what it does is actually target the 'next' breakpoints  md: 768px  and lg: 992px.
I'm just trying to understand the reason for this, when you see @media-breakpoint-between how do you interpret it? Maybe it'll help me get my head around it.

I'm also confused of this (using BS 4.1.3):
      @include media-breakpoint-between(sm, md) {
        grid-template-columns: 1fr 1fr;
      }

and it's outputting
@media (max-width: 991.98px) and (min-width: 576px)
.representative-sheet-options-container .representative-sheet-options.culoare-esarfa, .representative-sheet-options-container .representative-sheet-options.culoare-esarfa-sef-de-promotie {
    grid-template-columns: 1fr 1fr;
}

I was expecting between 576 and 768.

I'm also confused of this (using BS 4.1.3):
      @include media-breakpoint-between(sm, md) {
        grid-template-columns: 1fr 1fr;
      }

and it's outputting
@media (max-width: 991.98px) and (min-width: 576px)
.representative-sheet-options-container .representative-sheet-options.culoare-esarfa, .representative-sheet-options-container .representative-sheet-options.culoare-esarfa-sef-de-promotie {
    grid-template-columns: 1fr 1fr;
}

I was expecting between 576 and 768.

I was caught up on this for a second, but it's because the md range goes (in your case) from 768 to 991. When you specify media-breakpoint-between, you are considering the entire ranges of the breakpoints.
That is from 576 for the start of the sm breakpoint to 991.98, the end of the md breakpoint range. The next breakpoint range for lg starts at 992.


S3TransferUtilitySample - Make public an uploaded file from Android

Hi everybody!
Thanks for the excellent sample! it worked great!
Does anybody has an example that let me put the uploaded file in a public state in order to get access from an url?
Thanks!

There are a couple of ways to do it.

changes the ACL of the object via AmazonS3.setObjectAcl(String, String, AccessControlList). This requires an additional network request after the object is uploaded. See Managing Access with ACLs for more information.
manages the bucket policy. See Using Bucket Policies and User Policies and Bucket Policy Examples.

Thanks fosterzhang! it worked inmediately with:
manages the bucket policy. See Using Bucket Policies and User Policies and Bucket Policy Examples.
thanks!

This issue is closed. thanks.


No component factory found for DetachedLoader

Environment
Provide version numbers for the following components (information can be retrieved by running tns info in your project folder or by inspecting the package.json of the project):

CLI: 6.2.1
Cross-platform modules: 6.2.1
Android Runtime: 6.2.0
iOS Runtime: 6.2.0
Plugin(s):
NativeScript-Angular:8.20.3
Angular: 8.2.13

I've recently upgraded to NS 6.2 and I can't get the app to run on release mode.
On release I'm building the project using AoT.
Here's the runtime exception:
No component factory found for DetachedLoader. Did you add it to @NgModule.entryComponents?

Error: No component factory found for DetachedLoader. Did you add it to @NgModule.entryComponents?
    at ZoneAwareError (<embedded>:116749:33)
    at noComponentFactoryError (<embedded>:18833:17)
    at CodegenComponentFactoryResolver.push.CodegenComponentFactoryResolver.resolveComponentFactory (<embedded>:18880:19)
    at new PageRouterOutlet (<embedded>:83425:47)
    at createClass (<embedded>:21830:20)
    at createDirectiveInstance (<embedded>:21699:20)
    at createViewNodes (<embedded>:30059:36)
    at callViewAction (<embedded>:30375:13)
    at execComponentViewsAction (<embedded>:30294:13)
    at createViewNodes (<embedded>:30087:5)

It works fine when running in debug mode.
Any ideas?

HI @sebestindragos,
I tested your case with a newly created project. However, I was unable to recreate such an issue, while building it for release with AoT. As a reference, I am attaching my sample project. On that matter, it will help if you can send us your project and the exact steps that you've followed while building the app.
Archive.zip

Hi @tsonevn and thanks for the quick reply. I wasn't able to reproduce either using a blank project. Our production NS app has around 7 other plugins so that might be the issue. I can share the repository with you, just need your BitBucket account.


allow styles to change overflow

E.g.
I have a textarea for editing that needs autosize but also is affixed.
When the content is too long I set a maxHeight so that the affix doesn't drop off the bottom of the page.. but now I wish to set overflow: auto so that textarea can be scrolled.

I'd prefer not to set overflow: hidden when we have maxHeight or rows set and content overflows the textarea. That way you request would be fulfilled and it won't require manual style fixing.

I don't follow? Right now the autosized-textarea is setting overflow: hidden which cannot be changed by the implementation.
Assuming the overflow: hidden is there for a reason? Might it not be more simple to just allow implementations to explicitly override this hidden setting instead of having the textarea respond to edge cases?

Fix released in 2.1.3.


HIVE-24409 : Use LazyBinarySerDe2 in PlanUtils::getReduceValueTableDesc

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

LGTM. +1


"y" property of text ignored

Hello ! The "y" property of the text seems to always be ignored. Example:
<text  x="0"  y="0" dominant-baseline="hanging" v-wrapper="{ text: myTextValue, width: 200, align: 'none'}" />

Will give the same result as:
<text  x="0"  y="50px" dominant-baseline="hanging" v-wrapper="{ text: myTextValue, width: 200, align: 'none'}" />

will the text should be 50px lower

After further investigation, it appears to be because the x and y value of the each generated tspan is set to 0, while just not setting these value would have them inherit from the parent text

Good point, I am not sure why i set the y to 0, but your point is definitely correct. I fixed it.

Awesome reactivity, thank you very much !


[Forwardport] GKE Provisioning, unable to create a subnet in network vpc with an existing subnet

Forwardport rancher/rancher#21079

Created by mistake. closing.