Authorization with owner policies via groups does not work with REST auth enabled

Describe the bug
We are currently running into an issue with policies for dataset owners and METADATA_SERVICE_AUTH enabled where the indirect membership via groups seems not to be resolved correctly and therefore the user is not able to view the dataset details.
The setup looks like the following:

there is a dataset A
dataset A does not have any usage or profiling information
there is a policy P which allows owners of datasets to view all asset details
group X is technical owner of dataset A
user T is member of group X
we have enabled METADATA_SERVICE_AUTH in our deployment

Now if the user T wants to access the entity he gets a 500 in the UI and gets no details as he has no permissions to view the asset.
In the GMS backend we get following error messages (just an extract of the most relevant lines):
ERROR c.l.m.filter.RestliLoggingFilter:38 - Rest.li error:
com.linkedin.restli.server.RestLiServiceException: User is unauthorized to query usage.
	at com.linkedin.metadata.resources.usage.UsageStats.queryRange(UsageStats.java:382)
[...]
ERROR c.l.common.callback.CallbackAdapter:90 - Failed to convert callback error, original exception follows:
com.linkedin.r2.message.rest.RestException: Received error 401 from server for URI http://localhost:8080/usageStats
[...]
ERROR c.l.d.g.r.d.DatasetStatsSummaryResolver:75 - Failed to load Usage Stats summary for resource urn:li:dataset:(urn:li:dataPlatform:kafka,A,PROD)
com.linkedin.r2.RemoteInvocationException: com.linkedin.data.template.RequiredFieldNotPresentException: Field "value" is required but it is not present
[...]
ERROR c.l.d.g.e.DataHubDataFetcherExceptionHandler:21 - Failed to execute DataFetcher
java.util.concurrent.CompletionException: java.lang.RuntimeException: Failed to load Usage Stats for resource urn:li:dataset:(urn:li:dataPlatform:kafka,A,PROD)

Things I tried:

removing the privileges to view profile and view usage but it does not make a difference
adding the user T explicitly as owner to the dataset A does not work
Adding the user T explicitly to the policy works
after disabling METADATA_SERVICE_AUTH  it works

To Reproduce
Steps to reproduce the behavior:
see description of setup above
Expected behavior
I would expect that the authorization module would resolve the group permissions of the actor and allow the user to access the dataset details.
Or if he does not have the privileges to see usage stats they are not fetched from the backend and greyed out.
Versions

DataHub v<IP_ADDRESS>
Helm chart 0.2.182

@RyanHolstien yes, fetched the policy via graphql and looks like this. Does this help?
{
  "urn": "urn:li:dataHubPolicy:owner-policy",
  "type": "METADATA",
  "name": "Owner Policy",
  "state": "ACTIVE",
  "privileges": [
    "VIEW_ENTITY_PAGE",
    "EDIT_ENTITY_TAGS",
    "EDIT_ENTITY_GLOSSARY_TERMS",
    "EDIT_ENTITY_OWNERS",
    "EDIT_ENTITY_DOCS",
    "EDIT_ENTITY_DOC_LINKS",
    "EDIT_ENTITY_STATUS",
    "EDIT_DOMAINS_PRIVILEGE",
    "EDIT_DEPRECATION_PRIVILEGE",
    "EDIT_ENTITY_ASSERTIONS",
    "EDIT_ENTITY",
    "EDIT_DATASET_COL_TAGS",
    "EDIT_DATASET_COL_GLOSSARY_TERMS",
    "EDIT_DATASET_COL_DESCRIPTION",
    "EDIT_TAG_COLOR",
    "EDIT_GROUP_MEMBERS",
    "EDIT_USER_PROFILE",
    "EDIT_CONTACT_INFO",
    "MANAGE_ENTITY_TAGS_PRIVILEGE",
    "MANAGE_ENTITY_GLOSSARY_TERMS_PRIVILEGE",
    "MANAGE_DATASET_COL_GLOSSARY_TERMS_PRIVILEGE",
    "MANAGE_DATASET_COL_TAGS_PRIVILEGE",
    "EDIT_ENTITY_ASSERTIONS_PRIVILEGE",
    "VIEW_DATASET_USAGE",
    "VIEW_DATASET_PROFILE"
  ],
  "description": "Owners have privileges ONLY for assets they own.",
  "resources": {
    "type": null,
    "resources": null,
    "allResources": false,
    "filter": {
      "criteria": [],
      "__typename": "PolicyMatchFilter"
    }
  },
  "actors": {
    "users": [],
    "groups": [],
    "resourceOwners": true,
    "allUsers": false,
    "allGroups": false
  },
  "editable": true
}

Yes! Thank you, will reproduce and try to fix the issue.

Hi,
I think I also encountered the same issue which blocked our work, I have already raised a PR with the fix


Update the generated go code for pkg/api/v1alpha1

PR #3181 updated the CloudStack types but did not trigger make generate
to update the zz_generated.deepcopy.go. This causes a few unit tests
to fail when others try to regenerate the file. This PR updates the generated
code and fixes the broken testcases for CloudStackMachineConfig.

/assign wongni

/ok-to-test

/approve

/retest-required


Add prerun method

Currently, the strategy run() method is executed at every trade. When it come to using the same model on multiple trades, it's a burden and a method load/train a model before the first run would be required.

Actually, there is no need since it can be done in the constructor of the strategy.


rsync crashes – malloc error: "pointer being freed was not allocated"

Same issue as folks over at MacPorts experience with rsync's -X option:
https://trac.macports.org/ticket/50350
Version 3.1.2 is affected, previous version 3.1.1 (rev 6bb7624c0d1b361a71b316be6735c690a8a166df) not.
The problem appears when adding the 'X' option to copy extended attributes:
# Successful case
# -------------------
$ rsync -aP remoteserver:/sample-data .
receiving incremental file list
sample-data/
sample-data/foo.txt
              7 100%    6.84kB/s    0:00:00 (xfr#1, to-chk=2/4)
sample-data/hello.txt
             14 100%   13.67kB/s    0:00:00 (xfr#2, to-chk=1/4)
sample-data/mydate.txt
             29 100%   28.32kB/s    0:00:00 (xfr#3, to-chk=0/4)

# Now the error
# -------------------
$ rsync -aXP remoteserver:/sample-data .
receiving incremental file list
sample-data/
sample-data/foo.txt
              7 100%    6.84kB/s    0:00:00 (xfr#1, to-chk=2/4)
sample-data/hello.txt
             14 100%   13.67kB/s    0:00:00 (xfr#2, to-chk=1/4)
rsync(2689,0x7fff7bbac300) malloc: *** error for object 0x7fa300600070: pointer being freed was not allocated
*** set a breakpoint in malloc_error_break to debug
rsync: connection unexpectedly closed (157 bytes received so far) [generator]
rsync error: error in rsync protocol data stream (code 12) at io.c(226) [generator=3.1.2]

Do you know if this has been reported upstream? If not, could you do so? Thanks!

The X option is pretty common so you're probably going to need to get a lot more specific for upstream to diagnose this.

I'm encountering issues as well with 3.1.2, on of my rsync script that worked for over a year has stopped working, encountering sudden inflate returned -3 (1809 bytes) errors, workaround for now is to disable compression.

@mgcrea Are both ends 3.1.2?

@ilovezfs, nop server is 3.1.0 (up-to-date Ubuntu 14.04.4)
$ rsync --version
rsync  version 3.1.0  protocol version 31

Reverting to rsync 3.1.1 on my mac does fix the issue.

I'd be very curious whether the issue is reproducible if both sides are at 3.1.2, if you have the patience to test that configuration.

@ilovezfs what would be the easiest way to grab 3.1.2 on Ubuntu (ideally non-destructive)?

@mgcrea Linuxbrew, of course. (What did you think I would say!) :)

Closing this upstream issue.


updated parquet convert interface to support chunking of rowgroups with CLR types

Fixes
Issue #372
Description
Added an append flag overload to ParquetConvert.Serialise to enable streaming

[ ] I have included unit tests validating this fix.
[ ] I have updated markdown documentation where required.
[ ] I understand that successful approval of my pull request requires reproducible tests as per Contribution Guideline.

This should not go into master as there is no test coverage.

I've reverted the last commit from master, the test doesn't validate append (see inline comments). It would be easier if it goes through review first ;)
      [Fact]
      public void Serialise_deserialise_with_append()
      {
         DateTime now = DateTime.Now;

         IEnumerable<SimpleStructure> structures = Enumerable
            .Range(0, 10)
            .Select(i => new SimpleStructure
            {
               Id = i,
               NullableId = (i % 2 == 0) ? new int?() : new int?(i),
               Name = $"row {i}",
               Date = now.AddDays(i).RoundToSecond().ToUniversalTime()
            });

         using (var ms = new MemoryStream())
         {
            Schema schema = ParquetConvert.Serialize(structures, ms, compressionMethod: CompressionMethod.Snappy, rowGroupSize: 5, append: false);

            //might be better to use different data set for append
            schema = ParquetConvert.Serialize(structures, ms, compressionMethod: CompressionMethod.Snappy, rowGroupSize: 5, append: true);

            ms.Position = 0;

            SimpleStructure[] structures2 = ParquetConvert.Deserialize<SimpleStructure>(ms); //contains 20 structures

            SimpleStructure[] structuresArray = structures.ToArray();
            for (int i = 0; i < 10; i++)  //compares first 10?
            {
               Assert.Equal(structuresArray[i].Id, structures2[i].Id);
               Assert.Equal(structuresArray[i].NullableId, structures2[i].NullableId);
               Assert.Equal(structuresArray[i].Name, structures2[i].Name);
               Assert.Equal(structuresArray[i].Date, structures2[i].Date);
            }

         }


Fix flakey tests: regenerate y4m file without artifacts

What

Update test video file (see why). Increase from 133KB to 1.1MB (to avoid artifacts)
In video ready also checked !paused to ensure video has loaded

Why
Current file has some artifacts at the edge:

This appears to be causing following flakiness:

Bigger screenshot:

This hasn't impacted us so far as it seems to fall outside playwrights threshold during our current tests, but adding landscape tests this does exceed the threshold
New File that doesn't appear to have any artifacts around the edge

How Tested
Ran a bunch of times and haven't see the artifacts

Looks to have introduced its own flakiness:

Will reopen when fixed

Flaky start screen still happening

New y4m file


feat: undo

This is first part of undo to land for repliear-- next part would be to add a toast notification for an undo or redo operation. We would like to also implement scan so we can undo a creation of an Issue.


invalid constant type Error

Hi,
Your product has performed very well among the available options, but there are some problems when using it, which I request you to check and fix. like the:
INFO:  Decompiling class com/adventnet/client/components/table/pdf/PropertySheetRenderer WARN:        Method getDollarReqParams (Ljava/lang/String;Lcom/adventnet/client/view/web/ViewContext;)Ljava/lang/String; in class com/adventnet/client/components/table/template/FillTable couldn't be written. java.lang.RuntimeException: invalid constant type: Ljava/lang/Object;         at org.jetbrains.java.decompiler.modules.decompiler.exps.ConstExprent.toJava(ConstExprent.java:347)         at org.jetbrains.java.decompiler.modules.decompiler.exps.AssignmentExprent.toJava(AssignmentExprent.java:146)         at org.jetbrains.java.decompiler.modules.decompiler.ExprProcessor.listToJava(ExprProcessor.java:858)         at org.jetbrains.java.decompiler.modules.decompiler.stats.BasicBlockStatement.toJava(BasicBlockStatement.java:65)         at org.jetbrains.java.decompiler.modules.decompiler.stats.IfStatement.toJava(IfStatement.java:199)         at org.jetbrains.java.decompiler.modules.decompiler.ExprProcessor.jmpWrapper(ExprProcessor.java:796)         at org.jetbrains.java.decompiler.modules.decompiler.stats.SequenceStatement.toJava(SequenceStatement.java:111)         at org.jetbrains.java.decompiler.modules.decompiler.ExprProcessor.jmpWrapper(ExprProcessor.java:796)         at org.jetbrains.java.decompiler.modules.decompiler.stats.CatchStatement.toJava(CatchStatement.java:178)         at org.jetbrains.java.decompiler.modules.decompiler.stats.RootStatement.toJava(RootStatement.java:37)         at org.jetbrains.java.decompiler.main.ClassWriter.methodToJava(ClassWriter.java:1084)         at org.jetbrains.java.decompiler.main.ClassWriter.classToJava(ClassWriter.java:334)         at org.jetbrains.java.decompiler.main.ClassesProcessor.writeClass(ClassesProcessor.java:480)         at org.jetbrains.java.decompiler.main.Fernflower.getClassContent(Fernflower.java:156)         at org.jetbrains.java.decompiler.struct.ContextUnit.lambda$save$0(ContextUnit.java:179)         at java.util.concurrent.Executors$RunnableAdapter.call(Unknown Source)         at java.util.concurrent.FutureTask.run(Unknown Source)         at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)         at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)         at java.lang.Thread.run(Unknown Source)

DefaultTransformer.zip
and in this file.

Hey, I deleted the attached class files as I wasn't sure if they were copyright materials or not. However, the described issue is known and will be fixed soon.

Thank you
I have tried many products but your product has better output than all of them but I need to fix this problem to move forward with my project. How long do you think it will take to fix this problem?

I'm not quite sure, but I will make sure to prioritize it. The next release (1.9.0) will come out over the next few months, but I'll make sure to update you here when it's fixed so that you can build the current dev tree from source if you wish.

I am not very professional in Java language. I am planning to add a local calendar to an application to be used in our company, so I need access to the application code.
Is it possible to guide me how to use decompiler source code?

You would just need to clone this repository and do ./gradlew.bat build. The resulting jar will be in the build/libs folder.

This issue should be resolved now, you can try building from the branch as I stated above and using that jar. If that doesn't work, please let me know.

Hi,
I checked the new version of the QuiltFlower 1.9 and this problem is still there but this problem is not present in the new version and previous versions of Fernflower.


Slurm support

Initial support for Slurm scheduler. Adds slurm scripts and templates to bootstrap and the required plugins for sqswatcher and nodewatcher.  Slurm support for dynamic nodes isn't great and so we use a dummy node in the slurm.conf to allow submission of jobs larger than the current cluster size. Also the process of adding nodes relies on the syntax of the slurm.conf - not a great solution and I will think about making it more elegant.
Note
When adding updated boostrap and cfncluster-node packages to the AMI you also need to update the slurm binaries (/opt/slurm). The version in the current AMI is 14.11.2 and there seems to be an issue with the "squeue" command causing a segmentation fault, updating to 14.11.9 resolves this. The cluster will come up with the older binaries but we use the squeue command as part of nodewatcher, this will fail.

Hi Mike, thank you very much for your commit. We recently made the switch to Chef for the installation and bootstrapping of CfnCluster. I will review your PR and see what we can pull in.

Hi Dougal, ok no problem - happy to wait for the first release of the chef version and then I can look at integrating Slurm with that.

I have pulled the changes and will also add some of the code to the https://github.com/awslabs/cfncluster-cookbook project.


Make API more idiomatic

Basically: Look at the methods for the fuse-mt::FilesystemMT trait. Note how those methods have actual return values, rather than having the C-like "do something with some variable passed in as an argument and then return void" API that this crate currently has.
I think changing this crate's API to be more like fuse-mt's would be an improvement.

The reason why the Filesystem trait in rust-fuse uses a Reply argument instead of returning is to allow for asynchronous response. If I'm understanding this feature request correctly, this would not be an improvement, but a setback :/.

Maybe then return a Future<Reply*>? See #80

error/ok take self by value, so you can't call them multiple times.

Hum, right XD

If rust-fuse moves to tokio/mio, then maybe it could make sense to return a Future<Reply>. But in the interim, I don't think it makes too much sense as it only imposes a specific kind of abstraction on the developer.
I have some WIP stuff on integrating tokio with rust-fuse and I think it's possible to do this without introducing breaking changes (by making a new FilesystemFutures trait and a wrapper struct). I still need to figure out how to allow the methods access the tokio event loop (need a Handle or something somewhere).

I'd also really like to see a more idiomatic API. Back when I wrote the Filesystem trait, it wasn't really clear how cases like these should be handled idiomatically. I like the idea to use Future<Reply> as return types, but there are some problems with that:

In general, I'd like to prevent dependencies that would enforce developers to use a specific I/O implementation, like tokio or mio. I think future-rs would be ok (mainly traits), but I wouldn't like to depend on tokio (don't get me wrong, I like tokio, but I don't want to force others to use it).

The current trait was designed to enforce the user to always reply (to prevent leaking unreplied requests. If the reply object is dropped, a default reply is sent). Having a sendable reply object allows implementations to reply asynchronously. This could be done with return types as well (and would be more idiomatic). Since  Future<T> is implemented by T, it would nicely accept immediate values being returned for synchronous operations and optionally some future value for asynchronous operations. However, the reply does need to know the request id when it's sent. Currently, the reply gets the request id when it's created by the request dispatcher. If the user creates the reply object himself, how would we know the proper request id when actually sending the reply?

I think a Filesystem trait using future return values would work, but I'm unsure how we can keep the request/dispatch loop encapsuled from the user without depending on a concrete async I/O implementation like tokio.

I've been thinking about how to make the API more idiomatic and have come up with some ideas.
We need to keep ensured that (a) the actual implementation can only reply once to each request and (b) an error reply is sent if the implementation forgets to reply at all.
A fs operation method currently looks like this:
fn lookup(&mut self, _req: &Request, _parent: u64, _name: &OsStr, reply: ReplyEntry);

This ensures (a) by passing the reply object which has self-consuming ok/error methods and (b) because the reply object sends an error if dropped without calling ok or error.
More idiomatic would be to return a result:
fn lookup(&mut self, _req: Request, _parent: u64, _name: &OsStr) -> Result<ReplyEntry, Error>;

Both (a) and (b) would be covered because the implementation must return exactly one result. Error would be a rust-fuse specific wrapper type for errno (similar to std::io::error::Repr::Os) probably with some conversion methods for convenience.
Since ReplyEntry needs request-specific information, it either needs to be created by calling a method on _req or by implementing From<Request> so that _req can be turned into a reply.
This however might allow people to mistakenly create a wrong reply type and mess things up if they manage to interchange replies between different requests to satisfy the return type.
ReplyEntry might better be named Entry or turned into generic type Reply<Entry> (I'd prefer Entry for less clutter)
To make it asynchronous, we can use futures instead of results. However since Future is a trait, we would require the "impl traits" feature, which is only available in nightly rustc yet.
fn lookup(&mut self, _req: Request, _parent: u64, _name: &OsStr) -> impl Future<Item=ReplyEntry, Error=Error>;

Without impl traits, we need to box the returned type. This works with stable rustc, but would require an allocation for every returned value (!)
fn lookup(&mut self, _req: Request, _parent: u64, _name: &OsStr) -> Box<Future<Item=ReplyEntry, Error=Error>>;

We could either implement asynchronous operations in a different trait (which would end up in duplicating the large Filesystem trait using slightly different method signatures. Or move to use futures exclusively (which would work fine even for synchronous implementations since every Result can be turned into a future that resolves immediately)
With futures, it's getting a little more complicated.. The Filesystem trait might be easy to switch to futures (with impl traits being the only blocker). To fully take advantage of asynchronicity, the session's run method needs to return a future that the developer can put into a reactor to run the session loop and dispatch filesystem operations.
Moreover, the kernel communication (channel) needs to be future-based. This is where it requires more than just future-rs. As far as I can see, this would require a dependency on tokio-core/tokio-io/mio for async RawFd communication which I wouldn't be very happy about.
Thoughts/ideas appreciated

I still don't quite get why using return (without any explict async io crates [e.g. tokio] in the API) makes it impossible to perform async response...
Does the function that calls these functions waits until these functions to return?

If yes, how does using parameter instead of return value makes it async?
If not, why can't it spawn asynchronously this function and any following code?

Any pointing to relevant code is appreciated.

The point is that if it's passed as a parameter, the driver can then push the request/response to a thread pool or an event queue. rust-fuse would still block on it returning, but the function could return immediately after pushing the request on a work-stealing task queue.

The reply parameter doesn't make it async. The key thing is, that it doesn't prevent you from making it async. Passing the reply parameter gives the implementor freedom to either handle it asynchronously or synchronously.

Synchronously: do processing, use reply parameter to send the result, done
Asynchronously: spawn thread/task passing the reply parameter, return immediately. Within the thread/task do the processing and send the result later.

That's also the reason why the function gets ownership of the reply parameter (it's moved into the function), it can pass it on to other threads. Providing the reply as return values would effectively prevent any async processing since the function would need to provide the reply value to finish. The internal kernel communication and dispatching is single threaded, so yes, the next operation can't be dispatched until the function returns.
This design decision is quite old. I think there was no tokio or anything async back that time. There's some discussion about it in #1 :)
Nowadays, with Rust getting native futures and an ecosystem of async stuff is building up, I think this API should be redesigned and use futures (which provide the same functionality in a more obvious way). I'm working on redesigning the dispatch so that we get a fully async and more idiomatic interface (currently in the modernize branch. Hopefully I'll find some time in the next days to bring it into a working state and merge it)
However, using async in Rust still has a few drawbacks, e.g. async fns can't be used in traits yet and it still needs a nightly Rust to be compiled, so rust-fuse 0.4.0 will probably stay alpha/beta until futures in Rust become available in stable.

(This is just for the period before async is fully integrated into rust. I agree using more native and explicit support is always better.)
So, I mean, I see waiting for functions to return is a sequential operation. But the caller can always spawn a thread (or whatever async methods) and call this function in that thread (as well as performing any further work there, after the function call returns).
I didn't dig into the detail of rust-fuse's implementation, but for whatever way you make it possible to allow the user to decide either to go async or not requires an async way of handing the value change (or consumption) of reply, don't you? So why not also put calling the function there?

But the caller can always spawn a thread ...

The caller of the Filesystem methods is the session loop of rust-fuse that dispatches requests from the kernel driver to the user code. Spawning a thread there would be possible, but that'd part of the rust-fuse code and the idea is to give downstream implementations the freedom to choose whatever concurrency technology and strategy they like, therefore it needs to be done in the function implementation.

... but for whatever way you make it possible to allow the user to decide ... requires an async way of handing the value change (or consumption) of reply, don't you? So why not also put calling the function there?

Not exactly. The reply type is crafted to be able to send the reply back to the kernel driver on its own. It doesn't get processed by an outer loop.

I have some thoughts on an approach that could help make everyone happy.
Firstly, keep the current interface. It's flexible for the reasons you stated, and works. The Filesystem trait isn't quite low level in FUSE terms, but it certainly feels lower than FilesystemMT. You could call it a mid level API if you want!
Second, let folks who want a fully synchronous API use fuseMT. You could mention it in the docs, or even merge it into the crate.
Finally, handle async io by not handling it, and instead offering ways to serialize / deserialize the messages from a byte buffer. This "network protocols sans i/o" philosophy is the most flexible way to do this, in my opinion. And if you want to get fancy for async's sake, you could have it work with N byte buffers for vectored io.
This is the design I'm going for with nine, which occupies a similar niche as rust-fuse. (I'd love to help with this, by the way. Especially since 9pfuse isn't packaged in homebrew anymore, so I want to make a replacement using rust-fuse!)

I did some cleaning and improvements on the code in the modernize branch, which now also has a new version of the Filesystem trait that's using return values for replying: filesystem.rs. Comments / feedback welcome (MR is #126).
It can't be used yet since it's still missing the request dispatch mechanism, but I'm working to get that done as well.
I'm in favor of making the API async (which can easily be done now by making every method async), however since Rust doesn't support async methods in traits yet, that'd mean to either use the async-trait crate or use boxed future types as return values (very inconvenient).
Unfortunately, it would be hard to have both versions, a sync one and an async one at the same time. Making the trait async only if a certain feature is turned on seems a bit cumbersome. Easiest would be to change the return type to a boxed future of a reply, but that'd incur the cost of boxing the return value for each method call. Maintaining two filesystems traits, one with sync methods and one with async methods, seems too inconvenient as well.


Don't use block height as a primary key

It's likely causing failures during reorgs or other transient failures that result in a rebroadcast. Found by @ianamunoz and @gmale.
https://github.com/zcash-hackworks/lightwalletd/blob/master/storage/sqlite3.go#L30

This basically boils down to "implement re-orgs" which is a feature we knowingly punted on, as a trade-off.

Related to and possibly a duplicate of #31

Relates to https://github.com/zcash/zcash-android-wallet-sdk/issues/23

No longer relevant due to updates to mentioned topics


Connection error with FRITZ!WLAN Repeater 1750E

Describe the bug
I updated the plugin for Homebridge to the newest Version. Now running v6.0.5
After the update vom v5 to v6 I got connection error to my FritzRepeater and I didn't get the status information
I tried different changes in the config

change the password to not include special characters
"ssl" true / flase
"oldFW" true / false
"tr064" true / false
include a empty "username" as a Repeater has no username

last config was
{
    "active": true,
    "name": "Repeater",
    "host": "<IP_ADDRESS>",
    "username": "",
    "password": "xxx",
    "ssl": true,
    "tr064": true,
    "igd": false,
    "manufacturer": "AVM",
    "model": "FRITZ!WLAN Repeater 1750E",
    "oldFW": false,
    "wifiUnits": 2,
    "connection": "repeater",
    "options": {
        "wifi_2ghz": "switch",
        "wifi_5ghz": "switch",
        "wifi_guest": "switch"
    }
}

Error message
[6/23/2021, 8:13:46 PM] [FritzPlatform] Repeater Wifi guest (wifi_guest): An error occured during getting state!
[6/23/2021, 8:13:46 PM] [FritzPlatform] Error: Unauthorized
    at Object.exports.request (/homebridge/node_modules/homebridge-fritz-platform/node_modules/@seydx/fritzbox/lib/request.js:78:21)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
    at exports.requestXml (/homebridge/node_modules/homebridge-fritz-platform/node_modules/@seydx/fritzbox/lib/request.js:213:20)
    at Service.execAction (/homebridge/node_modules/homebridge-fritz-platform/node_modules/@seydx/fritzbox/lib/service.js:133:20)
    at Handler.get (/homebridge/node_modules/homebridge-fritz-platform/src/accessories/router/router.handler.js:241:28)
    at Handler.poll (/homebridge/node_modules/homebridge-fritz-platform/src/accessories/extras/extras.handler.js:774:13) {
  title: 'Invalid Response',
  code: 401,
  url: 'https://false:passwordxxx@<IP_ADDRESS>:49443/upnp/control/wlanconfig3'
}

Try adding a username ("admin") to the config

thanks for the quick response, worked with "admin" as username👍🏼


Blank service cards and numbers

User Story
An error in the timewizard caused service cards in the sidebar to show up blank and the numbers to be incorrect/blank.
Repro steps
Take any service, publish with one status and then publish with a different status to recreate the bug. An error will show in the browser's debug console.

t shirt size medium

this was resolved with https://github.com/chef/automate/pull/2185, correct?


CLI does not work on ports other than 1234

With 0.3, CLI commands do not recognize the server on ports other than 1234, and will return an error; lms server start will also force start the server on port 1234 if it was already running on another port.
This is because the program attempts to read from a location ~/.cache/lm-studio/.internal/http-server-last-status.json that no longer exists in 0.3.x. The fix should read from ~/.cache/lm-studio/.internal/http-server-config.json.

Fixed by #76


Plugin broken again, after the update

I fixed this 26 hours ago, if it’s still not working for you, please upload your client.log


Logging updated to use os_log

Task/Issue URL: https://app.asana.com/0/276630244458377/733110334111734/f
Description: All logs are now processed with unified logging system. More info: https://developer.apple.com/documentation/os/logging
Steps to test this PR:

Make sure logs are printed to Xcode console and don't contain  string

Internal references:
Software Engineering Expectations
Technical Design Template

@bwaresiak Thanks for careful review!
To see debug messages in Console.app go to Action -> Include Debug Messages.
There are known issues when using Simulator and debug messages - https://forums.developer.apple.com/thread/82736. We can see them in Xcode console so I don't think it's a blocker. Hope Apple will solve it soon

@tomasstrba Yes, I had this option enabled, but it was apparently problem with the simulator. :)


update tab-context.css

Greetings. For some time now the context menu for tabs has an option called “Close Duplicate Tabs”, but it is not included in simpleMenuWizard/tab-context.css

Just add #context_closeDuplicateTabs, to the list in the CSS file.


The big sudo challenge: how can we fix the need to run PiGpio provider applications as sudo?

As the native PiGpio integration requires sudo-privileges, a lot of users have problems or don't know if this requirement although it is mentioned in the documentation.
Relateded issues

https://github.com/Pi4J/pi4j-v2/issues/211
https://github.com/Pi4J/pi4j-v2/issues/117

This is not just a documentation issue. This is a real deal-breaker and makes using Pi4J utterly unusable in any program that requires network listeners or similar permission sensitive services. Node and python both have ways of doing GPIO and SPI without sudo privileges and appear to do so without yet another daemon and yet another slow socket interface. Are there any supported alternatives to pigpio please?

The sudo requirement can be mitigated for select functions including GPIO, I2C, SPI, SERIAL with the implementation of Linux file system I/O providers opposed to the PiGpio implementation.  This is on my list of things TODO.
Accessing GPIO via the Linux (user) file system drivers does add some latency but likely not as much as a socket daemon.

Hi @savageautomate, would be great if you could get me started on this LinuxFS or if we could work together on it...
See https://github.com/Pi4J/pi4j-v2/issues/117

An interesting (and faster) alternative is to consider using /dev/gpiomem. It's there for exactly this purpose.

@hackerjimbo sounds promising :-)
you have some more info or example implementations of this approach?

The GPIO library used by the v1 version used it.

Essentially it's like /dev/mem but limited to the GPIO area.

So why did v1 switch to things like Wiring and now pigpio, which themselves both seem to have very limited / sporadic support?

Because the original one wasn't maintained.

indeed WiringPi got deprecated in 2019. That's why Pi4J V2 switched to PiGpio. These native libraries are the "gateway" between Java and the GPIOs.

Problem seems to be that these gateways all to often become unhinged. pigpio not only requires root privileges but also seems to block other programs from accessing unrelated gpio pins and does not release that control even when you terminate the daemon.

@sgparry
I have been working on the LinuxFS implementation of a GPIO provider based on the older Linux Filesystem GPIO driver via the /sys/class/gpio file.  You user account must have the gpio group assigned to it to use this without sudo.
You can see this changes and build from this branch if you would like to test them out.
https://github.com/Pi4J/pi4j-v2/commits/issues/%23117
Here are some basic examples I used to test with for Linux FS Digital Input, Digital Output, and PWM:
https://github.com/savageautomate/pi4j-remote-test/tree/main/src/main/java/com/pi4j/remote
Note, I did not get PWM working on my system without sudo;  however, this may just be a configuration or overlay issue.  I need to update to the latest OS version and try again.

@hackerjimbo

An interesting (and faster) alternative is to consider using /dev/gpiomem. It's there for exactly this purpose.

Yes, that would be something we could look into.  However, to start with I'm trying to target the generic Linux kernel defined APIs so that the implementation may potentially be useful for for other SoCs in the future.
If we implement any RaspberryPi specific code, we would do that in the RaspberryPi I/O providers (which are currently just stubs and not implemented.).
Currently, I'm prototyping the native code to allow Pi4J to use the newer dev/gpiochip* character Linux device driver opposed to the older proc FS (/sys/class/gpio) driver which apparently is getting deprecated.  (More info: https://elixir.bootlin.com/linux/v5.15/source/include/uapi/linux/gpio.h).  If I target the 5.15 kernel in the latest Raspbian OS release we may be able to include things like PULL UP/DOWN bias which was not previously supported via the Linux filesystem drivers.  I still need to test this out and see if it works but I have been able to get the basic input and output working in native code.  This still may not be as efficient as the  /dev/gpiomem approach but it would at least adhere to the generic Linux APIs for GPIO access.

We now have a new gpiod implementation, and this doesn't require sudo anymore. So this issue is resolved, check out the new provider: https://www.pi4j.com/documentation/providers/gpiod/

Thank you a lot for this release!
To be completely sure, does this release still require sudo for PWM access?

The pigpio provider still requires sudo
https://www.pi4j.com/documentation/providers/.
The linuxfs pwm provider operates without sudo


Time out urgent pregnancies edits

I rule and have completed some work on Case Manager that's ready for review!
Minor edits to some outstanding work by @charleshuang80 in #590 -- time out urgent pregnancies after five days
It relates to the following issue #s:

Fixes #119

Current coverage is 97.51% (diff: 40.00%)

Merging #598 into master will decrease coverage by 1.84%

@@             master       #598   diff @@
==========================================
  Files            24         24
  Lines           468        482    +14
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            465        470     +5
- Misses            3         12     +9
  Partials          0          0

Powered by Codecov. Last update ea23d1e...5817398

Made TODOs for later testing. Merging!


Property logging.file on configserver repository

I'm trying to store the logging.file property in the configserver repository but it is not taking effect.
Is there any issue on doing it?
If I store this config on classpath application.yml it works fine, but not working on remote repo.
My config looks like this:
logging.file: /var/log/spring-app/application.log

Did you try that with a snapshot of Spring Cloud (Commons - it's not a config client feature actually)?

@dsyer I did with the following dependency:
<dependency>
   <groupId>org.springframework.cloud</groupId>
    <artifactId>spring-cloud-starter-config</artifactId>
</dependency>

I just added spring-cloud-commons dependency but nothing different happened. I've Angel.SR3 spring-cloud-starter-parent.
All properties stored on configserver is working fine, just logging.file does not take effect.

Angel.SR3 is not a snapshot. Try Brixton.BUILD-SNAPSHOT if you can. I think this problem night go away there.

Just tried with Brixton.BUILD-SNAPSHOT and the behavior persists.

Fixed in master. You can work around it by adding a (valid) logging.config to your remote property source (or adding another one with that property value).


HTTP signatures?

Format: https://github.com/w3c/activitypub/issues/203#issuecomment-297553229
publicKey be a pem format. keyId would be the ID of the key above?
could be together with JWT and Kroeg could support both. need to rewrite JsonWebToken table
Interop with Mastodon?

For reference: https://github.com/tootsuite/mastodon/pull/4146


Simple: Forgetting to drop FrozenContext will cause use after free

The pointer to a frozen collector's stack is only guaranteed to be valid for the lifetime of the FrozenContext. We store it in a hashtable.
Ideally, the user would always unfreeze! through the standard API, which causes it to be removed.
However, if the user forgets to unfreeze (or runs mem::forget) the frozen context, the old pointer will remain in the set of PersistentRoots even if the underlying memory is freed (or moved).
Right now I actually even have this reflected in the API. If a FrozenContext is ever dropped without unfreezing first it will panic with a "TODO message".
The code is undergoing temporary insanity :construction:
See also c9d70d89625bfccec822f7444 for the unsafe implementation of freezing simple collectors that we currently use.
When we finally fix this issue, we need to remember to remove all the TODOs in the code (and the FrozenContext destructor that panics).

Fixed after I finished rewriting how collector states. Now the collector owns a boxed reference to the raw context, and the SimpleCollectorContexts unregister on drop. Worst case we never unregister and leak a reference to the context. This will deadlock any future collections, but it is technically safe.
I finished these changes around 7db75a10353ed0cfc27b35903117c97a96570678.
This is even less of an issue after I removed the unsound frozen roots.


Fix on pymodbus 3.7.x

Closes #17
I have not tested actual functionality, just confirmed that the client and server start without crashing.

This repo is @dhoomakethu, he wanted a separate repo so he could do the maintenance easier, so I will leave it to him.

Yes, I understand that.  I'm just requesting your review on this one to confirm I made the API changes right.  On the other changes he's the only reviewer :)

So does this mean, pymodbus no longer supports binary framer ?

https://github.com/pymodbus-dev/pymodbus/pull/2130

@alexrudd2 I have added some more changes to be in line with pymodbus3.7, Refer MR #21 . JFYI

Closing in favor of #21


Field level format, parse, and validate.

feature request
Would like guidance on implementing field level validation, parse, and format functions.

Yes, parse/format is functionality from RFF that wasn't implemented here. I'll think about how best to add it.
The field-level validation could be added easily via the "field config" prop to registerField().
Let me think on both of these... 🤔

Thanks for the update here Erik!  I'll check into the registerField() prop.

I didn't mean for you to close this. It's still a legit feature request.

Seconded. Please provide some examples.


Not extracting all translations

Hi,
I would like to use this tool to extract all the translations and compare with the current i18n file we have. Not all developers are removing them and the file becomes messy.
How difficult it will be to change the tool to retrieve such translation key formats?

{{ (isNew ? 'ADD-ASSESSMENT' : isDeleted ? 'VIEW-ASSESSMENT' : 'EDIT-ASSESSMENT' ) | translate }}

It seems that it doesn't extract translations with pipe

You could just use a variable and extract the key outside the html:

{{ someVar | translate }}

and
someVar = isNew ? marker('ADD-ASSESSMENT') : isDeleted ? marker('VIEW-ASSESSMENT') : marker('EDIT-ASSESSMENT')

On the other hand (however it's more "expressive" and thus a bit heavier (cpu wise), you can also do this:

{{ isNew ? ('ADD-ASSESSMENT' | translate) : isDeleted ? ('VIEW-ASSESSMENT' | translate) : ('EDIT-ASSESSMENT' | translate) }}

or use "ngIf" instead, like

  <ng-container *ngIf="isNew; else notNew">
    {{ 'ADD-ASSESSMENT' | translate }}
  </ng-container>
  <ng-template #notNew>
    <ng-container *ngIf="isDeleted; else notDeleted">
      {{ 'VIEW-ASSESSMENT' | translate }}
    </ng-container>
    <ng-template #notDeleted>
      {{ 'EDIT-ASSESSMENT' | translate }}
    </ng-template>
</ng-template>

Both of these variants should work properly with this package and it saves you an additional variable (and possibly additional logic) in the controller.
Which variant you choose depends on your personal preference.


Reduce message size on mobile and increase a bit UI text

I didn't intend to mix those together, but I happened to work in the same branch, and the results kind of depend on each other anyway.
Reduce message size on mobile and increase a bit UI text
Some feedback on the channel and on #1558 mentioned that the messages on mobile are too big.
I don't entirely agree as they respect current trends (for example, Telegram ships by default with a 16px size on Android) but I understand that the gap between the previous state and what is currently on master was too abrupt and threw people off, so I set 15px as a compromise.
There were discussions to make this configurable, but it's further down the road and this just tries to find a common ground right now.
Also make the rest of the app on mobile a tad bigger to match the message size and to make it more usable. The jump here is very small compared to messages so it will be more subtle.
Also removed some font-size overrides that were not needed anymore.
Make channel list links clickable on full width, simplify code

Close button and badge are now positioned using the same method. Previously one was float and the other was absolutely positioned.
Increase the close button size to match 2.4.0, as it was made smaller when moving to the native font stack.
Removed link margins on left/right of the channel links, and the 1px gap between links.
Removed hack/fix for drag-and-drop now made unnecessary

Oh goodness, more discussions about text sizes. Boy do I love that.

It's also a cleanup ;)


[DataStructure] Introduce the DataStructure component

TODO:

[x] Queue
[x] PriorityQueue
[x] Stack

Pull Request Test Coverage Report for Build 490

42 of 43 (97.67%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.06%) to 99.941%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Psl/DataStructure/PriorityQueue.php
28
29
96.55%

Totals

Change from base Build 488:
-0.06%

Covered Lines:
1692

Relevant Lines:
1693

💛  - Coveralls

ugh, i did git reset instead of git rebase 🤦🏼♂


Update change notes

Did we end up deciding if the nanovdb notes can be consolidated?

Did we end up deciding if the nanovdb notes can be consolidated?

No. I've done a first pass at updating the docs and am going to merge this PR now and re-publish them to make sure the process is all working and everything is ready to go, but I won't publish the 11.0.0 draft yet. I think it would be better if the NanoVDB notes are re-written, so can be a topic for the TSC tomorrow.


Document how multiple doc versions are kept

Describe:

how & when to edit the switcher.json file
how to edit versions in the pages.yml file
what to do for a new tag

Related to https://github.com/prjemian/pysumreg/issues/4

Related to https://github.com/BCDA-APS/apstools/issues/713


[CI binary release] Maintain a Release_Notes.md

Maintain a Release_Notes.md under the root of the repo. It stores all contents of all releases. People will modify it before starting the binary release processing. The binary release workflow will retrieve the latest note and use it to create a github release.
It may likes:
## WAMR-1.2.0

*Here is the note of the release*

----------------------------------------------
## WAMR-1.1.0

*Here is the note of the release*

----------------------------------------------
## WAMR-1.0.2

*Here is the note of the release*

RELEASE_NOTES.md

https://github.com/bytecodealliance/wasm-micro-runtime/pull/1599


AppInsights ignores updated Title tags

I am using AppInsights v2.0.1 with React-Router-Dom v4.3.1 and mapping over an array of objects that contains my Route components. In those Route components, there is a Render function where I am setting a new document title tag with document.title. It writes to the interface just fine. The problem is when I switch routes, AppInsights is reporting the previous page's title tag with the new page's URL.
I did a test using React Helmet v5.2.0 and setting the document title on each component. AppInsights was still behaving the same way.
Does any one know how we can get AppInsights to report the correct document title?

Hi @keithort - can you please try our new RC version which is fully rewritten in TypeScript and uses the new AI SDK? I think we noticed this issue during rewrite and added a 500ms delay on the trackPageView call to ensure that it records the values of the new page and not the old page.
You can install it by running this command:
npm install react-appinsights@beta

Thanks. It seems to work better however I am seeing one page view reported 2 or 3 times. I will dig into the issue and see if its my implementation or a bug. If it is a bug, I will open a new issue.


Update stream-analytics-login-credentials-inputs-outputs.md

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

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

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

TTYL, AZPRBOT;


merge package from arlac77/npm-package-template

package.json

chore(package<EMAIL_ADDRESS>

:tada: This PR is included in version 1.0.10 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


set "\r" "\r\n" "\n" is allowed at the end of this sentence.

when sentence end is "\r" "\n" "\r\n",minmea_check will return falase.This is not the expected result.We should let minmea_check return true when “\r” "\n" "\r\n" at end of sentence

Closed in favor of #41.


Fix intl mock for cost models table test

The recent PatternFly table upgrade is generating the test error below. The containerized build is failing, so new PRs are blocked from merging.

Codecov Report

Merging #2803 (00f54d9) into main (a197f42) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2803   +/-   ##
=======================================
  Coverage   65.74%   65.74%
=======================================
  Files         485      485
  Lines        9341     9341
  Branches     2158     2158
=======================================
  Hits         6141     6141
  Misses       3197     3197
  Partials        3        3

Continue to review full report at Codecov.

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


do not cache IProjectProvider

Good rollback.
I think caching IProjectProvider is more trouble than worth.
Since we have problems with slow GetProjectItem https://github.com/fsprojects/VisualFSharpPowerTools/blob/d3d386a6702cb761e3f40aa9b1ac4e4071c0c997/src/FSharpVSPowerTools.Logic/ProjectFactory.fs#L130, a better solution would be to use IVSRunningDocumentTable https://msdn.microsoft.com/en-us/library/bb166372.aspx


Microservice Communication for Graphql APIs

Hi Everyone,
Problem Statement : I have a GraphQl query defined which is responsible for fetching few details from the database.
This graphql schema let's say is defined in microservice-B . Now I want this garphql API to be called by an upstream microservice-A.
microservice-A is actually the one which is actually called by our Clients(UI or any other client) Via Gateway
So, all the business logic is thereby written in the resolver written in microservice-B only, microservice-A resolver is just acting like a proxy to make a call to microservice-B resolver.
The Flow is UI -> Gateway -> Service A -> Service B
UI will make a graphql request to service A , service A(resolver) should make a call to service B(resolver).
Now, for the microservice communication I am using OpenFeign Client.
For staying intact with the design pattern, the same graphql schema needs to be present in both the services A & B
Where I am stuck is 2 places :(

When I have to make a call to service B from service A , I need to pass the actual query resolved with the arguments to the client

This is how the resolver - B is defined in the feignClient UserAPI
User API Feign Client
 @PostMapping(value = "/graphql/users" ,consumes = MediaType.APPLICATION_JSON_VALUE, produces = MediaType.APPLICATION_JSON_VALUE)
    String getUsers(@RequestBody String query
    );

How do I get the actual query in the resolver A ? Please  refer to the commented line ?
Resolver of Service A
@Component
public class UserResolverServiceA implements GraphQLQueryResolver {

    @Autowired
    private UserAPI userAPI;
    public User getUser( String firstName, String lastName, String phoneNumber , DataFetchingEnvironment environment)  {

        //How to get resolved query in the resolver for passing it to downstream service
        String result = userAPI.getUsers(query);

       //Convert this to actual User object by stripping off the graphql wrapped response.
       //Make changes if needed specific to service A

    }
}

Now , since you see ideally for REST APIs the return type of Upstream Service , Feign Client & Downstream is same or superset of the JavaObject, but since GraphQL resolver responses are intercepted by GraphQL Server and it wraps it into graphql response, the return type from resolver B is actually not User object to the feign client  but a wrapped graphQL response String .

That's why the feign client will receive String as response from  the resolver B .
Resolver A should how remove the graphql wrapper response and retrieve the actual User Object ?
Resolver of Service B
@Component
public class UserResolverServiceB implements GraphQLQueryResolver {
    @Autowired
    UserService userService;

    public User getUser(String firstName, String lastName, String phoneNumber){

        try{
            User user = userService.getUsers(firstName, lastName, phoneNumber);
            return user;

        } catch (Exception e){
            // TODO - Add Error Handling
            throw e;
        }
    }

}

I'm using
Java17
Springboot - 2.7.3
graphql-starter - 14.0.0
graphql-java - 19.2.0

Have added schema , endpoint etc details in sprigboot properties hence not created GraphqlProvider explicitly.
Tried to get hold of the query by intercepting the request by one of the custom filter but that didin't work!
I don't think it should be that complicated  to achieve what I am expecting.
I'm sorry for the lengthy description. I'm new to GraphQL hence kind of struggling to how to address this problem.
Please help me out on this. And please do guide for addressing this issue more elegantly.

Closing in favor of https://github.com/graphql-java/graphql-java/discussions/3301
This is a very generic question and it does not relate to spring-graphql as you are not using this project.


fix(iam-role-for-service-accounts-eks): Return valid role_name_condition value

Description
When using the iam-role-for-service-accounts-eks module with the allow_self_assume_role set to true a dynamic statement is generated for the policy. This policy uses a local role_name_condition computed using the coalesce function as follows:
coalesce(var.role_name, "${var.role_name_prefix}*")
The issue comes from the fact that the var.role_name_prefix is null by default, and cannot be included in a string template. The error generated is:
The expression result is null. Cannot include a null value in a string template.

Motivation and Context
This change solves the issue generated when the allow_self_assume_role set to true. It is a fix to handle the different cases properly
The PR also fixes this open issue: #383
Breaking Changes
The PR does not introduce any breaking changes.
How Has This Been Tested?

[x] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects

Changes have been tested by creating a new example taking the module.cert_manager_irsa_role as a base and setting allow_self_assume_role to true. The new example is included in the PR to make sure this case is always properly handled, it is under module.cert_manager_irsa_role_self_assume
The original example has been tested with the new changes to make sure nothing is broken.

[x] I have executed pre-commit run -a on my pull request

Is this going to get merged and released?


Using the add function does nothing

Hi
When I use the add function and get it to scan the QR code, it gets the code fine and adds it to the secret key field, when I select the 2nd slot and hit 'Ok', it doesn't actually do anything or add the details,  when I unplug and plug the Yubikey NEO in again, it doesn't show the new credential nor on the Android app. When I add credentials on the Android app, they show both on the Windows app and Android app.

The desktop version of Yubico Authenticator support two different modes, the "normal" OATH functionality which is available in the NEO and YK4, which is also supported on Android over NFC, and the older "legacy" mode, which uses the two slots available on more basic YubiKeys as well. The desktop app SHOULD always use the newer OATH functionality, if available. If not, it will use the legacy mode as a fallback.
Now, as the dialog you get when you're adding a credential is referencing slots, this means that the app is unable to detect the newer OATH functionality and instead using the legacy fallback . This might be because you do not have CCID mode enabled, which is required for the newer functionality. If so, you can use https://developers.yubico.com/yubikey-neo-manager/ to enable it. If CCID mode is enabled and you're still having this problem, then please let me know and we'll take it from there.

Yes, I have CCID enabled on both keys: http://i.imgur.com/tsnpPRe.png
When I open Windows Authenticator, it brings up the multiple OATH credentials I have on the one slot so it's definitely storing them all within just slot 2 but when I click the refresh icon icon on the slot name, that's when it asks me to press my yubikey then does nothing when I do tap it. If that is part of the legacy mode then how do I make it so I have to tap to get further TOTP codes after the initial time is up as it keeps restarting the time after it runs out currently.

The slot based (legacy) mode only supports two unnamed credentials, one in slot 1 and one in slot 2. If you have more than two credentials then you're definitely using the "normal" mode. Also, the slot based credentials will be named "YubiKey slot1" and "YubiKey slot 2", anything else is NOT using the legacy slots.
Now, it seems that something else is interfering with the app detecting the "normal" OATH functionality. If you open the application settings, try disabling both slots and saving. That way the app will not try reading from the legacy slots which under some circumstances interferes with the normal mode.

Oh, and requiring touch is only supported for (legacy) slot based credentials as well as for the newer YubiKey 4 "normal" OATH functionality.

Adding and reading now works flawlessly after I disabled the read from the 2nd slot in settings so thanks a lot for that :)
It's a shame the touch requirement is only supported for the newer versions since the Android app has a version of it where you have to tap the yubikey again to get new codes after the timer is up.

Is there anyway you can make it stop generating codes until the yubikey is pressed again or replugged in for my NEO since it's such a big security hole if someone got into my computer somehow remotely and was able to generate codes if I accidentlally left the Yubikey plugged in while I was away. I know it currently doesn't exist as you say but is it a possibility to add?

Not for NEO, I'm afraid. As stated, the YubiKey 4 can be programmed to require a button press for each code generation, but the NEO does not support this. What you can do is not have the application remember the password. In that case, any time the window is hidden (closed or minimized) the cached password will be deleted, and you will need to re-enter it the next time you wish to generate a code.

Ah so even having the authenticator window minimised, will stop any passcodes being generated? Maybe a master password setting could be added that you need to enter each time you wish to generate codes?

That is correct. Once minimized the app will stop generating codes, and any non-persisted password will be "forgotten".
The problem with requiring a password for each time you want to generate codes is that each code is only valid for at most 30 seconds. Half of the time the code will be valid fro less than 15 seconds, likely requiring another code generation before you're able to get a code that is usable. Add in the time it takes to locate the correct code in a long list, and you easily get into a situation where you have to enter the password multiple times just to use it once, which would be a usability nightmare.


Discrete_uniform : Range of parameter

While I undertand it is due to the representation of floating point numbers in python and other languages, it would be good if the discrete_uniform range of parameter could be better handled:
Exemple:
trial.suggest_discrete_uniform('keep_prob', 0.8, 1.0, 0.1)
will produce the following correction:
 The range of parameter `keep_prob` is not divisible by `q`, and is replaced by [0.8, 0.9].

Thank you for sending your feedback.
We have already recognized the issue, but we plan to keep current implementation for a while. As you mentioned, this is the fundamental issue caused by the representation of floating point numbers, and we think we'll bring a potential risk of future bugs if we handle such values exceptionally.
As a workaround, you can use trial.suggest_int instead of trial.suggest_discrete_uniform.
For instance, you can rewrite your example code as follows:
# before
trial.suggest_discrete_uniform('keep_prob', 0.8, 1.0, 0.1)
# after
trial.suggest_int('keep_prob', 8, 10) * 0.1

Understood and thanks for the tip

Thank you for your understanding. Let me close this issue.
If you have further comments, please re-open it.

Hi, it may be useful to fix the documentation to reflect this limitation. The example there does not work at the moment:
x = trial.suggest_discrete_uniform('x', 0.1, 1.0, 0.1)
[W 2019-09-19 01:03:42,710] The range of parameter x is not divisible by q, and is replaced by [0.1, 0.9]

@fotisj FYI This issue will be resolved by https://github.com/pfnet/optuna/pull/505.


(maint): update references to zplugin & zdharma

Signed-off-by: Vladislav Doster<EMAIL_ADDRESS>

@pschmitt I can't find your email, but do you mind using PRs instead of just pushing directly to repository main branches? Not only is it an anti-pattern, but  I've had PRs open and you're duplicating work I've done and now it was a waste of my time.

Sorry for that, I've been running a bunch of find -exec sed -i 's#zdharma#zdharma-continuum#g' on the whole org.
Will try to stick to PR in the future, sorry!


Internal Athena errors should be returned with a 5xx error code

What happened:
We're often seeing random Athena errors like this one:

error querying the database: GENERIC_INTERNAL_ERROR: Cannot invoke "io.trino.execution.SqlTaskExecution$DriverSplitRunnerFactory.enqueueSplits(java.util.Set, boolean)" because "factory" is null

Those errors are impossible to predict and apparently also not avoidable. However, when such an error happens, it often comes back with a 4xx error code from the /api/ds/query endpoint. We are using this endpoint for some automated tests and kind of rely on the response code to decide whether to retry the test. Retrying a something that really is a client error (e.g. query with syntax error) doesn't really make sense. So it would be nice to be able to distinguish between real client errors and internal errors.
What you expected to happen:
I'd expect internal Athena errors to be returned with a 5xx response code for the /query endpoint. 4xx response code makes sense for real client errors like sending a query with a syntax error.
How to reproduce it (as minimally and precisely as possible):
Not really reproducible in a reliable way
Environment:

Grafana version: 10.2.3
Plugin version: 2.14.1

Hi @skuzzle, hmm this is a tough one. I would think that we would want to forward whatever status codes AWS returns to us and not get too opinionated with our error handling here but I am not familiar with that specific error. Do you have any more information on it? It does seem like many of these "GENERIC_INTERAL_ERRORS" are often syntax/client related, which makes me a bit hesitant to add something: https://repost.aws/knowledge-center/athena-generic-internal-error
I think for us to implement something like this we'd need more information about the error so we feel confident that it should in fact be treated like a 5xx in all instances. It sounds like in your case it probably makes sense, but I'm not confident that is true in all cases since I don't understand what it means. We can ask our contacts at AWS to see if they are more familiar, it's also possible they do respond with a 5xx and there's a bug in our code where we do not forward that along properly.
In the mean time it could be helpful if you also want to reach out to AWS, and do let us know if you learn anything more about this error. Thanks for reporting!

it's also possible they do respond with a 5xx and there's a bug in our code where we do not forward that along properly

This is what I suspected is happening. I'm not expecting you to fix Athena error handling on your side. So if you are already forwarding 5xx as 5xx and 4xx as 4xx then you probably should not change a thing and it's up to Athena folks to fix this.
We spoke to AWS, resp. with our cloud contractor about some of the odd failures and the gist was that those kind of errors can happen in a distributed system and they should be handled with a retry on client side. Now this obviously makes only sense if those errors do come back as 5xx

Ah I see what you mean @skuzzle its going to be hard for us to cause the specific error you're talking about to test since we don't have repro steps, but we certainly can stub out different error status codes from AWS and see what gets returned.
I just tried hard coding a fake error with a status of 500, and noticed that while we returned that in the response object, the actual status of the response was a 400. I'm guessing we probably have a bug here. I updated the title to have someone double check/update any places they can find where we might not be forwarding the error status code.
Thanks for talking it through and making the issue! I'm going to put this into our backlog for now since we don't have a specific timeline just yet on fixing it and don't want to overpromise a delivery date till we can figure it out, but will be sure to bring it up to the team. Also if you have any interest in contributing we'd be happy to review any prs :)

Thx for picking this up. For the sake of completeness, here are two further errors we encountered that came back with a 4xx status from Grafana but that clearly look like internal Athena errors:
error querying the database: [ErrorCode: INTERNAL_ERROR_QUERY_ENGINE] Amazon Athena experienced an internal error while executing this query. Please contact AWS support for further assistance. You will not be charged for this query. We apologize for the inconvenience.

error querying the database: Query exhausted resources at this scale factor


Cannot load minecraft with latest InvTweaks dev build

java.lang.NoClassDefFoundError: net/minecraftforge/fml/relauncher/IFMLLoadingPlugin
Updating from INVENTORYTWEAKS-1.59-DEV-156
to InventoryTweaks-1.59-beta-174.jar
causes FML to throw a bunch of errors and cannot load the game (no crash log) using ATLauncher and the latest version of RR3.
crash log: http://pastebin.com/8vWvjakT

Maybe read the change notes for a while back, or the mc version on curse forge?

Ok, thanks!  Sorry

Ok thanks! Got it. 1.7.10.
Sorry.


Search doesn't seem to parse inside Tenants

Environment

Python version: 3.6.8
Nautobot version:1.1.3

Searching doesn't seem to parse Tenants. For example, I have a tenant called "Banana Unit", 4 devices have "Banana Unit" set as Tenant. Searching for "Banana"  only shows devices which have Banana in somewhere in their fields (comments, description etc) it doesn't show devices that have only have "Banana Unit" as their tenant.
In the search results the following appears.
Devices
Any devices that have "Banana" in the comments, description, or other fields - not those that only have the Tenant as "Banana Unit".
Virtual Machines
Same issue as devices.
Prefixes
Same issue as devices.
Tenenats
The Tenant does show up here.
The search results side panel will only count the number of devices that show up in the search results - 1 in this case. But, if you open the Tenant, there are more devices than showing here.
It seems that the search is not parsing the Tenant.
Steps to Reproduce

Create a Tenant, add some devices to the tenant.
Search for the Tenant.
Only devices that have the Tenant name in their fields (description, name, etc) but not those with only the Tenant set, show up.

Expected Behavior
All devices with the Tenant searched for should show up if the Tenant name is assigned, or the name is in any of the device fields.

Observed Behavior
Only devices with the Tenant name in any of the fields show up. Those with just the Tenant set do not show up.

Added screen shots to help explain.
Forgot to circle VM and IP totals as well - these are wrong on search results as well.

Thanks for the report! As I understand it this comes down to how the search method is implemented for each individual model mentioned - for example VirtualMachine search, as currently implemented, only searches the "name" and "comments" fields in this table, not the "tenant" field. We could certainly add the "tenant" field to the search parameters for each of these models, though it will come at the cost of some decrease in global search performance.

Related: #38, as well as other discussions about the need to generally refactor/redesign/improve the global search feature.

This still seems to be an issue in v2.1.2. Any news on this?


Fix bug in git submodule branch resolution

Description
This PR fixes an issue in the branch name resolution logic in the vcs provider.
Basically, when you are within a git submodule the .git file points to the git directory and that is being returned from has_git_file whereas has_git_dir is returning the project root (the one that contains the .git within it. Later it is assumed that /.git should be appended however we don't want to that for submodules, instead we consistently return the git directory from both has_git_dir and has_git_file
Thank you for this awesome plugin!

this patch has bug. please test it then fix.


std.csv std.stdio.File.byLine example

IMO it it not obvious how to get the content of a file to the csvReader
the new example shows how to do it

Auto-merge toggled on

thanks


[DPM] NB-to-SB message transform design and codes refactor

Context
NB message carries most customer port information grouped by port while SB message carries customer + infra information grouped by host. This message transformation is not trivial requiring careful design and elegant implementation.
./services/data_plane_manager/src/main/java/com/futurewei/alcor/dataplane/utils/GoalStateManager.java
Request

Draw out the algorithm flowchat of message transform
Refactor and modularize the DPM codes so that it is easier to maintain

Linked to umbrella issue #439

As https://github.com/futurewei-cloud/alcor/pull/486 and https://github.com/futurewei-cloud/alcor/pull/472 have been merged to master, the refactor is almost done.
@chenpiaoping Could you pls draw a software architectural diagram to show how you redesign DPM v2.0? Thanks.


VS freezes up when trying to perform a semantic operation within the first 10 seconds of opening Monaco

Clone VS Code
Open a file
Try to get quick info
VS freezes

@DanielRosenwasser can you make a dump using procdump at the moment when VS stops responding?

this looks like a consequence of https://github.com/dotnet/roslyn/issues/7403, so resolving it as external


Cmp. https://colabti.org/irclogger/irclogger_log/perl6-dev?date=2019-07-09#l83

Hey @usev6, what can be our long-term plan for todo-ed and skip-ed tests?
For example, what if we make the next SQUASHathon to be about roast todo-ed tests? We can then populate a separate issue tracker with issues, one per test. We make a guide on what to do with them (e.g. if the test doesn't make sense anymore, then delete it; if there are a bunch of todo-ed tests for a feature, file a single ticket and reference it from the tests; etc.). I believe the contributors will be able to do a lot of great work. It also sounds kinda fun and relatively easy.
Similar squashathons:

2017-12-02 MOLD Removal SQUASHathon (results)
2018-08-04 Ecosystem unbitrot (results)
2019-07-06 Math sequences squashathon (results)

Hi @AlexDaniel,

what can be our long-term plan for todo-ed and skip-ed tests?

actually, I'm not sure about that. I think we have at least three different kinds of todo-ed and skip-ed tests:

tests for obvious bugs (those are mostly covered with existing tickets on RT or GH)
tests that expect a different result than the current implementation gives, but where it's not obvious if the tests are right
tests for not implemented features

IMHO there are a lot of tests where it's not easy (for many contributors) to see to which category they belong. Your idea to use a squashathon to sort out todo-ed and skip-ed roast tests sounds attractive, but I fear that (most of) this work needs to be done by devs from the "inner circle". (Since roast is the specification, evaluating tests sounds a lot like language design.) So, from my perspective there aren't (m)any low hanging fruits in this area. (Of course, I might be wrong.)

I see. Thanks

If no one objects, I'll merge this PR and close the mentioned placeholder tickets soon-ish.


Filter bug in CRUD Search model generator

when i set some attribute by deffault it's filtering by him, but if i put into filter not available value, after $this->validate() return false and search method return my $dataProvider whitout any filter, and show all data but it's not correct, i think if you set in filter incorrect data and other correct data, search method must apply correct filter data, and where incorrect data show error, but now if one data incorrect, search method think that all data incorrect and show all record from db.

Duplicate of https://github.com/yiisoft/yii2/issues/3665


Created an Alarm Clock

Closes #21

[x] I have updated the documentation accordingly.
[x] I have added a new Project Alarm Clock.
[x] I have added a README.md file for the Project.
[x] The title of my pull request is a short description of the requested changes.

Screenshots

Hey I am not able to understand the relevance of the files other than clock.py could you please document that in the readme file or add sufficient comments inside the code.

Nice work , Leave a ⭐ and try to contribute to more projects.

Sure, Thanks!


migration to new instrumentation class

I'm reaching out because you guys already wrote few instrumentation package for nodejs and i believe it's best to let you know that we implement a new base class called Instrumentation.
You can find our tracking issue here: https://github.com/open-telemetry/opentelemetry-js/issues/1074
I'm in the process of migrating the http/https plugin (i'll ping here with the PR once its done), there is already the graphql plugin that have implemented: https://github.com/open-telemetry/opentelemetry-js-contrib/blob/master/plugins/node/opentelemetry-instrumentation-graphql/src/graphql.ts.
There was no discussion yet about removing the old plugin mechanism as they still a lot of plugin to migrate but i will try to keep you updated on that

PR for the http(s) plugin is here: https://github.com/open-telemetry/opentelemetry-js/pull/1671

Hey @vmarchaud, thanks for the heads up.
We're working on the required adjustments.

New packages committed in PR https://github.com/aspecto-io/opentelemetry-ext-js/pull/61 and released to npm.


[WIP] bspwm module

[DO NOT MERGE YET]
This is the initial commit for the bspwm module. Right now, you can add bspc config directives, startup programs and raw configuration.
Based in this issue: #359

Hello, what's the advancement for this module ? Do you need some help ?

You can check the status of the PR at #359 . I stopped working on this a year ago, and I don't use bspwm now. Feel free to finish it!


add a hook to allow users to transform the materialized object

useful, e.g. when restoring singletons.
this cannot be less overengineered, I think ;)

I am green now :P
now, if that's not the correct place, it certainly looks like (from my small understanding). At least my usecase works like a charm now :P

Ok, I need to do one more cycle on it because that should be possible for serialization and for materialization

I add it to an integration branch so can add missing stuff


AES256 implementation

Good day.
I am searching for detailed AES256 kerberos algorithm implementation.
I am interested espeсially in nfold algorithm in it. I know, how nfold algorithm works, but it is very important for me to know, which constants except string "kerberos" are used in key derivation with the help of nfold algorithm. I have heard that these constants depends on the type of traffic packet (AS_REG, AS_REP, TGS_REP)
Could you please help to find it, it is very important?

Sorry, but this isn't the right forum for that question. Perhaps you can try asking on one of the Kerberos support channels listed here: http://web.mit.edu/kerberos/www/krb5-latest/doc/resources.html


NAS-126012 / 23.10.2 / Add ability to ignore files/dirs for directory item (by Qubad786)

Context
Users were having large dbufs files which were expensive to query.
Change
This PR adds the functionality to skip copying files and / or directories in artifacts' directory class.
Skips capturing the /proc/spl/kstat/zfs/dbufs file in debug to prevent unnecessary resource utilization.
Original PR: https://github.com/truenas/ixdiagnose/pull/148
Jira URL: https://ixsystems.atlassian.net/browse/NAS-126012

This PR has been merged and conversations have been locked.
If you would like to discuss more about this issue please use our forums or raise a Jira ticket.


Execution failed for task ':app:bundleReleaseJsAndAssets'.

Description
when create version aab to upload to google play Don't make the apk And return this error

What went wrong:
Execution failed for task ':app:bundleReleaseJsAndAssets'.

Process 'command 'cmd'' finished with non-zero exit value 1

React Native Version
0.67.1
Output of npx react-native info
System:
OS: Windows 10 10.0.19045
CPU: (16) x64 11th Gen Intel(R) Core(TM) i7-11800H @ 2.30GHz
Memory: 4.40 GB / 15.73 GB
Binaries:
Node: 14.15.1 - C:\Program Files\nodejs\node.EXE
Yarn: 1.22.19 - ~\AppData\Roaming\npm\yarn.CMD
npm: 6.14.8 - C:\Program Files\nodejs\npm.CMD
Watchman: Not Found
SDKs:
Android SDK: Not Found
Windows SDK: Not Found
IDEs:
Android Studio: AI-221.6008.13.2211.9514443
Visual Studio: Not Found
Languages:
Java: 11.0.17 - C:\Program Files\Common Files\Oracle\Java\javapath\javac.EXE
npmPackages:
@react-native-community/cli: Not Found
react: 17.0.2 => 17.0.2
react-native: 0.67.1 => 0.67.1
react-native-windows: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
cd Android
./gradlew bundleRelease
can show issue
Snack, code example, screenshot, or link to a repository

As the bot said, you're on a unsupported version + we would need the full debug log.

As the bot said, you're on a unsupported version + we would need the full debug log.

upgrade react native version ??

Correct, we offer support only for RN 0.69+:
https://github.com/reactwg/react-native-releases/

Closing as per my previous comments


Add footer to output floating window with release 10.0

When you click on the output window, it focuses, and because the southern border does not have a title, then the title is removed from the input window, see:
Expected:

Actual

fix
Add a footer to the output window options, but is only available in neovim 10.0, see PR

closing as it can be changed with a user defined option for chat window as per #52


Fabio UI not displaying when hit on a DNS name

We're upgrading to Fabio 1.5.12 from 1.5.11. There are three instances of Fabio running which all respond to the same DNS name (via DNS round robbin), i.e. the DNS name our.proxy.example.com points to the IP addresses of the three fabio instances.
With 1.5.11 the UI displays as expected. With 1.5.12 it is broken, but only in Firefox and only if you hit the 'shared' DNS name. If you hit one of the IP addresses directly it works just fine.
If you hit the UI via the DNS name it looks like this:

but if you hit the IP address directly it looks like this

In Chrome both look normal
This is in Firefox 69.0.2 on Windows 10

Could you provide more information via the Firefox developer tools?  Which resource is failing to load?

Actually I can't reproduce this at the moment :( I think overnight the caches cleared or Firefox had a change of mind or gremlins ... I shall close this ticket. Thanks for humoring me.


DSC unable to retrieve portal token after 1.1.0

Hi Everyone,
First, thanks for all the contributions and work on this project! I've run into an issue with a previously running build after going to 1.1.0, although I've changed the config since then too. When running the attached base deployment federation fails with the following error:

PowerShell DSC resource ArcGIS_Federation  failed to execute Test-TargetResource functionality with error message: Unable to retrieve Portal Token for 'AdminAccount' from Deployment 'https://306.[redacted].com'

I have verified that the account used is being created as an admin in the portal. I can also log into /home , /portaladmin, and /sharing using the credentials as well as /generateToken with it.
base.txt
307_mof.txt
ArcGISFederation-Error.txt
ArcGISFederation-Verbose.txt
I can run in debug mode and step through ArcGIS.psm1 but I'm not sure how to debug whats running on the nodes?
~Thanks

Hey Corey,
Looks like there are errors in your config file for version 1.1 of this module.

"Target" Parameter in SSL Certificate Block is now a required param
No node in Config file is assigned a Fileshare role. This is necessary since we create a file share for the deployment. As of now, we don't support external file shares.
FileShareLocalPath  should be a local file path, again looks like you are using an external file share. (It is in the roadmap but not fully tested).
As of now, the is configured for both server and portal. You can't use them with only one as you are using as of now. ContentDirectoryLocation in the portal config block also needs to be a file share path using the same file share path as the one given for ContentDirectoryLocation of the server.
You don't need the federation block. It is required only for federated servers with specialized roles.

PS. I think a combination of the following two config files can help

https://github.com/Esri/arcgis-powershell-dsc/blob/master/SampleConfigs/Base Deployment/BaseDeployment-DualMachine.json
https://github.com/Esri/arcgis-powershell-dsc/blob/master/SampleConfigs/Base Deployment/BaseDeployment-ThreeMachine-10.6-ssl-per-Node-Example1.json

@RQF4 you can email me at<EMAIL_ADDRESS>. We can discuss your specific use case on Skype.
Thanks,
Shailesh

Support for external file shares and the File Share only being used only for server or portal for "ContentDirectoryLocation" was added.


[Task] Enable and document Jenkins Github Login

Reference

harvester/tests/issues/1648

PR: harvester/harvester-baremetal-ansible/pull/89


华为p20  android 10 出现图片选择的时候出现闪退.

android.database.sqlite.SQLiteException: near "GROUP": syntax error (Sqlite code 1 SQLITE_ERROR): , while compiling: SELECT _id, bucket_id, bucket_display_name, _data, COUNT(*) AS count FROM files WHERE ((owner_package_name IN ( 'com.daylong.wunder' ) OR media_type=2 OR media_type=4 OR media_type=0 AND mime_type LIKE 'audio/%' OR media_type=3 OR media_type=0 AND mime_type LIKE 'video/%' OR media_type=1 OR media_type=0 AND mime_type LIKE 'image/%') AND (is_pending=0) AND (is_trashed=0) AND (volume_name IN ( 'external_primary' ))) AND ((media_type=? AND _size>0) GROUP BY (bucket_id)) ORDER BY datetaken DESC, (OS error - 2:No such file or directory)

升级到最新版本试试

升级到最新版本就没有这个问题了


Revert "Accept uniqueness-token option to add to HTTP_HEADERS"

Reverts nationbuilder/nationbuilder-rb#22
@sethtrain @kaip

:+1:


misjudge of error,some ut code not executed

Please provide a description of this PR:
in ut code, I think the error condition was misjudged, some ut code never executed.

Hi @usernameisnull. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test


Supplementary - Separate Msgs and Msg_attributes - Silver & Core

We have a great messages table that provides attributes alongside the events that occur. However, to adhere to Flipside's IBC Model Standards, we need to create 2 more message tables: msgs and msg_attributes where these events are separated out. They should follow the same data format as currently available for Axelar and Osmosis on the Flipside app. This is so IBC cross-chain analysis can happen across like tables.
We will keep the current messages table as-is. I propose we rename this ez_messages to avoid potential confusion with the 2 incoming msg tables.
This GH Issue is for both msgs and mst_attributes as they are closely intertwined and contain some column overlap.
To Do

[ ]  Assign primary & secondary analysts
[ ]  Design the schema for this table
[ ]  Get confirmation on schema
[ ]  Build schema

[ ] fact msgs
[ ] fact msg attributes
[ ] renaming fact messages to ez messages

[ ]  Schema build review
[ ]  Submit Pull Request for review by pod

Direction
https://docs.google.com/spreadsheets/d/15VjarJe6AYrCuuxgTvyTweqNMPnYTjeZXPff0SHS-ck/
CORE.FACT_MSGS

Column
Data Type
Tests
Documentation

block_id
number
not null, sequence gaps, number format
Also known as block height. The block_id, which indicates the length of the blockchain, increases after the addition of each new block.

block_timestamp
timestamp
One day recent, not null, timestamp format
The date and time the block was recorded.

blockchain
string
not null, string/varchar format
The name of the blockchain.

chain_id
string
not null, string/varchar format
The version of the blockchain.

tx_id
string
not null, string/varchar format
A unique key that identifies a transaction. Called "TxHash" on block explorers.

tx_succeeded
boolean
not null
TRUE if the transaction was successful, FALSE otherwise.

msg_group
string
N/A - this can be null
Value grouping different messages together to represent a single action or event. Format will include the numeric msg_group and msg_sub_group with a ":" seperator. The subgroup will always be 0 except for "Exec" actions. NULL group means messages are related to the header (overall transaction).

msg_index
number
not null, number format
Short for "message index," the position in which messages occur within a transaction

msg_type
string
not null, string/varchar format
A string containing information about the type of message occurring.

msg
json
not null
The underlying json from the message or event(s) occuring within the transaction.

CORE.FACT_MSG_ATTRIBUTES

Column
Data Type
Tests
Documentation

block_id
number
not null, sequence gaps, number format
Also known as block height. The block_id, which indicates the length of the blockchain, increases after the addition of each new block.

block_timestamp
timestamp
One day recent, not null, timestamp format
The date and time the block was recorded.

blockchain
string
not null, string/varchar format
The name of the blockchain.

chain_id
string
not null, string/varchar format
The version of the blockchain.

tx_id
string
not null, string/varchar format
A unique key that identifies a transaction. Called "TxHash" on block explorers.

tx_succeeded
boolean
not null
TRUE if the transaction was successful, FALSE otherwise.

msg_group
string
N/A - this can be null
Value grouping different messages together to represent a single action or event. Format will include the numeric msg_group and msg_sub_group with a ":" seperator. The subgroup will always be 0 except for "Exec" actions. NULL group means messages are related to the header (overall transaction).

msg_index
number
not null, number format
Short for "message index," the position in which messages occur within a transaction

msg_type
string
not null, string/varchar format
A string containing information about the type of message occurring.

attribute_index
number
!!! No or doc entry in Osmosis silver. not null, number format
The position in which attributes occur within a message

attribute_key
string
not null
The key from the key-value pair from the message attribute

attribute_value
string
N/A - can be NULL
The value from the key-value pair from the message attribute

Contribution Reward
Per the S3 approach, this falls into the project category and contribution incentive will be handled through Coordinape. Expected collaboration is 3-4 individuals with a primary Analyst, primary PR Reviewer and possible secondaries for either the Analyst or Review side of the work.
Issue reward pool: $1,600 in LUNA (token amount determined at time of payment). Notice this is slightly increased to the broader nature of this issue.

Taking this one


Ensure pinga fails on startup if it cannot connect to PG (BUG-68)

Ensure pinga fails on startup if it cannot connect to PG. SDF works similarly by failing upon migrating. This does result in a duplicate test of the database connection. However, we want to retain both tests because the first one is used by "si-data-pg" to warm up the pool. This second one is used to block the listener loop from occuring.

bors merge


We need to be able to represent the memory "better"

Some machines like the Commodore 64 have a lot of bank switching, and memory map I/O, etc etc. There's way more addressable memory than just 64K, and this includes UARTs, the sound chip, etc etc.
Then things like the Atari 2600 which have 128 bytes of RAM only, and so the stack and zero page all overlap.
From what I can see, currently this emulator implements memory as a flat array, so that precludes its use in an emulator of the above systems.
What we should do instead is define a trait that has the necessary functions for reading & writing to memory. This can then be impled by the host, and it can do what is right.

Yep, that's fine, I think that's a sensible default

Speaking of which, I still don't know how to implement this.
Especially what to do in cases like that:
https://github.com/mre/mos6502/blob/f3d3bf470f10a0d3789472df6321f5aae2001b47/src/cpu.rs#L53-L55
We would need to reset the memory as well and the type of Memory would depend on what was passed in the constructor.

We would need to reset the memory as well

Why do we need to so that? Memory doesn't have a reset pin like the CPU does

Also, why does CPU::reset() call CPU::new()?
It should do the same stack accesses as the real chip, and also leave most of the registers alone. An actual 6502 only initializes the stack pointer and the interrupt flag; maybe this emulator should do the same

Having a generic argument on CPU (i.e. CPU<M>) will also affect AddressingMode::process. The function signature will need to change. I want to suggest moving address calculations into the CPU instead of having the CPU's state leaking into AddressingMode::process function. What do you think? Can you see a better way?

Why do we need to so that? Memory doesn't have a reset pin like the CPU does

Oh, then we don't. I was confusing the (theoretically) correct handling of a memory reset with the practical implementation in hardware. Let's stay as close to the hardware as possible, yeah.

An actual 6502 only initializes the stack pointer and the interrupt flag when it's reset; maybe this emulator should do the same.

Is this the correct behavior?
https://github.com/gianlucag/mos6502/blob/master/mos6502.cpp#L820-L835
I guess we can do something similar.

Having a generic argument on CPU (i.e. CPU) will also affect AddressingMode::process. The function signature will need to change. I want to suggest moving address calculations into the CPU instead of having the CPU's state (in particular, the memory and registers X and Y) leaking into AddressingMode::process function. What do you think? Can you see a better way?

I don't have an opinion on this. So far I don't see a reason why we can't move the calculations into the CPU if that makes things easier. 👍

I changed my mind about this part:
impl Default for CPU {
    fn default() -> Self {
        Self::new(Memory::default())
    }
}

We probably don't want to have a Default for CPU as there is no reasonable default for Memory: it depends on the hardware that is emulated. We should probably be explicit about it and require the user to set up the correct amount of memory.


feat: Add custom dialog

Fixes Issue
My PR closes #177
👨💻 Changes proposed(What did you do ?)
Create a new Dialog component that will be used throughout the project instead of using browser's native alert. Screenshot below show what the dialog looked like added to the logout and chat close function.
✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title and description of the PR is clear and explains the approach.

Note to reviewers

📷 Screenshots

Doesn't look good on mobile

Also pls pull the latest changes to this and make sure they work properly

edit doesn't work
typing status is buggy

Just to be safe

Thank for the feedback, I'll check on the point that you mentioned

@Dun-sin As you mentioned

I have fixed the dialog to be more responsive
Refactor it to has typehint
I'm not quite sure what do you mean by edit doesnt work but if you could elaborate it here, I'm happy to work on it. Sorry if I make a lot of mistake I'm new to open-source contribution, so if you could bare with me.

@Manethpak great work, but why not just use the dialog component provided by RSuite?
It's mobile responsive and can be customized, plus it already being used in the app...

@mathiasayivor That was my bad, I didn't know you had a UI library installed. I read through the project's code and mostly the UI are all written by hand that's why I decided to write one as well

@mathiasayivor That was my bad, I didn't know you had a UI library installed. I read through the project's code and mostly the UI are all written by hand that's why I decided to write one as well

@Manethpak still doesn't look good on mobile


FEATURE test.fireAppEvent and test.fireComponentEvent

abbreviates tests

This should also include an attribute/helper for recording events for assertions as well


Add MongoDB cheatsheet

Issue Description
with reference to issue #94, I wish to add a MongoDB Commands cheat sheet. Please assign me this issue if #94 is accepted OR if you are willing to add it to the present resources setup as a new resource, please instruct about the same as a comment below. Thank you.
Resource Details
The issue needs the creation of a new MongoDB cheatsheet in a web-friendly format.
Topic or Category
New Resource for Proposed CheatSheets section.
Benefits of Adding this Resource
As discussed in issue #94, Usually, as a learner, it is hard to grasp the large number of resources we have, and most miss creating concise notes for each of these. In such cases, a cheat sheet can give a quick reference or a recap that can be a valuable resource for the learner. I propose to add the same to this website.

Please do not spam!


64bit support?

Hi All,
I have a file that's nearly 4GB, but I can't search through it:
> ag 10008488 master.tx
ERR: File master.tx failed to load: The parameter is incorrect.

I'm using ag-2019-02-17_2.2.0-11-g4f7aca7-x64.zip from
https://github.com/k-takata/the_silver_searcher-win32/releases
Is this because 4GB files won't work with ag?

https://github.com/ggreer/the_silver_searcher/blob/4f7aca7b01c157866f5b708341bbccad0d7a6256/src/search.c#L317-L334
It seems that the code doesn't support a larger file than 4 GB, but I'm not sure about nearly 4 GB.

Thanks for the reply, @k-takata.
Do you think the ag output above is because of the file size? The file is simply a text file, which it parses just fine when it's not so large, i.e. same file at 1GB has no problem.

Do you think the ag output above is because of the file size?

Yes, but it's not sure which API failed: CreateFileMapping or MapViewOfFile.


Unsafe assignment to innerHTML

      "file": "src/components.js",
      "line": 139

      "file": "src/components.js",
      "line": 151

https://stackoverflow.com/questions/30076616/how-do-i-replace-text-without-using-innerhtml


Auto complete still not working on double quoted tables and columns

I reported this on #2191 but was marked as a duplicate of #2169 .
My tables and columns are double quoted, the last version the auto complete on columns is working is 4.1.2 (I'm still using that version because of it).

Looks like 2169 was another issue with autop-complete. Anyhow I can't reproduce problem with quotes.
Please post some example (table DDL) and how to reproduce this problem.
Thanks.

CREATE TABLE dbo."TestTable" (
"Id" int4 NOT NULL,
"CategoryId" int4 NOT NULL DEFAULT 0)
;
When you start typing:
update dbo.t -> (dbo."TestTable" will appear)
But there's no autocomplete on:
update dbo."TestTable" set c -> (I'm expecting "CategoryId", but nothing showing)
Tested on Mac OS X.

Thanks for details.
It'll be fixed in the next version


Fix FPS in the encoded stream, latency and quality

Thanks for your contributions @oreh-a . Seems like good proposals. However, would you mind submitting them in separate PR, so that they can be reviewed and merged independently?

Can you please also explain what is meant by your Passing FPS to SetPortAndWindowHandler instead of multiplier makes mdhd/time_scale incorrect. And as a result, receiver recognizes stream like FPS^2 instead of real FPS statement?`
I thought that the MP4FragmentEditor::m_time_scale_multiplier already fixed this issue by scaling up the "mdhd" values from 1FPS (incorrect framerate reported to Media Foundation) to 25FPS in the generated bitstream.

Yes, you are right with FPS from encoder multiplied by FPS time_scale gives FPS^2. With FPS = 1 from encoder it fixes your 'lying'. With correct FPS from encoder it needs to be set to 1.

Yes, you are right with FPS from encoder multiplied by FPS time_scale gives FPS^2. With FPS = 1 from encoder it fixes your 'lying'. With correct FPS from encoder it needs to be set to 1.

Good. Then please update the commit comment so that is not longer misleading.

Thanks a lot for the suggestion @oreh-a. Closing this PR after merging #16 that proposes the same changes with more details and test results.