Api docs deployment

Current status:

github pages hosted on master branch
every file is downloaded on it's own
slow initialization speed
once we release we would have to use develop branch and we won't be able to see progress

Ideas:

add to ci task to bundle all files into single blob (this would be useful for converting other formats as well)
publish current development render from gh-pages branch and using bundle instead of separate files (faster loading speed)
have some hosting service/server where we would host released api docs (are we gonna have some stable releases?)
publish every PR somewhere and use some github bot to comment url of build

@djrtwo does EF already has some services to host static files?
I can create PR for CI to bundle and publish master on gh-pages (to render current development) but we would still require some server to host site with some stable version of api.

cc: @Souptacular

If you care to do versioned releases of the API spec here on Github, you could host the bundle file with the release. You could do it all from CI, too. Not sure if that suits your requirements.
https://help.github.com/en/github/administering-a-repository/about-releases

If you care to do versioned releases of the API spec here on Github, you could host the bundle file with the release. You could do it all from CI, too. Not sure if that suits your requirements.
https://help.github.com/en/github/administering-a-repository/about-releases

Thats an option too, in that case we would deploy swagger-ui on gh-pages and use top bar dropdown to choose bundle dev/v1/v2 🤔

Having versioned releases would be quite nice for my use-case (API conformance testing tool). Not necessary, but nice.


Make PostgreSQL maintenance playbooks more flexible

Allow for manual initiation and disabling of maintenance mode outside of a reboot.
Also add clearing of proxy cache, which was necessary but missing.
This is a minor change I had hanging around and had forgotten about until now, but will be convenient to have merged before making some upcoming changes to production.  It makes it more convenient to put the frontend into maintenance mode for something other than a server reboot.  It also fixes the fact that the proxy cache was not being cleared.

:100:


Bug: PoseStamped.idl: Illegal identifier: geometry_msgs::msg::pose is already defined

Summary
This bug reports a illegal identifier when trying to generate PoseStamped. One theory is because Pose and pose are different cases, but treated the same.
Versioning
OS: Ubuntu 22.04
Hash: 0eed89a87427d3faca9a05e61bf31509529d627d
Message: ROS2 geometry_msgs/msg/PoseStamped
Steps to reproduce

Take the ROS2 IDL's that make up PoseStamped, dump them in a single folder, and edit the includes to not have folders
Run the genretor on messages sequentially to build up PoseStamped.

 microxrceddsgen Quaternion.idl
 microxrceddsgen Point.idl
 microxrceddsgen Time.idl
 microxrceddsgen Header.idl
 microxrceddsgen Pose.idl
microxrceddsgen PoseStamped.idl

Observe this error:

$ microxrceddsgen libraries/AP_XRCE_Client/Idl/PoseStamped.idl
openjdk version "11.0.18" 2023-01-17
OpenJDK Runtime Environment (build 11.0.18+10-post-Ubuntu-0ubuntu122.04)
OpenJDK 64-Bit Server VM (build 11.0.18+10-post-Ubuntu-0ubuntu122.04, mixed mode, sharing)
Loading templates...
Processing the file libraries/AP_XRCE_Client/Idl/PoseStamped.idl...
libraries/AP_XRCE_Client/Idl/PoseStamped.idl:15:35: error: Illegal identifier: geometry_msgs::msg::pose is already defined (Definition: com.eprosima.idl.parser.tree.TypeDeclaration@6aeb35e6)

Message Definition
ROS2 .msg
$ ros2 interface show geometry_msgs/msg/PoseStamped
# A Pose with reference coordinate frame and timestamp

std_msgs/Header header
        builtin_interfaces/Time stamp
                int32 sec
                uint32 nanosec
        string frame_id
Pose pose
        Point position
                float64 x
                float64 y
                float64 z
        Quaternion orientation
                float64 x 0
                float64 y 0
                float64 z 0
                float64 w 1

IDL
$ PoseStamped.idl
// generated from rosidl_adapter/resource/msg.idl.em
// with input from geometry_msgs/msg/PoseStamped.msg
// generated code does not contain a copyright notice

#include "Pose.idl"
#include "Header.idl"

module geometry_msgs {
 module msg {
   @verbatim (language="comment", text=
     "A Pose with reference coordinate frame and timestamp")
   struct PoseStamped {
     std_msgs::msg::Header header;

     geometry_msgs::msg::Pose pose;
   };
 };
};

You need to run the generator with the case sensitive -cs option.
I have added it on this PR: https://github.com/eProsima/Micro-XRCE-DDS-Gen/pull/61

Closing as the fix is merged, feel free to reopen if any other problem appears

I have run into a similar issue trying to generate .h and .c files for ROS packages installed by default with the ros-base Humble variant.
If I copy all .idl files from /opt/ros/humble/share to ~/idl:
mkdir ~/idl
pushd /opt/ros/humble/share
find . -name "*.idl" -exec cp --parents {} ~/idl \;

I then get the following structure:
❯ ls
rmw_dds_common
std_msgs
nav_msgs
map_msgs
...

For example:
Processing the file idl/rmw_dds_common/msg/ParticipantEntitiesInfo.idl...
In file included from idl/rmw_dds_common/msg/NodeEntitiesInfo.idl:5:0,
                      idl/rmw_dds_common/msg/ParticipantEntitiesInfo.idl:6:0:
idl/rmw_dds_common/msg/Gid.idl:8:12: error: 'uint8' was redefined
In file included from idl/rmw_dds_common/msg/NodeEntitiesInfo.idl:5:0,
                      idl/rmw_dds_common/msg/ParticipantEntitiesInfo.idl:6:0:
idl/rmw_dds_common/msg/Gid.idl:9:15: error: Illegal identifier: rmw_dds_common::msg::Gid is already defined (Definition: com.eprosima.idl.parser.tree.TypeDeclaration@70cf32e3)
In file included from idl/rmw_dds_common/msg/NodeEntitiesInfo.idl:5:0,
                      idl/rmw_dds_common/msg/ParticipantEntitiesInfo.idl:6:0:
idl/rmw_dds_common/msg/Gid.idl:13:0: error: Unexpected input '}'

If I just generate from .idls in geometry_msgs, everything works as expected:
find idl/geometry_msgs -name "*.idl" -exec bash -c './micro-xrce-dds-gen/scripts/microxrceddsgen -cs -I idl -d $(dirname "$1") -replace "$1"' x {} \;

The packages with issues:

rmw_dds_common
nav_msgs
map_msgs
tf2_msgs
visualization_msgs
sensor_msgs
pcl_msgs
stereo_msgs
trajectory_msgs


update for studio v3

@runeb How do you feel about this PR?

FWIW I've been using this branch in my local v3 development and the changes work just fine.

Changes incorporated in #22, full commit intact. Thanks so much for this, Kristi 🙏
Closing in favor of that branch.


Request for DOT aggregate functionality

It would be nice if InfluxDB offered a dot-product functionality, specifically for continuous queries. Something like:
SELECT DOT(valueA, valueB) FROM my_measurement WHERE time>now()-1h AND time<now()

Do you mean
"dot-product functionality" === if  vector valueA is perpendicular with vector valueB, then the dot product is equal to zero
R> a <- c(1,2,3,4,5)
R> b <- c(2,4,6,8,10)
R> dot <- acos( sum(a*b,na.rm=TRUE) / ( sqrt(sum(a * a,na.rm=TRUE)) * sqrt(sum(b * b,na.rm=TRUE)) ) )
R> dot
[1] 0

Yes that is correct, although I admit I don't really follow your example in R. I'm looking for something similar to numpy.dot() for Python. So,
In [1]: import numpy

In [2]: a = 1, 2, 3, 4, 5

In [3]: b = 2, 4, 6, 8, 10

In [4]: dot = numpy.dot(a, b)

In [5]: dot
Out[5]: 110

@amancevice both valueA and valueB are scalars. InfluxDB doesn't allow you to store vectors or arrays.

@mjdesa Apologies I must not be explaining myself. Say I have a query that looks like this:
SELECT DOT(valueA, valueB) AS dot FROM my_measurement GROUP BY time(1d)

I would expect InfluxDB to select valueA and valueB from the measurement, group them into 1d groups, and apply the aggregation DOT() on each group where the inputs are the columns, valueA and valueB.
So, if one of the groups in my series looked like this:
time                valueA valueB
1357088400000000000 1      2
1357088400000000000 2      4
1357088460000000000 3      6
1357088520000000000 4      8
1357088520000000000 5      10

I would expect the aggregate row to look like this:
time                dot
1357084800000000000 110

Does that make sense? Perhaps there is already a way to get this result but I thought including a DOT() function would be useful enough to include on its own.

@amancevice ah I see what you're saying.
Would the following query work?
SELECT sum(valueA *  valueB) AS dot FROM my_measurement GROUP BY time(1d)

I get the following error trying SUM(valueA * valueB):
ERR: error parsing query: expected field argument in sum()

Ah thats a problem. Looks like we don't support arithmetic in aggregate functions.

Great, thanks for opening the feature request!

Any idea when this might make it into a release? The ability to do arithmetic in queries -- specifically I'm thinking of continuous queries -- is something of a deal-breaker for using InfluxDB in production.

@amancevice at the moment we're rewriting our query engine which will serve as the foundation for building out this sort of functionality. Once the query engine has been rewritten, we'd gladly accept a PR for the functionality you're looking for.

Sounds good

As mentioned in my post to the mailing list we are experimenting with simplifying our open GitHub Issues. This feature request has been rolled into an aggregate issue for all function requests, so that we can close this issue until we are ready to work on it.
You may continue to make comments here. Closing the issue does not mean we are rejecting this idea.

@pauldix


🚀 Feature: Add emojis from emoji-mart (Bluesky)

Bug Report Checklist

