ObservationFromRay doesn't return u'LineOfSight' dictionary/json until a couple of moves

Steps to reproduce:

Add the line <ObservationFromRay/> to the mission XML file
E.g. For Python_Examples/tutorial_6.py, the corresponding XML section for agent is as shown below

        <AgentSection mode="Survival">
          <Name>Cristina</Name>
          <AgentStart>
            <Placement x="4.5" y="46.0" z="1.5" pitch="30" yaw="0"/>
          </AgentStart>
          <AgentHandlers>
            <DiscreteMovementCommands/>
            <ObservationFromFullStats/>
            <ObservationFromRay/>
            <VideoProducer want_depth="false">
                <Width>480</Width>
                <Height>320</Height>
            </VideoProducer>
            <RewardForTouchingBlockType>
              <Block reward="-100.0" type="lava" behaviour="onceOnly"/>
              <Block reward="100.0" type="lapis_block" behaviour="onceOnly"/>
            </RewardForTouchingBlockType>
            <RewardForSendingCommand reward="-1" />
            <AgentQuitFromTouchingBlockType>
                <Block type="lava" />
                <Block type="lapis_block" />
            </AgentQuitFromTouchingBlockType>
          </AgentHandlers>
        </AgentSection>

Start the example. Notice that until a couple of act calls, u'LineOfSight' dictionary is not returned.

Are there any preconditions that need to be satisfied for ObservationFromRay to return the LineOfSight JSON?
I noticed, for some particular configuration (which I suspect is standing very close to a "wall") the LineOfSight JSON is not returned.
From my experiment log:
[03-24-2017 22:56:51]: [MalmoBasicEnvironment.py:396:makeObservation]: DEBUG :: Received observation {u'PlayersKilled': 0, u'TotalTime': 13456, u'Life': 19.0, u'ZPos': 5.300000011920929, u'IsAlive': True, u'Name': u'ButterFingers', u'DamageTaken': 90, u'Food': 20, u'Yaw': 180.0, u'TimeAlive': 13414, u'XPos': 3.0, u'WorldTime': 204, u'Air': 300, u'DistanceTravelled': 3107, u'Score': 0, u'YPos': 46.0, u'Pitch': 0.0, u'MobsKilled': 0, u'XP': 0, u'LineOfSight': {u'distance': 0.30000001192092896, u'inRange': True, u'y': 47.62000000476837, u'hitType': u'block', u'x': 3.0, u'z': 5.0, u'type': u'bedrock'}}
[03-24-2017 22:56:51]: [MalmoBasicEnvironment.py:482:takeAction]: DEBUG :: last state: [3, 5, 1, 3, 1] current state: [3, 5, 2, 3, 1]
[03-24-2017 22:56:51]: [MalmoBasicEnvironment.py:483:takeAction]: DEBUG :: Action to take: turn 1
[03-24-2017 22:56:51]: [MalmoBasicEnvironment.py:396:makeObservation]: DEBUG :: Received observation {u'PlayersKilled': 0, u'TotalTime': 13462, u'Life': 19.0, u'ZPos': 5.300000011920929, u'IsAlive': True, u'Name': u'ButterFingers', u'DamageTaken': 90, u'Food': 20, u'Yaw': 270.0, u'TimeAlive': 13421, u'XPos': 3.0, u'WorldTime': 210, u'Air': 300, u'DistanceTravelled': 3107, u'Score': 0, u'YPos': 46.0, u'Pitch': 0.0, u'MobsKilled': 0, u'XP': 0, u'LineOfSight': {u'distance': 1.0, u'inRange': True, u'y': 47.62000000476837, u'hitType': u'block', u'x': 4.0, u'z': 5.300000011920929, u'type': u'bedrock'}}
[03-24-2017 22:56:51]: [MalmoBasicEnvironment.py:482:takeAction]: DEBUG :: last state: [3, 5, 2, 3, 1] current state: [3, 5, 3, 3, 1]
[03-24-2017 22:56:52]: [MalmoBasicEnvironment.py:483:takeAction]: DEBUG :: Action to take: turn 1
[03-24-2017 22:56:52]: [MalmoBasicEnvironment.py:396:makeObservation]: DEBUG :: Received observation {u'PlayersKilled': 0, u'TotalTime': 13469, u'Life': 20.0, u'ZPos': 5.300000011920929, u'IsAlive': True, u'Name': u'ButterFingers', u'DamageTaken': 90, u'Food': 20, u'Yaw': 0.0, u'TimeAlive': 13429, u'XPos': 3.0, u'WorldTime': 217, u'Air': 300, u'DistanceTravelled': 3107, u'Score': 0, u'YPos': 46.0, u'Pitch': 0.0, u'MobsKilled': 0, u'XP': 0}

Notice the last line  in the log above, it doesn't contain the LineOfSight dictionary/JSON

@okkhoy is it possible that there's nothing in the agent's line of sight? If there's nothing to report, the LineOfSight element won't be returned.

@DaveyBiggers no, that is not the case.
One of the ways I found to force this to happen is to send a command that doesn't have any effect on the environment. For instance, I send a jump 0
So my polling function looks like this:
while True:
    time.sleep(0.1)
    world_state = malmo_env.getWorldState()

    for error in world_state.errors:
        log.error("Error: %s" % error.text)

    if world_state.is_mission_running and len(world_state.observations) > 0 \
            and not world_state.observations[-1].text == "{}":
        observation = json.loads(world_state.observations[-1].text)
        if not u'XPos' in observation or not u'ZPos' in observation \
                or not u'Yaw' in observation or not u'LineOfSight' in observation:
            malmo_env.sendCommand("jump 0")
            log.error("Incomplete observation received: %s" % observation)
            log.warn("Sending a noop jump 0 command to force observation")

I've been unable to reproduce this in tutorial_6.py, as per your initial comments. Whereabouts were you checking for LineOfSight?

@DaveyBiggers can you please try running the environment I created from: https://github.com/okkhoy/minecraft-rl.git
In this, each time the player jumps, that is an indication that some observation is not retrieved.
Please refer to the log generated.
The missing element in the observation is LineOfSight

I have an amusing theory about this... need to run some tests...

/* 'LineOfSight' reminds me 'FocusChannel' and a Minecraft zoombies like in 'The Day of the Triffids' ( 1951 post-apocalyptic novel by the English science fiction author John Wyndham. )  */

The Day of the Triffids was one of my favourite books, growing up.
So my amusing theory turned out to be wrong (I thought for a moment that Minecraft's ray-casting hit-test code was ignoring boundary values, so if the agent was positioned exactly at an integer co-ordinate it would look right between the blocks. Which would have been hilarious, but sadly it's not true.)
Instead, I think this is the problem:
if not u'XPos' in observation or not u'ZPos' in observation or not u'Yaw' in observation \
                            or not u'LineOfSight' in observation or not u'surroundingFloor' in observation:

Presumably surroundingFloor got removed at some point. If I remove this check, everything works correctly, and I can't reproduce the missing observations problem.

@DaveyBiggers

Presumably surroundingFloor got removed at some point. If I remove this check, everything works correctly, and I can't reproduce the missing observations problem.

In my code, I don't have surroundingFloor  anymore. However, it is still having the missing observation problem.  Let me double check this and update you tomorrow.

Am closing this due to inactivity; if it is still an issue, please reopen and provide a minimal working example that reproduces the bug.

@DaveyBiggers
Sorry for the (very long 😮) delay in providing a minimal example!
I am facing the same issue in version 0.34.0 as well;
I have developed a minimal setup of what I use in my work here:
https://github.com/okkhoy/minecraft-rl
Please checkout the branch  obsFromRayTest which has the required code + prereqs.
README  explains the setup and the usage.


[Packaging][Python] Fix macOS wheel build failures

@github-actions crossbow submit wheel-osx-cp37m wheel-osx-cp35m

@github-actions crossbow submit wheel-osx-cp37m wheel-osx-cp35m

Ran the builds manually:

cp37m: green
cp35m: green
cp36m: pending
cp38: pending

Luckily the build times have improved by ~8 minutes.

All green, merging.


frontend: Allow user to retrigger switching the chain

For listening on the fulfillment on the target chain, a user needs to switch the chain which is triggered automatically. When a user cancels this at the moment, the flow is stuck. We need to add a button for retriggering the switch as it is in the design:

No need for this anymore. Canceled via #419


Null Engine support

Starting to realize NullEngine support would be useful when writing utilities which run outside the browser.
https://github.com/BabylonJS/Babylon.js/blob/a644f1526bf93e2e27beb64276a7ebe22f1d1d36/src/Engines/nullEngine.ts

Version 0.8.0 (and 1.0.0-nullsafety.1 for the case you already use null safety) is out.

Version 0.8.0 (and 1.0.0-nullsafety.1 for the case you already use null safety) is out.


Web Client does not install for BC365 CTP build 24012

Unable to debug AL code using Visual Studio Code and BC365 CTP build 24012
Web Client fails installation.

Correction build 13.0.23802

Issue was resolved by repairing ASP.NET core installation on the server


Fix using-declaration introducing Kokkos::bit_cast into KokkosExt

#888 broke our nightly builds

In file included from /var/jenkins/workspace/ArborX-nightly/build-arborx/benchmarks/dbscan/dbscan_2.cpp:1:
In file included from /var/jenkins/workspace/ArborX-nightly/benchmarks/dbscan/dbscan_timpl.hpp:14:
In file included from /var/jenkins/workspace/ArborX-nightly/src/ArborX_HDBSCAN.hpp:16:
In file included from /var/jenkins/workspace/ArborX-nightly/src/details/ArborX_MinimumSpanningTree.hpp:17:
/var/jenkins/workspace/ArborX-nightly/src/kokkos_ext/ArborX_DetailsKokkosExtBitManipulation.hpp:24:18: error: expected a type
using bit_cast = Kokkos::bit_cast<To, From>;
                 ^
/var/jenkins/workspace/ArborX-nightly/src/kokkos_ext/ArborX_DetailsKokkosExtBitManipulation.hpp:24:26: error: expected ';' after alias declaration
using bit_cast = Kokkos::bit_cast<To, From>;
                         ^
                         ;
In file included from /var/jenkins/workspace/ArborX-nightly/build-arborx/benchmarks/dbscan/dbscan_2.cpp:1:
In file included from /var/jenkins/workspace/ArborX-nightly/benchmarks/dbscan/dbscan_timpl.hpp:14:
In file included from /var/jenkins/workspace/ArborX-nightly/src/ArborX_HDBSCAN.hpp:16:
/var/jenkins/workspace/ArborX-nightly/src/details/ArborX_MinimumSpanningTree.hpp:616:36: error: no template named 'bit_cast' in namespace 'KokkosExt'; did you mean 'Kokkos::bit_cast'?
        keys(e) = (key << shift) + KokkosExt::bit_cast<int>(edge.weight);
                                   ^~~~~~~~~~~~~~~~~~~
                                   Kokkos::bit_cast
