line 1538, in create_logical_drives_with_particular_physical_drives\n  File "/tmp/ansible_ilo_storage_payload_o5ww7v9e/ansible_ilo_storage_payload.zip/ansible_collections/hpe/ilo/plugins/module_utils/ilo_oem_utils.py", line 1253, in check_physical_drives\nTypeError: '<' not supported between instances of 'int' and 'str'\n",

Hello Team,
I am getting the error below when I try to use the role  create logical drive with particular physical drives,
Please note that I am using the get_physical_drives in order to collect server drives details and registred into variable, which in a second task will be processed in order to collect the datadrives Location and total capacity based in a specific condition,
line 1538, in create_logical_drives_with_particular_physical_drives\n  File "/tmp/ansible_ilo_storage_payload_o5ww7v9e/ansible_ilo_storage_payload.zip/ansible_collections/hpe/ilo/plugins/module_utils/ilo_oem_utils.py", line 1253, in check_physical_drives\nTypeError: '<' not supported between instances of 'int' and 'str'\n",
So I didn't have any issue with datadrives location variables neither on counting the disks that I will use to create the the RAID , but   the error has been raised on create logical drive with particular physical drives  as CapacityGB result is being rescated as string
when I use the set_fact , I have confirmed that  it's a normal behavior for set_facts since always recast the variable to string
using the debug: msg and after registring the count results it always end up as dict type
in both case forcing the CapacityGB variable to int using the jinja template filter | int didn't solve the issue and raise the same error above as it appears to be Variable is being cast back to String type.
Is there any way to force the type convert from string to int inside the ilo_oem_utils.py python script and avoid this
below the main playbook
`

name: check logical drives on HPE server through iLO
hosts: localhost
gather_facts: no
collections:

community.general
hpe.ilo
vars_files:
vars/main.yml
roles:
get_physical_drives
role: configure_raid
physical_drives: "{{ physical_drives_details }}"
role: create_logical_drives_with_particular_physical_drives
vars:
raid_details:
- LogicalDriveName: "LD1"
Raid: "Raid1"
CapacityGB: "{{ total_capacity_R1 | int }}"
DataDrives: "{{ datadrive_locations_raid1 }}"
`

role configure_raid:
`

name: Group RAID1 physical drives by location
set_fact:
physical_drives_int: "{{  physical_drives.ilo_storage.GetPhysicalDrives.msg.physical_drives['array_controller_0'] | map(attribute='CapacityGB') | map('int') | list  }}"

name: Count RAID1 Disk Total Capacity
vars:
total_capacity_R1: 0
loop: "{{  physical_drives_int }}"
loop_control:
label: "{{ item }}"
when: item | int == 300
set_fact:
total_capacity_R1: "{{ (total_capacity_R1 | int  + item | int) | int }}"
`
Thank you in advance

We are checking on it


convert a image to pdf

Problem Statement
convert a image to PDF File

I would love to work on this mini-project.

If no one else does this, I can try.

sure @Xlgd  i am assigning this issue to you


UnhandledPromiseRejectionWarning Cannot read property 'length' of undefined

Hello, I cloned the code and build the index.js file and put it in the correct folder. When restarting Directus it states that the Mailer extension is loaded. BUT I get this error when I try to use it:
(node:2933868) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'length' of undefined
1|xxx  |     at /directus/extensions/endpoints/Mailer/index.js:1:713
1|xxx  |     at /directus/extensions/endpoints/Mailer/index.js:1:1042
1|xxx  |     at Layer.handle [as handle_request] (/directus/node_modules/express/lib/router/layer.js:95:5)
1|xxx  |     at next (/directus/node_modules/express/lib/router/route.js:144:13)
1|xxx  |     at Route.dispatch (/directus/node_modules/express/lib/router/route.js:114:3)
1|xxx  |     at Layer.handle [as handle_request] (/directus/node_modules/express/lib/router/layer.js:95:5)
1|xxx  |     at /directus/node_modules/express/lib/router/index.js:284:15
1|xxx  |     at Function.process_params (/directus/node_modules/express/lib/router/index.js:346:12)
1|xxx  |     at next (/directus/node_modules/express/lib/router/index.js:280:10)
1|xxx  |     at Function.handle (/directus/node_modules/express/lib/router/index.js:175:3)
1|xxx  | (node:2933868) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 2)
1|xxx  | (node:2933868) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Any ideas?
Thnx!

I am using directus@latest.

Not passing a file array in the email POST body caused this error. Thanks for pointing out this bug!  You can swap in the updated files, and everything should work.
Sorry for the late reply. 😊


Document MapIdentityApi<TUser> endpoints

Fixes #31196

Internal previews

📄 File
🔗 Preview link

aspnetcore/security/authentication/identity-api-authorization.md
How to use Identity to secure a Web API backend for SPAs

BTW @tdykstra ... In case you didn't hear, Jeremy will be sending new logout code shortly.
https://github.com/dotnet/AspNetCore.Docs/issues/31205#issuecomment-1845775832

UPDATE: Replacement code is being discussed at https://github.com/dotnet/blazor-samples/pull/136/files.


ENH: add MAGICs

Add a helper function that registers an IPython magics to run call
the run engine.
This function can/should be called in the beamline configuration
once the 'canonical' runengine has been set up.
This is support work for a coming PR from @danielballan

My alternative suggestion:
import IPython
from IPython.core.inputtransformer import StatelessInputTransformer

@StatelessInputTransformer.wrap
def tr_run_engine(line):
    if line.startswith('>>'):
        line = line[2:].strip()
        return 'RE({})'.format(line)
    return line

def RE(value):
    print('runengine!', value)

ip = IPython.get_ipython()
ip.input_splitter.logical_line_transforms.append(tr_run_engine())
ip.input_transformer_manager.logical_line_transforms.append(tr_run_engine())

Result:
In [4]: >> 'test'
runengine! test

In [5]: foo = 'a'

In [6]: >> foo
runengine! a

This default input transformer is also rather interesting:
https://github.com/ipython/ipython/blob/master/IPython/core/inputtransformer.py#L39
In [1]: def fcn(*args, **kwargs):
   ....:     print('args is', args, 'kwargs is', kwargs)

In [2]: /fcn 1 2 3 a=4 b=5 c=6
args is (1, 2, 3) kwargs is {'b': 5, 'c': 6, 'a': 4}

I am skeptical that this is actually better than convincing people to type RE(...) in the end, but I think it is a worthwhile experiment. I am in favor of merging it and seeing how people like it.

Why not just a single <? There is still no ambiguity:
In [1]: < 1
  File "<ipython-input-1-732f6b4d3ec1>", line 1
    < 1
    ^
SyntaxError: invalid syntax

am skeptical that this is actually better than convincing people to type RE(...) in the end

I totally agree.  @yugangzhang seemed pretty happy to just throw plans at the run engine via the RE() syntax, which feels like big praise to the notion of plans.

but I think it is a worthwhile experiment. I am in favor of merging it and seeing how people like it

I'm pretty concerned that this experiment is going to result in bug reports like:
def foo():
    < ct()
    < ct()
    < ct()

and essentially cause more work in the short term.  That being said, experiments are always a good thing when we are trying to converge on the right user interface here and I think that doing this experiment will ultimately result in a better user interface.
Interestingly, I initially started this comment strongly in the "don't do this. this will result in more work" camp, but through the process of typing out my reasoning, I have convinced myself that it is worth doing because even though I think it will ultimately fail, we will learn important things about what the users actually want and is thus very worth doing.

Thanks for the comments, @ericdill. Now that we've all slept on this for awhile and the related SPEC API change is about ready to go, it's time to make a decision. Where is everyone standing on this?

Signs already point to this being maybe a bad idea, but I think we all agree that it might be worth trying. I'm merging it so we have it handy if/when this comes up.

I'm just glad git blame points to @tacaswell. 👍


Don't request the entire Aggregate Share just to check if it exists

Fixes #581

Add method for getting just the report count of an aggregate share
Use AggregateStore::ReportCount to check if a batch exists instead of loading the entire aggregate share

Don't we need to make the same change for MockAggregator in daphne::testing?

There is no MockAggregator in daphne::testing :thinking:

Don't we need to make the same change for MockAggregator in daphne::testing?

There is no MockAggregator in daphne::testing 🤔

Disregard, I was thinking of InMemoryAggregator, which used to be called MockAggregator. But this PR isn't changing the role traits, so it's not relevant to InMemoryAggregator at al.


⚠ Turnos 24x7 has degraded performance

In c497997, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 998 ms

Resolved: Turnos 24x7 performance has improved in be929e6.


[Kosmos-2] Inference Script

Great work by the team on the Kosmos-2 Project!
Is there an inference script (maybe in the form of a Colab notebook) already created? If yes, please could you share it? I wanted to infer the model just like in the HuggingFace Demo (given an input image, the model outputs the predictions for different tasks).
Thanks!

K-2 will be integrated into the Transformers library very soon, so you could call the module easily.
https://github.com/huggingface/transformers/pull/24709

Okay Thanks @donglixp !


Settings

Add persistent settings for the default wake up time. Possibly even a schedule of default times depending on day and whether shown times should be in military time or 12 hour time.

Added preferences just for visual has no functionality yet.

Preferences has a default wakeup time and whether to display values in military or standard time!


Demonstrate a complex QEC experiment

