zotero webdav client doesn't work with copyparty

Describe the bug
After configuring a volume /User with permissions 'rmda' for account @user, trying to configure the Zotero webdav sync to work with it fails with confusing (to me) log message.
To Reproduce
Copyparty config:
[global]
#  i: unix:770:caddy:/tmp/cparty.sock
  i: <IP_ADDRESS>
  p: 3309
  e2dsa  # enable file indexing and filesystem scanning
  e2ts   # and enable multimedia indexing
  ansi   # and colors in log messages
  ah-alg: argon2   # enable password hashing
  chpw
  http-only
  gsel
  q, lo: $LOGS_DIRECTORY/%Y-%m%d.log
  df: 2
  no-robots, force-js  # make it harder for search engines to read your server

# create users:
[accounts]
  user: ...

[/]
  /var/lib/copyparty/mnt/
  accs:
    r: *

[/User]
  /var/lib/copyparty/mnt/User
  accs:
    rwmd: user
  flags:
    daw
    davauth
    safededup
    pk

With server running as above, configure Zotero to connect to the webdav server with the url "https://[webdav.domain]/User/", any username and the appropriate password.
Expected behavior
Zotero should be able to access the subdirectory /User/zotero (created manually) and begin using it for syncing.
Instead it fails with an error about the server's response. The following lines in the server log correspond to this.
Server log
@2024-1018-070000.358 [root                 ] reloading config
@2024-1018-070000.359 [auth                 ] loaded 1 config files:
└/etc/copyparty.conf
@2024-1018-070000.361 [auth                 ] volumes and permissions:

"/"  /var/lib/copyparty/mnt
|    read:  everybody
|   write:  --none--
|    move:  --none--
|  delete:  --none--
|    dots:  --none--
|     get:  --none--
|   upGet:  --none--
|    html:  --none--
|  uadmin:  --none--

"/User"  /var/lib/copyparty/mnt/user
|    read:  user
|   write:  user
|    move:  user
|  delete:  user
|    dots:  --none--
|     get:  --none--
|   upGet:  --none--
|    html:  --none--
|  uadmin:  --none--

@2024-1018-070000.361 [auth                 ] hint: enable upload deduplication with --dedup (but see readme for consequences)
@2024-1018-070000.363 [up2k                 ] reload #3 scheduled
@2024-1018-070000.364 [up2k                 ] reload #3 running
@2024-1018-070000.364 [up2k                 ] uploads temporarily blocked due to indexing
@2024-1018-070000.365 [up2k                 ] uploads are now possible
@2024-1018-070000.365 [up2k                 ] online (reading tags) [/var/lib/copyparty/mnt]
@2024-1018-070000.365 [up2k                 ] online (reading tags) [/var/lib/copyparty/mnt/User]
@2024-1018-070000.365 [up2k                 ] 2 volumes in 0.00 sec
@2024-1018-070000.365 [up2k                 ] mtp finished in 0.00 sec (0:00)
@2024-1018-070101.043 [.... 36762 ] OPTIONS /User/zotero/ @user
@2024-1018-070101.147 [.... 36762 ] PFIND /User/zotero/ @user
@2024-1018-070101.208 [.... 36762 ] GET  /User/zotero/nonexistent.prop @user
@2024-1018-070101.216 [.... 36762 ] PUT /User/zotero/zotero-test-file.prop @user
@2024-1018-070101.216 [.... 36762 ] server HDD is full; -214 B free, need 1 B, User/zotero/zotero-test-file.prop

Server details
if the issue is possibly on the server-side, then mention some of the following:

server OS / version: Debian 6.1.112-1 (2024-09-30)
python version: 3.11.2
copyparty arguments: see config snippet above
filesystem (lsblk -f on linux):

NAME   FSTYPE FSVER LABEL UUID                                 FSAVAIL FSUSE% MOUNTPOINTS
sr0
vda
├─vda1 vfat   FAT32       FF8E-8A23                             499.4M     2% /boot/efi
└─vda2 ext4   1.0         21ab8887-05de-4962-b5de-ea33389b76bb   15.7G    27% /

Client details
if the issue is possibly on the client-side, then mention some of the following:

OS version: pop_os! 22.04
browser version: N/A

Additional context
Zotero version 7.0.7 installed from https://flathub.org/apps/org.zotero.Zotero

hey, thanks for trying copyparty :>
it looks like copyparty is failing to see how much free space you have on the drive, and assumes it is full. Could you try to remove the df option and see if that makes it work?
of course, that is not a fix -- I'd like to figure out why checking the disk space usage is not working on your machine, so I'll post a script we can use to figure this out later tonight.

Hi, thanks for the tip, removing df made it work!
I forgot to mention earlier that this error was only happening with the zotero built-in webdav client. Mounting the webdav with Nautilus and creating the sub-directory seemed to work okay (though I didn't try creating any files that way).

turns out this was a general issue with how df applied to PUT uploads, which is what webdav does... So thanks for catching this :>
was also a good opportunity to clean up how df only cared about files with a known filesize -- now it will also reject uploads of unknown size, if the disk space is already below the limit.
Let me know if you hit any other issues :pray:


BPA - Trim Object rule error

Hi,
On the new rule "Trim object names" i get an error when i try to ignore it.
This is when its on a calculation item.
I get the following error:
One or more of the selected objects does not support annotations.
For this reason, the rule cannot be ignored on these objects.
Kind regards,
Ben

Well, if you want the rule to ignore calculation items in general, simply remove calc items from the scope of the rule (the "Applies to" dropdown within the Edit Best Practice Rule dialog).
If you want more granular control, you'll have to edit the rule expression. For example, if you want to ignore a calc item with the name "YTD " (note the trailing space), you should modify the expression to:
(Name.StartsWith(" ") or Name.EndsWith(" "))
and Name <> "YTD "

If you create a rule within the model itself with the same ID as the rule from this repo (TRIM_OBJECT_NAMES), then that rule will take precedence over the rule defined in this repo, letting you add additional exceptions by modifying its expression as shown above.


Update to Tensorflow 2

On Tensorflow 2  I get some errors, e.g.:
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-1-2516a398efe0> in <module>
----> 1 from dbn.tensorflow import SupervisedDBNClassification
      2 import numpy as np
      3 import pandas as pd
      4 from sklearn.model_selection import train_test_split
      5 from sklearn.metrics.classification import accuracy_score

~/RBMpro/jupyter/tf-dbn/dbn/tensorflow/__init__.py in <module>
----> 1 from .models import BinaryRBM, UnsupervisedDBN, SupervisedDBNRegression, SupervisedDBNClassification

~/RBMpro/jupyter/tf-dbn/dbn/tensorflow/models.py in <module>
     17
     18
---> 19 sess = tf.Session()
     20 atexit.register(close_session)
     21

AttributeError: module 'tensorflow' has no attribute 'Session'

Please update

Hello, this is the fix I have used:
I went into the models.py and added this line to the end of the imports:
import tensorflow.compat.v1 as tf
tf.disable_v2_behavior()
This should switch you back to v1.x behaviour. This is a temporary placeholder. I'll fork this sometime and write it for 2.x


lastUpdatedDateTime property meaning

The property "lastUpdatedDateTime" represents in fact the last timestamp the user object itself has been updated, but not the registration details for the user, right? Would be great to have that clearly stated in the docs, as I was assuming it's the timestamp the user changed his/her registration methods.
Is there any timestamp for that populated or do i need to parse the audit logs for the corresponding activity?
Thanks

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

ID: 5772ab5c-c7cd-6968-fb9f-4a708f57dca6
Version Independent ID: 7d221f54-e79e-1ef5-8d1f-a88114055ac1
Content: userRegistrationDetails resource type - Microsoft Graph beta
Content Source: api-reference/beta/resources/userregistrationdetails.md
Product: identity-and-access-reports
Technology: microsoft-graph
GitHub Login: @egreenberg14
Microsoft Alias: MSGraphDocsVteam

Closing this per my last comment.


[Candidate 4v4 Umbrella Corps ZH v1] Oil Derricks near map center have inconsistent room for movement

Observed behavior
Oil Derricks near map center have inconsistent room for movement.
Expected behavior
Oil Derricks near map center offer consistent mobility opportunities.
Screenshots
1. Can move around

2. Can move around

3. Can move around

4. Can move around

5. Can move around

6. CAN NOT MOVE AROUND

7. CAN NOT MOVE AROUND

8. CAN NOT MOVE AROUND

9. Can move around

10. Can move around

11. CAN NOT MOVE AROUND

12. Can move around

13. Can move around

14. CAN NOT MOVE AROUND

15. Can move around

16. Can move around

fixed


Is it possible to let softmax() output a more entropy output?

I need to test a network before training, but the softmax always output a distribution like [0.0,1.0,0.0] which is quite biased with no entropy. I would like some output like [0.2,0.5, 0.3].
I notice that he reason for biased output the directly related to the output of layer right before the softmax layer, therefore maybe it is possible to prevent it by initializing the layer parameter specifically. But how?

The softmax output should never be 0 theoretically, as e^{x} in the nominator is never zero. In practice, it is possible to get 1s and 0s because of limited type precision.
use tch::Tensor;

fn main() {
    let t1 = Tensor::of_slice(&[f64::MAX, f64::MIN, f64::MIN, f64::MIN, f64::MIN]);
    let t2 = t1.softmax(0, tch::Kind::Float);
    t1.print(); t2.print();

    let t1 = Tensor::of_slice(&[f64::MAX_10_EXP, f64::MIN_10_EXP, f64::MIN_10_EXP, f64::MIN_10_EXP, f64::MIN_10_EXP]);
    let t2 = t1.softmax(0, tch::Kind::Float);
    let t3 = t1.softmax(0, tch::Kind::Double);
    t1.print(); t2.print(); t3.print();
}

Output (reformatted + comments):
# the magnitude of these numbers is very large
 1.7977e+308
-1.7977e+308
-1.7977e+308
-1.7977e+308
-1.7977e+308

# exponentiating them results in nan
-nan
-nan
-nan
-nan
-nan

# using maximum numbers such that 10^x is still within float, we get these numbers
 308
-307
-307
-307
-307

# looks like they yield softmax (1, 0, 0, 0, 0)
 1
 0
 0
 0
 0

# by going to doubles we can see that it's just because these numbers are very close to 1/0 that they are rounded to 1/0 when represented as floats
 1.0000e+00
 8.1076e-268
 8.1076e-268
 8.1076e-268
 8.1076e-268

I don't know the context of your usecase but if you need for whatever reason to not have 1/0s in your softmax, maybe you can perform some sort of smoothing? E.g. Laplacean/additive smoothing (in pseudocode):
t = (1+eps, 0+eps, 0+eps, 0+eps);
t = t/t.sum();

Thanks your ad

The softmax output should never be 0 theoretically, as e^{x} in the nominator is never zero. In practice, it is possible to get 1s and 0s because of limited type precision.
use tch::Tensor;

fn main() {
    let t1 = Tensor::of_slice(&[f64::MAX, f64::MIN, f64::MIN, f64::MIN, f64::MIN]);
    let t2 = t1.softmax(0, tch::Kind::Float);
    t1.print(); t2.print();

    let t1 = Tensor::of_slice(&[f64::MAX_10_EXP, f64::MIN_10_EXP, f64::MIN_10_EXP, f64::MIN_10_EXP, f64::MIN_10_EXP]);
    let t2 = t1.softmax(0, tch::Kind::Float);
    let t3 = t1.softmax(0, tch::Kind::Double);
    t1.print(); t2.print(); t3.print();
}

Output (reformatted + comments):
# the magnitude of these numbers is very large
 1.7977e+308
-1.7977e+308
-1.7977e+308
-1.7977e+308
-1.7977e+308

# exponentiating them results in nan
-nan
-nan
-nan
-nan
-nan

# using maximum numbers such that 10^x is still within float, we get these numbers
 308
-307
-307
-307
-307

# looks like they yield softmax (1, 0, 0, 0, 0)
 1
 0
 0
 0
 0

# by going to doubles we can see that it's just because these numbers are very close to 1/0 that they are rounded to 1/0 when represented as floats
 1.0000e+00
 8.1076e-268
 8.1076e-268
 8.1076e-268
 8.1076e-268

I don't know the context of your usecase but if you need for whatever reason to not have 1/0s in your softmax, maybe you can perform some sort of smoothing? E.g. Laplacean/additive smoothing (in pseudocode):
t = (1+eps, 0+eps, 0+eps, 0+eps);
t = t/t.sum();

Thanks your advice, I would give it a try.


Fixes float stringification

Fixes #843
should fully replicate byond

Running the unit test fails in both MoMMI and my local version of Dream Maker. I think the behavior here is different depending on your OS?

Running the unit test fails in both MoMMI and my local version of Dream Maker. I think the behavior here is different depending on your OS?

CultureInfo?


Create Powershell script to install prereqs

Ease on-boarding pain by automating as much of the prereq install as possible.

I may be interested in assisting with this item. Is there any additional information about what is desired?


[Bug] - TypeError: Cannot read property 'apply' of undefined

OS
Mac  OSX Sierra 10.12.4
Versions
@angular/cli: 1.0.1
node: 6.9.1
os: darwin x64
@angular/common: 4.0.3
@angular/compiler: 4.0.3
@angular/core: 4.0.3
@angular/forms: 4.0.3
@angular/http: 4.0.3
@angular/platform-browser: 4.0.3
@angular/platform-browser-dynamic: 4.0.3
@angular/router: 4.0.3
@angular/cli: 1.0.1
@angular/compiler-cli: 4.0.3
Repro steps.
npm i -g @angular/cli
ng new foo
ng s -o

The app renders, but check the developer tools console and you will see ...

I am using Chrome

It seems to be a zone.js problem. Using 0.8.7 instead of 0.8.8 works.
Duplicate of #6036 and #6047

Just tested by pinning to zones 0.8.7 and it works.

Supposedly fixed in: https://github.com/angular/zone.js/pull/747

Hi @johnpapa this is a dupe of https://github.com/angular/angular-cli/issues/6036

yep. i see that above

0.8.9 released with a fix :) https://github.com/angular/zone.js/blob/master/CHANGELOG.md#089-2017-04-25

