Transfer device to location not working

When trying to transfer a device from one location to another via the "transfer" function in the device settings, it perfectly stays at its location and a success-message prompts.
(my.ctapp.io)

Strange timing:
https://discuss.cucumberwifi.io/t/transferring-boxes-and-locations/589
We have disabled for the short-term and are implementing a verification system as per thread.

This is working again. Needs testing


[OR-1790] fix path

fix path

I'm sorry for my missing.
Thank you!

Thank you. Could you check the path in tokamak-public-release.yml#L80?

Do you mean this line ?
echo "VERSION=$(jq -r .version ./packages/tokamak/${{ matrix.package }}/package.json)" >> $GITHUB_OUTPUT

Thank you. Could you check the path in tokamak-public-release.yml#L80?

Do you mean this line ?
echo "VERSION=$(jq -r .version ./packages/tokamak/${{ matrix.package }}/package.json)" >> $GITHUB_OUTPUT

Yes. I think ./packages/${{ matrix.package }}/package.json is right path.

@boohyung
Should we deploy optimism's instead of ours?

@boohyung Should we deploy optimism's instead of ours?

Sorry I missed it. Let approve this PR. Thanks a lot, Steven!


Fixes failing bounds test

This PR fixes https://github.com/mapbox/geo-viewport/issues/17.
I've added a float-rounding helper to do the precision-based comparison of numbers in the tests.
I avoided any modern ES features, like let or array destructuring, as the rest of the codebase isn't using such things, and has no transpilation set up at this point in time.

@mapsam or @flippmoke If y'all have a spare moment, are you able to help me review this small change?
I'd like to use it as a base for exploring a fix for https://github.com/mapbox/geo-viewport/issues/5, now that node-sphericalmercator supports float-based zooms.

Thanks for the merge @mapsam


Update virtual-networks-name-resolution-ddns.md

In the paragraph for the Windows clients, we mentioned the following:  "Azure leaves the primary DNS suffix blank, but you can set the suffix in the VM"
However, this is not correct based on the note mentioned in the below doc:  https://learn.microsoft.com/en-us/azure/virtual-network/virtual-networks-name-resolution-for-vms-and-role-instances
"If you use Azure Provided DNS then appropriate DNS suffix will be automatically applied to your virtual machines. For all other options you must either use Fully Qualified Domain Names (FQDN) or manually apply appropriate DNS suffix to your virtual machines."

Learn Build status updates of commit 6cc9b21:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/virtual-network/virtual-networks-name-resolution-ddns.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@mbender-ms
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#Please-close


Fixes loading of LLM and linting

awesome, thanks
you've removed dependents to PromptWatch... it is not an ideal solution, I'm planning to make it optional in the future, it is currently a required dependency ... that change was reverted

Whoops! Sorry! Another reason I should have committed the code formatting
before making changes!
Glad to make it work better. Your decorator library is a good example of
having typed output easily and having retries.
On Sun, 11 Jun 2023, 12:28 ju-bezdek, @.***> wrote:

awesome, thanks
you've removed dependents to PromptWatch... it is not an ideal solution,
I'm planning to make it optional in the future, it is currently a required
dependency ... that change was reverted
—
Reply to this email directly, view it on GitHub
https://github.com/ju-bezdek/langchain-decorators/pull/3#issuecomment-1586123236,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACK33ZS6ANU45FW34VKFWTXKWTWDANCNFSM6AAAAAAZCHP3HU
.
You are receiving this because you authored the thread.Message ID:
@.***>


Issue126 clang format license check

Add Contributing.md guide to propose / document style guidelines.
Add scripts to check formatting and licensing at checkin.
Signed-off-by: Alan Jowett<EMAIL_ADDRESS>

Coverage decreased (-2.3%) to 80.715% when pulling c126ec19ad71f4fb2012cf5c3e0be9b687bf729e on Alan-Jowett:issue126-clang-format-license-check into 6bd288ba564487a0779096b6978ca521fa2ff65e on iovisor:main.


Quicksight analysis and dashboard unable to be created

Stack deploys fine, but the analysis and dashboard are unable to be created, doing a describe on both of these quicksight resources I have a status of CREATION_FAILED with ERROR message: "you do not have permissions to access the source", "ACCESS_DENIED".
I suspect the issue is because the source is in us-east region which is blocked for us. Are you able to add ca-central-1 region or any other option?

Thanks for opening the issue. The solution supports any region (including ca-central-1) as long as the underlying services used in the solution are supported in that region. It seems to me that the error you got might be with the permission of the account you ran the describe command instead of issues with the solution. Can you please elaborate more about your problem? As one of the pre-requisites, a QS enterprise account must be created prior to deploying the stack https://docs.aws.amazon.com/solutions/latest/devops-monitoring-dashboard-on-aws/automated-deployment.html. If you provided a valid QS account and the stack was created successfully, were you able to open the QS analysis and dashboard console in the region where the solution is deployed? What are you trying to achieve? Steps and screenshots would be helpful for understanding your case.

The issue is that I’m not able to access your us-east quicksight dashboard as we have SCPs that restrict access to Canada only. If you look at your code the template for quicksight dashboards are all hosted in us-east.
One possible resolution is that I’ll uhave to deploy your solution in a personal account, import your quicksight dashboards into ca-central then try update your code to point to this quicksight account to import.

Yes as a work-around. you will need to copy the QS dashboard template into an account (A) in ca-central to which your final target account (B) can have access. You don't need to deploy the solution though,  just call the QS create-template cli command in account A to create a QS template using the solution's QS template. Please make sure that you set up a QS enterprise account with necessary permissions in account A to guarantee that the cli command will run successfully. Once the QS template is in account A, then can deploy the solution in Account B using the QS template ARN in Account A.
https://docs.aws.amazon.com/cli/latest/reference/quicksight/create-template.html
For your convenience, here is the json content which you can save into a json file, and pass the file as value to --cli-input-json parameter in your create-template cli command.
{
    "AwsAccountId": "Your-CA-Account-A",
    "TemplateId": "solutions_aws-devops-monitoring-dashboard_v1_5_0",
    "Name": "solutions_aws-devops-monitoring-dashboard_v1_5_0",
    "Permissions": [
        {
            "Principal": "*",
            "Actions": [
                "quicksight:DescribeTemplate"
            ]
        }
    ],
    "SourceEntity": {
        "SourceTemplate": {
            "Arn": "arn:aws:quicksight:us-east-1:423991167869:template/solutions_aws-devops-monitoring-dashboard_v1_5_0"
        }
    }
}

Sample command:
aws quicksight create-template --cli-input-json path-to-your-json-file --region ca-central-1


create bibtex

Hi Hubert, love your work. Would you mind creating a bibtex for SNAC so we can cite it properly?

thanks! added the citation file, now you can find the bibtex in the repository's right pane


Text selection with round corner

Is your feature request related to a problem? Please describe.
No response
Describe the solution you'd like
Text selection with round corner like in VSCode.
Describe alternatives you've considered
VSCode
Additional context

Hi @wayou, this feature sounds like a great idea, but we need consistency throughout our app and it is supposed to have a mimicked UI of Xcode. Xcode doesn't have round text selection. I might reconsider opening a PR for this if you tell me why we need round text selection. Every CodeEdit feature is backed by why it was implemented. Also, please provide a detailed response for each question, not just a few words.