As noted in f6f29323ebf2544ae576c6866167026adbfa6359, the tqec package is currently able to correctly generate a memory experiment on a logical qubit.
This experiment is a first demonstration that the code is generating valid Stim files that can be used to simulate QEC experiments, but only replicates a result that has been known, generated and simulated plenty of times.
I think the tqec package should now evolves in two complementary ways.
First, the code base need to be cleaned up, some assumptions (#34 for example) should be lifted, some quality of life (see #36) features should be implemented, and some parts of the code should be made more robust (#33 and #35).
Alongside the clean-up, and as part of the process of making the library more robust, we should try to push the tqec library to its limits, and push back these limits to a level where we can generate QEC experiments that were never generated or simulated before.
The goal of this issue is to discuss the follow-up experiment(s) that will be implemented to test the robustness of the code already written, fix the bugs that will likely inevitably show up in the process, and end-up with a scientific result novel and interesting enough to be worth envisioning a publication.

This should be considered closed by #44 and replaced by #50, #51, #52 and #53.


docs: Add documentation for including branch/fork context into configuration

Fix #1187 by documenting configuration and security considerations.

View rendered docs/best-practices.md

Maybe we should document that in https://probot.github.io/api/latest/classes/context.html#config? The documentation is from this comment:
https://github.com/probot/probot/blob/6e45ad7aa06e1d5c3a858c4306909b5ebce39907/src/context.ts#L146-L188

:+1: good idea. Updated

I made the note slightly more concise


webman  console 应用程序，使用异步http-cilent 报错

PHP Fatal error:  Uncaught Error: Call to a member function add() on null in /mnt/c/users/twomiao/desktop/luckysheet-phpserver/vendor/workerman/workerman/Connection/AsyncTcpConnection.php:213 Stack trace: #0 /mnt/c/users/twomiao/desktop/luckysheet-phpserver/vendor/workerman/http-client/src/ConnectionPool.php(216): Workerman\Connection\AsyncTcpConnection->connect() #1 /mnt/c/users/twomiao/desktop/luckysheet-phpserver/vendor/workerman/http-client/src/ConnectionPool.php(76): Workerman\Http\ConnectionPool->create() #2 /mnt/c/users/twomiao/desktop/luckysheet-phpserver/vendor/workerman/http-client/src/Client.php(145): Workerman\Http\ConnectionPool->fetch() #3 /mnt/c/users/twomiao/desktop/luckysheet-phpserver/vendor/workerman/http-client/src/Client.php(95): Workerman\Http\Client->process() #4 /mnt/c/users/twomiao/desktop/luckysheet-phpserver/app/command/TestCase.php(44): Workerman\Http\Client->get() #5 /mnt/c/users/twomiao/desktop/luckysheet-phpserver/vendor/symfony/console/Command/Command.php(298): app\command\TestCase->execute() #6 in /mnt/c/users/twomiao/desktop/luckysheet-phpserver/vendor/workerman/workerman/Connection/AsyncTcpConnection.php on line 213
add() on null，这样提示似乎不太友好
判断如果是null，说明不在workerman 运行环境，throw new Exception("不在workerman 运行环境");

感谢反馈，如果可以的话请发pr吧，提示用英文提示


grid library conflict with jquery.ui

When using bootstrap4 & fontawesome for grid libraries along with jquery ui, the jquery ui file ( jquery-ui.css) prevents the fonts from showing in column type:'icon' in the grid.
the default icon '>' shows instead of the fontawesome icon.

This should be fixed in the new version 1.7.2 that is released today. Could you try the new version.

v1.7.2 still does not work with the icon column type.
below still shows the standard arrow up icon instead of the fa-pencil.
example:
{ title: '', field: 'Edit', width: 38, type: 'icon', icon: 'fa fa-pencil ', tooltip: 'Edit', events: { 'click': CompanyEquipment_Edit } },

The issue is probably somewhere else. It works in this example http://gijgo.com/Temp/f5e398fe-ddb6-4da6-a3bd-5dc9e749895c.html
You can always use http://gijgo.com/grid/configuration/column.renderer in order to add the column manually if type icon is problematic.

this issue is with jquery.ui
the example you have listed above does not have jquery ui in it.
this is the file that is causing the issue: jquery-ui.min.css.
I also understand I can use another technique, but wanted to report this as jquery.ui is a common plugin.

I included jqueryui in the example below, but everything looks good.
http://gijgo.com/Temp/90543f86-7dcf-4a88-9758-2081c422904d.html

I am using bootstrap 4. it still has a problem.
here are the script files I have tested with:
<script src="https://ajax.googleapis.com/ajax/libs/jquery/3.2.1/jquery.min.js"></script>
<link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/gijgo/1.7.2/combined/css/gijgo.min.css" />
<script src="https://cdnjs.cloudflare.com/ajax/libs/gijgo/1.7.2/combined/js/gijgo.min.js"></script>

<link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/4.0.0-alpha.6/css/bootstrap.min.css" integrity="sha384-rwoIResjU2yc3z8GV/NPeZWAv56rSmLldC3R/AZzGRnGxQQKnKkoFVhFQhNUwEyJ" crossorigin="anonymous">

<link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/font-awesome/4.7.0/css/font-awesome.min.css" />

<link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/jqueryui/1.12.1/jquery-ui.min.css" />
<script src="https://cdnjs.cloudflare.com/ajax/libs/jqueryui/1.12.1/jquery-ui.min.js"></script>

I saw the issue. We fixed that in the new development version.

Please?  help  grid library conflict with jquery.ui  I use Gijgo JavaScript Library v1.9.11 and jQuery UI - v1.12.1 (url)
after click Edit type DATE empty


Remove remaining mentions of #638 in the code

Closes #638
The trivia item from #638 is now separately tracked in #1020.

Thanks a lot for the reviews @OmarTawfik and for bearing with me over all of these issues/PRs along the way :sweat_smile: :pray:


FileMode.Open issue after update library

After upgrade the Azure Configuration Manager, you need to remove FileMode in HomeContoller.csc
//await blob.UploadFromFileAsync(files[i].FileName, FileMode.Open);
await blob.UploadFromFileAsync(files[i].FileName);

Created a PR with the fix.


IGNITE-14095 : Try to fasten cluster start in the ducktests with decreasing 'spi.reconnectDelay'

Try to speed up cluster start ath the ducktests with decreasing TcpDiscoverySpi.reconnectDelay

Won't work. Same results. No start speed up.

Won't work. Same results. No start speed up.


Directives with isolated scope  + one-way data-binding

Hi all,
I wanted to discuss about a particular behavior in directives with isolated scope.
With One-way data binding arguments, objects are not passed like in two-way data binding.
Examples :
http://plnkr.co/edit/bTpYV7dRMrHO0fkSGn1i?p=preview
There is few documentation about that. Is it a normal behavior ? If yes, is it possible to have more information about that?
Thanks

I have a related question. Recent Angular versions introduce the '=*' binding, which set the watchers using $watchCollection instead of $watch. Now the thing is... When am I supposed to use that binding instead of plain old '='? I mean, if you are changing a property/adding an item/whatever there is no need to update the reference the other scope has, unless I'm missing something.


About TSEM module

The code implementation of your TSEM module is not the same as in the paper, can you disclose the same code part as the paper, thank you~

As we present in the ReadMe.md, we later found that a multi-scale architecture could perform on par with what we report in the paper for TSEM, and thus implement it as such. If you want the original implementation, we will try to disclose it.


feature request: add options to rename character sheet tab and recipe compendium

your module works really well for things other than crafting, like skill trees and spell lists (here are some examples of how i'm using it)

it would be super neat if there were settings to change the name of the character sheet tab and recipe compendium to more generic things, like "Progression" or "Unlockables"
right now i'm patching game.translations to achieve this, but that's pretty hacky and official support via a setting would be preferable
thanks for all your work on this module!

Oh well nice. You are right, you can do so many things with it now that it shouldn't even be calle beavers-crafting but it is too late for that now.
I will make the tab editable and i should probably make the various usecases more prominent somehow. May I use your screenshot for that in my documentation ?

absolutely!

3.4.1 released
It will not yet cover Beaver's Recipe Compendium. For now it keeps the name Recipe. It is hard when everything is generic that anyone can follow a documentation while i can't name what i am talking about. At somepoint Recipe may be replaced with blueprint, draft or concept or i do not yet know. I have no good idea for a clear name.


Transformed circles sometimes break

Circle transform seems to break, probably not exactly 2pi anymore. Probably necessary to set flags on Arc/EllipticalArc of whether they are exactly containing all points.

@pixelzoom, is this needed for the redeploy, or can I un-tag it? Does not affect any deployed sims.

@jonathanolson If you're confident that no sims are using this feature, then feel free to remove the "Summer 2015 redeploy" label.  I have no idea when the problem was introduced, or whether there are sims that might begin to hit this problem when redeployed.  Out of curiosity, what are you searching for to determine that no sims are affected?

Just checked for all usages of ".transformed(", and rotations were only being applied to rectangles at the moment. Removing label for now.


cilium-cli: 0.15.8 -> 0.15.10

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/cilium/cilium-cli/releases.
meta.description for cilium-cli is: CLI to install, manage & troubleshoot Kubernetes clusters running Cilium
meta.homepage for cilium-cli is: https://www.cilium.io/
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.15.10 with grep in /nix/store/wmwq88635lj539km1l91aywlmrljkw2x-cilium-cli-0.15.10
found 0.15.10 in filename of file in /nix/store/wmwq88635lj539km1l91aywlmrljkw2x-cilium-cli-0.15.10

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
cilium-cli

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/wmwq88635lj539km1l91aywlmrljkw2x-cilium-cli-0.15.10 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cilium-cli https://github.com/r-ryantm/nixpkgs/archive/5db1749a5b2ed01c9a67272c44ed7126bc615c3c.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/5db1749a5b2ed01c9a67272c44ed7126bc615c3c#cilium-cli

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/wmwq88635lj539km1l91aywlmrljkw2x-cilium-cli-0.15.10
ls -la /nix/store/wmwq88635lj539km1l91aywlmrljkw2x-cilium-cli-0.15.10/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

cilium-cli

Maintainer pings
cc @humancalico @bryanasdev000 @qjoly for testing.

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

1 package built:

cilium-cli


Fix random_id.telem is empty tuple error when disable_telemetry = true

Overview/Summary
When setting disable_telemetry = true, we observe the following error:
╷
│ Error: Invalid index
│
│   on ..\..\..\locals.telemetry.connectivity.tf line 42, in locals:
│   42:       random_id.telem[0].hex
│     ├────────────────
│     │ random_id.telem is empty tuple
│
│ The given key does not identify an element in this collection value: the collection has no elements.
╵

This PR implements a fix to catch this error, allowing disable_telemetry = true to continue controlling the random_id.telem resource creation as desired.
This PR fixes/adds/changes/removes

Fixes the error above
Replaces instances of var. with local. for consistency

Breaking Changes
n/a
Testing Evidence
Snippet of errors before the fix:

Snippet of plan when disable_telemetry = false:

Snippet of plan when disable_telemetry = true:

As part of this Pull Request I have

[x] Checked for duplicate Pull Requests
[x] Associated it with relevant issues, for tracking and closure.
[x] Ensured my code/branch is up-to-date with the latest changes in the main branch
[x] Performed testing and provided evidence.
[x] Updated relevant and associated documentation.
[ ] Updated the "What's New?" wiki page (located in the Enterprise-Scale repo in the directory: /docs/wiki/whats-new.md)

/azp run unit


Interpreting mem_wstrb

Hi,
I'm trying to understand the PicoRV32 native memory interface. How should mem_wstrb be interpreted in the != 0 case?

The 4 bits of mem_wstrb are write enables for the four bytes in the addressed word. Only the 8 values 0000, 1111, 1100, 0011, 1000, 0100, 0010, and 0001 are possible, i.e. no write, write 32 bits, write upper 16 bits, write lower 16, or write a single byte respectively.
I've now also added this paragraph to REAMDE.md in commit 3ccbf6877e124989d415be9a3c8eaa38b02bdb7c.

Thanks!


Missing details for the verifiability of the ceremony

Description
When debugging a Trusted Setup Phase 2 Ceremony, in order to verify the ceremony itself, you will need several informations: hashes of each file, circuit compiler version (due to breaking changes issue), publicly available files.
Tasks:

[x] Explore other ceremony tools for grasping knowledge about ceremony verifiability.
[x] Define what's actually missing to improve ceremonies verifiability.
[x] Version and Commit Hash of the Circom compiler (to be asked during setup).
[x] Link, Commit Hash and template (file) hash to .circom source files (to be asked during setup).
[x] Store finalization Beacon (raw, hash, external link) and number of exponential iterations (to be stored during finalization command execution).
[ ] Store the final verification log after the complete finalization. Maybe we can group finalize command output and store it in a specific folder more than in the root folder in storage (to be stored during finalization command execution).
[ ] Blake2b hash of each file (content) used as input and produced as output.
[ ] Every file stored in our storage and gathered from external ones, must be publicly downloadable for verification purposes (from .ptau to .zkey).

Related Issues

[ ] #166

Already done https://github.com/privacy-scaling-explorations/groth-p0tion/issues/44


giant commit.  cutout download fixes, template formatting errors, and…

… host galaxy database validation
Fixes #  .
Description of the Change
Checklist
Please check all that apply to your proposed changes

[ ] HTML code change
[ ] Added package dependency
[ ] Added data
[ ] Changed django model
[ ] Documentation change
[ ] Added or changed TaskRunner

Additional context

After merging this branch with my other changes on the dev branch of my fork, I built hub.ncsa.illinois.edu/blast/blast:b62d36b and applied it to our production deployment. I'm not sure what the easiest thing to do it for this PR; perhaps you should merge my dev branch into this djones-cutout-fixes branch before we accept the PR.

You may want to change this to a draft to spare the CI/CD cycles as you push commits.


Building OpenROAD without OpenSTA sources

Greetings,
I've just started the OpenROAD porting to OpenBSD; in the official port tree we already have OpenSTA as standalone tool; I'm wondering if it's possible to build OpenROAD without compiling the OpenSTA sources too.
If that's is supported (or even not fully supported, but the effort is low), please give me some hints on how to proceed.
Thanks and Regards
--
Alessandro

It isn't supported and I don't think it would make sense as many parts of the system depend on opensta.


the navbar can be changed to look

Is your feature request related to a problem? Please describe.
the current navbar dosent look good
Describe the solution you'd like.
No response
Describe alternatives you've considered.
No response
Additional context.
No response
Show us the magic with screenshots
No response
Checklist

[X] I have checked the Existing Issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)

I have seen many navbar issues, and i have mentioned it before for contributors to not create issues regarding navbar and there should be no more navbar issues, so i have closed this issue, if you have any queries, you can join discord server for further updates and communication.
we are not looking for navbar updates for now, as it was done and changes many times before, please understand.
and already it is in updation, so please understand.
for now the navbar is okay, we don't require any updates.
please understand.


Add ability to enter subnets for ELBs and RDS DB

These changes add additional parameters to the necessary templates to allow specifying different subnets for the ELBs and/or the database.  If the parameters are left empty the original behavior of using the ServerSubnets for all resources is maintained.
These parameters must be added as type CommaDelimitedList because it is not possible to leave AWS-specific parameters like List<AWS::EC2::Subnet::Id> blank.
This change also removes the UsePublicIp condition and the associated logic in the Outputs section from automate.yaml.  This logic seems unnecessary since even when using the internet-facing ELB scheme, the Automate instance itself will never have a public IP address (and the logic actually fails when internet-facing is specified since there is no public IP address on the EC2 instance to retrieve).  The AssociatePublicIpAddress parameter in main.yaml is not passed to automate.yaml and the Automate EC2 resource does not include any configuration for using a public IP.

This is filed to resolve https://github.com/chef-customers/aws_native_chef_server/issues/61


Upgrade Symfony to 5.0

#hackdayParis

Having a weird issue with the event dispatcher, both on v4.4 and v5.0, when running the tests:
Lexik\Bundle\JWTAuthenticationBundle\Tests\DependencyInjection\LexikJWTAuthenticationExtensionTest::testEncoderConfiguration
Symfony\Component\DependencyInjection\Exception\ServiceNotFoundException: The service "lexik_jwt_authentication.security.authentication.provider" has a dependency on a non-existent service "event_dispatcher".

Tried to add symfony/event-dispatcher in the dev dependencies, but didn’t fix it.
Plus, I have two failing tests on my laptop, which seem to pass on the CI, and I don’t get why.
Failed asserting that exception message 'Failed to load public key:
 0909006C:PEM routines:get_name:no start line
 0909006C:PEM routines:get_name:no start line
 0909006C:PEM routines:get_name:no start line
 0909006C:PEM routines:get_name:no start line
 0909006C:PEM routines:get_name:no start line
 0909006C:PEM routines:get_name:no start line
 0909006C:PEM routines:get_name:no start line
 0909006C:PEM routines:get_name:no start line
 0909006C:PEM routines:get_name:no start line
 0909006C:PEM routines:get_name:no start line
 0909006C:PEM routines:get_name:no start line
 0909006C:PEM routines:get_name:no start line
 0909006C:PEM routines:get_name:no start line
 0909006C:PEM routines:get_name:no start line
 0909006C:PEM routines:get_name:no start line' contains '0906D06C:PEM routines:PEM_read_bio:no start line'.

2) Lexik\Bundle\JWTAuthenticationBundle\Tests\Services\KeyLoader\OpenSSLKeyLoaderTest::testLoadInvalidPrivateKey
Failed asserting that exception message 'Failed to load private key:
 0909006C:PEM routines:get_name:no start line' contains '0906D06C:PEM routines:PEM_read_bio:no start line'.

