TrainDataset.save default behaviour deletes everything

Description
TrainDataset.save deletes all files in a folder as default behaviour (i.e. overwrite = True by default). Providing "." as path deletes all files in cwd which can cause lots of trouble for someone unfamiliar with the code. Proposal would be to set default behaviour to overwrite = False.
To Reproduce
from gluonts.dataset.repository.datasets import get_dataset  # type: ignore
from pathlib import Path

dataset_name = "m4_hourly"
dataset = get_dataset(dataset_name, regenerate=False)
dataset.save(str("."))

get_dataset will already stores the dataset on disk and uses .save for it.
I think it makes sense to set overwrite to False, but set it to true when using get_dataset.
@nklingen Do you want to give this a try?


Compare (ghprbTargetBranch and ghprbSourceBranch) branches

The plug-in gives
ghprbSourceBranch
ghprbTargetBranch

I'm trying to do
git diff ${ghprbSourceBranch} ${ghprbTargetBranch}

and build fails with
fatal: ambiguous argument 'source_branch': unknown revision or path not in the working tree.

even though the branch names are correct.
Any idea of what I'm missing?

Did you look at the available parameters for that build?

Hmm, this is PR builder job, it has only ${sha} as param. Should be another params?
All I need from this PR builder is giving me list of changed files between two branches.

Sorry; I wanted to assess whether or not the Parameters were available at
build-time. Could you look at a recent build where you saw this problem and
click the "Parameters" link for that build and let me know which variables
(just the variables...not the values) you see for ghprb?

All below have values for recent failed build:

The values are correct.
Environment variables were following:
ghprbActualCommit	abae870fe305eb21403d2104313740b5ae9a9846
ghprbActualCommitAuthor	Zafar
ghprbActualCommitAuthorEmail<EMAIL_ADDRESS>ghprbPullAuthorEmail<EMAIL_ADDRESS>ghprbPullDescription	GitHub pull request #9 of commit abae870fe305eb21403d2104313740b5ae9a9846 automatically merged.
ghprbPullId	9
ghprbPullLink	https://my_git_repo/pull/9
ghprbPullTitle	FE only and then be
ghprbSourceBranch	fe-only
ghprbTargetBranch	master
ghprbTriggerAuthor	Me Me
ghprbTriggerAuthorEmail<EMAIL_ADDRESS>GIT_BRANCH	fe-only
sha1	origin/pr/9/merge
SHELL	/bin/bash
SHLVL	1
TERM	linux
USER	jenkins-slave
XFILESEARCHPATH	/usr/dt/app-defaults/%L/Dt

Beginning of the log are following:

00:00:00.002 GitHub pull request #9 of commit abae870fe305eb21403d2104313740b5ae9a9846 automatically merged.
00:00:00.050 Setting status of abae870fe305eb21403d2104313740b5ae9a9846 to PENDING with url http://my-ci.net/job/PR%20Builder/140/ and message: Merged build started.
00:00:00.353 [EnvInject] - Loading node environment variables.
00:00:00.382 Building remotely on my-ci.net (swarm) in workspace /home/jenkins-slave/workspace/PR Builder
00:00:00.418  > git rev-parse --is-inside-work-tree # timeout=10
00:00:00.423 Fetching changes from the remote Git repository
00:00:00.425  > git config remote.origin.url<EMAIL_ADDRESS># timeout=10
00:00:00.431 Pruning obsolete local branches
00:00:00.433 Fetching upstream changes from<EMAIL_ADDRESS>00:00:00.434  > git --version # timeout=10
00:00:00.438  > git fetch --tags --progress<EMAIL_ADDRESS>+refs/pull/*:refs/remotes/origin/pr/* --prune
00:00:01.887 Seen branch in repository origin/master
00:00:01.888 Seen branch in repository origin/pr/1/head
00:00:01.888 Seen branch in repository origin/pr/10/head
00:00:01.889 Seen branch in repository origin/pr/10/merge
00:00:01.889 Seen branch in repository origin/pr/2/head
00:00:01.889 Seen branch in repository origin/pr/3/head
00:00:01.890 Seen branch in repository origin/pr/4/head
00:00:01.890 Seen branch in repository origin/pr/5/head
00:00:01.891 Seen branch in repository origin/pr/6/head
00:00:01.891 Seen branch in repository origin/pr/7/head
00:00:01.891 Seen branch in repository origin/pr/8/head
00:00:01.892 Seen branch in repository origin/pr/8/merge
00:00:01.892 Seen branch in repository origin/pr/9/head
00:00:01.893 Seen branch in repository origin/pr/9/merge
00:00:01.895 Seen 29 remote branches
00:00:02.215 Checking out Revision aa74481b5e33ffcb5bb9ed6f18f45f0129e93682 (origin/master)
00:00:02.227  > git config core.sparsecheckout # timeout=10
00:00:02.233  > git checkout -f aa74481b5e33ffcb5bb9ed6f18f45f0129e93682
00:00:02.353  > git branch -a -v --no-abbrev # timeout=10
00:00:02.365  > git checkout -b master aa74481b5e33ffcb5bb9ed6f18f45f0129e93682
00:00:02.459  > git rev-list aa74481b5e33ffcb5bb9ed6f18f45f0129e93682 # timeout=10

Looks I'm missing something in between the steps. Mbe don't know how refer to branches

@benpatterson  any help really appreciated.

up
@benpatterson  any thoughts?

have the same question +1


Feature request: File generation editor improvements

[x] DISCUSS Support using association as scope field element in file generation editor - we should also take into account what we do in #640 as it provides an extension mechanism for what which element type we can add per generation type
[x] Use PureModelContextText for generation - see #640
[ ] Allow downloading generated files (and folders - using jszip - most likely)
[ ] Allow uploading files/folders for model loader
[ ] Inform users when an artifact generation type has no properties

Closing this in favor of https://github.com/finos/legend-studio/issues/783


fix interface being not carried out to final contract

This PR fixes the case where an interface referenced in the base contract was not being printed in the final contract, an alternate approach

#695


HEPMC3 missing in CMakeConfig file

With HEPMC3 being a required dependency, the cmake config needs to be updated to reflect this, so that the downstream packages that uses WCSim don't need to have included explicitly (unless they need it).
https://github.com/WCSim/WCSim/blob/5bf9a0bba53960643b49cbfe625e6c32b05e0874/cmake/WCSimRootConfig.cmake.in#L6C1-L6C16
We might need to be slightly clever about this, as for WCSimRoot build HEPMC3 is not needed.

Agreed that this would be very good!
On the other hand, it might just be WCSimFQTuner that needs to be a direct WCSim dependency (the others like fiTQun, TriggerApp, BONSAI, FLOWER, LEAF, etc. just need WCSimRoot). So if this turns out to be complicated to implement, it's probably not worth spending a lot of time trying

True, a possibility (a lot simpler) would be to have two config.cmake template (one for WCSim and one for WCSimRoot).
I will give it a try in the coming days then.

Thanks @guiguem !


Reversed order of tabs per request

Updating order of tabs per request in 2.5, 2.6 per request in Slack.

Thanks for the PR, @jtravee! I am having some difficulties with setting up the docs server locally in my environmental at the moment. I just wanted to check if anchored links work now.
For example, does visiting https://rancher.com/docs/rancher/v2.6/en/admin-settings/authentication/azure-ad/#3-set-required-permissions-for-rancher take users to the right place? Does it work on your end?

Yes, in my local test environment after the tab switch, this link works!


tinyobjloader: init at 1.0.6

Motivation for this change
I've also added the python bindings, which were a bit of a pain to package.
They require some headers from the tinyobj source, but simply adding this as a buildInput won't work.
Not sure if my solution is proper, very much open for feedback.
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 105543 run on x86_64-linux 1

3 packages marked as broken and skipped:

python37Packages.tinyobjloader-py
python38Packages.tinyobjloader-py
python39Packages.tinyobjloader-py

1 package built:

tinyobjloader

Result of nixpkgs-review pr 105543 run on x86_64-darwin 1

3 packages marked as broken and skipped:

python37Packages.tinyobjloader-py
python38Packages.tinyobjloader-py
python39Packages.tinyobjloader-py

Result of nixpkgs-review pr 105543 run on x86_64-linux 1

4 packages built:

python37Packages.tinyobjloader-py
python38Packages.tinyobjloader-py
python39Packages.tinyobjloader-py
tinyobjloader


Fix strip_large_tensor when tensor doesn't have raw_data

onnx.external_data_helper require raw_data field

/test
LGTM!
Also asking @take-cheeze for review 🙏

/test


fix(cherrypicker): fix failed to fetch private repo problem

When a conflict occurs in the process of cherrypicker pick PR, a fetch upstream operation is required, which will cause errors to the private repository before.

/merge

/rerere


Setup semantic release

Coverage remained the same at 100.0% when pulling ce30b234b2102def5648a221f847ae065a9f6597 on setup-semantic-release into b90207a7f54149f2b4a68e132cf917e19f14f0a0 on master.


[ISSUE #2931] Fix  Plugin hystrix bug

// Describe your PR here; eg. Fixes #issueNo

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

pls check ci？

This pr is coupled with the previous code and I'm going to close this pr and create a new one


Enforced arg prefixes (-) when matching flags.

Spent some time diagnosing the issue that turned out to be from Willie's code
ca9fe38
Here's a fix to it that will enforce the prefixes "--" on the flag values and prevent his false positives, though I can see you reverted his commit already.
Anyway, thought this might still be useful.  My scenario is reflected in the parse.test.ts unit test doesn\'t throw when 2nd char in value matches a flag char

Hey @mfreydl - you're too fast and too good 🏆
we've added integration tests to make sure we won't break something like this again. I've added this to the new flag validation PR that was merged in today
Thank you :D

Nice! oclif's been a great product for us - happy to help!


Added feature to convert to quarto presentation

The files and code in this pull request allows for conversion of pptx files to qmd files - quarto presentations (revealjs).
It also introduces a method to infer the number of columns of the presentation, and it organize the converted data into the founded number of columns.
The feature still have some errors, but gives a decent version of the presentation with up to three columns support style file is also added

Great work! Would you please add a few lines in README.md to explain the --qmd argument?
I have tried out this and everything looks fine. update the README and I will publish a new version.

Great work! Would you please add a few lines in README.md to explain the --qmd argument?
I have tried out this and everything looks fine. update the README and I will publish a new version.

Thank you for publishing this repositorie. It is very helpful. I updated the README.md file explaining the --qmd argument. I hope more people find it useful.
Best regards!

Thanks. version 1.7.1 published.


ARROW-15644: [C++][Gandiva] Implement Find_In_Set Function

Returns the first occurrence of str in strList where strList is a comma-delimited string. Returns null if either argument is null. Returns 0 if the first argument contains any commas.
For example, find_in_set('ab', 'abc,b,ab,c,def')
returns 3

@kou @pitrou Can you check and merge that PR? It is approved by Projjal and me

Hmm... Arrow has a proper List type, so why take a comma-delimited string?

Hmm... Arrow has a proper List type, so why take a comma-delimited string?

@pitrou there are two things:

Currently, the Gandiva project does not accept complex types(we need to plan to add them in future)
That function we got from the Apache Hive project and the signature is a comma-separated string: https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=27362046

In the future, after adding the support for complex types we can add support for it using lists


amplify push vs git push

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting questions.
[X] I have removed any sensitive information from my code snippets and submission.

App Id
d2kf2ltunv8xfd
AWS Region
eu-west-1
Amplify Hosting feature
Backend builds
Question
I am using AWS CodeCommit as git repo for my Amplify app.
What is the relation between "amplify push" and "git push"? When should I one and when the other?
The "backend" folder is part of the git repo, so when I change something in the backend code should I use "git push" or "amplify push"?
Should I use "amplify push" only after some "amplify add/remove" operations? and "git push" after other changes?
Should I use both of them in sequence?

Hi @mat-b 👋 , thanks for raising this!
Based on the information provided, I understand that you are trying to perform full stack deployments with Amplify Hosting. Since your amplify folder (backend) is part of the git repository, the best practice would be to use git push after modifying the backend code. This should then trigger the CI/CD pipeline in the Amplify Console and a new build will start.
Similarly, if you wish to add/remove categories to the existing backend, you can do so using the Amplify CLI via amplify add/remove category. To continue with full stack deployments, you will then need to use git push to send your local changes to the remote git repository instead of performing amplify push using the Amplify CLI.


Bump dependency versions in create templates

It would be nice if these were picked up by pnpm update -r -i --latest at some point.  Not sure if we could add them to the workspace.  Regardless, lets merge this and the Svelte 5 CLI now :)


ENYO-5885: Fix that 'removeEventListener' was undefined in VirtualList and Scroller samples

Checklist

[X] I have read and understand the contribution guide
[X] A CHANGELOG entry is included
[ ] At least one test case is included for this feature or bug fix
[ ] Documentation was added or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added
When unmounting the page including VirtualList or Scroller, there is the following error.

Cannot read property 'removeEventListener' of undefined

Resolution
I passed the wrong object and referenced its wrong property before.
So I fixed it and updated the guard condition.
Links
ENYO-5885

LGTM


Remove contact no on top of the site.

Its better to keep the contact no in the contact us page.

This is not valid issue


[bug?] oai.samvera content harvesting error PIL.UnidentifiedImageError for several collections

Mapper: oai.samvera
Registry ID: 154 - errors in 2 mapped index

Run ID: manual__2024-05-03T23:58:47+00:00

permalink to airflow gridview

permalink to map index 16 error log
PIL.UnidentifiedImageError for calisphere-id oai:library.ucla.edu:ark:/21198/zz00280jpg: cannot identify image file '/tmp/ark%3A%2F21198%2Fzz00280jpg'

permalink to map index 37 error log
PIL.UnidentifiedImageError for calisphere-id oai:library.ucla.edu:ark:/21198/zz00288png: cannot identify image file '/tmp/ark%3A%2F21198%2Fzz00288png'

Registry ID: 28108 - errors in 1 mapped index

Run ID: manual__2024-05-03T23:59:53+00:00
permalink to airflow gridview
permalink to map index 5 error log
PIL.UnidentifiedImageError for calisphere-id oai:library.ucla.edu:ark:/21198/zz002hpngp: cannot identify image file '/tmp/ark%3A%2F21198%2Fzz002hpngp'

Registry ID: 28111 - errors in 1 mapped index

Run ID: manual__2024-05-04T00:00:02+00:00
permalink to airflow gridview
permalink to map index 23 error log
PIL.UnidentifiedImageError for calisphere-id oai:library.ucla.edu:ark:/21198/zz0025pngj: cannot identify image file '/tmp/ark%3A%2F21198%2Fzz0025pngj'

@christinklez You can debug these by downloading the mapped metadata, searching for the item, and checking the is_shown_by. For example, for the first error, I can see in the log that the mapped metadata file is 154/vernacular_metadata_2024-05-03T23:59:02/mapped_metadata_2024-05-04T00:01:07/data/80.jsonl.
I download this file from S3 and search for oai:library.ucla.edu:ark:/21198/zz002hpngp. The is_shown_by for this record is https://iiif.library.ucla.edu/iiif/2/ark%3A%2F21198%2Fzz00280jpg . When I go to this URL in a browser, I get an info.json page. This is not actually an image, so the content harvester throws an error.
I'm not sure if this is a provider issue or a rikolti mapper error?

For collection 154, most of the objects have is_shown_by URLs with this kind of format:
https://iiif.library.ucla.edu/iiif/2/ark%3A%2F21198%2Fzz00288pmz/full/!200,200/0/default.jpg
The 2 objects that are failing have these is_shown_by URLs:
oai:library.ucla.edu:ark:/21198/zz00280jpg - https://iiif.library.ucla.edu/iiif/2/ark%3A%2F21198%2Fzz00280jpg
oai:library.ucla.edu:ark:/21198/zz00288png - https://iiif.library.ucla.edu/iiif/2/ark%3A%2F21198%2Fzz00288png
If I add /full/!200,200/0/default.jpg to the end, then I get a thumbnail image:
oai:library.ucla.edu:ark:/21198/zz00280jpg - https://iiif.library.ucla.edu/iiif/2/ark%3A%2F21198%2Fzz00280jpg/full/!200,200/0/default.jpg
oai:library.ucla.edu:ark:/21198/zz00288png - https://iiif.library.ucla.edu/iiif/2/ark%3A%2F21198%2Fzz00288png/full/!200,200/0/default.jpg

Look at the vernacular metadata to review the isShownBy thumbnail URLs.

Looked at the OAI for each of these records, and confirming that they do not have the full/!200,200/0/default.jpg at the tail of the IIIF URLs. We can plan to send a heads up to UCLA about these records.
In the meantime, we can consider ETL'ing these collections instead.

Thanks to #899, we're able to get these collections through to -stage! But these records (as expected) have broken thumbnails.
153
(map index 4, 5, 14, 35)
https://calisphere-stage.cdlib.org/item/ark:/21198/zz002dcpng
https://calisphere-stage.cdlib.org/item/ark:/21198/zz0002pngj/
https://calisphere-stage.cdlib.org/item/ark:/21198/zz002cpng1/
154
(map index 16, 37)
https://calisphere-stage.cdlib.org/item/ark:/21198/zz00280jpg/
https://calisphere-stage.cdlib.org/item/ark:/21198/zz00288png/
28108
(map index 5)
https://calisphere-stage.cdlib.org/item/ark:/21198/zz002hpngp/
28111
(map index 23)
https://calisphere-stage.cdlib.org/item/ark:/21198/zz0025pngj/
28230
(map index 29)
https://calisphere-stage.cdlib.org/item/ark:/21198/zz0002fpng/


Documentation should link API

All mentions of classes and methods in the documentation should link to API documentation.
For example, at http://docs.doctrine-project.org/projects/doctrine-dbal/en/latest/reference/query-builder.html the Doctrine\DBAL\Connection#createQueryBuilder should link to http://www.doctrine-project.org/api/dbal/2.3/class-Doctrine.DBAL.Connection.html#_createQueryBuilder.
Such links make it much easier to understand and use the documented features in real applications.

Don't think we'd be able to do that automatically.
Closing here btw, this is a feature that could be built on https://github.com/doctrine/doctrine-website-sphinx
Consider reporting it there if you have a plan on how to implement this. Automation is required for that though.

doc

Doxygen does this automatically, even in Markdown pages.


@param with name "foo", which was not used When Parameter Name Matches Excluded Property Name From Parent Class

Search terms
The signature Foo.constructor has an @param with name "bar", which was not used
Expected Behavior
The behavior from TypeDoc versions 0.26.1 and below is expected. The documentation should not exclude documented parameters, nor should it warn about unrelated properties.
Actual Behavior
When a parent class (A in this example) has an excluded property (a in this example) with a name that matches the name of a child class' (B in this example) constructor parameter, a warning is given and the parameter is excluded from the documentation.
Steps to reproduce the bug

Install TypeDoc version 2.6.2, 2.6.3, or 2.6.4.

Enable excludeInternal in your TypeDoc configuration.

Use a file with the following contents as an entrypoint.
export abstract class A {
  protected constructor(a: number) {
    this.a = a;
  }

  /** @internal */
  public readonly a: number;
}