@austincondiff oh ok

also I mentioned issue I sent to SwiftTerm in issue #615

Hi @maxsuaudeau, glad to hear your reply. I'm long for such an editor with native experience and Xcode like well designed UI. But Xcode lacks of some convenient behavior like rich language support, format on save, etc.
I think the round text selection looks so great which is much more mac style, so I wonder if we can have it in CodeEdit.


Throw exception when creating more than 1 unique entity

context.isPaused = true; is the generated api for [Unique] components.
var e = context.CreateEntity();
e.isPaused = true; // currently works

The code above should throw an exception to enforce unique entities

I dont think you should throw there unless the unique entity already exists. It would break a ton of my code.
I like to create a unique entity to hold several other components, e.g. a settings entity with several non-unique components attached as the actual settings data. This way i can have easy access to lots of data through context.settingsEntity.settingA.value but also get the benefit of reactive systems and groups that trigger when certain combinations of these components are added or removed. I use this extensively for scene management, input, etc, where i need reactive systems to be able to trigger on combinations of components on the same entity but i also want the benefit of a context-level reference to the entity.
Does that make sense?

Clarification:
e.isPaused = true should only throw, if there is an entity with this component already

I share this concern about enforcing unique components in order to avoid having to do a query if other entities are already applying the not-necessarily unique component.
In the case of a unique component, have you considered removing the generated API for adding and removing a component to an entity?
In this example, do not generate e.isPaused property.
Instead, the context already has context.isPaused property.
And it already has context.pausedEntity entity.
As a user, I would rather not have an API than have an API that punishes me for using it.
Since the context has a way to add the component and for accessing the entity, all the valid use cases are supported, and none of the invalid use cases are left as traps for a user to stumble into.


Include friendlyName/display name in the JWT

Background
it would prove useful if user's display name was included in the token -- thus, the caller would not have to look it up elsewhere
Idea brought up by @fateeand

Implemented as displayname (Our LDAP has this field as displayname) in #61.


makers: go.vim - change to cwd of file in buffer for govet

Why is this needed?

Seems to be required for go, too: https://github.com/neomake/neomake/issues/358#issuecomment-202449049.

This was included in #410.
Thanks!


After release: update containerregistry version

API change check
API changes are not detected in this pull request.


Where is the colors module used in the example

I've was checking out the examples and I found that there's a color module imported in the classic.py example. I don't see where it's used.
https://github.com/Mckinsey666/bullet/blob/f386e2f4b58400d71abec8478a358d2e9b524573/examples/classic.py#L2

Issue is fixed. Thanks!


Calendar Error!

Hello,
you have today changed the Calendar. But now i became a Error when i start the Magic Mirror.
I have looked from which data the error came. And it´s the "node_helper.js"
The exact error can you see on the picture.
I have today the Magic Mirror installed with your Autoinstaller data for the Raspberry Pi!

Just pushed a fix, can you give it a try?

I experienced the same issue just now and it worked for me, thanks for the fix!

@jferch Thanks for testing.


M32 memory check on load

As discussed in #16, this checks model memory on load for file names and loads the model from memory if possible. This has the single culprit that an external change to the model wont be reflected. This is why I added the reload flag to load a file as a new model even if the file exists

So if I understand this correctly, you want to cache previously loaded models.
The thing is, Python already has caching tools in the standard library. It's the @lru_cache decorator from the functools module. We could just slap that decorator on the load() method and it would do almost exactly the same thing with just one line of code. Only then caching would not be optional, which, as you point out, it should be, so as to not confuse users who may have modified the saved model file.
However, we can still invalidate the cache even with the built-in tool. Consider this example:
import mph
from timeit import default_timer as now
from functools import lru_cache

client = mph.Client()
client.load = lru_cache(client.load)   # Adds the cache.

for n in range(1, 6):
    if n == 4:
        client.load.cache_clear()
    t0 = now()
    model = client.load('tests/capacitor.mph')
    print(f'Load #{n} took {now()-t0:.1f} seconds.')

which produces this output:
Load #1 took 15.4 seconds.
Load #2 took 0.0 seconds.
Load #3 took 0.0 seconds.
Load #4 took 1.9 seconds.
Load #5 took 0.0 seconds.

Note how Comsol itself does quite a bit of caching already. The Python cache then gets the load time down to practically zero.
So I think this is not really needed. Users can opt in if they want this and all they need are the standard tool.
There is another difference. With @lru_cache, the cache is indexed by the file parameter which is passed in to load(), not the fully resolved file path. But I think this is a minor issue. However, on that note, it would make sense to expose the .getFilePath() method from the Comsol API as model.path() and client.paths().

Some remarks on this one:

I changed some naming and added the methods to model and client

Buffering will drastically deteriorate if there are results in the model (at least on macOS, (UNIX?))

Using relative pathes has some risk when working interactively, especially since ipython supports shell commands and a cd is issued quite fast. This would break the cache

If we have the two methods as described above, the overhead is 3 lines, no other tools needed. See my reasoning below.

Users can opt in if they want this and all they need is the standard tool.

The thing is (my opinion) - the default user won't. I think code should help any user get the most out of their work, and this can speed up tasks without interfering with the toolset for pro users.

I will make caching strictly opt-in (to go the path of least surprise) and add an access method caching() to the Client class.


(SERVER-876) Only call wrap-params-for-jruby once

In a previous commit, wrap-params-for-jruby was called twice for catalog
requests. This meant that the body of the request would be consumed by
the first call (via body-for-jruby), and then the body would be gone for
the second call. This commit avoids that problem by refactoring
root-routes and the wrap-with-code-id to call wrap-params-for-jruby only
once in all routes. It also adds a test for the failing case where
catalog POST requests had no body.

Looks like multiple Travis failures occurred against this PR...

Yay!

Updated to fix test failures. Take that travis.

:+1:

:+1:

just waiting on Mr. Travis

one travis cell failed fetching the facter gem for spec tests, so i rekicked it.

:+1: Tests pass locally so merging...


fix: bump cainome and revert temporary fix

Time spent on this PR:30min
Resolves: #1009
Pull Request type

Please check the type of change your PR introduces:

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Testing

What is the new behavior?

Does this introduce a breaking change?

[ ] Yes
[x] No

@greged93 i am guessing we just need to revert changes in provider.rs and not from test changes in:
https://github.com/kkrt-labs/kakarot-rpc/pull/954/files
correct?

Correct only the changes in provider, keep the test

@0xicosahedron this looks good and CI is ok, can you just rebase and we can merge


Display estimate gas fee for tokenize, withdraw and stake

Proposed Changes
display estimate gas fee for tokenize, withdraw and staking
Implementation

Calculated by running estimateGas in each contract method
I focused on the implementation of features, so we didn't consider UI/UX much

withdraw modal:

stake modal:

property detail page:

tokenize page:

TODO

[x] more integration tests

@hhatto Greate changes!
But, can you add a tooltip to explain how the gas fees are consumed correctly?
Like this:

