Errors in Grafana Dashboards after manual upgrade

Hi,
I updated a while ago to v1.11.0, had some issues with running the database migration due to having missed to make the database user superuser, fixed it and the migration reported successful.
Teslamate seems to be running normally but I have issues in Grafana.
Drives dashboard just has a red left top corner and the error message: pq: column start_geofence.address_id does not exist

Locations has this error: pq: column g.address_id does not exist

Carges also reports: pq: column g.address_id does not exist
Charging stats has this error: pq: column geofence.address_id does not exist

Locations shows no top destinations with this error: pq: column g.address_id does not exist
All other dashboards seem to work normally. Does anyone have any idea of what I have done wrong?
I run on FreeBSD 11.2 in a jail on FreeNas.
Grafana is version 6.3.5, updated at the same time as teslamate.
I run postgresql96-server-9.6.15, might have updated that also at the same time but I don't know.
Thankful for any help I can get! I don't want to lose all the data I have collected this far by starting over...
Best Regards
Sigfrid

Your dashboards are outdated. Re-import them as described here: https://github.com/adriankumpf/teslamate/wiki/Manual-Installation#importing-dashboards

Ah, thought I had done that, tried again and got this output in the shell:
Output:
{"id":14,"slug":"charge-details","status":"success","uid":"BHhxFeZRz","url":"/d/BHhxFeZRz/charge-details","version":8}
✔ Dashboard charge-details exported successfully.
Output:
{"id":13,"slug":"charges","status":"success","uid":"TSmNYvRRk","url":"/d/TSmNYvRRk/charges","version":9}
✔ Dashboard charges exported successfully.
Output:
{"id":12,"slug":"charging-stats","status":"success","uid":"-pkIkhmRz","url":"/d/-pkIkhmRz/charging-stats","version":8}
✔ Dashboard charging-stats exported successfully.
Output:
{"id":11,"slug":"degradation","status":"success","uid":"riqUfXgRz","url":"/d/riqUfXgRz/degradation","version":8}
✔ Dashboard degradation exported successfully.
Output:
{"id":10,"slug":"drive-details","status":"success","uid":"zm7wN6Zgz","url":"/d/zm7wN6Zgz/drive-details","version":9}
✔ Dashboard drive-details exported successfully.
Output:
{"id":9,"slug":"drives","status":"success","uid":"Y8upc6ZRk","url":"/d/Y8upc6ZRk/drives","version":9}
✔ Dashboard drives exported successfully.
Output:
{"id":8,"slug":"efficiency","status":"success","uid":"fu4SiQgWz","url":"/d/fu4SiQgWz/efficiency","version":9}
✔ Dashboard efficiency exported successfully.
Output:
{"id":7,"slug":"locations","status":"success","uid":"ZzhF-aRWz","url":"/d/ZzhF-aRWz/locations","version":8}
✔ Dashboard locations exported successfully.
Output:
{"id":6,"slug":"mileage","status":"success","uid":"NjtMTFggz","url":"/d/NjtMTFggz/mileage","version":8}
✔ Dashboard mileage exported successfully.
Output:
{"id":5,"slug":"position","status":"success","uid":"Dz6UUdzWk","url":"/d/Dz6UUdzWk/position","version":8}
✔ Dashboard position exported successfully.
Output:
{"id":4,"slug":"states","status":"success","uid":"xo4BNRkZz","url":"/d/xo4BNRkZz/states","version":8}
✔ Dashboard states exported successfully.
Output:
{"id":3,"slug":"updates","status":"success","uid":"IiC07mgWz","url":"/d/IiC07mgWz/updates","version":8}
✔ Dashboard updates exported successfully.
Output:
{"id":2,"slug":"vampire-drain","status":"success","uid":"zhHx2Fggk","url":"/d/zhHx2Fggk/vampire-drain","version":8}
✔ Dashboard vampire-drain exported successfully.
Output:
{"id":1,"slug":"visited","status":"success","uid":"RG_DxSmgk","url":"/d/RG_DxSmgk/visited","version":8}
✔ Dashboard visited exported successfully.
✔ 14 dashboards exported successfully.
Still not working...
Tried to manually import them and it worked. Something in the automated import did not work...
Found it! The automatic import is still looking in the wrong place, found my outdated files that I had copied last time and therefore updated the dashboards with outdated versions. Removed my copy of dashboards folder and automatic import dont work anymore...
Best regards
Sigfrid

👍

And now I realized that the setup of wizzy needed to happen in the grafana folder. Thank you for your patience with a noob like me :)


Custom Hook plugin

https://github.com/orndorffgrant/bnhook
Currently, you can overwrite existing code. This allows insertion of additional code. Uses extra space present in ELF format. Written to be fairly easy to extend to other architectures and executable formats.
Learned a lot while writing it; hopefully you think its useful!

Nice, thanks! We've actually got an open bug for "smart patching" to do that in the main product, but this looks to hit many of the same features.
Also, note that the new repo for plugins is over on community-plugins. I really need to update the main readme here.
I'll push this as a submodule over there shortly. Thanks!


Unify way of printing exceptions from within fibers

There was slight inconsistency how these exceptions were printed to STDERR (puts vs print), now they should be even. I've also added STDERR.flush missing from AtExitHandlers.run.
For the reference:
https://github.com/crystal-lang/crystal/blob/5e059ab5c24ed60426d58dda019b5904bcd33fca/src/kernel.cr#L404-L405
vs
https://github.com/crystal-lang/crystal/blob/5e059ab5c24ed60426d58dda019b5904bcd33fca/src/fiber.cr#L122-L128

Weird, CI flunked (but just once) with the error below:
./bin/crystal build  -o .build/std_spec spec/std_spec.cr
.build/std_spec

...............................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................*.................................................................................................................................................................................................................................................................................................................................................................................................................................................................................

Nil assertion failed (Exception)
  from src/class.cr:148:0 in 'not_nil!'
  from spec/std/signal_spec.cr:0:23 in '->'
  from src/signal.cr:255:3 in '->'
  from src/signal.cr:255:3 in 'process'
  from src/signal.cr:174:9 in '->'
  from src/fiber.cr:255:3 in 'run'
  from src/fiber.cr:29:34 in '->'
  from ???

FATAL: uncaught exception while processing handler for CHLD, exiting

Makefile:89: recipe for target 'std_spec' failed
make: *** [std_spec] Error 1

This CI error happens infrequently. Don't know why. It's not related to this PR at least.

@straight-shoota Out of these two I prefer print, since puts will add a newline before the Exception message which ain't look good (to my eyes at least).

This CI error happens infrequently. Don't know why. It's not related to this PR at least.

I believe it's related to one of these:
https://github.com/crystal-lang/crystal/blob/5e059ab5c24ed60426d58dda019b5904bcd33fca/spec/std/signal_spec.cr#L5-L7

Without a line break, this can become a pretty long line (consider a long fiber name and error message) and more difficult to read.
I'd prefer to add a line break after the "Unhandled exception" part.

Could this be reviewed and possibly included in the upcoming 0.26.1 release? It's a small change so it shoudn't take much time to get through it. @RX14 @bcardiff @ysbaddaden @asterite ?

Anyone care for 2nd review? 🏓 @asterite @sdogruyol


refactor/remove_dashboard

remove dashboard UI from ovos-shell integration
it will no longer show in the dropdown menu, it can be restored once gui extensions are made into plugins for some dedicated GUI extension that wants it
the dashboard plugin itself will still work, only the hardcoded dropdown UI integration has been removed

Closed in favor of GUI plugins PR


[Bug]: Can't install RStudio

Prerequisites

[X] I have written a descriptive issue title.
[X] I have searched all issues/PRs to ensure it has not already been reported or fixed.
[X] I have verified that I am using the latest version of Scoop and corresponding bucket.

Package Name
rstudio
Expected/Current Behaviour
scoop install rstudio fails as follows
PS> scoop install rstudio
Installing 'rstudio' (2022.12.0-353) [64bit] from extras bucket
Loading RStudio-2022.12.0-353.exe from cache
Checking hash of RStudio-2022.12.0-353.exe ... ok.
Extracting dl.7z ... done.
Linking ~\scoop\apps\rstudio\current => ~\scoop\apps\rstudio\2022.12.0-353
Creating shim for 'rstudio'.
Can't shim 'bin\rstudio.exe': File doesn't exist.

Steps to Reproduce
`scoop install rstudio`

Possible Solution
Since the relative path of rstudio.exe has changed, bin and shortcuts in rstudio.json should be fixed.
Scoop and Buckets Version
Current Scoop version:
v0.3.1 - Released at 2022-11-15

'extras' bucket:
7349ea767 (HEAD -> fix-rstudio-path, horigin/fix-rstudio-path) rstudio: Fix bin path

'java' bucket:
83922d06 (HEAD -> master, origin/master, origin/HEAD) graalvm-nightly: Update to version 23.0.0-dev-20221216_2127

'main' bucket:
7c40bb7d5 (HEAD -> master, origin/master, origin/HEAD) yq: Update to version 4.30.6

'scoop-viewer-bucket' bucket:
7d6f4e3 (HEAD -> master, origin/master, origin/HEAD) [scoop-viewer] Version 0.10.0

Scoop Config
alias last_update         scoop_repo                              scoop_branch
----- -----------         ----------                              ------------
      2022/12/17 15:33:12 https://github.com/ScoopInstaller/Scoop master

PowerShell Version
Name                           Value
----                           -----
PSVersion                      7.3.0
PSEdition                      Core
GitCommitId                    7.3.0
OS                             Microsoft Windows 10.0.22621
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Additional Softwares
No response

fix: https://github.com/privet-kitty/Extras/commit/7349ea767d81e7920df129eeff95afcfc17898c5
I've confirmed that installation succeeds and the bin and the shortcut are available.


ModuleNotFoundError: No module named 'SigProfilerExtractor'

This is not an issue per se but I think someone else would probably look at here to find a solution.
It seems there is a typo in the readme:
pip install sigproextractor

this should be:
pip install SigProfilerExtractor

otherwise then there is an error in R, python does not find the correct package.
Thanks!

This was indeed a typo, and it's corrected now. Thanks so much for pointing this out!


