Error when installing node-deflate

Having trouble installing node-deflate as a dependency of bogart:

npm ERR! error installing<EMAIL_ADDRESS>Error<EMAIL_ADDRESS>install: ./build.sh
npm ERR! error installing<EMAIL_ADDRESS>sh "-c" "./build.sh" failed with 1
npm ERR! error installing<EMAIL_ADDRESS>   at ChildProcess. (/usr/local/lib/node_modules/npm/lib/utils/exec.js:49:20)
npm ERR! error installing<EMAIL_ADDRESS>   at ChildProcess.emit (events.js:67:17)
npm ERR! error installing<EMAIL_ADDRESS>   at ChildProcess.onexit (child_process.js:192:12)
npm ERR<EMAIL_ADDRESS>install: ./build.sh
npm ERR! sh "-c" "./build.sh" failed with 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is most likely a problem with the deflate package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     ./build.sh
npm ERR! You can get their info via:
npm ERR!     npm owner ls deflate
npm ERR! There is likely additional logging output above.
npm ERR!
npm ERR! System Linux <IP_ADDRESS>-rscloud
npm ERR! command "node" "/usr/local/bin/npm" "install"
npm ERR! cwd /var/www/shaf/node_modules/bogart
npm ERR! node -v v0.4.12
npm ERR! npm -v 1.0.30
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     /var/www/shaf/node_modules/bogart/npm-debug.log
npm not ok

Any ideas?

Conclusion: the problem seems to be node-waf being different from the version of node installed.


updatePlaybackRate

I use animation.playbackRate = -1 to reverse an animation mid-way through.
This has recently stopped working in Chrome, but it does work if I change it to animation.updatePlaybackRate(-1)
However, that isn't polyfilled, so I have to write code like:
if (animation.updatePlaybackRate) {
  animation.updatePlaybackRate(playbackRate);
} else {
  animation.playbackRate = playbackRate;
}

I use animation.playbackRate = -1 to reverse an animation mid-way through.
This has recently stopped working in Chrome

Do you have a test case where this doesn't work in Chrome? I'm pretty sure it's a bug they'd like to know about it.
That said, updatePlaybackRate is preferred over setting the playbackRate since it will make sure the change happens smoothly for animations running on a separate thread/process.

👌I haven't raised an issue yet: https://jsfiddle.net/amk221/2n60su37/

I can reproduce the bug in Canary but not release. Do you need help filing the bug?

Sure, I’ve not raised one against Chrome before, ty
On Fri, 14 Feb 2020 at 01:43, Brian Birtles<EMAIL_ADDRESS>wrote:

I can reproduce the bug in Canary but not release. Do you need help filing
the bug?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/web-animations/web-animations-js/issues/218?email_source=notifications&email_token=AAC5I3IQDU32V3UFUV4MZBDRCXZLZA5CNFSM4KUVT5VKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELXILXA#issuecomment-586057180,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAC5I3MRRO36MSFJDIUNCB3RCXZLZANCNFSM4KUVT5VA
.

No problem, I've filed https://bugs.chromium.org/p/chromium/issues/detail?id=1052217 for this.
/cc @kevers-google @stephenmcgruer


[cmd/agent] Pass pointer to json.Unmarshal

What does this PR do?
Pass a pointer to json.Unmarshal instead of a value in makeRequest.
Motivation
Fix unit tests (go vet fails because of this).

Already fixed by #4058


Time_index enhancement (issue 172)

Updated time_index selection with three new steps (2, 5 and 6):
Selection strategy
We determine which attribute or metadata value to use as a time index for the entity being notified by selecting the first value found in the below list that can be converted to a datetime. Items are considered from top to bottom, so that if multiple values are present and they can all be converted to datetime, the topmost value is chosen.

Custom time index. The value of the Fiware-TimeIndex-Attribute. Note that for a notification to contain such header, the corresponding subscription has to be created with an httpCustom block as detailed in the Subscriptions and Custom Notifications sections of the NGSI spec.
Custom time index metadata. The most recent custom time index (the value of the Fiware-TimeIndex-Attribute) attribute value found in any of the attribute metadata sections in the notification.
TimeInstant attribute. As specified in the Fiware IoT agent documentation.
TimeInstant metadata. The most recent TimeInstant attribute value found in any of the attribute metadata sections in the notification. (Again, refer to the IoT agent documentation.)
timestamp attribute.
timestamp metadata. The most recent timestamp attribute value found in any of the attribute metadata sections in the notification. As specified in the Fiware data models documentation.
dateModified attribute.
dateModified metadata. The most recent dateModified attribute value found in any of the attribute metadata sections in the notification.
Current time. This is the default value we use if any of the above isn't present or none of the values found can actually be converted to a datetime.

@taliaga all probably we should measure the latency introduced by applying the strategy.
Feelings? (Not much for this addition, but in general)

Thank you @heikkilv, nice work!. I'll just drop some minor comments on the diffs. @c0c0n3 should too!.

probably we should measure the latency introduced by applying the strategy.

I don't believe this PR is adding "latency" to the strategy as it is now, but certainly we could measure how it changed from the days we only used dateModified. Unfortunately we don't have performance tests yet, but we could add some as a way to instrument a constant profiling.

Feelings? (Not much for this addition, but in general)

I think we could simplify a bit the way we explain this policy (given we always look at value first metadata then). But all in all it looks good enough, at least up to the use-cases I'm aware of.
When needed, to have a more "complete" coverage, we'd need some cross-attribute options to pick the time-index (like use the oldest/newest timestamp of any of the attributes, etc). So basically users would just need to pick at most 3 things:
1 - The name of the custom attribute they want to use (if any)
2 - To consider or ignore the content in "metadata" section.
3 - If he only cares of "latest" or "oldest" timestamp regardless of the attribute and whether it is value or metadata.
Combining these 3 it'll be possible to cover a wide range of cases I think.

in fact my comment was: "not for this addition, but in general". Point being that this way of selecting the attribute, in the "worst" case scenario, may introduce some delay, and it may better to introduce an option to define from the usecase perspective which "policy" order ql should use. just an idea... but if in all the cases the "latency" introduced is negligible, it will not be worth to add this feature.
As a todo, can't we think of a test suite that running a test for each "case" measure the time between data arrival and data population in the backend?


Background color not reaching the bottom of the page on large lists

The background color does not reach the bottom of the page when large lists are being rednered. The height of the background is fixed. Use "minHeight" instead.

Resolved in #13


Unable to sign in with 2FA (Steam deck / Arch)

