[1.10.2-<IP_ADDRESS>] Slime chunk display is incorrect

I was trying to prevent slime spawning in my base with the Dark Utilities anti-slime block, but it wasn't working. After some messing around, I've discovered that it's because the slime chunk display is actually incorrect. It's saying some chunks are slime chunks which are not, and vice-versa.
I've checked against online slime chunk finders, as well as Botania's Slime in a Bottle, and Quark's Slime in a Bucket. All disagree with this mod's display. And the fact that they were spawning in other chunks not marked as slime chunks by this mod, but that are in other mods/finders, also shows it's wrong.
Thanks! Let me know if you need anything else. :)
EDIT: Looking at the code here: https://github.com/Lunatrius/LunatriusCore/blob/f037dd939c02ddd9149717769aba31f84fec1447/src/main/java/com/github/lunatrius/core/world/chunk/ChunkHelper.java
Assuming the numbers are correct, I don't see anything about bit shifting, dividing, or otherwise changing the coordinates before feeding into the equation. Could that be it?

What coordinates is your base located at?
I need to check if Java actually casts int to long in that formula, it's most likely overflowing the MAX_INT value. I doubt that Vanilla changed the formula at all and I know that it worked fine when I tested it last time.

Sure, one chunk that is not a slime chunk, but is marked as one is at -857, 36, -632. Not even close to MAX_INT.
Conversely, -889, 36, -648 is a slime chunk, but is not marked as being one.
I'm getting my formula from http://minecraft.gamepedia.com/Slime . I get it, you >> 4 to get the chunk coordinates from the x and y. That makes sense, now.

What are pos.getX() and pos.getY(), longs or ints? If I do this, it says true, but is wrong:
int x = -857 >> 4;
int z = -632 >> 4;

However, declaring them longs, it says false, which would be correct. So perhaps it's causing things in the equation to be int, whereas making them long would work. Still messing around, but you're probably right about the issue's general cause.

Ok, ignore previous post. heh
This code works, what's in there now does not:
import java.util.Random;

public class SlimeTest {
    public static void main(String[] args)
    {
        System.out.println(isSlimeChunk(3610660844093074297L, -857 >> 4, -632 >> 4));
        System.out.println(isSlimeChunk(3610660844093074297L, -889 >> 4, -648 >> 4));
    }

    public static final Random RANDOM = new Random();

    public static boolean isSlimeChunk(long seed, int x, int z)
    {
        //RANDOM.setSeed(seed + (x * x * 4987142) + (x * 5947611) + (z * z * 4392871) + (z * 389711) ^ 987234911);
        RANDOM.setSeed(seed + (long)(x * x * 0x4c1906) + (long)(x * 0x5ac0db) + (long)(z * z * 0x4307a7L) + (long)(z * 0x5f24f) ^ 0x3ad8025f);
        return RANDOM.nextInt(10) == 0;
    }
}

I used this formula from that wiki page:
Random rnd = new Random(seed +
                        (long) (xPosition * xPosition * 0x4c1906) +
                        (long) (xPosition * 0x5ac0db) +
                        (long) (zPosition * zPosition) * 0x4307a7L +
                        (long) (zPosition * 0x5f24f) ^ 0x3ad8025f);
return rnd.nextInt(10) == 0;

The coordinates on their own are not large, but the result of the multiplication exceed the MAX_INT value by 11 bits. The values are the same, they're just in hexadecimal notation on the wiki. Fixed in https://github.com/Lunatrius/LunatriusCore/commit/7a81e435da559103567711c63692b99c565f5993

Yeah, exactly, some of them were int inside the parentheses, then were cast to long. The third term, they were cast to long, and multiplied with a long. I checked the int constants, and they're definitely correct.
However, adding your changes to my test program returns false for both coordinates. That still doesn't seem to fix it.
My test code with your calculations as per the patch:
import java.util.Random;

public class SlimeTest {
    public static void main(String[] args)
    {
        System.out.println(isSlimeChunk(3610660844093074297L, -857 >> 4, -632 >> 4));
        System.out.println(isSlimeChunk(3610660844093074297L, -889 >> 4, -648 >> 4));
    }

    public static final Random RANDOM = new Random();

    public static boolean isSlimeChunk(long seed, long x, long z)
    {
        //RANDOM.setSeed(seed + (x * x * 4987142) + (x * 5947611) + (z * z * 4392871) + (z * 389711) ^ 987234911);
        //RANDOM.setSeed(seed + (long)(x * x * 0x4c1906) + (long)(x * 0x5ac0db) + (long)(z * z * 0x4307a7L) + (long)(z * 0x5f24f) ^ 0x3ad8025f);
        RANDOM.setSeed(seed + (x * x * 4987142L) + (x * 5947611L) + (z * z * 4392871L) + (z * 389711L) ^ 987234911L);
        return RANDOM.nextInt(10) == 0;
    }
}

The parameters being longs here should work out the same. And I pasted your line directly from your changed code.
That returns false and false, but should be false and true.
Suggest leaving the x and z as ints, and using this for the equation. It returns the correct values for the test cases. If I make them longs, it breaks it:
        RANDOM.setSeed(seed + (long)(x * x * 4987142) +
                              (long)(x * 5947611) +
                              (long)(z * z) * 4392871L +
                              (long)(z * 389711) ^ 987234911);

I'm 99.999% sure that above equation is correct.

Have you checked whether the new build works correctly?

On the newest version of Lunatrius Core now. Still not working correctly. Definitely need to use the equation I posted, I think. Making things all longs was the first thing I had tried, but just breaks it in different ways. I believe overflow is built into some parts of the equation.
BTW, thanks a lot for looking into this, and for the really nice mods. I do appreciate it! <3

It should be fixed now (https://github.com/Lunatrius/LunatriusCore/commit/57636be0202a309bbc0dc77e970796c739a41801). I have copied the formula from the sources.

Perfect, corresponds to slime chunks in my base correctly. Thanks! :D


Allow overriding Content-Type

Fixes #3194
Closes #3185

@timneutkens thanks for fixing the issue. There might be another concern - if setting content-type in getInitialProps, would static exporting still work because of res is used in the component

Why default is still text/htm in sendHTML function? It should be text/html; charset=utf-8.
With this change, 99% of the time, you will not need to overwrite the Content-Type in getInitialProps.

@howks fair enought. Feel free to send a PR!


Source of FFHQ-256

Which source did you get the FFHQ-256 dataset from? The downsampling method can be a critical factor in FID evaluation. Thanks!

Hi, it is true that the downsampling method is important in FID evaluation.  We use the official FFHQ-1024 images and downsample them with PIL.Image.LANCZOS to get anti-aliasing scaled images.

Thanks!


Project Collaboration - Azure Usage Billing Portal

I found your codebase and NuGet packages when trying to see if anyone has improved on the multiple rates and cost calculation for Azure.  It looks like your project could inject some life into this Usage and Billing Portal.  It's a nice to have for those of us who do not have an EA.
https://azure.microsoft.com/en-us/blog/announcing-the-release-of-the-azure-usage-and-billing-portal/
The project hasn't been updated in a while, but perhaps overhauling it with your api, and a singular web application and inexpensive Azure SQL instead of CSV you could help out many like myself who are looking to save money on Azure and be able to help others do so as well.
Cheers!  -Matt

Hey Matt,
thanks for your message and sorry for the late response. I'm currently really busy, so I'll unfortunately not find time to contribute to that project. I already thought about to join such a project or to create a simple solution that just shows the costs in a simple web application. This idea is still in my head, but as already said, I'm currently really busy, so it's on my todo list for later on.
A few weeks ago, I build an Azure Function that reads data from the billing api and sends a weekly report via mail (https://codehollow.com/2017/09/weekly-azure-billing-report-per-mail-azure-functions/). It's by sure a simple solution and the initial setup requires some steps, but maybe it helps.
Cheers! - Armin


Installation not working

Having trouble installing this component - have downloaded via HACS and also tried manually copying custom_components folder.  Both have no entities showing after entering the sensor code into configuration.yaml and altering login details.  Email and password definitely correct and code set to 44 (uk).  Scooter_id is 0.
There is an integration which appears in devices & services > add integration called “Nui Scooters” [sic] but on attempting to add the integration it says “ This device cannot be added from the UI, You can add this device by adding it to your 'configuration.yaml'. See the documentation for more information.“
I’m a little unsure what to try next.  Can anyone help?

strange for me the integration works for me for more then 2 years, do you get any errors in your logs ?

do you get any errors in your logs ?

I do, yes:
Logger: homeassistant.components.sensor
Source: custom_components/niu/sensor.py:375
Integration: Sensor (documentation, issues)
First occurred: 10:34:28 (1 occurrences)
Last logged: 10:34:28
Error while setting up niu platform for sensor
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 304, in _async_setup_platform
await asyncio.shield(task)
File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
File "/config/custom_components/niu/sensor.py", line 375, in setup_platform
sn = get_vehicles_info(api_uri, token)["data"][scooter_id]["sn"]
IndexError: list index out of range
I’ve no idea what any of this means but happy to try any fixes to get it working.  Thanks 🙏

wat is the name of your scooter

wat is the name of your scooter

It’s a Niu NQi GTS - 2019

I update the version in HACS and with me it is working. Be aware you scooter name in your niu app can not have special characters

Im sorry maybe I misunderstood.  The model of the scooter is an NQi GTS but the name in the app is “Ngt demo”.  Would that have any effect?

I update the version in HACS and with me it is working. Be aware you scooter name in your niu app can not have special characters

Amazing, your update has allowed me to install and the repo looked totally updated in HACS.  Thank you.  I now have my credentials in and the integration has found the name of my scooter.  All entities showing “Unknown” at the moment but I guess this might take a little while to update.  Once this is resolved I’ll be up and running :)

Nil data persisting on all entities after installation.  Integration has pulled my scooter name from my account so login credentials are definitely correct.  I have no special characters in scooter name.  Have tried reloading integration and restarting HA, with no effect.
@marcelwestrahome what could I try?

Did you get this sorted out @jevburchell ? I am having precisely the same issue.

Did you get this sorted out @jevburchell ? I am having precisely the same issue.

It’s not working no, I’ve tried reinstalling multiple times, reloading the integration, changing the scooter_id, changing the country code from 44, nothing is having any effect.

Yes, precisely the same issue here despite trying various similar things. I certainly hope this gets a response so we can work on sorting this out. As it stands, this integration is completely non-functional.

Hi  I am on holiday at this moment. So can not look to this in detail. The older version is still working, so if you realy need the integration..
All names of sensors are the same for both versions. I installed the integration by myself and it is still working  in the Netherlands,  so may be it has something to do with location.

Thank you for the response. I was thinking it may have something to do with the country as well. I am in Canada so am using country code 1.
Enjoy your holiday! I look forward to more investigation of this issue once you're available.

Sorry to disturb you on holiday Marcel.  Happy to wait and assist in any way I can at a later date.  It’s a very useful integration (I’ll be automating the charging of my scooter from excess solar generation) so I’m looking forward to seeing it in action :)