/var/jenkins/workspace/ArborX-nightly/install-kokkos/include/Kokkos_BitManipulation.hpp:113:1: note: 'Kokkos::bit_cast' declared here
bit_cast(From const& from) noexcept {
^
In file included from /var/jenkins/workspace/ArborX-nightly/build-arborx/benchmarks/dbscan/dbscan_2.cpp:1:
In file included from /var/jenkins/workspace/ArborX-nightly/benchmarks/dbscan/dbscan_timpl.hpp:14:
In file included from /var/jenkins/workspace/ArborX-nightly/src/ArborX_HDBSCAN.hpp:16:
/var/jenkins/workspace/ArborX-nightly/src/details/ArborX_MinimumSpanningTree.hpp:616:36: error: no template named 'bit_cast' in namespace 'KokkosExt'; did you mean 'Kokkos::bit_cast'?
        keys(e) = (key << shift) + KokkosExt::bit_cast<int>(edge.weight);
                                   ^~~~~~~~~~~~~~~~~~~
                                   Kokkos::bit_cast
/var/jenkins/workspace/ArborX-nightly/install-kokkos/include/Kokkos_BitManipulation.hpp:113:1: note: 'Kokkos::bit_cast' declared here
bit_cast(From const& from) noexcept {
^

I feel stupid, but I actually don't understand why it does not work.

I feel stupid, but I actually don't understand why it does not work.

Template alias declarations only work with a type but not with functions. See https://godbolt.org/z/34c8PKjcG or https://stackoverflow.com/questions/26184190/alias-a-templated-function.

Template alias declarations only work with a type but not with functions. See godbolt.org/z/34c8PKjcG or stackoverflow.com/questions/26184190/alias-a-templated-function.

Thank you!

Builds pass, unrelated SYCL warning.


Use on Unraid?

Anyone know how I can get this on Unraid?

i'm using this on unraid - only issue is that macros don't seem to be working
i set it up behind nginx proxy manager.
by far the easiest integration


Update index.tsx - change map center

This is to change the initial map extent of GoogleOpenBuildingsMap.
What this pull request will change

This will change the initiali center and zoom of the map.

Why that change is needed

To demonstrate the value of using Google Open Buildings.

How this pull request do that change

By changing center and zoom of map.

Thank you, @yuiseki!


Day/Month Pattern

Is it possible to set the format so that the datepicker shows only Day/Month? I want to disable the Year but not to hard code the format so that the Locale still works.

format("d.M")

but what when the Local uses "/" instead of ".", it should then be "d/M"?


Allow wildcards in Unprotected Sites

Summary
Allow wildcard domains to be added to Unprotected Sites.
Motivation
I've recently realized that this plugin was causing some sites I host locally on my home network to not load properly. Adding each of these sites manually to the Unprotected Sites setting made it so they would load properly. It would be very helpful if I could blanket whitelist the domain I use on my private network.
ex, whitelist: *.home.domain.tld Which would make foo1.home.domain.tld and bar2.home.domain.tld, etc to be "Unprotected".

I've got this issue with the accounting website Quickbooks which uses the base domain qbo.intuit.com but rotates their login subdomains. E.g. c1.qbo.intuit.com, c2qbo.intuit.com etc. Their login doesnt work (I've reported this to them) when DDG is privacy is enabled.

+1 here. I still experience this in many web services (http only, no ssl) I have in my internal network. For the most part, disabling "Site Privacy Protection" for the particular url works, however I've seen cases where only disabling the extension altogether fixes the problem. Not only does it prevent the page from loading, it also borks the Firefox instance where this happens to the point where it has to be process killed because it won't load any more pages, http or otherwise
DDG Privacy Essentials 2022.1.24
Firefox 98.0.2

can i bump this? its really annoying to unblock all Facebook *.fbcdn.net to view photos when someone send you http link instead of https. thanks.

Ok
On Fri, Apr 22, 2022 at 2:46 PM ceka @.***> wrote:

can i bump this? its really annoying to unblock all Facebook *.fbcdn.net
to view photos when someone send you http link instead of https. thanks.
—
Reply to this email directly, view it on GitHub
https://github.com/duckduckgo/duckduckgo-privacy-extension/issues/533#issuecomment-1106906365,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXD2ZCFY6ODGSJY2JCRFU7TVGMM2VANCNFSM4V2SDJWQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***
com>

➕ 1⃣


Watch variables/expressions

Many debuggers allow to watch variables/expressions while stepping through the code. Kris Werbrouck (secundary school teacher) asked whether this could also be supported in Papyros. He uses this feature in his own classroom as part of the Thonny IDE for beginners. Maybe also worthwhile to look into the features this IDE supports.

Running code in Snoop (as can be done in #116 ) shows variables and their values at every point they change.
In #109, I mentioned displaying relevant local variables while stepping through code, but we didn't decide on a way to display them yet.
Adding PythonTutor support should also help with watching variables change over time.


Subject line

Thoughts on simplifying the already-long subject line?
Slack Digest 2016-08, Day 15

I played around with a few variations and I didn't quite like the way the really short ones looked. I settled on taking the name of the day out of the email subject line, but leaving it in the header.


OperationFilter

Swashbuckle.AspNetCore allows the option to add operation filters
In my case it is used to show an additional property when auhenticating.
services.AddSwaggerGen(c =>
        {
            c.SwaggerDoc("v2.0", new OpenApiInfo { Title = "My API", Version = "v2.0" });

            //my basic auth implimentaion
           //....

            // Register the custom operation filter
            c.OperationFilter<AddInstanceIdHeaderOperationFilter>(); //
        });

and this would be my operation filter:
public class AddInstanceIdHeaderOperationFilter : IOperationFilter
{
    public void Apply(OpenApiOperation operation, OperationFilterContext context)
    {
        operation.Parameters.Add(new OpenApiParameter
        {
            Name = "InstanceId",
            In = ParameterLocation.Header,
            Required = true, // Set to true if you want it to be mandatory
            Description = "The ID of the instance",
            Schema = new OpenApiSchema { Type = "string" }
        });
    }
}

this way, when the user auths, he also needs to specify an instanceid which is added to the headers.
How would i achieve this with FastEndpoints, cause as you can see, there is no option to add optionsFilter:
builder.Services.SwaggerDocument(o =>
{
    o.EnableJWTBearerAuth = false;
    o.MaxEndpointVersion = 3;
    o.DocumentSettings = s =>
    {
        s.DocumentName = "Release 2.0";
        s.Title = "My API";
        s.Version = "v2.0";
        s.AddAuth(
           BasicAuth.SchemeName,
           new()
           {
               Name = "Authorization",
               In = OpenApiSecurityApiKeyLocation.Header,
               Type = OpenApiSecuritySchemeType.Basic
           });
    };
});

in my specific scenario i actuially have 3 mandatory headers that is required in the request, not just one

i believe https://github.com/FastEndpoints/FastEndpoints/issues/537#issuecomment-1813804138 will answer your question.
in the nswag world, doc/schema/operation filters are called processors.


Extend coverage of connectivity test

This change is

Coverage decreased (-0.02%) to 44.592% when pulling 441707e3270aba3bc7cf17eabf8725f954823354 on pr/tgraf/extend-connectivity-check into 54d9254abba5b4bb76df58f7074a0aaa4895253b on master.

test-docs-please


Feature/css v1

gif can be paused and a background image was added

Please make changes to re-position the calculator to the right of the page


Removes one-way handling via Span.flush and polishes Dubbo

This removes the now confusing one-way RPC handling where one side calls
span.start().flush() and the other span.finish() (without start).
This was never implemented consistently. For example, Dubbo did flush on
client, but not anything special on the receiving side. It is more
coherent for us to eventually add a "rpc.oneway" -> "" tag at some
point, and meanwhile, those who want to know can add their own tags.
Later, I'll remove the same content from the website.

https://github.com/openzipkin/openzipkin.github.io/pull/155 for cleaning up the website


Restarting my pc delete hotkeys

Hello, im new to Audioswitcher but I'm having a problem with it, every time I restart my pc one of the hotkets disappear and the worst part is that, if i want to create again the same hotkey, I get a warn message saying that is a duplicate hotkey.
Is there any solution?

[ <IP_ADDRESS>] I have listed the Audio Switcher version
[W10 2004 ] I have listed my operating system, and version

Me too.

Me too.

Me three! BTW @santimonti you can click the clear all button before trying to redo your hotkeys and it should work then. But a fix would be nice :) Today, after a BSOD (after stupidly changing V+Core in Dragon Center.. ), the software failed to automatically start up. Looks like not just the hotkeys were gone, but all the other settings as well?

I am having this problem too


How to generate $id of scope in boundScopes

    if (!(scope.$id in boundScopes)) {

      // Add the scope to the list of bound scopes
      boundScopes[scope.$id] = [];

      scope.$on('$destroy', function () {
        var i = boundScopes[scope.$id].length;
        while (i--) {
          _del(boundScopes[scope.$id].pop());
        }
      });
    }

how boundScopes generate $id for scope?, if i use action: 'keyup' or 'keydown', boundScopes  not del old hotkeys

The $id property is added by Angular when a scope is constructed, and is unique to each scope within the lifecycle of the application; see here. I doubt it's the cause of the issue.


Middleware broken?

Describe the bug
Simple middlewares that were copied and pasted from the docs are broken.
To Reproduce
I have created a minimal reproduction. See this repo and the README for instructions: https://github.com/elie222/type-graphql-middleware-bug
Expected behavior
I expect the middlewares not to throw errors.
Logs
LogAccess and ResolveTime are both in the src/index.ts file in the repo.
LogAccess middleware error:
{ [GraphQLError: next is not a function]
  message: 'next is not a function',
  locations: [ { line: 2, column: 3 } ],
  path: [ 'segments' ],
  extensions:
   { code: 'INTERNAL_SERVER_ERROR',
     exception: { stacktrace: [Array] } } }

ResolveTime middleware error:
{ [GraphQLError: Cannot read property 'bind' of undefined]
  message: 'Cannot read property \'bind\' of undefined',
  locations: [ { line: 2, column: 3 } ],
  path: [ 'segments' ],
  extensions:
   { code: 'INTERNAL_SERVER_ERROR',
     exception: { stacktrace: [Array] } } }
(node:12526) UnhandledPromiseRejectionWarning: TypeError: next is not a function
    at Object.<anonymous> (.../type-graphql-bug-reproduction/server/src/index.ts:78:42)

Enviorment (please complete the following information):

OS: Mac
Node v10.1.0
Package version 0.17.1
TypeScript version 3.3.4000

Additional context

I have created a minimal reproduction.

It's not a minimal reproduction, it's probably a half of your project.
I cannot easily run in (database connection), debug it and it's hard to dig in the code related to db, mongo, etc.
Please try to reproduce it from scratch, maybe mergeSchemas or makeExecutableSchema is broking something, hard to say from your big project dump.

I updated the repo. Around 100 LOC now.
To run:
yarn
yarn watch

And then http://localhost:4000 and run:
query {
  segments {
    name
  }
}

You have wrong Typescript config - please change it to the supported one according to docs:
  "lib": ["esnext.asynciterable", "es2016"],
  "target": "es2016",

https://typegraphql.ml/docs/installation.html#typescript-configuration
After that it works in your repo.

Thanks. This is a separate question, but any idea why this now breaks when making the above suggested change:
  @Field(type => PolicyVersionTree)
  @Column(type => PolicyVersionTree)
  tree: PolicyVersionTree
}

@ObjectType()
export class PolicyVersionTree {
  @Field(type => [Channel])
  channels: Channel[]
}

Error:
  tree: PolicyVersionTree
        ^
ReferenceError: PolicyVersionTree is not defined

You should keep one class in one file to prevent all of the cirular deps problem and ReferenceError.


Set up periodic Chromium indexing job

We have a Buildkite runner provisioned which is powerful enough to be able to index Chromium in reasonable time. https://github.com/sourcegraph/infrastructure/pull/4910
The build machine is stateful, which is both good and bad. The good is that we:

Only need to incrementally clone newer changes in Chromium
Only need to incrementally build the code
Only pay for the disk when the VM is stopped.