When trying to sign in with 2FA on such as a phone authenticator or email a blank white box will appear (which I'm guessing in the authentication page)
I had to create a new account to get it working until this is fixed.
I'm on a Steam deck which uses Steam OS (which is based on arch Linux)

So far, i've never been able to login through the app itself, it's a bug with WebView2 and Wine that turns that window white. A workaround is to login into Roblox on the browser, click to join any game and leave, the account should be logged in into the app by this point and you can open the app normally.


Quitting the server results in an infinite blank screen loop.

When I left my singleplayer world, my screen went completely black and blank, apart from the RAM meter I have with another mod installed. While it was refreshing constantly and without any hiccups, the game itself froze in this state. Upon closing it threw me an error:
[00:28:45] [Server thread/INFO]: MobButcher lost connection: Disconnected
[00:28:45] [Server thread/INFO]: MobButcher left the game
[00:28:45] [CullThread/INFO]: [STDOUT]: Shutting down culling task!
[00:28:45] [Render thread/INFO]: Stopping worker threads
[00:28:45] [Render thread/INFO]: OpenGL debug message: id=1280, source=API, type=ERROR, severity=HIGH, message='GL_INVALID_ENUM error generated. Polygon modes for <face> are disabled in the current profile.'
[00:28:45] [Render thread/INFO]: OpenGL debug message: id=1280, source=API, type=ERROR, severity=HIGH, message='GL_INVALID_ENUM error generated. Polygon modes for <face> are disabled in the current profile.'
[00:28:45] [Render thread/FATAL]: Unreported exception thrown!
java.lang.NullPointerException: Cannot invoke "net.minecraft.class_1132.method_3725()" because the return value of "net.minecraft.class_310.method_1576()" is null
	at Not Enough Crashes deobfuscated stack trace.(1.17.1+build.31) ~[?:?]
	at com.minenash.seamless_loading_screen.FinishQuit.render(FinishQuit.java:87) ~[seamless-loading-screen-1.3.6+1.17.1.jar:?]
	at net.minecraft.client.render.GameRenderer.render(GameRenderer:874) ~[?:?]
	at net.minecraft.client.MinecraftClient.render(MinecraftClient:1112) ~[?:?]
	at net.minecraft.client.MinecraftClient.disconnect(MinecraftClient:2209) ~[?:?]
	at net.minecraft.client.MinecraftClient.disconnect(MinecraftClient:2185) ~[?:?]
	at net.minecraft.client.network.ClientPlayNetworkHandler.onDisconnected(ClientPlayNetworkHandler:712) ~[?:?]
	at com.minenash.seamless_loading_screen.FinishQuit.render(FinishQuit.java:99) ~[seamless-loading-screen-1.3.6+1.17.1.jar:?]```

Full log: https://paste.ee/p/eTptd.

Seamless Loading Screen 1.3.6;
Minecraft 1.17.1
Fabric Loader 0.11.6
Fabric API 0.37.0

Also, with this setup I have encountered #1 and #11, but it does not happen consistently.

Due to the age of this issue and the rewrite the mod took, I am closing this issue until such happens again. Please make a new issue if such dose


Update recycled PET process UUID.

User Story
Note: I didn't add a new test because I can't realistically add all possible combinations of input parameters.

r=@magopian


Do not insert WebTypeReplacements.typemap file into my project

Describe the bug
After an upgrade, I was not expecting to see a WebTypeReplacements.typemap file in my class library project.
To Reproduce

Make a new class library project
Run upgrade assistant on a class library project

Exceptions (if any)
None
Further technical details

Windows version:10 and bitness: 64-bit
Include the output of dotnet --list-sdks

3.1.410 [C:\Program Files\dotnet\sdk]
5.0.203 [C:\Program Files\dotnet\sdk]
5.0.204 [C:\Program Files\dotnet\sdk]
5.0.300-preview.21258.4 [C:\Program Files\dotnet\sdk]
5.0.300 [C:\Program Files\dotnet\sdk]
5.0.301 [C:\Program Files\dotnet\sdk]
6.0.100-preview.4.21255.9 [C:\Program Files\dotnet\sdk]

Include the output of upgrade-assistant --version

current version

Good catch, @KSchlobohm. On the one hand, I like having the typemap file come along with our analyzers so that it's easy to find and customize but you're definitely right it doesn't belong in the project root. I can think of two possible fixes here:

Move this file under Properties/My Project so that it's more subtle but still available.
Package this as an embedded resource in the analyzer project (users could still add their own typemaps). This has the advantage of not adding any extra files at all to the project but would mean that customers are unlikely to discover that they can customize type upgrades via config.

@dotnet/dotnet-upgrade-assistant-team, any opinions on what the best design here would be?

option1 seems the most flexible to me.  as an end user I would want to retain the ability to edit this map as well as add to the list.

Sounds good. PR incoming...


Level formula is described, but not defined

This is an issue with the documentation.  The wiki says what levelFormula is but fails to define it.  It's not clear where "stairs()" comes from, and it was a process to look up both what arguments it takes or and what it even does.  Additionally, while 'x' is probably the current level, that's also not defined.  I'm sure it makes sense to you, but as an option an end user would be expected to use or modify there's just not enough information here.
What should be included: the scope this runs.  Variables accessible from this scope, at minimum defining relevant variables like "x".  And, ideally, a description of the current formula and what it produces.  After digging around I found the stairs function in the DataAttributesAPI.  While this kind of helps, it's a custom function no one will be familiar with.  Since most users will likely just want to make simple edits, a short description of what should be modified to shift the results up or down wouldn't go amiss.
public static double stairs(final double x, final double stretch, final double steepness, final double xOffset, final double yOffset) {
    return steepness * stretch * (x - xOffset) - steepness * Math.sin(stretch * (x - xOffset)) + yOffset;
}

Of course, there's an issue here if that's true.  This function takes 5 arguments, and the default in the config provides 6.
stairs(x,0.2,2.4,17,10,25)

Hi there,
Are you sure that you've read the wiki? I feel that it does, in fact, cover all the points you've made. Please see here.
Thanks.

Wow, apparently it is.  I'm sure I poured over both the original wiki, this wiki and the data attributes but yup, that's precisely what I was looking for.  That said, what's up with the six arguments?  In the documentation it comes from the application of min, meaning it will choose the lower value of the formula versus the final argument, but in data attributes this doesn't seem to be the case?  Of course, I couldn't find the proper documentation here so maybe I also misidentified the function being called by stairs.


Better parsing emoji

Hello,
Some emoji are printed like that on IRC : <a:in_love:432887860270465028>
I think it should be : :in_love:

It looks like the a prefix is for animated emoji, if Emoji.toString() in discord.js is anything to go by.
I've done a pull request – #388 – which should fix this.


[text2vec-openai] 1.16.8 using wrong model for graphql get

When setting the model to ada and modelVersion to 002, calling /graphql Get seems to be using the old ada model. I am getting this error when making the /graphql call:

explorer: get class: vector search: object vector search at index document: shard document_HUM50X6K4c8M: vector search: knn search: distance between entrypoint and query node: vector lengths don't match: 1536 vs 1024

I can confirm this because on our OpenAI account all tests were charged using 'text-search-ada'.

Thanks for the report, will look into it.

I can confirm (and reproduce) this – looking into a solution now.

A fix is available in this preview image. Can you confirm that this is also fixed for you? Then we can release this as v1.16.9.

Looks like that fixed it!

Fixed in v1.16.9.


Just a question

Hi,
I've been using my own sensor for the last year. Command line sensor that polls the status. How is this different? One issue I do have is that the eskom status isn't stable. So even if it's stage 2 for the whole day, it will switch between 2 and 0 quite a few times. Not sure if you've managed to sort that out...

Hi @brafster,
The simple answer is that right now there is not much difference in terms of functionality between using a command line sensor and using this integration in terms of functionality, but there may be differences in stability.
That being said, I do plan to continue adding features to this integration, such as zone and schedule lookups based on the location configured in your HA instance.
With regards to stability, I have seen the same issues that you have mentioned. The Eskom API occasionally returns a value of 0 when it is queried, even though this isn't a valid state (since no loadshedding is represented as a 1). I have also seem issues where the server just drops client connections in the middle of a request.
To combat this, I have made this integration quite aggressive in how it retries failed requests, both in terms of networking errors, and in terms of sanity checks (like the 0 response you mentioned). Since I have been using this on my own system I have not seen the sensor fail to get a response (i.e. become unavailable), or return an invalid loadshedding status.
Hope that helps 👍

Thanks for the feedback. And thanks for this awesome piece of work! Will install it and give it a try and let you know.

@brafster no problem at all, hope it works well 👍
I'm going to close this issue, but feel free to open another or reopen it if you have any other questions/comments!


Using milliseconds in timer

Hi, is it possible to setup cron with milliseconds ?

Hi @bigHapiness
No node cron don't accept milliseconds.


How to install on HA?

Hello smokyflex,
I am a newbie to Home Assistant (HA), so the question: how to install this integration in HA?
An official integration or via HACS is not possible.
Thanks a lot for your help!
Greetings
Holger

Hi Holger, this project as it is, cannot be integrated into HA, it documents my first DIY efforts to integrate my Fröling boiler into HA via Modbus TCP and MQTT. I am new to HA myself, so I just recently realized that HA offers a builtin Modbus solution: https://www.home-assistant.io/integrations/modbus/, so the whole custom Modbus to MQTT script is not necessary. I have not yet updated this project accordingly. It is unlikely that I will write a working HACS integration for this because I have not the time at the moment. If you have any questions regarding Modbus hardware setup or my Modbus configuration.yaml feel free to contact me!

Hello smokyflex,
Thanks for the feedback!
Yes, I am interested in your Modbus configuration.yaml, because than I have a good starting point ;-)
Regarding Modbus hardware I will follow your suggestion with the "Waveshare Industrial RS232/RS485 to Ethernet Converter".
Question: have you connected the Fröling PE1 to LAN (router) with 1st cable for the "Fröling Connect App" and in addition the Ethernet Converter to LAN (router) with a 2nd cable as well? In total two LAN connections from router to the Fröling PE1?
Thank you!

Hi, I have updated the doc and added 2 yaml files of my working configuration where you should find all you need.
Regarding your question: Yes I have 2 ethernet connections from my PE1 to the router, one for the Fröling connect service and the second from the Waveshare converter.
Regards, Felix

Great, thank you very much for the information and support.
This will certainly not only give me a good start.
Greetings, Holger


Update index.html

@anohene1  thnx for contributing, try this year, and make sure you follow me and star this repo for updates about special swags I will be giving.
Contribute in October for Hacktoberfest 2020


SBOM without CPE is not matched to the advisory

Reproduce:
When DS1/DS3 dataset is ingested, ubi9-minimal should be listed as related sbom for CVE-2023-44487
http://localhost:3000/vulnerabilities/CVE-2023-44487
Analysis:
The purl status query is expecting sbom_package_cpe_ref entry. But also, it's filtered as out of context later on.

fixed by #1098

close?


WebSocketV1ClientBase.lastReceivedTime field data race

WARNING: DATA RACE
Write at 0x00c000158128 by goroutine 56:
  .../pkg/client/websocketclientbase.(*WebSocketV1ClientBase).readLoop()
     .../pkg/client/websocketclientbase/websocketv1clientbase.go:212 +0x23c

Previous write at 0x00c000158128 by goroutine 16:
 .../pkg/client/websocketclientbase.(*WebSocketV1ClientBase).startTicker()
     .../pkg/client/websocketclientbase/websocketv1clientbase.go:137 +0xbc

Just:
p.timeMutex.Lock()
p.lastReceivedTime = time.Now()
p.timeMutex.Unlock()

Same for WebSocketV2ClientBase

@filinvadim thanks for the feedback, usually this won’t happen, however it has possibility in theory as lastReceivedTime is written in two routines. I don’t want to introduce mutex, so just move the lastReceivedTime initialization to earlier places. You can check the latest master branch.

@eynzhang so data race is still in place. You need a mutex, sorry.

@filinvadim could you share how you reproduce this issue?

Sorry no, @eynzhang . Just go run --race ...
You have access to the variable lastReceivedTime from 2 threads - tickerLoop and readLoop. That's data race. Reopen please to warn other developers.


[BUG]publishActivitiesAsIndividualAccessories does not work on TV mode

First of all thank you so much for the hard work on this project.  My apologies if I misread and this is currently not supported in TV mode.
Describe the bug
publishActivitiesAsIndividualAccessories defaults as true; however, without it in my config, devices are not published as individual accessories.  if i hardcode it as true, the same behavior occurs.  If I hardcode it as false, behavior is same as if it is true
To Reproduce
{
"hubIP": "192.168.x.x1",
"TVPlatformMode": true,
"publishActivitiesAsIndividualAccessories": true
}
Expected behavior
I would expect that each activity (Watch Apple TV, Play Xbox One, etc) would be individual accessories and icons I could press on the Home menu, and add to command center shortcuts

Hi,
This option is only for switch mode currently, and makes no sense currently since activities are inputs in tv mode.
I plan to had an option for tv mode to publish devices as tv accessories but it is not yet available . See https://github.com/nicoduj/homebridge-harmony/issues/53 .

Thanks for the response and issue to the FR.
As to the why - I would enjoy being able to have a one click button available for friends/family, however I have found a work around by creating a scene for an input.
Thanks again for providing this plugin

As to the why - I would enjoy being able to have a one click button available for friends/family, however I have found a work around by creating a scene for an input.

Why don’t you also publish a switch in this case ?


Update Ubuntu install doc for 17.04

Test and update https://github.com/clearcontainers/runtime/blob/master/docs/ubuntu-installation-guide.md

This isn't release-gating, but see #495.


Delegate read_csv, read_json, and read_parquet to pandas IO if Spark throws an exception.

If the protocol in the specified path is not supported by Spark, Koalas fails even if pandas supports the protocol, e.g., http or https.
This PR proposes to add a config to enable to fallback to pandas with warnings.

Codecov Report

Merging #1358 into master will decrease coverage by 2.45%.
The diff coverage is 70.37%.

@@            Coverage Diff             @@
##           master    #1358      +/-   ##
==========================================
- Coverage   95.23%   92.78%   -2.46%
==========================================
  Files          34       34
  Lines        7576     7733     +157
==========================================
- Hits         7215     7175      -40
- Misses        361      558     +197

Impacted Files
Coverage Δ

databricks/koalas/config.py
98.98% <ø> (ø)

databricks/koalas/namespace.py
83.07% <70.37%> (-4.69%)
:arrow_down:

databricks/koalas/usage_logging/__init__.py
24.32% <0.00%> (-71.18%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50.00% <0.00%> (-50.00%)
:arrow_down:

databricks/koalas/__init__.py
78.72% <0.00%> (-14.90%)
:arrow_down:

databricks/conftest.py
88.67% <0.00%> (-7.55%)
:arrow_down:

databricks/koalas/frame.py
93.69% <0.00%> (-3.03%)
:arrow_down:

databricks/koalas/groupby.py
90.46% <0.00%> (-1.01%)
:arrow_down:

databricks/koalas/plot.py
93.33% <0.00%> (-0.96%)
:arrow_down:

... and 8 more

Continue to review full report at Codecov.

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

Let me close this as per the comment https://github.com/databricks/koalas/pull/1358#discussion_r411256145.


Type mapping not working

Detailed description
I tried to map a resource to a class using this example. However, this does not work. It seems like the package internals don't resolve the TypeMapper class correctly so there are actually no mappings defined when calling the hasMapping method on the TypeMapper.
 If so, can you post a full code example?

Closing this stale issue, feel free to reopen when the issue is still present!


Are you planning to add the function Standalone .exe creator soon?

Duplicate of #86.

I see that i can install the mods with emip files i just need to know how to do that with the command file
Can you explain to me how to do that  or link me a youtube video on how to do that process?
i never used command lines before


Add support for sk_shader_new_two_point_conical_gradient

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Column type not working for array

Howdy,
I have a number of columns named Activity[0]..Activity[9]. These cells within the rows are either 0 or 1 but the conversion treats them as strings so a row looks like this in the JSON record
"Activity":["1","1","1","0","0","0","0","0","0","0"].
I tried the number#Activity[0] to no avail and other numbers are handled OK.
Any thoughts as to what I need to do to get the integers in the JSON.
Cheers
Tony Benci

Thanks.
Patched in 0.5.1 please give a check.
~Keyang


Remove hail package pin

So we keep the fork-based package from the driver image instead of the upstream-based official package. Context: https://github.com/populationgenomics/production-pipelines/pull/172#issuecomment-1301575780

Doesn't work, I think I need to remove hail from reqs altogether.

@lgruen can you check again?

@leo still doesn't work: we have analysis-runner in deps, and analysis-runner depends on hail, which forces re-install. I'm a bit confused because I though pip would re-install only if --ignore-installed is set.
Any suggestions?

Doesn't work, I think I need to remove hail from reqs altogether.

That's so weird. Do you understand why? If I run pip3 install hail in the driver image, it doesn't overwrite anything.

Let me try building this directly in the driver image...

Okay, after moving both hail and analysis-runner into extras, it works!
(doesn't happen with gnomad package because it excludes hail from install_requires)

That's good to know, but I'm still trying to understand what's happening. pip is actually collecting hail 0.2.103, 0.2.104, and 0.2.105, even when removing the pin. I'm trying to get a better output from pip that would explain the dependency chains here, as something seems really odd. Must be analysis-runner, I guess, if moving it to extras works?

But also analysis-runner should already be included in the driver image...

pip is actually collecting hail 0.2.103, 0.2.104, and 0.2.105, even when removing the pin.

I actually saw this behaviour when updating analysis-runner locally: pip started pulling tons of different versions of analysis-runner for no reason. I wonder if there was some update in pip that caused this weird behaviour.
I'm off for a couple of hours, let me know how it goes and happy to pick up from you and continue digging into this issue.

After discussion in Slack, the solution was to remove requirements from analysis-runner that it doesn't use directly (we had hail's dependencies duplicated for reason we couldn't figure out)


Add routes from parent cluster to child cluster pod

specify different POD CIDR for every cluster
add routes which are to child clusters' pod on parent

Which issue(s) this PR fixes:

Fixes #508

cc @lmxia for another look

Looks good to me. Thanks for the patch.


the identity provider gets data from a local cm too

Summary
the config map is maintained by a controller and contains identities for the root API Exports.
the config map is used by non-root shards and provides higher fault tolerance than reading directly from the root shard.
if the config map doesn't exist or there were some errors during getting it we always fall back to the previous behavior of getting identities from exports directly.
Related issue(s)
xref: see item 1. add identities configmap in system:shard from  https://github.com/kcp-dev/kcp/issues/1225

/assing @stevekuznetsov @sttts

xref https://github.com/kcp-dev/kcp/issues/1225

Can we update the specific bullet point in the epic that this implements?

stevekuznetsov
Can we update the specific bullet point in the epic that this implements?

done, please see if you like it.

/lgtm
/approve


Stryker v4 Instrumentor breaks enzyme snapshots

Summary
When using Stryker v4, my snapshots (using enzyme) break because where in the main run it recognizes the component name, in the mutant run it just calls it Component.
Example:
const ErrorMessage = ({ message, id }: Props) => (
  <React.Fragment>
    {message.length > 0 && (

        {message}
        Error: {message}

    )}
  </React.Fragment>
);

export default ErrorMessage;

interface IFormLabelProps {
  label: string;
  forId: string;
  errorMessage?: string;
}

const FormLabel: FunctionComponent<IFormLabelProps> = ({
  label,
  forId,
  errorMessage
}) => {
  return (
    <>

        {label}

      {!!errorMessage?.length && (
        <ErrorMessage id={`${forId}-error`} message={errorMessage} />
      )}
    </>
  );
};

it("should render with error message", () => {
  const formLabel = shallow(
    <FormLabel
      label="test label"
      forId="testId"
      errorMessage="test error message"
    />
  );
  expect(formLabel).toMatchSnapshot();
});

In the main run it will return the snapshot
exports[`should render with error message 1`] = `
<Fragment>

    test label

  <ErrorMessage
    id="testId-error"
    message="test error message"
  />
</Fragment>
`;

While in the mutant it'll render
exports[`should render with error message 1`] = `
<Fragment>

    test label

  <Component
    id="testId-error"
    message="test error message"
  />
</Fragment>
`;

The difference being that ErrorMessage has been replaced with Component. This is because the instrumentor replaced the main definition of ErrorMessage with a ternary, and it appears that enzyme (or react, not sure) is looking for the variable name of the original variable that the component is assigned to.
If you replaced ErrorMessage with
const ErrorMessage = true ? ({ message, id }: Props) => (
  <React.Fragment>
    {message.length > 0 && (

        {message}
        Error: {message}

    )}
  </React.Fragment>
) : () => undefined;

Then the snapshot would also fail in the same way it does during mutation tests.
One possible fix is to temporarily assign the component to a variable with the original name inside the ternary. That way the component maintains its original variable name. Here's an example below:
let ErrorMessage;

const ErrorMessageInstrumented = true
  ? (ErrorMessage = ({ message, id }: Props) => (
      <React.Fragment>
        {message.length > 0 && (

            {message}
            Error: {message}

        )}
      </React.Fragment>
    ))
  : () => undefined;

export default ErrorMessageInstrumented;

Hi @Garethp thanks a lot for opening this issue. Are you still planning to add a e2e test with enzyme snapshots? It's optional of course, but it would help us a lot with solving this issue. It's also fine if you just add the files a new e2e/test directory (for example e2e/test/jest-with-enzyme-snapshots). If you need any help, don't hesitate to react here, or hit us up on slack

Yes I am, I just didn't find the time this weekend. I should have a PR in the next day or so

The problem is a little bit more broad even, as you can see here:
$ node -p "const Foo = class { }; Foo.name"
Foo
$ node -p "const Foo = true? class { }: ''; Foo.name"

I've been thinking about this and I think I know of a solution that works for most use cases.
I want to instrument these:
const foo = function () { }
const bar = () => {}
const Baz = class { }

Like this:
const foo = true? function foo () { }: ...;
const bar = true? (() => { const bar = () =>{}; return bar } {})() : ...;
const Baz = true? class Baz { } : ...;

What do you think of this?
I don't like your approach @Garethp, because it changes the functionality of the code too much and will be difficult to maintain.

That looks like it works quite  well. I'll test it on my work project when it's been pushed up

I've just released 4.0.0-beta.3 with this fix. Feel free to give it another go @Garethp 🌹

This is strange, but there are some cases where you might not want to be doing this? Take the following case:
export const SomeItem = () =>

;

For some reason in non-mutated code this gets exported without a name. In the mutated code it now does have a name? I didn't even notice this before.
Also, it looks like there's a bug where having a string that starts with https:// as a return seems to break the instrumentor and it spits out an unterminated string. I'll try to get that as a e2e test in the next couple of days


Callback system

Hi everyone!
Thank you for sharing this project.
I have a few ideas that I think might contribute to your library and before making a pr I thought of asking here first to check if you are interested.
The idea would be to take one more abstraction step and instead of having an Agent with an Actor and a Learner, an Agent would have an Actor and a list of Callbacks. The Learner would be itself a Callback, but also the Adder, the Logger, and even the Noise. The Callbacks would react at certain events similar to fastai callback system ('on_episode_begin', 'on_episode_end', 'on_feedback', 'before_select_action', 'after_select_action').
Callbacks may be disabled on demand. So after running the the env_loop, one could simply call agent._noise.disable() to remove noise.
The environment loop looks quite clearner this way too:
iterator = range(num_episodes) if num_episodes else itertools.count()

for _ in iterator:
    timestep = self._environment.reset()

    self._callbacks.call('on_episode_begin', timestep=timestep)

    # Run an episode.
    while not timestep.last():
        # Generate an action from the agent's policy and step the environment.
        action = self._agent.select_action(timestep.observation)
        timestep = self._environment.step(action)

        self._callbacks.call('on_feedback', action=action, next_timestep=timestep)

    self._callbacks.call('on_episode_end')

I forked your repo and started implementing it (it is actually almost done). It involves quite a few changes, so let me know if you would be willing to take a look at it.

Note: I'm not a contributor, just a random user.
I, personally, appreciate the lack of callbacks (callback hell). Makes code much easier to read.
But even if you wanted callbacks, wouldn't it just be another class similar to EnvironmentLoop?:
https://github.com/deepmind/acme/blob/f8867fe383c591d00f873323b308cd4e70f9f818/acme/environment_loop.py#L30

Hi! No, callbacks would be part of agents (much like actors and learners). The environment could also have its own callbacks like logs and so, but that's not the part I'm really interested in.

I'm also not a contributor, but I personally agree with @drozzy: callbacks can easily become sprawling code that reduces readability, even though they are supposed to be clean abstractions.
That being said, not all callbacks are bad - I'd love to see how you actually implement it @schouhy. Is there a public fork?

I'm still working on it but here it is https://github.com/schouhy/acme/tree/callback_refactor
Let me know your thoughts!

Hi schouhy, thanks for the interest! I haven't yet looked at your proposal in detail. Can you explain what problem it is you're trying to solve?

Hey schouhy, thanks for bring in new ideas, always exciting to see new perspectives!
I personally find callbacks to be tricky since they introduce informal API requirements that tend to make the codebase more difficult to understand. That said, the idea of introducing an Observer class, and a list of observers that is passed into the EnvironmentLoop does come up every now and then. The observers sit between the actor and environment, and receive data about the interactions between them (observers should not mutate any data). I believe this would be very similar to your 'on_feedback' callback. However I feel that these observers would almost always be used for logging (our loggers are basically observers), in which case calling them loggers and limiting their scope is more explicit.

Thank you guys for the prompt replies!
@aslanides I guess I'm not addressing any specific problem. My proposal is to keep going in the same direction you already took and further divide agents into building blocks that run independently. The benefits of it would be more flexibility to turn on and off certain functionality of the agent and the ability to add new ones even for already trained agents. This way the environment loop could also be easily used not only to train but also to evaluate the agent, just by disabling the learner, the noise, the adders, and by enabling bookkeeping of the steps and rewards. I imagine this could be a nice feature to have while running experiments.
@fastturtle I agree Observers would mainly be for logging, but I believe adders could also be observers and there could be observers that do the bookkeeping of rewards in an evaluation stage and provide nice plots, or also bookkeep metrics about the weights of the networks in a train stage. It may be a healthy refactor anyway.
Let me know your thoughts! And if callbacks are really off the table for you, then feel free to close the issue.

Hey guys, I created a discord for this project to be able to better contribute to the issues,share training results and more,  enjoy.
https://discord.gg/Rf6EU7

Coming back to this thread, the EnvironmentLoop now can be configured with an observer:
https://github.com/deepmind/acme/blob/master/acme/environment_loop.py#L66


make extension-down: Delete the validation webhook of the admission component

How to categorize this PR?

/area dev-productivity
/kind enhancement
What this PR does / why we need it:
Locally, I hit the following issue. I executed make extension-down while I still had a Shoot with enabled registry-cache extension. The admission helm chart was deleted by make extension down which under the hood runs skaffold delete. But the admission webhook was still present in the system (not part of the chart) and was blocking updates of the Shoot because there was no admission Pod running anymore.
After https://github.com/gardener/gardener-extension-registry-cache/pull/122 the validating webhook is no longer part of the admission chart but it is created on admission Pod startup. Hence, we have to delete it explicitly.
Which issue(s) this PR fixes:
N/A
Special notes for your reviewer:
N/A
Release note:

NONE

/hold
Maybe the same has to be applied for make remote-extension-down. Let me check.

/hold cancel


docs(github): add issue and pull request templates

Description
Include Issues templates and Pull request template for more concise description and pattern across all issues and PR.
Screenshots

Awesome work @marceloavf 👏


Multiple terminals with different materials

It seems like you cannot assign two different materials to two different terminals. I am trying to create a "typical" UI where the left part of the screen is the main map/game area and the right part (fixed width of 30 tiles) is the UI. What I want to do is assign a tileset of e.g. 8x16 to the UI and 16x16 for the regular UI.
When I do that, it seems like the first texture assigned is the one that is used.
Maybe this will work in the next release, since it looks like updating the font has been made easier?
Here is an example of how I am changing the font
fn load_ui_tileset_system(
	asset_server: Res<AssetServer>,
	mut materials: ResMut<Assets<TerminalMaterial>>,
	mut q_term: Query<&mut Handle<TerminalMaterial>, With<GameUiTerminal>>,
) {
	for mut material_handle in q_term.iter_mut() {
		let mut material = materials
			.get_mut(&*material_handle)
			.expect("Could not get material from terminal");

		let tileset: Handle<Image> = asset_server.get_handle("tilesets/VGA8x16.png");

		material.texture = Some(tileset);
		*material_handle = material_handle.clone();
	}
}

I was using the same material for all terminals, fixed for the next release. Thanks for the report!


A modified approach to static testing of node types

@tadhg-ohiggins:
An alternative to #147. This feel a bit more Pythonic, using ifs over implicit method selection. Only caveat here is that, if the content of the with short-circuits, we'll trigger the warning -- a little awkward.
In any event, please don't merge; let's discuss

I'm going to rewrite this and make another PR, but if there are any early comments, please post

Closing this -- see 18F/atf-eregs#157


🐛 map glob pattern to path

I had an issue to build plugin with swiftlint with specified inputpath not being [Path]
/.tuist/Cache/Plugins/7f778916b5141efe661ed06eb29384aa/Repository/ProjectDescriptionHelpers/TargetScriptExtensions.swift:44:25: error: cannot convert value of type '[FileListGlob]' to expected argument type '[Path]'
            inputPaths: inputPaths,

Solved by updating Tuist.


[DateTimePickerV2] inconsistency in placeholder text

What package is this for?

[x ] React
[ ] Angular

Describe the bug
In , we use placeholder text 'hh:mm XM' for 12 hours format and 'HH:mm' for 24 hours format
In , we use dateTimeMask as the format. There are 2 places indicate incosistency.

The meridian, XM vs A
When hours are in single h or H format, time picker is not able to show it in placeholder text.

To Reproduce
Steps to reproduce the behavior:

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

Please create a reduced test case in CodeSandbox:
https://codesandbox.io/s/github/carbon-design-system/carbon-addons-iot-react/tree/next/packages/react/examples/codesandbox

Expected behavior
A clear and concise description of what you expected to happen.
Environment/versions:

OS: [e.g. MacOS, Windows]
Browser: [e.g. chrome, safari]
carbon-addons-iot-react version: [e.g. v2.60.0]

Additional context
If applicable, add screenshots to help explain your problem.
Add any other context about the problem here.
Specific timeline issues / requests
Do you want this work within a specific time period? Is it related to an
upcoming release?
NB: The core contributors will try to work with your timeline, but it's not
guaranteed. The earlier you make a request in advance of a desired delivery
date, the better!

@robinrock Would you please confirm the placeholder text for DateTimePicker and TimePickerDropDown?
In the design of TimePickerDropDown, we have hh:mm XM for 12 hour format placeholder text. Should we do the same in DateTimePicker? Note that there might be format like h:mm am. We probably need to add a few more placeholder text for varies time formats.

Copied from Robin's comment from slack https://ibm-watson-iot.slack.com/archives/GNHE0BJ90/p1681393085516099?thread_ts=1681155517.241189&cid=GNHE0BJ90
"We don't currently have a different variation of placeholder text for that. Using placeholder text with only one "h" would be confusing to people needing to enter the two-digit hour 10, 11, or 12. Typically standard placeholder text for hours is "hh" - hope that helps!"
Robin recommended "hh" as the placeholder instead of 'h' considering 'h' is confusing for people. In DateTimePicker, we use dateTimeMask as the placeholder so that people can expect the time format.

Kevin will look into the design. We will recap during sync meeting.


New hotfix breaks community patch

Source: CommunityPatch
Method not found: 'Void TaleWorlds.CampaignSystem.ExplainedNumber.Add(Single, TaleWorlds.Localization.TextObject)'.
Inner exception callstack:
No inner exception was thrown
Outer exception callstack:
at CommunityPatch.Patches.Feats.BattanianForestAgilityPatch.Postfix(DefaultPartySpeedCalculatingModel __instance, MobileParty& mobileParty, Single& baseSpeed, StatExplainer& explanation, Single& __result)
at DMD?2973742::CalculateFinalSpeed_Patch3>(DefaultPartySpeedCalculatingModel this, MobileParty mobileParty, Single baseSpeed, StatExplainer explanation)
at TaleWorlds.CampaignSystem.MobileParty.ComputeSpeed()
at TaleWorlds.CampaignSystem.ViewModelCollection.Map.MapInfoVM.UpdatePlayerInfo()
at TaleWorlds.CampaignSystem.ViewModelCollection.MapVM.OnRefresh()
at TaleWorlds.CampaignSystem.ViewModelCollection.MapVM..ctor(INavigationHandler navigationHandler, IMapStateHandler mapStateHandler, MapBarShortcuts shortcuts, Action openArmyManagement)
at SandBox.GauntletUI.Map.GauntlerMapBarGlobalLayer.Initialize(MapScreen mapScreen)
at SandBox.GauntletUI.Map.GauntletMapBar.CreateLayout()
at SandBox.View.Map.MapScreen.AddMapView[T](Object[] parameters)
at SandBox.View.Map.MapScreen.OnInitialize()
at TaleWorlds.Engine.Screens.ScreenBase.HandleInitialize()
at TaleWorlds.Engine.Screens.ScreenManager.CleanAndPushScreen(ScreenBase screen)
at TaleWorlds.Core.GameStateManager.OnPushState(GameState gameState)
at TaleWorlds.Core.GameStateManager.OnCleanAndPushState(GameState gameState)
at TaleWorlds.Core.GameStateManager.DoGameStateJobs()
at TaleWorlds.Core.GameStateManager.CleanAndPushState(GameState gameState, Int32 level)
at SandBox.CampaignGameManager.OnLoadFinished()
at TaleWorlds.Core.GameStateManager.OnTick(Single dt)
at DMD?61844338::OnApplicationTick_Patch1>(Module this, Single dt)

same

SAME

Same

Same

Same

sAme

same

Same

Same here. Took some 2 hours to figure this out!! Kindly fix! The game is useless without this patch.

Same

Please fix... lovely community patch people.

Sadly, I have the same issue

Same

doesn't seem like the patch leadership is too active anymore. I will say that one of the devs here has a working fix waiting in the pull requests here: https://github.com/JoeFwd/MnB2-Bannerlord-CommunityPatch/tree/bump-fixes-after-new-1.4.1-hotfix
tested it out myself, and while i cant verify every perk is fixed, at least it runs!

@timchu90 can you post a build with this pull request here?

sure, i dont see why not.
https://drive.google.com/file/d/1DtngTA8ypDt7M7kOzNNTA3n8CqUlP2-J/view?usp=sharing

dumb questions... The gdrive link above has the updated CommunityPatch that is at least playable?

dumb questions... The gdrive link above has the updated CommunityPatch that is at least playable?

Yes. I havent encountered any crashes yet. But like I said, this is an interim build that hasn't been verified by other devs. I'm just too impatient to wait for an official release. No guarantee that it will work perfectly.

Sweet action! I'm not too worried about everything working as long as I can get my save to run. Thanks @timchu90

all credit goes to @JoeFwd


Add CLI as strava_py

This PR adds a CLI for creating visualisations, and puts it into an installable package, ready for distributions via https://pypi.org so it can be installed using pip. (I can help with how to do that later!)
First, I moved the strava_py directory into a src directory. This is commonly used for Python projects. The main benefits is to make sure when you're testing, you're testing against something that has been installed, and not something that happens to have the same directory name in your current dir. Much more:

https://packaging.python.org/en/latest/tutorials/packaging-projects/#a-simple-project
https://blog.ionelmc.ro/2014/05/25/python-packaging/
https://hynek.me/articles/testing-packaging/

The way to install from source:
pip install .

Or if you're developing, -e means an editable install, so you can make changes to your local source tree and they're reflected in what's run:
pip install -e .

Then I went for strava_py as the CLI name (this can be changed):
$ strava_py --help
usage: strava_py [-h] [-o OUTPUT_FILE] path

positional arguments:
  path                  Input path to folder with GPX and / or FIT files

options:
  -h, --help            show this help message and exit
  -o OUTPUT_FILE, --output_file OUTPUT_FILE
                        Output PNG file (default: plot.png)

Example run:
$ strava_py /tmp/my_strava_activities
Processing data...
Processing:<PHONE_NUMBER>.gpx
Processing:<PHONE_NUMBER>.gpx
/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/fitdecode/reader.py:909: UserWarning: 'field "native_field_num" (idx #0) not found in message "field_description"' (local_mesg_num: 0; chunk_offset: 142); adding dummy dev data...
  warnings.warn(msg)
/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/fitdecode/reader.py:909: UserWarning: 'field "native_field_num" (idx #0) not found in message "field_description"' (local_mesg_num: 0; chunk_offset: 162); adding dummy dev data...
  warnings.warn(msg)
/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/fitdecode/reader.py:909: UserWarning: 'field "native_field_num" (idx #0) not found in message "field_description"' (local_mesg_num: 0; chunk_offset: 203); adding dummy dev data...
  warnings.warn(msg)
/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/fitdecode/reader.py:909: UserWarning: 'field "native_field_num" (idx #0) not found in message "field_description"' (local_mesg_num: 0; chunk_offset: 249); adding dummy dev data...
  warnings.warn(msg)
/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/fitdecode/reader.py:909: UserWarning: 'field "native_field_num" (idx #0) not found in message "field_description"' (local_mesg_num: 0; chunk_offset: 285); adding dummy dev data...
  warnings.warn(msg)
/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/fitdecode/reader.py:909: UserWarning: 'field "native_field_num" (idx #0) not found in message "field_description"' (local_mesg_num: 0; chunk_offset: 326); adding dummy dev data...
  warnings.warn(msg)
/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/fitdecode/reader.py:909: UserWarning: 'field "native_field_num" (idx #0) not found in message "field_description"' (local_mesg_num: 0; chunk_offset: 351); adding dummy dev data...
  warnings.warn(msg)
/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/fitdecode/reader.py:909: UserWarning: 'field "native_field_num" (idx #0) not found in message "field_description"' (local_mesg_num: 0; chunk_offset: 397); adding dummy dev data...
  warnings.warn(msg)
/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/fitdecode/reader.py:909: UserWarning: 'field "native_field_num" (idx #0) not found in message "field_description"' (local_mesg_num: 0; chunk_offset: 443); adding dummy dev data...
  warnings.warn(msg)
/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/fitdecode/reader.py:909: UserWarning: 'field "native_field_num" (idx #0) not found in message "field_description"' (local_mesg_num: 0; chunk_offset: 487); adding dummy dev data...
  warnings.warn(msg)
/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/fitdecode/reader.py:909: UserWarning: 'field "native_field_num" (idx #0) not found in message "field_description"' (local_mesg_num: 0; chunk_offset: 531); adding dummy dev data...
  warnings.warn(msg)
/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/fitdecode/reader.py:909: UserWarning: 'field "native_field_num" (idx #0) not found in message "field_description"' (local_mesg_num: 0; chunk_offset: 575); adding dummy dev data...
  warnings.warn(msg)
/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/fitdecode/reader.py:909: UserWarning: 'field "native_field_num" (idx #0) not found in message "field_description"' (local_mesg_num: 0; chunk_offset: 619); adding dummy dev data...
  warnings.warn(msg)
/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/fitdecode/reader.py:909: UserWarning: 'field "native_field_num" (idx #0) not found in message "field_description"' (local_mesg_num: 0; chunk_offset: 663); adding dummy dev data...
  warnings.warn(msg)
/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/fitdecode/reader.py:909: UserWarning: 'field "native_field_num" (idx #0) not found in message "field_description"' (local_mesg_num: 0; chunk_offset: 707); adding dummy dev data...
  warnings.warn(msg)
Processing:<PHONE_NUMBER>.fit
Processing:<PHONE_NUMBER>.gpx
Processing:<PHONE_NUMBER>.gpx
Processing:<PHONE_NUMBER>.fit
Processing:<PHONE_NUMBER>.fit
Processing:<PHONE_NUMBER>.fit
Processing:<PHONE_NUMBER>.fit
Processing:<PHONE_NUMBER>.gpx
Processing:<PHONE_NUMBER>.gpx
Plotting facets...
Saved to plot.png

I'll also add a bunch of inline comments to explain what some things do, please feel free to ask more about any of this!

@hugovk Amazing, these are exactly the kinds of things I was going to work out later. Thanks so much!

You're very welcome, happy to help out!
I'll post tomorrow about how to put it on PyPI. Any thoughts for a name? strava_py?
(Side note: PyPI does name normalisation whereby dashes and underscores can be interchanged, and they show dashes by default, but both work. See for example https://github.com/pypa/setuptools_scm/ / https://pypi.org/project/setuptools-scm/ that as a project prefers underscores.)

Awesome, thanks again! Having given it a bit more thought, I think "stravavis" is a better name, as the package is focused on visualisation. There was some discussion about moving away from it being strava-centric, but I intend to create new visualizations from other data in the bulk export zip (not just gpx/fit activity files). So the package will be mostly aimed at Strava users, even if some of the visualisations still work with activity files exported from other sources. Let me know what you think.


pretty-print specifications lists

attach a class to the specs list and write a print function that compresses the specs into something more concise and quickly readable

done in

specs-class.R - function to attach S3 'specs' class, functions to print a specs object prettily
metab_model-class.R:setClass("metab_model") - metab_model class now stores specs as an object of class specs
metab_model-class.R:metab_model() - metab_model() initializer converts the passed-in specs list to a specs class if not already given that way
metab_model-class.R:show() - show function now asks the specs to print themselves


Add Mini Calender

@amanver45 elaborate your idea with us.

A "mini calendar" typically refers to a small, compact version of a calendar that displays a quick overview of dates, usually for a single month. It is commonly used in web applications, productivity tools, or desktop widgets to allow users to quickly view dates, select a day, or navigate between months. Here are some common functions of a mini calendar:
Date Display: Shows the current month with all days listed in a grid.
Day Highlighting: The current day is often highlighted, making it easier to see today’s date.
Navigation: Arrows or buttons allow users to move to the previous or next month.
Date Selection: Users can click on a date to select it, which is useful for picking dates in forms, scheduling apps, or booking systems.
Event Indicators: In some cases, days that have events or tasks may be marked with dots or other icons.
to create a mini calendar using HTML, CSS, and JavaScript:
HTML: Structure the calendar with a header for month/year navigation and a grid for days of the week and individual days.
CSS: Style the calendar for layout, colors, and hover effects to enhance the user interface and ensure it's visually appealing.
JavaScript: Add interactivity by handling month navigation, dynamically calculating the number of days in each month, and rendering the days accordingly.
This combination allows for a functional and interactive mini calendar.

@Tapabrata-Goswami @iamrahulmahato  pls assign me


Move templates and patterns to plugins

What
We provide some patterns and templates in the kit, such as:

task list
related links
mainstream guide

We'd like to consider moving them to plug-ins, installed by default
Why
If any of these change, they require a new release of the kit itself. This is especially a problem if it is a breaking change in the pattern or template as it makes the kit itself look like a major release.
It also puts us in a better position to support transition from our patterns to one in the design system (such as is happening to Task List)
Who needs to work on this

developers

Who needs to review this

developers

Done when

[x] We know how we want to tackle these (might be slightly different per pattern/template)
[ ] We've moved patterns and templates into plug-ins
[ ] Any plug-ins we want installed by default are installed by default
[ ] We've updated any necessary documentation

@nataliecarey please review and let us know how we want to proceed before the planning session next week. Is this something we want to include in v13?

I think we'd end up with 3 plug-ins (these arent finalised names):
govuk-prototype-kit-templates
The in-service templates like Question, Content, Check answers
govuk-publishing-templates
The govuk publishing templates, like Start page and Mainstream guide, and any necessary components like Related links
govuk-task-list
The task list template/component.
Possibly a 4th - the existing Step by step plugin
One issue with naming is, they will include both templates and components, so 'templates' alone might not be right?

We have 3 plugins:

Common templates
Task list
Step by step

We have to think about how we can manage the release of the kit where these are removed. If we do nothing prototypes will break as they rely on these plugins, so ideally we can auto install them at the same time.
We should also add these plugins to the starter package.json


[BUG] missing collapse-close classname

Describe the bug
Using collapse-close classname for collapse component (https://daisyui.com/components/collapse)

    Collapse title

    Collapse content

and run ESLint, it's not recognized
22:27  warning  Classname 'collapse-close' is not a Tailwind CSS class!  tailwindcss/no-custom-classname

Note: I don't know if this issue is related to daisyui or tailwindcss
To Reproduce
Steps to reproduce the behavior:

git clone https://github.com/rtritto/template-vite-solid-daisyui.git
cd template-vite-solid-daisyui
git checkout bug-collapse-eslint-tailwindcss
yarn
yarn eslint .

Expected behavior
tailwindcss/no-custom-classname rule doesn't apper on eslint output.
Environment (please complete the following information):

OS: Windows / Linux
Softwares + version used:

daisyui 4.12.10
tailwindcss 3.4.10
eslint 8.57.0
eslint-plugin-tailwindcss 3.17.4
@typescript-eslint/eslint-plugin 8.3.0
@typescript-eslint/parser 8.3.0
Node 20.16.0

eslint config file or live demo
.eslintrc.yml

👋 @rtritto! I did not find collapse-close in Tailwind docs: https://tailwindcss.com/docs/border-collapse#collapse
I guess that supporting Daisy UI is out of scope for this plugin. You can specify third-party class names via rule settings though:
https://github.com/francoismassart/eslint-plugin-tailwindcss/blob/master/docs/rules/no-custom-classname.md#whitelist-default-

Thanks, I fixed with:
.eslintrc.yml
rules:
  tailwindcss/no-custom-classname: [1, {
    whitelist: ['collapse-close']
  }]


Mirror article “How to choose a SIG as a non-code Kubernetes contributor”

This is a Blog issue

What would you like to be added
Copy article https://www.k8s.dev/blog/2021/07/09/how-to-choose-a-sig-as-a-non-code-kubernetes-contributor/ into the blog (folder content/en/blog)
Why is this needed
We mirror most articles from https://www.k8s.dev/blog/ to https://www.k8s.io/blog/
Comments
Set the canonicalUrl: field in the front matter to: https://www.k8s.dev/blog/2021/07/09/how-to-choose-a-sig-as-a-non-code-kubernetes-contributor/
/area blog
/language en
/good-first-issue
See https://github.com/kubernetes/contributor-site/pull/205#issuecomment-877324201
(I'm fine for someone else to do that copying)

Hello Tim! I would like to work on this. could be please tell in which file/folder inside the blog repository am I supposed to paste the copied content to? Apologies for asking such question as I am a beginner and not able to understand in which repository I should paste the content as there are multiple files and folders in the blog repository.

Hello @Shatakshi0805
Source file:

https://github.com/kubernetes/contributor-site/blob/master/content/en/blog/2021/non-code-contribution.md
(Raw: https://raw.githubusercontent.com/kubernetes/contributor-site/master/content/en/blog/2021/non-code-contribution.md )

Destination:
Since the original article has no images included,
website/content/en/blog/_posts/ seems proper directory to copy into.

Thanks @jihoon-seo.

/assign

@Shatakshi0805
@sftim 's suggestion is that
when you create 2021-07-09-how-to-choose-a-sig-as-a-non-code-kubernetes-contributor.md in website/content/en/blog/_posts/,
add canonicalUrl:  field.
e.g.
---
layout: blog
title: 'How to choose a SIG as a non-code Kubernetes contributor'
linkTitle: 'How to choose a SIG as a non-code Kubernetes contributor'
Author: Chris Short
date: 2021-07-09
slug: how-to-choose-a-sig-as-a-non-code-kubernetes-contributor
canonicalUrl: https://www.k8s.dev/blog/2021/07/09/how-to-choose-a-sig-as-a-non-code-kubernetes-contributor/
---

/close
We missed the boat :wink: on this one.


Scroll to bottom does not work properly when collectionView has more than one section

Scroll to bottom does not work properly when collectionView has more than one section because JSQMessagesViewController supposes that there is only one section.
ScrollToBottom method always scrolls to the last cell of section 0. It does not support multiple sections.

Of course, it is. If you looking at JSQMessagesViewController.m (or JSQMessagesViewController.swift), you'll realize that scrollToBottomAnimated and scrollToIndexPath methods only works well on 0th section.
Why do you need more section???

Hi, because I want to group my messages by a custom criteria.
For example, grouped by date:
[March 17] (section 0) (custom section header)
[message 1]
[message 2]
[message 3]
[Apr 2] (section 1) (custom section header)
[message 4]
[message 5]
.. and so on.

In that case, you just have to modified these method on your JSQMessagesViewController subclass
- (CGFloat) collectionView: (JSQMessagesCollectionView *) collectionView
                    layout: (JSQMessagesCollectionViewFlowLayout *) collectionViewLayout heightForCellTopLabelAtIndexPath: (NSIndexPath *) indexPath
{
    return 0.0f;
}

- (CGFloat) collectionView: (JSQMessagesCollectionView *) collectionView
                    layout: (JSQMessagesCollectionViewFlowLayout *) collectionViewLayout heightForMessageBubbleTopLabelAtIndexPath: (NSIndexPath *) indexPath
{
    return 0.0f;
}

- (CGFloat) collectionView: (JSQMessagesCollectionView *) collectionView
                    layout: (JSQMessagesCollectionViewFlowLayout *) collectionViewLayout heightForCellBottomLabelAtIndexPath: (NSIndexPath *) indexPath
{
    JPCMessage *message = [self.messages objectAtIndex: indexPath.item];
    JPCMessage *previousMessage = nil;
    JPCMessage *nextMessage = nil;

    // Hien thi label thoi gian o tin nhan dau tien cua user (trong 1 nhom message cua user)
    if (indexPath.item > 0)
    {
        previousMessage = [self.messages objectAtIndex: indexPath.item - 1];
    }

    // Hien thi label thoi gian o tin nhan cuoi cung cua user (trong 1 nhom message cua user)
    if ((indexPath.item + 1) < self.messages.count)
    {
        nextMessage = [self.messages objectAtIndex: indexPath.item + 1];
    }

    return [self configureCellLabelHeightAtIndexPath: indexPath
                                  withCurrentMessage: message
                                     previousMessage: previousMessage
                                         nextMessage: nextMessage];
}

#pragma mark - Incoming, Outcoming CellBottomLabel Configuration Method

- (CGFloat) configureCellLabelHeightAtIndexPath: (NSIndexPath *) indexPath
                             withCurrentMessage: (JPCMessage *) current
                                previousMessage: (JPCMessage *) previous
                                    nextMessage: (JPCMessage *) next
{
    // Hien thi chieu cao cua Cell Bottom Label cho message dau tien
    //    BOOL isFirstMessage = [self isFirstMessage: current andPrevious: previous];
    // Hien thi chieu cao cua Cell Bottom Label cho message cuoi cung
    BOOL isLastMessage = [self isLastMessage: current andNext: next];

    // Neu la tin nhan cuoi cung cua user thi cho hien thi CellBottomLabel
    // Default la: kJSQMessagesCollectionViewCellLabelHeightDefault
    // force: 13.2f
    // (trong 1 nhom message cua user)
    if (isLastMessage)
    {
        return 13.2f;
    }

    // Nguoc lai chieu cao cua CellBottomLabel la 0.0f
    return 0.0f;
}

Just modify the height, text, background color, ... of your needed label (top, bottom, or message bubble top label) for 'separating' your messages group.
If you still want to do as your choice, you must override these methods

finishReceivingMessage / finishReceivingMessageAnimated
scrollToBottomAnimated
scrollToIndexPath
in your JSQMessagesViewController subclass. It is a bit of trouble to do that. Be careful.

Thanks for your suggestion.
I find quite simple to override scrollToBottomAnimated and scrollToIndexPath to support multiple sections (finishReceivingMessage/finishReceivingMessageAnimated are not needed) because I only need to change 2 or 3 lines in the JSQMessagesViewController class. The other good reason to override that methods is that I don't need to implement indexpath calculations that mess my code.
I've submitted a PR with the necessary changes to support it.

man, changing directly the JSQMessagesViewController class is final option. I don't really recommend that work. It's cruel, you should find other ways to solve your problem.

Thanks @ngoclamnguyen !


Fix "method_complexity" issue in kyu_2/evaluate_mathematical_expression/evaluate.py

Function check_conditions has a Cognitive Complexity of 20 (exceeds 5 allowed). Consider refactoring.
https://codeclimate.com/github/iKostanOrg/codewars/kyu_2/evaluate_mathematical_expression/evaluate.py#issue_6721b1256578ac0001000401

Function check_conditions has a Cognitive Complexity of 20 (exceeds 5 allowed). Consider refactoring. New
def check_conditions(strings: list, string: str, temp: str) -> tuple[str, str]:
"""
Normalizing string input by checking conditions
:param strings: list
:param string: str
Severity: MinorFound in kyu_2/evaluate_mathematical_expression/evaluate.py - About 2 hrs to fix

Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
Code is considered more complex for each "break in the linear flow of the code"
Code is considered more complex when "flow breaking structures are nested"
Further reading
Cognitive Complexity docs
Cognitive Complexity: A new way of measuring understandability


Connecting to MySQL in VSCode

No description! Close.
Maybe referenced : https://github.com/Bajdzis/vscode-database/issues/104


Linux support?

Is Linux support possible with these trackers?

@ItsRogueRen it's planned!

if you were to add linux support it would be nice to make the drive compatible with 1.14 because if you ever used steamvr on linux....... Yeah also thank you for the hard work i hope the project goes well

I also would love to see Linux support!

I would also like it to be available on Linux.

Any updates on this.
I might try it myself but I have zero knowledge of C++ other than this book I have, that I haven't read in awhile.

Hey, I don't mean to rush y'all or anything, just wondering if there's any sort of timetable/estimate on this?
I'm currently debating whether to pre-order the coming wave of SlimeVR hardware, and I want to go all Linux since the most recent VRChat update that added EAC also makes it so the game now blocks my Windows virtual machine, and also makes it so that in-game video players now work on Linux (previously the only aspect of VRChat that wasn't working on Linux).

The plan is to add linux support before official slimes ship. There isn't much hurdle to do this, just no one did it yet.

Getting this to happen would be simple, but none of our current contributors have time for it due to other higher priority tasks. If you are interested in Linux support, we would welcome a PR! It should be very easy.
Simply add a unix socket version of bridge-windows-pipes.cpp.

Is this resolved now? There's a socket transport and a release made now.

Thanks, added to my VR on Linux doc on Gitlab and Steam.


Consider ts-engine.yml

Quite a lot of switches are being added now (see #50 and #67)
Might be worth thinking about a ts-engine.yml
type: node-app | library
react: true | false

not for now, can't see much api change for a while


Todiff handles urls improperly

Tags whose value contain a colon confuse todiff. Granted, it's ruled out by the todo.txt spec but the url: tag it quite useful to me.
Steps to reproduce

topydo add test url:http://example.org
git difftool ...

Output
New tasks:
 → 2018-03-26 test//example.org url:http:

No changed tasks.

Ah, it's probably todo-txt's fault ><

For the record, it's referenced upstream in https://github.com/sanpii/todo-txt/issues/5.


Bsweger/move code from infrastructure repo

Resolves Infectious-Disease-Modeling-Hubs/hubverse-cloud#68
This somewhat sizable PR ports some MVP test code to a stand-alone repo. There's some additional context below, and I'll annotate inline.
For this initial review, it would useful to have someone:

Follow the instructions for installing this module
Try the dev setup instructions and make sure you can run the test suite

I'm less concerned about the overall design of the class (frankly, there's a lot I don't love about it). However, it's time to get this code into its own repo so we can iterate on it properly.

Additional info
This PR creates the first iteration of a Python function that:

reads a single model-output file from S3 (for now, in parquet or .csv format)
adds three columns: round_id, team, and model
writes the updated information back to S3 in parquet format

This code was part of the "proof of concept" for using S3 event notifications as a way to trigger a Lambda function that processes model-output files being sent to S3. It's written in Python because AWS Lambda doesn't support an R runtime out of the box.
The diagram below shows how the "Lambda function triggered by a new S3 object" builds on top of the S3 sync already in place:

My one bigger question at this point is, can we guarantee that the function is reading and encoding NAs correctly? I know this can be an issue when using python to read in csv with values encoded as NAs in R so might be a good idea to set up a test that includes a model output csv file with e.g. a mean output type id and check that when the file is accessed again in the parquet format, the output_type_id value is still NA.

Heya @annakrystalli, that's a good call, and I just pushed a commit to make the test .csv/.parquet files represent a wider-array of potential model-ouput data (e.g., mean output_type with an empty output_type_id)
I pushed a commit to widen the universe of things we're unit testing, but your point about R's NA is better suited to an integration test., which I'm working on and can push later tonight. Preliminary testing looks promising (pyarrow seems smart enough to read in the NA and render it in Python as null)
As for writing the data back out to parquet...is NA R-specific? Asking because if we're standardizing on parquet as our model-output standard, I think we should use whatever the parquet standard is for representing null/na

My understanding is that Parquet files have their own way of encoding missing values that is language agnostic. If all has worked well in the transformation, when connecting to the hub and collecting in R, the original NA values should still be interpreted as NA. So indeed sounds more like an integration test and good to hear results so far look promising!

@annakrystalli this commit adds integration tests
 (it doesn't integrate directly with hubData, though that would be a good future improvement, if you want to add an issue)
Writing this revealed that we could indeed be doing more to normalize missing data when reading .csvs (you'll see that change in the commit linked above). Really appreciate the question!

Glad it was useful!
I will go ahead and approve the PR as my particular concern has been addressed but noting again that I haven't addressed the specific items you asked for feedback on so probably still a good idea to get @lshandross's input too?
In the meantime I'll open an issue re: integration test with hubData.

Noting that @lshandross found some setup issues that we should address (or document). However, we agreed to come back to those once they've had a resolution with IT about some specific problems.
Thanks for taking on the challenge of working through the README @lshandross. The goal is to minimize setup headaches, and ideally get to a place where someone without admin access to their machine can get the project set up.


Whats new for August

Create the what's new page for August.

closing and opening for a fresh build


[Java] JDBC Flight SQL driver should follow Flight RPC spec

Describe the bug, including details regarding any error messages, version, and platform.
The JDBC Flight SQL driver assumes that the endpoints are all the same as the main server connection, but this is wrong. It should connect to individual endpoints specified by GetFlightInfo.
#13900 tried to fix it but the original contributor isn't interested - this would be a starting point for future contributors
Component(s)
FlightRPC, Java

take


In a grid layout, gap configured in breakpoints won't work

Bug report
Describe the bug
When writing a stitches object, the grid column/row object won't apply on the element.
To Reproduce
The object looks like this:
const TablesGrid = styled('div', {
  display: 'grid',

  variants: {
    gap: {
      horizontal: {
        gap: 4,
        gridTemplateColumns: '1fr 1fr',
      },
      vertical: {
        gap: 16,
        gridTemplateColumns: '1fr',
      },
    },
  },
});

Expected behavior
Expected that the gap will be applied based on different breakpoint.
Screenshots

System information

OS: macOS
Browser (if applies) Chrome
Version of Stitches: 1.2.8
Version of Node.js: v16.18.1

This is the setup of the breakpoints.
And it did work if I setup like this:


Terminology: dump and load are unclear

It's very unclear what dump and load mean. I gather dump fetches the current data from upstream and writes it to local files, and load reads data from local files and figures out the commands to push that to upstream.
Instead of dump, how about pull, fetch or download.
Instead of load, how about push, publish or upload.
(I noticed load generates aws commands but doesn't run them. That might just mean the name should be generate-push-commands.)

I do agree this could be a lot clearer.
dump-to-yaml and generate-sync-cmds were the subcommands that were being used up to a few weeks ago, I'd still be for using those sub commands as they're pretty unambiguous
Thoughts @lox ?

I like pull and push personally. I'd have push actually execute the commands unless you had --dry-run.

:+1:

I'm hesitant to allow iamy push to simply run and overwrite whatever you have on AWS. Seems too dangerous, especially because it's very easy for your local yaml to be out of sync with AWS.
What about if iamy sync-cmds and iamy sync-cmds --exec ?

Ideal would be a diff and confirmation; iamy push fetches upstream state into memory, calculates the differences, presents it as a concise diff, prompts to apply the diff.
I'm against calling a one-way process “sync”.

Yep ok - I'll make push interactive and maybe have push -f or push -y for non-interactive

I'm against calling a one-way process “sync”

Why? sync operations are normally one way e.g. rsync. You're syncing the destination with your source by applying a diff

I guess rsync is one way at a time, although it can sync in either direction. iamy sync iam://something localdir/ vs iamy sync localdir/ iam://something would be equivalent. But when the direction of the operation is fixed, a word describing that direction (push vs pull) is much clearer.

Fixed in #9


Create group for leavers which can deny API access

Rather than deleting user accounts as they leave we apply a "soft delete" and
move them into an IAM Group which denies any AWS API calls.
This helps mitigate new joiners with the same username inheriting some
permissions from cross-account assume role trust policies. AWS mitigates this by
transforming usernames into principal IDs[0] but this is undone when an assume
role policy is modified.
[0] See the "Important" section of "Creating a Role (AWS CLI)"
http://docs.aws.amazon.com/IAM/latest/UserGuide/id_roles_create_for-user.html

We looked further into the behaviour of AWS when a user is removed from the user-management account. Once the user is removed in the user-management account the the trust policy in the trusting-account is altered and the JSON no longer references the user in the ARN format, it comes back referencing the user using the unique user identifier. Trust policies can also only be applied if the ARNs resolve to valid resources. So there is little risk of a policy being re-written to reference another user with the same name.
For these reasons we've decided to delete users from the user-management account when they leave the organisation. The changes proposed here are no longer useful, so it will be closed.


Use object as message on docblock for PSR functions

Driven by this: https://github.com/Seldaek/monolog/pull/1260
I looked at the PSR-3 to see if custom levels are allowed.. and yes they are... but on the next lines I saw this:

Every method accepts a string as the message, or an object with a __toString() method. Implementors MAY have special handling for the passed objects. If that is not the case, implementors MUST cast it to a string

An alternative could be to get rid of the docblocks on these methods and depend on the docblocks of PSR.

Opened one PR here too
https://github.com/php-fig/log/pull/60

Closing as per my comments on https://github.com/php-fig/log/pull/60


Create property for configuring how often it should sync

Currently it only syncs upon plugin loading but should have a sensible default and be configurable to the users liking (probably not more than every 15 minutes at most?

Done.

defaults to 30 and is in minute granularity


Multimedia Icon

I created this icon... What do you think? Unsure how it would work with your colors... I think it should be a simple re-color.
multimedia.zip

Ah, I do remember this, though I didn't get what the top part was in the version you sent me since it didn't have the rectangle. Also I believe the name was something like mm, which I didn't realize meant multimedia. I think this is not bad, but it makes me think of music or music videos. You had one on the forum like this:

Did you discard it? I think it probably works better for the category.


@electron/remote removed in next Theia version

Bug Description:
@electron/remote will be removed with the next theia version: https://github.com/eclipse-theia/theia/issues/2018
We have to adapt our code accordingly.
Steps to Reproduce:

Update to next version
Try to build

I will prepare this on the next branch and integrate it after the Theia Release

It's not easy, I try to upgrade1.37.0 but failed, hope you can make it right.

I will try to update the main branch on Tuesday.
In the meantime, you may have a look at this commit: https://github.com/eclipse-theia/theia-blueprint/commit/edfdfcc3eaa014aed7c77067e883b3c18d7a67d2

Thank you, very kind of you, I will try this commit soon.


Module Prevents DD Import from working

In version 7.3, this module prevents the user from uploading using the DungeonDraft Import module. https://foundryvtt.com/packages/dd-import/.
User receives this error in the console log.

I notified Moo Man in this issue as well:
https://github.com/moo-man/FVTT-DD-Import/issues/42
My current workaround is just disabling DF Scene Enhancement until I am done with map uploads and re-enabling during the game.

I haven't had a chance to test my modules on 0.7.X yet, so thank you for letting me know. I do plan on getting them updated as soon as I have some free time. I'm happy you have a work-around for the time being.

Side note. It is a much welcomed module for me and my players. Thanks for keeping the game looking clean.
Thank you,
Justin Payton
<EMAIL_ADDRESS>
On Oct 9, 2020, at 2:50 PM, Andrew Fryer<EMAIL_ADDRESS>wrote:
I haven't had a chance to test my modules on 0.7.X yet, so thank you for letting me know. I do plan on getting them updated as soon as I have some free time. I'm happy you have a work-around for the time being.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/flamewave000/dragonflagon-fvtt/issues/3#issuecomment-706347175, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJCC2UPP5H42LW7AVNIJDZLSJ5LO7ANCNFSM4SKNGVRA.

@justinthejaguar Could you retry this issue using v2.0? I have corrected the thumbnail creation code for 0.7.4+.

Tested. Fixed.


Add links to section 1

Reference terminology section at first appearance of document-specific phrases: "Entity Metadata Provider", "Entity Metadata Consumer"

Ugh, just realized the reference should probably be to #roles rather than #terminology.


Manhwa-latino dont pass verification but webview works

Source information
Manhwa-Latino 1.4.42
Source language
Spanish
Steps to reproduce
When you open the extension the content does not appear but the web view works
Expected behavior
be able to search content
Actual behavior
nothing appears
Mihon/Tachiyomi version
Beta r6604
Android version
Android 11
Other details
I hope it can be solved because it is a fairly well-known and used extension. Greetings
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.15.3.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[ ] I have admitted that I am a clown by having checked this box, as I have not read these acknowledgements.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

They also put CF in other parts of the page, try opening it in a WebView and browsing around the page (by tapping the search icon, opening a manga/chapter, etc). If that doesn't work, wait for their protection to decrease or migrate to other sources


Device Plugin daemonset keep restarting

When creating multiple SriovNetworkNodePolicy with different nodeSelector we observe that the sriov-device-plugin daemonset is unstable. After some investigation we found that the root cause is that the operator keep changing the order of nodeSelectorTerms in the daemonset.

I was using an older version, after testing on current master branch the daemonset was stable.


CMakePresets don't work with MSVC when using Remote/SSH, and are painful when running locally

Brief Issue Summary
To get presets that are designed for cross-platform projects (so, using ninja generator, etc - I haven't tried using the Visual Studio/vcproj generators because I do not want to use them and never had to use them before Presets) to work:

when running locally, I have to close all vscode windows, then launch it from within a terminal window that has the developer environment variables set. (I wrap Enter-VsDevShell with this https://gitlab.freedesktop.org/ryan.pavlik/powershell-enter-vsdevenv so I can get to it quicker, but it's still a pain)
When running over SSH using the official Remote SSH extension into a Windows machine, since I do not have system environment variables set to include the MSVC developer paths, I cannot use this extension's build support at all, because the presence of CMakePresets.json turns off existing toolchain-finding, and there's no way I know of to call Enter-VsDevShell in the vscode server's shell before it launches.

As a workaround, I either only build from the embedded terminal manually (where I can call Enter-VsDevShell), or I simply delete the presets file so I can get the old Kits behavior back.

I think the fundamental problem here is that the kits and CMakePresets are not inherently in conflict, but are treated as such by the extension. It might be OK to default to not setting up kits when you have a presets file, but I want to be able to tell the extension to please set up the environment for a kit before doing what the presets file says. This doesn't matter as much on Linux where there are standard system locations for a toolchain (though I would still like to pick which one), but on Windows with MSVC installed as usual, you can't just run cmake (at least using any non-vcproj generator) from a command line with no advance setup.
This is actively keeping me from using the otherwise nice CMakePresets functionality because it makes developing and contributing to projects on Windows so painful.
CMake Tools Diagnostics
No response
Debug Log
No response
Additional Information
No response

Related to this: https://github.com/microsoft/vscode-cmake-tools/issues/2912
Same basic pain point.

@rpavlik Thanks for your report.
A clarifying question: You mentioned that "the presence of CMakePresets.json turns off existing toolchain-finding". Are you referencing the finding of toolchains or are you referencing the usage and finding of kits?
Additionally, have you confirmed that MSVC is installed on your Windows remote machine?
Thanks for any information that can help us investigate.

Sorry, yes I meant kits. The stuff is definitely installed, I just don't always want to turn sideways to use my windows laptop so I see this when using ssh from my Linux desktop. Works fine without presets files, but put a preset file there and you've got trouble. The project I first noticed this with is https://gitlab.freedesktop.org/monado/monado but any with similar presets files will do it.


Adding

Pull Request
📖 Description
This PR adds a <FluentTooltip> component (and associated files) to wrap the <fluent-tooltip> web component
🎫 Issues
👩💻 Reviewer Notes
Besides the component itself (FluentTooltip.razor), pages have been added and modified to test the component. See the 'files changed' in this PR.
📑 Test Plan
Visist the FluentTooltipTest.razor page to see several variations on using the grid.
✅ Checklist
General

[X] I have added tests for my changes.
[X] I have tested my changes.
[X] I have updated the project documentation to reflect my changes.
[X] I have read the CONTRIBUTING documentation and followed the standards for this project.

Component-specific

[X] I have added a new component
[ ] I have modified an existing component

⏭ Next Steps
Review and determine if functionality is sufficient/stuff is missing

I noticed some warnings on this PR. This isn't something I've seen before.  Is this a GH Project thing that happens with .NET projects?
It would be nice if these warnings were eliminated. But, I understand they may not be caused by this PR.

These warnings are coming from the .NET compiler. We should resolve them by changing the project properties to enable Nullability settings. Involves quit a few changes but it is a good investment in stability and robustness of the library.

@vnbaaij @awentzel In that case, I think we should ignore these warnings for now. And then do a dedicated PR to change the project settings and fix these issues. So, let's not block on this for now.


fix: deprecate macos-12 from github actions

macos-12 getting deprecated from GitHub-actions

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

Project coverage is 79.04%. Comparing base (616bf2f) to head (7d1be5e).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #53   +/-   ##
=======================================
  Coverage   79.04%   79.04%
=======================================
  Files          14       14
  Lines        1021     1021
=======================================
  Hits          807      807
  Misses        145      145
  Partials       69       69

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


Format DateTime values in templates

Motivation
The project-reference.js and speaking-engagement.js templates display data of date & time content elements. They're currently formatted in the original ISO format.
Design guidelines
Format the date & time information in a user-friendly way.

/request

I've worked with Gatsby a bit recently. I'll take this one.

Implemented via #20 .


Updating user data based on UID fails

I have issue updating the user data based on user's bid. First, I retrieve the user's data by email, and I get what I expect. Then, I used the uid from the response for updating the user's data, but I keep getting the following error:
E/flutter (13587): [ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: FirebaseAuthAdminException: auth/missinguid: A uid identifier is required for the current operation.
E/flutter (13587): #0      validateResponse (package:_discoveryapis_commons/src/api_requester.dart:306:9)
E/flutter (13587): <asynchronous suspension>
E/flutter (13587): #1      ApiRequester.request (package:_discoveryapis_commons/src/api_requester.dart:72:16)
E/flutter (13587): <asynchronous suspension>
E/flutter (13587): #2      AccountsResource.update (package:firebaseapis/identitytoolkit/v1.dart:714:23)
E/flutter (13587): <asynchronous suspension>
E/flutter (13587): #3      _AuthHttpClient.setAccountInfo.<anonymous closure> (package:dart_firebase_admin/src/auth/auth_api_request.dart:966:24)
E/flutter (13587): <asynchronous suspension>
E/flutter (13587): #4      _AbstractAuthRequestHandler.updateExistingAccount (package:dart_firebase_admin/src/auth/auth_api_request.dart:733:22)
E/flutter (13587): <asynchronous suspension>
E/flutter (13587): #5      _BaseAuth.updateUser (package:dart_firebase_admin/src/auth/base_auth.dart:772:9)
E/flutter (13587): <asynchronous suspension>
E/flutter (13587): #6      LoginController.login.<anonymous closure>.<anonymous closure>.<anonymous closure> (package:stock_log/app/modules/login/controllers/login_controller.dart:162:23)
E/flutter (13587): <asynchronous suspension>
E/flutter (13587): #7      LoginController.login.<anonymous closure>.<anonymous closure> (package:stock_log/app/modules/login/controllers/login_controller.dart:149:13)
E/flutter (13587): <asynchronous suspension>
E/flutter (13587): #8      LoginController.login.<anonymous closure> (package:stock_log/app/modules/login/controllers/login_controller.dart:144:11)
E/flutter (13587): <asynchronous suspension>
E/flutter (13587): #9      LoginController.login (package:stock_log/app/modules/login/controllers/login_controller.dart:87:28)
E/flutter (13587): <asynchronous suspension>
E/flutter (13587):

Here is my code:
await auth.getUserByEmail(email.value).then((user) async {
            String uid = user.uid;
            await auth
                .updateUser(uid, ath.UpdateRequest(password: code.toString()))
                .then((value) async {
              print('USER UPDATED');
              print(value.toJson());
            });
          });

The exception occurred because the UID was missing in the update request (possibly missed implementation in the package). Added a PR to fix!
Thanks for raising this :)

The exception occurred because the UID was missing in the update request (possibly missed implementation in the package). Added a PR to fix!

@HeySreelal when we can expect the fix to be merged, so we can continue with our work that is dependent to this issue.
Thank you once again!

I'm actually unsure about the timelines. May be an admin can help you with that. Fix is actually a one liner, you can check this commit: 8315f5373ed7e72d93cba02fffa67e3cb69e3ea7. In addition to that, you could add it locally on your machine/server and continue your work while waiting for the PR to be merged. Best :)

Still facing the issue (currently using the 0.3.1 version
FirebaseAuthAdminException: auth/missinguid: A uid identifier is required for the
current operation.
This is the function
await auth.updateUser(
uid,
UpdateRequest(
email: emailCtrl.text,
password: passCtrl.text,
));

Hi @AbuBakar044, I can see that this fix hasn't been released to pub.dev yet. I'm not sure why though :)
Probably there'll be an update soon.


[Q] Ternary operator

Should I split the query, or will ArangoDB evaluate unused subqueries?
As I see in the explanation, the subqueries have been moved up
My Environment

ArangoDB Version:        3.7.11

AQL query (get groups by name or username)
let query = concat('%', @name, '%')

let groups = (for group in groups
    filter like(group.name, query, true)
    return group
)

let result = groups[0]? groups : (
    let user = first(for user in users
        filter like(user.name, query, true)
        return user
    )
    filter user
    for group in outbound user usersInGroups
        return group
)

for group in result
    return group

Explanation plan
 Id   NodeType                  Est.   Comment
  1   SingletonNode                1   * ROOT
 27   SubqueryStartNode            1     - LET #10 = ( /* subquery begin */
 29   SubqueryStartNode            1       - LET #6 = ( /* subquery begin */
 11   EnumerateCollectionNode     35         - FOR user IN users   /* full collection scan */   FILTER LIKE(user.`name`, "%влад%", true)   /* early pruning */
 24   LimitNode                    1           - LIMIT 0, 1
 30   SubqueryEndNode              1           - RETURN  user ) /* subquery end */
 16   CalculationNode              1       - LET user = FIRST(#6)   /* simple expression */
 17   FilterNode                   1       - FILTER user
 18   TraversalNode                1       - FOR group  /* vertex */ IN 1..1  /* min..maxPathDepth */ OUTBOUND user /* startnode */  usersInGroups
 28   SubqueryEndNode              1         - RETURN  group ) /* subquery end */
 25   SubqueryStartNode            1     - LET groups = ( /* subquery begin */
  4   EnumerateCollectionNode      1       - FOR group IN groups   /* full collection scan */   FILTER LIKE(group.`name`, "%влад%", true)   /* early pruning */
 26   SubqueryEndNode              1         - RETURN  group ) /* subquery end */
 21   CalculationNode              1     - LET result = (groups[0] ? groups : #10)   /* simple expression */
 22   EnumerateListNode          100     - FOR group IN result   /* list iteration */
 23   ReturnNode                 100       - RETURN group

Subqueries are always executed before the ternary operator evaluates the condition. There is no short-circuiting with subqueries.
Also see:
#14463
#1624
#2474


Cannot create new labels

Hi all,
I have a recurring issue when doing sequence labeling with Doccano. When creating a new project and adding labels to it, sometimes it tells me I cannot create a new label with the same name or same shortcut key, even though none of the other labels share it. It even happened with an empty project, with no other projects.

It works again after deleting all the projects and starting over, but not every time.
The app is deployed on Heroku.
System Information
OS: Windows 10
Doccano deployed on Heroku (master branch)

I fixed the label creation process in https://github.com/chakki-works/doccano/pull/347. Please check it.

It is still the case:

Here I could not create any label (but it was possible from another account)


nncp: 5.3.3 -> 5.4.1

Motivation for this change

upstream changed build system from make to redo

Things done

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

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

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

Result of nixpkgs-review pr 99304 run on x86_64-linux 1

1 package built:

nncp

Result of nixpkgs-review pr 99304 run on x86_64-darwin 1

1 package failed to build:

nncp

redo: install: exists and not marked as generated; not redoing.

Ah, case insensitive file system breaks this :-(

Result of nixpkgs-review pr 99304 run on x86_64-darwin 1

1 package built:

nncp

Result of nixpkgs-review pr 99304 run on x86_64-linux 1

1 package built:

nncp


Missing index returns 403 error instead of 404.

When abc index is present in elastic search and on firing a search query for abc*.
It gives back correct values with 200 status code.
On searching for abc* on elastic search with no abc starting indices in elastic search, it gives the following error.
Exception in thread "main" java.lang.AssertionError: assertion failed: ElasticSearch returned failure error code 403:
{"error":{"root_cause":[{"type":"security_exception","reason":"no permissions for indices:admin/get"}],"type":"security_exception","reason":"no permissions for indices:admin/get"},"status":403
This is not the expected response.

do you have permissions for an index abc*?

sg_roles.yml
sg_user01_role:
cluster:

CLUSTER_MONITOR
CLUSTER_SCROLL
WRITE_BULK
indices:
abc*:
"*":

INDICES_ALL

sg_roles_mapping.yml
sg_user01_role:
users:

user01

sg_action_groups.yml
CLUSTER_MONITOR:

cluster:monitor*
CLUSTER_SCROLL:
"indices:data/read/scroll"
INDICES_ALL:
"*"
WRITE_BULK:
"indices:data/write/bulk*"

user01 has the permissions mentioned above for the index abc*

This should be fixed in SG 16 together with https://github.com/floragunncom/search-guard/issues/374.
Can you verify whether it works with SG 16?

assume solved