[umbrella] Configuration enhancements

Description
As discussed in https://github.com/apache/incubator-gluten/issues/6970, we plan to enhance the gluten configuration, and the specific enhancement solutions can be found in section https://github.com/apache/incubator-gluten/issues/6970#issuecomment-2511452471.
This issue serves as an umbrella to record the task list that we need to include to achieve our goals, because enhance config involves a large number of files, splitting multiple tasks facilitates code review and reduces the risk of introducing other problems.

[ ] Introduce the ConfigEntry to make the config definition more flexible.
[ ] Split the configuration into individual modules that belong to each other.
[ ] Split configuration into two separate files: one static and one non-static.
[ ] Standardize configuration naming, log the deprecated config name if used.
[ ] Support generate gluten configuration doc and check it. https://github.com/apache/incubator-gluten/issues/8070

@yikf FYI
Here are some my own summaries about characteristics of configuration:

The configuration at startup cannot be dynamically changed
Configurations that can be changed per query(i.e. session)
Configurations that only affects the configuration of the driver
Configurations that needs to be passed to the worker, that is, the backend configuration, not the Spark configuration
Spark configurations that needs to be passed to backend, for example, spark.sql.caseSensitive


Build A Scrappy Stack With Me

Contact Details
@lauren.benezra (slack)
I have read the dbt Developer Blog contribution guidelines.

[X] I have read the dbt Developer Blog contribution guidelines.

Which of these best describes you?

[ ] I am a dbt Community member or partner contributing to the Developer Blog
[X] I work for dbt Labs and am creating this issue for a community or marketing approved piece.

What is the topic of your post?
I volunteer for a cat rescue organization called Feral Change as their Foster & Behavior Coordinator. As you can imagine, our data is a rat's-nest of Google Sheets, and I've finally started designing the long-desired proper data stack that we can implement. We want to build a framework that other rescues and non profits can benefit from.
I thought it would be super fun to document this process, thinking, and design, so other tiny start-ups or non profits can build their own modern data stacks on a budget. Obviously, this includes utilizing dbt.
Link to an initial outline.
https://www.notion.so/dbtlabs/DevHub-Blog-Scrappy-Cat-Stack-4d1b40228f644fcd8857afb796b289d6

Hey @lbenezra-FA , thanks for opening up this issue! So hyped for this post. Just assigned @krevitt as your primary editor/PM for this piece, so feel free to collab with him and chat through anything at devhub writing sessions :)

@lbenezra-FA closing this issue out to clean up our open issues, but please re-open if you want to get this blog post going again!


Add support for input type number and email on 'nypl-text-field' class

Right now, the 'nypl-text-field' class looks specifically for input[type='text'] to add styling, other types should be supported such as email and number.

fixed in 7d2db3d6ec5d56774ffc2ce14b2a24519bb9b5a8

(not in npm yet)


Fix for #289: IANA code for TLV

Signed-off-by: Frederic DUR<EMAIL_ADDRESS>

Hi,
please have a look to my comment in #289.
Regards

OK
From my point of view, I thought that the LWM2M_OLD_CONTENT_FORMAT_SUPPORT flag meant that only old code for TLV/JSON are supported.
Your remark makes sense: close the ticket and the pull request


Pull report creation code into its own interface

The migration code is getting a little cumbersome, as it's mixing logic for the creation new-schema models and the extraction of old-schema models. Pulling out this new-schema creation logic would make the migration code cleaner and more reusable.

A new structure would facilitate these changes: mongoose interfacing, report creation and loo generation could be fully contained inside a new loodb package.
This would strip the api package down to simple routes and peculiarities of transmitting and receiving loos and reports as JSON across the internet. It would also strip the migrate package down to code that focuses purely on transition of data between legacy and new schemas.
The newer modules would have greater focus on a single purpose and would be more useful as decoupled individual tools, e.g:

The loodb package could easily be used in code that would create custom data sets, without worrying about bringing in migration or web code
The api package could would require no modification to use a different backend in the case that we moved away from MongoDB


Epidemic Game leaves trail after recovery

Fixed in version 800. Not yet made default. Still broken in Epidemic Game Maker when run in 791 (NetLogo 5.3)


[Feature] Allow listing of members across bridges

I wonder whether we could create a shorthand for helping people find members across bridges.
An ephemeral slash command might work: (only seen by sending user)
https://api.slack.com/slash-commands
Feature as imagined is slack-specific, but need not be.
A Slack slash command is just an integration that would be set up in Slack UI to hit an endpoint -- it could run off a "hook", whose precedent was already set up rocketchat hook integration
Or does this this seem like scope creep? I could understand that perspective :)

Related to #191
I want one solution for all bridges, probably will be by talking and sending commands to the bot

Ok, I'm grateful that you see it as helpful too! Do you feel it's important that the command/response NOT be public when possible to avoid?

If you wanted a list, it seems important it should PM you -- but checking to see if someone is "online" could still be really useful and not as spammy ...

!whois Jaykul
Found Jaykul on Discord (idle) and IRC as well as (here on) Slack

cc @valvin1 @guakamole (from other thread)

Closed in favor of #191


Add e2e job to CI

Resolves #13

@bjhargrave do you think the suggested changes here should be done for the SDG implementation as well?

do you think the suggested changes here should be done for the SDG implementation as well?

Yes. You want all the checkouts done before pip caching to get all the cache-key files in place.
I detest using caching in CI jobs since it is a source of innumerable errors. But I can see that uncached installs of the cli take a while. So we should at least try to get the caching right.

I think the e2e tests might need this change: https://github.com/instructlab/eval/pull/17/files


fix(SDK): SteamVR defines

Type.GetType requires the use of an assembly name. This fix adds an improved version.

There were the following issues with your Pull Request

Commit: fca21618a4e3400aa4e5d605c106336d577eb6a6
Your commit message body contains a line that is longer than 72 characters

Guidelines are available at https://github.com/thestonefox/SteamVR_Unity_Toolkit

This message was auto-generated by https://gitcop.com

Interesting. So you compile the SteamVR plugin into its own .dll?

Ah I see. You're putting the SteamVR plugin into the Plugins folder probably. Note that you shouldn't do this, as it's a special folder.
Assembly-CSharp-firstpass is Phase 1 here.

Ah, SteamVR used to put openvr_api.cs into the Plugins folder. When upgrading that file needs to be deleted or it will do this.

That doesn't matter, as that file is just the OpenVR API. SteamVR is never put into the Plugins folder by default. You can see that it's put into the Assets/SteamVR folder here, which means you as user probably moved it into Plugins later instead.

@bddckr just to confirm, is this PR a valid fix?

@thestonefox It is! But we probably should update all the other SDK_*Defines classes, too.

@bddckr I'm going to look at doing a PR today re-implementing this work but for the other define classes too (as you suggested). Otherwise we may not get this ready in time for 3.2.0 as it's been 4 days since original commit and no sign from the author :(

@bddckr only SteamVRDefines uses Type.GetType
The other defines all seem to use Assembly.GetType

So does that mean for instance instead of doing
Type eventClass = typeof(SDK_SteamVRDefines).Assembly.GetType("SteamVR_Events");
we should do
Type eventClass = VRTK_SharedMethods.GetTypeUnknownAssembly("SteamVR_Events");

PR to supersede this PR https://github.com/thestonefox/VRTK/pull/1309


Create GenericTemplateWithHeader.md

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

csidevdocs-public/GenericTemplateWithHeader.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

#sign-off

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

csidevdocs-public/GenericTemplateWithHeader.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

csidevdocs-public/GenericTemplateWithHeader.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

csidevdocs-public/GenericTemplateWithHeader.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

#sign-off

#sign-off

#hold-off

#please-close

#please-close

#please-close


Fix label updating

If you scroll the calendar page with the finger without waiting for the full deceleration, the label update only at the very end.

I do not understand this one.
Let me know if i have described your problem accurately.

I begin to scroll with my finger
With my finger still on the screen, I have reached a new month
my finger is still on the screen

Are you saying that the date should now update?

No the finger can leave the screen. But if you go quickly it never update. I've attached an image to show you where I go from dec 2009 to jun 2010 with no label update in between.

@cl3m  I found current Demo just behave the same with the Gif.

@Natai yes. Thats because I have a bad example. :)
But the code should work properly i think.
Did you implement the scrollDidEndDecelerating delegate function?

@patchthecode  Yes, It works to me. Thanks!

@patchthecode I tested my code again. I found that if I implement the scrollDidEndDecelerating  like this:
func scrollDidEndDecelerating(for calendar: JTAppleCalendarView) {
        self.setupViewsOfCalendar(from: calendar.visibleDates())
}

sometimes calendar can not get the right visibleDates(eg: scroll between Nov and Dec). I don't know what is wrong.

@Natai didEndDecelerating gets triggered the very second that you lift your finger.
Therefore are you looking for the dates the very second that you lift your finger?
Or
are you looking for the dates then the scrolling from your finger comes to a complete stop?

@patchthecode Do yo mean I should implement both of them? It works, but didn't scroll smoothly...
func calendar(_ calendar: JTAppleCalendarView, didScrollToDateSegmentWith visibleDates: DateSegmentInfo) {
        self.setupViewsOfCalendar(from: calendar.visibleDates())
    }

    func scrollDidEndDecelerating(for calendar: JTAppleCalendarView) {
        self.setupViewsOfCalendar(from: calendar.visibleDates())
    }

@Natai I am not saying that I should implement both of them. What i want to know is, behavior do you want the Library to be?
If you let me know what you want and how you want it to behave, then we can think of a good solution. Then I can code the solution and we can both be happy.

@patchthecode sorry.....:]   I'm looking for the dates the very second that I lift my finger.

@Natai you say that when you implemented both of them, that the scrolling performance was bad.
There should not be any bad scroll performance.
Can you test something for me?
put this as your code:
func delayRunOnMainThread(_ delay: Double, closure: @escaping () -> ()) {
    DispatchQueue.main.asyncAfter(
        deadline: DispatchTime.now() +
            Double(Int64(delay * Double(NSEC_PER_SEC))) /
            Double(NSEC_PER_SEC), execute: closure)
}

func delayRunOnGlobalThread(_ delay: Double,
                            qos: DispatchQoS.QoSClass,
                            closure: @escaping () -> ()) {
    DispatchQueue.global(qos: qos).asyncAfter(
        deadline: DispatchTime.now() +
            Double(Int64(delay * Double(NSEC_PER_SEC))) /
            Double(NSEC_PER_SEC), execute: closure
    )
}

func calendar(_ calendar: JTAppleCalendarView, didScrollToDateSegmentWith visibleDates: DateSegmentInfo) {
            delayRunOnGlobalThread(0.1, qos: .userInitiated) {
            let visibleDates = self.calendarView.visibleDates()
            delayRunOnMainThread(0.1) {
                self.setupViewsOfCalendar(from: visibleDates)
            }
        }
}