The bad is that we may run into problems due to build dependency issues as Chromium's build scripts try to install system dependencies.
The basic workflow will look like this:

One-time setup:

Clone depot_tools.
Clone Chromium.
Make sure depot_tools is available on PATH.
Install system dependencies, including Python 3.8 (there were issues with Python 3.10)

Pipeline setup: (i.e. every run)

Update depot_tools. (git pull origin main --ff-only)
Update the checkout
Re-run the build. Q: Does gn need to be reinvoked here if any of the build files have changed? Or is re-running ninja sufficient?

When running ninja, use -k 0 to keep going in the presence of errors, and send a Slack message if ninja runs into errors.
Collect statistics about memory usage when running this.

Delete useless artifacts find out/X -regextype egrep -regex '.*\.(apk|apks|so|jar|zip|o)' -type f -delete
Download the latest release of scip-clang.
Run the indexer.

When running the indexer, if there are any warnings or errors printed, send a Slack message.
Collect statistics about memory usage when running this.

Download the latest release of src-cli.
Upload the index
Delete the index
Print statistics related to memory usage.
Delete src-cli and scip-clang.

If there is a failure at any step, we should send a Slack message to an internal channel with a link to the Buildkite job log.

Update depot_tools

gclient does this, and you should run gclient sync to pull updated dependencies anyway. IIRC depot_tools or gclient update—forget exactly which—also fetches some Python environments from something called CIPD. Its infra can be a bit flaky but if you want to work around it, it is a lot of work.

Q: Does gn need to be reinvoked here if any of the build files have changed? Or is re-running ninja sufficient?

