Add headers to every API call

If one has the whole API behind auth, then  all calls to it need the header(s) added to it.
This PR wraps the fetch() calls to the API to always add the headers.

Wow, this is amazing! 😊 Thank you for the PR.


add darkGrey and other avh4/elm-color colors

Looks like these are missing, there's just black and white:
, red, orange, yellow, green, blue, purple, brown
, lightRed, lightOrange, lightYellow, lightGreen, lightBlue, lightPurple, lightBrown
, darkRed, darkOrange, darkYellow, darkGreen, darkBlue, darkPurple, darkBrown
, white, lightGrey, grey, darkGrey, lightCharcoal, charcoal, darkCharcoal, black
, lightGray, gray, darkGray

would be nice to have these as my code that uses avh4/elm-color depends on them.

Hi @bburdette!
Thank you for this consideration!
I personally recommend creating your own module where you define these colors.
You could copy/paste them from avh4/elm-color, put them into their own module and use them along side this new library.
The reason is simply that I did not want to include any specific, subjective colors in this library but instead wanted to focus only on the abstractions.
Let me know if you are running into any other issues.
Best


Corpus level urns are inaccurate

Corpus level URNs listed on the results pages are not accurate.

The URN listed above should return ALL editions of the Gospel of Mark, not just the manually edited ones.  But it is being displayed as if it were the URN for just the manually edited chapters of Mark.
Likewise this URN is incorrect:

I am now checking all of them and will list them in the thread. Use the URNs in the inventory Bridget provided that are just above the document level urns in the hierarchy.

Likewise: urn:cts:copticLit:nt.1cor should be urn:cts:copticLit:nt.1cor.sahidica_ed

The URN urn:cts:copticLit:nt is being listed next to the unedited NT corpus name. This is misleading.  The URN urn:cts:copticLit:nt should resolve to ALL New Testament documents, whether they are in the unedited corpus, edited corpus, or anyhwere -- that is the URN for any Coptic New Testament document anywhere.  It is not a unique identifier for the corpus.
This needs to be fixed. @amir-zeldes created subcorpora in the big NT corpus for precisely this reason.  You should use URNs for the subcorpora.  so urn:cts:copticLit:nt.1cor.sahidica is the URN for all documents from 1 Corinthians in the unedited sahidica.nt corpus.
There is NO urn for the corpus.  Only for each book of the bible.

I've updated the URNs for the Gospel of Mark and 1 Corinthians. Let me know what you'd prefer to have listed next to the New Testament title--for the moment, I've hidden the "urn:cts:copticLit:nt" next to the title.

Great.  That's the right choice.  It's possible to list a urn for the subcorpora, but this is fine for now.


New lines on CLI (hugo server)

What version of Hugo are you using (hugo version)?
Hugo Static Site Generator v0.75.1-A4A7BAB7/extended linux/amd64 BuildDate: 2020-09-15T06:57:20Z

Does this issue reproduce with the latest release?
Yep.
Every now and then when running hugo server or hugo the first WARN (if any) is on the same line as "Building sites ...". Today for instance I had both cases while solving some issues:
[16:23:03][patrick@bookra] ~/Projects/Me/samui-samui.de (+1|%0|!12826|0)
❯ hugo server
Building sites … WARN 2020/09/18 16:24:01 .File.UniqueID on zero object. Wrap it in if or with: {{ with .File }}{{ .UniqueID }}{{ end }}
WARN 2020/09/18 16:24:01 .File.Dir on zero object. Wrap it in if or with: {{ with .File }}{{ .Dir }}{{ end }}

and
[16:53:52][patrick@bookra] ~/Projects/Me/samui-samui.de (+1|%0|!12806|0)
❯ hugo server
Start building sites …
WARN 2020/09/18 16:54:55 .File.UniqueID on zero object. Wrap it in if or with: {{ with .File }}{{ .UniqueID }}{{ end }}
WARN 2020/09/18 16:54:55 .File.Dir on zero object. Wrap it in if or with: {{ with .File }}{{ .Dir }}{{ end }}

What seems strange is that there is a "Building sites" and a "Start building sites" in those samples for the same/identical CLI call.
It might sound like hair splitting to always want a new line for warnings and errors, but my specific use case here is, that I want to add automatic tests for issues with my sites and having WARN and ERROR be the first characters on a line is easier testable (maybe even more performant?) than using a regexp if those terms occur on any line somewhere within the line.

Sorry, my fault. Somewhere between those two calls I updated my Hugo from 0.74 to 0.75. The current version does what I need. Sorry again.


fix(internet-header): accessibility issues and broken config

Preview environments are getting posted here, as soon as they are ready!


Module build failed: Error: Node Sass does not yet support your current environment: Linux 64-bit with Unsupported runtime (53)

I have 3 electron-webpack project, my and two boilirplate. When I have added sass-loader with node-sass to my project I have an error only in my project
Module build failed: Error: Node Sass does not yet support your current environment: Linux 64-bit with Unsupported runtime (53)
node sass version identical
mxm@pc ~/D/e/social_pool1> ./node_modules/.bin/node-sass -v
node-sass	4.5.2	(Wrapper)	[JavaScript]
libsass  	3.5.0.beta.2	(Sass Compiler)	[C/C++]
mxm@pc ~/D/e/social_pool1> node -v
v7.5.0

I try:

remove-install node-modules folder
rebuild node-sass
change version

...and after node-sass not work only for my project
plz help my

Same here, different env:
Error: Node Sass does not yet support your current environment: OS X 64-bit with Unsupported runtime (51)

