Render callbacks no longer gets routerParams as of version 0.12

Render callbacks (specifically renderFetched) used to get routerParams passed into them as part of the data prop. As of 0.12 this no longer happens. Specifically the change is here:
https://github.com/relay-tools/react-router-relay/commit/1c38a0e63e2e001080bfdee41ddb03da2de06097#diff-02147c9abf35e579edc77adcfc038997R44
We are using these router params to render the component (data.route.component) along with some additional data. I was wondering whether this was an intentional change, and if so what's the best way to override the rendering step now without having to declare each component twice in each route?

That's unintentional. I'll fix it shortly.

Released on v0.12.1 (and v0.12.2).


COM are not initialized by using the initalizion code in DirectXTex.md

Initialization
The library assumes that the client code will have already called CoInitialize, CoInitializeEx, or Windows::Foundation::Initialize as needed by the application before calling any DirectXTex routines. Most DirectXTex functions use the Windows Imaging Component which requires COM.
For a Universal Windows Platform (UWP) app, the Windows Runtime and COM is initialized by the C/C++ Run-Time. For a classic Windows desktop application you have to do this explicitly:
#if (_WIN32_WINNT >= 0x0A00 /*_WIN32_WINNT_WIN10*/)
    Microsoft::WRL::Wrappers::RoInitializeWrapper initialize(RO_INIT_MULTITHREADED);
    if (FAILED(initialize))
        // error
#else
    HRESULT hr = CoInitializeEx(nullptr, COINITBASE_MULTITHREADED);
    if (FAILED(hr))
        // error
#endif

In desktop win10,  when I try to use DirectX::LoadFromWICFile() function, it returns E_NOINTERFACE, I think it's imopssiable.
As I use debugger to find out why it returns this, I found
line 1103
IWICImagingFactory* pWIC = GetWICFactory(iswic2);
    if (!pWIC)
        return E_NOINTERFACE;

get into GetWICFactory(iswic2)
line 280:
    if (!InitOnceExecuteOnce(&s_initOnce,
        InitializeWICFactory,
        nullptr,
        reinterpret_cast<LPVOID*>(&g_Factory)))
    {
        return nullptr;
    }

into InitializeWICFactory()
line 78:
       HRESULT hr = CoCreateInstance(
            CLSID_WICImagingFactory2,
            nullptr,
            CLSCTX_INPROC_SERVER,
            __uuidof(IWICImagingFactory2),
            ifactory
        );

here, hr is E_NOTINITIALIZED,means COM are not initialized.
So, maybe we need to change this initializiation code?

I'm sorry, but I'm at a loss as to the cause of your problem. The Microsoft Docs covers this in detail, so hopefully something there makes sense to you.


Build Folder and Record in Quickstart Updates 0.6.0

Went ahead and found all places where code and copy needed to be updated to 0.6.0 (inline with Quickstart changes) and myRecord > record in Quickstart updated well.
One thing we didn't change yet in this PR was the Build > Apps > ToDo app because it's not a breaking change right now bc the app example pinned at the top is using 0.4.0 SDK.

Added some more fixes to Quickstart, squashed, put upstream in 1bdde5f3822fafc4dbd8daa990bc1b794210404d


Footer and Header not working

When I try to add a footer using this code nothing shows up.
$footer = "
Hello how are you
hhhh
hhhh
";
$pdf->addPage($top.$specHtml.$modelHtml.$picHtml, array('footer-html' => $footer));

Everything else is generated fine.
When I use the same code but with header-html it show a small white space at the top of my pages but no other content is inserted.

$pdf = new Pdf(array(
'header-html' => getcwd().'/footer.html',
'footer-html' => getcwd().'/footer.html',
));
why is this not working mikehaertl?

@cerpnjaki Does it work on the commandline with raw wkhtmltopdf? And what is $pdf->getError()?

mikehaertl tnx for you comment.
$pdf = new Pdf(array(
'no-outline',
'margin-bottom' => '5cm',
'footer-html' => 'http://www.google.com'
        ));
        $pdf->addPage('<!DOCTYPE html>somehtml');

        if(!$pdf->send()) {
            echo "Error: ".$pdf->getError();
            die;
        }

i tried twice: onetime with this parameter: 'no-outline', <- i get error as it is down
second time without 'no-outline', parameter. <- i do not get error and i do no get pdf
this is my error message:
Error: Unknown long argument --no-outline Name: wkhtmltopdf 0.9.9 Synopsis: wkhtmltopdf [OPTIONS]...  [More input files] Description: Converts one or more HTML pages into a PDF document, not using wkhtmltopdf patched qt. General Options: --collate Collate when printing multiple copies --copies Number of copies to print into the pdf file (default 1) --extended-help Display more extensive help, detailing less common command switches -h, --help Display help -O, --orientation Set orientation to Landscape or Portrait -s, --page-size Set paper size to: A4, Letter, etc. --password HTTP Authentication password -p, --proxy Use a proxy -q, --quiet Be less verbose --username HTTP Authentication username -V, --version Output version information an exit Reduced Functionality: This version of wkhtmltopdf has been compiled against a version of QT without the wkhtmltopdf patches. Therefore some features are missing, if you need these features please use the static version. Currently the list of features only supported with patch QT includes: * Printing more then one HTML document into a PDF file. * Running without an X11 server. * Adding a document outline to the PDF file. * Adding headers and footers to the PDF file. * Generating a table of contents. * Adding links in the generated PDF file. * Printing using the screen media-type. * Disabling the smart shrink feature of webkit. Contact: If you experience bugs or want to request new features please visit , if you have any problems or comments please feel free to contact me: see

Well, you try to use an option that is not supported by your wkhtmltopdf version:

Unknown long argument --no-outline Name: wkhtmltopdf 0.9.9

So you should upgrade to a newer version or - if you already have it somewhere - set the full path to that file in the binary option.

Also better avoid the google page as test. Their site is probably a bit more complex than it looks. Better try with a local header/footer file first.

ok mikehaertl, i will try and give you feedback! tnx for your realy fast replies!
LG, Igor

I'm closing this issue for now as the problem of OP is with wkhtmltopdf, not with this library.

I had simillar problems and I want to left some notes here.
Official documentation of wkhtmltopdf says that header and footer options are only available using patched QT and I didn't have patched QT.
I found instruction of installation this right here and now everything works:
http://stackoverflow.com/questions/35617491/how-to-install-wkhtmltopdf-patched-qt-without-compiling

the problem is about css float. try to remove it and render the pdf again

If testing with --header-html option, make sure you include the following as part of your string:
<!DOCTYPE html>
Once I added that, it works flawless in both wkhtmltopdf in command line or using phpwkhtmltopdf


feat: bootstrapping framework across all OpenStack charts

This feature requests that we support a consistent bootstrapping request framework across all charts.
Every OpenStack service has resources that can be created within the service once it is actually up.  Our charts deploy very vanilla versions of services, and to be sure, this is the correct path for us.  However, users wishing to deploy a functional cloud will need some basic items and we want to provide a mechanism to load the items at chart install.
A few examples:
Glance currently deploys with no images and Nova does not deploy with any usable flavors.  It should contain a bootstrap section in values.yaml containing the images to inject once the glance server is up (and the appropriate job to perform the action) and Nova should come up with usable flavors for deployment.
This is just an example:
bootstrap:
  images:
    ubuntu:
      name: Ubuntu Precise 64-bit
      source: http://cloud-images.ubuntu.com/precise/current/precise-server-cloudimg-amd64-disk1.img
      min_disk: 20
      image_type: qcow

Nova right now has no flavors, a bootstrap section in values.yaml could create the default out of the box flavors OpenStack users expect:
bootstrap:
  flavors:
    m1.tiny:
      name: m1.tiny
      id: auto
      ram: 512
      disk: 20
      vcpus: 1

In addition, there are other things we should be doing as part of the bootstrap process (off the top of my head):

Creating and setting cinder volume types (required for some horizon workflows)
Loading glance metadata schema (though we could include this in db migration)
Create initial external network in Neutron
Create appropriate initial User Roles in Keystone (and stop using admin for service accounts!)

So as a matter of consistency is it worth applying a similar approach to the keystone chart? We've been discussing this & can see pros and cons either way. Adding bootstrap functionality means that now there's been an arbitrary determination of 'mandatory' and 'optional' configurations. One needs a nova user for a functioning system but it will operate fine without a cirros image loaded. UNLESS you want to run tempest, in which case that optional image now becomes mandatory. I want to add a selenium helm test to the horizon chart which requires a couple users to exist in keystone, those are now mandatory for us but perhaps no one else. I guess the point is that whats mandatory or not will depend on the environment, so enforcing a strict definition might not make sense.
If we were to follow the current pattern of each chart configuring only the services it needs then we'd be copying and pasting job-g-* yaml into every chart that might want a image created. Currently users are configured in each individual chart and the job-ks* pods are replicated everywhere referencing the helm-toolkit common code. It would make a fair amount of sense to consolidate all the user / role / endpoint configuration to the keystone 'bootstrap:' values.yaml along with the associated templates and pods. This would decrease the startup time for any chart depending on keystone as they would have less to setup & lower the number of overall dependencies overall across the entire system.
To simplify my lengthy blathering stream of conscious - should charts configure themselves or should charts dictate to other charts what their needs are? Doing both on a per-case definitely seems like the wrong answer.
Just thinking out-loud here, curious what others opinions are.

@larryrensing pointed this out as something I could look at in Openstack-Helm. Following his Nova and Glance PR as examples I have been working on a Cinder bootstrap and learning the Helm development process through it. Once I wrap my head around one I hope to knock them out fast.


Cant install maps from ‘abatzill.de’

Stratux version: Linux stratux 5.15.32-v8+ #1538 SMP PREEMPT Thu Mar 31 19:40:39 BST 2022 aarch64

Stratux config: Raspberry Pi 3+
SDR

[ ] single
[ x] dual

GPS

[ ] yes
[x ] no
type:

AHRS

[ ] yes
[ x] no

power source: Ok
usb cable: Ok

EFB app and version: (e.g., WingX Pro7 8.6.2)
EFB platform: (e.g., iOS 9.2)
EFB hardware: (e.g., iPad Mini 2)

Description of your issue:

When trying to install maps using SSH connection, I get following error message:
BEGIN
pi@stratux:~ $ sudo -s
root@stratux:/home/pi# cd /overlay/robase/opt/stratux/mapdata
root@stratux:/overlay/robase/opt/stratux/mapdata# ./download_mapdata.sh -h
Download OpenFlightMaps VFR Charts Europe (~700 MiB)? [y/n]y--2022-04-05 03:47:09--  https://abatzill.de/stratux/openflightmaps.mbtiles
Resolving abatzill.de (abatzill.de)... 2a03:4000:29:5bb::1, <IP_ADDRESS>
Connecting to abatzill.de (abatzill.de)|2a03:4000:29:5bb::1|:443... connected.
ERROR: The certificate of ‘abatzill.de’ is not trusted.
ERROR: The certificate of ‘abatzill.de’ has expired.
The certificate has not yet been activated
mount: /overlay/robase: mount point is busy.
Download US Sectional VFR Charts (~4.9 GiB)? [y/n]y--2022-04-05 03:47:11--  https://abatzill.de/stratux/vfrsec.mbtiles
Resolving abatzill.de (abatzill.de)... 2a03:4000:29:5bb::1, <IP_ADDRESS>
Connecting to abatzill.de (abatzill.de)|2a03:4000:29:5bb::1|:443... connected.
ERROR: The certificate of ‘abatzill.de’ is not trusted.
ERROR: The certificate of ‘abatzill.de’ has expired.
The certificate has not yet been activated
mount: /overlay/robase: mount point is busy.
root@stratux:/overlay/robase/opt/stratux/mapdata# client_loop: send disconnect: Broken pipe
lmac-nid-dsy:~ nid$
END
If possible, enable "Replay Logs", reproduce the problem, and provide a copy of the logs in http://<IP_ADDRESS>/logs/stratux/ and http://<IP_ADDRESS>/logs/stratux.log.

Already fixed upstream. Wait until you have a GPS fix so the clock is set correctly, or run
sudo systemctl start systemd-timesyncd
First to set the system time via internet.

Thanks a lot Adrian.

Hello again,
Download and save went OK but I got a last error, "mount: /overlay/robase: mount point is busy."
Thanks in advance for your assistance.
Best regards.
Nestor

Thanks @b3nn0

Hello Adrian,
I had to reflash my Stratux Europe, and I get the same problem again :
mount: /overlay/robase: mount point is busy.
despite my letting the GPS get a fix or executing sudo systemctl start systemd-timesyncd
prior to installing the maps.
Thanks in advance for your support.
Best regards.
Nestor
@.:/overlay/robase/opt/stratux/mapdata# sudo systemctl start systemd-timesyncd
@.:/overlay/robase/opt/stratux/mapdata# ./download_mapdata.sh
Download OpenFlightMaps VFR Charts Europe (~700 MiB)? [y/n]y--2023-02-28 20:40:57--  https://abatzill.de/stratux/openflightmaps.mbtiles
Resolving abatzill.de (abatzill.de)... 2a03:4000:29:5bb::1, <IP_ADDRESS>
Connecting to abatzill.de (abatzill.de)|2a03:4000:29:5bb::1|:443... connected.
HTTP request sent, awaiting response... 304 Not Modified
File ‘openflightmaps.mbtiles’ not modified on server. Omitting download.
mount: /overlay/robase: mount point is busy.
Download US Sectional VFR Charts (~4.9 GiB)? [y/n]nmount: /overlay/robase: mount point is busy.
@.***:/overlay/robase/opt/stratux/mapdata#

Le 3 déc. 2022 à 16:54, Adrian Batzill @.***> a écrit :
Already fixed upstream. Wait until you have a GPS fix so the clock is set correctly, or run
sudo systemctl start systemd-timesyncd
First to set the system time via internet.
—
Reply to this email directly, view it on GitHub https://github.com/cyoung/stratux/issues/866#issuecomment-1336187367, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZCJZSIEZMDKG6VUUZ7PJSTWLNUKJANCNFSM6AAAAAASSZHB3E.
You are receiving this because you authored the thread.


Add note about when BACKUP, etc. commands return

Add note to BACKUP, RESTORE, IMPORT, IMPORT INTO that the statement returns when the job is finished or if it encounters an error.
Closes #5240.

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/9ecaced073881e4388bfcf7269797958502f7c11/
Edited pages:

v19.2/backup.md
v19.2/import-into.md
v19.2/import.md
v19.2/restore.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/c0a756da4dcb56c4b35bf73a798a1654a3e57d0b/
Edited pages:

v19.1/backup.md
v19.1/import.md
v19.1/restore.md
v19.2/backup.md
v19.2/import-into.md
v19.2/import.md
v19.2/restore.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/5b2e43df1c9c87b586d0021918aa4acba2ce99f7/
Edited pages:

v19.1/backup.md
v19.1/import.md
v19.1/restore.md
v19.2/backup.md
v19.2/import-into.md
v19.2/import.md
v19.2/restore.md


check more optional and simplify optional checking

This is fixed with your latest commits I believe. Closing :)


fix: scenario where branchPosition is not in upstream

When upstream is set to origin/master a bug in the code sets the maybeRemote to origin/mast if the branch isn't found due to how lastIndexOf works. This fixes the issue.

Thanks!


How to know the previous destination

Hello, I am trying to navigate to different screens based on previous windows. Is there a way to find the previous destination using DestinationsNavigator?

Hi!
You can do NavController.previousBackStackEntry.destination() and can get the NavController like the DestinationsNavigator, same thing 🙂
Let me know if this helped!


DB connection with Pokecord

Hello. I am trying to use this library to make a simple pokémon system, and I'm using quick mongo. I tried to connect with database in many ways, but it couldn't add the data to the database. Here is my code:
const { MessageEmbed } = require("discord.js");
const { Spawn } = require("pokecord");
const db = require("../database.js")

module.exports.run = async (client, message, args) => {

    const pokemon = await Spawn().catch(e => {});
    if (!pokemon) return message.channel.send("Opps! Something went wrong :(");
    const filter = m => m.author.id === message.author.id;

    const embed = new MessageEmbed()
        .setAuthor("Guess the pokemon")
        .setColor("#1eb346")
        .setImage(pokemon.imageURL);

    await message.channel.send(embed);

    message.channel.awaitMessages(filter, {
        max: 1,
        error: ["time"],
        time: 15000
    })
    .then(collected => {
        const m = collected.first();
        if (!m.content || m.content.toLowerCase() !== pokemon.name.toLowerCase()) return message.channel.send(`❌ | Incorrect guess! The answer was **${pokemon.name}**.`);
        return message.channel.send(`✅ | Correct guess!`)
    })
    .catch(() => {
        message.channel.send(`❌ | You did not answer in time. The correct answer was **${pokemon.name}**!`);
    })

    .then(collected => {
    const m = collected.first();

    if(pokemon.name === m.content.toLowerCase()) db.add('teste-' + `${pokemon.name}`, 1);
    })

};

not related to this lib


mbedtls: Fix build with TFM and PSA client

See #74201 for the error I am trying to fix, I don't know if this is the right way to fix this but seems like the file was not building because this dependency was missing in the build, this is a hotfix to unblock issue I found from ci running on #74026

should the fix for now be just to add this config to the jwt test prj.conf ? or is it a more general problem?

clearly not a fix based on ci results, closing


[Python 3.5] NameError: name 'long' is not defined

Hi there,
I'm not sure why this happens, but on my python 3.5 install on OS X you use the following code to detect python version:
mpmath/libmp/backend.py, line 36
try:
    xrange
    python3 = False
except NameError:
    python3 = True

but it turns out, it doesn't work as expected on my python 3.5:
try:
    xrange
    python3 = False
except NameError:
    python3 = True
print("is python3", python3)

is python3 False

I'm really no python3 expect, but below you use:
if sys.version >= "3.2":

Could this or some other way of detecting python3 be preferable?
(for the record)
Python 3.5.2 (default, Oct 11 2016, 04:59:56)
[GCC 4.2.1 Compatible Apple LLVM 8.0.0 (clang-800.0.38)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> xrange
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
NameError: name 'xrange' is not defined

@KristianOellegaard I've just seen the same issue.  In my case, pyqtgraph (v 0.9.10) was redefining xrange somewhere before I imported mpmath, so this check for xrange didn't fail.  Upgrading to pyqtgraph v 0.10.0 fixed this for me, but try narrowing down which libraries are imported before mpmath to determine where xrange might be redefined.

We where also having this issue. Seems like something caused xrange to be redefined when running some unittests (probably nose). A small change to the code seems to have resolved this.


Mrd 1788 e2e test migration v2

Thanks @DarrenW-HMPPS !
I meant to make this a draft PR :O
Is it ok to leave it unmerged for now?


Custom quantile for pre-defined metrics, example

Hi,
I'm trying to add custom quantile for pre-defined metrics
the code:
on_worker_boot do
   ActiveRecord::Base.establish_connection if defined?(ActiveRecord)
  if ENV.fetch('PROMETHEUS_ENABLED', 'false') == 'true'
    require 'prometheus_exporter/client'
    require 'prometheus_exporter/metric'
    require 'prometheus_exporter/instrumentation'

    client = PrometheusExporter::Client.new(host: 'localhost', port:9394)
    PrometheusExporter::Client.default = client

    PrometheusExporter::Instrumentation::Process.start(type: 'puma_worker')

    PrometheusExporter::Instrumentation::ActiveRecord.start(
        custom_labels: { type: 'active_record' },
        config_labels: [:database, :host]
    )
  end
end

In the README there are examples for custom metrics - https://github.com/discourse/prometheus_exporter/blob/master/README.md#custom-quantiles-and-buckets
How can I add quantiles for pre-defined metrics, like response time, i.e. override this constant and add 95 percentile:
DEFAULT_QUANTILES = [0.99, 0.9, 0.5, 0.1, 0.01]

Thank you

Changing the default is not cleanly overridable, you would have to redefine the constant. Open to a PR here that makes changing defaults cleaner.


Decrypting with wrong password

Hello, I installed this repo and I was checking it out to use it within a project. However, it turns out after I encrypt something and the time for decryption comes, the decryption never fails. No matter what password you choose it always decrypts. Please try this out and inform me if I might be doing something wrong or if your code needs editing.

Thanks for letting me know.
What version of python are you using?
Also, could you send me the output of pip freeze so i can see which versions of packages you are using?

Python 3.8.0
I just started on a fresh environment, these are my packages:
cffi==1.14.0
cryptography==2.9.2
pycparser==2.20
six==1.14.0
virtualenv==16.7.7

Perfect, thank you. I will take a look at it and respond in the next few days.

I have to apologize. I got distracted while testing. What happened was the file i was decrypting with a wrong password turned back into the original file. However, the data was corrupted, so it wasn't really decrypting with a wrong password. Sorry, I had a lot on my mind today.

No problem. I am glad that it is working for you and really I appreciate you following up to let me know.


Add Array Extension for filtered on the  KeyPath

public extension Array where Element: Equatable {

     // Returns a deduplicated array based on the optional key path.
    public func filtered<E: Equatable>(duplication path: KeyPath<Element, E>) -> [Element] {
        return reduce(into: [Element]()) { (result, e) in
            let contains = result.contains { $0[keyPath: path] == e[keyPath: path] }
            result += contains ? [] : [e]
        }
    }
}

Hey @fanglinwei, why not make a PR to add this feature? 😉
Also, a minor suggestion: Checking contains on an Array has complexity O(n), which means that your function is, at worst case where you have an array that won't need any filtering, O(n²). If you use a Set you can get O(n) complexity, but you lose the ordering. What do you think?
public extension Array where Element: Hashable {
    /// Returns a deduplicated array based on the optional key path.
    ///
    /// - Parameter path: <#path description#>
    /// - Returns: <#return value description#>
    public func filtered<E: Equatable>(keyPath path: KeyPath<Element, E>) -> [Element] {
        return Array(reduce(into: Set<Element>()) { (result, element) in
            if !result.contains { $0[keyPath: path] == element[keyPath: path] } {
                result.insert(element)
            }
        })
    }
}

This looks like a variant withoutDuplicates with a custom equality predicate.  That's an awfully specific use case but if we want to include it I would recommend something a little more flexible like a closure that takes two parameters and returns a boolean.

Hello @chulbert
I added withoutDuplicates with a custom comparator in #704.


WIP - Test for Whitespace token - Don't merge yet

This included test might be a way to "enforce" the usage of the Whitespace token. The trivial case of a single whitespace is lexed by every lexer in pygments.lexers.LEXER. The way it is implemented now, allows either two tokens of type Whitespace for ' ' and '\n' or a single Token for ' \n'of type Whitespace.  Having this test fully pass can be seen as the goal of #1905.
$ pytest tests/test_trivial_whitespace.py
[...]
============= 426 failed, 82 passed in 8.55s ========

There are so many fails right now. For this reason, I think it would be good to not merge this PR any time soon.
Any opinion on this?

Current state;
370 failed, 138 passed in 7.34s

With my latest PR, the current status is:
263 failed, 254 passed

Yes, I have an opinion (and thanks a lot for preparing this!) We'll want that eventually integrated to ensure that all lexers pass; potentially with an explicit whitelist of lexers for now so we can enforce this on new lexers. There's also https://github.com/pygments/pygments/blob/master/scripts/check_repeated_token.py which probably should end up in a test as well and whitelist the existing lexers. The more future additions can be automatically tested, the better.

(Setting "update needed" just to trim down the list of PRs to be reviewed, as this one isn't ready yet.)


[Bug] exclude_from_targets not always working

What happened?
Hi!
Creating this issue mainly to ask a question.
Recently I did this PR, and at the time the part in target_dep_resolver.rb worked fine because there was at least 1 ds at that piont that had a pkg set. However, after some changes in my project (getting rid of all subspecs) the pkgs at that point are all nil, meaning it does not work anymore.
I tried moving this filtering to def spm_pkg(name, options) in podfile.rb, but even though at that point the filtering works, the library is still linked to these targets.
Could you point me to the correct place where filtering of dependencies based on exclude_from_targets should be done? Or is there no single place and I should implement it separately in all post_integrate steps?
CocoaPods environment
No response
Anything else?
No response

Another option would be to keep the check at the place where it is but check on dependency instead of the pkg inside it. But the dependency there has a nil options. If there is a way to get options set by that point it would also solve the problem.

So you mean a package is still linked to the aggregate targets (ex. Pods-Foo) despite being declared in the exclude_from_targets options?
If so, may be you wanna try adding the filter in spm_dependencies_for/spm_targets_for
https://github.com/trinhngocthuyen/cocoapods-spm/blob/9e6ad0daba2a3c08c57149870c38756a845ce1b9/lib/cocoapods-spm/resolver/result.rb#L33-L44
This is the place we get spm dependencies/targets and make up the linker flags and other related build settings

Thanks! That works indeed.
https://github.com/trinhngocthuyen/cocoapods-spm/pull/120


Unit testing for components

This is more or less obvious for most of the components that makes up various displays.
We could also use this for any technical discussions related to testing process.

#79
#82

@el-komandante

This is more of developmental practice rather than any specific implementation.


Resolving #1413 - Fixing Issue where Redirect Manager Not Accessible on Publish

By making /etc/acs-commons/redirect-maps globally readable

@klcodanr this ACL effects both Author and Publish... there's no concerns with this right?
Is there anything special w/ filter.xml thats required?

@klcodanr if you have a sec - can you confirm there's nothing missing here? I have to run out for about an hour but would like to cut a release when i get back - @lrochefort has been patiently waiting for weeks for a new release :)

Coverage remained the same at 40.804% when pulling ece9830c28dd2ff3a1dd08bc18f7dd5a5de4269a on klcodanr:issue/redirect-map-manager-access-1413 into 23617912d35c57be5787b98d535c087c9953bbc0 on Adobe-Consulting-Services:master.

hey @davidjgonzalez sorry I was in an all day workshop.
So yes, the idea is for the ACL to be applied in Publish & Author. For author, the user still must be authenticated and for publish the Redirect Maps need to be accessible to be read by the Dispatcher server.
I just installed the package into a local instance and the ACL installs successfully.

Travis failure due to timeout of some of the builds.


feat: signout helper function for Playwright

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

Instead of calling the API, I decided to delete the cookies instead.


[Query] Application can acquire scopes in access_token even though the permission is not configured in API permissions

I use the Azure AD v2.0 Protocols - OAuth 2.0 Authorization Code Flow template of Postman to acquire access_token. However, during my experiments, I found that when I add scope parameters of some permissions that I don't configure in Azure Portal API permissions at first and also don't need admin consent, as long as I consent for that permission when I login to that application, then the application could acquire those permissions in the access_token as well, which I think is inconsistent with the description below:

If application can acquire some permissions that don't need admin consent without Portal configuration, then I am a bit confused about the meaning of configuring permissions in Azure Portal.

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

ID: 0ac43307-75d4-cb6f-e561-0dd9008f28c6
Version Independent ID: 8fc47e76-5ee1-6aeb-73e2-c64c44e3e4ed
Content: Microsoft identity platform scopes, permissions, and consent
Content Source: articles/active-directory/develop/v2-permissions-and-consent.md
Service: active-directory
Sub-service: develop
GitHub Login: @rwike77
Microsoft Alias: ryanwi

Here are the process of my experiment:

Acquire the authorization code, and you can see from the picture that I add one scope paremeter of https://graph.microsoft.com/Bookings.Manage.All.

Check Azure Portal for API permission of that application, you could see that I don't configure https://graph.microsoft.com/Bookings.Manage.All permission.

Acquire authorization code and then send access_token request:
a. Consent for https://graph.microsoft.com/Bookings.Manage.All permission when login.

b. Fill in authorization code and scopes in access_token request.

Check scopes in access_token, and you can see that https://graph.microsoft.com/Bookings.Manage.All is contained in access_token.

Or is my understanding of API permission wrong?

@yiliuTo Thanks for reaching out. We will investigate and update the thread accordingly.

@yiliuTo  The experience that you had with scopes are expected in this oauth flow, which is also known as dynamic or incremental flow of permissions.
The End user would get the prompt for consent for any new permission which is being asked via scopes. This is to get developer more custom options to query the same API/resource with different permissions to perform different activities.
For an example an admin can use the same resource with different incremental scopes, while a normal end user can just have the default ones.

Hi @vipulsparsh , thanks for you explanation and I can understand why application can acquire scopes without Portal configuration.
But I have another question now. What is the meaning of configuring API permissions of scopes that doesn't need admin consent?
During my experiments, I found that if I added one scope like https://graph.microsoft.com/Bookings.Manage.All which doesn't need admin consent in API permission, then used Postman authorization code flow to acquire an access_token like above, the access_token wouldn't contain that scope unless I select Grant admin consent for .... However, I thought the admin consent button is used to configure those scopes which require admin consent like https://graph.microsoft.com/AccessReview.Read.All.
To a nutshell, I found it useless configuring a scope in API permissions without seleceting Grant admin consent for .... Could you help me to understand the function of it?

@yiliuTo I just tested this and works totally fine at my side.
Here is what I did :

Request URL

Normal user consent page :

Exchanged the code for the access token and when I decrypt it I can see the booking.manage.all permission :

An admin consent is required only when the permissions which are listed for admin grant, any normal permission should not ask you for admin grant and should work fine with the normal user consent.
can you give it a try again and let me know.

Hi @vipulsparsh , oh I didn't describe my question clearly. I can have the same result as you listed, but what makes me confused is that no matter if I configure a scope in API permissions or not (of course here all the permissions we discussed don't need admin consent), the access_token can contain it as long as it is requested and consented by the login user. Thus I want to know what is the aim/value or proper usage case of configuring it in API permissions?
Below are my experiment steps to test, taking https://graph.microsoft.com/Device.Read as example.

Do not configure it in Portal, you can see it is not listed below.

Acquire the authorization code, and you can see from the picture that I add it in the scope.

Acqurie acces_token and check Device.Read is fetched.

@yiliuTo The API permission you define are basically the static permissions and are used mostly in scenarios where an application is trying to get the token. As the application wont be able to ask for new asked permission, hence the predefined scopes mostly help this scenario.
For any  usual user consent flow, you have the capability to put in a scope dynamically and user will be prompted for consent if not already given while authenticating. You may or may not add the permission before hand and can pass the permissions as scope while trying to get the access token as well.

Thanks!


Auto comments for PRs against master

Due to the not infrequent PRs for swearwords.go against master and the like, I was so free and set up a workflow which automatically comments on such pull requests.
A pull request to master without changes to swearwords.go will mention the user to please not do that and instead change the base branch.
A pull request updating swearwords.go (again against master) will tell the user that we will not update legacy automoderator, however if they still wish to request critical changes, that they please change the base branch.
Both messages include a short description on how to change the base branch.
Was only able to test it on a private repository with limited capacities, but it works fine as far as I can tell.

Now using multiline mode. Had to also use a workaround to get the modified files, because hashFiles() works differently than I initially understood.


Should clicking the Forum link on the navbar open a new tab?

Describe your problem and - if possible - how to reproduce it
The link to the forum on the upper right of the page opens the forum on the same tab. People going to the forum from that link might try looking for a link back to the main site from the forum.
Some suggestions:

The forum opens in a new tab
There's a link in the forum that goes back to the primary site

Relevant forum post:
https://forum.freecodecamp.org/t/how-do-you-leave-the-freecodecamp-forum-once-you-enter-it/196337

Possible fix for developers target="_blank" on that forum link

Yes. Agreed.

On Jun 4, 2018 1:35 PM, "mrugesh mohapatra"<EMAIL_ADDRESS>wrote:
Yes. Agreed.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/issues/17434#issuecomment-394436118,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATDZW7Xk8nOE_aniCZt195jit6YwyrTQks5t5W_-gaJpZM4UY5ht
.

Please remind to add rel="noopener noreferrer" when adding a target="_blank"

Please remind to add rel="noopener noreferrer" when adding a target="_blank"

I think that is irrelevant in this case? We are not linking to an external site but our own forum whose code we control? Unless I missed something?

I thought target="_blank" is ok for the current task.

@raisedadead Should we update the target to _blank is someother places since this is already in place the views [freeCodeCamp/server/views/partials/navbar.jade]
Also in the below target seems to updated from json file
[freeCodeCamp/common/app/Nav/Nav.jsx]

@raisedadead Ineed it's not an external site link but it's a good practice to add this. This may prevent security issues for future changes.


feature/TSS-2036-citb-remove-redundant-test-folders

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 91.20%. Comparing base (1ef1886) to head (5d68eda).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #195      +/-   ##
===========================================
+ Coverage    83.56%   91.20%   +7.63%
===========================================
  Files           78       70       -8
  Lines         2258     2069     -189
===========================================
  Hits          1887     1887
+ Misses         371      182     -189

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


Greeks Always 0 With BlackScholes and Integral Price Models

Expected Behavior

All price models should return non-zero greek values.
Actual Behavior

BlachScholes and Integral models return zero-value greeks.
Potential Solution

Possibly an issue with QuantLib library.
Reproducing the Problem

View the logs of this backtest.
System Information

Run in the QC web terminal. Windows 10.
Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues

[x] I have provided detailed steps to reproduce the issue

Hi, what is the timeline for a fix? We are considering QC for a suite of projects, and will need to get greeks when using Black Scholes.
TBH I'm quite surprised this doesn't seem to be a high priority issue.

Hi ikamanu,
There is not currently an ETA for this functionality.
Best,
Derek Melchin

I am also surprised Black Scholes, the most common model used, can't provide the greeks for options in QC.
It's a pretty limiting problem because the greeks don't seem to ever match the brokerages or other platforms in backtest?

The following issue, once resolved, would indirectly resolve this issue for live mode.
https://github.com/QuantConnect/Lean/issues/5289

Any ETA on this one? This bug is a dealbreaker. I do like the QC platform but unfortunately, I'd have to look to alternatives without this.

Any way we can get this prioritized? This bug is a dealbreaker. I do like the QC platform but unfortunately, I'd have to look to alternatives without this.

I honestly just implemented it myself. Their Greeks implementation and pricing is incredibly slow. I suggest calculating it yourself, there's lot of C# and Python Black-Scholes examples out there.

Thanks @automaton82 , I have considered doing this, but I wasnt sure how viable it was. Would you mind sharing any code or resources for a custom implementation? There are many of us that would appreciate it greatly. Could share either here in the thread (ideally for others to benefit) or privately to ikamanu[at]gmail. Thanks in advance.

The one drawback with a custom change is not being able to use the QC live servers.
My preference is that the greeks are pulled directly from the broker feed which then supports trading multiple underlyings without the limitation of warmup and so resolves the current limitations of the greek modelling. I've also suggested that to avoid differences between backtest and live, that the greeks are pre-calculated in the historical data, so that warmup is not required for backtest.
Afaik, only QC supports options algos for backtest and live, and so even with the limitations, there's nothing else out there. But if anyone is aware of alternatives, then please share.

@ikamanu you can just search Black-Scholes C# or Python, there are plenty of examples online
@Shaggleford you can use a custom implementation in live mode, I certainly do. The data warm-up occurs when you start it in live mode, just as it does in back-test. So if you want to start back-testing for January 1, 2020 it will warm-up with the X days before. Not sure what the issue is?
You only need the warm-up data to calculate volatility anyways. Personally I use ROCP and STD in QC to calculate it. There's still no good way to get the risk-free rate, and I don't know how to get the dividend yield yet so I've hard-coded those for the tickers I'm interesting in.
The benefit again is that a custom implementation is much faster than QC's. I agree it would be nice if they pulled it from the broker but I don't think broker's can give historical greeks. IBKR for example uses a 100-step binary tree, but if you try BinomialCoxRossRubinstein in QC I don't think you can control the number of steps, and furthermore it exhibits the same issue as BS (greeks are 0).
Best to implement it yourself.

@automaton82 I didn't think it was possible to deploy to the QC live servers if you're using a custom Lean implementation as the live nodes will be running the QC core Lean? Or is your custom implementation at the user algo level and not at the Lean level?
Agree that IBKR doesn't provide historical greeks, but it's possible to get pre-calculated Greeks in historical data - QC were looking at this possibility at one stage. The alternative is that QC does some post-processing on the current historical data to generate the greeks so that it doesn't need to be calculated on the fly.
I will definitely be rolling my own if there is no progress from QC.

@Shaggleford it's at the user level. I just implemented BS and Binomial in separate classes in my project that would work with any quant platform, not just QC. Then I feed it the data in the OnData method to calculated it for that tick for that option.


start wireguard earlier

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

let's begin with this to quickly improve the issue before starting any rewrite to systemd


css boilerplate

add basic size, position and styles for our pages to size, position and style.css.

I went beyond boiler plate and got the game board done. it is a 4x4 grid, which I made responsively designed.


Do not set cascade delete for resources not part of VM creation

What this PR does / why we need it:
This PR fixes a bug in the DeleteMachine function where cascade delete option was set for all resources attached to the VM. It should be set only for the resources that were created as a part of VM creation.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

Bugfix:- During VM deletion, the cascade delete option is set only for the resources part of VM creation.

Thanks for fix. We should add an Integration Test  backlog to cover this case later.


⚠ Bofrost.de has degraded performance

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

HTTP code: 200
Response time: 1192 ms

Resolved: Bofrost.de performance has improved in b1ff903.


add link for the oai sets to the registry page

closes #198

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4717

Relevant Lines:
7360

💛  - Coveralls


Does ctransformers support ollama models?

Does ctransformers support ollama models?
How do I specify the model in this code below?
llm = CTransformers(model="***where is the model file for a ollama model?",
model_type="llama",
max_new_tokens=512
https://github.com/marella/ctransformers/issues/204

Was able to make it work with ollama. Used the following code sample.
llm = CTransformers(model="/usr/share/ollama/.ollama/models/blobs/sha256:8934d96d3f08982e95922b2b7a2c626a1fe873d7c3b06e8e56d7bc0a1fef9246",
model_type="llama2",
max_new_tokens=512,
temperature=0.1)
the model is present under /usr/share/ollama/.ollama/models/blobs folder in Linux.


Added meta keywords for Better SEO

Thanks, but meta keywords aren't considered by search engines for SEO anymore.


Feat/localization

Localization Improvements: Implemented the PyBabel system to automate the translation process of marked strings, substantially improving our app's multilingual capabilities.
It includes setting up 'en' locale and updating 'sv' translations.
Phrase Feature Addition: Introduced a feature in 'phrase' with default value and replacement variable. Also implemented handling of default language.
Main Module Enhancement: An important unit 'set_language' introduced in the main module.
Test Additions: Passing tests were added.
Following are the steps to manage the Babel system with .po and .mo files:
PyBabel is a part of Babel library, which is a Python package that provides tools for internationalizing and localizing Python applications. To install the Babel package, see below.
poetry add babel
1 - .po File Generation: The Babel library provides a command pybabel extract -F babel.cfg -o messages.pot, which crawls through the source code and creates a .pot file with all marked strings.
2 - Translation Catalog Creation: After the .po file is derived from .pot template, a translation catalog for a language is created via pybabel init -i locale/messages.pot -d locale -l <language-code> where  is the code for the intended language like 'en' for English.
3 - .po File Update: The translation strings get filled in this file. Any updates to the original strings need the .po file to be updated, by running pybabel update -i locale/messages.pot -d locale -l <language-code> --ignore-obsolete -N
4 - .mo File Compilation: The .po file is compiled to a binary .mo file before deployment. Use the command pybabel compile -d locale -l <language-code>
Remember, .po files are human-readable and editable. These are the files to which translators add their translations. The .mo file is machine-readable and is the format used by the system to display translations in the application. Please refer to Python Babel documentation for more detailed instructions.

Failing tests when running Pytest:
1. Switch to branch `feat/localization`.

2. Launched Pytest with args `fai-backend/tests --no-header --no-summary -q`.

Error log:
    def translate(self, key: str) -> str:
        if not self.translation:
>           raise ValueError('Language not set. Call set_language() before calling translate()')
E           ValueError: Language not set. Call set_language() before calling translate()

../fai_backend/phrase.py:27: ValueError

Fixed!


Remove script class checks when getting function signature

Removes script class checks when getting function signature because it's already impossible to call non-static functions in GDScript instances.
Fixes #71994

Can you please rebase and force-push to retrigger the CI again? Just in case some tests need updating after https://github.com/godotengine/godot/pull/72608.

Thanks!

@akien-mga Can you revert this merge? I forgot to close this PR, it was not functioning as intended, it introduced other issues.
Sorry.

I don't think making a distinction between classes and scripts is a good idea (it is just hiding something from the users, which there's no reason to hide and no way to do it in all cases anyway).
I does fix the bug mentioned in the issue though, and doesn't alter current behavior, so for now it is an improvement.

But it introduced weird issues as we discovered when live testing. I think we should revert the merge.

Reverted with 98921d8fbaa83f53f0205ea77f522e37ba30ead2.


Question

Hi,
I have a few question to your project. How many fps can you achieve?
i tried to measure how many fps your capture would give. it only gets 6 fps... any idea? i tested it with windows 10 + NVIDIA GeForce GT 720M
// DXGICaptureSample.cpp : Defines the entry point for the console application.
//

#include "stdafx.h"
#include "DXGIManager.h"
#include <time.h>

DXGIManager g_DXGIManager;

void init() {
    CoInitialize(NULL);
}

int capture() {
    //printf("DXGICaptureSample. Fast windows screen capture\n");
    //printf("Capturing desktop to: capture.bmp\n");
    //printf("Log: logfile.log\n");

    g_DXGIManager.SetCaptureSource(CSDesktop);

    RECT rcDim;
    g_DXGIManager.GetOutputRect(rcDim);

    DWORD dwWidth = rcDim.right - rcDim.left;
    DWORD dwHeight = rcDim.bottom - rcDim.top;

    //printf("dwWidth=%d dwHeight=%d\n", dwWidth, dwHeight);

    DWORD dwBufSize = dwWidth*dwHeight*4;

    BYTE* pBuf = new BYTE[dwBufSize];

    CComPtr<IWICImagingFactory> spWICFactory = NULL;
    HRESULT hr = spWICFactory.CoCreateInstance(CLSID_WICImagingFactory);
    if( FAILED(hr) )
        return hr;

    int i=0;
    do
    {
        hr = g_DXGIManager.GetOutputBits(pBuf, rcDim);
        i++;
    }
    while (hr == DXGI_ERROR_WAIT_TIMEOUT || i < 2);

    if( FAILED(hr) )
    {
        printf("GetOutputBits failed with hr=0x%08x\n", hr);
        return hr;
    }

    //printf("Saving capture to file\n");

    CComPtr<IWICBitmap> spBitmap = NULL;
    hr = spWICFactory->CreateBitmapFromMemory(dwWidth, dwHeight, GUID_WICPixelFormat32bppBGRA, dwWidth*4, dwBufSize, (BYTE*)pBuf, &spBitmap);
    if( FAILED(hr) )
        return hr;

    CComPtr<IWICStream> spStream = NULL;

    hr = spWICFactory->CreateStream(&spStream);
    if (SUCCEEDED(hr)) {
        hr = spStream->InitializeFromFilename(L"capture.bmp", GENERIC_WRITE);
    }

    CComPtr<IWICBitmapEncoder> spEncoder = NULL;
    if (SUCCEEDED(hr)) {
        hr = spWICFactory->CreateEncoder(GUID_ContainerFormatBmp, NULL, &spEncoder);
    }

    if (SUCCEEDED(hr)) {
        hr = spEncoder->Initialize(spStream,WICBitmapEncoderNoCache);
    }

    CComPtr<IWICBitmapFrameEncode> spFrame = NULL;
    if (SUCCEEDED(hr)) {
        hr = spEncoder->CreateNewFrame(&spFrame, NULL);
    }

    if (SUCCEEDED(hr)) {
        hr = spFrame->Initialize(NULL);
    }

    if (SUCCEEDED(hr)) {
        hr = spFrame->SetSize(dwWidth, dwHeight);
    }

    WICPixelFormatGUID format;
    spBitmap->GetPixelFormat(&format);

    if (SUCCEEDED(hr)) {
        hr = spFrame->SetPixelFormat(&format);
    }

    if (SUCCEEDED(hr)) {
        hr = spFrame->WriteSource(spBitmap, NULL);
    }

    if (SUCCEEDED(hr)) {
        hr = spFrame->Commit();
    }

    if (SUCCEEDED(hr)) {
        hr = spEncoder->Commit();
    }

    delete[] pBuf;

    return 0;
}

int _tmain(int argc, _TCHAR* argv[]) {
    init();

    clock_t t1 = clock();
    int i;
    int iterations = 100;
    for (i = 0;i < iterations;i++) {
        capture();
    }
    clock_t t2 = clock();
    printf("%d iterations: %0.0f fps\n", iterations, iterations / ((double)(t2 - t1) / CLOCKS_PER_SEC));

    return 0;
}

Hi,
The main issue I see is that cature() is allocating pBuf every single frame, the same with converting the pBuf to image via gdiplus every frame - it's too much overhead.
I would try to rewrite the code so pBuf is allocated only once and then - looked at the code saving picture - I think it has too much overhead as well...

@pgurenko ok, how to improve it so it can record 2 seconds with 30 fps?

@Arnold1, sorry for the late reply. I would start with simple proof-of-concept like how many fps can we get from just using duplication.
The next code gives me about 30 fps so it look like it is possible to achieve it through output duplication.
// DXGICaptureSample.cpp : Defines the entry point for the console application.
//

#include "stdafx.h"
#include "DXGIManager.h"
#include <time.h>

DXGIManager g_DXGIManager;

int capture(RECT& rcDim, vector<BYTE>& buf, CComPtr<IWICImagingFactory>& spWICFactory) {
  DWORD dwWidth = rcDim.right - rcDim.left;
  DWORD dwHeight = rcDim.bottom - rcDim.top;
  DWORD dwBufSize = buf.size();

  HRESULT hr = g_DXGIManager.GetOutputBits(buf.data(), rcDim);
  if (FAILED(hr))
  {
    printf("GetOutputBits failed with hr=0x%08x\n", hr);
    return hr;
  }
  return 0;
}

int _tmain(int argc, _TCHAR* argv[]) {
  CoInitialize(NULL);

  g_DXGIManager.SetCaptureSource(CSDesktop);

  RECT rcDim;
  g_DXGIManager.GetOutputRect(rcDim);

  DWORD dwWidth = rcDim.right - rcDim.left;
  DWORD dwHeight = rcDim.bottom - rcDim.top;

  printf("dwWidth=%d dwHeight=%d\n", dwWidth, dwHeight);

  DWORD dwBufSize = dwWidth*dwHeight * 4;

  vector<BYTE> buf(dwBufSize);

  CComPtr<IWICImagingFactory> spWICFactory = NULL;
  HRESULT hr = spWICFactory.CoCreateInstance(CLSID_WICImagingFactory);
  if (FAILED(hr))
    return hr;

  clock_t t1 = clock();
  int i;
  int iterations = 100;
  for (i = 0; i < iterations; i++) {
    capture(rcDim, buf, spWICFactory);
  }
  clock_t t2 = clock();
  printf("%d iterations: %0.0f fps\n", iterations, iterations / ((double)(t2 - t1) / CLOCKS_PER_SEC));

  return 0;
}

dwWidth=3840 dwHeight=2160
GetOutputBits failed with hr=0x887a0027
GetOutputBits failed with hr=0x887a0027
GetOutputBits failed with hr=0x887a0027
GetOutputBits failed with hr=0x887a0027
GetOutputBits failed with hr=0x887a0027
GetOutputBits failed with hr=0x887a0027
GetOutputBits failed with hr=0x887a0027
GetOutputBits failed with hr=0x887a0027
100 iterations: 30 fps

I guessed, frame drops are ok - I don't think you really want enforce saving each and every frame.
After that, you will need save the  data in buf somewhere and that is another question. There could be a queue keeping few latest frames in the memory and few threads processing this queue and saving frames to JPEG or compressing to video stream....


fix typo: 'topt' and chore improvements

Again, thanks for this package! I cannot wait to add this anytime soon to my app.
I added my two cents:

change 'topt' to 'totp'
export hook on root at entry file -> better cherry picking for esm-module
export types on entry file export * from "./types": types from packages come in handy sometimes, like import { Application } from "@feathersjs/feathers"
added github action

I'm happy to discuss these points or explain more if needed. Please feel free to refuse changes, if you feel like.

Hi @fratzinger, thank for the PR! I will merge it now.
PS: Embarrassing spelling mistake of TOTP :see_no_evil: But at least I pulled off consistently :-)

😅👍


Make the logs argument to docker attach configurable

This pull request fixes #51 and is step 1 in fixing docker/compose#2448.

Thanks - see my note in https://github.com/d11wtq/dockerpty/issues/51#issuecomment-165778084 about showing a warning.
Not sure why CI is failing - it looks unrelated. Seems to be unable to find the busybox:latest image, even though we pull it during setup.

I'll add a deprecation warning about changing the default. I couldn't see why the tests failed either. I'll force push the deprecation warning in a few minutes.

It seems that the build fails because the tests.sh script can't find the cgroups-mount command.

As far as I can understand, the most likely culprit is that the failing build runs on a different Linux Kernel.
From the last working build:
Operating System Details
Distributor ID:	Ubuntu
Description:	Ubuntu 12.04 LTS
Release:	12.04
Codename:	precise
Linux Version
2.6.32-042stab090.5

From my failing build:
Operating System Details
Distributor ID:	Ubuntu
Description:	Ubuntu 12.04.5 LTS
Release:	12.04
Codename:	precise
Linux Version
3.13.0-63-generic

By downloading the travis-ci log and looking at it properly I discovered that the apt-get install line in .travis.yml failed. :)

{
  "status": "Pulling repository busybox",
  "errorDetail": {"message":"Could not reach any registry endpoint"},
  "error": "Could not reach any registry endpoint"}
}

Probable explanation: https://blog.docker.com/2015/10/docker-hub-deprecation-1-5/

Unfortunately, it seems that busybox:latest has changed now that we're getting the image from docker-hub v2. Thus there is no /etc/issue and some of the feature tests break.

There are test fixes in #48 that we need to get into master

Ah, look at that. @TomasTomacek also found a much nicer fix for the build issues. :+1:

@dnephin What's your preference?

I can cherry pick the neccessary changes from #48 into this PR
I can cherry pick the build changes from #48 into a new PR that only does the build stuff
I can get rid of the build related changes in this PR and then rebase it later when someone else fixes the build

Option 2 would be great

Regarding the mystery test (watch), I suspect this could be an attempt to check whether watch is able to update the existing text instead of adding to the bottom of the screen. I won't bother fixing that now however.

PR #53 fixes the build issues. I'll rebase this change on top of that.

Thanks for fixing the build! One typo, needs a rebase, then LGTM.

Thanks. Rebased and fixed.

LGTM

(would have split my PR if I saw this earlier, glad you solved it)


added speech parsing and MSA processing notebooks

Added notebooks for https://github.com/llmware-ai/llmware/blob/main/examples/Use_Cases/msa_processing.py and https://github.com/llmware-ai/llmware/blob/main/examples/Use_Cases/parsing_great_speeches.py

👍


Add lambda and sqs handlers

review plz?? @MacLikorne

thanks @similark for the PR 🙏

review plz?? @MacLikorne

Sorry I missed the PR.
Could you please also add new features to README (AWS features list + new flags in command line) ?
For the version, since it's brand new features, you can set it to 0.11.0.
Thx for the contribution !!!

Not sure what happened with the repeating conflicts - but what's the status on this? @MacLikorne

@MacLikorne any chance you will help @similark merge it?

@MacLikorne any chance you will help @similark merge it?

Gonna go it this week @cregev

@similark @cregev conflicts solved. Merging the branch and doing a release.
Thx for the contribution.


enable optional zeroconf for airplay and chomecast plugins

docker file amended to install avahi daemon and set environment variable ENABLE_ZEROCONF=NO
this will install needed compoents but ensure ZEROCONF is not enabled by default (folks hate boradcast things).
By changing the env var this can be enabled,
startup script amedend to run daemon and dbus service on prescense of ENABLE_ZEROCONF=TRUE
Note, i am not a coder (literally) i had to look up IF usage to make this work. i did this a)because i hate not trying to contribute and b)to lear about vscode and github - feel free to reject,
Note 2 the avahi service depends on dbus - i think this is already present in the docker image?

Thanks for the contribution. I'll take a look this weekend. Unfortunately it's not as simple as just using service to start avahi-daemon inside the container - you need to use a process manager (which I will look into) to make sure that the processes get handled correctly and zombie processes get reaped.

I've pushed a new branch of this repo that incorporates avahi-daemon and dbus. Closing this PR. Thanks for the idea/motivation!


Update README to indicate that "Install OS X [osx_version_name].app" is expected in a specific directory

Update README to indicate that Install OS X [osx_version_name].app is expected at iso/OS X [osx_version_name]/ relative to the project directory.

Done


bump version to 0.0.4

Bump the package version. I think we should wait for https://github.com/pymc-labs/pymc-marketing/pull/179

Seaborn itself seems to take care of the invalid numpy and matplotlib dependencies?
https://github.com/mwaskom/seaborn/blob/55a328ba4301f429aa454f5824861323caf91cd0/pyproject.toml#L26-L28
Would it be enough to mark seaborn >= 0.12.2 (and remove the specific matplotlib and numpy ones)?


Able to pass build arguments into DockerCompose?

This is NOT a bug report.
I am curious if one can pass build arguments into the DockerCompose. I need to pass docker-compose build arguments in addition to using a .env file.
Thanks for any guidance on this 👍

For future users who want to know...
You can simply call compose._call_command to build with --build-arg's


Replaces an array spread of a NodeList with Array.from because NodeList isn't iterable in Safari 9.

Failure in Safari 11 is due to something unrelated in the 11.1 release.

Heya, what's the status on this. Do you know when this is going to be released?


test uniq validation empty attribute

error unique validate if empty attribute

after save an empty string is converted to null in database, and validator created a query with string ""

That is not the problem, if you insert a string into a integer column, that does not work. You should ensure the column value is int by adding integer validator or filter validator. Merged your tests with slight adjustment. Thanks!


Adding new fields in profileedit.php

Hello,
I want to add some custom fields into the user profile page where someone puts address and bio , I am unable to find way to connect it with backend!
In which connectivity file I will need to make changes so that frontend box will push data to DB?

You have to make changes in both profileedit.php (to add more fields) and in /login/class/profiledata.php in function upsertUserInfo() in order to get data from those fields.


how get host info

I'm new to vsphere. When I use vc.getMORefsInContainerByTypePropertyArray(rootFolder, 'HostSystem', ['summary']) API. I got error. I don't why?
Can you help me?

What's the error?  Did you set rootFolder=vc.serviceContent.rootFolder ?  Is 'HostSystem' the name of a virtual machine?
More information is required.


👌 IMPROVE: center-align figure, output_scroll

Added css for center-aligning figures and handling output_scroll tag in code-cells.
outpur_scroll tag css addition is mostly for backward compatibility , as it has been replaced with scroll-output.

Looks like there were some simple rules missing re: alignment for figures?
Also let's go with scroll-output and drop the scroll_ouput tag. I will make a PR to myst_parser to suggest we update the docs there

@AakashGfude please merge when you're ready. Thanks


Add content title

WHO
As a reader
WHEN and WHERE
when I look at the top of an article
WHAT
I would like to be able to view the title of the article
WHY
so that I get a confirmation that I am on the right page

Proposed solution and tasks

Extract the title from the XML
Display the title like on eLife’s journal article (see screenshot below)

Tasks

[ ] Extract the title from XML and retain in article store
[ ] Query article store and return article data with title
[ ] Display article title to reader

Clarification needed and assumptions

We need to agree what the title is. What value is title from the XML?

article-title

Are we having an article store such as lax for libero for MVP?
We need to determine if for the MVP we can strip out any style and display plain text

eLife & IJM: Italic, superscript and subscript, greek letters
Bold and underlined text don’t need to be supported, colors won't be supported
If possible we should support MathML for Hindawi
Inline figures are required for RSC for non-unicode characters

Can we support Unicode characters?
Yes for RSC
Do we need to support line breaks?

Not for eLife
That would be a nice to have for IJM > Production team suggest that we use <subtitle> instead

There is no character limit. Titles won’t be truncated.

120 characters limit on EJP (editorial policy)
Editorial and production teams would usually manage any edge case

We need more examples of XML

We have 2 articles converted in XML. We could do a whole issue is required

Libero will support multi-lingual journals trans-title

Technical notes
/article/front/article-meta/title-group/article-title appears to be the article title value
User interface / Wireframes

What should happen to titles that contain MathML when MathML isn't supported? (eg HTML's <title>)

What does 'Inline figures are required for RSC for non-unicode characters' mean?

RSC were explaining that character symbols in chemistry (like triple bond) are actually represented as inline images.

Interesting, would be interested in seeing an example. Sounds like something to ignore for core now though, if they want to do that can add it in as a custom extension and maybe make it a Libero extension in the future.

@thewilkybarkid yes we can definitely drop that for the MVP. I will update the ticket description.


Closes #989 - adds option to disable click actions on code lens commands

Description
Closes #989 - adds option to disable click actions on code lens commands

adds option to disable click actions on recent changes and authors code lens commands
updates documentation in the README file to reflect the changes

The option to disable click actions can be applied individually to recent changes and to authors code lenses
Screen captures

Checklist

[x] I have followed the guidelines in the Contributing document
[x] My changes are based off of the develop branch
[x] My changes follow the coding style of this project
[x] My changes build without any errors or warnings
[x] My changes have been formatted and linted
[x] My changes include any required corresponding changes to the documentation
[x] My changes have been rebased and squashed to the minimal number (typically 1) of relevant commits
[x] My changes have a descriptive commit message with a short title, including a Fixes $XXX - or Closes #XXX - prefix to auto-close the issue that your PR addresses

This is awesome work! Thank you so much for your contribution!


 Simple example:
var a: {
	do1():(s:string) => void
}

a = {
	do1() {
		return s => {
			s. //should be string type
		}
	}
}

Strangely enough, this might be per spec:

4.19 Contextually Typed Expressions

[...]
In a contextually typed object literal, each property value expression is contextually typed by

the type of the property with a matching name in the contextual type, if any, or otherwise
for a numerically named property, the numeric index type of the contextual type, if any, or otherwise
the string index type of the contextual type, if any.

Notice that it doesn't mention method declarations, so the following actually will work:
var a: {
    do1():(s:string) => void
}

a = {
    do1: () => {
        return s => {
            s. //should be string type
        }
    }
}

The question is whether or not it should be. While methods are declarations, not expressions, this behavior leads to a pretty inconsistent experience. @ahejlsberg would be better suited to answer this.

@DanielRosenwasser anyway this example works fine:
var a: {
    do1(s:string):void
}

a = {
    do1(s) {
        s. //actually is the 'string' type
    }
}

so context typing works for parameter types and doesn't work for return type.

Yes - actually I just noticed the following from the same section of the spec:

In the body of a function declaration, function expression, arrow function, method declaration, or get accessor declaration that has a return type annotation, return expressions are contextually typed by the type given in the return type annotation.

So this is still per spec, but for a different reason. I'd still like to get @ahejlsberg's take.

@DanielRosenwasser I guess the main point is ' that has a return type annotation' in the part of spec.

Agreed, this is inconsistent. We already have it captured in #1610.

@DanielRosenwasser @ahejlsberg thank you!


Promise Error on Basic Use Case

Hi, I attempted to test hypernova without loading a component from a file like this (it handles JSX because I'm compiling server JS file w/ webpack):
import hypernova from 'hypernova/server'
import { renderReact } from 'hypernova-react'

const TestComponent = () =>
Hello Hypernova

hypernova({
  devMode: true,
  getComponent: function () {
    return renderReact('TestComponent', TestComponent)
  }
})

Then when I use this from Rails:
# view
# <%= render_react_component "Test" %>

# controller
around_action :hypernova_render_support

# initializer
if Rails.env.development?
  require "hypernova/plugins/development_mode_plugin"
  Hypernova.add_plugin!(DevelopmentModePlugin.new)
end

Hypernova.configure do |config|
  config.host = "localhost"
  config.port = 8080
end

I do see hypernova's div rendered, but the content is empty:

On the server, I get this error:
Warning: a promise was created in a handler but was not returned from it, see http://goo.gl/rRqMUw

I'm not sure how to go about debugging this. Any help would be much appreciated.

The string you pass to renderReact must match your hypernova key and the string you pass into render_react_component - you're passing TestComponent instead of Test.

@ljharb sorry - I didn't realize that those had to match. Here's why I got confused:


Unattributed Commits

Some commits that are attributed to @joshuagruenstein were made by me on Fido's pi implementation (Thing two)

sure they were


Feature/generate classes array

Re-added the return typing for generateClasses. Everything passes and the behavior with the new array syntax is as expected in the Nuxt playground project.

I think we are going to sunset this idea as it further complicates the classes and has not seen very much demand


Corrected token involved in newtype declarations in the spec description

This is a bugfix.
In the spec, in the description of newtype declarations, the grammar extract was wrong, as it did not use the correct token ( TYPE_IDENTIFIER was used instead of TYPE ).
The corresponding section in the grammar has not the same bug.

@mbudiu-vmw , @jnfoster
As this fixes a bug in the spec text, I think it should be part of version 1.1 of the spec.


msal-angular fails to login user on mobile browsers

[ ] Bug report
Browser:
Chrome for IOS V- 75.0
Chrome for Android v- 75.0
Safari for Ios v-12
Library version
"@azure/msal-angular": "0.1.2"
Current behavior
login with pop up succeeds but authService.getUser() returns null hence user is not logged in. This happens only in the mobile version of chrome and safari. Desktop version works fine.
I am not sure if it is because of difference in behavior in opening pop ups in these browsers.
the pop up is an entirely new tab.
Expected behavior
after login with pop has succeeded  authService.getUser() should return the logged in user on mobile version of chrome and safari.
Minimal reproduction of the problem with instructions
in mobile version of chrome or safari
const token = await this.authService.loginPopup();
and then call this.authService.getUser()

I have the same problem. It should be included, otherwise mobile devices can not validate.

Maybe, to use loginRedirect instead of loginPopup is the best way for mobile devices

Apologies for not responding to this issue. @danicomas did loginRedirect fix the issue?

Hi Team,
Any updates on this bug?

Hi,
I have the same issue in 2 scenarios:
When using chrome on mac in full screen.
Chrome opens a new tab instead of a popup. Because of this the redirect after login doesn't work.
OS: macOs Mojave
Browser: Chrome V 77
Run browser in fullscreen
When using safari on mac (Not full screen)
Safari opens a new tab, asks to login, closes popup but returns user as null. So login doesn't do anything.
OS: macOs Mojave
Browser: Safari V 13
Solution/Workaround
We now use loginRedirect instead of loginPopup.
This solved both the issues.

Apologies for not responding to this issue. @danicomas did loginRedirect fix the issue?

It is fixed using loginRedirect @jasonnutter

@danicomas Good to know, thanks! We'll investigate this as we work on Angular this quarter.

I have been struggling with this for a long time now.
const token = await this.authService.acquireTokenSilent(this.tenantConfig.b2cScopes);
does the b2cScopeslook something like
['https://AppName.onmicrosoft.com/access-api/user_impersonation']?
I get a user_login_error|User login is required error. even after I log in with the loginRedirect method.
I am on
**    "@azure/msal-angular": "^0.1.4",**

@jasonnutter Thanks ! Much appreciated!!
Ill be looking out for the new beta 👍

@luklew @abannsunny I have been unable to reproduce this issue with the new beta. Can you please try it out and let me know if the issue still occurs?

@jasonnutter I tried using the new library . I seem to be able to log in using iOS, but now I am getting issues authenticating with the backend API. The acquireTokenSilent() seems to be failing, and so the auth token doesn't get attached to the API requests.
Do you know a way to fix this?
Thanks

@luklew Could you please open a new issue with details on what you are observing, thanks!

Closing this, as it looks to have been addressed by the new version of MSAL Angular. You can also use loginRedict/acquireTokenRedirect if there are issues specifically with popup.

still reproduced on v1.0.0

@sergievskiy-a Can you please open a new issue with details, thanks!


Fix a few issues with PEF relocation and loading

Fix a bug where the repeatCount for the repeated block unpacking opcode is decremented by one rather than incremented by one. See Mac OS Runtime Architectures page 8-13 for how this is supposed to work.

Implement the RelocSmRepeat and RelocLgRepeat relocation opcodes. The basic approach is to go back over the preceding blocks the specified number of times and insert new Relocation instances for each iteration. This approach may not scale the best for large numbers of repeats, but for real-world scenarios it seems to work OK.

Implement the RelocLgSetOrBySection relocation opcode in the same manner as the RelocByIndexGroup opcode.

Fix a couple of incorrect masks for the large opcodes.

Some general comments.  If you need me to point out specific locations in the files, I can, but hopefully you can figure it out:
Ghidra source formatter - should be automatically fixing if() { }'s, line length wrapping.
There are some places that have space indents instead of tabs.
Need to avoid adding a static ParseState.  A static mutable state object will be a source of errors if threads get involved.  The best solution would be to avoid introducing ParseState to begin with.  (See lower down in the comment)
isMatch() - should not have side effects
Possibly add a method to base class that will do the work after isMatch() returns true, something like performXYZVerbNameHere(BinaryReader, parseState)
In RelocationFactory.getRelocation():
if ( relocation.isMatch() ) {
  relocation.performXYZVerbName(reader, parseState);
  return relocation;
}

You could then revert change to constructors, no need to save BinaryReader and ParseState in constructors
setPointerIndex() calculations that use consts like 2, or 4 should document what those values are:
Example: setPointerIndex( x - 2/*sizeof(short fieldXyz) */ ... );
The big thing is that I don't like introducing ParseState to track the looping.  Did you explore adding something like a Relocation.readChildren() method that allows the specific subclasses to do the stream seeking and re-reading of the previous chunks and returning it for the caller?
For example, in LoaderRelocationHandler:
	_relocations.add(reloc);
	_relocations.addAll( reloc.readChildren(reader) )

The implementations of Relocation.readChildren(BinaryReader reader) that do something will have a while loop that looks almost the same as the while loop in LoaderRelocationHeader's constructor.
If ParseState were to stick around, the RuntimeException would need to be an IOException because we are reading something and it failed.

Let me know if this approach looks cleaner.  It removes the ParseState class as well as scaling better for large numbers of repeats.

Sorry about the delay. I'm not seeing anything major and only 1 small thing that either I can change or you can change before I pull in your PR.
You changed the return type for LoadRelocationHeader.getRelocations() from List<> to ArrayList<>. This seems unnecessary and looks like it can be changed back to the generic list type, along with the decl of _relocations in the same class, and the List local var in PefLoader.processRelocations().

Please go ahead and make the change, thanks!


Meta configuration variables; where is an overview over possible values

Question to the Documentation
I need to configure the client transport protocol for an arduino client. In meta files I found configuration variables, but I do not find a list of possible values for the variables.
Can someone give advice how to set variables for serial transport?

Hi @Gleis1234, documentation on available parameters can be found here.
Anyway, you don't need to modify this file to use or configure the serial transport.
If you need help, detail your use case or your configuration needs and we will give you advice on how to proceed.

Hi @Acuadros95 thank you for your answer. I got my config to run by adding board_microros_transport = serial to platformio.ini
and init the serial port with:
Serial.begin(11520);
set_microros_serial_transports(Serial);

I was a bit confused by the lot of config parameters and could not find a clear description of their meaning.
Thank you a lot.


Feat/use sbs

[!NOTE]
Only merging after a new version of core is released with changes from canary
Changes fix the reason the CI was flaky sometimes pending_sub_watch_label was not unique per subscription

Changes

Use new blocking methodology from #88 on subscribe, delete, and update
Dependent on sbs property coming from notify server in the responses for the above functions
upgrade vitest because v1 is ridiculously better than v0, with more reliable stack traces
refactors tests to not depend on notify_subscriptions_changed (not all, more to come later)
Uses sbs from spec: https://github.com/WalletConnect/walletconnect-specs/pull/182/files
Using a core canary that implements https://github.com/WalletConnect/walletconnect-monorepo/pull/4123

[!WARNING]
We are not setting the mjv flag as we still want notify_subscriptions_changed for now, to maintain complete compatibility with web3inbox. Will refactor when we have more time

Spec TL;DR
Add sbs flag to subscribe, delete and update functions. This includes latest state of subscription, causing us to not need to depend on notify_subscriptions_changed for latest state. Furthermore, functions only resolve after latest state is synced to local state.
Reminder of architecture:

Looking solid and excellent PR context 🙏


Adjust element layering on mainpage

The animated background used to cover the texts on mainpage, it should be fixed if we merge this PR into main...
Please note that the hero__subtitle may still not appear in the correct layer. Currently working on a fix

yup


Provide a wasm implementation

This could be a pretty slick use case:
Use wasm to render TVG icons with canvas instead of using SVG images. This could decrease the load size of websites

See:
https://github.com/raulgrell/tetris/blob/master/src/platform/web/canvas.zig

Done


add colorbar

Add an example for scatter plot with colorbar

Added another plot for Lorenz attractor

Thank you for your contribution
I will add you as a contributor so you can merger contributions at will


iOS 11 issues

On iOS 11, if you drag a cell near the top or bottom of the screen, there are visual jumps and glitches.
Also, automatic cell sizing is enabled by default in iOS 11. This needs to be disabled manually by the library user (see #6). A note should probably be added in the 'usage' section of the readme.

https://github.com/adamshin/SwiftReorder/pull/13 fixes this.


Quick fix context bug

This fixes a bug where the context is not updated correctly because the initialMessages parameter of useChat uses these messages only for initialization. Subsequent updates are ignored by useChat.
I verified the bugfix with the WaterBnb and HelloWorld demos- the context was changing as expected.

@mme should we close this PR after the last update?

@ataibarkai closing


'allow_override'

Importing get_config results in:
from d2go.model_zoo.model_zoo import get_config
TypeError: __init__() got an unexpected keyword argument 'allow_override'

Promblem can be solved by removing allow_override in https://github.com/facebookresearch/d2go/blob/master/d2go/export/api.py#L208
ModelExportMethodRegistry = Registry("ModelExportMethod", allow_override=True)
However, I do not know if it is safe to remove the allow_override argument.

Thanks for the feedback. It is because the mobile_cv isn't updated yet. @wat3rBro is looking into it.

@petoor should be solved by https://github.com/facebookresearch/mobile-vision/pull/44

Yes, it is working.
Cheers.


Function Parameter being array does not work

Describe the bug
I can not use type: 'array', items: 'string' for example to supply arrays as parameters.
Is that a problem with the API itself? It conforms with JSON schema as far as I know
To Reproduce
{ name: 'channelIds', type: 'array', description: 'An array of channel IDs', items: { "type": "string" }, required: true }
Fetch a function call chat API call
Code snippets
No response
OS
Linux
Node version
v18
Library version
latest

This should be fixed in our upcoming v4; please give it a try and reopen if the problem persists.


Regression: order(nil) no longer works

Regression in 0.8.4
In 0.8.3 order(nil) would silently work. Now it tries to pass that onto ES resulting in a valid exception.
I'm not sure if this is/was expected behaviour but thought I'd raise it just in case. I think it's common for users to do something like...
UsersIndex.query(term: {name: 'foo'}).order(params[:order]) # sanitized params ideally where it could be left blank and result in nil.

[400] {"error":{"root_cause":[{"type":"illegal_argument_exception","reason":"malformed sort format, within the sort array, an object, or an actual string are allowed"}],"type":"search_phase_execution_exception","reason":"all shards failed","phase":"query","grouped":true,"failed_shards":[{"shard":0,"index":"test_app","node":"_apI_g7gQ4WT-vbWdiqyvQ","reason":{"type":"illegal_argument_exception","reason":"malformed sort format, within the sort array, an object, or an actual string are allowed"}}]},"status":400}

I don't remember I did something with DSL between releases, did you check elasticsearch-ruby for changed behavior?

No other gems or ES itself have changed.
I'll try and investigate further.

Would be perfect, thanks!

This is a very crude experiment but I forked both tags (v0.8.4 and v0.8.3) and changed one line in the kaminari spec: https://github.com/toptal/chewy/commit/0aebb6f3a2970f5922cdb2f6f6ab4a046006aeeb
Tests in v0.8.3 - https://travis-ci.org/levent/chewy/jobs/118462121
Tests in v0.8.4 - https://travis-ci.org/levent/chewy/jobs/118461886

Could I ask you to submit a fix please? I frankly have no idea when will I be able to fix this, definitely not earlier the next weekend. Yeah, regression seems to be exists somehow :( Have no idea how though.

I'll try and find some time this week :+1:

Would be great, thanks!

Apologies for the delay!
I finally found the issue. I wonder what you propose?
See https://github.com/toptal/chewy/blob/master/lib/chewy/query/criteria.rb#L117
sort? returns true when the value of sort is [nil]
This is happening because sort? is using sort.present? now instead of sort.any? (This applies to all the STORAGE methods. You may want to compact all the arrays first but this may go against the performance optimisations you made in the commit that introduced the bug.
 Could I ask you to change it back for sorting and maybe add a spec to secure this behavior in the future?

This issue is fixed by #388, we can close it now :]

Thanks!


Capitalize proper nouns like Dependabot, Renovate, and GitHub

Proper nouns should be capitalized properly.
Suggested replacements on *.md files:

dependabot -> Dependabot
renovatebot -> Renovate or Renovate bot
github -> GitHub

Thank you! Would you like to do a PR?

I'll let you guys do it. 😉

In ./docs/checks.md following lines need update
in line number 206

determines whether the most recent (~30) commits have a Github-approved review

in line number 263

logging github context and secrets, or use of potentially untrusted inputs in scripts.

in line number 279

can add their own content to certain github context variables that are considered

in line number 294

specifically dependabot or

in line number 295

renovatebot. Out-of-date

in line number 312

Signup for automatic dependency updates with [dependabot](https://docs.github.com/en/code-security/supply-chain-security/keeping-your-dependencies-updated-automatically/configuration-options-for-dep    endency-updates) or renovatebot and place the config file in the locations that are recommended by these tools. Due to https://github.com/dependa    bot/dependabot-core/issues/2804 Dependabot can be enabled for forks where security updates have ever been turned on so projects maintaining stable forks should evaluate whether this behavior is satisfactory before turning it on.

in line number 313

Unlike dependabot, renovatebot has support to migrate dockerfiles' dependencies from version pinning to hash pinning  via the [pinDigests setting](https://docs.renovatebot.com/configuration-options/    #pindigests) without aditional manual effort.

in line number 475

Github's dependabot

in line number 476

or renovate bot.

in line number 491

The checks currently looks for known Github apps such as

@naveensrinivasan can I take this issue and make the changes and create PR?
Thank You

@naveensrinivasan Please review PR-2061. I resolved the conflicts but current PR has my older changes also which are made with respect to the issue-1534. Looks like something I didn’t do correctly while committing new changes.
Please help.
Thank You

Updated the changes in checks.yaml file and build the doc.

Please review the PR https://github.com/ossf/scorecard/pull/2061
Thank You


Bug/3814/switch evaluation expand collapse remains

Fixes #3814

Signed-off-by: Elisabeth Lang<EMAIL_ADDRESS>

The temp reference expand / toggle did not work properly, so now the expand property is controlled by the component only.


Zeroes reported on Static Results page

Beginning sometime yesterday, I've started getting results pages with mostly zeroes in the analysis tables.  The "total liabilities by calendar year (change)" table has nothing but zeroes except in the two right-most columns.  The difference by expanded income bin table has all zeroes in the "all tax units," "tax units with tax cut," "tax units with tax increase," and "total tax difference" columns.

@siervicul ospc.org was updated yesterday to use a new version of Tax-Calculator. I suspect that is why the results are different. Can you post a link to the outputs page that you are referring to?

Thanks @siervicul. I can confirm that this is an issue on the ospc.org side and not with Tax-Calculator. Something is happening after the data from Tax-Calculator is processed and when the data is rendered into the table. I'm trying to resolve this now.


Let's Encrypt Too Many Certificates - Not Handled

There were too many requests of a given type :: Error creating new cert :: Too many certificates already issued for: madcore.cloud

We need to be able to track domain registration so can block domains that have been used up. I will ad link to letsencrypt limits. I think its 20 subdomains per week currently.


Touch Device, not mobile

Hi !
This is a really nice component. I really enjoy using it.
Unfortunately I'm not able to make it run properly on touch device. It works on mobile but not on desktop computer with a touch enable device.
On one side I test : http://hammerjs.github.io/ which works perfectly on the same touch enabled device.
On the other side I test vue-slider-component and I'm not able to drag the dot.
What could I do to debug this ?
Thanks,

I've been trying to fake the isMobile function with forcing returning true or false. It did not change anything.

Do you want to debug on the mobile device, if so, please tell me the phone model.
If you are debugging in chrome, whether the same situation with the issue #54 .

And, It is not working to modify isMobile after the component has been initialized.

I've modified the lib to return true/false and test. The lib is loaded with hardcoded value to test wether or not it change something.
It is a touch screen I'm trying to make it work on. Not a mobile device.
Thanks

Sorry, I did not find the problem where, but I found that hammer is also using touch and mouse events.
Or there is no more information to tell me

They also use pointer events. I was wondering if it could come from here. Another thing is that I'm not sure if the return truefor isMobile is sufficient to activate touch support.

I found that there is a pointers event in ie 10+ browser.
You can do the next test:
document.body.addEventListener("mousedown", () => console.log('mouse events'), false);
document.body.addEventListener("touchstart", () => console.log('touch events'), false);
document.body.addEventListener("MSPointerDown", () => console.log('pointer events'), false);


Allow Daffodil to build against Scala 2.12

DAFFODIL-1652

Turns out the new Scallop version broke some CLI tests, primarily due to ambiguities with the --validate option. Updated scallop hoping it would fix it, but it didn't. So added a commit which changes how the --validate option works, the 'mode' parameter is now required. This prevents ambiguities where the thing following --validate could be either a validate option or the input file. Now a mode must follow the --validate option. This breaks backwards compatibility, but is pretty minor.

I realized that the LICENSE and eclipse classpath files needed to be updated to the new scala and scallop versions. Just added a commit that fixes those.