Keras model save/load is totally wrong with sessions

Bug report: Keras model.save() does not work with sessions
Actually, it only works with the default session.
I know tf.keras is mainly targetting at TF 2.0, but it really doesn't play nice with conventional TF-1.x ways (e.g. Session). I encountered this bug and nailed down why it happens after a day. Arguably, I believe this is a severe bug and hard to realize when one encounters. This is also somewhat related to #26430.
In summary (the current behavior): if one is using a custom session created (say MonitoredSession, as it was a conventional way of opening and sessions in TF 1.x way), Keras model's load/save will be COMPLETELY not working.
System information

OS Platform and Distribution: Linux, but agnostic to Platform
TensorFlow installed from (source or binary): binary, pypi
TensorFlow version (use command below): 1.13.1
Python version: 3.6

Code to reproduce the issue
Please see this notebook:
https://gist.github.com/wookayin/0712261896799e0f655df50e910055df
Or in a code:
import tensorflow as tf

class MyModel(tf.keras.models.Model):
    def __init__(self):
        super(MyModel, self).__init__()
        self.layer = tf.keras.layers.Dense(5)

    def __call__(self, x):
        return self.layer(x)

model = MyModel()

# Build the model
model(tf.zeros([3, 10]))

# this is something like a train_op that "changes" the content of the variable.
assign_op = model.variables[1].assign( tf.ones_like(model.variables[1]) )

# Let's create a session
sess = tf.train.SingularMonitoredSession()
#sess.run(tf.global_variables_initializer())

# we can see the 'bias' variable is initialized to ZERO
assert sess.run(model.variables[1]).mean() == 0.0

# Now let's make the 'bias' variable to all one...
sess.run(assign_op)

# sure it is ONE ...
assert sess.run(model.variables[1]).mean() == 1.0

# Let's save the Keras model parameter. The bias is set to ONE, right ?????
# Since model.save_weights try to create a new op (another bug #26430)
# and the graph has been finalized, we will 'unfinalize' the graph with a bit of hack
sess.graph._finalized = False
model.save_weights('/tmp/keras-one.h5')
sess.graph.finalize()

# Let's see what is stored in the model file ....
import h5py
h = h5py.File("/tmp/keras-one.h5")
assert h['dense']['dense']['bias:0'].value.mean() == 1.0     # <------ This will fail
# Actual output is: array([0., 0., 0., 0., 0.], dtype=float32)

The above code does the following:

Build a Keras model
Initialize the layer (bias will be zero)
Update the trainable variable (e.g. bias: 0->1)
Save the model into a H5 File
Read the model file, but the bias variable is not 1 but 0 (same as the initialization)

Additional Information

tf.Session() behaves exactly the same as with tf.train.MonitoredSession(). In this case we need to run custom variable initializer ops.
However, when tf.InteractiveSession() is used, it works exactly as expected (the readout from keras-one.h5 is 1.0)
It would work well in the eager mode.

A work-around
If we change the code as follows, it would work.
with sess.as_default():
    model.save_weights('/tmp/keras-one.h5')

A quick note: 'SingularMonitoredSession' object has no attribute 'as_default'
This is because Keras only plays with the default session (see #26430 as well) and in this example the manually created session was never registered as default. Kinda makes sense as there is no way for model.save() function to be aware of the session... However, this leads to a very inconsistent and unexpected behavior. If no session is registered as default, model.save() should instead throw an error.

Any feedbacks? I think we should make tf.Keras NOT use its own default session (_SESSION) when integrated to TF core. However, it can also break many things that have been relying on this implicit Keras session.

This is an most important problem. Please check it .

@fchollet @ymodak Can we revisit this issue? As TF 1.x is expecting the last release soon, I hope this bug should be fixed.

If this is not fixed quite soon, one should at least introduce a warning and some documentation about this behavior, this was quite unexpected for me and wasted a few trainings.

Has this been solved for keras 2.x? I'm using 2.3.1 and I can still see this problem.
I'm using Jupyter notebooks. In one notebook, I'm saving a model using model.save() and later loading the model using load_model() and it works perfectly fine.
But when I try to open the same saved model from a different notebook, I get very random accuracy and predictions. From what I understand, the weights are getting reinitialized randomly. What is the workaround for this? Urgent.

@aashnajena Make sure the session you are using is registered as a "default" session. I already have put a code snippet for the workaround.

Hi There,
We are checking to see if you still need help on this, as you are using an older version of tensorflow which is officially considered end of life . We recommend that you upgrade to the latest 2.x version and let us know if the issue still persists in newer versions. Please open a new issue for any help you need against 2.x, and we will get you the right help.
This issue will be closed automatically 7 days from now. If you still need help with this issue, please provide us with more information.

This still needs to be fixed in 2.x.

@wookayin,
Can someone provide a reproducible code in Tensorflow 2.x so that it can be looked into? Thanks!


[Builtins] Inline SatInt stuff and costing monoids

Just something I noticed in the Core, needs benchmarking, so don't look here yet.

/benchmark validation

^ Just experimenting to see if running the benchmarks here uses the same machine that we use manually.  Looks like they do.  I've cancelled the benchmarking job now.

/benchmark plutus-benchmark:validation

/benchmark validation

/benchmark validation

/benchmark nofib


EditorPaths: Fix creating 'export_templates' folder.

Follow-up to #63093.

Another option is to use EditorSettings::get_singleton()->get_export_templates_dir() here.
Actually, now I'm wondering if EditorPaths should be responsible for having the path to the export templates dir. Since, you know, it's an editor path.

Yeah both would probably make sense. Feel free to refactor this further.


[Impeller] Engine gets confused when enable_impeller and enable_software_rendering are both true.

Steps:
Create an app that displays an image.
Open a simulator.
Run flutter run --enable-software rendering.
The image will fail to render.
This has the same root cause as https://github.com/flutter/flutter/issues/126768#issuecomment-1558165539
IMHO we need the following:

Better runtime behavior when enable_impeller is true but impeller can't/won't be enabled (because no valid MTLDevice is available or because the user explicitly requested software rendering).
Better behavior in the tool when a user requests both impeller (implicitly or explicitly) and software rendering (explicitly).

After @jason-simmons' patch, at least we get a better error message now:

══╡ EXCEPTION CAUGHT BY IMAGE RESOURCE SERVICE ╞════════════════════════════════════════════════════
The following _Exception was thrown resolving an image frame:
Exception: No Impeller context is available

When the exception was thrown, this was the stack

Image provider: AssetImage(bundle: null, name: "csl.webp")
Image key: AssetBundleImageKey(bundle: PlatformAssetBundle#e566b(), name: "csl.webp", scale: 1.0)
════════════════════════════════════════════════════════════════════════════════════════════════════

I also get Exception: No Impeller context is available when running tests with the simple command flutter test integration_test/main.dart -r expanded, so with impeller enabled but no software rendering as far as I am aware (however the emulator runs in github CI, so it could be that the emulator forces software rendering on impeller?)

Make sure you are using a recent macOS image - older github CI actions don't have GPU passthrough enabled.

We are using macos-latest which is 12.6.6, I also cant find anything on github CI actions having GPU pass-through at all though.

I'm using CircleCI wit xcode 13.3.0 and also getting
Exception: No Impeller context is available

I see the same thing with latest flutter-dart versions running on Ventura.  It stopped working when I recently upgraded to latest MacOS
XCode: Version 14.3.1 (14E300c)
Dart:  Dart 3.0.6 (stable) (Tue Jul 11 18:49:07 2023 +0000) on "macos_x64"
flutter doctor
[✓] Flutter (Channel stable, 3.10.6, on macOS 13.4.1 22F770820d darwin-x64, locale en-US)
[!] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
✗ cmdline-tools component is missing
Run path/to/sdkmanager --install "cmdline-tools;latest"
See https://developer.android.com/studio/command-line for more details.
✗ Android license status unknown.
Run flutter doctor --android-licenses to accept the SDK licenses.
See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.
[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.1)
[✓] VS Code (version 1.80.0)
[✓] Connected device (3 available)
[✓] Network resources
flutter pub outdated
Showing outdated packages.
[*] indicates versions that are not the latest available.
Package Name  Current  Upgradable  Resolvable  Latest
direct dependencies:
intl          *0.18.0  *0.18.0     *0.18.0     0.18.1
dev_dependencies: all up-to-date.
You are already using the newest resolvable versions listed in the 'Resolvable' column.
Newer versions, listed in 'Latest', may not be mutually compatible.
Code:
DrawerHeader(
child: emptyWidget,
decoration: BoxDecoration(
color: Colors.blue,
image: DecorationImage(
image: AssetImage("images/drawer-header.png"),
fit: BoxFit.cover)),

I am getting the same one on 3.10.4 and on the ios versions < 14, is there any workaround to this other than disabling impeller engine ?
I also have opened a new issue for the same -
https://github.com/flutter/flutter/issues/134273

This is no longer possible - the engine now crashes in this scenario.


Synchronize adding a child task to a TaskGroup with parent task

When a task is adding adding new children to a task group, we need to synchronize with the task status record lock of the parent task that has the task group, to prevent races with concurrent cancellation and escalation.
In the future, we'd probably want a lock for the TaskGroup itself and not have it be the group's parent task's status record lock but for now, this will get us the right functionality.

Resolves rdar://86311782

@swift-ci please smoke test

Failure was the unrelated
/home/buildnode/jenkins/workspace/swift-PR-Linux-smoke-test/branch-main/swiftpm/Tests/WorkspaceTests/RegistryPackageContainerTests.swift:421:39: error: type 'FingerprintCheckingMode' has no member 'none'
20:09:38             fingerprintCheckingMode: .none,
20:09:38                                      ~^~~~

from swiftpm that we fixed; rebuilding

@swift-ci please smoke test

@swift-ci please smoke test

rdar://86311782


Unknown error while trying to get game data: Expecting value: line 1 column 1 (char 0)

2024-09-23 14:51:56 | INFO     | 456 - samet | start after 4s
2024-09-23 14:52:03 | SUCCESS  | 208 - samet | Get access token sucessfully
2024-09-23 14:52:03 | INFO     | 269 - samet | Logged in
2024-09-23 14:52:03 | INFO     | 270 - samet | Total point: 12650 | Risk Passed: True | Qualified: True
2024-09-23 14:52:04 | INFO     | 354 - samet | Starting to play game...
2024-09-23 14:52:04 | INFO     | 356 - samet | Attempts left: 6
2024-09-23 14:52:04 | SUCCESS  | 364 - samet | Game 7eaba13aab754700f3ed0c19b9b573a3 started successful
2024-09-23 14:52:04 | INFO     | 368 - samet | Wait 45.04889296037665s to complete the game...
2024-09-23 14:52:50 | ERROR    | 226 - samet | Unknown error while trying to get game data: Expecting value: line 1 column 1 (char 0)
2024-09-23 14:52:50 | INFO     | 381 - samet | Sleep 6.668215609300199s...

fixed


can I use yt-dlp as a drop in replacement for youtube-dl?

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2023.07.06 (update instructions) or later (specify commit)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
can I use yt-dlp as a drop in replacement for youtube-dl?
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

Yes, you can.

ok got my answer closing thread


Keys seem to come through scrambled when typing in a Glutin window

When typing fast in a Coder, the keys seem to come out in an arbitrary order.

Also, I got into a situation inside a window in which the Command key seems to not be responding. However, I spawned a new Glutin window and there the Command key works fine. At the same time, in the old window, it continue to not work.

Turned out, pollEvents: does not poll all events.
wrapped it in
while had_event {
   poll_again { had_events = true }
}

should work now


ModuleNotFoundError: No module named 'attn_core_inplace_cuda'

I am using openfold as a dependency for another package. Whenever the dependency calls openfold/openfold/utils/kernel/attention_core.py, I get this error:
ModuleNotFoundError: No module named 'attn_core_inplace_cuda'
I've tried looking elsewhere, but I can't find any documentation for this module.
Steps to recreate:

clone the openfold repo
create openfold_env as per the openfold installation instructions
run the openfold/utils/kernel/attention_core.py with python, e.g. python3 openfold/utils/kernel/attention_core.py

I am working on a computing cluster with CUDA version 12.2.

Hey @pikruse ,
As @dingquanyu mentioned the attention kernel issue should be solved once you get OpenFold built and installed using python setup.py install. I am closing this for now but feel free to reopen if you face any further issues.


Does the Dynomite storage engine support data migration in case of schema changes?

Hey @MaksimRT, what schema are you referring to in Dynomite?

Hi @kishorebanala!
Thanks for response!
I know that NoSQL database such as Dynomite or Redis is schemaless, but you have a schema in the code, right?
So I want to know how do you handle appication upgrade.
For example, I have v2.7.2 (you could choose any other version), could I skip v2.8.0 and install v2.8.1?
What happen if I install v2.8.1 and then decide to rollback to v2.7.2?
I also have the same questions for MySQL  database as well.

@MaksimRT We follow Semantic versioning, and any minor version upgrade should be backwards compatible. Any breaking changes will go to a major version upgrade, and will be mentioned explicitly.


Can UI support emoji?

Private Use Area like \u{f1e2} can be used but not the ideal emoji

The UI supports emojis/unicode to the extent of the underlying font.
The default font is a patched Nerd Font, which means that these also work: https://www.nerdfonts.com/cheat-sheet

The UI supports emojis/unicode to the extent of the underlying font.
The default font is a patched Nerd Font, which means that these also work: https://www.nerdfonts.com/cheat-sheet

I tried dozens of them and half cannot be rendered, displaying an empty square ☐.

Yep, I can also confirm this. I wonder if the same issue persists with a custom URL?

Which ones did you use?
The recent version of nerdfonts have had to move a lot of code points around to accomodate larger icon sets. We are using a very old version of nerd fonts, so the codepoints are likely also incorrect

Is thay upwards arrow and rofl glyphs?

Which ones did you use?
The recent version of nerdfonts have had to move a lot of code points around to accomodate larger icon sets. We are using a very old version of nerd fonts, so the codepoints are likely also incorrect

I am pretty sure this one nf-md-hand_pointing_right won't work \u{f02c7}, I was trying to use that for killing broadcast.

Is thay upwards arrow and rofl glyphs?

→ 🤣


Revert ThreadName due to problems on Windows

High Level Overview of Change
Some problems showed up on Windows that previous testing hadn't revealed with these reverted commits.  The problems are not insurmountable.  But fixes should not be rushed through.  This commit simply reverts the "thread name" commits.

@manojsdoshi - are we ok to merge this to develop now?


Thread protocol support removed?

Description of defect
It's more of a question/problem with getting started with the mesh api.
I was looking to develop a Thread mesh network using the STM32WB55 Nucleo board and mbed-os, but I've noticed that Thread seems to be getting removed from all of the mbed documentation (For example: ARMmbed/mbed-os-example-mesh-minimal/#306).
Is there any particular reason for this?
What target(s) are you using?
NUCLEO-WB55RG
What toolchain(s) are you using?
Mbed Studio 1.3.0
What version of Mbed OS are you using (tag or sha)?
mbed-os 6.3.0
It would help if you could also specify the versions of any tools you are using?
The latest
How can we reproduce your issue?
N/A

@georgejourneaux thank you for raising this issue.Please take a look at the following comments:
Could you add some more detail to the description? A good description should be at least 25 words.
What target(s) are you using?
What toolchain(s) are you using?
What version of Mbed OS are you using (tag or sha)?
It would help if you could also specify the versions of any tools you are using?
How can we reproduce your issue?
NOTE: If there are fields which are not applicable then please just add 'n/a' or 'None'.This indicates to us that at least all the fields have been considered.
Please update the issue header with the missing information, the issue will not be mirroredto our internal defect tracking system or investigated until this has been fully resolved.

Similar question here: ARMmbed/mbed-os/#13624

@georgejourneaux it has been 5 days since the last reminder. Could you please update the issue header as previously requested?

@ciarmcom Apologies, I've updated it now!

@georgejourneaux it has been 5 days since the last reminder. Could you please update the issue header as previously requested?

@georgejourneaux it has been 5 days since the last reminder. Could you please update the issue header as previously requested?

@georgejourneaux it has been 5 days since the last reminder. Could you please update the issue header as previously requested?

@georgejourneaux it has been 5 days since the last reminder. Could you please update the issue header as previously requested?

@georgejourneaux it has been 5 days since the last reminder. Could you please update the issue header as previously requested?

Having had a bit of a deep dive into how the mbed connectivity APIs work, I believe I have two options:

Create a NanostackRfPhy driver for the STM32WB, as there currently isn't one (as described here)
Create a NetworkInterface for the STM32WB (as described here)

As the target technically has an off-board network-stack (user loaded on its M0+ core) with an IPCC transport layer to interact with it, I believe that I should be looking at creating a NetworkInterface not a NanostackRfPhy driver. The NetworkInterface I create would then pass the network calls from the mbed-os APIs to the off-board stack, via the IPCC transport layer.
To me it seems like this would make the best use of the STM32WBs' architecture, but please do let me know if I'm completely wrong here as I'm fairly new to mbeds connectivity APIs. In the meantime I'll close this issue as I think I've solved my problem.


add file api design(add to wishlist)

Thank you @obeidahr for your PR
good start,
There are some notes to take care of:

you don't to create new file, move your code to apis.yaml and delete [API Design] add to wishlist.yml file.
it would be great to re-read API Spec's contributing guideline and confirm your code.
you should use /wishlist/{productId} endpoint, with POST method for add
you only need to define productId parameter, in path, as int, no need for requestBody
your 200 response does not have to return content (there is nothing to return except status code 200)

Please replay if you see any point is not logical, you can also discuss it in the group chat.

Thank you @obeidahr
you are getting better 👍
however, we still need to add productId in path as /wishlist/{productId}
and use the method POST (for adding) instead of put (for update)
and use tags: wishlist instead of user
and add parameters like in delete from wisthlist
and finally, move components: section to the left one step (two spaces)


refactor: 3835 pylint interventions

Fixing two alerts for pylint intervention experiment
The plan was 3 interventions
Changes are minor

The first long line represented the output and should stay as is
Unneeded parentheses were removed when calling the del keyword
Add line breaks to two too long lines making them shorter

Each intervention was done in a dedicated commit with a message explaining it.
@fzipi

Fixes #3835

Thanks for the fixes @evidencebp!


Update php image to php 7.

The alpine package is naming the bin php7, should be just php for a normal 7 install right?

That makes sense to me, yeah.

Done, iron/php:7 and iron/php:7-dev


Add Octopus connector & converters

Description
Add Octopus connector & converters

Provide description here
The current commit is to add an octopus connector.

[ ] New feature
Completely new connector for octopus

Related issues

Fix #400
This is related to #400 issue.

@tovbinm This connector is completed. request you to please review this code.

Right now this PR only contains the connector functionality and does not have the Faros destination converter yet. Are you planning to add the converter in this PR as the title suggests Add Octopus connector & converters or were you going to break that out into its own PR? (if you are going to separate please update the title, thanks)

Right now this PR only contains the connector functionality and does not have the Faros destination converter yet. Are you planning to add the converter in this PR as the title suggests Add Octopus connector & converters or were you going to break that out into its own PR? (if you are going to separate please update the title, thanks)

I think @tovbinm only assigned the connector work i can do that if he assigns it to me.

For now we only requested the Octopus connector development. I updated the description of the PR to reflect the changes.


fix(useMediaQuery): On misconfiguration, cause hydration error instea…

What is the current behavior, and the steps to reproduce the issue?
When the hook is misconfigured, that is, when initializeWithValue is true during SSR, the hooks crashes instead of producing a hydration error, which is not in line with the behavior of the rest of the isomorphic hooks.
What is the expected behavior?
Hydration error instead of a crash.
How does this PR fix the problem?
If the rendering environment is not a browser, set initializeWithValue to false.
This solution makes me wonder: what is the purpose of the initializeWithValue option anymore? If it is set to false during SSR anyway, why have it at all? Is there a usecase for having the hook return undefined on first render when doing client-side rendering?
Checklist

[x] Have you read contribution guideline?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Is there an existing issue for this PR?

#1000

[x] Have the files been linted and formatted?
[x] Have the docs been updated to match the changes in the PR?
[x] Have the tests been updated to match the changes in the PR?
[x] Have you run the tests locally to confirm they pass?

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


chore: stable schema generation

PR in pgvecto.rs: https://github.com/tensorchord/pgvecto.rs/pull/254
PR in pgvecto.rs-docs: https://github.com/tensorchord/pgvecto.rs-docs/pull/8

Currently we rely on pgrx to generate SQL schema. However it's hard to track generated schema and write a update script.
We will:

do not use pg_* macros, expect for pg_extern attached to _vectors_*
write schema in finalize.sql
document schema in pgvecto.rs-docs

Will this affect user update? Or we'll maintain it manually?

It'll make maintaintance of user update easier.


serverless.Api refuses ResourcePolicy in Auth property

Hi,
Thank you for the time and effort you are putting in to Troposphere. I really appreciate it!
I'm trying to use Troposphere to create an API gateway via Serverless with a ResourcePolicy.

Minimal troposphere code to reproduce the issue
from troposphere import Template, serverless

template = Template()
template.set_transform('AWS::Serverless-2016-10-31')

api_gateway = template.add_resource(serverless.Api(
    "ApiGateway",
    StageName="Prod",
    EndpointConfiguration="PRIVATE",
    Auth=serverless.Auth(
        ResourcePolicy={
            'SourceVpcWhitelist': [
                'vpc-12345678',
            ],
        },
    ),
))

print(template.to_json())

Troposphere fails to accept ResourcePolicy as attribute in the Auth() object:

Auth object does not support attribute ResourcePolicy

This usage is covered in the AWS documentation. I've also verified it to work by manually editing the JSON. The problem seems to be with the class definition of the Auth class. This is already hinted in #1278 :

The Auth settings are a bit wonky.

I'll look in to fixing this myself and creating a Pull Request as follow up.

System details
macOS 10.14.6
$ python -V
Python 3.7.5
$ pip list
Package                       Version
----------------------------- ------------
attrs                         19.3.0
awacs                         0.9.6
aws-cloudformation-utils      0.1.0
aws-cloudformation-validation 1.0.1
boto3                         1.10.22
botocore                      1.13.22
botostubs                     <IP_ADDRESS>.20
central-helpers               5.11.0
certifi                       2019.9.11
cffi                          1.13.2
cfn-flip                      1.2.2
cfnresponse                   1.0.2
chardet                       3.0.4
Click                         7.0
cryptography                  2.8
custom-resources              0.7.0
docutils                      0.15.2
edgegrid-python               1.1.1
entrypoints                   0.3
enum34                        1.1.6
flake8                        3.7.9
flake8-builtins               1.4.1
flake8-commas                 2.0.0
flake8-docstrings             1.5.0
flake8-pep3101                1.2.1
flake8-string-format          0.2.3
flake8-tidy-imports           3.1.0
idna                          2.8
importlib-metadata            0.23
invoke                        1.3.0
ipaddress                     1.0.23
jmespath                      0.9.4
mccabe                        0.6.1
more-itertools                7.2.0
ndg-httpsclient               0.5.1
netaddr                       0.7.19
pip                           19.0.3
prometheus-client             0.7.1
pyasn1                        0.4.8
pycodestyle                   2.5.0
pycparser                     2.19
pydocstyle                    4.0.1
pyflakes                      2.1.1
pyminifier                    2.1
pyOpenSSL                     19.1.0
python-dateutil               2.8.0
PyYAML                        5.1.2
pyzabbix                      0.7.5
requests                      2.22.0
s3transfer                    0.2.1
setuptools                    40.8.0
six                           1.13.0
snowballstemmer               2.0.0
troposphere                   2.5.2
urllib3                       1.25.7
zipp                          0.6.0

Solved in https://github.com/cloudtools/troposphere/pull/1568

Closing. Thanks @ikben.


Add KCSAN config

Check out the KMSAN config generation script. I think it's better to do something similar for KCSAN.

I think we also want to disable these things:
https://github.com/google/syzkaller/blob/master/dashboard/config/bits-kmsan.config#L24-L44
These will slowdown execution and generate lots of crashes that will throw away anyway.

Codecov Report

Merging #1420 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1420   +/-   ##
=======================================
  Coverage   58.26%   58.26%
=======================================
  Files         131      131
  Lines       24878    24878
=======================================
  Hits        14495    14495
  Misses       9715     9715
  Partials      668      668

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 1e9788a...93ba046. Read the comment docs.

Done.

Added script to generate KCSAN config.
Added generated config.
Removed intermediate commit, since it should be clear from generator script what the changes are.

Also build/boot this config in parallel, so that we know it works.

LGTM

LGTM

Done.
Tested with local syzkaller, which is still finding plenty of races.


Minor discrepancies with POSIX behaviour.

This is a low priority list of things which appear out of step with POSIX style fs behaviour. For any other things like this I'll just add to this issue.

already mentioned on the forum, sn_fs reports directory sizes as '0' whereas they appear to be at least the minimum block size (e.g. 4096) on Ubuntu.
ls -al of a directory should include entries for '.' and '..' as below, whereas for sn_fs these are missing.

PS I've attached a script which I used to test my now defunct SAFE Drive, tweaked slightly to test sn_fs and it passes all tests on Ubuntu 20.04. :tada:
You'll have to rename the .txt to .sh to use it of course. test-sn_fs.sh.txt
It expects to be run from the root of a git repo with the script in a tests subdirectory, for example:
cd ~/sn_fs
mkdir -p ~/SAFE
sn_fs ~/SAFE &
./tests/test-sn_fs.sh mock|disk

I'll also try some of the manual tests which gave SAFE Drive problems.

thx for filing the issue and for the test script!   I'll try it out later.


Change banner color, up percentage

Upping the percentage of people seeing the banner.
Hoping to draw eyes w/ green color.

:+1:


[wip] Fine-tune unified scheduler loops by select_biased

TBD
Problem
Summary of Changes
Fixes #

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

Project coverage is 82.7%. Comparing base (94af1aa) to head (b353804).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1437   +/-   ##
=======================================
  Coverage    82.7%    82.7%
=======================================
  Files         872      872
  Lines      370361   370354    -7
=======================================
+ Hits       306478   306538   +60
+ Misses      63883    63816   -67


readme: Add Fedora 26 build badge and update existing ones

This commit adds the Fedora 26 build badge and fixes the links
to the ubuntu badges.
Fixes #594.
Signed-off-by: Salvador Fuentes<EMAIL_ADDRESS>

@iphutch can you take a look at this? Thanks

lgtm


API: mne.preprocessing.mark_flat --> mne.preprocessing.annotate_flat ?

With the relatively new Annotations in MNE, there are now (at least) two functions that work on raw data and return annotations:

mne.preprocessing.annotate_muscle_z_score
mne.preprocessing.annotate_movement

There is an older function operating in a similar style: mne.preprocessing.mark_flat
Would it make sense to change this to mne.preprocessing.anotate_flat to have a set of functions with a consistent API working on raw data and providing annotations?

+100

That function might mark bad channels and it might annotate, depending on the data and what percentage of time a given channel is flat. So maybe annotate_mark_flat?

So maybe annotate_mark_flat?

this is in fact the only function with mark_ in its name (although there is the MNE-C function mne_mark_bad_channels). So unclear to me that we need the longer, more complicated name.  It doesn't seem conceptually weird to me that a temporal-span annotation function might "give up" and just mark the whole channel in some cases. +1 for annotate_flat

+1 for annotate_flat

+1

I just realized all of these functions return Annotations, but do not add them to the passed Raw instance. To me, this behavior seems to kind of contradict the function name? i.e. when something is called "annotate", I'd expect it to actually annotate… 🧐

I know MNE methods should generally work in-place, but in the case of artifact processing I'd want an option to double-check the automatically generated annotations/bad channels. Therefore, I'd prefer such methods to return annotations and a list of bad channels.

but in the case of artifact processing I'd want an option to double-check the automatically generated annotations/bad channels. Therefore, I'd prefer such methods to return annotations and a list of bad channels.

… which is already the case for annotate_muscle_z_score and annotate_movement; and if we rename mark_flat anyway, we could also change its behavior accordingly.
Still, my issue remains: if I say "annotate something", I expect "something" to be annotated. Just like when I call list.remove(), I expect something to be removed from the list, without requiring an additional step from my end. – If, however, the function were called e.g. gen_movement_annotations, I would expect it to return annotations (and not necessarily work in-place).

What about str.upper() then?

What about str.upper() then?

"upper" is not a verb, so to me the behavior is unclear unless I check the docs.

(besides, strings are immutable; so the comparison is a little off anyway… but this is an implementation detail)

You could interpret it as a verb though - uppercase that string for me. And yes, strings are immutable, but I wanted to point out that even in the built-in types this is not consistent.
Regarding your expectation, if you annotate something, you could also get back the annotated object (instead of modifying in-place).

Regarding your expectation, if you annotate something, you could also get back the annotated object

Yes, in addition to the in-place modification. Otherwise it would be, "generate an annotation please" and not "please annotate".

(I've edited my answer above)

In compute_proj_ecg and compute_proj_eog
we  do as the function name indicates: compute the projs, and return them. The verb, here, makes total sense. OTOH, annotate_*, without actually annotating, does not make sense to me.

I just realized all of these functions return Annotations, but do not add them to the passed Raw instance. To me, this behavior seems to kind of contradict the function name? i.e. when something is called "annotate", I'd expect it to actually annotate

@hoechenberger what is the concrete proposal here?  Is it to rename the existing annotate_* functions to make_*_annotations, or to keep their names and change their behavior?  If it's the latter, please open a separate issue, and we can (I think?) move forward here with renaming mark_flat to annotate_flat.
If your proposal is to keep the behavior and change the names, then we should probably do here it all at once.  But that said, I'm not bothered by the annotate_* naming. To me "annotate" can mean either "make notes" or "attach notes". If you think about a literary work, annotating it does not necessarily mean writing in the margins of a physical book; a separate document with page and paragraph numbers attached to each note still counts as annotation IMO. That seems quite analogous to our Annotations object (which incidentally, a user might simply want to save to disk in tabular format --- returning the modified Raw is not helpful in that case).

@drammock

@hoechenberger what is the concrete proposal here? Is it to rename the existing annotate_* functions to make_*_annotations, or to keep their names and change their behavior?

Since my interpretation of "to annotate" seems to be a bit of an edge case, I'm okay with leaving these functions as is (I very much would prefer to rename them to make_*_annotations, personally.)
But we should make sure that the current mark_flat, if renamed to annotate_flat, matches the behavior of the existing annotate_* functions, namely, stop modifying the input in-place, and return Annotations instead.

But we should make sure that the current mark_flat, if renamed to annotate_flat, matches the behavior of the existing annotate_* functions, namely, stop modifying the input in-place, and return Annotations instead.

Agreed; I think that was part of @sappelhoff's original proposal.

Sounds reasonable to me

(if someone calls mark_flat in 0.21 it should do the same thing as in 0.20 but give dep warning, annotate_flat should do whatever we want annotate_* functions to do)

@sappelhoff will you be able to take a stab at this one in the next couple of weeks?

Offering my help here too 👍

Offering my help here too +1

In that case, I'd be happy if you took over @hoechenberger :-)

+1 to add this to the next release


How to enable read only mode?

Hi All,
How to enable real only mode in docker?
Based on #218 PR it is enable with readOnly environment variable.
In docker I have tried the following configuration:
-e readOnly=True
-e readOnly=true
-e readOnly='True'
-e readOnly='true'
-e KAFKA_CLUSTERS_0_READONLY=true
-e KAFKA_CLUSTERS_0_READONLY=True

In every cases "Add topic" and "edit setting" button on the UI was still active. When I click on these buttons I dont get any error message. In the "edit settings" menu when I click on the "submit" button I got no error message.
Thanks,
Andor

Hi, @andormarkus
-e KAFKA_CLUSTERS_0_READONLY=true should work for backend in version 0.0.10, frontend will be released in next version.

Hi @germanosin
I have adjusted the documentation according it. See my PR #247
Thanks,
Andor


Trello Clone

Description
I want to make Clone of Trello Website in html,css and javascript
Code of Conduct

[X] I follow Contributing Guidelines of this project.

/assign


Should 'curator inference' always be manual?

It seems to make sense that curator inference should always be used in a manual assertion.
We currently have many "automatic assertion" classes under curator inference. Does this make sense in any situation? If not, we should deprecate these classes.
That would only leave "manual assertion" classes. Could we merge the "manual assertion" classes and the raw evidence types? Will that cause issues for users, particular GO curators? Currently, "curator inference used in manual assertion" is equivalent to IC. The ID for this term would stay the same, but the parent class would be merged into it and the label would become "curator inference".
The same would happen with all the children terms. This is parallel to the other discussions in #240 and #242. I'd just like to make sure that these changes wouldn't break any downstream applications.

It seems to make sense that curator inference should always be used in a manual assertion.
I agree. I can't seem to think of a curator inference resulting in an automatic assertion.

We currently have many "automatic assertion" classes under curator inference. Does this make sense in any situation? If not, we should deprecate these classes.
The could easily be obsoleted.

That would only leave "manual assertion" classes. Could we merge the "manual assertion" classes and the raw evidence types?
I would not do that. I don't really see what it accomplishes and I believe it is better to preserve consistency across the ontology.

Ivan

I don't really see what it accomplishes and I believe it is better to preserve consistency across the ontology.

The consistency is a good point. The point of merging manual terms up would be so as not to have classes that only have a single child.

Right, but conceptually it's a cross-product, so I would keep it that way
for consistency.
On Fri, Nov 8, 2019 at 3:28 PM Becky Jackson<EMAIL_ADDRESS>wrote:

I don't really see what it accomplishes and I believe it is better to
preserve consistency across the ontology.
The consistency is a good point. The point of merging manual terms up
would be so as not to have classes that only have a single child.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/evidenceontology/evidenceontology/issues/244?email_source=notifications&email_token=AAHWLKMHZRUZYJ2FSF5GMTLQSVZQRA5CNFSM4JJ62XR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDSIT5Q#issuecomment-551848438,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHWLKO6TQPDPUQTKKEM4CTQSVZQRANCNFSM4JJ62XRQ
.

Seems fine to me. Closing for now.


Filter embargoed also by Flavor

https://progress.opensuse.org/issues/153334

@asmorodskyi

I'd like to propose a possible different approach to drastically simplify logic in filter_embargoed. What about to add a field in the metadata to opt-in the check?
    Azure-SAP-PAYG-Incidents:
      embargo_sensible: true       <<<-----
      aggregate_job: false
      archs:
        - x86_64
      issues: *test_issues_list
      required_issues: *required_list
      packages: *whitelist_ha_hana
      params_expand:
        PUBLIC_CLOUD_IMAGE_ID: 'suse:sles-sap-15-sp5:gen2:latest'


Highlight build output

To be used in free-range tasks, e.g. in GoodStuff after the Build tasks, so that students can see the errors.

Nvm


Weird behavior when clicking in bib information

Acknowledge the following

[X] I carefully read and followed the Getting Started guide.
[X] I read through FAQ and searched through the past issues, none of which addressed my issue.
[X] The issue I am raising is a potential bug in al-folio and not just a usage question.  [For usage questions, please post in the Discussions instead of raising an issue.]

Describe the bug
When clicking in bib, abs, or other button in the publications page, the button enlarges before returning back to previous size.
To Reproduce
Steps to reproduce the behavior:

Go to https://alshedivat.github.io/al-folio/publications/
Click on ABS, BIB, HTML, or PDF buttons

Additional context
Error introduced after #1562

@george-gca, this bug seems critical. if we can't figure out how to fix it today, please roll back #1562 until we figure it out (I can look into it this week).
note: if you click on the buttons multiple times, they grow uncontrollably lol

I am looking at it now. Did a lot of tests, but forgot about this little thing.
After clicking in the buttons, it adds some new classes temporarily:
Before clicking:

During weird behavior:


Date query is incorrect

For example, you can find all items created in the last week with the clause Created Date=@Today-7.

This is incorrect, it instead shows items created one week ago, to the day. Most likely what was intended is Created Date>=@Today-7

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

ID: 5a718268-1105-4def-5f9e-20ea21afc0d7
Version Independent ID: 0f08fa18-3075-4275-22c7-d5d1c1e260b0
Content: Query fields, operators, and macros/variables - Azure Boards
Content Source: docs/boards/queries/query-operators-variables.md
Product: devops
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

@Mini-Geek - Thank you for the correction. The article is getting updated.


Excel sheet to - Date Issue - XLSX Package

Hi Team ,
I am struggling with date  format  .
my package.json
{
"name": "test",
"version": "1.0.0",
"description": "",
"main": "index.js",
"scripts": {
"test": "echo "Error: no test specified" && exit 1"
},
"author": "",
"license": "ISC",
"dependencies": {
"xlsx": "^0.14.0"
}
}
find my xlsx file in the attachment
100264.xlsx
and i am using below code in node js to read the file but the date is one day before . Also numeric field are not coming properly . please suggest ho to handle these issues
const XLSX = require('xlsx');
var workbook = XLSX.readFile('100264.xlsx',{type: 'binary', cellDates: true, dateNF: 'mm/dd/yyyy;@'});
//var workbook = XLSX.readFile('100264.xlsx',{type: 'binary', cellDates: true});
//var workbook = XLSX.readFile('100264.xlsx',{type: 'binary',cellDates: true});
let active_sheet_name = workbook.SheetNames[0];
let fileData = XLSX.utils.sheet_to_row_object_array(workbook.Sheets[active_sheet_name]);
console.log(fileData);
the output of console.log is given below.
[ { 'Sales Date': 2018-01-30T18:30:00.000Z },
{ 'Sales Date': 2018-01-30T18:30:00.000Z },
{ 'Sales Date': 2018-01-30T18:30:00.000Z },
{ 'Sales Date': 2018-01-30T18:30:00.000Z },
{ 'Sales Date': 2018-01-30T18:30:00.000Z },
{ 'Sales Date': 2018-01-30T18:30:00.000Z },
{ 'Sales Date': 2018-01-30T18:30:00.000Z },
{ 'Sales Date': 2018-01-30T18:30:00.000Z },
{ 'Sales Date': 2018-01-30T18:30:00.000Z },
{ 'Sales Date': 2018-01-30T18:30:00.000Z },
{ 'Sales Date': 2018-01-30T18:30:00.000Z },
{ 'Sales Date': 2018-01-30T18:30:00.000Z },
{ 'Sales Date': 2018-01-30T18:30:00.000Z },
{ 'Sales Date': 2018-01-30T18:30:00.000Z },
{ 'Sales Date': 2018-01-30T18:30:00.000Z },
{ 'Sales Date': 2018-01-30T18:30:00.000Z },
{ 'Sales Date': 2018-01-30T18:30:00.000Z },
{ 'Sales Date': 2018-03-01T18:30:00.000Z } ]

The default passes dates as raw date objects.  To recover the text in the original format, use the option raw: false, as answered in https://github.com/SheetJS/js-xlsx/issues/703#issuecomment-432781209

Hi Reviewher Thanks for the reply but now i am getting exact value as in excel file but i need the actual as you can see last formats . there is no year .so basically i have applied different format for each cell and but in behind each cell have proper date as you can see in my excel file so i need the actual in mm/dd/ yyyy format or any other specific format. Please reply..
Thursday, January 12, 2012
2013-01-25
1/21
1/31/18
01/12/11
31-Jan
31-Jan-19
Jan-20
January-22
May 25, 2013
12/19/17 12:00 AM
1/31/18 0:00
J
J-18
J-18
J-18
M-18
01/03/2017

Sure, you additionally need cellText: false on the read part.  That will ensure the raw text isn't generated, so when the conversion happens it will use the new format.

Hi Reviewher ,
Thanks once again I have applied cellText: false  now it is taking every year is 2000 century for example:-
03/01/1901 in excel
03/01/2001 in node js


Change parseJson to perform type validation

Adds a second argument to the parseJson function: A TypeRep which the result from parsing the JSON is validated against. This closes #150.

This is very nice, Aldwin!

@davidchambers Did you see I did the thing?

Thanks, @Avaq. Now that we've merged plaid/sanctuary-def#38 I can publish a minor release of sanctuary-def and a patch release of this project (to fix #165). That will free us to start merging changes for the v0.10.0 release.

I see, so we're awaiting the release of sanctuary-def before we'll rebase and squash. Cool. :)

Please rebase this branch when you get a chance.

:information_source: @davidchambers Notice the rebase? I'm not sure if it would've notified you.

:deciduous_tree: Thanks very much, @Avaq!


[GHSA-m95q-7qp3-xv42] Zod denial of service vulnerability

Updates

Affected products
CVSS
Severity

Comments
The National Vulnerability Database describes the severity as High:
https://nvd.nist.gov/vuln/detail/CVE-2023-4316
I've taken the Vector string from that record.

Hi @RobinTail, thanks for reaching out about GHSA-m95q-7qp3-xv42. The severity is currently lower than the CVSS score on NVD because, as this blog post by a colleague of mine explains, it's common for ReDoS vulnerabilities to not be particularly serious. However, I see that you've published https://github.com/RobinTail/express-zod-api/security/advisories/GHSA-mvrp-3cvx-c325 to talk about how CVE-2023-4316 affects your repository. From the perspective of someone who mitigated CVE-2023-4316 in their own repository, do you think the CVSS score is appropriate for the seriousness of the ReDoS?

Hello, @shelbyc .
Thank you for the review.
I'm actually not a security expert to assess the severities, I'm just good at filling out forms.
However, the article you refer begins with the following words:

some ReDoS vulnerabilities can be very serious (particularly when they’re server-side and enable an untrusted remote attacker to DOS the server

Although zod is a package that can be consumed both by client- and server-side applications, I'd say that email validation on server side can be a popular practice among consumers of the library. Since entering the email is a common scenario during a sign up process of many websites and applications, the vulnerability can be exploited by unlimited number of individuals (not limited to the registered ones, for example).
Taking into account that anyone entering a malformed email address on a website signup form can suspend or restart its API, I'd assume that the severity of this issue might be above low.
What do you think, @shelbyc ?

I have a lot of opinions about how CVSS and vulnerability severity assessment in general works, but I'll try to keep it short. 😅
ReDoS tends to be overscored because CVSS assesses impacts to data confidentiality, integrity, and availability equally, while each of those factors affects different users in different ways. For some users, high availability impact (i.e. a crash) may be a nuisance, while high confidentiality impact (having data exposed to a threat actor) could be devastating, but those two situations can receive the same score.
CVSS tries its best, but it's very difficult, if not impossible, to use one metric to fit everyone's situation. ReDoS vulnerabilities affecting libraries are one of those situations where CVSS tries but doesn't fit well, just because it's one scoring method trying to be applied to a huge range of software vulnerabilities. Sometimes, I avoid using CVSS in an advisory because it doesn't fit well for a certain situation, but I'm open to hearing from community members if they believe a vulnerability is more or less serious than I initially assessed. 😺


QA - Report

March 6th QA,

[improvement] Category should be able to be disabled by click again. typical UX expectation when a btn (toggle alike), it should be able to toggle it off, in this case, we should allow user to click again and disable.

[improvement] might need to an indicator when a filter is applied. now after user set up some filter, it doesn't look like there is anything selected when the modal is closed.

[bug] reset btn doesn't reset the "active" state on the categories tag, although the URL is resetted

[bug] the price slider is not as responsive as expected. try to drag but its stuck on certain range.

[Improvement] "apply" btn can be bigger

[improvement] when input field is focus, and user finish enter the value, the page will stay as zoomed

https://github.com/VoiceDeck/app/assets/79813968/21199d84-92ff-4668-aeee-144517574fbb
updated: category selections can be toggled on and off, reset button now reflects when tag is not active.

updated: filter trigger shows icon when none applied, number of filters shown in blue circle when some. Resets to icon when Reset button clicked.

updated: the silder can now select amounts with a minimum range of $50. This is a slight improvement from before and hopefully seems less arbitrary, but it is not possible to make the range any smaller bc it has to accommodate amounts from $0-$1000 - which looks like this:

updated: larger "apply" buttons

updated: no screen zoom triggered when user taps into search bar on mobile.
the fix was to increase the text size, which i don't love that it's now bigger than tin he other elements in the reports header but i think this is the best way to go. Other possible fixes potentially mess with accessibility.

@ChialiT  complete on this round, maybe open a new issue with additional testing feedback?

closed by #41


Index: Cleanup low level unused removeKey:

The low level indexes (BTree and SkipList) can not implement a naive #removeKey:
#removeKey: is implemented on the level of IndexedDictionary and marks the entry as deleted. This can only be cleaned up later, one idea is to do it on save (see #459). At that point then we have to remove the deleted item, possibly merge pages and update the index.
This PR removes the #removeKey: implementation on Btree and SkipList, as it is very confusing

it is not called outside fo the low level tests (not used by IndexDictionary and can not be used by it)
does not update the indexes

closes  #272

But the indexes can actually do a removeKey: The key is just an integer that is managed by the SkipList/BTree. So I don't really understand the rationale here

It is implemented right now just on SoilIndexedDictionary>>#removeKey:ifAbsent:, here it does not remove but instead it adds removed entry.
removeKey: key ifAbsent: aBlock
	| iterator v |
	^ transaction
		ifNotNil: [
			"remove from newValues as there could be a new at:put: on that
			key but removing the key will remove the value again"
			newValues removeKey: key ifAbsent: nil.
			transaction markDirty: self.
			iterator := self index newIterator.
			v := iterator at: key ifAbsent: [^ aBlock value ].
			v isRemoved ifTrue: [ ^ aBlock value ].
			removedValues
				at: key
				put: v asSoilObjectId.
			iterator at: key put: (SoilObjectId segment: 0 index: 0) ]
		ifNil: [
			removedValues
				at: key
				put: (newValues removeKey: key ifAbsent: [ ^ aBlock value ]) ]

Yes I understand the use case in the IndexedDictionary case and also flagging removed values. But the Indexes are kind of dictionaries themselves where you should be able to remove an item, no?

Ok, I close this


chore(common): Fix returned buffer from method

Closes: #1199

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

v6.22.4
GitHub release

Your semantic-release bot :package::rocket:

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

v6.22.4
GitHub release

Your semantic-release bot :package::rocket:


Webhoook Event URL offset details

Hello,
Cody S from the escalation team sent me here after they couldn't answer our questions about ticket number 4414242.
Introduction
We want to be able to map each clicks on our URL in our mail template to provide more visual statistics to our marketing team.
But after trying to reverse engineer we couldn't find any easy way to do it using the url offset, though it looks like the perfect params to do it.
The example email we used as 23 url, confirmed by counting them and using the following JS command :
document.body.querySelectorAll('a:not([href^="mailto:"]').length

But the url_offset go up to 37.

Why does the URL offset goes up to 37 ?
We also have some URL (say the 5th, 6th and 7th) where the URL offset would be 12, 13 then 16 (a jump of one or multiple url). Why does it happens?
We also observed an url_offset having two URLs. How can this happens?
Finally, why the first url_offset is associated with a link which isn't present in the template?

Expected content
More information on the logic behind the url offset generation, so we can create an accurate mapping of the clicks in an associated email template.
Actual content
<a name="urloffset"`urloffset` - if there is more than one of the same links in an email, this tells you which of those identical links was clicked.

Link to page: https://sendgrid.com/docs/for-developers/tracking-events/event/#json-objects

Hello @Aridjar ,
Thank you for bringing this to our attention, and we're sorry you're having trouble. We've added it to our docs backlog and will reach out to the product team to better understand the URL offset or if there may be another way to approach the problem given the data we return. Thank you for your patience.
Twilio SendGrid Docs Team

Hello @Aridjar ,
Is your template setup to include both plain/text and html/text? If so, do you have click tracking set for both? A template set that way may explain the "jump" behavior. Let us know, and we can provide more details if needed.

Hello @astuteape
The template setup is to include both plain/text and html/text and the click tracking is set for both.

Hello @Aridjar
We increment URL offset indexes from top down in the mime file. This means if a mime has one text part and one html part, both containing click-tracked links, the text part will start the index at 0 (we place text/plain before text/html when the mime is created). When we rewrite the links for click tracking the html part, the index is NOT reset and continues the index from where it left off with the previous text/plain mime part. This may be why you are seeing a “jump.” We hope this is helpful information, but let us know if you need more info.
Twilio SendGrid Docs Team

Hello @astuteape
Thanks for your answer. After your response, I did check another set of click events in another mail and wrote every data I though might be of interest for you.
List of URL in the template

URLs in the template
html url_offset
text url_offset

URL n°1
0
none

URL n°2
1
none

URL n°3
2
none

URL n°4
3
9

URL n°5
4
10

URL n°6
5
11

URL n°7
6
none

URL n°8
7
12

URL n°9
8
none

Data from the Sendgrid Event, json format:
[{
    "url_offset": { "index" : 0, "type": "html" },
    "urls": [ "URL n°1" ]
}, {
    "url_offset": {  "index" : 1, "type": "html" },
    "url" : [ "URL n°2" ]
}, {
    "url_offset" : { "index" : 2, "type": "html" },
    "url" : [ "URL n°3" ]
}, {
    "url_offset" : { "index" : 3, "type": "html" },
    "url" : [ "ULR n°4" ]
}, {
    "url_offset" : { "index" : 4, "type": "html" },
    "url" : [ "URL n°5" ]
}, {
    "url_offset" : { "index" : 5, "type": "html" },
    "url" : [ "URL n°6" ]
}, {
    "url_offset" : { "index" : 6, "type": "html" },
    "url" : [ "URL n°7" ]
}, {
    "url_offset" : { "index" : 7, "type": "html" },
    "url" : [ "URL n°8" ]
}, {
    "url_offset" : { "index" : 8, "type": "html" },
    "url" : [ "URL n°9" ]
}, {
    "url_offset" : { "index" : 9, "type": "text" },
    "url" : [ "URL n°4" ]
}, {
    "url_offset" : { "index" : 10, "type": "text" },
    "url" : [ "URL n°5" ]
}, {
    "url_offset" : { "index" : 11, "type": "text" },
    "url" : [ "URL n°6" ]
}, {
    "url_offset" : { "index" : 12, "type": "text" },
    "url" : [ "URL n°8" ]
}]

As you can observe, the 4 first urls don't have click of type "text", which aren't taken in account by the index of the url offset. Then, the counter don't count the 7th url either.
Why does it happen ?
If you are interested by more data, I can do the same work for the first set of data we used and on which I opened a ticket.
Also, let me know if I can be of any other kind of help.

Hi @Aridjar ,
I apologize for the delay. I will ask our team about why this might be happening. Thank you for passing the information along and for your patience.
Best,
Twilio SendGrid Docs Team

Hi @Aridjar ,
Our team is wondering if you can send a full payload to us. This seems like less of a docs issue and something we may want to move to support. Would you mind opening a support ticket and passing me the support ticket number. I can then include our support team on the context we have here to get you more direct support.
Best,
Twilio SendGrid Docs Team

Hi @astuteape ,
I would love to send you a full payload, but I don't want to display any data on a public page, as they might contain private informations. Which way would you prefer ?
Thank you for proposing to contact the support about this problem, unfortunately I came from there, as stated in the second line of the first message:

Cody S from the escalation team sent me here after they couldn't answer our questions about ticket number 4414242

Have a nice day,

Hi @Aridjar ,
We definitely don't want you to post anything with private information. That's why I think support will be a better venue for the conversation. If you pass me the support ticket number, I can follow up with support and engineering internally to continue the conversation we've started here. That way you're not starting from scratch with the support team. I just need the support ticket number and can facilitate from there.
Best,
Twilio SendGrid Docs Team

Hello @astuteape
The ticket number is #4414242 .
I replied to Cody and the escalation team so they can be prepared to reopen the ticket at your signal.
Have a nice day,

Thank you @Aridjar ,
I'm going to close this issue and follow up internally with support.
Best,
Twilio SendGrid Docs Team


WIP: JSON validation

Fixes #8
Proposed changes

Implement JSON input validation through a set of "validation backends".
A jsonschema validation backend is built-in.
Allow to register extra validation backends using @api.json_validation_backend.

TODO:

[ ] Update changelog
[ ] Add docs

Does not seem relevant anymore. Closing for now.


Merge OpenAI Triton till Aug 16th

It is essential to keep up to date with OpenAI Triton, to get the latest features, and reduce the difficulty to upstream our changes to OpenAI Triton.
This ticket is continuation of:

#1665
#1590
#1448
#1298

Moved up from Aug 2 to Aug 10 Triton version from OpenAI. [https://github.com/intel/intel-xpu-backend-for-triton/pull/1841 ]


Cell Format (Font Color, Font Style, Cell back Color ..Etc)

Hi,
Thank you for your efforts, please does that support cell formatting? and if yes, please how can I call that?
Regards,
Firas

Hi @frahman2 sorry but this library is only for basic fast read and write. Formatting isnt going to be added to the project

Hi,
Thank you, thanks for your effort.
Regards,
Firas

Thankyou for your interest, you will find a fully featured package


Fix TOC + anchor scroll

Fix table of content links (introduced by a bug in my VS Code plugin)
Add JS so that when we click on an anchor (e.g. in table of content), the scroll is not hidden by the fixed header

Do we need (want to add) all this Javascript just to not have the page hide behind the menu?
I think this could be fixed by changing the HTML for the site.  Something with the navbar, maybe
https://getbootstrap.com/docs/4.0/components/navbar/

Good point. I looked at more solutions and found one in pure CSS. We make the header non scrollable. PR has been adjusted.


Strange output from tsc noEmit - unclear where error stems from within app,

Bug Report
Hi all. Relatively new to TypeScript, but just trying to understand this output from tsc but I can't seem to get to the bottom of it. Please see terminal output below after running tsc command - appears error is in library code, rather than in app - but I'd expect a stack trace or similar to find the type error, but in fact there doesn't seem to be one?
Typescript tools in VSCode show no error in any files - does anyone have any idea what is causing this?
🔎 Search Terms
JSON errors, noEmit, tsc type check, version info, circular structure, "parent", "statements"
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about tsc output with noEmit
Tested on 4.4.3 and typescript@next

💻 Code

> tsc --noEmit

/frontend/node_modules/typescript/lib/tsc.js:88155
        return JSON.stringify(buildInfo);
                    ^

TypeError: Converting circular structure to JSON
    --> starting at object with constructor 'Node'
    |     property 'statements' -> object with constructor 'Array'
    |     index 0 -> object with constructor 'Node'
    --- property 'parent' closes the circle
    at JSON.stringify (<anonymous>)
    at getBuildInfoText (/frontend/node_modules/typescript/lib/tsc.js:88155:21)
    at emitBuildInfo (/frontend/node_modules/typescript/lib/tsc.js:87960:67)
    at emitSourceFileOrBundle (/frontend/node_modules/typescript/lib/tsc.js:87926:13)
    at forEachEmittedFile (/frontend/node_modules/typescript/lib/tsc.js:87685:28)
    at Object.emitFiles (/frontend/node_modules/typescript/lib/tsc.js:87900:9)
    at Object.emitBuildInfo (/frontend/node_modules/typescript/lib/tsc.js:94075:33)
    at Object.emitBuildInfo (/frontend/node_modules/typescript/lib/tsc.js:96965:67)
    at Object.handleNoEmitOptions (/frontend/node_modules/typescript/lib/tsc.js:95814:25)
    at Object.emit (/frontend/node_modules/typescript/lib/tsc.js:97015:29)

🙁 Actual behavior
This should produce output showing where the error is, rather than displaying either this global error, or something is wrong in the library.
🙂 Expected behavior
I'd expect more detailed info, but I might be wrong...?

@tomdev10 it's not telling you that you have an error, it's tripping over itself and crashing in a way that it should never do. Can you provide a repo where this happens?

From the stack, we can tell that what’s happening is that a parse tree node is ending up in the data that we are attempting to serialize into a .tsbuildinfo file. But that should never happen, and finding out why it’s happening is likely to be very very difficult without a debuggable repro, and probably very easy with one.

Okay fair comment, that makes sense. I will do my best to get one up and running - thanks! Will try and get it done this week.
Inidentally, trying to figure this out myself I came across extendedDiagnostics and this showing that all the node_modules are being type chcked, wondering if this might be why? Or if there is any useful output avaliable from this aid where to look?

No, --extendedDiagnostics probably won’t tell you much. There’s no rational explanation you can conjure for why this is happening because it’s just a bug, probably caused by a silly mistake somewhere.

Hey,
I'm experiencing the same issue, using<EMAIL_ADDRESS>called by<EMAIL_ADDRESS>I've narrowed down the issue to being caused by using<EMAIL_ADDRESS>in our case. Versions 4.0.0 and 4.1.0 do not cause this issue.
Hope it will help reproducing this issue as I can't share my employer's code either.

Hi @val1984 + et al. I never actually traced this, or got to the bottom of what caused it.
However, I did solve it within my repo, by removing all package-lock.json files (it was a monorepo) and regenerating those. Some contributing to the repo had been runnning npm audit fix --force and this was causing dependancies to update in an uncontrolled fashion.
After this, it worked.
You've done a better job tracing than I have so far, so I'll leave the issue open
FYI @andrewbranch

@val1984 that’s a pretty good start. Did the problem not reproduce with tsc, or do you just not normally use tsc as part of your workflow?

tsc does not exhibit the issue with:
"compilerOptions": {
  "incremental": false,
  "noEmit": true,
  "pretty": true
}

But it does have the same error as soon as incremental is set to true.
I'm able to run my project when disabling incremental mode (and I have type errors on my project because<EMAIL_ADDRESS>included some breaking changes).

I can't reproduce this just by installing<EMAIL_ADDRESS>importing it, and compiling with incremental, but it feels like we're so close. The compiler shouldn't crash like this under any circumstance, so the best thing would be to get a minimal repro that does crash so I can investigate what's going wrong. @val1984 if you're able to extract that, it would be incredibly helpful. I could also probably send you a custom build that logs out everything it’s trying to serialize into build info and maybe that would narrow it down 🤔
Barring all that, I think we’ll just have to let this issue sit and see if someone comes up with a shareable repro.

I could also probably send you a custom build that logs out everything it’s trying to serialize

Actually let me go ahead and do this so you can try it before you get into a state where it doesn’t repro anymore. Here is a copy of tsc.js from<EMAIL_ADDRESS>with the line added at 86695:
console.log(require("util").inspect(buildInfo, { depth: 32 }));

If you have time to try it, you’ll need to download it and drop it into ./node_modules/typescript/lib/ so it can find the lib.d.ts files in the same folder. Then run tsc as normal, and it will spit out a huge blob of JSON-esque output. That blob is not supposed to contain any actual source code, but the crash indicates that it might—feel free to sanitize the output before sharing it; the overall structure should give me a clue as to what’s happening.

Hey @andrewbranch,
Sorry for the long delay in replying. My project has about 100k loc and it seems that parts of the output are cut with:
... xxx more items
Is it enough data for you to investigate? Otherwise, what should I do to get you the data you asked?
Thank you very much!
debug.log

@val1984 nothing in that log looks like a problem, so maybe it did truncate the relevant parts... just to confirm, did it crash with TypeError: Converting circular structure to JSON right after that output?

Ok, let’s try this
function getBuildInfoText(buildInfo) {
    try {
        return JSON.stringify(buildInfo);
    }
    catch (err) {
        console.log(require("util").inspect(buildInfo, { depth: 32, maxArrayLength: Infinity }));
        throw err;
    }
}

Can you replace getBuildInfoText in tsc.js with this? This will only log during calls that are going to crash, and will not truncate any arrays. (The output will be a lot bigger so I would suggest piping the CLI output to a file directly if you weren’t already doing that. I always forget I should do that until I’m trying to copy and paste something very unwieldy.)

@andrewbranch The file weighs 134MB, any idea how to transmit it to you? Or maybe I can filter out most data?

You can email it to me if you<EMAIL_ADDRESS>Or you could upload to Dropbox or OneDrive or Google Drive and send me a link?

@andrewbranch Sorry, I'm not yet able to share the log with you.
On another note, upgrading to<EMAIL_ADDRESS>solved the issue (coincidentally, it includes many changes to the provided Typescript types).
And<EMAIL_ADDRESS>still crashes with<EMAIL_ADDRESS>

@val1984 feel free to redact any source file text from the log that makes it a problem to share. What I need to do with it is figure out the path from the top level of that object to the Node object, which shouldn’t be in there. I’m guessing when you look at it, you see a huge repeating cycle. I just need to trace that cycle to the top.

Hey @andrewbranch ,
we are also experiencing this build crash. Our log with your code snippet from 15 Nov 2021 is 128MB big with more than 170k lines.
We have a NextJS application with formerly Preact working fine as intended. We are now willing to switch back to React, and now this error occurs during build. I also tried some older versions of<EMAIL_ADDRESS>and the current nightly (2nd January) but with no luck.
I might can send you the log via email, but I need to talk to team members before.

Hey @andrewbranch,
we figured out what the problem is. We had a component, which was included with dyanmic-Import from NextJS. In the imported Component we had a type children: Array<React.ReactChild>; which was false for our case, only single items could appear here.
When we resolved the dynamic import to a normal import, we get the correct message from typescript TS2745: This JSX tag's 'children' prop expects type 'ReactChild[] & ReactNode' which requires multiple children, but only a single child was provided.
Maybe @val1984 can approve, if he has similar code which leads to this error.
Best, Florian

@fmacherey I’m glad you found a workaround, but you didn’t find “the problem” per se. The problem is and will forever remain a mystery until someone can send me that gigantic log file.

@andrewbranch just to clarify, I used the normal import for debugging, but switched back to dynamic after fixing the types. I think typescript has problems with next' dyamic import to correctly resovle the types.

This really has nothing to do with imports, resolving types, Next.js, incorrectly supplying an array where a single child should go, or any particular piece of end-user code. There is a very strange bug with serializing complex state during incremental builds, and you’ve happened to find a lever on one end of a Rube Goldberg machine that toggles the problem from a mile away. What’s happening between between that lever and the real problem is not inferable by a human. All I can tell you is that the lever itself is not relevant on its own, or attached to any other machine except the precise one made by your program, and the only way we can understand what’s happening is by observing the whole machine. I have tried to encode clues about its operation in the log file generated by the patched TS version I made, but I’ve yet to see one of these logs ☹.

@andrewbranch I just hit this in a similar setup. I'm using Next.js with Yarn PnP. I was also messing with children in some component I was editing, and restricting the types of children that could be passed.
Here's a link to the output of running tsc with a replaced buildInfo . It's happening on this branch of my personal website repo if you want to explore source code! Thanks!
This is what happens without the logs:
/Users/dgattey/checkouts/dg/.yarn/cache/typescript-patch-0b90225f97-2e488dde7d.zip/node_modules/typescript/lib/tsc.js:88443
        return JSON.stringify(buildInfo);
                    ^

TypeError: Converting circular structure to JSON
    --> starting at object with constructor 'Node'
    |     property 'statements' -> object with constructor 'Array'
    |     index 0 -> object with constructor 'Node'
    --- property 'parent' closes the circle
    at JSON.stringify (<anonymous>)
    at getBuildInfoText (/Users/dgattey/checkouts/dg/.yarn/cache/typescript-patch-0b90225f97-2e488dde7d.zip/node_modules/typescript/lib/tsc.js:88443:21)
    at emitBuildInfo (/Users/dgattey/checkouts/dg/.yarn/cache/typescript-patch-0b90225f97-2e488dde7d.zip/node_modules/typescript/lib/tsc.js:88248:67)
    at ...

Thanks @dgattey—as I suspected, that file shows me exactly what the problem is 🎉


See which worker accounts are possibly banned?

If you have mutliple accounts set up it sometimes says "possible ban" in the status box. Is there a way of being able to see which account it is trying to use at that moment so that i can remove that account from the list.

Then when you START your server,
look at the codes at the top. Successful should look like this...
main.html:169 2016-09-19 07:21:59,016 [ search-worker-0][      auth_ptc][    INFO] PTC User Login for: pokemon12345678
main.html:169 2016-09-19 07:21:59,682 [ search-worker-0][      auth_ptc][    INFO] PTC User Login successful.
Look for something that might say unsuccessful or fail.

Also, FILTER and look for consistent 0 search results over long period of time
Parsing found 0 pokemons, 0 pokestops, and 0 gyms

I don't understand the point of the /status page. When I type in the password I set it just shows a blank screen. Is this a defect or is there just something wrong with my setup?


Reproducible Trunk inf loop. Try it yourself

Hi, I wanted to make use of css-modules while porting my react site to yew + trunk. I could not get this to not go into an infinite loop. I even tried setting it so that it doesn't reload automatically in Trunk.toml as well as telling trunk to ignore the dir with the css file that is created by css-modules.
$ trunk --version
trunk 0.16.0

Anyway, let's get to the code:
$ cargo new inf-trunk-test

./html.index
<!DOCTYPE html>
<html>
    <head>
        <meta charset="utf-8" />
        <meta name="viewport" content="width=device-width, initial-scale=1" /
        <title>inf-trunk-test</title>
        <!-- rust wasm compilation options for trunk -->
        <link data-trunk rel="rust" data-wasm-opt="s" />
        <!-- css files -->
        <link data-trunk rel="css" href="css/app.css" />
    </head>
</html>

./Trunk.toml
[serve]
port = 8080

[[hooks]]
stage = "pre_build"
command = "sh"
command_arguments = [
    "-c",
    # trigger cargo build script (which generates css assets) before running Trunk build, because
    # Trunk runs asset pipelines (css and rust) in parallel. specify same target that is used by
    # Trunk so it can reuse compilation results (ie. not recompile css and rust twice)
    "cargo check --target wasm32-unknown-unknown",
]

./cargo.toml
[package]
name = "inf-trunk-test"
version = "0.1.0"
edition = "2021"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[dependencies]
yew = "0.19"
css-modules = "0.5"

[build-dependencies]
css-modules = "0.5"

./build.rs
use css_modules::CssModules;

fn main() {
    let mut css_modules = CssModules::default();

    // Include all CSS files in the src dir:
    css_modules.add_modules("src/**/*.css").unwrap();

    // Compile all modules and export CSS into one file:
    css_modules.compile("css/app.css");
}

./src/main.rs
use css_modules::include_css_module;
use yew::prelude::*;

fn main() {
    yew::start_app::<App>();
}

#[function_component(App)]
fn app() -> html {
    let css = include_css_module!("test.css"); // relative path to your CSS

    html!(
        <div class={css["Test"]}>
            {"hi"}
        </div>
    )
}

./src/test.css
.Test {
    color: red;
}

Here is a snippet of what to expect:
$ trunk serve

Yea, there appears to be an issue with recursively copying directories right now. We are attempting to track down what is causing this. Thanks for the report!

I think I am having the same problem.

I have one more example. I was trying to configure dotenvy_macro = "0.15.7" to set up some env vars during compile time so that the production\test builds have correct values, but it resulted in infinity loop for the test build
Trunk.toml
[[hooks]]
stage = "pre_build"
command = "sh"
command_arguments = ["-c", "echo 'BACKEND_ENDPOINT=http://backend:8081' > $TRUNK_SOURCE_DIR/.env"]

[[hooks]]
stage = "post_build"
command = "sh"
command_arguments = ["-c", "echo 'BACKEND_ENDPOINT=http://localhost:8081' > $TRUNK_SOURCE_DIR/.env"]

it works fine when I run trunk build --release but trunk serve goes into an infinity loop

I believe the fix in https://github.com/thedodd/trunk/pull/516 should work here too. I have this running for a while and don't have those issues any longer.
Unfortunately that PR doesn't seem to get any attention, so am leaning towards forking the project and releasing this on crates.io, because we do need this in our team. (and please don't see this as a complaint! we all have limited time!) But the more interest there is, the more this makes sense to me.

I have a similar looping problem.
A build-dependency lib (in the same tree) reads some csv-files (within the lib-folder), according to trunk -v serve the one of the csv-files is always changed and triggers a build. The file is not copied to the web but used to create static data.
With an older version it stopped after some time, with 0.16.0 and 0.17.3 I've stopped it after 5 tries.

And there is a second way to fix this in PR https://github.com/thedodd/trunk/pull/569 … which also isn't getting merged :shrug:


Added field name in "label for" and "input id"

[Forms] When you enter more than one radio field of the same value, a conflict occurs because they have the same id.
Example:
Field: Display
Options: Yes (id: option-Yes) | No (id: option-No)
Field: Featured
Options: Yes (id: option-Yes) | No (id: option-No)

Thanks! 👍


Fix syscall 610 to return a proper process type

According to https://github.com/obhq/obliteration/issues/409#issuecomment-1775588844 it seems like we are currently booting the libkernel in system application mode, not a game mode. Right now we already have IDT implementation so it should not too hard to fix syscall 610 with a proper implementation.

So, I assume no syscall skip-aheads until this is implemented

Let's see how far we can go without a proper implementation of this.

...Well, good thing is my replacement Mobo is coming today. I got a new full-atx b550 board to replace my matx b450 board, however, the board was DOA x3

Fixed by #469.


How to add Audio to different scenes within this example I have viewed #964 still not getting

<!DOCTYPE HTML>

    <meta charset="utf-8">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <title>Tour</title>
    <link rel="stylesheet"<EMAIL_ADDRESS>

Copying and pasting an existing example into an issue accomplishes nothing. I have no idea what you've already tried or what issues you're having, so I can't provide any help.


Add argument parser for script automation

Describe the feature
Adding an argument parser to its initial functionality.
Motivation
As some might want to automate the steps to create label files, It'd be handy to be able to execute the tracklet parser via command line with arguments. E.g.
python parser.py --labels <tracklet_labels.xml> --frames <frame_list.txt> --out <output_dir>

Hi,
Thanks for the tool.
As I saw in the git library of the tracklet_parser, these is no file named parser.py.

Is it the file named tracklet_parser.py in the "https://github.com/holtvogt/tracklet_parser/tree/develop/tracklet_parser" directory which is to be used to run it from the command line?
Best Regards,
Ajay

It is not implemented yet, thus there is indeed no file named parser.py. It was an idea I wanted to capture when I worked on it during my bachelor thesis to simplify the parsing process. It should basically add the ArgumenParser so that the TrackletParser can be used from CLI

Thanks for the response @holtvogt.


Overwriting signInPopup function to make it work with firefox background page

I am writing a browser extension which has this library in the background page. It works fine on Chrome and signInPopup opens up popup correctly, but this does not work on firefox and the popup is blocked. So as a workaround I want to use firefox webextension API to open a new window instead of popup like this:
browser.windows.create({
    url: ["https://developer.mozilla.org"]
});

So my question is, how can I achieve this without forking oidc-client-js.
My thoughts were to replace some internal function, for example:
userManager._popupNavigator.callback = function(url) {
browser.windows.create({
    url: [url]
});
}

then I can call
userManager.signinPopup()
What are your thoughts?
Thank you

Actually I tried this, but calling sinInPopup() does not call _popupNavigator.callback
Can you please tell me where could I place my code without breaking other signIn methods to achieve my goal when calling signinPopup() ?

if anyone wondered how I solved this. I have reimplemented auth in my extension by myself without this library. And I am using webextensions API.
browser.identity -> https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/identity
browser.identity. launchWebAuthFlow -> this function launches a popup window to auth.

Sorry, did you ever sort this out?


add with_main_gpu to LlamaModelParams

This commit adds a with_main_gpu method to LlamaModelParams which allows the main GPU to be set.

Could you also add a main_gpu to allow it to be read as well?


SMTP with TLS Send Fails

I've got an SMTP server configured, running off master (so I have the latest initializers as in #160 ).
My worker log is indicating a failure to send:
[Worker(host:ip-172-31-31-156 pid:14671)] Job Issue#send_notifications (id=4) FAILED (0 prior attempts) with Net::SMTPAuthenticationError: 530 Must issue a STARTTLS command first
In theory by default the SMTP initializer should be using STARTTLS if its required, so I'm not sure why this is happening.

@adamcooke any chance you have any thoughts on this?


Docs for kubernetes tls integration not clear

Hi Guys
So I just implemented the TLS integration into a kubernetes cluster with traefik.
I followed the steps given by your docs
However I needed to add the following lines to the traefik daemonset:
               - containerPort: 443           hostPort: 443           name: https           protocol: TCP
Don't you think this should be added to the docs?

area/documentation

Closed by #6457.


tanzu apps deliverable get returns deliverable details

Description of problem
PLACEHOLDER -- details pending...

Proposed solution
<Please describe your proposed solution, preferably in the following style:>
Given <Some Condition>
When <Something Happens>
Then <This other thing should happen?>

Example
<Code snippets that illustrate the when/then blocks>
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Deliverable info is now included in workload get so we decided to not implement a separate sub-command for this for now.