Is it possible that’s related to my version of OpenSSL (OpenSSL 1.1.1d  10 Sep 2019)?
@chalasr WDYT?

How did you install it for Symfony 5?
Problem 1
- lexik/jwt-authentication-bundle v2.6.4 requires symfony/framework-bundle ^3.4|^4.0 -> no matching package found.

Thank you @Deuchnord.

Thanks @chalasr and @Deuchnord for your work on this 👍


Improve public API surface

Improves the public API surface of SymbolKit to better allow clients
to create SymbolGraphs using SymbolKit directly.
Also fixes a bug where the Location symbol mixin was not encoded as part
of Symbol.
Resolves rdar://88504344.

@swift-ci please test

@swift-ci please test


feat: create dialog component

Current dependencies on/for this PR:

next

PR #283

PR #284

PR #285

PR #286

PR #290

PR #288

PR #287

PR #289


Accept Chef License To Allow Continous Integration Build To Succeed

Signed-off-by: John Curcio<EMAIL_ADDRESS>Description
Accept Chef License To Allow Continous Integration Build To Succeed
Issues Resolved
Accept Chef License To Allow Continous Integration Build To Succeed
Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[] New functionality has been documented in the README if applicable
[X] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

@MrGiga Any reason for closing this?

It failed the integration test.


tooltip js is not included in the download build

If you download a fresh copy of foundation (release builds) the tooltip javascript is nowhere to be found

Good catch, frustrating that we missed that (for a while, too). This has been fixed in the download.


Fixes for new TLS versions and testability