func scrollDidEndDecelerating(for calendar: JTAppleCalendarView) {
            delayRunOnGlobalThread(0.1, qos: .userInitiated) {
            let visibleDates = self.calendarView.visibleDates()
            delayRunOnMainThread(0.1) {
                self.setupViewsOfCalendar(from: visibleDates)
            }
        }
}

Let me know if the performance is still bad.
And if the performance is still bad, then can you show me the code you have in this function?
func calendar(_ calendar: JTAppleCalendarView, willDisplayCell cell: JTAppleDayCellView, date: Date, cellState: CellState)

@patchthecode I tested it in my APP and your newest demo 8 hours ago. Now, I swipe right it behave normally, there is a little delay to update text conversely. Just like this:

And I found a interesting thing, if I lift my finger and just keep clicking mouse the calendar will not stop scroll:

My willDisplay func:
func calendar(_ calendar: JTAppleCalendarView, willDisplayCell cell: JTAppleDayCellView, date: Date, cellState: CellState) {
        let myCustomCell = cell as! CalendarCellView
        myCustomCell.dayLabel.text = cellState.text

        if currentCalendar.isDateInToday(date) {
            myCustomCell.todayView.backgroundColor = Color.todayBackgroundColor
        } else {
            myCustomCell.todayView.backgroundColor = UIColor.white
        }
        handleCellSelection(view: cell, cellState: cellState)
    }

ok let me check this out right now

Can you put this for your code?
    func calendar(_ calendar: JTAppleCalendarView, didScrollToDateSegmentWith visibleDates: DateSegmentInfo) {
        self.setupViewsOfCalendar(from: visibleDates)
    }

    func scrollDidEndDecelerating(for calendar: JTAppleCalendarView) {
        self.setupViewsOfCalendar(from: calendarView.visibleDates())
    }

Also I was not able to reproduce the second issue.
Also, if you are using the demo project,
Do you have this code in your viewDidLoad() ?
calendarView.scrollingMode = .stopAtEachSection

If you do, then delete it.

@patchthecode I had implement didScrollToDateSegmentWith and scrollDidEndDecelerating in your demo. As you say, everything behave the right way after I delete (include the second issue)
calendarView.scrollingMode = .stopAtEachSection

You are so kind. Thanks!

@Natai you're welcome. And thanks for the github star rating! 😄


chore: replace conformance by nodesecure/licenses-conformance

close #10

@all-contributors please add @Kawacrepe for code


n_queens.py added under backtracking_algorithms issure #56

global variables removed as requested @Abhijit2505

global variables removed as requested @Abhijit2505

There is already a minor change requested above.

@Abhijit2505, please review the code and tell me if I m missing something


Organize MRTK Content under a MixedRealityContent GameObject

The problem we are trying to tackle
One of the challenges of creating UI for Mixed Reality is aligning experiences across both AR and VR platforms. Previously, HoloToolkit provided a solution with the SceneContentAdjuster, however, this solution required users to explicitly add monobehaviors to objects they wished to be adjusted, and did not offer much in the way of adjusting user controls/object placement based on experience setting (for example, stationary experiences might expose different controls to the user compared to room scale experiences)
The proposed solution
To address this, I propose introducing a  SceneContentSystem with an associated MixedRealityContent Gameobject to the scene. The SceneContentSystem would be used to toggle different features and expose certain parameters for other systems (floorHeight for teleport, targetScale for boundary). while the MixedRealityContent object would provide a dedicated place for users to put their content so that it scales appropriately based on their desired experience. This object would have an equivalent and configurable SceneContentAdjuster monobehavior to accomplish this task.
Basically, if the user wants an element to be placed in the correct position across VR and AR experiences, they would have to make that element a child of the MixedRealityContent object. This would also give MRTK a designated place to spawn relevant UI control objects when adding them in editor.
How would this change affect user experience
In many of the example scenes, we already organize all of the MRTK content (panels, buttons etc.) under a single SceneContent object. This implementation would essentially make it so that when we use the "Add scene and configure" menu option, we instaniate the scene with the MixedRealityToolkit, MixedRealityPlayspace, and a new MixedRealityContent object, encouraging the same sort of gameobject heirarchy as our existing examples.

What if the user doesn't want to adhere to the structure?
The user can still instantiate objects outside of this MixedRealityContent object to have those objects stay in the same part of world space regardless of experience. This will result in behavior identical to the current MRTK experience.
They could also create more objects with the SceneContentAdjuster that manage themselves, with the option to link them to the broader SceneContentSystem.
Summary
The main point of the MixedRealityContent object is to guide the users to laying out their content in a certain hierarchy, and to provide a place for us to instantiate new MRTK objects and ensure they are aligned to the user experience settings (the floor height and experience scale)

@RogPodge Is this completed after https://github.com/microsoft/MixedRealityToolkit-Unity/pull/9691?

yep! closing this now


Fix #189: Select element "flickers" on first page load

Hi @boeserwolf, thanks for updating the tests. I really appreciate that.
However, it seems no tests are really running. I've tried to fail them, switching from true to false and so on, but they still passing through.
I believe karma doesn't wait for the dom to run the tests. I did some research about that, and I'll need to move the tests for a service.

http://stackoverflow.com/questions/27830126/jasmine-test-angular-elementdocument-ready
http://stackoverflow.com/questions/30357261/unit-test-an-angularjs-document-ready-function

Hi @leocaseiro, thanks for your reply. Sorry, my bad, I should really have tested that.
After doing some further research, it looks like there's just no need for $document.ready() or $timeout(), because Angular won't initialize until the DOM was loaded:
jqLite(window.document).ready(function() {
    angularInit(window.document, bootstrap);
  });

})(window);

So, as far as I understand, we should be good with just calling element.chosen(). I've updated the pull request correspondingly. Are you solving another specific issue by wrapping the statement with $timeout()?
Thanks!

Hi @boeserwolf,
Thanks again.
Unfortunately, I had to add $timeout to solve the issue #176
I'll need to work on this on the weekend. I'd like to have a solution for both issues.

Seems #213 solves it

Closes in regards to #213
Thanks for all help guys!


Attempting to choose filepath on Windows crashes

Pretty much any attempt to interact w/ the file objects crashes.
on Windows 10:

launch Nodezator
File | New --- create "test.ndz" in the user directory
right-click | Useful encapsulations | read_text_file
click on the green text folder to select a path
select an existing file on the desktop ("C:\Users\willa\Desktop\tolkienbooks.txt")

app vanishes
Are there crash logs or anything which will help?

Please, go to the location where nodezator is installed and inside the "nodezator" folder there should be a "logs" folder with the logs from your session.

Please, also try, to update your installation using pip install --upgrade nodezator (in my machine it only works when called 02 times), or download from github again, if you didn't install). Then, try the same steps you made again. This time a dialog should appear instead of the app crashing.
If you locate the logs upload them here so I can take a look, please.

Updating to 1.0.5 allows selecting a filepath on Windows. Thanks!

Oh, really? That's great news! May I close this issue them?

Actually, I'm getting an error:

I believe I get the problem now. The node is fine, it even has a field where you can specify the encoding. The problem is that the widget has to load the file in order to show you a preview and it does so using a 'utf-8' encoding. To solve this I guess I'll have to either analyse the file and then choose the appropriate encoding used to load it for the preview or just keep using the utf-8 encoding by use a more lenient loading method that ignore unicode errors (the pathlib.Path.read_text() method has a parameter to allow this if I'm not mistaken).
I'll look into this right away and upload a new patch.

Ok, please, try updating your Nodezator version to 1.0.6 now, and repeat the steps that led to your original problem. See if no errors are raised anymore.
Also, if the text is previewed in the widget, click the preview and see if the file is displayed in the text viewer inside the app, some characters should probably not render well, but apart from that it should be fine.

That worked!
I'd suggest making the encoding entry a pop-up, and having the program determine the valid options and only afford them.

Yes, setting the errors parameter to ignore encoding errors was just a stopgap measure to prevent the app to crash on the users and allow them to keep working. I'll think of a better solution. and put this problem in my to-do list.
Since I believe the problem (crashing) was solved, if you don't mind I'll be closing this issue.


Feature/api observability through pod annotations

Added optional configuration for DataDog using pod annotations:
The API operator for Istio can be configured to use DataDog to monitor the API. The API Operator can take an environment variable OPENMETRICS_IMPLEMENTATION which defaults to ServiceMonitor (for prometheus operator). If you set OPENMETRICS_IMPLEMENTATION to DataDogAnnotations, the API Operator will add the DataDog annotations to the pod running the metrics API (which DataDog uses to scrape custom metrics) instead of creating the ServiceMonitor resource.
The environment variable is set in the configmap  named canvas-configmap in the canvas namespace`. e.g.
apiVersion: v1
kind: ConfigMap
metadata:
  name: canvas-configmap
  namespace: canvas
data:
  COMPONENT_NAMESPACE: components
  KEYCLOAK_BASE: http://canvas-keycloak.canvas:8088/auth
  KEYCLOAK_REALM: myrealm
  LOGGING: "20"
  OPENMETRICS_IMPLEMENTATION: DataDogAnnotation

Closes #93


Generate Camel Java routes in package

For now, Camel routes written using Java DSL are generated without a Java package structure. Allow user to specify a target package with a generation property.

Done!


How to Upload images to Cloudflare Images

Hi,
I've read the documentation many times but still didn't figure it out on how to upload image to cloudflare ?

I am using Cloudflare Pages to host the Nuxt.js 3 - works fine

On Local Dev when I use DevTools I can see OG-Images beeing rendered as I it should which is great.
However, since my website is dynamic, I need to dynamically "upload" those OG-Images to Cloudflare
When I deployed my Nuxt.js 3 app to Cloudflare pages I got this in sourc:

Any hint help is needed please..

Hi, not too sure what you mean.
The images are generated dynamically at runtime, they shouldn't be uploaded anywhere. If you want to use a persistent cache storage see Caching Images.

Hi @harlan-zw
Here is an example:
I upladed the project on Cloudflare Pages:
https://aihive-front.pages.dev/ai-tools-news/10-cutting-edge-ai-tools-transforming-social-media-and-content-creation-in-2024
And this is the generated image on runtime:
view-source:https://aihive-front.pages.dev/ai-tools-news/10-game-changing-ai-tools-transforming-businesses-in-2024
Which is not working
and this is inside script setup:
defineOgImageComponent('Article', {
  articleTitle: aiarticle.value?.name || 'Default Article Title',
  brandImageSrc: 'https://imagedelivery.net/Wi6_E62agkkvW3RsmnEvAw/0285ec60-08e2-44d8-082f-c42f687bff00/100',
  brandImageAlt: aiarticle.value?.name || 'Default Article Name',
  brandImageWidth: 200,
  brandImageHeight: 200
});

useHead({
  title: `${aiarticle.value.name} - AI Hive Tools`,
  htmlAttrs: {
    lang: 'en',
  },
  link: [
    {
      rel: 'canonical',
      href: `https://aihive.tools/ai-tools-news/${slug}`,
    },
  ],
  meta: [
    { name: 'description', content: `${aiarticle.value.name}` },
    { property: 'og:title', content: `${aiarticle.value.name} - AI Hive Tools` },
    { property: 'og:description', content: `${aiarticle.value.name}` },
    { property: 'og:type', content: 'article' },
    { property: 'og:url', content: `https://aihive.tools/ai-tools-news/${slug}` },
    { property: 'og:image', content: '' },
    { name: 'twitter:card', content: 'summary_large_image' },
    { name: 'twitter:title', content: `${aiarticle.value.name} - AI Hive Tools` },
    { name: 'twitter:description', content: `${aiarticle.value.name}` },
    { name: 'twitter:image', content: '' },
    { property: 'og:locale', content: 'en_US' },
    { property: 'og:site_name', content: 'AIHive.tools' },
    { name: 'robots', content: 'index, follow' },
  ],
});

