Merge featureLayerProvider geometries in the results

I have a case where searching for certain ZIP codes returns 2 separate polygons (because the ZIP code spans 2 counties). I need to be able to merge them to return a single polygon. Before I switched to this geocoder I returned the features and used L.esri.Util.arcgisToGeojson to push the geometries into a feature collection and then merged them using turf.js.
getCityZipBoundary: function(data){
     var df = $q.defer();
     var features = data.features;
     var featureCollection = {
         type: 'featureCollection',
         features: []
     };
     for (var i = features.length - 1; i >= 0; i--) {
         var feature = L.esri.Util.arcgisToGeojson(features[i]);
         featureCollection.features.push(feature);
     }
         var mergedPolygon = turf.merge(featureCollection);
         df.resolve(mergedPolygon);
         return df.promise;
}

Now that I'm using this geocoder plugin I'm not sure how I would be able to merge the geometries.
What I was going to try next but I'm extremely hesitant:
It looks like I would need to add code to the suggestions function that checks to see if the input text matches suggestions.text and if they do then change the magicKey value to null.
suggestions: function(text, bounds, callback) {
            var query = this.query().where(this._buildQuery(text)).returnGeometry(true);
            if (bounds) {
                query.intersects(bounds)
            }
            if (this.options.idField) {
                query.fields([this.options.idField].concat(this.options.searchFields))
            }
            var request = query.run(function(error, results, raw) {
                if (error) {
                    callback(error, [])
                } else {
                    this.options.idField = raw.objectIdFieldName;
                    var suggestions = [];
                    var count = Math.min(results.features.length, this.options.maxResults);
                    for (var i = 0; i < count; i++) {
                        var feature = results.features[i];
                        suggestions.push({
                            text: this.options.formatSuggestion.call(this, feature),
                            magicKey: feature.id
                        })
                    }
                    <<<Insert code here to change magicKey value to null?>>>
                    callback(error, suggestions.slice(0, this.options.maxResults).reverse())
                }
            }, this);
            return request
        }

Then, in results function, since you wouldn't have a key it would make a generic search and then I could merge the geometries the way I did before.
results: function(text, key, bounds, callback) {
            var query = this.query();
            if (key) {
                query.featureIds([key])
            } else {
                query.where(this._buildQuery(text))
            }
            if (bounds) {
                query.within(bounds)
            }
            return query.run(L.Util.bind(function(error, features) {
                var results = [];
                <<<Insert code here to merge features?>>>
                for (var i = 0; i < features.features.length; i++) {
                    var feature = features.features[i];
                    if (feature) {
                        var bounds = this._featureBounds(feature);
                        var result = {
                            latlng: bounds.getCenter(),
                            bounds: bounds,
                            text: this.options.formatSuggestion.call(this, feature),
                            properties: feature.properties,
                            geojson: feature
                        };
                        results.push(result)
                    }
                }
                callback(error, results)
            }, this))
        },

Am I on the right track or is this a terrible idea or am I completely missing the obvious solution?

thanks for taking the time to outline your goal and progress so thoroughly.
it sounds like you are saying that when you see duplicate search results listed as suggestions you'd like to have access to the entire collection after making a selection.  is that correct?
its just a gut reaction, but i think we should probably make sure no duplicates appear in the list of suggestions to begin with.

@AnnieGitUrGun i'm happy to try and provide assistance with your issue.  are you still interested and available to answer my previous question, troubleshoot and discuss further?

Apologies for not following up sooner.
Well, these aren't exactly duplicates but in a list they would appear as such. Let me give you a concrete example. In the example below I've searched for White Hall.

The White Hall ZIP code spans 2 counties- Baltimore and Harford. The picture below shows the ZIP code in its entirety (and also what I was able to do before switching to this geocoder).

Unfortunately, the data provider decided to clip the ZIP codes to the county boundaries (which makes no sense but that's what I'm stuck with for now). This is why there are two entries for White Hall, MD 21161. If I select one or the other I will only get one half of the ZIP code when I need to get the whole thing:

So in my initial question I was trying to figure out how I could merge the suggestions and the geometries.
Thanks for your help!

thanks for confirming that my understanding of the workflow was correct.
the first (and best, solution) would be to post process the data you purchased prior to publishing your service.  all you'd need is a quick dissolve.
if this isn't an option for you, i feel like the geocoder should be smart enough to not display list items if the formatted text is identical to an item that has already been added.
once an item is selected by the user, all results with identical formatted text should be passed through in the results.
you are seeing the problem when using the featureLayerProvider, correct?
can you make a sample service publicly available for me to test with (even if it only includes a small subset of the original data)?

Correct. Below is the service I'm hitting:
http://geodata.md.gov/imap/rest/services/Boundaries/MD_PoliticalBoundaries/FeatureServer/3

i've proposed a fix to the geosearch control.  would you be kind enough to test it out and confirm that it solves the problem at hand?

I've tested and can confirm it solves my problem. Thank you!

awesome.  thanks!


fix: incorrect go to def invocations & go to def for both imported and modules in js block

This PR:

[x] Fixes: incorrect go to definition invocations as reported in https://github.com/ashish10alex/vscode-dataform-tools/pull/51

[x] Feat: Adds support for go to def when a module is clicked. Takes user to js block if module is defined there. Otherwise takes them to the file where module is defined

https://github.com/user-attachments/assets/83c92965-9045-4407-a6b0-39014a5effc6

Nice solution @ashish10alex! This looks good to me :-)
Just a heads up, is this PR supposed to go into main?
While testing i found another case where we would not get any defintions since the regex wont match, for example this:
  const constants = {
    VAR: true,
    TABLE: 'table2'
  }
}

with table1 as (
  select 1 as id
)

SELECT
  1 AS COL,
FROM table1
${when(constants.JOIN, `left join ${constants.TABLE} on ${constants.TABLE}.id = table1.id`)},

But i suppose this is not that common and can probably be fixed at a later time to not hold up on the release.

thanks , I forgot the change branch while creating the PR.
Yes, there will be definitely be mare other edge cases. Javascript is a wild beast. For now, the implementation has good coverage for the most common uses cases so lets go for it

Sounds like a good plan


Back restore operator

Tracker for https://github.com/rancher/backup-restore-operator/pull/277

It is not closed yet

Finally it was merged

covered in rancher/elemental-operator#9


Restore some plumbing capabilities.

"plumber" wasn't built by default, "plumb" now gets a build file,
added plumber to the harvey profile invocation, and restored the
overzealously-removed /rc/bin/B
Signed-off-by: John Floren<EMAIL_ADDRESS>

LGTM
let's make this system usable again.

You already have an LGTM and I'm saved from urticary :-)


Dealing with non-root $PWD

cd ./example
mkdir tmp
cd ./tmp
nix run

This creates the data diretory in ./example/tmp/data (instead of ./example/data as one would expect -- ie. in flake root). What is the best way to address this? In nammayatri we use flake-root + mission-control module to side-step the problem.
One possible solution here is to have services-flake require the use of flake-root module, and have the data directory use the flake root directory determined by the latter module.

