Always put OAuth user info in the ExternalLoginUser table

Never set OAuth in User.LoginSource, which keeps referring to
the username/password interpretation. Fixes #1124
\cc @willemvd, @morrildl, @lunny, @bkcsoft
I think this should be done before 1.1.0

It's really to need a proposal document to describe all the login.

If a document is needed, it should be produced before introducing
the OAuth2 support, rather than let it add confusion to the code base
(and changes to production databases)

Note that doing this after 1.1.0 is out will require a new migration to convert existing records with OAuth2 in User.LoginSource

@strk but even if it is done before, what happens to the installs from master builds (such as my own)?

@geek1011 good question, I guess those would be just left with inconsistent database values. You might end up having users with OAuth2 LoginSource and no way to login with those OAuth2.

I like this idea. There are two kinds of external Login Source. One is provide password authentication, another is for link.

I raised this issue in the OAuth2 PR before, but my concern was not
considered. Now I don't know if I have time to fix this myself.
I think it should be done by the author of the OAuth2 code.
It's great to be more "open" in development, but we all need to pay
more attention to contributions and give proper importance to
responsibility. Whoever introduces a feature should be available to
maintain it.

@strk, when you review a PR, if you think your opinion should not be ignored, you could use github's request changes, then the merger should ask your confirmation before merge it. Our rules only consider two reviewers, lost this rule.

I will try to add migration for this PR when I have time.

@lunny Did you have time to work on this? Otherwise I suggest we close this as "out of date"

@bkcsoft I'm afraid I have no time to work on this. I still have serval PRs in v1.3 and v1.4 need work.

It looks like this is related to https://github.com/go-gitea/gitea/issues/4226 and https://github.com/go-gitea/gitea/pull/5029

Closing as outdated. Please re-open if needed.

Note I CANNOT reopen. I can not modify any github ticket, although I'm supposedly a "maintainer" (I think this should change)

Thanks @lunny, I'm force-pushing a version with conflicts resolved (but didn't try it again)

@zeripath I've formatted the code, to make Drone CI happy (hopefully) -- in case you want to give this branch a try

Still relevant, not sure why Travis is sad

As per integration tests there is error in template:
template: user/settings/account:10:45: executing "user/settings/account" at <.SignedUser.IsOAuth2>: can't evaluate field IsOAuth2 in type interface {}

Refactors moved  routers/auth.go to routers/user/auth.go and the IsOAuth2 method from models/user.go to models/user/user.go but the logic changed by this PR remains the same. @strk it looks like your initial motivation for creating this PR is still valid, right?

Yes, the motivation remains valid. Unfortunately the high version of Go required to build Gitea made me in a position to have more problems contributing (can't build w/out installing a foreign package in my otherwise all-distro-based system, and I'm short on disk space). For this reason I can't test my change much. Will gladly accept a PR to my branch if you want to work on the refactoring porting.

It's stale for petty long time.
What's the root problem behind the issue/PR?
Is there any valuable information in this context?

It was mostly a design concern, a way to separate "external" authentication sources from "internal" ones. One problem being resolved would be described in https://github.com/go-gitea/gitea/issues/1124#issuecomment-284766818 but that issue was closed by stale bot too

Could some maintainers help to re-open that issue? @techknowlogick @lunny

I think this PR/issue could be kept open, until there is a clear conclusion

https://github.com/go-gitea/gitea/issues/1124 is opened so I think we could close this one because it's outdated.

I also think we can close this and keep the issue open for further discussion


Allow gaps in timeline

First off: Nice work! This is a very nice library.
I've got a request for a new feature though:
Currently the timeline can have four states:

Not started yet
Data available
No data available
Ended

It would be great if you could have intermediate gaps, i.e ranges where you don't know what happened.
This could be useful for cases where you are recording data every other month and within the month you'd want to be able to know if there's data or not and then the next month would be blank (white).
So basically a differentiation between "there's supposed to be data, but no data is available" and "there is no data, but there was never supposed to be any in the first place":
Something like this basically:

If this is already possible, then I didn't find out how to do it.

Hi @WACMemphis!
Thank you for your nice and well explained feedback!
I agree, the ability to display time gaps would be useful. I just pushed a commit that serves that purpose. Check out the updated readme and the time gap example.
Let me know if this is what you were looking for. Thanks!

Excellent work! And quick as well. Thanks.


SWT is sometimes referred to as SWM

Hi Bookkypoobah,
your wiki covers the SWT as well. In the text though it is sometimes referred to as SWM, which is not correct. It is called SWT = SWarm city Token
It would be great if you could make the proper ammendments.
Thank you.
Bernd

Corrected. Please check.

Checked it. Now it's perfect. Thank you very much.
Sent from my iPhone

On 22 Feb 2017, at 10:30, BokkyPooBah<EMAIL_ADDRESS>wrote:
Corrected. Please check.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Great!


HTTP Request Returned 412 Precondition Failed: No such cookbook: git

Occurs during Test Kitchen run.

-----> Cleaning up any prior instances of
-----> Destroying ...
Finished destroying  (0m0.00s).
-----> Testing
-----> Creating ...
Bringing machine 'default' up with 'virtualbox' provider...
==> default: Importing base box 'bento/ubuntu-16.04'...
==> default: Matching MAC address for NAT networking...
==> default: Checking if box 'bento/ubuntu-16.04' is up to date...
==> default: Setting the name of the VM: kitchen-server-base-cookbook-default-ubuntu-1604_default_1501782435127_36275
==> default: Fixed port collision for 22 => 2222. Now on port 2200.
==> default: Clearing any previously set network interfaces...
==> default: Preparing network interfaces based on configuration...
default: Adapter 1: nat
==> default: Forwarding ports...
default: 22 (guest) => 2200 (host) (adapter 1)
==> default: Booting VM...
==> default: Waiting for machine to boot. This may take a few minutes...
default: SSH address: <IP_ADDRESS>:2200
default: SSH username: vagrant
default: SSH auth method: private key
default:
default: Vagrant insecure key detected. Vagrant will automatically replace
default: this with a newly generated keypair for better security.
default:
default: Inserting generated public key within guest...
default: Removing insecure key from the guest if it's present...
default: Key inserted! Disconnecting and reconnecting using new SSH key...
==> default: Machine booted and ready!
==> default: Checking for guest additions in VM...
==> default: Setting hostname...
==> default: Mounting shared folders...
default: /tmp/omnibus/cache => /Users/sean/.kitchen/cache
==> default: Machine not provisioned because --no-provision is specified.
[SSH] Established
Vagrant instance  created.
Finished creating  (0m32.71s).
-----> Converging ...
Preparing files for transfer
Preparing dna.json
Resolving cookbook dependencies with Berkshelf 6.2.2...
Removing non-cookbook files before transfer
Preparing solo.rb
-----> Installing Chef Omnibus (12)
Downloading https://omnitruck.chef.io/install.sh to file /tmp/install.sh
Trying wget...
Download complete.
ubuntu 16.04 x86_64
Getting information for chef stable 12 for ubuntu...
downloading https://omnitruck.chef.io/stable/chef/metadata?v=12&p=ubuntu&pv=16.04&m=x86_64
to file /tmp/install.sh.1559/metadata.txt
trying wget...
sha1	c2b25c5ed62ec32ea61c045bef0afe25e9ce220f
sha256	93f7c1374fd82e4dd55fb8258ba6ebc53398e0cca534d2f065c8a5344a9230fd
url	https://packages.chef.io/files/stable/chef/12.21.3/ubuntu/16.04/chef_12.21.3-1_amd64.deb
version	12.21.3
downloaded metadata file looks valid...
/tmp/omnibus/cache/chef_12.21.3-1_amd64.deb already exists, verifiying checksum...
Comparing checksum with sha256sum...
checksum compare succeeded, using existing file!
Installing chef 12
installing with dpkg...
Selecting previously unselected package chef.
(Reading database ... 37882 files and directories currently installed.)
Preparing to unpack .../cache/chef_12.21.3-1_amd64.deb ...
Unpacking chef (12.21.3-1) ...
Setting up chef (12.21.3-1) ...
Thank you for installing Chef!
Transferring files to
[2017-08-03T17:47:45+00:00] INFO: Forking chef instance to converge...
Starting Chef Client, version 12.21.3
[2017-08-03T17:47:45+00:00] INFO: *** Chef 12.21.3 ***
[2017-08-03T17:47:45+00:00] INFO: Platform: x86_64-linux
[2017-08-03T17:47:45+00:00] INFO: Chef-client pid: 1643
[2017-08-03T17:47:45+00:00] INFO: The plugin path /etc/chef/ohai/plugins does not exist. Skipping...
Creating a new client identity for default-ubuntu-1604 using the validator key.
[2017-08-03T17:47:47+00:00] INFO: Client key /tmp/kitchen/client.pem is not present - registering
[2017-08-03T17:47:47+00:00] INFO: HTTP Request Returned 404 Not Found: Object not found: chefzero://localhost:8889/nodes/default-ubuntu-1604
[2017-08-03T17:47:47+00:00] INFO: Setting the run_list to ["recipe[server-base::default]"] from CLI options
[2017-08-03T17:47:47+00:00] INFO: Run List is [recipe[server-base::default]]
[2017-08-03T17:47:47+00:00] INFO: Run List expands to [server-base::default]
[2017-08-03T17:47:47+00:00] INFO: Starting Chef Run for default-ubuntu-1604
[2017-08-03T17:47:47+00:00] INFO: Running start handlers
[2017-08-03T17:47:47+00:00] INFO: Start handlers complete.
[2017-08-03T17:47:47+00:00] INFO: HTTP Request Returned 404 Not Found: Object not found:
resolving cookbooks for run list: ["server-base::default"]
[2017-08-03T17:47:47+00:00] INFO: HTTP Request Returned 412 Precondition Failed: No such cookbook: git
   ================================================================================
   Error Resolving Cookbooks for Run List:
   ================================================================================

   Missing Cookbooks:
   ------------------
   No such cookbook: git

   Expanded Run List:
   ------------------
   * server-base::default

   System Info:
   ------------
   chef_version=12.21.3
   platform=ubuntu
   platform_version=16.04
   ruby=ruby 2.3.1p112 (2016-04-26 revision 54768) [x86_64-linux]
   program_name=chef-solo worker: ppid=1637;start=17:47:45;
   executable=/opt/chef/bin/chef-solo

No longer an issue.


Rename parsers.py to parser.py

The old datasets use the parsers.py convention but the template uses parser.py (which I believe is a better name)
So we need to rename the files to parser.py and update their respective __init__.py file.

Hi @lgvaz, I was looking to contribute to this project but couldn't find some suitable beginner issues. I still would need to understand the codebase for making future contributions, so I think this issue would be a good starting point for me :)
Could you assign me to this issue?

For refactoring the code, I think these are the files which need to be changed:

icedata/datasets/voc/parsers.py
icedata/datasets/coco/parsers.py
icedata/datasets/pets/parsers.py
icedata/datasets/birds/parsers.py
icedata/datasets/fridge/parsers.py
icedata/datasets/pennfudan/parsers.py

updating their respective __init__.py files.

Those are the necessary changes, correct =)
Thanks for the PR! I'll check it later on today


Add install_options to package spec(?)

Hi, have been having major headaches with installing on a CentOS server where an older version of varnish is in one of the repositories. Finally got round it by patching your module to add install options on the varnish package spec, and then passing a parameter to disable the repo in question.
I don't know if that's the best approach: any other ideas would be welcome. I'm fairly new to Puppet. Puppet version is 3.8.

Hi,
Apologies for taking over a year to get to this issue - hope you managed to get things sorted!
Admittedly dealing with pre-existing Varnish installations would be out-of-scope for the module, my advice would be to manually uninstall any old version of Varnish before running Puppet.
That said, if there are modifications you'd like to make to the module, please do raise a Pull Request and we'll review them :)
Thanks,
Craig


False positive when named arguments in function call enclosed in quotes

CFLint reports a MISSING_SEMI, exactly End of statement(;) expected instead of ) for the following line of code:
foo.bar("argNameOne" = argValueOne, "argNameTwo" = argValueTwo);

Seems to be triggered by the argument names enclosed in quotes, which to the best of my knowledge is valid ColdFusion code.

fixed in cfparser.


consistency with work

simple but not visible.
i am using windows 10 for office, windows 10 pro for my personal and other if i set something default setting for my office WSL and other same settings is not work on other no meter same id is used for all

I'm sorry? What?

Maybe this has to do with us not roaming settings? Like they want the settings to roam from their work PC to their personal one?


How to setup Newtonsoft.Json-for-Unity Using Burst Compiler and Jobs Package that uses NewtonSoft<EMAIL_ADDRESS>

<EMAIL_ADDRESS>Hey so Unity 2020.1.15f1 is using this package, I guess it's their internal package, and it basically causes some dll issue, when you delete one of them, it doesn't really build.
Mind you I am very new to unity not sure how this would be resolved.
I want to use both the packages, since I was using them before, I introduced the ... burst compiler and jobs package thus breaking most of my code.
These are the two I am looking to use:
https://github.com/jilleJr/Newtonsoft.Json-for-Unity
https://github.com/jilleJr/Newtonsoft.Json-for-Unity.Converters
Could you explain to me how I would get this to work with alongside with
<EMAIL_ADDRESS>

Hello @ArEnSc! Thanks for reporting this. I've yet to updated the documentation on the README.md's on the different repos.
Unity's package, the<EMAIL_ADDRESS>one, is actually this package but I think version 12.0.201 or something. I'm trying to talk this over with Unity so we could try find a solution where either I redirect my users to their or the other way around, because this mix is not that healthy atm.
Given that though, you can safely rely Unity's Newtonsoft.Json package instead, especially since the Burst compiler/jobs packages seem to indirectly reference it.
The converters package, https://github.com/jilleJr/Newtonsoft.Json-for-Unity.Converters, doesn't actually have a hard dependency on the package from this repo, https://github.com/jilleJr/Newtonsoft.Json-for-Unity. You can mix the "Newtonsoft.Json" DLLs however you want. Being using the converters package together with raw DLLs you've added into your Assets directory, using SaladLab's package https://github.com/SaladLab/Json.Net.Unity3D, using Unity's own package<EMAIL_ADDRESS>or using this repo. All should work fine.
So to resolve your dependency issues, go into the YourProject/Packages/manifest.json file and replace the package reference of this repo with the Unity one, essentially doing this:
diff --git a/Packages/manifest.json b/Packages/manifest.json
index 49a3afa..f0edd27 100644
--- a/Packages/manifest.json
+++ b/Packages/manifest.json
@@ -18,7 +18,7 @@
     "com.unity.collab-proxy": "1.2.16",
     "com.unity.test-framework": "1.1.22",
     "com.unity.ugui": "1.0.0",
-    "jillejr.newtonsoft.json-for-unity": "13.0.102",
+    "com.unity.nuget.newtonsoft-json": "2.0.0-preview",
     "jillejr.newtonsoft.json-for-unity.converters": "1.0.0",
     "com.unity.modules.ai": "1.0.0",
     "com.unity.modules.androidjni": "1.0.0",

Hope that resolves your issue!

Thanks!, I am wondering why they just don't adopt your library since it solves all the problems more or less with serialization in unity

They kind of have already. If you get the 2.0.0 version of the package and check the FileVersions of the DLLs, they've actually just taken the exact same DLLs from my 12.0.301 release, and in the Third Party Notices.md they've correctly stated that it's based on work by both me and the other that I've referenced in my THIRD_PARTY_NOTICES.md.
They have just adopted this package. They actually asked first in an email, which is really kind of them. I'm hosting it as OSS just so other can take it and host it elsewhere.
The big question that I'm working on resolving with them is why we still have two different repos for it still. One hosted by Unity and one here at GitHub. There is a plethora of ideas swimming around, and nothing finalized, but hopefully 2021 will be a year of good Newtonsoft.Json progress for making these kinds of descisions. Would be grand if it was still hosted as OSS, perhaps over at https://github.com/Unity-Technologies, and then I can just archive this repo and refer to the officially hosted repo.
Want to restate that nothing is decided yet. We'll see how this turns out. So far, Unity has taken small steps in this because they have very limited resources on this topic. But as more and more users and customers of Unity is starting to complain, maybe they will find some time for this.

kk ill give your fix a shot thank you for the quick replies!


Just a heads up on a new spoofing add-on

https://github.com/snapper26/shapeshifter
https://addons.mozilla.org/en-US/firefox/addon/shape-shifter/
According to the .js user agents shouldn't be spoofed, but I found this add-on that does a bit more than that and I just wanted to know your thoughts on it. Thanks.

Thanks @lazerstrike
Generates randomised values ... No configuration needed
Several issues with this (haven't had time to look at anything)

No control over the spoofing list which can lead to site breakage. That said, if it limits itself to
Firefox, that may be OK.
resistFP will probably override this (FF56)
does this cover buildID, because even though the spoof is random, if the buildID doesn't match a one of those for that version & OS, then it sticks out
I am firmly in the camp that it is far better, as per TBB's draft, where possible, to lower entropy vs randomly-always raising it
does this pass the test for all the navigator leaks and over iframes etc
more to come, watch this space --> <-- this one

The real browser is revealed over iFrame method. So no use of shapeshifter... yet.

Hi there
I am the author of this addon. Thank you for your feedback on this matter. I had a sneaking suspicion that  would cause a leakage however it is impossible to catch all problems at once. I have just released this addon and have plans to fix the issues mentioned above with  as well as another issue with CSP (Content Security Policy) which causes one of my injected scripts to be blocked from being injected causing the spoofing to fail. I try to normalise things like the navigator to common values in order to blend in.
As for buildID, I have only spoofed properties that appear under the standard list here https://developer.mozilla.org/en-US/docs/Web/API/Navigator so far but am open to spoofing anyother properties that need to be spoofed. As for spoofing other things such as batterymanager, canvas, webgl, audiocontext and client rectangles, the reason is because I believe them to be fingerprinting vectors that if left untouched could render the addon pointless. It is hard to give something like a canvas image a "default" value (maybe transparent black?) so what I do is generate a seed for each session and each different domain/origin which is used to create a unique ()but consistent) canvas for that session, it then changes when the session changes.

Hey @Snapper26 : ) Welcome
What I wrote above was a quick 2 minutes of thought off the top of my head, followed by some edited points after I had a quick look at the repo readme. Nothing is meant to be be dismissive, just ideas for discussion / follow-up. Rome wasn't built in a day :)
Some random thoughts below...

As for buildID, I have only spoofed properties that appear under the standard list...

Right. So the buildID leaks which gives away your exact version and OS. I've have to check now, but I think Tor Uplift just blanks it (there's actually a FF pref that does this), but that means you are now a KNOWN spoofer. Tor Uplift doesn't care because they just want uniformity (enforced). If 1% of FF's 100 million users turn on resistFingerprinting (there will be a UI option for it at some stage, and I expect it to even be added to PB mode windows down the track), then that is 1 million users. That's a good start.
Sticking to UA only, I think there are too many holes and overlap. That said, those holes may or may not exist (eg tor uplift navigator spoof may close some of these but the vast bulk of browser IDing remains - eg feature detect, eg CSS line height can detect OS, scrollbars can do the same) whether you spoof or not. If you're always rapidly changing then who cares....  there's no right or wrong answer - except the value of being a spoofer or not is itself a value - i.e if you are caught out in a lie then you are unique, otherwise they do not know you spoofed. I call this creating a problem that didn't exist, or making it more complex :)
Tor Uplift in FF56+ enforces spoofed values for UA and for items relating to UA. Have you checked if your extension overrides theirs - I will assume it does, as script injection should come last, right?. If it doesn't then that's a major problem.

I do not think referers should be in this AT ALL, and people can already use extensions such as uMatrix spoof all, or dedicated web ext such as smart referer etc - bit of a transition going on between legacy and WebExt, so its a bit messy. Re-inventing the wheel and making your addon more complex.
For this reason alone - interfering with referers and giving me no control - I would not use your extension: i.e granular control is the best solution, and this extension screws that up, so to speak. At the very least options down the track to turn features on and off.

Long story short .. I am not an expert, anti-FPing is really really hard, there are often no right or wrong answers... the Web Ext is at initial release and will prototype as we go ... good to have you around :)

As of 12 August 2017 shapeshifter uses the value "Mozilla/5.0 (Windows NT 6.1; rv:52.0) Gecko/20100101 Firefox/52.0" for navigator.userAgent which means that the build ID should be "20100101"? If so then I have fixed this today and made navigator.buildID equal to "20100101". If the user agent changes then this might need to change too. As for the referrer, sometimes it is good to get usability feedback which can help guide you in the right direction. I would like to in the not too distant future add a proper HTML pop up GUI which more detailed configuration options. At the moment I am just going to focus on fixing the issues brought up above such as buildID, iframes and the CSP script injection issue I am facing.

navigator.buildID -> http://browserspy.dk/showprop.php -> mine is<PHONE_NUMBER>3124 (I am not hiding it) => windows FF55 time stamp (seems weird its naming convention indicates august 3rd)
general.buildID.override overrides this in FF and the consensus has always been to use 20100101 (because the pref landed in 2010, you see). This is a simple test to see if someone is faking their agent :)
A more comprehensive approach (and I'm not saying to do it) is to incorporate the actual real values into every UA used. This adds complexity and overhead and time wasting ... not that I don't applaud such efforts :)  If you spoof as Linux version 54 language UK - then the build id should match linux 54 (and language could be English-UK, timestamp could be UK, date formats should be uk not that backwards-assed american format, and so on). That said, we already know you;'re spoofing, so big deal - its randomized frequently - the plausibility matters way less than the values, in fact plausibility might even be zero - I wouldn't get too hung on this aspect :)
If I think of things I will bring them up in your repo - I'm already a watcher