The estimated gas fee will vary depending on the congestion of the Ethereum network. However, Ethereum will often return about 20~40% of that amount to you unused. This is because it is difficult to predict in advance the exact computing resources required for an Ethereum transaction. Hence, a wallet asks for slightly more gas money but returns the unused portion to you.

add tooltip for estimated gas fee


[SPARK-6612] [MLLib] [PySpark] Python KMeans parity

The following items are added to Python kmeans:
kmeans - setEpsilon, setInitializationSteps
KMeansModel - computeCost, k

Test build #30809 has started for   PR 5647 at commit 9903837.

Test build #30809 timed out     for PR 5647 at commit 9903837     after a configured wait of 150m.

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

Could anyone please tell us why this timeout has occurred?

Jenkins, retest this please

Test build #30912 has started for   PR 5647 at commit 9903837.

Test build #30912 has finished for   PR 5647 at commit 9903837.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

Test build #30986 has finished for   PR 5647 at commit 9903837.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.
This patch removes the following dependencies:

RoaringBitmap-0.4.5.jar
activation-1.1.jar
akka-actor_2.10-2.3.4-spark.jar
akka-remote_2.10-2.3.4-spark.jar
akka-slf4j_2.10-2.3.4-spark.jar
aopalliance-1.0.jar
arpack_combined_all-0.1.jar
avro-1.7.7.jar
breeze-macros_2.10-0.11.2.jar
breeze_2.10-0.11.2.jar
chill-java-0.5.0.jar
chill_2.10-0.5.0.jar
commons-beanutils-1.7.0.jar
commons-beanutils-core-1.8.0.jar
commons-cli-1.2.jar
commons-codec-1.10.jar
commons-collections-3.2.1.jar
commons-compress-1.4.1.jar
commons-configuration-1.6.jar
commons-digester-1.8.jar
commons-httpclient-3.1.jar
commons-io-2.1.jar
commons-lang-2.5.jar
commons-lang3-3.3.2.jar
commons-math-2.1.jar
commons-math3-3.1.1.jar
commons-net-2.2.jar
compress-lzf-1.0.0.jar
config-1.2.1.jar
core-1.1.2.jar
curator-client-2.4.0.jar
curator-framework-2.4.0.jar
curator-recipes-2.4.0.jar
gmbal-api-only-3.0.0-b023.jar
grizzly-framework-2.1.2.jar
grizzly-http-2.1.2.jar
grizzly-http-server-2.1.2.jar
grizzly-http-servlet-2.1.2.jar
grizzly-rcm-2.1.2.jar
groovy-all-2.3.7.jar
guava-14.0.1.jar
guice-3.0.jar
hadoop-annotations-2.2.0.jar
hadoop-auth-2.2.0.jar
hadoop-client-2.2.0.jar
hadoop-common-2.2.0.jar
hadoop-hdfs-2.2.0.jar
hadoop-mapreduce-client-app-2.2.0.jar
hadoop-mapreduce-client-common-2.2.0.jar
hadoop-mapreduce-client-core-2.2.0.jar
hadoop-mapreduce-client-jobclient-2.2.0.jar
hadoop-mapreduce-client-shuffle-2.2.0.jar
hadoop-yarn-api-2.2.0.jar
hadoop-yarn-client-2.2.0.jar
hadoop-yarn-common-2.2.0.jar
hadoop-yarn-server-common-2.2.0.jar
ivy-2.4.0.jar
jackson-annotations-2.4.0.jar
jackson-core-2.4.4.jar
jackson-core-asl-1.8.8.jar
jackson-databind-2.4.4.jar
jackson-jaxrs-1.8.8.jar
jackson-mapper-asl-1.8.8.jar
jackson-module-scala_2.10-2.4.4.jar
jackson-xc-1.8.8.jar
jansi-1.4.jar
javax.inject-1.jar
javax.servlet-3.0.0.v201112011016.jar
javax.servlet-3.1.jar
javax.servlet-api-3.0.1.jar
jaxb-api-2.2.2.jar
jaxb-impl-2.2.3-1.jar
jcl-over-slf4j-1.7.10.jar
jersey-client-1.9.jar
jersey-core-1.9.jar
jersey-grizzly2-1.9.jar
jersey-guice-1.9.jar
jersey-json-1.9.jar
jersey-server-1.9.jar
jersey-test-framework-core-1.9.jar
jersey-test-framework-grizzly2-1.9.jar
jets3t-0.7.1.jar
jettison-1.1.jar
jetty-util-6.1.26.jar
jline-0.9.94.jar
jline-2.10.4.jar
jodd-core-3.6.3.jar
json4s-ast_2.10-3.2.10.jar
json4s-core_2.10-3.2.10.jar
json4s-jackson_2.10-3.2.10.jar
jsr305-1.3.9.jar
jtransforms-2.4.0.jar
jul-to-slf4j-1.7.10.jar
kryo-2.21.jar
log4j-1.2.17.jar
lz4-1.2.0.jar
management-api-3.0.0-b012.jar
mesos-0.21.0-shaded-protobuf.jar
metrics-core-3.1.0.jar
metrics-graphite-3.1.0.jar
metrics-json-3.1.0.jar
metrics-jvm-3.1.0.jar
minlog-1.2.jar
netty-3.8.0.Final.jar
netty-all-4.0.23.Final.jar
objenesis-1.2.jar
opencsv-2.3.jar
oro-2.0.8.jar
paranamer-2.6.jar
parquet-column-1.6.0rc3.jar
parquet-common-1.6.0rc3.jar
parquet-encoding-1.6.0rc3.jar
parquet-format-2.2.0-rc1.jar
parquet-generator-1.6.0rc3.jar
parquet-hadoop-1.6.0rc3.jar
parquet-jackson-1.6.0rc3.jar
protobuf-java-2.4.1.jar
protobuf-java-2.5.0-spark.jar
py4j-<IP_ADDRESS>.jar
pyrolite-2.0.1.jar
quasiquotes_2.10-2.0.1.jar
reflectasm-1.07-shaded.jar
scala-compiler-2.10.4.jar
scala-library-2.10.4.jar
scala-reflect-2.10.4.jar
scalap-2.10.4.jar
scalatest_2.10-2.2.1.jar
slf4j-api-1.7.10.jar
slf4j-log4j12-1.7.10.jar
snappy-java-<IP_ADDRESS>.jar
spark-bagel_2.10-1.4.0-SNAPSHOT.jar
spark-catalyst_2.10-1.4.0-SNAPSHOT.jar
spark-core_2.10-1.4.0-SNAPSHOT.jar
spark-graphx_2.10-1.4.0-SNAPSHOT.jar
spark-launcher_2.10-1.4.0-SNAPSHOT.jar
spark-mllib_2.10-1.4.0-SNAPSHOT.jar
spark-network-common_2.10-1.4.0-SNAPSHOT.jar
spark-network-shuffle_2.10-1.4.0-SNAPSHOT.jar
spark-repl_2.10-1.4.0-SNAPSHOT.jar
spark-sql_2.10-1.4.0-SNAPSHOT.jar
spark-streaming_2.10-1.4.0-SNAPSHOT.jar
spire-macros_2.10-0.7.4.jar
spire_2.10-0.7.4.jar
stax-api-1.0.1.jar
stream-2.7.0.jar
tachyon-0.5.0.jar
tachyon-client-0.5.0.jar
uncommons-maths-1.2.2a.jar
unused-1.0.0.jar
xmlenc-0.52.jar
xz-1.0.jar
zookeeper-3.4.5.jar