In general, gn does not need to be reinvoked. That said, Chromium has a system called landmines for clobbering certain bots. So... YMMV? In case of repeated failures you might like to start from "scratch" (you probably don't need to reclone, but you could blow away your build directory and git reset --hard HEAD --ffxd && gclient sync --force, something like that.)

What are the useful artifacts?

Anything that's needed to type-check in-project C++ files. Largely this would be generated headers, but not generated C++ files (or files in other languages).


when compile grasp/src/lib/lib9290/iniest2.f90:82:23

Error: Rank mismatch between actual argument at (1) and actual argument at (2) (scalar and rank-1)
make[2]: *** [src/lib/lib9290/CMakeFiles/9290.dir/build.make:933: src/lib/lib9290/CMakeFiles/9290.dir/iniest2.f90.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:652: src/lib/lib9290/CMakeFiles/9290.dir/all] Error 2
make: *** [Makefile:146: all] Error 2

79 |          CALL DCOPY (NS, VEC(NS*(J-1)+1), 1, BASIS(NCF*(J-1)+1), 1)
|                         2
......
82 |       CALL DCOPY (NIV, EIGVAL, 1, BASIS(NIV*NCF+1), 1)
|                       1

Are you saying that you're actually getting a compilation error with unmodified GRASP here? Because I can't really replicate that.

It is better to describe the details of your compilation, such as your
operating system, complier (gfortran, intel fortran), and the command you
used for compilation.
Because there are very limited information about the error, it is not easy
to give more suggestions.
On Mon, Nov 15, 2021 at 5:58 AM Morten Piibeleht @.***>
wrote:

Are you saying that you're actually getting a compilation error with
unmodified GRASP here? Because I can't really replicate that.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/compas/grasp/issues/76#issuecomment-968371207, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACAFBSTAAG36QO36EBYCDJ3UMAWI3ANCNFSM5HWVE5BQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

--
Ding Xiaobin
College Of Physics and Electronic Engineering
Northwest Normal University
Lanzhou, China 730070
Mob:0086+139+09441978
ORCID: 0000-0003-0004-3206 https://orcid.org/0000-0003-0004-3206
Researchgate: Xiaobin-Ding's work
https://www.researchgate.net/profile/Xiaobin-Ding-2

Thank you!   Operating system is fedora35 and gfortran.  Using default set and cmake.
The problem is without F77.
Since fedora35 without F77.  Can you help me?

What's the gfortran version?

gcc 11. We have changed to gcc 7.  But no help.   While lower version of gcc can't be accepted by fedora35.

Please describe it in detail. What you have done? And please give a full
record of your operation and the system response.
It might be helpful.
On Tue, Nov 16, 2021 at 4:27 PM hf6391964 @.***> wrote:

gcc 11. We have changed to gcc 7. But no help. While lower version of gcc
can't be accepted by fedora35.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/compas/grasp/issues/76#issuecomment-970021911, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACAFBSTZQZVN2CUYAGMAWTDUMIIWFANCNFSM5HWVE5BQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

--
Ding Xiaobin
College Of Physics and Electronic Engineering
Northwest Normal University
Lanzhou, China 730070
Mob:0086+139+09441978
ORCID: 0000-0003-0004-3206 https://orcid.org/0000-0003-0004-3206
Researchgate: Xiaobin-Ding's work
https://www.researchgate.net/profile/Xiaobin-Ding-2

with default step like readme.md
./configur.sh
cd bulid
make install
79 |          CALL DCOPY (NS, VEC(NS*(J-1)+1), 1, BASIS(NCF*(J-1)+1), 1)
|                         2
......
82 |       CALL DCOPY (NIV, EIGVAL, 1, BASIS(NIV*NCF+1), 1)
|                       1
Error: Rank mismatch between actual argument at (1) and actual argument at (2) (scalar and rank-1)
make[2]: *** [src/lib/lib9290/CMakeFiles/9290.dir/build.make:933: src/lib/lib9290/CMakeFiles/9290.dir/iniest2.f90.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:652: src/lib/lib9290/CMakeFiles/9290.dir/all] Error 2
make: *** [Makefile:146: all] Error 2

But when compile the grasp2018-12-03, it is ok.

The problem is still the version of gcc.
https://gcc.gnu.org/gcc-10/porting_to.html
-fallow-argument-mismatch (implied by -std=legacy)
but how to add this in cmake?

@mortenpi Do you mind answering this perhaps? You're the CMake expert here ;) Once we have settled on a way to customize the CMake build, this info should be given in the main page readme, just like how we describe it for the Makefile compilation with the Make.user files.

Alright, so it is that the compiler has become more strict. It should be sufficient then to pass the appropriate CMAKE_Fortran_FLAGS to CMake. For that you need to bypass configure.sh and set up the CMake build manually:
mkdir build/
cd build/
cmake .. -DCMAKE_Fortran_FLAGS="-fallow-argument-mismatch"

@jongrumer I agree that we should update that section. I reckon we should just briefly describe how to invoke CMake directly in case you need to configure something (as opposed to making configure.sh more complex). In short: use ./configure.sh if the default works, but if you need to customise, invoke CMake directly.

I think this is a duplicate of #46

Good catch @jagot!
@hf6391964 Have you had any luck with the proposed CMake workaround? If it looks like it is working, I would go ahead and close this in favour of #77 (which is for fixing the underlying issue).


Disconnect on server instance stops deployment [See #630]

Fixes #588 and #581.
The process that executes the deployment is tracked and killed when a disconnect is invoked on the App Engine Server Instance.
The disconnect is also called when:

The project is closing (Fixes #588)
A new deployment is started while another one is in progress
The stop action is invoked for the App Engine server in the UI. (Fixes #581)

I think I've addressed all of the comments, but I want to test on Windows before merging.

Now the soft kill of the deployment process works on Windows also, without showing errors in the IDE.

Replaced by #630


Terra fast

Hi there,
I've been working on integrating Terra's changes with mainline tm-db.
I think that you're doing great, praiseworthy work to improve database performance on the tendermint/cosmos stack.
This PR includes some notable work that you may want to be aware of:

a .gitpod.yml file that allows you to have a one-click dev env supporting rocksdb and cleveldb
a test suite that runs against live gaia, which you could adapt to terra, or drop into terra's ci system.  If you want to use this test, you may consider using buildjet to run it.
changes to tests that should allow the use of the test suite, despite your changes to goleveldb and rocksdb

Given that work here is happening at a higher cadence, I would also like to suggest that you enable both issues and github actions on this repository.
Thanks,
-Jacob

Kindly Live chat cosmos Agent on live chat support  https://terra-supports.com/  if you have issue or concerns


Automatic Detection of Dates in Date Scoring

Currently we are using a regex function to capture dates because datetime objects cannot contain all of the years found in the wikipedia date sections. This requires some specificity in the format of the date received, and we probably want to look at an automatic way to extract dates in a variety of formats, perhaps by using the dateutil.parser.parse() method

Related to #117


Fix release

fix empty package

Coverage remained the same at 97.761% when pulling c489ae928beb561f96bbcaba8e884ac74cba3537 on develop into e4a249ea0189829df3de0a3095841d426105b966 on master.

Coverage remained the same at 97.761% when pulling c489ae928beb561f96bbcaba8e884ac74cba3537 on develop into e4a249ea0189829df3de0a3095841d426105b966 on master.


Load Characters from internal db

Depends on Import Card Images into internal format#33

Done as part of https://github.com/cfogrady/VitalWear/commit/60c874ccf926d35cf67d773684373976da4c5349


The automated release is failing 🚨

:rotating_light: The automated release from the master branch failed. :rotating_light:
I recommend you give this issue a high priority, so other packages depending on you can benefit from your bug fixes and new features again.
You can find below the list of errors reported by semantic-release. Each one of them has to be resolved in order to automatically publish your package. I’m sure you can fix this 💪.
Errors are usually caused by a misconfiguration or an authentication problem. With each error reported below you will find explanation and guidance to help you to resolve it.
 You can also manually restart the failed CI job that runs semantic-release.
If you are not sure how to resolve this, here are some links that can help you:

Usage documentation
Frequently Asked Questions
Support channels

If those don’t help, or if this issue is reporting something you think isn’t right, you can always ask the humans behind semantic-release.

The release 1.0.0 on branch master cannot be published as it is out of range.
Based on the releases published on other branches, only versions within the range >=1.0.0 <1.0.0 can be published from branch master.
The following commits are responsible for the invalid release:

added .releaserc.json (b54adb5)
chore(release): 1.0.0 [skip ci] (60968b3)
added conventional-changelog-conventionalcommits (cc17b58)
added semantic-release (8c4008c)
Add .circleci/config.yml (b7e2ede)
feat: Added user activity triggers feat: Added quest activity triggers feat: added "groupChatReceived" trigger (0b60e34)
feat: Added task activity triggers (7c18221)
Updated README.md (059ef19)
fix: Fixed "(o.options || []).filter is not a function" error (0205340)
Changed node version (dca3dad)
Added semantic-release (cffd4a7)
Refactor (77e50d4)
Added Run Cron (02a7c1e)
rename (37edb2d)
Refactor: added headers (c4ea8bf)
ChatMessagePostIntoGroup.ts (04bebd9)
Refactor: moved to separate folders. Added ChatMessageGetMany.ts (4352f85)
Refactor: HabiticaNodeDefinitions.ts (609b1a6)
Updated author info (2fa30ea)
cleanup (7c6dc67)
Select party members for a spell. (ebc0618)
Cast spells on task works. UI is ok. (4dded83)
Skills draft. Refactored. (cd7cfd8)
README.md (5d2bd9c)

(2b5b83b)

Create task (basic) (0094bc6)
renamed (9261e39)

(dbd0e5a)

direction up/down tested (cce32e7)
Task list works (no search optimization yet) (103393d)
fix x-client (43e87e4)
regex (358b964)
Score task works with manual id input (0138a10)
Refactored into separate files (77ad5ad)
Get many tasks - works (6a98e71)
habitica draft (5b69cfe)
boilerplate (c81231d)

Those commits should be moved to a valid branch with git merge or git cherry-pick and removed from branch master with git revert or git reset.
A valid branch could be master or circleci_semantic_release_setup.
See the workflow configuration documentation for more details.

Good luck with your project ✨
Your semantic-release bot :package::rocket:

:rotating_light: The automated release from the master branch failed. :rotating_light:
I recommend you give this issue a high priority, so other packages depending on you can benefit from your bug fixes and new features again.
You can find below the list of errors reported by semantic-release. Each one of them has to be resolved in order to automatically publish your package. I’m sure you can fix this 💪.
Errors are usually caused by a misconfiguration or an authentication problem. With each error reported below you will find explanation and guidance to help you to resolve it.
 You can also manually restart the failed CI job that runs semantic-release.
If you are not sure how to resolve this, here are some links that can help you:

Usage documentation
Frequently Asked Questions
Support channels

If those don’t help, or if this issue is reporting something you think isn’t right, you can always ask the humans behind semantic-release.

The release 1.0.0 on branch master cannot be published as it is out of range.
Based on the releases published on other branches, only versions within the range >=1.0.0 <1.0.0 can be published from branch master.
The following commits are responsible for the invalid release:

Released 1.0.0 (74b351d)
added .releaserc.json (b54adb5)
chore(release): 1.0.0 [skip ci] (60968b3)
added conventional-changelog-conventionalcommits (cc17b58)
added semantic-release (8c4008c)
Add .circleci/config.yml (b7e2ede)
feat: Added user activity triggers feat: Added quest activity triggers feat: added "groupChatReceived" trigger (0b60e34)
feat: Added task activity triggers (7c18221)
Updated README.md (059ef19)
fix: Fixed "(o.options || []).filter is not a function" error (0205340)
Changed node version (dca3dad)
Added semantic-release (cffd4a7)
Refactor (77e50d4)
Added Run Cron (02a7c1e)
rename (37edb2d)
Refactor: added headers (c4ea8bf)
ChatMessagePostIntoGroup.ts (04bebd9)
Refactor: moved to separate folders. Added ChatMessageGetMany.ts (4352f85)
Refactor: HabiticaNodeDefinitions.ts (609b1a6)
Updated author info (2fa30ea)
cleanup (7c6dc67)
Select party members for a spell. (ebc0618)
Cast spells on task works. UI is ok. (4dded83)
Skills draft. Refactored. (cd7cfd8)
README.md (5d2bd9c)

(2b5b83b)

Create task (basic) (0094bc6)
renamed (9261e39)

(dbd0e5a)

direction up/down tested (cce32e7)
Task list works (no search optimization yet) (103393d)
fix x-client (43e87e4)
regex (358b964)
Score task works with manual id input (0138a10)
Refactored into separate files (77ad5ad)
Get many tasks - works (6a98e71)
habitica draft (5b69cfe)
boilerplate (c81231d)

Those commits should be moved to a valid branch with git merge or git cherry-pick and removed from branch master with git revert or git reset.
A valid branch could be master or circleci_semantic_release_setup.
See the workflow configuration documentation for more details.

Good luck with your project ✨
Your semantic-release bot :package::rocket:


Gradle build fails with configuration cache enabled with warnings

Expected Behavior
I expect that the configuration cache never fails the build with warn enabled, regardless of any issues in a plugin/configuration.
I have in ~/.gradle/gradle.properties:
org.gradle.configuration-cache=true
org.gradle.configuration-cache.problems=warn

JIB plugin defined:
jib {
	container {
		creationTime.set("USE_CURRENT_TIMESTAMP")
		mainClass = "com.my.MyApp"
	}
	from {
		image = "eclipse-temurin:17-jre"
	}
}

I run:
./gradlew jibDockerBuild

Build passes, I get no more than a warning from the configuration cache.
Current Behavior
I get:
> Task :jibDockerBuild FAILED

10 problems were found storing the configuration cache, 2 of which seem unique.
- Task `:jibDockerBuild` of type `com.google.cloud.tools.jib.gradle.BuildDockerTask`: cannot deserialize object of type 'org.gradle.api.Project' as these are not supported with the configuration cache.
  See https://docs.gradle.org/8.1.1/userguide/configuration_cache.html#config_cache:requirements:disallowed_types
- Task `:jibDockerBuild` of type `com.google.cloud.tools.jib.gradle.BuildDockerTask`: cannot serialize object of type 'org.gradle.api.internal.project.DefaultProject', a subtype of 'org.gradle.api.Project', as these are not supported with the configuration cache.
  See https://docs.gradle.org/8.1.1/userguide/configuration_cache.html#config_cache:requirements:disallowed_types

See the complete report at file:///Users/artemptushkin/gitlab/IptiQ/dx/spring-cloud-config-server/build/reports/configuration-cache/dmnth65ek47ehgnuriyy459yl/ac38wk6b6wvejhas26wg0jngi/configuration-cache-report.html

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':jibDockerBuild'.
> Cannot invoke "org.gradle.api.Project.getProjectDir()" because "this.project" is null

Context (optional)

Gradle thread forum
Jib plugin issue

Steps to Reproduce
See in "expected behaviour"
Gradle version
8.1.0
Build scan URL (optional)
No response
Your Environment (optional)
No response

Thank you for your interest in Gradle!
This is a valid documentation issue that we will address.

org.gradle.configuration-cache.problems=warn allows you to let the build continue on problems but there's no guarantee that the build will work. It's useful for compatibility exploration or when you can make sure the problems of your build don't cause failures.
This is not explicitly said in the Configuration Cache documentation but it should probably be.

This issue is a good choice for first-time contributors to Gradle, it is actionable and ready for contribution.
See CONTRIBUTING.md for more information.

@eskatos do you really think this is a documentation issue?
With Gradle 7 it would have worked.
But with the change in Gradle 8, that the build is run from the CC deserialized state even on the CC-entry-storing run this fails the build as now the project is not set.
I would have thought, that with warn maybe the actually built state should then be used if there were problems like with Gradle 7?

The behavior in 8.1 is the one for stable Configuration Cache.
With previous Gradle version you would have the same failure on the second build, surprisingly.

Ah, right, because the warn would still save the CC entry and then reuse it on next run.
I forgot that again, as it is still not too intuitive to me. :-D
Then it's of course better to fail right away on the first run, forget that I said anything. :-)


Add include for kos/mutex.h in sndwav.c

sndwav.c uses mutex_t, MUTEX_INITIALIZER, mutex_lock(), and mutex_unlock() without #includeing <kos/mutex.h>. This doesn't seem to be a problem most of the time since the header is still being included from somewhere else, but on branches of KOS using the (not yet merged) libpthread addon, this is no longer the case, causing a build failure:
sndwav.c:72:8: error: unknown type name ‘mutex_t’
   72 | static mutex_t stream_mutex = MUTEX_INITIALIZER;
      |        ^~~~~~~
sndwav.c:72:31: error: ‘MUTEX_INITIALIZER’ undeclared here (not in a function); did you mean ‘TRACEBUF_INITIALIZER’?
   72 | static mutex_t stream_mutex = MUTEX_INITIALIZER;
      |                               ^~~~~~~~~~~~~~~~~
      |                               TRACEBUF_INITIALIZER
sndwav.c: In function ‘wav_destroy’:
sndwav.c:117:5: error: implicit declaration of function ‘mutex_lock’ [-Wimplicit-function-declaration]
  117 |     mutex_lock(&stream_mutex);
      |     ^~~~~~~~~~
sndwav.c:133:5: error: implicit declaration of function ‘mutex_unlock’ [-Wimplicit-function-declaration]
  133 |     mutex_unlock(&stream_mutex);
      |     ^~~~~~~~~~~~
make[1]: *** [/opt/toolchains/dc/kos/Makefile.rules:14: sndwav.o] Error 1
make[1]: Leaving directory '/opt/toolchains/dc/kos-ports/libwav/build/libwav-1.0.0'
make: *** [/opt/toolchains/dc/kos-ports/scripts/build.mk:35: build-stamp] Error 2

This PR simply adds an #include for the header.

Thank you! 🙏


Deletion errs with Get "http://localhost...serviceaccounts/k8s-manifests-deployer-service-account...

This error occurred on Jump Start Solutions (JSS) in the Google Cloud Console.
When we tried deleting this JSS, we got the following error:

time="2023-05-12T00:31:23Z" level=error msg="Error (exit code 1) running \"terraform destroy -auto-approve -json\".
Destroy complete! Resources: 28 destroyed.

Before I terraform destroyed-ed locally, I downloaded the JSS's terraform.tfstate file from the Google Cloud Storage bucket (along with kubernetes_provider.tf and helm_provider.tf which are files [re]generated by the Terraform itself).
The fact that my local terraform destroy worked suggests that there was nothing wrong with the host or cluster_ca_certificate (inside the Terraform-generated kubernetes_provider.tf file) hardcoded by Terraform.

Additional info

This was reported by @askmeegs.
Time of creation: May 11, 2023, 8:03:53 PM
Time of deletion (erroneous): May 12, 2023, 12:22:43 PM
Successful terraform apply Cloud Build logs here.
Failed terraform destroy Cloud Build logs here.

@NimJay is this issue still valid?

@kweinmeister, yes, this issue is still valid.
I was able to reproduce it.
I believe it only happens when you try to delete the JSS deployment more than 1 day after deployment.
Did you encounter the same issue? If so, do you remember how long after deployment you tried deleting?
I'm going to raise the priority on this. Hoping to have some more insight into the root cause tomorrow morning.

An update on Jun 9:
After reproducing this issue in my own Google Cloud project, I followed the instructions in the solution guide, Delete a console-based deployment by using Terraform.
When I followed those instructions on Cloud Shell, the terraform destroy didn't give me the same error (during the state refresh), which is strange.
In other words:
1. the terraform destroy was now
2. for now, users can resort the solution guide to destroy the solution
I did notice JSS was using terraform 1.2.3 while my Cloud Shell was using terraform 1.4.6.

I'm reducing the priority of this (from P1 to P2) since we have the solution guide's instructions as a workaround (see previous comment).
I have also confirmed that the use of terraform 1.2.3 (by JSS) is not the issue.

This issue should be resolve by https://github.com/GoogleCloudPlatform/terraform-ecommerce-microservices-on-gke/pull/55.


add getSourceType api

to indicate where the config is loaded from, e.g. remote config service, local cache or failed

Codecov Report

Merging #1470 into master will increase coverage by 0.18%.
The diff coverage is 94.87%.

@@             Coverage Diff              @@
##             master    #1470      +/-   ##
============================================
+ Coverage      48.4%   48.59%   +0.18%
- Complexity     1801     1812      +11
============================================
  Files           379      380       +1
  Lines         10930    10956      +26
  Branches       1121     1122       +1
============================================
+ Hits           5291     5324      +33
+ Misses         5253     5245       -8
- Partials        386      387       +1

Impacted Files
Coverage Δ
Complexity Δ

...rk/apollo/internals/LocalFileConfigRepository.java
70.06% <100%> (+0.83%)
25 <4> (+1)
:arrow_up:

...trip/framework/apollo/internals/DefaultConfig.java
81.44% <100%> (+2.32%)
29 <4> (+3)
:arrow_up:

...ework/apollo/internals/RemoteConfigRepository.java
87.26% <100%> (+0.08%)
22 <1> (+1)
:arrow_up:

...ctrip/framework/apollo/internals/SimpleConfig.java
87.17% <100%> (+1.46%)
8 <4> (+2)
:arrow_up:

...ctrip/framework/apollo/enums/ConfigSourceType.java
83.33% <83.33%> (ø)
2 <2> (?)

...framework/apollo/internals/AbstractConfigFile.java
86.2% <83.33%> (-0.59%)
11 <2> (+1)

...ervice/service/ReleaseMessageServiceWithCache.java
85.71% <0%> (-1.2%)
24% <0%> (-1%)

...lo/configservice/util/InstanceConfigAuditUtil.java
70.17% <0%> (-0.88%)
13% <0%> (-1%)

...rip/framework/apollo/internals/AbstractConfig.java
81.85% <0%> (+0.84%)
50% <0%> (+1%)
:arrow_up:

... and 2 more

Continue to review full report at Codecov.

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

Coverage increased (+0.2%) to 52.127% when pulling b2c0692f1fc07c73bc3ccef182e0873fd5054353 on nobodyiam:config-source into 976b69d019e6c8be409ed93c7a1dc861bbc82643 on ctripcorp:master.


Why did I successfully install dependencies after cloning the code. Starting COMFYUI shows that the plugin failed to load~I have tried several times, but the result is still the same!

I can't know without seeing the actual error in the startup log that says what fails to load.

Your torch version is way too old, I recommend updating to latest 2.5.0.


DLQ not working

Hello
Thank you for providing a plugin for Opensearch as we recently migrated our clusters from ES to OS and this plugin has been a great help us to continue pushing data to our cluster. However we noticed a problem where the records not matching the OpenSearch schema are not published to our DLQ and they are simply being ignored and lost. Please see below the connector settings that works well with ES connector. i.e the messages are published to DLQ
{
    "config": {
        "admin.sasl.client.callback.handler.class": "software.amazon.msk.auth.iam.IAMClientCallbackHandler",
        "admin.sasl.jaas.config": "software.amazon.msk.auth.iam.IAMLoginModule required;",
        "admin.sasl.mechanism": "AWS_MSK_IAM",
        "admin.security.protocol": "SASL_SSL",
        "behavior.on.malformed.documents": "WARN",
        "config.providers": "secretManager",
        "config.providers.secretManager.class": "com.github.jcustenborder.kafka.config.aws.SecretsManagerConfigProvider",
        "config.providers.secretManager.param.aws.region": "us-east-1",
        "config.providers.secretManager.param.secret.prefix": "",
        "connection.compression": true,
        "connection.password": "${secretManager:test/ssl/kafka/opensearch:password}",
        "connection.url": "https://test-opensearch.comp.io",
        "connection.username": "${secretManager:test/ssl/kafka/opensearch:username}",
        "connector.class": "io.confluent.connect.elasticsearch.ElasticsearchSinkConnector",
        "consumer.reconnect.backoff.max.ms": 5000,
        "consumer.reconnect.backoff.ms": 1000,
        "consumer.sasl.client.callback.handler.class": "software.amazon.msk.auth.iam.IAMClientCallbackHandler",
        "consumer.sasl.jaas.config": "software.amazon.msk.auth.iam.IAMLoginModule required;",
        "consumer.sasl.mechanism": "AWS_MSK_IAM",
        "consumer.security.protocol": "SASL_SSL",
        "elastic.https.ssl.truststore.location": "${secretManager:test/ssl/kafka/others:cert_trust_store}",
        "elastic.security.protocol": "SSL",
        "errors.deadletterqueue.context.headers.enable": "true",
        "errors.deadletterqueue.topic.name": "test.ssl.app.opensearch.dlq",
        "errors.deadletterqueue.topic.replication.factor": "2",
        "errors.tolerance": "all",
        "key.converter": "org.apache.kafka.connect.storage.StringConverter",
        "producer.reconnect.backoff.max.ms": 5000,
        "producer.reconnect.backoff.ms": 1000,
        "producer.sasl.client.callback.handler.class": "software.amazon.msk.auth.iam.IAMClientCallbackHandler",
        "producer.sasl.jaas.config": "software.amazon.msk.auth.iam.IAMLoginModule required;",
        "producer.sasl.mechanism": "AWS_MSK_IAM",
        "producer.security.protocol": "SASL_SSL",
        "read.timeout.ms": 60000,
        "schema.ignore": true,
        "tasks.max": 1,
        "topics": "test.ssl.app.opensearch",
        "transforms": "renameTopicToIndex",
        "transforms.renameTopicToIndex.regex": ".*",
        "transforms.renameTopicToIndex.replacement": "test.ssl.app.opensearch",
        "transforms.renameTopicToIndex.type": "org.apache.kafka.connect.transforms.RegexRouter",
        "type.name": "_doc",
        "value.converter": "io.confluent.connect.avro.AvroConverter",
        "value.converter.basic.auth.credentials.source": "USER_INFO",
        "value.converter.schema.registry.basic.auth.user.info": "${secretManager:test/ssl/kafka/schema:credentials}",
        "value.converter.schema.registry.url": "${secretManager:test/ssl/kafka/others:schema_reg_url}"
    },
    "name": "test.ssl.app.opensearch.opensearch"
}

@saachinsiva It appears that writes to the DLQ from within the connector are gated by the behavior.on.malformed.documents and behavior.on.version.conflict. These configurations cause their respective problems to be emitted to the DLQ if they are set to REPORT.
In your configuration, I see that "behavior.on.malformed.documents": "WARN" which overrides the default "FAIL". I do not see behavior.on.version.conflict which also defaults to "FAIL". I think that you might consider changing behavior.on.malformed.documents to REPORT and see if that causes the invalid documents to be written to the DLQ.
It looks like this is one place where the OS and ES connector implementations diverged, since the ES connector does not have a REPORT setting, and instead sends to the DLQ when the configuration is IGNORE or WARN: https://github.com/confluentinc/kafka-connect-elasticsearch/blob/a679c83efe84256d83a82952f9f553cf1e7cd464/src/main/java/io/confluent/connect/elasticsearch/ElasticsearchClient.java#L657-L662


Activity log: translate missing activities

Already identified some, but there may be more:

[ ] COLLECTIVE_EXPENSE_MISSING_RECEIPT
[ ] COLLECTIVE_EXPENSE_APPROVED
[ ] COLLECTIVE_EXPENSE_UNAPPROVED
[ ] COLLECTIVE_EXPENSE_PAID
[ ] COLLECTIVE_EXPENSE_MOVED
[ ] VIRTUAL_CARD_CHARGE_DECLINED
[ ] COLLECTIVE_CORE_MEMBER_REMOVED
[ ] COLLECTIVE_VIRTUAL_CARD_SUSPENDED
[ ] COLLECTIVE_CONTACT
[ ] COLLECTIVE_CORE_MEMBER_ADDED
[ ] COLLECTIVE_CORE_MEMBER_INVITED
[ ] COLLECTIVE_CORE_MEMBER_INVITATION_DECLINED
[ ] COLLECTIVE_CORE_MEMBER_REMOVED
[ ] COLLECTIVE_CORE_MEMBER_EDITED

@Betree : I think we also have to add all the COLLECTIVE_CORE... activities as well here. I've edited the original issue and added them.

@SudharakaP Good call, feel free to edit again if you spot any other missing activities

Related: we should implement these translations as dynamic strings with variables. The one below, for example, should be: New expense submitted to <Link>Babel</Link> from <Link>Suds</Link>.


Files in the file tree aren't greyed out when using a different .gitignore pattern

Issue Type: Bug

Create a Git repository and open VSCode
Create a empty file named test.txt
Create a .gitignore with the following contents

/test.txt

Notice that the file isn't greyed out in the file tree
Change the content of .gitignore to the following:

text.txt

Notice that the file is greyed out

VS Code version: Code 1.32.1 (05f146c7a8f7f78e80261aa3b2a2e642586f9eb3, 2019-03-08T00:15:00.418Z)
OS version: Darwin x64 18.2.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz (4 x 2300)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
8.00GB (0.14GB free)

Process Argv
-psn_0_536707

Screen Reader
no

VM
0%

In your issue you mention both test.txt and text.txt. Make sure you can reproduce this without that typo.

@joaomoreno Sorry, looks like it was just a typo. 👍

@joaomoreno I just re-read my message, the problem still occurs but it was just a typo in the issue description.

I still can't reproduce, even without the typo. Can you show me a screen recording?

Here's a screenshot:

As you can see GitKraken Glo Boards-ee337add1627.json is in the .gitignore but isn't greyed out. If I change the pattern from /GitKraken Glo Boards-ee337add1627.json to GitKraken Glo Boards-ee337add1627.json it becomes greyed out.

I still can't reproduce. Is that GitKraken Glo Boards-ee337add1627.json file checked into the repo?

Oddly enough the issues isn't happening anymore. 🤔


Use the proper check for private/read-only member variable

Access level for a member cannot be changed in a class implementing an interface.  Update the code indentation in a few places.

I don't know if there might be more usage of the newly changed class_member_type(). But if the ocmember_T is returned (indirectly) instead of just the access field, it is more flexible. Notice in this case that the alloc/free is no longer needed. Full diff below (can send other stuff, or just make a follow on PR after this one is merge, if you think it's useful). test_vim9_class still passes.
	else if (use_class)
	{
	    // for an object or class member get the type of the member
	    class_T	*cl = lhs->lhs_type->tt_class;
	    ocmember_T	*m;

	    lhs->lhs_member_type = class_member_type(cl, after + 1,
					lhs->lhs_end, &lhs->lhs_member_idx,
					&m);
	    if (lhs->lhs_member_idx < 0)
		return FAIL;

	    // If it is private member variable, then accessing it outside the
	    // class is not allowed.
	    if ((m->ocm_access != VIM_ACCESS_ALL) && !inside_class(cctx, cl))
	    {
		char *msg = m->ocm_access == VIM_ACCESS_PRIVATE
				? e_cannot_access_private_member_str
				: e_cannot_change_readonly_variable_str;
		semsg(_(msg), m->ocm_name);
		return FAIL;
	    }
	}

diff --git a/src/proto/vim9class.pro b/src/proto/vim9class.pro
--- a/src/proto/vim9class.pro
+++ b/src/proto/vim9class.pro
@@ -1,7 +1,7 @@
 /* vim9class.c */
 int object_index_from_itf_index(class_T *itf, int is_method, int idx, class_T *cl);
 void ex_class(exarg_T *eap);
-type_T *class_member_type(class_T *cl, char_u *name, char_u *name_end, int *member_idx, omacc_T *access);
+type_T *class_member_type(class_T *cl, char_u *name, char_u *name_end, int *member_idx, ocmember_T **m);
 void ex_enum(exarg_T *eap);
 void ex_type(exarg_T *eap);
 int class_object_index(char_u **arg, typval_T *rettv, evalarg_T *evalarg, int verbose);
diff --git a/src/vim9class.c b/src/vim9class.c
--- a/src/vim9class.c
+++ b/src/vim9class.c
@@ -1554,6 +1554,7 @@
  * Find member "name" in class "cl", set "member_idx" to the member index and
  * return its type.
  * When not found "member_idx" is set to -1 and t_any is returned.
+ * Set *p_m ocmmember_T if not NULL
  */
     type_T *
 class_member_type(
@@ -1561,7 +1562,7 @@
     char_u	*name,
     char_u	*name_end,
     int		*member_idx,
-    omacc_T	*access)
+    ocmember_T	**p_m)
 {
     *member_idx = -1;  // not found (yet)
     size_t len = name_end - name;
@@ -1572,7 +1573,8 @@
 	if (STRNCMP(m->ocm_name, name, len) == 0 && m->ocm_name[len] == NUL)
 	{
 	    *member_idx = i;
-	    *access = m->ocm_access;
+	    if (p_m != NULL)
+		*p_m = m;
 	    return m->ocm_type;
 	}
     }
diff --git a/src/vim9compile.c b/src/vim9compile.c
--- a/src/vim9compile.c
+++ b/src/vim9compile.c
@@ -1866,27 +1866,22 @@
 	{
 	    // for an object or class member get the type of the member
 	    class_T	*cl = lhs->lhs_type->tt_class;
-	    omacc_T	access;
+	    ocmember_T	*m;

 	    lhs->lhs_member_type = class_member_type(cl, after + 1,
 					lhs->lhs_end, &lhs->lhs_member_idx,
-					&access);
+					&m);
 	    if (lhs->lhs_member_idx < 0)
 		return FAIL;

 	    // If it is private member variable, then accessing it outside the
 	    // class is not allowed.
-	    if ((access != VIM_ACCESS_ALL) && !inside_class(cctx, cl))
+	    if ((m->ocm_access != VIM_ACCESS_ALL) && !inside_class(cctx, cl))
 	    {
-		char_u	*m_name;
-		char	*msg;
-
-		m_name = vim_strnsave(after + 1, lhs->lhs_end - after - 1);
-		msg = (access == VIM_ACCESS_PRIVATE)
+		char *msg = m->ocm_access == VIM_ACCESS_PRIVATE
 				? e_cannot_access_private_member_str
 				: e_cannot_change_readonly_variable_str;
-		semsg(_(msg), m_name);
-		vim_free(m_name);
+		semsg(_(msg), m->ocm_name);
 		return FAIL;
 	    }
 	}
@@ -2097,10 +2092,9 @@
 	   return FAIL;

 	class_T	*cl = lhs->lhs_type->tt_class;
-	omacc_T	access;
 	type_T	*type = class_member_type(cl, dot + 1,
 					   lhs->lhs_end, &lhs->lhs_member_idx,
-					   &access);
+					   NULL);
 	if (lhs->lhs_member_idx < 0)
 	    return FAIL;