@johnpapa Working as expected no errors. 😄

confirmed fixed with zones 0.8.9.
thanks!


Phoenix.ActionClauseError is reported when using Bandit, despite returning 400 http status code

Any exception for which Plug.Exception.status(exception) is less than 500 should not be reported. As is working with Plug.Cowboy.

This in part caused because for Bandit we're listening for Telemetry events instead of log events. And Bandit emits telemetry events for all exceptions regardless of it's Plug.Exception.status result.


Nuxt3 rc13: useFetch(...) POST ignoring header Content-Type ( Client side app )

Environment
RootDir: /home/xxx/yyy                                           01:48:06
Nuxt project info:                                                                                01:48:06

Operating System: Linux
Node Version:     v18.12.1
Nuxt Version:     3.0.0-rc.13
Nitro Version:    0.6.1
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      debug, runtimeConfig, modules, app
Runtime Modules:  -
Build Modules:    -

👉 Report an issue: https://github.com/nuxt/framework/issues/new                                                01:48:06
👉 Suggest an improvement: https://github.com/nuxt/framework/discussions/new
👉 Read documentation: https://v3.nuxtjs.org
Reproduction

await useFetch(() => "/my/post/url", {
        method: 'POST',
        mode: 'no-cors',
        headers: {
	    	"Content-Type": "application/json"
	  	},
        body: add_to_cart_data,
        initialCache: false,

        onResponse({ request, response, options }) {
    	    // Process the response data
    	},

    } );

Developer tools "network inspection" in Chrome:
POST /addToCart HTTP/1.1
Accept-Encoding: gzip, deflate, br
Accept-Language: de-DE,de;q=0.9,en-GB;q=0.8,en;q=0.7,en-US;q=0.6
Connection: keep-alive
Content-Length: 43
Content-Type: text/plain;charset=UTF-8
DNT: 1
Host: localhost:8080
Origin: https://localhost:3000
Sec-Fetch-Dest: empty
Sec-Fetch-Mode: no-cors
Sec-Fetch-Site: cross-site
User-Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
accept: application/json
sec-ch-ua: "Google Chrome";v="107", "Chromium";v="107", "Not=A?Brand";v="24"
sec-ch-ua-mobile: ?0
sec-ch-ua-platform: "Linux"
Developer tools in Firefox:
POST /addToCart HTTP/1.1
Host: localhost:8080
User-Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:106.0) Gecko/20100101 Firefox/106.0
Accept: application/json
Accept-Language: de,en-US;q=0.7,en;q=0.3
Accept-Encoding: gzip, deflate, br
Content-Type: text/plain;charset=UTF-8
Content-Length: 43
Origin: null
Connection: keep-alive
Sec-Fetch-Dest: empty
Sec-Fetch-Mode: no-cors
Sec-Fetch-Site: same-site
Describe the bug
Content type should be the value set in the function call, here "application/json".
Additional context
REPRODUCTION  (added 14.11.2022): https://stackblitz.com/edit/nuxt-starter-d5y9os
Logs
No response

I don't think this is a bug. no-cors only supports content-types of application/x-www-form-urlencoded,multipart/form-data or text/plain.

I don't think this is a bug. no-cors only supports content-types of application/x-www-form-urlencoded,multipart/form-data or text/plain.

https://github.com/nuxt/nuxt/issues/27371


Lower minSdkVersion

Hello.
Is there a chance, that you will reduce minSdkVersion?
To 10 for example.
Regards.

@olivierg13 did that here: https://github.com/OrangeGangsters/android-autofittextview/commit/efa48edc6cb32acfc29221bb2eb8a3b09ec504b2
I'm not sure what the repercussions are of disabling OnLayoutChangeListener, but it seems like you'd lose the ability to refit the text on layout changes. You could possibly call autofit() from your view's onLayout() method manually in this situation.

Hello @pawelantczak, @grantland
Indeed, our app support SDK version 10 (beurghhhhh ^^), so I was forced to do it.
The OnLayoutChangeListener is not anymore called, but we don't really care because we only "support" version 10, so even if the experience is a bit bad, we don't care as soon as it doesn't crash.
If you want to still use it, you can run something like:
if (getView() != null) {
                    getView().post(new Runnable() {
                                       @Override
                                       public void run() {
                                           //TODO refresh autofittextview
                                       }
                                   }
                    );
}

Running a runnable on the view is a way to run a code as soon as the view is visible.
Cheers,
Olivier

Thanks!
@grantland for lib
@olivierg13 for backport


customizing librechat.yaml for advanced configuration

Hi @thunderbug1 , I really like your terraform deployment. Thanks for sharing.
I was wondering if you can point me out how to configure the webapp using the librechat.yaml?
I want to be able to configure TTS en STT and assistants for example.
Thanks very much!

I would like to learn the same thing!


Landing Page Re-designs

Thoughts/Questions regarding Weapons Landing Page:
-Could we replace the overview page with an organized grid of the images by categories? For example (similar images, geolocation, most searched, most frequent, abnormal, saved searches, specific keywords?)
-Can we show a listing of the most common keywords? Wordcloud or some sort of automated topic modelling thing?
-Link directly to popular gun ad websites
-How might we integrate weapon keywords in search?
-How can we show the user trends in images?
-How might we save bundles of images to specific projects/cases for investigators?
Grid Layout Options (8/14)
Linear results with a left hand side pane and no filtering

No left hand side pane, has filtering options.

Left hand side page and has filtering

No left hand side pane, but has filtering

I think we agree we should add case-building at some point, but this is currently out of scope for this SUG/pilot.

incorporated into redesign and done


Unify no tests found message when running MPR

currently, if there are no test found matching a pattern when running jest in Multi project runner mode we get a very noisy output for each separate project. we need to unify it into one single message for all projects.
example:

I also find this super noisy
How about something like this?
In /path/to/root
  781 files checked across 8 projects. for more details run with `--verbose`

running with --verbose would give as similar output as today's behavior.

Unrelated note: Once displayName gets merged it might be a good idea to it to this output also
In PROJECT_1 /some/path/to/project
  X files checked.
  testMatch: **/__tests__/**/*.js?(x),**/?(*.)(spec|test).js?(x) - X matches
  testPathIgnorePatterns: /node_modules/ - X matches

In PROJECT_2 /some/path/to/project2
  Y files checked.
  testMatch: **/__tests__/**/*.js?(x),**/?(*.)(spec|test).js?(x) - Y matches
  testPathIgnorePatterns: /node_modules/ - Y matches

Agree with @rogeliog, both of these suggestions make a lot of sense to me.

I'd like to work on this. I'll have the PR shortly.

Awesome!


Validate user options

Suggested by @radumereuta
Radu: This started from an error Alexey ran into: https://runtimeverification.slack.com/archives/C03ER623BS4/p1671002632034109
By using KORE_EXEC_OPTS='--debug-equation ToDebug' he didn't get the expected behavior.
It turned out that labels needed the module name prefixed.
The haskell tools should validate all the input given by the user and fail with a descriptive error message.
It saves users a lot of time in the long run and it boosts confidence in the tool.

@ana-pantilie I can look at the code, but I don't have access to the linked Slack thread. Could the relevant portions be copied to the ticket?

You should see the message from Alexey from Dec 14, 2022 in the #serokel-rv channel.
But the issue has a broad scope. Everything that can be specified by the user as a command line option or through environment variables should be validated.
Here are some options I added for the front-end: https://github.com/runtimeverification/k/pull/2834


Turn of logging in test suite

Hi There,
Anybody has any  solution to turn of logging in tests?
Examples that didn't work:
RestClient.class_eval do
    def self.log
      ''
    end
  end