@FlytxtRnD Checking in: What's the status of fixing the tests?  Thanks!  It will be great to get this merged for 1.4 (deadline this Friday!)
Btw, the earlier test timeout was from Jenkins having a bunch of issues, but hopefully those are resolved now.

@jkbradley, we are facing some issues with python 3 support. We are working on it and will fix it asap.

Test build #31114 has started for   PR 5647 at commit 7ecfd00.

Test build #31114 has finished for   PR 5647 at commit 7ecfd00.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

@FlytxtRnD That tiny issue is the only one I see.  After that, I think this PR will be good to go.  Thanks!

Merged build triggered.

Merged build started.

Test build #31233 has started for   PR 5647 at commit d6d3a09.

Test build #31233 has finished for   PR 5647 at commit d6d3a09.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

Merged build finished. Test PASSed.

After the seed update, it really will be it.  Thanks!

Merged build triggered.

Merged build started.

Test build #31382 has started for   PR 5647 at commit 0319821.

Test build #31382 has finished for   PR 5647 at commit 0319821.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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/31382/
Test PASSed.

@jkbradley , could you please check if this is ready to merge?

LGTM.  I'm rerunning tests once more for safety sake and then will merge this.  Thank you!

Test build #758 has started for   PR 5647 at commit 0319821.

Test build #758 has finished for   PR 5647 at commit 0319821.

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

unrelated failure...

Test build #763 has started for   PR 5647 at commit 0319821.

Test build #763 has finished for   PR 5647 at commit 0319821.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Test build #764 has started for   PR 5647 at commit 0319821.

Test build #764 has finished for   PR 5647 at commit 0319821.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build triggered.

Merged build started.

Test build #31828 has started for   PR 5647 at commit 8aac002.

Test build #31828 has finished for   PR 5647 at commit 8aac002.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
abstract class Evaluator extends Params
abstract class PipelineStage extends Params with Logging
class BinaryClassificationEvaluator extends Evaluator with HasRawPredictionCol with HasLabelCol
case class Not(child: Expression) extends UnaryExpression with Predicate with ExpectsInputTypes
case class And(left: Expression, right: Expression)
case class Or(left: Expression, right: Expression)
abstract class BinaryComparison extends BinaryExpression with Predicate
trait StringRegexExpression extends ExpectsInputTypes
trait CaseConversionExpression extends ExpectsInputTypes
case class Substring(str: Expression, pos: Expression, len: Expression)

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

Merged build finished. Test PASSed.

@FlytxtRnD The update confused github.  Can you please try closing and re-opening this PR to force github to recompute the diff?  Thanks!

@jkbradley ok, will close and reopen this.  Could you please tell us why this happened?

Hm, that didn't seem to fix it.  It looks like you merged this branch with the current master; sometimes, that confuses Github.  In general, you shouldn't need to merge with master unless Jenkins posts a notice that the PR can't be merged cleanly.
Can you please try rebasing your branch off of the current master?  Perhaps that will fix it.

@jkbradley , we merged it with branch-1.4 to get rid of the failed Mima tests in jenkins.

"Can you please try rebasing your branch off of the current master? Perhaps that will fix it."
We didn't get your point.. To which branch should we rebase to?

Looking at those logs, I don't think the MIMA tests actually failed.  Basing this off of "master" should be fine.  Could you please try that?
If that does not work, then perhaps you could identify your commit hashes, reset this branch to master,  cherry pick your commits, and then force push to your remote branch to update this PR.

@jkbradley , ok we'll try your steps.

Merged build triggered.

Merged build started.

Test build #31851 has started for   PR 5647 at commit b9e451b.

Test build #31851 has finished for   PR 5647 at commit b9e451b.

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/31851/
Test FAILed.

Jenkins, retest this please

Merged build triggered.

Merged build started.

Test build #31865 has started for   PR 5647 at commit b9e451b.