In some cases (where you are providing services to external users to be run via, say, nix run github:user/repo#blah), there is no such concept of flake root - so we should respect $PWD in these cases.
If we are using flake-root to solve this, then there should be a way to disable it, such as via a cdToProjectRoot option (cf. https://github.com/Platonic-Systems/mission-control/pull/29).

How about resolving the dir in the following order:

if PRJ_ROOT is set, use that
otherwise try flake root
finally fallback on $PWD

If we are using flake-root to solve this, then there should be a way to disable it, such as via a cdToProjectRoot

:+1:

I'm happy to add support for setting PRJ_ROOT in github:srid/flake-root if that env var is widely used. So, your sequence is narrowed down to (1) and (3) only.
The problem with resolving dir in an order like this is that the behaviour is no longer deterministic within the process-compose configuration. If you use flake-root, it behaves one way. If you don't, the same config.process-compose.<name> behaves differently. That's a bit of magic I'd like to avoid, and instead go for an explicit and local approach.

Maybe this should be solved upstream, in process-compose-flake. See https://github.com/Platonic-Systems/process-compose-flake/pull/29 by @MrFoxPro

Is this really a problem that's worth solving in process-compose-flake / services-flake? I'm getting convince that we should not. And instead document this, pointing users towards using mission-control + flake-root for a devShell experience.


Is -f supposed to be ignored with --diff-against?

The following command creates mame0234_1.dat instead of 210to234.dat.
SabreTools.exe -ud --diff-against --base-dat="mame0210.dat" -ot=cmp -f="210to234.dat" mame0234.dat
Is it supposed to ignore the output filename like that?

Yes, this was originally intentional that it doesn't fully function for base replacement or diff against functionality. I'll have to think about how to best allow that, since I want to ensure that the output DATs don't get odd.

I think it might indeed be problematic with various dats being used. @mnadareski would it be possible to port the code from Logiqx's MAMEDiff to a single argument (eg. --mamediff) that could only be used with a pair of dats, thus allowing -f for the output result.

I'm probably not going to be porting an entirely extra set of code just for a 2-DAT case. I'm fairly certain I can figure out a better way to handle this internally.


osk.TETR.IO version 8.0.0

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

A better PackageIdentifier would be osk.tetr or ...


Support Enum for specifying return type

Found out currently we don't have a way to set the return type for Enum type. We would like to display a message when a wrong type is inserted to the enum property.
For example from PowerApps:

Related to #1125.  May need to get more details here.

Closing the issue since I cannot reproduce the behavior in PowerApps :)


chore(release): prepare for release

Description
Update for android_alarm_manager_plus based on #2864

We have the problem with android_alarm_manager_plus and android_intent_plus version resolution in release PRs for quite some time. I haven't found the reason yet on why melos bootstrap doesn't work properly for these 2 plugins.


ODC-7466: Fix design issue as per the UX design

Story: https://issues.redhat.com/browse/ODC-7466
Screenshots:
--Before---

--After---

/cc @jerolimov

Loader should be align with the title. Needs to add left padding
Resolved it

Can we make the Number of PipelineRuns chart to cover the 100% width of the card when it move to next line on small screen.
I looked into it. If I don't specify the width, svg width is getting set to 450 and this width is not getting changed even if parent div width increases and there is no separate class defined for svg to manually set the width based on screen size. I will take a look at it again. Please guide here if you have any idea.

Column sort indicator arrow color should be light check the other tables in console.
Class and the values are proper but PF variable is not getting added. May be if we upgrade the PF this issue will be resolved.

/retest

/lgtm


[spawn-web] modernize web dev environment

Creates a more modern development environment for web UI work:

moves source assets into a new hydra-web/ folder
builds assets to the existing hydra-main/web folders using webpack
build step is now required to update web-ui
enables newer JS language features (es6, es7)

And as a cheap way to fix a weird AMD/webpack incompatability w.r.t. requesting
data from the server:

streaming processing of the initial /setup/update response with Oboe

Please update NOTICE if these projects are no longer being vendored.

Nothing was removed, things were just put in different places. A lot of the 'vendor' things are now listed in package.json.

If we are not including the files verbatim, distributed them, or otherwise doing something that would trigger an attribution requirement, we don't need to put them in NOTICE.  Stuff that a build tool downloads or is required at runtime (like the transitive graph of all maven dependencies) does not need to be there. Keeping of track of dependencies in a flat there is a pain, so making it close to the minimal required set eases maintenance.  From the 20 lines added and 14k removed I think this commit is removing the verbatim copies from source control, and using a modern build tool to assemble the web dependencies.


[Migrated] Refactor Django/Flask-specific behaviors into separate modules

Originally from: https://github.com/Miserlou/Zappa/issues/1318 by rgov
It's really powerful that Django and Flask apps are automatically detected and adapted to work on AWS out of the box.
The functionality for this is woven throughout core.py, cli.py, and utilities.py which increases complexity for testing, debugging, adding new features, etc.
An improved design would probably involve refactoring the behaviors into separate modules that are less strongly-coupled. With a defined interface for how to tune Zappa for different frameworks, it would be easier to add support for others. Plus, the Zappa core and the framework plugins could be tested separately.

PRs welcome


initializeDateFormatting('es_ES', null) filePath can't be null

i'm trying to initializaDateFormatting but the filePath can't be null, i'm using null safety. I tried with ' ' in filePath but i have this error.

FileSystemException (FileSystemException: Cannot open file, path = 'symbols/es_ES.json' (OS Error: No such file or directory, errno = 2))

anybody can help me to fix this error? thanks

code is broken suddenly. is there a fix to this????

Also, as pointed out by https://stackoverflow.com/q/68882861/, the README.md file has an outdated example that calls initializeDateFormatting with a null filePath.

The DateFormat documentation also contains multiple examples that pass null.

CC @natebosch to try to get eyes on this.

This package has multiple methods named initializeDateFormatting. If you import the one in date_symbol_data_file.dart then you'd hit this error. The example in the README uses date_symbol_data_file.dart which has an optional second argument so that looks like it could be correct.
It does look like the DateFormat documentation passes a null for all 3 examples, and likely at least 2 of those will need updates. PRs welcome.

Ah.  You mean that the version from date_symbol_data_local.dart has optional parameters.  That's a bit confusing; there's no point for the _local.dart version to have the optional, ignored parameter when it's incompatible with the signatures from the other versions. =/


feat(cli): add namespace info to mesh list cmd

Resolves #3295.
Signed-off-by: Johnson Shi<EMAIL_ADDRESS>
Description:

Affected area:

New Functionality      [X]
Documentation          [ ]
Install                [ ]
Control Plane          [ ]
CLI Tool               [X]
Certificate Management [ ]
Networking             [ ]
Metrics                [ ]
SMI Policy             [ ]
Security               [ ]
Tests                  [ ]
CI System              [ ]
Demo                   [ ]
Performance            [ ]
Other                  [ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?

Codecov Report

Merging #3334 (771115d) into main (91de38d) will increase coverage by 0.09%.
The diff coverage is 25.00%.

@@            Coverage Diff             @@
##             main    #3334      +/-   ##
==========================================
+ Coverage   66.68%   66.77%   +0.09%
==========================================
  Files         166      166
  Lines        7979     7989      +10
==========================================
+ Hits         5321     5335      +14
+ Misses       2625     2621       -4
  Partials       33       33

Flag
Coverage Δ

unittests
66.77% <25.00%> (+0.09%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

cmd/cli/mesh_list.go
26.58% <0.00%> (-1.42%)
:arrow_down:

cmd/cli/namespace_list.go
50.94% <40.00%> (-6.51%)
:arrow_down:

pkg/configurator/validating_webhook.go
81.81% <0.00%> (+8.48%)
:arrow_up:

Continue to review full report at Codecov.

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


Added User-Presence +UPDATES on Stories, etc.

About [Added User-Presence / Online-Status] More Info At #370

Tests&More-Infos probably coming soon on comments [...]

Important
⚠ Updates are a Fast forward implementation
⚠ Updates are not Fully Tested yet
⚠ I am not a profesional programmer yet
⚠ I am new to java
💛 Thanks you for understanding
Updated Files

InstagramItem.java

filter_type
image_versions2

StoryPollItem.java

Added Files

@GiorgosXou
Hello! I think majority of changes to Stories here have been addressed by #352

@jvogit
Hello!
InstagramGetUserReelMediaFeedRequest request = new InstagramGetUserReelMediaFeedRequest(292864121);
InstagramGetUserReelMediaFeedResult result = instagram.sendRequest(request);

am i wrong or the request returns  InstagramtUserReelMediaFeedResult instead of InstagramGetUserReelMediaFeedResult ?

@jvogit
Hello!
From InstagramGetUserReelMediaFeedRequest.java:
...
    public InstagramUserReelMediaFeedResult parseResult(int resultCode, String content) {
        return parseJson(resultCode, content, InstagramUserReelMediaFeedResult.class);
    }

From #352:
...
InstagramGetUserReelMediaFeedRequest request = new InstagramGetUserReelMediaFeedRequest(292864121);
InstagramGetUserReelMediaFeedResult result = instagram.sendRequest(request);

am i wrong or the request returns InstagramtUserReelMediaFeedResult instead of InstagramGetUserReelMediaFeedResult ?
Also forgive me but i am kind of confused, because i haven't found any way to get most of the things that i've added in InstagramItem.java class with any other way [...]

It returns InstagramUserReelMediaResult . It was outdated example that I'll update now!
What things specifically you need in InstagramItem?

the issue seems to be with InstagramGetUserReelMediaFeedRequest.java and not with your example because InstagramtUserReelMediaFeedResult has nothing compared to InstagramtUserReelMediaFeedResult

the issue seems to be with InstagramGetUserReelMediaFeedRequest.java and not with your example because InstagramtUserReelMediaFeedResult has nothing compared to InstagramGetUserReelMediaFeedResult
(except now that i've added pretty much everything into InstagramtUserReelMediaFeedResult [...])

What things specifically you need in InstagramItem?

story_polls , story_sliders, story_questions, story_poll_voter_infos, story_slider_voter_infos, story_question_responder_infos , etc. whitch i've added in InstagramtUserReelMediaFeedResult

InstagramUserReelMediaFeedResult is returned. It was changed from InstagramGetUserReelMediaFeedResult .
The changes are present in InstagramStoryItem

i am so confused, so your request wasn't aprroved yet so it wasn't in the main-master branch?

Omg i did all  this work and now i see that it was just this InstagramUserReelMediaFeedResult

Nop , i am again confused, anyways , thanks

@jvogit
I am really sory but just a Last question, what do i have to do to get the changes you have made? (because if i am right they are not merged yet)

@jvogit
I am really sorry but just a Last question, what do i have to do to get the changes you have made? (because if i am right they are not merged yet)
Do i have to manually download and copy paste them or is there any faster way?
(i am going AFK, to sleep)

I believe you can git fetch (my branch for stories) then merge them into your repository if you want the changes. You should do this in another separate branch.

Addressed in rewrite or no longer relevant.


Serde support?

Would be nice to be able to serialize the models. E.g. for displaying purposes.

Yes. That makes an awful lot of sense. I'll add a serde feature. :+1:

Sweet! 😀

I just added serialization support and was wondering if I should use the original Sunspec identifiers for the JSON format or stick with the "rustified" names?
Sunspec
{
    "AID": "...",
    "StVnd": "...",
    "Evt": "GROUND_FAULT | TEST_FAILED",
    "Ctl": "AUTOMATIC",
}

Rust
{
    "aid": "...",
    "st_vnd": "...",
    "evt": "GroundFault | TestFailed",
    "ctl": "Automatic",
}

Personally I prefer the rust style as it's much easier on the eyes and it's just the "default" serialization of an already rustified model definition.
Do you have actual plans for using this?

The bitflags format works in both directions. It's just that it's stringly typed inside the JSON and parsing it with a different language will be a bit annoying. As there are some vendor specific things inside those flags I think it's best to just use it like that and accept the default serialization format bitflags uses.

Okay good! I agree with you that it's not optimal. Feels a bit dirty. But since that's bitflags default approach I think it's best to follow that approach.

Renaming struct fields and enum variants was a no-brainer. Though the bitflags don't seam to support renaming.
In the mean time I'll add serialization with "rustified" names. If anyone needs a JSON format with the actual SunSpec identifiers we need to decide how to serialize Enums and Bitfields. The string format is probably not the best to begin with.

Sounds like a good start!
I would love to see a new release with this as well as tokio-modbus 0.11 support!

I just released the sunspec 0.4.0 crate on crates.io:

https://crates.io/crates/sunspec/0.4.0


Explicit parse.ts types

Context
Currently, there are a few inline types that make using typescript casting hard and verbose. For example,
type RawTransactionDescription =
    | {
        type: 'generic',
        ...
        computePhase: RawComputePhase,
        ...
    }
    | {
        type: 'storage',
        ....
    }
    | {
        type: 'tick-tock',
        ...
    };

When working with typescript, in order to be able to reach the computePhase property, we have to cast to the full type description.
This PR
This change adds explicit types for each such that:
export type RawTransactionDescription =
  | GenericTransactionDescription
  | StorageTransactionDescription
  | TickTockTransactionDescription

...

Which will make casting in typescript much easier.

Actually, I wonder if we're willing to make a breaking change and replace Raw... types with union types directly.
Otherwise, it's when a function returns RawTransactionDescription it's not easy to know whether it in fact translates to multiple types.

@shaharyakir can you please remove changes to package.json? It will go to the next release

@dvlkv done

And CHANGELOG also as well


New UI Widget: The UI Link Widget

I am opening this Issue in hope that someone will create a new UI Widget: A Link.
The UILink will be a widget used when people need to input a hyperlink. The widget may have fields similar to UIText. The only extra field shall be a "URL" or "link" field which can be used to enter hyperlinks (HTTP, HTTPS) to pages (Web pages to begin with, but later probably even in-game descriptive pages). Another optional field could be "image". One could use this field to set an image which, if clicked, would lead to a web page or an in-game page.
When seen, the link could be of a different color (preferably blue since that is usually the color of links) and if clicked, would lead to the hyperlink or in-game screen in the "URL" or "link" field.
The need for such a widget arose when I was creating an InformativeLinksScreen interface for the user. If you wish to share links like Terasology's main page, Github page, facebook page, or twitter page, it is recommended that the link you share can be clicked to lead to the actual page. If that is not the case, the amount of people who actually use that link merely out of curiosity drastically reduces due to the inconvenience.
Other than the example shared above, there are a multitude of places where the UILink widget would come in handy. As I mentioned before, in-game links may be helpful. For example, if people wish to update the "help" command to open a page with "links" to the various in-game descriptive pages, that would improve Terasology's UI interface.
Some APIs that this widget may use are:

createLink()
getLink()
setColor()
setLinkText()
getLinkText()
setLinkImage()
getLinkImage()

Images of uses of similar widgets are boundless. Some examples are:

In chat
As an image

Probably should ask for user confirmation before opening a link, since link in chat could be used for malicious purposes.

Ah, apologies smsunarto. I gave an in-chat picture but I only meant it to show how a link can be used "in text" (Edited just now). I do not intend this for this widget to be used by the user himself. The user may only access the link which has been written by Terasology programmers themselves. If people were to push code to Terasology with harmful link widgets contained, I am sure their merge will not be approved. That is why I did not mention the need for a user confirmation request.
That is still a very good idea. Thanks!


Annotations are not exposed to the Clear Container

From @jodh-intel on Jul 18, 2016
See https://github.com/opencontainers/runtime-spec/blob/master/config.md#annotations.
Copied from original issue: 01org/cc-oci-runtime#21

From @grahamwhaley commented on Jul 29, 2016
More a bug than an enhancement or feature, as we have missing functionality. May need to raise priority depending on feature demands.

From @amshinde on Jul 29, 2016
I am not clear how annotations are going to be used by the container. A quick search through runc, shows that they are simply stored and read bacc from the state file.

From @jodh-intel on Aug 30, 2016
The spec is not clear on how this data should be made available to the container so it would be interesting to see what runc does with it.

We expose annotations in the container state, virtcontainer uses them. I think we can close this one. Do re-open if you think otherwise.


Match ItLiftAndShiftFromOnPremDomain with latest WDT behavior

This PR modifes ItLiftAndShiftFromOnPremDomain to support 1) downloading WDT using custom URL inside the weblogic pod where WDT DiscoverDomain tool runs; 2) updating domain yaml file generated by DiscoverDomain tool with replica count
Using default WIT/WDT URL (latest WDT release doesn't include the fixes from Richard and Johnny therefore ItLiftAndShiftFromOnPremDomain is still failing, same for ItMiiDynamicUpdatePart1.testMiiRemoveTarget)
https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/11358/
ItRemoteConsole passed at: https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/11367
Jenkins job using WIT/WDT custom URL (main branch including Richard's latest WDT fix, ItLiftAndShiftFromOnPremDomain passed)
https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/11359/

LGTM


Spring client with access to multiple GIT repositories

I do have requirement to use pcf config service in one of the  spring boot cloud config client app to serve application configuration in deserved format like json. Application name will be requested dynamically to client app as part of the request. For example
Branding application - Which is using pcf config service
/rest/v1/branding/{appName}/{environment}.
This controlled method should retrieve configuration from GIT report based on environment and
send as JSON response to consumers
Spring cloud config server is retrieving configuration based on spring.application.name and will it support multiple application names

@tvijayakannan is it possible you have this repo confused with Spring Cloud Config Server?
This repo is for Bosh's idea of a contract between some credential-generating/storing service and the bosh director. Credhub is a popular implementation.

@h4xnoodle yes i am confused with Spring cloud config server.. i was checking that code and thought open it over there but somehow it's landed here. closing it now. Thanks for your comments


usage: watchdog.py [-h] [--log LOG] [--interval INTERVAL] [--port PORT]                    k8s_api watchdog.py: error: unrecognized arguments: --ca --bearer /var/run/secrets/kubernetes.io/serviceaccount/token http://my-IP:8080

Short summary about the issue/question:
After I start service, the program hanging with some notices about waiting for watchdog. About 20min later, the k8s web-page report an error :
"usage: watchdog.py [-h] [--log LOG] [--interval INTERVAL] [--port PORT]
k8s_api
watchdog.py: error: unrecognized arguments: --ca --bearer /var/run/secrets/kubernetes.io/serviceaccount/token http://<IP_ADDRESS>:8080"
Brief what process you are following:
I am following the one-machine deployment guide and doing the step-5

Operating type: Initial deployment
Following deployment process: single-box deployment

OpenPAI Environment:

OpenPAI version: V0.12.0
Cloud provider or hardware configuration: hardware
OS (e.g. from /etc/os-release): ubuntu18.04
Kernel (e.g. uname -a):
Hardware (e.g. core number, memory size, storage size, GPU type etc.): tesla-V100
Others:

Anything else we need to know:
I find a similar issue with narrow information (#2122 ). The author said updating the watchdog to latest version but I found that the tag of newest watchdog in docker.io/openpai is not 'latest' but 'v0.12.0'. I am really confused about that! And I don't know how to fix the image-tag with script.
Any one can help??

Finally, I solve the problem by editing the 'tag' as related version (v0.12.0 for me).


Game Card Arena Preview

BEFORE SUBMITTING AN ISSUE CHECK THE FOLLOWING:

[X] I checked existing issues.
[X] This is not a support question (which can be answered on Slack).
[X] This issue is on the web client (not API, Lambda, or Bots)

FEATURE REQUEST
- Describe the feature you would like to see
I would like to see a preview of the arena on the games page (currently a static arena image is shown).
- Why you need it
Seeing a static image is confusing when the arena doesn't match up.
- How it should work
Receive a minimal arena preview from the backend and display it on the client.

blocked by @oconn https://github.com/willowtreeapps/wombats-api/issues/334


can return index of input data?

hello ,I really like this algorithm, I want to return index of concave point in original data, What a good advice?

I created a python binding for this: https://concave-hull.readthedocs.io/en/latest/
It can return index of hull points. impl: https://github.com/cubao/concave_hull/blob/b67c38ebc4869ac51513ae48b776f12312aa8e8f/src/concaveman.h#L719


refactor: move browser scripts into own workspace

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Cherry picked (and slightly tweaked) from https://github.com/freeCodeCamp/freeCodeCamp/pull/49281.
I want to create a new script to help run python in the browser, and this can be merged separately from the React migration.

Actually, this isn't great. This shouldn't be working, because the dependencies I thought were unused are needed by webpack. It's only working because webpack resolves the client's dependencies.
The fix is to de-nest the workspaces, so I'll do that.

I've continued to work on this in https://github.com/freeCodeCamp/freeCodeCamp/pull/50913, so we can either merge this separately or I can just close it and do everything in the python PR. Whichever works.

Lets make it easy and do it in the Python Pr. Less hassle for all of us.


Invoking Android Native Modules

Do you have an example to invoke android module from this framework?

basically i want to invoke


Improve autocompletion for indices

Fixes #76955
Fixes #83074
Todo:

~include dictionary keys for const dictionaries in the auto completion (see https://github.com/godotengine/godot/issues/83074#issuecomment-1786989398)~ Done

@HolonProduction The 4.2 release is coming very soon, so I'm postponing this and your other autocomplete PRs to 4.3. At the moment we want to focus on critical bugs. Thanks for your patience!

Will need another rebase.

Rebased again and also did some changes to the behavior:

When using [] on an identifier whose type is unknown, identifiers from the current context are suggested. I made this change, since I think array lookups are the most common use case, which should also work well without static typing.
When using [] on an not builtin type, the properties of this object are now quoted. Otherwise auto-completion would complete invalid code.

Also dictionary keys with spaces in it (e.g. "Little guy") can't be accessed using .. Filtering such options out might make sense.

Yeah but you can keep that in mind for another time.

I checked the linked issues and tested this briefly, but I'm not convinced about what the desired completion should be.
I understand users want to be able to use local or member variables as indices, and this PR adds them back. But the rest still doesn't seem to make much sense to me:

IMO we should aim to limit the completion suggestions to only the things which actually make sense. Functions shouldn't be shown when using the [] operator. And random class properties like owner or class names like Object or whatever or is seem hardly relevant for array indexing.
Overall I find it hard to review autocompletion changes as it seems like what should be a good output is not that well defined.

The autocompletion uses _find_identifiers which adds all this stuff. This is in line with how function parameters work.
We could replace it with _find_identifiers_in_class but we would also need _find_identifiers_in_suite and we'd need to add things like true or false again, so basically we would need to reimplement large parts of _find_identifiers. If we did that for all places it would hurt maintainability quite a bit.
Furthermore I don't think it is that easy to say which identifiers are needed in which place.

Functions shouldn't be shown when using the [] operator.

Why not? Take the following example:
const enemy_scenes = {
    EnemyType.FIRE: preload("..."),
    EnemyType.WATER: preload("..."),
    EnemyType.PLANT: preload("..."),
}

func summon_enemy():
    var enemy = enemy_scenes[pick_fair_enemy()].instantiate()

pick_fair_enemy picks an enemy type at random, but also does rejection sampling to prevent unfair situations.
What I'm trying to say with this, is that there are a lot of things that someone could want to put in there. Instead of limiting suggestions on a case by case basis for every context, which could miss use cases, we suggest a lot of stuff and rely on the sorting. It ensures, that users still get suggested the stuff they want. E.g all the native types would never end up above a variable from the current script so they won't be a problem for users that don't want them, while they are still there in the rare case that they are needed.
At least that's how I understand the design philosophy of the autocompletion we currently have.

Cool, does it also fix class name / enum access?
Ex when trying to access an enum stored in a pure class:
# HeroEnums.gd
class_name HeroEnums

enum HeroType {
	SPEED,
	FLY,
	POWER,
}

# example.gd
func _ready():
	var hero_levels: Array[int]
	print(hero_levels[HeroEnums.HeroType.FLY])  # typing HeroEn... will NOT autocomplete, but HeroEnums.HeroTy... will autocomplete

	# another issue
	var hero_types: Array[HeroEnums.HeroType]  # reverse: typing HeroEn... will autocomplete, but HeroEnums.HeroTy... will not

Thanks!

Turns out I didn't fully test after the rebase, the dictionary key part does not work due to #94082 right now. I will fix it in a followup and add some test cases as well.


feat: add google analytics (ga4) and cookie consent.

This PR adds GA4 and consent to cookies.
To log into the Google Analytics account, the credentials of the<EMAIL_ADDRESS>account are stored in bitwarden.
To run the site locally, you must refer to /docs/README.md.
warn: In order to comply with GDPR, users must be able to change their cookie settings at any time. For this, a link "Change cookie settings" has been added in the footer (see video #167).

Note that the cookie manager proposes to activate or not a github cookie. This material is needed to get the information from github for the link on the top right : https://github.com/squidfunk/mkdocs-material/discussions/4666

Tests performed tests performed on Chrome, FFX desktop and Safari iOS.

creation of ga cookies, or not, depending on consent: OK.
send data to ga4: OK.
reception in ga4 (acquisition, page views, geoloc, search terms, etc.): OK.

Note: Tracking is more and more complicated to test as our devices are now equipped with anti-trackers (e.g. on my iPhone, even in non-private browsing, a lot of information is no longer transmitted, or are false).
fix #167

Works for me with chrome and firefox and in the GA4 dashboard


load is not happening

The bordeaux background is displayed in the Sumway extension. in the middle upper part of the screen a small horizontal status bar can be seen. It shows the load has just started (see screenshot) but it is not progressing at all what ever time you wait.
SWF: e51f1029f4819ec2944bce05c9540fc690e81ad583233ae33c1c976a5eae133d (Direct Link)

Works for me now. Thanks.


timeout-minutes, ff-timeout-minutes and max-parallel

Added timeout-minutes, ff-timeout-minutes and max-parallel.
This can be an update in V2 or a new V3.

@slawekjaranowski
Would you pls check this? Thx

@slawekjaranowski
Thx, so i will push it to the v2 branch.


Split IAM policy for ECR push per GitHub repo.

This generates a nasty quantity of IAM guff, but this is the next simplest thing I can think of since the 2 kbyte assume-role/trust policy limit scuppers the obvious solution of just listing all the repos in a condition.
This way also restricts a GH repo to writing to its own repo in the container registry, so at least we get something in return for the clutter.
Second attempt at #1346.

 Ugh I guess I just need to make this a map.

Ok I think that should cover the existing permissions now.

Copying this here from Slack since it's relevant:

sengi:
Medium-term we'd probably be better off having container builds go to GitHub Packages (ghcr.io) and make ECR a pull-through cache of ghcr. Then we wouldn't need a shedload of IAM garbage just to stop Actions from being able to write under other repos in the registry.
and that has the major bonus that the built images get the same visibility as the source code, which should help a lot with lowering the barrier to entry for development
...
It'd be really cool (from a security perspective at least) if with that plus switching from webhooks to having Argo watch for tag changes (Sean's suggestion from earlier) we could get rid of most (all?) of the authnz complexity and coupling between GitHub and AWS


K

When pop up saying thus which one do I hit?t
it just sit there wont find my list

use latest release: https://github.com/Airkek/Youtube-Viewers/releases/tag/3.0

u use pre-release

Yes im using new pre release
On Sat, Dec 12, 2020, 3:53 PM Keijia<EMAIL_ADDRESS>wrote:

u using pre-release
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Airkek/Youtube-Viewers/issues/43#issuecomment-743857134,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASBZ26M3XQJGPG5L5FOYXR3SUPJ3RANCNFSM4UYT65HQ
.

it's pre-release, I drafted release 3 min ago

I'm so sorry lol its working
On Sat, Dec 12, 2020, 3:56 PM Keijia<EMAIL_ADDRESS>wrote:

it's pre-release, I drafted release 3 min ago
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Airkek/Youtube-Viewers/issues/43#issuecomment-743860885,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASBZ26OBZ4DJGKVBUUNXXOLSUPKJHANCNFSM4UYT65HQ
.


default config-dir based on go-gh

As title suggested, let's use official go-gh package form gh cli and do not do the same work what they do :) If they add/change/remove any envs in the future then gh-profile is set without any extra code maintenance.

bonus 1: it solves issue described here: https://github.com/gabe565/gh-profile/issues/3#issuecomment-1508081734
bonus 2: added basic vscode debug settings
bonus 3: let's treat it as go-gh pkg opener for https://github.com/gabe565/gh-profile/issues/21 :)

Looks great! Thank you 👍


[VERBOSE-2:ui_dart_state.cc(157)] Unhandled Exception: 'package:example/main.dart': error: Unexpected tag 32 (ConstConstructorInvocation) in ::.main, expected expression

I want to track the creation of all the widgets under release, so refer to the track_widget_constructor_locations.dart implementation. Through AOP, I implemented one of my own transform, adding a parameter to the widget constructor, but when I executed the "flutter run", an error occurred:
[VERBOSE-2:ui_dart_state.cc(157)] Unhandled Exception: 'package:example/main.dart': error: Unexpected tag 32 (ConstConstructorInvocation) in ::.main, expected expression
#0      main (package:aspectd_impl/aspectd_impl.dart:5:16)
#1      _runMainZoned.<anonymous closure>.<anonymous closure> (dart:ui/hooks.dart:239:25)
#2      _rootRun (dart:async/zone.dart:1126:13)
#3      _CustomZone.run (dart:async/zone.dart:1023:19)
#4      _runZoned (dart:async/zone.dart:1518:10)
#5      runZoned (dart:async/zone.dart:1502:12)
#6      _runMainZoned.<anonymous closure> (dart:ui/hooks.dart:231:5)
#7      _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:307:19)
#8      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:174:12)

The following is“flutter doctor -v”:
[✓] Flutter (Channel stable, v1.17.1, on Mac OS X 10.15.4 19E287, locale
    zh-Hans-CN)
    • Flutter version 1.17.1 at /Users/wangziyang/flutter/flutter
    • Framework revision f7a6a7906b (4 months ago), 2020-05-12 18:39:00 -0700
    • Engine revision 6bc433c6b6
    • Dart version 2.8.2

[!] Android toolchain - develop for Android devices (Android SDK version 29.0.1)
    • Android SDK at /Users/wangziyang/Library/Android/sdk
    • Platform android-29, build-tools 29.0.1
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_202-release-1483-b49-5587405)
    ! Some Android licenses not accepted.  To resolve this, run: flutter doctor
      --android-licenses

[✓] Xcode - develop for iOS and macOS (Xcode 11.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.4.1, Build version 11E503a
    • CocoaPods version 1.9.1

[✓] Android Studio (version 3.5)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 44.0.1
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build
      1.8.0_202-release-1483-b49-5587405)

It's been bothering me for days，Looking forward to reply！Thanks。

When I modify the two parameters, I can perform the flutter run, but this is not the result I want。
return _constructLocation(
location,
parameterLocations: new ListLiteral(
parameterLocations,
typeArgument:
new InterfaceType(_locationClass, _currentLibrary.nonNullable),
isConst: false, // false is ok, true is wrong
),
);
return new ConstructorInvocation(
_locationClass.constructors.first,
new Arguments([], named: arguments),
isConst: false,// false is ok, true is wrong
);

Hi @fightcoder
Looks like you're running an older stable
Can you please upgrade to the latest  dev and try again
flutter channel dev
flutter upgrade --force
flutter doctor -v
If the problem persists, Can you please provide your flutter doctor -v with the latest stable and flutter run --verbose logs, and a minimal complete reproducible code sample
Thank you

Sorry, if I upgrade Flutter, the Aspectd hook function will not be available because the author has not adapted to the new version（@kangwang1988 ）。I can give you the transformed dill file, such as' MyHomePage ', and I have modified the constructor and created it.
app.aspectd.dill.txt

@TahaTesser Sorry, if I upgrade Flutter, the Aspectd hook function will not be available because the author has not adapted to the new version（@kangwang1988 ）。I can give you the transformed dill file, such as' MyHomePage ', and I have modified the constructor and created it.

@TahaTesser Sorry, if I upgrade Flutter, the Aspectd hook function will not be available because the author has not adapted to the new version（@kangwang1988 ）。I can give you the transformed dill file, such as' MyHomePage ', and I have modified the constructor and created it.
app.aspectd.dill.txt

Hi @fightcoder
It seems you are asking for assistance on personal code,
I implemented one of my own transform, adding a parameter to the widget constructor, but when I executed the "flutter run"  then it's a not bug in Flutter itself
Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you

@TahaTesser Sorry, if I upgrade Flutter, the Aspectd hook function will not be available because the author has not adapted to the new version（@kangwang1988 ）。I can give you the transformed dill file, such as' MyHomePage ', and I have modified the constructor and created it.
app.aspectd.dill.txt

hello ,  Have you solved this problem, I also encountered the same problem。

@TahaTesser Sorry, if I upgrade Flutter, the Aspectd hook function will not be available because the author has not adapted to the new version（@kangwang1988 ）。I can give you the transformed dill file, such as' MyHomePage ', and I have modified the constructor and created it.
app.aspectd.dill.txt

hello ,  Have you solved this problem, I also encountered the same problem。


Update build script to add new projects to solutions

Description

To add items to a solution, the "easiest" way is to use ENVDTE. However those are not easily available in PowerShell. Thus we generate a C# class, compile it, load it's assembly and call the main function to run its functionality and add the projects to the solutions.
Motivation and Context

Closes #1836
How Has This Been Tested?

Ran script and open solutions.
Screenshots (if appropriate):

/azp run

@Teap or @llongley want to take a quick look?

/azp run

This is really cool!  I hadn't actually known previously that there was a way like this to programmatically interact with VS project and solution files.  Thanks! :)

This is really cool! I hadn't actually known previously that there was a way like this to programmatically interact with VS project and solution files. Thanks! :)

I agree! I didn't know about this either! :)

This is really cool! I hadn't actually known previously that there was a way like this to programmatically interact with VS project and solution files. Thanks! :)

I agree! I didn't know about this either! :)

I didn't thought that that would be a way to do this upon researching this.
The only thing that bothers me a bit is the performance of this, but I guess unless we actually try to write the sln file manually, there won't be a faster way to do this. ¯\(ツ)/¯

Looks good!  I wouldn't worry about perf for something like this - something that's only run once when a user requests it to run is no big deal if it's slow.

/azp run

/azp run

/azp run

/azp run


Revert client tag to package-composite-v5

ARM (Control Plane) API Specification Update Pull Request
PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

 Click here to see the details of Step 1
Breaking changes review (Diagram Step 1)
If the automation determines you have breaking changes, i.e. Step 1 from the diagram applies to you,
you must follow the breaking changes process.
IMPORTANT This applies even if:

The tool fails while it shouldn't, e.g. due to runtime exception, or incorrect detection of breaking changes.
You believe there is no need for you to request breaking change approval, for any reason.
Such claims must be reviewed, and the process is the same.

 Click here to see the details of Step 2
ARM API changes review (Diagram Step 2)

If this PR is in purview of ARM review then automation will add the ARMReview label.
If you want to force ARM review, add the label yourself.
Proceed according to the diagram at the top of this comment.

 Click here to see the diagram footnotes
Diagram footnotes
[1] ARM review queue (for merge queues, see [4])
The PRs are processed by time opened, ascending. Your PR may show up on 2nd or later page.
If you addressed Step 1 from the diagram and your PR is not showing up in the queue, ensure the label ARMChangesRequested
is removed from your PR. This should cause the label WaitForARMFeedback to be added.
[2] https://aka.ms/azsdk/support/specreview-channel
[3] List of SDK breaking changes approvers in pinned Teams announcement
[4] public repo merge queue, private repo merge queue (for ARM review queue, [1])
If you need further help with anything, see Getting help section below.

Purpose of this PR
What's the purpose of this PR? Check all that apply. This is mandatory!

[x] New API version. (If API spec is not defined in TypeSpec, the PR should have been generated using OpenAPI Hub).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix swagger quality issues in S360.
[x] Other, please clarify:

Revert client tag to package-composite-v5. This is an unexpected change for SQL RP in auto swagger version creation. Thus, we need to revert back to "package-composite-v5" as that is used in main branch

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the Diagram Step 2, "ARM API changes Review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
Swagger-Suppression-Process
to get approval.

Getting help

For general PR approval workflow, see the diagram at the top of this comment.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

"Swagger Avocado" failures are expected 'cause SQL RP uses package-composite-v5  as default client SDK tag.

/azp run

/azp run

/pr RequestMerge


[Gekidou MM-46195] Fix team picker height

Summary
This PR fixes the team picker height to account for the bottomSheet margins and title separator margins
The main reason for this PR was to fix the calculation for the bottom sheet height calculation.  You in the before pictures, only one team is showing, when there are actually 2 available.  This was do to the bottom sheet top margin not showing and the title separator heights not added.
In addition to that fix, you can observe that the at touchable area (green) was not extended from one team item to another. Now, each touchable item abuts to the next without leaving any area that was untouchable (blue).
QUESTION FOR ALL: Why to we need the bottom screen to have the horizontal padding? I would think we should want the items to extend to the edge of the bottom screen.
I also created a ticket https://mattermost.atlassian.net/browse/MM-46241 to convert the TeamListItem to use the OptionItem component.  The enhancement would only need to allow for passing a custom icon (the team in this case). Currently, OptionItem accepts only CompassIcon names.

before
before highlighted
after highlighted
after

Nothing visible changes in these two before/after screenshots. The only difference is that the touchable area extends vertically for each item

before
after

Join Another Team (From Team Sidebar)

has other teams
Not additional teams to join

Join Another Team (From Team Sidebar)

has other teams
Not additional teams to join

Ticket Link
https://mattermost.atlassian.net/browse/MM-46195
Checklist

[ ] Added or updated unit tests (required for all new features)
[ ] Has UI changes
[ ] Includes text changes and localization file updates
[ ] Have tested against the 5 core themes to ensure consistency between them.

Device Information
This PR was tested on:
Screenshots

Release Note

QUESTION FOR ALL: Why to we need the bottom screen to have the horizontal padding? I would think we should want the items to extend to the edge of the bottom screen.

I think you're right. The padding shouldn't be added the the bottom sheet itself, but to the items within so they span the full width of the sheet

I added a ticket to make the padding change to the bottom sheet. https://mattermost.atlassian.net/browse/MM-46247

I added a ticket to make the padding change to the bottom sheet. https://mattermost.atlassian.net/browse/MM-46247

There's a ticket to look at all bottom sheets for consistency as well: https://mattermost.atlassian.net/browse/MM-44138

That's a great document :)

iPad screenshots?
Alsoplease do share screenshots of the add team bottom sheet (by pressing the + in the team sidebar) including there are no more teams to join for both phones and iPad.
Thanks!

iPad screenshots?
Also please do share screenshots of the add team bottom sheet (by pressing the + in the team sidebar) including there are no more teams to join for both phones and iPad.
Thanks!

I added the screenshots and made one change inside add_team_slide_up.tsx, with a PR comment describing why.

@enahum, those screenshots are from a user that is a sysadmin.  When I login as a member user, I don't see any team sidebar, which seems wrong. Shouldn't a member user also be able to change/join other teams?

@enahum, those screenshots are from a user that is a sysadmin.  When I login as a member user, I don't see any team sidebar, which seems wrong. Shouldn't a member user also be able to change/join other teams?

If the teams are opened then yes, check the team settings

@enahum, those screenshots are from a user that is a sysadmin.  When I login as a member user, I don't see any team sidebar, which seems wrong. Shouldn't a member user also be able to change/join other teams?

If the teams are opened then yes, check the team settings

I have 3 teams set to Anyone Can Join

I am getting this warning Warning: An empty or undefined "groups" array has been passed to the handleGroups method.  Perhaps that is the offender.

@enahum, those screenshots are from a user that is a sysadmin.  When I login as a member user, I don't see any team sidebar, which seems wrong. Shouldn't a member user also be able to change/join other teams?

If the teams are opened then yes, check the team settings

I have 3 teams set to Anyone Can Join

Open a ticket, as it seems that if you are not a member of two teams the + option to join another team is not available.
@matthewbirtch if you can comment on the ticket where the option should be visible that'll be nice

I am getting this warning Warning: An empty or undefined "groups" array has been passed to the handleGroups method.  Perhaps that is the offender.

Not at all, that means that we fetched groups the server returned an empty array, the warning is totally fine

@enahum, those screenshots are from a user that is a sysadmin.  When I login as a member user, I don't see any team sidebar, which seems wrong. Shouldn't a member user also be able to change/join other teams?

If the teams are opened then yes, check the team settings

I have 3 teams set to Anyone Can Join

Open a ticket, as it seems that if you are not a member of two teams the + option to join another team is not available.
@matthewbirtch if you can comment on the ticket where the option should be visible that'll be nice

Ticket created and I verified adding a user to two teams via the webapp adds the team sidebar on mobile.  https://mattermost.atlassian.net/browse/MM-46248

@matthewbirtch if you can comment on the ticket where the option should be visible that'll be nice

In the case where you are only part of one team, the team sidebar does not show. However, the team menu (which is supposed to open when you tap on the team name at the top of the Home Screen - but this is not done yet) was where a menu option will be to "join or creat another team".


Support for all keys in a path as a secret

in my current setup we have a vault path for every service, so developers can add secrets in vault and those will be in injected automatically to there service.
i wonder if i can do something like that ?
kind: Secret
apiVersion: v1
metadata:
  name: example-secret
  annotations:
    avp.kubernetes.io/path: "path/to/secret"
type: Opaque
dataString:
<range $key, $value := .>
 <$key>: <$value>
<end>

This was discussed before and ultimately we opted to not support this feature. See discussion here: https://github.com/argoproj-labs/argocd-vault-plugin/pull/281

Thanks for the suggestion @roee-hershko! I agree with @jkayani here. We take a declarative approach, in the sense that if you want to add a new item to a secret, it should be done in git. This prevents the situation of not knowing what secrets are in Kubernetes when looking in git.


Merge develop into master

@OsvaldoRosado

@jjjordanmsft,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Added deprecared CamelCase methods as removed in "uia" backend to tell users correct names

Old examples are still pursuing us. So we should set a defense to decrease number of typical questions like "Where is my SetText method for UIA?" or "Where is my control?".
ClickInput = removed(click_input) will be for UIA backend first (in 0.6.6), then we will switch to removed decorator for all backends in 0.7.0+.

One more thing to consider is how to hide all these "removed" methods from auto completion menu. BTW, deprecated methods should be hidden as well.
I have only one thought: handle all such methods in __getattribute__ method and return non-deprecated method instance if deprecated one is requested. Also raise a warning or an error in __getattribute__ as well. Of course this idea requires maintaining some hidden list/dict of deprecated/removed methods or method names.
As a conclusion the deprecated methods won't pre-exist but would be created on-the-fly.


Allowing for default branch seems to have broken oh-my-zsh

Just ran zgen update and pulled in the most recent commits, including the "use default branch instead of master" change. Now I'm seeing these errors anytime I create a new shell:
/Users/sprice/.zgenom/sources/ohmyzsh/ohmyzsh-___/oh-my-zsh.sh:source:12: no such file or directory: /Users/sprice/.zgenom/sources/ohmyzsh/ohmyzsh-master/tools/check_for_upgrade.sh
/Users/sprice/.zgenom/sources/ohmyzsh/ohmyzsh-___/oh-my-zsh.sh:cd:63: no such file or directory: /Users/sprice/.zgenom/sources/ohmyzsh/ohmyzsh-master
/Users/sprice/.zgenom/sources/ohmyzsh/ohmyzsh-___/oh-my-zsh.sh:source:74: no such file or directory: /Users/sprice/.zgenom/sources/ohmyzsh/ohmyzsh-master/lib/compfix.zsh
/Users/sprice/.zgenom/sources/ohmyzsh/ohmyzsh-___/oh-my-zsh.sh:76: command not found: handle_completion_insecurities
/Users/sprice/.zgenom/sources/ohmyzsh/ohmyzsh-___/oh-my-zsh.sh:100: no matches found: /Users/sprice/.zgenom/sources/ohmyzsh/ohmyzsh-master/lib/*.zsh
/Users/sprice/.zgenom/sources/ohmyzsh/ohmyzsh-___/plugins/kubectl/kubectl.plugin.zsh:5: no such file or directory: /Users/sprice/.zgenom/sources/ohmyzsh/ohmyzsh-master/cache/kubectl_completion

I think this is coming from oh-my-zsh's default settings:
❯ echo $ZSH
/Users/sprice/.zgenom/sources/ohmyzsh/ohmyzsh-master
❯ echo $ZSH_CACHE_DIR
/Users/sprice/.zgenom/sources/ohmyzsh/ohmyzsh-master/cache

Relevant lines from my zgen setup script:
zgen oh-my-zsh
zgen oh-my-zsh plugins/kubectl

any advice on how to avoid? Should I specify the master branch for oh-my-zsh explicitly? How do I do that?

Sorry my bad. I'll see that I get a fix out today.
In the meantime you could try to set this before you source zgenom: ZGEN_OH_MY_ZSH_BRANCH=___

@steeef I think I've fixed it.
Could you please switch from main to Fix-oh-my-zsh-prezto-default-branches to verify.
When you switched branches, run zgen reset and start a new shell the issues should be gone 👍

@jandamm Yup, that did it. oh-my-zsh loading works fine now. Thanks!

Thanks for the confirmation 👍


✨Website header#1

Description
This pull request addresses the issue #1, where the website header's responsiveness was not working properly on smaller screens. The changes include updating the CSS styles to ensure the header elements adjust appropriately for different screen sizes.

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

laptop

tablet

mobile

How Has This Been Tested?
I tested this change on various devices and browsers to ensure the header displays correctly and is responsive.
Tested on desktop (Chrome)
Tested on tablet (iPad, Android)
Tested on mobile devices (iPhone, Android)
Test Configuration: (optional)

Firmware version:
Hardware:
Toolchain:
SDK:

Checklist

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] 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

Good attempt closing this pr as it is already done, please pick another issue


feat(c/driver/postgresql): handle non-SELECT statements

Before we tried to infer the query schema before COPY by wrapping it in a "SELECT * FROM (...) LIMIT 0".  This broke if the query was (for example) a CREATE or UPDATE.  Instead, use a prepared statement to infer instead.  If we find that there are no result columns, then execute without the COPY path.
Also, test what happens with "INSERT INTO ... RETURNING" (this works with COPY).
Fixes #701.

Which PQexecParams are you referring to? The one in ExecuteUpdateQuery? I can rearrange the control flow so that we always prepare before calling that.
(We can't use it in ExecuteQuery since apparently you can't prepare a query involving COPY.)

I added Python tests on a whim and found https://github.com/apache/arrow-nanoarrow/issues/203 and https://github.com/apache/arrow/issues/35760 so I'm going to go shave some yaks here first.

Which PQexecParams are you referring to?

Sorry. I should have mentioned it explicitly.

The one in ExecuteUpdateQuery?

Yes.

(We can't use it in ExecuteQuery since apparently you can't prepare a query involving COPY.)

Oh, I didn't notice it.

+1


Add chunkwm

Project URL

Category

Description

Why it should be included to Awesome macOS open source applications  (optional)
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Appropriate language icon(s) added if applicable
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

@gdancel Thank for your contribution! 👍


actions[type] is not a function

I have this code with v3:
<Scene key="companies"
                           initial={loggedIn ? true : false}
                           hideNavBar={true} {...styles} type={ActionConst.BACK} component={Companies}/>

and I have a scene that calls Actions.companies({refresh: { goBack: true }})
that was working fine on v3, but on v4 I get actions[type] is not a function.
If I remove type={ActionConst.BACK}, then I no longer get the error, but the scene transitions from the wrong side. It seems like it's being added on the stack, but it should behave like pop() - removing the current scene to show the one below.

try using type={'back'} instead.

Not 'back', but 'pop'. I'm not sure about support of 'refresh' inside pop. Could you just raise two actions?

yep, it's pop and not back. But if I do use type={'pop'} navigating to this scene behaves like pop -> it just pops the last scene from the stack.
And since on the scene I am I have a tabbar -> when you navigate between the tabs and then hit the actions.companies() buttons you first switch between the tabs and once they are off the stack you navigate back to hte companies page. And I just want to directly transition back to companies no matter how many screens there are on the stack.
I tried popTo, but it seems to cause some endless loop or so... The app freezes :)

You should pass child scene to pop to, not container one.

okaay, so adding type={'popTo'} to the scene seems to work fine.
Using Actions.popTo({scene: 'companies'}) seems to cause the endless loop

Because 'companies' is not child.

Why does companies need to be a child? It's a parent in this case and I would like to navigate back to the parent?
And what's with refresh now? Can we use it in v4 or not?
Actions.companies({
	refresh: {
	  goBack: true
	 }
 })

This doesn't seem to pass goBack as property to the previous scene. No matter if I use type={'popTo'} or not. It just doesn't seem to pass the property back to the previous scene. ComponentWillReceiveProps is never fired.

'Companies' is stack. Stack cannot be presented, you always see some child within the stack.
'refresh' inside pop looks a little bit strange, have you tried just two calls - popTo & refresh?

Sorry, I'm lost. I don't understand what you mean with " Stack cannot be presented, you always see some child within the stack."
companies is the first scene in the stack.
I discovered that I can remove the type={'popTo'} from the scene definition. The way then to popTo this scene is by using
Actions.popTo('companies')
instead of v3 syntax
Actions.popTo({scene: 'companies'})
unfortunatly passing params as second argument doesn't do anything.
I tried it with 2 calls like this:
Actions.popTo('companies');
Actions.companies({wtf: true});

wtf was auto added to props.navigation.state.params. So we no longer need to use the refresh object?

Have you tried Actions.popTo('companies');Actions.refresh({data}); ?

Yep,that also seems to work.
Are those 2 the same in behavior? Calling refresh on actions is somehow strange for me as I don't know on which scene the refresh is going to be called.
Actions.popTo('companies');
Actions.refresh({data});

Actions.popTo('companies');
Actions.companies({data});

On current scene. After popTo currentScene is changed.


Switch item doesn't work with accessibility

I am having issues getting accessibility working with the switch item.  The surrounding TouchableHighlight doesn't allow focus on the Switch and thus cannot be tapped on.
Is there a workaround?

https://github.com/evetstech/react-native-settings-list/pull/16


TypeError: 'NoneType' object is not subscriptable.

TypeError: 'NoneType' object is not subscriptable.
In the crop_and_save() , the above issue is getting. Please help by resolving the issue soon.

hello,
Please check whether the mask images are being loaded correctly. You can confirm this by displaying the mask image in a cell. Do let me know the outcome.
Thanks

Hello, I have exactly the same issue. Have you managed to solve it?


Clean up (tests, travis, tox)

Python2.6 support
Move tests out of app
Migrate to using py.test/py.test-django as the test runner
Implement tox to start paving the road for testing multiple python/Django/DRF version support
Travis-CI integration
Codecov.io integration
Update README to be accurate with (current) supported versions

@marcosmoyano

I already know I'm going to need make some tweeks to .travis.yml to run tests through tox, but then run coverage to generate the coverage report that codecov.io needs.
This PR should at least allow anyone to clone the repo, install requirements and run tests successfully with tox.

:+1:


Inconsistency in C++ array property accessors and mutators

https://github.com/real-logic/simple-binary-encoding/blob/master/sbe-tool/src/main/java/uk/co/real_logic/sbe/generation/cpp/CppGenerator.java#L1266
defines put for null-terminated strings and uses the size of the type multiplied by the array length as the total byte length for copying.
https://github.com/real-logic/simple-binary-encoding/blob/master/sbe-tool/src/main/java/uk/co/real_logic/sbe/generation/cpp/CppGenerator.java#L1254
Does the same for the accessor.
While the std::string and std::string_view versions
https://github.com/real-logic/simple-binary-encoding/blob/master/sbe-tool/src/main/java/uk/co/real_logic/sbe/generation/cpp/CppGenerator.java#L1309
https://github.com/real-logic/simple-binary-encoding/blob/master/sbe-tool/src/main/java/uk/co/real_logic/sbe/generation/cpp/CppGenerator.java#L1325
https://github.com/real-logic/simple-binary-encoding/blob/master/sbe-tool/src/main/java/uk/co/real_logic/sbe/generation/cpp/CppGenerator.java#L1280
https://github.com/real-logic/simple-binary-encoding/blob/master/sbe-tool/src/main/java/uk/co/real_logic/sbe/generation/cpp/CppGenerator.java#L1291
only copy the length of the array in bytes.
I think we need to multiply with the size of the array elements in both cases.
Ideally, the c string versions could be called from the std::string and std::string_view versions to ensure common implementation.

Is SBE spec describe size of char? std::string and std::string_view added only if field is array of chars.
There is a problem in bit compatibility in case of size of char not equal to 1

Sorry, didn't notice the condition around the block. This issue can be disregarded.


Fix memory leak in MeshoptCompression

As discussed in https://forum.babylonjs.com/t/meshoptdecoder-memory-leak/49696
By using web workers to decode the buffer we can terminate the web worker which releases the wasm heap. In the future, memory.discard may be a better way of solving this, but this API is not available yet.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/14995/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14995/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14995/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14995/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14995/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14995/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14995/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14995/merge#BCU1XR#0

WebGL2 visualization test reporter:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14995/merge/testResults/webgl2playwright/index.html

Visualization tests for WebGPU (Experimental)
Important - these might fail sporadically. This is an optional test.
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14995/merge/testResults/webgpuplaywright/index.html

I really would not recommend doing this for every call... this results in inefficient resource utilization as the workers are continuously created and destroyed, and doesn't improve peak memory consumption. The correct pattern should include calling useWorkers(0) at some point after all scenes have been loaded (in a streaming scenario you probably never want to do this as you are constantly going to decode buffers anyway.
My understanding fwiw is that KTX2 decoding runs into the exact same problem, and a more centralized solution might be valuable in context of Babylon.JS (e.g. a single app-level hook that can be called when the loading is finished, or at least "temporarily finished", and resources can be relinquished)

KTX2 is using our own implementation or a workerpool, that maintains a worker in memory for a certain amount of time. if a worker is idle after the predefined idle time it deconstructs.
If you have a nice way of solving the issue here, we will be very happy to review it!

For reference, this is the code @RaananW is referring to:
https://github.com/BabylonJS/Babylon.js/blob/master/packages/dev/core/src/Misc/workerPool.ts#L89-L156
This is being used by Draco and KTX2 already. We can probably also use it for Meshopt?


Improve Trait Tool

Must make Will, can use a relevant skill, using IQ instead of DX
-1 per 5% increase in trait (will not increase damage)
+5 if motivated by fear, anger, concern for loved one (GM decision, usually failed control role)
Critical success no FP cost
Critical failure lose HP equal to FP spent on the attempt
Natural 18, immediate HT roll to avoid acquiring a temporary disadvantage

done


Fix "Introduction" link of README.md

name: "\U0001F4DD Documentation Fix"
about: Fixing a problem in an existing docs page
Checklist

[ ] Is there an existing issue for this PR?

link issue here

[ ] Have the files been linted and formatted?

What docs page needs to be fixed?

README.md

What is the problem?
"Introduction" link was page not found.
What changes does this PR make to fix the problem?
Update link to Introduction/getting-started

Thanks!


Add ES2015 @@isConcatSpreadable support, Array .concat() and .map() trailer change

Add support for ES2015 @@isConcatSpreadable in Array.prototype.concat(). Also remove the config options for non-standard trailing element behavior in Array.prototype.concat() and Array.prototype.map(). The non-standard behavior matches real world behavior of other engines, and the specification "bug" in ES5.1 was fixed in ES2015 so the options are no longer necessary.
Tasks:

[x] @@isConcatSpreadable support
[x] Remove non-standard config option DUK_USE_NONSTD_ARRAY_CONCAT_TRAILER
[ ] Remove non-standard config option DUK_USE_NONSTD_ARRAY_MAP_TRAILER
[ ] Resolve FIXMEs
[ ] Testcase update
[ ] API doc clarification for duk_concat() re: @@isConcatSpreadable behavior
[ ] Releases entry

The non-standard behavior matches real world behavior of other engines, and the specification "bug" in ES5.1 was fixed in ES2015 so the options are no longer necessary.

Does this mean that the "non-standard" behavior is, in actuality, standard?

It's non-standard in ES5.1, standard in ES2015.

For Array.prototype.map() the DUK_USE_NONSTD_ARRAY_MAP_TRAILER was added to match real world behavior of other engines. However, it seems ES5.0 already has the "real world" behavior included in the algorithm: the final result length is 1:1 with the input because the result array is created based on the input length. The subsequent index writes don't affect the length, and trailing gaps don't thus cause the result length to be shorter either. So the option was in fact just fixing a Duktape bug.


Enable HTTPS in all blazor templates

Can we set the project templates to default to HTTPS ?

How can I setup blazor to properly handle HTTPS manually? Currently getting "405 - HTTP verb used to access this page is not allowed." Seems like the Blazor client is not setup properly so it is routing requests using HTTP rather than HTTPS.

No, that error shouldn't have anything to do with HTTPS. It can mean, for example, that you're trying to POST to a resource that only accepts GET.
Are you using client-side or server-side Blazor? If the latter, is this on IIS, and do you have WebSockets support enabled?


Makes test_memory_limit_exceeded_cook_script more reliable

Changes proposed in this PR

adding -9 as an allowable exit code for the python script
removing the skipUnless that keeps the test from running locally

Why are we making these changes?
We've seen the exit code -9 happen in the wild. As for removing the skipUnless, this test passes reliably in my local environment, and I think it would be good for devs to be running it.

@DaoWen Do you have any feedback on this?


Drop signs queue

Drop signs queue: place them immediately
This removes neomake#signs#PlaceVisibleSigns and neomake#signs#RegisterSign.
Fixes: https://github.com/neomake/neomake/issues/834
Closes: https://github.com/neomake/neomake/pull/471
Ref: https://github.com/neomake/neomake/issues/240
Ref: https://github.com/neomake/neomake/pull/893

I expect for some issues to pop up - otherwise I'm surprised that it was that easy.. ;)


The default branch points to an outdated version

This makes the install according to the instruction given not working out of box

Hello @zh217 👋
What do you mean?
Can you please detail what you did to install Oh my tmux!?

Oh I see!
Weird. Changing that

Done


Improve warning message when manifest fields are invalid

Print the name of the manifest member when an unsupported value is specified.
Note: while testing this I noticed that the warnings are printed twice, but it doesn't seem related to this PR.
BUG=XWALK-5262

Please start the commit message with Manifest:
So the topic of the commit is clear.
For the message itself, I think it could just be
Unsupported value '" + ... + "' for 'orientation' in manifest.json
"Member" seems to be an unnecessarily specific term.
Thanks.

@r0b5t4 updated


Publish on PyPI

Everything is in place to publish, all that's needed is a bit more testing and cleanup, documentation, and closing all issues before publishing.

Published


Look into superlinter and if it can be useful for repo

Seems like a good idea, but after struggling to run it locally i'm gonna park this for now. It is primarily built for running with github actions so local setup is quite limited for the time being. Will check this again after gathering some more info and waiting for input from colleagues


Add Docker support to infping

Changed the URL from influxdb to influxdata to fix the client import
errors, at some stage this will need changing to v2.
Changed the configuration option from ./config.toml to
/etc/config.toml.
Add Dockerfile which will allow us to run this from Docker containers
and allow for us to deploy this at scale

Sorry, meant to do this in a personal repo - for some reason github changed to yours ;(

Thats ok, thanks
On Thu, Aug 25, 2016 at 12:58 PM, Dan Offord<EMAIL_ADDRESS>wrote:

Sorry, meant to do this in a personal repo - for some reason github
changed to yours ;(
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/torhve/infping/pull/5#issuecomment-242336908, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AI6p1_zJgVQQBwE9k34VS-VVT_rxp4nxks5qjWdIgaJpZM4Js5UO
.


redraw workflow stepper on resize

https://github.com/parodos-dev/backstage-parodos/assets/118328/8291a49a-a9f8-4d3e-8373-9c824fc31559

perfect!!!


fixed date with time in LuftfahrtBundesAmtBridge

Hi,
I fixed a new date format that was introduced by the website.
Kind regards
Holger

NICE


Fix typos in description of forecast endpoint

Fix small typos in the openapi description of endpoint /emissions/forecasts/current

Was experimenting with the api and saw the typos and thought I'll fix them shortly.
So its a super small PR :)

Thank you


No-compressed png encoder fails on Firefox

The output resulted from no-compressed png encoder of demo, even though can be displayed on Chrome/Edge, failed to do so on Firefox with error "Image corrupt or truncated". More over, it also fails when passed into createImageBitmap on Chrome with "DOMException: The source image could not be decoded" (yet when passed into image tag, it still loads ok on Chrome). I think the encoder is broken somewhere?

Thanks for bringing this up. Could you point to the file where something fails?!

Thanks for bringing this up. Could you point to the file where something fails?!

https://github.com/libjxl/libjxl/blob/b1dc4ff8cb2c73fb1e46a6bff79d4a8a09857f65/tools/wasm_demo/no_png.cc#L168-L215
I think it's somewhere here. I checked the output png file with pngcheck and the output is something like this:
File: image.png (4917485 bytes)
chunk IHDR at offset 0x0000c, length 13
1280 x 1280 image, 24-bit RGB, non-interlaced
chunk iCCP at offset 0x00025, length 550
profile name = 1, compression method = 0 (deflate)
compressed profile = 547 bytes
chunk IDAT at offset 0x00257, length 4916866
zlib: deflated, 256-byte window, superfast compression
CRC error in chunk IDAT (computed 33609e4c, expected 00000000)
ERRORS DETECTED in image.png

It's definitely because of the  Fake CRC32 at the end of IDAT. Fixed it. But would it cost much performance-wise?

@eustas probably can give you an answer
Can this be closed?

IIRC we had that checksum calculation in the past, but removed it for the sake of performance.
I can re-add it as a build option. That way "careless" browsers will be able to get max performance, while others will not fail.
Ping me regularly to make that happen, please.

It would be more useful to make this a runtime thing, and have some detection code to test whether correct CRCs are needed or not, computing them only when needed. There's no point in a wasm polyfill that doesn't work on some browsers...


Cannot rename items with same name when switching between cases

Hello. I am unable to rename items to switch between case. For example, "My API Artifact" accepts the change to "My API artifact" but does not actually save it. I have to clone and rename to something else, then delete the old and replace the cloned name with what I want it to look like.
Excellent app, by-the-way. You are doing hero's work.

I am also on v0.27.2.

Interestingly enough, I was able to update my collection name. Even got the Toast notification. I just created a test with all words capitalized in a collection, folder, and request. My test was to try renaming the last word in each item to lower case. Only the collection was successful. And only the collection gave me the toast notification.

FWIW, I am on a Mac with Ventura 13.6

I'll check on a Mac when I get a chance.

There is already an open issue for this problem: #510 and a PR #531 to fix it.

Solid. Thanks for the follow-up. I searched for similar issues, but not well enough. Apparently.

Duplicate of 510


Migrate to CircleCI 2.0

Migrate to CircleCI 2.0.
In the previous configuration of CircleCI, the version of ruby was specified as 2.1.3, but this time I am using 2.1 ruby docker image.
The reason is that, as a result of testing the build at hand several times, ruby version 2.1.3 image does not cache hit almost every time.
https://discuss.circleci.com/t/cache-docker-image/12790
Currently, ruby 2.1.10 p492 (2016-04-01 revision 54464) is used in CircleCI 2.0.

Ah yeah, I'm chill on the Ruby version - I use 2.1.3 because I liked that version of ruby a few years ago, and I enjoy the consistency, which ever version works is fine for me

          1 Message

:book:
@serima is not a contributor yet, would you like to join the Danger org?

  Generated by :no_entry_sign: Danger

Thanks @serima - if you're happy, I'm good to merge?

@orta Thanks! Yeah I'm happy. Good to merge 🚀

rockin'
Want to join the org?

@orta yea, I definitely want to join 😄

Sent 👍


Better Design around Fixie's TeamCity output

When the --TeamCity option is specified, Fixie will use that value to produce TeamCity styled formatting in its output. When the value is omitted, Fixie checks for an environment variable. This part isn't of any concern to Cake, but this should be better documented in the FixieRunner usage.
Also, for better discoverability and ease of use, the TeamCityOutput enum could be simplified to Nullable<bool>.

Sounds good to me! Thinking on it now, is this something that should be true when TeamCity.IsRunningOnTeamCity == true?

:+1:

I'm grabbing this issue :smile:

@RichiCoder1 Good catch, but it looks like Fixie automatically do this if running on TeamCity by looking for the TEAMCITY_PROJECT_NAME environment variable, so I don't think we need to do that.


No support for servers without curl

I was unable to get oauth2-client to work on app engine without cURL. I presume this is down to using Guzzle v3 which is two major versions behind. I think v4+ is able to use other handlers to send requests.

Version 1.0 will use https://github.com/egeloen/ivory-http-adapter, which allows you to use any number of HTTP adapters, as you like, including Guzzle 5.


Naming of mapPartial

mapPartial isn't a a name that will be understood without explanation. Should we rename it to something else? Are there other good options, like a set-builder notation?

We discussed some alternative options like map*, map? and mapPartialFun. Classical notation for mapping a set with a function would be f(X) or f[X] where X is a set, but that doesn't really apply here.
For now, we decided to leave it as it is and add a description for mapPartial in the Notation Section.

mapMaybe?

I guess that's another option, but it doesn't save us from having to explain it. In which case, sticking with mapPartial is a bit easier.

It still requires explanation but it's Haskell terminology so should be more familiar to some...


feat(lang): add python support

@jakub-kozlowicz this is a super nice addition. I just would like to add I cannot run the full pytest test suite using the LazyVim-provided <leader>tT. Does this work for you?
I had to override this for neotest-python:
    keys = {
      {
        "<leader>tT",
        ":lua require('neotest').run.run({ suite = true })<CR>",
        desc = "Run all tests (override LazyVim)",
      },

@jakub-kozlowicz this is a super nice addition. I just would like to add I cannot run the full pytest test suite using the LazyVim-provided <leader>tT. Does this work for you?
I had to override this for neotest-python:
    keys = {
      {
        "<leader>tT",
        ":lua require('neotest').run.run({ suite = true })<CR>",
        desc = "Run all tests (override LazyVim)",
      },

@fredrikaverpil For me the config is working just fine. I can run the whole suite of test using the <leader>tT. I do not have to add anything to the neotest-python configuration.

This config is simple and is working very well.

Merged. Thank you!


Testing SemanticKITTI with unlabels pc returning all 0's in labels - am I looking in the wrong place?

Is there any future where you add a test run for semanticKITTI testing data?
You're are currently using validation which has labels in it as the TEST_DATASET/TEST_DATALOADER. This does not represent testing as it gets put in with the actual labels and not with a list of 0's. I'm struggling to get testing to work myself.
Whenever I try and test the model with unlabeled pointclouds it seems to not even change the labels on the output. These remain 0 even when a min_possibility of 0.5 is reached over the whole testing dataset. Am I wrong to look in the labels on the output? Am I suppose to look somewhere else?

Hi, thanks for your attention to the reimplementation.

I have not provided the testing code yet, but it will come soon in next few days. You can also follow the testing code in my another repository and modify it to suit the RandLA testing(I will also write the testing code in this way). Because the testing code is similar in both papers.

I actually only evaluate the model on the validation dataset.

As for the problem you mentioned, I don't know what changes you have make to the code.
Did you specify a check point path during testing?
If you feed a list of full 0 as test labels to the network, maybe in this function the code will ignore your 0's label.

Is there any future where you add a test run for semanticKITTI testing data?
You're are currently using validation which has labels in it as the TEST_DATASET/TEST_DATALOADER. This does not represent testing as it gets put in with the actual labels and not with a list of 0's. I'm struggling to get testing to work myself.
Whenever I try and test the model with unlabeled pointclouds it seems to not even change the labels on the output. These remain 0 even when a min_possibility of 0.5 is reached over the whole testing dataset. Am I wrong to look in the labels on the output? Am I suppose to look somewhere else?

Hi, ThysEmiel.
I have released the test code for semantickitti, but I have not test my model and generate the predicted label yet, because It will spend some time while voting.
You can try the test code testing you model and I think the code should work correctly.
Remenber to modify the check point path before testing or the code would raise an error.
Good luck to you.


Start documentation /user manual

I think it would be useful for us to have a place to start putting together instructions and documentation for the Toolkit. Even if it's just placeholder text.
I'm debating between the Wiki in this org/repo and a Google doc. Thoughts? Which will be easier to migrate to another platform later? Other ideas?

I think a doc is easiest to migrate between platforms. I like readthedocs as an endpoint, and I'd be happy to set that up now, but we can definitely hold off / choose something else.

I don't have much experience with RTD. Is it "easy" to setup? Would the source live in this repo or a different one?

It's easy-ish -- I could certainly have it done by end of Tuesday. As long as we're not trying to do anything fancy (e.g., auto-executing Jupyter notebooks after new commits), at least.

Since we have the option to start with it, let's do it! I'll learn!

Sounds great! (And the source can indeed live in this repo)

We're set up over here!

Decided to put the docs in astrodb_scripts.


Redesign the landing page for the wall

Have a big call to action button to start creating a wall
Links to other loklak pages
Show a list of features offered by the wall

This issue will be updated as the wall grows.

Shelved temporarily.


Update podSpec to 8.0.4?

Would it be possible to update the podSpec to 8.0.4 so that at least the warning from Xcode (specifically #1307) would be brought down? Not an important issue, but I'd love to be able to remove the deprecation warnings.
Thanks!

Same applies for creating a tag for SPM

Please send in a PR, and i'll merge.
I am in between many things at the moment.

It's not a PR that's needed. Just a tag w/ a new version. The issue is that the current release is almost two years old and doesn't contain recent commits.

Understood. In that case, i have some account resolving to do.
This account is back by 2 factor auth.
Phone is fried. Please give me some time to resolve.

@patchthecode any update?

finally got access back to my account.
updated the tag.


chore: migrate api-category into this repo

Description of changes

mv the package from the cli repo into the amplify-category-api repo.
Update dependencies to use caret versioning for now-remote deps, and specific versions for local ones (which will be managed by lerna).
Update tsconfig file to remove references.

Relevant PR in CLI core https://github.com/aws-amplify/amplify-cli/pull/10493
Note: as implemented this is not a history-preserving activity. I'm currently exploring that as an option.
Issue #, if available
N/A
Description of how you validated changes
yarn setup-dev && yarn test, also verified amplify-dev init -y && amplify-dev add api -y && amplify-dev push -y succeeded with no issues.
Checklist

[x] PR description included
[x] yarn test passes

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Closing in favor of https://github.com/aws-amplify/amplify-category-api/pull/489


Add nix language syntax highlighting

Uses https://github.com/replit/codemirror-lang-nix

Thanks!


[QA Steps Required] Translate 'Home' breadcrumb in page meta resolvers

The word 'Home' was translated only in the breadcrumb.component in PR https://github.com/SAP/cloud-commerce-spartacus-storefront/pull/2463/files#diff-9ed96335f7d036e3662479d2f9049d91R42 , but we need to also translate it in all page meta resolvers (classes extending PageMetaResolver), which can provide custom breadcrumbs array.
The translation key common.home can be reused.

How to test this please?

Talked to @mszczepaniak about QA steps.
Tested on product, search and category pages - all of them now contains proper homepage label translation in breadcrumbs.


Update instructions for manual testing in DG

[x] Launching of Application
[x] Add Command
[x] List Command
[x] Edit Command
[x] Find Command
[x] Delete Command
[x] Schedule Command (@YuZizhen )
[x] Unschedule Command (@YuZizhen )
[x] Reschedule Command (@Dethada )
[x] Remark Command
[x] Add Tag Command (@Arixeyeion )
[x] Delete Tag Command (@Arixeyeion )
[x] Clear Command
[x] Exit Command
[x] Help Command

Let's try to follow this in order which is the sequence in our UG.

Closed in #330


Update Dawn to latest revision as of Nov 30, 2021

Update to upstream@dcf57433957731e9139358a5634f078912f1a26c

Process for updating:

git remote add upstream https://dawn.googlesource.com/dawn
git checkout main
git checkout -B update-nov-30 (replace date with current date)
git fetch upstream
git merge upstream/main
Send PR to merge update-nov-30 into main of hexops/dawn
Review that PR and make GitHub review notes of: any changes to (BUILD) .gn files, .gni files
gclient sync, gn args out/Release (set use_system_xcode=true) and ninja -C out/Release to build Dawn.
Review the changes to the third_party/ directory, consider altering add-dependencies.sh to remove files that should not be committed.
Run add-dependencies.sh, review the staged changes and commit.


Add repository url

You should add repository url to https://marketplace.visualstudio.com/items?itemName=Orta.vscode-twoslash-queries
e.g

Thanks (and thanks @with-heart) - done


Fix NA Treatment and Temperature codes

as well as NA cores...see diagnostic plots.

Fixed; Picarro registers some valve numbers of zero (now filtered out), and treatments.csv now has ambient entries.


fatal error: Wrong parameters for Exception([string $exception [, long $code [, Exception $previous = NULL]]])

Fatal error: Wrong parameters for Exception([string $exception [, long $code [, Exception $previous = NULL]]]) in /home/ckworklab/public_html/wp-content/plugins/fantastic-elasticsearch/vendor/ruflin/elastica/lib/Elastica/Exception/ResponseException.php on line 41
got this error on option.php page while trying to save the server url from ADmin panel

I am getting the same error on OSX MAMP.

Just started a new branch that is for ElasticSearch 2.x, the old version only worked with < 1.x
https://github.com/parisholley/wordpress-fantastic-elasticsearch/tree/upgrade
Let me know if that solves your issue, otherwise give me more environment details


Upgrade 2020-09-01-hybrid sdk to T2

Fixes microsoft/vscode-azureresourcegroups#296
View changelog

We may also need to do microsoft/vscode-azuretools#1180 as well. Though I'm not entirely sure these -profile-2020-09-01-hybrid packages are even needed anymore...

We need these packages to support custom clouds, which includes Azure Stack. We actually need to add these packages to the Functions and App Service extensions in the future so those extensions support Azure Stack too.
Related:
https://github.com/microsoft/vscode-azurefunctions/issues/2259
https://github.com/microsoft/vscode-azureresourcegroups/issues/298

VSCode extensions for Azure Account, Resources, Storage, and VM are all officially supported- https://docs.microsoft.com/azure-stack/user/azure-stack-dev-start-vscode-azure
Using a supported version of "hybrid" API profiles- https://docs.microsoft.com/azure-stack/user/azure-stack-profiles-azure-resource-manager-versions
As already mentioned by @alexweininger it would also be great to have working extensions for other resources available on Stack Hub, including App Services and Functions.