@andrefa, I have changed the version but nothing has changed(

Sorry, Electron isn't a runtime that is supported. You can read the various issues previously opened https://github.com/sass/node-sass/search?q=electron&type=Issues&utf8=✓

Module build failed: Error: Node Sass does not yet support your current environment: Windows 64-bit with Unsupported runtime (64)


Use PWMO files

Hello! Currently when using Chitbux after slicing I only get to save as a pwmo-file. Not cbddlp-file, so therefore I can't open the file in Photon File Validator. My printer is an Anycubic Photon Mono and Chitubox is configured for this printer. Is there any way to open pwmo files? Or can I somehow convert a pwmo file to cbddlp?

PFV is no longer developed
use UVTOOLS instead
No dia terça-feira, 16 de fevereiro de 2021, Martin Sonesson <
<EMAIL_ADDRESS>escreveu:

Hello! Currently when using Chitbux after slicing I only get to save as a
pwmo-file. Not cbddlp-file, so therefore I can't open the file in Photon
File Validator. My printer is an Anycubic Photon Mono and Chitubox is
configured for this printer.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Photonsters/PhotonFileValidator/issues/94, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUR56TLA3NNWEYKIXX73PLS7G67FANCNFSM4XVRRZOQ
.

--
Com os melhores cumprimentos,
Vinicius Silva


[BD-46] fix: use currentTarget in FormAutosuggest component to get selected option

Description
Fixes a bug with Form.Autosuggest component where clicking on MenuItem's children would pass incorrect selected value to the onSelected function
Deploy Preview
Include a direct link to your changes in this PR's deploy preview here (e.g., a specific component page).
Merge Checklist

[ ] If your update includes visual changes, have they been reviewed by a designer? Send them a link to the Netlify deploy preview, if applicable.
[ ] Does your change adhere to the documented style conventions?
[ ] Do any prop types have missing descriptions in the Props API tables in the documentation site (check deploy preview)?
[ ] Were your changes tested using all available themes (see theme switcher in the header of the deploy preview, under the "Settings" icon)?
[ ] Were your changes tested in the example app?
[ ] Is there adequate test coverage for your changes?
[ ] Consider whether this change needs to reviewed/QA'ed for accessibility (a11y). If so, please add wittjeff and adamstankiewicz as reviewers on this PR.

Post-merge Checklist

[ ] Verify your changes were released to NPM at the expected version.
[ ] If you'd like, share your contribution in #show-and-tell.
[ ] 🎉 🙌 Celebrate! Thanks for your contribution.

Thanks for the pull request, @viktorrusakov!
When this pull request is ready, tag your edX technical lead.


Feature/steam 838

This PR implements_ input components for a dynamic server URL and a list of patient ids, making UI modifications to handle these new input components and a few other changes. Beyond the implementation of the task, there are notable changes this PR introduces. It:

Refactors the fetching logic into a new util file;
Adds a message when there is no data found for a given patient id;
Adds Feather Icons dependency as our icon library;
Adds better empty-data logic to individual DataView components;
Moves all components associated with the Data View into a corresponding folder;
Moves all components associated with the Request Form into a corresponding folder;
Removes unnecessary logo file.

I think that's entirely reasonable. I took that approach in my most recent commit, making things look a bit better on my display. Let me know if they still look wonky on your end!


Bug report: Docker quick start broken

Describe the bug
When trying to use the quick start docker instructions from the README I get an error instead of a working image.
To Reproduce
Steps to reproduce the behaviour or a link to the recipe / input used to cause the bug:
$ docker run -it -p 8080:80 ghcr.io/gchq/cyberchef:latest
Unable to find image 'ghcr.io/gchq/cyberchef:latest' locally
docker: Error response from daemon: Head "https://ghcr.io/v2/gchq/cyberchef/manifests/latest": denied: denied.
See 'docker run --help'.

Visiting the URL https://ghcr.io/v2/gchq/cyberchef/manifests/latest gives
{"errors":[{"code":"UNAUTHORIZED","message":"authentication required"}]}

Expected behaviour
It should just work, looks like someone has locked it down too much by accident.
Desktop (if relevant, please complete the following information):
Running on latest Ubuntu, but that won't affect anything.

My fault, I've got a docker config file with creds in to access a private GitHub repo, for some reason with that in place I get denied access. Moving that file aside I can pull fine.
For anyone else on Linux who gets this, the file is ~/.docker/config.json and to get the image, just rename the file then put it back after.


Peer Review - Enhance README Documentation

Enhance README Documentation
The README provides some useful guidance, but it could be improved by including a step-by-step example like screenshot of one specific analysis. This would help users, especially those unfamiliar with the tools and setup, to follow along more easily and understand the process better.

Step by step screenshot to launch and run the anylysis added


Fix servers?dsID=<> route to return mids correctly

What does this PR (Pull Request) do?

[x] This PR fixes #5493

Which Traffic Control components are affected by this PR?

Traffic Ops
CI tests

What is the best way to verify this PR?

create a ds that employs both the edge and mid tier. i.e. a DNS delivery service
add a required server capability to the ds
add the same server capability to a handful of edge servers (ensure those edge servers are in a cache group that is parented by a mid cache group) and assign those edges to the ds
GET /servers?dsId={{dsId}} and make sure that the mids are not returned because they dont have the server capability
If this is a bug fix, what versions of Traffic Control are affected?

master

The following criteria are ALL met by this PR

[x] This PR includes tests
[x] This PR does not include documentation
[x] This PR does not include an update to CHANGELOG.md
[x] This PR includes any and all required license headers
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Additional Information

NOTE: This si the part that is necessary to fix this behavior in the 5.1 milestone. I'll create another PR for API 4.0(TC 6.0).

I had forgotten to add a WHERE clause, should be fixed now.

Should be fixed now


Paypal & Stripe Support

Support for Stripe and Paypal Payments

Hey. We will try our best to integrate.

hey @seedarren,
It my pleasure to inform you that strip is now supported by J2pay version 2.8.8.

Thanks
Sent from Yahoo Mail on Android
On Thu, Jan 3, 2019 at 3:58 PM<EMAIL_ADDRESS>wrote:
hey @seedarren,
It my pleasure to inform you that strip is now supported by J2pay version 2.8.8.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

dear @seedarren,
According to our understanding paypal required to redirect on paypal website, while we are only focused on server to server communication and for this paypal provide payflow pro that is already integrated in our library.
waiting for your response. I have to close this issue.


[1.17.1[ Soy Milk to substitute cow's milk

Hey there,
very interesting mod, you can almost live vegan, except for the milk.
Have you ever considered adding Soy Milk?
Thank you for your great work.
Another question:
What exactly is com.kwpugh:pugh_lib_fabric_1.17.1:1.1.5 ? I tried to fork the mod and add it myself (to use it privately) but my IDE couldn't find it and neither did I really.
~mycf / DruDo

I have been thinking about Soy Milk.
PughLib is a private library in my local Maven that handles the recipe remainder for some of the tools (i.e. tools stay in crafting grid and take durability).  I do not plan to make that public to avoid mod conflicts.

prerequisites: grow soybeans and make tofu
So, 1st we make a bucket of Raw Soy Milk

2nd, we heat it up on a campfire

3rd, we craft stuff with it

The cooked Soy milk bucket will have 16 uses

What do you think?

Hmm, the tags seem to work with Croptopia, nice

Add to v1.1.17, enjoy.

Thank you very much! Nice work!


Setup daily pipeline to catch any future incompatibilities with robotframework

Robot has already changed their API once (robotframework 3.2). Aside from that we're using a few non-official functions of theirs. Setup a test pipeline that will run daily, running the tests against the latest versions of OS(s), Python and robotframework, so that an email is received if it fails. This will help to quickly spot any issues further down the line and issue a patch quickly.

Daily pipeline setup that runs against the latest python and robotframework. Any future issues will be spotted via email notifications


Fix class not found error: com/nvidia/spark/rapids/GpuScalar

closes #11196
Bug
The bug is that GpuColumnVectorUtils  accesses shimmed GpuScalar from  unshimmed GpuColumnVectorUtils.
Two class paths in Rapids jar:
./com/nvidia/spark/rapids/GpuColumnVectorUtils.class
./spark-shared/com/nvidia/spark/rapids/GpuScalar.class
fix
Move related utility functions to GpuCaseWhen.
Signed-off-by: Chong Gao<EMAIL_ADDRESS>

build

build

I found a local repro. The key condition to trigger the bug is to add the plugin jar via the Spark system classloader using *extraClassPath instead of spark.jars. In our integration tests this happens automatically when we test RapidsShuffleManager. Please label one of your tests as a shuffle_test  to catch this regression in premerge. For instance:
$ git diff -w
diff --git a/integration_tests/src/main/python/conditionals_test.py b/integration_tests/src/main/python/conditionals_test.py
index b95ed53f3..0a9496b10 100644
--- a/integration_tests/src/main/python/conditionals_test.py
+++ b/integration_tests/src/main/python/conditionals_test.py
@@ -18,7 +18,7 @@ from asserts import assert_gpu_and_cpu_are_equal_collect, assert_gpu_and_cpu_are
 from data_gen import *
 from spark_session import is_before_spark_320, is_jvm_charset_utf8
 from pyspark.sql.types import *
-from marks import datagen_overrides, allow_non_gpu
+from marks import *
 import pyspark.sql.functions as f

 # mark this test as ci_1 for mvn verify sanity check in pre-merge CI
@@ -310,6 +310,7 @@ _case_when_scalars = [
     ['CAST(1.1 AS DECIMAL(12,2))', 'CAST(2.2 AS DECIMAL(12,2))', 'CAST(3.3 AS DECIMAL(20,2))', 'CAST(4.4 AS DECIMAL(12,2))', 'CAST(5.5 AS DECIMAL(12,2))'], # decimal(12) and decimal(20)
     ['CAST(1.1 AS DECIMAL(20,2))', 'CAST(2.2 AS DECIMAL(20,2))', 'CAST(3.3 AS DECIMAL(20,2))', 'CAST(4.4 AS DECIMAL(20,2))', 'CAST(5.5 AS DECIMAL(20,2))'], # decimal(20)
 ]
+@shuffle_test
 @pytest.mark.parametrize('case_when_scalars', _case_when_scalars, ids=idfn)
 def test_case_when_all_then_values_are_scalars(case_when_scalars):
     data_gen = [

Should GpuColumnVectorUtils be moved to sql-plugin-api so this type of issue cannot happen in the future? If so, can be a followup issue to track.

This should be trivial now, that most of the code is moved out of the class. As for the reason GpuColumnVectorUtils is kept  unshimmable is that it is referenced in https://github.com/dmlc/xgboost/blob/a6a8a55ffab4eb518998c7f1d47bbff99086827c/jvm-packages/xgboost4j-spark-gpu/src/main/scala/ml/dmlc/xgboost4j/scala/rapids/spark/GpuUtils.scala#L34

Should GpuColumnVectorUtils be moved to sql-plugin-api so this type of issue cannot happen in the future? If so, can be a followup issue to track.

There are other unshimed classes, if we move GpuColumnVectorUtils, it's better to move all the classes:
com/nvidia/spark/GpuCachedBatchSerializer*
com/nvidia/spark/ParquetCachedBatchSerializer*
com/nvidia/spark/rapids/ColumnarRdd*
com/nvidia/spark/rapids/GpuColumnVectorUtils*
com/nvidia/spark/rapids/ExplainPlan.class
com/nvidia/spark/rapids/ExplainPlan$.class
com/nvidia/spark/rapids/ExplainPlanBase.class
com/nvidia/spark/rapids/PlanUtils*
com/nvidia/spark/rapids/SQLExecPlugin*
com/nvidia/spark/rapids/ShimLoaderTemp*
com/nvidia/spark/rapids/SparkShims*
com/nvidia/spark/rapids/optimizer/SQLOptimizerPlugin*
org/apache/spark/sql/rapids/AdaptiveSparkPlanHelperShim*
org/apache/spark/sql/rapids/ExecutionPlanCaptureCallback*

Filed an issue to track how to prevent this kind of issue:
https://github.com/NVIDIA/spark-rapids/issues/11208


Is it working with UIButton?

I can see that, you've made by inheriting UIView. What if I want to do the same for UIButton?

My bad: UIButton <- UIControl <- UIView. So it should work.

You can place the UIButton within a NAUIViewWithBorders and pin it to each side using constraints.  If you want to try to use the code to make a UIButton do essentially the same thing, go ahead and give it a shot.  However, UIButtons have some weird internal workings that might cause conflicts.  I've tried to subclass UIButtons in the past and ran into various unpredictable issues when trying to manipulate its layout.  Anyway, my first suggestion would be the easiest.


mined gas is worth 0

infinite money glitch no longer real

but it's not infinite money because the only way to sell gas is via gas tanks and gas tanks cost like 1k each

im watching atmos in real time take a canister, fill it up with 650 thousand mols of oxygen and sell it

oh well if they're getting 65k per can out of pure oxygen and it's less labor than frezon or smth then yeah this is good
although i don't think co2 miners are ever mapped so maybe keep co2 sell price

almost every map has a co2 miner

i stand corrected

there definitely should be  cl

cmb cooled oxygen :trollface:


Rename the gpt-4-turbo aliases

Currently: https://github.com/simonw/llm/blob/ab1cc4fd1f5ea1c037a8d8c56b455d11448e5577/llm/default_plugins/openai_models.py#L30-L33
But quite recently OpenAI made gpt-4-turbo a valid reference to the latest GPT-4 Turbo (including vision) API model.

I'm just going to swap gpt-4-turbo-preview to be an alias, not a model name.
This does mean people with existing log databases will have old gpt-4-turbo-preview records and new gpt-4-turbo records but I think that's OK, I'll mention that in the release notes.


Minutes to Hours

Example Problem:
Convert 230 minutes to Hours&Minutes
Example Solution:
minutes_to_hours(230) = 3h:50m
Further explanation (optional):
This converts the time (in minutes) into the hour-minute format
Would you like to be assigned to this:
Yes, I would like to be assigned this

Yes, I would like to be assigned this


Add Code of Conduct, issue and PR templates.

As recommended by the Community profile

Heh. I hope we won't have a war over this. It seems like the idea of having a Code of Conduct is rather controversial, and I'm also sitting on the fence.

The first article in the Code of Conduct is not to go to war over Codes of
Conduct.
I'm happy one way or the other. Slightly tipped towards having one, since
GitHub advises us to do so.

Well, here's a TODO list they give:

[ ]  Research different codes of conduct designed for open source projects. Choose one that reflects your community's standards.
[ ]  Consider carefully whether you are willing and able to enforce it.

And they propose these:

https://contributor-covenant.org/
https://www.djangoproject.com/conduct/
http://citizencodeofconduct.org/

IF we were to add a code of conduct, which one would it be?

The second issue is way more important than the first. I don't know how we
would do that.
Besides, there's the issue of having a code just for this site, which is
osmotically related to the rest of the Perl6 repos. It's the whole
community that should take that decision, not us in this corner of the
Perl6 universe.
I think we should better leave that aside for the time being.

IMO that particular template is completely useless, but realistically I hate all issue templates…

I'm reporting … … …

Well, that's why there are labels, but people can't add these themselves. So if in the end someone will have to look at the issue and add appropriate labels anyway, so why should we make it harder for bug reporters?

+<!-- describe the problem, the page where you have found it -->

Well, dah, that's exactly why the user ended up here anyway.

If applicable, state what could be done to solve the problem, such as "Rephrase the description" or "Use an example program that actually runs"

Could be left there, but still rather obvious.
The only part I have no problems with is:

+<!-- Thanks a lot for your contribution. -->

And yes, this is not minimalistic to me. It's some bloated paperwork that I as a contributor will have to read for no good reason.

The COC was discussed last year. The consensus was this:
"We are a bunch of volunteers developing Perl 6 and Perl 6 applications, and try real hard to be nice to each other."
https://perl6.org/community/

We already dismissed the CoC...

This issue has been open for some time now. It's better if you say your
opinion before somebody acts on it, because if you do it after, it's
useless work done by someone.
You are right about the TODOs. It's a nuisance. You are also right about
the stub. Another nuisance. Both things can probably be fixed, but we can
also just take templates away.
Everyone is still on time to decide on the PR template. If no one is
strongly for it, I'll close this issue removing the existing template and
that's that.

It's better if you say your opinion before somebody acts on it

I agree, of course, but there are many tickets across many repos and I'm not omnipresent unfortunately. Sorry.
FWIW we do have a template in rakudo repo, which I also don't like… but most people seem to be OK with it, so maybe they'll also be OK with this doc issue template (as long as mentioned problems are resolved of course).

I might go for using the initial sentence, and also eliminating the ticks,
which might be useful for the PR template (if eventually adopted)
In fact, the issue that has used the template (besides mine) is a good
example of what I was looking for. It provided all the information needed
to fix it, and I could act straight away. I'm looking at old issues, and
it's almost impossible in many cases to figure out what is actually what is
expected from us.

Just to round up the discussion, the code of conduct has been raised repeatedly in the #perl6 IRC channel. Here's one of the earliest references.

I'm restoring the original title and reopening the issue.

There is something like a code of conduct as a draft in the specs repo. Never made it to the specs, apparently.

The draft sounds good.
Also might be worth a mention here: GNU Kind Communications Guidelines

https://github.com/Raku/Raku-Steering-Council/blob/main/papers/Raku_Code_of_Conduct.md


NonEditingAuthor

The persission DeleteOwnedItems ist missing at the role NonEditingAuthor.
Should be changed from:
NonEditingAuthor: CreateItems, FolderVisible, ReadItems
to:
NonEditingAuthor: CreateItems, FolderVisible, ReadItems, DeleteOwnedItems

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

ID: f1a893c8-66de-1f8a-f8d2-aa573379e778
Version Independent ID: 1c32c637-d003-820c-594f-a9adc9c353d9
Content: Add-MailboxFolderPermission (ExchangePowerShell)
Content Source: exchange/exchange-ps/exchange/Add-MailboxFolderPermission.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@mark-tsc, confirmed.

Also affects the following topics, so I updated them, too:

Set-MailboxFolderPermission
Add-PublicFolderClientPermission
Remove-PublicFolderClientPermission

Thanks for the feedback.
Closing


Mount options for disk device mounts

Currently you can mount a device into a container by adding it as disk to your containers configuration using: lxc config device add <container> <name> disk source=/dev/<device> path=<relativepath>
While this works great for basic filesystems like ext4, this makes using specific mount options difficult for anything more complex like btrfs.
I'm trying to specifically mount a btrfs subvol inside of a container, and while I can always mount it on the host manually, or just include it in the fstab pointing into /var/snap/lxd/common/lxd/containers/<container>/rootfs/<mountpoint> with my mount options, it seems like a messy solution especially when you get into multiple mount points, and possibly migrating containers.
Unless I've missed already implemented somewhere, or I should be approaching this differently, I believe this would be a useful feature to add.

So I'm still not fond of the idea (this has been suggested before) but as we don't really have a better answer to it, I suspect a raw.mount.options config option on the disk device would probably be okay.
Once we start implementing project restrictions (#6170) this would then get restricted in the same way a raw.lxc config option would be.

Hi, Lily and I would like to claim this issue for our group.

Excellent, for this one, I expect roughly the following commits:

api: Add container_raw_mount API extension

Add that string to shared/version/api.go
Add a matching entry to doc/api-extensions.md

lxd: Add support for `raw.mount.options:

Add the new option to lxd/device/disk.go
Make the logic in disk.go add the mount options to the returned RunConfig
Add matching logic to pass the flags through to forkmount for hotplug

tests: Add test for raw.mount.options

Extend test/suites/container_devices_disk.sh to test the new configuration option. One idea to test it would be to use a tiny vfat filesystem on a loop device, add it to the container with some mount options (I'd suggest using uid= and gid=) and confirm that this is properly used in the container.
For loop device handling, you'll want to use truncate to create a small (10MB should do) file, then use mkfs.vfat on top of that file, lastly map it to a loop using our configure_loop_device helper, cleaning things up after the test with deconfigure_loop_device.
The test should validate that this works both on at boot time and when hot plugging the device into the container.

@tomponline should be able to help you with any question you may have regarding this issue.

@splett2 Yep let me know if you have any questions.

@tomponline
I'm guessing most of the logic we're implementing for lxd: Add support for `raw.mount.options:
would be in the Start() function, and I see that there are already a number of mount options. If a user decides to pass in options via raw mount options, should we ignore any other potential mount parameters?

I think raw.mount.options should get appended to anything we already came up with internally

Hi, sorry for the radio silence, we ended up busy with projects and other deadlines, we're going to start working on this now.

@splett2 no worries, let us know if you have any questions

Hello, I'm working with David on this project
We saw two places to add logic for the mount options stemming from Start() in disk.go:
1.) Start() calls createDevice() which calls device_utils_disk.go’s DiskMount() which performs a unix.Mount(). We’re thinking of adding the raw.mount.options to this unix call, do you think that is needed? It looks to be creating the device on the host which is unmounted later.
2.) We see that we’re to add the options to the RunConfig, to handle this added logic, is changing insertMountLCX() and insertMountLXD() in the right direction?

@Lilyjjo Indeed the main spot to pass the extra mount options is the first unix.Mount call in DiskMount. I'd suggest adding an extra argument to DiskMount allowing for arbitrary mount options to be passed through, passing those to unix.Mount.
I don't think you actually need to do anything about RunConfig as that part of the mount handling is only about telling liblxc to bind-mount the device mount into the container. We don't really care about the mount options on that bind-mount, what matters is the mount options on the original mount which is what createDevice and DiskMount do.

(I realize I'm the one who mentioned RunConfig/forkmount, as it turns out, we only use those two approaches to inject a bind-mount into the container, so don't actually need to touch those at all so long as the initial mount of the device is correct)

Hi,
David and I are a little stuck. We’ve changed the device config and passed the raw.mount.options to the unix.Mount() commands inside of DiskMount() but we’re not seeing the changes reflected in the container.
Here’s how we’re setting up the loop device on the host:
Make a new file file.txt, truncate it to 10M
run: sudo losetup —show -f file.txt (This creates dev/loop9)
run: sudo mkfs.vfat /dev/loop9
(That’s all we did to set up the loop device)
Then we’re running these commands for lxc:
lxc config device add first vfat1 disk source=/dev/loop9 path=/mnt/test raw.mount.options=gid=123,uid=456
And then run:
lxc exec first — stat /mnt/test
We’re not seeing the gid/uid changes being made. We’re not sure why this is the case.
We’re not sure if we need to mount the loop device on the host or modify any fstabs. We’re wondering if you see anything wrong with what we’re doing.
Thanks

@Lilyjjo What does grep loop9 /proc/mounts show on the host? Do you see uid/gid being set there?
Does everything show up as nobody/nogroup in the container?
If so, you need to test with a privileged container to make this easier.
lxc config set first security.privileged true && lxc restart first should take care of that.

@stgraber
Hi, we were able to test with the privileged container and verified that the raw mount options are getting passed in correctly. We also wrote an integration test, and we're trying to run the bash scripts for integration testing, but we're getting an error:
==> Checking for dependencies
==> Available storage backends: dir btrfs lvm
==> Using storage backend dir
==> Setting up directory backend in /home/lily/go/src/github.com/lxc/lxd/test/tmp.RiI/VlR
==> Spawning lxd in /home/lily/go/src/github.com/lxc/lxd/test/tmp.RiI/VlR
==> Spawned LXD (PID is 7132)
==> Confirming lxd is responsive
lxd: error while loading shared libraries: libdqlite.so.0: cannot open shared object file: No such file or directory
lxd: error while loading shared libraries: libdqlite.so.0: cannot open shared object file: No such file or directory
==> Cleaning up
cat: /home/lily/go/src/github.com/lxc/lxd/test/tmp.RiI/macaroon-identity.pid: No such file or directory
./main.sh: 28: kill: Illegal number:
==> Killing LXD at /home/lily/go/src/github.com/lxc/lxd/test/tmp.RiI/VlR
==> Tearing down directory backend in /home/lily/go/src/github.com/lxc/lxd/test/tmp.RiI/VlR
We aren't sure why this is happening considering we're able to run LXD without issue manually, but it fails to load when running the test script.

@Lilyjjo hmm, not sure what's causing that one. Could be that some of the export are missing, especially the LD_LIBRARY_PATH one?
You can probably sort out your commits and open a pull request, then I can have a look at the tests and if all looks fine, have jenkins run them.

@stgraber we've made the pull request for this issue, it's named raw_mount_options https://github.com/lxc/lxd/pull/6429


log in error raised

Traceback (most recent call last):
File "", line 1, in
File "singa-auto/singa_auto/client/client.py", line 101, in login
'password': password
File "singa-auto/singa_auto/client/client.py", line 792, in _post
return self._parse_response(res)
File "singa-auto/singa_auto/client/client.py", line 837, in _parse_response
raise SingaAutoConnectionError(res.text)
singa_auto.client.client.SingaAutoConnectionError
...
FATAL:  remaining connection slots are reserved for non-replication superuser connections\n\n(Background on this error at: http://sqlalche.me/e/e3q8)\n","error_code":500,"message":"System Internal Error","success":1

fixed in https://github.com/nusdbsystem/singa-auto/pull/107


[CLI] Extend training support to all trainers

Feature request
The CLI currently supports training models with SFT/DPO/KTO: https://github.com/huggingface/trl/blob/6859e048da601fec181997a324e7b351fc997a33/trl/commands/cli.py#L24
It would be good to extend this support to all trainers so that we have a consistent API and also learn which parts of our scripts need refactoring to support this usage.
This could be tackled in separate PRs to keep things lightweight, and I'll track here the trainers in terms of priority to add (based on Hub usage):

orpo
reward modelling
ppo
rloo
online dpo
gkd
cpo
bco
xpo
nash-md

Motivation
It is somewhat annoying that one cannot train a model through the CLI as this is helpful for fast debugging / iterations.
Your contribution
Happy to open PRs, but this could be a good first issue for new contributors!

Duplicate #1811, closing it in favour of this one

Hi, it looks like it'd be just the extension of the SUPPORTED_COMMANDS constant?
as for instance orpo would already be there https://github.com/huggingface/trl/blob/main/examples/scripts/orpo.py
I didn't check for the others but compared the scripts of kto with orpo for instance.

@grumpyp yes I think this is all one needs - a better solution would be to have an automated way to populate this constant by globbing all the _trainer.py files (e.g. make cli_commands). This way, anytime someone adds a new trainer, we automatically get support for it in the CLI :)

@grumpyp yes I think this is all one needs - a better solution would be to have an automated way to populate this constant by globbing all the _trainer.py files (e.g. make cli_commands). This way, anytime someone adds a new trainer, we automatically get support for it in the CLI :)

yes definitely! If you want, assign the issue to me please. I'll try to get a PR out today.

assigned! thanks for the offer to help 🤗

hi @lewtun
I didn't want to manipulate py-files via the Makefile so I went a slighly different approach.
It now creates the commands dynamically using a utility function which is cached.
Let me know if that works for you or if it needs changes. Thanks for the opportunity to contribute.


Wrong button label

Into users "reserve books" page, the search button label is "add book", instead of "search"
Also, when searching the content of the search bar is cleaned up, which is a weird behaviour

Closed by #80.
Also since the Search button did not exactly made sense there, we removed it.
If not clear for the user, it can be re-added.


simplify segment pruning

Segment pruning feels over-engineered:

DataSchemaSegmentPruner and ValidSegmentPruner are just one liners which can be applied when necessary
ColumnValueSegmentPruner and SelectionQuerySegmentPruner are mutually exclusive so never both need to run, and these cases can be identified easily by examining the QueryContext
No new segment pruners have been in a very long time

This leads to inefficiencies like

None of the pruners inline
Lots of lists are created unnecessarily
We end up tracing one liner checks

This PR removes the two trivial pruners and applies them inline within the two remaining pruners. It adds a new method to identify based on the query context whether the pruner should run at all.

Codecov Report

Merging #8790 (853cdc9) into master (c4549e2) will decrease coverage by 48.69%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master    #8790       +/-   ##
=============================================
- Coverage     62.86%   14.17%   -48.70%
+ Complexity     4601      168     -4433
=============================================
  Files          1690     1688        -2
  Lines         89212    89211        -1
  Branches      13411    13415        +4
=============================================
- Hits          56082    12642    -43440
- Misses        29079    75623    +46544
+ Partials       4051      946     -3105

Flag
Coverage Δ

unittests1
?

unittests2
14.17% <0.00%> (-0.03%)
:arrow_down:

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

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

LGTM


How to set max.in.flight.requests.per.connection ?

I want to output data to kafka in order, so I want to set max.in.flight.requests.per.connection to 1.
How to do ?

https://github.com/cclient/logstash-output-kafka
modify and rebuild
  props.put(kafka::MAX_IN_FLIGHT_REQUESTS_PER_CONNECTION, max_in_flight_requests_per_connection.to_s)  unless max_in_flight_requests_per_connection.nil?
  props.put(kafka::ENABLE_IDEMPOTENCE_CONFIG, enable_idempotence.to_s) unless enable_idempotence.nil?

docker pull cclient/logstash:6.8.13
docker pull cclient/logstash:6.3.0


subprocess.run -> run_subprocess

Implement a run_subprocess method to remove most of the verbosity introduced by subprocess.run

Thanks for the reminder! Would indeed like to update this and wrap this up.

Should be good to go!

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

@merveenoyan are you aware of why this test might fail?
______ HubKerasFunctionalTest.test_save_pretrained_task_name_deprecation _______

self = <tests.test_keras_integration.HubKerasFunctionalTest testMethod=test_save_pretrained_task_name_deprecation>

    def test_save_pretrained_task_name_deprecation(self):
        REPO_NAME = repo_name("save")
        model = self.model_init()
        model.build((None, 2))

        with pytest.warns(
            FutureWarning,
            match="`task_name` input argument is removed. Pass `tags` instead.",
        ):
            save_pretrained_keras(
                model,
                f"{WORKING_REPO_DIR}/{REPO_NAME}",
                tags=["test"],
                task_name="test",
>               save_traces=True,
            )
E           Failed: DID NOT WARN. No warnings of type (<class 'FutureWarning'>,) matching ('`task_name` input argument is removed. Pass `tags` instead.') were emitted. The list of emitted warnings is: [FutureWarning('`task_name` input argument is deprecated. Pass `tags` instead.')].

Failed before and after rebasing on current main.


duplicate content in sample config file

file https://raw.githubusercontent.com/yinqiwen/ardb/master/ardb.conf
line 285-292 duplicate via line 311-318

line 151-162 duplicate via line 272-283


Ignore NavigationBar in Popup

popup with background not ignore navigationview, is it possible to cover all screen while keep using navigationview ?

Hey @rifqifadh, it doesn't seem to be possible to force any view to be shown above navbar view. Navbar just takes priority. What you can do is move your popup up your view hierarchy to be "above" your navigation view. Like this:
NavigationView {
            ...
        }
        .popup(isPresented: $showingTopToast, backgroundColor: .black.opacity(0.5)) {
            Color.red.frame(width: 50, height: 50)
        }

It's been a while so I'm closing this one for now, please do let me know if you have any further issues. Have a good day!

Hey @rifqifadh, I think I found a solution using Apple's fullScreenCover, please check it out:
extension View {

    func transparentFullScreenCover<Content: View>(isPresented: Binding<Bool>, content: @escaping () -> Content) -> some View {
        fullScreenCover(isPresented: isPresented) {
            ZStack {
                content()
            }
            .background(TransparentBackground())
        }
    }
}

struct TransparentBackground: UIViewRepresentable {

    func makeUIView(context: Context) -> UIView {
        let view = UIView()
        DispatchQueue.main.async {
            view.superview?.superview?.backgroundColor = .clear
        }
        return view
    }

    func updateUIView(_ uiView: UIView, context: Context) {}
}

Use like this:
        .transparentFullScreenCover(isPresented: $viewModel.isPresentedSettings) {
            Color.black.opacity(0.5).edgesIgnoringSafeArea(.all)
                .popup(isPresented: $viewModel.isPresentedSettings, type: .toast, position: .bottom) {
                    DraggableCardView() {
                        SettingsView(viewModel: ProfileViewModel())
                    }
                }
        }

Sorry if I'm a bit late, have a great day!

Is there any way to achieve this without using fullScreenCover?

Is any way to pass user interaction when using fullScreenCover,
is it possible to manage that by presenting a new UI Window above the current window
@f3dm76

Hey @JoeOCT91, this is a pure SwiftUI library though. If using UIKit is acceptable for your project, then please use one of the number of great UIKit popup libraries

@JoeOCT91 Why are you tagging me?


data cannot be used in stats graphs

i tried to use stats graphs in ha but none of the sensors from miscale shows.
bodymiscale.userxxx needs to have state_class: 'measurement' set
default is none
see more here https://www.home-assistant.io/more-info/statistics/

ok according to this the base sensors shall have it https://github.com/esphome/esphome/blob/dev/esphome/components/xiaomi_miscale2/sensor.py

ok i start to understand... platform template sensors do not have it.
one needs to use integration template. I will post an example in the card repo

ok i think is fixed... i still need to see the data showing practically config for user sensors needs updating
  - platform: template
    state_class: measurement
    name: Weight Alin
    id: weight_alin
    unit_of_measurement: 'kg'
    icon: mdi:weight-kilogram
    accuracy_decimals: 2
  - platform: template
    name: Impedance Alin
    state_class: measurement
    id: impedance_alin
    unit_of_measurement: 'ohm'
    icon: mdi:omega
    accuracy_decimals: 0


How to access headers to build request context in 0.2.0

I had this code previously:
let app = Router::new()
    .nest(
        "/api/rspc",
        rspc_router
            .endpoint(|req: rspc::integrations::httpz::Request| {
                let headers = req.headers();

                let auth_token = headers.get("Authorization").map(|header_value| {
                    let bearer = Bearer::decode(header_value).unwrap();
                    let token = bearer.token().to_string();
                    token
                });

                RspcContext {
                    admin_service,
                    auth_token,
                    jwt_verifier: jwk_auth,
                    metrics_query_service,
                }
            })
            .axum(),
    )
    .layer(
        ServiceBuilder::new()
            // Other layers
            .layer(Extension(prisma_client)),
    );

In 0.2.0 you don't have access to req in this context, the closure takes no arguments. Like this:
        rspc_axum::endpoint(rspc_router.clone(), move || RspcContext {
            admin_service: admin_service.clone(),
            auth_token: None,
            jwt_verifier: jwk_auth.clone(),
            metrics_query_service: metrics_query_service.clone(),
        })

I thought, how about I make this a middleware instead? No luck unfortunately, the request attached to the middleware context only has kind and path, no headers.
Is what I'm trying to do possible with 0.2.0?

Figured it out from reading the source for rspc_axum, turns out you can do this:
        rspc_axum::endpoint(rspc_router.clone(), move |parts: Parts| {
            let headers = parts.headers;
            let auth_token = headers.get("Authorization").map(|header_value| {
                let bearer = Bearer::decode(header_value.into()).unwrap();
                let token = bearer.token().to_string();
                token
            });
            RspcContext {
                admin_service: admin_service.clone(),
                auth_token,
                jwt_verifier: jwk_auth.clone(),
                metrics_query_service: metrics_query_service.clone(),
            }
        }),

Happy days!
I know the docs are actively being updated, +1 to include this little tidbit in the docs 😃

You beat me to it but gonna post this response I was typing anyway incase it's useful.

I am going to improve docs around this over the next week or so but with this new release rspc has full support for Axum extractors that do not access the HTTP body.
From a technical perspective this means anything that implements Axum's FromRequestParts.
You can change your closure to |parts: axum::http::request::Parts| and it will work for what you need.


Option to configure the ZFS block size on volumes

Custom volumes on ZFS (both filesystems and zvols) should expose control of their recordsize or volblocksize properties respectively.
This is commonly used to "tune" (read: fix) performance of applications like databases on ZFS, where it can have a huge impact due to its COW nature.

Yeah, should be easy enough to add a new ZFS-specific volume property.
volume.zfs.blocksize when configured on a POOL, zfs.blocksize when configured directly on a volume.

volume.zfs.blocksize when configured on a POOL, zfs.blocksize when configured directly on a volume.
Isn't this the other way around?

no

On a pool, the config keys that will impact volumes are all prefixed with volume. whereas no such prefixing is needed on volume keys since they can only affect volumes :)

Thank you for clearing this up @stgraber.
Would like to give this one a shot to implement.
Do you have some tips/guidance on this? I'm still getting familiar with the code base.

The easiest is likely to based it off another option we have and look for that in the codebase.
In this case, I'd look for volume.zfs.use_refquota which is mentioned in:

doc/storage.md
lxd/storage_pools_config.go
lxd/storage/utils.go
lxd/storage/drivers/driver_zfs_volumes.go
lxd/storage/drivers/driver_zfs.go
scripts/bash/lxd-client

It's a bit different in that this one will take a size instead of a boolean and it doesn't apply during quotas but instead just at volume creation and update time, so should be a smaller delta than refquota.
@tomponline how difficult to you reckon it would be to get rid of those entries in lxd/storage_pools_config.go and lxd/storage/utils.go, only relying on the drivers to expose their pool and volume configs?
The current way we have things feels like a bit of layering violation, resulting from us having never fully completed the migration over to the new storage packages.

@stgraber shouldnt be difficult, as we've already made provision for taking that out at some point.
I think this section is dead code now that all storage drivers are ported over: https://github.com/lxc/lxd/blob/master/lxd/storage/utils.go#L307-L344
Which means we could then get rid of https://github.com/lxc/lxd/blob/master/lxd/storage/utils.go#L296 entirely as its only called from one place anyway.
The storagePoolValidateConfig and storagePoolValidate are a bit more involved, they may need a generic storage.Validate() function being added that loads the driver internally and calls its .Validate() function before the DB record is created.
Shall I add a task in trello for this?

@stgraber one thing I was a bit caution around was the use of storagePoolValidateConfig in the old patches for storage pool, as didn't want to interfere with the historical conversion process.

Yeah, seems worth adding a card to clean things up when we have some time.
For things being used from patches, we can move what we must keep to patches_utils.go

@TimRots are you still looking into this?

@TimRots are you still looking into this?

@TimRots are you still looking into this?

Yes. I would like to pick this up again over the upcoming weekend.

Excellent!

@TimRots hey, any progress on this?

Hi @stgraber as I have recently worked on another volume related issue, do you mind if I give this one a shot?

Sure!

Un-assigning due to inactivity, we're starting to run low on easy issues so putting this one back in the pool.

Hello, I am a student at UT Austin taking Virtualization - Could me and @BlakeRomero I try tackling this issue?

done!

Could I get assigned as well since we will be working together on this?

@stgraber I was preparing for job interviews and didn't really have time to work on this. Sorry about ghosting and the inconvenience I've caused.

@zakufish no worries!

I've been able to add zfs.blocksize and volume.zfs.blocksize as config options, but I am a bit confused on how I can actually provide the desired functionality to these config options.
Do you have any tips on where in the code I should be looking?
So far, I have found that in lxd/storage/drivers/driver_zfs_volumes.go, CreateVolume references a defaultBlockSize, and GetVolumeUsage references a stat.Bsize. I am unsure of whether these two pieces of code should be changed, and if they should be changed, how I should go about altering them to get the desired behavior.

So the code paths that would need to be covered for this are the volume creation ones:

CreateVolume
CreateVolumeFromBackup
CreateVolumeFromCopy
CreateVolumeFromMigration

And because I "think" those properties can be changed later on, probably also add support for blocksize to:

UpdateVolume

I suspect the easiest here would be to add a setBlocksize(vol Volume) function to the zfs struct in driver_zfs_utils.go.
That function would apply the value from zfs.blocksize to the volume, if the volume is of type Filesystem, then it would be applied as recordsize, if the volume is of type Block, then apply as volblocksize.

Ah, so I just played with them a bit. recordsize (for filesystem volumes) can be set to any value that's a power of 2 between 512B and 1M, it can be changed at any time.
volblocksize can only be set at creation time (zfs create), it cannot be modified at any other point.
So for recordsize, what I said above will work just fine, but for volblocksize, it will only be possible to do it from CreateVolume, in all other cases, it should be skipped.

Thanks for the tips! I was able to add the necessary code for each of the places you mentioned, and I have written a setBlocksize function as well. I do have a couple more questions:
First, I think it would make sense to include some logic for a default value for this property, but I am unsure of what that default value should be and how I would go about including this logic. Do you have any pointers towards relevant code or information about this?
Second, I am having trouble finding references in the code to the recordsize and volblocksize properties that you have mentioned. Where can I find more information on these?
Thank you again for your help!

Please note that users can use a tunable to be able to go above 1M for recordsizes.

Default should be not to set either of those properties.
recordsize and volblocksize are ZFS properties which you pass to zfs create or to zfs set

@andrewli6504 @BlakeRomero hello, just wanted to check if you still intend to work on this or if we should unassign it so someone else can take a crack at it?

Yes, I plan to hopefully finish this up soon.

Sounds good, thanks!

@stgraber I can work on this if you don't mind.

Absolutely!

@stgraber,
I think that I'm very close to finish this one but I encountered one problem. We try to use one property (blocksize) to control two others: recordsize for volumes of type Filesystem and volblocksize for volumes of type Block. It wouldn't be a problem if not the fact that this two properties can have different range of values. volblocksize can have value from 512 to 128k, power of 2. And recordsize can have values from 512 to 1M, power of 2. So if we set 1M on the pool and then try to create volume of type Block, it would be a problem with creation.
What do you think? What will be the best approach to solve this?

So I think what we can do is:

Allow recordsize to be anything in the 512 to 1M range (I'm assuming M here is MiB)
When setting volblocksize, we cap it to 128K so any value larger than it will just cause us to set 128K instead.

@stgraber I've just open PR. Should we allow for 16 MiB and allow to zfs throw the error if value is too big?

Yeah, that's fine.


Minimum Interval Does not work

Date picker minimum interval for 30 minute doesn't work

Do you mean the minuteInterval property of UIDatePicker? Works great for me if I set it in the demo project in -[RMViewController openDateSelectionController]:105.

Thanks..


Bug | Rubocop 0.89 | uninitialized constant RuboCop::Cop::TooManyLines

Hi guys, i have met the issue with updating of base rubocop version from 0.88 -> 0.89
File with module RuboCop::Cop::TooManyLines has been removed in this commit
All the places with this constant usage have been replaced with Rubocop::Cop::CodeLength.
Also module CodeLength has been patched to handle both of cases - previous behaviour of CodeLength module and TooManyLines functionality
Expected behavior
RuboCop is running successfully
Actual behavior
Error is raised during rubocop starting
uninitialized constant RuboCop::Cop::TooManyLines
/Users/my_user/.rvm/gems/ruby-2.6.5@my_project/gems/rubocop-graphql-0.4.0/lib/rubocop/cop/graphql/resolver_method_length.rb:12:in `<class:ResolverMethodLength>'

Steps to reproduce the problem
Run the rubocop v0.89.0 with included rubocop-graphql gem
RuboCop version
0.89.0 (using Parser <IP_ADDRESS>, running on ruby 2.6.5 x86_64-linux)

Proposed steps to fix

Bump up the version of Rubocop-Graphql (major(preferred) / minor), set required version of rubocop greater or equal than 0.89 and rename the module included to resolver_method_length Cop
Make the condition inside the resolver_method_length Cop with check the version of rubocop and including correct constant(this check could be done at the start of rubocop running - not in this file)

Hi @dzemlianoi-double, thanks for the report! I'll take a look at the issue when my vacation ends 🏖

Fixed as part of PR #24 and released a new version


Enrollment notifications

The user is able to receive package enrollment notifications and consent to them. Unsupported notifications are no longer fired.

:+1:


*.release.flatcar-linux.net DNS not resolving

Description
DNS for *.release.flatcar-linux.net isn't resolving:
$ host stable.release.flatcar-linux.net
Host stable.release.flatcar-linux.net not found: 3(NXDOMAIN)
$ host beta.release.flatcar-linux.net
Host beta.release.flatcar-linux.net not found: 3(NXDOMAIN)
$ host alpha.release.flatcar-linux.net
Host alpha.release.flatcar-linux.net not found: 3(NXDOMAIN)
$ host lts.release.flatcar-linux.net
Host lts.release.flatcar-linux.net not found: 3(NXDOMAIN)

I'm getting NXDOMAIN even when querying the domain's nameservers directly:
$ host stable.release.flatcar-linux.net ns.inwx.de
Using domain server:
Name: ns.inwx.de
Address: 2001:67c:1bc::104#53
Aliases:

Host stable.release.flatcar-linux.net not found: 3(NXDOMAIN)
$host stable.release.flatcar-linux.net ns2.inwx.de
Using domain server:
Name: ns2.inwx.de
Address: 2001:67c:10b8::104#53
Aliases:

Host stable.release.flatcar-linux.net not found: 3(NXDOMAIN)
$ host stable.release.flatcar-linux.net ns3.inwx.eu
Using domain server:
Name: ns3.inwx.eu
Address: 2a02:d500::53#53
Aliases:

Host stable.release.flatcar-linux.net not found: 3(NXDOMAIN)

Impact
Unable to download new releases

Yes, I've also noticed that and tried to fix it and while it looks good with dig @<IP_ADDRESS> bincache.flatcar-linux.net it didn't propagate yet.

(Looks to be a fallout from https://github.com/flatcar/Flatcar/issues/958)

At least for release.flatcar-linux.net things should be back but I just did a temporary workaround.

Others also should be back (with the same workaround)


Not found fontFamily Ionicons

When execute the app using yarn start and then selected iOS or Android this error appeared (Only when I rename the root folder "react-native-starter-kit" works fine with name react-native-starter-kit)
fontFamily 'Ionicons' is not a system font and has not been loaded through Expo.Font.loadAsync.

If you intended to use a system font, make sure you typed the name correctly and that it is supported by your device operating system.
If this is a custom font, be sure to load it with Expo.Font.loadAsync.
node_modules/expo/src/Font.js:33:10 in processFontFamily
... 31 more stack frames from framework internals

Help us reproduce - tell us about your environment

Mac
Android or iOS
Latest version from master. Download zip.

Steps to reproduce

run yarn install
run yarn start
press "i" or "a"

Expected result

show app icons

Actual result

Log

fontFamily 'Ionicons' is not a system font and has not been loaded through Expo.Font.loadAsync.

If you intended to use a system font, make sure you typed the name correctly and that it is supported by your device operating system.
If this is a custom font, be sure to load it with Expo.Font.loadAsync.
node_modules/expo/src/Font.js:33:10 in processFontFamily
... 31 more stack frames from framework internals

I had the exact same issue. only renaming back to react-native-starter-kit works which seems super strange.

You need to do this in your index.js... You might have to change it to a react class
componentWillMount() {     this.loadFonts();   }   async loadFonts() {     await Expo.Font.loadAsync({       Roboto: require("native-base/Fonts/Roboto.ttf"),       Roboto_medium: require("native-base/Fonts/Roboto_medium.ttf"),       Ionicons<EMAIL_ADDRESS>    });   }

I've added something similar into the repo on master branch - let me know if it doesn't solve the issue, and I'll re-open.
https://github.com/mcnamee/react-native-starter-kit/commit/7494bda1f53c56acfd86087ecac6f39f8bb540a9


Default to non-debug output level

Default to the non-debug output level in .resource_for_path , and add a .ruby-gemset file for developer convenience.

+1 from me, thanks!

+1, thanks!

+1, thanks!


Allow configuring the terminal visual bell duration

3 seconds right now which is quite long.

terminal.integrated.bellDuration?

or terminal.integrated.bellDelay to match editor.hover.delay and workbench.hover.delay?


npm uninstall doesn't work in WSL with open file in VS Code

Type: Bug
Summary
I cannot uninstall an npm package in my WSL environment while there is an open file in that project in VS Code.
Steps to Reproduce:

In the WSL environment (using the Microsoft terminal), create a new directory and cd into it.
mkdir wsl-npm-test
cd wsl-npm-test

Initialize a nodejs project.
npm init -y

Install an npm package and see that it does work.
npm install react

Uninstall an npm package and see that it does work.
npm uninstall react

Install the npm package again.
npm install react

Open VS Code from the terminal in the current directory.
code --disable-extensions .

Create a javascript file in VS Code and add some content in it.
// src/index.js
// this is a comment

While the file is still open and active in VS Code (double click on the file name in the tab if it is italicised), switch back to the Microsoft terminal app and try to uninstall the npm package.
npm uninstall react

Unfortunately, this results in an error for me. The error looks something like this:
npm ERR! code EACCES
npm ERR! syscall rename
npm ERR! path /mnt/c/Users/username/Documents/wsl-npm-test/node_modules/react
npm ERR! dest /mnt/c/Users/username/Documents/wsl-npm-test/node_modules/.react-aQBsuRBV
npm ERR! errno -13
npm ERR! Error: EACCES: permission denied, rename '/mnt/c/Users/username/Documents/wsl-npm-test/node_modules/react' -> '/mnt/c/Users/username/Documents/wsl-npm-test/node_modules/.react-aQBsuRBV'
npm ERR!  [Error: EACCES: permission denied, rename '/mnt/c/Users/username/Documents/wsl-npm-test/node_modules/react' -> '/mnt/c/Users/username/Documents/wsl-npm-test/node_modules/.react-aQBsuRBV'] {
npm ERR!   errno: -13,
npm ERR!   code: 'EACCES',
npm ERR!   syscall: 'rename',
npm ERR!   path: '/mnt/c/Users/username/Documents/wsl-npm-test/node_modules/react',
npm ERR!   dest: '/mnt/c/Users/username/Documents/wsl-npm-test/node_modules/.react-aQBsuRBV'
npm ERR! }
npm ERR!
npm ERR! The operation was rejected by your operating system.
npm ERR! It is likely you do not have the permissions to access this file as the current user
npm ERR!
npm ERR! If you believe this might be a permissions issue, please double-check the
npm ERR! permissions of the file and its containing directories, or try running
npm ERR! the command again as root/Administrator.

npm ERR!  This time it does work.
npm uninstall react

Open VS Code again from the terminal. It should remember that you were last editing the src/index.js file. Close the file so that there are no files active in VS Code. Then, close VS Code with the X.
code --disable-extensions .

Install the npm package again from the Microsoft terminal.
npm install react

Open VS Code again from the terminal. This time, there should be no open and active files in VS Code.
code --disable-extensions .

Uninstall the npm package from the terminal. This time it now works.
npm uninstall react

I believe this shows that there is a problem with uninstalling an npm package while having an open file in VS Code while using the WSL environment. Perhaps the files in the node_modules folder is also open in VS Code and the remote environment is causing some permission problems, but I am not entirely sure.
Other important versions

npm version: 8.19.2
nvm version: v18.12.1
node version: v18.12.1

VS Code version: Code 1.75.1 (441438abd1ac652551dbe4d408dfcec8a499b8bf, 2023-02-08T21:32:34.589Z)
OS version: Windows_NT x64 10.0.19045
Modes:
Sandboxed: No
Remote OS version: Linux x64 4.4.0-19041-Microsoft
Remote OS version: Linux x64 4.4.0-19041-Microsoft

System Info

Item
Value

CPUs
AMD Ryzen 7 5800X 8-Core Processor              (16 x 3800)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
31.92GB (17.94GB free)

Process Argv
--disable-extensions --folder-uri=vscode-remote://wsl+Ubuntu/mnt/c/Users/nathaniel/Documents/wsl-npm-test --remote=wsl+Ubuntu --crash-reporter-id 0299ac97-b781-41bf-93af-ad4a3e29647b

Screen Reader
no

VM
0%

Item
Value

Remote
WSL: Ubuntu

OS
Linux x64 4.4.0-19041-Microsoft

CPUs
AMD Ryzen 7 5800X 8-Core Processor              (16 x 3801)

Memory (System)
31.92GB (17.94GB free)

VM
0%

Item
Value

Remote
WSL: Ubuntu

OS
Linux x64 4.4.0-19041-Microsoft

CPUs
AMD Ryzen 7 5800X 8-Core Processor              (16 x 3801)

Memory (System)
31.92GB (17.94GB free)

VM
0%

Extensions disabled
A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdf:30486550
c4g48928:30535728
dsvsc012cf:30540253
azure-dev_surveyonecf:30548226
vscccc:30610679
pyindex848:30662994
nodejswelcome1cf:30587006
3biah626:30602489
pyind779cf:30662993
89544117:30613380
pythonsymbol12cf:30657549
vsccsb:30662443

See https://code.visualstudio.com/docs/remote/troubleshooting#_i-see-eacces-permission-denied-error-trying-to-rename-a-folder-in-the-open-workspace


Design feedback & sanitize / validate text records BAPP-262

https://linear.app/coinbase/issue/BAPP-262/customize-profile-add-validation-on-social-text-records-inputs
Validate textRecords

Sanitize textRecords

Tidy up modal:

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1


JavaScript Guess the Number Game

You have 100 Chances to guess a number between 0-100,
You will be given 100 points at start and with each wrong guess you will lose one point,
when u guess the correct number your final score will be displayed!
at each guess you will be notified if your guess was greater or lesser than the answer
good luck!

Can you add your project a directory?

done sir.

Thanks for the PR. It has been merged.


Short-circuit TableScan when constraint is false

@electrum i think you uncovered this recently in #589. Would you be able to add a regression test for this one?

i think you uncovered this recently in #589. Would you be able to add a regression test for this one?

It's going to be hard to do that, since the regression is only in the shape of the plan, but it only happens for connectors that implement the new applyFilter API. I don't think we have a way to test plan shapes in the context of a connector.

the very basic would be to fire up an EXPLAIN and see that it does not contain table scan and it contains values instead.
also, you should be able to use getQueryRunner().createPlan(getSession(), "SELECT ...", WarningCollector.NOOP) in eg TestPostgreSqlIntegrationSmokeTest


Trying to connect opensearch link but getting error - 502 Bad Gateway nginx.

Hi All,
I am trying to connect Opensearch 2.2.0/2.3.0 database with link - https://opensearch.opensearch-import-test.localhost.cloud  but getting error - 502 Bad Gateway. Logs shows following errors-
[opensearch-cluster-master-0] path: /bad-request, params: {}
java.lang.IllegalArgumentException: invalid version format: ?  8À,À0 ?Ì©Ì¨ÌªÀ+À/ ?À$À( KÀ#À' GÀ
at io.netty.handler.codec.http.HttpVersion.(HttpVersion.java:116) ~[netty-codec-http-4.1.79.Final.jar:4.1.79.Final]
at io.netty.handler.codec.http.HttpVersion.valueOf(HttpVersion.java:78) ~[netty-codec-http-4.1.79.Final.jar:4.1.79.Final]
[2023-01-06T09:49:01,998][DEBUG][r.suppressed             ] [opensearch-cluster-master-0] path: /bad-request, params: {}
java.lang.IllegalArgumentException: text is empty (possibly HTTP/0.9)
at io.netty.handler.codec.http.HttpVersion.valueOf(HttpVersion.java:65) ~[netty-codec-http-4.1.79.Final.jar:4.1.79.Final]
at io.netty.handler.codec.http.HttpRequestDecoder.createMessage(HttpRequestDecoder.java:124) ~[netty-codec-http-4.1.79.Final.jar:4.1.79.Final]
It is working fine when trying in version 1.3.2. Problem is only with opensearch 2.2.0/2.3.0.
Please help to resolve the issue.
Regards
Rajesh Kumar.

Hi Daniel,
Thank you very much for your support.
Please find the answers below -

Curl also getting the same error-
Tried below commands
curl https://opensearch.opensearch-import-test.xxx.xxxxxxxxx.cloud/
curl XGET https://opensearch.opensearch-import-test.xxx.xxxxxxxxx.cloud/

502 Bad Gateway

502 Bad Gateway
nginx

2)SSL is not enabled because we set
network.host: <IP_ADDRESS>
plugins.security.disabled: true
3)We deploy the instance through HELM charts and kubernates on cloud.
4)curl -XGET http://localhost:9200
{
"name" : "opensearch-cluster-master-0",
"cluster_name" : "opensearch-cluster",
"cluster_uuid" : "wgQfy7CySAWlzrLCNwQ1TA",
"version" : {
"distribution" : "opensearch",
"number" : "2.2.0",
"build_type" : "tar",
"build_hash" : "b1017fa3b9a1c781d4f34ecee411e0cdf930a515",
"build_date" : "2022-08-09T02:27:25.256769336Z",
"build_snapshot" : false,
"lucene_version" : "9.3.0",
"minimum_wire_compatibility_version" : "7.10.0",
"minimum_index_compatibility_version" : "7.0.0"
},
"tagline" : "The OpenSearch Project: https://opensearch.org/"
}
Login into pod and run above command.

Hi Daniel,
Any update on the issue. Please help me to resolve the issue because it is blocker for me.
Regards
Rajesh Kumar

Hi @Rajeshdaneti let me know if this is still a blocker.
This seems like an issue for the core, transfer.
Thanks.


Measured a few bulbs

PowerCalc is a really cool idea!  I installed it and it supported exactly 1 model of bulb in my environment!  So I decided to fix that…
Notes about these measurements:
Kasa KP115 (www.amazon.com/gp/product/B08LN3C7WK)
GU10 to E26/E27 adapters (www.amazon.com/gp/product/B0185I3Z1E)
Lamp: No dimming (consistent power), staged switch cycles through 1, 2, or 3 bulbs being on or off

Dummy Load - Philips 9.5W, measured at 9.096W:

MR16 to E26/E27 adapters (www.amazon.com/gp/product/B00SQGIDUY)
Note that this adapter does NOT convert to the GU5.3 required 12V (I learned this the hard way and now only have 4).  I compensated for this by soldering on an old 12V DC power supply I had in my closet:

One concern, I noticed the bulbs using HASS (ZHA) controller (2022.8.0) when brightness=1, the bulbs appeared off!  Further complicating this, I was getting a lot of 0W readings.  When I added the dummy load the 0W readings went away but they still appeared off.  I have long suspected that HASS has a bug relating to low absolute brightness values.  Is this normal?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2308

Relevant Lines:
2466

💛  - Coveralls

PowerCalc is a really cool idea! I installed it and it supported exactly 1 model of bulb in my environment! So I decided to fix that…

Thanks a lot for all your work!

staged switch cycles through 1, 2, or 3 bulbs being on or off

What do you mean with this?

Dummy Load - Philips 9.5W, measured at 9.096W:

I am not sure if LED lights could be used as dummy load, but looking at your measurements they look pretty solid. @nepozs @RubenKelevra Can you say something about this?
Maybe you could also do some verifications for different measure points without a dummy load and see if they are similar.
Did you use this dummy load for all the measurements or only some?

One concern, I noticed the bulbs using HASS (ZHA) controller (2022.8.0) when brightness=1, the bulbs appeared off! Further complicating this, I was getting a lot of 0W readings. When I added the dummy load the 0W readings went away but they still appeared off. I have long suspected that HASS has a bug relating to low absolute brightness values. Is this normal?

Not sure about this. I don't have zigbee based lights myself.

One concern, I noticed the bulbs using HASS (ZHA) controller (2022.8.0) when brightness=1, the bulbs appeared off!  Further complicating this, I was getting a lot of 0W readings.  When I added the dummy load the 0W readings went away but they still appeared off.  I have long suspected that HASS has a bug relating to low absolute brightness values.  Is this normal?

Not a (HA) bug. My lamps do this from time to time too, if I don't use the dimming function (transition=0ms) they work fine. If I let them transition down to below 4 points (which is 1 pct) they do turn off.
If I use transition and turn them up below 4 points they just do nothing.
I suspect they may actually use floating point percentage values and just have rounding issues.
So it's likely a similar issue with your bulbs.
Make sure the default transition is set to 0ms in HA (turned off) for the devices you're testing. Some devices do still make a tiny transition phase, some don't. 🤷♂

@bramstroker wrote:

Not sure about this. I don't have zigbee based lights myself.

You def should change that :)

@bramstroker wrote:

Not sure about this. I don't have zigbee based lights myself.

You def should change that :)

Haha correction. I have a lot of hue lights (Just use the bridge for that). Also 8 zwave dimmers. And some esp32 modules with esphome and wled). Actually all lights in my house are smart.

@bramstroker maybe we wanna send 0ms transition via HA commands by default to avoid such issues? 🤔

Has already been changed 14 days ago https://github.com/bramstroker/homeassistant-powercalc/pull/941.

staged switch cycles through 1, 2, or 3 bulbs being on or off

What do you mean with this?

Just trying to be overly thorough in my description of the tools used.  Pictured lamp has a switch with "4" different positions, off, 1 bulb, 2 bulbs, 3 bulbs...  Probably irrelevant, and extra data...

Dummy Load - Philips 9.5W, measured at 9.096W:

I am not sure if LED lights could be used as dummy load, but looking at your measurements they look pretty solid. @nepozs @RubenKelevra Can you say something about this?

Sure thing, will do.  Might take me a bit to get to it.

Make sure the default transition is set to 0ms in HA (turned off) for the devices you're testing. Some devices do still make a tiny transition phase, some don't. 🤷♂

@bramstroker maybe we wanna send 0ms transition via HA commands by default to avoid such issues? 🤔

Has already been changed 14 days ago #941.

So, I just verified, I do have this change in my local environment but these bulbs were still behaving this way.  Not sure if the docker container uses a different version of the scripts?  Anyways, it was really just the LTA008's that I noticed doing this. I suppose I could try to re-measure them if we're low confidence... The measurements looked reasonable when I added a dummy load.
Worst case I could move these to one of my Hue hubs, it's just a few extra steps I didn't want to take.

Note that this adapter does NOT convert to the GU5.3 required 12V (I learned this the hard way and now only have 4)

@bramstroker
Documentation really should contain some warning about possible electrocution, not everyone has enough knowledge about electricity to be safe…
In case above the losses were small, but completely unnecessary
picture from link above
https://m.media-amazon.com/images/I/610krua5CTL._AC_SX679_.jpg

Note that this adapter does NOT convert to the GU5.3 required 12V (I learned this the hard way and now only have 4)

@bramstroker Documentation really should contain some warning about possible electrocution, not everyone has enough knowledge about electricity to be safe…
In case above the losses were small, but completely unnecessary
picture from link above https://m.media-amazon.com/images/I/610krua5CTL.AC_SX679.jpg

Well, I have no one to blame but myself... RTFM!

So, I just verified, I do have this change in my local environment but these bulbs were still behaving this way. Not sure if the docker container uses a different version of the scripts? Anyways, it was really just the LTA008's that I noticed doing this. I suppose I could try to re-measure them if we're low confidence... The measurements looked reasonable when I added a dummy load.
Worst case I could move these to one of my Hue hubs, it's just a few extra steps I didn't want to take.

The change for the transition time was released in v0.24.7. You are using v0.24.8 of docker image so you should be fine.
I think it's not that important to have the power at brightness level 1 100% correct. As no one will have the light at 1% in practice. Would be nice if you can verify at some levels without the dummy load. For example; take as sample at 20, 60, 100, 150, 200, 255 and compare these values to the CSV with the dummy load. When they are similar (~2%) than I think we are fine.

Note that this adapter does NOT convert to the GU5.3 required 12V (I learned this the hard way and now only have 4)

@bramstroker Documentation really should contain some warning about possible electrocution, not everyone has enough knowledge about electricity to be safe…

What do you like to add exactly? The wiki here is free to edit. https://github.com/bramstroker/homeassistant-powercalc/wiki/Contributing-new-lights.

So, I just verified, I do have this change in my local environment but these bulbs were still behaving this way. Not sure if the docker container uses a different version of the scripts? Anyways, it was really just the LTA008's that I noticed doing this. I suppose I could try to re-measure them if we're low confidence... The measurements looked reasonable when I added a dummy load.
Worst case I could move these to one of my Hue hubs, it's just a few extra steps I didn't want to take.

The change for the transition time was released in v0.24.7. You are using v0.24.8 of docker image so you should be fine. I think it's not that important to have the power at brightness level 1 100% correct. As no one will have the light at 1% in practice. Would be nice if you can verify at some levels without the dummy load. For example; take as sample at 20, 60, 100, 150, 200, 255 and compare these values to the CSV with the dummy load. When they are similar (~2%) than I think we are fine.

Ran a quick sampling setting the bulbs to 153 mireds.  Looks to me like LTA010 is in the ballpark, but I think I'll need to re-measure LTA008.  Not sure if it was the two bulbs PLUS dummy load or what, but these measurements aren't consistent with what I'm seeing sans dummy.
If you agree I'll go ahead and update the PR to remove LTA008 and put re-measuring those on my to-do list.
LTA010:

Brightness:
21
61
101
151
201
255

Original (w/Dummy Load)
0.59
0.78
1.27
2.21
3.49
5.25

New (w/o Dummy Load)
0.6
0.82
1.23
2.18
3.4
5.13

Difference
-1.68%
-5.00%
3.20%
1.37%
2.61%
2.31%

LTA008:

Brightness:
21
61
101
151
201
255

Original (w/Dummy Load)
0.79
0.27
0.64
1.34
2.31
3.64

New (w/o Dummy   Load)
0.61
0.73
0.77
1.8
2.64
4.04

Difference
25.71%
-92.00%
-18.44%
-29.30%
-13.33%
-10.42%

The wiki here is free to edit

So I will prepare some warning.

As a dummy load you should use something with ONLY resistive characteristic, like incadescent bulb, but it can't be LED bulb (they have build-in impulse power supply, so it can interfere with measured bulb).

To be clear, do you want me to drop both these measurements from the PR and try them again?

If it is possible use resistive dummy load, I would like to say: don't delete results, but they all could be wrong…
I can see second problem with low voltage MR16  - they probably should be measured with dedicated power supply…
Accurate measurement is more complex then it looks like.

OK
The power supply on the photo just not looks like power supply dedicated to any lamp.
In reality power supply should be measured separately and low voltage bulbs/lamps separately (or results sustracted).
As example Ikea products (I don't have similar bulbs to yours, so I suspected that manufacturer sells dedicated power supply like Ikea does).

Well, clearly it isn't an off the shelf lamp, I made it. The power supply has a 0.6W load that is subtracted from all the measurements. I could tear the "real" one out of my ceiling and plug into the Kasa but I doubt that 12V DC power converter is much better behaved.

@bramstroker using aliases for the Wiz bulb stops working. Powercalc no longer creates entities for this bulb. No errors in the logs.

As a dummy load you should use something with ONLY resistive characteristic, like incadescent bulb, but it can't be LED bulb (they have build-in impulse power supply, so it can interfere with measured bulb).

Yes I already thought only fully resistive loads can be used as a dummy load. Makes sense to also add a note for that in the WiKi

@ejpenney I think PR is ready for merge now right? I don't see any issues anymore. The problematic bulbs (which used LED dummy load are removed from the PR).
Agreed or do you have anything else to have a look at?

@ejpenney I think PR is ready for merge now right? I don't see any issues anymore. The problematic bulbs (which used LED dummy load are removed from the PR).
Agreed or do you have anything else to have a look at?

I'm ready if you all are satisfied!


Colour CMYK properties

Colour objects should have read-only cyan, magenta, yellow, and key properties. And maybe c,y,m,k properties too.
This issue comes from: https://github.com/l2wilson94/Habitat/issues/19

Cancel this - I don't care about CMYK


Ephemeral buildkit container fails to consistently spawn with kraft run

Describe the bug
When using kraft run I often get an error similar to:  E  unpacking the image: opening layer: open /Users/fhuici/.local/share/kraftkit/runtime/oci/digests/sha256/09613022ba9985f35083918e11f1e2fd8352183bf5ee761d50c2b88817bc5483: no such file or directory
I notice that if I re-run the command (see output in the other box), sometimes it does work, saying " i  creating ephemeral buildkit container". I'm guessing the issue is that for some reason this buildkit container isn't always spawned, given this is a newly introduced feature, but I don't know if this is what's going on.
I also don't know if this is a mac-only issue.
Steps to reproduce
Go to the KC examples repo, cd into http-c and run:
kraft run  --plat qemu --arch x86_64 --log-level debug --log-type basic -p 443:8080 .
Expected behavior
Output such as:
[...]
Powered by Unikraft Telesto (0.16.2~687131d)
Listening on port 8080...
en1: Set IPv4 address <IP_ADDRESS> mask <IP_ADDRESS> gw <IP_ADDRESS>
Which architectures were you using or does this bug affect?
x86_64
Which operating system were you using or does this bug affect?
macOS
Relevant log output
➜  http-c git:(main) ✗ killall qemu-system-x86_64 ; sudo kraft run  --plat qemu --arch x86_64 --log-level debug --log-type basic -p 443:8080 .
No matching processes belonging to you were found
 W  using hardware emulation
 i  pulling unikraft.org/base:latest (x86_64/qemu)
 E  unpacking the image: opening layer: open /Users/fhuici/.local/share/kraftkit/runtime/oci/digests/sha256/e3f208309e226ee8f6bdeca7985d583ea26d1dbae46350ff9f02734cb90f6741: no such file or directory

➜  http-c git:(main) ✗ killall qemu-system-x86_64 ; sudo kraft run  --plat qemu --arch x86_64 --log-level debug --log-type basic -p 443:8080 .
No matching processes belonging to you were found
 W  using hardware emulation
 i  pulling unikraft.org/base:latest (x86_64/qemu)
 E  unpacking the image: opening layer: open /Users/fhuici/.local/share/kraftkit/runtime/oci/digests/sha256/09613022ba9985f35083918e11f1e2fd8352183bf5ee761d50c2b88817bc5483: no such file or directory

➜  http-c git:(main) ✗ killall qemu-system-x86_64 ; sudo kraft run  --plat qemu --arch x86_64 --log-level debug --log-type basic -p 443:8080 .
No matching processes belonging to you were found
 W  using hardware emulation
 i  creating ephemeral buildkit container
 i  #1 [internal] load build definition from Dockerfile
 i  #1 transferring dockerfile: 426B done
 i  #1 DONE 0.0s
 i
 i  #2 [internal] load metadata for docker.io/library/gcc:13.2.0-bookworm
 i  #2 DONE 1.8s
 i
 i  #3 [internal] load .dockerignore
 i  #3 transferring context: 2B done
 i  #3 DONE 0.0s
 i
 i  #4 [internal] load build context
 i  #4 transferring context: 1.80kB done
 i  #4 DONE 0.0s

I can't reproduce this anymore on linux/mac


wet floor sign allows suit storage

About the PR

if you wear a wet floor sign you can now put things on your suit storage slot
Why / Balance

without this the only way non sec can have access to suit storage is if they break into a sec outpost and hack the secdrobe, if there even is one, to take a regular armor vest, or take a hardsuit. this gives crew and antagonists a low risk way of obtaining suit storage without having much armor while remaining mobile
and it has armor values, so it should be considered armor in this way
weapons like rifles and spears have slings on their sprite when you wear it on suit storage, but suddenly doesn't work when you don't have an armor vest on. (perhaps this can be solved by adding a sling item and craftable makeshift sling that allows you to wear guns and spears on the suit storage slot regardless of outerclothing?
Media

Requirements

[x] I have read and am following the Pull Request and Changelog Guidelines.
[x] I have added media to this PR or it does not require an ingame showcase.

Changelog
:cl: JoeHammad

tweak: wet floor signs, when worn, now allows suit storage

Summary by CodeRabbit

New Features

Introduced the advanced mop with enhanced functionalities for water reagent management.
Added a golden plunger with illumination effects.
Defined a new Wet Floor Sign Mine Explosive with explosive properties.
Expanded the definitions of janitorial items, including damp rag and wet floor sign with updated functionalities.

These updates enhance gameplay by adding new items and improving existing ones.

funny fulldiff

webedit moment

yeah, trying to fix that

reopen when you've fixed this


Publishing .Net Core 2.2 app broken after upgrading to VS2019 16.3.1

After upgrading to VS2019 16.3.1, I can no longer publish my .Net Core 2.2 app.
I'm using 2.0.76
It throws the following error:
Restore operation started...
C:\Users\euan\.nuget\packages\microsoft.web.librarymanager.build\2.0.76\build\Microsoft.Web.LibraryManager.Build.targets(35,9): Error MSB4018: The "Microsoft.Web.LibraryManager.Build.RestoreTask" task failed unexpectedly.
System.MissingMethodException: No parameterless constructor defined for this object.
   at System.RuntimeTypeHandle.CreateInstance(RuntimeType type, Boolean publicOnly, Boolean noCheck, Boolean& canBeCached, RuntimeMethodHandleInternal& ctor, Boolean& bNeedSecurityCheck)
   at System.RuntimeType.CreateInstanceSlow(Boolean publicOnly, Boolean skipCheckThis, Boolean fillCache, StackCrawlMark& stackMark)
   at System.RuntimeType.CreateInstanceDefaultCtor(Boolean publicOnly, Boolean skipCheckThis, Boolean fillCache, StackCrawlMark& stackMark)
   at System.Activator.CreateInstance(Type type, Boolean nonPublic)
   at System.Activator.CreateInstance(Type type)
   at Microsoft.Web.LibraryManager.Build.Dependencies.<>c__DisplayClass10_0.<GetProvidersFromReflection>b__1(TypeInfo fac)
   at System.Linq.Enumerable.WhereSelectArrayIterator`2.MoveNext()
   at System.Collections.Generic.List`1.InsertRange(Int32 index, IEnumerable`1 collection)
   at Microsoft.Web.LibraryManager.Build.Dependencies.GetProvidersFromReflection()
   at Microsoft.Web.LibraryManager.Build.Dependencies.Initialize()
   at Microsoft.Web.LibraryManager.Build.Dependencies..ctor(IHostInteraction hostInteraction, IEnumerable`1 assemblyPaths)
   at Microsoft.Web.LibraryManager.Build.RestoreTask.Execute()
   at Microsoft.Build.BackEnd.TaskExecutionHost.Microsoft.Build.BackEnd.ITaskExecutionHost.Execute()
   at Microsoft.Build.BackEnd.TaskBuilder.<ExecuteInstantiatedTask>d__26.MoveNext()

This should be fixed with the 2.0.96 version.  Can you upgrade your package and give it a try?

Upgrading to 2.0.96 fixed the problem for me.

Great, thanks for confirming!


Delegate does not implement -readerSessionDidBecomeActive: method

Hi,
with version 1.3.0 when I scan a nfc tag I receive on Xcode console the following error:
-[NFCNDEFReaderSession _callbackDidBecomeActive]:220 Delegate does not implement -readerSessionDidBecomeActive: method
the issue seems to be in the empty body of this function:
public func readerSessionDidBecomeActive(_ session: NFCNDEFReaderSession) {
}
It is possible to solve it?

Hi @methods-inaz, Which Xcode version are you using?

You may reopen this issue if you still need help ♥


Cleanup

Cleaned up docstrings and some minor tweaks.

Good catch, I need to textwrap.dedent

Other than that seems mostly fine to me. Some well needed changes. Not sure why you reverted the pre tag from a multi-line comment? That seemed much nicer to me. If you were having issues with getting it to render you should try wrapping the string in textwrap.dedent().


Error message on webpage triggers "found free vaccinations dates"

The situation is the following: Impfbot searches for free appointments at a specific location with a Vermittlungscode.
The webpage throws an error "Es ist ein unerwarteter Fehler aufgetreten":

Impfbot, however, triggers a "found free vaccinations dates" event:

This "unerwarteter Fehler" never happened to me, so its not considered. Can be easily fixed by checking this warning box in addition. I event can't say what it means and what to do then. But for the bot and the whole process isn't  a big deal. You would get no email, and the bot would try to book this event an the next try again.

I ran into the same issue. Chrome console prints "Failed to load resource: the server responded with a status of 429 (Too Many Requests)". I am trying to work around the rate limit but no success yet.

This error happens when the backend of the Impftermin site is overloaded and cannot process any requests.
Usally what follows is the Virtual waiting-room. just try it again later.

same error and unfortunately every time a slack message with "found free vaccination dates..." but this is not true, it's a mistake :(

Had the same issue multiple times already this morning.
Took me a moment to figure out why I get the "found vacciation dates" slack message all the time.
Btw: thanks for this bot!!! Really usefull so far

To counteract possible limits, I made some timing settings configurable in the properties. In addition, the "Virtual Waiting Room" is now also recognized. And thanks to @ncface's fix, the error window is now also taken into account.

The HTTP request to buchung sometimes fails with 404 (can happen even after the timeouts), the "Terminsuche" overlay is never shown and the bot still believes that dates have been found. Only when you click "Termine suchen" again manually, you get another error message:
SyntaxError: Unexpected token < in JSON at position 1 I guess this can be recognized similary

The HTTP request to buchung sometimes fails with 404 (can happen even after the timeouts), the "Terminsuche" overlay is never shown and the bot still believes that dates have been found. Only when you click "Termine suchen" again manually, you get another error message:
SyntaxError: Unexpected token < in JSON at position 1 I guess this can be recognized similary

I have the same issues and tried to adjust the fix from @ncface with an 'or' statement on the unexpected token error.
But I think your right, the error only appears when you press the button manually..

I still get lot of false-positives when the page where the buttom "TERMINE SUCHEN" is loadable but when the click event happens the server is in "virtueller Warteraum". Does anyone have similar issues?
Since there is the bookingPage.isDisplayingVaccinationDates() check, I think there is something not working correctly? Any suggestions?


The ADLS Gen2 container doesn't auto refresh after checking the option 'Auto-refresh blob editor' on Setting Panel

Storage Explorer Version: 1.9.0
Build: 20190822.2
Platform/OS: Windows 10/ Linux Ubuntu 19.04
Branch: master
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Launch Storage Explorer.
Go to Settings Panel -> Make sure the option 'Auto-refresh blob editor' is checked.
Open one ADLS Gen2 blob container which contains one blob at least.
Try to delete one blob -> Check the prompt for refresh shows up or not.

Expect Experience:

The prompt for refresh doesn't show.
The blob container can be refreshed automatically.

Actual Experience:

The prompt for refresh shows up.
The blob container doesn't refresh automatically.

This issue has been fixed on the latest master build 20190827.11. So close it.


sub0_stim.nii.gz is not a gzip file

Hi, Clarke, I am so interested in the topic of functional MRS, I tried to study the code you shared in the "main_demo.ipynb", but come across some problems. When I load the demo data, It is reported that "sub0_stim.nii.gz is not a gzip file". I also can't open the demo data with FSLeyes. appreciate it if you can give me some tips. details are listed below:

from fsl_mrs.utils import mrs_io
data represents a NIfTI-MRS file in memory
data = mrs_io.read_FID('simulated_data/sub0_stim.nii.gz')
what's in the data?
print(data.shape)

ImageFileError                            Traceback (most recent call last)
Cell In[9], line 4
1 from fsl_mrs.utils import mrs_io
3 # data represents a NIfTI-MRS file in memory
----> 4 data = mrs_io.read_FID('simulated_data/sub0_stim.nii.gz')
6 # what's in the data?
7 print(data.shape)
File ~/.conda/envs/fsl_mrs/lib/python3.8/site-packages/fsl_mrs/utils/mrs_io/main.py:88, in read_FID(filename)
86     filename = Path(os.readlink(filename))
87 try:
---> 88     return nifti_mrs.NIFTI_MRS(filename)
89 except (nifti_mrs.NotNIFTI_MRS, fslpath.PathError):
90     data_type, id_ext = _check_datatype(Path(filename))
File ~/.conda/envs/fsl_mrs/lib/python3.8/site-packages/fsl_mrs/core/nifti_mrs.py:106, in NIFTI_MRS.init(self, *args, **kwargs)
103     args = list(args)
104     filename = Path(args[0]).name
--> 106 super().init(*args, **kwargs)
108 # Store original filename for reports etc
109 self._filename = filename
File ~/.conda/envs/fsl_mrs/lib/python3.8/site-packages/fsl/data/image.py:1238, in Image.init(self, image, name, header, xform, loadData, calcRange, threaded, dataSource, loadMeta, dataMgr, **kwargs)
1236 if isinstance(image, (str, Path)):
1237     image      = op.abspath(addExt(image))
-> 1238     nibImage   = nib.load(image, **kwargs)
1239     dataSource = image
1240     saved      = True
File ~/.conda/envs/fsl_mrs/lib/python3.8/site-packages/nibabel/loadsave.py:105, in load(filename, **kwargs)
103 matches, msg = _signature_matches_extension(filename, sniff)
104 if not matches:
--> 105     raise ImageFileError(msg)
107 raise ImageFileError(f'Cannot work out file type of "{filename}"')
ImageFileError: File /home/rshuang/Amy/test/fsl_mrs_fmrs_demo/simulated_data/sub0_stim.nii.gz is not a gzip file

Hi,
I think you might need to install git lfs before cloning the repo. See https://git-lfs.github.com/
Will
From: Amysays930 @.>
Date: Friday, 2 December 2022 at 12:30
To: wtclarke/fsl_mrs_fmrs_demo @.>
Cc: Subscribed @.***>
Subject: [wtclarke/fsl_mrs_fmrs_demo] sub0_stim.nii.gz is not a gzip file (Issue #1)
Hi, Clarke, I am so interested in the topic of functional MRS, I tried to study the code you shared in the "main_demo.ipynb", but come across some problems. When I load the demo data, It is reported that "sub0_stim.nii.gz is not a gzip file". I also can't open the demo data with FSLeyes. appreciate it if you can give me some tips. details are listed below:

from fsl_mrs.utils import mrs_io
data represents a NIfTI-MRS file in memory
data = mrs_io.read_FID('simulated_data/sub0_stim.nii.gz')
what's in the data?
print(data.shape)

ImageFileError Traceback (most recent call last)
Cell In[9], line 4
1 from fsl_mrs.utils import mrs_io
3 # data represents a NIfTI-MRS file in memory
----> 4 data = mrs_io.read_FID('simulated_data/sub0_stim.nii.gz')
6 # what's in the data?
7 print(data.shape)
File ~/.conda/envs/fsl_mrs/lib/python3.8/site-packages/fsl_mrs/utils/mrs_io/main.py:88, in read_FID(filename)
86 filename = Path(os.readlink(filename))
87 try:
---> 88 return nifti_mrs.NIFTI_MRS(filename)
89 except (nifti_mrs.NotNIFTI_MRS, fslpath.PathError):
90 data_type, id_ext = _check_datatype(Path(filename))
File ~/.conda/envs/fsl_mrs/lib/python3.8/site-packages/fsl_mrs/core/nifti_mrs.py:106, in NIFTI_MRS.init(self, *args, **kwargs)
103 args = list(args)
104 filename = Path(args[0]).name
--> 106 super().init(*args, **kwargs)
108 # Store original filename for reports etc
109 self._filename = filename
File ~/.conda/envs/fsl_mrs/lib/python3.8/site-packages/fsl/data/image.py:1238, in Image.init(self, image, name, header, xform, loadData, calcRange, threaded, dataSource, loadMeta, dataMgr, **kwargs)
1236 if isinstance(image, (str, Path)):
1237 image = op.abspath(addExt(image))
-> 1238 nibImage = nib.load(image, **kwargs)
1239 dataSource = image
1240 saved = True
File ~/.conda/envs/fsl_mrs/lib/python3.8/site-packages/nibabel/loadsave.py:105, in load(filename, **kwargs)
103 matches, msg = _signature_matches_extension(filename, sniff)
104 if not matches:
--> 105 raise ImageFileError(msg)
107 raise ImageFileError(f'Cannot work out file type of "{filename}"')
ImageFileError: File /home/rshuang/Amy/test/fsl_mrs_fmrs_demo/simulated_data/sub0_stim.nii.gz is not a gzip file
—
Reply to this email directly, view it on GitHubhttps://github.com/wtclarke/fsl_mrs_fmrs_demo/issues/1, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFJJTTSIDMLND6IVV5SFTODWLHTVBANCNFSM6AAAAAASR3V7OI.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Got it . Thanks, Clarke, it has been figured out. I can run the code.