[Fixes #110590256, #111178990]

Note that this fixes some of the existing test failures in this gem, but not all.

LGTM


Move backtracking into OneOf

This PR significantly changes how backtracking is done in the library, hopefully for the better!

All backtracking work has been removed from individual operators (with the exception of Peek and Not)
Backtracking has been inserted in between each step of OneOf

Up till now it's been the responsibility of the parser to "be a good citizen" and clean up after itself on failure. This means if a parser failed to do so, you could get into a bad state or make a bad assumption.
By centralizing this work in OneOf, most parsers are freed of this responsibility and no longer have to worry about undoing changes to input. That is now simply be the responsibility of OneOf instead.

If I understand correctly, this is a huge change of behavior for end-users of the library. Until now, the input was freely available to reuse somewhere else if parsing did fail, but it can now be partially consumed. Issues can be caught with appropriate tests, but many may trip on this otherwise. Aside a prominent documentation/communication about the change, maybe there's a safer way to handle the transition? For example by adding a .parse() overload or new variant that won't consume the input?

@tgrapperon Thanks for the feedback!

If I understand correctly, this is a huge change of behavior for end-users of the library.

We do realize the behavior change is significant, but believe the impact to be small and we will call it out in the release notes.
Of all the parsers we've written we only found one to have a problem with this change, and that's the XCTest log parser in our benchmarks. The problem was that a parser was written in a very strange way that assumed a compactMap would undo its work. The fix itself was an improvement in readability:
https://github.com/pointfreeco/swift-parsing/pull/108/files#diff-27f107d6fe057661c029f42de5d29456e46036fff3acee94ee831d337ec1ec31

Until now, the input was freely available to reuse somewhere else if parsing did fail, but it can now be partially consumed.

While this was true of many of the parsers in the library, there has never been any guarantee that a parser defined outside of the library would undo its work, nor have we documented that parsers undo their work (all documentation around backtracking is additive in this PR). In the end, expecting a parser to undo upon failure isn't something we could ever truly trust it to do. Moving this logic to only OneOf naturally falls out of that.

Aside a prominent documentation/communication about the change, maybe there's a safer way to handle the transition? For example by adding a .parse() overload or new variant that won't consume the input?

We could keep the other backtracking behavior in for now with the plan to remove it in the future, but that feels like we're just kicking the can down the road and will have the same problem whenever we do change the behavior. We'd like to rip the band-aid off early.
Would you like to run any parsers you've written on main to see if anything unexpected happens?

@stephencelis I agree. I was just worried by the transition, not the behavior. Fortunately, the library often works on derived types like Substring's or their views, and it is very probable that the user created the input's value specifically for the parser.
Maybe adding a - Note: or - Warning: in the final parser() documentation would help the information to stand out.
I have a case where I'm trying to parse a String with a parser, and if fails, I send it in another function that is using ML and a more fuzzy approach. But I just checked and the "Substring protection" did kick-in here, as the ML function accepts a String argument (and the parser even works with utf8 views).
I wrote very few parsers with an explicit parse function myself, and I would always backtrack completely if some internal parser did failed in this case, so I guess there won't be behavioral issues. My concern was really about top-level uses, but it turns out I was probably over-worried!
This is a good change overall, and I can't wait for what's coming in the next weeks/months!

I did test it on our project and it did perform as expected 👍🏽

In general I think this is find. One additional "good citizen" might be Many, particularly if it fails to match the terminator, but I think it is already built that way.

Might be worth adding relevant documentation about Many also backtracking? Particularly if it affects performance.


PWA-207 Added usage of Ellipsis component to Headline

Pull Request Test Coverage Report for Build 513

0 of 2 (0.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.007%) to 38.348%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

components/Headline/index.jsx
0
2
0.0%

Totals

Change from base Build 511:
-0.007%

Covered Lines:
1730

Relevant Lines:
4386

💛  - Coveralls


Mod operator

Could you add a function that takes in two macros x and y and it returns the remainder of x divided by y.

I have been contemplating a way to do it and I have yet to come up with anything, although I will try to come up with something

I will also take a look into this, if I get the time

added with https://github.com/gibbsly/gm/pull/13


Breaks the domain when adding entries in bulk

When you try to add multiple A records, on one FQDN, of one transaction, it starts randomly deleting records from the domain database, including A, NS, SNAME, etc.
If you add one A record per transaction, then everything is applied normally. But adding one record at a time, when you need to add about 100 records, is extremely inconvenient.

Can you show the steps with examples to reproduce this bug? As I understand, you cannot add multiple A records at the same time. However, It looks good on my test.

This problem is observed when adding new A records, provided that the record to which we are adding is already in the database.
For instance:
I have these A records:
host.example.com IN A <IP_ADDRESS>
host.example.com IN A <IP_ADDRESS>
host.example.com IN A <IP_ADDRESS>
I want to add a new entry:
host.example.com IN A <IP_ADDRESS>
After clicking apply, all entries
host.example.com IN A <IP_ADDRESS>
host.example.com IN A <IP_ADDRESS>
host.example.com IN A <IP_ADDRESS>
Completely disappear.
As I understand it, this happens because the PATCH method is used to access the pdns api when I need to add something to an existing domain.
Is it possible to implement the method of adding with a preliminary check for the presence of such an entry in the database, and if such an entry exists, then parse them and automatically add them to the pdns request so as not to lose the existing entries?
And if you use the update function, or replace the record, then so that it does everything as before, i.e. completely replaced the old record in the database with a new one?

host.example.com IN A <IP_ADDRESS>
host.example.com IN A <IP_ADDRESS>
host.example.com IN A <IP_ADDRESS>

Just want to clarify, your 3rd record IP should be <IP_ADDRESS> right?
I tried with your sample, and it does work for me. No record got deleted.
Can you show the "recent history detail" after you apply? It would show more information.
Regarding the PATCH method, it is required by the PDNS API. You can see the docs here

Just want to clarify, your 3rd record IP should be <IP_ADDRESS> right?

Yes. Absolutely right

Can you show the "recent history detail" after you apply? It would show more information.

Unfortunately no. This information is already lost.

For instance:
I have these A records:
host.example.com IN A <IP_ADDRESS>
host.example.com IN A <IP_ADDRESS>
host.example.com IN A <IP_ADDRESS>
I want to add a new entry:
host.example.com IN A <IP_ADDRESS>
After clicking apply, all entries
host.example.com IN A <IP_ADDRESS>
host.example.com IN A <IP_ADDRESS>
host.example.com IN A <IP_ADDRESS>
Completely disappear.
There is one feature in this example that I forgot to mention at the beginning. All NS records are also deleted from the domain in which we added records, only one NS record remains, instead of 8

Hello,
I'm in the same issue, as I can know without further investigation (I'm on it), when you add multiple new records, randomly, add to the rrsets deletions, on my case, the MX records and many more (A records, NS, including subdomains)
The issue is harder when you try to add a duplicate record (for example, add an A record with same name and IP that one existing yet), because the powerdns.admin show an error, and no log on history nothing, but, the first operation that the app does is delete the record, and then tries to create with the same IP twice, and the pdns API trows an error, and the result is that the first record is deleted, and no info on the history about it.
Best regards!

I have some news about my last comment, I don't know that is the same issue, I think yes, is not only when you add bulk records, the issue is reproducible when you add one or more records, then apply changes and then add more records and apply again.
More schematic:
Go to "Manage domain"
Add record
apply changes
add record
apply changes
This second "apply changes" do more operations that was made by the user on the interface.
@ngoduykhanh , do you want to add a new issue with this specific case?
Thank you

More schematic:
Go to "Manage domain"
Add record
apply changes
add record
apply changes

@pprietosanchez I don't get it. What is the problem with this procedure? I only see the issue you mentioned in #752

@ngoduykhanh is another issue than #752 , in this case the issue occurs when you follow the steps described on https://github.com/ngoduykhanh/PowerDNS-Admin/issues/744#issuecomment-639308554
The relationship involved two issues is when both occurs at the same time, in an schematic example:
1 Go to "Manage domain"
2 Add record
3 apply changes
4 add duplicate record (reproducing #752 )
5 apply changes
If these steps occcurs, the result of this is the lost of arbitrary records on domain, including NS, MX, etc....
I hope I've been clear enough

Hello,
This request is stale for too long. Regarding our policy to cleanup this project, we are closing it. Feel free to open a new issue later if you still encounter this problem.
Regards
PowerDNS Admin Team
See #1153 for the policy


bug(cnpg): type "vector" does not exist

After this pull request https://github.com/tensorchord/pgvecto.rs/pull/273 in v0.2.0, we need to set search_path when packing the docker image.

Fix in https://github.com/tensorchord/cloudnative-pgvecto.rs/pull/26


Add docs that provide general directions for using a web framework in webviews

Extension/feature description
It would be great if the general ideas/guidance around incorporating a web framework (i.e. React, Vue, etc.) into webview-based extensions were captured in stand-alone documentation for developers to reference.
Use case
It will be impossible for us to keep up with and maintain an ever-growing catalog of sample extensions that cater to new web frameworks. It would instead be better if the directions on how to use any web framework in webview extensions were provided.
Misc
I think the currently maintained framework samples (i.e. React) and future planned framework samples (i.e. Svelte, Vue, Angular, and maybe SolidJS) can be still included as a tangible example/reference to aid these docs. But after those are completed I imagine we would no longer actively create/maintain future framework-based samples––only vanilla JS/TS samples.
The one exception that might be considered is if someone from the community contributes a PR with a new web framework sample or a sample demonstrating a helpful scenario/VS Code API in a web framework.
Reference
I've already written about this topic a bit on Twitter, so hopefully, this can act as a starting point for documentation.

cc @daviddossett for thoughts

I think this could make sense if it's possible to be broad enough to capture many/most frameworks without needing to endlessly update it.
Perhaps just hitting the on the core concepts and gotchas is a good place to start.

@hawkticehurst  @daviddossett thanks for the examples you are adding. I've been trying to wrap my head around how to listen to a postMessage sent from my extension to my webview in React (with Vite). Can you give me any pointers to this and also include it in the documentation when it comes about?
For context, I've tried adding the window.addEventListener call in several places, from the root index.html, the index.tsx, to the individual page's useEffect, but to no avail - the postMessage is being sent from the extension but not being picked up by the event listener in my webview.
Please let me know if I should create a new issue for this instead to keep the issues organized. Thank you!


Feature : The generated web app should have share button

The webapp generated should have buttons to share the website links to social networking websites.
@mariobehling @Princu7 @aayusharora What are your thoughts on this?

@geekyd Yeah it's great idea. Will you like to work for this ?

Yes i would love to work on this 😄
What should we go with in your opinion ?
Should we add share links for all the sessions or a share button to share only event website link ?
I think we should add share button to all the sessions.
@aayusharora @Princu7 @mariobehling
What do you think?

@geekyd Great idea. I think sharing link for the whole event rather than a particular session would be more better. Other event management systems also provide this option.

@Princu7 thanks for the feedback
@aayusharora What do you think?

Exactly !!  sharing event sounds good :)

@geekyd Can you update me on this :)

@aayusharora sorry about the delay, I had examination.
I will start working on this asap.

@aayusharora I have successfully added the share button for facebook and twitter.
Check out
https://open-event-web-appp.herokuapp.com/
I need your suggestion about where to place the social links.
One option is to provide the links on all the pages with a small slider present on every page for the social media websites.
something like this.

@geekyd Nice. Yeah. That is the standard norm.

@geekyd We have a lot of space above search bar on right side. Can you try this mockup.

@aayusharora Sure we can try that.
I will create two mock-up designs and we can choose the better one, before PR.

Will be awesome @geekyd !! Looking forward :)

@aayusharora
Here are the two designs
I think we should implement a side hover bar. What do you think?
Buttons

Side hover bar

@geekyd  The seconds sounds good on desktop view but the first is better on mobile UX as we can allocate space from left and right sides on mobile.

Implement the second view on Desktop.
Implement the first one on Mobile Screen.
Make sure the search bar must remain inline along with date nav bar.

Thanks

@aayusharora please check http://open-event-web-appp.herokuapp.com/

This issue has been solved.

Please have a look at the follow up issue here: https://github.com/fossasia/open-event-webapp/issues/1055


Change use of "an" to "a" in line 60

The line in the docs: "In terms of the CAP theorem, this is an CP system...", change to "In terms of the CAP theorem, this is a CP system..."

This change is

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

v20.1/multi-active-availability.md

Can you update 20.2 docs too?

Thanks for this fix, @dilapitan. I've opened a new PR making this change across all versions of our docs: https://github.com/cockroachdb/docs/pull/11789. Sorry it took so long.


Kinetic Python SDK

Build a Kinetic SDK for Python that can be used on a server.
A Kinetic SDK is a small convenience layer on top of an existing Solana SDK (like this one) following the Kinetic Sdk Standard.
It adds helper methods for creating Solana Transactions, drastically reducing the amount of code you need to write to transact KIN and other SPL tokens.
The SDK acts as an interface to communicate with the Kinetic API, which handles the fee payment. This is done by partially signing the transaction on the device, and sending that over to the Kinetic API.
Deliverables (code all MIT licensed on GitHub):

[ ] Kinetic Python SDK based on spec (TBD), installable as Python Package
[ ] Demo app that shows full usage of the SDK
[ ] Documentation on how to install the SDK and use it
[ ] Working demo + docs presented for delivery in a call

This is done: https://github.com/kin-labs/kinetic-python-sdk


