It can be use? i download and blank show!

It can be use? i download and blank show!
Please help me setup and use it!

Did you follow the installation instructions? I just tried installing it as per instructions and it worked as expected.
Simply install Composer, navigate to the directory your project is in, run
composer require alorel/dropbox-v2-php

or, if you already have a composer.json file in your project, add this package to your requirements:
{
    "require": {
        "alorel/dropbox-v2-php": "^0.3"
    }
}

Make sure your installation meets the requirements, i.e. PHP 5.6 and newer. In the end your directory should look something like this.

Thank man
I not found the liblary alorel/dropbox-v2-php and i think maybe it not done.
Thank you a lot for only 1 API v2 dropbox on github.
Can i ask a question, should do i use v1 or v2 dropbox api for better performance, i only need upload via API of dropbox but so hard to find a simple example to upload dropbox via api. example almost by native code or library not via API.
Thank again for beautiful code. :)

It's just the way Composer works. :wink:  This SDK is as close as you can get to raw API calls - all it does is wrap HTTP requests in PHP code. Have a look at the upload and UploadSession examples and their documentation here and here. The arguments these methods receive map directly to the ones the Dropbox API expects.
As for v1 vs v2, definitely go for v2 - it's newer and more actively developed, Dropbox want you to use v2, too.
Let me know if you have any more questions - if not, I'll be closing the issue.

Can you tell me how  to use  /save_url.
i use https://github.com/Alorel/dropbox-v2-php/blob/master/examples/Raw/Files/SaveURL.md
but have this error

`<?php
use Alorel\Dropbox\Operation\Files\SaveUrl\CheckJobStatus;
use Alorel\Dropbox\Operation\Files\SaveUrl\SaveURL;
use Alorel\Dropbox\Operation\Files\ListFolder\ListFolder;
use GuzzleHttp\Exception\ClientException;

$saveURL = new SaveURL();
$resp = json_decode($saveURL->raw('/google.html', 'https://www.google.com')->getBody()->getContents(), true);

if (isset($resp['async_job_id'])) {
    $check = new CheckJobStatus();

    do {
        sleep(3); //Wait a bit
        $r = json_decode($check->raw($resp['async_job_id'])->getBody()->getContents(), true);
	} while (isset($r['.tag']) && $r['.tag'] == 'in_progress');

    //You're done - URL saved
	} else {
    //You're done - URL saved
}	`

Please follow the installation instructions on the Readme and look at the examples.
This package requires knowledge of Composer and namespacing in PHP. Please look at how both of those are used before raising any more questions that aren't tied to this package.

Thank Alorel for  the answer :)
i had success run your example

But some error when i use  some namespace
<?php

    use Alorel\Dropbox\Operation\Files\ListFolder\ListFolder;  //is okay
    use GuzzleHttp\Exception\ClientException;//is okay

    use Alorel\Dropbox\Operation\Files\SaveUrl\CheckJobStatus;//is okay
    use Alorel\Dropbox\Operation\Files\SaveUrl\SaveURL; //return not found

Hope i don't disturb you.
Can you help me once time, please!

You'll find your answer in Composer's getting started documents. Everything works, otherwise it wouldn't have passed the unit tests on Travis-CI.
<?php

    use Alorel\Dropbox\Operation\Files\SaveUrl\CheckJobStatus;
    use Alorel\Dropbox\Operation\Files\SaveUrl\SaveURL;

    require_once 'vendor/autoload.php';

    SaveURL::setDefaultToken('Your token');
    SaveURL::setDefaultAsync(false);

    $saveURL = new SaveURL();
    $resp = json_decode($saveURL->raw('/google.html', 'https://www.google.com')->getBody()->getContents(), true);

    if (isset($resp['async_job_id'])) {
        $check = new CheckJobStatus();

        do {
            echo 'Not saved yet - waiting for 3 seconds before checking for the job status' . PHP_EOL;
            sleep(3); //Wait a bit
            $r = json_decode($check->raw($resp['async_job_id'])->getBody()->getContents(), true);
        } while (isset($r['.tag']) && $r['.tag'] == 'in_progress');

        echo 'URL saved';
    } else {
        echo 'URL saved';
    }

Thank for your kindness.
I had use composer before.

But still error, may be i wrong somewhere.
Thank again for your help and great lib

Composer on its own won't do anything - you need to include the autoloaded as I did in the example above - that way simply referencing a class by name will be enough to automatically load the file containing that class.


Static analysis in REPL shows invalid items

Uncheck "Hide static analysis suggestions"
Type in REPL:

from datetime import datetime
n = datetime.now()

Type "n._":

I wonder how this differs if you do from datetime import datetime as DT; n = DT.now()? Also if you disable typeshed. Either of these could be leaking an extra type in (also our regular analysis of the _datetime module may be getting some types wrong, since they lie about their source, so clearing the cache in %LOCALAPPDATA%\Microsoft\VisualStudio\15.0\Python may help).

This is actually a valid item - the datetime.datetime constructor sets the _day attribute.
If you are expecting underscore prefixed members to be hidden, that is not how we do it in VS. VSC does its own thing here, but we only hide double-underscored members and only when the "Hide advanced members" option is enabled.

And there is actually no %LOCALAPPDATA%\Microsoft\VisualStudio\15.0\Python folder

I meant it's valid for static analysis. If you only want to see items on the actual object, reenable the option you started by posting about :)
And it'll be the 15.0_[InstanceId] folder, not the 15.0 one.

I was able to get this _day field with that option enabled right now, but that is most likely a different bug ;)
Do you have a code sample that would set some value to _day so that n._day doesn't show an error?

>>> import sys
>>> sys.modules['_datetime'] = None
>>> from datetime import datetime
>>> dt=datetime(2018, 8, 13, 13, 27, 37, 814143)
>>> dt._day
13
>>>

Since when you import datetime it overwrites itself with the contents of _datetime, you need to be on a platform where the _datetime module is not available. On Windows, this is not the case, but on some platforms it may be (and even on Windows you could build Python without it to reduce size).
The general rule here is "given multiple definitions for a class/function under different conditions, do we use one of them or all of them?" and the current answer is "all of them". We could reliably choose one in some cases, but not all cases, so we let our multiple-value handling merge them together.


Tock 2.0 full system call API

This updates tock.h to have a full set of Tock 2.0 system call functions that provide all of the return results.
There's now allow_readwrite and allow_readonly, subscribe2, and command2.
syscall_return_t command2(uint32_t driver, uint32_t command, int data, int arg2);
subscribe_return_t subscribe2(uint32_t driver, uint32_t subscribe, subscribe_cb cb, void* userdata)
allow_rw_return_t allow_readwrite(uint32_t driver, uint32_t allow, void* ptr, size_t size);
allow_ro_return_t allow_readonly(uint32_t driver, uint32_t allow, const void* ptr, size_t size);

Because command2 can return anything, its return value presented as a uint32_t[3] and it's up to later code/other library functions to present a cleaner API.
It also changes tock_strerror into tock_strrcode since it's take a ReturnCode not an ErrorCode. There is now also a tock_strerr which takes an ErrorCode.
This PR also updates console to use the new calls.
The update to memop is in the memop PR. Updated to yield will be in a yield PR.
This includes support for both ARM and RISCV, although RISCV is not tested.

bors r+


Change Cardano (ADA) background color

Thank you for this project, the Cardano (Ada) background feels wrong, please reconsider it. Should be much darker.

You are right, the color is used for the font, but never as background for the logo. Please see the logos on:
https://cardanoroadmap.com/
https://daedaluswallet.io/
https://forum.cardano.org/
https://coinmarketcap.com/

Personally I think it should be #0d1e30 (matches also the offical sub on Reddit) or even a little brighter like #0f2230.

@katomm thanks, I'm not too up to date with Cardano, do you know any Cardano dev/designers GH handle that might be interested in giving us their opinion here?

No, I don't.

Ok, I think you're right though, looking through their branding #0d1e30 seems to be more appropriate. Will flag this for updating.

Thank you @lukechilds. Is there a certain way how colors are selected? Because after digging into a few other logos it looks like many others are completely off-brand.
Even the big ones like Ethereum and Bitcoin Cash. Please see:
https://www.ethereum.org/images/logos/Ethereum_Visual_Identity_1.0.0.pdf
https://www.bitcoincash.org/graphics.html

@sindresorhus, Cardano has changed it’s brand color to #0033ad (see https://cardano.org/ and https://www.coinbase.com/price/cardano).

We don't accept pull request contributions for new or updated icons at this time. Open an issue and we'll do the icon for you.

Reporting this here so one of you can hopefully set background color to #0033ad (btw, SVG appears to be broken… file is messed up in Illustrator).

@spothq are the new maintainers of this repo, it's no longer me or @sindresorhus.


mkFit in HLT: timing tests (from Mario&Brunella) and mkFit speedup (CMSSW_14_1_X )

this is for review/discussion, combining

various HLT setups prepared by Mario in https://github.com/mmasciov/cmssw/commits/mkFitHLTOct2024ForTiming

taken from https://github.com/mmasciov/cmssw/commits/mkFitHLTOct2024ForTiming/ ; the instructions were for CMSSW_14_1_4_patch5, but I realized late that the base of the branch is 14_0_X

speedups in mkFit (input) converters and the producer e476c0ab003c8b8f4c9b559c1b5a3e3e715a4659...caf148cba7d0e53a613be0f68d4606b9f3b9c9c1

RecoTracker/MkFit/plugins/MkFit*HitConverter.cc and convertHits.h : save mkFit layer index per hit to be used for a faster MkFitEventOfHitsProducer; use more variables available for a given Det to avoid their recomputation per hit (including hit local->global conversion); access clusters from the reference collection by index directly instead of dereferencing an OmniClusterRef
RecoTracker/MkFit/plugins/MkFitEventOfHitsProducer.cc uses pre-computed per-hit layer index
RecoTracker/MkFit/plugins/MkFitProducer.cc (and customization functions) : disable (by config) additional/redundant re-check of the input cluster charge (already applied during the hit creation); this should work OK for an mkFit setup where CCC is the same in all iterations
RecoTracker/MkFitCore/interface/Hit.h : vdt, replace (slow) std::hypot with explicit sqrt(x*x+y*y); use SMatrixSym33 for direct covariance storage to avoid conversion from SVector6
RecoTracker/MkFitCore/src/MkFinder.cc, RecoTracker/MkFitCore/src/MkFitter.cc, RecoTracker/MkFitCore/src/PropagationMPlex.cc, RecoTracker/MkFitCore/src/Track.cc vdt, replace (slow) std::hypot with explicit sqrt(x*x+y*y)