Try to do some research with my phone, @jevburchell,@Zamboniman, you both not in Europe, right. Its seems the interface (the url) that is used works only for europe @pikka97 you any comments.
I am based in the Netherlands, for me the plugin is working well, so makes me difficult to solve the issue :-(

I’m in Scotland 🏴

And I am in Canada

Try to do some research with my phone, @jevburchell,@Zamboniman, you both not in Europe, right.

So given that I'm in Europe, the url should work?  Possibly there is another bug in the mix

Thanks @syssi
The URL it's attempting is https://ncsi-fk.niu.com on port 443
I can send more details if required.

Could you provide the full capture by mail<EMAIL_ADDRESS>

Could you update the files at your custom_components/niu folder and apply these changes just for testing?
https://github.com/marcelwestrahome/home-assistant-niu-component/pull/49/files

Could you update the files at your custom_components/niu folder and apply these changes just for testing?
https://github.com/marcelwestrahome/home-assistant-niu-component/pull/49/files

I have edited the code in api.py and const.py as specified here.  Reloaded all YAML, restarted HA, Reloaded Integration and Restarted HA again with no change - all entities still Unavailable

Made the changes requested. Unfortunately, the integration is not working at all now. No entities. I'm just double checking now to make sure I didn't mess up the editing of api.py and const.py

Ah, I now have the same effect as @Zamboniman - all entities are gone.  I've deleted the integration and reinstalled and now it's empty - no device or entities within the integration

Thanks! Let me know if there's anything else I can do to help.

I've pushed some more changes. You can download the updated version here:
https://github.com/syssi/home-assistant-niu-component/archive/refs/heads/update-api-version.zip
Please override the content of the custom_components/niu folder and restart Home Assistant for testing.

I've pushed some more changes. You can download the updated version here:
https://github.com/syssi/home-assistant-niu-component/archive/refs/heads/update-api-version.zip
Please overwrite the content of the custom_components/niu folder and restart Home Assistant for testing.

Amazing, this has worked for me.  I see the entities and they are populated with data.  Thank you kind sir for your excellent support.  It is highly appreciated.  This will be extremely useful :)

This update seems to have done the trick. Most of the entities now have data. However, none of the 'last track' entities are working, all still show 'unknown'.

Also, 'max times charged' shows unknown, and latitude and longtitude show '0'

please create a new issue


Latest Documentation

Hello l have some problemi with the latest documentation,
l'm following all steps but there is something wrong, how can l install this application?
Can you help me?
l need to download server and ui separately?

Same issue for me.


refactor: use markdown preview

What does this PR do and why?
Provide a description of what this PR does. Link to any relevant GitHub issues, Notion tasks or Discord discussions.
Screenshots or screen recordings
If applicable, provide screenshots or screen recordings to demonstrate the changes.
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] (if relevant) I have tested this PR in all supported browsers.
[ ] (if relevant) I have tested this PR in dark mode and light mode (if applicable).

Current dependencies on/for this PR:

main

PR #3877

PR #3878

PR #3879

PR #3880   👈

This comment was auto-generated by Graphite.

@wraeth-eth started a stack merge that includes this pull request via Graphite.

Graphite rebased this pull request as part of a merge.

@wraeth-eth merged this pull request with Graphite.


Would not connect to integration after 1.15.4 update

Problem/Motivation

: After upgrading from 1.15.3 to 1.15.4 the zwavejs integration in home assistant would not connect to the  zwave js ui

Expected behavior

The integration to connect automatically or connect after the integration is reloaded.

Actual behavior

got this error: Config entry 'Z-Wave JS' for zwave_js integration not ready yet: Failed to connect: Cannot connect to host a0d7b954-zwavejs2mqtt:3000 ssl:default [Connect call failed ('<IP_ADDRESS>', 3000)]; Retrying in background

Steps to reproduce

All I did was install 1.15.4
Rolling back to 1.15.3 seem to have solved the issue

Proposed changes
none

Duplicate of #553


scons --help should not parse Python files

Describe the bug
Following the "Principle of least astonishment" it would make sense that invoking scons --help does not try to interpret the SConstruct.py file in the current directory and possibly fails to interpret it if it is not currently valid Python code (i. e. interpretation fails).
scons --version does not seem to parse Python files.
Required information