Can you open a PR after this PR is merged with your changes?


Frontend: change background and make color scheme better and attractive.

I'm gssoc'21 participant would like to make all the colors in sync and make it colorful and impressive.
I want to work on this issue
Can you assign issue to me ?
i'll submit before deadline.

@lakshita15 The pull request #358 has been merged recently and that worked on this exact same issue, so, please update your fork, give the updated website a look and then if you have any further improvements, create a new issue focusing on those exact improvements.


⚠ Bofrost.de has degraded performance

In 2522811, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1065 ms

Resolved: Bofrost.de performance has improved in 83c6e85.


portico: Align plans on /for/education with /plans.

These changes add an extra bullet, which is not actually showing up on the page. @amanagr , if it's not too hard, could you please tweak the CSS to make it work?

Updated

Thanks, works for me in manual testing! @timabbott over to you.

Merged, thanks @alya and @amanagr! Note that I almost missed this because the "WIP" in the PR title was still there.


Add ability to specify an authentication header.

We are using a third party vendor for our prometheus setup and they require that we provide a Token: foo header for authentication. This PR adds the ability to do this. I didn't follow the PR template because there are, as far as I can tell, no integration tests for me to run.

Welcome to Steampipe @gcbirzan-plutoflume and thanks for raising a PR to specify authentication headers.
The initial structure looks good, however, we had a couple of questions/suggestions before we could merge the changes -

