Update Get-RetentionCompliancePolicy.md

https://github.com/MicrosoftDocs/office-docs-powershell/issues/4583

Docs Build status updates of commit 660ad71:
:clock10: Full build: the average full build time is 11 min(s) 52 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 660ad71:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/policy-and-compliance-retention/Get-RetentionCompliancePolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

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

Docs Build status updates of commit cea292e:
:clock10: Full build: the average full build time is 11 min(s) 52 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit cea292e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/policy-and-compliance-retention/Get-RetentionCompliancePolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

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

Docs Build status updates of commit c9505ac:
:clock10: Full build: the average full build time is 11 min(s) 52 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit c9505ac:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/policy-and-compliance-retention/Get-RetentionCompliancePolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

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


Resample-move SMC

Preliminary prototype of resample-move SMC according to Adam Scibior's idea of using a list of continuations as the state of Metropolis transition.
Changes

Replaced module Trace to use a list of continuations as the state of the Markov chain, implemented via a coroutine with suspension points at primitive distributions. Alternative states (i. e., types in the class RandomDB) are deleted.

Added Inference.smcWithResampler, the common template between smc and smcFast, implemented according to the paper draft.

Added Inference.smcrm according to the paper draft. The only subtle point is that mhStep is specialized to Population m to discard new weight after transition. Only then is the posterior preserved.

I just added some notes on Trace to the draft in section 2.7. Could you check how this compares to your new implementation of the Trace transformer? In particular, can we have an analogue of mhstep in implementation? I'm not sure if it's possible to do things this way, but it would be very convenient if that was the case.

Thanks for the pointer, I could work on #13 after the resample-move prototype is done.
About §2.7: I will try this approach. Right now I can't make Tr a monad and the work-around is less appealing. It involves posterior-preserving natural transformations into functors that are not monads.

Is there a simple explanation for why Tr could not be a monad?

Tr saves a list of continuations from each primitive together with an answer. It is not clear how to combine the continuation lists sensibly in monadic bind.

Can't we extend the continuations on the left with the RHS of >>=, leave the continuations on the right alone, and concatenate the two lists?

Each continuation takes the value of one primitive variable and re-executes the program from that point onward, reusing previous samples whenever permissible. If continuations on LHS are to be combined with f, they must also retain the exact primitive samples during the run that generated the RHS.
Maybe I'm not defining T and Tr correctly. I will think about it more.

smcrm is now almost exactly as it appears in the paper draft.

Build 11.6 failed because Travis couldn't locate ghc, apparently due to $GHCVER not being set...

This is looking good - I'll take a more careful look later.
The problem with Travis is the script that caches Stack files across multiple builds. It usually works fine, but every now and then it can't find GHC. I don't think it is mmorph that's causing problems, but let's not add it to dependencies just yet.

Could you explain why mhStep' needs to be specialized to Particle m? In particular, why is there no mhStep' :: (MonadDist m) => Trace' m a -> Trace' m a? My intuition is that this would be sufficient for SMC-RM.
Also, why is Trace not an instance of MonadTrans?

why mhStep' needs to be specialized to Particle m?

The main reason is that we want to discard the weight produced by MH transition and restore the old weight. Otherwise MH transition does not preserve posterior. Here is a proof sketch.
Let P be the Metropolis kernel. Let π be the posterior distribution. P preserves π in the sense that
π(dy) = P(x,dy) π(dx) .

Let ρ be the prior and let f be the scoring function; then
π(dx) = f(x) ρ(dx) .

Substituting the 2nd equation into the 1st yields
π(dy) = f(x) P(x, dy) ρ(dx) .

Note that after the MH transition, we are sampling directly from P(x, dy) ρ(dx). To retain the posterior π(dy), the correct score is the old score f(x), not the new one f(y). Simply calling mhStep inside Trace' would use the new weight f(y) and would not be correct.
There could be a way to implement mhStep' without mentioning Population, probably along the line of
mhStep' :: (MonadBayes m) => Trace' m a -> Trace' m a  .

We need MonadBayes to be able to adjust the score after MH transition. So far I haven't managed to restore the old weight correctly. I will give it more attempts.

why is Trace not an instance of MonadTrans?

It easily could be. I will add the MonadTrans instance declaration.

I don't see how mhStep modifies the weight. There are two things we might call "weight".
One is the weight associated with a particle, which is kept in the Population monad. It is only modified when factor is called in Trace', which only happens during the first execution. It is not modified when factor is called in Trace, which happens in mhStep and mhStep'. So this weight is not modified by the MH transition and it correctly remains the old weight.
The other weight is that associated with a trace that's called mhPosteriorWeight and is kept in the Trace monad. This weight is modified by mhStep, but that is the correct behaviour. mhPosteriorWeight in only used for computing acceptance ratio, so it should be the new weight. marginal doesn't reinsert mhPosteriorWeight into the transformed monad, which is again correct, so there is no need for it to be in any way related to the old weight.
In more practical terms, I believe your current mhStep' would always produce newWeight = 1, so there is no harm in returning the product oldWeight*newWeight instead of oldWeight. And this is exactly what would happen if you simply lifted mhStep to Trace'.

I will add the MonadTrans instance declaration.

Please also write MonadBayes Trace' instance as factor k = Trace' $ do {lift (factor k); factor k} if this is indeed correct.

Your MonadBayes Trace' instance seems correct; it preserves the posterior for the sprinkler model.
Inside Trace', mhStep modifies the weight of the particle due to resuming a continuation. Trace' has to pass each call of factor to the underlying monad Population m so that particles may have different weights. But then the continuations inside each MHState carries those calls to factor inside Population m. After mhStep, the weight of the particle is modified by factors encountered after the first resampled primitive distribution.

I see. That is indeed a problem and I don't have a solution for it. But it would be great to have one.