Link to SCons Users thread discussing your issue.
bdbaddog asked me to file the issue in the Discord #scons-help channel
Version of SCons
v3.1.2
Version of Python
N/A
Which python distribution if applicable (python.org, cygwin, anaconda, macports, brew,etc)
N/A
How you installed SCons
Fedora package
What Platform are you on? (Linux/Windows and which version)
Fedora 31
How to reproduce your issue?  Please include a small self contained reproducer. Likely a SConstruct should do for most issues.
Create an invalid SConstruct.py file and invoke scons --help
How you invoke scons (The command line you're using "scons --flags some_arguments")
scons --help

Try scons -H, which does not check for options that could have been added by the scripts.
This is unavoidable for scons -h, as it must parse the sconscripts: they could be adding additional options with AddOption and -h (aka --help) promises to display those.

as @bdbaddog points out, catching errors if in help mode could still allow printing something useful even if the script isn't syntactically correct.

Okay, so there's special logic that makes --help work if there's no SConstruct, which would normally emit a failure message.  Can't really reuse that, because it happens up front, but maybe there's still a way to catch the error from a bad sconscript and be nice if we know help was requested. Is that worth pursuing?


cmd/ebitenmobile: the screen turns black when launching from a notification

Ebitengine Version
9bc5ed38479b9193961a339ff4b3a59cf243b16c
Operating System

[ ] Windows
[ ] macOS
[ ] Linux
[ ] FreeBSD
[ ] OpenBSD
[X] Android
[ ] iOS
[ ] Nintendo Switch
[ ] PlayStation 5
[ ] Xbox
[ ] Web Browsers

Go Version (go version)
go version go1.22.4 darwin/arm64
What steps will reproduce the problem?

Create an Android application with allowing notification
Launch the application and put it background
Send a test message to the application (https://firebase.google.com/docs/cloud-messaging/android/first-message)
See the application running

What is the expected result?
The application runs correctly
What happens instead?
The screen blacks out
Anything else you feel useful to add?
Maybe https://github.com/hajimehoshi/ebiten/commit/a30f075896b76b2f6fb1536c490aead650e4157f ?

Apparently, the entire GLSurfaceView is recreated at notification.

This happens when the view is created at Activity's onCreate. We should not do that.

Note to myself, if you want to prevent activity recreation on notification, see
https://stackoverflow.com/questions/7944338/resume-last-activity-when-launcher-icon-is-clicked
android:launchMode="singleInstance" worked in my case.


Implement turbiniactl workercheck command

Right now we don't have a way to check how many workers are connected, and this can be non-trivial because workers can connect from anywhere.  We should implement a 'turbiniactl workercheck' (or similar) command to run a quick check on all the workers.   PSQ has a Broadcast worker mechanism that we can use for this.

We can adapt and use the existing worker_stat.py task code for this.

Bump.   When there are unexpected workers listening to the same pubsub channel, it's pretty annoying to debug.   We should also add a 'worker kill' command so that we can do something about it remotely.

I chatted with @alimez about this, and I think rather than having a worker check command be implemented as a task, we'll have the workers write heartbeat/timestamp information into datastore keyed by their hostnames.  This will allow us to see what workers are running and where.  We can also use this to do some basic monitoring like in https://github.com/google/turbinia/pull/509 .
Cc: @wajihyassine for the monitoring angle since we could potentially use this in a dashboard.

We should still have a turbiniactl workercheck command or similar so that we can read out the heartbeat information easily.  If we follow the same pattern as other status commands we should have a corresponding cloud function that actually does the work.

@aarontp Is this something you'd still want to add to the new client? If so, pls assign to me.

@jleaniz I do think that we should have some kind of functionality that gives us insight into the actual running workers.  Now that we have just Celery, I think we can use functionality from there to do this instead of a separate heartbeat method that we maintain.  We had also talked about out of band about using the Celery web ui, so we could potentially use that instead if that would be easier.


When feature's default value is false it returns nil instead

Given a super simple feature defition:
{"defaultValue"=>false}

The SDK returns nil when checking its value, reproduction code:
features = {"simple_feature" => {"defaultValue" => false}}
gb = Growthbook::Context.new(features: features)
gb.on?("simple_feature") #=> false
gb.feature_value("simple_feature") #=> nil

Is this expected behaviour? The offending code is here: https://github.com/growthbook/growthbook-ruby/blob/main/lib/growthbook/context.rb#L137
Which I think is incorrect since: false || nil in Ruby will always yield nil. I think what author meant to write was:
feature.default_value.nil? ? nil : feature.default_value

I also tested the behaviour with JS SDK and it does return false as expected:
gb.loadFeatures({ autoRefresh: true }).then(function() {
	gb.getFeatureValue('simple_feature'); // => false
});

Thanks for catching this. A PR would be great!

PR here


ENH: Rotation equality checks

Is your feature request related to a problem? Please describe.
It would be useful to have a function to determine if two scipy.spatial.transform.Rotation objects are equal.
Describe the solution you'd like.
Since Rotations have floating point representations an approximate equality check is needed, and since the space of rotations is cyclic with a discontinuity in representation at +-pi, we can't simply check equality between the values of the quaternions of the two rotations. I believe the right way to do this is to calculate the magnitude of the angle between the two rotations, and check if it is near zero.
A simple way to implement this:
# a standalone function, or a method of the Rotation class (replace R1 with self)?
# atol measured in radians, 1e-12 as a default is notional
def rotation_approx_equal(R1, R2, atol=1e-12):
    Rdiff = R1* R2.inv()
    return Rdiff.magnitude() < atol

I think it would also be useful to take this chance to add a distance function as well:
def dist(self, R2):
    Rdiff = self * R2.inv()
    return Rdiff.magnitude()

def approx_equal(self, R2, atol=1e-12):
    return self.dist(R2) < atol

Huynh, "Metrics for 3D Rotations: Comparison and Analysis" describes a potentially more efficient distance function (note that @ is the dot product of the vector representation, not Hamilton multiplication): dist = 2 * arccos(abs(q1 @ q2))
Describe alternatives you've considered.
No response
Additional context (e.g. screenshots, GIFs)
No response

Given that most of Rotation's internals deal with quaternion representations, another possible implementation of this equality check could be
    def approx_equal(self, other: Rotation, atol=1e-12) -> bool:
        return np.allclose(self.as_quat(), other.as_quat(), atol=atol) # or maybe just self._quat and other._quat ?

    def __eq__(self, other: Rotation) -> bool: # enables `if R1 == R2:` syntax
        return np.array_equal(self.as_quat(), other.as_quat())

Hey @cgobat, unfortunately that approach runs into problems with the double cover of rotation space by quaternions. In other words, q and -q describe the same rotation. There's a lot of good discussion here: https://github.com/scipy/scipy/pull/17334
I do think the __eq__ operator might have its uses, though I'll note that it's inherently fragile for floating points and does work right now for identifying specific class instances:
>>> from scipy.spatial.transform import Rotation as R
>>> R0 = R.identity()
>>> R0 == R0
True
>>> R1 = R.identity()
>>> R0 == R1
False

Right, good point about $-q$ representing the same rotation as $q$. I guess there would have to be an extra step of making sure both of the quaternions being compared have positive scalar components, for instance, and flipping them if not.
I don't think the fragility with floats is a problem, per se. That "exact equality" behavior mirrors the behavior of comparing of two NumPy arrays using ==, (which is probably the wrong choice of operator for most use cases—but that's why things like np.isclose, np.allclose, and the proposed Rotation.approx_equal method are needed), but I would think we want to have that parallel in place. As you've constructed them, I think the intuitive value for R0 == R1 should be True, but of course R0 is R1 is and should be False. This holds for the NumPy analogy:
>>> import numpy as np
>>> A = np.eye(3)
>>> np.array_equal(A, A)
True
>>> np.array_equal(A, A.copy())
True
>>> B = np.eye(3)
>>> np.array_equal(A, B)
True
>>> A is A, A is A.copy(), A is B
(True, False, False)

Even ensuring the positive scalar components wouldn't work out, because there then becomes a discontinuity in the other components. Consider the small angular jump jump from eg q1 = (0, 0, 0.9999, 0.0001) to q2 = (0, 0, -0.9999, 0.0001). It's why the approach in https://github.com/scipy/scipy/pull/17460 calculates the angle by taking (R2 * R1.inv()).magnitude().
I do like what you're saying about that equality check.


fix: upgrade esm-loader & suppress globalPreload warning

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Question CasDoor Casbin

Question: https://github.com/casdoor/casdoor/issues/3033

@DemidRol713 plz don't create a new issue just to alert another issue.


failed to unpack image on snapshotter overlayfs: content digest not found

Description
Cannot pull some images from docker hub registry.

Steps to reproduce the issue:
# crictl pull prom/mysqld-exporter:v0.12.1
FATA[0000] pulling image failed: rpc error: code = Unknown desc = failed to pull and unpack image "docker.io/prom/mysqld-exporter:v0.12.1": failed to unpack image on snapshotter overlayfs: content digest sha256:0c18b88b8db3c193eefeede31a80d5d0c850306076b7166910ed239685dd20b0: not found

# crictl pull prom/pushgateway:v0.9.1
FATA[0000] pulling image failed: rpc error: code = Unknown desc = failed to pull and unpack image "docker.io/prom/pushgateway:v0.9.1": failed to unpack image on snapshotter overlayfs: content digest sha256:7351147077496a942c478cb3f4c850c1feb5c96a614472582c3feae07de959db: not found

Message from kubernetes kubelet:
  Warning  Failed     77s (x4 over 3m12s)  kubelet, gke-c1-pool-6-10d00b31-vmdm  Failed to pull image "prom/mysqld-exporter:v0.12.1": rpc error: code = Unknown desc = failed to pull and unpack image "docker.io/prom/mysqld-exporter:v0.12.1": failed to unpack image on snapshotter overlayfs: content digest sha256:0c18b88b8db3c193eefeede31a80d5d0c850306076b7166910ed239685dd20b0: not found

Describe the results you received:
Error message is generated, eg:
FATA[0000] pulling image failed: rpc error: code = Unknown desc = failed to pull and unpack image "docker.io/prom/mysqld-exporter:v0.12.1": failed to unpack image on snapshotter overlayfs: content digest sha256:0c18b88b8db3c193eefeede31a80d5d0c850306076b7166910ed239685dd20b0: not found

Describe the results you expected:
Image is pulled locally.
Output of containerd --version:
containerd github.com/containerd/containerd 1.2.7 85f6aa58b8a3170aec9824568f7a31832878b603

Any other relevant information:
Tested on GKE cluster 1.13.7-gke.24 with COS_CONTAINERD node images
crictl version 1.12.0

With docker pulling the same images works fine:
# docker pull prom/mysqld-exporter:v0.12.1
v0.12.1: Pulling from prom/mysqld-exporter
8e674ad76dce: Pull complete
e77d2419d1c2: Pull complete
1a6af4912274: Pull complete
Digest: sha256:9fe9938c4ac9216cc24005144338f14fac4f604f139b481cc541bead008db3c1
Status: Downloaded newer image for prom/mysqld-exporter:v0.12.1

Pulling older version of the image with crictl works fine:
# crictl pull prom/mysqld-exporter:v0.11.0
Image is up to date for sha256:e80442e91b90424ab62495669b0b4113222aeb8c8ca341d7091e782d0bd7f6d1

This is a multi-platform image issue.
Here is your image:
{
   "schemaVersion": 2,
   "mediaType": "application/vnd.docker.distribution.manifest.list.v2+json",
   "manifests": [
      {
         "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
         "size": 948,
         "digest": "sha256:3851cf4f9e48dbe2b3efab2a93487d718eadf57de22440f7d5dfb377e2ed7718",
         "platform": {
            "architecture": "amd64",
            "os": "linux"
         }
      },
      {
         "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
         "size": 948,
         "digest": "sha256:0c18b88b8db3c193eefeede31a80d5d0c850306076b7166910ed239685dd20b0",
         "platform": {
            "architecture": "amd64",
            "os": "linux"
         }
      },
      {
         "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
         "size": 948,
         "digest": "sha256:be7c6c0e9070377d739a20686ecb89dc2757cbc96c7bd25be0a2ac25f0aa264c",
         "platform": {
            "architecture": "amd64",
            "os": "linux"
         }
      }
   ]
}

When pulling image, we only pull and unpack content for one of the matched image. (The 1 at https://github.com/containerd/containerd/blob/master/pull.go#L88).
When getting the image information, there is no such limitation, so it tries to match all images. However, because other images are not downloaded during pull, the "content not found" error is returned.
I got the same error when working on windows support. @dmcgowan @jterry75
I'll send out a fix.

Hm, this should have been fixed in HEAD https://github.com/containerd/containerd/commit/1161409779f0275f5e65a697d587ff893f9f9188.
Let's cherry-pick the fix into 1.2?

It did end up in 1.2.8, and 1.2.9 will be out soon with some additional fixes (unrelated to that particular issue).

@r0bj We'll update containerd in GKE soon. And this should be fixed in newer GKE versions.

@Random-Liu thanks for info.

The issue should have been fixed in containerd 1.2.8.
Vendors will update containerd to fix the issue, e.g. GKE.
Close this for now. Feel free to reopen if you still need anything. :)

Seeing this on GKE with container 1.2.8 in a mixed way, where 2 nodes give this error and 1 doesn't
https://github.com/containerd/cri/issues/1565


Adding ability to enable ironic service in tempest

The ansible playbook for openstack doesn't allow enabling of ironic in the tempest.conf
This is updating the source for the os_tempest playbooks to a branch that adds this functionality.
Branch found here: https://github.com/dani4571/openstack-ansible-os_tempest/tree/feature/stable/mitaka/support_ironic_service

Hey @dani4571, can you please add a link to the upstream os_tempest issue or PR in a comment in ansible-role-requirements.yml?
This will make it easier to figure out when we can point back to upstream os_tempest.

Change has been updated with information about the upstream review and bug report that will fix our issue


ci(coverage): wait for CI until reporting status

Issue or need
Multiple reports need to be uploaded in order to provide a holistic coverage view (unit tests + E2E tests run for each Angular example app). However, current Codecov config doesn't wait to report the coverage status. So we get invalid statuses until all reports are uploaded

Proposed changes
Codecov to wait for CI to finish in order to show coverage results

Quick reminders

🤝 I will follow Code of Conduct
✅ No existing pull request already does almost same changes
👁 Contributing docs are something I've taken a look at
📝 Commit messages convention has been followed
💬 TSDoc comments have been added or updated indicating API visibility if API surface has changed.
🧪 Tests have been added if needed. For instance, if adding new features or fixing a bug. Or removed if removing features.
⚙ API Report has been updated if API surface is altered.

#698  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @davidlj95 and the rest of your teammates on  Graphite

Seems this doesn't solve the issue :(

Despite Codecov is aware of the nº of reports not matching

Seems we need the after_n_builds config as stated in docs


Force Rerendering of Component doesn't work

I've created a bootstrap Multiselect dropdown. Now i have problems to make blazor rerender the dropdown, when I reset the selected items via an additonal button

@using System.Collections.ObjectModel
@using System.Linq
@using Microsoft.JSInterop
@typeparam T

        <select style="float: left; width: 80%;"<EMAIL_ADDRESS>class="form-control selectpicker" multiple data-live-search="true"
                onchange="@(async x=> await OnSelectionChanged())">
            @{
                var selectedKeys = SelectedItems.Select(i => GetKey(i)).ToArray();
                foreach (var value in Items)
                {
                    var key = GetKey(value);
                    <option selected="@(selectedKeys.Contains(key))" value=@(key)>@(key)</option>
                }
            }
        </select>
        <button style="float: left; width:20%;" class="form-control" onclick="@(async x =>
            {
                this.SelectedItems.Clear();
                this.StateHasChanged();

            })" >Reset</button>