CONFIG_XUK_64_BIT_ABI is referenced but undefined (outside of tests)

No XUK_64_BIT_ABI Kconfig symbol ever existed. CONFIG_XUK_64_BIT_ABI is only set in arch/x86_64/core/xuk-config.h, which seems to be used for testing, so the #ifdef CONFIG_XUK_64_BIT_ABI in arch/x86_64/core/xuk.c is dead code outside of testing.
Trying to remove CONFIG_FOO references where FOO is not a Kconfig symbol, for a CI check.

Yeah, this was an evolution of the internal configuration values to kconfigs, except that Zephyr itself has never built in anything but x32, so it never got referenced.  Will make it internal again pending a work to make zephyr work with 64 bit pointer sizes.


Update prompt to be quicker

Change how the prompt is built so that we quickly render most of the vcs info, and asynchronously update the prompt with the git untracked/skipped status
After adding the vcs info and the custom git untracked/skipped info to the prompt I noticed that sometimes it took awhile for the prompt to be ready. To fix this I changed the vcs portion to update asynchronously. This is nice because I can quickly cd through dirs and get work done, but sometimes the vcs info would be stale until the async job finished and updated the prompt.
Eventually I realized 2 things:

The long delay only happened in big git repos
The only portion of the vcs update that was taking a long time was updating the git untracked status (ie. git ls-files --other --exclude-standard)

This commit changes the prompt back to adding the vcs info synchronously and only kicks off the git untracked/skipped status gathering asynchronously.
With these change:

I can move around the shell quickly
git branch, staged, unstaged and action messages (rebase, merge, etc) update quickly and are never stale
I still get my custom git untracked/skipped status

Tested on my work machine where I have a large git repo that's been giving me trouble.


[SPARK-16905] SQL DDL: MSCK REPAIR TABLE (follow-up)

What changes were proposed in this pull request?
This PR split the the single createPartitions() call into smaller batches, which could prevent Hive metastore from OOM (caused by millions of partitions).
How was this patch tested?
Will test this manually on millions of partitions.

Test build #63633 has started for PR 14607 at commit f2f3150.

Test build #63635 has started for PR 14607 at commit 21a6e1e.

Test build #63636 has started for PR 14607 at commit ec2d8da.

Test build #63633 has finished for PR 14607 at commit f2f3150.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #63635 has finished for PR 14607 at commit 21a6e1e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63635/
Test PASSed.

Test build #63636 has finished for PR 14607 at commit ec2d8da.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63636/
Test PASSed.

Test build #63646 has started for PR 14607 at commit c442b75.

Test build #63646 has finished for PR 14607 at commit c442b75.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63646/
Test PASSed.

Merged build finished. Test PASSed.

@yhuai @sameeragarwal @rxin I had updated the MSCK REPAIR TABLE to list all the leaf files in parallel to avoid the listing in Hive metastore, hopefully this could speed up it a lot (not benchmarked yet).

Test build #63707 has started for PR 14607 at commit b3797c9.

Test build #63708 has started for PR 14607 at commit 05ab7fc.

Test build #63709 has started for PR 14607 at commit 1c490ef.

Test build #63707 has finished for PR 14607 at commit b3797c9.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63707/
Test FAILed.

Test build #63708 has finished for PR 14607 at commit 05ab7fc.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63708/
Test FAILed.

Merged build finished. Test FAILed.

Test build #63709 has finished for PR 14607 at commit 1c490ef.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63709/
Test FAILed.

@davies  Can you create a new JIRA ticket for this change? It is a non-trivial follow-up.

Test build #63794 has started for PR 14607 at commit f30e387.

Test build #63793 has finished for PR 14607 at commit e2ca4e5.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63793/
Test FAILed.

Test build #63794 has finished for PR 14607 at commit f30e387.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63794/
Test FAILed.

Merged build finished. Test FAILed.

Test build #63800 has started for PR 14607 at commit a4d07db.

Test build #63800 has finished for PR 14607 at commit a4d07db.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63800/
Test PASSed.

Merged build finished. Test PASSed.

Test build #64017 has started for PR 14607 at commit 8a18bf7.

Test build #64017 has finished for PR 14607 at commit 8a18bf7.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64017/
Test FAILed.

Test build #64024 has started for PR 14607 at commit 0672c89.

Test build #64024 has finished for PR 14607 at commit 0672c89.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64024/
Test PASSed.

Test build #64310 has started for PR 14607 at commit 48ae071.

Test build #64311 has started for PR 14607 at commit 399e38e.

LGTM pending jenkins