Could you please attach a doc or reference link that mentions about how the authentication headers work?
Is it possible to set the headers using environment variables?
Could you also update the config/prometheus.spc file to include information about the authentication headers?

Welcome to Steampipe @gcbirzan-plutoflume and thanks for raising a PR to specify authentication headers +1.
The initial structure looks good, however, we had a couple of questions/suggestions before we could merge the changes -

Could you please attach a doc or reference link that mentions about how the authentication headers work?

I will document some examples, but the problem is that, for our specific usecase, the header isn't Authentication: foo, so it has to be flexible enough to handle any name/value.

Is it possible to set the headers using environment variables?

I'm not sure how useful it'd be, except to keep the value out of config files. For things like AWS, there's a lot of benefits, since the profile/region will allow it to Just Work. Let me know if you want me to do this, I'll have to check how it's done in other plugins.

Could you also update the config/prometheus.spc file to include information about the authentication headers?

Sure.

I've looked at the AWS plugin, and the env var comes from the AWS client (duh), so I'm not sure where else to check for a plugin that does this. In the mean time, doc and sample config.


Some CKEditor features only usable if div element is edited

Many CKEditor functions are only available, if the inline code is surrounded by an HTML div tag (see https://ckeditor.com/docs/ckeditor4/latest/api/CKEDITOR_editor.html#property-blockless and https://ckeditor.com/docs/ckeditor4/latest/api/CKEDITOR_dtd.html).
This means that common CKEditor plugins (e.g. justify, list) can currently often not be used.
Example:
If I want to center a headline, then I do not get this option for an h-element. I tried wrapping it with a div, but this does not help.
What is the solution for e.g. centering an h-element?

Any update on this?


Enable the use of prettify-symbols-mode

Make M-x prettify-symbols-mode prettify some symbols (e.g. <=, >=, ->) in Rust code.

The code to disambiguate the two uses of || has to be improved but, before, I'd like to know whether there is interest in thos feature.

I'd like to know whether there is interest in this feature.

I'm sure many users want this feature. But I think these changes should be optional. I'm actually not sure if we should make this a default. We should definitely document this in the readme. Most use doom-emacs etc. anyway so there will be probably a pull request for those distros soon.
I wonder if there's already some mode that provides this feature major mode independent. I think we should ensure this feature doesn't interfere with other packages.

But I think these changes should be optional. I'm actually not sure if we should make this a default.

The commit prepares the infrastructure for prettifying but nothing gets activated until the user issues M-x prettify-symbols-mode (or puts it in the mode hook or uses global-prettify-symbols-mode).  I've changed a comment to reflect that.  Prettify is the major-mode independent way of prettifying symbols (but it must be tailored to the mode).

 Can you do me a favor and a note to the readme ?

Done.

Thanks!


Added the scenario for linux users

This is the initial setup for the katacoda scenario.
For review purposes, the scenario is implemented here

/lgtm


rpi: add support for pi 400 & pi 4B 8GB

This PR adds support for the raspberry pi 400 board (tested on mine), and support for the 8GB RAM version of the pi 4B
Also:

add unit tests for revision 1.2 of pi 4B boards (2 and 4GB)
add unit tests for revision 1.4 of pi 4B boards (2, 4 and 8GB)
add unit tests for pi 400 (revision 1.0)

Codecov Report

Merging #11 (17106ad) into main (b2a8d7d) will decrease coverage by 2.5%.
The diff coverage is 100.0%.

@@           Coverage Diff           @@
##            main     #11     +/-   ##
=======================================
- Coverage   33.9%   31.4%   -2.5%
=======================================
  Files         49      45      -4
  Lines       5376    5248    -128
=======================================
- Hits        1823    1646    -177
- Misses      3427    3476     +49
  Partials     126     126

Impacted Files
Coverage Δ

rpi/rpi.go
29.7% <100.0%> (+0.6%)
:arrow_up:

pmem/alloc.go
38.1% <0.0%> (-52.4%)
:arrow_down:

cpu/cpu.go
59.1% <0.0%> (-31.8%)
:arrow_down:

distro/devtree.go
51.7% <0.0%> (-20.7%)
:arrow_down:

sysfs/sysfs.go
40.0% <0.0%> (-20.0%)
:arrow_down:

pmem/view.go
68.8% <0.0%> (-19.6%)
:arrow_down:

pmem/pagemap.go
84.2% <0.0%> (-15.8%)
:arrow_down:

distro/distro.go
75.4% <0.0%> (-10.8%)
:arrow_down:

sysfs/i2c.go
69.1% <0.0%> (-5.9%)
:arrow_down:

sysfs/thermal_sensor.go
77.7% <0.0%> (-5.4%)
:arrow_down:

... and 7 more

Continue to review full report at Codecov.

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

You'll have to wait a bit for me to fix the CI.

Do you mind to git fetch, git rebase and git push -f again? The CI should be fixed now. Thanks!

Would you mind signing the CLA?

@googlebot I signed it!

gohci

Oops ignore the failures, it's not your fault. Thanks!


Tasks in admonition block unclickable

Admonition version 6.7.2
Hi, I have been using admonition for quite some time now and it really amazes me that most of the dynamic functions work fine with the plug-in unlike with other kinds of code blocks.
However when I tried to put some tasks in admonition blocks I found them to be unclickable.
- [ ]  ddd
- [ ] ssss

Is this an issue that can be fixed? It would be extremely helpful if I can add tasks in admonition blocks.

I second that. You can change the task in edit mode but can't click them in preview mode

Hi, this should be fixed in 6.7.3. Sorry this took so long!

Lovely! Thank you so much!


Remove annotation to force scheduling the operator on amd64 nodes

Now, Prow builds multiarch images in the presubmits too

/test unit

/test e2e-gcp-multi-operator

/test e2e-gcp-multi-operator

For this to work we need the prow test to run on build10 too.


Banked XP calculator ignoring bonemeal and ashes

The banked XP plugin checks your bank for items and calculates how much xp is banked in a chosen skill. Currently when choosing prayer, it seems to look for bones, remains, ensouled heads and fossils. Currently, ashes and bonemeal are not checked for.
The reason I opened this request is because I know myself and many others stack bonemeal in their bank, from the daily morytania diaries reward.

That plugin is a hub plugin, not maintained by the RuneLite team. Please open this issue on the banked XP plugin's repository.


Consider with statements for too many branches lint

Resolves https://github.com/astral-sh/ruff/issues/11313
Summary
PLR0912(too-many-branches) did not count branches inside with: blocks. With this fix, the branches inside with statements are also counted.
Test Plan
Added a new test case.

Thank you for working on this! Welcome to Ruff :)


Add basic Cloud Logging support.

Add Cloud Logging support to daisy.

Cloud Logging is the default.

Logs will go to GCS when specified by the flag or when CloudLogging
is unavailable.

Removes stdout logging unless specified by a flag.

This was undesirable when running via gcloud.

Cloud logs are structured JSON logs.
Stdout and GCS logs are still strings for compatibility. However, the timestamp now contains timezone.

Cloud Logs example

I also think we should use cloud logging as the default instead of logging to both (cloud logging and gcs).

Cool. So to recap, let me know if this sounds right:

The default is cloud logging.

If cloud logging is unavailable (no api access, etc.) we fallback to using GCS.
If the new flag -log-gcs true is set, we use GCS instead.
The Stackdriver "LogName" will be daisy-workflow-[workflow.id]

We will update all our logs to make them structured.