@functions {

    private string Id { get; } = "id"+Guid.NewGuid().ToString("N");
    private async Task OnSelectionChanged()
    {
        var result = (string)await JSRuntime.Current.InvokeAsync<object>("eval", $"(''+ $('#{this.Id}').val())");
        var selectedKeys = result.Split(",");

        Api.Global.Log().Message("selectedKeys: " + selectedKeys.Aggregate());
        this.SelectedItems.Reset(this.Items.Where(i => selectedKeys.Contains(GetKey(i))));
    }

    protected override bool ShouldRender()
    {
        return true;
    }

    protected override async Task OnAfterRenderAsync()
    {
        await SelectPickerAsync(this.Id);
        await base.OnAfterRenderAsync();
    }

The StateHasChanged invocation in the button click seems not to trigger the rendering. How can I really enforce it?

I solved the issue by setting the val via javascript. But i still wonder how Blazor is determining which parts auf to be rerendered and which not. Actually I expected that the whole component will be rerendered if I invoke StateHasChanged().

Thanks for contacting us, @TFTomSun.
Can you please provide us with a repro project so we can investigate this?


Add Captcha to Sign Up Page

This would be in the MembershipForm Component, we need to prohibit the user from signing up without completing it.
A good start may be here by installing react-google-captcha.

I've done this many times in my projects using react-google-captcha. Here's some relevant info below. You can use V2 or V3. I use the V2 checkbox in my projects
Take a look at this boilerplate project I put together that uses it: https://github.com/arjohnston/boilerplates/blob/master/next-express/
Specifically, the API is set up in the server.js:
https://github.com/arjohnston/boilerplates/blob/master/next-express/server.js
The API is used on the contact form:
https://github.com/arjohnston/boilerplates/blob/master/next-express/components/ContactForm/ContactForm.js
And we have the config file holding the keys:
https://github.com/arjohnston/boilerplates/blob/master/next-express/config/config.example.js
You can learn more about recaptcha docs here:
https://developers.google.com/recaptcha/intro
And get started with getting the appropriate keys here:
https://www.google.com/recaptcha/intro/v3.html

@arjohnston Just saw this, thank you!!!


Error deploying Chirpstack V4 portainer stack - mosquitto.conf can't mount

In creating the chirpstack v4 image available in portainer, the process errors out when trying to start the mosquitto container. I've copied a portion of the event log below, but the full one (with really bad formatting, sorry) is here: https://pastebin.com/L9V9xe0E
Starting Container chirpstack-redis
Starting Container chirpstack-postgres
Starting Container chirpstack-certificates
Starting Container chirpstack-postgres
Started Container chirpstack-redis
Started Container chirpstack-certificates
Started Error response from daemon:
    failed to create task for container:
    failed to create shim task: OCI runtime create
    failed: runc create
    failed: unable to start container process:
        error during container init:
        error mounting "/data/compose/4/Stack/ChirpStack4/configuration/mosquitto/mosquitto.conf" to rootfs at "/mosquitto/config/mosquitto.conf":
            mount /data/compose/4/Stack/ChirpStack4/configuration/mosquitto/mosquitto.conf:/mosquitto/config/mosquitto.conf (via /proc/self/fd/6), flags: 0x5000:
                not a directory: unknown: Are you trying to mount a directory onto a file (or vice-versa)?
                Check if the specified host path exists and is the expected type

I can confirm the config file is on the host at the expected path:

/data/compose/4/Stack/ChirpStack4/configuration/mosquitto/mosquitto.conf
So for some reason it isn't mounting to the image, and I can't find much helpful info for fixing this error. It's usually caused by accidentally mapping a directory to a file or entering an incorrect host path (docker will just implicitly create a folder), but neither seems to be the case here. Anyone got any ideas?

Hi @Aweptimum , that stack mounts a local file to configure the mosquitto service. AFAIK this is not supported by Portainer CE, correct me if I'm wrong.
I'd suggest to use rakpios-cli or use docker container directly on the stack folder to bring it up.

Sorry for the long silence (life got busy)
I haven't used portainer before, so your guess is better than mine. Both the cli tool and docker compose worked just fine.
I was able to access the chirpstack web interface and ensure everything worked. Probably can close, but it would be nice if Portainer CE's lack of local mounting was confirmed and then added to the Portainer Services Documentation

You are right. Will add a notice to the documentation about it.
Closing the issue.

Cool, thanks :)

Actually, I have already pushed a change to the stack that solves the issue. Since named volumes work just fine with Portainer I bring up an initial service to copy the configurations to one such named volume shared with the rest of services, hence they can access the configuration data.


Csharp optional

@dvdsgl I have a feeling that some people might prefer our current way of not distinguishing between optional and nullable.  Should we make this into an option?

I can barely understand this distinction. I don't think we should make users reason about it.

This is the code we generate with this change:
        [JsonProperty("in", Required = Required.Always, NullValueHandling = NullValueHandling.Ignore)]
         public string In { get; set; }


added case id to the case exists api , test cases changes

added case id to the case exists api , test cases changes

Please update api specs of the case api pr

Done


Testing Salt formulas

Tested both in isolation and within the test suite with success.

Yes, what you suggest makes sense.
But if I have a look at the salt steps, I can see

features/step_definitions/salt_download_endpoint_steps.rb
features/step_definitions/salt_formulas_steps.rb
features/step_definitions/salt_minions_page_steps.rb
features/step_definitions/salt_software_states.rb
features/step_definitions/salt_ssh_steps.rb
features/step_definitions/salt_steps.rb

so it is already split across several files. Also, there is the risk to have a salt steps file that is too big.
I suggest we discuss this calmly after the GMC, as part of the many cleanups. Meanwhile, I'd suggest it can stay like this.
I'll add it to the "cleanup" card in Trello.

NO, please, no new errors. the past is the past, the present is to improve.
put the steps here features/step_definitions/salt_steps.rb

Question is, is it really an improvement?
Having better classified steps has also its advantages.

https://github.com/SUSE/spacewalk-testsuite-base#1-steps-should-be-grouped-by-topic-and-not-by-feature
This guideline was made by @dmacvicar
https://github.com/SUSE/spacewalk-testsuite-base/blob/master/docs/Pitfalls-test.md#meanless-step-names
We are not at the level but we want to go on the direction

As you wish.
I'll move them to salt_steps.rb tomorrow.

Éric no hurry take your time i reccomend you also the guide link i gave you it à nice doc

Done :-) .

Latest change requests applied, retested in isolation -> green. Tests in context not needed as only moving around and changing cucumber stances.

OK looks good thx!  We will merge it after beta1

Thank you Dario.

Yw


fix(versions): update Activiti/activiti-cloud-query versions into master

UpdateBot pushed helm dependency: common to: 1.1.15

UpdateBot commands:
updatebot push-version --kind helm common 1.1.15


Patch 1

Update .htaccess
Attempted changes (@saidfathalla feel free to let me know if these are incorrect in their implementation)

https://purls.helmholtz-metadaten.de/vp redirects to https://app.vocpopuli.com/ by default

PURLs of the form https://purls.helmholtz-metadaten.de/vp/[subdir1]/[subdir2] get redirected to https://app.vocpopuli.com/public_view_term/[subdir1]/[subdir2]

Requests with a "text/turtle" value in their HTTP_ACCEPT header to https://purls.helmholtz-metadaten.de/vp/cupgrin-41249667-vp/... point to https://gitlab.com/publish-vocabs/vocabulary-cup-grinding-v3/-/raw/main/SKOS_Cup_Grinding_v3.ttl

Requests with a "application/ld+json" value in their HTTP_ACCEPT header to https://purls.helmholtz-metadaten.de/vp/cupgrin-41249667-vp/... point to https://gitlab.com/publish-vocabs/vocabulary-cup-grinding-v3/-/raw/main/SKOS_Cup_Grinding_v3.jsonld

Please add contact information () at the beginning of the file (cf. seo).
# Contact person name: XXXX
# Contact person email: YYYY

This information is used by a tool we are developing to maintain PID persistence within our service.

I updated the .htaccess file with the needed contact info.


no maven repo found

Error:(258, 13) Failed to resolve: com.commonsware.cwac:netsecurity:0.4.4
Ive searched the maven repository and there seems to be no publishings for your code.

nvm realized you added a custom repo


Is there ideal solution to Daylight Saving Problem (One Feature Request)

Hi Quartz communities,
Because of the maturity and stability of Quartz, our work flow system is using Quartz library. We can't use UTC directly, since many of our jobs have to depend on PDT/PST. I've read through the daylight saving part of Quartz documentation. However, Both CronTrigger and SimpleTrigger can not meet our requirement.
We are using CronTrigger, but 2:00-3:00 job will be missed when daylight saving changes, you know. Do you have any hacks to let jobs delay to run (no skip) at that day, if the job is a daily job.

That's how we design the Cron and Simple Trigger to be. There are many other triggers available from the library as well, please ensure you read through them. The best way to avoid DST is not to use any Calendar base trigger (eg: SimpleTrigger)
If you must use Calendar based Trigger, then you can try create Listener to specially detect when DST occur, then you simply invoke the job manually in your listener.


ERROR    missing filename or label

I tried many files, But below Error happened, and I tried only one file, the result was the same.
I tried 'aocr dataset datasets/annotations-training.txt datasets/training.tfrecords', but the result was the same.(C:\Users\kimduknam\datasets\images include jungle.jpg)
'annotations-training.txt' file is below.
./datasets/images/jungle.jpg jungle
C:\Users\kimduknam>aocr dataset ./datasets/annotations-training.txt ./datasets/training.tfrecords
2018-02-05 17:39:28.312678: I C:\tf_jenkins\workspace\rel-win\M\windows\PY\36\tensorflow\core\platform\cpu_feature_guard.cc:137] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX AVX2
2018-02-05 17:39:28,310 root  INFO     Building a dataset from ./datasets/annotations-training.txt.
2018-02-05 17:39:28,310 root  INFO     Output file: ./datasets/training.tfrecords
2018-02-05 17:39:28,310 root  ERROR    missing filename or label, ignoring line 1: ./datasets/images/jungle.jpg jungle
2018-02-05 17:39:28,310 root  INFO     Dataset is ready: 1 pairs.
2018-02-05 17:39:28,310 root  INFO     Longest label (0):

Your windows machine uses backslashes in the path, while your annotations file has forward slashes.

@AdamWP OH, I missed it. Thank you for the comment! Without it, I'll never get it.

I tried on my Ubuntu 14.04 LTS, But The same error happened.
I used forward slashes.
I did the command at my home folder.
My home folder has datasets folder
datasets folder has images folder and annotations-training.txt file.
images folder has coley.jpg file
annotations-training.txt file is just like below
datasets/images/coley.jpg coley
(tensorflow)***~$ aocr dataset ./datasets/annotations-training.txt ./datasets/training.tfrecords
2018-02-05 21:01:17.813949: I tensorflow/core/platform/cpu_feature_guard.cc:137] Your CPU supports instructions that this TensorFlow binary was not compiled to use: SSE4.1 SSE4.2
2018-02-05 21:01:17,814 root  INFO     Building a dataset from ./datasets/annotations-training.txt.
2018-02-05 21:01:17,815 root  INFO     Output file: ./datasets/training.tfrecords
2018-02-05 21:01:17,851 root  ERROR    missing filename or label, ignoring line 1: datasets/images/coley.jpg coley
2018-02-05 21:01:17,852 root  INFO     Dataset is ready: 1 pairs.
2018-02-05 21:01:17,852 root  INFO     Longest label (0):

@nickppbd pushed a small fix that should help, please update the module and try again :)

@emedvedev Thanks, It works well!!! I aocr datasets two images file and created training.tfrecords over 5.0kB and I aocr trained for the first time!!!

Great, I'm glad it works for you now :) Closing the issue, but please feel free to re-open or open a new one if you have more questions.


Microsoft Dynamics 365 on premise integration with Business Central Online

Hi,
Is it possible to do Microsoft Dynamics 365 on premise integration with Business Central Online using the above steps. Or this article deals with On-Premise version of both? Please clarify.
Thanks,

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