Test build #31865 has finished for   PR 5647 at commit b9e451b.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
abstract class ShuffleHandle(val shuffleId: Int) extends Serializable
  *   class SomethingNotSerializable
      logDebug(s" + cloning the object $obj of class $
abstract class Evaluator extends Params
abstract class PipelineStage extends Params with Logging
class BinaryClassificationEvaluator extends Evaluator with HasRawPredictionCol with HasLabelCol
trait LDAOptimizer
class EMLDAOptimizer extends LDAOptimizer
class OnlineLDAOptimizer extends LDAOptimizer
class SaslEncryption
 static class EncryptedMessage extends AbstractReferenceCounted implements FileRegion
class SaslRpcHandler extends RpcHandler
public class SaslServerBootstrap implements TransportServerBootstrap
public class SparkSaslClient implements SaslEncryptionBackend
public class SparkSaslServer implements SaslEncryptionBackend
public class ByteArrayWritableChannel implements WritableByteChannel
class ParamGridBuilder(object):
abstract class Dialect
  class DialectException(msg: String, cause: Throwable) extends Exception(msg, cause)
case class Not(child: Expression) extends UnaryExpression with Predicate with ExpectsInputTypes
case class And(left: Expression, right: Expression)
case class Or(left: Expression, right: Expression)
abstract class BinaryComparison extends BinaryExpression with Predicate
trait StringRegexExpression extends ExpectsInputTypes
trait CaseConversionExpression extends ExpectsInputTypes
case class Substring(str: Expression, pos: Expression, len: Expression)
case class HiveDatabase(
abstract class TableType
case class HiveStorageDescriptor(
case class HivePartition(
case class HiveColumn(name: String, hiveType: String, comment: String)
case class HiveTable(
trait ClientInterface
class ClientWrapper(
class IsolatedClientLoader(
protected trait ReflectionMagic
 protected implicit class InstanceMagic(a: Any)
 protected implicit class StaticMagic(c: Class[_])

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/31865/
Test PASSed.

Merged into master and branch-1.4. Thanks!

@jkbradley , @mengxr  Thanks for the help!


Insert image/link/video is broken in action-out page after opening a condition or filter

Glitch in the page that prevents inserting link/video/image in the editor after the filter or one condition has been edited or created.

Solved in Version 2.6.1


connection not unlockable

no sign of unlocking it in code

in "the" code. sorry for bad typo

Press "c"

It was bugged! @mahdiyadiilavi1 fixed it in #184

It worked for me lol

pressing "c" works but the normal way to unlock it didn't

Oh, I just pressed random buttons on my keyboard until it worked lol


Unhandled rejection Error: ENOENT: no such file or directory

Get this error when building for OSX. Here is the complete error. The gulp task was working until I updated gulp-electron.
Unhandled rejection Error: ENOENT: no such file or directory, open '/Users/chrisgriffith/Documents/Development/ConfiGAP/release/v0.36.5/darwin-x64/ConfiGAP.app/Contents/Frameworks/ConfiGAP Helper.app/Contents/Info.plist'
at Error (native)
at Object.fs.openSync (fs.js:549:18)
at Object.fs.readFileSync (fs.js:397:15)
at /Users/chrisgriffith/Documents/Development/ConfiGAP/node_modules/gulp-electron/index.js:478:34
at Promise._execute (/Users/chrisgriffith/Documents/Development/ConfiGAP/node_modules/gulp-electron/node_modules/bluebird/js/release/debuggability.js:272:9)
at Promise._resolveFromExecutor (/Users/chrisgriffith/Documents/Development/ConfiGAP/node_modules/gulp-electron/node_modules/bluebird/js/release/promise.js:473:18)
at new Promise (/Users/chrisgriffith/Documents/Development/ConfiGAP/node_modules/gulp-electron/node_modules/bluebird/js/release/promise.js:77:14)
at distributePlist (/Users/chrisgriffith/Documents/Development/ConfiGAP/node_modules/gulp-electron/index.js:472:10)
at /Users/chrisgriffith/Documents/Development/ConfiGAP/node_modules/gulp-electron/index.js:284:18
at tryCatcher (/Users/chrisgriffith/Documents/Development/ConfiGAP/node_modules/gulp-electron/node_modules/bluebird/js/release/util.js:16:23)
at Promise._settlePromiseFromHandler (/Users/chrisgriffith/Documents/Development/ConfiGAP/node_modules/gulp-electron/node_modules/bluebird/js/release/promise.js:502:31)
at Promise._settlePromise (/Users/chrisgriffith/Documents/Development/ConfiGAP/node_modules/gulp-electron/node_modules/bluebird/js/release/promise.js:559:18)
at Promise._settlePromise0 (/Users/chrisgriffith/Documents/Development/ConfiGAP/node_modules/gulp-electron/node_modules/bluebird/js/release/promise.js:604:10)
at Promise._settlePromises (/Users/chrisgriffith/Documents/Development/ConfiGAP/node_modules/gulp-electron/node_modules/bluebird/js/release/promise.js:683:18)
at Async._drainQueue (/Users/chrisgriffith/Documents/Development/ConfiGAP/node_modules/gulp-electron/node_modules/bluebird/js/release/async.js:138:16)
at Async._drainQueues (/Users/chrisgriffith/Documents/Development/ConfiGAP/node_modules/gulp-electron/node_modules/bluebird/js/release/async.js:148:10)

Having the same issue here: https://github.com/mainyaa/gulp-electron/issues/38

Fixed by v0.1.3.


ftl::TaskScheduler allocator support

As we start to allocate more and more with various improvements, we need a way of getting an allocator in there. My ideal would be to use std::polymorphic_allocator<byte> + std::memory_resource but that's c++17. Then our options are to reimplement memory_resource etc. in abseil style (there is a boost license c++11 impl of both), or to embed the allocator into the type. As a task scheduler is basically a global, embedding it in the type isn't the end of the world but would also require everything to be in the header which is less than ideal.

I haven't read into polymorphic_allocator or memory_resource much, but they both look much better than the default std allocators. When I was first creating this library, I was looking at EASTL, due to the pain of std allocators.
With polymorphic_allocator, can this be used with std containers? Or would we need to replace the containers with something else?

A simpler, albeit less elegant, solution would be a C-style allocation callback.

The standard solution is to use std::pmr::vector etc. which are just aliases for std::vector<T, std::pmr::polymorphic_allocator<T>> and so on. It truly is the solution to the C++ allocator nonsense. The std::pmr::polymorphic_allocator accepts a std::pmr::memory_resource* which does the allocation through its protected polymorphic do_allocate, do_deallocate and do_is_equal. Override those 3, you have full polymorphic allocation, i.e. per object non-typed differences in allocation strategy.
The C++11 impl is here, provided by the guy who gave the talk that made me understand it: here (1hr). If you do want to adopt it (my preference), I would definitely recommend going the abseil strategy of just aliasing the actual type if it is available.
C callback would work, wouldn't particularly make me happy tho 😄 .

I'm working on a first attempt this weekend.

Good stuff, aiming for the allocator method?
Connor Fitzgerald
Sent from my Phone. Please excuse my brevity.
On November 17, 2018 9:40:48 AM EST, RichieSams<EMAIL_ADDRESS>wrote:

I'm working on a first attempt this weekend.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/RichieSams/FiberTaskingLib/issues/76#issuecomment-439622056

Yea. I'm using his C++11 impl of pmr. I'll have to figure out the best way to switch to the c++17 one if that's available

Take a look at Google's abseil library and how they implement std::any, std::variant, etc.
Connor Fitzgerald
Sent from my Phone. Please excuse my brevity.
On November 17, 2018 11:48:29 AM EST, RichieSams<EMAIL_ADDRESS>wrote:

Yea. I'm using his C++11 impl of pmr. I'll have to figure out the best
way to switch to the c++17 one if that's available
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/RichieSams/FiberTaskingLib/issues/76#issuecomment-439630564


Compliance Policy Review and Comparison with Federal Process

Describe the issue
Conduct a policy review of the federal government to understand the division of compliance work between the center and the agencies. This will help with the understanding of the process for the federal equivalent of corporate vs. ministry level compliance documents, and understand how that maps to BC's process. This will also help us understand how we are matching up with the experience we are describing as part of the Pathfinder experience - for example, that some of the compliance 'heavy lifting' will already have been done (as per the Shared Responsibility Model).
Which Sprint Goal is this issue related to?

Note: 'Milestone' is a ZenHub term that we use synonymously with 'Sprint'.

The 'Milestone' description (created in ZenHub) should clearly list Sprint Goals. This section should indicate which 'goal' this issue is related to.
Additional context

This task will provide important context and alignment with proposed components of the HADF strategy.
Julian & Shea have a lot of context over the GoC teams and how they are working

Definition of done

Stuart to share product lifecycle with Emily
Start with meeting with Julian & Shea for knowledge transfer
Summary document outlining federal process for providing services to their agencies, including any compliance work done up front and any work that remains for the agencies to take on.
Summary document will also include overview of provincial process including the policy background of compliance documents.
Cap to 3 this sprint (we could go waaaayyyy deeper)
factor in capabilities and capacities of the whole government, org capability vs team capacity

I think this work is no longer happening as Emily has moved on to other priorities.  Closing, can reopen if that's incorrect


feat: Add Deno.core.createCancelHandle()

This is a common utility used in ext/fs and ext/net in deno repo.
It's weirdly created in ext/web so it can be shared between the two extensions.

LGTM, but I am not sure we want to use this API forever -- it would actually be a lot better as a cppgc object. Happy to pull this code into deno_core for now at least.

That's a good idea. But let's pull the code to limit the blast radius in deno.


gptman: 0.8.0 -> 0.8.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/cecton/gptman/releases.
meta.description for gptman is: "A CLI tool for Linux to copy a partition from one disk to another and more."
meta.homepage for gptman is: "https://github.com/cecton/gptman"
meta.changelog for gptman is: ""
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
/nix/store/kfxsz1xc4cs87ca7gmf8qjxnkcgyvplg-gptman-0.8.2/bin/gptman passed the binary check.
1 of 1 passed binary check by having a zero exit code.
1 of 1 passed binary check by having the new version present in output.
found 0.8.2 with grep in /nix/store/kfxsz1xc4cs87ca7gmf8qjxnkcgyvplg-gptman-0.8.2
found 0.8.2 in filename of file in /nix/store/kfxsz1xc4cs87ca7gmf8qjxnkcgyvplg-gptman-0.8.2

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
docbookrx
gptman

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/kfxsz1xc4cs87ca7gmf8qjxnkcgyvplg-gptman-0.8.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A gptman https://github.com/r-ryantm/nixpkgs/archive/b574732548fcb92ca7c0c1736e5063e2ad709298.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/kfxsz1xc4cs87ca7gmf8qjxnkcgyvplg-gptman-0.8.2
ls -la /nix/store/kfxsz1xc4cs87ca7gmf8qjxnkcgyvplg-gptman-0.8.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

gptman

Maintainer pings
cc @akshgpt7 for testing.

Change looks good to me

Change looks good to me


packaging/optional_ovos_classifiers

ovos-classifiers is still in pre-alpha and wont have a stable release anytime soon, make it an optional dependency of ovos-core 0.0.8

Summary by CodeRabbit

New Features

Improved error handling for the answer selection process, ensuring service functionality even if the re-ranking component is unavailable.
Conditional loading of classifiers based on configuration settings to enhance performance and efficiency.
Enhanced entity extraction logic to prevent errors when necessary keywords are unavailable.

Bug Fixes

Adjusted logic to default to a random answer when the re-ranker is not available, avoiding potential failures.

Chores

Removed unnecessary dependency on ovos-classifiers, simplifying project requirements.
Added ovos-classifiers dependency to the testing requirements for enhanced testing capabilities.

closes https://github.com/OpenVoiceOS/ovos-core/issues/436


AMBARI-23778 p2 Ambari assigns /home for NameNode, DataNode and NodeM…

…anager directories
What changes were proposed in this pull request?
Bugfix of the previous patch
How was this patch tested?
unit test

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2212/
Test PASSed.

@swagle Please review
Created a new PR for this fix to avoid merge problems. This is meant to be on top of the previous fix, that was buggy.

Were committed alternative version of the patch


typo in clusterserviceversion file

BucketClass is written as BacketClass in the clusterseriviceversion file.
Signed-off-by: Santosh Pillai<EMAIL_ADDRESS>

This typo is coming from Nooba. Closing this PR.


Support for new macOS 13 API's

In macOS Ventura, Apple released several new Virtualization API's in beta.
Major one being support for VZEFILoader for running LinuxVM. It would be great if we can provide support for those new API's.
List of new API's,
General

[ ] VZVirtualMachineStartOptions
[ ] VZUSBMassStorageDeviceConfiguration
[ ] Consoles
[ ] Clipboard Sharing

Linux

[ ] VZEFIBootLoader
[ ] VZEFIVariableStore
[ ] VZLinuxRosettaDirectoryShare
[ ] VZVirtioGraphicsDeviceConfiguration

MacOS

[ ] VZMacTrackpadConfiguration

@balajiv113 Thanks!
I will support them after Apple released macOS 13.x.

macOS 13 has been released! I work for this!


Geocoder and Google

Anyone aware of changes (presumably on Google's side) since Mon Jan 4 which are now breaking the geocoder.google() method in Geocoder?

Hey Bahman, I've tested the google provider on my computer and restarted the Travis CI build and it seems to work fine. You might have reached the rate limit that Google has, you can only request 2500/requests/day.
https://travis-ci.org/DenisCarriere/geocoder/jobs/100031243#L245
Do you have any errors?
g = geocoder.google("Ottawa, ON")
print g
 <[OK] Google - Geocode [Ottawa, ON, Canada]>
print g.error
''

Exactly what I was going to say Bahman, most likely a rate limit issue, I
am having no issues getting 300 out of 300 rooftops returned just this
minute.
On Tue, Jan 5, 2016 at 4:32 PM, Denis<EMAIL_ADDRESS>wrote:

Hey Bahman, I've tested the google provider on my computer and restarted
the Travis CI build and it seems to work fine. You might have reached the
rate limit that Google has, you can only request 2500/requests/day.
https://travis-ci.org/DenisCarriere/geocoder/jobs/100031243#L245
Do you have any errors?
g = geocoder.google("Ottawa, ON")print g
<[OK] Google - Geocode [Ottawa, ON, Canada]>print g.error''
—
Reply to this email directly or view it on GitHub
https://github.com/DenisCarriere/geocoder/issues/169#issuecomment-169139941
.

--
Colin Johnson
C:703.829.6243 (google voice)
C:434.989.2091 (verizon)
<EMAIL_ADDRESS>

It is not a rate limit issue. I am familiar with those, haven't been hitting their API at all lately, and can reproduce the issue with different IP addresses. (I am not using an API key or anything, so I'm guessing the only identifier would be the IP address.)
It does not come back with any error. In fact, it does not come back at all. I have to terminate the process manually. The traceback is pasted below. It gets stuck at self._connect().
All other service providers seem to work fine, by the way.
/usr/local/lib/python2.7/site-packages/geocoder/base.pyc in _initialize(self, **kwargs)
116         self.content = None
117         self.encoding = kwargs.get('encoding', 'utf-8')
--> 118         self._connect(url=self.url, params=self.params, **kwargs)
119         self._build_tree(self.content)
120         self._exceptions()
/usr/local/lib/python2.7/site-packages/geocoder/base.pyc in _connect(self, **kwargs)
91                 self.status_code = 200
92         except KeyboardInterrupt:
---> 93             sys.exit()
94         except requests.exceptions.SSLError:
95             self.status_code = 495

Which python version are you using?? There's some SSL issues with the
versions below 2.7
Try using the latest version of python.
On Jan 5, 2016 4:46 PM, "Bahman Engheta"<EMAIL_ADDRESS>wrote:

It is not a rate limit issue. I am familiar with those, haven't been
hitting their API at all lately, and can reproduce the issue with different
IP addresses. (I am not using an API key or anything, so I'm guessing the
only identifier would be the IP address.)
It does not come back with any error. In fact, it does not come back at
all. I have to terminate the process manually. The traceback is pasted
below. It gets stuck at self._connect().
All other service providers seem to work fine, by the way.
/usr/local/lib/python2.7/site-packages/geocoder/base.pyc in
_initialize(self, *
*kwargs) 116 self.content = None 117 self.encoding =
kwargs.get('encoding', 'utf-8') --> 118 self._connect(url=self.url,
params=self.params, **kwargs)
119 self._build_tree(self.content)
120 self._exceptions()
/usr/local/lib/python2.7/site-packages/geocoder/base.pyc in _connect(self,
**kwargs)
91 self.status_code = 200
92 except KeyboardInterrupt:
---> 93 sys.exit()
94 except requests.exceptions.SSLError:
95 self.status_code = 495
—
Reply to this email directly or view it on GitHub
https://github.com/DenisCarriere/geocoder/issues/169#issuecomment-169143317
.

I've updated the Geocoder PyPi version (1.7.1), I've had this type of error before and I fixed it in the pass few Github versions.
$ pip install geocoder --upgrade

Or download the latest version from Github
$ git clone https://github.com/DenisCarriere/geocoder
$ cd geocoder
$ pip install .

The upgrade(s) helped.
Thank you for your prompt response!


Improve authentication mechanisms

This somewhat-large PR (sorry! 😊) does the following:

Adds support for API tokens as an authentication mechanism
Supports the new web session ID cookies required for crumb-based authentication as of Jenkins 2.190.1 (see https://jenkins.io/security/advisory/2019-07-17/#SECURITY-626)
Adds a new jenkins_auth variable to determine which authentication mechanism to use

Also adds a new none authentication type (for jenkins_auth) which won't bother with API tokens or crumbs, if requested

Adds a new section about authentication and security in the README
Adds a new molecule scenario test for API token-based authentication, which also lays the groundwork for testing other things like this

Fixes https://github.com/emmetog/ansible-jenkins/issues/47, ping @emmetog

Thanks for the PR @nre-ableton !


"All Posts" should appear above "Meta" in menu

The "ALL POSTS" menu item in the top left nav menu should appear above the "META" and "ABOUT" items

Seems reasonable. Just pushed this on devel, should be pushed soon.


Support consuming binary messages

In #29 support was added for producing raw buffer messages. However, as far as I understand, support for consuming binary messages is missing. The decoder converts message values to strings by assumption before calling the callback.
There should be a configuration parameter that controls whether to interpret messages as strings or not, or alternatively, the possibility to provide a custom message decoder function.

We can add a option to determine whether to convert message to string or not.

The consumer support encoding option now, when set it to buffer, the message value will be a raw buffer

Works beautifully. Thanks!


Bug: Can't install on macos 10.14.6

I can't install on my macos 10.14.6. I get this error:
$ opam pin add --yes https://github.com/CraigFe/oskel                             Tue Jan 07 15:25 00:00:01
Processing: [oskel: http]
[oskel] downloaded from https://github.com/CraigFe/oskel
[ERROR] Could not retrieve Unknown archive type:
/private/var/folders/wg/zz5h9br17xl76_k2755c5wtr0000gn/T/opam-86976-9b8556/oskel
I've been trying to get an ocaml setup to learn the language, but I'm having a hard time getting things to work.

Thanks for the issue. This was a typo I made when creating the installation instructions: you need a .git prefix for Git HTTPS remotes, so something like:
opam pin add --yes https://github.com/CraigFe/oskel.git

should work for you. I've fixed this in https://github.com/CraigFe/oskel/pull/3.

Feel free to re-open if you still have issues installing :slightly_smiling_face:

It still will not install. I had to redo my ~/.opam directory in order to run these commands. But, as you can see, they are erroring:
$ opam pin add --yes https://github.com/CraigFe/oskel.git                         Tue Jan 07 20:52 00:00:00
Processing: [oskel: git]
[oskel] synchronised from git+https://github.com/CraigFe/oskel.git
Package oskel does not exist, create as a NEW package? [Y/n] y
[oskel.~dev] synchronised from git+https://github.com/CraigFe/oskel.git
oskel is now pinned to git+https://github.com/CraigFe/oskel.git (version ~dev)
The following dependencies couldn't be met:

oskel → unix
unknown package

[NOTE] Pinning command successful, but your installed packages may be out of sync.
$ opam install oskel                                                              Tue Jan 07 20:53 00:00:00
<><> Synchronising pinned packages ><><><><><><><><><><><><><><><><><><><><>  🐫
[oskel.~dev] no changes from git+https://github.com/CraigFe/oskel.git
The following dependencies couldn't be met:

oskel → unix
unknown package

No solution found, exiting

That one was a silly mistake on my part, fixed by https://github.com/CraigFe/oskel/commit/58cf8a39cac78fc2c756449a0c3361d7a98a0c64. Currently in the process of adding CI to this repository...

It’s now dying with:
[ERROR] The compilation of oskel failed at "/Users/raguay/.opam/opam-init/hooks/sandbox.sh build dune build -p
oskel -j 3 @install".
#=== ERROR while compiling oskel.~dev =========================================#
context     2.0.5 | macos/x86_64 | ocaml-system.4.08.1 | pinned(git+https://github.com/CraigFe/oskel.git#58cf8a39)
path        ~/.opam/default/.opam-switch/build/oskel.~dev
command     ~/.opam/opam-init/hooks/sandbox.sh build dune build -p oskel -j 3 @install
exit-code   1
env-file    ~/.opam/log/oskel-76288-e18f3c.env
output-file ~/.opam/log/oskel-76288-e18f3c.out
output
File "bin/dune", line 3, characters 55-63:
3 |  (libraries cmdliner fmt fmt.cli fmt.tty logs logs.cli logs.fmt oskel))
^^^^^^^^
Error: Library "logs.fmt" not found.
Hint: try: dune external-lib-deps --missing -p oskel -j 3 @install
<><> Error report <><><><><><><><><><><><><><><><><><><><><><><><><><><><><>  🐫
┌─ The following actions failed
│ λ build oskel ~dev
└─
┌─ The following changes have been performed
│ ∗ install cmdliner     1.0.4
│ ∗ install conf-m4      1
│ ∗ install dune         2.0.1
│ ∗ install fmt          0.8.8
│ ∗ install ocamlbuild   0.14.0
│ ∗ install ocamlfind    1.8.1
│ ∗ install seq          base
│ ∗ install stdlib-shims 0.1.0
│ ∗ install topkg        1.0.1
└─

Apologies. More missing configuration in dune-project, now added.
Unfortunately, Dune doesn't check that it's only relying on dependencies that you've told it about via dune-project.

Thanks. It's working now. Glad to help debug. Now, on to learning Ocaml.
On Tue, Jan 7, 2020 at 10:44 PM Craig Ferguson<EMAIL_ADDRESS>wrote:

Apologies. More missing configuration in dune-project, now added.
Unfortunately, Dune doesn't check that it's only relying on dependencies
that you've told it about via dune-project.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CraigFe/oskel/issues/2?email_source=notifications&email_token=AAS7JR2F5Y4GLFLSDUIKETLQ4SPOXA5CNFSM4KDUQVT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIJJFXY#issuecomment-571642591,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAS7JR6TTCH26QVPK5XF3LTQ4SPOXANCNFSM4KDUQVTQ
.


Before sending the actual handshake packet the peer must obtain a cookie

What actual handshake packet? There has been no previous mention of a handshake packet, why it would be needed, what its purpose would be. This section needs a heading and an introduction.

line numbers man!

For reference this line is located here.


What's the difference between roboto-android and roboto-unhinted?

Google says:

Hinted fonts
Hints are the instructions embedded in a font on how to modify (distort) a glyph
to look better on low-resolution displays. As a tradeoff, a hinted font consumes
more space than the unhinted version.
Both Roboto and Noto have hinted and unhinted versions. Google recommends:

Use the unhinted versions on Android and on Mac OS X, which doesn’t implement hints.

Use hinted fonts on Chrome OS, Windows, and Linux.

What's the difference between roboto-android and roboto-unhinted?
see also: http://stackoverflow.com/questions/38118283/whats-the-difference-between-roboto-android-and-roboto-unhinted

See here for a detailed description of hinting:
https://en.wikipedia.org/wiki/Font_hinting
But practically speaking, this is what happens if you use unhinted Roboto on Windows:
https://github.com/google/roboto/issues/174
This is because the hinted versions contain more data to help Windows fit contours to pixels. But since this data makes the font files significantly larger, you should probably try to use the unhinted versions whenever possible. I actually would not recommend using hinted fonts on any platform other than Windows, including Linux. Unhinted Roboto looks fine on Linux in my experience.

Whoops, misread your question as the difference between hinted and unhinted. The differences between roboto-unhinted and roboto-android are: 1) The Android version has had glyph outlines optimized and some data dropped which Android doesn't use anyways. You can see this in the smaller font file sizes. 2) The metrics of the font are manually set to be compatible with older versions of Roboto, with values that the Android operating system expects.


[Bug] Manifest generation failure

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the issue
DBT compilation error when run in Fivetran dbt Core transformation.
Code is executed successfully in local machine
Relevant error log or model output
21:06:55  Encountered an error:
'<' not supported between instances of 'NoneType' and 'int'
21:06:55  Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/dbt/cli/requires.py", line 91, in wrapper
    result, success = func(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/dbt/cli/requires.py", line 76, in wrapper
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/dbt/cli/requires.py", line 169, in wrapper
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/dbt/cli/requires.py", line 198, in wrapper
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/dbt/cli/requires.py", line 245, in wrapper
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/dbt/cli/requires.py", line 271, in wrapper
    ctx.obj["manifest"] = parse_manifest(
  File "/usr/local/lib/python3.8/dist-packages/dbt/parser/manifest.py", line 1790, in parse_manifest
    manifest = ManifestLoader.get_full_manifest(
  File "/usr/local/lib/python3.8/dist-packages/dbt/parser/manifest.py", line 318, in get_full_manifest
    manifest = loader.load()
  File "/usr/local/lib/python3.8/dist-packages/dbt/parser/manifest.py", line 478, in load
    self.parse_project(
  File "/usr/local/lib/python3.8/dist-packages/dbt/parser/manifest.py", line 674, in parse_project
    parser.parse_file(block)
  File "/usr/local/lib/python3.8/dist-packages/dbt/parser/base.py", line 483, in parse_file
    self.parse_node(file_block)
  File "/usr/local/lib/python3.8/dist-packages/dbt/parser/base.py", line 436, in parse_node
    config: ContextConfig = self.initial_config(fqn)
  File "/usr/local/lib/python3.8/dist-packages/dbt/parser/base.py", line 397, in initial_config
    config_version = min([self.project.config_version, self.root_project.config_version])
TypeError: '<' not supported between instances of 'NoneType' and 'int'

Expected behavior
Run without compilation erros
dbt Project configurations

# Name your project! Project names should contain only lowercase characters
# and underscores. A good package name should reflect your organization's
# name or the intended use of these models
name: 'fivetran_dbt'
version: "1.0"

# This setting configures which "profile" dbt uses for this project.
profile: 'fivetran_dbt'

# These configurations specify where dbt should look for different types of files.
# The `model-paths` config, for example, states that models in this project can be
# found in the "models/" directory. You probably won't need to change these!
model-paths: ["models"]
analysis-paths: ["analyses"]
test-paths: ["tests"]
seed-paths: ["seeds"]
macro-paths: ["macros"]
snapshot-paths: ["snapshots"]

clean-targets:         # directories to be removed by `dbt clean`
  - "target"
  - "dbt_packages"

# Configuring models
# Full documentation: https://docs.getdbt.com/docs/configuring-models
vars:
  ad_reporting__amazon_ads_enabled: False # by default this is assumed to be True
  ad_reporting__apple_search_ads_enabled: False # by default this is assumed to be True
  ad_reporting__facebook_ads_enabled: True # by default this is assumed to be True
  ad_reporting__google_ads_enabled: True # by default this is assumed to be True
  ad_reporting__linkedin_ads_enabled: False # by default this is assumed to be True
  ad_reporting__microsoft_ads_enabled: False # by default this is assumed to be True
  ad_reporting__pinterest_ads_enabled: False # by default this is assumed to be True
  ad_reporting__reddit_ads_enabled: False # by default this is assumed to be True
  ad_reporting__snapchat_ads_enabled: False # by default this is assumed to be True
  ad_reporting__tiktok_ads_enabled: False # by default this is assumed to be True
  ad_reporting__twitter_ads_enabled: False # by default this is assumed to be True

Package versions
packages:

package: fivetran/ad_reporting
version: [">=1.7.0", "<1.8.0"] # we recommend using ranges to capture non-breaking changes automatically

What database are you using dbt with?
databricks
dbt Version
1.8.1
Additional Context
No response
Are you willing to open a PR to help address this issue?

[x] Yes.
[ ] Yes, but I will need assistance and will schedule time during our office hours for guidance
[ ] No.

Hi @leandrocentr thanks for opening this issue. If the code is running successfully locally, but failing when deployed to Fivetran Transformations, then I assume this error is similar (albeit not exactly the same) to Issue #108 and needs to be addressed by the Fivetran Transformations orchestration team.
Unfortunately, our team managing the open source data models does not have full visibility into the orchestration component. I would recommend opening a Support Ticket to have the Fivetran Transformations team take a look at this to see why the manifest is failing when deployed to orchestration.
Let me know if you have any questions. Thanks!

@leandrocentr I saw you were able to create the support ticket! I will close this issue so all future correspondence may be consolidated in the new support ticket you created. Thanks!


Deploying Diego to Bosh-Lite README is out of date.

In the README for generating and deploying bosh-lite, it talks about uploading the garden-runc release and not the garden-linux release. If you then generate the manifest with USE_SQL='mysql' ./scripts/generate-bosh-lite-manifests to deploy, it fails saying that 'garden-linux' is not uploaded.  You need to run USE_SQL='mysql' ./scripts/generate-bosh-lite-manifests -g. But the README also never talks about uploading the Garden-Linux release.
Also to run ./scripts/generate-mysql-bosh-lite-manifest, the cf-mysql release needs to exist in your workspace. It would be nice to have the actual location of where it is looking.

Seems like develop has this fixed with garden-runc as the default. Closing the issue

Thanks, @DanLavine! Yeah, we should be fixing all of that as part of https://www.pivotaltracker.com/story/show/133000417.


dotfiles v1.2.0 QA

### Tasks
- [ ] https://github.com/ec-intl/dotfiles/issues/43
- [ ] https://github.com/ec-intl/dotfiles/issues/48

Done


Add/Modify/Improve Cypher Functions

Excited to work on Cypher Functions!

please assign this to me, I'll work on randomUUID().

Assign it to me as well! I'll work on:

toBoolean()
toInteger()
toFloat()


Added M_INFO field

According to version 5.0 of the documentation from 12 Aug 2024 M_INFO field is required for transactions with VISA cards

Thank you for the PR.

According to the documentation, the data should be in this format: {"email":"user@sample.com","cardholderName":"CARDHOLDER NAME","mobilePhone":{"cc":"359","subscriber":"8939999888"}}. At least this is how the example is given. You've mentioned it incorrectly in README.md.
Will there be any issues if this field is not filled in with the current version?
The tests are failing and we will need to update them to accommodate this parameter.

We need to resolve these issues before we can merge the PR.

Have added backward compability and updated information in the README file

@rosen4o Thank you for the PR. It has been merged.