[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Overview
Mentioned by @vladimyr.bsky.social: https://bsky.app/profile/vladimyr.bsky.social/post/3kobo3z4ieo2b:

I was wondering what Bluseky's webapp uses and found another potential platform data source: github.com/missive/emoj...
It includes related keywords:
$ curl -sL 'https://github.com/missive/emoji-mart/raw/main/packages/emoji-mart-data/sets/15/native.json' | jq '.emojis.airplane_departure'

Yes!! This is a great idea. I should have thought of searching Bluesky 😄. Accepting PRs!
https://github.com/missive/emoji-mart
Additional Info
I'd mentioned in https://github.com/muan/emojilib/issues/194#issuecomment-2009621140 that a few emojis didn't have keywords in any existing platforms:

🐙 went from 7 keywords to 1: octopus
...
None of the platforms in emoji-platform-data have more than 1-2 keywords for them. Adding in a more rich platform would fill back in those missing keywords. For example, asking the native macOS emoji picker for sea includes 🐙 in the results. I added emoji-platform-data issues labeled platform support.

Looks like emoji-mart has a lot of good keywords, including 🐙!

"octopus": {
  "id": "octopus",
  "name": "Octopus",
  "keywords": ["animal", "creature", "ocean", "sea", "nature", "beach"],
  "skins": [{ "unified": "1f419", "native": "🐙" }],
  "version": 1
}

Co-authored-by: @vladimyr

@allcontributors add @vladimyr for ideas


[release-1.2] Remove skipped test

Remove test that is anyways skipped, no where used -and hard coded to the "consolidated" dispatcher.

/cherry-pick release-1.4

/cherry-pick main

/retest

/retest

/close


In Edit field, I only want to update data to model when I click on Save button. How can I do?

At present, when I enter data in Edit field, system automatically updates data to model. However, I want to cancel change that I have just done some minutes ago but I can't do. So, what should I do?

Find
    Formbuilder.options = {       BUTTON_CLASS: 'fb-button',       HTTP_ENDPOINT: '',       HTTP_METHOD: 'POST',       HTTP_SAVE_ACTION: '',       HTTP_RESET_ACTION: '',       AUTOSAVE: true,
and change AUTOSAVE to false


[Debug] Search secret doesn't have ca.crt

ca.crt is removed from the search secret with cert manager removal
Debug with comment out secret.Data["ca.crt"]

/test all

/test all

/assign @qiujian16

/lgtm
/assign @zhiweiyin318 leave to you for approve

/hold

/unhold
/approve


dwave.embedding package

Merges the contents of https://github.com/dwavesystems/chimera-embedding under a dwave.embedding namespace

Do you get this?
From: Alexander Condello<EMAIL_ADDRESS>Sent: October 11, 2018 12:12 PM
To: dwavesystems/dwave-system<EMAIL_ADDRESS>Cc: Joel Pasvolsky<EMAIL_ADDRESS>Review requested<EMAIL_ADDRESS>Subject: Re: [dwavesystems/dwave-system] WIP: dwave.embedding package (#111)
@arcondellohttps://github.com/arcondello requested your review on: dwavesystems/dwave-system#111https://github.com/dwavesystems/dwave-system/pull/111 WIP: dwave.embedding package.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHubhttps://github.com/dwavesystems/dwave-system/pull/111#event-1899087107, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AgdtKqsPU-tM6jAoTrzwRhU3BjRmeMfnks5uj5f3gaJpZM4XWQ0T.

I don't know where that came from; it's definitely a typo if you're trying to import https://github.com/dwavesystems/chimera-embedding/blob/master/src/polynomialembedder.py

From: Joel<EMAIL_ADDRESS>Sent: Thursday, October 11, 2018 1:45:40 PM
To: dwavesystems/dwave-system
Cc: Kelly Boothby; Review requested
Subject: Re: [dwavesystems/dwave-system] dwave.embedding package (#111)
Do you get this?
From: Alexander Condello<EMAIL_ADDRESS>Sent: October 11, 2018 12:12 PM
To: dwavesystems/dwave-system<EMAIL_ADDRESS>Cc: Joel Pasvolsky<EMAIL_ADDRESS>Review requested<EMAIL_ADDRESS>Subject: Re: [dwavesystems/dwave-system] WIP: dwave.embedding package (#111)
@arcondellohttps://github.com/arcondello requested your review on: dwavesystems/dwave-system#111https://github.com/dwavesystems/dwave-system/pull/111 WIP: dwave.embedding package.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHubhttps://github.com/dwavesystems/dwave-system/pull/111#event-1899087107, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AgdtKqsPU-tM6jAoTrzwRhU3BjRmeMfnks5uj5f3gaJpZM4XWQ0T.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHubhttps://github.com/dwavesystems/dwave-system/pull/111#issuecomment-429112234, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAixNn9pSEeVTfy9di_ywztbF00-L2c9ks5uj63zgaJpZM4XWQ0T.

I'd like to do away with the terms "evil" and "eden," and "processor."  An "eden processor" is an interface to a chimera graph where every internal coupler exists,  whereas a "processor" is an interface to a chimera graph where some internal couplers are missing. Moreover, these interfaces are made to support embedding by considering bundles of parallel qubits
proposal:
processor -> bundle_embedder
eden_processor -> bundle_embedder_nid -- where NID stands for "no internal defects"
evil_edges -> internal_defects
Additionally, folks may want to discuss the name of "bundles."  They're connected sets of shores.  Keeping with the nautical theme, some folks may want to call these "fjords."  So basically I'll consider any serious suggestion; not fjord.

Another little quirk -- this project is one of many which implements its own chimera adjacency.  I'm not sure if it makes sense to pull that from dwave_networkx, since that might introduce a cyclic dependency down the road.  Thoughts?

@boothby re: dnx, I don't think we'll ever want to make dwave-system a dependency of dwave-networkx. At least not while we aspire to merge dwave-networkx into NetworkX? Anyway, right now dnx is more general then dws, and it makes sense to just use that functionality from dnx, not reimplement it.


Deploy to Heroku

deployed Heroku.

waiting to deploy a functioning site.

site works locally. next deployment to heroku should deploy successfully


caves does not implement hello?

A java kv integration test fails on something (simplified):
    MutationResult insertResult = collection.insert(id, "Hello, World");
    assertTrue(insertResult.mutationToken().isPresent());

Looks like mutation tokens are not yet implemented for caves?

Naturally, this makes all persistTo/replicateTo tests fail as well.

It appears the real cause here is that the mock does not reply with the actual list of supported features.

passing now.


az webapp log tail shows duplicate log entries for function apps

Describe the bug
A user reported in this issue that they were seeing duplicate log entries in az webapp log tail when used on a function app. I believe the fix that the portal took in that issue needs to be taken in the CLI too. The nature of the fix is described in this comment - the logstream should be pointed specifically at the LogFiles\Application\Functions\Host directory using /api/logstream/application/functions/host.
To Reproduce
Run az webapp log tail on a function app, then execute a function.
Expected behavior
Log entries appear once.
Actual behavior
Log entries are duplicated. Example:

Workaround
Until this is fixed, you can use the provider command to force the stream to be scoped to the correct directory. e.g. --provider application/functions/host

webapp

Thanks for reporting. Will take a look.

@paulbatum i am guessing you see the same behavior if you use az functionapp log tail?

Is this command available? I tried it and got this error:
paul@Azure:~$ az functionapp log tail --name pbatum-v3js-win --resource-group pbatum-v3js-win
Unable to load extension 'mesh: unrecognized kwargs: ['min_profile']'. Use --debug for more information.
'tail' is misspelled or not recognized by the system.

Is this command available? I tried it and got this error:
paul@Azure:~$ az functionapp log tail --name pbatum-v3js-win --resource-group pbatum-v3js-win
Unable to load extension 'mesh: unrecognized kwargs: ['min_profile']'. Use --debug for more information.
'tail' is misspelled or not recognized by the system.

Interesting all webapp commands should have an equivalent functional command.

Any update on this? I was seeing double logging in "az webapp log tail" output for a function this week.


Index - Needs work

the index.rst needs some work so that we have sections and the logical groups have group headings.  Someone needs to head-up learning RST format and getting us a good start on MonsterDocs formatting.

Also please figure out how we do previews of these docs so we do not need to develop off of main to preview.  @venomboss985 , just tagging you for awareness on this, if you know how to do this, please let us know.

For preview of docs use a pre-release version (you can change this in vs code extensions) of the esbonio extension https://docs.esbon.io/en/latest/. Also, there needs to be a python interpreter integrated into the vs code project. It does not seem to work with the readTheDocs theme, however by publishing the page on github we can always check.

The section breaks have been added on the branch sectionBreaks which is of dev. The problem was the spacing because the caption does not appear if there is a line break in between the :maxdepth: and the :caption:

@SahanRULES , thank you for figuring out how to fix the index.  This makes the docs a lot better.  I have gone ahead and merged your updates into the main and that automatically built the new version of the docs.  I will checkout that preview as well.

Item moved to In Review while I review the preview item.  Thanks for that as well.


Updated definition of a stack - grammar issue

'Stacks is a First In Last Out' changed to 'A stack is a First In Last Out'.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

LGTM  and congratulations on your first PR 😄🎉
Thanks for contributing !


passing state to start

I'm seemingly-unable to supply some state to the Fresh server.
Might it be useful to make it possible to add an additional prop to start's opts, which would then be accessible on route ctx? Here is my use case:

In a given main.ts, what if (A) there is some state initialized before starting the server and (B) routes need to make use of that state? For instance:
const someOtherServer = new SomeOtherServer()
someOtherServer.listen()

await start(manifest)

In this case, someOtherServer might contain a port, about which the fresh routes need be aware in order to make requests of someOtherServer. Ideally we could do something like...
const someOtherServer = new SomeOtherServer()
someOtherServer.listen()

- await start(manifest)
+ await start(manifest, {
+   globalState: { port: someOtherServer.port },
+ })

Then, within a given route, we could access that port, and use it to connect to someOtherServer.
import { Handlers } from "$fresh/server.ts"

export const handler: Handlers = {
  GET(_req, ctx) {
    const conn = Deno.connect({
      port: ctx.globalState.port,
      ...
    })
  },
}

I'd imagine there are limitations to what kind of data one could supply as globalState...
Would this feature be desirable? Is there another approach to solving the same problem? (I'm quite new to Fresh... wouldn't surprise me if there's another approach about which I'm unaware). Thank you!

Why not just use a module?
// serverConnection.ts
const someOtherServer = new SomeOtherServer()
someOtherServer.listen()

export const port = 1111

// routes/example.ts
import { Handlers } from "$fresh/server.ts"
import { port } from "../serverConnection.ts"

export const handler: Handlers = {
  GET(_req, ctx) {
    const conn = Deno.connect({
      port: port,
      ...
    })
  },
}

Perhaps the port state was a bad example. A better example might be a cache.
main.ts
// ...

await start(manifest, {
  globalState: { cache },
})

routes/example.ts
import { Handlers } from "$fresh/server.ts"
import { port } from "../serverConnection.ts"

export const handler: Handlers = {
  GET(_req, ctx) {
    const possiblyCachedValue = ctx.globalState.cache.getOrInit(someKey, someInitFn)
    // ...
  },
}

Different requests could make use of the same cache instance, supplied to serve.

Maybe I'm not understanding something, but couldn't this also just be a module?
// cache.ts
class Cache {
    // ...
}

export const cache = new Cache()

// routes/example.ts
import { Handlers } from "$fresh/server.ts"
import { port } from "../serverConnection.ts"
import { cache } from "../cache.ts"

export const handler: Handlers = {
  GET(_req, ctx) {
    const possiblyCachedValue = cache.getOrInit(someKey, someInitFn)
    // ...
  },
}

Good point! I suppose I was thinking about state as if working with an Oak server (where one must specify how state is passed between middleware)...
But yes... when one wants handlers to simply share the same piece of data, a singleton would do! Thank you for bearing with me.

Let's say we want to package a Fresh website as a library with some configuration that needs to be accessed by handlers. Ideally, we could expose the site by wrapping the start call.
export interface Props extends StartOptions {
  somePropUsedByHandlers?: string
}

export function theSite(props: Props = {}) {
 return start(manifest, props)
}

How would one access the somePropUsedByHandlers prop from within handlers?

Sidenote: perhaps this merits a new issue?: "packaging Fresh site as configurable library"

To me that seems like an uncommon use case to distribute a website as a library. I'm happy to be proven wrong though if enough people are interested in this feature.
A valid option to pass data around when starting multiple instances of fresh are environment variables. For as long as each fresh instance has their own process, you can pass different env variables to them.


Removing deprecated Code From XSS-Functions

In AEM 6.2 use of com.adobe.granite.xss.XSSAPI is deprecated and org.apache.sling.xss.XSSAPI should be used instead.
However https://adobe-consulting-services.github.io/acs-aem-commons/features/xss-functions.html is still using deprecated API even in verion 3.6.x.
Deprecated code needs to be updated in Taglib for XSSFunctions.
Here:
https://github.com/Adobe-Consulting-Services/acs-aem-commons/blob/acs-aem-commons-3.6.2/bundle/src/main/java/com/adobe/acs/commons/xss/XSSFunctions.java

Fixed via e05b7e4


fix: mod descriptions when perk info available

What
fix: mod descriptions when perk info available

:tada: This PR is included in version 1.8.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


feat(input): input type range

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] If applicable, have a visual design approval

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] clarity.design website / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: #112
What is the new behavior?
The input type="range" was implemented.
The dev page for input type="range" was implemented.
The gemini tests were implemented.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

@florin-borceanu can you squash your commits?

Closing in favor of #4137.


[Bug]: snowflake_grant_account_role fails on apply due to Root resource was present, but now absent.

Terraform CLI Version
1.5.3
Terraform Provider Version
0.91.0
Terraform Configuration
resource "snowflake_grant_account_role" "service_bot_id_roles" {
  provider  = snowflake.id
  for_each  = toset(local.id_snowflake_bot_roles)
  role_name = each.value
  user_name = module.service_bot_id_name[0].user_name
}

### This for_each pattern works in other places but when packaged in module fails.

Category
category:grants
Object type(s)
resource:grant_account_role
Expected Behavior
I would expect that the resource would be created for each of the existing roles passed in the for_each statement to the explicitly provided user_name.
Actual Behavior
The plan looks entirely normal but upon apply the following error occurs.
produced an unexpected new value: Root resource was
│ present, but now absent.
Here's a debug file with security keys and tokens removed.
apply_failure.txt
Steps to Reproduce
This may be a result of leveraging an in repo module which implements this.  However I have found that this unique case I am seeing a plan that looks exactly what I want:
module.service.snowflake_grant_account_role.service_bot_id_roles["PUBLIC"] will be created

resource "snowflake_grant_account_role" "service_bot_id_roles" {

id        = (known after apply)
role_name = "PUBLIC"
user_name = "SERVICE_DEIDDEV_BOT"
}

But upon apply the apply fails.
How much impact is this issue causing?
Medium
Logs
No response
Additional Information
I am unable to understand the
Would you like to implement a fix?

[ ] Yeah, I'll take it 😎

Hey @kcarrillorhoads 👋
It's happening, because the PUBLIC role is a "pseudo-role" (more on that here) that is already assigned to every role and user, so there's no need to grant it through terraform. If you're having an issue with removing the resources please use terraform state rm <id> to remove the resource from the state (and you can safely remove the configuration). Also, for a test, I granted the PUBLIC role to myself in the worksheet and got this message: "Granting role PUBLIC has no effect.  Every user and role has role PUBLIC implicitly granted.".

Closing due to long inactivity (+ not really a bug, but rather general misuse of Snowflake/Provider)


Scale newtxtt

Altough the manual of newtxtt reads that there is no scaling necessary, the font reads too large:

When using the patch, it renders as follows:

This was my MWE:
\documentclass[english]{lni}

\usepackage{mwe}

\lstset{
  language=SQL,
  basicstyle=\normalsize\ttfamily,
  showstringspaces=false
}

\begin{document}
\title{title}

\blindtext{}
And some more text with an SQL statement \lstinline!select * from MyTable;! without any further meaning.
\blindtext{}

\begin{lstlisting}
select Name, Total
from Professors, (select sum(ECTS) as Total
                  from Courses
                  where PersId = Lecturer) C
\end{lstlisting}

\blindtext{}

\end{document}

Thank you.


Ignore projectile-project-root errors

(projectile-project-root) is not an autoloaded function, declaring it forward doesn't make a difference. Even if the entire package is required, when (projectile-project-root) can't a project root, it'll raise an error, which results in blowing away the entire mode line. This patch fixes it.

Sorry, I'd already got this fix on my machine with some other changes when you opened the PR.
Thanks though 🙂


Name change from Accounts Receivable to Sales Ledger

Hi,
When looking for Accounts Receivable, as stated in the online doc. I couldn't find the Accounts Receivable-field.
I did find the Sales Ledger-field, under which the >Invoices>All Free Text invoices 10.0.14
Kind regards,

:white_check_mark: Organization membership check passed
Requester is not a member of the MicrosoftDocs organization.


Add cagliostrolab/animagine-xl-3.1 to list of known XL models

Added support of cagliostrolab/animagine-xl-3.1
Tested with following command:
python text_to_image_generation.py
--model_name_or_path cagliostrolab/animagine-xl-3.1
--prompts "1girl, souryuu asuka langley, neon genesis evangelion, solo, upper body, v, smile, looking at viewer, outdoors, night"
--negative_prompts "nsfw, lowres, (bad), text, error, fewer, extra, missing, worst quality, jpeg artifacts, low quality, watermark, unfinished, displeasing, oldest, early, chromatic aberration, signature, extra digits, artistic error, username, scan, [abstract]"
--num_inference_steps 50
--guidance_scale 7
--num_images_per_prompt 50
--batch_size 5
--height 1216
--width 832
--image_save_dir /var/work/optimum-habana-clean/examples/stable-diffusion/stable_diffusion_images_animagine
--scheduler euler_discrete
--use_habana
--use_hpu_graphs
--gaudi_config Habana/stable-diffusion-2
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

LGTM.

@regisss can you please provide your review here?

Not needed right now


Documentation for ScrolledWindow::new misleading

"Usually you want to pass None for the adjustments, which will cause the scrolled window to create them for you."
Pass 'None' for the adjustments leads to
error[E0283]: type annotations required: cannot resolve _: gtk::IsA<gtk::Adjustment>
--> src/main.rs:159:42
|
159 |     let scrolled_window:ScrolledWindow = ScrolledWindow::new(None, None);
|                                          ^^^^^^^^^^^^^^^^^^^
|
= note: required by gtk::ScrolledWindow::new
You have gtk::NONE_ADJUSTMENT defined. Why not mention that in the documentation instead of 'None' and avoid leading users down this incorrect path?

You can also pass None::<&gtk::Adjustment> FWIW. These problems exist because functions can't declare default types for generic parameters yet.


Auto update samples

Description
Please include a summary of the change and/or which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Target Release
Please specify which release this PR should align with. e.g., 1.0, 1.1, 1.1 Preview 1.
Checklist

[ ] Samples build and run using the Visual Studio versions listed in the Windows development docs.
[ ] Samples build and run on all supported platforms (x64, x86, ARM64) and configurations (Debug, Release).
[ ] Samples set the minimum supported OS version to Windows 10 version 1809.
[ ] Samples build clean with no warnings or errors.
[ ] [For new samples]: Samples have completed the sample guidelines checklist and follow standardization/naming guidelines.
[ ] If I am onboarding a new feature, then I must have correctly setup a new CI pipeline for my feature with the correct triggers and path filters laid out in the "Onboarding Samples CI Pipeline for new feature" section in samples-guidelines.md.
[ ] I have commented on my PR /azp run SamplesCI-<FeatureName> to have the CI build run on my branch for each of my FeatureName my PR is modifying. This must be done on the latest commit on the PR before merging to ensure the build is up to date and accurate. Warning: the PR will not block automatically if this is not run due to '/azp run' limitation on triggering more than 10 pipelines.

/azp run SamplesCI-All.yml

/azp run SamplesCI-All

You should add to https://github.com/microsoft/WindowsAppSDK-Samples/blob/main/docs/samples-guidelines.md#checklist
that if you have a new feature area, you should add it to the featureareas parameters in the SamplesCI-All pipeline .


Climate: Ocean currents

Might work similar to global trade winds to a degree where the major streams point the same direction. HOWEVER: There are counter-streams at the borders of the streams.
See: https://en.wikipedia.org/wiki/Ocean_current
There is some interesting work by for-h-in-hexes on that topic:
https://forhinhexes.blogspot.com/search/label/Currents
In theory, we can initialize a vector field similar to the global wind vectors, then walk the currents until we hit land, then walk our way back when the ocean currents are deflected by landmasses. Then we interpolate the vector field using the inverse distance weighted something-something.
https://en.wikipedia.org/wiki/Inverse_distance_weighting

I have something hacky that

builds a voronoi diagram using points only in the ocean
throws away paths that cross land
divides the planet into Northern/Southern hemispheres
divides each hemisphere into vertical slices
for each slice finds a path (if possible) between the pole & equator
alternates between making the path 'hot' (carrying warm water toward the pole) or 'cold' (carrying cold water towards the equator)

Just a thought.

That's a pretty smart idea... kinda like a mix of pathfinding and how water flux works when you find the downhill neighbor for each point to chart the flow of rivers?
If you alternate the direction and flip the sort order, the currents would start circulating... Clever idea. Would be worth a look :) It might be also a lot faster than the vector based approach. My usual way of doing it would be to implement both approaches in parallel using two different properties in the structs and then just comparing which one works best. I think your suggestion, if there is not something obvious that we're missing, should be the solution causing the least issues, since there is way less math involved where things could go wrong :)

I tried a few approaches and this was the least awful I found that was fairly simple to compute :smile:  I'm sure there are cleverer ways but eh :man_shrugging:
The main issues;

Reusing paths. I don't think (??) it makes sense for currents to flow over each other .. although in the case of cold water (should be deeper?) and hot water (shallower?) it actually might be possible .. my high school physics says it makes sense, but then I was never that good at physics
The inability to form a current because the land just happened to cross an entire slice .. technically this isn't an issue per se, since if there's no ocean then there's no ocean currents, but it can make for unequal currents in the rest of the oceans -- although if we modelled current strength we could simply infer that the one (say) cold current flowing downward carried more water than the two carrying hot water flowing upward.

Yeah, one way or another, the currents in the top layer are the most important anyway. :) So I think it's safe to disregard the third dimension. I was also able to get away with the global winds to only flow in two dimension :D
Well, in theory you can sort all regions from north to south and south to north and use a queue to work off all ocean regions, so if you don't reach all ocean regions in the first pass, you just repeat the process for all regions (points) that remain unvisited after each pass or so until you run out of tries or the queue is empty :) If I understand it correctly... I'd have to do a sketch on a napkin to be sure


Ingress v1.1.3 is calling waf-regional/GetWebACLForResource even though waf annotations are not specified

Controller version: 1.1.3 (not observed in v1.1.2)
I am running Ingress controller inside a private subnet. When  create a new ingress resource with following annotations:
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  name: alb-ingress
  annotations:
    kubernetes.io/ingress.class: alb
    alb.ingress.kubernetes.io/scheme: internal
    alb.ingress.kubernetes.io/target-type: ip
    alb.ingress.kubernetes.io/subnets: subnet-11111,subnet-2222

in the ingress controller logs I can see following sequence:
ec2metadata/GetMetadata
ec2/DescribeSecurityGroups
elasticloadbalancing/DescribeLoadBalancers
elasticloadbalancing/CreateLoadBalancer
elasticloadbalancing/DescribeLoadBalancerAttributes

waf-regional/GetWebACLForResource

My question is:
Is it possible to not call <service/Action> unless an annotation is specified ?
Also note this is not the case in controller v1.1.2.
Thanks

Hi, there is no behavior change around this between v1.1.2 and v1.1.3.
You add an additional arg to controller "--feature-gates=waf=false" to disable waf support.
The reason to query for WAF even if there is no annotation specified is the controller need to clean-up WAF on alb if it's already there.

Thanks for looking into this @M00nF1sh . Adding additional arg to controller helped.
Is this defaulted to 'false' for version 1.1.2 ? and where can I find a list of all arguments (and their impact)

@fmmasood it's default to true in 1.1.2 too. So i supposed u have that flag in the old controller.
This flag is missing from the docs, but will be supported in future versions too. (need to find some time to update the docs)


Enable colorized gcc errors output

Colored error messages are easier to parse.

 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.


Get or create vector version of LCP lab logo

Check if we have a vector version of the LCP logo. If not, create one.

I made it, I have a vector version.  Will dig it up for you tomorrow.
K
On 9/1/15 10:15 PM, Tom Pollard wrote:

Check if we have a vector version of the LCP logo. If not, create one.

Reply to this email directly or view it on GitHub:
https://github.com/MIT-LCP/mimic-website/issues/3

Great, thanks Ken.


Need to apply JavaPlugin, not just assert it

I think:
plugins {
  id 'a'
  id 'b'
}

does not guarantee application of a before b.
In a project where I tried to apply j2objc, I did the usual README incantation of java first then j2objcgradle.  However our assertion failed because j2objcgradle was applied before java.
We should just Apply the JavaPlugin ourselves; that is how other plugins (like GroovyPlugin) work.

https://github.com/gradle/gradle/blob/master/subprojects/plugins/src/main/groovy/org/gradle/api/plugins/GroovyPlugin.java#L35

fixed by #453


Vite >=3.1.5 breaks Jotai babel plugins

Describe the bug
Vite 3.1.5 changed the processing of package exports. This breaks two Jotai plugins (debug-label and react-refresh).
vite dev chokes with the following error:

Reverting to 3.1.4 makes the error go away.
The author of Jotai thinks this may be a vite problem, not a problem with Jotai itself.
See discussion with initial analysis here: https://github.com/pmndrs/jotai/issues/1475
Reproduction
https://github.com/gunters63/jotei-vite-import-error
Steps to reproduce
pnpm install, then pnpm dev
System Info
System:
    OS: Windows 10 10.0.22621
    CPU: (16) x64 AMD Ryzen 9 6900HS with Radeon Graphics
    Memory: 6.27 GB / 31.26 GB
  Binaries:
    Node: 18.10.0 - C:\Program Files\nodejs\node.EXE
    npm: 8.19.2 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.22621.608.0), Chromium (106.0.1370.37)
    Internet Explorer: 11.0.22621.1
  npmPackages:
    vite: 3.1.7 => 3.1.4

Used Package Manager
pnpm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

@antfu about this, isn't there a problem with the versioning on npm ?
The CHANGELOG.md mentions last stable to be 3.1.3. But on npm there are 3.1.4 to 3.1.7

It appears this is still an issue, as #10528 and #10683 only affect config bundling.

I think the plan is to allow users to omit "module" from the resolve.conditions option in order to prevent it from being used (assuming I fully understand the intent of #10683).
In my opinion, we shouldn't use the module condition by default, and users can add it to resolve.conditions if they need it.

I think the plan is to allow users to omit "module" from the resolve.conditions option in order to prevent it from being used (assuming I fully understand the intent of #10683).
In my opinion, Vite shouldn't use the module condition by default, and users can add it to resolve.conditions if they depend on a misconfigured package.

I agree with you.

@aleclarson Isn't this issue about config bundling only? module being used in the normal Vite pipeline isn't part of this issue.

I think the plan is to allow users to omit "module" from the resolve.conditions option in order to prevent it from being used (assuming I fully understand the intent of #10683).

I don't think that's the intent of the PR. It's fixing an issue with config bundling only.

I think the plan is to allow users to omit "module" from the resolve.conditions option in order to prevent it from being used (assuming I fully understand the intent of https://github.com/vitejs/vite/pull/10683).

I implemented in a way that we could do that. But that is not a main intent of that PR. The intent was to make module not be used when config bundling.
I think removing module condition from the normal Vite pipeline is a different thing from this issue. (IMO Vite should use module condition for isRequire: false because of the intent of this condition: https://github.com/webpack/webpack/issues/11014, https://github.com/rollup/rollup/issues/3514, https://github.com/rollup/plugins/tree/master/packages/node-resolve#exportconditions)
About chaging the resolve.conditions, I've left explanations about it in #10917.

Isn't this issue about config bundling only?

Ah ok, my mistake!

I think the plan is to allow users to omit "module" from the resolve.conditions option in order to prevent it from being used (assuming I fully understand the intent of #10683).

I implemented in a way that we could do like that in future. But that is not the main intent of that PR. The intent was to make module not be used when config bundling.

 I wasn't saying that it was the only or even the main intent of #10683, but I noticed it was implied in the overrideConditions description.


OneDrive File Picker throwing: The icon "custom" was used but not registered

Project: OneDrive File Picker v7.2 (local & CDN)
Bug: I'm seeing the below error when testing the OD File Picker v7.2 via compiled source and published CDN. Seems like a change in Fabric may be causing a break.
Repro:

Create a sample application following https://docs.microsoft.com/en-us/onedrive/developer/controls/file-pickers/js-v72/open-file
Open the OneDrive File Picker
Select a file
Click "Open"
The warning & stacktrace are printed to console, the "Open" button is disabled, but the popup doesn't close.

May need to register icons if this is still happening -- https://github.com/OfficeDev/office-ui-fabric-react/wiki/Using-icons

This is an issue with the hosted page, not the SDK itself. Closing as author of the issue.


Metrics Operators Index

Made changes as per the Open Source Docs Migration Plan.
Purpose of this pull request
This pull request (PR) to edit the Metrics Operators Index under Metrics.
Issue number:

Select the type of change:

[x] Minor Changes - Typos, formatting, slight revisions
[ ] Update Content - Revisions and updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - Updates, maintenance, and new packages for the site, Gatsby, React, etc

@kimsauce
Seems like a Docusaurus issue. I'm troubleshooting it. Will ask Jagadish or Megha for help.


feat: add subscribe RegisterOracle Tx and execute ApproveOracleRegistration Tx

Closes #3
Closes #4
Actually, I missed that the two issues were separated. Sorry for about that.

Add a subscribe RegisterOracle Tx and add a register-oracle cmd in register_oracle.go.
Add a EventHandler in register_oracle_event.go so that executing ApproveOracleRegistration Tx.

Not related this PR, I think we subscribe tx to ApproveOracleRegistration


Added logout button

Issue that this pull request solves
Closes: #135
Proposed changes
Added a logout button in the navbar, which removes the token located in the local storage.
The button also redirects the user to the home page.
Types of changes
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Put an x in the boxes that apply

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] My changes does not break the current system and it passes all the current test cases.

Screenshots

Looks good to me.
Thanks for your contribution @matheusmosca :smiley:


Can we receive IR signal. I want to build 2 way control for my AC

I see that we can control our IR device well. But if we can find solution to receive ir signal i think we can sync remote state with hass
So can we do it, and how to do it
Thank so much

I'm not following, but assume are you requesting an enhancement for a particular IR device?

I'm not sure but it sounds to me like he wants to put the IR blaster next to the A/C unit so when you use the remote control to change the A/C the blaster will also pick up and report the signal.  The blasters really are not designed to receive continuously like that, but I don't see why you couldn't fake it by creating a loop that puts it back into 'learn' mode every time it receives something.  Running the learned signal through IRRemoteControlDevice.pulses_to_nec(...) or IRRemoteControlDevice.pulses_to_hex(...) should get you a usable code you can match against.


refactor: refine nothing type

Current dependencies on/for this PR:

main

PR #38   👈

This comment was auto-generated by Graphite.

Graphite Merge Job
Current status: 🔄 Waiting for checks to pass
This pull request is currently waiting for CI and mergeability checks to pass before being merged.
This comment was auto-generated by Graphite.

Job Reference: snArrRseLhXioPIul3WC


[cmake] Fix typo in SWIFT_INCLUDE_DIRS SwiftConfig definition

Correct the SwiftConfig definition of SWIFT_INCLUDE_DIRS to reference SWIFT_INCLUDE_DIRS (plural), not SWIFT_INCLUDE_DIR (singular). This will help the out of tree use case, for cmake builds that depend on libraries from Swift.
Note that the SwiftConfig definition of SWIFT_INCLUDE_DIR (singular) correctly references the plural SWIFT_INCLUDE_DIRS, as the comment indicates it should.
The plural SWIFT_INCLUDE_DIRS is defined in cmake/modules/CMakeLists.txt and includes both SWIFT_INCLUDE_DIR and SWIFT_MAIN_INCLUDE_DIR.
The singular SWIFT_INCLUDE_DIR is defined in the root CMakeLists.txt

CC: @Rostepher @gottesmm

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 374c2c82ce64d493eb78c81b7b5ad7c6895fada0

@swift-ci please test Linux platform

Build failure doesn't appear related.

Build failed
Swift Test Linux Platform
Git Sha - 374c2c82ce64d493eb78c81b7b5ad7c6895fada0

@swift-ci please test Linux platform

Thanks!


Can't create a new git branch

VSCode Version: 1.31.1 (1.31.1)
OS Version: MacOS Mojave 10.14.3 (18D109)

Please provide the concrete steps to reproduce the problem. Thanks!

/needsMoreInfo

I think this could be because the behaviour of the "create new branch" option has changed recently.
After entering a branch name and pressing enter the user is now presented with a second list to select select the ref in which to base the branch from. On quick glance it's easy to see the confusion.

Took me quite a while and several branches, created on command line to find the solution to my confusion here.
As a quick help for users, I would suggest, to change the first entry "HEAD" to something which better indicates the second use of this menue like "DERIVE FROM... HEAD

The quick open mentions "Select a ref to create a new branch from".
I'll close this and see whether more users hit this confusion. It might just be a small "getting used to it" bump.

@joaomoreno I believe the requirement to select the ref is an unnecessary step for the majority of users. I think VScode should by default support the most common path first.
I'm pretty sure I'm not alone in saying that I have NEVER found myself wanting to create a branch from anything else other than HEAD

I didn't even recognize that quickref...
I did not expect anything to stand there except stuff I already knew so I wasn't looking (User Expectancy)
We do several branches that depend on other branches, but we usually opened the correct branch and then created a new one. So, getting used to it.
Closing is OK, but there are at least two other issues with this theme, I think.
..and we shouldn't keep writing comments in a closed issue, so that's all from me.

@raldred @Markus-bt Addressed in https://github.com/Microsoft/vscode/issues/70812


[Feature] : Enhancement can be done in the contributor form-tag of [Create Empty Threat Model] Page

Describe what problem your feature request solves:
Currently the Added Contributor alignments are not so good in terms of UI. which we can see in below images
[Create Empty Threat Model] Page:

[Contributors form tag]:

Describe the solution you'd like:
I'd Like to have the added contributors in the [contributor form-tag] more clear and differentiable Like shown below:

Additional context:

provided by commit 60e18e1b7b57de343a2f0111f966442877a86e40


Create branches for multiple active gateway development

Request branches be created for development of the multiple active gateway enhancement. Multiple people will work on this enhancement, and we believe that having public branches will enable collaboration and improve visibility into the changes being proposed.
Proposed branch name: ""multi-active-gw"
Branches are needed in the following repos:

submariner
submariner-operator
shipyard

Request write permission for the branches for the following people:

@anfredette
@Billy99
@astoycos
@nerdalert

@skitt @Billy99 @astoycos @nerdalert -- FYI

I’ve created the branch on submariner-operator and redirected https://github.com/submariner-io/submariner-operator/pull/1951. https://github.com/submariner-io/submariner-operator/pull/1953 sets up CODEOWNERS so that you can handle merges on your own.
The branch protection rules for jobs which should pass aren’t set up yet.

Whats the plan on reviewing work being done on that branch? Is it following the code review guidelines or is it something else?
How about merging back to main branch, is there going to be one single big pull request?

Thanks, @skitt.  Good questions @mkolesnik.
We've reviewed the general approach with @sridhargaddam and @aswinsuryan.  The initial goal is to do a POC to prove things out and work out the details. After which, we'll do a more detailed review with interested parties. As mentioned in the request, we wanted these branches to facilitate collaboration and to do this initial work in public so interested people can look at it as they have time.  We intend to submit changes in reviewable chunks and follow the review guidelines, but we want to be able to do this initial POC work more quickly than would happen if we required reviews by the current code owners. When it comes time to merge back to main, we'll have to decide as a team whether the existing set of commits can be merged as a whole, or if we need to restructure the changes into more logical chunks for reviewing and merging into main.  I'm expecting it will be the latter.
Sound okay?

Sounds good, I think we need to set clear expectations here eg:

Reviews from all Submariner members are encouraged but not required for merging on the feature branch
Work from feature branches should be often merged back to main branch when it's stable and ready
If work from a feature branch is too big for a single PR (I think our last discussed limit was 500 LOC changed, but we can discuss and define a clear one) then it should be broken down into smaller PRs

Also I guess feature branches will have different (or no) branch protection rules?

Also I guess feature branches will have different (or no) branch protection rules?

For now I’m replicating the devel branch protection rules, to ensure that we don’t miss anything which would result in significant effort to merge the resulting PRs when ready — whether it’s “basic” stuff like linting rules, or ensuring that E2E still passes.

Thanks for the help here @skitt Do you by chance have PR's up to do this for https://github.com/submariner-io/submariner and  https://github.com/submariner-io/shipyard as well?

Thanks for the help here @skitt Do you by chance have PR's up to do this for https://github.com/submariner-io/submariner and https://github.com/submariner-io/shipyard as well?

The branch already exists on Shipyard; I’ll take care of the branch on Submariner and the two setup PRs today.

Thanks!

Shipyard: https://github.com/submariner-io/shipyard/pull/759
Submariner: https://github.com/submariner-io/submariner/pull/1743

This is all done, please open new issues if you run into problems. I’m aware we still need to describe how to update the feature branches with changes from devel.


micro-frontends All In One

micro-frontends All In One

https://zhuanlan.zhihu.com/p/78764100
https://www.thoughtworks.com/radar/techniques/micro-frontends
https://assets.thoughtworks.com/assets/technology-radar-vol-22-cn.pdf
http://rbrtsmith.com/2015/08/a-modern-frontend-workflow
https://martinfowler.com/articles/micro-frontends.html

https://micro-frontends.org/

https://blog.bitsrc.io/state-of-micro-frontends-9c0c604ed13a
https://itnext.io/11-micro-frontends-frameworks-you-should-know-b66913b9cd20
https://single-spa.js.org/
https://github.com/umijs/qiankun

https://webpack.docschina.org/concepts/module-federation/


Allow Import and Export Routing Policy at the AS Level

Environment

Peering Manager version: master

Proposed Functionality
When peering with an organization that has several peering IP addresses, it would be convenient to be able to have policy scoped at the AS level.
Use Case
I'm thinking that import policy like "tag routes learned from AS1234 with 65000:1234" would be best configured at the AS level, not at the peer level

+1 however a peer level policy for session <IP_ADDRESS>/AS1234 should probably override an AS level policy that would already have applied a policy for session <IP_ADDRESS> / AS1234.

+1 to both of you :)

The policy weight is not really defined by any logic in PM. I mean, it sure makes sense that IX Policy < AS Policy < Session Policy, but that logic is up to the user to be defined in the template.
That said it is interesting to have policies per AS to be defined. This kind of policies could also be used for direct peering sessions as well.

We are a juniper shop, so we have many import and export policies defined for each peer. Only a few of them are really different between AS peers. We have an import policy that tags all routes learned at the IX with a "we learned this route in $region" community, then we have AS-level filters like the import filter from irrpt, and we tag with a "we learned this route from $as_num" community. This is in addition to the boilerplate no-small-communities, no-bogons, etc kinds of filters.
I'm looking forward to seeing the IRR integration features once they come out. That's the real reason I'm interested in this tool. It's a lot easier to teach a tool like this to a new NOC engineer than to describe how to ssh into a machine, run irrpt_fetch and irrpt_pfxgen, copy files to the router, etc...


endless loading even when I reopen the game

endless loading even when I reopen the game
I tried to reinstall the game but it does not work

did it end up working for you? mine does the same right now

Do you still have this issue? If yes, give more details, like your machine.


added code for distinct ways to climb stair problem

Added a file named distinct_ways_to_climb_stairs.cpp containing

The question at the top of the file.
Code solving the problem statement

Related issue #124
Solved the question on interviewbit with all the test cases passing.
What types of changes does your code introduce? Put an x in all the boxes that apply:

[ ] Bug fix (change which fixes an issue with the algorithm)

[x] New Algorithm (non-breaking change which adds functionality)

[ ] Documentation

Checklist:
Go over all the following points, and put an x in all the boxes that apply.
If you're unsure about any of these, don't hesitate to ask. We're here to help!

[ ] My code follows the code style of this project.

[ ] My change requires a change to the documentation.

[x] I have updated the documentation accordingly.

[x] I have read the CONTRIBUTING document.

@tstreamDOTh added proper directory structure


Allow asynchronous passcode checking

[x] I have read this issue template and provided all possible information.
[x] I'm using CocoaPods and have run pod update before filing this issue.

Hardware / Software
Which version of the library were you using?  0.0.2
Which version of iOS are you running?  11.2.6
What model of iOS device were you testing on? iPhone 6
If using CocoaPods, which version of CocoaPods are you on?  1.3.1
Goals
What is the outcome result you want to achieve with this library?
A custom child-lock passcode to restrict certain settings such as in-app purchases from kids.
Expected Results
What did you expect to happen?
When I finish inputting the passcode my passcode checking code is called and the UI does not freeze
Actual Results
What happened instead?  (Please attach a screenshot/screen recording if possible)
Because the passcode checking is done in a blocking method on the main thread the UI pauses for about a second with the last dot of the passcode not filled, and then it suddenly animates away or performs the reset animation
In my opinion a good solution for this would be another delegate method, maybe passcodeViewController: performCodeCheck: that, if present, would immediately return out of didCompleteEnteringPasscode in the TOPasscodeViewController. There would have to be a method in TOPasscodeViewController that you could call to message a success or failure, such as didCheckPasscodeSuccess:, that would then call the same things as didCompleteEnteringPasscode: currently does on success/failure.
Edit: Currently I think you can sorta hack a solution by always succeeding and making didInputCorrectPasscode NOOP, then manually calling your success code or passcodeView.resetPasscode, but that method means that the passcodeSuccess variable is always going to be YES when dismissing the controller. I haven't tested this solution yet, but it may work.

That's a fantastic idea! I'll have to have a think about how to implement that one.
Some Apple APIs implement an asynchronous callback mechanism by providing a block to the delegate that needs to be called once the asynchronous logic has finished. That might be the best way here.

I am also looking for this.
My thought would be to have two Delegates, the existing TOPasscodeViewControllerDelegate and aTOPasscodeViewControllerAsyncDelegate. It would be an "Exclusive Or" on which one you can implement and the controller would call whichever is set.


updating azure k8s cluster configuration

Signed-off-by: barinek<EMAIL_ADDRESS>Fixes #148

Codecov Report

Merging #153 (a7a6030) into main (846b8cf) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #153   +/-   ##
=======================================
  Coverage   91.82%   91.82%
=======================================
  Files          38       38
  Lines        1542     1542
=======================================
  Hits         1416     1416
  Misses         85       85
  Partials       41       41

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 846b8cf...a7a6030. Read the comment docs.


mautrix-meta: update to 0.4.0

This update cannot be done by the nixpkgs bot, as
the structure of the project has been changed, so why wait.
The mautrix-meta project has been moved under "cmd/mautrix-meta" There is also "cmd/lscli", but since this package is mainly about mautrix-meta, I think we can stay with this specific cmd. If we wanted, we could switch to both of them by removing this subPackages attribute.
CC: @eyJhb

@NixOS/nixpkgs-merge-bot merge

Anyways, I'm running it on my server for the past 10min, and I didn't notice any issues. :)

I am trying to test this, but running into an issue with libolm being marked as insecure.
I added this to my system .nix configuration:
nixpkgs.config.permittedInsecurePackages = [ "olm-3.2.16" ];

But when I build, I still get the following message:
error: Package ‘olm-3.2.16’ in /nix/store/va8pfd91qy9zizlfgrnnsapwyg21fyaj-source/pkgs/development/libraries/olm/default.nix:26 is marked as insecure, refusing to evaluate.

Any tips?

I am trying to test this, but running into an issue with libolm being marked as insecure.
I added this to my system .nix configuration:
nixpkgs.config.permittedInsecurePackages = [ "olm-3.2.16" ];

But when I build, I still get the following message:
error: Package ‘olm-3.2.16’ in /nix/store/va8pfd91qy9zizlfgrnnsapwyg21fyaj-source/pkgs/development/libraries/olm/default.nix:26 is marked as insecure, refusing to evaluate.

Any tips?

How are you testing/building it? If you're doing nix-build/nix build/nix shell, etc. then you could do something like NIXPKGS_ALLOW_INSECURE=1, if however you've patched your nixpkgs and is building with this commit in, then it seems like what  you added is fine. But needs more information.

How are you testing/building it? If you're doing nix-build/nix build/nix shell, etc. then you could do something like NIXPKGS_ALLOW_INSECURE=1, if however you've patched your nixpkgs and is building with this commit in, then it seems like what you added is fine. But needs more information.

Thank you for the follow-up! I should have provided more context: I am running mautrix-meta as a system service on a flake-based system. Thus, I can't (or at least don't know how to) do a temporary 'nix shell' since I need it to run as a nixos service.
In flake.nix, I have:
inputs = {
    mautrixmeta.url = "github:Rutherther/nixpkgs/mautrix-meta/update";
    ...

outputs = { ..., mautrixmeta }:
    let
      overlays = final: prev: {
        inherit (mautrixmeta.legacyPackages.${prev.system}) mautrix-meta;
      };
      ...

And then in the system service configuration:
{ pkgs, nixpkgs, ... }:

let
  readYAML = pkgs.callPackage ./read-yaml.nix {};
in
{
  nixpkgs.config.permittedInsecurePackages = [ "olm-3.2.16" ];

  services.mautrix-meta.instances.messenger = {
    enable = true;
    ...

Then I run sudo nixos-rebuild build --flake .#$HOST and get the error message.

Closing in favor of #342988 :)

@presto8 sorry, I have no experience with flakes. Maybe ask in the Matrix help room :)

@presto8 legacyPackages import nixpkgs instance without permitted insecure packages. You need to make an instance yourself, and get mautrix-meta from that instance. You might use config = config.nixpkgs.config to get the config you configure with nixpkgs.config options. Moreover, I would suggest not using overlays for this, but instead to use services.mautrix-meta.package with the package

@eyJhb Thanks for trying to help anyway, I appreciate it!
@Rutherther Thanks for the tips. I still have much to learn about flakes. I will do some offline study and see if I can figure it out!


Replace features with cfg and automatic version detection

The user is no longer required to set the target library version, the builds script detects it from pkg-config output and sets the appropriate cfg directives.

cc @EPashkin

Thanks !


low heap mode

There are many environments where heap usage comes at a significant premium (for me, a Solana program which has only 32KB of heap), and I think it would be good to reduce heap usage. It might also help reduce cache footprint for large-ish circuits.
There's a lot of instances in this library in which unnecessary Vecs are used where slices would suffice. Unfortunately, this might requires breaking interface changes.
To avoid breaking existing stuff, we can have a low_heap feature with alternative types and function signatures (e.g. returning iterators, taking mutable references to slices). Another option is to duplicate code and have a separate lite_verifier crate/module that re-implements just the stuff needed by the verifier.
Examples:

to_fri_openings can return an iterator over the openings instead and save a pretty significant amount of heap memory
Challenger's input_buffer gets quite large when observing FRI openings, and we can set a maximum size absorb buffered inputs whenever it reaches that size
eval_unfiltered in Gate can take a ConstraintConsumer (or a new analogous thing) instead of allocating and returning a Vec every time.

I'm already working on this stuff because I need it, and I'm happy to open a PR. Just want to gague what approach is preferred and how OK we are with breaking interface changes here/there

Just to reiterate, I'd prefer if we can reduce memory use of the main verifier, to avoid splitting its code.

My understanding is the Solana verification effort has stopped for now, so this probably isn't needed


Minor StaticHTML reporter code improvements

See individual commits.

Once done, I plan to do some improvements to the StaticHtml report.

generally, I'd be interested for what features you're still using it, and whether it makes more sense to migrate any missing feature to the web app reporter instead of adding more feature to the static HTML reporter. Maybe a good topic for the community meeting to get a feeling for which reasons, if at all, people are still using the static report.


Incompatibility with BluetoothSerial library on LilyGo T-Display-S3

Attempting to use the BluetoothSerial library with the LilyGo T-Display-S3 board results in compilation errors. The same code compiles and runs as expected on a standard ESP32 board, indicating the issue is specific to the LilyGo T-Display-S3 board.
Here is the code:
#include "BluetoothSerial.h"
BluetoothSerial SerialBT; // Object for Bluetooth
void setup() {
Serial.begin(115200); // Start the built-in serial port, possibly to communicate with the host computer
//SerialBT.begin("ESP32test"); // Start BluetoothSerial with the name "ESP32test"
Serial.println("Bluetooth device is ready to pair");
}
void loop() {
if (SerialBT.available()) { // Check if we receive anything from Bluetooth
Serial.write(SerialBT.read()); // Send what we receive to the built-in serial port
}
if (Serial.available()) { // Check if anything is coming from the built-in serial port
SerialBT.write(Serial.read()); // Send what we receive to Bluetooth
}
}
Here is the error:
c:/users/talha/appdata/local/arduino15/packages/esp32/tools/xtensa-esp32s3-elf-gcc/esp-2021r2-patch5-8.4.0/bin/../lib/gcc/xtensa-esp32s3-elf/8.4.0/../../../../xtensa-esp32s3-elf/bin/ld.exe: libraries\BluetoothSerial\BluetoothSerial.cpp.o:(.literal._ZL8_stop_btv+0x18): undefined reference to esp_spp_disconnect' c:/users/talha/appdata/local/arduino15/packages/esp32/tools/xtensa-esp32s3-elf-gcc/esp-2021r2-patch5-8.4.0/bin/../lib/gcc/xtensa-esp32s3-elf/8.4.0/../../../../xtensa-esp32s3-elf/bin/ld.exe: libraries\BluetoothSerial\BluetoothSerial.cpp.o:(.literal._ZL8_stop_btv+0x1c): undefined reference to esp_spp_deinit'
c:/users/talha/appdata/local/arduino15/packages/esp32/tools/xtensa-esp32s3-elf-gcc/esp-2021r2-patch5-8.4.0/bin/../lib/gcc/xtensa-esp32s3-elf/8.4.0/../../../../xtensa-esp32s3-elf/bin/ld.exe: libraries\BluetoothSerial\BluetoothSerial.cpp.o: in function _stop_bt()': C:\Users\Talha\AppData\Local\Arduino15\packages\esp32\hardware\esp32\2.0.14\libraries\BluetoothSerial\src/BluetoothSerial.cpp:714: undefined reference to esp_spp_disconnect'
c:/users/talha/appdata/local/arduino15/packages/esp32/tools/xtensa-esp32s3-elf-gcc/esp-2021r2-patch5-8.4.0/bin/../lib/gcc/xtensa-esp32s3-elf/8.4.0/../../../../xtensa-esp32s3-elf/bin/ld.exe: C:\Users\Talha\AppData\Local\Arduino15\packages\esp32\hardware\esp32\2.0.14\libraries\BluetoothSerial\src/BluetoothSerial.cpp:715: undefined reference to `esp_spp_deinit'
collect2.exe: error: ld returned 1 exit status
exit status 1
Error compiling for board LilyGo T-Display-S3.

Perhaps you don't know that ESP32s3 doesn't support classic Bluetooth, only supports ble


🐞 - 4.0.0-rc.4 - TuiInputNumberModule Stepper doesn't work

Playground Link
No response
Description

example repo:
https://github.com/KhaledGabr/ssr-test/tree/main/src/test/number
UPDATE:
Removed unrelated console log error from the post.
Angular version
18
Taiga UI version
4.0.0-rc.4
Which browsers have you used?

[ ] Chrome
[ ] Firefox
[ ] Safari
[ ] Edge

Which operating systems have you used?

[ ] macOS
[ ] Windows
[ ] Linux
[ ] iOS
[ ] Android

I'm confused as to where this code comes from and why is it relevant to input number when it says .tui-image-editor-download-btn 🤔

@KhaledGabr You imported NG_EVENT_PLUGINS but didn't add them to app config. It should get added by migration script/ng add, probably an issue with RC not ready yet with all the scripts. How did you create this app, what scripts did you run?


Updating UCP license requires updating DTR license manually

@joaofnfernandes commented on Wed Nov 16 2016
A few users where surprised by the fact that on install DTR will get the license from UCP, but when you update the UCP license you need to manually upload the license to DTR.
Mention this in the docs to avoid surprising users.

https://docs.docker.com/datacenter/dtr/2.3/guides/admin/configure/license-your-installation/


doc: add rebar3_gpb_plugin to rebar3 doc

Not sure if community plugins should be documented in rebar3 doc, however this one offers the same functionality that can be had today in rebar2, could be useful to people migrating from rebar2

This is covered here https://github.com/rebar/rebar3/issues/962

yes, but in rebar2 there are two protobuf providers, erlang_protobuffs and gpb, #962 offers the basho one, in rebar2 you can choose between the two via configuration, i think it makes sense to offer the same in rebar3

Oh, woops, my bad. OK, agreed.

Cool, i'm available to write the doc, how do i go about it?

Same question as the other one though -- do we have a test app to demo it with? I guess if you use it personally that's good enough, but it would be nice to ensure things still work in the future.
To write documentations, you'll need a readme.io account and to propose an edit on the plugins page of rebar3.org.
If you'd rather avoid that, put the content you'd like in here and I can make sure it's gonna make it on the page.

There's no test app that i know of in github, we use gpb (and rebar's facility for it) extensively at work.
I'll propose the documentation changes through readme.io

Documentation change suggestions have been submitted on readme.io

Merged.


Vickel: Blacklist vairt.com

Vickel requests the blacklist of the website vairt\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

vairt\.com has been seen in 5 true positives, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


npm has 9.8 as latest version

on npm it has 9.8 as the latest version, but this repo has it set as 9.10, could someone push it up to npm please? Thanks

If you submit a PR, we can publish it :+1:

someone committed it already, someone just needs to push up the changes to npm

Ah, thanks. Done.


How to get a correct href right away [router.afterEach]

Version
2.7.0
Reproduction link
https://jsfiddle.net/6rzLab69/1/
Steps to reproduce
click button
What is expected?
after first click,
window.location.href is https://fiddle.jshell.net/home?index=1;
What is actually happening?
after first click
but window.location.href is https://fiddle.jshell.net/home;

after first click
router.query == { index: 1 };
but window.location.href is https://fiddle.jshell.net/home;
setTimeout output is https://fiddle.jshell.net/home?index=1
Promise.resolve output is https://fiddle.jshell.net/home?index=1

Dup of #1197

meeting the same condition


I can't export coordinates of maps.map object

I have the Next code for exctract the coordinate from map
from pygbif import maps
x = maps.map(familiaKey = 4690, year = 2000, bin = "hex", country='CO',srs='EPSG:4326',
             hexPerTile = 500, style = "classic-noborder.poly",format = ".mvt")
x.response
x.path
x.img # None
import mapbox_vector_tile
result = mapbox_vector_tile.decode(x.response.content)

Next, I visualizate the data
result['occurrence']['features'][0]
# Output
{'geometry': {'coordinates': [[[2425, 2312],
    [2423, 2307],
    [2417, 2307],
    [2415, 2312],
    [2417, 2316],
    [2423, 2316],
    [2425, 2312]]],
  'type': 'Polygon'},
 'id': 0,
 'properties': {'total': 150},
 'type': 3}

My problem is in item for coordinate. Because I need in this format:
{'geometry': {'coordinates': [[[-75.3, 4.35],
[-73.2423, 4.2307],
[-75.2417, 5.2307],
[-75.2415, 5.2312],
.........
How can i make the trasfomation? I need the coordinate in this way because I will create a GeoDataFrame with Geopandas.
This is the link of my notebook in Google Colab.
https://colab.research.google.com/drive/1noZlE6fFq39Qnhml2mTJk5BI2ksYRITA

Thanks for your question. I don't know how to convert the data to lat/long values. We do this in the R package rgbif, through the protolite pkg here https://github.com/jeroen/protolite/blob/master/R/mvt.R - i'll ask some folks if they know how to do this in python

I do think https://github.com/mapbox/mercantile or https://github.com/tilezen/mapbox-vector-tile#decoding could be useful, but haven't tried this myself.

thanks @stijnvanhoey
he is already using mapbox_vector_tile.decode above - but it returns tile coordinates, what he wants is the lat/long coords.
mercantile might be useful here

I use one option to fix it, then is to apply the function
plt.hexbin(x,y,cmap=color_map,edgecolors='k',extent=(-81.735793,-66.870391,-4.250167,13.394803),                        gridsize=bins,mincnt=1),
where x and y are longitude and latitude, make this plot:

If you wanna see how I make this, I give you a notebook link for you:
https://colab.research.google.com/drive/16FPv6Ko3qFUqAyAaXO-aBHHJ57aDjy72
However, i see than my request is incomplete, because when I use: occurrences.count(taxonKey=7707728, country='CO')
This return value of 2105267 registers, but when I call a quest like this:
occurrences.search(phylumKey= 7707728, country='CO', limit=2e6)
only get 300 datapoints..... anyone knows how can I get all the occurrences for my query?

the answer for the pagination is that GBIF allows 300 records per request. in rgbif we handle that pagination internally for the user, but here we don't. python tends to be a more technical set of users, so didn't do pagination for the user here. there's probably a better way to do it, but here's one way:
from pygbif import occurrences
def gbif_search():
    limit = 300
    offset = 0
    endOfRecords = False
    out = []
    while not endOfRecords:
        x = occurrences.search(taxonKey = 9206251, country='MX',
            limit=limit, offset=offset)
        endOfRecords = x['endOfRecords']
        offset = sum([len(z['results']) for z in out]) + len(x["results"])
        out.append(x)
    return out

x = gbif_search()
sum([len(z['results']) for z in x])

I'll have a look at the notebook

I used your code and only change this part for my propose:
x = occurrences.search(taxonKey=7707728, country='CO',             limit=limit, offset=offset)
But I get the next error:
HTTPError: 400 Client Error: Bad Request for url: https://api.gbif.org/v1/occurrence/search?taxonKey=7707728&country=CO&limit=300&offset=199800
This error is the same when I try to get all occurrences with a for loop.

Sorry, didn't tell you the other information, that 200,000 is the max for offset+limit, see  https://www.gbif.org/developer/occurrence  - You can separate your requests into smaller bins with .search, or you can switch to the .download method - it's different syntax from .search, but is more flexible, and has the advantage that there's no limit on records

@juanpac96 so it looks like you figured out the mapping issue - given your comment in https://github.com/sckott/pygbif/issues/58#issuecomment-566341827 ?

Yes, I finally I did it. with download requests, I get it and found the solution to my issue.

Great, nice work


Type revamp

type method has been revamped

Dears , @MohabMohie , @MahmoudElSharkawy , @magdyheiba , @MhmdElGazzar this message for reference , it's the sudo code
https://shaft-engine.slack.com/archives/C04SAA8BWCT/p1700142762303159?thread_ts=1700127523.414399&cid=C04SAA8BWCT

Great PR ya @zanoon2020. I added some small comments. Last this I would suggest is to remove the wording 'new' to the methods name as this will be the current type methods, just to minimize any confusion in the future.

dear @MahmoudElSharkawy the word new is temp as i intend to remove it once i launch the beta version , as i suggest to go with beta version first after this PR

@MahmoudElSharkawy are we ready to merge this PR yet? Feel free to raise your findings please.

@MahmoudElSharkawy are we ready to merge this PR yet? Feel free to raise your findings please.

I'm good :D

@MohabMohie @MahmoudElSharkawy is it ok to merge 😊do i need to do any kind of update for the branch before merging

@zanoon2020 you can check the pipeline reports, if they are passing then we can merge and have a release 😃🙏


Added Core 0.14.2 and 0.14.2-uasfsegwit0.3

My first contribution, hope this works

not working... ill review


Client loses connection after several scene loads in Unity.

I am working on a game in Unity that has a single hub scene which gets loaded after every mini game. The first time through everything works fine, but once the mini game is complete and I load back into the hub scene I can no longer communicate with the server.
My ParseInitializer has DoNoDestroyOnLoad and I only ever have one in the game. Im at a loss as to what to do here.

Can you please provide additional details as to the issue you're observing. Ideally this would be logs so we can help you pin things down. Closing this out in the meantime as this is nearly a year old... If you are still having an issue let us know and we'll open this back up.


[SPARK-28093][SQL][FOLLOW-UP] Update trim function behavior changes to migration guide

What changes were proposed in this pull request?
We changed our non-standard syntax for trim function  in #24902 from TRIM(trimStr, str) to TRIM(str, trimStr) to be compatible with other databases. This pr update the migration guide.
I checked various databases(PostgreSQL, Teradata, Vertica, Oracle, DB2, SQL Server 2019, MySQL, Hive, Presto) and it seems that only PostgreSQL and Presto support this non-standard syntax.
PostgreSQL:
postgres=#  select substr(version(), 0, 16), trim('yxTomxx', 'x');
     substr      | btrim
-----------------+-------
 PostgreSQL 11.3 | yxTom
(1 row)

Presto:
presto> select trim('yxTomxx', 'x');
 _col0
-------
 yxTom
(1 row)

How was this patch tested?
manual tests

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/testing-k8s-prb-make-spark-distribution-unified/12044/
Test PASSed.

Test build #106827 has started for PR 24948 at commit ae2c16e.

Test build #106827 has finished for PR 24948 at commit ae2c16e.

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

Yes. We need a flag

I'm hesitant about a flag; it's a bug fix right? it was always supposed to work as it does now. And happens for Spark 3 only. It's an important behavior change to note, to be sure. I'm not super against it but I'd also consider this something users have to consider and test when updating to Spark 3.

Based on my experiences, end users could call some library that depends on the old behavior and they are unable to change the library directly. Thus, adding a flag is always safer and nicer to end users.

Retest this please.

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/testing-k8s-prb-make-spark-distribution-unified/12419/
Test PASSed.

Test build #107270 has started for PR 24948 at commit ae2c16e.

Test build #107270 has finished for PR 24948 at commit ae2c16e.

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

Since this contains a change on stringExpressions.scala together, I'll merge this.
Now, as @wangyum commented in the PR description, this syntax becomes consistent among Spark/PostgreSQL/Presto. Only old Spark 2.x has that behavior.
If there is Spark 3.0.0 really need to have that old behavior for backward compatibility, please make another PR for that legacy flag, @MaxGekk and @gatorsmile . That PR can be reviewed as a separate item.

Merged to master.


Question: How to read properly in typescript the _imageRegistry property ?

Hi. I'm using your great module to print PDF on node.js. I'm using typescript and in my code, I'm looking sometimes at the _imageRegistry property of the document (PDFDocument). But there is no function or getter defined in the pdfkit typescript types to have access to this property ( I need sometimes to know the calculated height of an image added by the doc.image function)
Is there a way to have access properly in typescript to this property (_imageRegistry) ?
Thank you in advance for your answer.

If you want the height of an image you can do instead:
var img = doc.openImage(url);
doc.image(img, x, y, options);
var height = img.height

You will have access to the same properties of the image as in the _imageRegistry: it will be the height of the original image file and not the height of the image in the PDF document


Stack Smashing protect Failure - getCharacteristics();

Hello everybody,
I'm actually using Neil Kolban's library ESP32_BLE_Arduino, but i have a little problem...
In my program i would to :

Scan BLE servers(done)
Connect to a remote BLE Server (done)
Once connected, find all services available on the server (done)
Find all characteristisc of all services (PROBLEM)
Read and write characteristics values (not done yet)

Once i'm connected to the server and recovered service map i try to recover all the characteristics map with the BLERemoteService::getCharacteristics(); but i have a STACK SMASHING PROTECT FAILURE
Someone has an idea why this function works with some of my BLE server services but not with all ?
There are some pics to make it more understandable...
Picture 1 : code extract in the part i'm having trouble
(this part of code is just after i get successfully connected on the server)

Picture 2 : Serial Monitor with "stack smashing protect failure"

Picture 3 : My BLE server when i'm connected on it with nRF Connect

I would appreciate any hints or helps :)
Have a nice day,
Brice ANNETTE

Stack smashing error is when your task stack is too small. You have 2 options:

increase current task stack (im guessing it is looprTask),
create new task and use it for BLE

Hello chegewara,
Thank you for your quick answer.

increase current task stack (im guessing it is looprTask),

How can i increase the task stack ? Have it to be done with ESP-IDF?
Have a nice day,
Best regards

You can call ‘Serial.println(uxTaskGetStackHighWaterMark(NULL)) ‘ in your function to check how much stack space you have left. If it reports below 500 or so you’ll either have to put your function in a task with ‘xTaskCreate()’ and set it’s stack size higher, or find the main.cpp file and increase the task stack size there.

Hello h2zero,
Thank you very much for your answer, i tried what you said with the uxTaskGetStackHighWaterMark(NULL); function but my stack size is higher than 500.
As you can see on this picture :

I tried several times and it's always much higher than 500 (it's around 6000);
Do you have other ideas ?
I'll try to use xTaskCreate() to be sure and i'll report you soon.
Have a nice day.

So I tried what you said : I created a new task for the function that I had trouble with and it still deosn't work and I have to admit i'm a bit distraught.
this is the code that I used to create my new task : xTaskCreate(getServicesAndCharacteristics, "MyFunction", 100000,NULL,1,NULL); where getServicesAndCharacteristics is my function.
I tried with several others stacks depth values and it still doesn't work.
Have you other ideas of where it should come from ?
Thank you in advance.
 The main task stack is 8192 so it’s big enough for most things.
I’ll try to do some testing as I haven’t used the function you are before.

Just tested your code snippet and copied the service uuid you posted as created 12 other services and it seems to work fine. Not sure what else is happening in your code that could cause the issue, but the library seems to be fine.

Ok thank you for this information h2zero.
I'm asking myself if there are some kind of data protection in the BLE protocol ?
Because the device i'm interacting with (the server) is a Tyre Pressure Moniroting System TPMS for short. A device that is inside the tyre of recent cars.
The four services Generic Access, Generic Attribute, Battery Service and Device Information seems to behave correctly but not the 3 others in particular it is the characteristics of these services that make my program fail (when i do try to get the map of characteristics)
And it is those characteristics that contain the "sensitive" data such as pressure, temperature, acceleration, state of the device etc...
This what bring me to this reflexion...
I'll try to interact only with the characteristics of the services Generic Access, Generic Attribute, Battery Service and Device Information to be sure.

Hi,
probably its because you are using arduino library with bug (maybe more than 1), in this library its already fixed:
https://github.com/nkolban/esp32-snippets/blob/master/cpp_utils/BLERemoteService.cpp#L129

Ok thank you Chegewara i'll try it !

Good catch @Chegewara, the arduino library would do well to keep up with this repo, unfortunately that might break some folks currently running projects.

Thanks to you two !
If i understood well i have to replace the files of the src folder in my computer at :
C:\Users\MyName\AppData\Local\Arduino15\packages\esp32\hardware\esp32\1.0.2\libraries\BLE
by all the files *.cpp and *.h at _https://github.com/nkolban/esp32-snippets/blob/master/cpp_utils_ ?
I'm not very comfortable with Arduino yet so i prefer to ask you :-(

Yes, just download this repo and copy/ paste the files from cpp_utils folder into the the libraries/BLE folder and you’ll be using this library instead. You might have to make some adjustments to your code to accommodate this library, such as using an advertised device pointer in the connect() call instead of address. Just check the examples.

Hi,
probably its because you are using arduino library with bug (maybe more than 1), in this library its already fixed:

Well it seems like you were right !
To resolve my problem i did what you said : i updated the library at :
C:\Users\MyName\AppData\Local\Arduino15\packages\esp32\hardware\esp32\1.0.2\libraries\BLE
By the library at :
_https://github.com/nkolban/esp32-snippets/blob/master/cpp_utils_
I only replaced the files that were on both library, i didn't add the other files present in cpp_utils because it made my code uncompilable.
And now, it works perfectly !!!
Thank you very much chegewara and h2zero for your time :)
Have a nice day.

Hi,
probably its because you are using arduino library with bug (maybe more than 1), in this library its already fixed:
https://github.com/nkolban/esp32-snippets/blob/master/cpp_utils/BLERemoteService.cpp#L129

Hi, I have a question about ‘esp32-snippets’ repository. I found some bugs in the arduino-esp32  that have been fixed in this repository. Why does not pull to arduino-esp32.


Provide a third logical node type "NOT" to negate sections of the intermediate query tree

Believed benefit here is that though individual operators often come in polarized pairs, sometimes it's more natural to express "not all these things" instead of "factoring not" throughout.

This is possible for every backend except for the rsql variation. So far I haven't added anything that isn't able to be implemented in every backend to core, so I think I'll continue that for now.


Request to update operation-key value for notify-controller-attribute-value-changes

Problem Description:
As per specification NotificationProxy+config.json "operation-name": "/v1/notify-controller-attribute-value-changes" is having     operation-key present as "n.a." under operation-server-interface-configuration  as mentioned below
 {
            "uuid": "np-1-0-0-op-s-is-011",
            "ltp-direction": "core-model-1-4:TERMINATION_DIRECTION_SOURCE",
            "client-ltp": [
            ],
            "server-ltp": [
              "np-1-0-0-http-s-000"
            ],
            "layer-protocol": [
              {
                "local-id": "0",
                "layer-protocol-name": "operation-server-interface-1-0:LAYER_PROTOCOL_NAME_TYPE_OPERATION_LAYER",
                "operation-server-interface-1-0:operation-server-interface-pac": {
                  "operation-server-interface-capability": {
                    "operation-name": "/v1/notify-controller-attribute-value-changes"
                  },
                  "operation-server-interface-configuration": {
                    "life-cycle-state": "operation-server-interface-1-0:LIFE_CYCLE_STATE_TYPE_EXPERIMENTAL",
                    "operation-key": "n.a."
                  }
                }
              }
            ]
          }

Proposal :

Updating operation-key to the default as "Operation key not yet provided."

We would support this proposal. However, this is a change to the specification that can only be implemented with a later update.

We would support this proposal. However, this is a change to the specification that can only be implemented with a later update.

Thank You!

In the CONFIGfile:

Values of the operationKeys of individual services (it was not just /v1/notify-controller-attribute-value-changes) changed from "n.a." to "Operation key not yet provided.".
Values of the operationKeys of ODL clients changed from "Operation key not yet provided." to "n.a.".

Don't understand comment above:
"However, this is a change to the specification that can only be implemented with a later update."
It is a value in the CONFIGfile.
What would be the purpose of a CONFIGfile, if changing the values would require another software release?


Add postnote/prenote functionality to \sidecite and add \sidetextcite and \sideparencite

Add support for a postnote and a prenote to \sidecite (backwards compatible), basically making citing a more biblatex-like experience and addressing issues brought up in #46. Also, add support for other biblatex-like commands:

\sidetextcite: like \sidecite but uses a \textcite in-text citation
\sideparencite: like \sidecite but uses a \parencite in-text citation
\sideparencite*: like \sidecite but uses a \parencite* in-text citation

Tested fairly thoroughly with numeric and apa (like authoryear) citation styles.

Note: I didn't update the Citations section of examples/documentation/chapters/references.tex because it looked out of date on other stuff (example below):

[\sidecite] does not have an offset argument yet, but it probably will in the future.

Here are the tests I ran to make sure these side citing commands match their corresponding biblatex commands:
before \sidecite[][pre][]{Peters2017book} after

before \cite[pre][]{Peters2017book} after

before \sidecite[][post]{Peters2017book} after

before \cite[post]{Peters2017book} after

before \sidecite[][pre][post]{Peters2017book} after

before \cite[pre][post]{Peters2017book} after

before \sidetextcite[][pre][]{Peters2017book} after

before \textcite[pre][]{Peters2017book} after

before \sidetextcite[][post]{Peters2017book} after

before \textcite[post]{Peters2017book} after

before \sidetextcite[][pre][post]{Peters2017book} after

before \textcite[pre][post]{Peters2017book} after

before \sideparencite[][pre][]{Peters2017book} after

before \parencite[pre][]{Peters2017book} after

before \sideparencite[][post]{Peters2017book} after

before \parencite[post]{Peters2017book} after

before \sideparencite[][pre][post]{Peters2017book} after

before \parencite[pre][post]{Peters2017book} after

before \sideparencite*[][pre][]{Peters2017book} after

before \parencite*[pre][]{Peters2017book} after

before \sideparencite*[][post]{Peters2017book} after

before \parencite*[post]{Peters2017book} after

before \sideparencite*[][pre][post]{Peters2017book} after

before \parencite*[pre][post]{Peters2017book} after

And here is the corresponding output with citestyle=numeric:

And here is the corresponding output with citestyle=authoryear:

Excellent!


resort half heartedly

🚀 Deployed on https://663d0b3bcc85d328b4e02098--cozy-fox-0defec.netlify.app


Map fixes

Closes #44
Nerfs armory, adds a few new places, changes up the emergency shuttle & Central Command, small bug fixes.

I may have accidentally removed the entirety of Frigid and CentComm