So if 20100101 gives me away as trying to hide, then that is a fingerprint in itself. I know you didn't say to do it, however it might be worth the extra bit of effort to use the real buildID for each release. This would make it much more believable. Also I have already spend lots of time making this extension so yeah.

If I think of things I will bring them up in your repo - I'm already a watcher

This would be greatly appreciated as sometimes I get so deep in development that I miss the simple things.

Leave it so, most UA Strings contain it. A partial list is here and a huge one here.
Also recent Pale-Moon and K-Meleon give 20100101

Leave it so, most UA Strings contain it. A partial list is here and a huge one here
.
Also recent Pale-Moon and K-Meleon give 20100101

Most UA Strings contain it. A partial list is here and a huge one here.
Also recent Pale-Moon and K-Meleon give 20100101

Yep so I probably should just use 20100101 then? Also @crssi you mentioned about iframe.contentWindow still revealing the real browser attributes. Shape Shifter is a web extension which means that I can't use the obsolete/deprecated preferences service to set say general.useragent.override. This makes spoofing in iframe's quite tricky unless someone knows an easy way that I don't :)

Closing this. Am adding the extension to out addon sticky, so we do not forget about it.


Typo (Learn -> Train)

Coverage remained the same at 99.764% when pulling cc018422a65e486596f15c1ae2067855b244f3b6 on bzaar:patch-1 into 0fccf53f711edbaa75c847197cfd1184d5418db1 on Alexander-Ignatyev:master.

Thanks!


Command plt service init not found

As by title, the service init command is not found, even though this command is mentioned on the documentation:

When you run platformatic service init, a new JSON $schema property is added in platformatic.service.json

Running npm create platformatic@latest instead, allows generating the initial config for a Platformatic service.

This should be removed from the docs, folks should just use create-platformatic.

Done on https://github.com/platformatic/platformatic/pull/767


train downstream ASR using own upstream

Hi, I want to use the pertained model for downstream ASR task, however in the s3prl/downstream/asr/feat/ directory, there is no config file, is the ASR task properly configured? Thanks.

Hey, I'm facing a similar issue in using asr for downstream. can someone please help me with this?

Hey, I'm facing a similar issue in using asr for downstream. can someone please help me with this?

Hi,
Since ASR is a complicated system including LM training and beam search decoding during testing. Instead of reinventing the wheel on this repo, we decided to incorporate s3prl pretrained models into the framework of an existing well-organized ASR repo Alexander-H-Liu/End-to-end-ASR-Pytorch, which is created by a senior member from our lab. Currently we create a fork from that repo to support the pretrained models: s3prl/End-to-end-ASR-Pytorch. We modified several components to enable finetuning:

Delay the feature extraction part from dataloader to training loop, and accelerate it by GPU either for fbank extraction or pretrained feature extraction.
Support distributed data parallel to enable large batch size finetuning, since some pretrained models can be huge like wav2vec2

One can refer to the README in s3prl/End-to-end-ASR-Pytorch for more details and hope this answer your question!
Thanks!
Sincerely,
Leo

Hi,
Since ASR is a complicated system including LM training and beam search decoding during testing. Instead of reinventing the wheel on this repo, we decided to incorporate s3prl pretrained models into the framework of an existing well-organized ASR repo Alexander-H-Liu/End-to-end-ASR-Pytorch, which is created by a senior member from our lab. Currently we create a fork from that repo to support the pretrained models: s3prl/End-to-end-ASR-Pytorch. We modified several components to enable finetuning:

Delay the feature extraction part from dataloader to training loop, and accelerate it by GPU either for fbank extraction or pretrained feature extraction.
Support distributed data parallel to enable large batch size finetuning, since some pretrained models can be huge like wav2vec2

One can refer to the README in s3prl/End-to-end-ASR-Pytorch for more details and hope this answer your question!
Thanks!
Sincerely,
Leo

Hi Leo, thanks for the reply. I read the End-to-end-ASR-Pytorch project, but I do not fully understand how to link up pertaining with the ASR task. I guess the pertained model is slot in before step 2 training the end-to-end ASR? Will there be a detailed procedure?

Hi Leo, thanks for the reply. I read the End-to-end-ASR-Pytorch project, but I do not fully understand how to link up pertaining with the ASR task. I guess the pertained model is slot in before step 2 training the end-to-end ASR? Will there be a detailed procedure?

Hi,
The pretrained model is plugged at here  in s3prl/End-to-end-ASR-Pytorch. The pipeline is like:

The dataloader prepares batches of waveforms.
The pretrained upstream model extracts features from the input waveforms.
The downstream ASR model uses pretrained features as input to train CTC or LAS loss (or hybrid).

There might not be detailed procedure in one or two months, but for ASR since the doc is quite comprehensive in the original repo, so it might work for you after some code tracing. You can search for Finetune pretrained models with multiple GPUs in the README.md of s3prl/End-to-end-ASR-Pytorch, that might be the most relevant part for you.
Sincerely,
Leo

Hi,
The pretrained model is plugged at here  in s3prl/End-to-end-ASR-Pytorch. The pipeline is like:

The dataloader prepares batches of waveforms.
The pretrained upstream model extracts features from the input waveforms.
The downstream ASR model uses pretrained features as input to train CTC or LAS loss (or hybrid).

There might not be detailed procedure in one or two months, but for ASR since the doc is quite comprehensive in the original repo, so it might work for you after some code tracing. You can search for Finetune pretrained models with multiple GPUs in the README.md of s3prl/End-to-end-ASR-Pytorch, that might be the most relevant part for you.
Sincerely,
Leo

Thanks Leo, I see the upsteam model is loaded withupstream = torch.hub.load('s3prl/s3prl', 'tera')

Oh I get it! If you wish to use your own pretrained model, indeed you

Hi!
This is a good point! I haven't thought about this situation. This indeed shall be improved, to make the customized pretrained models easily evaluated. I will dive into this and thank you for the insightful comments. This is a must have feature.
Sincerely,
Leo

Thanks Leo, or maybe you can share how you save the pretrained model to torch.hub

Hi,
Actually it is quite straight forward!
Here is the reference: https://pytorch.org/docs/stable/hub.html

Hi Leo, I tried write hubconf.py to upload the self pertained model, but I do not know how should I initialise the model. Below is the hubconf.py I write, but it has the error of "NameError: name 'model' is not defined". Should I initialise it from the class runner in s3prl/pretrain/runner.py?
import os
import torch
dependencies = ['torch']
def mkj(*args, **kwargs):
"""
pretrained libri100 mockingjay
"""
dirname = os.path.dirname(file)
checkpoint = os.path.join(dirname, 'result/pretrain/mkj_100/states-epoch-100.ckpt')
state_dict = torch.load(checkpoint)
model.load_state_dict(state_dict)
return model

Hey,
Are you also using this toolkit for the pretraining? If this is true, then there might be a simpler solution.
In our ASR repo you can try using arguments like:
python3 main.py --config config/libri/finetune.yaml --upstream mockingjay_local --upstream_ckpt [your path]

Hi Leo, I am not too sure where is the main.py file located.
I try to use the pertained model for another ASR toolkit. I load the pretrained model using " torch.load(ckpt_pth, map_location='cpu') ". However, there is always an error of
_pickle.UnpicklingError: A load persistent id instruction was encountered,
but no persistent_load function was specified.
I run shell file to call python function, is this the reason?

Hi,
Could you fork that repo and make your change visible on GitHub? (create a minimal shell script to reproduce your issue will be nice.)
Perhaps I need more information to understand the situation.
Thanks!

We are including downstream ASR into s3prl currently on asr branch, so I am closing this issue. It will be soon merged into master, thank you for your patience.


A way to click in gui

Some servers u need to click i  gui's to teleport n stuff ops this is not issue

In my opinion there is no module in mineflayer to do it


author is not available

p = Axlsx::Package.new(:author => 'Randy')
How can read an author name ? It seems there is no other way than
p.instance_variable_get('@core').creator
Would ne nice to have 'author' available on Package or Workbook or both.

Thanks Ochko that is a great suggestion.
I'll add accessors for app and core members to the package in the next release.
But I think you will still need to do
p.core.creator to read/write that value.


BigQuery Data Transfer: ListDataSourcesRequest not documented / not linked to

See: https://googlecloudplatform.github.io/google-cloud-java/latest/apidocs/com/google/cloud/bigquery/datatransfer/v1/DataTransferServiceClient.html#listDataSources-com.google.cloud.bigquery.datatransfer.v1.ListDataSourcesRequest-
I would expect com.google.cloud.bigquery.datatransfer.v1.ListDataSourcesRequest to be a hyperlink to the ListDataSourcesRequest class, but it is not. A search within that page for ListDataSourcesRequest does not show any documentation for that class.

Many other classes are also undocumented. com.google.cloud.bigquery.datatransfer.v1.DataSource is another example of a class that is referenced but not documented.
Possibly these are all the underlying Protocol Buffer resource classes?

Root cause should have been fixed by #2862 and take effect the next release. I'll resolve this for now.