Timing reduction (based on callgrind) in MC ttbar with PU onDemand=False; mkFit, max # candidates / seed=3, maxClusterSize=4 there is overall 30% reduction in mkFit-related modules (somewhat arbitrary units, callgrind counters on 300 events/1e9):

MkFitEventOfHitsProducer : 16.54 -> 9.66 (0.58)
MkFitSiStripHitConverter: 9.65 -> 6.24 (0.65)
MkFitSiPixelHitConverter: 1.43 -> 0.87 (0.60)
MkFitProducer: 12.35 -> 10.59 (0.86; 7% is the faster math, 7% from not updating the cluster mask with CCC)

This still needs to be tested in the standard offline setup to confirm minimal changes in physics performance

MkFitProducer: 12.35 -> 10.59 (0.86; 7% is the faster math, 7% from not updating the cluster mask with CCC)

I'm guessing the first 7% should stay with the offline setup, while the latter will fractionally go to ~0.

@srlantz
in case you think the vectorization may be affected,
would you be able to test the changes if vectorization gets affected?
Please let me know.
Thank you.

@srlantz in case you think the vectorization may be affected, would you be able to test the changes if vectorization gets affected?

Yes, I should be able to test this, but not until next week.

The simplest test is just to see if the offline mkFit runs faster or slower with these changes.
But also, I want to understand how @osschar incorporated vdt into Matriplex, in case that's a better way to do it.
On the other hand, it looks like the current MiniPropagators.cc code already incorporates some of the vdt transcendentals in a manner similar to this PR.

Yeah, I wrote mini-props before Matriplex stuff -- in fact, it was an inspiration for it :)


performance improvement for MigrationVersion.compareTo

issue described in https://github.com/flyway/flyway/issues/563
in general moved logic of parsing to construction phase and make some reusing of
cached patterns
result of micro-brenchmarking are promising compareTo ~ 33% of old implementation where construction is faster but not so significant.

@pwielgolaski Could you get in touch with me via email (axel at boxfuse.com)? Thanks!


5.6 | Ensure access to the su command is restricted tasks do not account for lines commented out in pam.d

The tasks for this CIS criteria first grep the /etc/pam.d/su file, and do not take into account a line that may be commented out, so the tasks get skipped ie.
# auth       required   pam_wheel.so
      - name: "SCORED | 5.6 | PATCH | Ensure access to the su command is restricted | Check for pam_wheel.so module"
        command: grep 'auth.*required.*pam_wheel' /etc/pam.d/su
        changed_when: false
        failed_when: false
        register: ubtu18cis_5_6_pam_wheel_status

      - name: "SCORED | 5.6 | PATCH | Ensure access to the su command is restricted | Set pam_wheel if does not exist"
        lineinfile:
            path: /etc/pam.d/su
            line: 'auth       required   pam_wheel.so use_uid group={{ ubtu18cis_su_group }}'
            create: yes
        when: ubtu18cis_5_6_pam_wheel_status.stdout == ""

Seems an update is needed to check if the line is commented out and uncomment it so that the control is applied properly.

FYI here is an example from the GSA how they handle this criteria in their role using lineinfile:
https://github.com/GSA/ansible-os-ubuntu18/blob/master/tasks/section5.yml#L809
- name: "SCORED | 5.6 | PATCH | Ensure access to the su command is restricted"
  lineinfile:
      state: present
      dest: /etc/pam.d/su
      regexp: '^(#)?auth\s+required\s+pam_wheel\.so'
      line: "auth           required        pam_wheel.so use_uid"
  when:
      - ubuntu1804cis_rule_5_6
  tags:
      - level1
      - patch
      - rule_5.6
      - scored

Hello,
Thanks for raising the issue. I have added a little more robust regex to control a bit more of what it is looking for. One issue I discovered is if there are no existing or commented out instances where do we put the line. With that I noticed that there was an error in the task where it was always going at the bottom of the file, which is the wrong place for that required pam_wheel element. Since rootok should generally be there I have it inserting below that if there are no instances of auth required pam_wheel. Thanks for raising the issue and I will close this ticket once I get it into devel. Thanks again
George


Comments are successfully posted but only shown after a long delay

This seems to have been one of the main reasons of duplicate comments happening on CW.
When a comment is posted, sometimes it is shown to be succeeded (the top bar vanishes) but the newly posted comment aren't shown at all, at least not until like 10 minutes. Hence users post it again thinking the comment isn't actually being posted, and ends up with 2 identical comments as a result.
Shouldn't comments that have been posted shown immediately after querying for them? It should happen automatically unless comment write are cached, but there's no reason to do that.

I seem to have the same problem, it shows on my own account but not on the thread, even hours later.


yml parse error?

when you define vars with double quote, it seems to ignore yml config? The result is the same when single quote.
desc: basicauth
vars:
  user: "hoge"
  password: "pass@?$"
steps:
  basicAuth:
    bind:
      basicToken: base64encode(vars.user + ':' + vars.password)
  output:
    dump: basicToken

$ runn run ./book.yml --debug
Generated by `runn new` ... ok

1 scenario, 0 skipped, 0 failures

if you remove double quote, it works well
desc: basicauth
vars:
  user: hoge
  password: pass@?$
steps:
  basicAuth:
    bind:
      basicToken: base64encode(vars.user + ':' + vars.password)
  output:
    dump: basicToken

$ runn run ./book.yml --debug
Run 'bind' on 'basicauth'.steps.basicAuth

Run 'dump' on 'basicauth'.steps.output
aG9nZTpwYXNzQD8k
basicauth ... ok

1 scenario, 0 skipped, 0 failures

Seems to be an issue with https://github.com/k1LoW/expand
https://github.com/k1LoW/runn/blob/51e59249aa179bc0668279a319f6e7aa8f0ce6d4/runbook.go#L66
I'll check it out.

Looks like k1loW/expand is responding to $.

you're right. the below works well
desc: basicauth
vars:
  user: "hoge"
  password: "pass@?"
steps:
  basicAuth:
    bind:
      basicToken: base64encode(vars.user + ':' + vars.password)
  output:
    dump: basicToken

Fixed in v0.49.3.


New version: Alibaba.AlibabaCloudCLI version 3.0.214

Created by 🥟 Dumplings in workflow run #6675.

Log
Updated: 3.0.213 → 3.0.214
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-07-17 23:23:43.611 [FAIL] Installer failed security check. Url: https://github.com/aliyun/aliyun-cli/releases/download/v3.0.214/aliyun-cli-windows-3.0.214-amd64.zip Result: 0x80004005

Automated error analysis suggests -1978335187 may mean Installer failed security check. Another script should retry this every 18 hours.
(Automated response - build 886.)

/azp run

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-07-18 18:35:34.086 [FAIL] Installer failed security check. Url: https://github.com/aliyun/aliyun-cli/releases/download/v3.0.214/aliyun-cli-windows-3.0.214-amd64.zip Result: 0x80004005

Automated error analysis suggests -1978335187 may mean Installer failed security check. Another script should retry this every 18 hours.
(Automated response - build 887.)

/azp run

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-07-19 14:32:00.143 [FAIL] Installer failed security check. Url: https://github.com/aliyun/aliyun-cli/releases/download/v3.0.214/aliyun-cli-windows-3.0.214-amd64.zip Result: 0x80004005

Automated error analysis suggests -1978335187 may mean Installer failed security check. Another script should retry this every 18 hours.
(Automated response - build 887.)

/azp run

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-07-22 10:35:28.852 [FAIL] Installer failed security check. Url: https://github.com/aliyun/aliyun-cli/releases/download/v3.0.214/aliyun-cli-windows-3.0.214-amd64.zip Result: 0x80004005

Automated error analysis suggests -1978335187 may mean Installer failed security check. Another script should retry this every 18 hours.
(Automated response - build 887.)

/azp run

/azp run


[Key Vault] Make test resource cleanup script asynchronous

Resolves #16473.
This makes test resource cleanup significantly faster. Certificates are deleted before keys and secrets so that certificate-related key and secret deletion requests don't raise service errors.

/azp run

/azp run python - keyvault - ci


Docker compose build is stuck on setting ownership

Checklist

Have you pulled and found the error with jc21/nginx-proxy-manager:latest docker image?

Yes

Are you sure you're not using someone else's docker image?

Yes

Have you searched for similar issues (both open and closed)?

Yes

Describe the bug
The docker image is stuck on setting ownership when starting.
Nginx Proxy Manager Version

Login does not start.
To Reproduce
Steps to reproduce the behavior:

Run docker-compose file.
Watch build
Start containers
Stuck on

app_1  | ❯ Configuring npm user ...
app_1  | ❯ Configuring npm group ...
app_1  | ❯ Checking paths ...
app_1  | ❯ Setting ownership ...

Expected behavior
I would expect it to start normally.
Operating System
Ubuntu 22.04

what was the problem last time @xfy777 ? I'm facing the same issue on kubernetes

@arsa-dev  Chmod777 everything in the directory and essentially just gotta wait it out!

same problem for me

Here's a solution for people running slow HDD pools on TrueNAS Scale:
https://github.com/truenas/charts/issues/1212
It worked for me.


Fix #924

Close #924

Summary by CodeRabbit

New Features

Enhanced wallet connection logic for the Argent mobile app.
Improved expiration status display for identities in the gallery.

Bug Fixes

Updated tooltip messages for expired and expiring domains in the identities gallery.

Chores

Added .env to the .gitignore file for better version control practices.
Reformatted licensing text for improved readability.

wen ready ? @emarc99

I am still working on it, trying to detect where errors occur by passing statically domain_expiry values in precise as the response from api.starknet.id and using relevant functions in within the project.

wen ready ? @emarc99

I am still working on it, trying to detect where errors occur by passing statically-generated domain_expiry values in precise format as the response from api.starknet.id and using relevant functions within the project.

need to finish asap for OD hack

@Marchand-Nicolas please help test. I tested by mocking your public address.


added db_consistency checker gem in gem-file

Fixes: #3703
Describe the changes you have made in this PR -

Added database_consistency gem in gem-file from source : https://github.com/djezzzl/database_consistency
Then execute bundle install to install gems for the same using : bundle install
This checker can be used with the command : bundle exec database_consistency , this would give you a log on your local setup of the inconsistencies.

Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

 Also add bundle exec database_consistency to Github actions

Closing due to inactivity, feel free to re-open once the issues are addressed


Safari 9.0 double download / Flash Of Wrong Image™ ([if IE 9] no longer works?)

Hi, looking at the demo here: https://scottjehl.github.io/picturefill/examples/demo-02.html, which includes the pattern recommended here: https://github.com/scottjehl/picturefill/issues/536.
Safari 9.0 seems to double-download the right size and 1 size down:

@rayshan
If you read through the #536 you see that we actually discuss another pattern to prevent exactly this.
<picture>
  <!--[if IE 9]><video style="display: none;"><![endif]-->
  <source srcset="large.jpg" media="(min-width: 1200px)">
  <source srcset="small.jpg" >
  <!--[if IE 9]></video><![endif]-->
  <img class="header-image__img" alt="Image">
</picture>

The idea is not to add a src or a srcset to the image.

Ahh this worked. I tried it but typed <source src="small.jpg" > by copying src from img over, but it has to be <source srcset="small.jpg" >, or small.jpg won't load. Thanks.


Does not work with Babel >= 5.5.0

Source, es6.js:
function foo () {
  function bar () {}
}

Babel configuration:
{
  "plugins": ["closure-elimination"]
}

Error:
      throw err;
            ^
TypeError: es6.js: undefined is not a function
    at PathHoister.run (node_modules/babel-plugin-closure-elimination/lib/index.js:135:48)
    at NodePath.enter (node_modules/babel-plugin-closure-elimination/lib/index.js:185:17)
    at NodePath.call (node_modules/babel/node_modules/babel-core/lib/babel/traversal/path/context.js:122:26)
    at NodePath.visit (node_modules/babel/node_modules/babel-core/lib/babel/traversal/path/context.js:146:8)
    at TraversalContext.visitMultiple (node_modules/babel/node_modules/babel-core/lib/babel/traversal/context.js:85:16)
    at TraversalContext.visit (node_modules/babel/node_modules/babel-core/lib/babel/traversal/context.js:115:19)
    at Function.traverse.node (node_modules/babel/node_modules/babel-core/lib/babel/traversal/index.js:67:17)
    at NodePath.visit (node_modules/babel/node_modules/babel-core/lib/babel/traversal/path/context.js:163:26)
    at TraversalContext.visitSingle (node_modules/babel/node_modules/babel-core/lib/babel/traversal/context.js:105:12)
    at TraversalContext.visit (node_modules/babel/node_modules/babel-core/lib/babel/traversal/context.js:117:19)

@julien-f thanks yes the Babel API changed a lot in the last week or two and broke all our plugins, will fix them asap

That sounds like a semver violation on Babel's end... Were you using undocumented/private functions?

@pygy basically the whole of the plugin API is/was undocumented so yeah. Haven't had chance to fix these up yet but I'll get to them as soon as I can.

It does appear that parts of it are starting to stabilize now.

Any news on this project?

@novacrazy basically I have no time to work on this at the moment, happy to accept PRs though

this was fixed by @JexCheng  in #2


Bug Fix: Hit File tab from room details view make Riot crash

#931

Done
Assign to @jeangb for testing


[branch bhai] source activate

I'm trying to run the branch bhai. The command "source activate wa-web" isn't working ("bash: activate: File or directory not found"). If I ignore this error, the program wa.py runs, opens the browser, I instantiate a session on the mobile, but commands do not work right.
For example:
#gc -- getChats = "Failed to get messages", probably because there is no directory named chat.
This is normal? What can I do?

Sorry for my bad english

Hey,
Are you running the command inside the folder. Also, make sure you've installed virtualenv
Run the command pip install virtualenv
Also did you login via the browser?

Hi,
I'm using now, the environment seems to work. I'm authenticating through the browser. The main program still doesn't respond what I'm expecting. Do the commands work there? If you type:
Bot start
Are the messages answered by their XML?

Hey, so the bot functionality doesn't work! I'll probably get around it one day! Still need to write that AIML Code but other features of sending a message does work out!

Thanks! =)

Hey pull the latest code and I've created the folder as well! You can send a Pull Request for the Get Chats feature.


[feature request] untangle gradio dependency from minichain "core"

gradio is a fairly heavyweight dependency and may not be necessary in production deployments. currently, import minichain requires gradio to be installed via
https://github.com/srush/MiniChain/blob/b79ebc51bdedb836c9265eec2fcc21cd60b17327/minichain/__init__.py#L16
and
https://github.com/srush/MiniChain/blob/b79ebc51bdedb836c9265eec2fcc21cd60b17327/minichain/gradio.py#L6
would it be possible to untangle gradio from minichain core and make it optional? more generally, would it be possible to build a minichain "core" which additional components then build off of?

Yup, will do.


ci: Test the build on an old nightly

Add a CI job that runs cargo xtask build with an old nightly. Currently the oldest nightly that works is 2022-04-18. There's nothing particularly magic about that version; we can require a newer version when we deem it appropriate. Having this CI job just makes it easy for us to identify in advance when a PR is going to require a newer nightly, while still allowing other CI jobs to use latest nightly.
This change is related to (but not a fix for): https://github.com/rust-osdev/uefi-rs/issues/510
Checklist

[ ] Sensible git history (for example, squash "typo" or "fix" commits): See the
Rewriting History guide for
help.
[ ] Update the changelog (if necessary)

So this is basically the nightly-MSRV, isn't it? 😁


SetObjectProperties not working?

I have been experimenting with the postman sample for Studio, thanks for that, and the ones provided are working as expected.
However, I am trying to add one for SetObjectProperties, is this supported via JSON?

I added the call 'SetObjectProperties' to the collection. I then tested the call using:
{
"method": "SetObjectProperties",
"id": "1",
"params": [{
"Ticket": "{{Current_Ticket}}",
"ID": "{{Object_ID}}",
"MetaData": {
"BasicMetaData": null,
"RightsMetaData": null,
"SourceMetaData": null,
"ContentMetaData": null,
"WorkflowMetaData": {
"Comment": "This is my comment 2",
"classname": "WorkflowMetaData"
},
"ExtraMetaData": null,
"classname": "MetaData"
},
"Targets": null,
"classname": "WflSetObjectPropertiesRequest"
}],
"jsonrpc": "2.0"
}
The above works. I think it's the commenting  (i.e. the '//' that was causing the problem.


imenu: handle lifetimes in impl blocks.

fix #277

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @pnkfelix (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

Thanks for the patch!


Lock In Python 3.12.2

Problem
The version of python to be used with the project is not locked in. The dockerfiles use 3.12.0, the CI uses 3.9 & the developer may choose to use another version as the documentation does not guide the developer on this.
Solution
The latest LTS version of Python, v3.12.2, has been locked in via the following:

The README now recommends using Python 3.12.2
The dockerfiles use Python 3.12.2
The CI uses Python 3.12.2
Instructions on setting up a virtual environment have been added to the README.

Testing

The unit tests have been verified to work with these changes.
The dockerfiles have been verified to build with these changes.

When running unit tests in the python 3.12.2 virtual environment I ran into a segmentation fault when a test with the 'with patch(...)' syntax was encountered. I'm curious if this is happening for anyone else or if this is just an issue with my machine. When unit tests are run locally with python 3.12.2 they're all working correctly.
I have encountered this error before, but it was when running the unit tests locally with Python 3.12. I did not run into this error when testing these changes, however, so it appears to be intermittent.

When running unit tests in the python 3.12.2 virtual environment I ran into a segmentation fault when a test with the 'with patch(...)' syntax was encountered. I'm curious if this is happening for anyone else or if this is just an issue with my machine. When unit tests are run locally with python 3.12.2 they're all working correctly.

I have encountered this error before, but it was when running the unit tests locally with Python 3.12. I did not run into this error when testing these changes, however, so it appears to be intermittent.
Which unit test in particular is failing? We may want to consider moving away from the 'with path(...)' syntax if this is the cause of the error.

For me all of the unit tests using the with patch(...) syntax were resulting in a segmentation fault. I'm not sure if this is something that would require an additional work item as I am able to run the tests fine locally but might be worth looking into if others encounter this error.


Error while starting

error while starting integration, choosing masovian is unavailable

Can't reproduce this problem

Problem only on iOS HA App, works ok via www.Wiadomość napisana przez github-actions[bot] @.***> w dniu 30.09.2022, o godz. 10:58:
Stale issue message
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


different way to get chromium revision from puppeteer package

hi all.
it seems that not much ppl are using the build.js but it seems that there was a change how one can get the revision from puppeteer and this sample here https://github.com/puppeteer/puppeteer/issues/8203#issuecomment-1109891159 is IMO a now working way to do it.
br, d

hi @bmblb , any update on this?

@rostacik Hello. I completely missed your message. I will apply the PR


Failed to play cast file on github pages

it's time for some serious debugging. 💪

I'll take a look at this issue today.


[BUG] Apps with incorrect names

^ For example.
Thankfully, I was able to find it using the game id. ;)
This might or not be related to issue #81

Also, I'm sure this is a new(-ish?) issue. I'm sure it wasn't called "Mabinogi" back when I added it to my backlog.

Hello @MakoSipper
You are right, this problem has been caused due to recent changes to the scraping system.
I made those changes to remove demos and other non-games apps from database, as well as reduce database size to increase loading times and include store links to games. In the process, a new bug appeared when the system updates a game that is removed from steam. All the following games updated in that batch have been some attributes changed.
This is something I detected and solved last week, but now some games are affected with this until I solve it manually or until they are scraped again (every game is updated every 3 months or so).
Sorry for the "technical" talk, but I wanted to explain the reason of this. You can see more of this if you search "Borderlands 2" in "all games".
I will manually update that game, and any other that i can see.
If you detect any more of those wrong games, please, let me know.
I'm starting to think in creating a page with a form where users could change wrong data.
What do you think about it?
Thanks a lot

I'm starting to think in creating a page with a form where users could change wrong data.
What do you think about it?

You could try, if it's not too much trouble. If people start abusing it or if you're having way too much trouble dealing with those, you can always take it down :)

I've also noticed that there are many games wrongfully name "Mabinogi".

You could try, if it's not too much trouble. If people start abusing it or if you're having way too much trouble dealing with those, you can always take it down :)
I don't think that people would abuse a helping form, and it could help a lot to fill wrong data for removed games from steam or the infamous problem with game release dates (do you remember?)

I've also noticed that there are many games wrongfully name "Mabinogi".
Ok, tomorrow i will refresh the scraping for any game called borderlands 2 and mabinogi.

I will keep this issue open for a while and if you spot any other game with wrong name, just let me know. I will close the issue once i think all the data is fixed.
Thanks

Update

8 Apps called Mabinogi updated
Appids: 212200, 409320, 344040, 541160, 673000, 280040, 477160, 212210
212200 and 212210 have been removed from db.

Those changes should be visible after db is updated and cache is refreshed.
I will test in 24hours in incognito mode.

Update

21 apps called "Borderlands 2 RU" updates
Appids: 217610, 217630, 229150, 232990, 229160, 229170, 257330, 239680, 249920, 268610, 217490, 217550, 248270, 254170, 238300, 217570, 545250, 21730, 351970, 254180, 217590
217610, 217630, 229150, 229160, 229170, 257330, 239680, 217490, 217550, 254170, 217570, 254180 and 217590 have been removed from db.

Those changes should be visible after db is updated and cache is refreshed.
I will test in 24hours in incognito mode.

Update

21 apps called "Trials Rising" updated
Appids: 641080, 928770, 1003010, 1040900, 1041670, 1039370, 809230, 1011470, 1020430, 924180, 1041430, 1020440, 924840, 1028520, 998570, 413610, 1042090, 1008300
_  928770, 924180 and 924840, have been removed from db._

Those changes should be visible after public db is deployed and cache is refreshed (24h)

Update

21 apps called "La Corda d'Oro Octave - 金色のコルダ オクターヴ" fixed

Appids: 1040040,1044640,1011350,1028760,1033620,1042310,1021830,742510,846590,623870,413950,1001470,387580,826620,419580,1005410,1019230,1033820,1041240,1036120,1041750,1015620,997950,1040680,1037860,1042460,1019420,1033490,1035280,998320,998310,998290,998280,998260,998250,998240,1041660,998140,998230,1040200,998210,998200,1027120,998190,998180,998170,998160,1011210,998000,978310,998030,998040,998050,862630,998060,998070,931520,998080,998090,998100,998110,998120,998130

846590, 623870, 413950, 1001470, 387580, 826620, 419580, 998310, 998290, 998280, 998260, 998250, 998240, 998140, 998230, 998210, 998200, 998190, 998180, 998170, 998160, 1011210, 998000, 978310, 998030, 998040, 998050, 862630, 998060, 998070, 931520, 998080, 998090, 998100, 998110, 998120 and 998130  have been removed from db.

Those changes should be visible after public db is deployed and cache is refreshed (24h)

Update "Skullgirls ∞Endless Beta∞"

5 apps fixed
Appids: 275170, 340450, 12770, 471010, 208610

Those changes should be visible after public db is deployed and cache is refreshed (24h)

Update "(Depreciated)"

5 apps fixed
Appids: 295060, 303250, 301710, 292750

Those changes should be visible after public db is deployed and cache is refreshed (24h)
Those changes should be visible after public db is deployed and cache is refreshed (24h)

Update "- Arcane preRaise -"

5 apps fixed
Appids: 609796, 609795, 609794, 609793, 609792, 609791, 603780
609796, 609795, 609794, 609793, 609792, 609791  have been removed from db.

Those changes should be visible after public db is deployed and cache is refreshed (24h)

Update "3DCoat"

4 apps fixed
Appids: 257610, 255050, 234570, 219210
219210 have been removed from db.

Those changes should be visible after public db is deployed and cache is refreshed (24h)

Update "Music Creator 6 Touch"

15 apps fixed
Appids: 297140, 245810, 215850, 261160, 257320, 222710, 293080, 285630, 285620, 285610, 241090, 241490, 241230, 241970, 241960
293080, 285630, 285620, 285610, 241090, 241490, 241230, 241970 and 241960 have been removed from db.

Those changes should be visible after public db is deployed and cache is refreshed (24h)

Update "Agarest: Generations of War"

60 apps fixed
Appids: 372710, 505060, 307940, 437220, 306660, 567780, 374240, 504800, 46560, 372960, 438240, 251080, 265130, 259240, 222630, 268710, 255140, 208790, 259220, 242270, 249910, 246050, 237070, 215310, 237890, 241150, 240890, 240120, 241140, 239860, 240880, 240110, 241130, 240870, 240100, 241120, 240860, 240850, 240080, 240840, 240070, 240060, 240040, 241060, 241050, 240020, 241040, 240990, 239950, 239940, 240950, 239920, 240940, 240930, 239900, 240150, 239890, 240910, 240140, 239880, 240130
241150, 240890, 240120, 241140, 239860, 240880, 240110, 241130, 240870, 240100, 241120, 240860, 240850, 240080, 240840, 240070, 240060, 240040, 241060, 241050, 240020, 241040, 240990, 239950, 239940, 240950, 239920, 240940, 240930, 239900, 240150, 239890, 240910, 240140, 239880 and 240130 have been removed from db.

Those changes should be visible after public db is deployed and cache is refreshed (24h)

Update "Sins of a Dark Age"

4 apps fixed
Appids: 267830, 254260, 238900, 237620, 215860, 251970, 270900
270900 have been removed from db.

Those changes should be visible after public db is deployed and cache is refreshed (24h)

Update "Who Wants To Be A Millionaire?"

4 apps fixed
Appids: 257610, 255050, 234570, 219210

Those changes should be visible after public db is deployed and cache is refreshed (24h)

Update "Tomb Raider"

24 apps fixed
Appids: 213900, 210060, 267660, 208790, 8000, 391220, 225320, 225300, 225020, 225000, 208800, 849310, 905320, 237450, 849170, 849180, 974630, 541750, 750920, 8009, 7000, 8030, 849260, 7030, 849300
7030, 8030, 237450, 541750, 849170, 849180, 974630 have been removed from db.

Those changes should be visible after public db is deployed and cache is refreshed (24h)

Update "Flow DJ Software"

3 apps fixed
Appids: 277770, 257290, 264970

Those changes should be visible after public db is deployed and cache is refreshed (24h)

Update "America's Army: Proving Grounds"

3 apps fixed
Appids: 249730, 203290, 239490

Those changes should be visible after public db is deployed and cache is refreshed (24h)

Update "Resident Evil 4"

4 apps fixed
Appids: 235140, 273540, 254700, 272260
272260 have been removed from db.

Those changes should be visible after public db is deployed and cache is refreshed (24h)

Update "Scrabble"

4 apps fixed
Appids: 258420, 240500, 236660, 234100

Those changes should be visible after public db is deployed and cache is refreshed (24h)

Update "Dead island riptide JP"

2 apps fixed
Appids: 244060, 240220

Those changes should be visible after public db is deployed and cache is refreshed (24h)

@MakoSipper
I have finished to refresh all detected games with wrong games.
Database will be refreshed tomorrow, and any spare game that needs to be updated but I haven't detected, should be automatically updated at most by September, even those are mostly DLCs
Closing the issue


Feature/dvc dag to get grants and generate validation data

Description
It adds a new independent dag to generate the validation data, useful in case we want to change anything:
+------------+              +-------------------------------------------+
| get_grants |              | data/interim/mesh_pipeline_result.csv.dvc |
+------------+***           +-------------------------------------------+
                 ***                  ****
                    ****          ****
                        **      **
           +----------------------------------+
           | generate_validation_data_xlinear |
           +----------------------------------+

dvc repro won't be up-to-date.
Checklist

[ ] Linked to Notion or GitHub issue
[ ] Added tests
[ ] Updated README
[ ] DVC up to date

Release checklist

[ ] DVC repro up to date
[ ] Models synced in S3
[ ] Version updated
[ ] CHANGELOG.md updated
[ ] Model and package version aligned

To release:

make build
make deploy

dvc.lock seems to have changed is that because you run the step you added? You can get dvc repro up to date with your change as soon as #168 merges

Also do you intend for the notebook to be added?

And lastly you are adding one script that runs the pipeline, assuming the pipeline is there and adding the output. I wonder to which extent this belongs here. We have a tag grants script here to not rely on running the pipeline. If you think running the pipeline is the "de facto" way, this probably belongs to the pipeline repo. What is the use case?

⚠ Note that I switched to main for the base branch. I switched the base branch on the PR but for tests to work you need to cherry pick 469b116038bedae879f1ab7068832d198cf12f1d. Just run git cherry-pick 469b116038bedae879f1ab7068832d198cf12f1d if you aim to do more commits and want to run tests against latest main and not the soonoutdated master

And lastly you are adding one script that runs the pipeline, assuming the pipeline is there and adding the output. I wonder to which extent this belongs here. We have a tag grants script here to not rely on running the pipeline. If you think running the pipeline is the "de facto" way, this probably belongs to the pipeline repo. What is the use case?

It's a tricky one. I want to do evaluation with exactly the same dataset that is produced in prod, and I don't want to copy-paste the code in this repository here. So the dvc dag needs one dataset which is generated by an external pipeline, (trigger_pipeline.sh) which I need to add manually, and then it needs to do some ML logic.
Ideally, the prod code wouldn't contain any data processing logic, it would just call the code in here, but this is another discussion I think.

And lastly you are adding one script that runs the pipeline, assuming the pipeline is there and adding the output. I wonder to which extent this belongs here. We have a tag grants script here to not rely on running the pipeline. If you think running the pipeline is the "de facto" way, this probably belongs to the pipeline repo. What is the use case?

It's a tricky one. I want to do evaluation with exactly the same dataset that is produced in prod, and I don't want to copy-paste the code in this repository here. So the dvc dag needs one dataset which is generated by an external pipeline, (trigger_pipeline.sh) which I need to add manually, and then it needs to do some ML logic.
Ideally, the prod code wouldn't contain any data processing logic, it would just call the code in here, but historically this is how it is, so I think it's another discussion.

Well i did make that suggestion to Jeff in #73. Its worth reading what he said.
In any case if you want to run that pipeline just also add instructions on how a user can run it somewhere i.e. the steps to get the code in, something like git clone into that path etc etc

In any case if you want to run that pipeline just also add instructions on how a user can run it somewhere i.e. the steps to get the code in, something like git clone into that path etc etc

I think that's a good idea.

Whether it should be part of the main DAG

Yep, maybe. The question is what I'm using the dag for. I don't think a general user of the package has to fiddle with the dag whatsoever, so no risk in having stuff there that only we can run. Very similar to the other evaluation tasks btw, that require access to private Wellcome S3.

The name of the file that comes from running the pipeline and why we dont use that and instead fetch from 42

That's because the result of the pipeline only returns Grant_ID + Classification, but we need to join this with some extra columns with extra metadata to generate the file for evaluation.

Whether it should be part of the main DAG

Yep, maybe. The question is what I'm using the dag for. I don't think a general user of the package has to fiddle with the dag whatsoever, so no risk in having stuff there that only we can run. Very similar to the other evaluation dags btw, that require access to private Wellcome S3.

The name of the file that comes from running the pipeline and why we dont use that and instead fetch from 42

That's because the result of the pipeline only returns Grant_ID + Classification, but we need to join this with some extra columns with extra metadata to generate the file for evaluation.

👍


Remote URL sources not supported?

I'm getting a 404 for images in this:
{# _components/site-hero.webc #}

<template webc:type="11ty" 11ty:type="njk">
  <div class="hero">
      <picture>
        <source
        media="(max-width: 280px)"
        sizes="(max-width: 640px) 100vw, 640px"
        srcset="
        https://files.cdn.com/assets/images/{{ folder_hero_images }}/webp/portrait/320.webp 320w,
        etc...
<link webc:bucket="hero" rel="stylesheet" href="../assets/css/components/hero.min.css">
</template>

{# index.njk #}

{% renderTemplate "webc" %}
<site-hero></site-hero>
<style @raw="getBundle('css', 'hero')" webc:keep></style>
{% endrenderTemplate %}

Does it have to do with issues/15?

It seems that Nunjucks variables are ignored. Shouldn't 11ty:type="njk" make them work?

Where is folder_hero_images defined? What URL is being requested?

folder_hero_images are defined in front-matter of each page; the URL request images stored in Backblaze bucket and served with Cloudflare Transform Rules


Updated: wireshark 3.2.4

Newer version of wireshark (3.2.5) is now available in master. 6bd39cc4b58e74dfb2c6ab8501cc4d7db0ebb6fd


Implemented New Resources View

Removed test resources, added links to the slack, the website, and the depots

fixes #3


test(ChunkMeshWorker): initial attempt at using reactor-test

Use reactor-test methods to write these tests of ChunkMeshWorker's Flux. Targets #4786.
Key Points to Review:

Are the tests legible? Can you tell what they are supposed to be testing and what they expect?
I tried to centralize the responsibility for calling mesh.dispose by putting that in Chunk#setMesh : 72b0f67 (#4987), does that make sense?

Is that an appropriate use of AtomicReference? Or is it giving me a false sense of security without preventing any real race condition?

 Reactor: ReadWriteStorageManager engine (persistence)

 Reactor: ChunkMesh engine (rendering)

There are still some tests left to complete, but as this is a PR-to-a-PR and I think the tests are more or less legible now, I think this is ready for review.

Brain dumping on ChunkMeshWorkerTest as it's been a couple weeks:
ChunkMeshWorker has a few collections of state that say something about the view of the world or work-in-progress

Set chunkMeshProcessing
List chunksInProximityOfCamera

in addition to

Chunk.isDirty

Those are the things I'm concerned with having tests for, to make sure we're updating them correctly and they are being used effectively.
One of the things chunkMeshProcessing is being used for is avoiding duplicate work:
if two events throw the same chunk (or location) in the queue twice before the first is processed,
we only need handle it once. Doing it twice would be redundant at best,
as it would be working from the same data at approximately the same time.
It might also be serving as a lock to avoid race conditions?
Things with mutable state that might get racy:

a Chunk has block data
a Chunk has a Mesh, which may be replaced or removed
a ChunkMesh may or may not have some initialization data (vertexElements) and its own buffers and GL buffers which manages.

Some things are not a big deal if they are temporarily inconsistent, like a slight lag in reflecting the current block data.
Other things scare me more, like trying to regenerate ChunkMesh VBOs at the same time another thread is uploading its buffers.

 ChunkMesh tests

I got to talk this through with @outofculture. The conclusion we came to (which isn't 100% solid but seems like the intended design) is that ChunkMeshWorker always makes new ChunkMesh objects. And the ChunkMesh that ends up on RenderableChunk.mesh is, in effect, a read-only ChunkMesh.
(In practice, it looks like LodChunkProvider does do chunk.mesh.updateMesh() on a previous mesh, but do LoD chunks get mixed in with other chunks?)
If that's the case, I think I can replace some of the test cases I'm worried about writing with notes to use read-only views.

Thoughts on separating out the mutable methods of ChunkMesh are in #4998. It seems like it's probably viable, but some FallingBlocks-related interfaces make it outside the scope of including in this PR.

the ways lod's work is kind of not great tbh. not sure if it should be generated as part of a chunk and cached together with the chunk. having to regenerate the chunk seems like it adds quite a lot of bandwidth to the gpu.


docs: 📝 增加场景demo「想如何度过一天」

👀 PR includes

🔧 Chore

[ ] Test case
[x] Docs / demos update
[ ] CI / workflow
[ ] Release version
[ ] Other ()

🖼 Screenshot

🐂


Currency mismatch with payment methods balance comparison in the Contribution flow

Describe the bug
If you have an Open Collective Gift Card with you for $150 and you want to donate a project INR165 ($2 approx), the donation panel shows that you don't have sufficient balance to donate INR 165 from $150.
It is assuming INR = USD. The donation amount in whatever currency it might be, open collective is comparing it with USD and not letting the user to donate more than the value of that Gift Card which is in USD.
Briefly, USD 150 = INR 12,270, but you can't donate more than INR 150 at a time using that gift card.
To Reproduce
You must have a Gift Card with some balance
Steps to reproduce the behavior:

Go to Donation, and click on Contribute.
Click on Other and add a value in your local currency which is in figures more than the Gift Card value (eg. donate INR 150 from $160).
Proceed further and you'll see it will say that the balance is not sufficient in you gift card.

Expected behavior
Every time someone tries to donate in their local currency, open collective should first convert it into USD and then check the balance from the Gift Card (whether it is sufficient for the donation or not)
Screenshots

Desktop (please complete the following information):

OS: Windows 11
Browser Chrome
Version 107.0.5304.107

Additional context
N/A

Triage Template (core team only)
This template is for members of the team to triage for prioritisation. For more guidance see https://www.loom.com/share/369ab467fbc64dec848085d38ff57ca0:
P1 high frequency, high impact
P2 low frequency, high impact
P3 high frequency, low impact
P4 low frequency, low impact
Examples of high impact - a problem affects users during an essential process (expenses and payments > onboarding and registration > contributing) with no workaround.
High frequency - >10% of users affected (measured as a proportion of total potential users for this case).

@itsnitish22 What's the collective you're trying to donate to? DigitalOcean gift cards are limited to initiatives hosted by the Open Source Collective, so it's likely that the contribution will fail anyway.
The currency issue does exist though. Using the same approach we've used for expense's amountV2, we could fix it by accepting a  currency argument on PaymentMethod.balance and making sure we fetch something as:
loggedInUser {
  paymentMethods {
    balance(currency: $collectiveCurrency) {
      value
      currency
    }
  }
}

@Betree I am contributing to one of my collectives which has Open Source Collective as a fiscal host, so I don't think there would be a problem anyways donating to the collective.
Not a pro in JS, but got the idea how it would be rectified.

@itsnitish22 Can you link the abovementioned collective so that we can investigate?


misisng yarax binary in /target/release after building

Hi creators of yarax, i am new to linux and currently i want to try yarax after learning yara.
so the problem i am facing is after i clone the yarax github repository, i try to build the release, and i get the finish release [optimized] at the end, but i cant see yarax inside the target/release.
may i know how to install yarax inside my oracle linux please.
Thank you

The final binary for the command-line tool is not named yarax, it's simply yr and you can see it in the target/release directory.


Merging the cluster_config branch

Is the cluster_config branch containing PR #15 able to be merged into the master branch? Autorestarting of docker swarm services would be very useful.

I have just realised that the same thing could be achieved by using the DEPLOY_HOOK environment variable like: DEPLOY_HOOK="docker service update --quiet --force <service name 1>; docker service update --quiet --force <service name 2>"

Yes you can. But in this case, the reload is done for any domain on one specific service. If you want to do it per domain and service basis, you need to use something else.
I am in fact working on the autorestart support on swarm, it will be here soon.

I just release 2.14.0: it includes the autorestart on Swarm feature.

Thanks. Can confirm it works well.


macOS 11

Just wanted to let you know a PR to provide Vagrant / VirtualBox on macOS 11 has been closed unmerged :-(
https://github.com/actions/virtual-environments/pull/4010
Let's hope they figure out how to fix the video driver issues mentioned in the comments before abandoning macOS 10.15!

Oh no, that's sad!

https://github.com/actions/virtual-environments/issues/4060#issuecomment-950736374
https://github.com/actions/virtual-environments/issues/4060#issuecomment-950757410


foobar2000 returns artist twice - once with just text, second time with attribute - XMLMapper can't access second array item

Good day,
I found a problem when trying to connect to a foobar2000 server.
The server returns:
<upnp:artist>The 54 All-Stars</upnp:artist>
<upnp:artist role="AlbumArtist">Various Artists</upnp:artist>

And that's ok, because XMLMapper treats it like an array.
However, I can't access the second item, no matter what I try.
If I map it to a String, I just get the first item. Ok, that's fine, but the second item gets lost.
So then I tried creating a new class:
class SOAPItemAlbumArtist: XMLMappable {
	required init?(map: XMLMap) {}

	var nodeName: String!

	var artistName: String?
	var artistRole: String?

	func mapping(map: XMLMap) {
		artistName <- map.innerText
		artistRole <- map.attributes["role"]
	}
}

And then the definition is as such:
var artist: [SOAPItemAlbumArtist]?
That causes a crash in XMLMapper.swift:90
if var object = klass.init(map: map) as? N {
I tried everything I could to figure this out, including tracing through the source code but I'm not familiar enough with it to determine exactly how to fix it, or if I'm just setting up my mapping incorrectly.
Has anyone seen this issue? Is it a bug, or user error?

@mgainesdev thank you for using this library.
I tested you example and this doesn't happen on my side.
Could you please share your XMLMapper version?
Also, perhaps sharing bigger part of your model and raw XML will help identify the issue.

Sure. I didn't want to post TOO much data.
                    <item id="0/1/0/17/0I" parentID="0/1/0/17" restricted="1">                         <dc:title>Studio 54</dc:title>                         <dc:creator>The 54 All-Stars</dc:creator>                         <dc:date>1998-01-01</dc:date>                         <upnp:artist>The 54 All-Stars</upnp:artist>                         <upnp:artist role="AlbumArtist">Various Artists</upnp:artist>                         <upnp:album>54: Music From the Miramax Motion Picture, Volume 1</upnp:album>                         <upnp:albumArtURI>http://<IP_ADDRESS>:56923/albumart/1287a8209c2f26c8785dc9d15e82227f                         </upnp:albumArtURI>                         <upnp:longDescription>ExactAudioCopy v1.5</upnp:longDescription>                         <upnp:originalTrackNumber>1</upnp:originalTrackNumber>                         <res duration="0:03:48.000" size="40219244" bitrate="176400" bitsPerSample="16"                             sampleFrequency="44100" nrAudioChannels="2" protocolInfo="http-get:*:audio/wav:*">                             http://<IP_ADDRESS>:56923/content/1287a8209c2f26c8785dc9d15e82227f.wav?profile_id=0&convert=wav&subsong=1                         </res>                         <res duration="0:03:48.000" size="40219244" bitrate="176400" bitsPerSample="16"                             sampleFrequency="44100" nrAudioChannels="2" protocolInfo="http-get:*:audio/wav:*">                             http://<IP_ADDRESS>:56923/content/1287a8209c2f26c8785dc9d15e82227f.wav?profile_id=1000&convert=wav&subsong=1                         </res>                         <res duration="0:03:48.000" size="40219200" bitrate="176400" bitsPerSample="16"                             sampleFrequency="44100" nrAudioChannels="2"                             protocolInfo="http-get:*:audio/L16;rate=44100;channels=2:DLNA.ORG_PN=LPCM">                             http://<IP_ADDRESS>:56923/content/1287a8209c2f26c8785dc9d15e82227f.l16?profile_id=1001&convert=lpcm&subsong=1                         </res>                         <upnp:class>object.item.audioItem.musicTrack</upnp:class>                     </item>
The version I'm using is 1.6.1

Again, sorry for the late response.
Your model is correct and has as a result the correct mapping of the provided XML:
class SOAPItemAlbumArtist: XMLMappable {
    required init?(map: XMLMap) {}

    var nodeName: String!

    var artistName: String?
    var artistRole: String?

    func mapping(map: XMLMap) {
        artistName <- map.innerText
        artistRole <- map.attributes["role"]
    }
}

class SOAPRes: XMLMappable {
    required init?(map: XMLMap) {}

    var nodeName: String!

    var duration: String?
    var size: String?
    var bitrate: String?
    var bitsPerSample: String?
    var sampleFrequency: String?
    var nrAudioChannels: String?
    var protocolInfo: String?
    var url: URL?

    func mapping(map: XMLMap) {
        duration <- map.attributes["duration"]
        size <- map.attributes["size"]
        bitrate <- map.attributes["bitrate"]
        bitsPerSample <- map.attributes["bitsPerSample"]
        sampleFrequency <- map.attributes["sampleFrequency"]
        nrAudioChannels <- map.attributes["nrAudioChannels"]
        protocolInfo <- map.attributes["protocolInfo"]

        url <- (map.innerText, XMLURLTransform())
    }
}

class SOAPItem: XMLMappable {
    private static let dateFormatter: DateFormatter = {
        let dateFormatter = DateFormatter()
        dateFormatter.dateFormat = "yyyy-MM-dd"
        return dateFormatter
    }()

    required init?(map: XMLMap) {}

    var nodeName: String!

    var id: String?
    var parentID: String?
    var restricted: String?

    var title: String?
    var creator: String?
    var date: Date?
    var artist: [SOAPItemAlbumArtist]?
    var album: String?
    var albumArtURI: URL?
    var longDescription: String?
    var originalTrackNumber: Int?
    var res: [SOAPRes]?
    var `class`: String?

    func mapping(map: XMLMap) {
        id <- map.attributes["id"]
        parentID <- map.attributes["parentID"]
        restricted <- map.attributes["restricted"]

        title <- map["dc:title"]
        creator <- map["dc:creator"]
        date <- (map["dc:date"], XMLDateFormatterTransform(dateFormatter: SOAPItem.dateFormatter))
        artist <- map["upnp:artist"]
        album <- map["upnp:album"]
        albumArtURI <- (map["upnp:albumArtURI"], XMLURLTransform())
        longDescription <- map["upnp:longDescription"]
        originalTrackNumber <- map["upnp:originalTrackNumber"]
        res <- map["res"]
        `class` <- map["upnp:class"]
    }
}

Apart from some minor changes that I made for the model to be prettier, everything else left intact.
And here are the result from the debugger:

One last question: Are you using cocoapods, Carthage or SPM?

@mgainesdev is there any news on this? Did you tried the proposed model for your mapping?

@mgainesdev is there any news on this? Did you tried the proposed model for your mapping?

Closing this issue as stale. Feel free to reopen it if the problem persists.


[docs] add Statamic setup guide

The Issue
Recently Discord issue highlighted the need for current setup instructions.
There is older article, Setup Statamic via DDEV which is outdated and has issues as discussed in Discord thread.
How This PR Solves The Issue
Statamic is built on top of Laravel, so the instructions are almost identical.
Manual Testing Instructions
Work through setup instructions.
Related Issue Link(s)
There is currently an "unoffical" addon ddev-addon-statamic-please.
This does not include the ddev-get topic and is therefore not disoverable by DDEV. An issue was created to address this. However, the addon is stale and I am unsure if it's being actively developed/maintained.
Not sure if we want to include addon in documents.
Release/Deployment Notes

looking good thank you!
ps: a handy line I learned could be added to automatically get the correct URL in the .env:
ddev exec 'sed -i "/APP_URL=/c APP_URL=$DDEV_PRIMARY_URL" .env'

With type 'laravel' I think that gets updated automatically in current ddev v1.21.4 @hansipete ?

Wondering if this should be included in the side menu?

Wondering if this should be included in the side menu?

I struggled with that a bit. It’s almost identical to Laravel but also obscured by it as a top-level choice—and this is shortly after we cleaned up that top-level menu so it’s easier to browse. Again I think we’re fighting how tabs are used in this context.
I think having Statamic instructions in there and searchable is a positive move, otherwise no strong feelings about where it lives. I’d make broader changes that are out of scope here.

I think having Statamic instructions in there and searchable is a positive move,

Yes. That was the conclusion I came to too.
Merge away.

Merge away.

I assume this part is for @rfay.


Falco 0.11.0 based container fails to load kernel module

Tested on Debian Jessie (k8s-1.4-debian-jessie-amd64-hvm-ebs-2016-10-21 ami from AWS, kernel 4.4.26-k8s) and Ubuntu 16.04 (GKE provide AMI, kernel 4.13.0-1008-gcp) with the Falco 0.11.0 docker container.
* Setting up /usr/src links from host
* Unloading falco-probe, if present
rmmod: ERROR: Module falco_probe is not currently loaded
* Running dkms install for falco

Kernel preparation unnecessary for this kernel.  Skipping...

Building module:
cleaning build area...
make -j1 KERNELRELEASE=4.13.0-1008-gcp -C /lib/modules/4.13.0-1008-gcp/build M=/var/lib/dkms/falco/0.11.0/build....
cleaning build area...

DKMS: build completed.

falco-probe.ko:
Running module version sanity check.
Error! Module version 7640717FDE125E312760246 for falco-probe.ko
is not newer than what is already found in kernel 4.13.0-1008-gcp (7640717FDE125E312760246).
You may override by specifying --force.

depmod...

DKMS: install completed.
* Trying to load a dkms falco-probe, if present
* Trying to load a system falco-probe, if present
* Trying to find precompiled falco-probe for 4.13.0-1008-gcp
Found kernel config at /host/boot/config-4.13.0-1008-gcp
* Trying to download precompiled module from https://s3.amazonaws.com/download.draios.com/stable/sysdig-probe-binaries/falco-probe-0.11.0-x86_64-4.13.0-1008-gcp-8f812ada63c6c5ca48357a632afbb4aa.ko
Download failed, consider compiling your own falco-probe and loading it or getting in touch with the sysdig community
Wed Jul 25 14:28:57 2018: Falco initialized with configuration file /etc/falco/falco.yaml
Wed Jul 25 14:28:57 2018: Loading rules from file /etc/falco/falco_rules.yaml:
Wed Jul 25 14:28:57 2018: Loading rules from file /etc/falco/falco_rules.local.yaml:
Wed Jul 25 14:28:57 2018: Unable to load the driver. Exiting.
Wed Jul 25 14:28:57 2018: Runtime error: error opening device /host/dev/falco0. Make sure you have root credentials and that the falco-probe module is loaded.. Exiting.

Error in dmesg is [78962.345914] module: falco_probe: Unknown rela relocation: 4.
Manually trying to modprobe the module built by the container gives the error.
$ sudo modprobe falco-probe
modprobe: ERROR: could not insert 'falco_probe': Exec format error

RPM based install works fine. Also, running falco-probe-loader from the RPM install with the kernel module compiled from the container image loads the kernel module.

Btw, the latest version of sysdig has the same problem:
...
DKMS: install completed.
* Trying to load a dkms sysdig-probe, if present
* Trying to load a system sysdig-probe, if present
* Trying to find precompiled sysdig-probe for 4.13.0-26-generic
Found kernel config at /host/boot/config-4.13.0-26-generic
* Trying to download precompiled module from https://s3.amazonaws.com/download.draios.com/stable/sysdig-probe-binaries/sysdig-probe-0.22.0-x86_64-4.13.0-26-generic-94a76b51a68ec35679d1401155803cb6.ko
Download failed, consider compiling your own sysdig-probe and loading it or getting in touch with the sysdig community
$ grep sysdig_probe /var/log/syslog | tail
Jul 25 14:37:00 ubuntu kernel: [ 2539.557399] module: sysdig_probe: Unknown rela relocation: 4
Jul 25 14:37:00 ubuntu kernel: [ 2539.580730] sysdig_probe: driver loading, sysdig-probe 0.21.0
Jul 25 14:37:00 ubuntu kernel: [ 2539.580805] sysdig_probe: failed to find page_fault_user tracepoint, disabling page-faults
Jul 25 14:37:00 ubuntu kernel: [ 2539.580806] sysdig_probe: failed to find page_fault_kernel tracepoint, disabling page-faults
Jul 25 14:37:12 ubuntu kernel: [ 2551.695765] sysdig_probe: driver unloading
Jul 25 14:37:59 ubuntu kernel: [ 2598.196968] module: sysdig_probe: Unknown rela relocation: 4

And as a workaround, on debian-based distros you may be able to upgrade to a kernel that has the backport, which will allow the kernel module to be loaded.


Switch gears on Markdown/syntax highlighting

I've ripped prism and react-markdown back out in favor of highlight.js and remarkable. By combining the two we have a component that handles markdown and syntax highlighting in one shot.

Hi @egid, please rebase this to pick up #365, which just landed.


Error in data download from Markets Data API V1 - 4

Extract from JSON download:
{
"symbolInput": "GB00B8BZ3226",
"basic": {
"symbol": "GB00B8BZ3226:GBP",
"name": "Threadneedle Monthly Extra Income Fund Z Income GBP",
"exchange": "Lipper",
"exhangeCode": "LIP",
"currency": "GBP"
},
"quote": {
"lastPrice": 137.51,
"previousClosePrice": 137.89,
"change1Day": -0.38,
"change1DayPercent": -0.27558199,
"timeStamp": "2017-02-20T05:00:00"
}
"currency" should either be "GBp" and symbol suffix ":GBX", or prices should be divided by 100.

duplicate #50


Allow users to configure tasks location

Right now, when you install a new task, it's located by default under app/tasks and {test|spec}/tasks. In legacy systems, this might not be a convenient location, and I think it would be worth it to expose this location for configurability, since the typical location would be app/jobs/migration instead.
EDIT: Sorry, I meant app/jobs/maintenance instead of app/jobs/migration

I don't see a compelling reason to support a configurable path for Tasks. Just as any other generator, the maintenance_tasks:task generator will create Tasks under app/tasks, which is where Tasks are supposed to be found in a Rails app that uses our gem. It wouldn't make sense to place Tasks under app/jobs either, since Tasks and Jobs are different things.

I don't see a compelling reason to support a configurable path for Tasks. Just as any other generator, the maintenance_tasks:task generator will create Tasks under app/tasks, which is where Tasks are supposed to be found in a Rails app that uses our gem. It wouldn't make sense to place Tasks under app/jobs either, since Tasks and Jobs are different things.

Ok. So let's close this PR. Thank you so much for the contribution.

Ok. So let's close this PR. Thank you so much for the contribution.


Questions about spine phantom data

Hi,
Thank you for your awesome work! I noticed in the spine_phantom2 folder there are left1, left1_1, left2_2 and left3. I'm wondering what the meaning of the notes is. Are they actually the same sequence? Do the data still need to be recalibrated?

Hi, they are calibrated and registered. Name suggests the direction of scanning from the transverse perspective. The numbers tell you that they are overlapping in the middle.


use vscode.workspace.fs.stat API for file existence checks (instead of ripgrep)

The new implementation is simpler (uses a standard VS Code extension API, no dependency on ripgrep, no exec'ing a subprocess) and faster.
The previous implementation was more lenient and applied an implicit **/ glob prefix to each file path. For example, if a workspace contained only the file x/a/b.txt, then the previous implementation would report that the path a/b.txt exists. That leniency reduced the rate of false hallucination reports. TODO See PR comment below...
For the query Which files define SAML auth? on the sourcegraph/sourcegraph repository (same as that used in the original benchmark at https://github.com/sourcegraph/sourcegraph/pull/52541):

Previous vscode.workspace.findFiles implementation: 2784ms (just to confirm I repro'd the terrible slowness initially reported from this impl)
Current ripgrep implementation introduced in https://github.com/sourcegraph/sourcegraph/pull/52541: 64ms
New implementation (using vscode.workspace.fs.stat): 2ms

All operated on 5 paths in Cody's response.
Test plan
TODO

@beyang Actually, I didn't realize initially that it was intentional that the previous implementation was lenient. (Eg it would report that a/b.txt existed if ANY file in the workspace matched ROOT/**/a/b.txt/**, not just if a file ROOT/a/b.txt existed.) That makes the previous implementation make a LOT more sense, heh.
How important is it actually to have this lenient behavior? All file paths mentioned by Cody in prose should be relative to the root anyway (not partial); otherwise, the user couldn't click on it, the partial path could be ambiguous (could refer to multiple paths), and it could be a hallucinated partial path that happens to exist elsewhere. In practice, I don't recall ever myself seeing any intentional partial paths in Cody's responses.
So, I propose we change the behavior of the file existence checker to be stricter to simplify this code by a lot, and then we use prompts and context to make Cody mention only paths that resolve unambiguously to the root.


Icon size improvement

Hello,
This is a very nice extension!
I tried it and I immediately noticed an issue with the icons being too small compared to the original ones.
I added the following in main.js (after const newSVG = document.createElement("img");):
newSVG.style.width = "20px";
newSVG.style.height = "20px";
newSVG.style.marginTop = "1px";
newSVG.style.marginBottom = "-1px";

This makes it so much better IMO. Please try it out / consider this.

Hey @christosnc . Thanks for the suggestion and the code , I tried it out and I do indeed like the slight increase. I am a bit afraid of increasing the size though because all users are likely already used to the current size and a sudden increase might be the kind of change that is weird to people who have been using the extension for a long time.
Let me think about this for a second and see if I can come up with a smooth way to allow for this change or if I'm just overthinking it.
Sorry about the delay as well, I know it's been a while!

Hi, I would be interested in seeing this implemented. Perhaps there could be an extension setting that allows users to select the icon size from a list of options, "small", "medium", and "large". The current icon size that many users are used to could be the default size and maybe the "small" option. @christosnc's proposed adjustment could be "medium".

@elijaholmos That's a good idea. I'm working on an options page where this setting can be selected, along with some other options that would unblock a couple of other issues. My first goal is to fix the auto-release actions (because I'm bound to forget to check the version in the stores again ...) but this is the immediately next priority.

I generally agree that making them bigger would be a nice improvement! Even just 18x18px looks a bit better.  There are two things to keep in mind though.

Different Git providers use different default sizes (Bitbucket icons are large by default), so if you added an option for changing the sizes you'd need to make sure the custom sizes made sense for each site they were being applied to
Generally this extension is good about replacing the icons before the first render, but in the situations where it doesn't, it can be a weird look if the icon is so big that it either pushes the text over after the first render, or changes the rows height. This extension is an example that uses larger icons that expand the row's height, but only after it first renders the original icons, and its not a great look: https://chrome.google.com/webstore/detail/atom-file-icons-web/pljfkbaipkidhmaljaaakibigbcmmpnc. In most cases this won't be a problem if you don't make them too large, but its definitely something to keep in mind!

@csandman those are very good points.
Looking at our current providers I've been thinking about defining a multiplier/factor for each provider, and using that to multiply the "size" of the icons chosen by the user. This way we could have a "medium" icon that is slightly different in each provider to ensure it matches the size of the row.
Agreed that the default size should not affect row sizing in any way. That is very jarring

@csandman solid proof of concept! the approach you described sounds efficient, nice job.
I agree that different size settings per site would be convenient. perhaps there is a default setting that applies to all sites, then site-specific overrides that could be enabled. i.e. a user prefers the extra large icons on all sites by default (which may be different than the default that is shipped with the extension), so each time this user visits a new site, they would have to enter the settings to change the size from the extension's default to their preferred. if the extension's default icon size could be modified by the user, then this annoying process will be eliminated for this group of users, while also satisfying users who want different icon sizes depending on which site they're visiting.

can you please increase the icon size? or make an option to set a custom size for all the icons

@Claudiohbsantos I also realized that using a shared size across all git provider websites might be totally fine considering most people only really spend most of their time on one site.  Just a thought anyway!

@csandman I agree, but I got a tad excited and went all out lol
I just merged a popup based on the branch you started (thanks for that, it was a massive head start) and an options page that allows providers to be customized individually.

@elijaholmos I took a stab at a default setting that is applied to all providers. Providers can have custom settings that override the global default but the default can be easily set as a base preference for a user.
This should all be released on the next store push, which I'm hoping to get out sometime this week.
I'll close this issue since I believe the request is fulfilled by the changes but feel free to tag me here if I missed anything/any follow up discussion comes up.

@Claudiohbsantos I'm glad you got some use out of my branch!  The changes look great!
One potential idea that you might want to implement is reloading the page when the icons are enabled/disabled.  It's simple enough to do using the chrome.tabs.reload function:
chrome.tabs.query({ active: true, currentWindow: true }, (tabs) => {
  const activeTabId = tabs[0]?.id;
  if (activeTabId) {
    chrome.tabs.reload(activeTabId);
  }
});

Up to you, just thought it might simplify things a bit!

@csandman I actually had some similar code in at some point but decided to pull it out because I think unannounced page reloads have the potential to be a pain. For example: You are in the "Go to file" page and have a long complex filter typed that you've been using to narrow down to a specific set of files. You open the extension popup to increase the size of the icon because you want to take a screenshot but accidentaly tap the disable checkbox and the page reload. Boom. Your filter is gone and now you hate me deeply for a couple of seconds :laughing:
I admit the downsides are not that intense and it would be more elegant to automatically change icons but ultimately I decided the best way to do it would be to find a way to restore the icons without reloading the page but that it also wasn't worth the effort for now. Since I don't expect users to frequently toggle the extension it didn't seem like a priority :shrug:

That's a fair point and something I had thought about, but my mind went more to writing our an issue, pr, or comment on one of those two.  And I realized GitHub actually saves the state of those text fields when you reload, which would reduce the likelihood of causing problems.  But thats Github and I have no idea how the rest of the providers would handle a reload.
I also would generally think that most people wouldn't be changing things while they're in the middle of typing something out.  But you're right, it doesn't really offer that much benefit vs the potential pain point if it messes someone's progress up.  And yeah, changing the icons back without reloading would of course be the perfect solution, but with the way icons are being replaced right now, that seems like a larger undertaking than its probably worth haha.


simplify if condition

refactor cluster.go
Reduce elseif statements and distinguish between errors.IsNetwork and errors.IsMoved
		if errors.IsNetwork(err) {
			...
			break
		}
		moved, ask, addr := internal.IsMovedError(err)
		if moved {
			...
		} else if ask {
			...
		} else {
			...
		}

Thanks!

This change is

Hi,
This looks good, but can you please rebase this on v5 branch?

@vmihailenco
rebased, and pushed.
Thanks for your quick reply!

:+1:


Added support for using middleware without gRPC

Issue describing the changes in this PR
resolves #808
Pull request checklist

[X] My changes do not require documentation changes

[ ] Otherwise: Documentation issue linked to PR

[ ] My changes should not be added to the release notes for the next release

[X] Otherwise: I've added my notes to release_notes.md

[X] My changes do not need to be backported to a previous version

[ ] Otherwise: Backport tracked by issue/PR #issue_or_pr

[X] I have added all required tests (Unit tests, E2E tests)

@bbieniek we've been following up on the issue, but this duplicates some of the surface we have int the Core assembly. Are those not sufficient? Thanks!

Thanks for the explanation in the other ticket. This can be closed not


Updates based on review.

Assuming the Beta-since support is now live, pls update the PR with @Beta(SinceVersion.V1_1_0):

on top of each of the new interfaces (just the models, not the fluent definition/update ones),
the new ACS/ACR Manager classes,
any new methods exposing ACS-related functionality added to already previously released interfaces or objects (e.g. Azure)


Fix issue with duplicate field for WaPo in Solr.

Fixing bug introduced by https://github.com/castorini/anserini/commit/c5ee9af442c500ec43fc28808903cfca2417ac12

Great! Go ahead and merge when Jenkins turns green.

So this only works because of converting the value to String, which might work out correctly assuming both types resolve the same string value - otherwise there could be an order effect, and different documents could get different values.

@r-clancy ?

So this only works because of converting the value to String, which might work out correctly assuming both types resolve the same string value - otherwise there could be an order effect, and different documents could get different values.

The implementation of stringValue() doesn't convert arbitrary objects to a String (using toString() for example). It looks like the implementation does this for a Number only (and keeps Strings, of course):
@Override
  public String stringValue() {
    if (fieldsData instanceof CharSequence || fieldsData instanceof Number) {
      return fieldsData.toString();
    } else {
      return null;
    }
  }

However, your comment has made me realize that the order of the checks in this PR should be flipped as the numericValue() method has stricter checking:
@Override
  public Number numericValue() {
    if (fieldsData instanceof Number) {
      return (Number) fieldsData;
    } else {
      return null;
    }
  }

Our code should be this:
if (field.numericValue() != null) {
    solrDocument.addField(field.name(), field.numericValue());
  } else if (field.stringValue() != null) { // For some reason, id is multi-valued with null as one of the values
    solrDocument.addField(field.name(), field.stringValue());
}

The ordering within a document collection would already be consistent, but I think this just makes more sense: try the numerical value first and fallback to the String. Since fields are added to Lucene documents in a fixed order in the Generator classes and iterating over fields happens in a deterministic order, this should be fine.

Sounds right, tnx for the clear explanation!


QB-target menu

I don't get it to work, and also the UI it's impossible to close

Same issue here

I dont updated the script for a long time, you have to edit yourself.

Could you tell me the command to close the menu?


VPS Install

"sudo -u postgres createuser -D -A -P stringer"
This instruction is not correct, there is no such option "-A"
_root@stringer:~# createuser --help
createuser creates a new PostgreSQL role.
Usage:
createuser [OPTION]... [ROLENAME]
Options:
-c, --connection-limit=N  connection limit for role (default: no limit)
-d, --createdb            role can create new databases
-D, --no-createdb         role cannot create databases (default)
-e, --echo                show the commands being sent to the server
-g, --role=ROLE           new role will be a member of this role
-i, --inherit             role inherits privileges of roles it is a
member of (default)
-I, --no-inherit          role does not inherit privileges
-l, --login               role can login (default)
-L, --no-login            role cannot login
-P, --pwprompt            assign a password to new role
-r, --createrole          role can create new roles
-R, --no-createrole       role cannot create roles (default)
-s, --superuser           role will be superuser
-S, --no-superuser        role will not be superuser (default)
-V, --version             output version information, then exit
--interactive             prompt for missing role name and attributes rather
than using defaults
--replication             role can initiate replication
--no-replication          role cannot initiate replication
-?, --help                show this help, then exit
Connection options:
-h, --host=HOSTNAME       database server host or socket directory
-p, --port=PORT           database server port
-U, --username=USERNAME   user name to connect as (not the one to create)
-w, --no-password         never prompt for password
-W, --password            force password prompt_

VPS really needs a lot of work... most of instructions fails...
Missing components...
"Unable to load application: ExecJS::RuntimeUnavailable: Could not find a JavaScript runtime."

foreman export systemd systemd-services -a stringer -u stringer - Error!
Missing OpenSSL, sudo, nodejs packages
SECRET_TOKEN=  ... incorrect lenght
...

@vdias would you mind making a PR to improve the documentation?

@mockdeep I can take this issue as well


doesn't check X-Google-Dkim-Signature

perhaps instead you should use https://github.com/jhermsmeier/node-dkim

https://bugs.launchpad.net/dkimpy/+bug/1729884


Using Data Helper 'primary_tag' returns empty

Issue Summary
Using Data Helper 'primary_tag' inside page with a post returns empty instead of url (neither relative nor absolute work).
To Reproduce

Added this code to header template (After ):

{{#is "post"}}

        {{post.primary_tag.name}}

{{post.title}}

{{/is}}

Renders:

    General but Relevant

Added the #post data helper:

{{#is "post"}}{{#post}}

        {{primary_tag.name}}

{{title}}

{{/post}}{{/is}}

Exact same result. All the other code I have works, just the primary_tag.url syntax loads empty.

I forgot to mention, I have a custom route for tags (instead of '/tag/slug/' it's '/slug/'). And the strange thing is that inside the post body it does get the correct url, but it's because it uses the tags method:
{{#if tags}}

{{tags separator=" "}}

{{/if}}

Technical details:

Ghost Version: 2.5.0
Node Version: 8.12.0
Browser/OS: Docker Alpine on Ubuntu 18
Database: MariaDB

Hey @foleman!
{{#post}}
    {{#primary_tag}}
        {{url}}
    {{/primary_tag}}
{{/post}}

Many questions can be answered by reviewing our docs for self-hosters, our theme API, or our public API. If you can't find an answer then our forum is a great place to get community support, plus it helps create a central location for searching problems/solutions.

@kirrg001 Yes, I've read them.
So you are telling me that all the data helpers work just fine (post.title, post.primary_tag.name, author.name,... etc) but accessing post.primary_tag.url not? Why can I not access post.primary_tag.url with that syntax, but can access post.primary_tag.name and every other object (as I said, post.author, post.title, post.slug, ....).
And outside a post context too. If I'm in an author page I can access author.name, author.slug.. etc. I don't think it's natural that post.primary_tag.url doesn't return the url, but does return it's other attributes.

@foleman Raised a few valid questions that deserved a response. I am also seeing this issue in the default RSS Feed configuration where the  element containing {{primary_tag}} does not render.


SimpleDependencyGraphPlugin does not capture plugin dependencies

Following the example in the readme, when executing the following, plugin/buildscript dependencies are not being captured.
gradle -I init.gradle --no-configuration-cache --no-configure-on-demand :ForceDependencyResolutionPlugin_resolveAllDependencies

It would appear that support for this was added to the GithubDependencyGraphPlugin here.

Why do you believe these dependencies are not being captured? My testing indicates that they are.
Can you please provide a project that allows me to reproduce this issue?

@bigdaz here's a repo reproducing what I'm seeing and explaining what I think should be happening https://github.com/brianmcgee/github-dependency-graph-plugin-bug

Thanks for the reproducer. I'm not sure what's happening on your end, but when I run the exact command against that repo I do see the Spotless Gradle plugin listed in the dependencies:

You might find it useful to add --scan to your invocation: the generated build scan captures all resolved dependencies and is a good way to share your results.

I updated my reproducer to include the report I'm generating, which matches what you listed above. I also added a second plugin to demonstrate the issue better.
The report above shows an entry for com.diffplug.spotless:spotless-plugin-gradle:6.25.0 which is sourced from https://repo.maven.org.
But it does not include an entry for com.diffplug.spotless:com.diffplug.spotless.gradle.plugin:6.25.0 which is sourced from https://plugins.gradle.org.
Similarly, you will see in my updated example that com.github.johnrengelman.shadow:com.github.johnrengelman.shadow.gradle.plugin:8.1.1 is not included in the report.
Here is a scan report for the latest run: https://scans.gradle.com/s/iitusscge3aao

The specific dependencies you mention are referred to as plugin marker dependencies. The don't contain any code, and simply help gradle resolve a plugin id to an actual plugin dependency artifact.
These are deliberately and explicitly excluded from the report: https://github.com/gradle/github-dependency-graph-gradle-plugin/issues/111

Ah, that makes sense.
Essentially, if you have a dependency of the form <group>:<module>-plugin-gradle:<version> you can reasonably assume the plugin marker to be <group>:<group>.gradle.plugin:<version>.
In answer to why I'm interested in these, I'm using the dependency report from the plugin to construct a lock file for use with Nix that allows me to construct a local maven repository containing all the build dependencies so the build can be run in offline mode within the Nix sandbox.
This was the last piece of the puzzle and it looks like I have it working.
Thanks for your help :pray:

If it's important to you, I'd be open to a PR that made it possible to retain the plugin marker artifacts via some sort of env var.
Here's where the code is: https://github.com/gradle/github-dependency-graph-gradle-plugin/blob/321383e803910581d412d92e05f6496173cea82c/plugin/src/main/kotlin/org/gradle/dependencygraph/extractor/DependencyExtractor.kt#L248-L262
Any PR would require test coverage.

I think my immediate use case is covered, but I'm happy to look at creating a PR to capture them properly.
I'll try and find some next week to give it a go.


Back to development: 3.5.1

Coverage remained the same at 94.541% when pulling 44fd421f81e80caf21347732fc9b3421b55ee71f on btooood into 3c74cc80fb25de72488c68f02d62b3aa79168499 on master.


Egg-Fertilization does not always apply

There are two issues at the moment,

If eggs are not (quickly) fertilized, they remain unfertilized. This happened particularly often with vaginally placed eggs.
Insect Eggs (from animals with ovipositors) are just shat out and not fertilized.

Ok, it seems to be ok.
There is a block on fertilizing high-gestation eggs in the base rjw function which is why my eggs are not fertilized.
Especially if (for test settings) egg pregnancy duration is set shorter, it's very common that the Gestation too high and then nothing happens (which is expected).
I updated some logs and I guess that's it.

I made some prints, but it would not need to be fixed here.


fix windows 10 error

fix windows 10 error message box
#38 #40

I made my own commit because here you remove quite important security (which can break Windows 10), but thank you it really helped me with the project!


Pass the mem_len argument into the uBPF VM so it can be used by programs.

Currently the ubpf_exec function takes both a pointer to a memory structure and the length of that structure as inputs. However only the pointer is passed into the VM itself (via the R1 register). The length is used for range checking in the verifier step but not for anything else.
It would be useful when running programs to have the length of the input memory structure available. I propose that we add that via the R2 register. The eBPF spec allows R1-R5 to be used for such purposes.
Longer term a more generic approach to populating these registers would be nice. But we can start with mem_len and go from there.
I'll try and work up something for this.

I see this is a duplicate of PR #22 which was never merged. I am happy to update my PR with the right tests if someone points me in the right direction for how to do that.


htop: fix config merging

formatMeters unfortunately does not make it easy to change the priority, so this is what I came up with.
If there is a better way to do this, please let me know.
I did not test this, so please do.

It took a while before I went back to updating the configuration. :D
I solve the problem in a slightly different way by using lib.mkForce on the override side.
https://github.com/name-snrl/nixos-configuration/commit/667495a34bf141db73fcf1029fc4b016987ba409#diff-41489789006ed24b44970ea370c0dd891d3aed42b0e06c4c2fb39ccfd9c226ef
But this PR showed me the way, thank you.

This is indeed nicer, thank you!


JDA doesn't restore the currently playing game after reconnect

I'm using 2.0.0_245 and my game/status gets reset to null after the bot auto-reconnects.
No errors are shown in the console, apart from the disconnection message.

If I am correct:
Create a new listener (refer to the examples if needed). There is a method called onDisconnect and onReconnect. Use these to manage what will happen in the case of a disconnect or reconnection.

JDA used to take care of game statuses upon reconnecting. Not sure why this stopped being the case, I'm pretty sure it's unintentional.

We know of the issue but are not quite sure of the cause. Will continue to test this issue.