Once again, the image is being generated on DevTools great, but when I deploy the web app now I don't have any image.
Am I missing something ?

The image is working correctly https://aihive-front.pages.dev/og-image/image/ai-tools-news/10-cutting-edge-ai-tools-transforming-social-media-and-content-creation-in-2024/og.png
The problem is you have the site URL set to a different domain

@harlan-zw omg, time to sleep, I was playing with this for more than 12 hours setting everything up and I knew I'm missing something.
Thank you!


Feature/az/api entrypoints coverage

This PR depends on https://github.com/basho/riak_pb/pull/106, in turn depending on the bunch of PRs by @zeeshanlakhani.

It still merges cleanly, so no changes; just a rebase onto HEAD of develop.

I disagree generally with the approach. We already have preflist calculation as an API request, why not simply provide the nodename + host/port/protocol for every interface?
The client can cache the network information and simply request the preflist for the desired bucket-type/bucket/key, and then match up the nodenames to existing connections or new connections to be opened. Obviously, the client should also periodically re-request for the live nodes' interfaces, or by be triggered some signal from the server.
Also, I'm doubful about how you're deriving the host interface information, because it is totally valid for a listener to start up with <IP_ADDRESS> or ::0 as the address, which binds all interfaces but is not routable. Also, you don't have adequate information about how the client can (or cannot) be routed to the bound interfaces.

@seancribbs The issue you point out concerning listening ip where the external ip is meant, is now quite embarrassingly obvious to me. It's a blunder I've just committed (no, I have sat on it feeling good for a week!). What's needed here is some code path eventually calling getifaddrs() and picking the addr of the interface we assume to be the one externally accessible.
I will review the whole thing tomorrow and report back.

Reworked, now the functions return the proper external ip instead of <IP_ADDRESS>.
@seancribbs concerning your suggestion to return the nodename and host/port, much as I'd like to just go with that, the java guys seem to only want the external ip and the port(s), not the nodenames proper or the part after '@' in those. I've been scratching my head hard, but other than calling getifaddrs on location, I just don't see how one can figure out the ip to which external clients can connect. I'll be happy to stand corrected, but at any rate, the scope of DP-6 is precisely about that.
I grepped the entire riak/deps/ and found no matches on getifaddr (except an odd one in erlang_js).
I am provisionally force-pushing the amended commit, for @srgg to continue his work on riak java client for the Thursday demo. There are a couple of issues to resolve, notably what 'heuristics' to apply in the case of multihomed hosts (such as those with infiniband in addition to plain eth0).

I suppose this is fine for a demo, but before this merges, I'd like to see the following changes:

The host interface information is at a fundamentally different granularity than the preflist information. Since we already have requests for fetching the preflist, there's no reason to duplicate that effort here; all we need is the host/port/protocol information.
The code in this pull request RPCs to every node in the preflist on every request. The host interface information doesn't change often enough to warrant this, and under load RPC will become a bottleneck. Instead, each host should publish its listener information in cluster metadata, which will be gossiped around automatically. Fetching the data is an inexpensive, local operation.

To go the whole hog, I now see I'll have to:

Provide a function to enumerate the interfaces and export the bound addresses to the preflist (still have to figure out how to do the latter).
For the first time, call that function at listener app start, then call it once every minute (or not, with a big fat warning to whoever will be moving hosts around, to take care).
If a listener is configured to listen on an address other than <IP_ADDRESS> that ends up being different from what has been discovered, reconfigure that listener.
Make my api calls to just pick the relevant bits from preflist.

I don't know what the preflist has to do with the bound addresses, but yes, you will need to export them to cluster metadata.
I don't think you need to check every minute. Why would a listener address be changing that often? Or are you talking about from the client? If the latter, then perhaps that could be configurable. Polling might be ok, or triggered on a failed connection to a previously discovered host.
Not sure what you mean here. Can you elaborate?
When you say "coverage map", what do you mean exactly? "Coverage" usually means "a list of partition/node pairs covering the entire keyspace", and doesn't require a bucket/key to seed it. The "preflist (preference list)" usually refers to the replicas calculated for an individual key, based on consistent hashing.

@seancribbs Thanks for commenting and dropping precious hints.
I. My line of reasoning was about this call: https://github.com/basho/riak_kv/blob/develop/src/riak_kv_web.erl#L115. It returns (on my devrel setup):
$ curl  localhost:10018//buckets/B/keys/K/preflist
{"preflist":[{"partition":29,"node":"dev5@<IP_ADDRESS>","primary":true},...]}
where I expect these new properties to appear: "http_ip":"<IP_ADDRESS>", "http_port":10018 (plus same for pbc). I see I used the word 'preflist' meaning 'cluster metadata' in my previous comment, inaccurate me. But, are these new entry point details now in the proper place as returned by this call?
II. Indeed, let it be triggered by a dropped connection (and failure to establish one initially, too). Then, when a client times out or sees it gone, it will re-request the entire entry points map, possibly with &force_refresh=1 (or some such) to cause it to be regenerated. And then no periodic autorefresh is necessary.
III. Suppose riak.conf initially has, e.g, listener.protobuf.internal = <IP_ADDRESS>:10017, and then the host migrates from <IP_ADDRESS> to <IP_ADDRESS>. If the configuration had <IP_ADDRESS>:10017, the listener would just keep going. Otherwise, the listener would listen on an address which is no longer routed (it would even fail to restart, correct?).
My proposition is to check whether the configured listening address, unless it is <IP_ADDRESS>, does match the bound address on some ethX in the system, and take steps to correct the situation.
IV. It's simply the complete list of all API entry points into the cluster. The idea is to extend this list with details on what key ranges are quickest accessed via each entry point -- and thus attain the meaning of "coverage". Properly naming things goes a long way, indeed.

Another reworking effort to address @seancribbs latest comments. Due to massive changes over what went into this branch, I pushed the new work into https://github.com/basho/riak_kv/tree/feature/az/api-entrypoints-coverage-rebased.

Work and further discussions moved to https://github.com/basho/riak_kv/pull/1088; closing this one then.


Relicense to MIT-0

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

(FP for missing NOTICE file deletion)


Support for short path syntax

If you want to write optimised output to the same file you have to specify the same path twice. It happens when it’s not a source file and generated by a preprocessor:
csso: {
	task: {
		files: {
			'styles/screen.css': 'styles/screen.css'
		}
	}
},

But in grunt-autoprefixer it’s possible to use src shortcut for such cases:
autoprefixer: {
	task: {
		src: 'styles/screen.css'
	}
}

But when I try to use the notation for grunt-csso, I see following error:
grunt csso
Running "csso:task" (csso) task
Warning: Unable to write "undefined" file (Error code: undefined)

Could you please implement support for the same shortcut? Thank you.

I published<EMAIL_ADDRESS>so please use it. :)

Thank you!


ACM-14185 Application edit page crashed with multi subscriptions

Signed-off-by: John Swanke<EMAIL_ADDRESS>

/lgtm


DNS nameservers queried in wrong order