export class B extends A {
  /** @param a */
  public constructor(a: number) {
    super(a);
  }
}

Execute typedoc.

The following warning messages will be displayed:
[warning] The signature B.constructor has an @param with name "a", which was not used
[info] Documentation generated at ./docs
[warning] Found 0 errors and 1 warnings

Additionally, the generated documentation will exclude a from the constructor of B:

Environment

TypeDoc version: 2.6.4
TypeScript version: 5.4.5
Node.js version: 22.3.0
OS: Windows 11

Wow, that's a weird one. Caused by 16b2694c270ae0bc3c3a092e0c9a1b2d383c1d3b.


10676 DEV Remove moveToGrisham function from admiral upgrade flow.

https://github.com/Shippable/pm/issues/10676

This is complete.
Closing.


Move kubecf-apps-dns code to cloudfoundry-incubator

kubecf-apps-dns code should be in cloudfoundry-incubator
We need to have all the kubecf code in the cloudfoundry-incubator org.
https://github.com/SUSE/kubecf-apps-dns
should be
https://github.com/cloudfoundry-incubator/kubecf-apps-dns

Also check https://github.com/SUSE/kubecf-kubectl as well as the source for the database instance group image.


username not shown on action logs

Imported from trac ticket #69,  created by phelinek on 12-19-2013 at 14:30, last modified: 12-19-2013 at 15:26

This was a bug I introduced when changing the buttons to links for tkt #50 - I forgot to include the input parameter with the href (uid=ALL)
Trac comment by phelinek on 12-19-2013 at 15:26


Bookmark / Session Restoration Bug

Seems this commit be8bfe8513bc609e78ec7ac3de3472714e30b8e3 broke my bookmark and session autoloading setups.
My startify screen looks like this:

If I press 0, Vim would CD into that directory, and autoload my Session.vim file in that directory.
Now when I press 0, I get this:

It doesn't CD anymore and instead looks like it just tried to perform an :edit Sites/directoryname instead.
If necessary, I can post my Startify .vimrc settings as well.

Yup, please list your startify settings as well.

  " Hacky way to disable Powerline in Startify
  autocmd BufNew * set laststatus=2|highlight CursorLine guibg=NONE
  autocmd FileType startify set laststatus=0|highlight CursorLine guibg=#000000|setlocal cursorline
augroup END

The commit above should make it work for you.
Funnily enough I noticed a bug a neovim where :edit /tmp would switch to <cwd>/tmp instead of /tmp. :edit /tmp/ works, though. In Vim both variants switch to /tmp.

Worked like a charm! Thanks a bunch!

Thanks for reporting!

Looks like I found another issue (that I think is related to this last change, thus re-opening).
If I just press e from Startify, to simply create a blank doc, I get the following error:

After I press a key though, I get the blank document, so it still works, just the error.

The commit above changes :lcd %:h to :lcd %:p:h which would be enough to prevent this problem, but it made no sense to check the user options for [e] in the first place.
This is done only for regular files now.

Fix works great! Thanks again!

Hi mhinz - Can you help with the following problem?
I am able to save and restore sessions, however I get the following error every time.
Error detected while processing function startify#open_buffers..<SNR>77_open_buffer.
.startify#session_load..<SNR>77_create_last_session_link:
line    2:
E484: Cannot open file /tmp/vllxRPo/2
startify: Can't create 'last used session' symlink.