ID: 9d13606d-1b5f-8061-ddbb-65bad9f63e29
Version Independent ID: ef4faad8-2f82-cd9c-100f-1dc181db3974
Content: Preparing for Dynamics 365 for Sales for Integration - Business Central
Content Source: dev-itpro/administration/prepare-dynamics-365-for-sales-for-integration.md
Service: dynamics365-business-central
Product: dynamics-nav-2018
GitHub Login: @jswymer
Microsoft Alias: solsen

Business Central On line to Dynamics 365 Sales on premise integration is not supported due to authentication mechanism available in Business Central Online. Only supported authentication mechanism for Business Central Online to connect to Dynamics 365 Sales in through Office 365 Authentication. As you can see in this article here only supported authentication types for Dynamics 365 in on premise environment as AD, IFD and oAuth, but not Office 365.

Thanks for the feedback! We will close this issue for now. If you have any additional feedback, let us know.


difference() does not work with distinct()

Bug report
System info:

InfluxDB v1.5.1 (git: 1.5 cdae4ccde4c67c3390d8ae8a1a06bd3b4cdce5c5)
Amazon Linux 2017.09

Steps to reproduce:
The query that includes difference(distinct()) like this.
select difference(distinct("value")) from example where id = 'a' and time > now() - 1h group by time(1m) fill(previous)

Expected behavior:
Above query works.
Actual behavior:
Not works.
Additional info:
I had set this query on Grafana with InfluxDB v1.3.7 and it worked correctly with that old version. This Query ceased to work after upgrading to v1.5.1 .
Perhaps, this issue is related to #9144 ?

Thank you for your quick and polite response !


Not working. Make a much super detailed guide video in english please

Yeah also my computer deleted the files from it because it is a virus

In progress.


Fixed white image on white background

I couldn't find anything else to ship this with, so..

Hooray Jenkins reported success with all tests good!

Thanks for remembering this! I forgot and yeah that's kinda nice to have fixed with GCI started .. :D


Thumbs - recheck on change to justification text

Don't just change on blur.

I do not want to give credit for a single character of justification. Perhaps there is a useful compromise - pick some arbitrary number of characters, such as 5, and give credit if there are at least that many characters in the justification input box.


feat: modifiy ReceiptWithBloom and associated impls to use with Reth

Unblocks: https://github.com/paradigmxyz/reth/issues/12701

@mattsse As discussed, before updating ReceiptWithBloom type, I have updated TxReceipt trait and also changed the associated impls. Wdyt?

@mattsse I think we are good.

lgtm,
this should be simple to port to op-alloy, I believe

On it.


FaF Support Request - Modify eligibility check so MATs are eligible

Changes in this PR

allow users belonging to groups (MATs and SATs) to use the service

 #234 FaF Support Request - Modify eligibility check so MATs are eligible


Problem

Not working :( says

Ah drat, not CORS issues again. Thanks for bringing this to my attention, i'll investigate

Is it fixed by now? I am having quite the same issue...


thieves can now throw cans and shoot non-lethal guns

About the PR
made thieves able to throw containers with liquids in them (does not include beakers as they break on impact) and shoot non-lethal guns like disablers, practice guns, syringe guns and pneumatic cannons
Why / Balance
disablers do 0 damage now so there's no reason for thieves not to be able to fire them
space cola cans are very cool for slipping seccies
Changelog
:cl:

tweak: Thieves can now throw things with liquids in them, shoot syringe guns and disablers

Not sure about syringe gun, but disablers and disabler SMG are good

a syringe gun can be very lethal

Also, change thieves in changelog to pacifists.

a syringe gun can be very lethal

So is the hypopen, though that does at least require being in close range. Thieves being "pacifist" has always been silly, because noct hypopen + poison of choice = death.


added external custom domain support to Api construct

Added support for adding custom domains to the Api construct that are outside of Route 53.
Sample code example:
const api = new sst.Api(this, "Api", {
  customDomain: {
    domainName: "api.sst.sh",
    isExternalDomain: true,
    certificate: acm.Certificate.fromCertificateArn(
      this,
      "Certificate",
      "arn:aws:acm:us-east-1:000639264671:certificate/xxxx"
    ),
  },
  routes: {
    "GET /": "src/lambda.main",
  },
});

Thank you @Manitej66 and @fwang


Formatter undocumented deviation: No blank line added between decorated inner classes in pyi files

class Foo:
    @foo
    class Bar: ...
    @foo
    class Baz: ...

With default settings (no settings in pyproject.toml) ruff format . leaves this code unchanged. black . adds an empty line between the two inner classes. Reproduced with Ruff 0.1.6 and Black 23.11.0.
Unfortunately I can't give you a playground reproduction for Ruff since it assumes files are .py, but here's one for Black as its playground supports it: https://black.vercel.app/?version=stable&state=_Td6WFoAAATm1rRGAgAhARYAAAB0L-Wj4ACqAGFdAD2IimZxl1N_WlXnON2nzNX9HFvHx6cBLwL3Qwwh1eRAPLFFFWljgvmpOKOffkrecPNUT9AKbB5TWcHByQZl7G4uT9jIT7lav7JhqiztyvUmJIr2-kI3P8tLXaxrmogstgAAAAAANPCqLcjD5SoAAX2rAQAAACADosuxxGf7AgAAAAAEWVo=

Interesting! They don't add a newline between decorated functions, but do add a newline between decorated classes. Thanks for reporting.

I'm wondering if this is intentional or a bug in Black.

Mostly because it's not that consistent between functions and classes: https://black.vercel.app/?version=stable&state=_Td6WFoAAATm1rRGAgAhARYAAAB0L-Wj4AFyAIFdAD2IimZxl1N_WlXnON2nzNX9HFvHx6cBLwL3Qwwh1eRAPLFFFWljgvmpPX9pLqTnCMqdh3N3EyMEBEzXO-2vBynVUJMtmULtassHbhRrn1oJcaBh0ocW84sbYX4q7g1ul3plgihpMni7ANfRx9SPRrj7QHiDkG4Yihi-PnL71L5GAAAAAACFb5eQO-sQawABnQHzAgAA15WayLHEZ_sCAAAAAARZWg==


[Search & Sorting] Insertion Sort [Java]

I would like to work on this

@jenistenxavier  can i work on this?


Query on LibriSpeech Character Error Rate

Hello Sir,
I have trained the end-to-end ASR on LibriSpeech corpus using the examples provided in the repository.  However the RESULTS are provided only for the phoneme based lexicon units at the output CTC layer. I have tried to repeat the experiments for character based CTC output by modifying the scripts (as given in asr_egs/wsj/local/wsj_prepare_char_dict.sh). However, the CER obtained is ~11% , and is greater than the WER (~8.6%). Is there anything I am missing in obtaining the actual CER? It would be of great help if the RESULTS section also reports the actual CER.
I hope you do the needful.
Thanking You
With Regards
Pradeep Rangan
PhD Scholar
Indian Institute of Technology Kharagpur

The Character Error Rate (CER) is usually greater than the Word Error Rate (WER), and the character-based system will have higher error rates than the phone-based systems. I am not sure if someone has reference runs for these cases ready, and could provide them for addition to the README?

Thanks for your information sir.


Support go-to-definition on type names in inlay hints

Rust-analyzer supports go-to-definition on type names (where the type has a definition in the source) in inlay hints.
LSP 3.17 appears to support this using an optional feature called "interactive and composite labels", where InlayHint.label is an array of InlayHintLabelParts (which can have associated locations) rather than a string.
It would be nice to add similar support to clangd.

Interesting idea. Are you working on this right now? If not, perhaps I could have a try :)

Please feel free, I probably wouldn't get to this for a while :)

(Apologies for the late reply, I've been little occupied recently.
I made some modification locally and it works for certain context where type hint occurs. I just want to ensure if I'm heading in the right direction: Can we just directly migrate the type of label field, just as defined in InlayHint protocol, from 'string' to 'InlayHintLabelPart[]'? I mean we may have to concern about compatibility if clients with InlayHint support (not only vscode) is not capable to parse the field as an array but only for string. It's highly unlikely such client exists but just in case, should we do anything to keep compatibility?

It's a good question. Usually, the way this would work is that the client capabilities include an option that indicates whether the client supports the richer format.
However, since there is no such option in InlayHintClientCapabilities, and since the InlayHintLabelPart feature was part of the initial version of the inlay hints feature added to the spec in LSP 3.17, I think it means it's safe for a server to assume that a client that supports inlay hints at all supports InlayHintLabelPart as well.

I've hit another problem. IIUC, according to current LSP specification we don't have access to position information for linkable inlay hints. As a result, we may miss user's intended cursor location. For example, for class template with template parameters, we're unable to tell which declaration that user want to inspect.
auto f(auto p) { return p; }

struct C {};

void foo() {
  auto r = f(std::vector<C>()); // r: std::vector<C>
}

We could only provide one location for hint : std::vector<C>, which would either be definition of std::vector or that of C. This limitation restricts the use case for the feature and may lead to confusion for users.
Or we don't provide any clickable links on templates, at least before LSP specification provides cursor information for inlay hints.

I think the idea behind having an array of InlayHintLabelParts is that for a hint like std::vector<C>, we can have four label parts: std::vector, <, C, and >, with the std::vector and C parts having (different) locations.

So, I guess I'm in a pickle here and I'm looking for a (possible) solution.
As previously said, I'm trying to extract type hint to InlayHintLabelParts in order to make every possible class/struct jumpable. This is trivial for non-template types as we could simply put such type name into a single InlayHintLabelPart by calling QualType::getAsString(), which is currently in use. But what makes everything complicated is template types, for which we have to extract every template argument from (possibly deduced) a type and then apply locating logic on it.
Currently I'm rewriting a visiting logic(which is mostly borrowed from TypePrinter, a private class defined in a cpp file.) that attempts to inspect each parameter, find declaration, and print brackets and commas. Soon I find it hard to maintain because I have to reconsider every corner case that TypePrinter handles. And I don't think it is a promising approach which essentially is duplicating (part of) the TypePrinter.
I'm considering a workaround and here is what I've come up with:

Just rewrite the processing logic for templates, which is I'm doing now and appears to be quite hard to maintain.
Add a callback interface (similar to PPCallbacks) that would be passed into QualType::getAsString(), which intercepts  possible write logic, then we could handle these symbols separately. (Cons: Interfaces might be not that generic to handle all the cases for simplicity)
Parse the template name manually by maintaining a stack, then extract every parameter that we want to get its location. (It's strange as we've already had QualType)
Write a custom raw_ostream that intercepts every streaming request to get symbol name. (Like 2, but we could leave TypePrinter as-is)

Do you have any suggestion? Or could we resort to any other facility that provides more flexibility on type names? Thank you very much in advance :)