I've managed to make mhStep' generic (610fa6f317f1c49cd7d66b7b6b83e034eeff3605). The trick is to add a WeightRecorderT layer to save all the factors passed to the underlying monad, and later pass reciprocals of factors we want to revoke.
newtype Trace' m a = Trace' { runTrace' :: Trace (WeightRecorderT m) a }

Nicely done!


Have @usage decorator apply to namespaces

Clear and concise description of the problem
We have @usage for individual models and properties: we want to be able to apply this globally through a decorator that takes in a namespace / interface. More granularity down the line wins out
| Decorator State                   | Model with Public Usage | Model with Private Usage | Model without Usage |
|-----------------------------------|-------------------------|--------------------------|---------------------|
| `@usage(Usage.None)`              | Used                    | Used                     | Not Used            |
| `@usage(Usage.Input/Usage.Output)`| Used                    | Used                     | Used                |

Checklist

[X] Follow our Code of Conduct
[X] Check that this issue is about the Azure libraries for typespec. For feature request in the typespec language or core libraries file it in the TypeSpec repo
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

the table is unclear. inner @usage will overwrite outer @usage, right?


GVRF reference counting

GVRF uses a (native code) std::shared_ptr based reference counting scheme. It works well enough in 'normal' cases, but does break down in the not-so-special case where you have an object (we see this commonly with scene objects and textures, but it could happen with other objects, too) that goes in and out of the scene graph: When GVRF sees that an object has no native references (just the reference from the Java wrapper) it will 'forget' about the native wrapper and/or recycle() the GL resource. This can lead to consequences like eye picking returning a GVRSceneObject instead of the subclassed instance (holding app values) that you put into the scene graphs, or even to GVRF crashing when it dereferences null in native code.
We have what amounts to work-arounds for this: we can manually add references (as per the JavaDoc for GVRShaders.setMainTexture()) and we can set the GVRHybridObject.setKeepWrapper() flag. Perhaps it would be better to add explicit
long GVRHYbridObject.touch(); // addReference()?
void GVRHYbridObject.release(long); // removeReference()?

methods?

I think we need to go back and investigate how other scene graphs handle this issue.  Grant it, we might have a bit of a special case by having this division between native code and java code.  But I don't remember encountering these kinds of problem with other scene graphs.

How about something like register/unregister mechanism or add flag in constructor to let developer to say if it should be reserved without recycling?

There are a lot of constructors; I don't think we want to change each and every one to handle something that only applies to maybe 1% to 5% of all GVRF objects.
How does a "register/unregister mechanism" differ from touch/release?

Can’t it be more natural to keep object intact by default unless the user explicitly call recycle(), I mean the opposite approach? From the app developers’ perspective, I think they’d better don’t know about the reference counts in the framework. As Tom suggested, one way would be learn best practices for it from investigating other SGs.

Leaving resource lifecycle management to the app is one heck of a burden, Jason. It would slow app development tremendously, and most app developers would get it wrong: They'd leak some resources by not freeing them, and have dangling reference issues from freeing other resources prematurely.

Huh - I thought I replied to Tom's reply. I think it may indeed be the case that we are in a very sparsely populated part of the 'state space'.
It's 100% common for managed code (Java or C# or whatever) to have classes that wrap native code. But - generally - there's a 1:1 relationship between native instances and managed wrappers, and most of the action happens in the 'wrapped side'. An Android Bitmap, for example, does not have native references to components that also have Java wrappers, the way that a GVRF GVRSceneObject does. In this sort of environment, one can rely on managed side garbage collection and finalizers (and explicit close() methods) in a way that we simply can't, in GVRF.
An all-C or all-C++ game engine isn't going to have the issues that we have.
The Unity docs talk about a Mono runtime, and supporting ".NET languages" that compile to a DLL. That - to me - suggests that what's going on in Unity is a pretty standard .NET scenario: the managed (C#  or 'equivalent') objects are the masters, and the native objects are implementation details. (In GVRF, by contrast, the native objects are the masters, and the Java objects are mere tails,  attached in a way that was designed by someone who clearly was not a Java programmers.)
That is, if I had to place a bet, I'd bet that we'll be better served by looking at the design we inherited and trying to solve its problems in the cleanest way possible, than by spending time digging into the details of other game engines that will (I suspect) turn out to have little in common with the GVRF scenario.

A different - though much more programmer-intensive - way to solve this problem is to totally ditch the native reference counting, and let Java drive the recycling. We would only free a native object when its Java object is finalize()d (or, perhaps, explicitly close()d). This would eliminate all need for the 'object identity' code, manually manipulating reference counts, and setKeepWrapper().
We would keep all the lists of references (in GVRScene and GVRSceneObject &c) in Java, though we would probably let the native implementations keep native lists as well, as an optimization.
Things like traversing the scene graph would be a lot cheaper in Java, but this isn't currently a major expense: Overall, we'd be spending a lot of effort (40 hours, maybe more) to provide a clean solution to a problem that may not justify the effort. Otoh, we do seem to keep running into it, and we may still have race conditions: For example, what if a low-priority background thread creates a scene object, but the wrapper is released before the creating code can link it into the scene graph or call setKeepWrapper()?

Actually I was thinking of finalized() also. But I guess somehow we would lose direct control for the gabbage collection because those JNI object will be released only when GC cycle is triggered. And also, the whole mechanism for our current reference counting recycling may need to be restructured.

Chao wrote:

I guess somehow we would lose direct control for the gabbage collection because those JNI object will be released only when GC cycle is triggered.

In some ways, this is good: we currently poll for dead objects, 60 times a second. Leaving most of that work to the much less frequent garbage collection will leave more cycles for rendering.
Plus, adding a manual close() method lets app code release transient resources well before gc would get to them. (This, too, is Industry Standard.)

the whole mechanism for our current reference counting recycling may need to be restructured.

No, not "restructured": it would be gutted.

https://github.com/Samsung/GearVRf/pull/41


mkl CI config

Adds MKL job to macOS

@gf712 cherry pick the commits of that branch into this one and we'll see right away ;)

@gf712 cherry pick the commits of that branch into this one and we'll see right away ;)

There is an issue with unit-FLDATest, the values seem to be symmetric though...

@vigsterkr seems to be working now. Just need to sort the ccache now right?


tabindex on dom-repeat

Expected behavior
I can tab through the list of hastags/deals
Actual behavior.
I can't tab

done


Use sort.Slice from Go 1.8

We generally only sort slices, so all of this interface-satisfying boilerplate:
type bySum []result

func (b bySum) Len() int           { return len(b) }
func (b bySum) Swap(i, j int)      { b[i], b[j] = b[j], b[i] }
func (b bySum) Less(i, j int) bool { return b[i].sum > b[j].sum }

func foo() {
    sort.Sort(bySum(b))
}

can be replaced with an inline less func:
func foo() {
    sort.Slice(b, func(i, j int) bool { return b[i].sum > b[j].sum })
}

Candidates

[ ] hugolib.mapKeyValues
[ ] hugolib.menuSorter
[ ] hugolib.orderedTaxonomySorter
[ ] hugolib.pageSorter
[ ] hugolib.WeightedPages
[ ] langs.Languages
[ ] metrics.bySum
[ ] media.Types
[ ] output.Formats
[ ] related.ranks
[ ] releaser.contribCounts
[ ] tpl.collections.pairList

Note that sort.Slice will be slightly slower, not that is too important I think -- but most of the above will have to wait for my current working branch to go in -- or it will become a fairly painful merge.

Also, I think we use the stable sort today, which I think we need to keep, so `sort.SliceStable´ is the right choice.

I agree on both points.  Also, a few of the candidates I listed don't really lend themselves to this simplification.  I mostly wanted to leave notes here for later.


[stable/ark] optional backup schedule

FEATURE REQUEST:
I would like to automatically create an ark backup schedule when installing the chart. The schedule would not be created by default, but there would a Helm property to enable the creation of the schedule and properties to define the various schedule parameters.

See https://github.com/helm/charts/issues/6438#issuecomment-412560311 - will be much cleaner to implement once helm 2.10 is released.


[bug 1130762] Alerts app, models and modelfactories

This is the gist of the Alerts app.
r?

This doesn't have any tests. Mostly it's a db schema.
When reviewing, just make sure it has the appropriate things for Django models. Otherwise this is sort of a no-op.

New app LGTM. I ran the migration and it worked.
Bring on the Alerts! r+

0fa0aab [bug 1130762] Alerts app, models and modelfactories

Thank you!


[Enhancement] Add "demo" folder, can be used for testing/example usage

Is your enhancement request related to a problem? Please describe.
Currently we don't have a 100% way of checking/verifying that the library works "live".
Describe the solution you'd like
We should add a demo folder+app that uses TS and runs as its own app, loading a given version of the app and doing some basics. This can have its own package.json to help test/debug versioning live.
Describe alternatives you've considered
This might be overkill, but having separate versions of demos would help prevent regression, but it would be a lot more work.
Additional context
This could also be its own repo, but I like the idea of keeping it in this repo for simplicity sake.

Not going to do this.


Restore SharedMap to public API surface

Reverts microsoft/FluidFramework#20584 + removes the deprecation.

(Committed PR #20651 instead.)


Document <complex.h> escape hatch macro _CRT_USE_C_COMPLEX_H

As discussed in https://github.com/microsoft/STL/issues/3280 and https://github.com/microsoft/STL/pull/3347, the UCRT's <complex.h> contains an escape hatch macro _CRT_USE_C_COMPLEX_H that bypasses its inclusion of C++'s <ccomplex> and expands the contents of the C header normally (note: this area is very confusing):
#if (_CRT_HAS_CXX17 == 1) && !defined(_CRT_USE_C_COMPLEX_H)
#include <ccomplex>
#else // ^^^^ /std:c++17 ^^^^ // vvvv _CRT_USE_C_COMPLEX_H vvvv

In certain situations, where code is using <complex.h>'s non-Standard types _Dcomplex etc. which are documented on this page (and which are not provided by the STL's <ccomplex>), defining the escape hatch macro is the only way to get code to compile. However, the escape hatch macro is not currently documented!
FYI @amyw-msft.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 837f9388-5530-798a-5b6d-aaa1a5c0e285
Version Independent ID: fc31aaec-0539-bfc5-7d24-de55eb1d4a4c
Content: C complex math support
Content Source: docs/c-runtime-library/complex-math-support.md
Product: visual-cpp
Technology: cpp-ucrt
GitHub Login: @TylerMSFT
Microsoft Alias: twhitney

Moved this issue to an internal work item tracking system because we are moving away from using GitHub issues to track C++ doc work.


Helm install fails at kong.renderEnv

I was trying to get kong installed via our CD pipeline (fluxv2) and would always get the following error:
 Helm install failed: template: kong/templates/deployment.yaml:69:10: executing "kong/templates/deployment.yaml"
at <include "kong.controller-container" .>: error calling include: template: kong/templates/_helpers.tpl:383:4:
executing "kong.controller-container" at <include "kong.ingressController.env" .>:
error calling include: template: kong/templates/_helpers.tpl:599:15:
executing "kong.renderEnv" at <.>: wrong type for value; expected map[string]interface {}; got string   5s

After digging a bit more into helper.tpl I've managed to narrow the issue down to the following line:
{{- $_ := set $autoEnv "CONTROLLER_KONG_ADMIN_TLS_SKIP_VERIFY" "true" -}}

The above sets the value to "true" (string), which is then merged with default value (bool) from values.yaml
    kong_admin_tls_skip_verify: true

This makes mergeOverwrite fail silently and produce an empty string which, when passed down to renderEnv results in the original error message.
I still haven't figured out why the behaviour is different between helm CLI client and the library, it seems like they both call the same functions, however in helm CLI the above set command results in the actual boolean value being assigned to that key. Not sure why.
I thought it'd be good to have a fix here by simply changing the above set command to:
{{- $_ := set $autoEnv "CONTROLLER_KONG_ADMIN_TLS_SKIP_VERIFY" true -}}

This way the merge operation succeeds and helm chart gets installed correctly. Happy to do a PR if you think that solution is ok.

closing as a duplicate of #224


[BUG]: Drizzle Studio Postgres showing NaN for uuid primary key with Postgres.

What version of drizzle-orm are you using?
0.30.8
What version of drizzle-kit are you using?
0.20.16
Describe the Bug
Drizzle studio shows NaN for uuid primary key with Postgres. I ran raw SQL just in case, DB is indeed creating and storing uuid as expected.
The id for the third row in the picture is: 94a0efe8-1207-4ba0-ad7f-d71e106c6d98.
I tried flushing db and drizzle-kit push:pg again, same issue.
Drizzle Studio:

Expected behavior
Should show correct uuid value, not NaN
Environment & setup
Schema:
export const evaluationTable = pgTable("evaluation", {
  id: uuid("id").primaryKey().defaultRandom(),
  hirespaceId: uuid("hirespace_id")
    .references(() => hirespaceTable.id)
    .notNull(),
  description: varchar("description", { length: 256 }).notNull(),
});

Postgres Docker Image:
# Use postgres/example user/password credentials
version: "3.1"

services:
  db:
    image: postgres
    volumes:
      - pgdata:/var/lib/postgresql/data
    environment:
      POSTGRES_USER: postgres
      POSTGRES_PASSWORD: testtest
      POSTGRES_DB: dbdb
    ports:
      - 4321:4321
    command: -p 4321
volumes:
  pgdata:

Another person on discord reported the same issue, no code provided from the person though.

Yep, same issue here. Drizzle said I had to upgrade drizzle-kit to keep using it, and started having this problem.
I have text ids with a custom generator function, not uuids.

Same here! defaults values are not being generated from studio from the last version

@dkmooers @dialnco it should be fixed now in<EMAIL_ADDRESS>


Non-cookie token storage

Is your feature request related to a problem? Please describe.
I am implementing an Ionic app using @nuxtjs/ionic. I'd like to use Supabase as my API, but that means I need a way to store authentication tokens using the native plugins.
Describe the solution you'd like
Being able to have my own (async) getter and setter methods in the config would allow me to do this.
Describe alternatives you've considered
I can use cookies, though I worry that they won't be as durable as using dedicated local storage.
Additional context
I am happy to make/work on the change if that's useful!

If you create your own storage, you can pass that into your config.
import { defineNuxtConfig } from 'nuxt'

export default defineNuxtConfig({
  // ...
  supabase: {
    client: {
      auth: {
        storage: YourCustomStorage
      }
    }
  }
}

See https://github.com/supabase/gotrue-js/blob/master/src/lib/local-storage.ts for a generic implementation of getItem setItem and removeItem. Replace the code inside of each method with your own.

If you create your own storage, you can pass that into your config.
import { defineNuxtConfig } from 'nuxt'

export default defineNuxtConfig({
  // ...
  supabase: {
    client: {
      auth: {
        storage: YourCustomStorage
      }
    }
  }
}

See https://github.com/supabase/gotrue-js/blob/master/src/lib/local-storage.ts for a generic implementation of getItem setItem and removeItem. Replace the code inside of each method with your own.

I should have mentioned, I tried this strategy, but because the nuxt config gets serialized to JSON, this way doesn't work.

@dts well that stinks. That's a good reason why they should have the dev create the client - even if it's via a library-provided function. Then you can pass in your custom storage. This is how other auth helpers do it.

Well, I thought other auth helpers used to support custom storage. Maybe it was only the storageKey. The SvelteKit one doesn't support either these, so I was wrong.

I've just gone down this rabbit hole today myself, trying to implement a custom storage mechanism for Ionic.
@dts what did you eventually use as an alternative?
In a previous project a few months ago I chose not to use this nuxt plugin and just used Supabase directly, I then used Pinia with a localStorage adapter. However, I'd love to reduce the number of moving parts and just use this module for my Ionic builds.

🙈 ```import { Preferences } from "@capacitor/preferences";
export async function getItem(key: string): Promise<string | null> {
const v = (await Preferences.get({ key })).value;
return v;
}
export async function setItem(key: string, value: string): Promise {
return await Preferences.set({ key, value });
}
export async function removeItem(key: string): Promise {
return await Preferences.remove({ key });
}
window.NUXT.config.public.supabase.client.auth.storage = {
getItem,
setItem,
removeItem,
};
export default defineNuxtPlugin((nuxtApp) => {
nuxtApp.hook("app:beforeMount", () => {});
});


Feature/move multi box

This PR enables the moving of multiple Boxes at once with the Boxtribute Qr-Reader.
please check tests 3.4.6 as reference
@pylipp I have not validated if a Box which was scanned is already in the location that you want to move it to. What happens in the backend if a send a graphQL mutation with updateBox and the new location id is what the box already has?

 2.0 QrOverlayMultiTab: view for move Box action

Functional review: works :+1: (I did not test the error cases)

@pylipp I have not validated if a Box which was scanned is already in the location that you want to move it to. What happens in the backend if a send a graphQL mutation with updateBox and the new location id is what the box already has?

I tested this case and I can see the notification (a box was successfully moved). The BE executes the change on DB level but won't write an entry to the history table.

@aerinsol Do you think that we should handle this case or are you happy?


how to connect orientdb cluster via mybatis

how to connect orientdb cluster via mybatis. What is the orientdb cluster ip and port?

Config the mybatis enviroments property in config.xml, the code like:
<environments default="development">
    <environment id="development">
    <transactionManager type="JDBC"/>
        <dataSource type="POOLED">
        <property name="driver" value="com.orientechnologies.orient.jdbc.OrientJdbcDriver"/>
        <property name="url" value="jdbc:orient:remote:<IP_ADDRESS>:2425/douban_all;jdbc:orient:remote:<IP_ADDRESS>:2426/douban_all" />
        <property name="username" value="admin"/>
        <property name="password" value="admin"/>
        </dataSource>
    </environment>
</environments>

You just need to connect to one of the servers in the cluster (port 2424, the default one).

i connected successfully but it return nullpointEx to me. is it can use the formal sql?


ocaml_ns version 2

Namespace support inescapably involves a circular dependency.  On the one hand, submodules must depend on the ns module for two things:

submodule compilation must -open the namespace module
in the current implementation, at least, submodules get their namespace prefix from the ns module

On the other hand, the ns module depends on the submodules, since it needs to know which modules to alias.
Version 1 resolves this by having the submodules depend on the ns module via the ns attribute of ocaml_module, and by having the ns module depend on the submodule source files rather than the submodule build targets.  This works, but it's inelegant and error prone, since the build developer must ensure the two sets of deps match up.
The goal is to have the ns module depend on the submodule targets, period.  The proposal is to use a transition function together with a label-valued build setting to support this.  Sketch:

Submodule names can be anything; there is no requirement that the filenames include an ns prefix. All that matters is that the alias point to the right filename.  That is, if the ns is Foo, and the submodule is Bar, the convention is to rename bar.ml to Foo__Bar.ml, and alias it with module Bar = Foo__Bar.  But this is not a requirement. We can give the submodule file any name we please, so long as the alias statement points to it.  In particular, we can use the package name to construct a prefix for the submodule filename.  So if foo.ml and bar.ml are in package //a/b/c, we can rename bar.ml to A__B__C__Bar.ml and alias it thusly:  module Bar = A__B__C__Bar.  In other words, there is no necessary relation between submodule name paths (e.g. Foo.Bar) and filesystem names (Foo.Bar maps to A__B__C__Bar).
We can use a transition function on the submodules attribute of rule ocaml_ns to notify submodules that they are being namespaced. Transition functions set build settings, which are in effect global variables. When a transition function on an attribute sets a build setting, it converts it to a kind of local variable; the setting becomes visible only to the transitive dependency graph rooted at the attribute.  So the ocaml_ns instance will assign a value to build setting @ocaml//ns, and the submodules (ocaml_module instances) will read it to determine whether or not they are being used as submodules. If so, they will construct their filenames using their package names as a prefixes, as noted above.
We still need to compile the ns module so that the submodule can -open it.  The proposal - and I'm not yet sure this will work - is to have the transition function compile the ns module and pass it as the value of @ocaml//ns.

I think this will work, because the dependency of the ns module on its submodules is actually a pseudo-dependency.  The ns module does not depend on the compiled submodules, it only depends on their names.  So it should be possible for the transition function to obtain those names from the submodules attribute of ocaml_ns, write a source file containing the necessary aliases, compile it, and pass it along as the value of @ocaml//ns.

Another possibility: always use the package name as the namespace prefix.  That way both the ocaml_ns and the submodules know what the prefix is, so long as they are in the same package. Actually even if they are in different packages it would work, since the ocaml_ns would be able to extract the package name for all its deps.
The catch is that the submodules need to know that they should use the ns prefix. Simplest way to do this is to make the ns attribute a boolean.  The drawback with this is just the need to add this attrib to each submodule.  The alternative is to use a transition function to set the ns value.

Another possibility: always use the package name as the namespace prefix.  That way both the ocaml_ns and the submodules know what the prefix is, so long as they are in the same package. Actually even if they are in different packages it would work, since the ocaml_ns would be able to extract the package name for all its deps.
The catch is that the submodules need to know that they should use the ns prefix. Simplest way to do this is to make the ns attribute a boolean.  The drawback with this is just the need to add this attrib to each submodule.  The alternative is to use a transition function to set the ns value.

The main unresolved problem is how to make the compiled ns module available to the submodules, who need it for compilation.  We may be able to use the transition function for this as described above.  Not sure how efficient that is.  Another possibility is to have each submodule have a hidden dependency on a hidden package-global ns module, whose only purpose is to support submodule compilation. It would contain an alias statement for every module in the package.  But I'm not sure "hidden ns module" is even possible.
Maybe a hidden attribute whose default value is 'glob(["*.ml"])' ?  If ns is True, compile the helper ns module. No, then each submod would compile it.  We need them all to depend on one target. An ocaml_ns_helper rule?
Best solution would be to put the alias statments on the compiler command line.

The main unresolved problem is how to make the compiled ns module available to the submodules, who need it for compilation.  We may be able to use the transition function for this as described above.  Not sure how efficient that is.  Another possibility is to have each submodule have a hidden dependency on a hidden package-global ns module, whose only purpose is to support submodule compilation. It would contain an alias statement for every module in the package.  But I'm not sure "hidden ns module" is even possible.
Maybe a hidden attribute whose default value is 'glob(["*.ml"])' ?  If ns is True, compile the helper ns module. No, then each submod would compile it.  We need them all to depend on one target. An ocaml_ns_helper rule?
Best solution would be to put the alias statments on the compiler command line.


⚠ Processors has degraded performance

In 8683dc7, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 48 ms

Resolved: Processors performance has improved in 99f00c5.


Functional testing of the search service

[x] Implement and iterate on functional testing framework (PR 36, PR 37, PR 40)
[ ] Add functional tests that hit the key features of the search service

[ ] V3 Search

[ ] Paging (skip + take)
[ ] Count only
[x] Prerelease true/false (PR 36)
[ ] Feed parameter
[ ] All query fields
[x] Schema (PR 40)
[x] Rank by download count (PR 40)

[ ] V2 Search

[ ] Paging (skip + take)
[ ] Count only
[x] Prerelease true/false (PR 40)
[ ] Feed parameter
[ ] Sort by parameter
[ ] All query fields
[ ] Schema
[x] Rank by download count (PR 40)

[ ] Autocomplete

[ ] Paging (skip + take)
[ ] Prerelease true/false
[ ] "q=" vs "id="

[ ] Investigate whether /find or /rankings should be tested or are just diagnostic tools.

[ ] Add search quality tests based on user reports of bad searches
[ ] Design and implement integration testing that can run against DEV, INT, and maybe PROD.

@joelverhagen Can you remember which items were tackled and which not?

@shishirx34 , you implemented some functional tests. Anything we can check off?

Ticked the last one, apart from that I don't think our integration tests verify anything new.

Split out auto-complete functional tests: https://github.com/NuGet/NuGetGallery/issues/7232
Split out V2 search functional tests: https://github.com/NuGet/NuGetGallery/issues/7233

@joelverhagen , can we close this one?

@skofman1, we still need functional tests on the V3 search endpoint (/query) and I left this one to cover it.


Create an ADOPTERS.md to track who is using hydrophone

Something like https://github.com/containerd/containerd/blob/main/ADOPTERS.md ?

/assign

I think this has been fixed with #214.


Fix metadata inference for groupby.agg

Previously this would result in incorrect metadata for multi-indices. We
now fallback to using the automatic inference in aca and
map_partitions.
Fixes #2278.

I pushed a new commit that adds the index name checks to assert_eq and fixes the places where metadata inference was off. This could use a quick review.


Compilation errors in a isolated hello_world app

Hi when i compile the examples from this repo works fine, but in a isolated custom app failed with the same code, i think is a version problem:
error[E0277]: the trait bound `glium::Display: glium::backend::Facade` is not satisfied
  --> src/support/mod.rs:25:24
   |
25 |     let mut renderer = Renderer::init(&mut imgui, &display).expect("Failed to initialize renderer");
   |                        ^^^^^^^^^^^^^^ the trait `glium::backend::Facade` is not implemented for `glium::Display`
   |
   = note: required by `imgui_glium_renderer::Renderer::init`

error[E0277]: the trait bound `glium::Frame: glium::Surface` is not satisfied
   --> src/support/mod.rs:127:18
    |
127 |         renderer.render(&mut target, ui).expect("Rendering failed");
    |                  ^^^^^^ the trait `glium::Surface` is not implemented for `glium::Frame`

my Cargo.toml looks like:
[dependencies]
imgui = "*"
glium = "*"
imgui-glium-renderer = "*"

Thanks in advance!!!

May be you can show the content of your Cargo.lock so that we know exactly what versions you have installed.
I would advise you to work from master rather than the released crates for the time being. You can clone the repository and fill Cargo.toml as below. This should work.
[dependencies]
imgui = { path = "imgui-rs" }
imgui-glium-renderer = { path = "imgui-rs/imgui-glium-renderer" }
glium = { version = "0.20", default-features = true }
imgui-sys = { path = "imgui-rs/imgui-sys", features = ["glium"] }

Effectively your solution works very well. Thanks for your time!!!

This happens because the current release of imgui-glium-renderer depends on glium 0.19, but glium = "*" picks up glium 0.20 instead and you then have two incompatible gliums. You've got two choices:

use the master version of all imgui crates. imgui-rs master uses glium 0.20 so this works
use glium = "0.19" dependency instead of glium = "*"

Closing old issue


what are winrt::composing and winrt::composable ?

I don't understand winrt::composing and winrt::composable?,  where can i find any documents?
i need to resolve a problem:
idl define:
interface IOnly
{
    ...
}

unsealed runtimeclass Only : IOnly
{
     Only(Object inner);
}

namespace winrt
{
class HotKey : public winrt::implements<HotKey, composing, Only, IClosable>
{
}
}
if i did not authorize a runtime class HotKey , how can i compose Only  runtime class?

@DefaultRyan

I would highly recommend letting C++/WinRT implement this for you (rather than using implement directly) via the -component flag. It knows where these markers go - I don't recall as its a rather complicated dance.

I'm a bit late to the party, but the short of it is that a WinRT type that can be instantiatable as a base class across the ABI boundary, is referred to as composable in WinRT parlance. One part of implementing this scheme uses COM Aggregation, which requires lighting up some extra logic in both the base (inner) class and the derived (outer) class.
winrt::composable is generated by cppwinrt to mark that a type can function as a composable base class, and winrt::composing is generated by cppwinrt to mark that a type is subclassing a composable bass class. Both of these light up the extra COM aggregation logic needed for this scenario, and you should avoid using them manually.


Mac 10.15 Ansible PR Builder fails due to incorrect usage of brew cask install

TASK [Common : Install Build Tool Casks] ***************************************
failed: [localhost] (item=adoptopenjdk10) => {"ansible_loop_var": "item", "changed": false, "item": "adoptopenjdk10", "msg": "Updating Homebrew...\nError: Calling brew cask install is disabled! Use brew install [--cask] instead."}
failed: [localhost] (item=packages) => {"ansible_loop_var": "item", "changed": false, "item": "packages", "msg": "Error: Calling brew cask install is disabled! Use brew install [--cask] instead."}

Please set the title to indicate the test name and machine name where known.
To make it easy for the infrastructure team to repeat and diagnose, please
answer the following questions:

test suite/name?
Is there an existing issue elsewhere covering this?
Which machine(s) does it work on?
Which machine(s) does it fail on?
Do you have a link to a Grinder re-run if the test with the failure?

Any other details:

This is a bug with the upstream ansible cask module. Checking if it's been fixed yet

This is a bug with the upstream ansible cask module. Checking if it's been fixed yet

It has been fixed - as of Ansible 2.10.5 (see: https://github.com/ansible-collections/community.general/issues/1524#issuecomment-755126741 ). For some reason, however, the PR-Builder installs an older version of Ansible, despite 2.10.5 being the latest version (https://formulae.brew.sh/formula/ansible)

It has been fixed - as of Ansible 2.10.5 (see: https://github.com/ansible-collections/community.general/issues/1524#issuecomment-755126741 ). For some reason, however, the PR-Builder installs an older version of Ansible, despite 2.10.5 being the latest version (https://formulae.brew.sh/formula/ansible)

Okay, as of https://github.com/AdoptOpenJDK/openjdk-infrastructure/pull/1870/checks?check_run_id=1782760737 , it appears to be installing the correct version of Ansible.
New Issue:
Run brew untap homebrew/cask-versions
  brew untap homebrew/cask-versions
  brew cask uninstall adoptopenjdk8
  brew cask uninstall adoptopenjdk11
  brew cask uninstall adoptopenjdk12
  brew cask uninstall adoptopenjdk13
  brew cask uninstall adoptopenjdk14
  shell: /bin/bash -e {0}
Error: No available tap homebrew/cask-versions.
Error: Process completed with exit code 1.

Okay, as of https://github.com/AdoptOpenJDK/openjdk-infrastructure/pull/1870/checks?check_run_id=1782760737 , it appears to be installing the correct version of Ansible.
New Issue:
Run brew untap homebrew/cask-versions
  brew untap homebrew/cask-versions
  brew cask uninstall adoptopenjdk8
  brew cask uninstall adoptopenjdk11
  brew cask uninstall adoptopenjdk12
  brew cask uninstall adoptopenjdk13
  brew cask uninstall adoptopenjdk14
  shell: /bin/bash -e {0}
Error: No available tap homebrew/cask-versions.
Error: Process completed with exit code 1.

Fixed as of #1873

Fixed as of #1873


descriptor: fix wrong handling of missing origin in multisig

Example (second key is missing an origin):
wsh(multi(2,[00000001/48'/0'/0'/2']tpubD6NzVbkrYhZ4WaWSyoBvQwbpLkojyoTZPRsgXELWz3Popb3qkjcJyJUGLnL4qHHoQvao8ESaAstxYSnhyswJ76uZPStJRJCTKvosUCJZL5B/0/0,tpubDFHiBJDeNvqPWNJbzzxqDVXmJZoNn2GEtoVcFhMjXipQiorGUmps3e5ieDGbRrBPTFTh9TXEKJCwbAGW9uZnfrVPbMxxbFohuFzfT6VThty/0/0))
Before, the fingerprint and origin keypath of the second key was the
same as the first one, as the loop parsing it did not reset it.
Now, the fingerprint and origin keypath of the second key is None.

cc @ben-kaufman , according to the git history you are the best person to tag :)

I have a complete reimplementation of the Descriptor class that should resolve this problem. Currently how descriptors are implemented is incredibly hacky and isn't very extensible.

I see. I am currently implementing multisig support for the BitBox02 in HWI, making use of the the descriptor param in display_address(). When do you expect it to land? Should I build on what's on master or on your PR/branch?


Improve article 2401.18058

Exact location of issue
Article fails to convert.
Problem details
From the log report:
(Processing content /dev/shm/UXYIL1NNL3/acl_latex.bbl... 1.23 sec)
(Processing content /dev/shm/UXYIL1NNL3/060appendix.tex...
Fatal:timeout:token_limit Token limit of 249999999 exceeded, infinite loop?
 at 060appendix.tex; line 32 col 0 - line 32 col 15
 In Core::Gullet[@0x5593abeddfb8] /dev/shm/UXYIL1NNL3/060appendix.tex; from line 32 col 0 to line 32 col 15
(Building...
(Loading compiled schema /usr/local/share/perl/5.34.0/LaTeXML/resources/RelaxNG/LaTeXML.model... 0.01 sec) 0.11 sec)
(Rewriting... 0.00 sec)
(Finalizing... 0.00 sec)
Conversion complete: 4 warnings; 1 fatal error; 2 missing files[utfsym.sty, fontawesome.sty]

The article needs support for utfsym.sty and fontawesome.sty and (possibly independently) encounters an infinite loop in the appendix, after loading a .bbl file raw.

Thank you for the report! I have updated the issue description with details from the log.


No bluetooth in console image

Describe the bug
Bluetoth service fails to start.
systemctl status reports a degraded state.
systemctl list-units --failed reports:
  UNIT                        LOAD   ACTIVE SUB    DESCRIPTION
● bb-wl18xx-bluetooth.service loaded failed failed BB WL18xx Bluetooth Service

Describe which image you are running
bone-debian-10.3-console-armhf-2020-03-16-1gb.img.xz
Describe your hardware
BeagleBone Green Wireless
Describe any Capes Installed
None
Output of /opt/scripts/tools/version.sh
eeprom:[A335BNLTGW1ABBGW18042503]
model:[TI_AM335x_BeagleBone_Green_Wireless]
dogtag:[BeagleBoard.org Debian Console Image 2020-03-16]
bootloader:[eMMC-(default)]:[/dev/mmcblk1]:[U-Boot 2019.04-00002-g07d5700e21]:[location: dd MBR]
UBOOT: Booted Device-Tree:[am335x-bonegreen-wireless-uboot-univ.dts]
UBOOT: Loaded Overlay:[AM335X-PRU-RPROC-4-19-TI-00A0]
UBOOT: Loaded Overlay:[BB-ADC-00A0]
UBOOT: Loaded Overlay:[BB-BBGW-WL1835-00A0]
UBOOT: Loaded Overlay:[BB-BONE-eMMC1-01-00A0]
kernel:[4.19.94-ti-r37]
/boot/uEnv.txt Settings:
uboot_overlay_options:[enable_uboot_overlays=1]
uboot_overlay_options:[uboot_overlay_pru=/lib/firmware/AM335X-PRU-RPROC-4-19-TI-00A0.dtbo]
uboot_overlay_options:[enable_uboot_cape_universal=1]
pkg check: to individually upgrade run: [sudo apt install --only-upgrade <pkg>]
pkg:[bb-cape-overlays]:[4.14.20200312.0-0rcnee0~buster+20200312]
pkg:[bb-wl18xx-firmware]:[1.20200226.1-0rcnee0~buster+20200226]
pkg:[kmod]:[26-1]
WARNING:pkg:[librobotcontrol]:[NOT_INSTALLED]
pkg:[firmware-ti-connectivity]:[20190717-2rcnee1~buster+20200305]
groups:[debian : debian adm kmem dialout cdrom floppy audio dip video plugdev users systemd-journal netdev gpio pwm eqep remoteproc admin spi iio i2c docker tisdk weston-launch xenomai bluetooth cloud9ide]
cmdline:[console=ttyO0,115200n8 bone_capemgr.uboot_capemgr_enabled=1 root=/dev/mmcblk1p1 ro rootfstype=ext4 rootwait coherent_pool=1M net.ifnames=0 rng_core.default_quality=100 quiet]
dmesg | grep remote
[    1.548058] remoteproc remoteproc0: wkup_m3 is available
[    1.550064] remoteproc remoteproc1: 4a334000.pru is available
[    1.550562] remoteproc remoteproc2: 4a338000.pru is available
[    1.717622] remoteproc remoteproc0: powering up wkup_m3
[    1.717741] remoteproc remoteproc0: Booting fw image am335x-pm-firmware.elf, size 217168
[    1.721414] remoteproc remoteproc0: remote processor wkup_m3 is now up
dmesg | grep pru
[    1.550064] remoteproc remoteproc1: 4a334000.pru is available
[    1.550176] pru-rproc 4a334000.pru: PRU rproc node pru@4a334000 probed successfully
[    1.550562] remoteproc remoteproc2: 4a338000.pru is available
[    1.550718] pru-rproc 4a338000.pru: PRU rproc node pru@4a338000 probed successfully
dmesg | grep pinctrl-single
[    1.072213] pinctrl-single 44e10800.pinmux: 142 pins, size 568
dmesg | grep gpio-of-helper
[    1.084583] gpio-of-helper ocp:cape-universal: ready
lsusb
Bus 001 Device 002: ID 05e3:0610 Genesys Logic, Inc. 4-port hub
Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub
END

To Reproduce
Boot the board
Expected behavior
Bluetooth services start and bluetooth LED is ON.
Additional context
I've fixed this by installing bluez package.
I think that bluez package should be installed by default in this image.

debian@beaglebone:~$ df -h | grep mmc
/dev/mmcblk0p1  866M  458M  348M  57% /
debian@beaglebone:~$ sudo apt install bluez
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following additional packages will be installed:
  libasound2 libasound2-data libdw1
Suggested packages:
  pulseaudio-module-bluetooth libasound2-plugins alsa-utils
The following NEW packages will be installed:
  bluez libasound2 libasound2-data libdw1
0 upgraded, 4 newly installed, 0 to remove and 0 not upgraded.
Need to get 1450 kB of archives.
After this operation, 5244 kB of additional disk space will be used.
Do you want to continue? [Y/n]

@JoseGoncalves , yeah i'm fine with this, it only add 5MB, thanks for the issue!


Save the database shape in history.

Is your feature request related to a problem? Please describe.
when we draw a database shape, then click the undo action, but it is still remain on the board.

Fixed


New sample data loader and ldap entries for demo users

Resolves https://github.com/eclipse-pass/main/issues/386
Resolves https://github.com/eclipse-pass/main/issues/373
Resolves https://github.com/eclipse-pass/main/issues/389
Add data necessary for some demo users with the usual password. Username is our GH username

aforward
amoore
bbranen
grant-mcs
jabrah
jaredgalanis
jgara
jrmartino
kineticsquid
markpatton
springstim
tsande16

Need to add grants to these users
Known Bugs
Simplest way to add grants is to add all new users as coPis to the same grant. This leads to some bad UI behavior where each coPi on the grant can see the submissions from other users associated with the grant. This means that draft PRs from one coPi can be seen and edited by other coPis :)
Workaround should now be in place

Need to update the UI with this bugfix: https://github.com/eclipse-pass/pass-ui/pull/1157

Currently deployed https://demo.eclipse-pass.org/ but the new users can't seem to be used when logging in. May have to use the docker removeall script

Had forgotten the disconnect between the deployed environments and the local dev environment (eclipse-pass.base.yml vs demo.yml)


Scripts for Running HumanEval Benchmark

Is there a script that can reproduce the results in the paper on HumanEval benchmark?

Is there a script that can reproduce the results in the paper on HumanEval benchmark?

Same I would like to get access to the scripts to reproduce humaneval results.

Hi, I would like to ask about the schedule for the release of this script

@stellaHSR

Thank you for your patience. Due to limited staffing, we are currently working through the test stages and finalizing the scripts. Once everything is validated, we will release the scripts and update this thread as soon as possible.


How to visualize the tracking result on avi vedio file?

in the readme:
where "img_folder" is a folder containing the original KITTI tracking images
Do I need to transfer the avi vedio file (from CityFlow dataset)  to image?

Yes, our tool only supports visualizing on images, so you would need to extract the video frames first

Closing this for now, please reopen if needed


pre-training for the Imitation learning

Hi, all. I am a Ph.D. student from NTU, Singapore and also work in the mapless navigation research direction. I must admit this is an amazing project, however, I am struggling with a problem. Are you training both Actor and Critic neural networks in the Imitation learning process? Look forward to your soon response. Thanks.

@lukewenMX hi, I am an undergraduate in NUS, doing a project based on this paper as well and having some troubles including loading the pre-trained weights. If you don't mind, are there any way I could reach out to you for further discussions?

The pre-training weights are only used to initialize the actor. I admit, this could be stated more clearly in the paper. Initializing the critic with the same weights would not make sense. The critic is trained from scratch in every training run.

@lukewenMX @kermeed
I ran the code but met a problem.
The problem is that the loaded robot model can't run within the map.
I guess it's caused by that the coordinate system's origin of /odom and /map is mismatching, however I can't solve it.

Did you met the problem? Looking forward to your reply. Thanks.

@minded-hua
Hello,  I want to ask about the operation of the experiment. Whether I initialize or retrain, I never succeed once, and the robot always fails to reach the target point. I think it's probably a problem with my 2D simulator, stageros. I used the stageros simulator that came with ROS kinetic, but its clock was so slow that the training time was too long. So, I modified the time parameters, but I also found a problem, that is, the simulator can't send a stall signal, I don't know if it's because of this that my robot can't reach the target.

@minded-hua
Hello,  I want to ask about the operation of the experiment. Whether I initialize or retrain, I never succeed once, and the robot always fails to reach the target point. I think it's probably a problem with my 2D simulator, stageros. I used the stageros simulator that came with ROS kinetic, but its clock was so slow that the training time was too long. So, I modified the time parameters, but I also found a problem, that is, the simulator can't send a stall signal, I don't know if it's because of this that my robot can't reach the target.

@minded-hua
Hello,  I want to ask about the operation of the experiment. Whether I initialize or retrain, I never succeed once, and the robot always fails to reach the target point. I think it's probably a problem with my 2D simulator, stageros. I used the stageros simulator that came with ROS kinetic, but its clock was so slow that the training time was too long. So, I modified the time parameters, but I also found a problem, that is, the simulator can't send a stall signal, I don't know if it's because of this that my robot can't reach the target.

@minded-hua
Hello,  I want to ask about the operation of the experiment. Whether I initialize or retrain, I never succeed once, and the robot always fails to reach the target point. I think it's probably a problem with my 2D simulator, stageros. I used the stageros simulator that came with ROS kinetic, but its clock was so slow that the training time was too long. So, I modified the time parameters, but I also found a problem, that is, the simulator can't send a stall signal, I don't know if it's because of this that my robot can't reach the target.

@minded-hua
Hello,  I want to ask about the operation of the experiment. Whether I initialize or retrain, I never succeed once, and the robot always fails to reach the target point. I think it's probably a problem with my 2D simulator, stageros. I used the stageros simulator that came with ROS kinetic, but its clock was so slow that the training time was too long. So, I modified the time parameters, but I also found a problem, that is, the simulator can't send a stall signal, I don't know if it's because of this that my robot can't reach the target.

@minded-hua
Hello,  I want to ask about the operation of the experiment. Whether I initialize or retrain, I never succeed once, and the robot always fails to reach the target point. I think it's probably a problem with my 2D simulator, stageros. I used the stageros simulator that came with ROS kinetic, but its clock was so slow that the training time was too long. So, I modified the time parameters, but I also found a problem, that is, the simulator can't send a stall signal, I don't know if it's because of this that my robot can't reach the target.

@minded-hua
Hello,  I want to ask about the operation of the experiment. Whether I initialize or retrain, I never succeed once, and the robot always fails to reach the target point. I think it's probably a problem with my 2D simulator, stageros. I used the stageros simulator that came with ROS kinetic, but its clock was so slow that the training time was too long. So, I modified the time parameters, but I also found a problem, that is, the simulator can't send a stall signal, I don't know if it's because of this that my robot can't reach the target.

@minded-hua
Hello,  I want to ask about the operation of the experiment. Whether I initialize or retrain, I never succeed once, and the robot always fails to reach the target point. I think it's probably a problem with my 2D simulator, stageros. I used the stageros simulator that came with ROS kinetic, but its clock was so slow that the training time was too long. So, I modified the time parameters, but I also found a problem, that is, the simulator can't send a stall signal, I don't know if it's because of this that my robot can't reach the target.

@minded-hua
Hello,  I want to ask about the operation of the experiment. Whether I initialize or retrain, I never succeed once, and the robot always fails to reach the target point. I think it's probably a problem with my 2D simulator, stageros. I used the stageros simulator that came with ROS kinetic, but its clock was so slow that the training time was too long. So, I modified the time parameters, but I also found a problem, that is, the simulator can't send a stall signal, I don't know if it's because of this that my robot can't reach the target.

@minded-hua
Hello,  I want to ask about the operation of the experiment. Whether I initialize or retrain, I never succeed once, and the robot always fails to reach the target point. I think it's probably a problem with my 2D simulator, stageros. I used the stageros simulator that came with ROS kinetic, but its clock was so slow that the training time was too long. So, I modified the time parameters, but I also found a problem, that is, the simulator can't send a stall signal, I don't know if it's because of this that my robot can't reach the target.

@minded-hua
Hello,  I want to ask about the operation of the experiment. Whether I initialize or retrain, I never succeed once, and the robot always fails to reach the target point. I think it's probably a problem with my 2D simulator, stageros. I used the stageros simulator that came with ROS kinetic, but its clock was so slow that the training time was too long. So, I modified the time parameters, but I also found a problem, that is, the simulator can't send a stall signal, I don't know if it's because of this that my robot can't reach the target.

@minded-hua
Hello,  I want to ask about the operation of the experiment. Whether I initialize or retrain, I never succeed once, and the robot always fails to reach the target point. I think it's probably a problem with my 2D simulator, stageros. I used the stageros simulator that came with ROS kinetic, but its clock was so slow that the training time was too long. So, I modified the time parameters, but I also found a problem, that is, the simulator can't send a stall signal, I don't know if it's because of this that my robot can't reach the target.

@minded-hua
Hello,  I want to ask about the operation of the experiment. Whether I initialize or retrain, I never succeed once, and the robot always fails to reach the target point. I think it's probably a problem with my 2D simulator, stageros. I used the stageros simulator that came with ROS kinetic, but its clock was so slow that the training time was too long. So, I modified the time parameters, but I also found a problem, that is, the simulator can't send a stall signal, I don't know if it's because of this that my robot can't reach the target.

@minded-hua
Hello,  I want to ask about the operation of the experiment. Whether I initialize or retrain, I never succeed once, and the robot always fails to reach the target point. I think it's probably a problem with my 2D simulator, stageros. I used the stageros simulator that came with ROS kinetic, but its clock was so slow that the training time was too long. So, I modified the time parameters, but I also found a problem, that is, the simulator can't send a stall signal, I don't know if it's because of this that my robot can't reach the target.

@lukewenMX hi, I am an undergraduate in NUS, doing a project based on this paper as well and having some troubles including loading the pre-trained weights. If you don't mind, are there any way I could reach out to you for further discussions?

Hello, do you have a WeChat ID?  We can talk with WeChat.
------------------ 原始邮件 ------------------
发件人<EMAIL_ADDRESS>发送时间: 2020年9月2日(星期三) 中午11:05
收件人<EMAIL_ADDRESS>抄送<EMAIL_ADDRESS><EMAIL_ADDRESS>主题: Re: [ethz-asl/rl-navigation] pre-training for the Imitation learning (#5)
@minded-hua
Hello,  I want to ask about the operation of the experiment. Whether I initialize or retrain, I never succeed once, and the robot always fails to reach the target point. I think it's probably a problem with my 2D simulator, stageros. I used the stageros simulator that came with ROS kinetic, but its clock was so slow that the training time was too long. So, I modified the time parameters, but I also found a problem, that is, the simulator can't send a stall signal, I don't know if it's because of this that my robot can't reach the target.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

@sumingzs11 Do you have a WeChat ID? We can talk with WeChat.

Sorry, I was just having lunch. If you are willing to communicate on WeChat, that is really great. My wechat id is
ibelieveyouonly
---Original---
From<EMAIL_ADDRESS>Date: Wed, Sep 2, 2020 12:12 PM
To<EMAIL_ADDRESS>Cc: "Mention"<mention<EMAIL_ADDRESS>Subject: Re: [ethz-asl/rl-navigation] pre-training for the Imitation learning (#5)
@sumingzs11 Do you have a WeChat ID? We can talk with WeChat.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

------------------ 原始邮件 ------------------
发件人:                                                                                "ethz-asl/rl-navigation"<EMAIL_ADDRESS>发送时间: 2020年9月2日(星期三) 中午12:12
收件人<EMAIL_ADDRESS>抄送: "苏铭"<847366070<EMAIL_ADDRESS>主题: Re: [ethz-asl/rl-navigation] pre-training for the Imitation learning (#5)
@sumingzs11 Do you have a WeChat ID? We can talk with WeChat.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


manage_master_user_password_rotation does not disable rotation

Description
It seems that the manage_master_user_password_rotation = false option does not disable the rotation.
Here's my RDS setup:
module "rds" {
  source  = "terraform-aws-modules/rds/aws"
  version = "6.9.0"

  identifier     = "my-rds"
  family         = "postgres16"
  engine         = "postgres"
  engine_version = "16.4"
  instance_class = var.rds_instance_class

  db_name  = local.db_name
  port     = local.db_port
  username = local.db_user
  manage_master_user_password          = true
  manage_master_user_password_rotation = false
  iam_database_authentication_enabled = false

  vpc_security_group_ids = [module.rds_sg.security_group_id]
  db_subnet_group_name   = aws_db_subnet_group.database.name

  allocated_storage = 20
  storage_encrypted = var.rds_storage_encrypted
}

After the RDS is created, the secret has the default rotation of 7 days.
The issue was reported in the past (#547 , #538), but it seems that it doesn't work with the latest version. Also, I tried to fall-back to 6.5.5 since the comments in those issues suggested that it was fixed in there, but still no success.

[X ] ✋ I have searched the open/closed issues and my issue is not listed.

Versions

Module version [Required]: 6.9.0

Terraform version: v.1.9.5

Provider version(s):  5.65.0

Reproduction Code [Required]
module "rds" {
  source  = "terraform-aws-modules/rds/aws"
  version = "6.9.0"

  identifier     = "my-rds"
  family         = "postgres16"
  engine         = "postgres"
  engine_version = "16.4"
  instance_class = var.rds_instance_class

  db_name  = local.db_name
  port     = local.db_port
  username = local.db_user
  manage_master_user_password          = true
  manage_master_user_password_rotation = false
  # password                            = var.rds_master_user_password
  iam_database_authentication_enabled = false

  vpc_security_group_ids = [module.rds_sg.security_group_id]
  db_subnet_group_name   = aws_db_subnet_group.database.name

  allocated_storage = 20
  storage_encrypted = var.rds_storage_encrypted
}

Steps to reproduce the behavior:

No

Yes

Expected behavior
The secret created by RDS does not have rotation set

Actual behavior
The secret that is created has the default rotation of 7 days

Terminal Output Screenshot(s)

Additional context

This comment is applicable here I believe: https://github.com/terraform-aws-modules/terraform-aws-rds/issues/538#issuecomment-1965606613

I'm having same issue on version 6.10.0


CNV-43068: Catalog -> "Add volume" should be enabled for regular user

📝 Description
Allow a regular user to create volume to his namespace
🎥 Demo
Before:

After:

/cherry-pick release-4.16


job-dsls: add image sync job

Added tools jobs for image sync between clouds.

How to retest a PR or trigger a specific build:

* a pull request please add comment: Jenkins retest this

Superseded by https://github.com/kiegroup/kie-jenkins-scripts/pull/1160 - closing.


DBZ-4920 Use Kafka broker as bootstrap server

https://issues.redhat.com/browse/DBZ-4920

Any idea what the build failure is about, @jpechane?

@gunnarmorling The missing method was renamed as part of DBZ-4801 - but this was not update in Debezium UI.

@gunnarmorling Fixed in https://github.com/debezium/debezium-ui/pull/551


Add assert to Table to check for rows with null children

Description
Add a simple assert that checks for TableRow instances with null children being added to a Table.
Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I signed the [CLA].
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read [Handling breaking changes].

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: https://flutter.dev/go/breaking-changes-template Replace this with a link to a pull request that adds the migration guide to https://flutter.dev/docs/release/breaking-changes

Looks good, it should be fairly simple to add a test for this. Can you add one? Thanks.

Looks good, it should be fairly simple to add a test for this. Can you add one? Thanks.

Done


improvement suggestions for search feature

Some small requests/suggestions regarding the replay vault search:

allow users to filter by ladder rating (currently you can filter by global only)
allow users to sort by review score/amount of reviews (would help casters and others who wanna watch nice replays)
change the featured game mode filter to a dropdown displaying the available game modes so people don't have to guess how the ladder game mode is called

another issue i stumbled across:
when looking for specific factions you get these results
cybran -> aeon
aeon -> uef
uef -> cybran
seraphim -> seraphim
i guess the main faction ids are messed up somewhere

Please always make one issue per issue. These are all seperate issues but as soon as i fix one i close the ticket and all the others are lost!!!!

Fixed in #2044


[ODS-4703][ODS-4704] Consistency in naming containers and volumes

Note Ods-Web was renamed to Ods-Web-Api, and I updated the docker build.


Failing e2e tests - scorecard-head on ossf-tests/scorecard-action

Matrix: null
Repo: https://github.com/ossf-tests/scorecard-action/tree/main
Run: https://github.com/ossf-tests/scorecard-action/actions/runs/4395398117
Workflow name: scorecard-head
Workflow file: https://github.com/ossf-tests/scorecard-action/tree/main/.github/workflows/scorecards-head.yml
Trigger: schedule
Branch: main

Is anyone working on it?
I can think of two things:

API doesn't work
Network can be an issue - but in this scenario, doesn't look like
Timeout is very less

I haven't looked in detail but let me know

Yeah, we set a 10s timeout. Normally the requests complete within 1-2s, but this run took 10.9s which is why the action gave up.


Update dependency skopeo to v1.16.0

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


nearby bug

Before creating an issue, please search the existing issues to see if a similar one has already been created. You can search issues by specific labels (e.g. label:nearby ) or just by typing keywords into the search filter.
Summary:
Summarize your issue in one sentence (what goes wrong, what did you expect to happen)
Steps to reproduce:
How can we reproduce the issue?
Add System logs:
Add logcat files here (if possible).
Expected behavior:
What did you expect the App to do?
Observed behavior:
What did you see instead?  Describe your issue in detail here.
Device and Android version:
What make and model device (e.g., Samsung J7) did you encounter this on?  What Android
version (e.g., Android 4.0 Ice Cream Sandwich or Android 6.0 Marshmallow) are you running?  Is it
the stock version from the manufacturer or a custom ROM ?
Commons app version:
You can find this information by going to the navigation drawer in the app and tapping 'About'
Screen-shots:
Can be created by pressing the Volume Down and Power Button at the same time on Android 4.0 and higher.
Would you like to work on the issue?
Please let us know whether you want to fix the issue by yourself. If not, anyone can get the issue assigned to them.

Sorry it was created by mistake


Broken link on Zed website

Check for existing issues

[X] Completed

Describe the bug / provide steps to reproduce it
Click the "Download" link in the footer of https://zed.dev
Environment
https://zed.dev
If applicable, add mockups / screenshots to help explain present your vision of the feature
No response
If applicable, attach your ~/Library/Logs/Zed/Zed.log file to this issue.
If you only need the most recent lines, you can run the zed: open log command palette action to see the last 1000.
No response

Thank you for opening this issue @Mattsi-Jansky. You can only access that page if you are logged in for the time being. But if you are logged in and it doesn't work, please let me know what you see.

If you are logged in you still get forwarded first to the page with link not found(404) and then redirected to the download page. If you are logged out, you only get the 404 page. This only happens with the link in the footer, not via the download button right top in the page.

I'm going to go ahead and close this issue out.  We will be allowing everyone to download Zed here shortly and so fairly soon, this button will no longer do any permissions checking in this future, and this 404 thing should just go away.


Add appcontext

We need to save runtime app configuration (work as a session, profile...).
Let's start with saving the current working directory, later registering available :whale: (layers, optimizers....)  type will be added
First thing that will be done is...

[x] Design #728

[x] add header
[x] add basic ~mock~ tests
[x] add implementation

[x] Basic working

[x] Current working directory #728
[x] wire app context to model #733

[x] Register factories

[x] register built-in objects #745

[x] change factory function spec #745

[x] test with optimizer

[x] change opt enum type to string #721
[x] try register custom optimizer #745
[x] add createObject function to adam and sgd #746
[ ] ~Add (sort of) virtual copy constructor~ Canceled as waiting for ccapi refactoring
[x] register built-in optimizer #746

[x] apply the behavior to the layers

[x] change layer type enum to string #725
[x] add createObject funtion to each object #753
[x] register built-in layers #753

[x] Dynamic loading support

[x] Add dynamic loader (resolving path and other stuff..) (#908)
[x] Add dynamic registerer format (with defined symbols for constructor, destructor) (#908)
[x] Add plugin loader (get path from conv, env) (#915)
[x] Add documentation (#915)

[x] Demonstration

[x] Add separated custom layer (say, pow2) (#779)

[ ] CAPI support (need discussion, nothing has set yet)

:octocat: cibot: Thank you for posting issue #716. The person in charge will reply soon.

Can you expand a little on this context about its purpose - creating a global session would require passing that session everywhere internally and externally, which is not convenient.

This is to add a configuration about additional (and dynamic) setup.
This is to resolve #370 and support registering a customized layer and setting up a LOG_DIR, possibly others (like LOG_DEBUG_LEVEL)..
I am thinking code like below, I am open to design suggestion :)
init_global_context(void) __attribute__((constructor)); # initiate and configure singletone instance for nntrianer::AppContext::Global

fini_global_context(void) __attribute__(destructor));

int main() {
# If want to use custom context
  NNTrainer nntr = new nntrainer::AppContext();
# if want to use global configuration
  NNTrainer nntr = nntrainer::AppContext::Global();
#

  nntr.setWorkingDirectory("/path/"); # to resolve #370
  nntr.setLogDirectory("/path/");

  nntr.register_layer("custom", CustomLayer);
  nntr.register_optimizer("custom_opt", CustomOptimizer);

  createModel(model, global_context=nntrainer::AppContext::Global());
}

How is AppContext different from the model itself.
int main() {
  createModel(model);
  model.addDynamicLibPaths("/path/"); // or with env variable
  model.setLogPath("/path/");
  model.setSavePath("/path/");

  model.register_layer("custom", CustomLayer);
  model.addLayer(std::make_unique<CustomLayer>());
  model.addLayer("custom", CustomLayer);
  model.register_optimizer("custom_opt", CustomOptimizer);
}

Registering layers/optimizers to a context creates issues that now the context must also be saved/shared among multiple models and creates more things to do from the user. Rather, if the model is assumed to be the global object which encapsulates everything, then those issues. Yes, with multiple models, custom layers/optimizers will have to be registered multiple times with each model (it's an overhead) but saves providing context everywhere.
Register functions can be outside models for nntrainer directly which affects the core nntrainer like below:
int main() {
  createModel(model);
  model.addDynamicLibPaths("/path/"); // or with env variable
  model.setLogPath("/path/");
  model.setSavePath("/path/");

  nntrainer::register_layer("custom", CustomLayer);
  model.addLayer(std::make_unique<CustomLayer>());
  model.addLayer("custom", CustomLayer);
  nntrainer::register_optimizer("custom_opt", CustomOptimizer);
}

When a user is creating a custom layer or optimizer, she won't be able to access the savePath or other configurable variable if we are to set those variable inside a (global) model. If we have a context parameter, user can access it because it will be evaluated in lazy fashion.

Register functions can be outside models for nntrainer directly which affects the core nntrainer like below:
The core idea of making AppContext is accessing the core nntrainer without any object declared in global space and everyone accessing it. Creating another session is more of an additional functionality as of now.

nntrainer::register_layer("custom", CustomLayer, context=Global_context())  or we can have `nntrainer::register_layer("custom", CustomLayer") as a synthetic sugar.

but saves providing context everywhere.
This will default to the global context so, it won't create an overhead if the user is not customizing anything but leaving room to customizing it.

When a user is creating a custom layer or optimizer, she won't be able to access the savePath or other configurable variable if we are to set those variable inside a (global) model. If we have a context parameter, user can access it because it will be evaluated in lazy fashion.

These variables are meant for the layers/optimizers. They are meant for the model. These custom objects are supposed to receive the file objects (processed objects) than direct values provided by the user.

The core idea of making AppContext is accessing the core nntrainer without any object declared in global space and everyone accessing it. Creating another session is more of additional functionality as of now.

The core idea is good, but this is usually required when in the global space things are being over-written (example - user wants to create multiple layers with the same name) or overhead is coming on, but I don't see either being an issue in nntrainer.

This will default to the global context so, it won't create an overhead if the user is not customizing anything but leaving room to customizing it.

This creates the issue that the model file is not complete itself anymore. It needs to be paired with the appropriate context file which just complicates things.

These variables are not meant for the layers/optimizers. They are meant for the model. These custom objects are supposed to receive the file objects (processed objects) than direct values provided by the user.

I think we are not sure at this stage if we need configuration for layers as well, say, if we have to restrict num threads globally or force it to cpu for debuging purpose? allow fused operations? setting debug level for a specific layer? Those can be done by passing a variable from a model, and it will couple model and layer, those changing layer spec will tackle model method as well. More or less, this is more flexible than only model having such configuration members.

This creates the issue that the model file is not complete anymore by itself. It needs to be paired with the appropriate context file which just complicates things.

From the user's point of view, nothing will be changed if user won't customize anything so nothing is more complicated for the user.

This encapsulates a configuration to an object and model can have the configuration (or function can use as an argument), which is less complicated and more orthogonal than having n members for each model.

for example, addDynamicPathLib should be at the same level as register_layer, let's say if we are running two models and model add dynamic path lib?

If you worry about it is complicating the user's code.
How about model has a context member? We can think of it as a bunch of member variables packed inside a configuration class. Only leaving flexibility to extend.
class Model {
public:
  Model(AppContext ac_ = AppContext::AppInstance(), other members...): app_context(ac_){
  }
}

We had an offline discussion to make appcontext but restrict its usage to be a member of Model for now and later decide details later

Now this is done except for the capi support. I'll make a separate issue for the capi support to keep it simple.


drude with dns-gen

Hi,
I am using drude with dns-gen on mac and on linux.  I think this configuration has some advantages against nginx-proxy and blinkreaction/dns-discovery  in an local multi project dev environment.
I can provide a installation instruction and a new vagrant file for mac and a installtion instruction for ubuntu 15.10
Maybe that can be an installaton option for drude? What do you think?
Thanks

Hi @christianwiedemann,
Would you share your setup and config details?
What are the local multi project use cases, that you think are not currently covered by Drude or can be handled better?

Hi @lmakarov
sorry for my late response. First I want to thank you for dsh. It has completely improved the local dev process at my company.
The advantage of the dns configuration is that you can access the containers directly. So it is easier and more straight forward to connect to mysql or any other databases. Also xdebug etc. can be accessed on port 9000.
As a consequence every non http service can be accessed by domain name. You need no longer static ips in your docker-compose files.
And it is possible to configure own domain names (Not important but nice).
The vagrant file is attached. I have listed a short description of what to do on linux and mac:
Linux:

append --bip="<IP_ADDRESS>/24 to your docker/default or your /lib/systemd/system/docker.service
@see https://hub.docker.com/r/jderusse/dns-gen/ "Start the container automatically after reboot"
Start the docker daemon.
docker run --daemon --name dns-gen
--restart always
--publish <IP_ADDRESS>:53:53/udp
--volume /var/run/docker.sock:/var/run/docker.sock
jderusse/dns-gen
Add <IP_ADDRESS> to your dns configuration

Mac:
Replace the vagrant file in your projects folder
Add a route
sudo route -n add <IP_ADDRESS>/16 <IP_ADDRESS>
Set the boot2docker ip as your dns
After dsh up clean your dns cache
sudo killall -HUP mDNSResponder
If you are interested, let me know. I can make a pull request and try integrate that into dsh.
vagrant.txt

@christianwiedemann, If I understand you correctly, you are proposing to add routing for the docker internal network, so containers can be addressed directly by their internal IPs?
The other part (DNS discovery) is already in place in https://github.com/blinkreaction/boot2docker-vagrant (and on Linux/Ubuntu it's done by dsh).
See https://github.com/blinkreaction/boot2docker-vagrant/blob/develop/docs/networking.md#dns-resolution


client: don't hang indefinitive when the connection break

When the connection closes all client task should terminate. But following example would hang even when the broker closes the connection:
mqcli = MQTTClient(config={'auto_reconnect': False})
await mqcli.connect('mqtt://localhost/')
await mqcli.subscribe([('some/topic', QOS_2),])
message = await mqcli.deliver_message()

The problem lies in the task.set_exception(). It should terminate all pending task. But the problem is Tasks doesn't support set_exception() and this call raise itself an exception.
As the docs state:
"asyncio.Task inherits from Future all of its APIs except Future.set_result() and Future.set_exception()."
However after implementing it. message will return None without any exception.

This PR only handles the non-auto-reconnecting case.


[string] Add _deconstructUTF8 for internal usage

Add string deconstruction into contiguous UTF-8 (allocating if
needed).

@swift-ci please test

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - edf0ea316a73585e365dfeb7cdd7d422c44ff537

Build failed
Swift Test OS X Platform
Git Sha - edf0ea316a73585e365dfeb7cdd7d422c44ff537

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - 1276478aa1e07cc3f5bb493131b637cdee6719e5

Build failed
Swift Test Linux Platform
Git Sha - 1276478aa1e07cc3f5bb493131b637cdee6719e5

@swift-ci please test

Just wondering. May be I missed it, does this PR have a public API (not meant for testing) that is meant for the os_swiftoverlay?

Just wondering. May be I missed it, does this PR have a public API (not meant for testing) that is meant for the os_swiftoverlay?

Right now it has one that's flagged plain public. I'm thinking about whether we want to restrict it to SPI os_swiftoverlay if we also want it to be a compiler intrinsic.

Build failed
Swift Test OS X Platform
Git Sha - 66f7c70366187f126d26b8bf7682e4ae810df71a

Build failed
Swift Test Linux Platform
Git Sha - 66f7c70366187f126d26b8bf7682e4ae810df71a


Secrets documentation

#33 started some documentation of our secrets. I wonder if we should have something like a key manager for MapLibre. The Governing Board members could all have access to that key manager.
In native, the situation is worrying me because we don't have any documentation on how secrets are managed and we will likely not be able to make releases with under the old accounts if something breaks. https://github.com/maplibre/maplibre-gl-native/issues/312

Figma has the MapLibre logo at https://www.figma.com/file/hVmulAVC8dMwYS14ovGd3w/OLD-%2F-MapLibre-Logo-%2F-Visuals

@wipfli , @klokan , can the Figma credentials be added to Bitwarden? Until we have a good media kit ready, the SVG Export plugin in Figma would be convenient to export single Frames.

I think we do not have a figma account. This was just with the designer's or with maptiler's figma account if I remember correctly...

We have an account now with the board email it can be transferred to.

I guess you just make a copy under the Figma account, right?
Not sure how the transfer would work otherwise.
On Mon, Sep 19, 2022, 10:55 AM Birk Skyum @.***> wrote:

We have an account now with the board email it can be transferred to.
—
Reply to this email directly, view it on GitHub
https://github.com/maplibre/maplibre/issues/63#issuecomment-1250745971,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAOPFEOVGKXTRJ37NBFBWDV7ATAXANCNFSM53L7ECMA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Update build tools

This simplifies the build process, also adds a more capable Dockerfile for debugging sidecar installations.

@codeboten what say you to this? It simplifies the Makefile a lot. I would test it with an upcoming release.


Fix shared album list and bump version

Details can be found here https://github.com/AnyImageProject/AnyImageKit/issues/128

Thanks


Allow user to create template from scratch, input data, save to database

Tracks changes in the edit text fields and saves it after they are changed
Populates RecylerView with exercise that user chooses
Allows user to add sets with by clicking the "add set" button
Saves template to the database

https://user-images.githubusercontent.com/83436163/175750449-996c94b0-9317-4049-809d-70d1f11d307d.mov

Looks great!! so exciting to see the workout creation flow working in the video


editor: smaller toolbar

fixes https://github.com/lockbook/lockbook/issues/2954
fixes https://github.com/lockbook/lockbook/issues/2956

Nice. I like that it's more coherent across platforms. Also, it'll never cover the text now, so I may keep the toolbar on even if I don't use it much. Highlighting the style under the cursor is very nice for learnability.
The only concern I have is whether or not to add a bottom border. Aesthetically I like it without, but this might hurt usability. Probably needs user feedback after some use.


Implement List Keystores endpoint

First key manager endpoint implementation to list public keys + tests
part of #439

@jframe I think I addressed all your comments, thanks for reviewing!

@jframe take a look when you get the chance 🙂


Categorize by free/paid?

Please tag the tools with free or paid, people gotta know what they are getting into.

@aolko 💯!
We'll make a legend soon.

hey @aolko what do you think about such legend?

some update:
we added free tool label to the Contribution Guidelines and now I will add labels to the into part in Readme file. So people will see that it's important to add labels with their submissions.
As there are 150+ tools over the repo, it might take some time to review all tools and add labels. We'll start doing it with @valianka.
As free\paid labels exist already, I'm closing the issue.

Existing tools are not entirely categorized, please fix


Component template content: html encoded chars in attributes are not properly decoded

Version
2.5.17
Reproduction link
https://codepen.io/chessydk/pen/ZqGqbr?editors=1010#0
Steps to reproduce
Hover over the "æøå".
What is expected?
I would expect html attributes to be html-decoded, just like text-nodes, so when you hover over the text you should see "æøå"
What is actually happening?
You see "æøå"

Context: Serving html from CMS backend as JSON and creating dynamic Vue component with the server-generated html, which can have danish characters in attributes (mostly in object-data bindings like :model="{prop: æøå}" , but the problem exists for all attributes.

Duplicate of #8805


Add configuration to allow adding a placeholder when path is null

Hi,
thanks for this awesome library, we are using it a lot in our projects and it's great.
As an enhancement, could a configuration be added to allow having a particular placeholder (or using the ErrorPlaceholder) instead of a transparent background when the path to load the image is null?
thanks again

Hi @fedemkr,
Do you use it with Xamarin.Forms or native? With Forms, you could just use custom IValueConverter which will return deafult file when null string is found.

Hi, I use it with Native
Thanks again

Hi @fedemkr,
Thanks for the positive feedback.
FFImageLoading should not handle such scenarios, these are very easy to handle from the app logic if you make some helper classes.


Fix Adds DaprClient in Dapr.AspNetCore from AddSingleton to TryAddSingleton

…gleton
Signed-off-by: zhenlei520<EMAIL_ADDRESS>Description
I hope that AddDaprClient can be changed to TryAddSingleton. Although the AddDaprClient method only adds DaprClient to the current class library once, if other class libraries also add DaprClient and then use Dapr.AspNetCore, DaprClient will be added again.

@zhenlei520 - Thanks for opening this! Though I think this is already solved in the code: https://github.com/dapr/dotnet-sdk/pull/867/files#diff-6b7b1ed59d457676840fddee293b56b4f8354d2b5bbad4df9c8d8fb11103ff15R41

@zhenlei520 - Thanks for opening this! Though I think this is already solved in the code: https://github.com/dapr/dotnet-sdk/pull/867/files#diff-6b7b1ed59d457676840fddee293b56b4f8354d2b5bbad4df9c8d8fb11103ff15R41

I am very happy to receive your reply, but I think there is a problem here. If you directly use the methods in the Dapr.AspNetCore package, both AddDaprClient and AddDapr support multiple additions, and DaprClient will not be added multiple times.  but DaprClient is the core capability of Dapr.Client. If I only need to use service calls, then I can do Add DaprClient myself, but if I do this, once I use AddDapr again later, it will cause DaprClient to be called add multiple times

https://github.com/dapr/dotnet-sdk/blob/452ccad335498361b229795229398e26a42d9802/src/Dapr.AspNetCore/DaprServiceCollectionExtensions.cs#L47
I think this should be replaced by TryAddSingleton, or the AddDaprClient method should be provided by Dapr.Client instead of Dapr.AspNetCore

@zhenlei520 - Is the use case you're talking about adding DaprClient to the services manually, without calling one of the helper methods?
Also, I don't think Dapr.Client should own this code path. The services model that we're going off of here is an AspNet behavior and thus the code belongs in the Dapr.AspNetCore package.

Also, I don't think Dapr.Client should own this code path. The services model that we're going off of here is an AspNet behavior and thus the code belongs in the Dapr.AspNetCore package.

We left this out of Dapr.Client to avoid extra dependencies. It doesn't feel worthwhile to add a dependency so that we can add a method that includes one line of code.

Also, I don't think Dapr.Client should own this code path. The services model that we're going off of here is an AspNet behavior and thus the code belongs in the Dapr.AspNetCore package.

We left this out of Dapr.Client to avoid extra dependencies. It doesn't feel worthwhile to add a dependency so that we can add a method that includes one line of code.

I now have a package only for sending messages. In fact, I only need to handle the injection of DaprClient by myself, and then pass it through DaprClient's Pub. I don't need to rely on Dapr.AspNetCore, but if I do , if the user calls the AddDapr method after using my package, it will cause the DaprClient to register multiple times

Any news on this?


refusing from /terminalsocket

Since the last few versions, it is possible to call %CSP.WebSocket from %CSP.REST.
As an example you can look at %API.Atelier.v1 method RunDebugger It redirects access to %Atelier.v1.XDebugAgent which in fact is WebSocket.

Hello Dima! Thanks!
But at least we need to leave /terminalsocket for the previous versions, right?

Depends on how much old version you would like to support. I suggest, that this way available at least since 2016.2 where API Atelier was appereared.

As far as I remember, WebTerminal is compatible with 2014.1+. So I think it would be a downgrade which isn't worth a change.

Your analytics collects information about Caché version where installed?

Unfortunately, no. Only the WebTerminal version and install status (any errors) are collected.
But anyway, if we compare the pros and cons of this change, there are no pros in implementing it (in usability), whereas there are cons of cutting two release versions.

So should we close it?

As you wish, I would implement both ways, at least would added socket endpoint to Router.
Least supported version is 2016.1
Any versions less, quite old already, when you are going to drop those versions?
You still use %ZEN.Auxiliary.jsonProvider, if you drop support for such an old version, you can make it work faster with native JSON support.

For sure, but I know which pain can it cause if anyone still uses v2015 for
example. During my last training I saw people having Healthshare 2015.
Nikita
On Thu, 12 Dec 2019 at 10:13, Dmitry Maslennikov<EMAIL_ADDRESS>wrote:

As you wish, I would implement both ways, at least would added socket
endpoint to Router.
Least supported version is 2016.1
https://cedocs.intersystems.com/ens20161/csp/documatic/%25CSP.Documatic.cls?PAGE=CLASS&LIBRARY=%25SYS&CLASSNAME=%25CSP.WebSocket
Any versions less, quite old already, when you are going to drop those
versions?
You still use %ZEN.Auxiliary.jsonProvider, if you drop support for such
an old version, you can make it work faster with native JSON support.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/intersystems-community/webterminal/issues/131?email_source=notifications&email_token=ABGCCSFZTXOK7C2Q5TIIN73QYHXAZA5CNFSM4JZPU6E2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGV2QAQ#issuecomment-564897794,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABGCCSAHEHWEVP4S4NZ3AUDQYHXAZANCNFSM4JZPU6EQ
.

@daimor
Yup, it turned out that WebTerminal collects ISC Product version indeed
So there are 17 installs out of 173 recorded for a couple of last months which use InterSystems products of version 2015 and lower.

~2,5% of users, still, something I wouldn't drop in favor of just updating things.


Fatal Error

Fatal error: Could not check compatibility between iRainDrop\CPS\Commands\CPSCommand::getOwningPlugin(): iRainDrop\CPS\Commands\Plugin and pocketmine\plugin\PluginOwned::getOwningPlugin(): pocketmine\plugin\Plugin, because class iRainDrop\CPS\Commands\Plugin is not available in phar:///home/container/plugins/CPS.phar/src/iRainDrop/CPS/Commands/CPSCommand.php on line 18
I'm using 2.0 and the latest pmmp versions

It has been fixed in the latest release


Opam installation updates

This fixes #345
When checking to see whether the ppx avoiding in #342 would fix the failing opam install from #345, I discovered that the #342's refactoring actually broke the 4.13 support just added in #340...
This PR thus

writes out the Seq.equal definition contributed in #340
updates the opam install workflow to run on all changes

The install workflows are quite fast, so I favor protecting against such breakage again now that we actually support 4.13 and 4.14.

CI summary:

The 32 bits test are failing as noted in #344
A macOS 5.1 STM int64 ref test parallel asymmetric failed to trigger
Cygwin tests are still ongoing (2 of them completed part 1 successfully at this point)

In the interest of restoring 4.13 support I'll therefore merge the PR.


Error while passing middleware to eel

Hi, I was just trying to pass a beaker middleware to eel (related to #199 ) but got the following error:

Traceback (most recent call last):
File "//run.py", line 52, in start_app
eel.start(start_html_page, mode='chrome', port=0, app=app)
File "//venv/lib/python3.6/site-packages/eel/init.py", line 161, in start
run_lambda()
File "//venv/lib/python3.6/site-packages/eel/init.py", line 151, in run_lambda
app.route(path=route_path, callback=route_func, **route_kwargs)
AttributeError: 'SessionMiddleware' object has no attribute 'route'

Code is same as in the linked issue

This should be fixed in the latest v0.12.2 release @RahulBadenkal