RestClient.log =  Logger.new('/tmp/restclient')

So, it turns out it was Http log.
For people having the same issues:
HttpLog.options[:url_blacklist_pattern] = /.*/


Don't exclude implementations of public classes in Doxygen

Brief problem description
Due to
set(DOXYGEN_EXCLUDE_PATTERNS "*/internal/*")
set(DOXYGEN_EXCLUDE_SYMBOLS "internal::*")

in doc/aspice_swe3_4/CMakeLists.txt most part of the public user API is not visible e.g. popo::Publisher

Tasks

Re-write regular expression so that implementations of popo::Publisher, popo::Subscriber are not excluded
Add set(DOXYGEN_INLINE_INHERITED_MEMB YES) so that inherited methods and members are displayed

@mossmaurice I had an inspiration. What do you think about having and impl or similar folder, e.g.
include
   |- iceoryx_posh
      |- popo
      |   |- impl
      |      |- subscriber_base.hpp
      |      |- subscriber_impl.hpp
      |- subscriber.hpp

We need maybe a better name, but I guess you know what I mean.
With this we would still hide the internal header and could get the documentation from the base class.

@mossmaurice this was fixed with #1315 from the workaround on the release branch. It's different than the proposal above but still works. Are you in favor of a different solution or are you fine with the workaround?

@elBoberido The workaround is ok for me long term. However, we should document this somewhere not to forget if e.g. new files are added. I suppose the CONTRIBUTING.md would be a good place. Can you add a few senteces there?

@mossmaurice @elBoberido Is here still something to do or not?


Using Vec<f64> as parameters only works when adding argmin-math as dependency

In a downstream crate, I added argmin 0.7.0 as a dependency (but not argmin-math). I then tried to setup a solver like this:
let solver = NelderMead::new(vec![
    vec![3.0f64, 3.0f64],
    vec![2.0f64, 1.5f64],
    vec![1.0f64, 0.5f64],
]).unwrap();

This fails with the error message that the trait ArgminAdd is not implemented for Vec<f64>. However, once I add argmin-math as a direct dependency of the downstream crate as well, it works (it is not necessary to import the trait).
Is this expected behaviour?

Yes, this is intended and documented in the book. Unfortunately there are currently some difficulties to actually find the book, depending on which path you took ;) I assume that your path was either via the API documentation or the examples, which both so far make no mention of the book. The former is fixed in #270 (but will only go live with the next release) and I will fix the latter by turning the examples into their own crates with dedicated Cargo.toml files hopefully soon.
It may seem a bit tedious having to add both argmin and argmin-math but this split approach has a couple of advantages, in particular in terms of maintainability and in terms of flexibility for the users. For instance, it allows one to cut down on compilation time and code size if needed by enabling one to explicitly specify which functionality is needed.
Apologies for the inconvenience and thanks for this issue, because it really helps improving the "docs experience".

Thank you for your fast and detailed answer. Your assumptions were correct, I went the path of API documentation + examples.
I think the split approach is the right one, keep up the good work :-) I consider this issue to be resolved.


PowerToys Run : control indexed folder

Summary of the new feature/enhancement
It will be great if we can control the list of indexed folder.
Proposed technical implementation details (optional)
Today, in function of search, a lot of non-pertinent items can be proposed.
For example, if I want to start be Eclipse IDE that is present in my start menu, the result is below:

I have to enter "eclipse.lnk" (completely) to view Eclipse shortcut :(.
Like Wox, the possibility to specify the folder of programs to use will be wonderful :)

this is related to #2041. this is a setting that we should add once we've completed 2041.

To unblock here, add folder via settings

To unblock here, add folder via settings

What about adding links to Windows Settings App in PT Settings.

@htcfreek we'll be doing that as part of this work.  Closing as this is part of #2041


Does the i18n function need to be global?

It looks like this plugin will only work if the i18n function is a global function. We are moving over a number of applications with large code bases to webpack and the way the applications currently implement i18n is like so:
var i18n = require('br/I18n');

...
			filterText = i18n('ct.grid.refine.refinersummary.filter.range', {
				fromValue: fromText,
				toValue: toText
			});
...

and in our en.properties file we have
ct.grid.refine.refinersummary.filter.range=[fromValue] - [toValue]
As i18n is bound from a require I can't use the webpack i18n plugin, that's all well and good but I'm curious how others actually use the global approach, I guess in your app entry point you export __() (or whichever function you configure webpack to use) to window so you don't trigger reference errors? Presumably the function that people export can do interpolation if they so require? Would it not make sense for this plugin to provide such a function, instead of having many similar copies in all the webpack projects?

Ah for crying out loud, you just replace the function, sorry for the questions. So this means that webpack i18n plugin functionality is limited to static strings...I guess if there was template string support then you could have interpolation support...OK well the way it currently works doesn't support our full set of use cases even with module local function support. It would also have to leave the function alone. I'll close this anyway as I'm not sure our use cases are something that you'd want to support. I guess most people deal with static translation text.


Add development instructions

This adds instructions on how to develop for apollo-server.
But this instructions are not complete
I'm getting this error when doing GraphQL calls. I confirmed that it's because I have multiple instances, the schema is ok. I think the /src folder is imported too.
Error: Schema must be an instance of GraphQLSchema. Also ensure that there are
not multiple versions of GraphQL installed in your node_modules directory.

So this PR is also a question on how to develop for this package

Thanks!

@helfer, but using this instructions I'm getting that error :/ how can I solve that?

I'm pretty sure the error doesn't have anything to do with the package. Make sure you only have one version of GraphQL installed, now preferably 0.7.0. What does npm list | grep graphql output for you?


Asking to open in a default browser when no browser

? You need to login to Axiom. Should I open your default browser?
Frankly it is kinda a dumb question when there is no browser installed (often on containers or VMs or servers, etc). Should check to see if the binaries it is planning to execute exist before prompting the question.

+1
Should be just a method like "auth via-api-token" from cli

install.sh or just a binary for cli
I don't want to make the binary from sources. I just want to use your binary

Should be just a method like "auth via-api-token" from cli

For the time being you can use axiom auth login --auto-login=false which guides you through the setup.


Switch to tracing for logging

changes lgtm, but would you rebase when you get a chance? i had to modify the lockfile after the last subtree sync and would rather have this branch be up to date since it's also touching the lockfile (caution/paranoia, same thing I guess :laughing:)

It looks like I could just use GitHub's ui for rebasing since there were no merge conflicts, so I did.

Hey, @calebcartwright @fee1-dead I'm not sure If i'm doing something wrong, but I'm unable to see any log output when running the latest master rustfmt (f2bad9c). Do we need to add use statements for tracing::trace!, tracing::debug!, and tracing::info! throughout the project?
Using rustfmt 1.5.3-nightly (f4b80cac 2023-06-30):
echo "fn main() {}" | RUSTFMT_LOG=rustfmt=DEBUG rustfmt
[2023-07-06T04:31:45Z DEBUG rustfmt_nightly::items] compute_budgets_for_params 7 Indent { block_indent: 0, alignment: 0 }, 0, SameLine
[2023-07-06T04:31:45Z DEBUG rustfmt_nightly::items] rewrite_fn_base: one_line_budget: 89, multi_line_budget: 95, param_indent: Indent { block_indent: 4, alignment: 0 }
[2023-07-06T04:31:45Z DEBUG rustfmt_nightly::formatting] track truncate: 14 2

However using rustfmt 1.6.0-nightly (f2bad9c7 2023-07-05) produces no log messages:
echo "fn main() {}" | RUSTFMT_LOG=rustfmt=DEBUG cargo run --bin rustfmt --

.. I forgot to test logging before opening this PR. This is my bad. I will put up a PR fixing this soon.

If desired, I can also add a compatibility layer for use of log::debug! to be redirect to tracing's implementation, but #5818 should fix this for the rustfmt crate at least.


Add advanced features for readtable()

We need to support all of the following to fully match R's and pandas' level of functionality:

[ ] Support reading from URL's
[ ] Support reading files with multicharacter delimiters
[x] Decompress files on the fly using GZip or BZip2
[ ] Automatically parse date and time columns
[ ] Implement skiprows, which specifies the indices of lines in the raw input to skip and defaults to Int[]. The current implementation has a placeholder that does nothing.
[ ] Implement skipend, which specifies the number of lines to skip at the bottom of a file  and defaults to 0.
[ ] Implement converters, which specifies a Dict of functions, each of which specifies how to convert the values in a named column to a desired type.
[ ] Implement encoding, which specifies whether a file is encoded in :utf8 or :latin1. The current implementation has a placeholder that does nothing.
[ ] Implement allowescapes and escapemark, which allow characters like \n or , to be escaped in unquoted fields.

Looks like skiprows is not working:
julia> anneal=readtable("anneal.tab", separator=' ',skiprows=[2, 3]) ERROR: ArgumentError: Argument 'skiprows' is not yet supported.  in readtable at /Users/henryolders/.julia/v0.4/DataFrames/src/dataframe/io.jl:823  in readtable at /Users/henryolders/.julia/v0.4/DataFrames/src/dataframe/io.jl:945

These are mostly implemented in CSV.jl; additional issues can be opened there for specific requests.


can the _prefix_use function be used for all libraries?

I'm just not sure about the prefix_use function because the lstm_cell.vhd has something like LIBRARY work; without use in before
Originally posted by @Annika10 in https://github.com/es-ude/elastic-ai.creator/pull/31#discussion_r789653677

hmm.... i guess a library declaration would then look something like this:
library <lib>;
use <lib>.<package>;
use <lib>.<other_package>;

So i guess in the lstm file we're seeing two libraries, whereas we got only one here. With this in mind i don't think my proposed code makes sense.

just repeated from the linked comment

hmm.... i guess a library declaration would then look something like this:
library <lib>;
use <lib>.<package>;
use <lib>.<other_package>;

So i guess in the lstm file we're seeing two libraries, whereas we got only one here. With this in mind i don't think my proposed code makes sense.

Yes, you are right about this. But I talked with Chao about it and for now it is enough to use use the ieee library and not the work library. So I think we can stick to your proposed code for now

my problem with the proposed code is that its structure does not exactly match the vhdl specification.
From the specification:
library_clause ::= library logical_name_list ;
logical_name_list ::= logical_name { , logical_name }
logical_name ::= identifier

So what i called library header instead is actually the library_clause. In fact the proposed code should be
class ContextClause:
  _library_clause = "library ieee;" # no idea if this is actually the library header, better look it up in the reference
  def __init__(self):
    self._use_clause = ["ieee.std_logic_1164.all", "ieee.numeric_std.all"]

  def __call__(self):
    yield self._library_clause
    yield from _add_semicolon_in_each_line(self.prefix_lines_with_USE(self.libs))

or even
class ContextClause:
  def __init__(self, library_clause, use_clause):
    self._use_clause = use_clause
    self._library_clause = library_clause

  def __call__(self):
    yield from self._library_clause()
    yield from self._library_clause()