Expected behavior
On Windows, the system default DNS nameservers should be queried in same order as in the Windows operating system
Actual behavior
The DNS nameservers are queried in reverse order
Steps to reproduce
Have a setup with multiple nameservers, e.g. one nameserver from your internet provider (192.168.x.x) and one from an OpenVPN connection (172.16.x.x). In the Windows operating system first the nameserver from the OpenVPN connection is queried, and afterwards, if this fails, the ISP nameserver is queried. In Netty the opposite search order is used.
Minimal yet complete reproducer code (or URL to code)
(code example is in Scala)
private val resolverGroup = new DnsAddressResolverGroup(classOf[NioDatagramChannel], DefaultDnsServerAddressStreamProvider.INSTANCE)`

and later:
   def clientConnect(context: ConnectContext.Client): ChannelFuture = {
      logger.debug(s"CONNECTING [${context.address.uri}]")
      val b: Bootstrap = new Bootstrap
      b.group(clientGroup).channel(classOf[NioSocketChannel])
      // Set the Bootstrap.resolver to a non-blocking DnsResolver (though getting a list of default dns servers itself may be blocking,
      // cite from the DefaultDnsServerAddressStreamProvider documentation: "This may use the JDK's blocking DNS resolution to bootstrap
      // the default DNS server addresses.")
      b.resolver(resolverGroup)

I think that the problem is, that in the Netty code the nameserver search list is determined by only calling via reflection
sun.net.dns.ResolverConfiguration.nameservers
but I think that also
sun.net.dns.ResolverConfiguration.searchlist should be used to sort the nameservers into the right order
JDK code snippet:
    /**
     * Returns a list corresponding to the domain search path. The
     * list is ordered by the search order used for host name lookup.
     * Each element in the list returns a {@link java.lang.String}
     * containing a domain name or suffix.
     *
     * @return list of domain names
     */
    public abstract List<String> searchlist();

    /**
     * Returns a list of name servers used for host name lookup.
     * Each element in the list returns a {@link java.lang.String}
     * containing the textual representation of the IP address of
     * the name server.
     *
     * @return list of the name servers
     */
    public abstract List<String> nameservers();

Netty version
4.1.86
JVM version (e.g. java -version)
IMPLEMENTOR="Azul Systems, Inc."
IMPLEMENTOR_VERSION="Zulu11.56+19-CA"
JAVA_VERSION="11.0.15"
JAVA_VERSION_DATE="2022-04-19"
OS version (e.g. uname -a)
Windows 11, Build 22H2

Ps: I just saw, in the Netty API documentation you find the line: "These defaults do not respect your host's machines defaults".
So this rather is a feature request, and not a bug report.
/**
 * A {@link DnsServerAddressStreamProvider} which will use predefined default DNS servers to use for DNS resolution.
 * These defaults do not respect your host's machines defaults.
 * <p>
 * This may use the JDK's blocking DNS resolution to bootstrap the default DNS server addresses.
 */
public final class DefaultDnsServerAddressStreamProvider implements DnsServerAddressStreamProvider {


【iOS】'RNCWebViewSpec/RNCWebViewSpec.h' file not found

Bug description:
I created the project using expo and opened the new architecture.
i config new architecture enable in app.json
[
        "expo-build-properties",
        {
          "ios": {
            "newArchEnabled": true,
            "flipper": true
          },
          "android": {
            "newArchEnabled": true
          }
        }
      ],

and run npm prebuild to create ios directory.
but when i run build on xcode, i got error what is 'RNCWebViewSpec/RNCWebViewSpec.h' file not found

Environment:

OS: iOS
OS version: 16.7
react-native version: 0.73.3
react-native-webview version: 13.8.1

I solved it, it was a version problem by package.lock


Update test docker container

Updates circle ci ruby image from the legacy image
Allows the entire test suite (unit and integration) to be run together by fixing some side effects from other specs
Use config.datadog_enabled for datadog related commands instead of Object.const_defined?("Racecar::Datadog")

looks like the 2.7 and 3.2 specs were hanging. how can I re-trigger the workflow?


Bullet or number ordered list not working with indent...

Bullet or number ordered list not working with indent and all numbers list start with 1....so please fix it if possible....

i'm working on it please wait for few week
thank you

I'm working on an indentation problem.
list sequence problem has been fixed in delta_to_html  0.2.2


Implementation of "Learning Audio-Text Agreement for Open-vocabulary Keyword Spotting" is not found.

Implementation of "Learning Audio-Text Agreement for Open-vocabulary Keyword Spotting" was not found in this repo. Any other sources are there for this approach?

In the paper, you are passing mel spectrogram to 1D convolution. How you are passing the 2D matrix to 1D array?
Are you flattening it? if mean it will loose the time information right?
Thankx,
kesav


chore!: merge main to v1.x

Overview
Merged main to see the diff and make sure we we're not missing anything consensus breaking (technically we weren't, as its not consensus breaking, but just avoids a bad bug #2423)
I don't know if we want to bother with keeping up with v1.x, or attempting to push through this change, as we might just want to start cutting releases off main since all commits should in theory be backwards compatible and this PR is essentially impossible to review

Codecov Report

Merging #2424 (9dfb0fe) into v1.x (6639c1a) will increase coverage by 0.10%.
Report is 1 commits behind head on v1.x.
The diff coverage is 5.47%.

@@            Coverage Diff             @@
##             v1.x    #2424      +/-   ##
==========================================
+ Coverage   20.52%   20.62%   +0.10%
==========================================
  Files         138      131       -7
  Lines       15761    15276     -485
==========================================
- Hits         3235     3151      -84
+ Misses      12219    11823     -396
+ Partials      307      302       -5

Files Changed
Coverage Δ

app/app.go
4.52% <ø> (+0.06%)
:arrow_up:

app/default_overrides.go
14.95% <0.00%> (-6.39%)
:arrow_down:

app/prepare_proposal.go
0.00% <0.00%> (ø)

pkg/user/tx_options.go
0.00% <0.00%> (ø)

test/cmd/txsim/cli.go
0.00% <0.00%> (ø)

test/e2e/node.go
0.00% <0.00%> (ø)

test/e2e/setup.go
0.00% <0.00%> (ø)

test/e2e/testnet.go
0.00% <0.00%> (ø)

test/e2e/util.go
0.00% <0.00%> (ø)

test/txsim/account.go
0.00% <0.00%> (ø)

... and 14 more

... and 17 files with indirect coverage changes

closing as this PR has served its purpose to demonstrate the diff between v1.x and main


[feature request] Avoid anonymous sections in case section defined with a block

It will be useful currently anonymous sections defined with block argument to have a class name.
Proposed solution: deduce_section_class to be rewritten similar to this:
   def deduce_section_class(base_class, &block)
       klass = base_class
       if block_given?
<EMAIL_ADDRESS>Class.new(klass || SitePrism::Section))
         klass =<EMAIL_ADDRESS>
         klass.class_eval(&block)
       end
       return klass if klass

       raise ArgumentError, "You should provide descendant of \
 SitePrism::Section class or/and a block as the second argument."
     end

where @_last_section_name is a name parameter from section/sections call.
Why?
I'm currently working on an internal framework (going to make it public someday) for mobile automation based on site_prism which identify arguments based on element/section name and section/page class name
For example, I have a page object classes defined like this
# module SitePrismDSLExtension extend and override/monkeypatch some basic site_prism DSL methods
class BaseSection < Section
   include SitePrismDSLExtension
end
class BasePage < Section
  include SitePrismDSLExtension
end

class Row < BaseSection
   set_default_search_arguments
   element :column1
   element :column2
end

class Home < BasePage
   element :header
   sections :rows1, Row

   sections :rows2, Row do
      element :column3
   end

   section :rows3, Row do
      element :column4
   end
end

This DSL extension identifies locator based on class name and element name. For example, a locator for column1, column2 and default search args for section Row will be looked in YAML file row.yml;
header, rows1 and rows2  in home.yml;
column3 will be defined in home.yml but nested
header:
   locator: header_locator
# rows1 may use default locator
rows2:
  locator: rows2_locator
  column3:
     locator: column3_locator

This allows to extract locators to another abstract layer and use config overlay to define locators based on device or platform (iOS and Android) without defining separate page object for same page structure in case locator differ for specific platform/device.
The logic of mapping depends on the class name. If the page class name is Home it will be looking for locators in home.yml. If section class name is Row it will be looking for locators in row.yml etc.
The problem arise for section definitions like rows2 with section name as well as a block. The actual class won't have a name (anonymous). Finding superclass of this class may slightly help, but will create another issue - row.yml will have to hold locators for all the possible extensions of Row section and it will hold no binding to the parent section/page. To avoid this problem 2 things are needed:

Section class should have a name
Name of the class should be based on parent section/page name + section name
For example, for our case, it can be Home::Rows2 < Row and Home::Rows3 < Row

This should not be a breaking change as it only changes internal implementation.

In principle this is ok. However I would probably advocate for the following....

"Anonymous" included in the name somewhere, as it is an anonymous section and that's what we're calling it.
If it didn't have a random element, then there's a possibility of a name conflict? Need to think about this
Probably need to log exactly what we're calling it


Split Atom suggestion entry to Type and Constructor

Pull Request Description

Changelog:

update: split Atom suggestion to Type and Constructor
update: gui API
update: JSONRPC doc

Important Notes

Checklist
Please include the following checklist in your PR:

[x] The documentation has been updated if necessary.
[x] All code conforms to the
Scala,
Java,
and
Rust
style guides.
All code has been tested:

[x] Unit tests have been written where possible.
[x] If GUI codebase was changed: Enso GUI was tested when built using BOTH
./run ide build and ./run ide watch.

So we shouldn't tell them that Maybe: Maybe

The returnType is only used internally and not exposed to the IDE in the JSONRPC protocol. We need it as an index in the suggestions database. And it is much cheaper to store it in a field than to compute it.


readinessProbe.sh-script fails

Describe the bug
The problem is that when the scripts fetch an empty list from /proc/net/tcp and since it doesn't get an exception it won't fetch /proc/net/tcp6 where the ports are really are. It then doesn't find any open ports.
The problem here is that the value-variable is not stripped before it is split. This results in all ports equals None and then it will always continue - no matter whether the ports are listening or not.

https://github.com/artemiscloud/activemq-artemis-broker-kubernetes-image/pull/80


[FRG] Support optionally rendering elements based on rendering flow

This is a feature idea.
I am loving the way the absolute positioning is working on the visual editor, but there's one particular instance where it can make things difficult. If I do not know the length of my data, and the data needs to wrap, it can overlap my next elements.
So, consider two elements
[Wrapping Text for 1–12 lines - absoluteY: 10]
[Next Print - absoluteY: 30]

In that case, "Next Print" would end up being overlapped as soon as the first element grew past 1-2 lines.

One idea I had that might work, is to add a boolean "☑ Use Flow-Based Rendering" (not set on the name). Then I would check that option on the "Next Print" line.
The benefit to this, is that the visual editor could still use Next Print's absoluteY for showing it on the editor, but at the point it was rendered, it would just be rendered after the previous element, similar to how the code-based version typically works.

This is a super low priority thing for us, I've only run into wanting this a couple of times and have always been able to work around it. I'm just throwing the idea out there in case you feel it has merit.

After some of us using the GUI more, our feeling is that flow-based rendering would ideally be the default, with absolute positioning being less common.


Selected Account and button have the same styling in Switch Accounts.

@neerajwahi @borisyankov Can you please suggest a style for the button. I would like to work on this.

@vishwesh3 This issue can be closed now.

yup! :+1:


Move containerrun out of cf-operator

This is a proxy card to track the work I'm doing in #172048264, mostly to ensure things don't get lost.
Acceptance Criteria:

[x] we need a new repo in the cloudfoundry-incubator extension.
[ ] should be moved to the stemcell image.
[ ] remove code that copies this binary around.

Currently waiting on SR#816074 (landing the package in the stemcell).

This is not blocked, unless you mean "blocked on somebody doing a review"

SR#816074 was now accepted.

Blocked for the moment by #941

Unblocked with #941 closed

If someone builts a RPM from the separate repo, we can include that in the stemcell at https://build.opensuse.org/project/show/Cloud:Platform:Fissile-Stemcell
Quarks can then queue another story to remove the copy code, remove the build code from Quarks Dockerfiles and so on.

Oh, sorry, should have more context here.
@manno It's already in the stemcell; landed via SR#816074.  All the images that kubecf currently uses (since the 13.9 bump) has a /usr/bin/quarks-container-run binary.  I was going to work on making the PR for quarks-operator to drop the copying, but things got in the way.

It looks like https://github.com/cloudfoundry-incubator/quarks-operator/pull/1074 would handle the actual operator side (that I kept not getting to because other ~shiny~ things kept getting into the way.  Ooops.

I'm finishing your item in the Quarks backlog and queue a new story for Quarks to make the tests pass.
https://www.pivotaltracker.com/story/show/172048264

@manno @mook-as this item seems to be done
we're tracking removal of the operator containerrun on the quarks board


Fixed #26698 -- Fixed a dbshell crashes on Postgres with an empty database name.

…y database name

Thanks for your contribution, could you possibly add a mention in the docs/releases/1.9.7.txt release notes as this will be backported to the stable/1.9.x branch.

Sure thing, done!

Just have to fix dbshell test suite

Tests fixed 👍

Buildbot, add to whitelist.

Tests pass locally for me... Looking at the Jenkins UI I can't really tell how my change is related to those failing tests.

merged in 19ff506878071ac93de684fe01328707e75e2b3a, thanks!


Apply static routes on change to master state

Refactored static routes for private gateways so they also get loaded when the router switches to master state. Otherwise they're lost and connections drop after fail over.

Can I get some code reviews on this PR?  I have added it to my CI queue...

Does the “CsHelper.execute” logs the commands being executed and their outputs?
If not, what about adding logs to show the routes that are being added? Also, after line 35, what about a log to state that the route is being revoked.

tag:needlove

We'll be pulling this one in.

So, Si (@kiwiflyer) and I have been testing this functionality against VPCs in our lab and we verified that static routes are indeed loaded when a VR goes from BACKUP to MASTER.  The relevant logs showing that is works are below.  We are however still having problems with the static routes not getting added on demand to the current MASTER VR.  Has anyone else had this issue while running this commit?
2016-05-06 20:32:20,795 DEBUG    {u'config': {u'router_id': u'2', u'baremetalnotificationapikey': u'QtWfDPFOrqddFe3iofWCS6Yhf6g2cCTwR6n3-L7TaaKl3ReBI4219jckJ0AfdX878jo2DpW4ohhNG_6Hlad62g', u'baremetalnotificationsecuritykey': u'B99mUf0oWKL7ATA5WF9iMb17nk1ePQ6kgg3YgWA0kJE2vCaGH52Z65EWDAyCaOYiZB1YPr40pWiwTlBt7_TTmg', u'name': u'r-28-VM', u'eth0mask': u'<IP_ADDRESS>', u'type': u'vpcrouter', u'dns1': u'<IP_ADDRESS>', u'privategateway': u'<IP_ADDRESS>', u'vpccidr': u'<IP_ADDRESS>/22', u'domain': u'cs2cloud.internal', u'redundant_state': 'BACKUP', u'host': u'<IP_ADDRESS>', u'disable_rp_filter': u'true', u'redundant_router': u'true', u'router_password': u'3838383966687309145665215760380045387957133810480803993986340699863084083818544695063990662780164604461468366180051318649230056294824943648327772148445755', u'redundant_master': False, u'port': u'8080', u'eth0ip': u'<IP_ADDRESS>', u'template': u'domP'}, u'id': u'cmdline'}
2016-05-06 20:32:20,796 DEBUG    Loading data bag type ips
2016-05-06 20:32:20,796 DEBUG    Setting router to master
2016-05-06 20:32:20,796 INFO     Will proceed configuring device ==> eth2
2016-05-06 20:32:20,797 DEBUG    Executing: ip link set eth2 up
2016-05-06 20:32:20,800 INFO     Bringing public interface eth2 up
2016-05-06 20:32:20,800 INFO     Adding gateway ==> <IP_ADDRESS> to device ==> eth2
2016-05-06 20:32:20,801 INFO     Will proceed configuring device ==> eth1
2016-05-06 20:32:20,801 DEBUG    Executing: ip link set eth1 up
2016-05-06 20:32:20,804 INFO     Bringing public interface eth1 up
2016-05-06 20:32:20,804 INFO     Adding gateway ==> <IP_ADDRESS> to device ==> eth1
2016-05-06 20:32:20,804 INFO     Checking if default ipv4 route is present
2016-05-06 20:32:20,804 DEBUG    Executing: ip -4 route list 0/0
2016-05-06 20:32:20,807 WARNING  No default route found!
2016-05-06 20:32:20,807 INFO     Adding default route
2016-05-06 20:32:20,807 DEBUG    Executing: ip route show default via <IP_ADDRESS>
2016-05-06 20:32:20,810 INFO     Add default via <IP_ADDRESS>
2016-05-06 20:32:20,810 DEBUG    Executing: ip route add default via <IP_ADDRESS>
2016-05-06 20:32:20,812 DEBUG    Configuring static routes
2016-05-06 20:32:20,813 DEBUG    Loading data bag type staticroutes
2016-05-06 20:32:20,813 DEBUG    Processing CsStaticRoutes file ==> {u'id': u'staticroutes', u'<IP_ADDRESS>/16': {u'revoke': False, u'ip_address': u'<IP_ADDRESS>', u'gateway': u'<IP_ADDRESS>', u'network': u'<IP_ADDRESS>/16'}}
2016-05-06 20:32:20,813 DEBUG    Executing: ip route show | grep <IP_ADDRESS>/16 | awk '{print $1, $3}'
2016-05-06 20:32:20,818 DEBUG    Executing: ip route add <IP_ADDRESS>/16 via <IP_ADDRESS>
2016-05-06 20:32:20,821 DEBUG    Executing: /usr/sbin/conntrackd -C /etc/conntrackd/conntrackd.conf -c
2016-05-06 20:32:20,824 DEBUG    Executing: /usr/sbin/conntrackd -C /etc/conntrackd/conntrackd.conf -f
2016-05-06 20:32:20,828 DEBUG    Executing: /usr/sbin/conntrackd -C /etc/conntrackd/conntrackd.conf -R
2016-05-06 20:32:20,831 DEBUG    Executing: /usr/sbin/conntrackd -C /etc/conntrackd/conntrackd.conf -B

CI RESULTS
Tests Run: 85
  Skipped: 0
   Failed: 1
   Errors: 0
 Duration: 9h 06m 23s

Summary of the problem(s):
FAIL: Test redundant router internals
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/data/git/cs1/cloudstack/test/integration/smoke/test_routers_network_ops.py", line 478, in test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false
    result = check_router_command(virtual_machine, nat_rule.ipaddress, ssh_command, check_string, self)
  File "/data/git/cs1/cloudstack/test/integration/smoke/test_routers_network_ops.py", line 61, in check_router_command
    test_case.fail("Failed to SSH into the Virtual Machine: %s" % e)
AssertionError: Failed to SSH into the Virtual Machine: [Errno 110] Connection timed out
----------------------------------------------------------------------
Additional details in: /tmp/MarvinLogs/test_network_KAUTZA/results.txt

Associated Uploads
/tmp/MarvinLogs/DeployDataCenter__May_11_2016_00_29_07_UR1IBW:

dc_entries.obj
failed_plus_exceptions.txt
runinfo.txt

/tmp/MarvinLogs/test_network_KAUTZA:

failed_plus_exceptions.txt
results.txt
runinfo.txt

/tmp/MarvinLogs/test_vpc_routers_N2UD4H:

failed_plus_exceptions.txt
results.txt
runinfo.txt

Uploads will be available until 2016-07-11 02:00:00 +0200 CEST
Comment created by upr comment.

LGTM. Tested in a hardware lab.

@rafaelweingartner yes, it will be forward merged.  Sorry I only just got around to answering your question.  I need one more code review on this one.  Thanks...

LGTM.  We tested this in our Lab on HW and all static routes on each PG came up as expected.

tag:mergeready

Thanks gents.  I will get this merged...


ctrl-v virtual mode can not enable column edit.

In some previous version. ctrl-v can enable virtual mode with column edit.
Now it can't but enable a block select.  Am I something wrong?

Just came here to report the same issue. Visual block mode doesn't work with ctrl-v

This is a dupe of #468

Visual block mode doesn't work with ctrl-v
Same here.

As stated in the README, this package is no longer maintained and is deprecated. We recommend that people use the vim-mode-plus package instead. Because of this, we are archiving this repository and closing all issues and pull requests. Thanks very much for your support and contributions!


Add ranges for warnings

https://github.com/stylelint/stylelint/blob/main/CHANGELOG.md#1470

This is automatically handled by using word in reporter.


Dark Light mode

[x]  Now we can switch between light n dark mode. Also cache remembers the priority of user.

closing it, bcz deployment failed


⚠ Bofrost.at has degraded performance

In 427339f, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1471 ms

Resolved: Bofrost.at performance has improved in cdc2cc4.


[Misc.] Bump azdev

Description

Check if new azdev works
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Misc.

@bebound, you have done an outstanding job with great attention to detail.


Noindex withdrawn content for crawlers

We publish a sitemap with down-weighting for withdrawn content, however this does not seem to affect the ranking of external search results as much as we might hope.
By no-indexing here, we hope to surface current information more readily in places like google.
Should users wish to find withdrawn content, then it'll be findable via site search or navigation.
https://trello.com/c/YcXJ8OFD/258-noindex-withdrawn-content

Look good - do we need any tests?

We've seen that this is only affecting a few pages, so we're not going to do this now


A one-line open preview like notion

Problem

Solution
Hello, could you update a single line open preview feature like notion、thanks
Additional details
No response

@trey-wallis  Hello, have time to design this


Improvements

Remove pure and local
clean tests
add RemoveAt to list
Add EnumFromNative util class
Remember to Add/Update the:

[ ] Sample scripts
[ ] Test scripts

Coverage: 92.834% (-0.04%) from 92.879% when pulling 331defecef50c83537dc1c132a88daa9eecf8e76 on improvements into aecef0d706b803b5b3ad8261b549fa34dd3cc2b0 on main.


Bump: Sentry Cocoa to 7.0.3

:scroll: Description
Bump Sentry Cocoa to 7.0.3
:bulb: Motivation and Context
New Cocoa Release
:green_heart: How did you test it?
With the example app
:pencil: Checklist

[x] I reviewed submitted code
[ ] I added tests to verify changes
[ ] I updated the docs if needed
[x] All tests passing
[x] No breaking changes

:crystal_ball: Next steps

Codecov Report

Merging #445 (d1bc8a2) into main (d0a7dbb) will increase coverage by 3.95%.
The diff coverage is n/a.

:exclamation: Current head d1bc8a2 differs from pull request most recent head 6a5cec8. Consider uploading reports for the commit 6a5cec8 to get more accurate results

@@            Coverage Diff             @@
##             main     #445      +/-   ##
==========================================
+ Coverage   89.91%   93.86%   +3.95%
==========================================
  Files          54        5      -49
  Lines        1685      163    -1522
==========================================
- Hits         1515      153    -1362
+ Misses        170       10     -160

Impacted Files
Coverage Δ

dart/lib/src/protocol/sentry_device.dart

dart/lib/src/throwable_mechanism.dart

dart/lib/src/protocol/sentry_id.dart

dart/lib/src/transport/noop_transport.dart

dart/lib/src/protocol/sentry_operating_system.dart

dart/lib/src/scope.dart

dart/lib/src/protocol/contexts.dart

dart/lib/src/sentry_exception_factory.dart

dart/lib/src/hub_adapter.dart

dart/lib/src/noop_hub.dart

... and 39 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d0a7dbb...6a5cec8. Read the comment docs.


Provide a way to break out metrics by a TLS dimension

One requirement for the proxy privacy project is that the UI should be able to break out metrics according to whether TLS was used. For example, we should be able to see separate request latency and separate success rate numbers for a pod's TLS traffic and for its non-TLS traffic. The user might also prefer to see these combined into overall metrics (TLS + non-TLS).

How should this be done in the Web UI?
How should this be done in conduit stat?

We should design the UI early on while waiting for the underlying foundational TLS work to be done, since the UI design itself might be non-trivial, and because the changes to the web UI and/or conduit stat to implement such API might be non-trivial.
/cc @wmorgan, who might suggest more specific requirements. I've set the target milestone to 0.4 based on my understanding of the desired prioritization.

@wmorgan / @olix0r Is this issue solved by the SECURED column in the CLI output and in the web UI stat tables? Or is there additional design work that needs to happen for the TLS feature?

Yes, the design work in this issue is captured by the changes already in master to the CLI, Grafana, and web UI.

Great, thanks @wmorgan -- am going to close this.


Improved Java and C# coloring

I would like to suggest some improvements to the coloring. The changes only affect the Java and C# languages.
I changed the light-orange color of "ordinary" Java/C# source code to the default (gray). I also changed the non-primitive typenames to light orange (to distinguish them from primitives like int, float, etc.).
Maybe it is just my preference, but I also decided to remove the purple coloring from import statement package names in Java (as it was way too much purple in one place for me :-). I made it gray, but I think it could also be light-orange so it matches class name color.
I also corrected the using statement in C# (it was uncolored).
Comparison:
Before:

After:

@pgkos Thanks. Looks a lot cleaner.
Would it make sense to scope the additions only to Java and C# and move it down into the Languages part: https://github.com/pgkos/one-dark-syntax/blob/master/styles/language.less#L256-L319?
Or if it's hard to scope them and they would benefit other languages too, maybe only move https://github.com/pgkos/one-dark-syntax/blob/master/styles/language.less#L30-L33 and https://github.com/pgkos/one-dark-syntax/blob/master/styles/language.less#L49-L51
Just so that the top part stays lean and somewhat language "independent".

I hope it will work correctly! I haven't moved the .markup selector as I'm not sure if it is specific to just one language.

:+1: Thanks.

I haven't moved the .markup selector as I'm not sure if it is specific to just one language.

I'm actually not sure either where that gets used. Could still be moved at a later point.

Something I'm wondering.. should the same be done in One-light-syntax? Basically keep the "logic" the same and only change the color values. To make that easier, color variables probably would need to be generic.. like @color-1, @color-2 etc. so that when making changes to one theme, you could just replace the whole file in the other theme and not have to worry about the variables names.


compatibility release 0.8.0 for 2023.1 (231)

Updated from upstream, bumped version numbers, built and verified the updated version with IntelliJ IDEA EAP 2023.1 (ideaIU-231) today.

So the GitHub build pipeline failed, although my local build in an actual upcoming IDEA EAP IDE has been working just fine and produced a valid theme plugin that works and verifies in the editor. So what, GitHub: "Merging is blocked".

How to unblock this process?

Merged manually on the command line.


[SPARK-36039][KUBERNETES] Fix executor pod hadoop conf mount

What changes were proposed in this pull request?
Fix executor pod hadoop conf mount.

Why are the changes needed?
Arg --conf spark.kubernetes.hadoop.configMapName for executor pod not working.

Does this PR introduce any user-facing change?
No.

How was this patch tested?
UT.

Can one of the admins verify this patch?


Testing out core-reviewers PR notification

This is for testing out core-reviewers PR notification

@italovalcy @ajoaoff could you double check if you have received an notification to review this PR on your GitHub notification inbox? I've made a slight change in the core-reviewers notification settings.

@italovalcy @ajoaoff could you double check if you have received an notification to review this PR on your GitHub notification inbox? I've made a slight change in the core-reviewers notification settings.

Received! thanks!

@italovalcy @ajoaoff could you double check if you have received an notification to review this PR on your GitHub notification inbox? I've made a slight change in the core-reviewers notification settings.

Received! thanks!

Awesome. Let's keep his new configuration then, it's simpler to assign the entire group. Thanks for confirming.


feat: support policy-optimizer

Extend the RBAC roles assigned to the policy-server Service Account.
Grant access to the resources required by policy-optimizer.
This is required to implement this RFC.
More work has to be done, that's why this is a draft PR filed against a feature branch

Closing, we're not going to go after this implementation


[ECS] Fatal error after updating to 7.3

Hello. I'm using prefixed version of easy coding standard and I faced a problem after updating composer dependencies.
Stack trace:
> ecs check --ansi

Fatal error: Uncaught _HumbugBoxc7ce9d53454b\Symfony\Component\DependencyInjection\Exception\RuntimeException: Class "_HumbugBoxc7ce9d53454b\PHPStan\Rules\Rule" not found while loading "Symplify\CodingStandard\CognitiveComplexity\Rules\ClassLikeCognitiveComplexityRule". in phar:///var/www/html/vendor/symplify/easy-coding-standard-prefixed/ecs/vendor/symfony/dependency-injection/Compiler/DefinitionErrorExceptionPass.php on line 49

_HumbugBoxc7ce9d53454b\Symfony\Component\DependencyInjection\Exception\RuntimeException: Class "_HumbugBoxc7ce9d53454b\PHPStan\Rules\Rule" not found while loading "Symplify\CodingStandard\CognitiveComplexity\Rules\ClassLikeCognitiveComplexityRule". in phar:///var/www/html/vendor/symplify/easy-coding-standard-prefixed/ecs/vendor/symfony/dependency-injection/Compiler/DefinitionErrorExceptionPass.php on line 49

Call Stack:
    0.2019     399824   1. {main}() /var/www/html/vendor/symplify/easy-coding-standard-prefixed/ecs:0
    0.5406    2451024   2. require('phar:///var/www/html/vendor/symplify/easy-coding-standard-prefixed/ecs/bin/ecs') /var/www/html/vendor/symplify/easy-coding-standard-prefixed/ecs:6
    0.7012   10360096   3. Symplify\EasyCodingStandard\HttpKernel\EasyCodingStandardKernel->boot() phar:///var/www/html/vendor/symplify/easy-coding-standard-prefixed/ecs/bin/ecs:69
    0.7048   10468816   4. Symplify\EasyCodingStandard\HttpKernel\EasyCodingStandardKernel->initializeContainer() phar:///var/www/html/vendor/symplify/easy-coding-standard-prefixed/ecs/vendor/symfony/http-kernel/Kernel.php:113
    0.8444   13160016   5. _HumbugBoxc7ce9d53454b\Symfony\Component\DependencyInjection\ContainerBuilder->compile() phar:///var/www/html/vendor/symplify/easy-coding-standard-prefixed/ecs/vendor/symfony/http-kernel/Kernel.php:440
    0.9151   13169208   6. _HumbugBoxc7ce9d53454b\Symfony\Component\DependencyInjection\Compiler\Compiler->compile() phar:///var/www/html/vendor/symplify/easy-coding-standard-prefixed/ecs/vendor/symfony/dependency-injection/ContainerBuilder.php:620

This seems to be caused by #1881 because there is no this problem in 7.2.20

Prefixed version has not been build correctly yet, see failin build for 7.3.*:
https://github.com/symplify/symplify/runs/637384371

Try 7.3.2, just released

Last patch fix this issue. Thanks a lot for you job!

Great! You're welcome


running cert rotate with rke can result in a mixed version k8s cluster

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):

create a cluster using rke v0.1.8
./rke_v018 up --config cluster.yml
run certificate rotate on cluster
./rke_v027 cert rotate --rotate-ca

Result:
This results in the control plane of the nodes runing a different component version from the kubelets
CONTAINER ID        IMAGE                                  COMMAND                  CREATED              STATUS              PORTS               NAMES
bf79ceb8294a        rancher/hyperkube:v1.14.5-rancher1     "/opt/rke-tools/entr…"   29 seconds ago       Up 4 seconds                            kube-scheduler
5992e1d2ed96        rancher/hyperkube:v1.14.5-rancher1     "/opt/rke-tools/entr…"   36 seconds ago       Up 10 seconds                           kube-controller-manager
5eb67a33c829        rancher/hyperkube:v1.14.5-rancher1     "/opt/rke-tools/entr…"   45 seconds ago       Up 16 seconds                           kube-apiserver
331f0e3d3c0b        rancher/rke-tools:v0.1.40              "/opt/rke-tools/rke-…"   About a minute ago   Up About a minute                       etcd-rolling-snapshots
1ef394c18ba7        rancher/coreos-etcd:v3.3.10-rancher1   "/usr/local/bin/etcd…"   About a minute ago   Up About a minute                       etcd
f4fdb369b56d        rancher/hyperkube:v1.10.5-rancher1     "/opt/rke/entrypoint…"   16 minutes ago       Up 3 seconds                            kube-proxy
cd77ddf9477b        rancher/hyperkube:v1.10.5-rancher1     "/opt/rke/entrypoint…"   17 minutes ago       Up 3 seconds                            kubelet```