let g:startify_bookmarks = [ {'c': '~/git/dotfiles/dots/vimrc'} ]
let g:startify_list_order = [
       \ ['   Files'],       'files' ,
       \ ['   Directory '],   'dir',
       \ ['   Sessions '],  'sessions',
       \ ['   Bookmarks '], 'bookmarks',
       \ ]

 let g:startify_skiplist = [
       \ 'COMMIT_EDITMSG',
       \ ]

Under this line: https://github.com/mhinz/vim-startify/blob/master/autoload/startify.vim#L761 add:
echomsg 'cd '. shellescape(s:session_dir) .' && ln -sf '. shellescape(fnamemodify(a:spath, ':t')) .' __LAST__'

hi again im not sure either, however I get the following output:
cd '/home/lasse/.vim/session' && ln -sf 'sessiontest' __LAST__
Error detected while processing function startify#session_save..startify#session_wri
te..<SNR>80_create_last_session_link:
line    3:
E484: Cannot open file /tmp/vq2Q5dr/3
startify: Can't create 'last used session' symlink.
Session saved under: /home/lasse/.vim/session/sessiontest
Press ENTER or type command to continue

Hmm, the cd '/home/lasse/.vim/session' && ln -sf 'sessiontest' __LAST__ looks all right.
Let's break it down. It fails at calling system(). That function usually opens a temporary file which seems to fail in your case.
This is the exact command system() is using:
echo &shell &shellcmdflag &shellxquote 'some command' &shellredir tempname() &shellxquote

Can you paste the output?

Hi.
:echo &shell &shellcmdflag &shellxquote shellescape('some command') &shellredir tempname() &shellxquote

using fish:
/usr/bin/fish -c  'some command' >%s 2>&1 /tmp/vSVWFv3/1

Then I tried changing my standard shell to bash:
/bin/bash -c  'some command' >%s 2>&1 /tmp/vcGrABL/3

Press ENTER or type command to continue
:echo system('ls')

Works just as expected

And you get the same error with 'shell' being bash, right?
What about :echo system('ls && cat')?

Same error yep, however I am not able to do
:echo system('ls && cat')  using fish as shell.
I can do :
:echo system('ls ; cat') <-- fish syntax
and get the right output.

Changing line https://github.com/mhinz/vim-startify/blob/master/autoload/startify.vim#L763
to
       \ .' ; ln -sf '. shellescape(fnamemodify(a:spath, ':t')) .' __LAST__')
did the trick.
Thanks for the help.

@poohzrn I assumed every shell would support &&. :\
The latest plugin should make it work without issuing any warning messages.

@mhinz to be fair, that is a reasonable assumption.
I just just updated and it works like a charm. Thanks for a great plugin :+1:

@poohzrn Thank you for the feedback!


Add pager duty alerts

For this to work you need 2 more arguments:
--pagerDutyKey YOUR_KEY
--groupTopicMaxLagConfigs group1:topic1=100,group1:topic2=1000
For simplicity, the alerts are not stored in the database, so an alert can be triggered multiple times if the issue is not solved quickly.

+1


chat immer 1

This moves the root of chat to immer but not any of the inner objects

[ ] after release


Select is out of sync in 0.10.0.

When having a controlled value, I see the select being out of sync in 0.10.0. It can be reproduced in the documentation page.

Go to Select Controlled value example
Select Apple
Deselect Apple
Select box still shows Apple, controlled value is empty (eg. it shows Your favorite fruit is: .). This happens most of the time for me, but not always.

Luckly for me I need disallowEmptySelection: true, so that issue is currently not relevant for me (I had to specify this explicitly, since the default value changed in 0.10.0. This is the reason I started to look into this).
Originally posted by @jcmonnin in https://github.com/kobaltedev/kobalte/issues/229#issuecomment-1689992206

Sorry, unfortunately I still got an issue related to out of sync controlled value in single select.
Please check example here and follow the explanations on the page:
https://stackblitz.com/edit/github-vhpnzf-y71c4s?file=src%2FApp.tsx
In a nutshell, it looks like the Select value needs to be null in order have correct behavior when unselecting an element, however it's not what TypeScript expect (eg. it expects Fruit | Fruit[] | undefined).
Generally, I think null should be avoided:
https://medium.com/@hbarcelos/why-i-banned-null-from-my-js-code-and-why-you-should-too-13df90323cfa
Therefore, in my opinion, onChange should give undefined when nothing is selected, and value should expect undefined for unselected state.


Aart Bik

Talking about 106 Aart Bik has the proof you need for the smp error.  You can contact him at his website.  He works for google.
I thought he had sent you the dump along time ago. I am just trying to help you.  The problem is clearly in pvtable. Cxxdroid jut does not like the macros for smp

Aart Bik has the proof you need for the smp error. You can contact him at his website. He works for google. I thought he had sent you the dump along time ago. I am just trying to help you.
If you save my first post you will have lots of info to help find problems. Short version test vice in chess for
Android and make a dump.

Reminder the U64 Randon64Poly[781];  Not a recognize name type in polykey.h
The print statement in polykey..c that needs to be changed to %zu

My make file:
Clang
-pipe
-fvectorize
-freciprocal-math
-mlittle-endian
-Marm

When the smp problem is fixed, I can send you the finished 64 bit ViceSmpNeonMath.  It uses neon math files for fastest execution on armv8  snapdragon 8 core.

personal preference
increased these 3 things
inputbuffer=8
inf_bound=34000
ab_bound=32000
This was not included with the orignal dump.

I cannot find mirror.epd on the internet.  Please include the file with the source code.


Provide a way to get the metric according to the status of the failure

A user may want to get the metrics about 500 and 408 respectively.
Currently, a user can get only two metrics which are success and failure by default.

#1172 fixes this.


feat: detect packageManager field in package.json

prepare for https://github.com/pkg-size/action/issues/49

@privatenumber  Can you fix the CI error first and then I can do a rebase?
https://github.com/privatenumber/ci/runs/7227941544?check_suite_focus=true

Thanks for handling the merge upstream, was going to do that today

When will this be released?

Working on adding tests before the release because its a big change but pretty booked out for the weekend. Likely Monday. Thanks for your patience 🙏

Thanks for the promising schedule!

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[recnet-api] Add announcement api

Description
Design Doc: https://www.notion.so/Announcement-DB-schema-API-0b6d6f4c28334f3684116219992578bb

Add the following APIs:

GET /announcements
POST /announcements
PATCH /announcements/:id

Create Announcement table in DB
Add Announcement to recnet-api-model

Related Issue

https://github.com/lil-lab/recnet/issues/227

Notes

TODO

[ ] Paste the testing link
[x] Clear console.log or console.error for debug usage
[ ] Update the documentation recnet-docs if needed
[ ] Version bump in package.json if needed

Remember to revisit the design docs and update it


PowerPC Instruction Alignment Change