3Dtile is at the wrong place

Hello,
I've made a custom 3DTile from an obj file with Obj2Tiles but when placed on the globe, the mesh is actually thousands of kilometers away from the origin.
How the tileset appears on the scene view :

Position of an object created nearby the mesh :

It seems to be caused by a difference between the CRS used for the obj file and the one used by Cesium.
Is there any other CRS supported by Cesium on Unity other than ECEF ?

Hi @ThomasBillaud2 , would you mind asking these kinds of usage question on our forum?
https://community.cesium.com/c/3d-tiles/16
https://community.cesium.com/c/cesium-for-unity/13
(also, it may have already been asked)
Thanks!


updated readme and fixed for sddm

I made some fixes in the readme

Now it works with sddm ( issue id #96 )

acpi isnt needed btw

acpi isnt needed btw

I will remove it!

and what are these? :thinking:

and what are these? :thinking:

Chmod +x at the file


Validity check on geometry interactions.

It is currently possible create invalid geometries by editing or drawing.
We need to add a validity check for geometries as a gatekeeper in the client to prevent invalid geometries ever being sent to the backend.

Control on valid geometry now works for adding polygon as new feature.

Added for editing polygons and linestrings.


Is it possible to derive  a method of a rust struct in Dart?

For example, for a struct
pub struct VideoDoc {
    pub ids: Vec<VideoId>,
}

impl VideoDoc {
    pub fn get_uuid(&self){
        // dosomething with self.ids
    }
}

I found the class VideoDoc in dart, but no getUUid there.  How to derive it?  And is this behavior encouraged or discouraged?

https://cjycode.com/flutter_rust_bridge/feature/lang_methods.html

https://cjycode.com/flutter_rust_bridge/feature/lang_methods.html

I found that the methods would only be generated when their structs are defined in the API file. If structs are defined in a module,  then methods would be generated. Is this designed to be like this?

That seems to be a lack of feature. Feel free to PR!

That seems to be a lack of feature. Feel free to PR!

If a struct is defined in a module, then there would be extra code in generated API file, like:
// Section: impl IntoDart

impl support::IntoDart for SharedStruct {
    fn into_dart(self) -> support::DartAbi {
        vec![
            self.id.into_dart(),
            self.name.into_dart(),
            self.num.into_dart(),
        ]
        .into_dart()
    }
}
impl support::IntoDartExceptPrimitive for SharedStruct {}

What is it used for?

What is it used for?

IntoDart. i.e. convert the rust struct into dart wire type. used when you want to return a SharedStruct object

https://cjycode.com/flutter_rust_bridge/contributing/design.html

What is it used for?

IntoDart. i.e. convert the rust struct into dart wire type. used when you want to return a SharedStruct object

If a shared struct is used as a return type for multiple blocks, where should it be defined?  Should everything related to it be defined in a new rust file?

Hmm I am not sure. As long as it does not conflict IMHO it is OK.

reopen since #1145 is going to fix this

v2 seems to have this feature :) If you find bugs, feel free to reopen or create a new issue!


Pre staging

Added date and math tasks by @bkb2135  in #88
Improved dataset and context structure by @steffencruz  in #109
Enhanced neuron logging by @steffencruz  in #108
Added mock utilities to unit tests by @steffencruz  in #107
Fixed typos in README by @dougsillars  in #105
Implemented decay score by @steffencruz  in #111

Tracked experiment: https://wandb.ai/opentensor-dev/alpha-validators/runs/y78dnu4y/overview?workspace=user-opentensor-steffen
Acceptance criteria:

Each task has the same occurrence rate (25% for all four tasks)
Each task has stable rewards distribution
Rewards for both old and new tasks are consistent with other validators over same time period
Status codes are nominal and consistent with other validators over same time period


Link new generic sample for Connector configuration

Link new generic sample for Connector configuration in Connectors section.

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

ID: ebc0dbbb-a5af-6028-cb66-65cc711eb9f0
Version Independent ID: 891b08f2-827b-c5e2-8070-4489650acff9
Content: Microsoft Teams code samples - Teams
Content Source: msteams-platform/tutorials/code-samples.md
Product: msteams
GitHub Login: @laujan
Microsoft Alias: lajanuar