Executing rke up with the recent version will upgrade the cluster to a consistent version.


allow extension for all urls?

I am trying to allow the extension to work on all websites.
I modified the manifest.json (including the dev version) adding necessary permissions:
"permissions": [ "contextMenus", "management", "tabs", "storage", "https://*/*", "<all_urls>" ],
but it doesn't help, it won't work. Apparently it only works on github, and the matter is dealt with by this variable, which, would only work for github.com (tried replacing it with other websites, won't work)
const arrowURLs = ['^https://github\\.com'];
any clues?
Thanks,

I did this:
chrome.tabs.onUpdated.addListener(async (tabId, changeInfo, tab) => {
  if (changeInfo.status !== 'loading') {
    return;
  }

  const result = await isInjected(tabId);
  if (chrome.runtime.lastError || result[0]) return;

  loadScript('inject', tabId, () => console.log('load inject bundle success!'));
});

@nickweinberg Do you think you could expand on this? Not sure what this code does, and would really like to know the reasoning behind it!

Yeah you can modify chrome/extension/background/inject.js.

I got stuck as this did not work for me

I got stuck as this did not work for me

try to add this on background.js (i pass empty object and this helped me)
chrome.runtime.onInstalled.addListener(function () {         chrome.declarativeContent.onPageChanged.removeRules(undefined, function () {             chrome.declarativeContent.onPageChanged.addRules([{                 conditions: [new chrome.declarativeContent.PageStateMatcher({})],                 actions: [new chrome.declarativeContent.ShowPageAction()]                 }             ]);         });     });


problem in training

As you see after starting the training i have this erro can you help me to solve it please ???
299/300 [============================>.] - ETA: 0s - loss: 0.1770 - acc: 0.9238Epoch 00001: loss improved from inf to 0.17682, saving mod300/300 [==============================] - 80s 268ms/step - loss: 0.1768 - acc: 0.9239
14/14 [==============================] - 2s 124ms/step
/home/hala/anaconda3/envs/py35gpu/lib/python3.5/site-packages/skimage/io/_io.py:141: UserWarning: data/membrane/test/0_predict.png is a low contrast image
warn('%s is a low contrast image' % fname)
Lossy conversion from float32 to uint8. Range [0, 1]. Convert image to uint8 prior to saving to suppress this warning.
/home/hala/anaconda3/envs/py35gpu/lib/python3.5/site-packages/skimage/io/_io.py:141: UserWarning: data/membrane/test/1_predict.png is a low contrast image
warn('%s is a low contrast image' % fname)
Lossy conversion from float32 to uint8. Range [0, 1]. Convert image to uint8 prior to saving to suppress this warning.
/home/hala/anaconda3/envs/py35gpu/lib/python3.5/site-packages/skimage/io/_io.py:141: UserWarning: data/membrane/test/2_predict.png is a low contrast image
warn('%s is a low contrast image' % fname)
Lossy conversion from float32 to uint8. Range [0, 1]. Convert image to uint8 prior to saving to suppress this warning.
/home/hala/anaconda3/envs/py35gpu/lib/python3.5/site-packages/skimage/io/_io.py:141: UserWarning: data/membrane/test/3_predict.png is a low contrast image
warn('%s is a low contrast image' % fname)
Lossy conversion from float32 to uint8. Range [0, 1]. Convert image to uint8 prior to saving to suppress this warning.
/home/hala/anaconda3/envs/py35gpu/lib/python3.5/site-packages/skimage/io/_io.py:141: UserWarning: data/membrane/test/4_predict.png is a low contrast image
warn('%s is a low contrast image' % fname)
Lossy conversion from float32 to uint8. Range [0, 1]. Convert image to uint8 prior to saving to suppress this warning.
/home/hala/anaconda3/envs/py35gpu/lib/python3.5/site-packages/skimage/io/_io.py:141: UserWarning: data/membrane/test/5_predict.png is a low contrast image
warn('%s is a low contrast image' % fname)
Lossy conversion from float32 to uint8. Range [0, 1]. Convert image to uint8 prior to saving to suppress this warning.
/home/hala/anaconda3/envs/py35gpu/lib/python3.5/site-packages/skimage/io/_io.py:141: UserWarning: data/membrane/test/6_predict.png is a low contrast image
warn('%s is a low contrast image' % fname)
Lossy conversion from float32 to uint8. Range [0, 1]. Convert image to uint8 prior to saving to suppress this warning.
/home/hala/anaconda3/envs/py35gpu/lib/python3.5/site-packages/skimage/io/_io.py:141: UserWarning: data/membrane/test/7_predict.png is a low contrast image
warn('%s is a low contrast image' % fname)
Lossy conversion from float32 to uint8. Range [0, 1]. Convert image to uint8 prior to saving to suppress this warning.
/home/hala/anaconda3/envs/py35gpu/lib/python3.5/site-packages/skimage/io/_io.py:141: UserWarning: data/membrane/test/8_predict.png is a low contrast image
warn('%s is a low contrast image' % fname)
Lossy conversion from float32 to uint8. Range [0, 1]. Convert image to uint8 prior to saving to suppress this warning.
/home/hala/anaconda3/envs/py35gpu/lib/python3.5/site-packages/skimage/io/_io.py:141: UserWarning: data/membrane/test/9_predict.png is a low contrast image
warn('%s is a low contrast image' % fname)
Lossy conversion from float32 to uint8. Range [0, 1]. Convert image to uint8 prior to saving to suppress this warning.
/home/hala/anaconda3/envs/py35gpu/lib/python3.5/site-packages/skimage/io/_io.py:141: UserWarning: data/membrane/test/10_predict.png is a low contrast image
warn('%s is a low contrast image' % fname)
Lossy conversion from float32 to uint8. Range [0, 1]. Convert image to uint8 prior to saving to suppress this warning.
/home/hala/anaconda3/envs/py35gpu/lib/python3.5/site-packages/skimage/io/_io.py:141: UserWarning: data/membrane/test/11_predict.png is a low contrast image
warn('%s is a low contrast image' % fname)
Lossy conversion from float32 to uint8. Range [0, 1]. Convert image to uint8 prior to saving to suppress this warning.
/home/hala/anaconda3/envs/py35gpu/lib/python3.5/site-packages/skimage/io/_io.py:141: UserWarning: data/membrane/test/12_predict.png is a low contrast image
warn('%s is a low contrast image' % fname)
Lossy conversion from float32 to uint8. Range [0, 1]. Convert image to uint8 prior to saving to suppress this warning.
/home/hala/anaconda3/envs/py35gpu/lib/python3.5/site-packages/skimage/io/_io.py:141: UserWarning: data/membrane/test/13_predict.png is a low contrast image
warn('%s is a low contrast image' % fname)
Lossy conversion from float32 to uint8. Range [0, 1]. Convert image to uint8 prior to saving to suppress this warning.

I have solved the same problem using this method.


use defer to unlock mutex

kind cleanup

What is the reason for the change?
Defer is not free, albeit it has become cheaper with time. I really see no reason to change this unless there is a reason.

I think the original motivation of the author of the change here was
general code hygiene: 1) (on the current code) two lines of lock management
next to each other instead of three spread throughout the function is
cleaner 2) (on the future code) chances of forgetting to free the lock in a
possible later (not yet written) branch later is very low if it's already
done by the defer statement.
When it comes to performance, yes defer is more expensive. In this code, if
I'm reading the logic correctly, the pushError is not in a tight loop, but
called about once if error happens in a part where we shortly tear down the
pipeline because of it. Paying a one-off of defer when we take down
goroutines, close channels etc seems negligible and I would emphasize the
correctness and readability of it. That said, I'm not feeling strongly
about this (the old code is totally acceptable to my eye as well).
On Sun, Jun 20, 2021, 16:50 Klaus Post @.***> wrote:

What is the reason for the change?
Defer is not free, albeit it has become cheaper with time. I really see no
reason to change this unless there is a reason.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/klauspost/pgzip/pull/42#issuecomment-864566201, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AFSXSBZ7XY3BH5OSTPGISADTTX53PANCNFSM47ACF7EA
.

Toni Timonen

I think the original motivation of the author of the change here was general code hygiene: 1) (on the current code) two lines of lock management next to each other instead of three spread throughout the function is cleaner 2) (on the future code) chances of forgetting to free the lock in a possible later (not yet written) branch later is very low if it's already done by the defer statement. When it comes to performance, yes defer is more expensive. In this code, if I'm reading the logic correctly, the pushError is not in a tight loop, but called about once if error happens in a part where we shortly tear down the pipeline because of it. Paying a one-off of defer when we take down goroutines, close channels etc seems negligible and I would emphasize the correctness and readability of it. That said, I'm not feeling strongly about this (the old code is totally acceptable to my eye as well).
On Sun, Jun 20, 2021, 16:50 Klaus Post @.***> wrote: What is the reason for the change? Defer is not free, albeit it has become cheaper with time. I really see no reason to change this unless there is a reason. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#42 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFSXSBZ7XY3BH5OSTPGISADTTX53PANCNFSM47ACF7EA .
Toni Timonen