Test build #64311 has finished for PR 14607 at commit 399e38e.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
          logWarning(s\"All labels belong to a single class and fitIntercept=false. It's a \" +
   * Set thresholds in multiclass (or binary) classification to adjust the probability of
class MultinomialLogisticRegression @Since(\"2.1.0\") (
          logWarning(s\"All labels belong to a single class and fitIntercept=false. It's\" +
  /** Margin (rawPrediction) for each class label. */
  /** Score (probability) for each class label. */
  class MultinomialLogisticRegressionModelWriter(instance: MultinomialLogisticRegressionModel)
 class ALSWrapperWriter(instance: ALSWrapper) extends MLWriter
 class ALSWrapperReader extends MLReader[ALSWrapper]
 class GaussianMixtureWrapperWriter(instance: GaussianMixtureWrapper) extends MLWriter
 class GaussianMixtureWrapperReader extends MLReader[GaussianMixtureWrapper]
 class IsotonicRegressionWrapperWriter(instance: IsotonicRegressionWrapper) extends MLWriter
 class IsotonicRegressionWrapperReader extends MLReader[IsotonicRegressionWrapper]
 class LDAWrapperWriter(instance: LDAWrapper) extends MLWriter
 class LDAWrapperReader extends MLReader[LDAWrapper]
class JavaClassificationModel(JavaPredictionModel):
class LogisticRegressionModel(JavaModel, JavaClassificationModel, JavaMLWritable, JavaMLReadable):
class DecisionTreeClassificationModel(DecisionTreeModel, JavaClassificationModel, JavaMLWritable,
class RandomForestClassificationModel(TreeEnsembleModel, JavaClassificationModel, JavaMLWritable,
class GBTClassificationModel(TreeEnsembleModel, JavaPredictionModel, JavaMLWritable,
class NaiveBayesModel(JavaModel, JavaClassificationModel, JavaMLWritable, JavaMLReadable):
class MultilayerPerceptronClassificationModel(JavaModel, JavaPredictionModel, JavaMLWritable,
class LinearRegressionModel(JavaModel, JavaPredictionModel, JavaMLWritable, JavaMLReadable):
class DecisionTreeModel(JavaModel, JavaPredictionModel):
class TreeEnsembleModel(JavaModel):
class RandomForestRegressionModel(TreeEnsembleModel, JavaPredictionModel, JavaMLWritable,
class GBTRegressionModel(TreeEnsembleModel, JavaPredictionModel, JavaMLWritable, JavaMLReadable):
class GeneralizedLinearRegressionModel(JavaModel, JavaPredictionModel, JavaMLWritable,
class JavaPredictionModel():
class SubstituteUnresolvedOrdinals(conf: CatalystConf) extends Rule[LogicalPlan]
case class UnresolvedInlineTable(
case class UnresolvedTableValuedFunction(functionName: String, functionArgs: Seq[Expression])
case class UnresolvedOrdinal(ordinal: Int)
case class Literal (value: Any, dataType: DataType) extends LeafExpression with CodegenFallback
abstract class PlanExpression[T <: QueryPlan[_]] extends Expression
abstract class SubqueryExpression extends PlanExpression[LogicalPlan]
case class ListQuery(plan: LogicalPlan, exprId: ExprId = NamedExpression.newExprId)
case class Exists(plan: LogicalPlan, exprId: ExprId = NamedExpression.newExprId)
case class With(child: LogicalPlan, cteRelations: Seq[(String, SubqueryAlias)]) extends UnaryNode
case class SubqueryAlias(
class QuantileSummaries(
  case class Stats(value: Double, g: Int, delta: Int)
class JDBCOptions(
class JacksonParser(
abstract class ExecSubqueryExpression extends PlanExpression[SubqueryExec]
class OrcFileFormat extends FileFormat with DataSourceRegister with Serializable

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64311/
Test PASSed.

Test build #64310 has finished for PR 14607 at commit 48ae071.

This patch passes all tests.
This patch does not merge cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64310/
Test PASSed.

Build finished. Test PASSed.

Merging into master and 2.0 branch.


fix(jar): update keycloakify to latest to fix js jar function for key…

…cloak
Resolves: https://github.com/keycloakify/keycloakify/issues/331 (Thank you @lordvlad!)

Great! Thanks!


Allow users to specify redirect_uri during OAuth flow?

Issue Type

[ ] Bug
[x] Enhancement / Feature request
[x] Question
[ ] Documentation

Description
I have been using your excellent SDK to do two things:

Support installing an app
Support "Sign in with Slack"

My use case treats those as separate concerns in how my application works. My solution for supporting logging in with Slack OAuth involves defining 2 separate com.slack.api.bolt.App instances, with slightly different configuration, so that I can handle OAuth URLs that are distinct for each use case. One App instance is for installing the app, and one is for signing in with Slack.
This strategy has been mostly very practical; I don't have to write any custom OAuth logic since your SDK handles the important parts of the flow. I do need to replace a few components with subclasses to modify behavior slightly for my (perhaps unusual) approach here.

I've subclassed FileOAuthStateService so that I can name the session cookies different things for either Slack App
I need to subclass App itself in order to append a redirect_uri as a query param to the URL used to exchange an OAuth code for an access token

The method com.slack.api.bolt.App::getOauthInstallationUrl builds URLs for exchanging an OAuth code for an access token. Slack's API documentation suggests that if there are multiple possible OAuth redirect URIs configured for an app, then using the redirect_uri query param helps Slack know which OAuth callback URI to send the user to after they accept the scopes.
Here's my question: would it make sense to include the redirect URI as configured on the AppConfig object when building that "OAuth Installation URL"? Defaulting to whatever the user configured on the AppConfig for their app seems sensible for a couple reasons:

If someone is trying to do something like what I'm after here, they will likely have multiple Slack App instances, and the configuration will likely vary in terms of the OAuth URIs
Not being explicit about which redirect URI to use seems potentially risky, because a user might reconfigure their app, and Slack seems to pick the first option in the list unless it's been explicitly told to use one or the other

For now, I've solved my own problem, but I'm wondering whether a general solution might be useful for others.
My proposal would effect this code: https://github.com/slackapi/java-slack-sdk/blob/master/bolt/src/main/java/com/slack/api/bolt/App.java#L317
    public String getOauthInstallationUrl(String state) {
        AppConfig config = config();
        if (config.getClientId() == null || config.getScope() == null || state == null) {
            return null;
        } else {
            String scope = config.getScope() == null ? "" : config.getScope();
            if (config.isClassicAppPermissionsEnabled()) {
                // https://api.slack.com/authentication/migration
                return "https://slack.com/oauth/authorize" +
                        "?client_id=" + config.getClientId() +
                        "&scope=" + scope +
                        "&state=" + state +
						"&redirect_uri=" + config.getRedirectUri(); // Change here
            } else {
                String userScope = config.getUserScope() == null ? "" : config.getUserScope();
                return "https://slack.com/oauth/v2/authorize" +
                        "?client_id=" + config.getClientId() +
                        "&scope=" + scope +
                        "&user_scope=" + userScope +
                        "&state=" + state +
						"&redirect_uri=" + config.getRedirectUri(); // Change here
            }
        }
    }

Requirements

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

@timrourke
Thanks for flagging this. Indeed, the redirect_uri is not yet supported in authorize URL generation (although verification with an oauth.v2.access API call works even without doing so). We can definitely update the logic to support the missing piece.
When it comes to a bit details of the implementation, as the redirect_uri is an optional parameter, we should have null check before appending the param.
This seems to be a relatively easy task. If you're interested in making a pull request to fix this issue, I'm happy to review it. If you're busy, I can work on this probably tomorrow.

@seratch I'll open a PR shortly.

@seratch here is the relevant PR: https://github.com/slackapi/java-slack-sdk/pull/492

Thanks for reporting and fixing this. The change will be included in the next patch version - https://github.com/slackapi/java-slack-sdk/milestone/5

Multiple redirect url not working in slack sso
When we try sso it works fine for the first redirect URL and when doing sso for the second one we are getting {"ok":false,"error":"bad_redirect_uri"}. Please help us in resolving the issue

Multiple redirect url not working in slack sso
When we try sso it works fine for the first redirect URL and when doing sso for the second one we are getting {"ok":false,"error":"bad_redirect_uri"}. Please help us in resolving the issue

Having the same issue. While redirect URL works fine for the first URL, it is throwing {"ok":false,"error":"bad_redirect_uri"} for the second url


testnet

Was going to attempt to test mining on testnet, but it didn't seem to connect, bad POW at block 151 on my testnet nodes, here is the debug.log (may not be very important in the grand scheme of things):
2016-03-02 22:10:38 Myriad version v<IP_ADDRESS>-a195a52-rc1 (2016-03-02 19:40:37 +1300)
2016-03-02 22:10:38 Using OpenSSL version OpenSSL 1.0.1k 8 Jan 2015
2016-03-02 22:10:38 Using BerkeleyDB version Berkeley DB 5.3.28: (September  9, 2013)
2016-03-02 22:10:38 Default data directory /home/xxx/.myriadcoin
2016-03-02 22:10:38 Using data directory /home/xxx/.myriadcoin.testnet2/testnet3
2016-03-02 22:10:38 Using config file /home/xxx/.myriadcoin.testnet2/testnet/myriadcoin.conf
2016-03-02 22:10:38 Using at most 125 connections (65536 file descriptors available)
2016-03-02 22:10:38 Using 8 threads for script verification
2016-03-02 22:10:38 scheduler thread start
2016-03-02 22:10:38 Binding RPC on address :: port 22889 (IPv4+IPv6 bind any: 1)
2016-03-02 22:10:38 Using wallet wallet.dat
2016-03-02 22:10:38 init message: Verifying wallet...
2016-03-02 22:10:38 CDBEnv::Open: LogDir=/home/xxx/.myriadcoin.testnet2/testnet3/database ErrorFile=/home/xxx/.myriadcoin.testnet2/testnet3/db.log
2016-03-02 22:10:38 Bound to [::]:22888
2016-03-02 22:10:38 Bound to <IP_ADDRESS>:22888
2016-03-02 22:10:38 Cache configuration:
2016-03-02 22:10:38 * Using 12.5MiB for block index database
2016-03-02 22:10:38 * Using 29.9MiB for chain state database
2016-03-02 22:10:38 * Using 57.6MiB for in-memory UTXO set
2016-03-02 22:10:38 init message: Loading block index...
2016-03-02 22:10:38 Opening LevelDB in /home/xxx/.myriadcoin.testnet2/testnet3/blocks/index
2016-03-02 22:10:38 Opened LevelDB successfully
2016-03-02 22:10:38 Opening LevelDB in /home/xxx/.myriadcoin.testnet2/testnet3/chainstate
2016-03-02 22:10:38 Opened LevelDB successfully
2016-03-02 22:10:38 LoadBlockIndexDB: last block file = 0
2016-03-02 22:10:38 LoadBlockIndexDB: last block file info: CBlockFileInfo(blocks=1, size=278, heights=0...0, time=2014-02-20...2014-02-20)
2016-03-02 22:10:38 Checking all blk files are present...
2016-03-02 22:10:38 LoadBlockIndexDB: transaction index enabled
2016-03-02 22:10:38 LoadBlockIndexDB: hashBestChain=0000017ce2a79c8bddafbbe47c004aa92b20678c354b34085f62b762084b9788 height=0 date=2014-02-20 06:06:33 progress=0.000027
2016-03-02 22:10:38 init message: Verifying blocks...
2016-03-02 22:10:38  block index              26ms
2016-03-02 22:10:38 init message: Loading wallet...
2016-03-02 22:10:38 nFileVersion = 110200
2016-03-02 22:10:38 Keys: 102 plaintext, 0 encrypted, 102 w/ metadata, 102 total
2016-03-02 22:10:38  wallet                    6ms
2016-03-02 22:10:38 init message: Activating best chain...
2016-03-02 22:10:38 mapBlockIndex.size() = 151
2016-03-02 22:10:38 nBestHeight = 0
2016-03-02 22:10:38 setKeyPool.size() = 101
2016-03-02 22:10:38 mapWallet.size() = 0
2016-03-02 22:10:38 mapAddressBook.size() = 1
2016-03-02 22:10:38 init message: Loading addresses...
2016-03-02 22:10:38 Loaded 0 addresses from peers.dat  0ms
2016-03-02 22:10:38 dnsseed thread start
2016-03-02 22:10:38 Loading addresses from DNS seeds (could take a while)
2016-03-02 22:10:38 net thread start
2016-03-02 22:10:38 opencon thread start
2016-03-02 22:10:38 addcon thread start
2016-03-02 22:10:38 msghand thread start
2016-03-02 22:10:38 init message: Done loading
2016-03-02 22:10:38 0 addresses found from DNS seeds
2016-03-02 22:10:38 dnsseed thread exit
2016-03-02 22:10:38 GUI: PaymentServer::LoadRootCAs: Loaded  174  root certificates
2016-03-02 22:10:38 receive version message: /Satoshi:<IP_ADDRESS>/: version 70003, blocks=384, us=<IP_ADDRESS>:60041, peer=1
2016-03-02 22:10:38 Added time data, samples 2, offset +0 (+0 minutes)
2016-03-02 22:10:38 ERROR: ContextualCheckBlockHeader: incorrect proof of work at height 151
2016-03-02 22:10:38 Misbehaving: <IP_ADDRESS>:20888 (0 -> 100) BAN THRESHOLD EXCEEDED
2016-03-02 22:10:38 ERROR: invalid header received
2016-03-02 22:10:38 ProcessMessages(headers, 19036 bytes) FAILED peer=1
2016-03-02 22:10:39 receive version message: /Satoshi:<IP_ADDRESS>/: version 70003, blocks=384, us=<IP_ADDRESS>:41673, peer=2
2016-03-02 22:10:39 Added time data, samples 3, offset +0 (+0 minutes)
2016-03-02 22:10:39 ERROR: ContextualCheckBlockHeader: incorrect proof of work at height 151
2016-03-02 22:10:39 Misbehaving: <IP_ADDRESS>:20888 (0 -> 100) BAN THRESHOLD EXCEEDED
2016-03-02 22:10:39 ERROR: invalid header received
2016-03-02 22:10:39 ProcessMessages(headers, 19036 bytes) FAILED peer=2
2016-03-02 22:11:39 Adding fixed seed nodes as DNS doesn't seem to be available.
2016-03-02 22:11:55 scheduler thread interrupt
2016-03-02 22:11:55 addcon thread interrupt
2016-03-02 22:11:55 opencon thread interrupt
2016-03-02 22:11:55 msghand thread interrupt
2016-03-02 22:11:55 net thread interrupt
2016-03-02 22:11:55 Shutdown: In progress...
2016-03-02 22:11:55 StopNode()
2016-03-02 22:11:55 Shutdown: done

ah yes, that should be on the known issues list, the parameters for testnet (hard forks etc) in no way match what was in <IP_ADDRESS>/17. there would need to be quite a few changes to chainparams.cpp under the testnet section to bring them in line.

understood, perhaps I'll submit a pull request when I have some time...

fixed finally in commit 9b7f415
Please test and advise. my two testnet nodes (one is 0.11.2 the other <IP_ADDRESS>) are up and talking to yours.

I see your nodes and have connections, thanks.
Works good, was able to mine all 5 algos (no aux-pow)...
so if I understand this correctly:
consensus.nStartAuxPow = 1402000;
consensus.nBlockTimeWarpPreventStart1 = 740500; // block where time warp 1 prevention starts
consensus.nBlockTimeWarpPreventStart2 = 766000; // block where time warp 2 prevention starts
consensus.nBlockTimeWarpPreventStart3 = 1048320; // block where time warp 3 prevention starts
are quite far down the chain, I would guess testnet would be more useful if these started earlier? Maybe that doesn't jive with 0.9.16 testnet... but I don't have a longer testnet to use... Or maybe a plan to reset the testnet to start those earlier at a later date?

Yes, the settings for testnet currently are not good. This is due to what testing 8bitcoder was doing when introducing the changes to block time etc. I would suggest we release a <IP_ADDRESS> version as a maintenance release, which contains changes to testnet to make all of the above more sane for testnet purposes. then we make the same changes in 0.11.2 when it releases.
having a stable testnet would be beneficial, not only for other services, but also for the core wallet code. there's some things i want to try using the consensus machanisms, and testnet is the place for them.

Understood, good idea. I'll take a look at that when time permits.
Well done and Thanks!


Plus sign is not allowed in component names

...this does not let to enter components such as, for example, xerces-c++. As a workaround, it can be entered, and is showing, as xerces-c%2b%2b.

I've confirmed this is not an issue in the 3.0 development branch.


Implement HA clusters on AWS with kops

kops helps you create, destroy, upgrade and maintain production-grade, highly available, Kubernetes clusters from the command line.
Currently if the worker node dies, pods have nowhere to reschedule to, which has been causing site cyclical outages.

This should be fixed with the migration to the new kops cluster.


ETH Pool liquidity is wrong

See below

@alexwhte what's wrong here?

lol it's too late. i was reading in USDC like TVL


How to submit form?

Hey there,
Amazing project, thanks for all the great work!
Just wanted to ask how to submit the form? I am running the destack starter repo locally.
I have

added a component with a form
made a file pages/api/submit.js
selected the form and changed the component settings to:

Method: POST
Action: /api/submit
Async: ON (checked)

selected the Button and set it to SUBMIT.

Hi @apugoneappu,
Glad you liked Destack. Through your issue I realised I missed a bit of config in pages/api/submit.js
To get the form submission to work head back to pages/api/submit.js to find the corrected api route.


Makes the ChannelBuilder build a public method

This PR removes the unnecessary private modifier for the Channel build method.
@vejeta ^^

LGTM

Codecov Report

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

@@           Coverage Diff           @@
##           master      #45   +/-   ##
=======================================
  Coverage   12.01%   12.01%
=======================================
  Files          20       20
  Lines         283      283
  Branches        2        2
=======================================
  Hits           34       34
  Misses        249      249

Impacted Files
Coverage Δ

rpc/src/main/scala/client/package.scala
0% <ø> (ø)
: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 fe137f0...414e059. Read the comment docs.


docs: add k8s & fargate service examples in cd onboarding guide

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?

[ ] Issue
[ ] Feature
[X] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[X] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://657145b3cc7b952ffcabb639--harness-developer.netlify.app


QGIS新建Shapefile图层时，使用中文作为表头新建成功保存后，表头成为乱码，UTF-8、GBK编码都无法转回中文

QGIS新建Shapefile图层时，使用中文作为表头新建成功保存后，表头成为乱码，UTF-8、GBK编码都无法转回中文

@dt123456789 Please report in English, and notice that this repository is for managing QGIS website, not the software

@DelazJ Excuse me, where should I give feedback on my questions? Please guide me. Thank you.

@dt123456789 I do not understand your initial message so hard to tell; Translate and I can try to help.

@DelazJ    When QGIS creates a new Shapefile layer, it uses Chinese as the header of the table. After it is successfully saved, the header becomes garbled, and the UTF-8 and GBK codes cannot be converted back to Chinese. What is the problem, or my operation problem, please guide me, thank you

Clear now.
Can you try on a new QGIS user profile (https://docs.qgis.org/3.22/en/docs/user_manual/introduction/qgis_configuration.html#working-with-user-profiles)? If the bug still occurs, the right place is at https;//github.com/qgis/QGIS/issues but please fill all the requested information there about your environment (OS, QGIS version (better if it is 3.22 or 3.28), exact steps to reproduce,...). Hope that helps.

@DelazJ Please tell me where the feedback is. I can't find the entrance. After entering the entrance on the upper right, I don't know where to enter

The first option (bug/crash...)

@DelazJ
This?

Yes.
Did you try to fix the issue with a new QGIS user profile, as suggested above?

@DelazJ
The problem has not been solved. Thank you for your help. My English is not good. Please understand. Thank you

@DelazJ Where are you from?

@DelazJ For QGIS, this is a very good software. I originally wanted to use the original version, but I can't use English. I can only download the local version. What time is it in France now

This issue report could be closed. See https://github.com/qgis/QGIS/issues/51867.


OptiFine compatibility

It would be nice if OptiFine could be installed along this mod. Right now the game refuses to load if you install OptiFine and Authentic Adventure together.
I assume the lighting fixes of AA interferes with the lighting optimization of OptiFine.
These are the classes in minecraft,jar that conflict between these two mods:

acq.class
lr.class
zq.class
hu.class
jy.class
vl.class
aaw.class
l.class
ct.class

For reference, I'm using OptiFine_1.2.5_HD_A.

Due to complications with OptiFine's licensing, I won't be doing this.


Set default time zone for pool

Sql query: SET TIMEZONE TO 'UTC' not working, I receiving time in my local TZ 2021-03-22 14:57:49.834961+03.
But it work with connect psql:
# select now();
              now
-------------------------------
 2021-03-22 15:02:39.242053+03
(1 row)
# SET TIMEZONE TO 'UTC';
SET
# select now();
              now
-------------------------------
 2021-03-22 12:03:10.843069+00
(1 row)

The issue here it seems is that the connection returned from the pool may be a new connection instance. I can reproduce this with both ext-pgsql and ext-pq, as using SET TIMEZONE TO 'UTC' sets it only for that session, not any subsequent sessions.
This could be done for each connection returned from the pool. There's no way to do this currently, Pool would need a mechanism to specify a set of queries to run on each new connection.
As a workaround, you can use transactions, as that will allow you to perform a series of statements that is guaranteed to use the same connection. Otherwise of course you can of set the timezone on the server.

Revisiting this issue while preparing for the v2.0 release and realized I was incorrect in my last comment, this is currently possible in both 1.x and 2.x. Pass a custom implementation of PostgresConnector (or Connector in v1.x) instance to the pool object, which may then execute queries on the new connection before it is returned to the pool.
As no code changes need to be made, I'm closing this issue. If you have any questions on creating a custom connector, please open a new issue or reach out in chat.


Add option to not print file comments?

I am currently running wp-pot in a watch build task, and php-edits are causing noise in the pot files due to line numbers changing. Would you be open to adding an option to surpress generation of the comments? If so I could create a pull request.

That's a good idea I think, I'm away at vacation right now but I can look into it when I'm back

Added in 1.4.0

Thanks!

Just published gulp-wp-pot 2.1.0 which requires wp-pot 1.4.0


Resource-Level Actions Not Working

Description
Disclaimer
I don't know much about how resource-level Actions are supposed to work, so I'm not much help here and I also could be missing something obvious. Just couldn't find any info in Runway or Statamic docs, on this particular type of action. Very possible I'm just wiring this up wrong. Could also be an issue with Statamic, more so than Runway.
Related PRs

Statamic #10471
Runway #564

Issue
When running an Action at the resource-level, the Action throws an exception that sounds like it's still expecting selected/specific models to be passed through, which are not available (to my understanding) with resource-level Actions.
Steps to reproduce

Setup a simple model/resource
Setup a really simple Action—can just log something in the run method
Don't restrict Action's visibility, so it's available to the resource itself
Attempt to run it from the resource, see error below

Environment

Environment
Environment
Application Name: Statamic
Laravel Version: 11.34.2
PHP Version: 8.2.5
Composer Version: 2.5.4
Environment: local
Debug Mode: ENABLED
URL: runway-sandbox.test
Maintenance Mode: OFF
Timezone: UTC
Locale: en
Cache
Config: NOT CACHED
Events: NOT CACHED
Routes: NOT CACHED
Views: CACHED
Drivers
Broadcasting: log
Cache: file
Database: mysql
Logs: stack / single
Mail: log
Queue: sync
Session: file
Statamic
Addons: 1
Sites: 1
Stache Watcher: Enabled (auto)
Static Caching: Disabled
Version: 5.41.0 Solo
Statamic Addons
statamic-rad-pack/runway: 7.x-dev

What does your action class look like?

Nevermind, I can see it now. I'll fix it shortly.

Yeesh, crazy-fast turnaround—thanks so much, Duncan! 🎉


feat: Introduce kraftfile-runtime builder

This PR introduces a new build context (builder) to the kraft build subcommand which recognizes contexts where a Kraftfile has a runtime element.
In some cases, the user may still wish to use this command (often out of habit) and expect an action to have occurred.  It is possible in this circumstance to build the rootfs which is traditional considered a build step.  Whilst the rootfs may still be constructed at package-time, this introduction offers a convenient mechanism for performing the same action.

Rebased removing the lingering comments, but did not include any new tests.


[nfc] Revert indentation changes

Revert some (apparently) accidental indentation changes to support files, to somewhat simplify the hopefully-upcoming upgrade to RN 0.60.
(This PR contains only whitespace changes.)

The invariant I want to maintain for formatting like indentation is that we can always have the IDE auto-format the code (with one keystroke), and it doesn't make any changes. That minimizes formatting churn, and I also often find it convenient while writing code.
For our main codebase, we check this invariant in CI. For the Android codebase we don't, and so it's incompletely adhered to, but it's generally close. For this file, it looks like if I do that at the tip of this branch, it

reverts this branch's changes;
expands getConversations out from its single line.

That story on Android isn't as well documented or fully automated as in the main codebase, mainly because over the past year or so the Android work has mostly been just me. So it's probably getting to time to fix that. :slightly_smiling_face: There's some discussion in the commit messages of
83134808803920467488fa333b70bdd970d87bab
8c86b820b564f2a0d3d5556fc6b0ec64959ac66c
436800ecdaeab9000005892292995ac17d6849a7
So I think for the Android file here, we should just leave it as is.

For the Info.plist file:
Inadvertently changed in c8818f0a33224f6226667cff4d73e3a5f3832875.
(Understandably so -- this file, uniquely, uses tabs as indentation.)

It looks like that commit actually made the same indentation change you're making.
Then 456352b82 changed it back.
Before that, e5cbe3cc5 also changed it.
I don't really care which it is, but I would like it to stop changing. :-) What happens if you go into Xcode and edit one of the values in this file?
I don't believe the name "React Native standard"; I think what you're describing is just "the sample app in the react-native init template". RN doesn't care what the indentation is.

(Stale, and Chris has the RN 0.60 upgrade anyhow.)


Fix Cascading Bugs in Huggingface and Ollama Embedder

Description
Note
This pull request builds upon the previous PR #1548 , which addressed Bug 1. During further testing, additional bugs were discovered and have been documented and resolved in this PR.
Summary
This pull request addresses a series of bugs in the embeddings especially while using 'huggingface' as embedder. Each fix revealed the next issue, which has been documented and resolved in sequence. I already made PR for Bug1 beforehand itself but discovered series of bugs later on as mentioned below.
Details

Bug 1: ModuleNotFoundError: No module named 'embedding'

Files: mem0/embeddings/huggingface.py and mem0/embeddings/ollama.py
Description: Discoverd the incorrect import of the EmbeddingBase from non-existing directory.
Fix: Fixed the import of the EmbeddingBase class with correct parent directory.

Bug 2: Value error, Unsupported embedding provider: huggingface

File: mem0/embeddings/configs.py
Description: After fixing the Bug 1, config was throwing error for hugginggface as provider in embedder config. 'huggingface' was not included in the validator in EmbedderConfig.
Fix: Added huggingface to the provider checklist in EmbedderConfig.

Bug 3: TypeError: Can't instantiate abstract class HuggingFaceEmbedding with abstract method embed

File: mem0/embeddings/huggingface.py
Description: After fixing the Bug 2, HuggingFaceEmbedding was not being instantiated. The abstract method embed was missing.
Fix: Renamed the get_embedding method to embed in HuggingFaceEmbedding

Bug 4: AttributeError: 'HuggingFaceEmbedding' object has no attribute 'dims'

File: mem0/embeddings/huggingface.py
Description: After fixing the Bug 3, HuggingFaceEmbedding was not being instantiated. The attribute dims was missing.
Fix: Added dims attribute to HuggingFaceEmbedding using get_sentence_embedding_dimension method of SentenceTransformer

Impact
These fixes, by addressing multiple cascading issues, improve the reliability and performance of mem0 while using the custom embedder config (like huggingface, ollama), preventing crashes and unexpected results.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Please delete options that are not relevant.

[x] Test Script (please provide)

Test Script
This code initializes the Memory class with the specified configuration, ensuring that the embedding provider 'huggingface' and the model 'multi-qa-MiniLM-L6-cos-v1' are correctly set up and functional.
from mem0 import Memory

config = {
    "embedder": {
        "provider": "huggingface",
        "config": {
            "model": "multi-qa-MiniLM-L6-cos-v1"
        }
    }
}

m = Memory.from_config(config)

Steps to Reproduce

Bug 1: Try using "huggingface" or "ollama" to the embedding provider in config.
Bug 2: After fixing Bug 1, try to instantiate Memory class using from_config method.
Bug 3: After fixing Bug 2, try to instantiate Memory class using from_config method.
Bug 4: After fixing Bug 3, try to instantiate Memory class using from_config method.

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

Maintainer Checklist

[ ] closes #xxxx (Replace xxxx with the GitHub issue number)
[ ] Made sure Checks passed

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

Closing this PR as changes have already been applied.


The % Enhanced Damage stat is bugged.

Modifying the % Enhanced Damage stat doesn't work at all. The item editor only displays stat 17: item_maxdamage_percent. Modifying this value will change the X% Enhanced Damage text to "Adds Y-X damage", with Y being the value you just set.
So increasing 20% enhanced damage to 30% will change the stat to "Adds 30-20 damage". This is invalid so the change is reverted by the game.
Adding item_maxdamage_percent to an item instead of changing a preexisting one will read:
"Adds Y-undefined damage."
Adding item_mindamage_percent as well to modify the other value will instead read:
"Adds Y-undefined damage.
Enhanced minimum damage."

I just modified a Superior weapon, using the editor I changed it to 15%.  Loaded it up ingame, and instead of 15% enhanced damage, I have 15% enhanced max damage.  Can confirm it is bugged.


Godep gives a godep: "Package not found" error when trying to add a new library to an project already using godep

I have a project where I am using godep. I started the project from scratch, i did a go get for all my dependencies, i did a godep save and godep moved all the dependencies to the vendor folder and every thing works as expected.
But when I add a new dependency and tried to run godep save again, it gives me a  "Package not found" error for packages already moved to the vendor folder by godep previously.
Am i doing something wrong here??

@nithin-bose are you getting the message godep: Package (context) not found?
I'm asking this because I've this problem and I guess it's the same as yours.
All I'm trying to do is to run a godep save on a clean new project, and I got this error.

@galeone See #451, which was merged as v61.

I'm getting a similar error to @galeone . I'm on v61 and go 1.6.1
Trying to re-run godep save gives me godep: Package (fmt) not found

Hmm. 'fmt' is a stdlib package and should be easily found. Is your go install special in any way? How was it installed? Are symlinks involved?
Sent from my iPhone

On Apr 21, 2016, at 06:40, David Dollar<EMAIL_ADDRESS>wrote:
I'm getting a similar error to @galeone . I'm on v61 and go 1.6.1
Trying to re-run godep save gives me godep: Package (fmt) not found
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub

No symlinks. Both godep and go installed from Homebrew.

@ddollar I can't seem to replicate this with the same install. Can your provide a link to a copy of the output from godep save -d -v <pkg spec> >log 2>&1 ? The output may be huge, so it may not attach here.

 If it shows back up again I'll grab a log.

Both godep and go installed from Homebrew.

(Offtopic) pro tip: don't install Go from homebrew (or any package manager). Download a tarball from https://golang.org/dl/ and unzip it into /usr/local/go.

Had this problem with stdlib packages in v61 as well. In v63 it only complained about third party packages. Funny thing is: I ran godep update $mypackage and it complained about another package that was in fact missing in my GOPATH.
@kr Why?

@stroborobo go get obeys local arch. Since code can be deployed to different architectures godep looks at all dependencies. So when you godep save cross arch deps may be missing. I'm not sure what the best solution to that is.

Hey @freeformz :)
Sorry, but how is this relevant for any of the two? In the first case I tried to update a specific package and godep complained that another package is missing. Why would it even check for it's existence? In @kr's suggestion it's about precompiled binaries, but both the homebrew and the golang.org version of Go are precompiled, so where's the difference? And shouldn't godep installed via homebrew rely on GOPATH no matter on which machine is's been compiled on or where it's located on disk?

@stroborobo my comment was wrt third party packages, not the stdlib. godep relies on the $GOPATH reguardless of installation method. As I mentioned above I wasn't able to replicate this with the "same install", meaning I removed my own godep / go and installed them via homebrew. Furthermore I was trying to clarify how godep analyzes packages. It always relies on $GOPATH, but can pull in extra transitive deps across architectures vs. go get. This is important if you develop on OSX, but deploy to Linux, which is a very common case.

How to reproduce with third party packages, note how I the updated package and the missing one are different:
bo-mpb:bo [~/src/go/src/github.com/stroborobo/godep-not-found]I> < Godeps/Godeps.json
{
        "ImportPath": "github.com/stroborobo/godep-not-found",
        "GoVersion": "go1.6",
        "GodepVersion": "v63",
        "Deps": [
                {
                        "ImportPath": "github.com/stroborobo/ansirgb",
                        "Rev": "c938d9fb2e25c965c50f2440483f7c6107b6c71b"
                },
                {
                        "ImportPath": "github.com/stroborobo/myfmt",
                        "Rev": "2320937f75d54eff996d1d4b1bff673a2e76bfec"
                }
        ]
}
bo-mpb:bo [~/src/go/src/github.com/stroborobo/godep-not-found]I> ..
bo-mpb:bo [~/src/go/src/github.com/stroborobo]I> mv myfmt/ myfmt.bak
bo-mpb:bo [~/src/go/src/github.com/stroborobo]I> popd
bo-mpb:bo [~/src/go/src/github.com/stroborobo/godep-not-found]I> godep update github.com/stroborobo/ansirgb
godep: Package (github.com/stroborobo/myfmt) not found
bo-mpb:bo [~/src/go/src/github.com/stroborobo/godep-not-found]I>

Doesn't go get download the complete source code regardless of architecture? How does that differ from godep exactly? I thought godep copied the needed packages into vendor/, stripping subdirectories and stuff.

If $GOPATH/github.com/stroborobo/myfmt isn't there then you will get not found because the package isn't in $GOPATH. The way godep currently works is that to update / save deps the packages need to be in the $GOPATH, even if they are already saved. This is for various reasons.
Re: go get. Yes and No. Yes, in that go get fetches all the source code for your current os/architecture. No, in that the same code can require additional transitive dependencies on different os/architectures. I'm not sure if that's what happened here, but it's likely.

So does godep check the vendor folder for dependencies?? Especially in go 1.6 and above??

@freeformz

The way godep currently works is that to update / save deps the packages need to be in the $GOPATH, even if they are already saved.

I see, assumed it's a bug.

No, in that the same code can require additional transitive dependencies on different os/architectures.

Of course, thanks! ;)
@nithin-bose It does, you'll still have a Godep/ dir though, but it only contains the JSON file (and a Readme).

@stroborobo Maybe I should treat it as a bug.

@stroborobo the only difference I can see while init-ing a project and adding a new dependencies to an existing project using godep is that in the first case, the dependencies are in the src folder, whereas in the second case, godep has already moved existing dependencies to the vendor folder...

I have the same problem here :(

@hadv context is part of go1.7 and therefore requires tip or a go1.7 build.

@freeformz so, how can we use godep for go1.6? Thank you!

@hadv what version of godep are you using? I added support for handling go version build tags a few months ago, so this should work if the lib you are importing is conditionally importing context behind a build flag (unless your godep version is old).
Otherwise the package you are using is just blankly importing context no matter of go version and the package needs to be fixed.

@freeformz thank you, I updated the latest version of godep then it's ok now!

I upgraded my go version to go1.7rc1, and godep stop working:
godep save ./...
zsh: correct './...' to './..' [nyae]? n
godep: Package (github.com/google/gofuzz) not found

Same problem here with go1.7rc2:
$ godep save ./...
godep: Package (bytes) not found

Same issue
$ go version
go1.6.3 darwin/amd64

@Gnouc does anything use it transitively ?
@jingweno I can't replicate, can you provide a step by step reproducer using go1.7rc3 and a clean gopath?
@tejasmanohar This issue talks about several different problems AFAICT, can you be more specific?

$ godep save ./...
godep: Package (encoding/json) not found

Sounds like a problem with the version of go you are using or your install
of go.
On Thu, Jul 21, 2016 at 5:44 PM Tejas Manohar<EMAIL_ADDRESS>wrote:

$ godep save ./...
godep: Package (encoding/json) not found
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tools/godep/issues/447#issuecomment-234425413, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAAAZ_T-Xln2rTvM4TS1bxTwY2RgKmNkks5qYBJXgaJpZM4H-fKt
.

I encounter a similar error to @nithin-bose, i git clone the K8s project(used godep manager packages), and when I add a new package(locally) and exec godep save -d -v nvidia-docker/nvidia > jian.log 2>&1 command, errors as the following:
 versionString() godep v74 (linux/amd64/go1.6.2)
majorGoVersion go1.6
VendorExperiment true
sep /vendor/
dotPackageImportPath: k8s.io/kubernetes
dotPackageDir: /root/code/kubernetes/src/k8s.io/kubernetes
godep: Go Version: go1.6
godep: Finding dependencies for [nvidia-docker/nvidia]
LoadPackages [nvidia-docker/nvidia]
listPackage nvidia-docker/nvidia
findDirForPath nvidia-docker/nvidia <nil>
searching /usr/local/go/src/nvidia-docker/nvidia
searching /root/code/kubernetes/src/nvidia-docker/nvidia
godep: Package (nvidia-docker/nvidia) not found

Am I doing something wrong here? How to solve it?

something weird for me to , i think this might be the same error
➜  kubernetes git:(config-e2e-file) ✗ go version
go version go1.7 darwin/amd64
➜  kubernetes git:(config-e2e-file) ✗ godep version
godep v74 (darwin/amd64/go1.7)

godep save -d -v nvidia-docker/nvidia is telling godep to analyze and
save the deps for the nvidia-docker/nvidia package. Is that what you want?
On Wed, Aug 17, 2016 at 10:11 AM jay vyas<EMAIL_ADDRESS>wrote:

something weird for me to , i think this might be the same error
➜  kubernetes git:(config-e2e-file) ✗ go version
go version go1.7 darwin/amd64
➜  kubernetes git:(config-e2e-file) ✗ godep version
godep v74 (darwin/amd64/go1.7)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tools/godep/issues/447#issuecomment-240480131, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAAAZ-WF5M0UZOPYkeZZT0xrFjQbTUJqks5qg0CygaJpZM4H-fKt
.

Similar issue:
$godep save ./... godep: Package (github.com/kr/logfmt) not found
Versions:
$go version
go version go1.7 darwin/amd64
$godep version
godep v74 (darwin/amd64/go1.7)

Why is that?
godep: Package (github.com/hashicorp/go-cleanhttp) not found

The package is transitively required, possibly in a way where go get may
not fetch it. go get github.com/hashicorp/go-cleanhttp it directly. and
re-save. I the error happens again with the same package there is a
problem. If it happens again with a different package, go get that
package. Rinse and repeat until save. You should only need to do this until
all of the transitive deps are local.
On Wed, Oct 5, 2016 at 4:59 PM Ömer Ufuk Efendioğlu <
<EMAIL_ADDRESS>wrote:

Why is that?
godep: Package (github.com/hashicorp/go-cleanhttp) not found
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tools/godep/issues/447#issuecomment-251833368, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAAAZxIv3kS_lUs_9WWe_-W3Q8qf4DtXks5qxDnEgaJpZM4H-fKt
.

Running into this issue:
$ godep save ./...
godep: Package (github.com/stretchr/testify/assert) not found

$ go version
go version go1.7.1 darwin/amd64
$ godep version
godep v75 (darwin/amd64/go1.7.1)

go get github.com/stretchr/testify/assert; godep save ./...
On Thu, Nov 3, 2016 at 12:18 AM Alyssa Quek<EMAIL_ADDRESS>wrote:

Running into this issue:
$ godep save ./...
godep: Package (github.com/stretchr/testify/assert) not found
$ go version
go version go1.7.1 darwin/amd64
$ godep version
godep v75 (darwin/amd64/go1.7.1)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tools/godep/issues/447#issuecomment-258076815, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAAAZ1oUBiW1z79A1xlykeQoTXTKLYC_ks5q6YrHgaJpZM4H-fKt
.

Same here
$ go version
go version go1.7.1 linux/amd64
$ godep version
godep v75 (linux/amd64/go1.7.1)

I use a workaround method to get all missing packages, but I don't think it's a good idea.
#!/bin/bash
rc="$(godep save 2>&1)"
while echo $rc | grep "not found";
do
  package=$(echo $rc | sed -n "s/godep: Package (\(.*\)) not found/\1/p")
  go get $package
  rc="$(/home/build/go/bin/godep save 2>&1)"
done

godep works off the packages in your $GOPATH. if it can't find a package in
your $GOPATH, then it can't save it, so you need to manually go get it
before saving.
On Wed, Nov 9, 2016 at 7:09 PM yhung124<EMAIL_ADDRESS>wrote:

Same here
$ go version
go version go1.7.1 linux/amd64
$ godep version
godep v75 (linux/amd64/go1.7.1)
I use a workaround method to get all missing packages, but I don't think
it's a good idea.
#!/bin/bash
rc="$(godep save 2>&1)"
while echo $rc | grep "not found";
do
package=$(echo $rc | sed -n "s/godep: Package ((.*)) not found/\1/p")
go get $package
rc="$(/home/build/go/bin/godep save 2>&1)"
done
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tools/godep/issues/447#issuecomment-259590944, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAAAZyXO16l9ohbLXqmbgzbl5BsmDO_Xks5q8orYgaJpZM4H-fKt
.

Same here. I am adding a new package to the same code, and getting two different errors from two different Mac OSX machines.
On one machine, I simply cannot run godep save ./..., it will report error godep: dirty working tree error. Tried it on another machine and now I am getting godep: Package (context) not found.

What version of godep are you on?
If the working tree of a dep is dirty is should say what the dep is. That
needs to be resolved as godep won't know what to vendor if the dep's
working tree is dirty.
I'm pretty sure the restriction on the local project being dirty was
removed many versions ago.
On the other machine I am assuming you have an older version of go/godep as
well.
On Mon, Nov 14, 2016 at 4:16 PM Yi Chen<EMAIL_ADDRESS>wrote:

Same here. I am adding a new package to the same code, and getting two
different errors from two different Mac OSX machines.
On one machine, I simply cannot run godep save ./..., it will report
error godep: dirty working tree error. Tried it on another machine and
now I am getting godep: Package (context) not found.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tools/godep/issues/447#issuecomment-260506661, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAAAZ1dd04WT4d8NpSBZEc8KDNFkBSd9ks5q-PnlgaJpZM4H-fKt
.

Hi,
I am using recent go and godep versions on both Mac OSX machines.
If the working tree of a dep is dirty is should say what the dep is. That
needs to be resolved as godep won't know what to vendor if the dep's
working tree is dirty.
I'm pretty sure the restriction on the local project being dirty was
removed many versions ago.
On the other machine I am assuming you have an older version of go/godep as
well.
On Mon, Nov 14, 2016 at 4:16 PM Yi Chen<EMAIL_ADDRESS>wrote:

Same here. I am adding a new package to the same code, and getting two
different errors from two different Mac OSX machines.
On one machine, I simply cannot run godep save ./..., it will report
error godep: dirty working tree error. Tried it on another machine and
now I am getting godep: Package (context) not found.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tools/godep/issues/447#issuecomment-260506661, or
mute
the thread
<https://github.com/notifications/unsubscribe-auth/
AAAAZ1dd04WT4d8NpSBZEc8KDNFkBSd9ks5q-PnlgaJpZM4H-fKt>
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/tools/godep/issues/447#issuecomment-260828878, or mute
the thread
https://github.com/notifications/unsubscribe-auth/APRAhg1scSU9uDjUBk6-dauRqewhYVd5ks5q-mCTgaJpZM4H-fKt
.

So the packages in google.golang.org/cloud have local, uncommitted changes.
Or something else is going on. If you "cd /Users/yichen/go/src/
google.golang.org/cloud" and run "git status ", what does it say?
On Tue, Nov 15, 2016 at 19:50 Yi Chen<EMAIL_ADDRESS>wrote:

Hi,
I am using recent go and godep versions on both Mac OSX machines.
If the working tree of a dep is dirty is should say what the dep is. That
needs to be resolved as godep won't know what to vendor if the dep's
working tree is dirty.
I'm pretty sure the restriction on the local project being dirty was
removed many versions ago.
On the other machine I am assuming you have an older version of go/godep
as
well.
On Mon, Nov 14, 2016 at 4:16 PM Yi Chen<EMAIL_ADDRESS>wrote:

Same here. I am adding a new package to the same code, and getting two
different errors from two different Mac OSX machines.
On one machine, I simply cannot run godep save ./..., it will report
error godep: dirty working tree error. Tried it on another machine and
now I am getting godep: Package (context) not found.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tools/godep/issues/447#issuecomment-260506661, or
mute
the thread
<https://github.com/notifications/unsubscribe-auth/
AAAAZ1dd04WT4d8NpSBZEc8KDNFkBSd9ks5q-PnlgaJpZM4H-fKt>
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/tools/godep/issues/447#issuecomment-260828878, or
mute
the thread
<
https://github.com/notifications/unsubscribe-auth/APRAhg1scSU9uDjUBk6-dauRqewhYVd5ks5q-mCTgaJpZM4H-fKt
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tools/godep/issues/447#issuecomment-260846754, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAAAZz_gsfw9wQi76MSEzHG7pMQ4LT_oks5q-n2VgaJpZM4H-fKt
.

Aha so "git status" says the "HEAD detached at abc5503". Did "git checkout
master" and now godep save works!
I didn't realize the message dirty working tree meant the dependencis, not
my working repo. Thanks for the help!
On Tue, Nov 15, 2016 at 8:32 PM, Edward Muller<EMAIL_ADDRESS>wrote:

So the packages in google.golang.org/cloud have local, uncommitted
changes.
Or something else is going on. If you "cd /Users/yichen/go/src/
google.golang.org/cloud" and run "git status ", what does it say?
On Tue, Nov 15, 2016 at 19:50 Yi Chen<EMAIL_ADDRESS>wrote:

Hi,
I am using recent go and godep versions on both Mac OSX machines.
If the working tree of a dep is dirty is should say what the dep is.
That
needs to be resolved as godep won't know what to vendor if the dep's
working tree is dirty.
I'm pretty sure the restriction on the local project being dirty was
removed many versions ago.
On the other machine I am assuming you have an older version of
go/godep
as
well.
On Mon, Nov 14, 2016 at 4:16 PM Yi Chen<EMAIL_ADDRESS>wrote:

Same here. I am adding a new package to the same code, and getting
two
different errors from two different Mac OSX machines.
On one machine, I simply cannot run godep save ./..., it will report
error godep: dirty working tree error. Tried it on another machine
and
now I am getting godep: Package (context) not found.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tools/godep/issues/447#issuecomment-260506661,
or
mute
the thread
<https://github.com/notifications/unsubscribe-auth/
AAAAZ1dd04WT4d8NpSBZEc8KDNFkBSd9ks5q-PnlgaJpZM4H-fKt>
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/tools/godep/issues/447#issuecomment-260828878, or
mute
the thread
<
https://github.com/notifications/unsubscribe-auth/APRAhg1scSU9uDjUBk6-
dauRqewhYVd5ks5q-mCTgaJpZM4H-fKt
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tools/godep/issues/447#issuecomment-260846754, or
mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAAAZz_
gsfw9wQi76MSEzHG7pMQ4LT_oks5q-n2VgaJpZM4H-fKt>

.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/tools/godep/issues/447#issuecomment-260851707, or mute
the thread
https://github.com/notifications/unsubscribe-auth/APRAhudB4cYxutxZX4-0lyXnqFWg-e87ks5q-odxgaJpZM4H-fKt
.

Just leaving here a tip that was helpful for me when running godep save I was getting godep: Package (fmt) not found.
I ran go get github.com/tools/godep and then godep save and it worked.
Probably I was running some old version.

Had Package (fmt) not found error. both go and godep were installed via brew.
$ go version
go version go1.7.5 darwin/amd64

The problem was that I was using godep v78, after updating to v79 via
$ brew unlink godep
$ brew install godep

error is gone. Hope it will help someone.

Same issue
godep: package ( {various packages} ) not found

godep will not find packages in vendor?

It does, but you should consider migrating to dep or govendor instead.

The solution to this issue is that, Some packages are missing your default GOPATH where go get <package link> command clones them to.
So run, godep restore to clone again all the missing  dependencies then continue with the process of adding the new dependency.

Thanks @dmigwi. I run a godep restore first. For me, after that the godep save started working.


Raidboss: varis-ex Translation Fr

Can you resolve the conflicts here between your patch and Akurosia's that was already merged?

I put (?) In the name, because they are in the timeline and I wanted to respect the French punctuation.
I did not try to see what it would give.

Thanks!

It's ok. For blade pulse, I put the name of the spam click. I don't know what to put others. Thank you.


Bug fix to negative stock holding

@BruceYanghy @spencerR1992 @XiaoYangLiu-FinRL

This is to solve the bug I found in: https://github.com/AI4Finance-Foundation/FinRL/issues/630

Only the calculation of available_amount should be revised. Why did you revise other codes?

Hi, I just changed the line of calculating the available_amount. I also noticed that the git history showed I changed the whole file, but it was what it was when I committed my change.

Could you please pull the code from master again, and revise, and push it again?

Your codes may be an older version....how about revising the latest codes, and then submitting them?  Thanks very much.


Visual Studio can not find the Microsoft.SqlServer.Dac.Extensions file/assembly

DacFx Version: Microsoft.SqlServer.DacFx 162.0.52
.NET Framework Version: 4.7.2
Environment (local platform and source/target platforms): Windows 10

Steps to Reproduce:

Install Visual Studio 2022 with SSDT
Create new SSDT project
Add .NET Framework 4.7.2 , console project
Install Microsoft.SqlServer.DacFx 162.0.52 package to the .NET 4.7.2 project
Right click the SSDT project, click Schema Compare
Select ANY 2 databases
Save the file
Create a Schema Compare object in the .NET Framework 4.7.2 project with a path to the Schema Compare file that was just created in the previous step
Run the program; you will get the following error

Could not load file or assembly 'Microsoft.SqlServer.Dac.Extensions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified.
Did this occur in prior versions?  If not - which version(s) did it work in?
This error appears to be happening in older versions as well.  Using the deprecated version, DacFx64, I was getting this error:
Could not load file or assembly 'Microsoft.SqlServer.Dac.Extensions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified.
There appears to be a problem with the Nuget package dependencies.  It is not installing Microsoft.SqlServer.Dac.Extensions, or it is in some mystery directory that the Visual Studio devenv process can not find.

(DacFx/SqlPackage/SSMS/Azure Data Studio)

I downgraded all of my projects to .NET Framework 4.6.2 from .NET Framework 4.7.2, and now I am getting a different error message:
Cannot open database "<databaseName>" requested by the login. The login failed.
Login failed for user '<computerName>\<userName>'

Here is the full connection string that I am using:
Data Source=.;Initial Catalog=<databaseName>;Integrated Security=True;Persist Security Info=False;Pooling=False;Multiple Active Result Sets=False;Encrypt=False;Trust Server Certificate=True;Application Intent=ReadOnly;Command Timeout=0

I have tried running the process in CMD, still no cookie.

Is this because the SqlAuthenticationMethod enum only has Active Directory, and SQL password?  Was Integrated Security deprecated?  Anyone have an article or an announcement on this?  Everything on StackOverflow is 10 years old now.  Is any of this stuff going to even work in 10 more years, or am I wasting my time, and wasting my life?

So, apparently this error:
Cannot open database "<databaseName>" requested by the login. The login failed.
Login failed for user '<computerName>\<userName>'

Was because I had a typo in the database name.  What a horribly misleading error message.

As for the original issue, I have worked around the fact that this project does not appear to be supported by .NET Framework 4.7.2 by downgrading all of my projects to 4.6.2.
The way I figured this out was by going to %UserProfile%\.nuget\packages\microsoft.sqlserver.dacfx\162.0.52\lib where I only saw folders for .NET 6.0, 6.2, and Standard 2.1 - nothing for .NET 4.7.2.

I'll leave this open in case anyone from Microsoft wants to add a build target for .NET 4.7.2 on this project.

@Mattnificent I'm trying to repo this issue but I need more info about it. Can you please share part of you C# code to do the compare? I assume you run the Schema Compare in SSDT and the use the scmp file in a C# code. If I'm wrong please let me know. Thanks

That is correct.  Here is the gist of the code:
SqlConnectionStringBuilder SourceDatabase = ...
SqlConnectionStringBuilder DestinationDatabase = ...
var schemaCompare = new SchemaComparison(
    new SchemaCompareDatabaseEndpoint(SourceDatabase.ConnectionString),
    new SchemaCompareDatabaseEndpoint(DestinationDatabase.ConnectionString)
);
SchemaComparisonResult comparisonResult = schemaCompare.Compare();

@Mattnificent in general there shouldn't be any problem with using target 4.7.2 when using DacFx. 4.6.2 dlls should just work. You can see in other packages like https://www.nuget.org/packages/Microsoft.Data.SqlClient/ that they also have target 4.6.2 and you can use it in applications targeting 4.7.2. Now your issue might be related to something else. I tried to repro it with the test app you provided but mine works fine. I just created a simple c# console app and targeted to 4.7.2 and added DacFx nuget. Used you sample code, and it runs just fine. Can you verify the DacFx Extension dlls is added your project is coming from the 162.0.52 Nuget? and is there anything in the assembly binding that might cause any loading issue?

Now that I think of it, when I changed all of my projects to run as .NET 4.6.2, it required me to install the .NET 4.6.2 SDK, if I recall correctly.  But the issue was happening on 2 different computers, so I think the NuGet packages were not resolving correctly or fully for some reason - probably because I created the source project on 2-3 generations of VS ago.  I couldn't even find the dll that VS was referencing.

that can be the issue. Please make sure you have the SDK installed and use latest version of VS. I'm closing this as I don't think this is a DacFx issue but feel free to reopen if you think otherwise.


Updated Neighbors VS Rules Stats considering ASes in ASRelDB but not IRR

-1 for neighbor if no data in AS Relationship DB. -1 for import/export if no data in IRR.

Total: 95911 ASes.

IPython.
In [14]: df[(df['import'] == -1) | (df['export'] == -1)]
Out[14]:
       aut_num  neighbor  import  export
0        37591         2      -1      -1
3        62685         4      -1      -1
6       399566         1      -1      -1
16       11471         1      -1      -1
19       14444         2      -1      -1
...        ...       ...     ...     ...
95883    31957         2      -1      -1
95884    52860         1      -1      -1
95885   264164         2      -1      -1
95895   393635         2      -1      -1
95909   265651         1      -1      -1

[16960 rows x 4 columns]

In [15]: df[(df['neighbor'] == -1)]
Out[15]:
       aut_num  neighbor  import  export
11       55704        -1       0       0
33      202322        -1       3       3
44      199337        -1       2       2
45       44502        -1       1       1
48       24788        -1       0       0
...        ...       ...     ...     ...
95896   142031        -1       0       0
95899    17902        -1       3       3
95900    28529        -1       0       0
95901   208690        -1       0       0
95905   211088        -1       9       9

[20468 rows x 4 columns]

16960 ASes appear in ASRelDB but not IRR.

Wild. Do you mean ~17K ASes have no import/export rules, or that there are no aut-num objects at all?

Wild. Do you mean ~17K ASes have no import/export rules, or that there are no aut-num objects at all?

No aut-num in IRR databases we use.

[ ] whois some of the AS not in IRR.

I realized a potential problem, @cunha. We are using the AS Relationship DB from 2013, which is probably outdated.
Maybe we should update to one of CAIDA's other DBs such as as-relationships?

No longer latest stats after fixing default.

Covered by #95.

I realized a potential problem, @cunha. We are using the AS Relationship DB from 2013, which is probably outdated.
Maybe we should update to one of CAIDA's other DBs such as as-relationships?

We should definitely be using a newer database. Something to match the IRR datasets we're using. Did you do this at one point?

No, we are not using databases from 2013. It is just under that folder. It has been undated regularly since and we have been using the July 2023 one.

Sounds good. Just checking.


[request] Batch Requests

Process requests in batches

Is this different from request collapsing #155 ? if so, how?


rq: add hash aggregate exec

Signed-off-by: hi-rustin<EMAIL_ADDRESS>

We have to use a hashmap for all rows.
Look like: map[each-rows][acc-for-each-cols]
Try change the get value to be a generic method.


Vulnerability

According to https://blog.zimperium.com/freertos-tcpip-stack-vulnerabilities-put-wide-range-devices-risk-compromise-smart-homes-critical-infrastructure-systems/ there are multiple CVE in the amazon FreeRTOS. Which part of the software is affected ? has this been corrected ?

Hi dkalog,
The CVEs listed in the blog post relate to vulnerabilities identified in the FreeRTOS+TCP TCP/IP stack and the Secure Sockets library implementation for FreeRTOS+TCP. We released patches for all identified vulnerabilities on August 22, 2018, as part of Release Version 1.3.2 of this repository. The fixes were also posted to the FreeRTOS kernel repository on SourceForge, and the Amazon FreeRTOS AWS console downloads. We are grateful to the researcher Ori Karliner of Zimperium zLabs, who disclosed these issues to Amazon early.
Version 1.3.2: https://github.com/aws/amazon-freertos/commit/12ea022d0f52df211111a79e0aca1ca7b04639d6
Since Amazon has taken on stewardship of FreeRTOS, we have been adding security measures to proactively identify and fix potential security issues using techniques that include automated checks and penetration testing.
If you discover a potential security issue in this project, we ask that you notify AWS/Amazon Security via our vulnerability reporting page: http://aws.amazon.com/security/vulnerability-reporting/.
Thanks
Shyam


Create a Subdomain

Opening the first PR for one of the 2 tutorials on subdomains.
This one is on how to set up a subdomain
The ultimate goal is to create a learning pathway that helps people to discover the communities existing (the sig tuto), then if needed set up a subdomain, then how to create tools and finally how to add those tools to the subdomain (a 2nd tuto in a another PR)
An idea also was to add a tuto in the final Learning pathway on Galaxy Codex and on the GalaxyLabs (tuto needs to be don here)

Hi all ! Is this tuto ready to ber merged then ? @hexylena @bgruening ?


select function contains error when options share a prefix

The bug
The select function does not execute when the options share a prefix.
To Reproduce
Here is the shared setup:
import guidance

api_key = ... # openai api key

guidance.llm = guidance.llms.OpenAI(
    "gpt-3.5-turbo", api_key=api_key
)

prompt = guidance(
    """
{{#system~}}
You are a helpful and terse assistant.
{{~/system}}

{{#user~}}
{{message}}
{{~/user}}

{{#assistant~}}
{{select 'response' options=response_options}}
{{~/assistant}}
"""
)

The following will not work:
response =prompt(
    message="Hello",
    response_options=[
        "This message contains a greeting.",
        "This message contains financial terminology."
    ]
)

it returns
{{#system~}}
You are a helpful and terse assistant.
{{~/system}}

{{#user~}}
{{message}}
{{~/user}}

{{#assistant~}}
{{select 'response' options=response_options}}
{{~/assistant}}

However, this works fine:
response =prompt(
    message="Hello",
    response_options=[
        "This message contains a greeting.",
        "There is financial terminology in this message."
    ]
)

As with GPT acceleration does not work, I cannot integrate the common prefix into the prompt via:
{{#assistant~}}
This message contains {{select 'response' options=response_options}}
{{~/assistant}}

Presumably, this is a bug with the common prefix acquisition.
While this can be worked around by simply using different prefixes, I think that at the very least the code should throw an error indicating that this is a known problem. As of right now there is no error, just the prompt is returned.
System info (please complete the following information):

Windows 11
python = 3.10.6
guidance==0.0.64

Hopefully this is fixed in the new release, where we fixed a ton of select bugs. Sorry it took us so long to get to it!


Compatibility issues with DRF

I am using:
djangorestframework==3.9.4
django-axes==5.0.9
Django==2.2.3

Problem seems to be here:
https://github.com/jazzband/django-axes/blob/master/axes/helpers.py#L371
def get_client_cache_key(request_or_attempt: Union[HttpRequest, Any], credentials: dict = None) -> str:
    """
    Build cache key name from request or AccessAttempt object.
    :param request_or_attempt: HttpRequest or AccessAttempt object
    :param credentials: credentials containing user information
    :return cache_key: Hash key that is usable for Django cache backends
    """

    if isinstance(request_or_attempt, HttpRequest):
        username = get_client_username(request_or_attempt, credentials)
        ip_address = get_client_ip_address(request_or_attempt)
        user_agent = get_client_user_agent(request_or_attempt)
    else:
        username = request_or_attempt.username
        ip_address = request_or_attempt.ip_address
        user_agent = request_or_attempt.user_agent

    filter_kwargs = get_client_parameters(username, ip_address, user_agent)

    cache_key_components = ''.join(value for value in filter_kwargs.values() if value)
    cache_key_digest = md5(cache_key_components.encode()).hexdigest()
    cache_key = f'axes-{cache_key_digest}'

Instead of having an django.http.request.HttpRequest I have an rest_framework.request.Request
If I refactor:
from rest_framework.request import Request

...

def get_client_cache_key(request_or_attempt: Union[HttpRequest, Any], credentials: dict = None) -> str:
    """
    Build cache key name from request or AccessAttempt object.

    :param request_or_attempt: HttpRequest or AccessAttempt object
    :param credentials: credentials containing user information
    :return cache_key: Hash key that is usable for Django cache backends
    """

    if isinstance(request_or_attempt, (HttpRequest, Request)):
        username = get_client_username(request_or_attempt, credentials)
        ip_address = get_client_ip_address(request_or_attempt)
        user_agent = get_client_user_agent(request_or_attempt)
    else:
        username = request_or_attempt.username
        ip_address = request_or_attempt.ip_address
        user_agent = request_or_attempt.user_agent

    filter_kwargs = get_client_parameters(username, ip_address, user_agent)

    cache_key_components = ''.join(value for value in filter_kwargs.values() if value)
    cache_key_digest = md5(cache_key_components.encode()).hexdigest()
    cache_key = f'axes-{cache_key_digest}'

    return cache_key

It works.
But this refactoring creates a funky dependency to rest_framework. How could this be properly implemented?
Note: the problem was not present with:
django-axes==5.0.9

This is mentioned in the docs:
https://django-axes.readthedocs.io/en/latest/6_integration.html?highlight=DRF#integration-with-django-rest-framework
Does this address your issue?

Hey, and thanks for reporting! Seems like we could swap out the type check and
if isinstance(request_or_attempt, AccessAttempt):
    ...
else:
    ...

Which should check the types about correctly, and also pass with DRF Request objects.
Would you be interested in opening a PR and checking if that would fix the issue?
We can't depend on DRF there so this has to be implemented without it :)

Fix has been released in version 5.0.11.

This is mentioned in the docs:
https://django-axes.readthedocs.io/en/latest/6_integration.html?highlight=DRF#integration-with-django-rest-framework
Does this address your issue?

Not, it is unrelated.
I have anyway a recent DRF, as stated in my explanation.

Fix has been released in version 5.0.11.

Wow, thanks, that was quick! It has even hit pypi! :heart_eyes:


Make process as module

closes #161
Hey @richarddavison , please review carefully, I'm not as good in rust as in js and ts.
But I covered implementation with tests and it looks good to me,
if you see any improvements to make, pls let me know.

I'm a bit concerned about repetition of code, that I have to explicitly remap all functions from global process to module process, but maybe it is the only way in Rust, I'm open to suggestions .

Awesome first PR!


impl topaz config info get

This PR adds the ability to extract values from the info result set
Extract config object out of info result set
topaz config info config
{
  "topaz_certs_dir": "/Users/gertd/.local/share/topaz/certs",
  "topaz_cfg_dir": "/Users/gertd/.config/topaz/cfg",
  "topaz_db_dir": "/Users/gertd/.local/share/topaz/db",
  "topaz_dir": "/Users/gertd/.config/topaz"
}

Extract topaz_certs_dir value out of config sub-object
 topaz config info config.topaz_certs_dir
"/Users/gertd/.local/share/topaz/certs"

Extract topaz_certs_dir value out of config sub-object with --raw | -r option to remove quotes from string.
topaz config info config.topaz_certs_dir -r
/Users/gertd/.local/share/topaz/certs

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1132

Relevant Lines:
4949

💛  - Coveralls


cannot import name _imaging

I've been trying to run for two days this build but after looking for some info, asking to Jimmy and searching through the web I have not luck for a solution. I tried to do another user account in my Mac to restart all the process but neither I had any luck.
The same error is showing to me:
File "gltf2usd.py", line 16, in
from PIL import Image
File "/Library/Python/2.7/site-packages/PIL/Image.py", line 93, in
from . import _imaging as core
ImportError: cannot import name _imaging
Any idea?
Thanks.

@Betbe looks like you may have issues with your install of pillow (https://github.com/python-pillow/Pillow/issues/3088).
You can try installing with pip to see if that works better.