It would be handy if there existed a version of TypePrinter that printed the type into a virtual buffer that could be referenced via SourceLocations, and also constructed a TypeLoc representation of the type, with its SourceLocations pointing into the virtual buffer. Then, we would get most of what we want for free, by running ExplicitReferenceCollector on the TypeLoc, which would traverse the TypeLoc and identify every place in it which is a reference to some Decl. (To think about it a different way, these are the places that would get go-to-def targets if the type were printed into an appropriate context in the source code.)
Unfortunately I'm not aware of such a thing and it would probably be somewhat involved to write, so for now we may need to do something more manual.
One general thought: the solution doesn't need to be perfect / handle all edge cases. Any solution that provides links to type names in some common cases, is an improvement over the status quo. So, if it makes the task easier to "bail out" when encountering some edge cases and just fall back on printing a particular type component as a string with no links, I think that's fine.
Some thoughts on your specific options:

I think (4) may not work because raw_ostream doesn't make the needed functions virtual for overriding
(2) is probably the most powerful/flexible approach, but since it involves modifying an upstream clang API, it would need some discussion and buy-in on the LLVM Discourse first

Does the TypeVisitor interface help at all? My initial suggestion would probably be to use TypeVisitor to implement a traversal of the type that handles common cases (e.g. references, pointers, qualifiers, template specialization type) and falls back to getAsString() (with no links) for more complicated cases.

Thank you for sharing me ideas and my tentative implementation is using TemplateArgumentVisitor and TypeVisitor. I think I should take a further learning at clang CXX types, since I'm always cringed that I might mess up type printing here =)

...by running ExplicitReferenceCollector on the TypeLoc, which would traverse the TypeLoc and identify every place in it which is a reference to some Decl.

FWIW, I'm not using ExplicitReferenceCollector but locateSymbolForType from XRef.cpp to inspect declarations of symbol at the moment. I took a rough look at the collector and IIUC, the basic idea is to collect all of the type declarations once and then we could make it faster for query(i.e., generate hints for each type), right? Compared to locateSymbolForType, which is more optimal do you think? (I assume it is a trade-off as the collector may takes more space while the locateSymbolForType has a higher time complexity, though)

ExplicitReferenceCollector is not really useful for us in the absence of a way to get a TypeLoc for the type, because it operates on AST nodes that have associated SourceLocations, including Expr and Decl and TypeLoc (but not plain Type). In the hypothetical scenario where we have a TypeLoc, the utility of ExplicitReferenceCollector is that it could do the work of identifying the places in the type string that refer to some declaration (as well as getting those declarations), in essence replacing the need for new code to traverse over the type.
For the use case of getting a target declaration for a Type, locateSymbolForType is a good choice.

@zyn0217 what do you think about making some incremental progress here by starting with a simple initial change, e.g. always producing just one InlayHintLabelPart for a type hint which takes you to a single target location (e.g. for vector<C> it might be <vector>)?

Ah, sure! Let me sort out the patch. And sorry for procrastinating...

Just have sent out a patch of the protocol part to reduce the review workloads. This part looks separated and backward compatible.