Yes, thanks Timonen. BTW, "effective Go" says "...It's an unusual but effective way to deal with situations such as resources that must be released regardless of which path a function takes to return. The canonical examples are unlocking a mutex or closing a file.".

Sure. Not changing this unless it fixes a problem.


cannot

mod lib has an error and won't install

Could you provide more details about the error ? The crash report file, game version, etc...

how to make a cake ?
idk just cook it or something

Once again this information is absolutely useless, I need you to reproduce the issue and send the actual error m


Update dev prereqs

@aileenzeng, I added a new subsection to README.MD called "Dev Prereqs"--can you help me fill this out?

Yup! I don't think we need to add anything else since everything else is installed in the Docker container. (Django is also included)
I threw in Git for good measure.

Great! Thanks.


Specify submodule when using orchest functions

Otherwise throws an error like: Error in py_get_attr_impl(x, name, silent): AttributeError: module 'orchest' has no attribute 'output_to_disk'

 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.

Thank you for fixing the R docs @andtheWings! Please don't forget to add yourself to the contributors section in the README, we would be happy to have you listed there :smiley:

Awesome fix!


Allow all BLE controllers to choose phy support

It is possible to consider adding BT_CTLR_2M_PHY_SUPPORT and BT_CTLR_CODED_PHY_SUPPORT helper configs.

@cvinayak , @carlescufi can we merge this PR?

@rugeGerritsen  we are in freeze for 2.2


PR | Sample data |

Description
implemented sample data of pins via a js file and exporting it.
Due to TS we also creatd the interface for the pins.
We restrucured the files to fit in the new global.d.ts
Card Link
https://3.basecamp.com/5802516/buckets/37608217/card_tables/cards/7440870365
Type of change
Please delete options that are not relevant.

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

How Has This Been Tested?

[x] consolelog tested

Checklist before requesting a review

[x] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

@J-Ariola Yes, my lord


rename debugging environment variables

I think these variables should be namespaced. I just went ahead and made the PR since it's so small rather than create an issue first.

DEBUG_COMMANDS → GIT_TOWN_DEBUG_COMMANDS
DEBUG → GIT_TOWN_DEBUG

@kevgo @charlierudolph @ricmatsui

Why do you think these variables should be namespaced? That makes them much harder to use in my opinion, and I don't see a scenario where there would be unintended conflicts.

True. The issue was if the environment variables were already defined in the shell by some other process or set in a *.rc file for use by another process, they would be consumed unintentionally by git town.
On second thought, this is probably more trouble than it's worth.


fix: memory leak in WebAudioSoundManager

Please do not update the README or Change Log, we will do this when we merge your PR.
This PR (delete as applicable)

Fixes a bug

Describe the changes below:
unlockHandler was not removed because this was not binded to bodyRemove function, causing serious memory leak. As you see performance debugger with chrome, event listeners count get continuously increased as user presses keyboard.
This fixes the above bug by binding document.body to bodyRemove function.

Merging, because I don't think this has any side effects, but will keep an eye on it.


Error: BrowserModule has already been loaded.

I updated to the new version 1.5.0
And I'm getting this error
Error: BrowserModule has already been loaded. If you need access to common directives such as NgIf and NgFor a lazy loaded module, import CommonModule instead.