class UseClause:
  def __init__(self, items: list[str]):
    self._items = items

  def __call__(self):
    def prefix_use(line: str):
      return f"use {line}"
    yield from _add_semicolon_in_each_line(map(prefix_use, items))

class LibraryClause:
  def __init__(self, logical_name_list: list[str]):
    self._logical_name_list = logical_name_list

  def __call__(self):
    yield "library {}".format(", ".join(self.logical_name_list))

We will have to reuse these code generators a lot, so it is crucial to make sure the terminology matches exactly the reference. For simplicity's sake we do not have to model every left side in the ebnfs as its own class and we do not have to model the full grammar, but only the subset we actually need. But it has to be correct in the formal sense, otherwise we'll run into trouble later.

okay I see your point and will change it like that. You have a different bock for the vhdl specification than me, it seem more detailed. The book I used do not mention the library definition that detailed :) I will add your book to the readme as well

So when I use your implementation approach and want to define the libraries, I do it like this?:
        lib = ContextClause(
            library_clause=LibraryClause(logical_name_list=["ieee", "work"]),
            use_clause=UseClause(
                selected_names=[
                    "ieee.std_logic_1164.all",
                    "ieee.numeric_std.all",
                    "work.all",
                ]
            ),
        )

yep, that doesn't mean the end user will have to do it like that though, but that's an issue for later


checkout ParticleNet to branch from latest master

This is my version of the ParticleNet with global features. I patched the ParticleNet.py file, which was the first version and is not used by anyone.

Making a new PR with finalized changes.


Provide esm version

With more and more build tools (vite, nuxt, etc) going towards an esm-first build, it would be nice if a proper esm interface could be provided, i.e. transpile and distribute esm version, including correct export declaration in package.json.

@tobiasdiez yeah, I think we knew this should happen eventually - thanks for the nudge. Got a PR open for it. I'll test out the builds next week, should be able to get it merged and published in a number of days. If you don't mind trying out the builds that would be great!

Hey @tobiasdiez I'd like to nudge again. this fix would cut 100kb from my app bundle size. Thank you

@floydspace the changes on the linked PR take after Apollo Server's approach, which has been updated once already and is awaiting another fix from a contributor who has offered to implement the change. I'd like to wait until that change lands, then I can follow up with an equivalent update on #259.

Hey @trevor-scheer, I see that the other fix is nnot really moving for quite a while. Any chance we can get the ball running on this one?

My #259 PR follows Apollo Server's pattern. I don't think Apollo Server has it right currently, so I'm reluctant to adopt the same pattern until we figure out the right one. I unfortunately haven't had time to investigate this and I'm not sure when that might happen.
Ref: https://github.com/apollographql/apollo-server/issues/7625
If anyone feels motivated to solve the TS + CJS/ESM dual-package problem in a monorepo I would love to see that. The results of my limited investigation up to this point are documented in the linked issue. There's potentially some tooling that could help us there if we bend it to our will but I haven't tried anything out yet.


Is it possible to exclude a specific check from being required for deployment?

Details
We use branch-deploy to deploy to our dev environments and part of that deployment is to run our integration test suite. This test suite creates a check in github to indicate whether it succeeded or not, and this check is required to succeed for us to merge this branch into main.
Now, branch-deploy also requires all checks to pass before running the deployment, so we have a bit of a catch 22 if the integration tests fail, we cannot re-run the deployment without touching the branch to reset the failed check, which could trigger lengthy builds due to the content in the pull request.
In theory this isn't something that should be required, the tests fail, you should fix them before trying to re-run them, but unfortunately it can happen due to the complexity of the system and the tests that a test can fail due to an issue with an external dependency.
I think in effect what I'm asking for is an inverse to required_contexts, a way to opt out of a check being required for the deployment (even though it's required in the branch protection settings of the repository).

👋 Hey @redoz thank you for opening this issue and sorry for the delay, I was traveling. I think your suggested approach makes a lot of sense. You are probably looking for a new feature where we have something like ignored_contexts in addition to the existing required_contexts so that certain CI checks can be disabled.
That could potentially take some time to implement. In the meantime, have you looked into the checks: "required" field? Perhaps you could make your acceptance test (for your dev env) not required and then use this branch-deploy setting.

i am looking forward for this feature, some checks is required to deploy o specific environment but not all of them.

Hi @GrantBirki no worries, I think the inclusion of an ignored_contexts property would indeed fix the issue. But unless I've misunderstood something (quite possible) I don't think the checks: required property would help us, we use a monorepo setup where only changed projects are rebuilt. The only required check in the main branch protection is the integration test suite.
So branch-deploy wont run unless the builds pass (that are triggered by watching for path changes), and by default you can't merge the branch even if the builds pass, because the integration test suite hasn't run yet.
Once we run the branch-deploy onto one of our dev environments it executes the integration tests and will add the pass/fail check to the PR/Commit.
If it fails, I cannot re-run the branch-deploy because no one of the checks are failing (which is slightly annoying, because in order to release that check, we have to push to the branch, which triggers all changed components to rebuild again). But also, I cannot merge the PR because the required branch protection rule has failed, which is exactly what we want.
I don't think I can list all component builds as required for branch-deploy, because if they didn't change, they wont rebuild, so they by definition didn't pass then, correct? And if that assumption is true I'd then be forced to rebuild every component on any change in the PR which would be quite inefficient.

@redoz Thank you for the clarity here! So it does indeed sound like a ignored_contexts input would be the solution here. This would be a great feature to have then so I'll add this to the backlog. Thank you!

Upon further research into this feature, I have decided against making an ignore_contexts option as this would actually be quite confusing to the required_contexts option that already exists. Instead, I will be making adjustments to the existing checks input and adding a new ignored_checks input. The reason for this is that the required_contexts input adjusts the payload that is sent to the GitHub API upon deployment. This feature is baked into this Action for those that really need it but in the vast majority of cases you shouldn't have to touch this option.
So in practice, the features that people are really going to want is being able to control which CI checks (not contexts) control whether their deployment can proceed or not.
Currently, the checks input option only supports the two following values:

all - All CI checks must pass for a deployment to start
required - Only CI checks marked as "required" via your branch protection settings are required

I am going to make it so that the checks option now also accepts CI job names (in addition to all / required) so that those names can be used as a "filter" as to what CI jobs should truly block deployments.
To complement these changes, I will also make a new input option called ignored_checks which does the inverse. Rather than list out all the checks you want to pass, with ignored_checks you can simply provide the ones that you don't care about. Is your lint job failing, and you don't want it to block you from shipping a critical bug fix? No problem... ignored_checks: lint

@GrantBirki that sounds like it would fit my use case perfectly, thank you!

@redoz These changes are merged into main if you want to check them out. Otherwise, a new major release (v10) will be coming out shortly 🎉

Here is the latest RC containing the changes if you want to test it out!
v10.0.0-rc.1

Here is the latest RC containing the changes if you want to test it out!
v10.0.0-rc.1

Just changed our workflow to use the RC and the new ignore_checks parameter and it does exactly what we want, thank you so much for this!

Wonderful! Changes are now live for v10.0.0 as well so I will officially close out this issue. Thank you for the feedback 🙇


Allow AES256 Encryption

Allow AES256 Encryption

:tada: This issue has been resolved in version 3.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Hovercards all link to Jeffalo

Console error:
addon.js:47  Uncaught (in promise) ReferenceError: loggedinUser is not defined
    at fillInHoverCardTemplate (addon.js:47:36)

Oh, I just fixed that issue. I will push the fix soon!

@mybearworld Can you confirm whether this bug is fixed in the latest version? Thanks!

It's fixed!


feat: added co2 prediction. resolves issue #31

Related Issue

Info about the related issue

[x] GSSOC Participant

[x] Contributor

Closes: #issue number #31
Describe the changes you've made
Give a clear description what modifications you have made
Type of change
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Describe how have you verified the changes made
Checklist:

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added tests that prove my fix is effective or that my feature works.
[x] Any dependent changes have been merged and published in downstream modules.

Screenshots

Original
Updated

Original Screenshot
Updated Screenshot

@miraj0507 , @avijit1999 , @adithya-s-k , @ADITYADAS1999 ,  please can you add appropriate labels to my pr and issue.
Doe to the incorrect label my pr isn't being considered for the leaderboard.


schedule.mjs / schedule widget code should only be executed when on the dashboard page

Currently the code for the schedule widget is executed on every page because it's not wrapped in a conditional if statement.
This results in an error on each page but the dashboard page and stops the script from running #

This line should be wrapped in a condition checkin if the page is dashboard
https://github.com/WesselSmit/education-portal/blob/1631aa80991e7e34dbacf2423f543860e919ee36/src/js/index.js#L13


Getting Error when setup connection string to MSSql Database

Can I Use a MSSql database in this project
we did change connection string in appsettings.Development.json in DefaultConnection in the project keycloak-authorization-services-dotnet

This library doesn't interact with databases in any way, that is a concern of your application only


[minor fix] add link to login with Amazon

this fixes a hole compared to the other login options in that we dont offer instructions to our own docs.
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Validated url.


Add new tests for v200 schema

What does this PR do? Add schema verification tests.
What issues does this PR fix or reference? https://github.com/devfile/api/issues/186
Is your PR tested? Consider putting some instruction how to test your changes.
Docs PR

Interestingly the new tests are run automatically as part of the pull request build.

I noticed an error in the test logic which was giving some false positives. Going to cancel this pull request until I have addressed the issues

Fixed the issues and added a commit to the pull request so have re-opened it.

Will close the pull request again and re-open later once the endpoint proticol issue has concluded and test modified accordingly.

@sleshchenko - please let me know when you consider your branch complete

@sleshchenko - please let me know when you consider your branch complete

@amisevsk The main goal of these tests is to make sure that we don't change the API and breaking the existing devfiles. In case there is an error occurs, it is either some changes have been introduced that breaks existing devfiles. It is true that the document should match but there is no way for us to confirm that the doc matches the code to enforce compatibility. These tests are meant for catching that. For intentional changes on the schema, I'd expect the test change will go with the code changes to reflect those.

@amisevsk The main goal of these tests is to make sure that we don't change the API and breaking the existing devfiles. In case there is an error occurs, it is either some changes have been introduced that breaks existing devfiles. It is true that the document should match but there is no way for us to confirm that the doc matches the code to enforce compatibility. These tests are meant for catching that. For intentional changes on the schema, I'd expect the test change will go with the code changes to reflect those.

Further to Angels comments about complexity I simplified the directory structure and renamed the test to make thing more obvious. Also I set it up so when we want to create a new set of tests for a new version of the schema it is easy to re-use this test directory structure as a starting point - I added instruction for this in the README.

Further to Angels comments about complexity I simplified the directory structure and renamed the test to make thing more obvious. Also I set it up so when we want to create a new set of tests for a new version of the schema it is easy to re-use this test directory structure as a starting point - I added instruction for this in the README.