What is the feature you'd like to have?
I would like to change the instruction alignment form current value 4 to 2 (https://github.com/Vector35/binaryninja-api/blob/6929009578958634304404179659515738797945/arch/powerpc/arch_ppc.cpp#L328).
Is your feature request related to a problem?
If the alignment will be set to 2, it would be easy to create a ppc architecture extension to support VLE encoding which adds 2-byte instructions. Currently, if the alignment is set to 4, branches fail to resolve.
Are any alternative solutions acceptable?
I am extending PPC architecture to support VLE instructions with this (currently heavily in progress) plugin: https://github.com/Martyx00/PowerPC-VLE-Extension
Alternative would be to treat PowerPC with VLE instructions as a separate architecture.
Additional Information:
None

The best bet would be to make the PPC-VLE a separate architecture, as changing the alignment could have some negative consequences for things like linearsweep.


Deleting unwanted prebuilds for Yocto

This is kind of a follow on to this: https://github.com/prebuild/prebuildify/issues/54
We use a package (msgpackr-extract) that relies on prebuildify to do it's prebuilding.
The problem we have is that prebuildify leaves the various prebuilt binaries on the OS, even the ones that cannot possibly run (like 64 bit ones on a 32 bit system).
This is obviously normally not a problem, except for when you try and use it with a system like Yocto to build an OS.
Yocto has various QA systems and uses objcopy to move the binaries across. It blows up when you try and copy a 64 bit binary to a 32 bit OS because it knows it could never possibly run.
My current solution is rm -rf node_modules/msgpackr-extract/prebuilds after npm install has finished, to wipe out all the prebuilt binaries (the one we need isn't shipped anyway).

Is there any way that this situation could be handled differently to make it more compatible with build systems?

Nothing in prebuildify for that no, but you could make a prune tool / option that removes all non-relevant builds


Update 11_Pipelines.ipynb

Made a couple of small grammar corrections to the documentation

thanks for the contribution ❤


About DETR Align

How does DETR perform alignment operations?
Can I find a detailed description in the paper?
Where is the code?

DETR works the same as other methods, except for a cluster-based bipartite graph matching and cross-entropy loss.
You can refer to Section B.5. for the detailed description in the paper.
Please refer to DETR pre-training config for more pre-training details, we provide an example to pre-train and fine-tune Mask R-CNN. The DETR is similar to it, and you should replace the related config.

Close this issue due to no response for 2 days. Feel free to re-open this issue if you have more questions.


Channel Method for getLocationAccuracy not implemented

🐛 Bug Report
Geolocator.getLocationAccuracy() does not have a channel method implemented on Android and is throwing a MissingPluginException.
E/flutter (26906): [ERROR:flutter/lib/ui/ui_dart_state.cc(199)] Unhandled Exception: MissingPluginException(No implementation found for method getLocationAccuracy on channel flutter.baseflow.com/geolocator)
Expected behavior
A default value of LocationAccuracyStatus.precise to be returned. I think this is currently the behavior for iOS?
Reproduction steps

Call Geolocator.getLocationAccuracy() on Android device

Configuration
Version: 7.20+1
Platform:

[ ] :iphone: iOS
[x] :robot: Android

@helixbooks the getLocationAccuracy() method is only supported on iOS and is indeed not implemented on Android, hence the error.
I noticed the documentation doesn't mention this, so I will leave this issue open so we can track the documentation update.

@helixbooks I changed the documentation, so that it is clear the method only supports iOS devices. Also, when executing the method on Android devices LocationAccuracyStatus.unknown will now be returned. See: https://github.com/Baseflow/flutter-geolocator/pull/769


Support ABI overload for fe contract.

What is wrong?
There's no method overloading in Fe. However for some standard EIP we need overloading method. E.g safeTransferFrom for ERC721 token. We should support it as well.
How can it be fixed
🤔

I think that Fe doesn't support overloading because is an anti-pattern in Rust, and Fe is trying to stay close to Rust-like syntax like Vyper is to Python. Abstraction without overhead: traits in Rust


fetchgit is non-deterministic when given leaveDotGit = true and fetchSubmodules = false

nix-prefetch-git is non-deterministic if given both leaveDotGit = true and fetchSubmodules = false.
Here are a couple runs back-to-back:
~/src/nixpkgs (pleasant-ruby) λ nix-prefetch-git https://github.com/bundler/bundler.git a2343c9eabf5403d8ffcbca4dea33d18a60fc157 --hash sha256 --leave-dotGit 2>/dev/null
git revision is a2343c9eabf5403d8ffcbca4dea33d18a60fc157
d77a4f6c4f57af463bb481c9ea54ad059434e79f7fbfdd4b5a7f71208f2e8383

~/src/nixpkgs (pleasant-ruby) λ nix-prefetch-git https://github.com/bundler/bundler.git a2343c9eabf5403d8ffcbca4dea33d18a60fc157 --hash sha256 --leave-dotGit 2>/dev/null
git revision is a2343c9eabf5403d8ffcbca4dea33d18a60fc157
2816a1f50ec43a30c2c01f9e4ac4121f2b6f07e5d5abf2191b960d8865eb8afb

~/src/nixpkgs (pleasant-ruby) λ nix-prefetch-git https://github.com/bundler/bundler.git a2343c9eabf5403d8ffcbca4dea33d18a60fc157 --hash sha256 --leave-dotGit 2>/dev/null
git revision is a2343c9eabf5403d8ffcbca4dea33d18a60fc157
ddc4ae29374585987a1e6cf0a8e66ba96bb690bd474a887caef2dce0b3b0f9a7

However, when I make this leaveDotGit = true and fetchSubmodules = true:
~/src/nixpkgs (pleasant-ruby) λ nix-prefetch-git https://github.com/bundler/bundler.git a2343c9eabf5403d8ffcbca4dea33d18a60fc157 --hash sha256 --leave-dotGit --fetch-submodules 2>/dev/null
git revision is a2343c9eabf5403d8ffcbca4dea33d18a60fc157
34caf39724cfa72a136869d97aa74eacca9f9fb16a1e335e7fa6158aad38f1e4

~/src/nixpkgs (pleasant-ruby) λ nix-prefetch-git https://github.com/bundler/bundler.git a2343c9eabf5403d8ffcbca4dea33d18a60fc157 --hash sha256 --leave-dotGit --fetch-submodules 2>/dev/null
git revision is a2343c9eabf5403d8ffcbca4dea33d18a60fc157
34caf39724cfa72a136869d97aa74eacca9f9fb16a1e335e7fa6158aad38f1e4

~/src/nixpkgs (pleasant-ruby) λ nix-prefetch-git https://github.com/bundler/bundler.git a2343c9eabf5403d8ffcbca4dea33d18a60fc157 --hash sha256 --leave-dotGit --fetch-submodules 2>/dev/null
git revision is a2343c9eabf5403d8ffcbca4dea33d18a60fc157
34caf39724cfa72a136869d97aa74eacca9f9fb16a1e335e7fa6158aad38f1e4

Interestingly, fetchgit itself is deterministic, despite nix-prefetch-git being non-deterministic (at least from the command line).
This is pretty curious.

Perhaps environment variables come into play? Does the shell script take care to clear the environment like a nix build does?

So, I tried diffing two of the outputs:
~/src/bundix ((detached from fa9302f)) λ diff -rq /nix/store/fh3hmfij0pgwl9fxba96wn8dxz69q73l-git-export /nix/store/wybf5dkiiilhqzz4vxqr3cag60m6dwh4-git-export
Only in /nix/store/fh3hmfij0pgwl9fxba96wn8dxz69q73l-git-export/.git: tags.17333
Only in /nix/store/wybf5dkiiilhqzz4vxqr3cag60m6dwh4-git-export/.git: tags.17419

That looks an awful lot like part of my git hooks that automatically generate tags. Sure enough:
~/src/bundix ((detached from fa9302f)) λ HOME=/homeless-shelter nix-prefetch-git https://github.com/bundler/bundler.git a2343c9eabf5403d8ffcbca4dea33d18a60fc157 --hash sha256 --leave-dotGit 2>/dev/null
git revision is a2343c9eabf5403d8ffcbca4dea33d18a60fc157
20167f627ad30a90324a6d35b19f189672889fd9462f6df0988a6762b3d411b8

~/src/bundix ((detached from fa9302f)) λ HOME=/homeless-shelter nix-prefetch-git https://github.com/bundler/bundler.git a2343c9eabf5403d8ffcbca4dea33d18a60fc157 --hash sha256 --leave-dotGit 2>/dev/null
git revision is a2343c9eabf5403d8ffcbca4dea33d18a60fc157
20167f627ad30a90324a6d35b19f189672889fd9462f6df0988a6762b3d411b8

I'm of the opinion that we should override $HOME in the nix-prefetch-scripts package, to prevent these kinds of problems.
Thoughts?

Seems like those nix-prefetch-* scripts in general should almost completely clear out your environment

Yeah, I don't know how much to get rid of, considering that someone may want to get the hash of their private repos or something.


Python install from source

Having issues with the SAT solver after installing from source:
from ortools.sat.python import cp_model
gives the following error
from ortools.sat import cp_model_pb2 ImportError: cannot import name 'cp_model_pb2'

I do not know, we have been compiling or-tools continuously for the last
week and many different setups and platforms, and we did not see that
problem.
Which platform?
What is the command line to run the example?
Are you adding a PYTHONPATH? are you using make rpy?
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
2017-11-28 10:28 GMT+01:00 ciarán<EMAIL_ADDRESS>
Having issues with the SAT solver after installing from source:
from ortools.sat.python import cp_model
gives the following error
from ortools.sat import cp_model_pb2 ImportError: cannot import name
'cp_model_pb2'
Is there a fix imminent for this?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/534, or mute the thread
https://github.com/notifications/unsubscribe-auth/AKj17Q67zbmZV7PeThXZ4sSkJAU-7RN8ks5s69I-gaJpZM4Qs7SX
.

Hi,
it is on mac os x 10.12.6.
I follwed the exact guide here using make python.
It is in the python terminal session when I try to do the import, even when adding directory to PYTHONPATH

My bad... seems like there was an issue with my SWIG installation. Fixed that and it's all good now, thanks!


Fix chunk loading error in local env

jira: https://issues.redhat.com/browse/RHCLOUD-12799
This pr adds the webpack proxy to the webpack config (currently its just copied from: https://github.com/RedHatInsights/frontend-components/tree/master/packages/config and added some minor tweaks because this project is not using the common config, it will be extracted to a standalone dependency to prevent any webpack versions issues).
@dlabrecq can you please test if you still see the webpack chunk loading issues? Currently, it does not support local API and it's setup only for ci-beta env. It will be configured properly if it helps with the problem.
Start it with yarn start:beta and open https://ci.foo.redhat.com:8002/beta/openshift/cost-management (https is required for now)
Changes

use webpack proxy instead of insights proxy for local development
add an interactive start script to pick required development environment
changed build directory to dist

I have kept the spadnx and local spandx configs intact so it can be still used with insights proxy.

@Hyperkid123 I tested the PR and was able to successfully debug using Chrome – the webpack chunk loading issues were not detected. I could not debug without the PR in Chrome.

@dlabrecq is ok if I finish this PR to switch you from insights proxy to webpack proxy? It would mean using the webpack utils package (not the config one, this one just includes all the partial plugins etc.) and adding a setup for using local API. It will also enable dev server hot-reload and you will have a bit more control over the proxy config in general.

@Hyperkid123 That would be very much appreciated. Zack Allen modified the current webpack config to help get us this far.
I assume we will be required to restart the dev server in order to develop with local environments other than ci-beta; for example, qa-beta, prod-beta, etc.?

I assume we will be required to restart the dev server in order to develop with local environments other than ci-beta; for example, qa-beta, prod-beta, etc.?

@dlabrecq for now yes. But I will add this requirement to the list of future features.

@Hyperkid123 from a first look this looks good, i am testing it locally now.

Been testing and things are looking good. However, I'm not able to reach https://prod.foo.redhat.com:8002/beta/openshift/cost-management
page cost-management not found, fallback to index.html
serving locally /beta/openshift/cost-management > cost-management ---> /Users/dlabrecq/Work/hcm/repos/koku-ui/dist/index.html

I found this in the Chrome console, but Firefox doesn't load either:

@dlabrecq the prod env should be working now as well.


Online Customizer for 4.x

Hello,
Just wanted to mention that the lack of this feature https://getbootstrap.com/docs/3.4/customize/ is the only reason that stops me from migrating to version 4.x of Bootstrap.
And to be more specific - I'm talking about the component selection... the whole theming section is not that important (we can always override something we don't like in a custom css file) but being able to choose which css and js components to include is crucial, especially nowadays when everyone is using mobile devices and bandwidth is important (plus google penalizes slower loading sites).
You can all agree that Bootstrap is quite a large project and most of the time not everything that comes with it is needed for every site... for example why should I make the visitors download code for modals and forms if the site they open doesn't have any?
I know that you've said in the past that you won't add this feature to version 4, but please reconsider it for performance reasons... if not - could you at least publish the source of the current customizer so someone else could make it work with the latest version (I tried searching for it but couldn't find anything).

@Johann-S created this one: https://johann-s.github.io/bs-customizer/.

Thank you, that is exactly what I needed!
Just a small suggestion - since this is made by an official team member you guys might want to add it to the main site - for example the reason I posted here is because I couldn't find it after a few different google searches... and back in the day when I was looking around for a framework to replace what I had cobbled on my own, the customizer was the cherry on top that made me choose Bootstrap :)

same here, will submit a PR to the doc because it's really useful...
On only need a custom grid system, don't want to setup webpack loaders just for building it once for all


Remove map.reload, in favor of vis.reload

To remove some indirection and (hopefully) troubleshoot failed map instantiations easier.
We had this relationships before:
  Layers (CartoDBLayer and TorqueLayer) -> Map#reload
  Analysis -> Map#reload
  Dataviews -> Map#reload
  Map#reload -> Vis#reload

This PR removes Map#reload and makes objects reload the vis directly:
  Layers (CartoDBLayer and TorqueLayer) -> Vis#reload
  Analysis -> Vis#reload
  Dataviews -> Vis#reload
  Map -> Vis#reload

As a follow up, I think it'd make sense to centralice all calls to vis.reload in a single class, so that it's easier to control when the map gets instantiated...
@viddo what do you think? Thx!
@viddo what do you think?

Sounds reasonable and code looks legit 👌


SyntaxError: Unexpected token @

Hi, I'm trying to start my azk env and I getting this error.
MacBook Pro (Retina, 15-inch, Mid 2015)
Node: 0.12.7
npm:2.12.1
$ azk agent start
azk: Wait, this process may take several minutes
azk: Loading settings and checking dependencies.
azk: Checking version...
azk: azk 0.14.5 detected

/usr/local/Cellar/azk/0.14.5/node_modules/vboxmanage/lib/parse.js:97
      return JSON.parse("{" + line + "}");
                  ^
SyntaxError: Unexpected token @
    at Object.parse (native)
    at to_item (/usr/local/Cellar/azk/0.14.5/node_modules/vboxmanage/lib/parse.js:97:19)
    at Array.map (native)
    at Object.exports.machinereadable_list (/usr/local/Cellar/azk/0.14.5/node_modules/vboxmanage/lib/parse.js:99:30)
    at /usr/local/Cellar/azk/0.14.5/node_modules/vboxmanage/lib/machine.js:47:37
    at Object.callback (/usr/local/Cellar/azk/0.14.5/node_modules/vboxmanage/lib/command.js:72:16)
    at next (/usr/local/Cellar/azk/0.14.5/node_modules/vboxmanage/node_modules/async/lib/async.js:720:43)
    at /usr/local/Cellar/azk/0.14.5/node_modules/vboxmanage/node_modules/async/lib/async.js:24:16
    at ChildProcess.<anonymous> (/usr/local/Cellar/azk/0.14.5/node_modules/vboxmanage/lib/command.js:64:20)
    at ChildProcess.emit (events.js:98:17)
    at maybeClose (child_process.js:766:16)
    at Process.ChildProcess._handle.onexit (child_process.js:833:5)

@agutoli
You are using VirtualBox 5.0, right?
This problem was corrected in #495, but has not yet released a version with it.

Yes @gullitmiranda , I'm using the 5.0.2. I will try to use a 4.x version. tks!

Hey @gullitmiranda , with Virtualbox 4.x works for me!... awesome!

nice


Pausing sessions

After updating to focus version 1.4.0, pausing a session with ctrl-c sometimes marks that session as completed.

@Blackbeing Thanks, I noticed that bug too — happens rarely but not ideal. Working on a fix

Closed by https://github.com/ayoisaiah/focus/commit/c8ee01fc5a7c549eaeeec90153d416b239789256
Please try v1.4.1

This still occurs in v1.4.2

@Blackbeing I haven't experienced it since I implemented that fix.
How can I reproduce it?

Uploading focus.log…
That's the latest log file after completing session when paused.
Using latest version
git clone https://github.com/ayoisaiah/focus && cd focus && go install ./...

@jonnieey Could you please re-upload the log file

Uploading focus.log…

@jonnieey I suggest deleting all your existing timers with focus delete-timer --all, and let me know if it happens again


Chain-ID validation notes/tests

Description
I've added some notes and tests so if the value of the chain-id changes, at least the current maintainers of this codebase can be aware that this may be problematic
There isn't much that can be done in the case that a counterparty upgrade to the tendermint version before the other chains are even aware this value may be changed, but it is good for folks to be aware so they don't launch a chain that cannot interact with IBC until everyone upgrades
closes: #177

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (see CONTRIBUTING.md)
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Code follows the module structure standards.
[x] Wrote unit and integration tests
[x] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[x] Added relevant godoc comments.
[x] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[x] Re-reviewed Files changed in the Github PR explorer
[x] Review Codecov Report in the comment section below once CI passes

I think a chain ID validation function should be incorporated daemon setup process. This way you could avoid this issue entirely and enforce chains that use IBC to validate their chainID before initializing the chain.

The IBC client cannot assume the counterparty validates its own chain-id. The counterparty could be malicious. Without this check, an user could create an IBC client with no max chain-id length


[PE-D][Tester A] Empty table of contents in UG

Table of content in UG is empty.

Labels: type.DocumentationBug severity.Low
original: ChinJunAn/ped#2

Duplicate issue. Refer to #102.


build(buck2): create a prettier_check rule for use in lang-js

Prior to this change, no source formatting was being enforced by a build system and thus started to drift in consistency.

bors merge


[nodejs] load local sources using require() even if action is not zip/binary

Some projects have a build step that takes it from a normal commonjs code to a single action javascript (no zip) that is Ioaded using eval by the OpenWhisk nodejs runtime. However, when loading the original or intermediary sources (via --on-build), require()-based loading is required.
This could be controlled through a command line option, although that would be nodejs specific. Maybe there is also an automatic way.

Went with the automatic way - checking if there is a require() in the code.


Sink to BigQuery with google pubsub sink

Is your feature request related to a problem? Please describe.
Given the level of integration with pubsub between the different GCP components (cloud run, eventarc, dataflow, bigquery, workflows, ...), having a pubsub sink would be highly beneficial.
I'm currently missing this to take advantage of the pubsub bigquery CDC feature as documented here: https://cloud.google.com/pubsub/docs/bigquery.
Describe the solution you'd like
I'd like a sink wose definition would look like:
CREATE SINK sink1 FROM mv1
WITH (
   connector='google_pubsub',
   topic='topic1'
)
FORMAT PLAIN ENCODE JSON; -- possibly AVRO  [1]

[1] https://cloud.google.com/pubsub/docs/samples/pubsub-publish-avro-records
Describe alternatives you've considered
I tried to use the native RW bigquery sink, but it's using the legacy tabledata.insertAll method, which is not compatible with the CDC message ingestion.
Additional context
opening the issue here as discussed in https://github.com/orgs/risingwavelabs/discussions/15237

https://github.com/risingwavelabs/risingwave/pull/16363 will resolve this issue.


Legend Symbol Parameter [New Feature]

New Feature proposed to display symbol on legend
Legend Symbol Parameter [New Feature] #4804

@touletan you can push updates to an existing PR. you didn't really have to close https://github.com/chartjs/Chart.js/pull/4806. please make updates on the same PR in the future. let us know if you're not sure how to do that

The options logic path seems a bit unclear and I don't think the helpers.canvas.drawPoint should be changed (it's a public method and modifying its signature like you did is a breaking change). Though, I'm not sure to get the result of that feature very well, @touletan can you please post screenshots of the different use cases you are trying to implement?

@simonbrunel , my specific need is to use a large line as legend symbol while keeping the dot as point style on the lines.

I did few additional changes to be able to use other symbols that were always available as points.

All the logic about usePointStyle is to avoid breaking that current functionality.
Agree about the helper signature, it could break external plugin using it. I propose the rename the new method and keep the same logic in it. The code from the current method will be changed to call the new method from there and parameter values will be defaulted/updated so the outcome is the same.

I think it's fine to add a new helpers.canvas.(draw)Symbol that split the radius into width and height and have drawPoint calling this method. Though, I don't like the extra isLineWidthZero parameter, which seems to only prevent ctx.stroke.
What about a new helpers.canvas.symbol method that simply creates the shape (no ctx.fill or ctx.stroke)? The calling code would be responsible to fill and/or stroke this shape:
drawPoint: (...) {
    helpers.canvas.symbol(ctx, style, w, h, x, y); // would be exports.symbol(...)
    ctx.stroke();
    ctx.fill();
}

The legend would use only the helpers.canvas.symbol method.
Then I would maybe expose the following dataset/legend options:
legend: {             // nested options under the dataset
    symbol: 'rect',   // default to rect
    boxWidth: ...
}

if usePointStyle is true, it would override symbol and boxWidth.

sounds like a plan. @etimberg, any other comments before I start working on it?

@simonbrunel, ctx.fill() is not required for each symbol, so it should stay in the method logic.

@touletan I have no other comments. 😄

@etimberg , @simonbrunel , @benmccann , new version completed and ready for review.

@simonbrunel have you had a chance to review this since it was last changed?

@touletan I removed trailing spaces from all of our docs https://github.com/chartjs/Chart.js/commit/bba29e591604fed8c1e3bc4cf911273d4c5e0f42. If you rebase then you will not need all the formatting changes in this PR which may make it slightly easier to review

@touletan can you rebase this PR so that it's easier for us to review? thanks!

@touletan this PR needs to be rebased before it can be merged

Hi
could you please provide guidelines to do this correctly.
thank you.

De : Ben McCann<EMAIL_ADDRESS>Envoyé : 15 juillet 2018 23:23
À : chartjs/Chart.js
Cc : touletan; Mention
Objet : Re: [chartjs/Chart.js] Legend Symbol Parameter [New Feature] (#4811)
@touletanhttps://github.com/touletan just a reminder that this PR needs to be rebased
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/chartjs/Chart.js/pull/4811#issuecomment-405126000, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AE_7QfxoNbPpFU_pwLBG_C0_qXaPrjK3ks5uG87VgaJpZM4Ps-yk.

I would recommend developing each feature on a new branch. You are on the master branch which I would recommend you not to use in the future
This might not be exactly right, but hopefully will work or be pretty close:
git remote add upstream git@github.com:chartjs/Chart.js.git
git fetch upstream
git merge upstream/master (at this point you will have merge conflicts and will need to fix them)
git add . (to mark all merge conflicts as fixed)
git rebase --continue
git push -f

@touletan Oooh. That seems like it didn't work unfortunately as the PR is now saying 136 files edited
At this point I think the best bet would be to delete your fork on GitHub, create a new fork, clone the code, create a new branch, and apply the changes on a new branch

Does it look better now or should I start from scratch as you proposed?

De : Ben McCann<EMAIL_ADDRESS>Envoyé : 19 juillet 2018 04:47
À : chartjs/Chart.js
Cc : touletan; Mention
Objet : Re: [chartjs/Chart.js] Legend Symbol Parameter [New Feature] (#4811)
@touletanhttps://github.com/touletan Oooh. That seems like it didn't work unfortunately as the PR is now saying 136 files edited
At this point I think the best bet would be to delete your fork on GitHub, create a new fork, clone the code, create a new branch, and apply the changes on a new branch
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/chartjs/Chart.js/pull/4811#issuecomment-406153392, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AE_7QdSAU6BVpYM85_Gj3bcoQJ0suVDCks5uIA9ZgaJpZM4Ps-yk.

Much better! Thank you!!

After additional testing, I removed the code related to the point rotation, since it was causing issue with the legend symbol position. I need help to figure out how to re-introduce point rotation ....

that version is good for review.

@touletan I'm very sorry this PR will need to be rebased again. Would you be able to update it?

@touletan just a reminder that this PR will need to be rebased before we can move forward with it

Thanks @touletan! And sorry again for the delay on getting this in. We are trying to get through the backlog of open reviews. I'm afraid it's hard for us all to find enough spare time, but hopefully we will make it through them all eventually

@touletan would you be able to rebase this PR?

@touletan I just wanted to check in one last time to see if you still have an interest in moving forward with this PR. It's been needing to be rebased for quite awhile. We'll close it as inactive if we don't hear back soon though you're always welcome to reopen it in the future when you find time to work on it again

I'll have a look.

De : Ben McCann<EMAIL_ADDRESS>Envoyé : 4 avril 2019 00:28
À : chartjs/Chart.js
Cc : Dominic Jean; Mention
Objet : Re: [chartjs/Chart.js] Legend Symbol Parameter [New Feature] (#4811)
@touletanhttps://github.com/touletan I just wanted to check in one last time to see if you still have an interest in moving forward with this PR. It's been needing to be rebased for quite awhile. We'll close it as inactive if we don't hear back soon though you're always welcome to reopen it in the future when you find time to work on it again
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/chartjs/Chart.js/pull/4811#issuecomment-479745373, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AE_7QTEeFOwaWyN0dC0-JIjcNoWhfPZXks5vdX9TgaJpZM4Ps-yk.

@touletan I'm going to close this PR as inactive for the moment to help us keep track of which PRs need review, but please feel free to reopen if you get a chance to rebase and we'll be happy to take a look at it

I'd be in need of this boxHeight option... thanks in advance!


File Write/Read returning array of null bytes

Been struggling to get this to work in my own local project, and I finally realized there is a demo of this project on netlify.
However the netlify instance of the project doesn't seem to be working either.
It seems either the File Write or File Read operation is broken.
In my own testing, I found that writing and a file and reading it always returns an array of null bytes (00 00 00 00...). I assume it's working to some extent, as the file size remains exactly correct, but the actual content of the file is useless.
Is this something wrong with ffmpeg.wasm? Or is this a FFmpegBlazor issue?

If you're able to point me in the right direction, I can try to fix the issue in a pull request.

Hi, thanks for helping with it.
Few things I that might require some testing are:

Write function unmarshalled Interop call C# and JS side : https://github.com/sps014/FFmpegBlazor/blob/b0588125fe1020886fed0bcc6c460b87e72b4313/FFmpegBlazor/FFMPEG.cs#L76
https://github.com/sps014/FFmpegBlazor/blob/b0588125fe1020886fed0bcc6c460b87e72b4313/FFmpegBlazor/wwwroot/blazorFfmpeg.js#L50

Similar there can be issue in read  function as well
https://github.com/sps014/FFmpegBlazor/blob/b0588125fe1020886fed0bcc6c460b87e72b4313/FFmpegBlazor/FFMPEG.cs#L54
https://github.com/sps014/FFmpegBlazor/blob/b0588125fe1020886fed0bcc6c460b87e72b4313/FFmpegBlazor/wwwroot/blazorFfmpeg.js#L33

API documentation for FFMPEG wasm Read can be found here:
required doc
Blazor unmarshalled Interops
https://docs.microsoft.com/en-us/aspnet/core/blazor/javascript-interoperability/call-javascript-from-dotnet?view=aspnetcore-6.0#unmarshalled-javascript-interop
if you find anything feel free to create PR.

Seems like the following line isn't updating the byte array in FFMPEG.cs.
https://github.com/sps014/FFmpegBlazor/blob/b0588125fe1020886fed0bcc6c460b87e72b4313/FFmpegBlazor/wwwroot/blazorFfmpeg.js#L47
I don't know enough about how Blazor handles array pointers or how chrome shares memory, but I have a sneaking suspicion that this is an issue with SharedArrayBuffer. The fact that the demo broke without any changes except browser version further supports this theory.

Not entirely sure but one of possible reason could be this breaking change.

Hi, i am able to fix the issue, there was a breaking change in Blazor array interop.
We do longer need to deal with raw buffer pointers manually.
In blazorffmpeg.Js we can have our new js interop codefor readFileProcess as

        readFileProcess: (h)=>
        {
            return readFileBuffers[h];
        },

and in FFMPEG.cs we can directly invoke
       public async Task<byte[]> ReadFile(string path)
       {
           _ = reference.InvokeUnmarshalled<FileConf, bool>("readFileFFmpeg", new FileConf()
           {
               Hash = Hash,
               Path = path
           });

           await Task.Delay(5);

           var array =reference.Invoke<byte[]>("readFileProcess",Hash);
           return array;
       }

you can still send PR with these changes if you want.
thanks.


When player makes a move somtimes the wrong card moves

Sometimes when the player drags one card to the board another one is moved to the board and the one that the player selected is moved back to the player.

This bug occured because indexes weren't updated in time


Enquiry about resources for referenced projects

I have two Bridge projects (an application and a component) and a C# MVC application as a host project. The Bridge application project references the component project. My component has a CSS file and my application also has a CSS file.
What I'm trying to achieve is for my component to specify how and where (relatively) it's CSS file is output. Likewise I want my application to be able to specify where it's CSS file is output and not have to look in it's bin folder to find the CSS file for the component.
If I don't specify those CSS files as resources explicitly for either Bridge projects I end up my neither CSS files being output in the 'Output' folder for my C# hosting app. This is expected as I haven't specified where I want these files to end up.
If I specify the CSS files explicitly in the 'resources' of the bridge.config for the component project, the '*.meta.js' isn't included in the output. I believe this is because when you specify resources explicitly you're providing a whitelist, however the generated JS file is output, as I'm specifying it as the 'fileName' property in my component's bridge.json.
Here's the bridge.json for my component:
{
	"output": "$(OutDir)/",
	"cleanOutputFolderBeforeBuild": true,
	"fileName": "Component.js",
	"resources": [
		{
			"name": "Main.less",
			"output": "./stylesheets",
			"files": [ "Stylesheets/Main.less" ]
		}
	]
}

So here my 'Main.less' file ends up in the './stylesheets' folder as expected, and the Component.js ends up in the 'Output' directory, although because I haven't included 'Component.meta.js' explicitly I'm getting serialisation errors in my Bridge application.
I suppose what seems like the sanest solution to this problem would be to be able to specify 'extra resources'. This way all the JS would be dumped in the output folder as usual, except you could specify where CSS files in particular end up.
Does it seem like I'm doing this correctly?
Apologies if this is a duplicate.
Here's the page I've been referencing:
https://github.com/bridgedotnet/Bridge/wiki/global-configuration#resources

I don't know if this might be related to https://github.com/bridgedotnet/Bridge/issues/1960#issuecomment-255373750
Dan asked:

unless there was a way to use wildcards in the "files" entries within resources, so I could just specify "*.js"

In this case it was about dependencies, but I guess the same question applies for this project. How can we add additional resources to a project, without it "replacing" the original list of resources (which means we have to then explicitly add all of the default files).
I guess another question is how Bridge decides which files to extract from the DLL - why do the JS files get pulled out automatically, but not the .less files? (It looks like they are embedded in the DLL, just not extracted).


Set gthesheep TikTok tap variant to default

Checklist

[x] Add/update the file in the appropriate folder (/taps or /targets). The name of the file should match the name of the tap. If there is already one, add a descriptor to the name such as -search.
[x] Add/update the PNG logo image in /assets/logos/<taps or targets>. The image name must match the YAML file name.
[x] Tag @tayloramurphy or @pnadolny13 to flag it for review. Or post to the #hub channel on Meltano slack.

Reviewer Checklist

[ ] Validate file against JSON Schema. https://www.jsonschema.net/home is an option.
[ ] Build website locally and validate everything works.

@pnadolny13 @tayloramurphy - the difference between the 2 taps appears to be the inclusion of reservation ads based on the schema, I've added a task to my repo to add these as I'll be working on the tap this week anyway to add audience stats

I take it that's not how to set defaults? 😂

@GtheSheep thanks for this! Yeah we used to set defaults inside the plugin file but now the way to do it is to set it in the default_variants.yml.


Add six in README and setup.py

:x: Build SMT 0.1.261 failed (commit https://github.com/SMTorg/SMT/commit/b8186d7388 by @bouhlelma)

:x: Build SMT 0.1.262 failed (commit https://github.com/SMTorg/SMT/commit/27d595302f by @bouhlelma)

:x: Build SMT 0.1.264 failed (commit https://github.com/SMTorg/SMT/commit/a174dd9363 by @bouhlelma)

:x: Build SMT 0.1.265 failed (commit https://github.com/SMTorg/SMT/commit/ca472b05bc by @bouhlelma)

:x: Build SMT 0.1.266 failed (commit https://github.com/SMTorg/SMT/commit/83a088d5c6 by @bouhlelma)

:x: Build SMT 0.1.267 failed (commit https://github.com/SMTorg/SMT/commit/3f6d47984c by @bouhlelma)

:x: Build SMT 0.1.268 failed (commit https://github.com/SMTorg/SMT/commit/c7abe85538 by @bouhlelma)

:x: Build SMT 0.1.269 failed (commit https://github.com/SMTorg/SMT/commit/36fc649d8a by @bouhlelma)

:x: Build SMT 0.1.270 failed (commit https://github.com/SMTorg/SMT/commit/e6b6bc22de by @bouhlelma)

:x: Build SMT 0.1.271 failed (commit https://github.com/SMTorg/SMT/commit/268f424651 by @bouhlelma)

:x: Build SMT 0.1.276 failed (commit https://github.com/SMTorg/SMT/commit/6d212f2814 by @bouhlelma)

:x: Build SMT 0.1.277 failed (commit https://github.com/SMTorg/SMT/commit/1a7e1294ce by @bouhlelma)

:white_check_mark: Build SMT 0.1.278 completed (commit https://github.com/SMTorg/SMT/commit/7e3b113875 by @bouhlelma)

:white_check_mark: Build SMT 0.1.279 completed (commit https://github.com/SMTorg/SMT/commit/8c30b7bd52 by @bouhlelma)


Fixed NullReferenceException in OpenXmlElement RawOuterXml field

The current version of your code:
It sets _rawOuterXml to string.Empty if value is null, but instead of leaving the setter immediately, it resets the field to null at the end of the setter
The changes I made should resolve the issue.
Thanks.

@aidilumarov Thanks for the catch! Can you add a regression test so that it will be caught in the future?

@aidilumarov I'd like to get a 2.12.0 release out this week. If you can get some tests in, we'll have this as part of that release.
Also, can you update the CHANGELOG.md to include this?

@twsouthwick Sorry for taking long. I was off for several weeks. I have made the changes you asked for. Kind regards.

I've taken the liberty of updating the branch so it can be merged

I've taken the liberty of updating the branch so it can be merged


SARS and COVID-19 death proportions are identical

On https://covid19dashboards.com/comparitive-analysis/#Deaths, SARS and COVID-19 proportions are identical. I assume this is an error, as e.g., Italy had 0 deaths to SARS in 2002-2003.

in particular in [12] c_lat_grp is used twice (plot 1 and 3), the second use should be s_lat_grp I assume.

Thanks for reporting this.  I fixed this in c51d916ad15ca005a2510c76da7874021d20fd70
cc: @imdevskp


Fix broken link to Babel's plugin inline-requests in blog post 'JavaS…

…cript Unit Testing Performance'
Summary
Fixes the broken link in JavaScript Unit Testing Performance to the plugin inline-requires
Test plan

thanks!


Decode fix

Fix issues with GET requests in get_authentication_tokens and get_authorized_tokens returns value in bytes and needed to be decoded before response data can be worked with.
The response.content would return an dict with the keys and values encoded;
{b'oauth_token': b'Yh...token...EQ', b'oauth_token_secret': b'YH...token...gO', b'oauth_callback_confirmed': b'true'}

No worries about being new! This looks better, though! Can you please verify that it works in Python 2.7.x as well and I'll be happy to merge in :)

I tested it on python 2.7 in the console, and it works nicely. It seems that in 2.7 it auto converts or ignores the fact that its bytes and will allow the dict to be worked with normally.
The newer fix, with if isinstance() is the appropriate fix and will allow continued support for 2.7 and fix the issue that my teams project experienced.

Alright, thanks for this!

python-tumblpy 1.0.4 is now available via pip


Adding desc content type.

Adding section long_description_content_type to setup.py to fix these errors:
HTTPError: 400 Client Error: The description failed to render in the default format of reStructuredText.
After merging to master, I think we can use these to upload to pypi:
$ pip install --upgrade setuptools wheel twine
$ python setup.py sdist
$ python setup.py bdist_wheel
$ twine upload dist/*whl dist/*gz

Coverage remained the same at 27.862% when pulling 6383b33ec7d62427693f4312cfe3bdb4c81d78e6 on scriptsrc:fix_readme_for_pypi into 78dee83209c16b3cb06fd7a6d77e43ec974c6db4 on Netflix-Skunkworks:master.


Unexpected error when running pub install

sethladd:~/dart/dart_server (master)$ cat pubspec.yaml
name: dart_server
description: A sample command-line application
#dependencies:
unittest: any
sethladd:~/dart/dart_server (master)$ pub --version
Pub 0.5.11+1.r23200
sethladd:~/dart/dart_server (master)$ pub install
The null object does not have a method 'forEach'.
NoSuchMethodError : method not found: 'forEach'
Receiver: null
Arguments: [Closure: (dynamic, dynamic, dynamic) => dynamic]
#0      Object.noSuchMethod (dart:core-patch/object_patch.dart:19:25)
#1      LockFile._parse (file:///Volumes/data/b/build/slave/dart-editor-mac-trunk/build/dart/sdk/lib/_internal/pub/lib/src/lock_file.dart:26:29)
#2      LockFile.LockFile.load (file:///Volumes/data/b/build/slave/dart-editor-mac-trunk/build/dart/sdk/lib/_internal/pub/lib/src/lock_file.dart:15:27)
#3      Entrypoint.loadLockFile (file:///Volumes/data/b/build/slave/dart-editor-mac-trunk/build/dart/sdk/lib/_internal/pub/lib/src/entrypoint.dart:75:12)
#4      Entrypoint.installDependencies.<anonymous closure> (file:///Volumes/data/b/build/slave/dart-editor-mac-trunk/build/dart/sdk/lib/_internal/pub/lib/src/entrypoint.dart:47:74)
#5      Future.Future.sync (dart:async/future.dart:10:31)
#6      Entrypoint.installDependencies (file:///Volumes/data/b/build/slave/dart-editor-mac-trunk/build/dart/sdk/lib/_internal/pub/lib/src/entrypoint.dart:46:12)
#7      InstallCommand.onRun (file:///Volumes/data/b/build/slave/dart-editor-mac-trunk/build/dart/sdk/lib/_internal/pub/lib/src/command_install.dart:15:42)
#8      PubCommand.run.<anonymous closure> (file:///Volumes/data/b/build/slave/dart-editor-mac-trunk/build/dart/sdk/lib/_internal/pub/lib/src/command.dart:69:32)
#9      Future.Future.sync (dart:async/future.dart:10:31)
#10     PubCommand.run (file:///Volumes/data/b/build/slave/dart-editor-mac-trunk/build/dart/sdk/lib/_internal/pub/lib/src/command.dart:65:5)
#11     main.<anonymous closure> (file:///Volumes/data/b/build/slave/dart-editor-mac-trunk/build/dart/sdk/lib/_internal/pub/bin/pub.dart:72:50)
#12     _ThenFuture._sendValue (dart:async/future_impl.dart:265:24)
#13     _FutureImpl._addListener.<anonymous closure> (dart:async/future_impl.dart:190:30)
#14     _asyncRunCallback._asyncRunCallback (dart:async/event_loop.dart:15:17)
#15     Timer.Timer.<anonymous closure> (dart:async-patch/timer_patch.dart:9:15)
This is an unexpected error. Please run
    pub --trace 'install'
and include the results in a bug report on http://dartbug.com/new.

This issue has been moved to dart-lang/pub#570.


Can not run Bert with Colab TPU

I try to upload credentials to TPU
 with open('/content/adc.json', 'r') as f:     auth_info = json.load(f)   tf.contrib.cloud.configure_gcs(session, credentials=auth_info)
I got this error :

InternalError: From /job:tpu_worker/replica:0/task:0:
The filesystem registered under the 'gs://' scheme was not a tensorflow::RetryingGcsFileSystem*.
[[{{node GcsConfigureCredentials}}]]
During handling of the above exception, another exception occurred:
InternalError                             Traceback (most recent call last)
/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/client/session.py in _do_call(self, fn, *args)
1382                     '\nsession_config.graph_options.rewrite_options.'
1383                     'disable_meta_optimizer = True')
-> 1384       raise type(e)(node_def, op, message)
1385

I got the same thing. Yesterday it still worked. Is it possible google changed something in colab?

I got the same thing when I tried authenticating my google cloud storage account with a TPU instance, but when I switched to a GPU based-one it worked fine

go to Cloud IAM  and try to grand permission for TPU service account manually or use gsutil command . it works .
Here is the free TPU service account email :

<EMAIL_ADDRESS>


Deploy a wheel, not just an sdist tarball.

For these reasons.
https://github.com/scikit-hep/scikit-hep-testdata/pull/33#issuecomment-648090503

Done: https://pypi.org/project/uproot4/0.0.20/#files


makefile is not valid on openbsd

it is GNU make, we should use portable POSIX make only.

https://github.com/rain-1/s/blob/970be176aa377f04f3da228865f2af43a5fb82b3/Makefile
please go back to older simpler makefile.
I could not make it work on openbsd.

solved. we dont use makefiles anymore.


fix: config flow errors

Codecov Report

Merging #39 (877186e) into dev (2e9ab8a) will decrease coverage by 0.21%.
The diff coverage is 60.00%.

@@            Coverage Diff             @@
##              dev      #39      +/-   ##
==========================================
- Coverage   91.09%   90.88%   -0.22%
==========================================
  Files           5        5
  Lines         337      340       +3
==========================================
+ Hits          307      309       +2
- Misses         30       31       +1

Impacted Files
Coverage Δ

custom_components/openei/config_flow.py
88.07% <60.00%> (-0.44%)
:arrow_down:


GitLab 16.8 Release

GitLab 16.8 has just been released
16.9 is planned for 2024-02-15

:thought_balloon: Not :100: sure for 16.7 eol and support

Duplicate of #4470


for miss test case

Fixes #ISSUSE_ID.
Changes proposed in this pull request:

Pull Request Test Coverage Report for Build 6074

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 72.507%

Totals

Change from base Build 6073:
0.0%

Covered Lines:
9531

Relevant Lines:
13145

💛  - Coveralls


Documentation error? - Port for Unifi Controller - 8443 or 443

Hi
I have just got unifi-poller up and running in a Docker container - all good.
But I followed the instructions, and specified port 8443 for the Unifi controller (https://<IP_ADDRESS>:8443).  This failed.  Changing it to 443 worked.

Yup. it's 443 on the UDM! (and I think the cloudkey now too) 8443 for most other folks. I'll add some more data to the docs/config file to call this out.

Thanks!
I am on a Cloud Key Gen 2 - with firmware 2.0.26 running the new(?) Unifi OS
Thanks for all
Andrew

On 2 Feb 2021, at 11:56, David Newhall<EMAIL_ADDRESS>wrote:
Yup. it's 443 on the UDM! (and I think the cloudkey now too) 8443 for most other folks. I'll add some more data to the docs/config file to call this out.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/unifi-poller/unifi-poller/issues/306#issuecomment-771584685, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASPZL3VQGGXKZUL4OAEERF3S47R53ANCNFSM4W6QBO5Q.

I just took the update tonight:
UniFi-CloudKey-Gen2-Plus, Firmware | 2.0.27
Unifi Network Controller, Current Version 6.0.45 (Build: atag_6.0.45_14358)
The update deleted my unifi-poller account. I had to re-add it in the CloudKey users area, and I had to change the port from 8443 to 443 in the up.conf to establish communication. I set the new account to local admin, unifi network read only.
Hope this helps someone.

This has been heavily flagged on the new docs.


Entity API Approach

I noticed in your post on the Cesium blog that you mentioned trying the Entity API before shifting to the WebGL/custom primitive approach.  I'm curious if you would be willing to share the Entity API approach that you tried?
Additionally, apart from performance with high numbers of particles, did you encounter any other issues with the Entity API Approach?
Thanks in advance!

Oh, I forgot some detail.
What I remember is that I use the entity PathGraphics to draw a moving line. You can either create PathGraphics objects directly or generate a CZML file which contains all the entity data and load it into Cesium.

Additionally, apart from performance with high numbers of particles, did you encounter any other issues with the Entity API Approach?

Apart from performance, I am satisfied with Entity API since it is easier to understand and use

Ok, thank you for the information.  I was heading down the same path with the entity PathGraphics, similar to the sandcastle Interpolation demo.
One issue I'm running into is that I don't seem to be able to decouple the animation of the entity/path from the "main" Cesium clock.  So if the scene is "paused", the paths don't continue to animate.
The CustomDataSource allows provisioning of a clock, but I'm not having much luck getting it to work.
Do you have any experience with this aspect of things?  I'm planning on posting to the Cesium Forum as well, but figured I'd ask you too!

Prior to posting, I searched the forum and came across this post
I think this is the solution I'm looking for, but if you have any additional thoughts, I'm definitely interested!
By the way - I never mentioned it ☝, but my reason for pursuing the Entity API approach is that we have a requirement for the application to work on mobile devices.  In reading some of the closed issues it sounds like modifying the demo to work on mobile (i.e. removing the EXT_frag_depth extension) is a complicated change.  Is that still the case?

In reading some of the closed issues it sounds like modifying the demo to work on mobile (i.e. removing the EXT_frag_depth extension) is a complicated change. Is that still the case?

yes, and it seems that it will take a long time for most mobile devices to have a complete WebGL2 support


Content Ideas & Drafts

This spreadsheet has ideas listed by theme and topic for potential blog post articles.
If you would like to take a crack at drafting some of these articles please feel free to do so, also notify us on here if you are so that we can make sure that we are getting adequate coverage.
This spreadsheet is subject to change as more info on the messaging comes in, but you are safe to start drafting your take on most of these topics if you'd like. if you have any questions, feel free to tag me.

Provide link to spreadsheet plz

woops, ok description updated with the spreadsheet

Thanks @patrick727 . I'll make an attempt at some of the topics. To scale this effort to the broader community you may want to consider implementing the 'Community Reward Program' I drafted.

Hi Patrick - I would be interested in curation and authorship of Identity
related articles and papers. This is a big subject so we would want to set
some parameters and RChain specific direction for the effort. Let me know
how you would want to proceed.
Cheers,
On Tue, Aug 8, 2017 at 4:51 PM, patrick727<EMAIL_ADDRESS>wrote:

Assigned #67 https://github.com/rchain/Members/issues/67 to @Ken-dahl
https://github.com/ken-dahl.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rchain/Members/issues/67#event-1198150989, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AaCFaX4pZXD4Lqp4KXYG2Slfp6RwwhNQks5sWOaOgaJpZM4OxY1E
.

--
Kendall Williams
Divvy DAO Ltd.
Proven Secure Solutions, Inc.
office:<PHONE_NUMBER>
Cell:<PHONE_NUMBER>
<EMAIL_ADDRESS>Confidentiality Statement:
This email/fax, including attachments, may include confidential and/or
proprietary information and may be used only by the person or entity to
which it is addressed. If the reader of this email/fax is not the intended
recipient or his or her agent, the reader is hereby notified that any
dissemination, distribution or copying of this email/fax is prohibited. If
you have received this email/fax in error, please notify the sender by
replying to this message and deleting this email or destroying this
facsimile immediately.

Thanks Patrick, exactly which topic do you think I could contribute most to? Row number 67 is empty.

On 09-Aug-2017, at 4:21 AM, patrick727<EMAIL_ADDRESS>wrote:
Assigned #67 https://github.com/rchain/Members/issues/67 to @rishabhkapoor8711 https://github.com/rishabhkapoor8711.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/rchain/Members/issues/67#event-1198150987, or mute the thread https://github.com/notifications/unsubscribe-auth/AZmIAB8ng6_CxuIs2TnGgQStfRegUN5Cks5sWOaOgaJpZM4OxY1E.

If you are not confident in your writing abilties, in particular to these topics, please don't worry drafting the copy, because we could better use skillsets else where. It would be nice to simple input some of your ideas to help fill this out further.
@rishabhkapoor8711 I apologize I meant to link someone else.
@kdvalentine yes the drafting, feedback, and editing are all three great subtasks that can take place in such a reward program. As I remember you are interested int he business development side of things, today in the debrief Greg and Ed both have mentioned music related plays for decentralization. if you are confident in your writing an article that doubles as market research and presentation for music DApps.
It could a long form post that goes indepth into how the internet disrupted the industry, its current business models, and then the business models of the future and particular how Rchain will enable these specific DApps. or any other topic that you feel passionate about really.
@Ken-dahl hey! thats great, and you've got the in through divvy with more interviews, we could leverage the interviews with Andreas Fruend, set one up with Rouven(lead at uport), who just showed interest in RChain(sovrgn, diglife ,etc). We could do essentially the same formula, history of web identity, whats broken about it, current solutions, and then the new solutions coming out and how RChain plans to enable them(could be an interview with Ed and Glenn). We could go in depth about why your identity is needed. If you would like to start out in the sheet, we can list out all of the potential topics and subtopics within identity and then propose some content ideas.
Note* i'm going to change this to content ideas & drafts issue, because these theme's, topics, and messaging will dessiminate through all of our channels and can come out in many forms. Please input ANY idea for content that engages the community in the sheet, including interviews such as the identity meeting and the work study group, to ebooks, to AMA's to meme's etc, etc.

More than happy to do a few content articles for Rchain. Do I need to pick something from the Content Ideas Sheet or can I go with my own?

@traviagio either or, if you have your own idea, please add it to the content sheet, change the field color to yellow and comment your working on it.


Postgres Compatibility

[x] Sample database.yml
[x] Installation instructions in readme

Why this feature has not been integrate since its request ?
PostGreSQL is suitable for that kind of apps and could host big data ?

Why this feature has not been integrate since its request ?
PostGreSQL is suitable for that kind of apps and could host big data ?


Rework security flow

The PR addresses some issues I had when trying to use connexion for a production-ready product.

Use token_info['sub'] instead of token_info['uid'] to better align with RFC7662 (oauth token introspection)
x-tokenInfoUrl is only kept for compability reasons. IMHO it's only useful for a small amount of people, because it hardcodes bearer authentication (to the authorization server) and the response format is not standardized. I suggest providing an example instead, how RFC7662 can be used with a custom x-tokenInfoFunc.
validate_token_info was not customizable and simply compared two sets of strings.
In our setup, we want to add more logic, e.g. scope 'ressource' is superior to 'ressource:read'. One can now set a custom scope validator.
If security definitons don't match the connexion implementation (only one definition supported), connexion silently ignors ALL definitions and allows full access to the api.
Basic authentication / apiKey authentication had to be done manully as function decorator instead of using the security definitions from the swagger definition. Now one can define callbacks similar to x-tokenInfoFunc.
Bonus: If both, oauth and basic auth / apiKey is specified for an operation, the new x-basicInfoFunc / x-apikeyInfoFunc also receives the required oauth scopes. This allows using oauth scopes for authorization while using basic auth / apiKey for authentication.

It's not yet fully tested and documentation/examples are not updated yet, but I would like to get some general feedback on that approach.

Do you have any objectives on breaking changes?
Especially the change in FlaskRequestContextProxy. It's not really needed, but aligns better to flasks recommended approach on storing data.
What do you think about keeping/abandoning x-tokenInfoUrl?
I'm currently thinking about creating the verify_* callbacks on api level (instead of creating them for each operation). They only depend on the security_scheme, scope is passed later at call-time.
That would also fix auth_all_paths for openapi 3, which currently uses AbstractAPI.security_definitions, which is only set for swagger 2.

Do you have any objectives on breaking changes?

We don't have a hard policy about breaking changes, but 2.0 already includes big backward incompatible changes, and any new breaking change will make it harder for people to migrate.

What do you think about keeping/abandoning x-tokenInfoUrl?

I would keep it with a DeprecationWarning to minimize disruption, and then we can remove it at a later time (what do you thing @hjacobs).

Updated documentation and examples.

👍

Do you want me to squash again and also add a note about the change in flask.request?

@cziebuhr That would be perfect.

Done and updated readme.

👍


Getting error while running docker

While running the docker-compose getting this error.
➜  docker-elk-filebeat git:(master) docker-compose build
ERROR: Named volume "f:\logs:/logs" is used in service "logstash" but no declaration was found in the volumes section.
Can you please have a look.

@parameshnalla  You should change this one where your logs stored for example I changed to - ./logs:/logs. It worked well.

Thank you @atahanceylan


Support cc_library

https://bazel.build/reference/be/c-cpp#cc_library

[x] Add example and fixture test: https://github.com/buildbuddy-io/rules_xcodeproj/pull/133
[x] Support alwayslink
[x] Support defines
[x] Support include_prefix: https://github.com/buildbuddy-io/rules_xcodeproj/pull/1656
[x] Support includes: #310
[x] Support implementation_deps: https://github.com/buildbuddy-io/rules_xcodeproj/pull/1664
[ ] Support linkstatic
[x] Support local_defines
[x] Support nocopts (support removed in Bazel, so not doing anything for it un rules_xcodeproj)
[x] Support strip_include_prefix: https://github.com/buildbuddy-io/rules_xcodeproj/pull/1656

@brentleyjones As I mentioned in Slack, the issue with keith/swift-syntax-bazel not working is related to rules_swift support for cc_library being consumed as a Swift module, if it is tagged. Do you want me to track that work on this issue? If not, I will create another one.

No need for a new issue, you can relate the PR to this one and update the description to link back to that PR. Thanks!

We are missing an example for linkstatic, but unless someone reports an issue for it, I'll assume we support it.


Addressed an issue with class members

Class members that were not parameters caused an error (regression
in 2.4.0). This addresses this issue.
This patch also clarifies and makes explicit that modifying class
variables is not safe; an error message is now raised.

Needs test. Please look it over though and let me know what you think. In particular, the exclusions that I avoid for the class variables.


bug arrise in b0606b54d0e7abde5676ba4914b5d339b6c53618

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to navigation panel
Click on 'all navigation buttons'
See all buttons are not working due to preventDeafault.

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. Mint20.1]
Browser [e.g. chrome]
Version [e.g. 89]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Bug in the b0606b54d0e7abde5676ba4914b5d339b6c53618

got it. :+1:

Great, Work on it


  I have proxy settings configured in vscode, already.

I think cargo needs an HTTP_PROXY environment variable.

See https://doc.rust-lang.org/cargo/reference/config.html#httpproxy.

@lnicola I have the same issue. Cargo is working. It's the plugin installation which doesn't work. Even with environment variables set properly, and the proxy in the settings.json.

Also this should ideally not fail silently.

See https://code.visualstudio.com/docs/setup/network#_proxy-server-support for proxy configuration in Code.

@lnicola It's still not working. The extension marketplace works, the C/C++ extension can download its language server. Only the rust-analyzer plugin fails (silently, at least some feedback would be nice, where it's stuck, but that's a UX issue with almost every VSCode extension).

Does the plugin start? What OS/platform are you using?

I mean it starts, it says it couldn't find the language server 0.2.182 even if I download it directly to the folder mentioned in the install guide with:
curl -L https://github.com/rust-analyzer/rust-analyzer/releases/0.2.182/download/rust-analyzer-linux -o ~/.config/Code/User/globalStorage/matklad.rust-analyzer/rust-analyzer

I'm on
code 1.45.1 5763d909d5f12fe19f215cbfdd29a91c0fa9208a x64
rustc 1.43.1 (8d69840ab 2020-05-04)
cargo 1.43.0 (2cbe9048e 2020-05-03)
rust-anaylzer 0.2.182

You might want to run chmod +x ~/.config/Code/User/globalStorage/matklad.rust-analyzer/rust-analyzer.

Thanks, but still it's asking for version 0.2.182

Also, the https://github.com/rust-analyzer/rust-analyzer/releases/0.2.182/download/rust-analyzer-linux URL from your cURL invocation returns 404.

Ah yeah, I tried with latest again, which worked, but then it doesn't recognize it as version 0.2.182

@Veetaha I'm not really familiar with the update process. I've noticed that the extension version changes after installation, but 0.2.182 seems to be the latest one.
@SoftwareApe do you have any screenshots or logs showing the issue? I assume the rust-analyzer binary runs fine from the command line.

@lnicola On the latest version I can run rust-analyzer --version, which yields rust-analyzer fbb8b88. Otherwise there's no other feedback from the extension.
the only thing I can screenshot is that it asks me if I want to install it. But no matter if I click Download Now or X, the same popup reappears and there's no progress bar or failure report.

There should be logs in Help  > Toggle Developer Tools > Console.
However, you cannot install the extension to the directory specified like that. The extension stores the binary version in its internal persistent state.
As a workaround you should explicitly set rust-analyzer.serverPath. I saw the code of vscode-cpptools where they deal with http proxies but didn't wonder what was that and why this compexity is required...

@Veetaha Using the manual serverPath saved me for now. I hope we can port the vscode-cpptools proxyserver solution.

@SoftwareApe would you share the logs from Help > Toggle Developer Tools > Console for more observability on the error?

It seems vscode-cpptools uses this library:
https://github.com/TooTallNate/node-https-proxy-agent

@Veetaha  I get the following log
abstractExtensionService.ts:396 Activating extension 'matklad.rust-analyzer' failed: Got response 403 when trying to fetch release info for 2020-05-25 release.
_logMessageInConsole	@	abstractExtensionService.ts:396

The 403 error is #3688. Beware of the crates extension, since that also tends to fire a lot of API requests.

@SoftwareApe
Sorry, I forgot that logging is disabled by default, could you please recheck with
{
    "rust-analyzer.trace.extension": true
}

@empath-75, @SoftwareApe are you still having issues?

@empath-75, @SoftwareApe are you still having issues?

@lnicola since i haven't use the extension in a while i haven't had this issue.

@lnicola since i haven't use the extension in a while i haven't had this issue.


maybe typo

@hecongy : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Remove netstandard1.x assets from SBRP

Many projects in the source-build-reference-packages repository currently target netstandard1.x. Targeting netstandard1.x is now discouraged with the latest .NET SDK and we should remove those from SBRP.
This issue is blocked until all repositories that consume SBRP removed their netstandard1.x assets.

@MichaelSimons would you mind sharing an up-to-date list of the netstandard1.x packages that are still consumed and the components that bring them in?

These repos still bring netstandard1.x packages in:

[ ] Runtime: https://github.com/dotnet/runtime/pull/104139
[ ] Roslyn: https://github.com/dotnet/roslyn/pull/74199
[ ] NuGet: https://github.com/NuGet/NuGet.Client/pull/5885
[ ] Roslyn-analyzers
[ ] Sdk
[ ] Source-build-externals
[ ] VSTest

Repositories that eliminated their netstandard1.x packages (for at least source-build) should then set this switch so that this doesn't regress over time: https://github.com/dotnet/roslyn/blob/dabd07189684b5cda34b3072326a12b18301a012/eng/Versions.props#L86-L87.


Shadowrun Sixth World v.86 - 1 Bugfix, 1 New Feature

[x] The pull request title clearly contains the name of the sheet I am editing.

[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).

[x] The pull request makes changes to files in only one sub-folder.

[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

[x] I have followed the Character Sheets Standards when building this sheet.

[ ] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Bugfix - Essence Mod works again
Added Omega, Exoware and Gammaware to Augmentation grade drop downs and essence calculations.  They do not however add the bad luck quality or extra condition monitor boxes when selected.

Character Sheet Info Roll20 Internal Use only.