I'll look into what the actual structure of the logs can be to make it easy to track sub-workflows, steps, etc.
Serial logging and GCS logging will flatten the structured logs to maintain the current log schema. (Or is that not necessary? I'm not sure how much string matching might already be in place around these log entries for monitoring purposes.)

I'll leave the serial output logs alone for now. We can fold them in afterwards if we decide that's what we want to do.

Yeah that sounds good. Keeping the current logging would be best so as not
to confuse users, writing a stringer for the structured log format you come
up with shouldn't be too difficult. If anything this will make the logging
logic way more understandable, also keep in mind that console logs may need
to look a bit different, workflows themselves log and the Daisy binary logs
as well (cloud logger probably only needs to be in the workflow logging
logic)
On Wed, Apr 18, 2018, 10:57 AM ryanwe<EMAIL_ADDRESS>wrote:

Cool. So to recap, let me know if this sounds right:

The default is cloud logging.

If cloud logging is unavailable (no api access, etc.) we fallback
to using GCS.
If the new flag -log-gcs true is set, we use GCS instead.
The Stackdriver "LogName" will be daisy-workflow-[workflow.id]

We will update all our logs to make them structured.

I'll look into what the actual structure of the logs can be to
make it easy to track sub-workflows, steps, etc.
Serial logging and GCS logging will flatten the structured logs
to maintain the current log schema. (Or is that not necessary? I'm not sure
how much string matching might already be in place around these log entries
for monitoring purposes.)

I'll leave the serial output logs alone for now. We can fold them in
afterwards if we decide that's what we want to do.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/compute-image-tools/pull/380#issuecomment-382474813,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQWyS5So-AB48qNUeDxOs8zDW8GOsAgkks5tp36AgaJpZM4TaXYV
.

I ended up adding another flag for enabling/disabling console logs which should make Daisy play nice with gcloud. I updated the PR summary to reflect these decisions and the changes made.


feedback: relative String() for an RR

I'm trying to output an RR relative to an origin; to write out a zone file. This turns out to be non-trivial, mostly because an @ is a zonefile construct and doesn't exist in actual DNS data. I.e current attempt to rewrite the domain name itself returns:
\@	3600	IN	TXT	"blaat"

I wonder if there is something that can be done here, or that I just need to be more creative.
Sidenote: why not output the zone as-is? Well, the zone may be shared between multiple origins when reading it. Outputting it fully qualified means multiple copies and also bloats the file unnecessary.

I have thought for a while that it might be useful to add something like a RelativeString(origin string, relativeRdata bool) method to RR. An origin field might also be useful, either separately or in conjunction.

[ Quoting<EMAIL_ADDRESS>in "Re: [miekg/dns] feedback: relative ..." ]

I have thought for a while that it might be useful to add something like a RelativeString(origin string, relativeRdata bool) method to RR. An origin field might also be useful, either separately or in conjunction.

Thanks for your thoughts.
I think adding another field might be prohibitive from a memory standpoint.
I've seen other requests for something like this, so maybe a callback function
might work better, but then you know what rdata field your working on. So maybe
that overkill and the above is the only thing that's needed.

Adding #992 for some code to discuss

I can't use this for the use case I had in mind (signing a relative zone) as the signer's name is included in the signature data.
From dnssec.go:
// For signing, lowercase this name
sigwire.SignerName = strings.ToLower(rr.SignerName)

This might still be useful; but it's a big change, so I won't push for it.

I don’t like the idea (#992) of every RR type having to have two String implementations that then have to be kept in sync.
Doesn’t something like this work? (Or could we make something like this work).
r2 := dns.Copy(rr)
r2.Header().Name = "@"
r2.String()

That converts @ into @ on string conversion.
On Sun, 7 Jul 2019, 03:36 Tom Thorogood<EMAIL_ADDRESS>wrote:

I don’t like the idea (#992 https://github.com/miekg/dns/pull/992) of
every RR type having to have two String implementations that then have to
be kept in sync.
Doesn’t something like this work? (Or could we make something like this
work).
r2 := dns.Copy(rr)
r2.Header().Name = "@"
r2.String()
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/miekg/dns/issues/991?email_source=notifications&email_token=AACWIW2QN4ZZY3BMKK4VA6TP6FCC7A5CNFSM4H47HD52YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZLC77I#issuecomment-508964861,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACWIWZMT2XFXJHUIWPJYKDP6FCC7ANCNFSM4H47HD5Q
.

If there will always be a single @, this should work:
r2 := dns.Copy(rr)
r2.Header().Name = "@"
strings.Replace(r2.String(), "\\@", "@", 1)

While it’s not exactly elegant, I think having to add a second String method should be a deal breaker so the above might be the best option.

I can't use this for the use case I had in mind (signing a relative zone) as the signer's name is included in the signature data.

That's why I suggested a two-parameter RelativeString(origin string, relativeRdata bool), with the second parameter controlling whether or not RDATA should also be presented relative to the specified origin. But that gets into @tmthrgd's concern:

I don’t like the idea (#992) of every RR type having to have two String implementations that then have to be kept in sync.

Which I agree in principle, but in practice changes to serialization should be so rare that it may not matter.

r2 := dns.Copy(rr)
r2.Header().Name = "@"
strings.Replace(r2.String(), "\\@", "@", 1)

That works for the specific case of owner name matching zone origin (as would e.g. r2.String()[1:]), but such manipulation seems to be a sign that something has gone wrong.
That said, I like the direction of the changes but don't know if I'm convinced that they are warranted.

closing this, thanks for the discussion.


Posibility to change Click for Alt+Click

What would you like to be added?
Posibility to change Click for Alt+Click.
Why is this needed?
fast way to enter to note.

Can you elaborate more on your request? Like where you want to apply the alt + click

If I Click on Project Title, this open the edit menu

I want to Click and goes to file


Universal Semantic Version Handling

We have previously used our own implementation of Semantic Versions but my new dependency resolution code uses an OTA implementation.  We need to unify this so we are using the same implementation across all of impact.  Furthermore, I think we should actually define a short hand type for semantic versions and the various features we need from them so we could actually switch implementations behind the scenes or at least abstract away any awkward aspects of the implementations API.
I'm currently using this implementation of Semantic Versions in the Go implementation.  So far, the main problem I have had is that it favors the use of pointers to versions rather than version values.  There are a few cases where this is awkward.  But it isn't a fatal flaw.
There are many issues in here that are related.  I will try to at least comment in each on and reference this issue since I think many could be aggregated into a single set of fixes.

OK, we use semantic versions everywhere now.  And the semver package I was using go rid of their pointer based approach as well.  So things actually look pretty good.  About the only remaining issue I see is how we serialize and de-serialize the version information.  But that isn't this issue, so I'm closing.


[CERT-TEST-FAILURE] [TC-S-1.1] Test failure in case if optional LastConfiguredBy attribute isn't supported by DUT.

Feature Area
Area:Scenes
Test Case
TC-S-1.1
Reproduction steps
Run TC-S-1.1 from TestHarness version 2.10-beta2.2+spring2024 on DUT without support LastConfiguredBy attribute
Bug prevalence
Whenever I do this.
GitHub hash of the SDK that was being used
0ff709f25f6a7af68314c1b0a147a76c30c234f6
Platform
raspi
Anything else?
Here is TestHarness log. FYI: PICS filled correctly.
TC-S-1.1_huebridge_TH_log.log

Fixed by https://github.com/project-chip/connectedhomeip/pull/32003


Fix wrong confirm button label

Fixed the wrong confirm button label.
"Yes" button was "No" xD

Yeah it does

I just went through the whole intro, without that happening.

Could you provide a screenshot of that happening? I couldn't find it so far.

@jcgruenhage
You are before my changes?

@jcgruenhage
Before:

After:

Straight from F-Droid:

@jcgruenhage
Strange

Yep. Your commit totally makes sense, and it should be broken, if that is actually the code that went into the apk I have on my phone. /me will dig around a bit.

@jcgruenhage
Okay

Okay, now this makes sense. The extraction of those strings was done after the release of v0.9.6, so currently (on F-Droid/master, not develop), those are hardcoded strings.

Thanks for your contributions :)

Oh okay, thanks!


namespace object generates the wrong Web IDL spec in references

See https://console.spec.whatwg.org/#references. Web IDL 1 doesn't even define namespace objects. And that's the URL for Web IDL, not Web IDL 1.

Hm, this is how the biblio data sees things:
-
a:webidl
WebIDL
webidl-1
-
d:webidl-1
WebIDL-1
15 September 2016
PR
WebIDL Level 1
https://www.w3.org/TR/WebIDL-1/
https://heycam.github.io/webidl/

Cameron McCormack
Boris Zbarsky

It aliases webidl to webidl-1, and then you end up getting the "current" links, which is to the correct spec.
The w3.org url is the most correct, recent "snapshot" url, too, so I'm not sure what's wrong with this?

What's wrong is that in https://console.spec.whatwg.org/#references it says "Web IDL Level 1" and "PR" instead of "Web IDL" and "ED".

The most recent snapshot level for the spec is indeed PR.  And calling the spec as a whole "level 1" is fine imo - what's wrong with it?
Note that WebIDL is still officially a W3C spec.

Right, but we're not linking to Web IDL level 1, the spec located at https://www.w3.org/TR/2016/PR-WebIDL-1-20160915/ whose level is PR. We're linking to Web IDL, the spec located at https://heycam.github.io/webidl/ whose level is ED.

Yes, I'm aware of all the issues around WebIDL snapshotting. But the facts are still that WebIDL is a W3C spec, it has snapshot versions, the latest such snapshot is a PR, and it has links to both the snapshot and the current version. And, since WHATWG has a default policy of linking to the current versions, you see only the current versions in your biblio anyway.
The only problem seems to be that you would like the reference to be unlevelled (it's a W3C spec, so the name is determined by the W3C, and they say it's called "Web IDL Level 1" in their biblio data), and that you dislike the fact that "PR" is mentioned in the biblio (despite it being a true fact that its most recent snapshot is a PR).  Since they're both correct as written, and working around it would be some sort of terrible hack, I'm not super excited about trying to "fix" anything there.

From my perspective, a spec is a series of { name, status, url } tuples. We'd like to reference a specific { name = "Web IDL", status = "ED", url = "https://heycam.github.io/webidl/" } tuple.
However, what we've ended up with is a hybrid. It's talking about a spec with name = "Web IDL Level 1", status = "PR", but with url = "https://heycam.github.io/webidl/". There is no such spec! The biblio entry is internally inconsistent with itself! The URL doesn't match the name and status fields! In other words, "you see only the current versions in your biblio anyway" is not true---we're seeing a hybrid of the old snapshot version, and the current version!
Reading between the lines of your replies, it seems like Bikeshed has some kind of policy that whenever you reference a spec, it wants to put information on whatever the latest status of that spec is in the biblio entry. But I argue this doesn't make sense, and is actively misleading for readers, when the URL it gives is in fact for a different-status spec.

Hmm, you're right that, if I'm just using the "current" URL, talking about the snapshot's status doesn't matter (and is kinda confusing). I'll fix that.

Done. If you prefer current URLs (and got one), it omits the date and status of the snapshot.

Thank you!


Event Code NAN

Fixes https://github.com/JeffersonLab/jmyapi/issues/9

Added UNDEFINED to fromInt() method as well.


Please check for the following default values in nova/neutron

We want to consider the following neutron and nova settings around database and RPC pool management as default in RPC. I personally would have seen that in OSAD but I'm pretty sure a bug would have been down voted :-1 there. Lastly we need to either make sure that these settings are set or are actively being set during the installation
keystone_db_max_pool_size: 120
keystone_db_pool_timeout: 30
neutron_db_pool_size: 120
neutron_db_pool_timeout: 30
nova_db_max_pool_size: 120
nova_db_pool_timeout: 30
rpc_conn_pool_size: 180
rpc_response_timeout: 180
rpc_thread_pool_size: 180
nova_rpc_conn_pool_size: '{{rpc_conn_pool_size}}'
nova_rpc_response_timeout: '{{rpc_response_timeout}}'
nova_rpc_thread_pool_size: '{{rpc_thread_pool_size}}'
BTW, the nova_rpc settings have not been implemented yet in OSAD (https://bugs.launchpad.net/openstack-ansible/+bug/1471926)

Can you give an idea of a priority? This looks fairly straightforward.

should be implemented for kilo, prio 3 is ok

Possibly something that duplicates this, or this is the more up to date request for RPC:  https://bugs.launchpad.net/openstack-ansible/+bug/1471926

I opened this issue here to run our clouds with this special setup and the osad with a more generic parameter configuration. In the end I have no real preference where we set these parameters as long as they are pushed as default configuration for our RPC clouds

Sounds like we have 2 issues (not using acronyms because I think we're overloading them here):

Get the oslo.messaging remote procedure call settings into OSAD as variables
Define values for Rackspace Private Cloud usage.

@nrb that sounds correct.

This should be on 10.1.11 and 11.0.1 milestones

db settings don't need to be added upstream
db == db in juno
db == database in kilo/master

Also seems, the defaults in kilo can set set in rpc-openstack which will automatically update (I think)
The defaults for juno may land squarely on changing the local config in the installation.

thought we had user_* stuff we could override with, or is that not in juno?

oh ya, upstream is juno but we are tracking it here :(

keystone_db_max_pool_size: 120
keystone_db_pool_timeout: 30
neutron_db_pool_size: 120
neutron_db_pool_timeout: 30
nova_db_max_pool_size: 120
nova_db_pool_timeout: 30
nova - max_pool_size && pool_timeout in database section
  nova_db_max_pool_size && nova_db_pool_timeout
neutron - max_pool_size && pool_timeout in database section
  neutron_db_pool_size && neutron_db_pool_timeout
keystone - max_pool_size && pool_timeout in database section
  keystone_db_max_pool_size && keystone_db_pool_timeout

juno will need to go upstream

the options need readding in kilo

https://review.openstack.org/#/c/205678/
https://review.openstack.org/#/c/205675/
https://review.openstack.org/#/c/205671/
go do the reviews, kthnx
all needed after that is overriding the values for the RPC stuff in the userconfig


Revert tweak

reverting this https://github.com/pointfreeco/swift-prelude/pull/26/commits/fb7ad5d9bb5f589b49e8653ceeffd52151be71fe

imma close for now cause I just used parens to get around this. we can revisit later


Replaced "burnbright/silverstripe-shop" dependency with "silvershop/core"

This is required to keep from having two instances of the SilverShop add-on.

This is great but the dependency should be ~1.0 || ~2.0 as 2.0 is coming pretty soon.

Is the Ajax add-on compatible with 2.0 though? Would need to make sure both the main Ajax and Shop-Ajax add-on's are compatible. At least with just a 1.0 compatibility check, you're sure that there won't be issues.

2.0 really only potentially breaks internationalization (just changes some translation keys) and some custom omnipay integrations (i.e. if you're replacing OrderProcessor with your own implementation and/or doing other custom stuff). Majority of shop installations won't even notice.

That's cool. I'll go ahead and update it and redo the pull request.


[Meteor 1.3-beta.12] (Cross-Post-Issue) Error building with raix:push - Package configured via .json config file

Hello, i've posted the following error to the Meteor Issue Tracker already, see link below.
The thing is, I think it's possible to configure raix:push without the .json - File, which would be a workaround, but I don't get exactly how to configure the client side. The docs lead to the recommendation to use the configuration of the cordova push package: https://github.com/raix/push/blob/v3.x/docs/ADVANCED.md#note-outdated , but the supplied link https://github.com/phonegap/phonegap-plugin-push#pushnotificationinitoptions doesn't seem to lead to an explanation of the config anymore.
Maybe somebody can give a hands-on example of how to deal with it?
Or maybe it's possible to move this to Meteor.settings or upgrade the Push.Configure - call to set the required parameters for phonegap-plugin-push?
This is the error I posted:
Link: https://github.com/meteor/meteor/issues/6411
Text:

I receive this of error since updating from 1.3 Beta 11 to 1.3 Beta 12:
Error: sourcePath not a file: push.config.web.browser.js
    at ImportScanner._ensureSourcePath (/tools/isobuild/import-scanner.js:208:13)
    at /tools/isobuild/import-scanner.js:101:28
    at Array.forEach (native)
    at ImportScanner.addInputFiles (/tools/isobuild/import-scanner.js:100:11)
    at /tools/isobuild/compiler-plugin.js:709:15
    at Array.forEach (native)
    at Function.computeJsOutputFilesMap (/tools/isobuild/compiler-plugin.js:689:19)
    at ClientTarget._emitResources (/tools/isobuild/bundler.js:749:8)
    at /tools/isobuild/bundler.js:516:12
    at /tools/utils/buildmessage.js:359:18
    at [object Object].withValue (/tools/utils/fiber-helpers.js:89:14)
    at /tools/utils/buildmessage.js:352:34
    at [object Object].withValue (/tools/utils/fiber-helpers.js:89:14)
    at /tools/utils/buildmessage.js:350:23
    at [object Object].withValue (/tools/utils/fiber-helpers.js:89:14)
    at Object.enterJob (/tools/utils/buildmessage.js:324:26)
    at ClientTarget.make (/tools/isobuild/bundler.js:507:18)
    at /tools/isobuild/bundler.js:2232:14
    at /tools/isobuild/bundler.js:2322:20
    at Array.forEach (native)
    at Function._.each._.forEach (/Users/spaceman/.meteor/packages/meteor-tool/.1.1.13-beta.12.1p7kgab++os.osx.x86_64+web.browser+web.cordova/mt-os.osx.x86_64/dev_bundle/lib/node_modules/underscore/underscore.js:79:11)
    at /tools/isobuild/bundler.js:2321:7
    at /tools/utils/buildmessage.js:271:13
    at [object Object].withValue (/tools/utils/fiber-helpers.js:89:14)
    at /tools/utils/buildmessage.js:264:29
    at [object Object].withValue (/tools/utils/fiber-helpers.js:89:14)
    at /tools/utils/buildmessage.js:262:18
    at [object Object].withValue (/tools/utils/fiber-helpers.js:89:14)
    at /tools/utils/buildmessage.js:253:23
    at [object Object].withValue (/tools/utils/fiber-helpers.js:89:14)
    at Object.capture (/tools/utils/buildmessage.js:252:19)
    at Object.exports.bundle (/tools/isobuild/bundler.js:2213:31)
    at /tools/runners/run-app.js:586:36
    at Function.run (/tools/tool-env/profile.js:489:12)
    at bundleApp (/tools/runners/run-app.js:576:34)
    at AppRunner._runOnce (/tools/runners/run-app.js:629:35)
    at AppRunner._fiber (/tools/runners/run-app.js:881:28)
    at /tools/runners/run-app.js:406:12

After a lot of humming and hawing, it seems to boil down to this:
raix:push is structured to be configured using a file config.push.json in the meteor projects' root directory.
Then there seems to be some kind of build plugin which is supposed to read the file and actually add some additional javascript. The whole thing is registered here: https://github.com/raix/push/blob/f3a5a653dc2d1d7cd1af0b4d6478d23bcbd0644f/plugin/push.configuration.js#L185
...AAAND that's how far I can isolate the error. Something (a lot I feel?) changed in the build process from 1.3 Beta 11 to 1.3 Beta 12 and this is one issue which probably needs to be patched somewhere.
Reproduction Repo: https://github.com/DanielDornhardt/meteor13Beta12errorBuildingWithRaixPushPackage

just remove or rename the file config.push.json to see the build error disappear.

We have the same issue. Would be awesome if Meteor could resolve this ASAP.

I havent gone into depth but could this be just an issue with the json configuration?

Hello @raix , I don't think so. Please take a look at the config.push.json-file in the reproduction repo - it's the same file we're using in our app, except the keys and filenames have been changed.
The values - and the file - was always working for us with meteor versions < 1.3 .

We might want to replace the json config way of doing it in favor of using the import/export pattern eg. allowing:
push.config.client.js
export default {
  "gcm": {
    "projectNumber":<PHONE_NUMBER>1111
  },
  "production": false,
  "badge": true,
  "sound": true,
  "alert": true,
  "vibrate": true,
  "iconColor": "#FFFFFF"
};

push.config.server.js
import clientConfig from './push.config.client.js';
export default Object.assign({ // xxx: this extend might not be 100% correct...
  "apn-dev": {
    "passphrase": "test",
    "key": "example.pem",
    "cert": "example2.pem"
  },
  "apn": {
    "passphrase": "test",
    "key": "example.pem",
    "cert": "example2.pem"
  },
  "gcm": {
    "apiKey": "00000000000000000000000000000",
  }
}, clientConfig);

app.client.js
  import clientConfig from './push.config.client.js';
  Push.Configuration(clientConfig);

app.server.js
  import serverConfig from './push.config.server.js';
  Push.Configuration(serverConfig);

Hello, thank you for the example code, that gives us something to work with.
I actually didn't get why you used the .json - file in the first case instead of the Meteor.settings - is it because you wanted to prevent the keys of every platform to "leak" to every client?
Are you trying to package the key and cert files for each platform only for the builds of that specific platform?

the nice thing about the json config is that all configuration can be done in one place - the package will compile it into a client and a server configuration. (should not leak server keys etc.)
Having said that using Push.Configuration directly might be the way to go in larger setups making it possible to have the push send server as a separate microservice

Hi,
not leaking Server Keys could also be done by using the /private/ meteor special directory and / or the non-public keys of the settings mechanism, right?
Or maybe even let users provide a path to a file somewhere, and one possible delivery mechanism would be the /private/ directory, which gets included in the build for the server?
I don't know how you read the files, you probably depend on some other package to read the files, but I think that might be good meteor-compatible options?

private is for the cert/key but true have the files in separate client/server folders would do the trick too

Btw, this seems to have been fixed / worked around in Meteor and might be good again in the next Meteor 1.3 Beta: https://github.com/meteor/meteor/issues/6411

Thanks @DanielDornhardt good to know

Hey, I'd also be excited about having these settings in settings.json.  I already have a system setup to manage dev/stage/prod settings via settings.json, and this extra config is extra work.

Agreed that having keys in settings.json would be ideal.


Allow external code to change value of MaskedInput (InputMask)

Sometimes we need to change the value of maskedinput from external component.
For example we had the following scenario: user enters something in maskedinput and presses submit button. After that modal opens where user confirms his action. When user confirmed action modal hides and the entered value in maskedinput must be cleared.

Merge conflict. I’ll open a fresh PR with this change and merge this in today. Thank you.


Add ability to use Google fonts

Please add the ability to use fonts directly from fonts.google.com, without having to install them first.
Why?
To achieve a consistent look, I want my diagrams to match the font of my documents. As I'm using Google Docs, I naturally want to use fonts from Google, like Roboto or Open Sans. These are unfortunately not available by default. You can install them locally and use them via the "Custom font" option, but that is rather cumbersome and has the disadvantage that the fonts will differ when sharing the drawing.

You can use a plugin to load all the google fonts, so that the google fonts are always installed on load of draw io.
I have added to the custom fonts "Roboto", "Open Sans" & "Libre Franklin". But you can add/change this setting if you want. (you need to fork oid the repo)
Because now all the fonts are installed, you can add custom fonts in Draw.io by entering the name of the font family.
Draw.io Plugin URL:
https://yacdn.org/serve/https://raw.githubusercontent.com/CoolZeroNL/draw-io-plugins/master/%23 Draw-io-plugin-Google-Fonts/google-fonts-v1.js?maxAge=10

You can now use the built-in Font picker: