Add-in Error: This add-in is no longer available....

Article URL
https://docs.microsoft.com/en-us/office/dev/add-ins/tutorials/excel-tutorial#protect-a-worksheet
Issue
Completed the tutorial up to the Protect A Worksheet section.  I have followed the instructions to delete caches files (I've deleted all of the files and sub directories from all of the mentioned locations) but am getting an error in Excel which says, "Add-in Error.  This add-in is no longer available.  If this add-in's manifest was removed, please restore it to the previous location and try again."  I've not moved any files, I've only made the changes as per the instructions (as far as I know).
MacOS: 10.14.6
Excel: 16.41
Output of npm start:

<EMAIL_ADDRESS>start /Users//Documents//Projects/office_add_ins/My Office Add-in
office-addin-debugging start manifest.xml

Debugging is being started...
App type: desktop
The dev server is already running on port 3000.
Sideloading the Office Add-in...
Debugging started.
I'd also like to add that it would be very helpful if the getting started linked off to details for "why" each step is being performed.  There really isn't any explanation about why deleting a cache is necessary in this case, when it would be necessary in any case, so its hard to know (especially as someone trying this for the first time) to know where to look next.

Hi @octocat1000,
Thanks for the feedback. We are looking into this, but I wanted to check in with you about one thing. Your npm start output included "The dev server is already running on port 3000." That leads me to believe the web server is still running, even after the cache clear. Do you encounter the same problem if you give the background process a couple minutes to shutdown (or even restarting your machine) after closing Node.js? That information will help us diagnose the root problem.
Thanks!

Hi @AlexJerabek,
Thanks for getting back.  Just now, I think I've resolved this.  Here is what seems to have been the issue and the things that I tried:
I think that the root culprit was a typo in my manifest wherein I had transposed a couple letters is one of my resid.  After spotting this, deleting all of the cache files and then reloading the app it worked.
I tried turning on debug logging as per the instructions here but no log file was generated.  I also run the manifest validation as per the instructions here and it passed.  So... not really sure what is a good way to find or debug these kinds of errors...  Any ideas?

Let me assign @Rick-Kirkham to investigate the manifest validation issues. @octocat1000, can you please share the issues in your manifest, so we can recreate the problem? Thanks!

Hi @Rick-Kirkham.  I believe that I had just been following the instructions in the tutorial here.  I was on the step to add second taskbar items to the manifest and, if I recall correctly, I had transposed the "el" in "ProtectionButtonLabel".
In this section:
   <Supertip>                     <Title resid="ProtectionButtonLabel" />         <Description resid="ProtectionButtonToolTip" />     </Supertip>

I'm not clear on the issue. Were you expecting that the manifest would not validate given the typo? I don't think that would prevent the manifest from validating. A resid is just a string that you choose. The validator will accept any string.

Hi Rick, I have no particular expectations of the manifest validation tool.  However, my question at the start of this thread was how should I be able to find such issues.  In my case, there was no error message anywhere that I could find that led me to understand the cause of the add-in not loading.  So, if the validation tool doesn't catch such errors, how does one come to understand when they occur?  Not sure who is best to answer this.

I think that I must be missing something.  No system can read a programmer's mind and know what they intended to type.  If a typo causes an error, the system can't tell you that you typed "le" when you meant to type "el".
Maybe this is why bug forms have sections labelled "Expected" and "Actual". What is your "expected" behavior that you would like to see when you have a typo in a resid?

Rick, it might be good if you read the whole thread.
Ultimately, it came down to a typo in the manifest.  However, the initial error was, "Add-in Error. This add-in is no longer available."  It happened in the same point in the tutorial that talks about having to clear caches by deleting files in multiple locations.  So, at first, I wasn't sure at all why I was seeing this message (i.e. had I delete the wrong files or something).
So, the question now is, how to debug or find logs in order to find this sort of problem.  As I mentioned, I didn't find any logs or anything that pointed me to the cause.  At least, I would hope, there would be a log message somewhere that says there is an error occurring, some clue, something.  As also mentioned, I'm working on a Mac and maybe there is less logging and, it seems, fewer debugging options vs. on Windows?

This might make me look worse in your eyes, but I actually have reread the whole thread before writing each of my comments. As far as troubleshooting the error in the issue title, I searched for it in Bing and found a number of causes, mostly connected to resids. I think the best I can do is add a section to this article, Troubleshoot development errors, about this error and describe the causes/solutions to the extent they are known. Does that sound like a good idea?
BTW, in case  you didn't know this, we do have a system of runtime logging that works on Mac and can catch some issues with manifests. See Debug your add-in with runtime logging.

Removing activity status, so the robot won't delete this prematurely.

I have the same issue. Trying to start the add-in on Mac but getting the following error:

I'm running the command office-addin-debugging start manifests/office-dev.xml desktop. It used to work but stopped working for no apparent reason.

Hi @ralphtheninja,
Thanks for reporting this. Can you please create a new issue on the https://github.com/OfficeDev/Office-Addin-Scripts repo to track this? That'll ensure the right people from yo office are looking into the problem.

@AlexJerabek How about re-opening the issue and transferring it to the other repository?

Quick re-cap and background
I have the following scripts in my package.json:
"scripts": {
  "start:excel": "office-addin-debugging start manifests/office-dev.xml desktop --app excel",
  "start:powerpoint": "office-addin-debugging start manifests/office-dev.xml desktop --app powerpoint",
  "start:word": "office-addin-debugging start manifests/office-dev.xml desktop --app word",
  "start:outlook": "office-addin-debugging start manifests/outlook-dev.xml desktop"
}

Currently Word is working great, npm run start:word and boom word shows up with the add-in.
But when I try to start the addin for Excel or PowerPoint I get the error that the add-in is no longer available. If there's some old stuff laying around that should be cleaned up I could simply just try to wipe that information before executing the script as a workaround, if I only knew what needs to be done to get into a good state.
The best way would probably have the office-addin-debugging do the clean up needed.
Word works all the time, and I can reproduce the errors in Excel and PowerPoint 100%.

Also see https://github.com/OfficeDev/office-js/issues/256

Following the troubleshooting section here https://docs.microsoft.com/en-us/office/dev/add-ins/testing/troubleshoot-development-errors, I couldn't get any of the paths to fix my problem.
But I found a quick and dirty solution. On Mac I removed the complete ~/Library/Containers/com.microsoft.Excel folder and it started working again. Doing this is like starting Excel for the first time, since you're greeted with a lets get started dialog.
Similarly, removing ~/Library/Containers/com.microsoft.Powerpoint solved the problem for PowerPoint. I'm guessing it works for Word as well.

Sure thing @ralphtheninja.
@millerds, can you please take a look at this issue, paying particular attention to the information from @ralphtheninja starting on June 3rd? Thank you!

@ralphtheninja deleting files in that folder is not working for me either. While creating Office add-in task pane project using react framework works fine, every time I try create excel custom functions add-in project I get the same error.. add-in no longer available, manifest removed...
I am a mac user. Whats strange is that I can swap in the manifest of the task pane react framework for the manifest created by the custom excel add-in and then excel finds the manifest again. But that manifest then doesnt contain the option to write functions. Similar to original issue in https://github.com/OfficeDev/office-js/issues/256#thread-subscription-status its still fine if only changing certain parts of manifest but then get the error again when changing other parts.
I dont know why excel keep pointing to the manifest created for react framework even though I have deleted the manifest, and all cache. Manifest debugger also says the new manifest has no errors.

Well................... I'll be damned. I solved this by setting Excel to be Run as Administrator, even when it is being opened by visual studio is debug mode and the error is gone.


Update bypassed status when checking role/user checkbox

Currently the bypassed status does not update by checking the box.
Example: request gets replayed with lower access, however it's supposed to be available for the access level expressed in the tickbox, the status should change to "enforced" without having to run analyze again

Agreed!

Closed via https://github.com/caido-community/authmatrix/pull/20


Speedup Features - Start with quest items

Option to start with random amount, % chance to have them on slider (so 100% would be guaranteed to start with all of them)
Each one with separate slider?
Runes
Stones
Three Part Keys
Bell, Book, Candle

but...but...why even play the game then?

It is a request from Twitch Streams. They want to be able to play the entire game/randomized game in a shorter amount of time to fit in a single stream. Shorter also opens the possibility of streamers racing each other. Which would be cool if it ever happened.

...okay... ?

shrugs There are lots of options to make things harder. I am okay with adding some to make it easier too. Then people can mix and match the options to their heart's content.

This is done.


Pivot Table Format issue - 1.4.1

Facing format issue for the pivot chart. Screenshot attached, if change the zoom size of the browser +/- the format is getting corrected.
Actual results

Environment
(please complete the following information):

browser type and version:
superset version: 1.4.1

Checklist

Closing this because it's stale, because it's regarding the legacy/deprecated pivot table, and because I can't seem to cause this in either the new or deprecated chart anyway. Happy to re-open if anyone is still facing this.


Set Scale lower limit for vertical axis of Hemoglobin violin plots

When hemoglobin is really low or ~ 0, scale goes to 5 decimal points, obscuring axis label on the left.

Should be fixed in e7b17bd56f9f87741363c633174928347da354cf.


Improve start cake script to store remote server settings

The remote startup configuration exports (username, database, password,etc ) may be saved and loaded as the developer needs the script. A user prompt reads for new values and saves them locally or load defaults from the local script settings. (Start-cake.Sh)

PR #18


[CONTROLLER/DB] adds npb_tunnel field

This PR is for:

Server

adds npb_tunnel field
Checklist

[ ] Added unit test.

Backport to branches

main

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.zhengya seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Fix numbered list issue.

The numbering of an ordered list appears to get reset after a code block in GitHub markdown. So this PR fixes the issue by using an unconventional numbering method that is not affected by the code blocks.

@cmpolis

@cmpolis


Added round function to gles2

Fixes: https://github.com/godotengine/godot/issues/24850
Implements the round() function in GLES2. Something similar can be done with a few of the other GLES3-only features found in http://docs.godotengine.org/en/latest/tutorials/misc/gles2_gles3_differences.html#shading-language
Putting this up first to see if this is the implementation we want to follow. Then I will make a PR with a few others.
Implementation Notes:
Right now, I have copied the implementation from the mix(a, b, bool_type) override. The shader compiler replaces calls to round() with calls to roundx(), which are defined in stdlib.glsl. This implementation works and is safe to use. However it may be unnecessary.
An alternative would be to just provide an overloaded round() function in stdlib.glsl without changing anything in the compiler. This seems to work, but some GLES2 devices that deviate from spec allow users to use round(), so I'm worried that it would crash on those devices. That being said, it is a much more elegant solution.

My only fear with adding more code to the base shaders is that versions will take longer to compile because more needs to be parsed. Maybe having some ifdef around these being used may work?

Thanks!


SSL errors on monit 5.20

php /home/graph/web/cron.php
[MonitGraph] /home/graph/monit-graph/web/monit-graph.class.php line 154: cURL Error (35): error:14094410:SSL routines:SSL3_READ_BYTES:sslv3 alert handshake failure
Commenting out lines:

curl_setopt($ch, CURLOPT_SSLVERSION, 3);
if($verify_ssl) curl_setopt($ch, CURLOPT_SSL_VERIFYHOST, 2);
else curl_setopt($ch, CURLOPT_SSL_VERIFYHOST, 0);

and add line:

curl_setopt($ch, CURLOPT_SSL_VERIFYHOST, 0);

Fixed problem. At this moment, I think ssl version should be configurable per connection and default should be TLS 1.0 or 1.1, not SSL v3.
Thanks for this software :+1:

Thanks! I've removed CURLOPT_SSLVERSION option entirely, so the default is used instead (not to mention SSL v3 is horribly outdated and vulnerable).
You can set the verify_ssl to false to prevent it from checking:
[0]
name                    = "localhost"
server_id               = 1
config[url]             = "localhost:2812"
config[ssl]             = true
config[http_username]   = "admin"
config[http_password]   = "monit"
config[verify_ssl] = true

Heads up: I'll probably rename this value in the future, planning to get a full rewrite done as soon as I've the time!


unknown driver gojdbc forgotten import

I'm attempting to use go-jdbc to access and Ingres database using iijdbc.jar, but getting the error "sql: unknown driver "gojdbc" (forgotten import?"
Will you provide a usage example?
package main
import (
"database/sql"
"log"
_ "github.com/japettyjohn/go-jdbc"

)
// NewDB establishes a new Ingres JDBC Connection.
// The are no args. NewDB returns *sql.DB.
func NewDB() *sql.DB {
dbh, err := sql.Open("gojdbc", "jdbc:iijdbc://HOSTNAME:II7/DBNAME;UID=ark;PWD=secretPassword")
if err != nil {
log.Fatal("DB connection problem", err)
//panic(err.Error())
}
return dbh
}
// CloseDB closes the database connection.
// The argument is the database handle to close.
func CloseDB(dbhandle *sql.DB) error {
err := dbhandle.Close()
return err
}

The driver is registered as "jdbc", not "gojdbc". You can see that here.

Thank you so much. I read line 22 of driver_test.go and completely missed that.

Use the URL, if it's ijdbc - https://www.dbschema.com/ingres-jdbc-driver.html.

Hi all, i see this questions was closed here. I am also trying to use this repo to connect to ingres. My question is where and how do i specify the JAR file path and the JDBC_DRIVER_CLASS?
I am sure the code would need these setting to connect?
For now this is what I have..
db, err := sql.Open("jdbc", "jdbc:ingres://:/;user=;password=")
if err != nil {
log.Fatal("DB connection problem", err)
}
I do now get and error but when i try and use the db lets say 'err := db.Ping()' I get this error
dial jdbc: unknown network jdbc
Pretty sure i am missing something


Macbook pro M3 kuburnetes performance very slow

Starting maybe a month ago, kuburnetes is running very slow on my macbook.
Deployment to orbstack is very fast. Once inside the api container when i try to run installs, for example installing python requirements, the speed is incredibly slow.
My Macbook specs =>

Macbook Pro M3 2023
18GB memory
Sonoma 14.4 (23E214)
ARM 64, 12 cores

I tested this in various ways

Inside of Kubernetes api container. Ran installs, for example, python requirements install. Incredibly slow, I'm talking 15-20 minutes.
Outside of kubernetes api container, I spun up a docker compose and opened a container throught this. Same installs were incredibly fast, like 5-10 seconds.
Inside of kubernetes container, pinged a url. Ping was very high and rate of new pings inconsistent.
Outside of container, in my terminal, pinged same website. Ping was low and consistent.

Diagnostics report
OrbStack info:
Version: 1.5.0
Commit: afc6a0a9a22761c33329835c75a3b42a4ceb29b8 (v1.5.0)
System info:
macOS: 14.4 (23E214)
CPU: arm64, 12 cores
CPU model: Apple M3 Pro
Model: Mac15,6
Memory: 18 GiB
Full report: https://orbstack.dev/_admin/diag/orbstack-diagreport_2024-03-27T06-31-16.086408Z.zip

How to replicate

Deploy Kubernetes
Once deployed, open api container in VScode
in container terminal run 'make install' from make file

install:
   apt-get update -y
   apt-get install build-essential -y
   pip3 install -r requirements.txt
   cd /mnt/my-app && make

the install takes a very long time. Then I run 'make dev' which is to start up the dev container environment for the api and this will also take quite awhile =>

dev: format
	export DJANGO_SETTINGS_MODULE=settings.dev && \
		export API_PORT=8001 && \
		cd ${MAKEFILE_DIR} && \
		python3 main.py

Please share a diagnostic report from Help > Upload Diagnostics as required by the issue template. Also share exact commands to reproduce this from scratch.

Please share a diagnostic report from Help > Upload Diagnostics as required by the issue template. Also share exact commands to reproduce this from scratch.

Ok updated

How are you "deploying Kubernetes"?

Deploying kubernetes with a 'python3 localhost.py' script, I cannot share the script but it deploys our dashboard, api and schema. It logs in to AWS ECR cn-north-1 and pulls the images and applies a k8s config.
My colleague also on macbook, but on intel version has no issue with kubernetes and the same processes I outlined above are very fast for him.

traceroute to api (API IP), 30 hops max, 60 byte packets
 1  19X.XXX.XXX.X (19X.XXX.XXX.X)  0.263 ms  0.014 ms  0.004 ms
 2  19X.XXX.XXX.X (19X.XXX.XXX.X)  0.117 ms  0.012 ms  0.008 ms
 3  * * *
 4  * * *
 5  * * *
 6  * * *
 7  * * *
 8  * * *
 9  * * *
10  * * *
11  * * *
12  * * *
13  * * *
14  * * *
15  * * *
16  * * *
17  * * *
18  * * *
19  * * *
20  * * *
21  * * *
22  * * *
23  * * *
24  * * *
25  * * *
26  * * *
27  * * *
28  * * *
29  * * *
30  * * *

And yeah i believe we are using the same proxy settings and same VPN

Please share a screen recording of the issue. (feel free to email<EMAIL_ADDRESS>if sensitive)


LOG4J2-2025: JUL bridge handler (incl. unit test and site doc)

Solution for LOG4J2-2025 (as alternative to JUL-LogManager).
Rest see JavaDoc and site-doc (index.md)
This runs with log4j 2.11, so maybe can be even release with 2.11.2 (and not 3.0).

This overall seems reasonable, but I'm concerned about licensing. If any code is copied from Logback, then it's not allowed inside an Apache project. The EPL and LGPL, while neat, are both weak copyleft licenses which can only be used in optional dependencies in Apache projects.
We would either need this contribution to be fully ALv2 (or compatible), or this would need to remain a 3rd party component unfortunately.

Documentation is enhanced and I removed the author tags.
Regarding logback and license: I did not think about this, but your are right. Currently the level propagation is not implemented, so nothing from logback is used (this the outcommented author tag for it); it is just a hint for later. Even if this comes, there will be no problem because the implementation is quite trivial and has to be adjusted to log4j's internals, so jut the idea will be from logback's implementation.

Some questions to the log4j experts regarding the automatic log level propagation:
My idea is to have an interace LoggerListener or maybe better named LoggerConfigListener that is called by log4j's implementation at the relevant places:

configApplied():  initialisation of log4j and readout of configuration (log4j2.xml ...) is finished and applied, so one get get the current info via ((org.apache.logging.log4j.core.LoggerContext) LogManager.getContext()).getLoggers() Maybe the LoggerContext may be given as parameter. Method name could also be initFinished().

Following methods are only called after initial creation, i.e. after calling above method:

added(Logger):  the given logger has been added, called after its configuration (log level) is set
remove(Logger):  the given logger will be removed, so JUL logger level can be set to null - if removal is possible in log4j at all
setLevel(Logger):  log level has been set/changed, thus no extra parameter because logger.getLevel() may be used

There must be a method (in LogManager?) to set/add this listener. E.g. the init. code of Log4jBridgeHandler will call it.
I have not found any possibility to hook into the relevant places of log4j's implementation to allow log level propagation. So this is a generic solution. What do you think?

A rainy and cold weekend in northern Germany helped to complete this.
From my point, the implementation is finished - but do your review, please.

I just looked at the statistics and 13% of last month's downloads were for 2.3 so I guess a fair number of people are still stuck at Java 6. Although I have no problem providing support for bugs we would have to discuss whether we want to add new features. It has been so long since this was released I am not even sure how much of the release process has changed, if at all.
I personally have no problem providing a link to your project. However, you should not be using the org.apache.logging package name as that may confuse users into thinking it is supported by Apache Logging Service. Of course, if we agree to incorporate it then no change is required it is integrated into Log4j.

Okay Ralph, nice to get this number (my Java6 project is now already migrated to Java8, but I have this code lying around).
If you want to create a "2.3.1" release with my stuff included, fine. I will create a pull request then.
But if this is to complicated because the release process "has gone", no problem - log4j is an open source volunteer project.
I will adjust the package of my addon-project then to prevent wrong support thinking.
Simply tell me your (or log4j's team) intent.


docs(README): point SBFspot link at "new" site

Because CodePlex has been in archive mode since 2017 and is supposed to
have been taken down in July this year already.

Please create a pull request in the upstream repo https://github.com/kellerza/pysma

Ah, sorry to have bothered you then.
FYI: I got to this specific repo because of the link on this page https://www.home-assistant.io/integrations/sma/#sensors

FYI: I got to this specific repo because of the link on this page https://www.home-assistant.io/integrations/sma/#sensors

Thanks! Will fix the docs


Minor patch for DocumentSelection

Found a small bug where DocumentSelection MenuItem was using key instead of value for the value attribute. So the type value would be correct with "Bank Statement" instead of "BankStatement", "Driver's License" instead of "DriversLicense", etc. These changes should fix things without breaking anything.
https://github.com/codeforpdx/PASS/assets/14917816/de1cd54c-f993-45e8-b5ef-d5f143f9de9e

Closing PR. Could potentially use type for querying purposes.

A good enhancement to make here would be to find RDF types to save instead of these strings. We could research what would be the best types for that.


Upgrade heroku stack

Heroku says:
Cedar-10 will reach end-of-life on November 4th, 2015.
!   Upgrade to Cedar-14 at your earliest convenience.
!   For more information, check out the following Dev Center article:
!   https://devcenter.heroku.com/articles/cedar-14-migration

Done


Left-right hotkey allows you to go past the time range in the graph

You can go to the right into the future, past the time displayed in the line chart.  Can we limit it to be so that it stops at the left and right boundaries of the chart which is displayed?
If this is hard, it isn't critical for July release, and we can tackle later.

@Alaatohamy @dlinah @dctanner, would be great to briefly discuss this in todays meeting.
Not a problem if you don't have time.
This is the only one above Cypress testing

Im just messing with this on app.nowcasting.io, and im now sure this is working yet. seem to still be able to go past the end.


Add support for Azure, OpenAI, Palm, Anthropic, Cohere Models - using litellm

I'm the maintainer of litellm https://github.com/BerriAI/litellm - a simple & light package to call OpenAI, Azure, Cohere, Anthropic, Replicate API Endpoints
This PR adds support for models from all the above mentioned providers (by creating a class liteLLM)
Here's a sample of how it's used:
from litellm import completion

## set ENV variables
# ENV variables can be set in .env file, too. Example in .env.example
os.environ["OPENAI_API_KEY"] = "openai key"
os.environ["COHERE_API_KEY"] = "cohere key"

messages = [{ "content": "Hello, how are you?","role": "user"}]

# openai call
response = completion(model="gpt-3.5-turbo", messages=messages)

# cohere call
response = completion("command-nightly", messages)

# anthropic call
response = completion(model="claude-instant-1", messages=messages)

cc @ciuzaak can you take a look at this when you get a chance ? Happy to add tests and docs based on your feedback

Very good work, I will try to put it together when I have time, thank you

@ciuzaak was litellm missing something to be useful here?
Any feedback would be helpful!


str object not callable

Encounter the following error:
jiraffe -h
Traceback (most recent call last):
  File "/usr/local/bin/jiraffe", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.7/dist-packages/jiraffe/__main__.py", line 48, in main
    parser = argparse.ArgumentParser(formatter_class=argparse.RawDescriptionHelpFormatter, description=style.GREEN(banner) + style.RESET(''), usage=style.GREEN("jiraffe ") + style.YELLOW("[-h] [-t {}]").format(style.UNDERLINE("https://example-jira-instance.com") + style.RESET('') + style.YELLOW('')) + style.RESET(''))
TypeError: 'str' object is not callable

Quickfix on Line 8:
        if sys.platform.lower() == "win32" or "linux":
Somehow something with the color setup is going worse...

I had the same problem on my Kali, but I had to use "linux2" instead of "linux"

Implemented a hot-fix, should do the job.

Please, add "darwin" to the list to fix the issue for OS X users.

I'm also getting the same error "str object is not callable " while running "Jiraffe" on Mac. Please suggest how to resolve the issue. Thanks!


JoinedLoad of labels and notes

In a few methods we are not using a joinedload on labels and notes. See

_get_concept_scheme: Returns notes and labels but doesn't load them with the concept. Forcing 2 extra db queries
get_by_id: Returns a concept. Joined loading the most needed relations (labels and notes) would save 2 queries.
get_by_uri: Returns a concept. Joined loading the most needed relations (labels and notes) would save 2 queries..

@BartSaelen If you have no objections, I'd let @TalissaJoly handle this.

Some quick tests show that this improves performance with a postgresql database. But it might reduce performance for an in-memory SQLite database. Still, that's not a common usecase.


Integrate puppet-lint into our linter suite

I just did a bunch of cleanup to our puppet manifests using the standard puppet-lint package.  We should integrate it into our tools/lint-all linter framework, using its exclude options for the few things we haven't fixed yet.

@zulipbot claim


cancel button + function

bdd cancel function

Codecov Report

Merging #101 (74a1373) into main (913c2f1) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #101      +/-   ##
==========================================
+ Coverage   94.13%   94.15%   +0.01%
==========================================
  Files           5        5
  Lines         341      342       +1
==========================================
+ Hits          321      322       +1
  Misses         20       20

Impacted Files
Coverage Δ

service/routes.py
94.61% <100.00%> (+0.04%)
:arrow_up:

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 913c2f1...74a1373. Read the comment docs.


[main] Upgrade to latest dependencies

v1 -markusthoemmes
/cc knative-sandbox/client-writers
/assign knative-sandbox/client-writers
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/client 09d48d5...a1cf5bf:
  > a1cf5bf Update Serving and Eventing versions in E2E tests (# 1507)
  > da6f164 Remove lookup-plugins flag (# 1506)
  > c1e3589 Fix config init for non-writable locations (# 1502)
  > 3d43c13 Update changelog for 1.0 release (# 1504)
  > d0c13a2 upgrade to latest dependencies (# 1503)
  > 7de7338 Fix fetching of Istio latest release (# 1505)

/approve
/lgtm


Added Japanese Capabilities

I did a raw translation of the English file to Japanese using google translate. I then went through the entire document and deleted all words which were translated incorrectly. I did simple tests to make sure the system was working, and it captures negative/positive Japanese sentences fairly effectively.

Awesome, thanks @smantri7!


global is undefined while polyfilling the native promise

For some reason,  I have to compile the monaco-editor by myself.
Recently I update the monaco and found that the editor crash while loading the monaco.
It seems that we need polyfill the promise after we set the global.

Thanks


Update gitops multi-cluster pattern to use eks pod identity and cluster access a management (CAM)

Update gitops multi-cluster pattern to use eks pod identity and cluster access a management for hub ArgoCD instance to connect to spoke clusters

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

What is the outcome that you are trying to reach?

Describe the solution you would like

Describe alternatives you have considered

Additional context

I will work on this.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_39xwt5d5fu/clone/recipe/meta.yaml:22: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_39xwt5d5fu/clone/recipe/meta.yaml:61: missing_dev_url: The recipe is missing a dev_url
ERROR: /tmp/abs_39xwt5d5fu/clone/recipe/meta.yaml:61: http_url: http://github.com/frictionlessdata/frictionless-py is not https
WARNING: /tmp/abs_39xwt5d5fu/clone/recipe/meta.yaml:61: missing_description: The recipe is missing a description
ERROR: /tmp/abs_39xwt5d5fu/clone/recipe/meta.yaml:61: missing_documentation: The recipe is missing a doc_url or doc_source_url
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_40ee7cw0jw/clone/recipe/meta.yaml:22: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_40ee7cw0jw/clone/recipe/meta.yaml:61: http_url: http://github.com/frictionlessdata/frictionless-py is not https
ERROR: /tmp/abs_40ee7cw0jw/clone/recipe/meta.yaml:61: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_40ee7cw0jw/clone/recipe/meta.yaml:61: missing_dev_url: The recipe is missing a dev_url
WARNING: /tmp/abs_40ee7cw0jw/clone/recipe/meta.yaml:61: missing_description: The recipe is missing a description
Errors were found


Document what npm build does and pushState

The User Guide does not actually document what npm run build does. The docs also don't describe how exactly to "deploy" the resulting build directory.
This PR describes what npm run build does and what it means to deploy the build in the Users Guide.
Additionally, it describes the pushState gotcha which was previously only described in reference to specific servers (eg github pages).

Slightly tweaked and merged. Thank you so much!


Failing to upgrade to Kubernetes 1.14 or 1.15, please advice

Some useful command about namespace management is missing in 1.11, so I tried to upgrade to 1.14 and 1.15 in many ways but they all failed. I don't understand fully, but some complain about systemd, and cgroup plug-in.

I can try bumping to 1.14 soon. Thx for reporting.

Bumped it to 1.14. Let me know how it goes


Email notifications

Description
Email Notification API works in the command line but am unable to test as form submission doesn't work on main
Tests
PyTest, Unittest, and Cucumber/Selenium, does not work well at the moment
UI Changes
N/A
Issues
Testing is still an issue and the form submission doesn't work
Additional Notes
N/A

Had horrible heroku issues, needed to revert a lot to be able to salvage this branch and close it. the emailing works, just not on button click submission


Linux kernel modules can't be licensed under Apache 2

Apache 2 isn't a good kernel license (having to use "proprietary" as a module license string should have been a big hint.)
Please just make these GPLv2 so we can all use and contribute to them.

Can you elaborate on why this licensing scheme means not everyone can use this or contribute to it?

Most, if not all, kernel developers employed by companies are not allowed to contribute to non-GPLv2-compatible kernel code for obvious reasons (hint, you wouldn't want me to either).
I'll turn it around, why does anyone think this licensing scheme is a good idea, or, at the very least, even legal?  :)

To the extent that proof of concept code isn't a derivative of the Linux kernel it would be very useful to other projects if the license isn't GPLv2-only, e.g. BSD/GPLv2 dual license like the i915 graphics drivers.

@gregkh @emaste we're working on an answer here -- expect more after the holidays -- but I can definitely say our intent is to make sure everything (remains) available with a permissive license option.

@gregkh the kernel modules (along with the rest of the repository) are now available as GPLv2.

Wonderful, thank you so much for doing this!


fix #5256

What is the purpose of the change
fix #5256
Brief changelog
MessageOnlyChannelHandler add sendFeedback

Codecov Report

Merging #5724 into master will decrease coverage by 0.03%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #5724      +/-   ##
============================================
- Coverage     61.43%   61.39%   -0.04%
+ Complexity      497      496       -1
============================================
  Files           930      930
  Lines         38079    38034      -45
  Branches       5481     5464      -17
============================================
- Hits          23393    23352      -41
+ Misses        12149    12145       -4
  Partials       2537     2537

Impacted Files
Coverage Δ
Complexity Δ

.../dispatcher/message/MessageOnlyChannelHandler.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...bo/config/event/listener/LoggingEventListener.java
37.5% <0%> (-25%)
0% <0%> (ø)

...ava/org/apache/dubbo/config/DubboShutdownHook.java
65.95% <0%> (-14.9%)
0% <0%> (ø)

.../apache/dubbo/rpc/protocol/AsyncToSyncInvoker.java
79.16% <0%> (-8.34%)
0% <0%> (ø)

...mmon/threadpool/support/fixed/FixedThreadPool.java
80% <0%> (-7.5%)
0% <0%> (ø)

...moting/transport/netty4/NettyEventLoopFactory.java
30% <0%> (-6.37%)
0% <0%> (ø)

.../threadpool/support/limited/LimitedThreadPool.java
83.33% <0%> (-5.56%)
0% <0%> (ø)

...ng/exchange/support/header/HeartbeatTimerTask.java
73.68% <0%> (-5.27%)
0% <0%> (ø)

.../apache/dubbo/remoting/transport/AbstractPeer.java
58.69% <0%> (-4.35%)
0% <0%> (ø)

...on/threadpool/support/cached/CachedThreadPool.java
85.71% <0%> (-4.29%)
0% <0%> (ø)

... and 33 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 5a62b55...2b3b312. Read the comment docs.


hgroup is no longer a valid html5 specs element

Hi, as title, do we still want it to be in the stylesheet? I asked because my editor highlight it as an error.

gh-437

@cheahkhing: hi! please see this.

hi, guys, thanks for pointing me to the comments. forgive me for not searching first and asked.
But yet, after so many requests on removing, it is still being kept for supporting those sites that using obsolete tags? I still don't get it.

@cheahkhing See this comment : https://github.com/necolas/normalize.css/pull/180#issuecomment-16005485

Hi, thanks again for the references, since the owner doesn't want to exclude that, i'll just close this and manually remove it from my usage. Thanks.


Terraform apply taking ages

Description
First of all I would like to  say big thanks to contributors who have taken time to implement such a wonderful module. Much appreciated. I'm trying to use the example as it is and I'm noticing terraform apply is not deploying any resource, basically nothing happens for almost 20+ mins and then I stop by pressing ctrl-c. I did retry couple of times and still the same issue, infact I tried using different version of terraform (v0.12.31, v0.13.1, v0.14.0), still no luck. I literally deleted and re-initialised.
I have used other modules such as vpc and could successfully create VPC and EC2 instances. I'm trying to spin up EKS cluster in EU-west-2 region.
⚠ Note
Before you submit an issue, please perform the following first:

Remove the local .terraform directory (! ONLY if state is stored remotely, which hopefully you are following that best practice!): rm -rf .terraform/
Re-initialize the project root to pull down modules: terraform init
Re-attempt your terraform plan or apply and check if the issue still persists

Versions

Terraform:
0.12.31, 0.13.1, 0.14.0
Provider(s):
AWS
Module:
EKS & VPC

Reproduction

Steps to reproduce the behavior:
Are you using workspaces? No
Have you cleared the local cache (see Notice section above)?  Yes
List steps in order that led up to the issue you encountered terraform init followed by terraform apply
Code Snippet to Reproduce
 I'm using basic example from the [git hub](https://github.com/terraform-aws-modules/terraform-aws-eks/tree/master/examples/basic)
Expected behavior

I'm expecting terraform apply to spin up the resources in my AWS account and create EKS cluster.
Actual behavior

Successfully initialised terraform, however terraform apply script just doesn;t do anything, I can control -C and it quits. Its not like hanging, because I could ctrl-c. I waited for almost 20+ mins, but no response/logs appear in the console.
Terminal Output Screenshot(s)

base) Manjunaths-MBP:learn-terraform-modules manjunathg$ terraform init
Initializing modules...
Initializing the backend...
Initializing provider plugins...

Reusing previous version of hashicorp/template from the dependency lock file
Reusing previous version of hashicorp/local from the dependency lock file
Reusing previous version of hashicorp/null from the dependency lock file
Reusing previous version of hashicorp/random from the dependency lock file
Reusing previous version of hashicorp/aws from the dependency lock file
Reusing previous version of hashicorp/kubernetes from the dependency lock file
Installing hashicorp/local v2.1.0...
Installed hashicorp/local v2.1.0 (self-signed, key ID 34365D9472D7468F)
Installing hashicorp/null v3.1.0...
Installed hashicorp/null v3.1.0 (self-signed, key ID 34365D9472D7468F)
Installing hashicorp/random v3.1.0...
Installed hashicorp/random v3.1.0 (self-signed, key ID 34365D9472D7468F)
Installing hashicorp/aws v3.38.0...
Installed hashicorp/aws v3.38.0 (self-signed, key ID 34365D9472D7468F)
Installing hashicorp/kubernetes v1.13.3...
Installed hashicorp/kubernetes v1.13.3 (self-signed, key ID 34365D9472D7468F)
Installing hashicorp/template v2.2.0...
Installed hashicorp/template v2.2.0 (self-signed, key ID 34365D9472D7468F)

Partner and community providers are signed by their developers.
If you'd like to know more about provider signing, you can read about it here:
https://www.terraform.io/docs/plugins/signing.html
Terraform has been successfully initialized!
You may now begin working with Terraform. Try running "terraform plan" to see
any changes that are required for your infrastructure. All Terraform commands
should now work.
If you ever set or change modules or backend configuration for Terraform,
rerun this command to reinitialize your working directory. If you forget, other
commands will detect it and remind you to do so if necessary.
(base) Manjunaths-MBP:learn-terraform-modules manjunathg$ terraform apply
Additional context

I think I figured it out, why terraform apply was taking time. Basically I started commenting the code and working towards the minimal set of code and finally at one point terraform complained about credentials. I did pass the credentials as part of variables.tfvar, but still it was complaining. After some googling I found this link https://github.com/hashicorp/terraform-provider-aws/issues/13057, I followed the steps by exporting the variables. Once the variables were exported, I could see terraform spinning up and creating the resources.

Please see my earlier comment and happy to close the ticket


Consider alternative name for "modules"

We chose the name "modules" for two main reasons:

It's the same as the R term which makes transitioning a bit easier
Shiny modules are pretty similar to generic Python modules so it made sense to use the name

The name does seem to create confusion in some users and is difficult to explain. For example I ended up saying: "the Shiny module_ui function is in Shiny's UI module".
According to chatGPT the most common term for this is "Component" so maybe we should go with @component.ui and @component.server.

Web Framework
Reusable Elements

React
Components

Angular
Components

Vue.js
Components

Ember.js
Components

Backbone.js
Views or Templates

Meteor.js
Templates or Components

Express.js
Middleware or Router Middleware

Next.js
Components

Svelte
Components

Preact
Components

Django
Templates or Views

Dash
Components

Streamlit
Components or Widgets

+1 for component over module, definitely seems closer to the right concept. Maybe a rename plus better positioning in the docs would be enough: for me, "modules" felt so far advanced into package development that I didn't even open the docs page to see that it actually was the solution for encapsulating a feature for reuse.

Good suggestion. On a related note, I had hoped to be able to use classes and methods for reuse in py-shiny. This would seem particularly useful In a bigger apps where you may be able to inherit a lot from a class and only have to overwrite a few methods
I'm not sure if @wch's suggested at the end of issue below is what would be needed to make that work now or if this shows a best-case-scenario for the future.

Component doesn't seem right either since the way most projects use that term is closer to what Shiny would call a binding/widget than it is to the concept of a Shiny module. So, if we are going to change the name, I'd probably prefer something like "Template" or "Namespace".

@cpsievert I'm not sure that's right, for example if I'm creating a React component I encapsulate some React code into something which can be used by other parts of my application. This seems exactly the same as what we do in Shiny for Python. Similarly, Solara and Pynecone both use component to refer to what we would call modules.
Template to me sounds a bit more like a code snippet or HTML template which isn't quite right. Namespace is probably the most accurate, but it's not that evocative and we'd have the similar problem of being the only project that uses the term.

I agree that "template" and "namespace" don't feel right, and also shared @vnijs's experience that what I was really looking for was "class" because this feels like a natural thing for OOP.


confusing error message when directory is not a bundle directory

$ imgpkg push -b k14stest/dk-test-repo-bundle -f examples/packaging-demo/repo
Error: Expected one '.imgpkg' dir, got 0:

seems like we can improve error message to indicate that this directory is not a bundle and needs to have a .imgpkg if it wants to be one. also what's up with trailing :?

Agreed. This error message should be improved, I was able recreate this issue by running from the imgpkg repo root dir:
imgpkg push -b k14stest/gc-test-repo-bundle -f examples/basic-step-1
Error: Expected one '.imgpkg' dir, got 0:

When adding an .imgpkg dir with no imagesLock file to the basic-step-1 dir, and then running the same command, I get:
mkdir examples/basic-step-1/.imgpkg
imgpkg push -b k14stest/gc-test-repo-bundle -f examples/basic-step-1
Error: Reading path /Users/garrettcheadle/dev/imgpkg/examples/basic-step-1/.imgpkg/images.yml:
open /Users/garrettcheadle/dev/imgpkg/examples/basic-step-1/.imgpkg/images.yml: no such file or directory

I think this error is better, but I feel the message should also include explicit information about bundles requiring an
.imgpkg/images.yml file.

👋 I'm going to take a stab at this.

let us know if you need any help 😄

Just a qq: I noticed at least some of those test could be in a table test structure. Is it ok to do that or you are trying to avoid it for these tests?

*some of the tests in the push_test.go, which is where I'm adding cases to test the validation logic I'm adding.

We have been trying to move our tests into a t.Run style, like https://github.com/vmware-tanzu/carvel-imgpkg/blob/develop/pkg/imgpkg/bundle/bundle_images_lock_test.go, and replace manual t.Fatalf with testify assertion framework.
I think we are ok with a table test format in general, just let's make sure we gain readability in these tests 😃


Provide a way to acquire TLS certs even if Sandstorm isn't the main web server

I have an Apache running on port 80 and port 443 to serve other services on my domain. It'd be awesome if I could somehow configure Sandstorm to acquire a wildcard cert for my Sandcats domain, even though it isn't the main webserver, so my Apache could then use it to serve Sandstorm.

One thing you can do is configure this in the sandstorm.conf:
HTTPS_PORT=4430
and probably get the behavior you want, but note that the certs would
expire weekly, so you'd still have to automate Apache restarting.
The certs are also in a JSON format, not in raw CRT/etc. format.
So this is just to say that I don't have a full solution to your problem,
but now you know everything that is possible for now.

@paulproteus thanks! I'll leave this open for a proper solution.


Content does not load when I use ng-repeat and the class "item" on demo project

Hello all !!
I'm using the demo project for testing features and when I loaded the contents of friends page using ng-repeat nothing happened. No item were loaded.
I noticed that class "item" on "a" element is causing this, but I don't know why.
Any idea?
Thank's and sorry for my bad english.

I have seen this before. If you put the js for calling the material transitions after you have retrieved your data for the ng-repeat it should work fine

Thank you for reply, but it not working yet.
Look my controller:
.controller('FriendsCtrl', function($scope, $stateParams, $timeout, ionicMaterialInk, ionicMaterialMotion) {
    // Set Header
    $scope.$parent.showHeader();
    $scope.$parent.clearFabs();
    $scope.$parent.setHeaderFab('left');

    $scope.friends = [
        {
            name : "Jon Snow",
            description: "Da illest illegitimate"
        },
        {
            name: "Daenerys Targaryen",
            description: "Dragon mommy"
        }
    ];

    // Delay expansion
    $timeout(function() {
        $scope.isExpanded = true;
        $scope.$parent.setExpanded(true);
    }, 300);

    // Set Motion
    ionicMaterialMotion.fadeSlideInRight();

    // Set Ink
    ionicMaterialInk.displayEffect();
})

and my template:
<ion-view view-title="Friends">
    <ion-content ng-class="{expanded:isExpanded}" class="animate-fade-slide-in">

{{friend.name}}

{{friend.description}}

    </ion-content>
</ion-view>

I realized that in lists using the ionic directives, I can not run the ripple effects.
However, that your case, where you are using , simply call the function "ionicMaterialInk.displayEffect ();" after "ng-repeat" being fully charged.
You can find the solution here: #46

Ok. Thank you for your reply.
I'll try the proposed solutions.


Expand view opens but does not work the first time you install Openbit

Describe the bug
Can't open expand view the first time install Openbit
To reproduce the bug

Version test : http://<IP_ADDRESS>:8900/s/tkH4isbmSTJWe4f
Validation passed on Chrome, Brave


Support interceptors

Hi Roberto,
Another request - I've been trying to use the standard Quasar axios boot file, but had problems due to the very obscure way I'm using it. Your extension has resolved the issue I had, but it would be great to use interceptors
The use case is to pop a notification to the user to say there is an API error. I have one global interceptor for this, rather than having to examine every response and then have duplicated code everywhere.
Regards,
Andy

Hi,
Thank you for using the module.
I will take a look on interceptors and more use cases so I can support them correctly.


[Question] How to deprecate cachedRead for Ledger

Vinnie has a comment to deprecate cachedRead.  There are only a few call sites.  Is it preferred to just always call ledger.read(keylet::unchecked(key))?

We should simply inline the appropriate codepath (depending on whether we know the type or not) in the right place and remove this function.


Adds ability to delete unused script tags

Adds function to delete script tags
Automatically delete unused script tags from shop

Thanks for this! Looks fine. Just to confirm though, memory is rusty, the API call only has access to the apps current script tags correct? I wouldn't want it to delete some other app's scripttags.

Thanks, merging.

@niveshsaharan This has been an ask for webhooks as well, do you think a similar process for webhooks would be beneficial?

@osiset Maybe but I never needed to do this for webhook as we don't normally need our apps to remove existing webhooks.

I may duplicate your process and implement this for webhooks as well.
From: Nivesh<EMAIL_ADDRESS>Sent: Friday, July 31, 2020 03:11
To<EMAIL_ADDRESS>Cc: Tyler<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [osiset/laravel-shopify] Adds ability to delete unused script tags (#543)
@osisethttps://github.com/osiset Maybe but I never needed to do this for webhook as we don't normally need our apps to remove existing webhooks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/osiset/laravel-shopify/pull/543#issuecomment-666935942, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AASO4ORHNJZQ4VEMWIADATDR6JKPHANCNFSM4PNUXFPQ.

Sounds good. Thank you for the awesome work you're doing.


Starting the Lookout Style Guides

Removing placeholders and slightly modifying the README to be more
org specific.

I agree with all other changes however.

made all changes, merging.


Updated og tags

I've been busy working on another project. I will look at this PR over the weekend. Thanks for the couple of updates.


Use springfox instead of the old swagger-springmvc for API documentation

Since the documentation doesn't work, tagged as bug

I add springfox dependency but i have this :
[ERROR] Failed to execute goal org.springframework.boot:spring-boot-maven-plugin:1.3.1.RELEASE:run (default-cli) on project standalone: An exception occurred while running. null: InvocationTargetException: Error creating bean with name 'workerResource': Injection of autowired dependencies failed; nested exception is org.springframework.beans.factory.BeanCreationException: Could not autowire field: private com.fasterxml.jackson.databind.ObjectMapper io.mandrel.endpoints.internal.WorkerResource.objectMapper; nested exception is org.springframework.beans.factory.NoUniqueBeanDefinitionException: No qualifying bean of type [com.fasterxml.jackson.databind.ObjectMapper] is defined: expected single matching bean but found 2: mapper,_halObjectMapper -> [Help 1]


Error: application token is invalid

Ive tried 2 new tokens, however still not working. Any ideas?
Error: application token is invalid
at Pushover.errors (/Users//Documents/delivery-slot-bot/node_modules/pushover-notifications/lib/pushover.js:146:13)
at IncomingMessage. (/Users//Documents/delivery-slot-bot/node_modules/pushover-notifications/lib/pushover.js:250:12)

User error! Ignore


Set scene scale

Viser currently assumes the scene is roughly 10x10x10 which makes navigation difficult when your scenes are a very different scale. For example, nerfstudio is +-1. Users currently scale everything on the server side, but we should add the functionality to set the scene scale in viser.
One api possibility is,
viser_server = ViserServer()
viser_server.set_scale(10)

So I think there are two high-level options for what happens we call .set_scale():

Track the scale entirely on the viser server side in Python. After .set_scale() is called, any positions we read or write can be scaled in Python by those coordinates. Sounds like a lot of boilerplate to me.
Alternatively, send the scale value over to the client, and have the client just scale a threejs group that wraps everything. This could be roughly prototyped by adding a scale={[10.0, 10.0, 10.0]} here. The viewer camera isn't currently in this group and won't be scaled, but it seems possible to move it in.

Does this make sense? Interested in thoughts from @cnnmon and/or @tancik


Tag filter is applied only to first tag

The current behaviour when using tag filter is to compare it against the first tag of each image.
This matters for use cases on images with multiple tags.
In my view, it is more intuitive to have an option to say it should match any of the tags on the image or all of the tags on the image.
If you care about backwards compatibility (i.e. this was a feature and not a bug), there could also be an option to only match the first tag...
In my particular use case, the image should be deleted only if the filter to matches all the tags on the image. I actually think this is the most sensible default for the library because it's the most strict (matches the least) which is a good idea when the effect of the operation is permanent deletion.

The reason I went with checking the first tag was:

Across (public) Container Registry and Docker Registry at the time, only 14% of tagged images had more than one tag. This could have changed since, but it didn't seem like there were a compelling number of images being multi-tagged.
The logic to an end-user becomes incredibly confusing: Do I want to delete iff all tags match? Do I want to delete iff any tags match? There's another thread about inverse matches (only when tags don't match), in which case how would users expect the logic to be inverted?

Across (public) Container Registry and Docker Registry at the time, only 14% of tagged images had more than one tag. This could have changed since, but it didn't seem like there were a compelling number of images being multi-tagged.

I fail to see how this motivates not implementing this, when there is a demand?

The logic to an end-user becomes incredibly confusing: Do I want to delete iff all tags match? Do I want to delete iff any tags match? There's another thread about inverse matches (only when tags don't match), in which case how would users expect the logic to be inverted?

IMHO it is more confusing that the filter only applies to the first tag. Would the cases you're describing rather not require a less restrictive filter, or a "match any" feature which considers all tags?

I fail to see how this motivates not implementing this, when there is a demand?

There have been exactly 3 requests for this over the lifetime of the project, and GCR Cleaner is used millions of times each day. My point is that the complexity required to implement this correctly and in a backwards-compatible way, and I don't think there's reasonable demand given existing use. That doesn't mean we'll never do it, it just means it hasn't been a priority.

IMHO it is more confusing that the filter only applies to the first tag. Would the cases you're describing rather not require a less restrictive filter, or a "match any" feature which considers all tags?

The current behavior is predictable for all possible inputs, which was the goal. I think there's room for a more powerful filtering system (one that let's users express complex expressions), but that's going to require a design that maintains BC and supports all possible use cases; I don't want to engineer a stop-gap solution.


devicemapper: autoloopfile distinction

This set introduces a change to default behavior of the devicemapper driver, and introduces a new flag that delivers the previous [transparent] expected behavior.
This needs a bit of discussion, even though I feel like is a nicely supportable use case.
Many complaints around the performance of the devicemapper driver (or just "performance" in general because folks are not even aware of storage driver differences), are due to the io traversal of the loopback file.
With this change, devicemapper driver will expect (and fail without) a block device being provided, OR with daemon flag --storage-opt dm.autoloopfile=true it will automatically create files and attach them on loopback.

Can you please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "vbatts-dm_auto_loop_flag" git@github.com:vbatts/docker.git somewhere
$ cd somewhere
$ git rebase -i HEAD~3
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

This needs additional testing that it doesn't break the usage of options like dm.thinpooldev.
/ ping @rhvogyal

@vbatts
I am still wondering that what's wrong with default behavior of setting loop files. Yes it is not a very good configuration from performance point of view but it gets user going quickly. That's the part I like about docker most. Without too much of configuration in, one can easily play with it. And once somebody needs more performance out of it, you need to look into various configuration options to tune it better.
IOW, setting loop files automatically by default sounds just a good default to me. May be we can just document it better that it will not be very good for performance and setup a lvm thin pool for better performance.

IOW, from ease of use point, it is very convenient that a first time user of docker can simply run docker without having to worry about various storage options or without worrying about having to learn about dm pools and loopback files. They can just type "docker -d" and get going. If we introduce this then they need to know a lot more before they can start.

What is graphdriver
Device mapper is default graph driver.
What are different storage options
How storage can be setup in simplest way to get going etc.

I personally feel that current default of automatically setting up loop file makes lot of sense. And those who want more performance out of storage, they need to pass in right --storage-opt options.

if this means that a user downloading the get.docker.com or master.dockerproject.com binary to a centos box can't have it just work, then this PR needs user facing documentation explaining why, and what they need to do to get it going.

I agree on all points. Though the inspiration for this is due to the lack of understanding from users that @rhvgoyal has laid out. Perhaps this PR would be an end goal functionality, but we could make incremental steps to get there.
Next steps could be, leave this flag here, but default to false, and add a NOTICE to the log performance concerns when running on devicemapper on loopback?

@vbatts
Did you mean default to "true" for this flag? If you default to "false" then user will have to specify it explicitly and then docker does not work out of the box.
An Info log which says that a user might be able to get better performance by lvm thin pool should be ok. But IMHO, introducing one more flag sounds unnecessary.

@vbatts
Solution to the performance issue is writing a blog to educate users better about better storage setup.
Also project atomic is shipping a service "docker-storage-setup" to automatically setup lvm thin pool or dm data and metadata devices for docker.
https://github.com/projectatomic/docker-storage-setup
I think documenting and writing a blog about better storage setup will help users.

+1 to adding info to the docs - we really need it. Blog posts are good for highlighting those docs.

@rhvgoyal understood. I realize this PR is a bit drastic, it also seems that users aren't involved enough in what they're running. I'm less inclined to pursue this PR, which breaks the 'automagic' experience on devicemapper. I'll instead open a PR for perf warning when running on loopback file


Improve test module

Add more fine-grain and coarse-grain tests

This is done now (thanks to Dmitry) so I close this ticket.


(GH-1207) Run task function should respect _noop option

When running a task from a plan, the run_task function should respect
the _noop option. Now when the run_task function is called either
with an executor in noop mode or with the _noop option it will run in
noop.
Closes #1207

I linked to a kind of ambiguous commend https://github.com/puppetlabs/bolt/issues/1207#issuecomment-539149226 trying to convey the resolution for https://github.com/puppetlabs/bolt/pull/1237#pullrequestreview-296465282 . We thought that cleaning up that dead code path (or actually implementing it fully) would be resolved with this ticket.

I removed adding [:noop] to the plan run parameters if you pass in --noop, since it sounds like supporting that for plans is an entirely separate bit of work.

CLA signed by all contributors.


[chloggen] Disallow whitespace-only values

Resolved #164

Closing as duplicate of #165

@djaglowski If you want you can reopen it as https://github.com/open-telemetry/opentelemetry-go-build-tools/pull/165 got stale and I closed it.


The symbol "inWebWorker" has already been declared (TurboRepo)

Which project does this relate to?
Router
Describe the bug

I am currently running vite  react-ts in a Turbo Repo project.
Once I setup tan stack router with file-based routing following the docs here I get the above error.
$ node -v
v20.10.0
$ npm -v
10.2.3
"vite": "^5.4.10"
"typescript": "~5.6.2",
"react": "^18.3.1",
"react-dom": "^18.3.1",
Your Example Website or App
n/a
Steps to Reproduce the Bug or Issue

Create a turbo repo project
cd into the apps directory
run npm create vite@latest my-app -- --template react-ts to create  a vite project in ./apps
cd my-app and setup tanstack router following the guide here

Expected behavior

it should display the index.lazy.ts page

Screenshots or Videos

Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

can you please provide a complete reproducer GitHub repo?

https://github.com/Emyboy/tensillabs
there you go, I made i public.

@Emyboy you need to remove this instance of the react-refresh plugin since you've already got it set up after the TanStackRouterVite plugin.
This is the line that needs to be removed - https://github.com/Emyboy/tensillabs/blob/ab9212931bf1ecc841b10faa1c914a1f26e00e9e/apps/workspace-app/vite.config.ts#L9

@SeanCassiere Link dead.  Next time please provide text solution.

@paulm17 the solution was specific to the OP's repo.

To summarize, the issue was the removal of a couple lines of code in the OP's vite.config.ts.
// vite.config.ts
import { defineConfig } from 'vite'
import viteReact from '@vitejs/plugin-react';
- import reactRefresh from '@vitejs/plugin-react';
import { TanStackRouterVite } from '@tanstack/router-plugin/vite';

export default defineConfig({
	plugins: [
		viteReact(),
		TanStackRouterVite(),
- 		reactRefresh()
	]
})

this error seems to come up again when the vite.config.ts says
import { defineConfig } from "vite";
import react from "@vitejs/plugin-react";
import viteReact from "@vitejs/plugin-react";
import { TanStackRouterVite } from "@tanstack/router-plugin/vite";

// @ts-expect-error process is a nodejs global
const host = process.env.TAURI_DEV_HOST;

// https://vitejs.dev/config/
export default defineConfig(async () => ({
  plugins: [react(), TanStackRouterVite(), viteReact()],
  esbuild: {
    logOverride: { "this-is-undefined-in-esm": "silent" },
  },
  optimizeDeps: {
    include: ["@tanstack/react-router"],
  },
  define: {
    __dirname: '""',
    __filename: '""',
  },
  build: {
    commonjsOptions: {
      transformMixedEsModules: true,
    },
  },

  // Vite options tailored for Tauri development and only applied in `tauri dev` or `tauri build`
  //
  // 1. prevent vite from obscuring rust errors
  clearScreen: false,
  // 2. tauri expects a fixed port, fail if that port is not available
  server: {
    port: 1420,
    strictPort: true,
    host: host || false,
    hmr: host
      ? {
          protocol: "ws",
          host,
          port: 1421,
        }
      : undefined,
    watch: {
      // 3. tell vite to ignore watching `src-tauri`
      ignored: ["**/src-tauri/**"],
    },
  },
}));

It is unclear which should be removed in this case...
removing the standard react() plugin seems to have fixed it... not sure if this is the best way.

@Dax911 both react() and viteReact() refer to the same thing. It's what causing the error since the react-refresh plugin is attempting to transform code that has already been transformed earlier.

Ahhh. Learn something new everyday. Thanks for clarifying.


WIP: Custom made DL

Looks good!

See my api PR

On 21 Nov 2021, at 20:09, pancake @.***> wrote:

Looks good!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.


Bug - Black screen showing instead of map (Repo provided)

I'm submitting a ...  (check one with "x")
[ ] question
[ X ] any problem or bug report
[ ] feature request
The plugin version:  (check one with "x")
[ X ] 2.0-beta3 (github)
[ ] 2.0 (npm)
If you choose 'problem or bug report', please select OS:  (check one with "x")
[ ] Android
[ X ] iOS
Current behavior:
I have 2 maps in my ionic app, the first one shows when you open the app (which works fine), you click on the button, it will open a modal and it will drive you to Contact Page from that page when you click on Go to About Page you'll see a black screen where it is supposed to see the map !
Expected behavior:
The map should show instead of the black screen.
Steps to reproduce:
Please refer to the following<EMAIL_ADDRESS>

This is not a bug.  I'm sure you don't understand How does this plugin work? section.
https://github.com/mapsplugin/cordova-plugin-googlemaps#how-does-this-plugin-work
And did you search the past issues?
https://github.com/mapsplugin/cordova-plugin-googlemaps/issues/1678

The first of all, the map view is NOT html element. It means the map view is not DOM element.
So the map view is located under the browser, NOT in the browser view.
That's why the map view is under the browser, and the browser background color MUST BE TRANSPARENT

The <ion-modal> is a popup window, which means the <ion-modal> is located upon the other elements.
This is the structure what happens in your app.

The reason of the black map is
.
This element is the background for the modal dialog.
However this element is not parent element of the map div, the plugin does not change the background of this element.
That's why the page is black.
For the evidence, if I change the background as red, the app displays red.

Even if you set background-color:transparent for the
 element,
you can not use the map.
Because the <ion-modal> is displayed over the <page-home>.
The <page-home> element keeps the background as white, you only see the white page (like regular page).
If you change the background of the <page-home>, you will see the two maps you generated.
But, you don't remove the map1, the map2 is displayed upon the map1.

If you handle these background-color, display:none for the other pages, and remove the previous map, you can use a map in <ion-modal>.
Otherwise, you can not use the map.

Since you are CTO, and consultant for several startups, please debug, study as much as possible before reporting as bug.

Basically, I guess that's Ionic Team work ! They need to add a fix for it.
Anyways, thanks for the clarifications.


TF Lite enable Flex delegate C++ API (with CMake of Bazel)

System information

Linux Ubuntu 18.04):
TensorFlow installed from source:
TensorFlow version commit 47e07ba0d68c55dba62bff5b8486291086840097:

Hi, I want to run simple conv2d layer training using C++ API of TF Lite. I’ve used examples/minimal project as reference. Calling trainer->invoke() leads to next error:
ERROR: TensorFlow Lite Error: Select TensorFlow op(s), included in the given model is(are) not supported by this interpreter. Make sure you apply/link Flex delegate before inference. For the Android, it can be resolved by adding “org.tensorflow:tensorflow-lite-select-tf-ops” dependency…
ERROR: Node number 48 (FlexConv2DBackpropFilter) failed to prepare.
I didn’t find any good tutorial on Flex delegate. Please, tell me, how to build TF Lite with Flex using CMake and how to make this interpreter use Flex’s implementation of this node. Thanks

Hi @alex19999 !
Sorry for the late response.
Currently we can build Select ops flex delegates through Bazel. Not sure on Cmake .  A similar feature request has been raised here.#55536.
I gave a try building select ops run time in cmake by changing the build file though. Could you give it a try with your model. Attached gist for reference  (Please change the build file before running the code)

   deps = [
        "//tensorflow/lite:framework",
        "//tensorflow/lite/kernels:builtin_ops",
        '//tensorflow/lite/kernels:select_ops"
    ],

Hi @alex19999 ! Sorry for the late response. Currently we can build Select ops flex delegates through Bazel. Not sure on building flex delegate through Cmake . A similar feature request has been raised here.#55536.
I gave a try building select ops run time in cmake by changing the build file though. Could you give it a try with your model. Attached gist for reference (Please change the build file before running the code)

   deps = [
        "//tensorflow/lite:framework",
        "//tensorflow/lite/kernels:builtin_ops",
        '//tensorflow/lite/kernels:select_ops"
    ],

@mohantym adding //tensorflow/lite/kernels:select_ops entry in deps wouldn't end up in compilation error because there is no such target available part of tensorflow/lite/kernels package?
@alex19999 I've tried just building tensorflowlite_flex library and linking that library while building application level binary using CMake. That's been working fine for Mac OS.
Bazel command to build tensorflowlite_flex library
bazel build -c opt --config=monolithic tensorflow/lite/delegates/flex:tensorflowlite_flex

Sample CMake snippet
add_executable(minimal minimal.cpp)
target_link_libraries(minimal tensorflowlite tensorflowlite_flex)

@alex19999 !
That is great to hear.  Please let us know after you results with Ubuntu too.
Thank you!

@dhruvkakadiya
Could you please share details for this?

@alex19999 I've tried just building tensorflowlite_flex library and linking that library while building application level binary using CMake. That's been working fine for Mac OS.

I tried to do the same. Maybe I've missed something.
I built tensorflowlite-flex.so using:
bazel build -c opt --config=monolithic tensorflow/lite/delegates/flex:tensorflowlite_flex

Here's my CMake for linking:
add_library(tensorflowlite_flex SHARED IMPORTED)
set_property(TARGET tensorflowlite_flex PROPERTY
        IMPORTED_LOCATION "${CMAKE_CURRENT_SOURCE_DIR}/tensorflow-prebuilt/libtensorflowlite_flex.so")

target_link_libraries(minimal tensorflow-lite tensorflowlite_flex)

It fails with an error:
ERROR: Select TensorFlow op(s), included in the given model, is(are) not supported by this interpreter. Make sure you apply/link the Flex delegate before inference. For the Android, it can be resolved by adding "org.tensorflow:tensorflow-lite-select-tf-ops" dependency. See instructions: https://www.tensorflow.org/lite/guide/ops_select
ERROR: Node number 3 (FlexTensorArrayV3) failed to prepare.

Error at /home/kyrylomykhailov/Workspace/tensorflow/tensorflow/lite/examples/minimal/minimal.cc:61

Thank you!

Ok @alex19999 !
Thanks for sharing your result in Ubuntu. It is still a pending feature request then.
Thank you!

Hello @MykhailovKyrylo, I've tried same process with Linux OS but yes as you observed it's not working so seems like Flex support is missing for Linux/amd64 or ARM64 like targets(with CMake) and only supported for Mac OS as @mohantym mentioned.
@mohantym can you please confirm missing feature thing for sure because then documentation in here seems misleading or are we missing some information from other documentation references?
Per table in here, can find information for ARM based computers that "Select TF Ops" is not supported with CMake. So that's related in this context and should we expect in future having that feature available?

@dhruvkakadiya !
I have already forwarded this feature request to internal team . Will keep you posted.
Thank you!

@alex19999 !
Moving this to closed status as it is duplicate to #55536 .  Please comment in the same issue.
Thank you!


Why we are setting ignoreLineBreaks to true by default

Here, https://github.com/onelogin/java-saml/blob/master/core/src/main/java/com/onelogin/saml2/util/Util.java#L116
we are setting the system property "org.apache.xml.security.ignoreLineBreaks" to true by default which will instruct xmlutils to ignore line breaks.
Is there a way that we can override this value dynamically !
How one should say to NOT ignore line breaks in the signature verification ?

@complanboy2 are you still interested in providing such PR?


Using sampler2D as a function parameter and trying to debug at the same time causes a crash.

For some reason, if I pass sampler2D into a function and pixel inspect and fetch, it crashes.
Simply adding this function in without even calling it will cause the issue.
float test(sampler2D testMap) {
    return 0.5;
}

Thanks for reporting this! I'll try to include a fix for this issue in the next release (should be out in 1-2 weeks).

Thanks once again for reporting this. Should be completely fixed!