I'm using the BrowserAnimationsModule in the AppModule to animate my routes, I also have a SharedModule where I export the MaterializeModule. It would look something like this angular/angular#15829
In the previous version everything worked
Versions
@angular/cli: 1.2.2
node: 8.1.3
os: win32 x64
@angular/animations: 4.3.1
@angular/common: 4.3.1
@angular/compiler: 4.3.1
@angular/core: 4.3.1
@angular/forms: 4.3.1
@angular/http: 4.3.1
@angular/platform-browser: 4.3.1
@angular/platform-browser-dynamic: 4.3.1
@angular/platform-server: 4.3.1
@angular/router: 4.3.1
@angular/cli: 1.2.2
@angular/compiler-cli: 4.3.1
@angular/language-service: 4.3.1

Thank you for the excellent abstraction of the materialize css that you have been carrying out

We have the same error on our internal application. So, I am investigating on it.

You can update your ng2-materialize version to 1.5.1 in your application.


[dev-tool] Test alternative method of executing samples

Packages impacted by this PR

@azure-tools/dev-tool

Issues associated with this PR

NA

Describe the problem that is addressed by this PR
This PR changes the samples execution logic to run the published TypeScript samples instead of the samples in samples-dev. It runs these samples by actually installing the SDK package (using a relative path) and building the TypeScript. I am hoping this will

Fix the broken samples run step for ESM-migrated packages
Make the samples running step a bit more thorough (i.e. we are now checking if the samples will run against something that is much closer to the published package).

/azp run js - keyvault-certificates - tests

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

/azp run js - keyvault-certificates - tests

/azp run js - keyvault-certificates - tests

/azp run js - keyvault-certificates - tests


Fix Docs spacing/indentation and RFC3339 time

What does this PR (Pull Request) do?

[x] This PR is not related to any Issue

This PR fixes mixed tabs and spaces in indention, as well as instances of using tabs in alignment within the documentation.
This PR also changes several places in the API documentation that referred to dates/times being formatted according to RFC3339 that, in fact, use the custom lib/go-tc.TimeLayout format ("2006-01-02 15:04:05-07"). These times are now referred to as being "ISO-like".
Which Traffic Control components are affected by this PR?

Documentation

What is the best way to verify this PR?
Make sure documentation builds with no errors or warnings, ensure indentation and alignment are consistent and in accordance with the relevant documentation guidelines. Ensure all date/times that are not presented in RFC3339 format are not documented as though they were, and that those that are are correctly documented as being RFC3339.
If this is a bug fix, what versions of Traffic Control are affected?

master
5.x probably
4.x probably

The following criteria are ALL met by this PR

[x] Tests are unnecessary for documentation changes
[x] This PR includes documentation
[x] An update to CHANGELOG.md is not necessary
[x] This PR includes any and all required license headers
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY

"An ISO-like format" is vague and does not accomplish the purpose of documentation, in this case, which is to allow a user to anticipate the date format.

"an ISO-like format" is consistent with how that format is described throughout most of the documentation, though I agree it's overly vague.

The whitespace updates all look fine, so if you want, if you remove the RFC3339 time commit from the PR and throw it in a separate one for more discussion, I'll gladly merge the whitespace fixes.

Both changes ought to be done, so I don't really care much about splitting them up, unless you're saying I should do that.
Here's my first thought on improving that: change "an ISO-like format" to a link to a label in the API index section that describes the custom format in more detail, and also probably links to the format's GoDoc on go.pkg.dev.

Both changes ought to be done, so I don't really care much about splitting them up, unless you're saying I should do that.

Oh, splitting would only be if you want to avoid unnecessary merge conflicts if something else is merged before the RCF3339 part is merged, keeping them together works for me.

Here's my first thought on improving that: change "an ISO-like format" to a link to a label in the API index section that describes the custom format in more detail, and also probably links to the format's GoDoc on go.pkg.dev.

The links sound good. I think providing a term to describe it would still be helpful, even if it means inventing one.


Some APIs do not have terminal response definition - Added code to fix it.

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[ ] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] Validation errors from the Linter extension for VS Code have all been fixed for this spec. (Note: for large, previously checked in specs, there will likely be many errors shown. Please contact our team so we can set a timeframe for fixing these errors if your PR is not going to address them).
[x] The spec follows the patterns described in the Swagger good patterns document unless the service API makes this impossible.

Related to issue: https://github.com/Azure/azure-sdk-for-ruby/issues/566
@amarzavery @veronicagg @vishrutshah @salameer Please review.

Hi @sarangan12, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (sarajama). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@sarangan12 - Can you please resolve the comments made in the PR?

@amarzavery Can you please review the updated swagger?

@amarzavery
Snapshots attached to indicate the output as a result of this change.

 https://github.com/Azure/azure-sdk-for-python/pull/958

 https://github.com/Azure/azure-sdk-for-ruby/pull/657

 https://github.com/Azure/azure-sdk-for-node/pull/2029


trouble running OpenGL applications from nixpkgs on Ubuntu 16.04

Issue description
After nix-env -i glxinfo, I attempted to run glxinfo and glxgears:
(DevContainer)glxgears
glxgears: error while loading shared libraries: libXext.so.6: cannot open shared object file: No such file or directory
(DevContainer)glxinfo
glxinfo: error while loading shared libraries: libXext.so.6: cannot open shared object file: No such file or directory

I attempted to correct this by doing nix-env -i libXext, but the same error message results after successful installation of libXext. I'm not very familiar with how library loading works with nixpkgs, so maybe this is a user error.
Technical details

System: (NixOS: nixos-version, Ubuntu/Fedora: lsb_release -a, ...)

(DevContainer)cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=16.04
DISTRIB_CODENAME=xenial
DISTRIB_DESCRIPTION="Ubuntu 16.04.3 LTS"

Nix version: (run nix-env --version)

nix-env (Nix) 1.11.14

Nixpkgs version: (run nix-instantiate --eval '<nixpkgs>' -A lib.nixpkgsVersion)

"18.03pre115884.b5a5d0ba84"

Sandboxing enabled: (run grep build-use-sandbox /etc/nix/nix.conf)

(DevContainer)grep build-use-sandbox /etc/nix/nix.conf
grep: /etc/nix/nix.conf: No such file or directory

On NixOS opengl drivers are located at /run/opengl-driver I expect many packages expect to load stuff from there.

Probably related: https://github.com/NixOS/nixpkgs/issues/9415

Right, better keep OpenGL on non-NixOS in that thread.


Complete MediaStreamTrack with Spec

see comment in commit:
this PR:
close #51
close #48
close #28
Complete MediaTrack member and method

pass correspond properties to js when create track.

implement track.enabled = true/false which can be used to mute a track.

MediaStream.removeTrack() will invoke native method now.

implement track.stop()
maybe unnecessary to invoke it if we explicitly release media stream.
but some implementation would use it when hangup.
this also supported in chrome and firfox.
I'm not exactly understand what this means, but here are some hints:
Google Developers - MediaStream Deprecations
W3C track.stop() procedure
W3C detached meaning
what exaclty detached means in doc?

NOTICE:
I'm confusing about what's the differences of behavior between MediaStream.removeTrack() and MediaStreamTrack.stop() ?
Currently my understanding is, MediaStream.removeTrack() will invoke native one removeTrack, thus the track will remove from stream. In the webrtc java source code, it iterate all tracks from mediaStream and dispose() ( aka free() ) each track.
MediaStreamTrack.stop() is just stop the track itself, will not be removed from mediaStream.
in short: both track.stop() and mediaStream.removeTrack(track) will STOP playing the track, and set enable = false;
if someone have a clear understanding, please fix those actions to consistent with the spec. thanks!
Error Handling Object and Event
[js] add MediaStreamError Event type
this is just added the type object at the very first stage and nothing
todo with currently code base for now.
for future compatible with spec. maybe when we started to implement
constrainable pattern interface
and
error
handling

OK! Thanks.
I will spend some time implementing iOS part. Or any kind person would like to do it?

haha, I will look into it.
Will ping you if I encounter any trouble.

I added some of the ios code here : https://github.com/oney/react-native-webrtc/pull/51
I only had time to do what was necessary for my project (mediatrack setEnabled method). So it might be best to just use the objective c code I have and merge it into this architecture here.
Cheers,

I updated the PR, commit together with @dguillamot
please see the NOTICE in the PR comments.

@oney
I don't know how to get readyState on ios. ( method not found, btw it's readonly according to spec )
so hard-coded as "live" when we got the track first time.
since we don't use readyState for now so it won't break or hurt anything.

Awesome PR! I will test it later.


Add OpenSRP2 Web documentation to docusite

IMPORTANT: Where possible all PRs must be linked to a Github issue
Fixes #3082
Engineer Checklist

[ ] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[ ] I have added any strings visible on UI components to the strings.xml file
[ ] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[ ] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[ ] I have built and run the FHIRCore app to verify my change fixes the issue and/or does not break the app
[ ] I have checked that this PR does NOT introduce breaking changes that require an update to Content and/or Configs? If it does add a sample here or a link to exactly what changes need to be made to the content.

Code Reviewer Checklist

[ ] I have verified Unit tests have been written for any new feature(s) and edge cases
[ ] I have verified any strings visible on UI components are in the strings.xml file
[ ] I have verifed the CHANGELOG.md file has any notable changes to the codebase
[ ] I have verified the solution has been implemented in a configurable and generic way for reuseable components
[ ] I have built and run the FHIRCore app to verify the change fixes the issue and/or does not break the app

is what's in this PR the features? Is the user guide still on the confluent wiki?

I will merge this for now, we can add more later


Can WebGLNode share a canvas with renderer:webgl?

I'm not familiar with how scenery manages canvases.  Can WebGLNode share a canvas with renderer:webgl?  Likewise, can CanvasNode share a canvas with renderer:canvas?

Yes on both accounts. WebGLNode creates a WebGL drawable and likewise for CanvasNode.

Let me know if you need any more information, or if there's anything to do with this issue (close if not).

Question answered, thanks!  Closing.


chore(main): release 0.1.10

:robot: I have created a release beep boop
0.1.10 (2023-11-23)
Bug Fixes

log formatting + add disallow override file option to config (544767e)
magerun command now handles config override correctly (66eb5ea)
small fix for graceful interupts + some whitespacing fixes (229a649)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/furan917/MageComm/releases/tag/v0.1.10 :sunflower:


Recent Files & Folders

Feature
Store recently opened files & folders in db, and let user reopen them

Implemented in #223


Rename DescribeTableMergedResult to EncryptColumnsMergedResult

Pull Request Test Coverage Report for Build 22

0 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 58.201%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

encrypt-core/encrypt-core-merge/src/main/java/org/apache/shardingsphere/encrypt/merge/dal/EncryptDALResultDecorator.java
0
3
0.0%

Totals

Change from base Build 956:
0.02%

Covered Lines:
10546

Relevant Lines:
18120

💛  - Coveralls