File not exists:  VOT2019/*/motion_change.tag

when i download VOT2019, I got this issues, Why?

i meet the same question,  there is motion_change.label in my downloaded VOT2019 file.
how do u solve it?


[New package version]: Microsoft.ServiceFabricRuntime version 8.2.1363.9590

Package Requested

[X] I would like someone else to build the manifest.
[X] I would like help so I can submit the manifest.
[X] I have performed a search and couldn't find this package.
[X] I have checked that the installer for this package is not a standalone or portable executable or a .zip file
[ ] I think there is a new version available and I have provided the URL.

Please provide the following information
Current Package Identifier: Microsoft.ServiceFabricRuntime
Publisher: Microsoft Corporation
Package Name: Microsoft Azure Service Fabric
Package Version: 8.2.1363.9590
Package Url: https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-get-started
Not the correct download URL: https://www.microsoft.com/web/handlers/webpi.ashx?command=getinstallerredirect&appid=MicrosoftAzure-ServiceFabric-CoreSDK
Since the package is missing Metadata i'm not sure where to get find the download url

Looking for something like https://download.microsoft.com/download/b/8/a/b8a2fb98-0ec1-41e5-be98-9d8b5abf7856/MicrosoftServiceFabric.8.0.521.9590.exe

https://download.microsoft.com/download/b/8/a/b8a2fb98-0ec1-41e5-be98-9d8b5abf7856/MicrosoftServiceFabric.8.2.1363.9590.exe
Changelogs as well as Download URLs are available at https://github.com/microsoft/service-fabric/tree/master/release_notes

Ahaaa, i think i found the github page, but didn't look at the release notes..
Ty, however the installer is failing with exit code 1 for me


Swipe to dismiss on mobile

I have been playing with Sonner for some time and it's amazing. One thing i have noticed is that i frequently attempt to swipe the toast off the screen to dismiss it on mobile. It seems like a thing an average user would expect so, for a better ux, it should be implemented as a feature of Sonner.

Wdym? You can swipe it down to dismiss on mobile.

Wdym? You can swipe it down to dismiss on mobile.

Oh, I have tested it on the docs site and it doesn't seem to be working on Safari (iOS)

Wdym? You can swipe it down to dismiss on mobile.

Oh, i didn't know that, i thought i was supposed to swipe it to some horizontal side (left/right) to dismiss it. Just like the fact that most notification stacks are dismissible by side unless there's a single popup notification.
Looks amazing though!


Link to documentation is not valid anymore

Link http://pixate.github.io/pixate-freestyle-ios/ is not working anymore. Any plans to re-publish it somewhere?

Well, the likelihood of hearing back from the developers seems slim.
Here's a mirror of the docs: https://uzegonemad.github.io/pixate-freestyle-ios/

Please bring back the object documentation. Without it, I have to depend on sample code to figure out what attribute is working. Thanks.

We restored the docs, but it looks like the Developer Reference is still down. We're working on this now.

OK, the dev documentation has been restored: http://pixate.github.io/pixate-freestyle-ios//dev-reference.html.
Let us know if you see any more issues there.

Thanks Andrew.
On Fri, Oct 9, 2015 at 12:35 PM, Andrew Holt<EMAIL_ADDRESS>wrote:

Closed #183 https://github.com/Pixate/pixate-freestyle-ios/issues/183.
—
Reply to this email directly or view it on GitHub
https://github.com/Pixate/pixate-freestyle-ios/issues/183#event-431765238
.


⚠ Bofrost.de has degraded performance

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

HTTP code: 200
Response time: 1393 ms

Resolved: Bofrost.de performance has improved in 68c69a6.


Some test cases time out on Intel-IoT-Refkit image

Description
I tried to run the tests of this repository on Intel-IoT-Refkit image but some tests always time out. It seems these test cases are OCF C/S-communication-related. Adding the timeout  setting to 2 times / 3 times of the default value doesn't help. Ports 5683/5684 for IPv4/IPv6 were opened.
No such problem on Ubuntu.
Steps to reproduce:

Flash and boot Intel-IoT-Refkit gateway image build#211 on a Joule 570x device.
Find the iotivity-node vesion built in the image and copy the related version tests and dependencies to Joule.
Run  ./node_modules/grunt-cli/bin/grunt test --force .
See the iotivity-node testing log.

Environment:

Image node version: v6.11.0
Image iotivity-node version: v1.2.1-1
iptables settings:

root@intel-corei7-64:~# iptables -S
-P INPUT ACCEPT
-P FORWARD ACCEPT
-P OUTPUT ACCEPT
-A INPUT -p udp -m udp --dport 5683 -j ACCEPT
-A INPUT -p udp -m udp --dport 5684 -j ACCEPT
-A INPUT -p tcp -m tcp --dport 8000 -j ACCEPT
root@intel-corei7-64:~# ip6tables -S
-P INPUT ACCEPT
-P FORWARD ACCEPT
-P OUTPUT ACCEPT
-A INPUT -s fe80::/10 -p udp -m udp --dport 5683 -j ACCEPT
-A INPUT -s fe80::/10 -p udp -m udp --dport 5684 -j ACCEPT
-A INPUT -s fe80::/10 -p udp -m udp --dport 32768:60999 -j ACCEPT
-A INPUT -p tcp -m tcp --dport 8000 -j ACCEPT

Last week Refkit moved into nftables-based firewalling. Try using nft for opening the ports.
Try creating this file (call it iotivity-node.nft) and copy it to /etc/firewall/services:
#!/usr/sbin/nft

table inet filter {
    include "zones.ruleset"
    chain iotivity-node-udp {
        ip6 protocol udp saddr fe80::/10 accept;
        ip protocol udp accept;
   }
}

add element inet filter udp_service_map { 5683, 5684, 32768-60999 : jump iotivity-node-udp };
add element inet filter tcp_service_map { 8000 : accept };

I didn't have a chance to try to file out, so it might very well contain errors. :-)
If you just want to open the ports temporarily (firewall config is fully replaced whenever a configuration change or a large network change happens), see how iotivity firewall settings are done in tests for example here: https://github.com/intel/intel-iot-refkit/blob/master/meta-iotqa/lib/oeqa/runtime/core/iotivity/base.py

See here for current Refkit firewall documentation: https://github.com/intel/intel-iot-refkit/blob/master/doc/security.rst#firewall-support

@ipuustin , this works! Thanks very much!
I added iotivity-node.nft and reboot the system. Then all the tests pass completely.

Since this is a firewall-related issue and fixed, close it.

Sorry, there was a bug. I forgot that we couldn't have ranges in map keys because of a known bug in nft (already fixed upstream but not released). Please use this file instead:
#!/usr/sbin/nft

table inet filter {
    include "zones.ruleset"
    chain iotivity-node {
        type filter hook input priority 0; policy accept;

        ip6 saddr fe80::/10 udp dport {5683-5684, 32768-60999} mark set $accept_packet;
        ip protocol udp udp dport {5683-5684, 32768-60999} mark set $accept_packet;
        tcp dport 8000 mark set $accept_packet;
    }
}

(For future reference, here's a bit faster version that we can take into use later when the range interval flag fix gets released in nftables 0.8):
#!/usr/sbin/nft

table inet filter {
    include "zones.ruleset"
    chain iotivity-node {
        ip6 saddr fe80::/10 accept;
        ip protocol udp accept;
   }
}

add element inet filter udp_service_map { 5683-5684 : jump iotivity-node };
add element inet filter udp_service_map { 32768-60999 : jump iotivity-node };
add element inet filter tcp_service_map { 8000 : accept };

Thanks! I updated the nft file.
Currently, the version of nft is 0.7 in the image. I'll use the faster version when 0.8 is ready in the image.


README: Recommend brew install stern from homebrew-core

As of https://github.com/Homebrew/homebrew-core/commit/7d44ae9f44d112cde586258a28b8bbed22b43f2a, this fork of stern, with its latest version, ships in Homebrew's core tap (on macOS and shortly on Linux).

@issyl0 Great! Thank you as always for maintaining Homebrew Formula. 🥳

wow! thank you man


add Remark

Codecov Report

Merging #24 (2d38f8a) into master (041e38d) will decrease coverage by 3.63%.
The diff coverage is 40.00%.

@@             Coverage Diff              @@
##             master      #24      +/-   ##
============================================
- Coverage     72.15%   68.51%   -3.64%
+ Complexity      399      392       -7
============================================
  Files            70       73       +3
  Lines          1232     1277      +45
  Branches        125      130       +5
============================================
- Hits            889      875      -14
- Misses          311      366      +55
- Partials         32       36       +4

Impacted Files
Coverage Δ

...java/seedu/address/logic/commands/EditCommand.java
61.76% <0.00%> (-35.26%)
:arrow_down:

...va/seedu/address/logic/commands/RemarkCommand.java
0.00% <0.00%> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
88.88% <0.00%> (-11.12%)
:arrow_down:

...eedu/address/logic/parser/RemarkCommandParser.java
0.00% <0.00%> (ø)

.../java/seedu/address/model/util/SampleDataUtil.java
20.00% <ø> (ø)

src/main/java/seedu/address/ui/PersonCard.java
0.00% <0.00%> (ø)

...c/main/java/seedu/address/model/person/Remark.java
62.50% <62.50%> (ø)

...a/seedu/address/logic/parser/AddCommandParser.java
100.00% <100.00%> (ø)

...ain/java/seedu/address/logic/parser/CliSyntax.java
85.71% <100.00%> (+2.38%)
:arrow_up:

...c/main/java/seedu/address/model/person/Person.java
97.82% <100.00%> (+0.20%)
:arrow_up:

... and 5 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


batchsize

Under which py file can set batchsize?
Thank you!

I'm not sure I understand the question. For insetGAN, we always optimize a single exemplar at a time.


Draft: AWS Batch tutorial

Codecov Report
Patch and project coverage have no change.

Comparison is base (d8e7d7d) 51.03% compared to head (d8a663d) 51.03%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #199   +/-   ##
=======================================
  Coverage   51.03%   51.03%
=======================================
  Files          30       30
  Lines        3656     3656
  Branches      502      502
=======================================
  Hits         1866     1866
  Misses       1732     1732
  Partials       58       58

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[BUG] Consider removing innerHTML usage

Expected Behavior
Firefox addons being accepted without warnings around Chart.js.
Current Behavior
Submitting an addon to the Firefox store gives:

⚠ Unsafe assignment to innerHTML
Warning: Due to both security....

When searching through master, I get one offending line:
https://github.com/chartjs/Chart.js/blob/75aa44eef677b790768262638df0de223fd01a56/src/platforms/platform.dom.js#L187-L205
At first glance this seems to be the same usage in the minified build.
Possible Solution
Unsure, but perhaps there's another way to do the same thing in that line?
Steps to Reproduce (for bugs)

Follow the Firefox addon submission wizard for an addon that includes Chart.min.js as a content_script

Context
Reviews of such addons tend to take longer, or the addon might even be rejected based on this.
Environment

Chart.js version: 2.7.3
Browser name and version: n/a
Link to your project: n/a

Thx for letting me know, and indicating that a PR is welcome. Will have a go at creating one some time soon.

Converting the whole innerHTML block as createElement, appendChild, setAttribute, etc. (as @benmccann suggested) will be a mess if the inline static style is not first converted to CSS classes and injected when initialized (fixing #5208 btw). But that means it would break even more projects using Chart.js within shadow DOM, so we first need to fix #5763 before considering removing this innerHTML.
@jeroenheijmans I may have a look on these issues, knowing pretty well that part of the code.

Okay, thank you for the ping @simonbrunel, I'll hold off for the moment then. Let me know if I can help (e.g. test).

@jeroenheijmans if Firefox doesn't complain about .cssText then what @benmccann suggested could be a viable solution since we will not have to convert the style to JS properties or CSS classes.

@jeroenheijmans can you build #5909 and verify if it fixes this issue?

Built that branch and verified my app keeps working as expected, and that the Mozilla add on store doesn't give any warnings anymore.
Thanks!


Reproducibility Issue

Hi,
I am currently trying to reproduce the MNIST-U, MNIST-N clustering experiments. However, for some reason, I am not able to reproduce the numbers in your paper. Some of the possible reasons are follows:
1) Dataset : Although you have kindly provided MNIST-U, MNIST-N datasets, those datasets lack the labels for evaluating the clustering accuracy. Hence I have created my own datasets (using your configuration). Maybe this could be the reason for the discrepency.
2) Hyperparameters (including architecture of the model) : In your paper, you have not explicitly provided the detailed hyperparameters for MNIST-U, MNIST-N experiments (sorry in advance if I have missed something). So I was guessing the default configuration you provided in the README, is the setting you used in the paper. Maybe my guess was wrong.
Or, possibily, the difference is due to some other reasons. Hence I would be grateful if you could provide the detailed configuration for MNIST-U and MNIST-N experiments.
Sincerely,
Joo Young Choi

Hi,
I think it is very likely that this issue is happening because of the discrepancy in the labels. For MNIST, we use the dataset from torchvision which also contains the semantic labels and works with the clustering_mnist.py file. You can get this mnist by these commands:
mnist_train = torchvision.datasets.MNIST('data/', train=True, download=True)
mnist_test = torchvision.datasets.MNIST('data/', train=False, download=True)
The command from Readme file, should give you the results for z_dim=2 and P8. However, for higher clustering accuracy, you can increase the z-dim to 32 by using –z-dim 32, and for better rotation prediction, you can use –groupconv 16.
Best,
Ali Nasiri

Thank you for the reply. Just to clarity some points,

I have been using z_dim = 32 groupconv=16.

I am little confused, but are you saying when clustering, you used mnist (without rotation and translation)? In my case, I used your dataset (MNIST-N and MNIST-U) and used the labels from MNIST provided by torchvision. (For testset, the order of the samples were same for your dataset and torchvision dataset).

However, according to my experiments, the clustering accuracy seems to vary a lot depending on the run. For MNIST-U, the average was around 67% with maximum of 72% and mimimum of 61%. Did you experience the similar statistics, or am I doing something wrong.

We get our best results on MNIST-U with z-dim=32 and groupconv=16, so your setting for these values are correct.
For clustering MNIST, you can just the labels from the MNIST provided by torchvision. I was saying the same thing.

The reason for the high variance in the results is using agglomerative clustering. You can use k-means instead by adding --clustering k-means to your clustering command. I modified the code so now the k-means is the default clustering method  for MNIST.
Also I modified the code for RandomFourierEmbedding2d in the src/models.py, I modified the code so that the weight and bias won't change during the training which could have been another source of the variation in the clusterin result. Please use this new code for training the model.
There is about 2 - 3 percent variation in the clustering result for the mnist, but it shouldn't be higher than that.


steinbock duplicates ROIs

Hi @jwindhager
as reported vie email and verified by me, steinbock occasionally duplicates ROIs from single .mcd files.

 Does that help?

There is another option:

Keep the current strategy, but begin with matching the longest file names

Thanks for looking into this! And that totally makes sense - sorry for not correctly reading the docs.
I think we definitely need option 2. The .txt filename should exactly match the .mcd filename + _{acquisition_id}. I experienced the issues when a broken acquisition was continued in a second file named {mcd_file}_v2.mcd.

Thinking about it again, I'm actually in favor of strategy 3. Strategy 2 would potentially raise a lot of issues in cases like the one described above, where additional infixes are added to the txt files. Thoughts?

Discussed offline, going for strategy 3 (to not rely on user-provided descriptions that may be prone to file name encoding issues)


Single quote in double-quoted string gives error

Before reporting a bug, please test the beta branch!
Bug description
When building from a schema with a single quote in a double quoted string, like this:
type Thing {
  id: Int!
  exists: Boolean @deprecated(reason: "A thing's existence cannot be proven")
}

I get errors like this:

Could not format because the source could not be parsed:

line 11, column 2 of .: Expected to find ')'.
   ╷
11 │  }
   │  ^
   ╵

If I remove the single quote it works as expected.
I expect the double-quoted string to allow a single quote character.
Specs
Artemis version: 7.3.2-beta

build.yaml:
targets:
  $default:
    sources:
      - lib/**
      - graphql/**
      - schema.graphql
    builders:
      artemis:
        options:
          scalar_mapping:
            - graphql_type: Int32
              dart_type: int
          schema_mapping:
            - output: lib/graphql_api.dart
              schema: schema.graphql
              queries_glob: graphql/*.graphql
              naming_scheme: pathedWithFields

Artemis output:
❯ flutter pub run build_runner build --delete-conflicting-outputs --verbose
[ +127 ms] executing: sysctl hw.optional.arm64
[  +22 ms] Exit code 1 from: sysctl hw.optional.arm64
[        ] sysctl: unknown oid 'hw.optional.arm64'
[   +8 ms] executing: [/usr/local/Caskroom/flutter/2.10.2/flutter/] git -c log.showSignature=false log -n 1 --pretty=format:%H
[  +20 ms] Exit code 0 from: git -c log.showSignature=false log -n 1 --pretty=format:%H
[        ] 7e9793dee1b85a243edd0e06cb1658e98b077561
[        ] executing: [/usr/local/Caskroom/flutter/2.10.2/flutter/] git tag --points-at 7e9793dee1b85a243edd0e06cb1658e98b077561
[ +112 ms] Exit code 0 from: git tag --points-at 7e9793dee1b85a243edd0e06cb1658e98b077561
[        ] 2.10.3
[  +10 ms] executing: [/usr/local/Caskroom/flutter/2.10.2/flutter/] git rev-parse --abbrev-ref --symbolic @{u}
[  +10 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic @{u}
[        ] origin/stable
[        ] executing: [/usr/local/Caskroom/flutter/2.10.2/flutter/] git ls-remote --get-url origin
[   +9 ms] Exit code 0 from: git ls-remote --get-url origin
[        ] https://github.com/flutter/flutter.git
[ +126 ms] executing: [/usr/local/Caskroom/flutter/2.10.2/flutter/] git rev-parse --abbrev-ref HEAD
[  +13 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[        ] stable
[   +7 ms] executing: sw_vers -productName
[  +23 ms] Exit code 0 from: sw_vers -productName
[        ] macOS
[        ] executing: sw_vers -productVersion
[  +18 ms] Exit code 0 from: sw_vers -productVersion
[        ] 12.2.1
[        ] executing: sw_vers -buildVersion
[  +19 ms] Exit code 0 from: sw_vers -buildVersion
[        ] 21D62
[  +81 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[   +6 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'WindowsUwpEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[  +56 ms] Artifact Instance of 'MaterialFonts' is not required, skipping update.
[        ] Artifact Instance of 'GradleWrapper' is not required, skipping update.
[        ] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[        ] Artifact Instance of 'FlutterSdk' is not required, skipping update.
[        ] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'WindowsUwpEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FontSubsetArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'PubDependencies' is not required, skipping update.
[ +138 ms] Using /usr/local/Caskroom/flutter/2.10.2/flutter/.pub-cache for the pub cache.
[        ] executing: /usr/local/Caskroom/flutter/2.10.2/flutter/bin/cache/dart-sdk/bin/dart __deprecated_pub run build_runner build --delete-conflicting-outputs --verbose
[INFO] Entrypoint:Generating build script...
[INFO] Entrypoint:Generating build script completed, took 550ms

[WARNING] build.fallback:
The package `artemis_quotes_bug` does not include some required sources in any of its targets (see their build.yaml file).
The missing sources are:
  - $package$
[INFO] BuildDefinition:Initializing inputs
[INFO] BuildDefinition:Reading cached asset graph...
[INFO] BuildDefinition:Reading cached asset graph completed, took 69ms

[INFO] BuildDefinition:Checking for updates since last build...
[INFO] BuildDefinition:Checking for updates since last build completed, took 792ms

[INFO] Build:Running build...
[FINE] artemis:artemis on lib/$lib$:-> Class
[FINE] artemis:artemis on lib/$lib$:┌ [TypeName(name:r'Thing'), TypeName(name:r'RootQuery')][RootQuery][null null] ()
[FINE] artemis:artemis on lib/$lib$:|   [TypeName(name:r'Thing'), TypeName(name:r'RootQuery')][RootQuery][ClassName(name:r'Thing') ClassPropertyName(name:r'thing')] -> Thing$RootQuery$Thing
[FINE] artemis:artemis on lib/$lib$:|   -> Class
[FINE] artemis:artemis on lib/$lib$:|   ┌ [TypeName(name:r'Thing'), TypeName(name:r'RootQuery'), ClassPropertyName(name:r'thing')][Thing][null null] ()
[FINE] artemis:artemis on lib/$lib$:|   |   [TypeName(name:r'Thing'), TypeName(name:r'RootQuery'), ClassPropertyName(name:r'thing')][Thing][ClassName(name:r'Int') ClassPropertyName(name:r'id')] -> int
[FINE] artemis:artemis on lib/$lib$:|   |   [TypeName(name:r'Thing'), TypeName(name:r'RootQuery'), ClassPropertyName(name:r'thing')][Thing][ClassName(name:r'Boolean') ClassPropertyName(name:r'exists')] -> bool?
[FINE] artemis:artemis on lib/$lib$:|   └ [TypeName(name:r'Thing'), TypeName(name:r'RootQuery'), ClassPropertyName(name:r'thing')][Thing][null null] ()
[FINE] artemis:artemis on lib/$lib$:|   <- Generated class Thing$RootQuery$Thing.
[FINE] artemis:artemis on lib/$lib$:└ [TypeName(name:r'Thing'), TypeName(name:r'RootQuery')][RootQuery][null null] ()
[FINE] artemis:artemis on lib/$lib$:<- Generated class Thing$RootQuery.
[WARNING] artemis:artemis on lib/$lib$:
Closure: (VariableDefinitionNode) => void from Function 'visitVariableDefinitionNode':.
[SEVERE] artemis:artemis on lib/$lib$:

Could not format because the source could not be parsed:

line 11, column 2 of .: Expected to find ')'.
   ╷
11 │  }
   │  ^
   ╵
line 7, column 44 of .: Expected to find ','.
  ╷
7 │ @Deprecated('A thing's existance cannot be proven')  bool? exists;
  │                                            ^^^^^^
  ╵
line 7, column 66 of .: Unterminated string literal.
  ╷
7 │ @Deprecated('A thing's existance cannot be proven')  bool? exists;
  │                                                                  ^
  ╵
line 7, column 50 of .: Expected to find ','.
  ╷
7 │ @Deprecated('A thing's existance cannot be proven')  bool? exists;
  │                                                  ^^^^^^^^^^^^^^^^^
  ╵
line 7, column 34 of .: Expected to find ','.
  ╷
7 │ @Deprecated('A thing's existance cannot be proven')  bool? exists;
  │                                  ^^^^^^
  ╵
line 7, column 22 of .: Expected to find ','.
  ╷
7 │ @Deprecated('A thing's existance cannot be proven')  bool? exists;
  │                      ^
  ╵
line 7, column 24 of .: Expected to find ','.
  ╷
7 │ @Deprecated('A thing's existance cannot be proven')  bool? exists;
  │                        ^^^^^^^^^
  ╵
line 7, column 41 of .: Expected to find ','.
  ╷
7 │ @Deprecated('A thing's existance cannot be proven')  bool? exists;
  │                                         ^^
  ╵
package:dart_style/src/dart_formatter.dart 144:7     DartFormatter.formatSource
package:dart_style/src/dart_formatter.dart 72:12     DartFormatter.format
package:artemis/generator/print_helpers.dart 423:26  specToString
package:artemis/generator/print_helpers.dart 441:16  writeLibraryDefinitionToBuffer
package:artemis/builder.dart 200:7                   GraphQLQueryBuilder.build

[INFO] Build:Running build completed, took 390ms

[INFO] Build:Caching finalized dependency graph...
[INFO] Build:Caching finalized dependency graph completed, took 37ms

[SEVERE] Build:
Failed after 441ms
[+4016 ms] "flutter run" took 4,259ms.
[   +6 ms] pub finished with exit code 1
[   +1 ms]
           #0      throwToolExit (package:flutter_tools/src/base/common.dart:10:3)
           #1      _DefaultPub.interactively (package:flutter_tools/src/dart/pub.dart:416:7)
           <asynchronous suspension>
           #2      PackagesForwardCommand.runCommand (package:flutter_tools/src/commands/packages.dart:251:5)
           <asynchronous suspension>
           #3      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1161:27)
           <asynchronous suspension>
           #4      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           #5      CommandRunner.runCommand (package:args/command_runner.dart:209:13)
           <asynchronous suspension>
           #6      FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:281:9)
           <asynchronous suspension>
           #7      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           #8      FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:229:5)
           <asynchronous suspension>
           #9      run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:9)
           <asynchronous suspension>
           #10     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           #11     main (package:flutter_tools/executable.dart:94:3)
           <asynchronous suspension>

[  +51 ms] ensureAnalyticsSent: 49ms
[   +1 ms] Running shutdown hooks
[        ] Shutdown hooks complete
[        ] exiting with code 1

~/artemis_quotes_bug master*
❯

GraphQL schema:
schema {
  query: RootQuery
}

type RootQuery {
  thing(id: Int!): Thing!
}

type Thing {
  id: Int!
  exists: Boolean @deprecated(reason: "A thing's existence cannot be proven")
}

GraphQL query:
query Thing($id: ID!) {
  thing(id: $id) {
    id
    exists
  }
}

@martinolsson fixed in 7.3.3-beta


Add more failure info to QueryCompletionEvent

We need to know the task ID and the machine on which the failure occurred. This helps with finding logs related to a failure, correlating machines with higher than average failure counts, etc.

Let me try this one.
@electrum
Can you be more specific about machine information? I'm not sure what kind of information you have in mind. If you referred to nodeInstanceId[1] in TaskInfo as machine, it should be easy to add it to QueryCompletionEvent.
[1] https://github.com/facebook/presto/blob/c244ebaa629638fc0035f38ddc1c07948dba5100/presto-main/src/main/java/com/facebook/presto/execution/TaskInfo.java#L54


Importing OpenBCI data

Any plan to read/stream OpenBCI data? I use that hardware... so I'd be interested in incorporating it. Perhaps something along the lines of the following project: https://github.com/OpenBCI/OpenBCI_Python  or ideally just incorporating that project in MNE.

hi @ncullen93, we have a pull request, https://github.com/mne-tools/mne-python/pull/2671, that aims to incorporate OpenBCI streaming into MNE. feel free to join the discussion there, and if you would like, you can help contribute to help finish. we can re-stimulate its progress.
there was an older discussion before https://github.com/mne-tools/mne-python/pull/2462 about this. the data file that comes from OpenBCI_Processing GUI is a modified csv. we decided that it would not be advantageous to support it within the package for a host of reasons.
I wrote a reader for converting the Processing Data for MNE here.
Let me know if you have any more questions about this. I would happy to answer them. If not, feel free to close this issue.

I appreciate the info and apologize for the repeat - should have looked harder. Will close this.

no worries :)

Hey when I try to run the "from mne_openbci import read_raw_openbci"
I get an error saying that mne is needed but I already have mne installed and activated.

This module tries to import _BaseRaw, which doesn't exist (anymore?) in MNE. Try changing this in lines 17 and 21 to BaseRaw and it should work. @teonbrooks you might want to update your module. Also, I don't really understand why you set np =  None and mne = None?

@d3christ that mne_openbci package is deprecated. are you trying to read data in real time or are you trying to the export csv file?

I wrote the package like three years and haven't used it since. both mne and openbci has changed significantly since then


how do i add support for ethereum?

thanks

Simply put, you can't with MPOS Core: https://github.com/MPOS/php-mpos/issues/2506#issuecomment-226103881
RPC Calls for Geth are completely different to the implementation in MPOS and hence won't work at all unless you do some core changes. I have no idea what these are nor do I think they could be added in a 'generic' enough way to not break other coins. So either find a pool operator willing to share these changes or mine on existing pools.


Host helm.elastic.co not found: 2(SERVFAIL)

Get https://helm.elastic.co/helm/filebeat/filebeat-7.4.1.tgz: dial tcp: lookup helm.elastic.co on <IP_ADDRESS>:53: server misbehaving

$ host helm.elastic.co
Host helm.elastic.co not found: 2(SERVFAIL)

Other domains like www.elastic.co and docker.elastic.co are working

Hi @morganchristiansson,
Elastic Helm repository is working fine.
➜  ~ host helm.elastic.co

helm.elastic.co is an alias for nginx-k8s.elstc.co.
nginx-k8s.elstc.co has address <IP_ADDRESS>
➜  ~ curl helm.elastic.co | head
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   187  100   187    0     0    710      0 --:--:-- --:--:-- --:--:--   711

<title>308 Permanent Redirect</title>

308 Permanent Redirect

nginx/1.13.7

You should check your DNS config.

Works for me now.


Helm RBAC roles scaffolded are not idempotent

Bug Report
What did you do?
Generate the Helm Sample using the script with a Kind cluster up and without a cluster running.
What did you expect to see?

I would expect to see more roles when the plugin is able to get a kubeconfig than when it is NOT.
I would expect to see in all scenarios the verbs instead of *

What did you see instead? Under which circumstances?

The roles "better" populated when the kubeconfig is not found  (no cluster up).
The roles with * instead of the verbs when the kubeconfig is found.

Environment
Operator type:
/language helm
Kubernetes cluster type:
Kind
$ operator-sdk version
b9bf0f436b0eddce80ec6f6c41278cb8e2a77684
Possible Solution

Remove the NewDiscoveryClientForConfig and keep it with an idempotent result.

Additional context
Code;
https://github.com/operator-framework/operator-sdk/blob/24d73f782af630f26d3600530fb9b46e6878fc57/internal/plugins/helm/v1/scaffolds/internal/templates/config/rbac/manager_role.go#L97-L103
Changes/diff :

c/c @joelanford

I would expect to see more roles when the plugin is able to get a kubeconfig than when it is NOT.

When there isn't a cluster available, the role generator will use the default ruleset, which is fairly broad in order to allow most charts that use built-in APIs to work without further modification.
When there is a cluster available, the role generator will template the default chart, lookup each of the resources in the generated manifest using the discovery API, and then generate a rule set that is tailored to those resources.
There is no expectation that the generated rule set from the manifest and lookup will be smaller or larger than the default rule set because some charts will have a small number of resources, and others will have a large number of resources.

I would expect to see in all scenarios the verbs instead of *

Is there a reason that an explicit set of verbs is better than *, which means all verbs?

In an ideal world, we would use the discovery API during sample generation, which would give us the more tailored rule set. The implication is that we would need a cluster available during the sanity tests, which would definitely increase the time it takes to setup (and therefore run) the sanity tests.
We'll have to decide if that's a trade-off we're willing to make.

Hi @joelanford
Thank you for your reply. Following some comments:
**Keep the NewDiscoveryClientForConfig  based on (see principle of least privilege) shows go against the strattgey adopted **
If we are following the principle of least privilege) why we would grant cluster-scoped permissions instead of the namespaced one?
Shows not concise and accurate the project tries to persuade the goal of following the principle of least privilege) for Helm RBAC roles when granting cluster-scope permissions (it shows an inconsistent strategy across the board. Or we will follow this principle and be ok to make harder users getting started with since they usually will face issues or we are permissive related to the permissions and make it easier for them. However, in POV we need a consistence strategy here. Otherwise, the idea that they need to be customized the RBAC permission according to their business is not clear enough.
NewDiscoveryClientForConfig is not precisely anyway and the tool asks for users check the RBAC. So, what is the real value on keeping it for users and us? How helpful this behaviour is for them?
Check the msg/output when it is used:

WARN[0000] The RBAC rules generated in config/rbac/role.yaml are based on the chart's default manifest. Some rules may be missing for resources that are only enabled with custom values, and some existing rules may be overly broad. Double check the rules generated in config/rbac/role.yaml to ensure they meet the operator's permission requirements

They need to customize their RBAC permissions anyway.
IHMO: I am as users and/or maintainer would prefer an idempotent behaviour and a consistency strategy across the project

Is there a reason that an explicit set of verbs is better than *, which means all verbs?

See: Replace Role verb "*" with a list of verb strings in generated files so the Role is compatible with OpenShift and Kubernetes. (#2175)

See the issue #671 which motivates us to replace * by the verbs.

It seems like we could do this, but is generally unrelated to this issue.

If we are following the principle of least privilege, why we would grant cluster-scoped permissions instead of the namespaced one?

We made a broader decision based on Kubebuilder defaults (and the fact that CRDs and webhooks are cluster-scoped objects) to default operator projects to be cluster-scoped. Given that decision, the principal of least privilege would direct us to give the operator the minimal set of permissions to enable it to do its job across the cluster.

What is the real value on keeping it for users and maintainers?

Users get actual RBAC rules tailored to their operator. While not perfect, it gets them MUCH closer than the default rules do AND it does it without giving the operator unnecessary privileges.

How helpful or not is this behaviour for both?

IMO, very helpful.

Is theNewDiscoveryClientForConfig bringing advantages that justify the cost of maintaining this behaviour?

IMO, yes.
This is one of the killer features of the Helm operator. It enables users to focus their time and effort of developing their helm chart, and then using the operator-sdk to package it into an operator with extremely low effort (to the point that some users may be building and packaging an operator entirely in a CI pipeline).

Lastly, users running operator-sdk create api in a script to automatically generate operators from charts would be broken. Therefore getting rid of the role generation would be a breaking change, so we can't remove it until at least 2.0 anyway.

Closing since we can't make this change right now.


Messages containing HTML entities such as   cause a MessageParserException

Parsing any message containing HTML entities fails in the function get_text_content_from_message:
https://github.com/SymphonyPlatformSolutions/symphony-api-client-python/blob/a15462a88ea21bc95c5ca28d92f5069c1a0d7369/symphony/bdk/core/service/message/message_parser.py#L24-L28
This issue is that difusedxml won't accept non-XML entities by default.
This is a particular issue because tables copy/pasted from Excel into a symphony chat render empty cells as
For example:

Arrives as:

                    /bilat-barc-dev
                    Bid
                    Mid
                    Ask

                    1.114
                    1.893
                    0.291

I noticed that the same table pasted into Symphony 1.5 does not contain   so this might (also) be a platform bug.

                    /bilat-barc-dev
                    Bid
                    Mid
                    Ask

                    1.114
                    1.893
                    0.291

                    2
                    1.265
                    1.304
                    1.727

Hello @jimbodunc! thanks for highlighting this issue, your PR has been merged.
Thanks for your contribution!

@jimbodunc we released a new BDK 2.0 version, your fix is now available on 2.0b5 version https://github.com/SymphonyPlatformSolutions/symphony-api-client-python/releases/tag/2.0b5

@symphony-mariacristina Great, thanks!


record and rerun failed cases

Hi - is it possible to use gradle config with your plugin to record and rerun failed test cases.
Thanks

This is not a current feature of this plugin.

Any plans to implement it?

No. But we are open to pull requests if your interested in adding it.


Error generating schema from the vscode debug protocol

The code below reproduces the problem:
def gen_debugger_protocol():
    import os.path

    json_file = os.path.join(os.path.dirname(__file__), 'debugProtocol.json')
    if not os.path.exists(json_file):
        import requests
        req = requests.get('https://raw.githubusercontent.com/Microsoft/vscode-debugadapter-node/master/debugProtocol.json')
        assert req.status_code == 200
        with open(json_file, 'wb') as stream:
            stream.write(req.content)

    import json
    import schemapi

    with open(json_file, 'rb') as json_contents:
        contents = json.loads(json_contents.read())
        api = schemapi.JSONSchema(contents)
        api.write_module('debug_protocol_schema')

if __name__ == '__main__':
    gen_debugger_protocol()

The error I have is:
  File "C:\tools\Miniconda\envs\tests_py36\lib\site-packages\schemapi\jsonschema.py", line 466, in write_module
    return utils.save_module(spec=self.source_tree(),
  File "C:\tools\Miniconda\envs\tests_py36\lib\site-packages\schemapi\jsonschema.py", line 413, in source_tree
    schema_content = template.render(cls=self, classes=classes)
  File "C:\tools\Miniconda\envs\tests_py36\lib\site-packages\jinja2\asyncsupport.py", line 76, in render
    return original_render(self, *args, **kwargs)
  File "C:\tools\Miniconda\envs\tests_py36\lib\site-packages\jinja2\environment.py", line 1008, in render
    return self.environment.handle_exception(exc_info, True)
  File "C:\tools\Miniconda\envs\tests_py36\lib\site-packages\jinja2\environment.py", line 780, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "C:\tools\Miniconda\envs\tests_py36\lib\site-packages\jinja2\_compat.py", line 37, in reraise
    raise value.with_traceback(tb)
  File "<template>", line 12, in top-level template code
  File "C:\tools\Miniconda\envs\tests_py36\lib\site-packages\schemapi\jsonschema.py", line 344, in object_code
    return trait_extractor.object_code()
  File "C:\tools\Miniconda\envs\tests_py36\lib\site-packages\schemapi\trait_extractors.py", line 191, in object_code
    for ref in self.schema['allOf']]
  File "C:\tools\Miniconda\envs\tests_py36\lib\site-packages\schemapi\trait_extractors.py", line 191, in <listcomp>
    for ref in self.schema['allOf']]
  File "C:\tools\Miniconda\envs\tests_py36\lib\site-packages\schemapi\jsonschema.py", line 333, in trait_code
    return trait_extractor.trait_code(**kwargs)
  File "C:\tools\Miniconda\envs\tests_py36\lib\site-packages\schemapi\trait_extractors.py", line 482, in trait_code
    in self.schema.wrapped_properties().items()}
  File "C:\tools\Miniconda\envs\tests_py36\lib\site-packages\schemapi\trait_extractors.py", line 481, in <dictcomp>
    trait_codes = {name: Variable(prop.trait_code) for (name, prop)
  File "C:\tools\Miniconda\envs\tests_py36\lib\site-packages\schemapi\jsonschema.py", line 336, in trait_code
    "keys {0}".format(tuple(self.schema.keys())))
ValueError: No recognized trait code for schema with keys ('type', 'description')

Hi – I'm planning on deprecating this version of schemapi, and updating to a version not based on traitlets (see https://github.com/altair-viz/altair/tree/master/tools/schemapi)
I didn't know anyone was using this... what's your use-case?

Thanks, makes sense.
For what it's worth, it was issues like this one that led me to abandon the traitlets-based approach in the end. I couldn't see any way to fix them given the constraints of traitlets and the assumptions of jsonschema.

But the updated version within altair works quite well, if you want to try it.


feat: added S3 built-in CDN/Edge support for tarball.

**Type:**feature
The following has been addressed in the PR:

There is a related issue?
This is part of verdaccio/monorepo#250, see it for details.
Unit or Functional tests are included in the PR
Yes, new unit tests was added.

Description:
This PR work along with PR https://github.com/verdaccio/monorepo/pull/249, enabled serving tarball via S3 built-in CDN/Edge integration (i.e. CloudFront for S3, DO CDN for digitalocean spaces).

@juanpicado I understand, the PR breaks the principle of isolate. I originally implemented it in s3 plugin only. But the convertDistRemoteToLocalTarballUrls method always overrides the tarball url from package json. I don't know the purpose for this method, since the overridden tarball url are the same in the package.json. I guess the intention of convertDistRemoteToLocalTarballUrls may be one of below,

Make it easier to switch deployment details without data migration (i.e. change url_prefix, domain name.)
Override upstream package json tarball to serve the package tarball locally.

To solve the convertDistRemoteToLocalTarballUrls issue, I propose an alternative

Check current store's managedTarballUrl property or disallowTarballUrlOverride or anything more make sense for you.
If true, skip the convertDistRemoteToLocalTarballUrls logic, and leave the tarball url no change.
If false, null or undefined, run convertDistRemoteToLocalTarballUrls as usual.

How about it?


Add CalendarKit

Project URL
https://github.com/richardtop/CalendarKit
Description
CalendarKit is a fully customizable calendar library written in Swift. It was designed to look similar to iOS Calendar app out-of-the-box, but allow complete customization when needed. To make modifications easy, CalendarKit is composed of multiple small modules. They can be used together, or on their own.
Why it should be included to awesome-ios (optional)
Because there is no such library here yet.
Checklist

[X] Only one project/change is in this pull request
[X] Addition in chronological order (bottom of category)
[X] Supports iOS 9 or later
[X] Supports Swift 3
[X] Has a commit from less than 2 years ago
[X] Has a clear README in English

          1 Error

:no_entry_sign:
Found 1 link issue

Link issue by awesome_bot

Line
Status
Link

1087
Error
https://www.tapglue.com  Connection refused - connect(2) for "www.tapglue.com" port 443

  Generated by :no_entry_sign: danger

Thx


Card brand "elo" no detected

The card of brand "elo" no is detected.
Examples of numbers this card find in https://gist.github.com/erikhenrique/5931368

Some intend to add support for brand "elo"?

this can help to add support for ELO brand
https://github.com/Fivell/credit_card_validations/commit/7ee9a809c5682760a363a334109846e3d5a0c37a

@darlanmendonca @reginaldojunior i think that would be nice if you could implement this, @jessepollak would be glad =D

right, I will make a pull request following logic and structure of @jessepollak with:

regex to idetify elo cards
draw of brand in css only

:+1:
When possible create a pull request'm kind of out of time now.

GREAT, SCOTT!!!
@darlanmendonca you had to modify this little buddy here https://github.com/jessepollak/payment/blob/master/src/payment.coffee

Guys i am try add this brand, but I don't know where in files scss to compile automatic.

Done!
:smile:

awesome @reginaldojunior... did you added your regex to payment.coffee?

The units tests did stay wrong I am try to fix. :bug:

Will be fixed in v1.2.0!


tests for _core_cuts

_core_cuts is a function in utils that ought to be tested

closed by https://github.com/rethinkpriorities/squigglepy/commit/3e6fd741fbd4b4c3635365fcf4743cd4406e5501


Alphanumeric sorting initial sort

Description
We're using the DataTable component and have a column that is using the 'alphanumeric' sort. Clicking the column header sorts the table correctly. Howver, on load, we have set this column as the initialSortColumn and the table is not initially sorted using the 'alphanumeric' sort, but using the 'basic' algo.
Steps to reproduce

Implement a new DataTable component. Adding columns, with one column using the 'alphanumeric' sort
Have data that has integers in the values, like var1, var11, var103, var2, var22, var203 etc.
On the DataTable component, set the initialSortColumn to the column that has the 'alphanumeric' sort set
On load the table will be sorted using a 'basic' algo. Clicking on the table column header will make it sort using the alphanumeric algo.

Version
latest
Browser
No response

Thanks for filing this bug!
@joshblack could you provide an estimate of size for this and we'll see if we can prioritize? I'm wondering if we might be able to fit this into an upcoming FR cycle for a PRC engineer.

Hi 👋 I hope I'm not treading on any toes here by opening a PR for this issue but, as I mentioned in the linked PR, I wanted to explore the code a bit and thought I'd grab an open issue as a starting point.
Thoughts and feedback on #3967 would be appreciated, thanks 🙂

Hello! I was interested in contributing to Primer and I have a fix for this issue on a local branch. It seems like I might need permission to push a branch, because I received a 403 response when I tried. If this is user error, feel free to let me know and I'll look into it.
If you aren't interested in a contribution, no worries!

👋 @joshblack , I see you reviewed a previous PR related to this issue so I wanted to let you know that I attempted to fix with  https://github.com/primer/react/pull/4602.

Hi there, thanks so much for your patience 🙏 I see @TylerJDev was tagged as reviewer on https://github.com/primer/react/pull/4602. We'll be follow up soon! Please feel free to nudge us again here if we haven't gotten back to you by next week.

Thanks @lesliecdubs !


Adding multi language support system✨

I want to add multi language support system to the website which can help us understand in their native language.
@yatulearn can you assign me this issue.

@yatulearn can you assign me this issue.


rectify images

perhaps use SurfRCaT from @conlin-matt... or get help from MC himself...

This is awesome! Which cameras are you trying to use? Some look like they view features that could be seen in airborne lidar (e.g. NC12 @ Buxton), others don't (e.g. NC12 @ Temporary Bridge Inlet). And what info are you trying to pull out of the rectified products?

every 10 minutes I am scraping:
Mirlo
Ocracoke
N Hatteras
Buxton
NewInlet
CanalZone
these sites tend to flood — so it would be nice to have estimates of  flood/overwash/washover sand deposit dimensions...

Here is an example of Mirlo flooding:

It woudl be nice to know how big that breach is....
ALSO! for Mirlo,  i have surveyed in many of the poles and signs in the image... (and can go out and do the same for any of the sites...).. but i haven;t spent the time learning to/doing the rectificaiton on the images... your tool seems to make that easy.
All of the cameras move around in the wind, adn sometimes get pointed in other direcitons (i.e., Mirlo ispointed South now, not north (as it as in the image here)

Lastly, I bet there are more traffic cameras like this in other states.... which supplements the surf cameras... and explicilty look at flooding/overwash in developed places (this is an ongoing project with @edlazarus)

Nice. Sorry, just looked at your code. Of the 6 cameras you are using currently, my assessment of SurfRCaT usability is:
Mirlo: Yes
Ocracoke: No
N. Hatteras (if this is the ferry dock): Yes
Buxton: Yes
New Inlet: Can't find this one?
Canal Zone: No
However, any camera that has points surveyed-in should be rectify-able. SurfRCaT won't work out of the box for surveyed-in points; SurfRCaT routines would need to be generalized a bit but this is something I've been meaning to do anyway.
I'd love to help out with this- I think it's super cool!

N Hatteras is a parking lot... here is a pic of it flooded

awesome, let me ping you over email and we can discuss..


Documentation about how to get a stock toolchain that works with this project

Now that #251 has landed, it would be great to update the README with instructions, on each platform, for acquiring a toolchain that works for this project.

Here's the first prebuilt stock toolchain that should work with this project: https://storage.googleapis.com/s4tf-kokoro-artifact-testing/prod/s4tf/toolchain-ubuntu-docker/dev-stock-ubuntu18.04/11/20201216-031245/swift-tensorflow-DEVELOPMENT-stock-ubuntu18.04.tar.gz
In the next week or two we'll start publishing toolchains and instructions that work on more platforms.

But is that information written down anywhere publicly, e.g. in the README?
On Wed, Dec 16, 2020 at 11:39 AM marcrasi<EMAIL_ADDRESS>wrote:

Here's the first prebuilt stock toolchain that should work with this
project:
https://storage.googleapis.com/s4tf-kokoro-artifact-testing/prod/s4tf/toolchain-ubuntu-docker/dev-stock-ubuntu18.04/11/20201216-031245/swift-tensorflow-DEVELOPMENT-stock-ubuntu18.04.tar.gz
In the next week or two we'll start publishing toolchains and instructions
that work on more platforms.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/borglab/SwiftFusion/issues/258#issuecomment-746895769,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAKYIKNX4O7E5FDNXNYWX3SVEEGJANCNFSM4U5EQCQA
.

--
-Dave

No, but it's not ready for that yet. I'm still working on automating builds of the toolchains and making sure that everything works with them, so I don't think they should be in the README for a bit longer.


ODP-1550 - Fixing Knox Log4j vulnerability (CVE-2019-17571) detected after ODP upgraded to <IP_ADDRESS>-2

ODP-1550 - Fixing Knox Log4j vulnerability (CVE-2019-17571) detected after ODP upgraded to <IP_ADDRESS>-2

creating another PR with Base ODP-<IP_ADDRESS>-2


Add retry mechanism to gateway client

Description
We want to retry a few times in the gateway client in case of spurious errors.
Changes:
See commits.
Types of changes

New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)

Testing
Requires testing: Yes
Did you write tests??: Yes
Documentation
If this requires a documentation update, did you add one? No, No

Made a few tiny adjustments (three lines). This is ready for review.

Rebased on develop. Needed to refactor a few things to work nicely with testsource.


FIX #113: service raise error when parameter unknown

Error thrown when :

unknown parameter on creation
unknown or not modifiable parameter on update
redesign AutosleepServiceInstanceServiceTest according to @gberche-orange showed last meeting....Hope I understood well... Please don't let him be misunderstood :musical_note:

What about adding an acceptance test for it?

Acceptance should not be a duplicate of unitary, shouldn't they?


Allow application serializer to serialize POJOs and POJAs

This is necessary since we're no longer using filenames for multiword formatting (e.g. /fixtures/some_contacts.js used to output some_contacts in the response JSON. This was a bad idea.)

#607 closes


HomeSlider Question

Might be a dumb question, but how do I remove the "container" class parent div from the HomeSlider element? I tried removing it from homeSlider.js in both the dist and src folders, but on page load it is still being included. I'm unsure of where the tree is determined in this setup. The "container" class is still a parent element of the "home-slider" class after making this change.
My goal is to have the HomeSlider element take up the width of the screen rather than being confined by the "container" class without making changes directly to the "container" class and throwing all the other page alignments off.

Hi,
As far as I tested, you have to modify the "theme/src/components/homeSlider.js" file to modify this page.


Marker crash after when new scene is loaded

When we load a new scene, we rebuild all the markers against that new scene and we end up with two cases:

a marker is attached to a style that exists in both the previously loaded and new scene,
the marker doesn't find a new style to be built against

Inconsistency can happen in both those cases; a style can be found with the same name but have  different resources, where we result in a bad access or, the marker wouldn't show if a style was available in the previous scene but not in the new one.
To resolve this inconsistency, the best solution seems to drop all markers whenever a scene gets reloaded, since those are in fact tightly coupled with the scene itself when they get created.

To add to @karimnaaji, we can document the behavior such that the user handles readding of any marker in the asyncSceneLoad/Update callback.


Add French audio files

Hello,
I just record and propose french audio files.

Sorry I'm so late but I have added them to the download page https://www.phpcaptcha.org/downloads/audio/securimage_audio-fr.zip and the WordPress plugin will now pull this set instead of the old one.
Thanks so much!


Hardset path for zpool

Hi,
It seems that in 90zfsbootmenu/module-setup.sh the path for zpool is set statically. This generates the following issue upon trying to generate an image with dracut:
ldd: /usr/bin/zpool: No such file or directory
I believe the fix should be the following (?):
116c116
<   if ldd /usr/bin/zpool | grep -qF 'libgcc_s.so'; then
---
>   if ldd $(whereis -b zpool | awk '{print $2;}') | grep -qF 'libgcc_s.so'; then

Best regards,
vhns

What distribution are you running and where is zpool on your system?

I am running gentoo. It's in /sbin/zpool.

This has been fixed in 2d03ff5 . We're entering a feature freeze this week and will be testing/tagging/building a new release later this month.


Upgrade bitcore-lib and explicitly increase bitcore-mnemonic

New releases of bitcore-lib and bitcore-mnemonic in combination with the version constraints in the package.json caused the following error:
Uncaught Error: More than one instance of bitcore-lib found. Please make sure to require bitcore-lib and check that submodules do not also include their own bitcore-lib dependency.
This change solves those issues.

Thanks for this @roderik, merging!

I will merge this in, update the RELEASE-NOTES etc and then republish on npm. :)

Version 2.5.4 is tagged and published to npm.

👍


Fix vocab_emb.rst

Job PR-108/2 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-108/2/index.html


Distribute bug- some destination wells display as mixed

In a user test Piul was taking Ingredient 1 and distributing it into wells that had two states: empty or Ingredient 2.
In the case where Ingredient 1 went into empty wells the side panel showed that Ingredient 1 was going into a well with Ingredient 1 in it.
In the case where Ingredient 1 was going into wells with Ingredient 2, the side panel showed that Ingredient 1 was going into a well that already contained Ingredients 1 & 2.
It seems to me like distribute was displaying the destination well state after the distribute, not their state before the distribute.

It happened again in Cindy's usertest in two different ways:

@IanLondon Feel free to edit this bug ticket to make more sense as to what went wrong technically

@howisthisnamenottakenyet Looks good to me! I put it in the upcoming sprint


Android 4.3 Cordova PDF doesn't load

I can't get a PDF to load on Android 4.3, in the app I have created using cordova, the screen loads blank, and I get the same issue if I use the default Samsung browser. However, loading the app and site on newer versions of Android the PDF loads fine.
I've tried to debug using the promises set up in your plugin, but I only get the following message in console, which is less than helpful

I'm using the latest files:
pdf.js
pdf.worker.js
compatibility.js
Below is the code I'm using to call pdf.js, I call it in a angular controller:
window.pdfDoc = PDFJS.getDocument(url);
        console.log(
            "URL: " + url + '\n' +
            "PDFJS: " + PDFJS + '\n' +
            "PDFElement: " + JSON.stringify(window.pdfDoc)
        );
        window.pdfDoc.then(function (pdfDoc_) {
            console.log("SUCCESS: " + pdfDoc_);
            pdfDoc = pdfDoc_;
            document.getElementById('page_count').textContent = pdfDoc.numPages;

            // Initial/first page rendering
            renderPage(pageNum);
        }, function(error) {
            console.log("ERROR: " + JSON.stringify(error));
        });

The site shows no errors, but the status of the promise stays at 0, which means it never resolves. We have searched the tickets here but found no one else having this issue.
Thanks

Solved at http://stackoverflow.com/a/34027997/1765767 . Closing as answered.


Private GitHub repo

I'm trying to use prebuild install with a private GitHub repo.
Is there any way we could have a similar idea where you can add your token to .prebuildrc?

I think for that to work, prebuild-install has to use the GitHub API (rather than the public release URL), which may involve a few roundtrips to find the right asset. Starting from:
GET /repos/:owner/:repo/releases/tags/:tag

Yep that's how I've implemented it currently.

Oh, that's great :) Is your implementation public?

I'll get back to you about that tomorrow.
My version is based on the following with some tweaks to use the GitHub API instead.
https://github.com/WilixLead/iohook/blob/master/install.js

My implementation is quite messy so I'll do a fork of this repo and do a proper version for a PR.

I have added #75


WorkItemLinks query - Cannot read property 'map' of undefined.

I was running into an issue with running a workitemlinks query to an on prem TFS  Version 14.0.24712.0. After looking around it looked like this shouldn't have been an issue (as far as the query went) and handled correctly. I went through and found the following object is returned for the workitemlinks query I was attempting to run:

The cause of the cannot map undefined is due to the following line of code that does the mapping for the returned query in the query.ts:
let workItemIds = result.workItems.map(val => val.id).slice(0, Math.min(200, result.workItems.length));
I was able to fix the code so the query I was attempting to execute would map correctly, but am unsure if there may be a way around this or whether this is just a bug.

I fixed the error in 2d73e865f4ae5cdd54a56c0d66a3540393bb613e.
However, more work will be needed to make the link output really good. For now I just resolve source and then target items in order.


check for widescreen image before we use it

We don't always have a widescreen image - this is to do a check before we use it to stop articles erroring.
Found from dotorg explosions./

Merging as it's affecting all wordpress articles.


Migrate more stuff

Closes #13998
Closes #13954
Doing both at once to save on testing time :)
CR
Sorry, but this has some major test changes. I found a lot of issues while writing my tests, and decided to fix the test, which led to fixing some bugs:

Duplicated tests
Skipping api keys in all tests :scream: Most likely as a hack because the fixture was wrong (user model api key did not match master api key)
Bug: Static notifications was not backwards compatible (importer failed if not present). The test fixture was incorrect.
A lot of fixtures were identical to the full export one because they deleted keys from the main hash, instead of the user/organization subhash

Acceptance tips
We have tests for low-level. So let's test at high level. For example, create a dataset synced via dropbox, move the user to a different cloud and check that the sync keeps working. Similar for connectors.

🚀 deploying!