NoMethodError: undefined method `remote' for nil:NilClass

What did you do?
I ran the command /Users/loberhub/.rbenv/versions/2.1.6/bin/bundle

What did you expect to happen?
I expected Bundler to...package

What happened instead?
Instead, what actually happened was...it crashed with the error below after having completed bundling.

Error details
NoMethodError: undefined method `remote' for nil:NilClass
  /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.4/lib/bundler/source/rubygems.rb:388:in `fetch_gem'
  /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.4/lib/bundler/source/rubygems.rb:192:in `cached_built_in_gem'
  /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.4/lib/bundler/source/rubygems.rb:175:in `cache'
  /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.4/lib/bundler/runtime.rb:113:in `block in cache'
  /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.4/lib/bundler/runtime.rb:110:in `cache'
  /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.4/lib/bundler/cli/install.rb:108:in `run'
  /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.4/lib/bundler/cli.rb:162:in `install'
  /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.4/lib/bundler/vendor/thor/lib/thor/command.rb:27:in `run'
  /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.4/lib/bundler/vendor/thor/lib/thor/invocation.rb:126:in `invoke_command'
  /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.4/lib/bundler/vendor/thor/lib/thor.rb:359:in `dispatch'
  /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.4/lib/bundler/vendor/thor/lib/thor/base.rb:440:in `start'
  /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.4/lib/bundler/cli.rb:10:in `start'
  /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.4/bin/bundle:20:in `block in <top (required)>'
  /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.4/lib/bundler/friendly_errors.rb:7:in `with_friendly_errors'
  /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.4/bin/bundle:18:in `<top (required)>'
  /Users/loberhub/.rbenv/versions/2.1.6/bin/bundle:23:in `load'
  /Users/loberhub/.rbenv/versions/2.1.6/bin/bundle:23:in `<main>'

Environment
Bundler   1.10.4
Rubygems  2.2.3
Ruby      2.1.6p336 (2015-04-13 revision 50298) [x86_64-darwin14.0]
GEM_HOME  /Users/loberhub/projects/chopin/vendor/bundle/ruby/2.1.0
GEM_PATH
Git       2.4.4

Bundler settings
path
  Set for your local app (/Users/loberhub/projects/chopin/.bundle/config): "vendor/bundle"
disable_shared_gems
  Set for your local app (/Users/loberhub/projects/chopin/.bundle/config): "1"
jobs
  Set for your local app (/Users/loberhub/projects/chopin/.bundle/config): "4"
cache_all
  Set for your local app (/Users/loberhub/projects/chopin/.bundle/config): "true"
cache_all_platforms
  Set for your local app (/Users/loberhub/projects/chopin/.bundle/config): "true"

Can you share the output from bundle env?

I think this is a duplicate of #3158.
I have a hypothesis of what the problem is:
This failure occurs when a gem is included in the Gemfile that is already built into ruby. I was trying to work around gems that are no longer included in ruby 2.2 but bundling in ruby 2.1.6. For me it was test-unit. Does that sound right?

This example from #3158 reproduces my problem exactly:
mkdir /tmp/bundler_repro

# I am using rbenv, plz adjust to your setup
echo "2.1.6" > /tmp/bundler_repro/.ruby-version

# This gem version matches exactly the version number of io-console
# in the ruby 2.1.6 builtin gemset.
# Probably explains the presence of 'cached_built_in_gem' in the callstack.
echo "gem 'io-console', '= 0.4.3', :require => false" > /tmp/bundler_repro/Gemfile

# boom
cd /tmp/bundler_repro && bundle package

- What did you do?

  I ran the command `/Users/loberhub/.rbenv/versions/2.1.6/bin/bundle package`

- What did you expect to happen?

  I expected Bundler to...

- What happened instead?

  Instead, what actually happened was...

Error details

    NoMethodError: undefined method `remote' for nil:NilClass
      /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.6/lib/bundler/source/rubygems.rb:390:in `fetch_gem'
      /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.6/lib/bundler/source/rubygems.rb:194:in `cached_built_in_gem'
      /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.6/lib/bundler/source/rubygems.rb:175:in `cache'
      /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.6/lib/bundler/runtime.rb:113:in `block in cache'
      /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.6/lib/bundler/runtime.rb:110:in `cache'
      /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.6/lib/bundler/cli/install.rb:108:in `run'
      /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.6/lib/bundler/cli/package.rb:32:in `install'
      /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.6/lib/bundler/cli/package.rb:16:in `run'
      /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.6/lib/bundler/cli.rb:265:in `package'
      /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.6/lib/bundler/vendor/thor/lib/thor/command.rb:27:in `run'
      /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.6/lib/bundler/vendor/thor/lib/thor/invocation.rb:126:in `invoke_command'
      /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.6/lib/bundler/vendor/thor/lib/thor.rb:359:in `dispatch'
      /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.6/lib/bundler/vendor/thor/lib/thor/base.rb:440:in `start'
      /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.6/lib/bundler/cli.rb:10:in `start'
      /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.6/bin/bundle:20:in `block in <top (required)>'
      /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.6/lib/bundler/friendly_errors.rb:7:in `with_friendly_errors'
      /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0/gems/bundler-1.10.6/bin/bundle:18:in `<top (required)>'
      /Users/loberhub/.rbenv/versions/2.1.6/bin/bundle:23:in `load'
      /Users/loberhub/.rbenv/versions/2.1.6/bin/bundle:23:in `<main>'

Environment

    Bundler   1.10.6
    Rubygems  2.2.3
    Ruby      2.1.6p336 (2015-04-13 revision 50298) [x86_64-darwin14.0]
    GEM_HOME  /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0
    GEM_PATH  /Users/loberhub/.rbenv/versions/2.1.6/lib/ruby/gems/2.1.0:/Users/loberhub/.gem/ruby/2.1.0
    Git       2.4.4
--- TEMPLATE END ----------------------------------------------------------------

Unfortunately, an unexpected error occurred, and Bundler cannot continue.

First, try this link to see if there are any existing issue reports for this error:
https://github.com/bundler/bundler/search?q=undefined+method+%60remote%27+for+nil%3ANilClass&type=Issues

If there aren't any reports for this error yet, please create copy and paste the report template above into a new issue. Don't forget to anonymize any private data! The new issue form is located at:
https://github.com/bundler/bundler/issues/new

And here is bundler env:
Environment

    Bundler   1.10.6
    Rubygems  2.2.3
    Ruby      2.1.6p336 (2015-04-13 revision 50298) [x86_64-darwin14.0]
    Git       2.4.4

Gemfile

    gem 'io-console', '= 0.4.3', :require => false

Gemfile.lock

    GEM
      specs:
        io-console (0.4.3)

    PLATFORMS
      ruby

    DEPENDENCIES
      io-console (= 0.4.3)

    BUNDLED WITH
       1.10.6

Fixed by https://github.com/bundler/bundler/pull/3896.


Recognition not so accurate

Hello bro, its great effort to build the application.
But some times the app behaves wearied  , will recognize unknown persons aswel and for a single person the app is returning two names . How can i compare the best one among those two names ?
i am always getting confidence 0.0 for all faces.
I have tried changing
if (length(face_descriptors[i]-rec_face_descriptors[j]) > 0.1)
up to 1.5
and i have also tried increasing INPUT_SIZE
Can you please help me to fix this issue . Thank you in advance

bro mine is not even recognizing can you tell how you achieved


remove outermost parentheses in C

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
Remove outermost parentheses
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
(Write your answer here.)
#955

@iamrajiv please have a review at my PR

@atarax665 @Abhijit2505 could you please review my PR

@atarax665  i have made the required changes. Please verify it

@atarax665 i have made the required changes. Please verify it

@atarax665 @Abhijit2505 please review my pr

@atarax665 @Abhijit2505 @Kajol-Kumari @SKAUL05 please review my PR


Add marko support to storybooksJS

What I did
Marko support for storybooksJS.
How to test
Is this testable with Jest or Chromatic screenshots?
Not sure.
Does this need a new example in the kitchen sink apps?
Yes, i have added marko-cli in examples folder.
Does this need an update to the documentation?
It does. I have added some preliminary docs (basically just copied from other apps and edited/added some basic stuff). Will keep adding as we go.
Do let me know what might be missing. I still had a quick question around arranging my stories in marko-cli, and once i get that answered i might change up some stuff. Also, couldn't add any tags because i don't think i have permissions for the same.

related: https://github.com/storybooks/storybook/issues/2841

Working on lint erros 👍

Please integrate the changes from https://github.com/storybooks/storybook/pull/3472

Hello all, I think i have taken care of most issues. After the new refactor the render.js has indeed become a lot simpler.
Few open items i can think of which i might need some help with

Should my package.json version for @storybook/marko be 4.0.0-alpha.4? If i change it to that and do a yarn bootstrap i get some error

error An unexpected error occurred: "https://registry.yarnpkg.com/@storybook%2Fmarko: Not found".

Live examples (netlify) ?
Probably also the "CLI" integration ?
Don't see any examples for tests in other apps?
Add config.js for hierarchal story separation. (right now it displays the separator '|')?  What exactly is this, and why do we need it?

Hey guys, i think i have got it to a point where i knew what needs to fixed. Let me know if we're good to go. Feel free to slack me (or #markojs) if thats quicker.

Plus https://github.com/storybooks/storybook/pull/3520 please

Live Example
@nm123github "no preview" story shows an error https://deploy-preview-3504--storybooks-marko.netlify.com/?selectedKind=Hello&selectedStory=with No Preview!&full=0&addons=1&stories=1&panelRight=0&addonPanel=storybook%2Factions%2Factions-panel

Also, thanks @Hypnosphi for taking care of some of the pending items.

Let me know if we're good to go

Regarding the addons, if you think you won't get to them in this PR, let's remove the unneeded deps in the example. Also, configuring Storyosurce is easy, so probably worth adding it here.

Congrats @storybooks/team @nm123github Great work!

released as 4.0.0-alpha.6


Updates to AdaNet tutorial notebooks

@cweill I made some updates to the tutorials. I'm not sure if there's a good way to diff .ipynb files, but heres what I added:

In both notebooks: a cell with pip install adanet for folks running these in Colab
In adanet_objective: a caveat that exact MSE may vary depending on where you run training
In customizing_adanet: I added a cell to generate predictions on the first 10 values from the test set. Also added a predict input_fn to the cell where you create the dataset.

Thank you for this PR, @sararob.


[SPARK-15319][SPARKR][DOCS] Fix SparkR doc layout for corr and other DataFrame stats functions

What changes were proposed in this pull request?
Doc only changes. Please see screenshots.
Before:
http://spark.apache.org/docs/latest/api/R/statfunctions.html

After

(please ignore the style differences - this is due to not having the css in my local copy)
This is still a bit weird. As discussed in SPARK-15237, I think the better approach is to separate out the DataFrame stats function instead of putting everything on one page. At least now it is clearer which description is on which function.
How was this patch tested?
Build doc

@shivaram @sun-rui

Test build #58596 has started for PR 13109 at commit 756262c.

LGTM. Thanks @felixcheung

Test build #58596 has finished for PR 13109 at commit 756262c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58596/
Test PASSed.

This looks better. but the roxygen style is a little bit deviated. The previous is like:
#' function name
#' description
Current is like:
#' function name - description
We may need a consistent roxygen style documentation. At least for two styles:
one function for one RD
multiple functions for one RD
And also if you type '?corr' in R, only corr() for Column functions is displayed.  Since R is function oriented, I think two corr() descriptions better to be displayed together in one page?

right - that's why my comment on SPARK-15237 is that we should have a different rd for each of "statsfunctions" instead of having all of them on one rd. To clarify, currently, we have

column function corr in its rd file
DataFrame function corr, crosstab, cov, freqItems in one rd file

What I think we should have instead is

DataFrame function corr in one rd file
DataFrame function crosstab in one rd file etc..
column function corr should group with other column functions (or least column stats functions) in one rd file with appropriate "multiple functions in one rd" formatting as you have suggested

I think it is rather confusing to put both DataFrame corr on the same pages as column function corr?
Thoughts?

My opinion is that since R supports generic function and a generic function can have multiple methods for it, it is natural to put both corr() in the same page. It is a mechanism that descriptions can be aggregated even if methods of the same name are distributed in different RD files?

that's fine, I don't know the history of putting stats column function into one rd page though.
I agree it is fine to group function by the name corr DataFrame & corr column going to the same rd page.

Chiming in a little late here -- from my R usage, I've definitely seen two patterns commonly used

sharing the same function for multiple generics and just documenting how they differ for each input (for example something like https://stat.ethz.ch/R-manual/R-devel/library/base/html/rowsum.html)
collating multiple related functions in the same rd file (example https://stat.ethz.ch/R-manual/R-devel/library/stats/html/cor.html)
So I am not sure that having a separate rd file for each (function, input type) pair is the right approach.

Right now my take is the following:

For more complex functions like corr (i.e. where we have DataFrame and column versions), lets group by function name and lets exclude them from the generic stats column rd file
For simple functions like mean or sum lets keep them in a stats functions rd.

Let me know what you think of this proposal.

@shivaram, I checked your two examples. It seems that the rule is that:
For a generic function, methods for it are documented together
For a non-generic function (isGeneric() returns FALSE), methods of related functionalities can be documented together.
I have no strong opinion on this. so +1

@felixcheung Is this PR still relevant ?

I think it does. I will try to update this today.

Updated cov, corr into their rd page and kept others.

Test build #60897 has started for PR 13109 at commit df0851c.

Test build #60897 has finished for PR 13109 at commit df0851c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60897/
Test PASSed.

cc @dongjoon-hyun

Test build #60905 has started for PR 13109 at commit 9104bbc.

Oh, the root cause exists in generics.R. Nice catch!

It's nasty! 😄

In line 333 of functions.R, @rdname covar_pop -> @rdname cov?
#' covar_pop
#'
#' Compute the population covariance between two expressions.
#'
#' @rdname covar_pop
#' @name covar_pop

That's intentional - covar_pop has a separate page.
cov == covar_samp
putting covar_pop on the same rd would have 2 different descriptions here. (I think we really should avoid putting multiple things in one rd..)

Yes. Indeed, we had better keep each function on own RD generally.

Test build #60905 has finished for PR 13109 at commit 9104bbc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #60908 has started for PR 13109 at commit 10a4dba.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60905/
Test PASSed.

Test build #60908 has finished for PR 13109 at commit 10a4dba.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60908/
Test PASSed.

Test build #60910 has started for PR 13109 at commit 3760d03.

I just checked the generated R doc and I felt that we shouldn't group many methods together. For example, in this PR, the DESCRIPTION section looks okay because we used crosstab - ... and freqItems - .... But the Arguments section becomes very messy and the Value section is even worse. The example code are chained together. See attached screenshots. summary is another example here. It might be better to only group methods are closely related to each other and share the same arguments.

LGTM. This version looks good to me. Thanks for iterating on this. Will wait for Jenkins and then merge.

@mengxr Yes - this is true and in #13798 we are making a few more of the methods into individual Rd files.  At a high level there is a tradition in R to group together similar methods (https://stat.ethz.ch/R-manual/R-devel/library/base/html/colSums.html for example) but with roxygen2 that leads to some issues.
I think we can even split the statfunctions ones as they are named differently -- For summary I think its more tricky as the function name is the same ?

Methods documented in colSums share the same parameters and each was only documented once. Roxygen2 supports that if each param doc only appears once in the comment. That grouping looks okay to me, which is different from statfunctions or summary.
For summary, I'm thinking about documenting the corresponding summary methods in the fit methods such as spark.glm, spark.naiveBayes. And then put see alsos in the generic summary doc page.

Test build #60910 has finished for PR 13109 at commit 3760d03.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60910/
Test FAILed.

@mengxr @felixcheung Can we open a new issue of the form Separate out rd files for SparkR functions ? We can then make a list there of everything thats sharing a rd file right now and see what is the way around things.
@mengxr - also let me know if you think this is good to merge. I think for 2.0 RC1 having this PR is better than not having it ?

Jenkins, retest this please

Right @shivaram @mengxr I agree it would be nicer to put predict, summary or even write.ml to their corresponding mllib methods.

Test build #60912 has started for PR 13109 at commit 3760d03.

Yes, we should merge this PR first and discuss the grouping later.

Created https://issues.apache.org/jira/browse/SPARK-16090 to follow up.

Test build #60912 has finished for PR 13109 at commit 3760d03.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Cool. Thanks - LGTM. Merging this to master, branch-2.0

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60912/
Test PASSed.

Merged build finished. Test PASSed.


Update variables.tf

Apparently k8s only support 3 entries for DNS servers.
Check the validation here:  https://github.com/kubernetes/kubernetes/blob/c970a46bc1bcc100bbbfabd5c12bd4c5d87f8aea/pkg/apis/core/validation/validation.go#L2944-L2953
To avoid these warning "Nameserver limits were exceeded, some nameservers have been omitted, the applied nameserver line is: <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS>"
randomly removed the last entry ;)

@Wuppie007 Thanks for this. Please don't hesitate to tweak the kube.tf.example file, and the description of the variable to inform of that limit.
If you are feeling adventurous, see how validations are done for some other variables (or just use chatgpt 4 🤐) to add a validation for 3 max values, it's just another attribute in the variable object.

I would argue to use both cloudflare's DNS servers and one Google, or even Quad9 as the third instead of Google

Updated now for three different companies: ["<IP_ADDRESS>","<IP_ADDRESS>","<IP_ADDRESS>"]

Released in v2.0.2, thanks for your contribution both. FYI, I finally opted for @aleksasiriski's recommendation to put Cloudflare first.


fix: Remaining pages for workspace

Details

Follow up https://github.com/Expensify/App/pull/6543
Fixed Issues

$ https://github.com/Expensify/App/issues/6496
Tests | QA Steps
Same as preceding PR.
Tested On

[x] Web
[x] Mobile Web
[x] Desktop
[x] iOS
[x] Android

Screenshots

Web

Mobile Web

Desktop

iOS

Android

https://user-images.githubusercontent.com/24370807/144223270-4ff0800a-0ab4-42d5-8e2a-aefa181a7534.mp4

@Gonals Please review...

@thienlnam I don't know why E2E test failed but we should be good to merge here.

Hmm, are E2E tests failing on other PRs? If not then I don't think we should merge this.  Going to re-run the test to see if it was just a flakey test

We are good.. @thienlnam

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @thienlnam in version: 1.1.17-8 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@parasharrajat Do we need to do any additional QA for this PR on top of QA for  #6543?
if yes, can you share steps?

I think this PR is possible breaking the VBA flow -> https://github.com/Expensify/App/issues/6620
Did a git bisect and it led me here.

🚀 Deployed to production by @roryabraham in version: 1.1.18-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Refactoring code html and css

Looks great however there seems to be some merge conflicts probably because I merged some pull requests before this one. If you have time to fix it or make a new pull requests I'll happily merge it :)

Great 👍

No problem :)


BNB USDC is 18 decimals

Describe the bug
BNB USDC is 18 decimals, UI should display accordingly.

fixed.


feat(date-picker): adds default-calendar-start-time prop when type is date\datetime\week

type 为 date\datetime\week时，可以设置 default-calendar-start-time 展示面板日历默认开始的月份时间戳
closes #4493

Codecov Report
Attention: Patch coverage is 0% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 60.94%. Comparing base (82fccfb) to head (335fc67).

Files
Patch %
Lines

src/date-picker/src/panel/use-calendar.ts
0.00%
0 Missing and 1 partial :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5701      +/-   ##
==========================================
- Coverage   60.95%   60.94%   -0.01%
==========================================
  Files        1125     1125
  Lines       23875    23876       +1
  Branches     6243     6244       +1
==========================================
  Hits        14552    14552
  Misses       7705     7705
- Partials     1618     1619       +1

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


kata publication is going wrong, sometimes...

Hi,
See here: https://www.codewars.com/kata/unknown-math-symbols/discuss/java
We got another user who's publishing his homework so that we can solve them for him. The question is: how could he even publish the kata, since the solution of the author is doing nothing? My guess is that it comes from a kumite converted to a kata.  If that's the way he did it...
cheers

@Blind4Basics because kata publish never performs validation. It's a very old bug that I reported recently: #1369
Also I've retired the kata. Maybe close this and keep track at #1369?


Visual Design

We have a vibrant community. We need some design/Ux friends to help us with the aesthetic vision

With a top user experience trend being to "search", it would be ideal for the new version to group results, and give consideration to different languages when the results are grouped as well.


Fix/downcast

Fixes the issue reported by @juliusfrost regarding downcasting of int to int

Note that the teSt cases here don't actually fail without the fix because of the UPLC package being too lenient.


Fix LGTM Usless Variable Assigment in C# files

Draft cause there's a lot of them.  Also opening this pr so that lgtm can run through it.
Also if anyone is reading this, my Windows CI seems to have become sad because of
error MSB8036: The Windows SDK version 10.0.14393 was not found. Install the required version of Windows SDK or change the SDK version in the project property pages or by right-clicking the solution and selecting "Retarget solution"
in building ACE.  Did something change that I missed?  Was using Visual Studio 2017, should be trying to add that SDK version in Visual Studio installer?

-DCMAKE_SYSTEM_VERSION=10.0.16299.0, see Mike's PR here: https://github.com/apache/geode-native/pull/717

-DCMAKE_SYSTEM_VERSION=10.0.16299.0, see Mike's PR here: https://github.com/apache/geode-native/pull/717

Codecov Report

Merging #728 (bc3fa6b) into develop (bc3fa6b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #728   +/-   ##
========================================
  Coverage    74.99%   74.99%
========================================
  Files          645      645
  Lines        49779    49779
========================================
  Hits         37331    37331
  Misses       12448    12448

Continue to review full report at Codecov.

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

Codecov Report

Merging #728 (bc3fa6b) into develop (bc3fa6b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #728   +/-   ##
========================================
  Coverage    74.99%   74.99%
========================================
  Files          645      645
  Lines        49779    49779
========================================
  Hits         37331    37331
  Misses       12448    12448

Continue to review full report at Codecov.

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


Cannot find module 'webpack'

Bug report

What is the current behavior?
When running latest webpack (4.29.6) via npm under git bash / command prompt (windows 10) I receive an error
Error: Cannot find module 'webpack'

Running the same repository under Ubuntu (under Windows 10) works fine.
If the current behavior is a bug, please provide the steps to reproduce.
Can be reproduced using the following repo https://github.com/gnesher/phaser-capacitor (bug was reported by a user, I can reproduce it on my machine). It's a repository for a real project but it's tiny and the webpack.config.js is very small / easy to read
To reproduce simply clone, run npm install and then try to run npm run build

What is the expected behavior?
I should be able to run webpack via npm without installing it globally

Other relevant information:
webpack version:  4.29.6/4
Node.js version: 11.12
Operating System: Windows 10
Additional tools:

Can't reproduce, i think problem related to npm/yarn than webpack, please provide more information - npm or yarn, version of package manager, version of windows, thanks

npm v6.7.0
yarn 1.12.1
windows 10 - 10.0.17763 Build 17763

@gnesher still can't reproduce, are you sure problem not related to your environment?

If I run the same project through bash for windows (the Ubuntu shell) in the same folder it works fine so this is probably tied to something with Windows - but I'm not sure if this is just an odd edge case that Webpack isn't handling vs something broken with my Windows installation

macOS Mojave 10.14
npm 6.5.0
node v9.1.0
webpack 4.34.0
webpack-cli 3.3.4

but I use npm install -g<EMAIL_ADDRESS>
Running this is not recommended. webpack should be installed locally, not globally. This should be called as part of a npm script which uses node_modules/.bin/webpack then.

but I use npm install -g<EMAIL_ADDRESS>
Running this is not recommended. webpack should be installed locally, not globally. This should be called as part of a npm script which uses node_modules/.bin/webpack then.

ok！thanks~


[init] Making random deterministic

Adding determinism to the random number generators in the library
DataBuffer has multiple threads but single thread of train/valid/test
which run in sequence in my understanding
Note: this PR does not applies to the applications using their own random number generators
Resolves #167
Signed-off-by: Parichay Kapoor<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #179. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.

:octocat: cibot: @kparichay, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/179-202006161814400.73836898803711-b29fa02137dce3c5065a76716ee2138ea54f2760/.

:octocat: cibot: @kparichay, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/179-202006161820230.37381601333618-8df45ad8f57bf86d53d0ca2a5ad5cac6769bd555/.

:octocat: cibot: @kparichay, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/179-202006161908140.58319306373596-200aba8d5aa1cb57fdc0d23db095e45c920b4c55/.


[Feature Request] Multiple FancyZones

Multiple FancyZones

Hi all,
I would like to have the possibility to activate a whole different layout by holding another set of keys. Example: 1) holding Shift key while dragging a Template previously attributed to that command would appear; 2) holding Ctrl + Shift key while dragging another Template for that command would appear; and so on...
I couldn't find that request anywhere. Sorry, by any means, if this is a duplicated suggestion...
<!--

@Lombrilope
closing as duplicate of https://github.com/microsoft/PowerToys/issues/1411
Thanks.


Not updating in real time.

I have a bicycle that I put insane mass on to test whether it works to edit by setting the mass to normal but it doesn't, is it because it's a bicycle or am I doing something else wrong?

UPDATE: Tried on cars and still didn't work


feat: 🎸 Fixes submit button, reset button, helper text behavior

Apologies for the large PR but I didn't want to split this up into multiple PR's because each would be its own version that would introduce new bugs. The re-work required all of this work to be together. Also of note, no breaking changes were needed.
This PR closes 3 issues, possibly more when I go through the issues to see if they are related.
It Fixes the submit button to be disabled when it shouldn't. Fixes the reset
button to reset to the proper initial errors state. Fixes the helper
text to always be in a correct state.
This PR doesn't affect the buttons on the Stepper component. Those buttons do not use the useSQButton hook because the Stepper component has to account for all the steps and forms as a whole.
In addition to the loom explanations, the Date picker and DateTimePicker were not properly rendering the helper text error icon when in the error state. This PR fixes that slight UI issue as well.
Looms:
https://www.loom.com/share/bb878c53cfb447c38ac144bcab0c8ef0
https://www.loom.com/share/7faa684ed86247248cd419ac7b65b026
https://www.loom.com/share/9a5d63dbaed045f3a42403b90a047298
✅ Closes: #59, #83, #47

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


spring-zeebe-starter is not using the configuration from application.yml

I think there is a problem with spring-zeebe-starter, as it's not working as intended.
It isn't using the configured properties declared in application.yml that maps to io.zeebe.spring.client.properties.ZeebeClientConfigurationProperties.
I think it's just using the defaults configured in: io.zeebe.spring.client.config.ZeebeClientSpringConfiguration#DEFAULT
How to reproduce this problem:
If you start the example (io.zeebe.spring.example.StarterApplication) without having an instance of zeebe's broker running, you'll see this error:
Caused by: io.netty.channel.AbstractChannel$AnnotatedConnectException: Connection refused: /<IP_ADDRESS>:26500

If you change the port in the example's configuration:
zeebe:
  broker:
    contactPoint: "localhost:26700"

The error keeps showing the default value:
Caused by: io.netty.channel.AbstractChannel$AnnotatedConnectException: Connection refused: /<IP_ADDRESS>:26500

Looking in the structure of ZeebeClientConfigurationProperties, the configuration should be:
zeebe:
  client:
    broker:
      contactPoint: "localhost:26700"

But that produces the same wrong result as before.
The problem seems to be that the AutoConfiguration (io.zeebe.spring.client.config.ZeebeClientStarterAutoConfiguration) is doing nothing with
the ConfigurationProperties (io.zeebe.spring.client.properties.ZeebeClientConfigurationProperties)
Am I not understanding or using in a bad way the spring-zeebe-starter or there is a problem with the dependency?
Also, I think there is a problem/discrepancy with the worker default name configuration.
io.zeebe.spring.client.annotation.ZeebeWorker declares this:
String name() default "${zeebe.worker.name}";

But ZeebeClientConfigurationProperties declares this:
zeebe:
  client:
    worker:
      name:

Thanks for your great work

The version tested is 0.4.0, with Spring Boot 2.0.3 and Java 11

Hi @vvalencia-cl,
thanks for reporting this issue. It seems the problem is that the configuration is ignored and instead a default Zeebe client is build
https://github.com/zeebe-io/spring-zeebe/blob/a00daa9e62bccfc796419b0ed59401afd345ee63/client/spring-zeebe/src/main/java/io/zeebe/spring/client/config/ZeebeClientSpringConfiguration.java#L31-L33
https://github.com/zeebe-io/spring-zeebe/blob/a00daa9e62bccfc796419b0ed59401afd345ee63/client/spring-zeebe-starter/src/main/java/io/zeebe/spring/client/config/ZeebeClientStarterAutoConfiguration.java#L21-L25
@jangalinski can you confirm that this not intended behavior of these classes?
Cheers,
Sebastian

Yes, obviously a bug.


Be more rust idiomatic

Many of the binding currently don't really do the idiomatic rust thing. I thought it might be a good idea to start gathering them here, and then hopefully change them if it can be agreed upon:
Connection

get_setup currently panics when the pointer is null. Should return Result instead, and Err when that happens.
flush current returns a bool on failure. The rust way would be to do a Result<()>, with an Err. This way the clippy, or even the compiler itself can indicate a point of failure.

get_setup currently panics when the pointer is null. Should return Result instead, and Err when that happens.

This should be (more or less) unreachable. The only way to get an xcb_connection_t with xcb_get_setup(c) == NULL is when xcb_connect() failed and returned a connection in an error state. Such a connection cannot easily be constructed correctly (connect() certainly cannot and if the error check is moved into from_raw_conn(), then I think it should be entirely impossible to get this).
Well, there is one other possibility...
One could also have a very old version of libxcb. Namely, before https://gitlab.freedesktop.org/xorg/lib/libxcb/commit/c4e40f646b8da4fd112ea54a612c880be5e942a8, `xcb_get_setup` started returning NULL when a connection entered an error state. This commit is part of libxcb 1.12 which was released in May 2016, so is not really "ancient old".

For flush failing: Well... I would argue that no one cares about that error. Especially since error states in libxcb are "sticky" and ones a connection had an error, it will always have it.

v1.0.0 is released.


Added notes for EKS users to watch out for hostnework issue

Signed-off-by: Abel Perez Martinez<EMAIL_ADDRESS>Thank you for helping make the Dapr documentation better!
Please follow this checklist before submitting:

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[ ] Read the contribution guide
[ ] Commands include options for Linux, MacOS, and Windows within codetabs
[ ] New file and folder names are globally unique
[ ] Page references use shortcodes instead of markdown or URL links
[ ] Images use HTML style and have alternative text
[ ] Places where multiple code/command options are given have codetabs

In addition, please fill out the following to help reviewers understand this pull request:
Description
Added notes for EKS users to watch out for hostnework issue
Issue reference
This is a follow up from issue https://github.com/dapr/dapr/issues/4074 on dapr main repo

@msfussell I'd like to, just I'm only familiar with the kind of setup I have which is EKS using Calico as CNI, I'm not aware of any other issues with other CNI plugins including the default VPC CNI.
At the moment my priority is to focus on https://github.com/dapr/dapr/issues/4141 so it can give more flexibility to anyone using the same setup. As I learn more, I'd like to keep adding more to the docs.


Multiple / Repeating fields in a form

Starting to think about how I would build a form that supports multiple fields. Wondered if anyone had any suggestions or thoughts?
IE The user can add 2 addresses
The obvious solution would be to append an index to the field name but that wont scale well. Another thought I had was creating an address component that receives setProperty that would manage its own state and push to an array property on the model but I fear that might be too complicated.

I would personally go with index for these kind of situations. Input has to have a unique name.
What are your concerns about scaling it?
If you need an array of addresses in the model you can try to use middleware configuration to aggregate it from streetAddress1, streetAddress2, etc.

So I think the name is a small part of this, and the more important part is the validation.
Take this sandbox as an example: https://codesandbox.io/s/n90z3o8054
If one wants streetName{n} to be valid if displayed, how would they go about it? In that example, both streetName1 and streetName2 are required so unless the user adds a second street name then the form is invalid.
What about using the <Control ... /> method defined in #49 but with the validators defined there?
<Control name="email" validators=[isRequired] shouldValidate={false} {...this.props}>
    <TextField
        style={{ marginBottom: '20px' }}
        label="Enter your email"
        fullWidth={true}
        error={!!this.dirtyInputError('email')}
        helperText={this.dirtyInputError('email')}
    />
</Control>

Got the problem now, sandbox helped.
sounds like some improvements for form-container v2
:)


Add support for encryption/decryption of data using AndroidKeystore

By request of @AdamBJohnsonx
This PR is work-in-progress

@AdamBJohnsonx Ready for another round 💪

@AdamBJohnsonx I didn't pull this API up into AsymmetricKey (or the associated RSA subinterface) -- trivial to do if you want that in this PR. Otherwise I can save it until we start using this code and we just expose it at the IDevicePoPManager level

@AdamBJohnsonx I didn't pull this API up into AsymmetricKey (or the associated RSA subinterface) -- trivial to do if you want that in this PR. Otherwise I can save it until we start using this code and we just expose it at the IDevicePoPManager level

Ooh, that would be good.

@AdamBJohnsonx I didn't pull this API up into AsymmetricKey (or the associated RSA subinterface) -- trivial to do if you want that in this PR. Otherwise I can save it until we start using this code and we just expose it at the IDevicePoPManager level

Ooh, that would be good.

You got it 👍

@AdamBJohnsonx I didn't pull this API up into AsymmetricKey (or the associated RSA subinterface) -- trivial to do if you want that in this PR. Otherwise I can save it until we start using this code and we just expose it at the IDevicePoPManager level

Ooh, that would be good.

You got it 👍

Pulled up. Added more tests: 789c0b1ff0e1d6b4a06e56f05c1892bf0e39f450


Use accelerate for finetuning model via multi node and multi gpu

Hallo,
Does anyone have suggestion for finetuning a model via slrum on multinode and multigpu. I have seen examples for model train in https://github.com/huggingface/accelerate/tree/main/examples but this a basic example.
But is there any option to combine this for finetuning model since it uses TrainingArguments api from transformers and SFTTrainer api from trl:
training_arguments = TrainingArguments(
        output_dir=model_details["output_dir"],  # directory to save and repository id
        num_train_epochs=num_train_epochs,  # number of training epochs
        ......................................
    )

  trainer = SFTTrainer(
      model=peft_model,
      train_dataset=train_dataset,
      eval_dataset=eval_dataset,
      ................................
  )

Your code won't change, what matters is how you launch the code. So just have your own scripts and then launch using the recommendations for slurm over in the examples https://github.com/huggingface/accelerate/tree/main/examples/slurm

Thanks for your answer i will check it out.

I was able to run model via multi node and multi gpu. Now i need to reduce my batchsize before i was able to train my model on 2 GPUs around 30 batch size, after runing on multi node i have to reduce my batch size to 5, which is very less and i feel, something is wrong.
Is there somthing i need to add changes in code so its traning faster, i am using BitsAndBytesConfig and flash_attention_2
    peft_config = LoraConfig(
        r=256,
        lora_alpha=512,
        lora_dropout=0.25,
        target_modules="all-linear",
        task_type="CAUSAL_LM",
        bias="none",
    )

quantization_config = BitsAndBytesConfig(
          load_in_4bit=True,
          bnb_4bit_use_double_quant=True,
          bnb_4bit_quant_type="nf4",
          bnb_4bit_compute_dtype=torch.bfloat16,
      )

gradient_checkpointing = True

model_kwargs = dict(
    attn_implementation="flash_attention_2",
    torch_dtype=torch.bfloat16,
    use_cache=False if gradient_checkpointing else True,
    device_map=get_kbit_device_map() if quantization_config is not None else None,
    quantization_config=quantization_config,

)

training_arguments = TrainingArguments(
        output_dir=model_details["output_dir"],  # directory to save and repository id
        num_train_epochs=num_train_epochs,  # number of training epochs
        ......................................
    )

  trainer = SFTTrainer(
      model=model_name,
      model_init_kwargs=model_kwargs,
      peft_config=peft_config,
      train_dataset=train_dataset,
      eval_dataset=eval_dataset,
      ................................
  )


fix: tools-2875 display all errors generated while using the validati…

…on command
Using a similar example to the ticket, validating unsupported Aerospike versions now displays an error.
./bin/asconfig validate -a 7.2.0 testdata/cases/server70/server70.conf
INFO[2024-03-27T09:34:25-07:00] Using config file: /etc/aerospike/astools.conf
ERRO[2024-03-27T09:34:25-07:00] error while validating config
ERRO[2024-03-27T09:34:25-07:00] failed to get aerospike config schema for version 7.2.0: unsupported version

Should the schema submodule 7.1.0 update be included in this PR? Maybe it's not a problem assuming you don't need the 7.1.0.json file reviewed

@jdogmcsteezy Good catch, the schema update should only be in the server 7.1 support PR. I don't think adding it here would have caused any issues, maybe a merge confict, but I reset it.


Add new Planet Upload Type

Add upload type to import Planet imagery. The key argument is going to be a list of scene IDs (or maybe URLs to the scene API?). The Airflow job should create scenes in Raster Foundry where the user who initiates the upload.
Also, relevant metadata should be added to the scene as much as possible.

Basic workflow for each scene id is:

initialize planet API client (as client from here)
client.get_item(id='<id>', item_type='REOrthoTile') (I'm assuming we'll usually want REOrthoTiles? I'm not sure how we'd expose item_type)
parse scene from the resulting json (.get() the response)


Date mismatch in all testing report data

The testing status day is mentioned as April instead of March.
https://api.rootnet.in/covid19-in/stats/testing/latest
https://api.rootnet.in/covid19-in/stats/testing/history

Not our API


Create favico for the dashboard

SVG required for the logo.

hello Kevin, is this one for the logo on our dashboard?

@caiyuan1222 I started working on this yesterday. Sorry, should've assigned it to myself.


VueCarousel-inner intermittently stays initialised with visibility=hidden

Once in every 10-20 page loads, approximately, the .VueCarousel-inner div retains a CSS property of visibility=hidden and thereby hides all items even if/when you interact with the (empty) carousel. Resizing the browser rectifies the visibility status.
I'm using vue-carousel v0.18.0-alpha downloaded this week.
Similar behaviour was described in https://github.com/SSENSE/vue-carousel/issues/138#issuecomment-369066595
Normal/correct load:

Intermittent/hidden load:

No errors in the console.
I could add a workaround to trigger $('.VueCarousel-inner').css('visibility', 'visible') on a timeout (which also triggers correctly updated flex-basis etc), but would prefer to make the underlying loading mechanism robust.

I have the same issue. However, I don't like the idea of using an manual timeout workaround.
Btw I think https://github.com/SSENSE/vue-carousel/issues/138#issuecomment-456382817 is a better workaround. At least for me it is.

Hey @DrMeers thanks for flagging! This is really weird. The resizing causes the carousel to recompute it's properties which explains the fix. I'm not really sure why the classes are persisting though.. I'll flag this as a bug and hopefully someone can investigate, otherwise I'll get to it when I can!


Fix PyCharm warnings in staff.py

I had some warnings in my PyCharm IDE so I tried to fix them up.
I also ran the quicktest.py and get 3 failures + 1 error but they doesn't seem to be related to my changes.

It is probably worth refactoring the code around get_ticket_from_request_with_authorisation() a bit.
Given that if "public" is set to True it just tries to get a ticket and the only time it is set to true is via the update_tcicket() call in helpdesk/views/public.py, it is pointless calling update_ticket from helpdesk/views/public.py at all and it should just call get_object_or_404(Ticket, id=ticket_id) directly.
That would simplify the logic in the get_ticket_from_request_with_authorisation() somewhat and "public" can be removed from the method signature.
The if statement seems to check stuff that is implicitly already checked as well and could be simplified:
   if not (public or (             request.user.is_authenticated and             request.user.is_active and (                 is_helpdesk_staff(request.user) or                 helpdesk_settings.HELPDESK_ALLOW_NON_STAFF_TICKET_UPDATE))):
The user can only be authenticated if the user is active  (this is done in the authenticator module) so and request.user.is_active  can be removed.
If the above removal of "public" is implemented then only authenticated user requests will ever invoke the get_ticket_from_request_with_authorisation() method so forcing the user to the login screen does not really make sense.
The reason why the ticket may not be found could be a configuration error in the app or it could be someone trying to hack the system and modify tickets they do  not have access to so it should possibly log the user Id used to try and access the record with a message like f"Possible attempt to access unauthorised ticket by user {request.user}" and return an error screen to the user.
Instead there should be a screen that


Online digital art exhibits

Here we collect web sites of online exhibitions

http://likelike.org/2020/04/14/the-online-museum-of-multiplayer-art/

https://rethread.art/browser-fingerprint/

science of the unseen
http://science-unseen.siggraph.org/

http://www.whitepagegallery.network/

The artists have created a program that selects in real time the five most-watched videos on a famous online content sharing platform. The program then takes these five sources, modifies them using random filters and aggregates them into a single video.
https://burningcollection.tv/?lang=en

home delivery service by Ars Electronica
https://ars.electronica.art/aeblog/en/2020/05/05/home-delivery/

The Lockdown Festival of Architecture
https://lockdownfestivalofarchitecture.org/index.html

DIGITAL POWER: ACTIVISM, ADVOCACY, AND THE INFLUENCE OF WOMEN ONLINE
http://http//digital-power.siggraph.org
A Journey through the SIGGRAPH Art Show Archives with Audience Participation
https://digitalartarchive.siggraph.org/

Connecting to art: How The Metropolitan Museum of Art used a public API to share 5,000 years of human creativity with audiences beyond its walls. https://increment.com/apis/metropolitan-museum-api-connecting-to-art/

MUTEK online exhibition
https://mutek.org/en/news/mutek-virtual-exhibition-an-exceptional-showcase-of-multidisciplinary-art/

Messages from the mine
An interactive art installation that excavates messages embedded in the Bitcoin blockchain
https://brangerbriz.com/portfolio/messages-from-the-mines

Poetics and politics of erasure
https://www.oneacre.online/?id=ingrid-monoskop-Y1600430938249

the history of digital art, at the digital art museum
https://dam.org/museum/exhibitions_ui/timelines/events/

exhibition space and artist residency that exists inside the game Minecraft
https://ender.gallery/

Hyperminer – Extracted Earth
https://frederik-de-wilde.com/project/hyperminer-extracted-earth/

Feral File


print debug logs and blitz help on terminal

https://github.com/rootzoll/raspiblitz/issues/2012
print debug logs fully and link option. Raspiblitz help menu
Hope this pull comes without another 999 pulls...

good idea, but couple of notes:
lndmanage (manage) will be not included in the GUI in 1.7
the users loop, pool and faraday will be deprecated and the install scripts to be left CLI only as well because all their interfaces will be the part of the lit user.

yes, since lndmanage is not supported currently (hopefully temporarily) I suggest to no put it into the help text either (same with the separate loop, pool and faraday)


Missing validation/completions when using existing with unresolved discriminated objects

Created from this discussion: https://github.com/Azure/bicep/issues/784#issuecomment-880907755
Example repro of this behavior:
resource normal 'Microsoft.DBForMySQL/servers@2017-12-01' = {
  name: 'asdfsadf'
  location: 'asdas'
  properties:  {
    administratorLoginPassword: ''
    administratorLogin: ''
    createMode: 'Default'

  }
}

// completions work on 'properties'
var normalAccess = normal.properties.version

resource existing 'Microsoft.DBForMySQL/servers@2017-12-01' existing = {
  name: 'asdfsadf'
}

// completions do not work on 'properties'
var existingAccess = existing.properties.version

My analysis from the comment thread:

With this particular resource - I think because properties is modeled with a DiscriminatedObjectType, without the createMode (discriminator) supplied, Bicep is unable to figure which discriminated path has been selected. Rather than displaying all of the properties, Bicep only supplies the one it knows for sure (createMode)

We should union all the types of this discriminated object in a best effort way.

in #4407 I left a TODO where a merge should occur. This is also a problem with existing resources that discriminator is based on a property, not a name.

Property merging should also occur for case in #4668


Error - return  Gekko version: v${util.getVersion()}

Hi,
I have an older version of gekko running, just watching some markets. It works fine on BTCe. Yesterday I pulled the latest stable branch into a new directory and tried to set it up.
When running node gekko I get the following error:
nohup: ignoring input
/Software/gekko/core/util.js:92
    return  `Gekko version: v${util.getVersion()}`
            ^
SyntaxError: Unexpected token ILLEGAL
    at exports.runInThisContext (vm.js:73:16)
    at Module._compile (module.js:443:25)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Module.require (module.js:365:17)
    at require (module.js:384:17)
    at Object.<anonymous> (/Software/gekko/gekko.js:18:12)
    at Module._compile (module.js:460:26)
    at Object.Module._extensions..js (module.js:478:10)

Is this some version issue? Should I purge the old files (currently still running, in a separate directory)?
Thanks!

Your node is too old most likely. What is the output of the following command?
node - v


not working

Please update this API on another website

The server has been changed. Now BD API also has its own domain.
Please update the rest API URL to https://bdapis.com/
For more details please visit: https://bdapis.com/


ARROW-5897: [Java] Remove duplicated logic in MapVector

Current implementation of MapVector contains much logic duplicated from super classes. We remove it by:

Making the default data vector name configurable
Extract a method for creating the reader

LGTM, CC @BryanCutler in case you want to take a look?


installer: start new bitcoind download if not finished previously

In case the user clicks on "Previous" before the bitcoind download has finished, this PR will clear the incomplete download (or a download that completed with errors) from InternalBitcoindStep.
If the user then returns to this step, either a new download will start or the finished download will be available still. This fixes a bug following #746 for users that already have bitcoind v25.0 installed. If they click "Previous" while downloading 25.1, then upon returning to this step, the existing 25.0 bitcoind will be started.

ACK bdd16c73bcae64e17931bd62d70f4d85b9a6af9a -- code makes sense to me and Kevin tested it fixed (one of) the bug he reported.


Question: funder_mnemonic = OS.get_environment("ALGODOT_FUNDER_MNEMONIC") does what exactly?

I commented out this code line because i could already generate my test account's mnemonic from Algod sandbox.
I'm not sure what this line does, Godot DOCs say it returns the value of an environment variable but I don't think on a test device like mobile or Linux, a user's account mnemonic would just be so easily accessible like that. Also, it cancels out whatever mnemonic string that's sent to the algod object so it's actually a hindrance rather than good line of code.
I commented it out, but in case it's useful in some way that i don't know, please enlighten me!

I see you're looking at the test project. This is only used for the purposes of automated testing of the library, and provides a useful way of setting a mnenomic to test with, for example when running CI with github actions. That way you don't need to change the testing script if you want to alter the test account that's being used.
It's not meant to show how to get the users mnemonic in a real life application - you could have the user enter a seedphrase and save it, or better yet generate a new seedphrase for them.
It does raise a good point of how to safely get access to a user's account within an application, and perhaps integration using services like walletconnect could be nice to look at.

Makes sense. I'm not familiar with github actions, so that went over my head. I included encrypt() and decrypt() functions in my code so user's mnemonics are protected. I'm building for a hackathon to build a game around a workable script, so i don't think I'll implement wallet connect, that's way too advanced for the scope of the project i'm trying to build. It's been nice talking with you dev, i"ll continue testing. I might push a PR once i'm done. I ended up breaking down alot of the test code into reusable functions, and i'm formatting them so no-code users and novice users can build gameplay on top of it. If the final code is beyond the scope of this repository, i'll make a fork and tag you in it.
Ciao


Remove nodejs-8_x for 19.09 release

Issue description
Right now nodejs-8_x is in maintenance  mode, while nodejs-10_x is the current active LTS version. According to the nodejs release schedule nodejs-8_x will reach its end of life in December 2019 because at this date also OpenSSL-1.0.2 will be end of life.

Seems like this has been fixed in #64336.


What happens if the server dies?

Currently, I don't feel confident that in the event the server goes down (perhaps from hardware degradation), that it could be brought back up quickly. It'd be nice to have a test of this, and see if a new instance can be brought up, and the DB switched over, while the have the old one to fall back on. In similar fashion to https://www.pagerduty.com/blog/failure-friday-at-pagerduty/, I'd rather know that we have issues bringing up a new server (perhaps due to outdated dependencies) while we still have a crutch to rely on if needed.

This has been validated during the switch from AWS to Digital Ocean.


Versioning dependency issue

I keep getting the below error when compiling with npm:
npm ERR! peer dep missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>My current version is<EMAIL_ADDRESS>

Thanks for the report. It surprises me that it wasn't raised before.
I'll fix that 👍

Because even I have been ignoring it for a couple of months (the warning can be ignored and compilation will run)
but the fix is easy so since I was on github... :)

:tada: This issue has been resolved in version 1.0.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

@estevanmaito facing issue like above but if when i install after tailwind installation, If install before tailwind installation its show just warning.
npm ERR! While resolving: undefined@undefined
npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/tailwindcss
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/tailwindcss-multi-theme
npm ERR!   tailwindcss-multi-theme@"*" from the root project

I believe this package is abandoned and should be marked as such. No updates for two years, tailwind dependencies are very outdated. Projects that are using this package cannot update anymore. It would be great if the person that developed this package offered an alternative.


Add Colemak layout support

Hello, I love Colemak and would love to contribute to the layouts, there are few common variants of Colemak. #45

Hey @delinx
all you would need to do, it replicate this format for the new layout
https://github.com/tris203/hawtkeys.nvim/blob/main/lua/hawtkeys/keyboards/qwerty.lua
everything is 1 indexed because lua.
Fingers are indexed from left pink to right pinky
https://github.com/tris203/hawtkeys.nvim/blob/main/lua/hawtkeys/keyboards.lua
then once your layout is in, add a line here.
If there are multiple variants, then we would need a few different files, but I think we are going to have to do that with qwerty for US/UK variants, as I would like to expand to numbers and symbols in the layouts too

Is there some sort of universal layout format that's used somewhere? Hopefully something not as complicated as zmk/qmk configs at least? If so we could maybe try auto-generating layouts with a script. Would be much easier than manually maintaining each one and we could keep them up to date with CI.

I have looked for a repo of keyboard layouts in some sort of JSON/YAML/XML format, but haven't found anything yet


Query: Memory footprint of multiple ibeam containers

Hi,
I have 10 users in my system, who fire trades independently using IBKR.
I have 10 ibeam containers on my server, 1 for each of the users.
What are the memory implications of this scenario ? How many containers can run seamlessly on my AWS server ( approx. ) ? Is there any such limitation in the first place ?

Hey @raj-crec good question! There's no data currently on how much memory does IBeam consume. If you'd like to conduct such study, please make sure to share your results here 😉 You could run a simple script to measure the memory available before and after IBeam is used on your host - this should be a good place to start.
As for deploying 10 containers on an AWS server - that's certainly possible, provided that your server has sufficient resources to do so. This isn't intrinsic to IBeam in any way.
I've seen other users using multiple IBeam instances, so theoretically this should be doable. Just make sure you map the ports accordingly to avoid conflicts 👍

Thanks much ... I'll update if we do the memory test anytime soon !

few datapoints from running iBeam on my local NAS (a Synology with 4GB RAM).
Each iBeam instance consumes 0.1% CPU load and 140 (+/- 5) MB RAM. These values are stable over at least a 14 days period.
I do run 4 instances and I could likely run 10+.


Fix parse_file_with_callback fp need fclose

on osx file open default is 256
> ulimit -n
256

run gen/bidirectional
./gen/bidirectional > gen/bidirectional.h
parse_file_with_callback: fopen 'data/UnicodeData.txt': Too many open files.

Thank you very much for spending the time to report this issue! I have fixed it in commit c8b34aa with a checked call to fclose().


Upload output directory to CyVerse rather than tarball

For season 11 data processing, we need the ability to access output data in an uncompressed format. As such we need to:

Move output directories to  directory
Move  directory to /xdisk location
Run a cronjob at a regular interval (on filexfer node)

Things to be cautious about:

If multiple people are running the same pipeline, the /xdisk location may fill up too quickly.
If data cannot be uploaded quickly enough, the /xdisk location will fill up.

I will write the code, have the team test it, and figure out if this approach is feasible.

Given that multiple people will be running the pipeline, I believe that adding all data to a directory and setting up a cron job could lead to too many errors. However, I have updated the code to:

Process data as usual
Move output and log directories (uncompressed) to a directory specified by yaml['paths']['cyverse']['upload_directories']['temp_directory']
Submit a small job (1 core, 24 hours) that will upload data and delete the directory

I believe this approach is best, as each process (single user running the pipeline) will still be responsible for downloading, processing, and uploading data.

@Travis-Simmons Please review and provide your thoughts on this issue and the proposed solution.


Fix compositing performance (3x improvement)

How to test
Comment out tickRelationshipOutlines(); in neo4jd3.js like this:
            tickRelationshipsTexts();
            //tickRelationshipsOutlines();
            tickRelationshipsOverlays();

This removes other bottlenecks. Compare the performance of dragging around icons before vs after this change.
Before (75ms total task time~, around 50ms is compositing):

After (25ms total task time~, around 1ms is composting):

Reference: https://stackoverflow.com/a/40533907/3991315

Improves #8


ssrplus ssUDP game mode not work

maybe it is ssrplus itself config bug,and it seem to be solved.

https://github.com/fw876/helloworld/pull/504

https://github.com/fw876/helloworld/issues/476

https://github.com/fw876/helloworld/pull/513


Publish the Helm chart tarball with each release

Is your feature request related to a problem? Please describe
Before using the Helm chart we need to clone the git repository.
Describe the solution you'd like
Instead, I would like Helm to download the Chart by itself.
This could be either from a chart repository or as a tarball published on the release page on Github.
Maybe using chart-releaser

Good idea. We use GitHub Actions for CI/CD. Are you able to try out / make a PR for https://github.com/helm/chart-releaser-action ?

Sorry, I don't know Github Actions and I don't have time to learn it right now.


🔌 Support plugins for CLI (roles/directives only)

This is a first draft of a plugin system for the mystmd CLI. It allows for new directives and roles to be created in javascript and run arbitrary code to pass into the renderer.
A few examples of things that this supports are:

documenting directives programatically (all directives are now documented as well!)
creating custom cross-reference roles (e.g. to a directive option, or a team member (#168 cc @choldgraf))
loading/creating content based on a YAML file (etc.) (e.g. team member, project listings, etc.)

This PR does not implement some obvious next steps for adding custom "transforms" (which can be async actions that transform the document AST (query an API, take a screenshot, read from a DB)) and it does not yet implement rendering extensions (e.g. a custom LaTeX export node type, or Word, etc.). Both of those require a bit more work on firming up an API that we are happy with.
The PR also only works with local ESM Nodejs files (as opposed to CJS). A package manager and discovery mechanism similar to the templates API is likely another next step. The ESM requirement means changes to some base libraries which incorrectly support ESM/CJS, and, as is common in the Node ecosystem, moving to support ESM only:

https://github.com/executablebooks/markdown-it-myst-extras/pull/2
https://github.com/executablebooks/markdown-it-dollarmath/pull/11
https://github.com/executablebooks/markdown-it-amsmath/pull/10

These PRs must be merged/released and updated here before any un-transformed ESM code can run (i.e. esbuild can figure it out, but NodeJS cannot at the moment).
Docs
A few snippets from the docs:

cc @fperez @minrk @choldgraf who were helpful in prompting this last push! The directives/roles are now relatively easy to add, and there is scope to expand this rather quickly to support programatic documentation (e.g. papyri @Carreau), but that needs the transform extensions -- I will hopefully get a pass of that going soon and some example of integrating the two projects.

Ok -- @fwkoch this is good enough for a first pass. 🚀

🎉 I don't know enough TypeScript to meaningfully comment on the code, but I wanted to thank and commend the team for this progress! This is wonderful to see, and puts us on the path for the real promise of MyST being about combining the best experience of Markdown with the extensibility of ReST/Docutils/Pandoc, but with a modern architecture and toolchain. Thanks so much folks! This project makes me really happy, and I only wish I had more bandwidth to devote to it.

Yes! And similarly the extensibility of LaTeX with a modern reading experience.

Fully agreed @nthiery - I see these tools as a viable replacement to LaTeX, reST/Sphinx and plain Markdown workflows for our community. We're not there yet but I see a clear path to that goal, and I think that goal is a worthwhile one. With the work this team is doing, I'm optimistic! :)

This is great! If it helps, here's one kind of sphinx extension that would block my migrating from sphinx-based jupyter-book to something based on mystmd.
It's a custom role points which tracks a sum of points (with some categories), and then a directive pointreport which renders the point totals (typically at the top of the page with the points). So as long as:

roles can populate a per-document data structure, and
directives can read that data structure, and
the directive can read it after all roles have populated it, regardless of order in document

I should be all set.

Having tested it out, I can make this work for the most part, excluding item number 3, but I think that the not-yet-implemented transforms plugin API is probably where the data should be collected so that it's available at the right time, rather than in the roles themselves.
It just so happens that directives get evaluated before roles, so I can't use my previous strategy of collecting in roles and reporting in a directive, but I could do the reverse. There also doesn't appear to be a per-document event to indicate a render at which point the totals should be set to zero, so when I use myst start, each page render causes the point totals to increase. Not super relevant for a real site, but it is hitting me while I test.
In general, there seems to be something missing for 'stateful' plugins like this, but I suspect that transforms will be the way to collect this info at the right time (including the reset from zero, perhaps?). Possibly renderers as well.

Thanks, this is great ! I'm still looking how to integrate myst into papyri for rendering.
I'm still unfamiliar with Typescript, but I think this will gave me some (many?) of the hooks I need for papyri.

Just voicing my support of this, I think it's a critical feature so I'm really happy to see a lot of voices providing their needs/experience/etc.

Thanks all for the encouragement, really exciting to see these usecases @minrk, we are working through some of the implementation and choices here for the transforms which will be needed! See #636
As part of that I am translating your points example @minrk:
https://gist.github.com/rowanc1/7a7a40ffb0ead018e2b441977380aa62
A few questions on what to expose, and how to export up the plugin, but the basics are there and it can confidently add up points (and bonus points...!). This will also work for all of the exports (tex, docx, typst, jats, etc.), which is nice.
@Carreau -- let me know how I can help you get up and running with the myst theme, or if you want me to jump into your repos. I put a demo together here with a few videos:
https://github.com/rowanc1/myst-papyri


disable buffer check on "testing" environment

Description
To clean up the buffer notice in "testing" environment which previously shows:
There were 3 risky tests:

1) CodeIgniter\CodeIgniterTest::testRunClosureRoute
Test code or tested code did not (only) close its own output buffers

2) CodeIgniter\CodeIgniterTest::testRun404OverrideByClosure
Test code or tested code did not (only) close its own output buffers

3) FeatureTestCaseTest::testEchoes
Test code or tested code did not (only) close its own output buffers

Checklist:

[x] Securely signed commits

From 3 down to 1:
There was 1 risky test:

1) FeatureTestCaseTest::testEchoes
Test code or tested code did not (only) close its own output buffers

Wow thanks for this, such an annoyance. I looked into it at one point but was baffled where the issue was. Unfortunate that it is in app/ but this should affect TravisCI more than most users.

We’re still getting one of these. @samsonasik any chance you could look at this last one since you seem to have dug into this issue?
1) FeatureTestCaseTest::testEchoes
366 Test code or tested code did not (only) close its own output buffers

I will check the last one when I have a chance ;)


[ASCII-1535] Update go version to 1.22.4

What does this PR do?

Update Go version to 1.22.4
Here is the PR for the build images repo https://github.com/DataDog/datadog-agent-buildimages/pull/605
Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Gradient Clipping w/ Multiple Optimizers

❓ Questions and Help
Before asking:
No docs helped.
What is your question?
I have a GAN with two optimizers, one generator one discriminator. I would like to clip only generator parameters.
What is the cleanest way to accomplish this?
Currently just using standard training_step and configure_optimizers with automatic optimization enabled.

@tchaton do you have a suggestion here?

I implemented it as follows:
    def training_step(self, batch, _batch_idx, _optimizer_idx):
        generator_opt, discriminator_opt = self.optimizers()

        # Train generator.
        generator_loss = self.generator_training_step(batch)
        self.manual_backward(generator_loss, generator_opt)
        torch.nn.utils.clip_grad_norm_(self.generator.parameters(), self.grad_norm_clip)

        self.manual_optimizer_step(generator_opt)

        # Train discriminator.
        discriminator_loss = self.discriminator_training_step(batch)
        self.manual_backward(discriminator_loss, discriminator_opt)
        self.manual_optimizer_step(discriminator_opt)

While setting automatic_optimization=False.
Is this the right approach? If so, this works for now, but perhaps the trainer gradnorm argument could be extended to allow a tuple of equal length to the length of self.optimizers().

IMO gradient clipping should be a part of LightningModule rather than the Accelerator, so that one can override it and make changes as per the use-case.

hey, I just had the same question and thought I share the solution that I found:
This is now possible without manual optimization by using configure-gradient-clipping of LightningModule.


TypeError: order must be str, not int

Hi!
Thanks for uploading your tool. I've tried to use it, but get following error.
Loading STL
Creating frames
Traceback (most recent call last):
File "STL-To-Gif.py", line 241, in
main(sys.argv[1:])
File "STL-To-Gif.py", line 232, in main
createFrames()
File "STL-To-Gif.py", line 96, in createFrames
scale = stl_mesh.points.flatten(-1)
TypeError: order must be str, not int
Do you have any idea what I'm doing wrong?
greets
Martin

Changing the parameter from -1 to 'C' worked for me.
scale = stl_mesh.points.flatten('C')

Worked for me as well!
Thank you.


[Bug]: py_layers.bzl's interpreter layer also fetches dependencies

What happened?
While adopting rules_oci for python using the macro py_oci_image, I noticed non-interpreter files in the interpreter layer. Some of the site-packages have compiled modules for cpython-*, which pass the regex for interpreter (screenshot attached).
.../bazel-bin/services/services_image.interpreter_tar_manifest.spec:
services/service_name/service.runfiles/pip_parse_name_sqlalchemy/site-packages/sqlalchemy/cimmutabledict.cpython-39-x86_64-linux-gnu.so uid=0 gid=0 time=1672560000 mode=0755 type=file content=external/pip_parse_name_sqlalchemy/site-packages/sqlalchemy/cimmutabledict.cpython-39-x86_64-linux-gnu.so

Should be a simple fix:
cmd = "grep '{}' $< | grep -v '{}' >$@".format(PY_INTERPRETER_REGEX, SITE_PACKAGES_REGEX),

Happy to open a PR!
Version
Development (host) and target OS/architectures:
Output of bazel --version:
Version of the Aspect rules, or other relevant rules from your
WORKSPACE or MODULE.bazel file:
Language(s) and/or frameworks involved:
How to reproduce
No response
Any other information?
No response

Has this been fixed by https://github.com/aspect-build/bazel-examples/pull/357 ?


Azure key vault and Azure devOps service connections

The implementation uses azure key vault and Azure devOps service connections.
fix #30

@hepsi204
Were you able to run the terraform apply step without the Azure variables (Client ID, Tenant ID, Subscription ID?
I keep getting this error during this phase:

@allantargino
it turns out, I still had reference to variable group JMETER_AZUR_PRINCIPAL in the pipeline YAML during my test and removed it after I tested it. I have tested 2 solutions today. First solution is I put back the variable group name and rectify the readme, second solution is adding the variable group entries to key vault and update terraform script below:
- script: terraform apply -auto-approve
  workingDirectory: ./terraform
  env:
    ARM_CLIENT_SECRET: $(arm-client-secret)
    TF_VAR_JMETER_IMAGE_REGISTRY_PASSWORD: $(acr-secret)
  displayName: 'SETUP: Run Terraform Apply (target=all)'

to this
- script: terraform apply -auto-approve
    workingDirectory: ./terraform
    env:
      ARM_SUBSCRIPTION_ID: $(arm-subscription-id)
      ARM_TENANT_ID: $(arm-tenant-id)
      ARM_CLIENT_ID: $(arm-client-id)
      ARM_CLIENT_SECRET: $(arm-client-secret)
      TF_VAR_JMETER_IMAGE_REGISTRY_PASSWORD: $(acr-secret)
    displayName: 'SETUP: Run Terraform Apply (target=all)'

@hepsi204
What option do you think it is the more appropriate?

@allantargino  I think prefer having ARM details in key vault. lets go with that. We will have to keep the one variable group for data that isn't as sensitive/or is needed before access to key vault is established.
I will push up following changes shortly:

set secretsFilter to '*' so every secret key doesn't need to be specified in key vault step
terraform apply stage updated to pass in ARM details via 'env' like in my previous comment
ReadMe updated to include instructions for creating service connection via Azure CLI

LGTM! Thank you so much for this amazing contribution @hepsi204!


Translate to bulgarian table in tab "Доброволци" in admin panel

Which area(s) of Podkrepi.bg are affected?
Desktop layout
Describe the Bug

Need translation to bulgarian name of columns
To Reproduce
1.Open https://podkrepi.bg/admin/supporters
2. Login with admin user
3. Check all columns names.
4.
Expected Behavior
Name of columns to be in bulgarian.
Which browser are you using? (if relevant)
Vivaldi

The issue is fixed here: https://github.com/podkrepi-bg/frontend/pull/1838
I will close the ticket as completed.


Cordova 4 updates

Good day, Arne,
I decided to use your Parse plugin for my latest Ionic / Cordova project, and there were a few minor updates that needed to be done for it to work for me. Some of the updates stem from the fact that I am using Cordova 4.

I updated the Cordova plugin requirement to v4. This is because of some changes in CordovaWebView that I needed to make, which break backwards compatibility on older versions of Cordova
the Cordova Google Play plugin has been changed, and the old one no longer works with Cordova 4
the ParsePlugin initialisation call was not returning on my devices until I moved the call onto the UI thread. It now initialises correctly
your plugin was not intercepting BOOT_COMPLETE in order to initialise Parse. I have added that intent filter to your ParseWakeUpReceiver

I don't know if you are still planning to keep this repo up to date, but thought that these updates might be helpful. Of the 2 Parse Cordova plugin architectures available, I find yours much easier to work with, which is why I have chosen to continue forward development on this one.
Kind regards,
Richard Le Mesurier
Glowworm Software
Cape Town, SA

Hi,
This repo is still active. It's supporting another plugin for the Construct2 game engine that must work with any changes to this repo. It must support Cordova 4 as that is the main build environment for the users.
Did you test your changes with Cordova 5 or 4?
/Arne

I am glad it is still active. It was the best starting point for me as a Parse plugin.
I am testing with Ionic 1.6.4, backed by Cordova 5.2.0.
Re: Google Play dependencies:
Some time back in the development I was getting an error about "com.google.playservices" no longer being supported, and the framework was automatically trying to fetch "cordova-plugin-googleplayservices" instead.
This is likely due to them moving Cordova plugins into the Node Package Manager repo. However today the old google plugin installs on my system perfectly fine. I must have changed something on my build system.
This is probably not important for your purposes.
Re: Cordova 4 compatibility
When compiling with Cordova 5.2.0, I get the following error from Gradle:
:compileDebugJava
......\platforms\android\src\se\frostyelk\cordova\parse\plugin\ParsePlugin.java:99: error: cannot find symbol
            webView.post(new Runnable() {
                   ^
  symbol:   method post(<anonymous Runnable>)
  location: variable webView of type CordovaWebView

This is due to a change in CordovaWebView, which I think I have tracked down to the 4.0.0 release. They updated the interface so that the webView no longer has a .post() method available.
This is probably something to test carefully on your side. Unfortunately I can't think of a way to support both pre-4 and post-4 situations, without using reflection in the Android code. For my purposes, I'm happy to rule out backwards compatibility, which simplifies things on my side.
Re: Parse initialisation on the main thread
Simply put - my devices were hanging with the init process never returning when run from the threadpool.
Since initialisation is a startup cost, I was happy to place it on the main thread as it does not impact my current app.
Re: initialisation on BOOT_COMPLETED
The plugin was crashing on reboot since Google Cloud Messaging was trying to initialise, hence Parse, but Parse had not yet been initialised.
Perhaps some devices throw USER_PRESENT before BOOT_COMPLETED, but in my case this was an issue.
I hope this helps.
Let if you want me to break this pull into smaller chunks, although its probably just easier to copy/paste anything helpful on your side, since the changes are pretty minor in each case.
Richard


Deploy TODO App on Netlify

Configure for upstream branch issue-47/update-sdk, switch to main when available
Dependencies:

#47
#53

npm run build has errors:
Error: Transform failed with 10 errors:
assets/index.8c49503e.js:8287:5719: ERROR: Big integer literals are not available in the configured target environment ("chrome87", "edge88", "es2020", "firefox78", "safari13" + 2 overrides)
assets/index.8c49503e.js:8287:5723: ERROR: Big integer literals are not available in the configured target environment ("chrome87", "edge88", "es2020", "firefox78", "safari13" + 2 overrides)
assets/index.8c49503e.js:8287:5732: ERROR: Big integer literals are not available in the configured target environment ("chrome87", "edge88", "es2020", "firefox78", "safari13" + 2 overrides)
assets/index.8c49503e.js:8287:5736: ERROR: Big integer literals are not available in the configured target environment ("chrome87", "edge88", "es2020", "firefox78", "safari13" + 2 overrides)
assets/index.8c49503e.js:8300:80181: ERROR: Big integer literals are not available in the configured target environment ("chrome87", "edge88", "es2020", "firefox78", "safari13" + 2 overrides)

Live at: https://main--unrivaled-crumble-56ce70.netlify.app/


Navbar add additional App

Prerequisites

[X] I have searched for duplicate or closed feature requests
[X] I have read the Modus style guidelines

Proposal
Hi,
Looks i cant configure the Navbar to add App in front on Trimble.

This is quite common for Trimble Applications. the avatar, i think its currently under proposal so would be nice to get it as soon as possible.
Thanks
Motivation and context
No response
What is the issue regarding ?
@trimble-oss/modus-web-components
Priority
High
What product/project are you using Modus Components for ?
Trimble Desktop
What is your team/division name ?
Tekla Team
Are you willing to contribute ?
None
Are you using Modus Web Components in production ?
No response

@jmecosta Thank you for submitting the issue. Kindly let us know if your team is interested in fixing the issue.

@msankaran0712 yes we can contribute

Thank you @jmecosta

For now it's only a text without interaction.  But u see other wanting to
add a log also.
Example Trimble Tekla with both product icons
On Wed, Aug 23, 2023, 17:07 Mahalakshmi Sankaran @.***>
wrote:

@jmecosta https://github.com/jmecosta Can you please confirm what is
that App ? is it a logo or a button that can be interacted with?
—
Reply to this email directly, view it on GitHub
https://github.com/trimble-oss/modus-web-components/issues/1638#issuecomment-1690032394,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH6HSA5ZARZ3PVIUDQXSKDXWYFCVANCNFSM6AAAAAA3PIMZIE
.
You are receiving this because you were mentioned.Message ID:
@.***>

@jmecosta It is okay to hide the profile icon if no TID implementation.

@jmecosta the issue is approved and ready to be worked on

@enowak1031 to check with Branding team whether it is okay to display "Trimble Logo | Product Logo | Application Logo "

@jmecosta A separate issue has been created for making the profile menu optional, https://github.com/trimble-oss/modus-web-components/issues/1782
Anybody who is interested can contribute a PR 👍🏼

Only one logo is allowed in the navbar

@jmecosta The profile menu is now optional. Use the showProfile property set to false to hide.  With the logo, the current guidance is to have a combined logo following the branding guidance.  Due to this, I will be closing this ticket.


[ENH] Add transmit and receive RF field maps

B1+ and B1- fields map are commonly used to correct for quantification errors stemming from transmit (excitation profile) and receive field (sensitivity profile) inhomogeneities.

As in the qMRI methods specified under the Anatomy Imaging Data section, applications to generate RF field maps also call for the use of grouping suffixes. Nonetheless, this type of data does not convey considerable structural information (on purpose). Therefore, they are placed at the fmap folder.

Update descriptions of B1plusmap and B1minusmap in the anatomy imaging data section and give hyperlink to the Appendix for detailed informarion re fieldmaps.
Additions to the qMRI appendix

Add explanation about the purpose of the application
Add grouping suffix table for RF field map inputs
Define entity mappings for the grouping suffixes
Add explanation about where to place derived RF field maps.

@ChristophePhillips @lazaral
I made the changes for the use of acq for the particular use case of RB1COR:

Body coil
Head coil
Use case

_acq-body
_acq-head
Single acquisition

_acq-bodyMTw
_acq-headMTw
MTw for MPM

_acq-bodyPDw
_acq-headPDw
PDw for MPM

_acq-bodyT1w
_acq-headT1w
T1w for MPM

I used bodyMTw instead of MTwBody to have body and head as the leading word in the value, so that the processing logic becomes more inclusive to non-MPM applications.
If this is reasonable, @Gilles86 I reflected on few other minor details we discussed in the meeting, should be ready to merge.

Looks good to me!

I can live with that. 😄


Invalid compaction on slow block writes

While trying to replicate #121, I triggered the following error:
invalid compaction: Err(
    InvalidCompaction,
)

On this checksum: 47ab5dae0359bac41fd15adc423c3a7456ba12c0
The only changes I made to SlateDB's code was this:
--- a/src/tablestore.rs
+++ b/src/tablestore.rs
@@ -11,9 +11,11 @@ use object_store::buffered::BufWriter;
 use object_store::path::Path;
 use object_store::{ObjectStore, PutPayload};
 use parking_lot::RwLock;
+use tokio::time::sleep;
 use std::collections::{HashMap, VecDeque};
 use std::ops::Range;
 use std::sync::Arc;
+use std::time::Duration;
 use tokio::io::AsyncWriteExt;

 pub struct TableStore {
@@ -306,6 +308,7 @@ impl<'a> EncodedSsTableWriter<'a> {

     async fn drain_blocks(&mut self) -> Result<(), SlateDBError> {
         while let Some(block) = self.builder.next_block() {
+            sleep(Duration::from_millis(100)).await;
             self.writer.write_all(block.as_ref()).await?;
             self.blocks_written += 1;
         }

The intent was to simulate slow-ish block writes to object storage.
Then, I ran this code with the modified slatedb library:
use std::{sync::Arc, time::Duration};

use object_store::{memory::InMemory, path::Path, ObjectStore};
use slatedb::{config::{CompactorOptions, DbOptions}, db::Db};
use tokio::time::Instant;

#[tokio::main]
async fn main() {
    let object_store: Arc<dyn ObjectStore> = Arc::new(InMemory::new());
    let options = DbOptions {
        flush_ms: 100,
        manifest_poll_interval: Duration::from_millis(100),
        min_filter_keys: 10,
        l0_sst_size_bytes: 1024,
        compactor_options: Some(CompactorOptions::default()),
    };
    let kv_store = Db::open_with_opts(
        Path::from("attempt3"),
        options,
        object_store,
    )
    .await
    .unwrap();

    println!("Writing for some time");
    let mut record_count = 0;

    // write a loop that inserts 1000 keys repeatedly or a duration of 1 minute
    let start = Instant::now();
    let write_options = slatedb::config::WriteOptions {
        await_flush: false
    };

    while start.elapsed() < Duration::from_secs(10) {
        let iteration = start.elapsed().as_secs();
        for i in 0..1000 {
            kv_store.put_with_options(format!("key-{}-{}",iteration % 5, i).as_bytes(), format!("value{}", i).as_bytes(), &write_options).await;
            record_count += 1;
        }
    }

    println!("Total records written: {}", record_count);

    // Close
    kv_store.close().await.unwrap();
}

The application failed with the InvalidCompaction message I listed above

I'm also sometimes seeing:
conflicting manifest version. retry write

at the tail end of the stdout logs (right before the "total records written" log line).

From @rodesai:

ah I think this is because I haven't yet had the compaction policy look at what compactions are running when selecting a compaction to schedule. So if it starts a compaction, and then the scheduler runs again before the compaction finishes you'll see this bug. The error is returned when the scheduler tries to schedule a compaction to a sorted run, but there's already an in-progress compaction to that sorted run.

@rodesai is it possible that this issue will be solved by #96?

After looking at #96 and the compaction doc, I think this issue is different. #96 is about validating that the inputs adhere to the rules set forth in the doc. This issue is just about making sure that a scheduled run does not attempt to get re-scheduled.

Yeah. #96 is just meant to track adding more validation. To fix this we need to fix the compaction policy to not submit redundant compactions. I've got a patch in flight to do just this, and fill in the policy with compaction beyond just l0->compacted. So I can just take this ticket.

Fixed by https://github.com/slatedb/slatedb/pull/147


Support react-native 0.47

Added readme instructions and increased package.json requirements
I had no problems building but I have a script that sets the gradle / sdkVersions for all .gradle files to avoid any versioning problems ;)

Ok, looks good! Thanks again :)


Error in Defender exclusion PowerShell script

In the PowerShell for Defender exclusions, this line has an extra line break, which causes a syntax error. Also, the variable is not used in the script. Please correct it:
  $StorageAcct = "
  storageacct"  # Storage Account Name

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

ID: 6386653c-9f82-1ea7-8f29-377be4e5e3b7
Version Independent ID: a5db2b47-afaf-5917-40cd-740168cad8b6
Content: FSLogix for the enterprise - Azure Architecture Guide
Content Source: docs/example-scenario/wvd/windows-virtual-desktop-fslogix.yml
Service: architecture-center
Sub-service: azure-guide
GitHub Login: @christiaanbrinkhoff
Microsoft Alias: pnp

Thanks for reporting - this issue is under review. This is a Microsoft Internal DevOps Bug ID # 527348

@JLambEJG Thank you for catching that error in the docs! I have submitted a PR to fix the issue, the change is merged internally and will go-live in the public docs shortly.


Doc fixes

Syntax  and consistency fixes.

@stoned thanks. please fix DCO issue.

@stoned thanks. please fix DCO issue.

yes of course ! My mistake...

@stoned Good job, Thinks!


[AutoDiff] Handle @differentiable(linear) on Types

PR adds support to mark types as @differentiable(linear), serialize/deserialize it, and print it in the AST when you run -dump-ast. However, currently if you mark a type as differentiable, I throw a diagnostic error. I added very few tests, as there aren't a lot of cases to consider.
Questions/Comments

Because of us currently creating the error diagnostic, I don't think I can test serialization since an error would be thrown.

@swift-ci please test tensorflow

@swift-ci please test tensorflow

@swift-ci please test tensorflow

@swift-ci please test tensorflow

@swift-ci please test tensorflow

@swift-ci please test tensorflow

@swift-ci please test tensorflow


add rule specific for consul

istioctl can't create rules for domain/namespace .service.consul and in pilot if a host field is not a FQDN, it appends .svc.<meta.domain>
https://github.com/istio/istio/blob/release-1.0/pilot/pkg/model/config.go#L548
so for now cannot use the same rules in the bookinfo example for both consul and k8s

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: GregHanson
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: frankbu
If they are not already assigned, you can assign the PR to them by writing /assign @frankbu in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

samples/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

related to https://github.com/istio/istio/issues/7367

Should we also open an issue for the bug where we append .svc.<meta.domain>, even for non k8s environments?

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: frankbu, GregHanson
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

samples/OWNERS [frankbu]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


⚠ Bofrost.at has degraded performance

In 535377a, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2992 ms

Resolved: Bofrost.at performance has improved in 7e230b3 after 312 days, 23 hours, 22 minutes.


Save/restore initial fg/bg colors

Writing CSI 0 m resets the colors to the terminals initial colors, not
what the settings are before PSReadLine starts.
Fix #774

How does this fix work on non-Windows, where the console colors cannot be read? Will we try to restore the state to something incomprehensible?

Ah, right, I knew I was forgetting something.

Non-Windows fixed via https://github.com/lzybkr/PSReadLine/commit/16ab1c6afe237dc6f38743c44d3047d498bf83bf


Make private repository aware

Currently, collect_changes_clone.py does not work with private repositories.

@wata727 Thank you for your PR.
I bit fix to non-config file user cause generating token is a bit bother.


[Bug]: The CSS of the new quote modal .

Contact Details
<EMAIL_ADDRESS>What happened?

The CSS seems messed up
Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Kindly assign this issue to me

@KushalGhosh9899  The input box is too small , u can make in longer , remove border(except bottom) and white background

Hi @SaraswatGit
If this task is being delayed, then I'd be glad to work on this issue.
I'd be grateful if you could assign this task to me under GSSoC'22.

Hey @Sukriti-m , glad to hear that you are very interested to solve this issue but this issue was on hold due to bug #243, now this bug is solved so I don't think resolving these issues will take much time.
I highly recommend you please look for other issues to solve or else you can wait 1 or 2 days after that you can start working on this issue as well.
Thank you

can i work on this issue


Toxins Specialist(?) and Chemist roles need re-added when job timers are added.

Description
This'll let us lock up toxins and chemistry to keep them from blowing up a ton.

Don't forget Atmos Tech.


Update 2022-Training-Workshop.md

Adding details of September training event.

Thanks, Alison - looks good to me.

Thanks David. Is it okay for me to merge this pull request myself, or could someone else (@davidhassell, @sadielbartholomew) do so please?