So, I have rewritten most of the logic (the previous implementation was sadly lost :( ), and the drafts are now on this branch. This is going to support links on template-type arguments for hopefully most common cases e.g.
                              vvvv   No link on basic types
std::map<MyClass, std::vector<bool>>
                  ^^^^^^^^^^^  link to the ClassTemplateSpecializationDecl
         ^^^^^^^ link to the CXXRecordDecl


/subreddits/mine/subscriber

https://www.reddit.com/dev/api#GET_subreddits_mine_subscriber

32cc5d1fd152026231d3397207bb2325145d6015


define types for a Progressive Audio Presentation

[ ] find types in the legacy data (see #1) that are general to all player presentation and define them

deriving the modern Presentation from the legacy JSON
The following table summarizes how the legacy JSON Presentation maps to new, needed types:

legacy part
maps to what?

Segment
Id [GitHub ], Title [GitHub ], Temporal [GitHub ]

LayoutMetadata
This is a big one!

Description Fragment
type Description = ┃ Description of DisplayText

Credits Fragment
(Role of DisplayText, Name of DisplayText)

Copyright Fragment
type Copyright = ┃ Copyright of DisplayText

playlist convention
(Title, Uri) list

intro-URI convention
type IntroUri = ┃ IntroUri of Uri


TimedeltaIndex construction is deprecated.

Fixed warning TimedeltaIndex construction is deprecated and will be removed in a future version.

If addressing an issue, link to it.

PR addresses #1820

#1820 is a different problem. This PR fixes #1837

On futher inspection, dev branch already has fixed.


Not working after computer start

Environment
Microsoft Windows [Version 10.0.18362.778]
PowerToys version:
v0.17.0

Steps to reproduce
Start computer
Expected behavior
When I start my computer I expect that windows would snap to the zones I defined in FancyZones.
I  also expect that I would be able to enter the settings of PowerToys.
Actual behavior
When I start my computer FancyZones is not working.
I see the PowerToys icon in the taskbar, but cannot access the Settings, seems unresponsive.
When I trry to run PowerToys from the Start menu nothing happens.
To get it to work I have to open the Task Manager, stop PowerToys Runner, and then I can start it from the Start menu
Screenshots

Same issue here since v0.17.0, everything was working as intended before that.

I am having the same problem with v0.17.0. Same fix as author said. Close and reopen PowerToys and it works again.

Hi @fundaypreben @Weyrlen @RABIGA7OR
do you have virtual desktops active when rebooting the systems?
If yes, can you test if closing all the virtual desktops before rebooting make any difference?
Thank you.

No virtual desktops here

@Weyrlen
if for you it's acceptable to install an unsigned version of PowerToys, can you try this one https://github.com/microsoft/PowerToys/files/4565289/PowerToysSetup-0.17.1-x64.zip
It was prepared for a different issue, but I wonder if it may also solve this problem.
Thank you.

I used to have a virtual Desktop but I have none currently.

Just installed it and it seems to work now.
Since even after restarting PowerToys it crashed randomly after some time, I'll let you know later if something happens again or if this version totaly solved the problem.
Thx @enricogior

@RABIGA7OR @fundaypreben
feel free to give it a try to the test build.

@enricogior
Yes this build fixed it. I have to grant permission to launch every time now but I guess that's because it's unsigned. Thanks for the help. I assume the fix will make it in the next official version?

@RABIGA7OR

I have to grant permission to launch every time now but I guess that's because it's unsigned

Try to turn off the the auto start and then on again, it may fix it.

@enricogior
Everything is working perferctly since 0.17.1.
Thx again !

@Weyrlen
good to know, thanks.

Is there an estimated release date for this fix?

@enricogior did this get added into 0.18?

@crutkas the fix will be in 0.18.

The fix is now available in 0.18 https://github.com/microsoft/PowerToys/releases

This still happens to me in 0.25.0

@pexpexpexy please open a new issue with logs so we can diagnose the issue


Subtracting two MayaDT objects gives incorrect delta.

print(maya.when('tomorrow') - maya.when('today'))

Prints out:
86400 days, 0:00:00

To fix this, in the MayaDT.subtract_date method, specify that you're setting the seconds of the timedelta object you create:
    def subtract_date(self, **kwargs):
        """Returns a timedelta object with the duration between the dates"""
        return timedelta(seconds=self.epoch - kwargs['dt'].epoch)

This will make at least one of your tests fail (same fix should be applied to it, pass the calculated seconds in as seconds:
@pytest.mark.usefixtures("frozen_now")
def test_mayaDT_sub():
    now = maya.now()
    then = now.add(days=1)
    assert then - now == timedelta(seconds=24 * 60 * 60)
    assert now - then == timedelta(seconds=-24 * 60 * 60)

Why not submit a PR if you've already written out the fix?

Thanks for the contribution! I've fixed it in 8468dd2
@jgaines you are more than welcome to submit a PR next time :beers:

Heh, sorry, end of a long day didn't have the steam left to do it proper. ;)

No problem :) :tada:


Add collision object to the demo

Description
Please explain the changes you made, including a reference to the related issue if applicable
Checklist

[ ] Required by CI: Code is auto formatted using clang-format
[ ] Extended the tutorials / documentation, if necessary reference
[ ] Include a screenshot if changing a GUI
[ ] Document API changes relevant to the user in the moveit/MIGRATION.md notes
[ ] Created tests, which fail without this PR reference
[ ] Decide if this should be cherry-picked to other current ROS branches
[ ] While waiting for someone to review your request, please consider reviewing another open pull request to support the maintainers

Thanks! Added a fix and cherry-picked directly to the target branch, so I'm closing this


Fix issue where very-long message gets cut off due to limited buffer size

Issue details in #80.

[X] change transport_readline function signature to allow retain newline during read
[x] check newline when reading and expand buffer if no newline found

@alistairking your comments should be addressed now. Could you take a another sweep? Thanks

This issue is caused by hex string parsing error, rather than buffer size. The issue should be addressed in d17ae2e099f1e74de41ac444623e9e618ab242da.


Can I get video thumbnail?

Can I get video thumbnail?
To get thumbnail, I used "react-native-create-thumbnail". but It is a little cumbersome.
I want to get thumbnail only using " react-native-community".

related to #168 , not closing as duplicate because video might need a specific implementation compared to images


Create event page

Basic Information
An event manager can click on an event in the dashboard to open up the event in more detail. This can be dynamically routed, or a modal.
It supports assigning volunteers to an event, and items to be repaired in the session.
Other Information
[Are there any additional information that we should know about?]

Partially resolved in #41


Add CastXML Travis build

To prepare for the migration to CastXML as being the default xml parser, we need a Travis build running CastXML.
Once solution would be to installed castxml like I did with gccxml, but the debian package is currently only available in Ubuntu Wily.
Castxml is now available in homebrew, so that seems to be fastest solution. The https://github.com/gccxml/pygccxml/tree/TravisCastXML branch is already up and running; but castxml currently crashes. Once this is cleared we should have a build to start with.
I could also download and use the precompiled castxml we are using in ITK (either on Mac or Linux). Let's see what works out best.

I download the binaries from Kitware in the .travis.yml for ompl: https://github.com/ompl/ompl/blob/master/.travis.yml
That seems to work.

Hey, super cool.
I think I will use that until I get the homebrew package to work on Travis. Thanks for the hint.


Bump all packages and actions

major changes:

python 3.11
xarray 2023.5.0
jupyterbook 0.15.2

All green! FYI In order to lock the environment for osx-64 I had to use conda 22.9 (https://github.com/conda/conda-lock/issues/408),,, seems like it'll be fixed for next release of conda.

for the sake of those that has the new Mac M1/M2 machines

I thought in theory the lock for -p osx-arm64 was for these fancy new machines :)
https://github.com/xarray-contrib/xarray-tutorial/blob/3eadc8afa59e9704d6a4cf81726fe80f459bffca/conda/README.md?plain=1#L16
@lsetiawan from the conda folder can you try conda-lock install -n xarray-tutorial or micromamba create -n xarray-tutorial -f conda-lock.yml I definitely think we should push people towards a binderhub for the scipy tutorial to avoid these complications.

Ah. I haven't used much of conda lock and it's Friday afternoon. I guess that's my issue 😉 Sounds good. I agree in pushing people towards binderhub. Thanks for the explanation 😄


[FIX] Update font awesome icons

resolves #38

updates the docs about FA usage
upgrade FA link in html_css_files to v6.1.1

Notes:
FA v5 seems to have had some "growing pains" in which the fa CSS class might have been required to set the font-family property. It'd probably be best to avoid using FA v5.

Codecov Report

Merging #64 (3b063ad) into main (2cdadca) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #64   +/-   ##
=======================================
  Coverage   88.90%   88.90%
=======================================
  Files          10       10
  Lines         910      910
=======================================
  Hits          809      809
  Misses        101      101

Flag
Coverage Δ

pytests
88.90% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out 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 2cdadca...3b063ad. Read the comment docs.


Add IcebergTableTest unit test

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[ ] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-1707 - "Add Iceberg support to DistCp"

Description

[ ] Here are some details about my PR, including screenshots (if applicable):

Add IcebergTableTest unit test.  This was promised earlier with https://github.com/apache/gobblin/pull/3559
Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Unit test included
Commits

[ ] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Codecov Report

Merging #3564 (7211ae0) into master (5ae4ec7) will decrease coverage by 3.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3564      +/-   ##
============================================
- Coverage     46.77%   43.75%   -3.03%
+ Complexity    10512     2054    -8458
============================================
  Files          2099      406    -1693
  Lines         81988    17528   -64460
  Branches       9132     2142    -6990
============================================
- Hits          38349     7669   -30680
+ Misses        40096     9005   -31091
+ Partials       3543      854    -2689

Impacted Files
Coverage Δ

...a/org/apache/gobblin/cluster/GobblinHelixTask.java
60.21% <0.00%> (-6.46%)
:arrow_down:

...tadata/provider/SimpleMetadataProviderFactory.java

...a/org/apache/gobblin/password/PasswordManager.java

...ogle/webmaster/GoogleWebmasterDataFetcherImpl.java

...ata/management/copy/converter/UnGzipConverter.java

...t/version/FileStatusTimestampedDatasetVersion.java

...nstrumented/writer/InstrumentedDataWriterBase.java

.../main/java/org/apache/gobblin/dataset/Dataset.java

...obblin/yarn/event/DelegationTokenUpdatedEvent.java

...orkunit/packer/KafkaSingleLevelWorkUnitPacker.java

... and 1685 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Show error (Challenge Required)

Give me screenshot

trying to follow celebgram   error ???
ITS THE SAME INSHACKLE tool you just changed the name and the banner and some celeb ID -.-


Bug shorthand :disabled in mode production

Sorry my english is not good.
Please preview video.

and source code test here https://github.com/tronghiep92/vue-bug

Hmm, I cloned your repo but cannot reproduce it... can you do a fresh npm install and see if this still occurs?

I was able to reproduce it with npm 2.11.2, browserify 6.1.0.

Same here. npm 3.8.3, browserify 3.39.0. The attribute value got stripped off:
;(typeof module.exports === "function"? module.exports.options: module.exports).template = "<button :disabled>Test D

html-minifier trims colon'ed boolean attributes. Config taken from lib/compiler.js#L20.
Test case: https://tonicdev.com/56ceae9cf07e650c00ae128d/5704bf946ea4ea1100b4b24f

First finding shows that this is a problem/config issue with html-minifier, more specifically: https://github.com/vuejs/vueify/blob/master/lib%2Fcompiler.js#L240. I believe I had experienced something similar with that lib in the past. Will explore more.

Oops, @simplesmiler is one step ahead of me :D

@yyx990803 Because I use Windows should install cross-env.
Fix it install: npm install -g cross-env and try again.
with Linux i think NODE_ENV=production browserify -t vueify -e src/main.js -o build.js

@tronghiep92 Not a problem. cross-env should work, duh, cross-env.

Oh well, this seems to be a regression on html-minifier's end... it was working as intended with 1.2, after updating to 1.4 it is showing this behavior.
On the other hand - 1.4 can now properly parse Vue's shorthand attributes, so the custom surround is no longer necessary. Simply removing that will work as intended: https://github.com/vuejs/vueify/blob/master/lib/compiler.js#L21

That's great to know :+1:  I'll go ahead and close the PR then.


MP3 playback clipping.

MP3 playback using the mp3 music example seems to be clipping something like a second of audio from the end of the audio stream. This is particularly apparent on short audio clips, with a 1 second duration MP3 not outputting any sound and a 2 second duration MP3 cutting out midway through playback.
This is on Windows 10 x64 using rodio 0.8.1.
examples.zip

Could you file an issue in the minimp3-rs crate? The following line might return an early EOF
https://github.com/germangb/minimp3-rs/blob/master/src/lib.rs#L78

Seems the minimp3 crate is returning an early EOF. Could you file a bug in https://github.com/germangb/minimp3-rs/issues ?
Thanks!


Issue with spacing between buttons in symptoms dialog

This will be fixed when we move creating symptoms to managesymptoms dialog instead.


[BUG] Servicelist Detail page does not show rejected applicants.

Describe the bug
Servicelist Detail page does not show rejected applicants.
What Did You Want
In the Servicelist Detail page, I wanted to list all the applicants to the service. Approved and Pending appliacations are listed as expected, but rejected applicants were not shown in the applicants table.
To Reproduce
Steps to reproduce the behavior:

Create a service with a user (creater).
Apply to this service with another user (applicant).
Login with creater and reject the application of the applicant.
Login as admin and go to Admindashboard-Services.
Find the service with rejected application and click to open detail.
See that the rejected application is missing.

Expected behavior
In the Applicants table of the Servicelist Detail page, all applicants should be seen with application status.
Result behavior
No error message. Missing item.
Screenshots
Desktop (please complete the following information):

OS: Windows
Browser: Chrome
Version: -

Additional context

isDeleted implementation is done for the models and views are changed accordingly.

Bug is fixed and can be tested via bug/68_73_74_92_93_100


[CT-3304] [Bug] Unclear error messaging returned on Snowflake quoting related syntax error

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
Sean at Flowcode reached out as they were seeing the following error message in their jobs:
14:26:26  on-run-end failed, error:
14:26:26   100069 (22P02): Error parsing JSON: missing comma, pos 124

The issue, however, was actually around some quoting they used in a .yml file. They’re using the Snowflake adapter (which is finicky about quoting). Please note that this user has quoting enabled for identifiers.
Expected Behavior
I would expect an error message around quoting on the specific line in the .yml file to be returned, rather than an error message referencing an entirely different part of the project (the on-run-end).
Steps To Reproduce
I haven't had time to reproduce this yet, but my assumption is that you would need to create a dbt project that uses the Snowflake adapter, has quoting enabled for identifiers, and has a sources.yml file with a freshness config that includes a config that uses double quotes. The customer had this in their project:
- name: flow_directory
        freshness:
          warn_after:
            count: 6
            period: hour
          error_after:
            count: 24
            period: hour
          filter: datediff('day', "DMS_S3_SYNCED_AT"::timestamp, current_timestamp) < 2
        loaded_at_field: "DMS_S3_SYNCED_AT"

The issue was caused by the double quotes around DMS_S3_SYNCED_AT.
Relevant log output
https://dbtlabsmt.datadoghq.com/logs?query=%40extra.run_id%3A211284387%2A%20&cols=host%2Cservice&event=AgAAAYuGIO6Ti1esowAAAAAAAAAYAAAAAEFZdUdJUF80QUFBUkxVVzRiMG5LdWdBSgAAACQAAAAAMDE4Yjg2MmQtMGE4Yi00MjliLTg1YWUtOTI4MjMxZmJhNzc1&index=%2A&messageDisplay=inline&refresh_mode=sliding&stream_sort=desc&viz=stream&from_ts=1698591902623&to_ts=1698851102623&live=true

Environment
- OS: This was surfaced by a dbt Cloud user.
- Python: NA
- dbt: 1.5

Which database adapter are you using with dbt?
snowflake
Additional Context
No response

Thanks for reporting this @sophiad96 !
The tough thing here would be tracing the database error all the way backwards to the YAML config that caused it. I'm not aware of a reasonable way we can do that currently.
One thing that may or may not be possible is linking the error message back to "source freshness" somehow so at least the user knows how to narrow it down a little bit better. But I'm sure if that is even reasonable or not.
It doesn't sound dbt is not behaving erroneously here, but rather not as helpful as we hope. So I'm going to re-categorize this as a feature request rather than a bug.
Marking this a "Refinement" for us to take a closer look to see if there's anything we can do or not.


Html data

Preparing to merge the htmlData branch.
The transform for creating web views of UW Sinopia data is at a fairly good stopping point, although to-dos do still exist, these will be documented for possible follow-up later.

Wow.
That was really ridiculously easy and all-GUI...
Even gave me a handy "delete branch" button once the merge was completed.


Add notification to the player when using the legacy item

Player using a legacy item should be notified of it being functionnal but deprecated, and soon-to-be-removed.
Actual state : Functionnal, no notif
Todo : Functionnal, notif
Deprecated : Non functionnal, notif
Final state : no legacy item left

This issue is linked to the radio items in addon gsri_radio only.

Fixed in 45705e7


Remove unused import

Class org.springframework.stereotype.Component is no more used.

I actually quite like using @Component for that (see aa1f874).


[FSDE-15209] Order progress info popover is displayed under content when positioned bottom

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-344/1/. :white_check_mark:

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-344/2/. :white_check_mark:

You can check this PRs instance in https://1414c7ff24661e0717d4a7bf94cfd3d162fd67f5-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-344-ux-chi.kubeodc-test.corp.intranet/ (internal)


Boot in subdirectory

These changes allow a userdev to boot a development image in a subdirectory of Dolphin's root directory. Now all application development activity can be isolated from coredev activity.

I'm investigating an alternative of updating the package paths on startup if the image has changed.

Be careful you don't build in anything that rules out Workflows A & B. That is, don't assume that the core sources always reside in the same folder as Dolphin7.exe. I think this needs to be optional. The idea discussed on Gitter where we have an option to create a new app directory might have legs. e.g. something like:

dolphin7 DPRO -init \MyHome\MyApp -sharecore

Might copy DPRO.img7 to \MyHome\MyApp\MyApp.img7 and somehow configure it to refer to the shared core packages in the original location. BTW, I don't necessarily like these option names.. just food for thought.

Based on concerns raised I'll withdraw this as we investigate alternate developer workflows.


Deployment of k8s cluster failing when using bastion host and Openstack driver

Bug
When creating a new cluster using Openstack driver with a bastion host, after provisioning the nodes on openstack rancher fails to connect to the nodes and cluster state is Provisioning: Waiting for etcd and controlplane nodes to be registered. In my understanding rancher doesn't take into account the bastion host as it tries to directly connect to the k8s-nodes which are only accessible via the bastion host.
Other important factoids:

Using the openstack driver i can deploy nodes and k8s clusters without an issue when not using a bastion host.
I can connect from the rancher node to the bastion and then to the target node without an issue
Security groups and ssh configs are set per rancher documentation. They are working as the next notes indicate
from the rancher node, i was able to deploy (after provisioning the nodes myself) a k8s cluster using rke and the bastion host, flow is correct etc
I was also able from rancher to create a cluster using existing nodes via the same bastion host. Nodes created manually etc.

Relevant logs:
From rancher-server logs
2021/02/08 11:36:01 [ERROR] error syncing 'c-bghd4/m-zhdqz': handler node-controller: Error creating machine: Error detecting OS: Too many retries waiting for SSH to be available.  Last error: Maximum number of retries (60) exceeded, requeuing

and
2021/02/08 11:35:46 [DEBUG] keyPath not found. The node is probably still provisioning. Sleep 1 second
2021/02/08 11:35:47 [DEBUG] keyPath not found. The node is probably still provisioning. Sleep 2 second
2021/02/08 11:35:49 [DEBUG] keyPath not found. The node is probably still provisioning. Sleep 4 second
2021/02/08 11:35:53 [DEBUG] keyPath not found. The node is probably still provisioning. Sleep 8 second

I'm not defining ssh_key_path on the cluster_config, but i pass the private key directly (ssh_key). I used the same config when i tried directly with rke and also when importing nodes and it worked.
From tcpdump on the rancher node:
sudo tcpdump -n -vvv -i ens3 dst port 22
tcpdump: listening on ens3, link-type EN10MB (Ethernet), capture size 262144 bytes
12:30:10.023769 IP (tos 0x0, ttl 63, id 7912, offset 0, flags [DF], proto TCP (6), length 60)
    <IP_ADDRESS>.59946 > <IP_ADDRESS>.22: Flags [S], cksum 0xd8fe (incorrect -> 0x63cf), seq<PHONE_NUMBER>, win 64240, options [mss 1460,sackOK,TS val 330017407 ecr 0,nop,wscale 7], length 0
12:30:12.039775 IP (tos 0x0, ttl 63, id 7913, offset 0, flags [DF], proto TCP (6), length 60)
    <IP_ADDRESS>.59946 > <IP_ADDRESS>.22: Flags [S], cksum 0xd8fe (incorrect -> 0x5bef), seq<PHONE_NUMBER>, win 64240, options [mss 1460,sackOK,TS val 330019423 ecr 0,nop,wscale 7], length 0
12:30:13.040447 IP (tos 0x0, ttl 63, id 38431, offset 0, flags [DF], proto TCP (6), length 60)

The target IP's are the IPs of the k8s nodes that are not accesible from rancher, only through the bastion host. On the bastion host i don't see any traffic flowing at all, neither on the k8s nodes. To me that indicates that for some reason the bastion flag is not taken into account. When deploying with rke and/or importing nodes i could see that the traffic was going through the bastion host as expected.
Steps to reproduce (least amount of steps as possible):

Enable the openstack driver
On openstack create necessary resources such as security group, keypair, private network & subnet, bastion host that is accesible from rancher and has access to the private network.
Create Openstack node template with the necessary info
Create new cluster using the created openstack node template and edit yaml to add the bastion_host snippet

Result:
Environment information

Rancher version:

Rancher | v2.5.5
User Interface | v2.5.4
Helm | v2.16.8-rancher1
Machine | v0.15.0-rancher49

Installation option (single install/HA): Single install using option A found here as we are evaluating rancher atm.

@nikParasyr did you managed to solve it ? I'm facing the same behavior with an aws pub/priv net setup and the ssh doens not seems to go trough bastion either.

@fredleger no i wasn't. It's been a while since then but i think from discussing on the rancher slack channel this seems to not be supported on rancher for any provider.

that was my guess too but the docs are not clear about it. Thats so sad then

Hello !
We are facing the exact same issue, which means rancher can't deploy a remote cluster using RKE in our environment.
We may not be using RKE due to this issue, no plan to get it fixed ?

hi @3fhtVrY1VzdKYCrJzGtgY71Nt the issue has been arround for a while and if am not wrong it's not only true for openstack. There was a gloabl issue thta was closed so i don't think there will be a fix for RKE 1 especially that RKE2 is planned now.

From what I understand it's related to https://github.com/rancher/machine/blob/master/libmachine/ssh/client.go which try to connect directly to the private IP of the node, but the client does not support SSH ProxyJump or ProxyCommand but I believe it should be possible to make it works.

no i wasn't. It's been a while since then but i think from discussing on the rancher slack channel this seems to not be supported on rancher for any provider.

We had this issue as well while trying to use a bastion host to provision an RKE cluster on a vSphere provider. Rancher support confirmed that it is not supported to use a bastion host when deploying from Rancher.


Add visualization of parent hierarchy [low priority]

Add an optional visualization for the parent hierarchy of atom types. Parent dictionary should be able to be used to recreate this tree. Here is a conda installable module that might be useful:
http://etetoolkit.org/docs/2.3/tutorial/tutorial_drawing.html
This is a low priority for now, but will be helpful in the future.

For the record, it may be helpful soon to have this along with the "score versus time" plots in #78 in order to understand why the score is changing when it does, at least for selected transitions. Clearly once we move to the ZINC parm@frosst set it's going to be very tricky to extract any useful insight from just browsing the log file, so being able to visualize the tree at points throughout the trajectory may be helpful.

I have this as a todo on my list right now, because I think we would like to have nice figures of these for the paper. I'll likely add whatever I come up with to the score_utils methods

I really like the style of the color wheel/tree just below the header "Node backgrounds"; not sure if it would work here, but I'm imagining something where the wheel is colored by element and different legs are labeled with their SMARTS/SMIRKS could be really cool if it could be made to work.

I'm working on this currently, I'm not sure how easy it will be to automate this process, but I have an ipython notebook generally showing how to do this in branch issue46 which has a work in progress pull request #214

I have some examples from this that I should put in utilities so other people can use it. I'll add cleaning up this notebook to my to do list and add it here.

The conda-installable networkx package can make circular tree plots, though I also came across a really nice example for making circular trees with d3.js using Python.

That is nice too, I used ete2 (linked above) to make the color coded circular plot David had in his ACS talk


PEP 541: transfer "jinja" to the "jinja2" project

https://pypi.org/project/Jinja/ had its last release in 2007 and https://pypi.org/project/jinja2/ has been actively developed since 2008 (last release in April 2019).
https://github.com/pallets/jinja/issues/752

Both those projects are owned by @mitsuhiko.  What is there to transfer?


Empower individual engineers to validate and test their changes

Only a few engineers on our core team are capable of deploying QA-testnets and other infrastructure necessary to test new features or bugfixes. This is a bottleneck that we aim to remove.
Friction points:

Infrastructure information is in a separate repository and cannot update in lockstep nor CI along with Coda's changing repo.
There is not enough knowledge shared around how to do this properly
There is a belief that if CI passes it's good to merge
Change culture around what is required to test a protocol or product change
We have many config.mlh profiles (most of which we don't use or can obsolete when we finish moving over to runtime genesis) -- hard to know which one is right for the task at hand

Potential ideas:

Discontinuous integration:

If I force push to env/staging1, automatically redeploy a new network with my configuration
An easy "one-click" deploy a staging network (in CI?)

Share knowledge and creds and culture
Config.mlh culling and renaming of profiles

Exit Criteria:

. >50% of protocol team and >50% of product team has successfully deployed QA-net-infrastructure without help
Solution for sharing keys (covered in #4722 )
Configuration of networks requires minimal knowledge of infra code -- at the most requiring just 1 or 2 top-level files being touched (covered in #4723)
Minimal config.mlhs, all unnecessary ones removed, files named properly (covered in #4756)

Relates to #4723

Enveloped by #4723


Can't retrieve Approval Processes!

Hi,
Thank you andrew, for stepping forward to help!
My use case is to retrieve Approval Processes from another organization. For this, I genarated my own MetadataAPI wsdl and MetadataService class for version 33. But still, I could not retrieve, or even list Approval Processes of any organization.
Could I request you to look into this since it's very urgent ?
Many Thanks!

I'm afraid i cannot give an guarantees on when i can look into this, this is an open source community project and as such subject to the availability of those that contribute to it. I do have on my list a task to update the library to API 33 and in doing so will for sure try your use case.
In the meantime is there any reason why you didn't try with the MetadataService.cls in this repository, this class has been enhanced to fix some issues the default one generated by the platform has, such as issues with the retrieveMetadata operation.

Here is an example of reading an Approval Process, please make sure to use the latest MetadataServices.cls from this repo.

Thank you Andrew, I really do appreciate your help.
2015-04-08 16:53 GMT+00:00 Andrew Fawcett<EMAIL_ADDRESS>
Closed #83 https://github.com/financialforcedev/apex-mdapi/issues/83.
—
Reply to this email directly or view it on GitHub
https://github.com/financialforcedev/apex-mdapi/issues/83#event-276414100
.

--
*Sara ELALAOUI *
Élève Ingénieur en Génie Informatique École Nationale des Sciences
Appliquées de Safi**Université Cadi Ayad - Maroc


Add skeleton

[x] Use schadcn Skeleton
[x] Should the whole page use Skeleton?

We want to use server side rendering, and putting Skeleton on the whole website would be a miss of Server Side Rendering.
[ ] Spend at least 1 Story point fixing Hydrations error, Issue #15

Moved until we have real data that we fetch. The skeleton should be only used for async data, server side rendering should not be reason to add skeleton


Exception not handled on SerpApiClient.get_json

I am experiencing unexpected behaviors when getting thousands of queries. For some reason, sometimes the API returns an empty response. It happens at random (1 time out of 10000 perhaps).
When this situation happens, the method SerpApiClient.get_json does not handle the empty response. In consecuence, the json.loads() raises an exception causing a JSONDecodeError.
I attach an image to clarify the issue.

It seems a problem with the API service. Not sure if the problem should be solved with an Exception handling, handling the code 204 (empty response), or if there is any bug with servers.
to reproduce the exception:
import json json.loads('')
Do you recommend any guidelines to handle the problem in the meanwhile you review the issue on the source code?
Thanks.

Thanks for raising this question.
Please share an extra information so we can investigate this issue:

A search ID of your successful searches
Search parameters which result into getting an empty response from serpapi.com


中文翻译更新

如题，crowdin 中的中文翻译已100%，等待更新

Hi @Tim-Paik thanks for the contribution! Please use English in GitHub issues.

@Tim-Paik also can you fix the wrong English text you mentioned in CrowdIn (by making PR in this repo)?