@Wajeed-msft - Clarification required, as we have moved out the Code sample section as a link in the TOC (as per PR#2654) changes. Need to confirm where this link needs to be added.

@KirtiPereira -

Please link link first two samples from this section to bottom of Creating Office 365 Connectors for Microsoft Teams doc under code samples sections.
Please link link last sample from this section to bottom of Create outgoing webhook doc under code samples sections.


Syncing doesn't work with auxiliary .meta files

Steps to reproduce:

Create tiddler of type text/plain
.txt, .meta files are generated
echo newtext >> textFile.txt
No change in client side

Currently it seems only .tid and .meta extensions are handled.
https://github.com/OokTech/TW5-Bob/blob/e40d5133445bf9858712347d77b7695d3fb6ed57/FileSystem/FileSystemMonitor.js#L90

I am running (check any that apply, put an x inside the [ ] to check a box, like this: [x]):

[x] Windows
[ ] OSX
[x] Linux
[ ] Other

and using

[x] The nodejs version
[ ] The single file executable

Before posting I read issue guidelines and:

[x] I am using the newest version
[ ] The answer to my question isn't listed in the documentation or this isn't
a question
[ ] This is not a duplicate issue
[ ] I have not done anything that required me to set acceptance to
I Will Not Get Tech Support For This

Thanks for the bug report, @tejasvi. I am currently doing a deep-read of the filesystem and syncing for BOB, but have not yet gotten to the FileSystemMonitor.js functionality. I will keep this in mind, thanks!

Thanks for the bug report, @tejasvi. I am currently doing a deep-read of the filesystem and syncing for BOB, but have not yet gotten to the FileSystemMonitor.js functionality. I will keep this in mind, thanks!


[BEAM-9164] Re-enable UnboundedSourceWrapper#testWatermarkEmission test

The stability issues should be resolved by d106f263e625e5d7c4f3848f16da301871f65142.

Tests passed (https://builds.apache.org/job/beam_PreCommit_Java_Commit/11297/) but running again to verify.

Run Java PreCommit

Run Python2_PVR_Flink PreCommit

Again, not flaky: https://builds.apache.org/job/beam_PreCommit_Java_Phrase/2133/ but testing again.

Run Java PreCommit

Again, no failure of that test. Assuming that the changes in d106f263e625e5d7c4f3848f16da301871f65142 fixed the root cause.

Thanks @je-ik!


[SSH] Various fixes for Github Issues

Add support for direct MSAL usage in newer azure cli (beta currently)
Add support for port option and ssh additional arguments option
Remove directory creation as part of ssh_config creation
Try .pub for public key if only private key is specified
Add --hostname to --ip argument
Add fallback to private IP with warning

This checklist is used to make sure that common guidelines for a pull request are followed.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

SSH

@N6UDP Any update for those comments?

If possible, consider splitting this PR to multiple PRs and link to corresponding issues. This simplifies the review process and our discussion.

I currently don't have enough time to fully check the PR. Let's publish it for now (with the preview talisman).


Add Extra Options to Vips?

Hi Florian, I have been playing some more with your awesome vips driver and adding in some extra options...
    ‘optimize_coding’=> true
    ‘optimize_scans’ => true
    ‘trellis_quant’ => true
    ‘quant_table’  => 3 #same setting used by mozjpeg I believe
    ‘overshoot_deringing’ => true

If you take a 1993 × 2491px jpg 1.2mb original colour photo with the options below you get a file of size of 231 KB - pretty good!

    ‘interlace’     => true
    ‘strip’         => true
    ‘quality’       => 50

But with the options below you get a file of 182 KB, a handy saving, at the expense of some processing time.

    ‘interlace’     => true
    ‘strip’         => true
    ‘quality’       => 50
    ‘optimize_coding’=> true
    ‘optimize_scans’ => true
    ‘trellis_quant’ => true
    ‘quant_table’  => 3 #same setting used by mozjpeg I believe
    ‘overshoot_deringing’ => true

Using mozjpeg on the same file at the same quality you get a file of 179 kb

A webp at Q50 is quite a bit less at 148kb but it maybe perceptually a bit different on photos - but maybe OK?

(Of course if you go the jpg route only (no webp) then there is only one set of images in multiple sizes… which may or may not be important.)
At least for photos, these extra options look like a good compromise between image quality and image size.
*It would of course be nice to use mozjpeg with vips rather than the default (jpeg or jpeg-turbo?). But the install option for mozjpeg with vips on homebrew has been removed… and I’m not sure I want to build vips from scratch with mozjpeg…I may do it if the lock down continues much longer...
So Florian what are your thoughts on these extra options?
OSX so all installed via brew

vips 8.9.2
php 7.4
caddy 1.4
mozjpeg 3.3.1
webp 1.1.0

Hi! Sorry i am not really sure what you are asking me to do. Do you want to change default settings to something else? Are there options that don't work if you put them in the config file?
Thanks!

Hi Florian, sorry to be unclear. I was just saying that these would be good options to add to your plugin as I have done on my local copy.

Hi, i've tried to address it here.
https://github.com/floriankarsten/kirby3-vipsthumbnail/blob/dev/Vipsthumbnail.php
Could you try if that works with your options? There is now array $allowed options that allows adding one word output options i am not sure if some other options need parameters. We need o gather all the required options to work.

@shoesforindustry ?

Very sorry Florian, I've been away from the Internet, I shall have a look tomorrow and report back. Thank you for your efforts.

OK, I have managed to have a quick look this morning and whilst the driver still works it did not as compress as much as my version. I changed my Vipsthumbnail.php in a slightly different way to you and add it below as well as my config which shows the options. I'm sorry but I am going to have intermittent internet access, so I may not respond right away, I hope this is OK ;-)
<?php

namespace Floriankarsten;

use Exception;

/**
 * TBA
 *
 */
class Vipsthumbnail
{
	public $options = [];
	public $src;
	public $dst;

	public function __construct(string $src, string $dst, array $options = [])
	{
		$this->src = $src;
		$this->dst = $dst;
  // option merging: plugin defaults > config options > options from image
    $this->options = array_merge($this->defaults(), option('thumbs'), array_filter($options));

		if ($this->options['log'] === true) {
			// $this->logMessage(json_encode($this->defaults()));
			// $this->logMessage(json_encode(option('thumbs')));
			// $this->logMessage(json_encode(array_filter($options)));
			$this->logMessage(json_encode($this->options));
		}
	}

	protected function defaults(): array
	{
		return [
			'bin'           => 'vipsthumbnail',
			'interlace'     => false,
			'autoOrient'    => true,
			'crop'          => false,
			'height'        => null,
			'strip'         => true,
			'quality'       => 90,
			'width'         => null,
			'optimize_coding'=> false,
			'optimize_scans' => false,
			'trellis_quant' => false,
			'quant_table'  => null,
			'overshoot_deringing' => false,
			'log'           => false
		];
	}

	function logMessage($log_msg)
	{
		// dumblog from stack overflow please no judging
		if (!empty($this->options['logdir'])) {
			$log_filename = $this->options['logdir'];
		} else {
			$log_filename = __DIR__ . "/logs";
		}

		if (!file_exists($log_filename)) {
			// create directory/folder uploads.
			mkdir($log_filename, 0777, true);
		}
		$log_file_data = $log_filename . '/vipthumbnaillog_' . date('d-M-Y') . '.log';
		file_put_contents($log_file_data, $log_msg . "\n", FILE_APPEND);
	}

	protected function autoOrient()
	{
		if ($this->options['autoOrient'] === true) {
			return '--rotate';
		}
	}

	protected function convert(): string
	{
		return sprintf($this->options['bin'] . ' %s', $this->src);
	}

	protected function interlace()
	{
		if ($this->options['interlace']) {
			return 'interlace';
		}
	}

  protected function optimize_coding()
	{
		if ($this->options['optimize_coding']=== true) {
			return 'optimize-coding';
		}
  }
  protected function trellis_quant()
	{
		if ($this->options['trellis_quant']=== true) {
			return 'trellis-quant';
		}
  }

  protected function optimize_scans()
	{
		if ($this->options['optimize_scans']=== true) {
			return 'optimize-scans';
		}
  }

  protected function quant_table()
	{
		if ($this->options['quant_table']) {
      return 'quant_table=' . $this->options['quant_table'];
		}
  }

  protected function overshoot_deringing()
	{
		if ($this->options['overshoot_deringing']) {
        return 'overshoot-deringing';
		}
  }

	public function process(): string
	{

		$command = [];
		$outputOptions = [];

		$outputOptions[] = $this->strip();
		$outputOptions[] = $this->optimize_coding();
		$outputOptions[] = $this->optimize_scans();
		$outputOptions[] = $this->trellis_quant();
		$outputOptions[] = $this->quant_table();
		$outputOptions[] = $this->interlace();
		$outputOptions[] = $this->quality();
		$outputOptions[] = $this->overshoot_deringing();

		$outputOptions = implode(',', array_filter($outputOptions));

		$command[] = $this->convert();
		$command[] = $this->autoOrient();
		$command[] = $this->resize();

v$command[] = $this->save($outputOptions);
		// remove all null values and join the parts
		$command = implode(' ', array_filter($command));

		if ($this->options['log'] === true) {
			$this->logMessage($command);
		}

		// try to execute the command
		exec($command, $output, $return);

		// log broken commands
		if ($return !== 0) {
			throw new Exception('The Vips convert command could not be executed: ' . $command);
		}

		return $this->dst;
	}

	protected function save($outputOptions): string
	{
		return sprintf('-o %s[%s]', $this->dst, $outputOptions);
	}

	protected function quality(): string
	{

		return 'Q=' . $this->options['quality'];
	}

	protected function resize(): string
	{
		// normalize crop
		// here it should be possible to take $this->options['crop'] "center" etc to make crops by direction
		// dirty weak check $this->options['crop'] == true if its either true or string "center" etc
		if (is_string($this->options['crop'])) {
			$this->options['crop'] = true;
		}

		// simple resize
		if ($this->options['crop'] === false) {
			return sprintf('--size %sx%s', $this->options['width'], $this->options['height']);
		}

		if ($this->options['crop'] === true && $this->options['height'] === 0) {
			// assume crop to square like ->crop(100)
			return sprintf('--size %sx%s --smartcrop attention', $this->options['width'], $this->options['width']);
		}

		if ($this->options['crop'] === true) {
			// assume crop with exact sizes
			return sprintf('--size %sx%s --smartcrop attention', $this->options['width'], $this->options['width']);
		}
	}

	protected function strip()
	{
		if ($this->options['strip']) {
			return 'strip';
		}
	}
}

And in my config.php I have this, I've listed the full set of options:
'thumbs' => [
  'mozjpeg' => '/usr/local/opt/mozjpeg/bin/cjpeg',
  'driver' => 'vipsthumbnail',
  'bin' => '/usr/local/bin/vipsthumbnail',
  'interlace' => true,
  'autoOrient' => true,
  'crop' => false,
  'height' => null,
  'strip' => true,
  'quality' => 90,
  'width' => null,
  'optimize_coding' => true,
  'optimize_scans' => true,
  'trellis_quant' => true,
  'quant_table' => 3,
  'overshoot_deringing' => true,
  'log' => true
],


Common message parts in different projects

Hi,
Here is my use case:
Project A defines a set of messages with a common base (BasePub is a  that is referenced in every sbe:message object in project A).
Project B which depends on A and uses the objects defined in A adds new messages that have the same base BasePub. Project A does not need to be aware of Project B objects, and project B need only be aware of the 'common' objects.
Is there a way to generate the objects of project B without copying all the common objects from project A in B's schema? (It appears that the SBE tool is able to generate codecs from a single schema.xml file.)
Thanks,
Vladimir

Unfortunately this is how the specification is defined. Version 2.0 is close to being ratified. It would be worth you voicing your requirements to the spec committee. https://github.com/FIXTradingCommunity/fix-simple-binary-encoding


Attention, Transposed Convolutions, Embeddings, LayerNorm

Merging #34 into master via this branch, so as to make some tweaks.
Updates relative to #34:

Added new layers to the documentation.
Added the mathematical details to the docstrings for MultiheadAttention and LayerNorm
Substantially overhauled the MultiheadAttention implementation. The implementation in #34 was basically mimicking PyTorch's implementation, which has a very bad API. The new API is much more consistent, and much more general. (For reference, I found the Haiku implementation of MultiheadAttention to be the cleanest of all previous implementations.)

Plus some other misc updates in support of these changes:

Dropout now takes a fast path when p=0.
Enabled MathJax in the documentation.
Bumped version number.
Updated custom_types.Array and custom_types.PyTree to follow the convention in Diffrax; namely that they're now subscriptable and the documentation handles this.
Standardised on the spelling "normalisation" (rather than "normalization") just because that's the convention I happen to follow.
Other misc doc tweaks.

CC @andyehrenberg (and @lucidrains ?) for any commentary on these changes prior to merging them in.

I agree that your overhauled MultiheadAttention has a better API. When I was implementing it, I was trying to make the Haiku implementation fit in with the PyTorch API, but I agree that it's pretty bad (with sort of misleading arguments). I've been putting more thought into ConvTranspose lately and should have some extensions ready soon that take inspiration from how https://github.com/google/jax/pull/5772 computes output sizes (mainly to deal with more output_padding and padding cases). It looks like I'm getting consistent outputs between this new implementation and Haiku's ConvTranspose but want to test things a bit more.

Excellent. I'm assuming you implemented MultiheadAttention on the basis that you need it for your own work? If you could give it a try and check that it looks like it's working for your use cases -- just to be sure -- then that'd be great.
As for transposed convolutions: staring at the implementation I'm not feeling convinced by it. These things jump out at me:

this implementation only supports padding = 0 or padding = 1 -- perhaps this is what you're describing above.
Would it be simpler to ignore lax.conv_transpose and work directly with lax.conv_general_dilated? Looking at the implementation of lax.conv_transpose, it's just a thin wrapper around lax.conv_general_dilated and I don't think we really hit any of the meaningful functionality that that wrapper provides.
We can probably compute dimension_numbers for all dimensions, just by working directly with lax.ConvDimensionNumbers instead of the simplified string replications. (It doesn't look that tricky.)

WDYT?

With the other branch merged in: let me know once you're satisfied that both MultiheadAttention and ConvTranspose work for your use cases and I'll merge this branch + do a new release.


Works on XU4

If I changed odroid_fan.14 to odroid_fan.13 it works on Odroid XU4 too.
FAN_MODE_FILE="/sys/devices/odroid_fan.13/fan_mode"
FAN_SPEED_FILE="/sys/devices/odroid_fan.13/pwm_duty"

Are you able to provide a pull request?
A change in the script that tries to test if *fan.13 or *fan.14 exist.
Based on that it uses one or another..

PR https://github.com/nthx/odroid-xu3-fan-control/pull/4


Scroll Snap intent to implement in Firefox 68

CSS Scroll Snap: https://caniuse.com/#feat=css-snappoints
As noted, Firefox supported properties from an older version of the spec.
We are updating our implementation, removing support for the old properties, and implementing support for the current properties:
https://groups.google.com/forum/#!msg/mozilla.dev.platform/s0rMvOBnO_4/AFjL0qHpAwAJ
With the intention of shipping in Firefox 68.

Also, the spec name has changed, and so "css-snappoints" as a token in the URL:
https://caniuse.com/#feat=css-snappoints
is out of date.
Do you update such things?

Not generally so as not to break existing links. Seems leaving it as is should be fairly harmless.

The data seems up to date (now):

I think this issue can be closed?

@jensimmons ^ ? :)

Yup.


Error: more than one library with package name 'com.reactlibrary'

i think you should change package name to match your plugin's name..

Hello, i will think about it, thanks

fixed in v2.1.0


Clearer "offline" UI treatment

Explore more explicit "offline" treatment .. for people and for instances

Some of the ideas discussed:

Don't show a button if the contact is both offline and you wouldn't be able to click on the button (e.g. don't show the grey'd out start getting access button)
Show a "(Offline)" by the instance name. Or by the username if the there is only one instance.

Another idea: offline instances are hidden under a fold (you click on the user to unfold), while online instances are always shown.

My recommendation the same as @dborkan's in this issue: https://github.com/uProxy/uproxy/issues/878
"We could also hide instances if they have not signed online in some time
frame, but not actually delete them.  Then if they ever appear online again
they could reappear in the buddylist with the same permissions as before."

This is an issue we're still seeing in UI studies. Users who sign in for the first time with FB may see only one offline friend. In that case, they don't have a basis for comparison to recognize the shading difference between offline and online. Perhaps we can just write the word "offline" under the buddy name. What do you think, @zahoriea?

Yep! Completely agree.
Here is an example from an old mock below:

Does this look okay?

Thanks Lucy! Would you be able to send over the font-size of
"Online/Offline" and line-height of Bob Uproxy -> Online?
On Fri, Sep 11, 2015 at 11:51 AM, lucyhe<EMAIL_ADDRESS>wrote:

Does this look okay?
[image: image]
https://cloud.githubusercontent.com/assets/8723127/9823511/a4987c90-5894-11e5-80dd-6909edc9549a.png
—
Reply to this email directly or view it on GitHub
https://github.com/uProxy/uproxy/issues/874#issuecomment-139626963.

CSS for Online:
padding: 0px 18px;
font-size: 11px;
line-height: 13px;
Bob Uproxy is font-size 16px;

Maybe a little closer to the mocks?
Bob Uproxy:
font-size: 15px;
line-height: 21px;
("Online" the same)

(oops tried to update to a retina SS, didn't realize it'd be huge!)

done with https://github.com/uProxy/uproxy/pull/1888


Add info about retention

🚀 Deployed on https://6740c836f74a11143dd14514--cozy-fox-0defec.netlify.app


React Plugin integration

I would like to know how to integrate react plugins, like this becuase I can't see how to do so from your expansive documentation.

This issue was moved to reactrb/reactrb#53


Where's the module unite_clarifications source/preprocessing

Hello! I can not find the module unite_clarifications.py　in  source/preprocessing

Thanks for pointing this out, I added the missing script: https://github.com/vered1986/self_talk/blob/master/source/preprocessing/unite_clarifications.py


Final steps before the release of @excalidraw/excalidraw

[x] Added Readme
[x] Added changelog
[x] Updated version

I haven't followed this package in general, but is there a plan to add a GitHub action to automatically publish on npm..
How the releases are going to work? Would be nice to automate as much as possible.

I haven't followed this package in general, but is there a plan to add a GitHub action to automatically publish on npm..
How the releases are going to work? Would be nice to automate as much as possible.

Yes, we should automate releases, I was thinking of using something like semantic-release which helps in creating release notes as well so we need not worry about changelog or deciding what should be the next version, etc.  We can discuss this with the team after the initial release.

Yes, we should automate releases, I was thinking of using something like semantic-release which helps in creating release notes as well so we need not worry about changelog or deciding what should be the next version, etc. We can discuss this with the team after the initial release.

That workflow still relies on people naming their commits correctly.
I haven't filly reviewed the readme, but I trust @ad1992 did a good job, and we can always make tweaks in the future. Feel free to merge.


cannot get slicknav to be visible at all

I hope someone can help. I've searched for so long for a menu exactly like this one so very happy when I found slicknav.  Unfortunately, I cannot get it to display at all.  I have been trying for 3 days and created all sorts of test pages and I cannot get to see it, even at the first stage before the media queries are set and it's meant to be visible with your own menu.  There must be a conflict with something I've got going on in my site becuase slicknav's instructions are clear and straightforward.  Please can someone take a look www.theterracewimbledon.co.uk/index2.html
Any advice so welcome!!!!  I don't want to give up on slicknav and try another one because this one is lovely!  Thank you

I took a look at the link and it looks like you went with a different solution. If you want to open this back up you could provide a jsfiddle so we can take a look at what the problem might be.

Thanks Josh, I had to give up with Slicknav as I couldn’t wait anymore for a response and had to find a solution quick!  I managed to find a pure css one which is working just fine, so removed all my experimental stuff. It was a shame though because I spent ages trying Slicknav but just couldn’t get a fix for it!
Thank you for a wonderful service though….I’ll be back!
Tracy
From: Josh<EMAIL_ADDRESS>Sent: 24 February 2015 15:26
To: ComputerWolf/SlickNav
Cc: charlieapple
Subject: Re: [SlickNav] cannot get slicknav to be visible at all (#68)
I took a look at the link and it looks like you went with a different solution. If you want to open this back up you could provide a jsfiddle so we can take a look at what the problem might be.
—
Reply to this email directly or view it on GitHub https://github.com/ComputerWolf/SlickNav/issues/68#issuecomment-75776720 .  https://github.com/notifications/beacon/AKczffkU_ELZfEZu2RTq7toq6z3AiRa9ks5nvI9qgaJpZM4DgZSi.gif


Rat Check fails for Deployment Sample Data

[17-4-2019 10:12] Wu, Y. (Yuefeng):

Summary
Generated at: 2019-04-17T09:40:26+02:00
Notes: 0
Binaries: 4
Archives: 0
Standards: 116
Apache Licensed: 0
Generated Documents: 0
JavaDocs are generated, thus a license header is optional.
Generated files do not require license headers.
1 Unknown Licenses

Files with unapproved licenses:
C:/Workspace/egeria/open-metadata-resources/open-metadata-deployment/sample-data/coco-pharmaceuticals/metadata/ibm-igc/ds_jobs_ibm-db2.isx

Hi Chris, thanks. I will revert my modification on POM file in my pull request and add exclusion locally.

If it's breaking on all Windows builds (which sounds like it is the case?) then I think we do want the fix to go into master (not just locally)...
Looking at your change, though, I think it overlaps a bit with the root project pom.xml -- which already excludes all .json and .csv.  Probably we should add the RAT exclusion to the list there, to ignore either **/*.isx or at least /open-metadata-deployment/**.isx?

Sure, I will add it. to the exclusion. :)

Maybe as a separate, super-simple PR as well?  Would like to ask @GuyAckermans to review it, as he seemed to face the same issue.


Update description of available languages for numpy.org

Thanks for catching this @melissawm. That’s an embarrassing mistake on my part. l guess a result of revising the text of that card dozens of times until I was no longer processing the text.

Is there a manual step for pages build and deployment? It looks like it was not triggered by this pr...

Ah, sorry. I forgot that deployment is still manual. I'll manually deploy and look into getting automatic deployment set up.

It's up now https://scientific-python-translations.github.io/


Added Checkout to Other User Functionality

(Added) Functionality to checkout to other users

Thank you for your contribution! It was merged on a dev-branch  and will be merged into main soon!


Performance concern regarding print function

The print function does not seem to implement block buffering. So if you print a lot of lines, you end up calling the write syscall for every line, which is quite costly.

false, print function collects all args and then prints them in a single syscall
https://github.com/Alonely0/Voila/blob/d259f22febb442613354f757b45c867ebfcb3f1e/src/interpreter/functions.rs#L61

false, print function collects all args and then prints them in a single syscall
https://github.com/Alonely0/Voila/blob/d259f22febb442613354f757b45c867ebfcb3f1e/src/interpreter/functions.rs#L61

Line buffering and block buffering are not the same thing. Rust's io::stdout does not do block buffering by default.
https://github.com/rust-lang/rust/issues/60673

In C, setvbuf(stdout, NULL, _IOFBF, 0); can be used to make stdout block buffered. By default, it is line buffered.

ah lol, didn't know that.


Revive crate, upgrade to new API

Rationale for reviving a deprecated crate

This crate is the first result when searching for mpv on crates.io and has a nice API and even an example for embedding into SDL2, it only needs to have a few changes to be perfectly working again.
The referenced crate libmpv does not currently support any rendering functionality.

List of changes

Upgraded bindings and documentation on binding generation; see mpv_handler.rs
Switched from deprecated opengl_cb API to the render_gl API
Adjusted Event enum to API changes
Removed deprecation notice

These changes are made while leaving the user-facing API unchanged, except of the following changes

Removal of the SubApi enum
Replacement of the mpv_opengl_cb_get_proc_address_fn with the GetProcAddressFn type

Simple question, why not use libmpv, written by ParadoxSpiral? It seems more stable to me. This was written in my early rust days, and I'm sure there are many flaws like this one.
If you can give me a good reason as of why this specific crate is useful to you, I'm willing to transfer this repository and give you crates.io access.
Frankly I don't have time to maintain this, I'm already trying my best to keep rust-sdl2 floating (which itself is barely alive because there is no one else), so if you want this crate it's all yours!

I chose this crate in particular because it was easy to find and is (to my knowledge) the only one which supports rendering into an OpenGL context.
It's also accessible with an example for SDL2, so you can get video playback going in a few lines of code.
However, I can understand your lack of time to maintain it, I appreciate your fast response though. :)
For now, I will use my fork and contribute a render API to ParadoxSpiral's crate when I have the time.


[ENH] Will dqc support a solid system with externel potentials?

Like implementing Pseudopotential approximation.

There is an experimental feature (undocumented) on solid system with gaussian basis using density fitting. However, the 3-centre integrals are very slow and I'm not sure if it propagates the gradient, so it is not ready yet.
We certainly have a plan to expand it to solid system, but currently it is not our top priority. If you want to make a pull request on solid system, you are welcome to do so.


Feat: GitHub action that locks closed issues, discussions, and PRs

Changes

Feat: GitHub action that locks issues, discussions, and PRs that have been closed and have not received interaction in the last 30 days (the 30-day window helps when we fix and close an issue, but it has not actually been resolved so the user can reopen it timely).

Thank you for the PR!


Book keeping files

AUTHORS
HISTORY -> for release updates? (I am unsure on this one)
Do we use these still with pyproject.toml? @carlsondev

I don't believe it is required, however, it may be more clear for repo visitors to have those files (or at least the AUTHORS).


Fix security issues with exampes.

This CR replaces package 'lite-server' with security issues
by 'http-server'.
Adding yarn-lock.json to the git in examples folder.
Resolves: OLPEDGE-2002
Signed-off-by: Oleksii Zubko<EMAIL_ADDRESS>

Codecov Report

Merging #316 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #316   +/-   ##
======================================
  Coverage    87.6%   87.6%
======================================
  Files          56      56
  Lines        1559    1559
  Branches      183     183
======================================
  Hits         1365    1365
  Misses        121     121
  Partials       73      73

Continue to review full report at Codecov.

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


Nuget problem

Hi,
I am getting error when try to install nuget package.
nstall-Package : An error occurred while retrieving package metadata for 'angular-file-upload' from source 'https://github.com/'.
At line:1 char:1

Install-Package angular-file-upload

  + CategoryInfo          : NotSpecified: (:) [Install-Package], Exception
  + FullyQualifiedErrorId : NuGetCmdletUnhandledException,NuGet.PackageManagement.PowerShellCmdlets.InstallPackageCommand

Post it on stackoverflow

@raysefo I think you did NOT specify the correct source. Try make sure you have the correct source configured. "https://www.nuget.org/api/v2/" is the official source for Nuget.


credentials_manager 0.16.2 has been removed

New Issue Checklist

[x] ~Updated fastlane to the latest version~ My issue is related to old version support and its removal from rubygems, so latest version does not apply here
[x] I have read the Contribution Guidelines

Issue Description
Builds were broken overnight due to removal of gems from rubygems.

Your bundle is locked to credentials_manager (0.16.2), but that version could not be found in any of the sources listed in your Gemfile. If you haven't changed sources, that means the author of credentials_manager (0.16.2) has removed it. You'll need to update your bundle to a different version of credentials_manager (0.16.2) that hasn't been removed in order to install

My legacy environments were fixed old fastlane version (1.103.0) and builds started breaking because credentials_manager version 0.16.2 has been removed from rubygems.
The release note says it was removed due to circular references (more specifically between fastlane and scan gems), but that was not my case environments as I was using older versions.
Having an immutable build environment is crucial and Fastlane has been a huge step towards that, so I believe removing old versions from rubygems in favor of new releases goes totally against it.
Environment
Here is a gist link to my Gemfile, Gemfile.lock and my travis build error output: https://gist.github.com/felipesabino/3f2a8510ff5b58987381a76a7c9a6837
Workaround
If you can't change the build environment right now (my case), you can workaround it by manually monkey patching the Gemfile to have credentials_manager and any other gem fixed to the github source
gem 'credentials_manager', '0.16.2', :git => 'https://github.com/fastlane/fastlane.git', :tag => 'credentials_manager/0.16.2'

In my case (Gemfile and Gemfile.lock attached) I also had to fix scan to 0.14.1

Just as an FYI, I did not added fastlane env output as this command was added only at 1.106.0 and my problem relates to 1.103.0 version and legacy support (Explanation at the issue description)
cc @fastlane-bot

This didn't just affect old builds. We updated to ~> 2.0 on Friday which was pegged to credentials_manager 0.16.2 so things that worked EOD Friday were busted Today morning.

I encountered the same issue today as well. Agree with @felipesabino about the build environment - progress is great and all, but intentionally breaking older installs is extremely problematic for a tool depended on by so many.

I had fastlane pinned to 1.111.0 which depended on credentials_manager (>= 0.16.2, < 1.0.0) which now fails. Should be easy enough for me to update but yanking gem versions like this seems like an unfriendly move and something that should be avoided.

My fastlane also pineed to 1.11.0 and failed start from this morning

The release note says it was removed due to circular references (more specifically between fastlane and scan gems

Yes, we've had lots of users getting a circular dependency error, so we had to do this change to fix their setup. See next reply for more details.

Should be easy enough for me to update but yanking gem versions like this seems like an unfriendly move and something that should be avoided

Yes, completely agree, this was not something we wanted to do, really sorry this happened. We wanted to avoid yanking any releases, however lots of people that don't use bundler ran into this error, forcing us to yank releases and push new major releases.
I actually added further information about this, over here.

Having an immutable build environment is crucial and Fastlane has been a huge step towards that, so I believe removing old versions from rubygems in favor of new releases goes totally against it.

I completely agree, and actually gave all of this a lot of thought. Taking parts of my reply from here:

True, but due to fastlane's nature of usually needing the most up-to-date spaceship release, we really want people to be on the latest release usually. While this is not ideal as builds might not be reproducible, there is no way (to my knowledge) to do that, with the third party API changing, and us needing to update spaceship regularly to stay up to date.

Due to spaceship we are forced to always have people running on the latest release, otherwise their build isn't immutable anyway (due to changing third party APIs out of our control).

My fastlane also pineed to 1.11.0 and failed start from this morning

I assume you don't mean 1.11.0? Could you update to 2.1.1 and see if that fixes your issue?

Furthermore, I looked into the yanked release, and it seemed like we yanked a really old release (https://github.com/fastlane/fastlane/releases/tag/credentials_manager%2F0.16.0), I'll try to push it up again, let's see if that fixes the issue.
If you have any other feedback, please do let us know. We're always happy to hear how we can make it better and easier for you all 👍

We wanted to avoid yanking any releases, however lots of people that don't use bundler ran into this error, forcing us to yank releases and push new major releases.

I understand, but removing already published gems creates a huge trust problem and seems like node's left-pad issue all over again.
If people don't use bundler and have version conflicts we need to ask them to use it, as you even suggested me before when I tried to solve a version conflict on cocoapods at #4627 the wrong way

Furthermore, I looked into the yanked release, and it seemed like we yanked a really old release (https://github.com/fastlane/fastlane/releases/tag/credentials_manager%2F0.16.0), I'll try to push it up again, let's see if that fixes the issue.

Restoring would be awesome and I had issues with both 0.16.0 and 0.16.2, not sure what other gem versions were removed.

I understand, but removing already published gems creates a huge trust problem and seems like node's left-pad issue all over again.

We didn't want to yank any of the published gems, I don't know yet why this happened. That was not planned.

If people don't use bundler and have version conflicts we need to ask them to use it, as you even suggested me before when I tried to solve a version conflict on cocoapods at #4627 the wrong way

Agreed, however lots of people don't want that, that's why we built the bundled fastlane for them a few weeks ago.

Restoring would be awesome and I had issues with both 0.16.0 and 0.16.2, not sure what other gem versions were removed.

It looks like RubyGems doesn't support unyanking of releases any more, so the only thing we can (and probably will) do is to release a credentials_manager 0.16.4 with the content of 0.16.2

not all the gems are getting updated due the error thrown by credential_manager circular dependencies. I have installed only GitHub (hub), Cocoapods and Fastlane gems in the machine.
Here's the fastlane env output
🚫 fastlane environment 🚫
Stack

Key
Value

OS
10.12.2

Ruby
2.3.1

Bundler?
false

Git
git version 2.11.0

Installation Source
/usr/local/bin/fastlane

Host
Mac OS X 10.12.2 (16C67)

Ruby Lib Dir
~/.rbenv/versions/2.3.1/lib

OpenSSL Version
OpenSSL 1.0.2h  3 May 2016

Is contained
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
8.1

System Locale

Error

No Locale with UTF8 found 🚫

fastlane files:
No Fastfile found
No Appfile found
fastlane gems

Gem
Version
Update-Status

credentials_manager
0.16.2
🚫 Update availaible

spaceship
0.39.0
🚫 Update availaible

fastlane
2.0.3
🚫 Update availaible

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.0.2

slack-notifier
1.5.1

CFPropertyList
2.3.4

claide
1.0.1

colored
1.2

nanaimo
0.2.3

xcodeproj
1.4.1

rouge
1.11.1

xcpretty
0.2.4

terminal-notifier
1.7.1

plist
3.2.0

public_suffix
2.0.4

addressable
2.5.0

multipart-post
2.0.0

highline
1.7.8

security
0.1.3

credentials_manager
0.16.2

multi_xml
0.6.0

faraday_middleware
0.10.1

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20161129

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

fastimage
2.0.1

babosa
1.0.2

xcode-install
2.0.9

word_wrap
1.0.0

excon
0.54.0

gh_inspector
1.0.2

uber
0.0.15

representable
2.3.0

retriable
2.1.0

mime-types-data
3.2016.0521

mime-types
3.1

hurley
0.2

little-plugger
1.1.4

multi_json
1.12.1

logging
2.1.0

jwt
1.5.6

memoist
0.15.0

os
0.9.6

signet
0.7.3

googleauth
0.5.1

httpclient
2.8.3

google-api-client
0.9.20

mini_magick
4.5.1

rubyzip
1.2.0

xcpretty-travis-formatter
0.0.4

dotenv
2.1.1

bundler
1.13.6

i18n
0.7.0

json
1.8.3

thread_safe
0.3.5

tzinfo
1.2.2

minitest
5.10.1

activesupport
<IP_ADDRESS>

commander
4.4.2

io-console
0.4.6

unicode-display_width
1.1.2

terminal-table
1.7.3

generated on: 2016-12-22

The latest version 2.2.0 solves circular dependency issues. [sudo] gem update fastlane was a smooth go today.

Great to hear 👍


Set children to null (having previously been set) the arrow remains pointing down

My use case is that the tree shows options near a certain location. When the location updates, I need to refresh the option list dynamically. I do this by setting the options children back to null when location updates.
This works and forces the tree to search again when the user interacts with the tree however you can see here, on HOTEL, that the arrow remains pointed down V (because there were children there before I set it back to null) rather than >

User also has to click several times - once to put the arrow back to > then a second time to make it search again (setting to V)
Expected behaviour
I would expect that when options are changed dynamically that the arrow state is also rechecked to make their open / closed state to match the newly updated children status. i.e. if there WERE children and now children is null, it should go back to > rather than stay on V

Currently, the internal logic of vue-treeselect is that, after being set children as an array (whether it's empty or not), a branch node will turn to "loaded" stated, and cannot go back to "unloaded" state. That's why you cannot reset state by setting children: null. I'll consider changing this behavior.
As a workaround, you can add a key to <treeselect />, its value should be generated based on user entered location. For example:
<div>
  Location: <input type="text" name="location" v-model="location" />
  <treeselect :key="location" ... />
</div>

Then every time the user enters a new location, a new instance of vue-treeselect will be created and its states will be fully reset (the user selected value will be kept).

Thanks for the idea!
Yeah better if you can address your end, if possible. Thank you!


Update Example

Version: 2.3.1
Bug: No
Description:
In your example, You set filtered count after sorting the columns. But this is not necessary because the number of records are the same and the query is slower.
So call first:

// Get filtered count.
$queryCount = clone $query;
....

Then;

// Order.
foreach ($request->order as $order) {
....

Thank You for your excellent bundle !!

You are right - I have updated the example, thank you!


[l10n] Improve et-EE locale

Ref #20656

[X] I have followed (at least) the PR section of the contributing guide.

@villuv thanks 🙏


for playdate import

hi im new to playdate
how can i take a song from midi and convert to json ,?
i see some sample json songs in folder but i cant seem to
import them into playdate direct? Thank you.

@zartan9 There is a new build available and updated instructions in the Readme file that should provide some clarity.


Address coverage and issues

Summary by CodeRabbit

New Features

Enhanced naming conventions across various classes and files for consistency and clarity.

Bug Fixes

None.

Refactor

Standardized naming of types and member variables, improving readability and maintainability.
Streamlined the implementation of certain functions to reduce overhead.
Improved memory management and encapsulation of observer behavior in specific classes.

Style

Updated all instances of variables and types to follow lowercase naming conventions where applicable.

Tests

Adjusted test cases to align with new method and signal names following refactoring.
Expanded test coverage for error handling and retry mechanisms in observables.

Chores

Minor housekeeping changes for improved code clarity without impacting functionality.

@coderabbitai full review
@coderabbitai summary

@coderabbitai full review

@coderabbitai summary