File/page not found

Dear,
I would like to visualize and download the BigWig files of experiment SRX666556.
However, I receive error 404: page not found when I click on download, and receive the following loading error when visualizing in IGV:

Can you help me with this please?
Thank you in advance!
Kind regards,
Astrid

Dear Astrid,
Thank you for your report on ChIP-Atlas error. I found that the data of your interest (SRX666556) may have been deleted by our error. I will correct the error by the next update of ChIP-Atlas. If you want the data in hurry, may I send you the .bw file?
Best,
Shinya

2021/09/21 22:41、ADReu @.***>のメール:
Dear,
I would like to visualize and download the BigWig files of experiment SRX666556.
However, I receive error 404: page not found when I click on download, and receive the following loading error when visualizing in IGV:
https://user-images.githubusercontent.com/91136466/134181060-5b6e274f-40c8-4fc3-9614-dace1e490bee.png
Can you help me with this please?
Thank you in advance!
Kind regards,
Astrid
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/inutano/chip-atlas/issues/44, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADD2CTJKFNUZZ4N5LMIQRLLUDCDPDANCNFSM5EOPM2PQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

?Dear Shinya,
Thank you for your fast reply.
It would be great if you could send me the .bw file please.
Thank you so much!
Kind regards,
Astrid

Van: Shinya Oki @.***>
Verzonden: woensdag 22 september 2021 00:14
Aan: inutano/chip-atlas
CC: Astrid De Roover; Author
Onderwerp: Re: [inutano/chip-atlas] File/page not found (#44)
Dear Astrid,
Thank you for your report on ChIP-Atlas error. I found that the data of your interest (SRX666556) may have been deleted by our error. I will correct the error by the next update of ChIP-Atlas. If you want the data in hurry, may I send you the .bw file?
Best,
Shinya

2021/09/21 22:41?ADReu @.***>????:
Dear,
I would like to visualize and download the BigWig files of experiment SRX666556.
However, I receive error 404: page not found when I click on download, and receive the following loading error when visualizing in IGV:
https://user-images.githubusercontent.com/91136466/134181060-5b6e274f-40c8-4fc3-9614-dace1e490bee.png
Can you help me with this please?
Thank you in advance!
Kind regards,
Astrid

You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/inutano/chip-atlas/issues/44, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADD2CTJKFNUZZ4N5LMIQRLLUDCDPDANCNFSM5EOPM2PQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/inutano/chip-atlas/issues/44#issuecomment-924427229, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVXKDUUGLS2Y46MUW6OWIDTUDD7U5ANCNFSM5EOPM2PQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
[ { @.": "http://schema.org", @.": "EmailMessage", "potentialAction": { @.": "ViewAction", "target": "https://github.com/inutano/chip-atlas/issues/44#issuecomment-924427229", "url": "https://github.com/inutano/chip-atlas/issues/44#issuecomment-924427229", "name": "View Issue" }, "description": "View this Issue on GitHub", "publisher": { @.": "Organization", "name": "GitHub", "url": "https://github.com" } } ]

Hi Astrid,
The .bw file of your request is now available from the following URL:
https://fsv.iimc.kyoto-u.ac.jp/public/eewYgAUSg8lAbioBKUp8rqkSzC2l4KWcHtQZfsUM2PNg
Best,
Shinya

2021/09/22 15:46、ADReu @.***>のメール:
?Dear Shinya,
Thank you for your fast reply.
It would be great if you could send me the .bw file please.
Thank you so much!
Kind regards,
Astrid

Van: Shinya Oki @.***>
Verzonden: woensdag 22 september 2021 00:14
Aan: inutano/chip-atlas
CC: Astrid De Roover; Author
Onderwerp: Re: [inutano/chip-atlas] File/page not found (#44)
Dear Astrid,
Thank you for your report on ChIP-Atlas error. I found that the data of your interest (SRX666556) may have been deleted by our error. I will correct the error by the next update of ChIP-Atlas. If you want the data in hurry, may I send you the .bw file?
Best,
Shinya

2021/09/21 22:41?ADReu @.***>????:
Dear,
I would like to visualize and download the BigWig files of experiment SRX666556.
However, I receive error 404: page not found when I click on download, and receive the following loading error when visualizing in IGV:
https://user-images.githubusercontent.com/91136466/134181060-5b6e274f-40c8-4fc3-9614-dace1e490bee.png
Can you help me with this please?
Thank you in advance!
Kind regards,
Astrid

You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub <https://github.com/inutano/chip-atlas/issues/44 https://github.com/inutano/chip-atlas/issues/44>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADD2CTJKFNUZZ4N5LMIQRLLUDCDPDANCNFSM5EOPM2PQ https://github.com/notifications/unsubscribe-auth/ADD2CTJKFNUZZ4N5LMIQRLLUDCDPDANCNFSM5EOPM2PQ>.
Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<https://github.com/inutano/chip-atlas/issues/44#issuecomment-924427229 https://github.com/inutano/chip-atlas/issues/44#issuecomment-924427229>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AVXKDUUGLS2Y46MUW6OWIDTUDD7U5ANCNFSM5EOPM2PQ https://github.com/notifications/unsubscribe-auth/AVXKDUUGLS2Y46MUW6OWIDTUDD7U5ANCNFSM5EOPM2PQ>.
Triage notifications on the go with GitHub Mobile for iOS<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
[ { @.": "http://schema.org http://schema.org/", @.": "EmailMessage", "potentialAction": { @.": "ViewAction", "target": "https://github.com/inutano/chip-atlas/issues/44#issuecomment-924427229 https://github.com/inutano/chip-atlas/issues/44#issuecomment-924427229", "url": "https://github.com/inutano/chip-atlas/issues/44#issuecomment-924427229 https://github.com/inutano/chip-atlas/issues/44#issuecomment-924427229", "name": "View Issue" }, "description": "View this Issue on GitHub", "publisher": { @.": "Organization", "name": "GitHub", "url": "https://github.com https://github.com/" } } ]
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/inutano/chip-atlas/issues/44#issuecomment-924635084, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADD2CTIIVD2HZZSRSDDUI5DUDF3VLANCNFSM5EOPM2PQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Thank you so much!
Kind regards,
Astrid


"Make sign-in more secure" does not exist as an option

When setting up MFA, "Make sign-in more secure" is not an option on my screen.  I'm therefore unable to follow these instructions.

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

ID: 3ed95865-eddb-4ad1-d2e2-5bd618151c1b
Version Independent ID: ef32bb26-e78a-caad-5b41-d7e49f6ef9b4
Content: Set up multi-factor authentication for Office 365 users
Content Source: Office365-Admin/security-and-compliance/set-up-multi-factor-authentication.md
Service: o365-administration
GitHub Login: @Sirkkuw
Microsoft Alias: sirkkuw

@OzmeisterO - Thank you for submitting feedback.
I think the best way forward is if you can open a service ticket in your tenant so this can get resolved ASAP. Based on outcome, let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:  https://docs.microsoft.com/office365/admin/contact-support-for-business-products
If you don't mind please keep us posted here on the resolution and I really want to get this information you are discovering back into the docs.


connectors: add warning to the SAML connector

Add a warning to indicate that this connector is unmaintained and likely
vulnerable to auth bypasses.
Regardless of the decision to deprecate the connector, I think this is
important to communicate to potential users.
https://github.com/dexidp/dex/discussions/1884

cc @justaugustus @sagikazarmark


Doesn't work since the Kobolds and Catacombs release

On the first try it give a script error once saying something was undefined. After restarting everything, the next time it got stuck on "pack found" and then upon restart again was stuck in a loop of trying to upload and failing.
@mlntn Can you please fix this up for us? Thank you! :) This is an awesome tool that makes pack openings much more fun.

We had a small bug on the server side that we fixed roughly an hour after release. It should work now. Did you check again?

It still appears to be stuck in a loop. Perhaps something is wrong with some cached data that it's trying to upload. How can I uninstall and start fresh?  Should be in %APPDATA%\Roaming\pack-o-bot

Thanks, clearing %APPDATA%\pack-o-bot (%APPDATA%\ is already the Roaming folder on Win10) fixed the issue.

Huh, you’re right about %APPDATA%. I should have checked it on my PC first. It seems wrong that it goes to Roaming.


animationClass 'toast-bottom-right' and 'toast-bottom-center' not working

The 'toast-bottom-right' and 'toast-bottom-centrer' both classes need top:auto property to avoid top:12px being set by 'toast-top-right' class.
Would love to submit a PR.

animations are not yet added to the library.  This is a future enhancement.

Closing since this is probably an improper use of the positionClass.


MultipleOptionsSearchBar: add aria-label

multipleOptionsSearchBar: add aria-label
Description
This PR endeavors to add an Aria label (with default value of "Search") to the search button, wherever MultipleOptionsSearchBar appears.
Checklist

[x] I'm aware of the code of conduct.
[x] I've created logical separate commits and followed the commit message format.
[ ] I've added relevant test cases.
[ ] I've added relevant documentation.
[x] I've marked translation strings (for relevant code).
[x] I've followed the CSS/JS and React guidelines (for relevant code).
[x] I've followed the web accessibility guidelines (for relevant code).
[x] I've followed the user interface guidelines (for relevant code).
[ ] I've identified the copyright holder(s) and updated copyright headers for touched files (>15 lines contributions).
[x] I've NOT included third-party code (copy/pasted source code or new dependencies).

Closes https://github.com/inveniosoftware/invenio-search-ui/issues/155

To confirm, you mean hardcode it with the translation, yes?

To confirm, you mean hardcode it with the translation, yes?

yes, exactly, without creating additional prop in the component

OK, all set. Now hard-coded (with translation option.)


Workaround for strange MIDI delays using RP2040 and the MIDI library on the hardware serial ports

I recently found a workaround for strange irregular delays in outputting MIDI clock through the hardware serial port on a RP2040 based boards like the Pico or the Waveshare RP2040 Zero.
I can pinpoint it to every read operation that is executed in the main loop lik : void loop(){ MIDI.read()} . Or a direct serial port read without the MIDI library.
USB MIDI does not have this problem when it is used solo, but both together with an hardware serial device it also this strange behaviour. The result is a BPM that is very irregular and way below the BPM you set with setTempo.
So the workaround I found after al lot of experimentation is using the dual core mode of the RP2040. This solves the problem.
// Running on core1
void setup1() {
// Initialize UART midi stack
MIDI.begin(MIDI_CHANNEL_OMNI);
setHWMidiHandlers();
MIDI_USB.begin(MIDI_CHANNEL_OMNI);
setMIDI_USBHandlers();
}
void loop1() {
MIDI.read();
MIDI_USB.read();
}

Do you have a simple example that demonstrates this, or can you post your full code?  I output serial MIDI clock in my RP2040+uClock project, and I don't think I've noticed anything strange like this so far...


please add troubleshooting

need to add troubleshooting for what happens when win 10 business upgrade didnt happen and, if in progress, show that

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

ID: ad8e657e-2aa7-230d-0d9d-675c0200ac18
Version Independent ID: 2028f003-2624-2bd3-34ea-6022d495e2c6
Content: Set up Windows devices for Microsoft 365 Business users
Content Source: microsoft-365/business/set-up-windows-devices.md
Service: o365-administration
GitHub Login: @Sirkkuw
Microsoft Alias: sirkkuw

@officedocsbot assign @mypil

thank you for your feedback - please check out this:
https://support.microsoft.com/en-us/help/4027322/windows-update-troubleshooter
I will link that to the docs.

@gkens - Thank you for submitting feedback.
From our understanding, the issue has been answered by Sirkkuw. If you feel it hasn't been resolved please re-open this issue.
Thank you for engaging with the community here for the docs.

That did not resolve the issue as it say to run if I get a particular error code - I received no error code

@gkens as stated for https://github.com/MicrosoftDocs/microsoft-365-docs/issues/321, please refference the below articles for troubleshooting steps or contact Microsoft support, so that specialized support engineers can look at your specific issue:
https://docs.microsoft.com/en-us/windows/deployment/upgrade/resolve-windows-10-upgrade-errors
https://docs.microsoft.com/en-us/windows/deployment/upgrade/troubleshoot-upgrade-errors
https://support.microsoft.com/en-us/help/10587/windows-10-get-help-with-upgrade-installation-errors

@officedocsbot close


Can you do your mod more easier or write new lite version

Custom Ore Gen is very interesting mod which help very much customize your modpacks. For example now, I create TerraFirmaCraft+GregTech modpack, but I can't use cog because it's to hard. I know that this mod have wiki but cog is hard enough and wiki isn't very clear. Create please lite version of this mod, for example, it doesn't have custom gui and all configuration are in config file. You can disable vanilla ores and ores from mod and for example something like this (config file):

VeinName(###)
Biomes:###^#, #####^#
Ore:###^#, ###^#
Height:###^###
VHeight:###
Area:##
Mass:###
Density:###

VeinName(GalenaOre)
Biomes: ExtremeHills^60, Forest^40
Ore: Galena^70, Antimony^30
Height: 40^80
VHeight:8
Area:3
Mass:40
Density:2

What this means? This means that in world will be generated ore vein with name “GalenaOre” with frequency 40/(40+n)*100%(“n” is mass of other veins). It will be spawn in biome ExtremeHills with frequency 60% or Forest with frequency 40%. It will be comprise 70% of Galena ore and 30% of Antimony ore (users must write in this pole id of some ores). It will be spawn on height 40-80(with the biggest frequency in the middle). Height of this vein in blocks is 8 and it will spawn on area 3^3. His density will be 2 blocks of ore per one block of cobble. All veins are cubic.

You are looking for Sprocket. https://github.com/reteo/Sprocket

Maybe.  Note that GregTech was mentioned.  Sprocket doesn't support NBT tags, so GT ores will not be configurable using the utility.

Maybe. Note that GregTech was mentioned. Sprocket doesn't support NBT tags, so GT ores will not be configurable using the utility.

This meanes that maybe you can do lite version of cog?

It means someone could improve Sprocket.

It's open to whoever wants to fork it.

Maybe still do it?( Without gui and variability, if user must do setting by the template it isn't very hard to write. In addition no one know how user must work with it(sprocket). I'm write to sprocket owner, but it will be better if you do it.

I am the maker of Sprocket, but I've retired from it.  It's in the GPL, so anyone is free to fork it and make their own version.

So, to how I must write?

The instructions are on the Sprocket main page, under the file list.

I do not speak englich well enough and I don't now how use pyrhon(

I'm afraid I can't help you, then.  Have you tried running it through Google Translate?

I know about translate, but it's very "crooked". Maybe it's can do like a fork cog owner?

Maybe at least post the NBT limitation as an issue on the Sprocket repo and if/when someone forks it, that person can decide to address it. Eventually, I'll have to fork it to get things working a bit better on recent versions of Minecraft (but I am personally sticking with 1.7.10 for now).

Well, I've added an issue, as well as a note in the documentation that NBT isn't supported.

Ok, thanks for very good support. Maybe when it makes sense to write this mod, but if you'll just write(in the near future), make it for version 1.7.10.

If you will do it write me on email.


fix: apigw api exist bug

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Jupyter-notebook background-color not correct when connected via SSH

Environment data

VS Code version: 1.53.0-insider
Extension version (available under the Extensions sidebar): v2021.1.502429796
OS and version: Macos
Python version (& distribution if applicable, e.g. Anaconda): 3,7
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda
Relevant/affected Python packages and their versions:
Relevant/affected Python-related VS Code extensions and their versions:
Value of the python.languageServer setting:

Expected behaviour
Vscode theme should be respected when using jupyter in python.
Actual behaviour
SSH into a server, then use either ipynbs or ipython interactive cells which causes the VScode theme to be ignored.
Steps to reproduce:
/\

Sorry but only the default themes are supported in a remote situation.
If you use VS code insider's, this problem shouldn't happen though.

Sorry but only the default themes are supported in a remote situation.
If you use VS code insider's, this problem shouldn't happen though.

I'm using vscode insiders

I'm using vscode insiders

Can you share a screenshot?

Can you share a screenshot?

The interactive window will still have this problem. Sorry should have mentioned VS code insiders only fixes it for notebooks.

The interactive window will still have this problem. Sorry should have mentioned VS code insiders only fixes it for notebooks.


Create Architecture Documentation

Document where things are in the code and how to navigate it
From the 2019 convening: https://docs.google.com/document/d/1_4x9x3eHxo4qevBRlpmCC9L2bVlUkAQuAlidT8wJDzI/edit#heading=h.264josm5lg7

We had noted this as high priority and tagged Albert; seems to be a developer issue so not sure if a docs issue.

It's for the advanced docs section.


DEVEX-775 Add preview link scraping for Apple Reminders

Added Apple Reminders as a scraper to prevent deferred linking.

👍


Documentation Edit Request

In the document, there is a minor correction in the  PowerShell command under the section “Convert standard managed disks to premium”
Mistake command: $vm = Get-AzureRmVM -Name $vmName -rgName $resourceGroupName
Correct command: $vm = Get-AzureRmVM -Name $vmName -ResourceGroupName $rgName

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@v-viksur : Thanks for your contribution to the Azure documentation! The author, @cynthn, has been notified to review your proposed change.

#sign-off thanks for the fix!


Remove 'var' from the codebase, and some modernization sprinkled here and there

Pull Request Test Coverage Report for Build 65

227 of 242 (93.8%)  changed or added relevant lines in 8 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.4%) to 89.192%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/transports/shared/tcp.js
21
22
95.45%

lib/transports/server/childProcess.js
23
26
88.46%

lib/transports/server/middleware.js
12
16
75.0%

lib/transports/client/childProcess.js
53
60
88.33%

Files with Coverage Reduction
New Missed Lines
%

lib/transports/shared/tcp.js
1
90.22%

lib/transports/server/tcp.js
1
94.52%

Totals

Change from base Build 62:
-0.4%

Covered Lines:
494

Relevant Lines:
531

💛  - Coveralls

Pull Request Test Coverage Report for Build 65

227 of 242 (93.8%)  changed or added relevant lines in 8 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.4%) to 89.192%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/transports/shared/tcp.js
21
22
95.45%

lib/transports/server/childProcess.js
23
26
88.46%

lib/transports/server/middleware.js
12
16
75.0%

lib/transports/client/childProcess.js
53
60
88.33%

Files with Coverage Reduction
New Missed Lines
%

lib/transports/shared/tcp.js
1
90.22%

lib/transports/server/tcp.js
1
94.52%

Totals

Change from base Build 62:
-0.4%

Covered Lines:
494

Relevant Lines:
531

💛  - Coveralls


Log activity

This dumps process activity to a docker-style JSON log, e.g.:
{"log": "container has started", "ts": "2016-07-22T13:14:09.416695+00:00"}
{"log": "container exceeded its memory allocation", "ts": "2016-07-22T13:14:10.577891+00:00"}
{"log": "container is restarting:", "ts": "2016-07-22T13:14:10.578461+00:00"}
{"log": "  PID     VSZ     RSS  STAT    COMMAND\n20628  132104  131600  D       /hog/hog", "ts": "2016-07-22T13:14:10.578717+00:00"}
{"log": "container has exited", "ts": "2016-07-22T13:14:16.425063+00:00"}
{"log": "container has started", "ts": "2016-07-22T13:14:16.425237+00:00"}
{"log": "container exceeded its memory allocation", "ts": "2016-07-22T13:14:17.974524+00:00"}
{"log": "container is restarting:", "ts": "2016-07-22T13:14:17.976428+00:00"}
{"log": "  PID     VSZ     RSS  STAT    COMMAND\n20994  132104  131604  D       /hog/hog", "ts": "2016-07-22T13:14:17.976775+00:00"}
{"log": "container has exited", "ts": "2016-07-22T13:14:23.432373+00:00"}

The goal is in turn to feed those to https://github.com/aptible/joecool so they'll appear in customer log drains 🎉

cc @fancyremarker

👏 Looks great, feel free to merge this and aptible/aptible-cookbook#273 at will.

@fancyremarker I'll merge in just a little bit; I have a final fix to make, which is that we should not emit "container has started" messages when starting Captain Comeback itself (i.e. we should not emit those messages on the very first sync).


fix: set DID when creating participant context

What this PR changes/adds
Correctly sets the participant's DID when converting from ParticipantManifest -> ParticipantContext
Why it does that
Briefly state why the change was necessary.
Further notes
List other areas of code that have changed but are not necessarily linked to the main feature. This could be method
signature changes, package declarations, bugs that were encountered and were fixed inline, etc.
Linked Issue(s)
Closes #347
Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.

@ ThomasRutger FYI


Add default attributes map optimization

We can use some saved map of serialized attributes when no modifiers provided
It is common use case to serialize all provided attributes, so it can be useful

Need to check performance first, not sure this will behave better

This behavior can lead to errors with cross references  or  errors if used together with depth_limit plugin. So won't do


random sentence

Source information
kolnovel
Steps to reproduce
i opened the source and The sentences in the lnreader are not the same sentences written on the site, but are random sentences
Expected behavior
the same sentence as site
Actual behavior
random sentence
LNReader version
1.1.15
Android version
android 10
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 1.1.15.
[X] I have updated all installed extensions.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

This has been fixed for the next release, you may obtain a debug release here > https://github.com/K1ngfish3r/lnreader/actions/runs/5302804022

It worked great, thanks

The link is expired bro 😭

Try latest debug bro https://github.com/LNReader/lnreader/actions/runs/6287180526


post-install hooks are not executed after resources were posted

If a Helm-chart defines a post-install hook the controller does not execute the job after the resources were loaded into kubernetes. Instead the controller waits for all resources to be available (e.g. all pods of the deployments are ready) until it executes the post-install jobs.
The chart I am using needs the post-install jobs to run in order for the pods to successfully start. For my case a workaround by setting disableWait to true which tells the controller to execute the jobs.
From my understanding the specification of the chart hook "post-install" states that it shall be executed after the resources were loaded and not after they become ready?

Helm does execute post-install hooks after it has waited for resources to become ready, see their logic here: https://github.com/helm/helm/blob/a499b4b179307c267bdf3ec49b880e3dbd2a5591/pkg/action/install.go#L348-L364
If this does not work for the chart you are working with, spec.disableWait is the right solution.

I see but from the documentation my understanding was different, so I go with the custom spec.


TypeError: Cannot read property 'value' of null

Is there a solution
This is my configuration
"dependencies": {
"@nuxtjs/auth-next": "^5.0.0-1616003482.75c20e6",
"@nuxtjs/axios": "^5.13.1",
"@nuxtjs/component-cache": "^1.1.6",
"@nuxtjs/proxy": "^2.1.0",
"@vue/shared": "^3.0.7",
"axios-extensions": "^3.1.3",
"core-js": "^3.8.3",
"element-ui": "^2.15.0",
"lodash": "^4.17.21",
"lru-cache": "^6.0.0",
"qiniu": "^7.3.2",
"nuxt": "^2.15.6",
"qrcodejs2": "^0.0.2",
"resize-observer-polyfill": "^1.5.1",
"swiper": "5.x",
"uuid": "^8.3.2",
"vue-awesome-swiper": "^4.1.1"
},
"devDependencies": {
"@nuxt/types": "^2.15.3",
"@nuxtjs/eslint-config": "^5.0.0",
"@nuxtjs/eslint-module": "^3.0.2",
"@nuxtjs/style-resources": "^1.0.0",
"@nuxtjs/stylelint-module": "^4.0.0",
"babel-eslint": "^10.1.0",
"babel-plugin-component": "^1.1.1",
"cross-env": "^7.0.3",
"eslint": "^7.18.0",
"eslint-plugin-nuxt": "^2.0.0",
"eslint-plugin-vue": "^7.5.0",
"postcss-preset-env": "^6.7.0",
"sass": "^1.32.8",
"sass-loader": "10",
"stylelint": "^13.9.0",
"stylelint-config-standard": "^20.0.0"
}

Was also experiencing this issue, resolved by upgrading eslint packages
$ yarn upgrade @nuxtjs/eslint-config @nuxtjs/eslint-module babel-eslint


Change "Server" header set location so that it is set only possibly once

1. What does this change do, exactly?
Change "Server" header set location so that it is set only possibly once.
I investigated #1926 a bit more and made additional tests locally with this caddyfile:
http://localhost:8084/
{
	root C:\webroot
	proxy / localhost:8085
}

http://localhost:8085/
{
	root C:\webroot
}

I noted that if I visit http://localhost:8084/2.jpg "Server" header is set twice.
This commit should correct the issue mentioned above as well as #1926 which is related.
Please review this PR to confirm if it may be a good solution for this issue.
2. Please link to the relevant issues.
#1926
3. Which documentation changes (if any) need to be made because of this PR?
None.
4. Checklist

[ ] I have written tests and verified that they fail without my change
[ ] I have squashed any insignificant commits
[X] This change has comments for package types, values, functions, and non-obvious lines of code
[X] I am willing to help maintain this change if there are issues with it later

The Server header having two values (or being set twice -- equivalent in HTTP) is a feature, I think, not a bug. It's useful to see which servers are handling the request. I rely on this when serving the Caddy forum, for example.
A solution for the linked issue would be to bring in the header middleware into the middleware chain that simply redirects HTTP to HTTPS. Or something like that.
Thanks for the PR! But I don't think it's something we're looking for.

Np. I didn't see that use-case before submitting my PR.


Resubmit adding vllm benchmarks

Description
Please include a summary of relevant context/issue and your changes.
Tests
Please describe the tests that you ran on Cloud VM to verify changes.
Instruction and/or command lines to reproduce your tests: ...
List links for your tests (use go/shortn-gen for any internal link): ...
Checklist
Before submitting this PR, please make sure (put X in square brackets):

[x] I have performed a self-review of my code.
[x] I have necessary comments in my code, particularly in hard-to-understand areas.
[x] I have run one-shot tests and provided workload links above if applicable.
[x] I have made or will make corresponding changes to the doc if needed.

If you could update the PR description for future reference, that will be great!


FinancialException in sendApiRequest can't be created when using plugin alone

Hello,
I try to use your plugin just for communicate with paypal but I got an error when paypal returns an error (ACK != Success).
I don't use the CorePaymentBundle model (FinancialTransaction, etc) so in sendApiRequest (PaypalPlugin) when the ACK is not equal to Succes it tries to create a FinancialException :
if (false === $paypalResponse->isSuccess()) {
$ex = new FinancialException('PayPal-Response was not successful: '.$paypalResponse);
======>   $ex->setFinancialTransaction($this->currentTransaction);  <=======
$this->currentTransaction->setResponseCode($paypalResponse->body->get('ACK'));
$this->currentTransaction->setReasonCode($paypalResponse->body->get('L_ERRORCODE0'));
        throw $ex;
    }

$this->currentTransaction is null and generate an error because I didn't initiate the transaction.
So do I have to use the model in CodePaymentBundle ? Or what solution do you recommand ?
PL
ps : please excuse my English ...

We could make this conditional i.e. add a check such as ìf (null !== $this->currentTransaction) { /** ... */ }.
In general though, I would recommend that you use the JMSPaymentCoreBundle as you most of the time need re-try functionality, tracking transactions, etc.

Thank you for reply !
Indeed, I added this quick patch at the beginning... Now, I use the model from CoreBundle, it's simpler and avoids dirty fix.


Can't install for Symfony 3.4.37

[root@beta01 beta]# composer require nelmio/cors-bundle
Do not run Composer as root/super user! See https://getcomposer.org/root for details
Using version ^2.0 for nelmio/cors-bundle
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Conclusion: don't install nelmio/cors-bundle 2.0.1
    - Conclusion: remove symfony/symfony v3.4.37
    - Installation request for nelmio/cors-bundle ^2.0 -> satisfiable by nelmio/cors-bundle[2.0.0, 2.0.1].
    - Conclusion: don't install symfony/symfony v3.4.37
    - nelmio/cors-bundle 2.0.0 requires symfony/framework-bundle ^4.3 || ^5.0 -> satisfiable by symfony/framework-bundle[v4.3.0, v4.3.1, v4.3.10, v4.3.11, v4.3.2, v4.3.3, v4.3.4, v4.3.5, v4.3.6, v4.3.7, v4.3.8, v4.3.9, v4.4.0, v4.4.1, v4.4.2, v4.4.3, v4.4.4, v5.0.0, v5.0.1, v5.0.2, v5.0.3, v5.0.4].
    - don't install symfony/framework-bundle v4.3.10|don't install symfony/symfony v3.4.37
    - don't install symfony/framework-bundle v4.3.11|don't install symfony/symfony v3.4.37
    - don't install symfony/framework-bundle v4.3.3|don't install symfony/symfony v3.4.37
    - don't install symfony/framework-bundle v4.3.4|don't install symfony/symfony v3.4.37
    - don't install symfony/framework-bundle v4.3.5|don't install symfony/symfony v3.4.37
    - don't install symfony/framework-bundle v4.3.6|don't install symfony/symfony v3.4.37
    - don't install symfony/framework-bundle v4.3.7|don't install symfony/symfony v3.4.37
    - don't install symfony/framework-bundle v4.3.8|don't install symfony/symfony v3.4.37
    - don't install symfony/framework-bundle v4.3.9|don't install symfony/symfony v3.4.37
    - don't install symfony/framework-bundle v4.4.0|don't install symfony/symfony v3.4.37
    - don't install symfony/framework-bundle v4.4.1|don't install symfony/symfony v3.4.37
    - don't install symfony/framework-bundle v4.4.2|don't install symfony/symfony v3.4.37
    - don't install symfony/framework-bundle v4.4.3|don't install symfony/symfony v3.4.37
    - don't install symfony/framework-bundle v4.4.4|don't install symfony/symfony v3.4.37
    - don't install symfony/framework-bundle v4.3.0|don't install symfony/symfony v3.4.37
    - don't install symfony/framework-bundle v4.3.1|don't install symfony/symfony v3.4.37
    - don't install symfony/framework-bundle v4.3.2|don't install symfony/symfony v3.4.37
    - don't install symfony/framework-bundle v5.0.0|don't install symfony/symfony v3.4.37
    - don't install symfony/framework-bundle v5.0.1|don't install symfony/symfony v3.4.37
    - don't install symfony/framework-bundle v5.0.2|don't install symfony/symfony v3.4.37
    - don't install symfony/framework-bundle v5.0.3|don't install symfony/symfony v3.4.37
    - don't install symfony/framework-bundle v5.0.4|don't install symfony/symfony v3.4.37
    - Installation request for symfony/symfony (locked at v3.4.37, required as 3.4.*) -> satisfiable by symfony/symfony[v3.4.37].

Installation failed, reverting ./composer.json to its original content.

- Conclusion: don't install nelmio/cors-bundle 2.0.1
- Conclusion: remove symfony/symfony v3.4.37

So I just can't use nelmio corsbundle with Symfony 3.4.37 ?
Am I forced to use Symfony 4 or 5 ??
Is there any old version to install?

# composer require nelmio/cors-bundle
Using version ^2.0 for nelmio/cors-bundle

Try composer require nelmio/cors-bundle:^1.5

# composer require nelmio/cors-bundle
Using version ^2.0 for nelmio/cors-bundle

Try composer require nelmio/cors-bundle:^1.5

Thanks @guilliamxavier

Thanks @guilliamxavier !!


on MacOS, node v4.4.2::  [Release/obj.target/protobuf_for_node/protobuf_for_node.o] Error 1

5 warnings generated.
  CXX(target) Release/obj.target/protobuf_lite/protobuf/src/google/protobuf/io/zero_copy_stream.o
  CXX(target) Release/obj.target/protobuf_lite/protobuf/src/google/protobuf/io/zero_copy_stream_impl_lite.o
../protobuf/src/google/protobuf/io/zero_copy_stream_impl_lite.cc:159:16: warning: comparison of integers of different signs: 'int' and 'size_type' (aka 'unsigned long') [-Wsign-compare]
  if (old_size < target_->capacity()) {
      ~~~~~~~~ ^ ~~~~~~~~~~~~~~~~~~~
../protobuf/src/google/protobuf/io/zero_copy_stream_impl_lite.cc:179:3: warning: comparison of integers of different signs: 'int' and 'size_type' (aka 'unsigned long') [-Wsign-compare]
  GOOGLE_CHECK_LE(count, target_->size());
  ^               ~~~~~  ~~~~~~~~~~~~~~~
../protobuf/src/google/protobuf/stubs/common.h:712:48: note: expanded from macro 'GOOGLE_CHECK_LE'
#define GOOGLE_CHECK_LE(A, B) GOOGLE_CHECK((A) <= (B))
                                            ~  ^   ~
../protobuf/src/google/protobuf/stubs/common.h:708:26: note: expanded from macro 'GOOGLE_CHECK'
  GOOGLE_LOG_IF(FATAL, !(EXPRESSION)) << "CHECK failed: " #EXPRESSION ": "
                         ^~~~~~~~~~
../protobuf/src/google/protobuf/stubs/common.h:705:5: note: expanded from macro 'GOOGLE_LOG_IF'
  !(CONDITION) ? (void)0 : GOOGLE_LOG(LEVEL)
    ^~~~~~~~~
2 warnings generated.
  LIBTOOL-STATIC Release/protobuf_lite.a
  CXX(target) Release/obj.target/protobuf_for_node/protobuf_for_node.o
In file included from ../protobuf_for_node.cc:22:
../node_modules/nan/nan.h:337:54: error: too many arguments to function call, expected at most 2, have 4
    return v8::Signature::New(nan_isolate, receiver, argc, argv);
           ~~~~~~~~~~~~~~~~~~                        ^~~~~~~~~~
/Users/ty/.node-gyp/4.4.2/include/node/v8.h:4675:3: note: 'New' declared here
  static Local<Signature> New(
  ^
In file included from ../protobuf_for_node.cc:22:
../node_modules/nan/nan.h:409:19: error: no type named 'ExternalAsciiStringResource' in 'v8::String'; did you mean 'ExternalStringResource'?
      v8::String::ExternalAsciiStringResource *resource) {
      ~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~
                  ExternalStringResource
/Users/ty/.node-gyp/4.4.2/include/node/v8.h:2210:19: note: 'ExternalStringResource' declared here
  class V8_EXPORT ExternalStringResource
                  ^
In file included from ../protobuf_for_node.cc:22:
../node_modules/nan/nan.h:408:36: error: redefinition of 'NanNew'
  NAN_INLINE v8::Local<v8::String> NanNew(
                                   ^
../node_modules/nan/nan.h:403:36: note: previous definition is here
  NAN_INLINE v8::Local<v8::String> NanNew(
                                   ^
../node_modules/nan/nan.h:427:3: error: redefinition of 'NanNew'
  NanNew<v8::StringObject, v8::Handle<v8::String> >(
  ^
../node_modules/nan/nan.h:420:3: note: previous definition is here
  NanNew<v8::StringObject, v8::Local<v8::String> >(
  ^
../node_modules/nan/nan.h:444:36: error: redefinition of 'NanNew'
  NAN_INLINE v8::Local<v8::RegExp> NanNew(
                                   ^
../node_modules/nan/nan.h:438:36: note: previous definition is here
  NAN_INLINE v8::Local<v8::RegExp> NanNew(
                                   ^
../node_modules/nan/nan.h:456:36: error: redefinition of 'NanNew'
  NAN_INLINE v8::Local<v8::RegExp> NanNew(
                                   ^
../node_modules/nan/nan.h:450:36: note: previous definition is here
  NAN_INLINE v8::Local<v8::RegExp> NanNew(
                                   ^
../node_modules/nan/nan.h:767:13: error: no member named 'smalloc' in namespace 'node'
    , node::smalloc::FreeCallback callback
      ~~~~~~^
../node_modules/nan/nan.h:777:12: error: no matching function for call to 'New'
    return node::Buffer::New(nan_isolate, data, size);
           ^~~~~~~~~~~~~~~~~
/Users/ty/.node-gyp/4.4.2/include/node/node_buffer.h:31:40: note: candidate function not viable: no known conversion from 'uint32_t' (aka 'unsigned int') to 'enum encoding' for 3rd argument
NODE_EXTERN v8::MaybeLocal<v8::Object> New(v8::Isolate* isolate,
                                       ^
/Users/ty/.node-gyp/4.4.2/include/node/node_buffer.h:43:40: note: candidate function not viable: 2nd argument ('const char *') would lose const qualifier
NODE_EXTERN v8::MaybeLocal<v8::Object> New(v8::Isolate* isolate,
                                       ^
/Users/ty/.node-gyp/4.4.2/include/node/node_buffer.h:28:40: note: candidate function not viable: requires 2 arguments, but 3 were provided
NODE_EXTERN v8::MaybeLocal<v8::Object> New(v8::Isolate* isolate, size_t length);
                                       ^
/Users/ty/.node-gyp/4.4.2/include/node/node_buffer.h:36:40: note: candidate function not viable: requires 5 arguments, but 3 were provided
NODE_EXTERN v8::MaybeLocal<v8::Object> New(v8::Isolate* isolate,
                                       ^
In file included from ../protobuf_for_node.cc:22:
../node_modules/nan/nan.h:781:12: error: no viable conversion from returned value of type 'v8::MaybeLocal<v8::Object>' to function return type 'v8::Local<v8::Object>'
    return node::Buffer::New(nan_isolate, size);
           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Users/ty/.node-gyp/4.4.2/include/node/v8.h:210:7: note: candidate constructor (the implicit copy constructor) not viable: no known conversion from 'v8::MaybeLocal<v8::Object>' to 'const v8::Local<v8::Object> &' for 1st argument
class Local {
      ^
/Users/ty/.node-gyp/4.4.2/include/node/v8.h:210:7: note: candidate constructor (the implicit move constructor) not viable: no known conversion from 'v8::MaybeLocal<v8::Object>' to 'v8::Local<v8::Object> &&' for 1st argument
/Users/ty/.node-gyp/4.4.2/include/node/v8.h:214:13: note: candidate template ignored: could not match 'Local' against 'MaybeLocal'
  V8_INLINE Local(Local<S> that)
            ^
/Users/ty/.node-gyp/4.4.2/include/node/v8.h:326:13: note: candidate template ignored: could not match 'S *' against 'v8::MaybeLocal<v8::Object>'
  V8_INLINE Local(S* that)
            ^
In file included from ../protobuf_for_node.cc:22:
../node_modules/nan/nan.h:788:26: error: no member named 'Use' in namespace 'node::Buffer'
    return node::Buffer::Use(nan_isolate, data, size);
           ~~~~~~~~~~~~~~^
../node_modules/nan/nan.h:1728:12: error: no member named 'IsExternalAscii' in 'v8::String'; did you mean 'IsExternal'?
  if (str->IsExternalAscii()) {
           ^~~~~~~~~~~~~~~
           IsExternal
/Users/ty/.node-gyp/4.4.2/include/node/v8.h:2174:8: note: 'IsExternal' declared here
  bool IsExternal() const;
       ^
In file included from ../protobuf_for_node.cc:22:
../node_modules/nan/nan.h:1729:23: error: no type named 'ExternalAsciiStringResource' in 'v8::String'; did you mean 'ExternalStringResource'?
    const v8::String::ExternalAsciiStringResource* ext;
          ~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~
                      ExternalStringResource
/Users/ty/.node-gyp/4.4.2/include/node/v8.h:2210:19: note: 'ExternalStringResource' declared here
  class V8_EXPORT ExternalStringResource
                  ^
In file included from ../protobuf_for_node.cc:22:
../node_modules/nan/nan.h:1730:16: error: no member named 'GetExternalAsciiStringResource' in 'v8::String'
    ext = str->GetExternalAsciiStringResource();
          ~~~  ^
../node_modules/nan/nan.h:1731:11: error: assigning to 'const char *' from incompatible type 'const uint16_t *' (aka 'const unsigned short *')
    *data = ext->data();
          ^ ~~~~~~~~~~~

I am having the same problem with node v6.2.1 and Mavericks.


typo. concret; -> concrete

@quickfur @JackStouffer thanks for the quick action. Remember to approve new contributors at the auto-tester though - otherwise it won't be tested and thus no auto-merge too (I just did so for @btbytes)


main.sh: update config output to add dashboard

@SamYuan1990 @vprashar2929 can we remove microshift test? It looks most test cases are similar with kind

@SamYuan1990 @vprashar2929 can we remove microshift test? It looks most test cases are similar with kind

Previously the microshift is setup by @sthaha , either fine for me, I will leave to @sthaha

Previously the microshift is setup by @sthaha

s/sthaha/vprashar2929 :-)


TypeError: Cannot call method 'trim' of undefined

Firstly thank you for this plugin, It seems to fit perfectly my needs :)
But I encounter an issue when I try to add a mcq :
{%mcq ans="o1", count=2%}
{%title%} This is a question?
{%o1%} First option
{%o2%} Second option
{%o3%} Third option
{%o4%} Fourth option
{%o5%} Fourth option
{%o6%} Fourth option
{%o7%} Fourth option
{%o8%} Fourth option
{%endmcq%}

This code throws an error :
Template render error: (/Users/admin/repo/javascript/introduction/invention.md)
  TypeError: Cannot call method 'trim' of undefined (In file "introduction/invention.md")

Currently I am quite busy with some other things, I will look into it later this week.

I have added the code above into my test gitbook (https://ymcatar.gitbooks.io/gitbook-test/content/v/ef3c1cf88f838469635f8a738f0618d230cbd743/testing_mcqx.html), but I can't reproduce the problem.

I got the same error, the version of gitbook which I'm using is 2.6.7

I copied the code at the top into a fresh new gitbook (ver: 2.6.7), but I can't reproduce the problem. Can you tell me which version of node.js and the plugin you're using?

Debian Jessie, Node.js v4.2.4, gitbook-plugin-mcqx 1.1.0
It can't work on gitbook.com as well, the same error

I'm sorry, this is not a bug of this plugin, after I removed plugin-exercises everything works well.

Glad that you solved the problem. I will close the issue then.


Possible UI bug fixed, model unit test added

Might fix this bug, it seems like a UI bug.
Possible reasons:

appStore.encointer.community?.symbol might be coming null, which is fixed
Models are converting fine, unit tested, so it might be back-end sending wrong or null data

This does not fix the error, it tested what happens when the symbol is null, and it simply prints 'null'. I think printing null is better than N/A as it is more universal. Having unit tests is nice though, and I will merge your PR, when you revert the other change.
I think I have found the issue behind the error and fixed it here: https://github.com/encointer/encointer-wallet-flutter/pull/1395


[Review Only] Use travis for auto deployment

@reiniergs No merge. Need to test this first after you added the env variables.
The only thing I am not sure is this part:
if [ "$TRAVIS_BRANCH" = 'master' ] && [ "$TRAVIS_PULL_REQUEST" == 'false' ]

Changes Unknown when pulling ce2a1e1446d891a90c64deb9b6b6ae684c8d1b1c on scottmo/ciDeployment into ** on master**.