support for aggregate 'phone' and 'tablet' detection

While working with this library, it became useful to have a grouping for 'phone' and 'tablet' to avoid having long conditional statements in app code.
I also added a very basic NPM manifest to allow locally installing the grunt libraries needed for tests/uglify, and added a Bower manifest file.  To allow registering with Bower for installation, a 0.3.0 tag will need to be created in this repo to match the manifest.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Sidebar on right side doesn't align with Close button of the window on Windows

VSCode Version: 1.33.1
OS Version: Microsoft Windows 10.0.17763

Steps to Reproduce:

"workbench.sideBar.location": "right",

Does this issue occur when all extensions are disabled?: Yes

👻

This is by design. Widening the window controls to match the sidebar is only effective on this side (right sidebar) and would make them look less native.


Add Version Info in the App

Hello Very Great App!!
It will be nice to see "App version release" inside the App.

Hello!
Thank you for your suggestion! I've added it to my backlog.
In the meantime, you can view the current version of the app in your phone's settings, typically under the Applications section.

Hello!
I have added the app version information to the Settings page in v1.12.4


Survey: How do you use this project?

Hi guys,
I am Magento core developer and may not know all the issues you may face as a community developer. To improve current project, I am trying to collect more information about the use cases community developers have in their daily work.
I would appreciate If you could share your typical project structure and how you use this vagrant project.
Maybe there are some missing features, which could make your life easier.
Thanks
Alex

@jasonheecs @CellFishy @nfourteen @wpoortman @sunel @MurKit @WillPS1989 @andertun @GerardKetuma @jwineman @dimstar @s2010 @henryhardy @tmos @nicolaspernot @samuelecarpene @PivitParkour94 @crantron @TapaniAla @degaray @kaynenh @rhssk @dazuaz @jasonalvis @apothegm @jatasya @createadrln @nobuttsalex @hvstudio @LCIJJ @crantron @Szaamaan @refaelgold @willboudle @Jrb1x @dlehren @CNSam @riconeitzel @aaronallen8455 @ClaudiuCreanga since you reported issues for this project some time ago, I assume you used it and would appreciate if you could share your experience, thanks.

I ended up using another one since I have nginx installed in my production site.

I ended up using another one since I have nginx installed in my production site.

Hello, I ended setting up valet, which was a lot faster for me in Mac

On Jul 18, 2017, at 7:32 PM, degaray<EMAIL_ADDRESS>wrote:
I ended up using another one since I have nginx installed in my production site.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I'm also using Valet. It's insanely fast. No VM's. just a Docker container for Redis and RabbitMQ.

I maintain Cloudflare's magento plugin.  We use vagrant internally for our development environment so I wanted a vagrant setup for magento as well to ease new developer's ability to get up to speed.
Pros:

Easy to get up and running
Easy to reset to known state

Cons:

I don't like the bash scripts that wrap bin/magento and composer.  They occasionally swallow errors and when something breaks I'm not sure if the environment (bash script) is broken or if my code is the root cause.  I would rather vagrant ssh and use the box but I assume the bash scripts exist for a reason.
Switching between magento versions isn't straight forward.  Its been a while since I did it but I remember being very confused trying to reproduce a bug on a different version of Magento.  It took me half a day to get the vagrant box on a different version.

I ended up going in another direction (Shopify). Magento 2 and the different environment setups I tried was too time consuming to get to work properly for the webshop I am building.

Wrote bash script to create / clean-up projects directly (create nginx configs from templates, MySQL users / databases etc.) and have been using that instead of Vagrant since the projects were significantly slower to work on than when running them without any virtualization.

Staying away from any virtual environments from now on. The file syncing between the host machine and virtual box/docker is always painfully slow.

I couldn't get it to work as I needed to - cloning the git repository worked but for whatever reason the Composer method wouldn't. I tried Magestead but had problems with that too. In the end I just used Scotchbox.
An all-in-one solution like this is still desired by me.

I'm very intrested in this solution. Under Windows 10 is very difficult to use vagrant.

I'm with @jwineman. His pros and cons mirror my own. I also really like how easy it is to install and uninstall sample data.
I have been looking into Valet as a means to speed things up. NFS is really slow and I quit using grunt watch because it was taking 15-20 seconds for it to pick up the file changes. I've upped the box to use 4GB, but I'm still getting pretty bad page load times (kind of understandable in developer mode), which I think is still an NFS problem.
Overall, this is a great project and saved me a lot of set up time. If I ever work with the enterprise edition of Magento, I could see where this box would be invaluable to setting up different caching layers and services. Thank you for putting this together and making it available to the community!

Hi. I've pretty much moved to use Docker for my Magento 2 projects as I've found it to be much faster.
I still think this is a great project though, and it's nice to have choices :)

Thanks everyone for the feedback!


Automation & DevOps section - Application delivery

Page section: Design Recommendations - App Installs in AVD Images

App-V is mentioned here, however MSIX AA is the direction of travel for app delivery.
Also refence to 3rd parties is outside of ESLZ design principles.

We need to:

Add or change guidance for App-V to MSIX AA to align with roadmap etc.
Remove references to 3rd parties.

Concur with removing third party options if native ones are available.
MSIX app attach is our recommendation, though not all applications are compatible so FSLogix application masking could be an option to manage user access.

Migrated to ADO


copy_and_edit.py to replace a command in a file?

Dear all,
Coming from this [1] discussion, how can I apply the plugin copy_and_edit.py [2] to replace all instances of lqrelease-qt5 with lqrelease in a specfic build file of a package [3] that I want to create using Snap?
Any help would be very appreciated!
Thankfully,
~Robert
[1] https://askubuntu.com/questions/794875/snapcraft-autoreconf-configure-ac-or-configure-in-is-required/794979?noredirect=1#comment1197730_794979
[2] https://github.com/ubuntu/snappy-playpen/blob/master/ubuntukylin-icon-theme/parts/plugins/copy_and_edit.py
[3] https://gist.github.com/orschiro/489710764173788d07c00b5a84ddaafe

@fcole90 can you maybe help out here?

Hi, the plugin was actually created by @caldav. You can see an example here: https://gist.github.com/caldav/9f0978ef0aa4cda6d4cdf92fced1e99b
I think something like this should work:
plugin: copy_and_edit
        source: https://github.com/vkohaupt/vokoscreen/archive/2.5.0.tar.gz
        source-type: git
        files:
            #src/* : dest/*
        edit:
            #src/my-build-file:
                "lqrelease-qt5": "lqrelease"

Thank you, @fcole90!
I am not sure if I understood your comment correctly but this is what I tried:
name: vokoscreen
version: "2.5.0"
summary: Vokoscreen Screencast Software
description: Snap package for Vokoscreen Screencast Software
  https://gist.github.com/orschiro/489710764173788d07c00b5a84ddaafe
  Currently, you have to manually replace all instances of
  lrelease-qt5 with lrelease in parts/vokoscreen/src/vokoscreen.pro
confinement: strict

apps:
  vokoscreen:
    command: vokoscreen

parts:
  vokoscreen:
    plugin: qmake
      qt-version: qt5
      source: https://github.com/vkohaupt/vokoscreen/archive/2.5.0.tar.gz
    plugin: copy_and_edit
        source: https://github.com/vkohaupt/vokoscreen/archive/2.5.0.tar.gz
        source-type: git
        files:
            #src/* : dest/*
        edit:
            #src/my-build-file:
                "lqrelease-qt5": "lqrelease"

Resulting in the following error:
Issues while validating snapcraft.yaml: mapping values are not allowed here on line 16 of snapcraft.yam

I haven't tried the above, but I think "#src/my-build-file:" is interpreted as a comment and ignored.

Dear @caldav,
Good idea! I removed the # from both lines but the same error still occurs.
Any other ideas?
~Robert

@orschiro have you had the chance to look further into the last issue you had? Another option that might get you more help is to post the question on the snapcraft mailing list.

Dear David,
I haven't yet had the chance to dig into this issue further. Currently, I
don't have time either to investigate this. Maybe someone will continue the
work based on the latest gist file [1].
Warmly,
~Robert
[1] https://gist.github.com/orschiro/489710764173788d07c00b5a84ddaafe
On Jul 14 2016, at 12:16 pm, David Planella<EMAIL_ADDRESS>wrote:

@orschiro have you had the chance to look
further into the last issue you had? Another option that might get you more
help is to post the question on the snapcraft mailing list.

—
You are receiving this because you were mentioned.
Reply to this email directly, [view it on GitHub](https://github.com/ubuntu
/snappy-playpen/issues/148#issuecomment-232624804), or mute the
thread.


Migrated to Swift Package

Converted to a Package so it can be used as a dependency with the Swift Package Manager.

Let's ignore Travis. Something to sort for another day.


File is not released

IPresentation presentation = SCPresentation.Open(HostingEnvironment.MapPath("~/Template/Education_Template.pptx"), isEditable: true);

presentation.Close();
presentation.Dispose();

after close or dispose method the file opened is not released.

Hi. Thank you for the report!

... is not released

You mean the file is not accessed? For example, in the code below, the closed file is successfully deleted:
const string filePath = @"c:\temp\test.pptx";
IPresentation presentation = SCPresentation.Open(filePath, true);
presentation.Close();

File.Delete(filePath);

No, I mean if the same method is called again. Exception is thrown, the file trying to open is already open.

Was able to reproduce! Will fix.
Many thanks.

Thanks

The bug has been fixed in the v0.21.1 version.
Please, feel free to re-open the issue if this still does not work correctly.

Thanks for the quick fix. I have tested it and is working as expected.

Hello @tboulis,
could you please provide a sample of how you add table.
It works for me, for example, when I do this in the following way:
var pres = new Presentation();
pres.Slides[0].Shapes.AddTable(10, 10, 2, 2);

pres.SaveAs("result.pptx");


GNOME buttons incorrectly styled when running Stylix as NixOS module.

I currently use Stylix as below, for my GNOME system. There is an oddity when enabled, in that certain buttons (namely the power options shown in the image) are incorrectly styled and hang off the screen.
With this code, I see the issue:
    # Apply Stylix styling to this system for colour scheming
    # and wallpaper. This applies globally to the system.
    stylix.enable = true;
    stylix.autoEnable = false;
    stylix.image = ../../../../assets/wallpapers/monokai.jpeg;
    stylix.imageScalingMode = "fill";
    stylix.polarity = "dark";
    stylix.base16Scheme = "${pkgs.base16-schemes}/share/themes/rose-pine.yaml";
    # Set the system cursor type. I like Bibata Modern Classic.
    stylix.cursor = {
      package = pkgs.bibata-cursors;
      name = "Bibata-Modern-Classic";
      size = 24;
    };

When I set autoEnable = false; so that Stylix applies no theming, I get the expected behaviour, which looks like this:

This is using the release-24.11 branch. Both physical and virtual machines experience this issue.

This was fixed for the master branch in #658. We should be able to backport this to release-24.11.

This was fixed for the master branch in #658. We should be able to backport this to release-24.11.

The master branch seems to have a different issue - if I used the code above for GNOME, I get the error:
error: The option virtualisation.vmVariant.home-manager.users.username.programs.kubecolor' does not exist. Definition values:`
I've tried manually disabling the target for kubecolor but it doesn't seem to help.

The master branch seems to have a different issue - if I used the code above for GNOME, I get the error:
error: The option virtualisation.vmVariant.home-manager.users.username.programs.kubecolor' does not exist. Definition values:`
I've tried manually disabling the target for kubecolor but it doesn't seem to help.

IIRC, the kubecolor module is not available on release-24.11. Consider manually disabling the module: https://github.com/danth/stylix/commit/9b61cc39b2c82f01b63bb0ae85865d7372697fc4.

This was backported in #670


Empty Values

Describe the bug
Joining a pool with empty values sends you back to the dashboard without a message or error.
To Reproduce
Steps to reproduce the behavior:

Click join
Do not enter any values
Submit

Expected behavior
Should stay on screen with an error message, or perhaps the pools page should default to 0 on both fields instead of empty fields
Screenshots
https://user-images.githubusercontent.com/39380100/103900998-6f067780-50f0-11eb-8d15-a73ffe288486.mov
Desktop (please complete the following information):

OS: MacOS 10.14.6
Browser Chrome and Firefox
Version Various

Additional context
Connected via Metamask
Wallet address
0xD4E5e3e9f936a59568C3A7c6159c5cFCE328D496

https://github.com/The-Poolz/Poolz-Back/issues/35
thanks for the report.

https://github.com/The-Poolz/Poolz-Back/issues/35
thanks for the report.


chore: test bundle code rather than all of ipfs

its very mock heavy, but our code depends on all the possible
ways of connecting to ipfs; our tests become horrible without some
judicious mocking out of dependencies.
It means we can now focus our tests on the logic of this module
rather than elaborate ipfs init tests. Also our tests no longer
fail if you happen to be running an ipfs daemon which is nice.
Also adds option to try accessing ipfs directly from companion
seperate from window.ipfs, and will try js-ipfs-api on the current
origin before trying the default value
License: MIT
Signed-off-by: Oli Evans<EMAIL_ADDRESS>

Fixes #4 as it now uses an ipfs.get with the empty dir as the connection check by default, and calling code can change it as required.

More try/catching to do...


Document updates not working when extending another document

I have the following setup :
A document BoosterTerm extending document Term, and every BoosterTerm can have a BoosterTerm as a parent.
Now the issue is when I get $boosterTerm->getParent() and update a value on it and then flush the value is not saved in the DB, there is no exception, no error, the values are simply not saved in the DB.
Here are some code examples :
Acme\Bundle\Document\Term:
    repositoryClass: Acme\Bundle\Repository\TermRepository
    collection: terms
    type: document
    inheritanceType: COLLECTION_PER_CLASS
    fields:
        id:
            id: true
        name:
            type: string
        parent:
            reference: true
            type: one
            nullable: true
            targetDocument: Acme\Bundle\Document\Term

Acme\Bundle\Document\BoosterTerm:
    repositoryClass: Acme\Bundle\Repository\BoosterTermRepository
    collection: booster_terms
    type: document
    fields:
        id:
            id: true
        creation:
            type: boolean
        parent:
            reference: true
            type: one
            targetDocument: Acme\Bundle\Document\BoosterTerm
            inversedBy: children
        children:
            reference: true
            type: many
            targetDocument: Acme\Bundle\Document\BoosterTerm
            mappedBy: parent

<?php
class BoosterTerm extends Term

After digging a while I think I found what the issue is, but not sure where the source of the issue is.
In Doctrine/ODM/MongoDB/UnitOfWork.php in function executeUpdates on line 1186 in the master-dev version.
if (get_class($document) == $className || $document instanceof Proxy && $document instanceof $className) {

evaluates as true for a BoosterTerm document when $className is actually Acme\Bundle\Document\Term and it continues until Doctrine/MongoDB/Collection.php line 1243 :
return $this->mongoCollection->update($query, $newObj, $options);

but after that line is executed nothing happens in the DB.
I've made it work only after I removed the Acme\Bundle\Document\Term from the $commitOrder array in UnitOfWork.
So there migth be three sources of the issue :

Acme\Bundle\Document\Term should not appear in $commitOrder as no Document with this type has been modified.
the order in the $commitOrder is not ok as Acme\Bundle\Document\Term appears before Acme\Bundle\Document\BoosterTerm
line 1186 in UnitOfWork should match only the instance of the child class and not the parent class also.

I hope this really is an issue and you can understand what is happening.
I would like to work on this issue if somebody would guide me a little between the 3 above solutions.

@lenardpalko would you mind creating failing test case?

CommitOrder is removed for some time now, but if the issue still exists please feel free to submit a failing test case


[tui] editor component - write lots of tests for new features that got added

Add tests for

[ ] select,
[ ] copy,
[ ] cut,
[ ] paste,
[ ] delete,
[ ] undo,
[ ] redo

Add tests for

[ ]  select,
[ ]  copy,
[ ]  cut,
[ ]  paste,
[x]  delete,
[ ]  undo,
[ ]  redo

Running status of testcases.

Running Status for testcases :

[ ]  select,
[ ]  copy,
[ ]  cut,
[ ]  paste,
[x]  delete,
[x]  backspace,
[ ]  undo,
[ ]  redo

Running status for Test-cases :

[ ]  select
[ ]  copy
[ ]  cut
[ ]  paste
[ ]  delete
[ ] backspace
[ ]  undo
[ ]  redo


ClusterLoader2 - Steady State Tests

What would you like to be added:
I want to run "steady-state" tests on kubernetes clusters to be able to compare regressions in clusters in context of different failure modes.
Some examples:

Create deployments/secrets/configmaps for 6 hours
Delete deployments/secrets/configmaps for 6 hours
List deployments/secrets/configmaps for 6 hours

Obviously it wouldn't be wise to just create a resource for 6 hours with out a corresponding delete, so users will have craft their configs accordingly.
Why is this needed:
It's difficult to test performance regression in conjunction with other infrastructure events. For example, comparing performance in a cluster during cluster upgrades. Say I run a load test during a 1.21 -> 1.22 upgrade on 2 clusters, and 1 cluster sees a lot more 5XX apiserver errors during the upgrade, I can't confidently say that that cluster has worse performance. The worse performance could've just been a lucky alignment of a particular step in the clusterlaoder test and a step in the cluster update.
Why is this needed:

/assign @marseel

Added a quick PR that highlights what I'm personally looking for. Feel free to use it as inspiration, but I'm not attached to it in anyway and open to other approaches.

I'm leaving it to Marcel, but my 2cents is that what you're asking for is kind of against the CL2 test API. The current idea behind the API is that you provide a "state the cluster should be in", via the phase step, and CL2 brings cluster to that state. IIUC what you're asking for, you'd like to have CL2 to execute some sequence of operations on the cluster for a provided time. This use-case might be valid, but if we want to support it, we should make it in a way that will keep our API consistent.
Just thinking out loud, but maybe instead of playing with tuning sets we should provide an option to execute given step or module N times. Then you could execute the "scale up/down" module (that @shyamjvs mentioned above) for M hours and it would generate a steady churn on the cluster. Would it work for you?

I agree with Matt here.
I think that what we're missing is exactly what @mm4tt described above, which is in the Phase API add some "repeat config" (we need a better name though):
https://github.com/kubernetes/perf-tests/blob/master/clusterloader2/api/types.go#L104
That would allow us to repeat certain things N times, for N hours or sth like that.


Make sure there is a readme in every major directory

Ensure that we have a readme for every major directory


New benchmark harness

This adds harness.py, a new harness for the entire benchmark workflow.
Its key features are:

Integrating setup steps directly into the harness itself, using a dedicated subcommand
Lots of error checking throughout, to detect misues/misconfiguration and produce meaningful errors
A mode for comparing the performance of two wasmtime revision against each other
Support for executing benchmarks that are entirely defined within a single .wat file (used for simple micro-benchmarks)
Deterministic execution: Invoking ./harness.py with the same options at the same revision of the benchfx repository (and therefore the same definitions in config.py) uniquely determines the exact versions of all dependencies as well as the flags they are built and run with.

I've updated README.md to document the new harness.
For the existing "macro" benchmarks, the harness re-uses make.generic.config for creating the merged .wasm files, by overriding certain values in make.config with values provided on the command line.

Once this is landed it would be easy to integrate it into the CI for the wasmfxtime repo. This way we could at least check that the benchmarks still work, or even better: Report benchmark results and/or fail if a PR slows down things by more than some threshold.


When has team route, remove leading slash

@srkirkland when it isn't using the team route, I left the leading slash in front of the api route.


Responsive UI Managed Metadata Picker Dialog Does Not Work

After installing the PnP Responsive UI to a classic SharrePoint Online site, the managed metadata picker dialog no longer shows the choices from the term set.
PnPResponsiveUI Disabled

PnPResponsiveUI Enabled

@VesaJuvonen could you set Responsive Label plz ?


Safety Manual added

Fixes: #230
Description:
Added Women's Safety tips with all required changes.
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] My changes generate no new warnings.

Kindly let me know if there are any further changes to be made. I'll make
all of them at the same time instead of doing minor changes repeatedly.
On Mon, May 24, 2021, 4:17 PM Swapnil Srivastava @.***>
wrote:

@.**** requested changes on this pull request.
Few changes:

Change the heading image with some good image.
Remove the png from the heading that is there at the last.
Make a Question as a heading and remove the question from all the
headings.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/swapnilsparsh/Rescue/pull/334#pullrequestreview-666618553,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANPN7TW3U5EG4VOXFGY6XFDTPIVEHANCNFSM45LZFZEQ
.

Sir, a few images that can be used for the heading are in the drive link below :
https://drive.google.com/drive/folders/1ZtP6O0j2St_jrw8gdUKLRZDDhdscbQcB?usp=sharing
Kindly go through the same and let me know which one to use .

Sir, a few images that can be used for the heading are in the drive link below :
https://drive.google.com/drive/folders/1ZtP6O0j2St_jrw8gdUKLRZDDhdscbQcB?usp=sharing
Kindly go through the same and let me know which one to use .

2nd photo without bg.


Remove node_modules from github repo

done, https://github.com/DutchmanNL/ioBroker.discovergy/commit/f0ca33bdeaad227b45a0d004a1b6e0bf01751b71


Fix product templates

In this i am fixing the product template as said
and all templates should be in the base DIR/template.

fixes #83

You're not actually explaining why you're doing this change. You're only saying that you said it somewhere (without saying where or what). I approve it as I trust that you know what you want to achieve with that, but please add some details.

Roger that,i will update the description!


Add a note to install-w-machine with pointers if hosted discovery is down

Unsticks beginners, addressing issue https://github.com/docker/swarm/issues/1503.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "docs-what-if-hosted-discovery-down" git@github.com:ahaurw01/swarm.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

The command "go test -v -race ./..." exited with 1.
...
Discovery service returned 500 HTTP status code

:laughing: :trollface:

ping @moxiegirl (this note might be useful for docker-machine users)

@ahaurw01 can you please update to use @moxiegirl's comment

@ahaurw01 ping - thanks!

Sorry, this fell off my radar. I'll update this weekend. Thanks for the reminder @amitshukla.

Updated.

Thank you @ahaurw01
LGTM
/cc @moxiegirl @vieux

LGTM

Thanks!


chore: build contracts and protocol circuits sequentially if not enough ram

#5426 but for building noir projects.

Benchmark results
Metrics with a significant change:

note_trial_decrypting_time_in_ms (32): 115 (+243%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit 4064e180 and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
676
676
676

l1_rollup_calldata_gas
6,400
6,412
6,412

l1_rollup_execution_gas
585,733
585,745
585,745

l2_block_processing_time_in_ms
1,315 (-1%)
4,842 (+3%)
9,125 (+1%)

note_successful_decrypting_time_in_ms
197 (+21%)
489 (-5%)
941 (-4%)

note_trial_decrypting_time_in_ms
61.7 (-9%)
:warning: 115 (+243%)
72.9 (-34%)

l2_block_building_time_in_ms
12,142 (+3%)
44,549 (+3%)
87,780 (+3%)

l2_block_rollup_simulation_time_in_ms
7,434 (+3%)
26,078 (+4%)
50,791 (+3%)

l2_block_public_tx_process_time_in_ms
4,688 (+3%)
18,423 (+3%)
36,893 (+2%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
14,585 (-2%)
26,729 (-5%)

note_history_successful_decrypting_time_in_ms
1,208 (-6%)
2,365 (-4%)

note_history_trial_decrypting_time_in_ms
65.9 (-50%)
121 (-18%)

node_database_size_in_bytes
18,636,880
35,070,032

pxe_database_size_in_bytes
29,859
59,414

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
184 (-1%)
44,379
28,214

private-kernel-ordering
169 (-1%)
52,880
14,296

base-parity
3,519 (+2%)
128
311

root-parity
1,234 (-2%)
1,244
311

base-rollup
15,010 (+2%)
165,760
861

root-rollup
49.3
4,359
725

private-kernel-inner
229 (-1%)
73,795
28,214

public-kernel-app-logic
95.9 (+2%)
35,252
28,217

public-kernel-tail
138 (+1%)
40,928
28,217

merge-rollup
8.56 (-6%)
2,568
861

public-kernel-teardown
92.5 (+2%)
35,252
28,217

public-kernel-setup
91.7 (+2%)
35,252
28,217

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
512 leaves
1024 leaves
2048 leaves
4096 leaves
32 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.0 (+1%)
16.1 (+2%)
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.6
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.583 (+1%)
0.498 (+2%)
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
46.4 (+1%)
71.8 (+2%)
230 (+3%)
445
872 (+4%)
1,717 (+3%)
N/A

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
96.0
159
543
1,055
2,079
4,127
N/A

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.476 (+1%)
0.443 (+2%)
0.419 (+3%)
0.416
0.415 (+4%)
0.412 (+4%)
N/A

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
54.0 (-1%)
106 (+1%)
340 (+3%)
666 (+1%)
1,305 (+3%)
2,599 (+3%)
N/A

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
105
207
691
1,363
2,707
5,395
N/A

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.477 (-1%)
0.480 (+1%)
0.461 (+3%)
0.458 (+1%)
0.454 (+3%)
0.454 (+4%)
N/A

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
61.0 (+1%)

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
109

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
0.533 (+1%)

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
14,968
495,320

Transaction size based on fee payment method

Metric
none fee payment method
native fee payment method
fpc_public fee payment method
fpc_private fee payment method

tx_with_fee_size_in_bytes
773
905
1,161
1,377

Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes
2 new note hashes

tx_pxe_processing_time_ms
1,789 (+3%)
1,198 (-1%)
4,440 (+2%)

Metric
0 public data writes
1 public data writes
2 public data writes
3 public data writes
4 public data writes
5 public data writes
8 public data writes

tx_sequencer_processing_time_ms
12.2 (+24%)
578 (+1%)
449 (+4%)
1,178 (+3%)
489 (+3%)
2,043 (+3%)
563 (+3%)

Setting the env var RAYON_NUM_THREADS to a lower value would help with this. The trouble is that nargo is trying to compile too many projects at once for your system so if we force it to use less parallelism then we should avoid it.
Long term fix is nargo being smarter about how it builds larger workspaces so it doesn't run out of RAM.


Integration workflow update to support all-in-one adapter testing

resolves #
Problem
When testing changes to adapters, particularly dbt-adapters changes but also some core changes, we need to execute integration tests on all of the adapters, with the ability to specify dbt-adapters branch, dbt-core branch, and sometimes the local adapter branch.
Solution
Add additional input fields for the "workflow_dispatch" event: "dbt_adapter_branch", and "name".   The name helps to identify individual runs in the lists of workflow runs, which can get lost. Also there's no good way to connect up a rest api workflow run or a gh cli workflow run with the matching run result, so adding a distinctive name assists in finding the correct results.
Checklist

[ ] I have read the contributing guide and understand what's expected of me
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX

Pull requests in other repos:
https://github.com/dbt-labs/dbt-postgres/pull/133
https://github.com/dbt-labs/dbt-bigquery/pull/1309
https://github.com/dbt-labs/dbt-redshift/pull/889
https://github.com/dbt-labs/dbt-spark/pull/1082


M04-LAB01: KQL queries

Contact Details
<EMAIL_ADDRESS>What should we change?
Since class inception, the aka.ms/lademo site has a SecurityAlert table, but it always has been and still is empty.
So why do we continue to query it?
So that leaves
Task 2 steps 8, 9, 11
Task 6 steps 6, 7,
as producing no output.  This can be verified by running
``KQL
union Security*
| summarize count() by Type
And take a look at the output.  there is no row returned with a table name of SecurityAlert.
Please update the md file to include the phrase The following will not produce results like we had in an older version of the md file.
Or modify those steps to include working examples.
Lab
Lab 04 Exercise 01 Create queries for Microsoft Sentinel using Kusto Query Language (KQL)
Relevant screenshots
paste here 😉

I will test this to confirm.

Thank you  @MSFT-MarcoEs for fixing the KQL queries lookback time to 90 days for SecurityAlert table.  The 7 day lookback is not long enough.  Many thanks!


merge working version from valentijnscholten

Not sure what works and doesn't work, but as discussed in the meeting we're going to merge my version into the official repo as it seems to be the version that works best with apiv2.

We're using your fork right now, so I would not know either, but it surely is probably the better one for v2, as you say. Let's merge and see if some others can even contribute!


Adding google scholar field and 2 other

Changes proposed in this pull request:- Added Google scholar and other field in profile page

-added 3 input field for google scholar , linkedin , github.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #232   +/-   ##
=======================================
  Coverage   51.73%   51.73%
=======================================
  Files          66       66
  Lines        3659     3659
  Branches      413      413
=======================================
  Hits         1893     1893
  Misses       1671     1671
  Partials       95       95

Impacted Files
Coverage Δ

src/app/components/profile/profile.component.ts
51.25% <ø> (ø)
:arrow_up:

Impacted Files
Coverage Δ

src/app/components/profile/profile.component.ts
51.25% <ø> (ø)
:arrow_up:

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 b03bdc4...f107c3c. Read the comment docs.

I fix the indent issue and removed unnecessary lines

i am opening new pr


Utilize Mesh Surfaces

Make use of Mesh Surfaces, in VoxelObjects, so as too add/remove/modify mesh directly via operations, as opposed to the recreating of the complete mesh on any single operation.

surfaces going to be used in #11

thinking of using surfaces for #21


fix chown: invalid user: ‘bot’

use root in docker
Fix unbuild fail,
chown: invalid user: ‘bot’

Thanks!


Using Backbone.Model.set with "silent" option doesn't clear out previous attributes.

From the 0.9.0 documentation it appears this is by design, but it gives an odd behavior. Here's an example:
var Foo = Backbone.Model.extend();
var instance = new Foo();
instance.bind('change', function (model, value) {
console.log(model.changedAttributes());
});
instance.set({ bar: 1 }, { silent: true });
instance.set({ baz: 2 });
// I get { bar: 1, baz: 2 } logged.
// I'd expect { baz: 2 }
I'd think of this as "staging" a change since that's really what it's doing. If I want to change a model property, not trigger any change events, but also not trigger change events in the future, what's the recommended way of doing that? Can I do that without creating a new model instance?

@bialecki see every issue I've raised over the past two weeks for more information on "why" they've repurposed the silent option for completely different functionality.
@dnathe4th I really like this idea. A Compat model for compatibility for any project which uses the old form of silent so functionality actually works in the same way as the previous, and this way only the module would have to be updated along with Backbone rather than a whole extra fork.

#981 contains a pull request for a simple replication of the old way silent used to work as well as a Gist with the extended model version that @dnathe4th suggested :)

@bialecki If you really have to do this you can use the methods suggested by @dnathe4th above.
From the change documentation:

Manually trigger the "change" event and a "change:attribute" event for each attribute that has changed. If you've been passing {silent: true} to the set function in order to aggregate rapid changes to a model, you'll want to call model.change() when you're all finished.

The behavior you mention is by design and is not new to Backbone 0.9.x.  Model attributes are meant to fire events when changed.  If you don't want to be notified of these events just don't observe them.

Thanks for the feedback guys. I know it's an intentional behavior and, as it turns out, it's not an issue for what I'm doing now so "issue resolved." :)
I think I'm used to the behavior I mentioned from other JS frameworks (YUI I what comes to mind), where passing silent on a model setter, caused the value to be set, but didn't notify listeners. Obviously, Backbone can (and may have good reason) to go a different direction.
I don't know where this request belongs, but there's nothing in the docs about why I'd want to use the silent option, so I'm now having a hard time thinking of a situation when I'd want to use it. Are there use cases demonstrating when I'd use this somewhere?

@bialecki You're right, I believe silent is a bit of a loaded term in Backbone.  Some clarification in the documentation for set is probably in order.  The docs for change hint at it but don't really explain the reasons for using silent.


::Rake.Application.last_comment needs to be removed

.bundle/ruby/2.1.0/gems/rspec-core-2.99.2/lib/rspec/core/rake_task.rb:143:        desc "Run RSpec code examples" unless ::Rake.application.last_comment

 bundle exec rake lint
rake aborted!
NoMethodError: undefined method `last_comment' for #<Rake::Application:0x00000000dbffb0>
/var/lib/jenkins/jobs/test/workspace/profile_base/.bundle/ruby/2.1.0/gems/rspec-core-2.99.2/lib/rspec/core/rake_task.rb:143:in `initialize'
/var/lib/jenkins/jobs/test/workspace/profile_base/.bundle/ruby/2.1.0/gems/puppetlabs_spec_helper-1.1.1/lib/puppetlabs_spec_helper/rake_tasks.rb:15:in `new'
/var/lib/jenkins/jobs/test/workspace/profile_base/.bundle/ruby/2.1.0/gems/puppetlabs_spec_helper-1.1.1/lib/puppetlabs_spec_helper/rake_tasks.rb:15:in `<top (required)>'
/var/lib/jenkins/jobs/test/workspace/profile_base/Rakefile:2:in `require'
/var/lib/jenkins/jobs/test/workspace/profile_base/Rakefile:2:in `<top (required)>'
(See full trace by running task with --trace)

rake 10.5.0 has alias for last_comment to last_description but this was removed in 11.0
.bundle/ruby/2.1.0/gems/rake-10.5.0/lib/rake/task_manager.rb:10:    alias :last_comment :last_description # :nodoc: Backwards compatibility

@cdenneen RSpec 2.99 was only released to help users upgrade to RSpec 3, please refer to this PR https://github.com/rspec/rspec-core/pull/2202 for more information.

I'm seeing this on rspec-core-3.5.0.beta1

Looks like this is fixed in 8e723fc805e901ac4fa5483837138b175d411d6e

Any chance of getting a 3.5.0.beta2 released for people running the rspec beta alongside rails betas?

master will always be the most recent version but I've just released a 3.5.0.beta2

what I did was added the gem 'rake', '11.1.0', and then updated rake "bundle update rake"


Moved call up to OptimizationPass

This is a good change, but I think we should delete OptimizationPass instead of modifying it. It was a mistake for me to add it in the first place, because it's nothing but a definition of a particular kind of Callable (and not even a complicated one).

I opened up an issue (#1241) to remove it; could we merge this in the meantime?

Automerge done.


Rspec Issue:  Missing locales in one spec breaks another

I'm having a weird error with the rspec tests in my app related to the I18n library.   This is a Ruby app, but not Rails.   I18n is version 0.6.1
If one of my tests calls I18n.t without first setting the load_path, it appears to somehow mess up subsequent tests that do actually set the load path.   To illustrate this failure in a very simple manner, I created two specs that don't even use any application code:
(just showing the specs for brevity; can provide the full source on request)
--test1.rb--
 it "should fail if a translation is not found" do
      puts "Translation fail"
      I18n.t('test').should eq "translation missing: en.test"
 end

--test2.rb--
it "should translate hello to english" do
  create_locale file_for_testing()
  I18n.load_path << Dir.pwd + "/en.yml"
  puts "Translate English"
  I18n.locale = :en
  I18n.t("hello").should eq "Hello world"
end

If I adjust the file names so that the "English" translation test runs first, everything's fine:
Translate English
.Translation fail
...
2 examples, 0 failures

If I adjust the file names such that the "fail" translation test runs first, it causes the "English" test to fail too!
Translation fail
.Translate English
F

Failures:

  1) foo should translate hello to english
     Failure/Error: I18n.t("hello").should eq "Hello world"

       expected: "Hello world"
            got: "translation missing: en.hello"
   ...
    2 examples, 1 failure

Clearly the first spec is somehow messing things up for the second spec.  But what?  And why?

Hi there,
It's been a while, but I don't suppose anyone has an app which reproduces this issue? That'd help tremendously in debugging this.
Thanks!

@radar    It's not an app, but here's a gist that illustrates the issue.
Just create the three files and run rspec.
rspec spec.rb works - both tests pass
rspec spec_failed.rb illustrates the problem - both tests fail
The only difference between the two spec files is the order of the tests.

Thanks @lynnfaraday!
I've been able to investigate this today and I've discovered a solution. However, it doesn't work with I18n.load_path <<, only I18n.load_path +=. I think to make it work with the former we'd need to re-define the << method on load_path, which may be a bit hacky. tbh I'd rather advise people to use I18n.load_path +=.

☝  there's a PR which will fix your issue. Here's an updated Gist of the code I used to verify this patch: https://gist.github.com/radar/c74cadff6dee7988f764536d667f44df

Just merged #348 which fixes this. Thanks @lynnfaraday!


category error

if a specific category is in d1 but not in d2, in the previous code the category wad deleted. Now we keep all the categories in d1 and d2

Thank you for your PR and finding my tool helpful.
Actually, the code is meant to merge only 2 files with same categories.
As I included in my updated README: "The reason I didn't mix categories, incase they are different, is to help annotators identifying any change in there categories."
In addition, the way you append the categories is not enough. You still need to check for duplicate category id. If that is the case, you need to update that to new category id and update the annotations of that file accordingly. Plus, Imagine a file containing Dog category as id 1 in first file and as id 15 in the second file. You'll then need to check also for "name" value in each category item so that you won't have duplicates.


Error: require GNU grep. Aborting.

This is the error i am getting while running the first command when testing
./sudomy -d hackerone.com
Error: require GNU grep. Aborting.

Hi Guys , i resolved this issue , by making a very tiny change in the code. GREP is working fine, but we need GNU GREP so i just added a "g" in the if condition of the code  where it is checking for grep and now the tool is running as expected.
so now it is ggrep.
Thanks


TRAVIS use neurodebian on Ubuntu 14.04

in order to have a numpy version that satisfies nistats requirements. Fix #105.
Instead of listing the build matrix in the env/matrix section, this uses the matrix/include section. This is the way shown in the Travis doc and this allows to run on Ubuntu 14.04 containers rather than the default 12.04 containers.

I'll wait for the build to be green and merge this.

I enabled CircleCI on PRs as well since it was not enabled.

Everything was green, so merged.

Awesome @lesteve, thanks for fixing this!

Thx !


Chore: maintains dependencies, fixes new lint rules

Coverage remained the same at 100.0% when pulling d2a87f188d8f906776392dc280003421a63dbe87 on chore-maintain-deps into 4ad335f2401d7a919176039bde76d9155fe5e58a on master.


06c Retrieving variables

Testing
Let's see how our program handles variables being used in place of values:
https://github.com/nyjc-computing/pseudo/blob/ecc694bbfabc210fd9c85bb336278320f9061780/main.py#L10-L14
Result:
Index
{'Index': {'type': 'INTEGER', 'value': 1}}

This is because evaluate() handles name tokens by returning their word value:
https://github.com/nyjc-computing/pseudo/blob/ecc694bbfabc210fd9c85bb336278320f9061780/interpreter.py#L5-L9

Get expressions
We need a new kind of expr: one that says "please look up this word in a frame, and get its assigned value". Let's call it a 'get' expr.
Handcrafting a get expr
So far we have been treating exprs as binary expressions, which involve a left expression/token, an oper operator, and a right expression/token. Let's see if we can continue this pattern so we don't have to invent a new kind of expr.
A get involves:

getting a frame (which contains mappings of names to values),
getting a name,
getting the value associated with the name from the frame

That's promising; get lends itself well to being a binary operator. We can try to define it like get(frame, name):
https://github.com/nyjc-computing/pseudo/blob/99b4d015362d2de1463f0ad04cc8650f0e0ca873/builtin.py#L62-L63

Parsing names
We add a new parser for names, which we call identifier() because we are using name as a variable name:
https://github.com/nyjc-computing/pseudo/blob/e15fb417c475fe94bc1debeb0bdbbe35fb373177/parser.py#L31-L36

Our declare and assignment statements also need updating so they get a name token and not a get expr for the first token:
[737a3e4137844f3045ae26dbb9b972c587363f5d]

Get parsing precedence
Where should we slot this 'get' expr in recursive descent parsing?
We are likely to be writing code like i <- i + 1, and want i to be treated as its assigned value rather than a string or keyword, so that means it should have the same precedence as value() (where we capture integers, reals, and strings too).
[26a13acbb321495649eb7dcc8581a952053a412b]

Frame insertion
For now we will insert the frame in the interpreter, when we evaluate() a get expr:
https://github.com/nyjc-computing/pseudo/blob/7835fc47de7d1ee83a6326a8d515bcdcf03778b3/interpreter.py#L11-L18

Don't forget to update the evaluate() calls to add the frame argument:
[bf07c087c1d2fa59222128c21dc54559f096819e]

Bug
It's not working:
  File "main.py", line 26, in main
    frame = interpreter.interpret(statements)
  File "interpreter.py", line 57, in interpret
    execute(frame, stmt)
  File "interpreter.py", line 47, in execute
    execOutput(frame, stmt)
  File "interpreter.py", line 23, in execOutput
    print(str(evaluate(expr)), end='')
  File "interpreter.py", line 19, in evaluate
    return oper(left, right)
  File "builtin.py", line 63, in get
    return frame[name]
TypeError: 'NoneType' object is not subscriptable

Suspect the line in evaluate() that handles get operator:
https://github.com/nyjc-computing/pseudo/blob/bf07c087c1d2fa59222128c21dc54559f096819e/interpreter.py#L13
With a breakpoint there:
> interpreter.py(15)evaluate()
-> left = frame  # <-- expr['left'] is None
(Pdb) expr
{'left': None, 'oper': {'type': 'symbol', 'word': '', 'value': <function get at 0x7f66262c99d0>}, 'right': {'type': 'name', 'word': 'Index', 'value': None}}
(Pdb)

'left' didn't receive a frame for a get expr.

Fix
Add frame into the evaluate() call in execOutput():
[f79b55564fd540674ce0033ba6069234cb375123]
Result:
{'type': 'INTEGER', 'value': 1}
{'Index': {'type': 'INTEGER', 'value': 1}}

Result:
1
{'Index': {'type': 'INTEGER', 'value': 1}}

Now our program can declare variables, assign values to variables, and subsequently retrieve (evaluate) them.

Refactor
We have an OPERATORS constant containing a list of operator symbols:
https://github.com/nyjc-computing/pseudo/blob/f609d3ab0dc29628c060820baf4776ac26bad510/builtin.py#L21-L24
And we have an operators dict mapping operator symbols to operator functions:
https://github.com/nyjc-computing/pseudo/blob/f609d3ab0dc29628c060820baf4776ac26bad510/builtin.py#L69-L80
We don't need both: [58e5ded13b83ea2471e905d554b2c29fdd3007d7]


Remove texts from Text Overview

Please include a feature to remove texts (that have been imported) from the text overview or the database, respectively.

This could be a useful feature. I just struggling to find a good position to place to delete button.
I think it should be in text page, since the user probably will check the text before deleting it. Having a delete button next to each text on the project page, is visually too disruptive.

You could also choose a design analogous to the import function. Put a delete icon next to the import and check buttons which could e.g. add check boxes next to the files as well as a delete button.

Thank is a good idea. Delete texts is no implemented: 81a5a29d0616a58ec57f43d815d0397dbe9e14d5
For now you can only delete a single text, but it is very easy and fast to delete a few texts in a row.


its very hard to get 3.2ETH from Goerli Test Network

maybe, its the last step and the hardest step to get 3.2ETH from Goerli Test Network。
I can only get 0.05ETH once a time... wow !!!
it means that I will request 64 times from https://goerli-faucet.slock.it/ page.
ti’s the hardest work...  😄

if someone keep enough ETH for Goerli Test Network .
please send 3.2ETH to me, thanks very much
0x35d415bF0E6bbf8e10ebA814e10325c46a2CF009

@netpi you can request it one go from the authenticated faucet
https://faucet.goerli.mudit.blog/

@sachayves
thanks, I have successfully deposited
https://goerli.etherscan.io/tx/0x636e2c271bd491dadf437f0cb3c48726c1963e23716ef91f4a330d1d2ebcd1f6


dart2js: Allow new String.fromEnvironment('foo')

main() {
  print(new String.fromEnvironment('hi'));
}
generates:
  main: function() {
    H.throwExpression(P.UnsupportedError$("String.fromEnvironment can only be used as a const constructor"));
  }
The VM does not have this restriction.
It is not always possible to contrive that String.fromEnvironment has a constant argument.
For example, https://code.google.com/p/dart/issues/detail?id=15593 would require the construction of the environment variable from the containing group.  This is not possible with 'const' since the parameters of a const constructor are not constants.

@vsmenon can this be closed as of const-update-2018? String.fromEnvironment can only be const now, right?

Yes.


Remove CartDataService from CheckoutPaymentService

QA: Checked that CartDataService is removed from CheckoutPaymentService and can go through checkout.


fix(iceberg): fix jni context class loader

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?

Try to resolve https://github.com/risingwavelabs/risingwave/issues/17296

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

I am thinking that shall we instead always set the context loader after we do attach_current_thread of jni?

I am thinking that shall we instead always set the context loader after we do attach_current_thread of jni?

It depends on whether the method needs to use ContextLoader. Currently, I only find debezium and iceberg catalog will use it.

I am thinking that shall we instead always set the context loader after we do attach_current_thread of jni?

It depends on whether the method needs to use ContextClassLoader. Currently, I only find debezium and iceberg catalog will use it.

Java libraries usually assume that the class loader is correctly set. So I think we'd better just always set the class loader when we do attach_current_thread to create a new JniEnv, so that we don't have to manually set the class loader one by one for these methods and can avoid similar exception thrown again when we use other java libraries in the future.

I am thinking that shall we instead always set the context loader after we do attach_current_thread of jni?

It depends on whether the method needs to use ContextClassLoader. Currently, I only find debezium and iceberg catalog will use it.

Java libraries usually assume that the class loader is correctly set. So I think we'd better just always set the class loader when we do attach_current_thread to create a new JniEnv, so that we don't have to manually set the class loader one by one for these methods and can avoid similar exception thrown again when we use other java libraries in the future.

Sounds good. Do you have any idea how to implement it?

I am thinking that shall we instead always set the context loader after we do attach_current_thread of jni?

It depends on whether the method needs to use ContextClassLoader. Currently, I only find debezium and iceberg catalog will use it.

Java libraries usually assume that the class loader is correctly set. So I think we'd better just always set the class loader when we do attach_current_thread to create a new JniEnv, so that we don't have to manually set the class loader one by one for these methods and can avoid similar exception thrown again when we use other java libraries in the future.

Sounds good. Do you have any idea how to implement it?

We can unify all our call to jvm.attach_current_thread() to a method like
fn new_jvm_env(jvm: &JavaVM) -> AttachGuard<'_> {
    let env = jvm.attach_current_thread();
    let thread = call_static_method!(env, {java.lang.Thread.currentThread()});
    let system_class_loader = call_static_method!(env, {java.lang.ClassLoader.getSystemClassLoader()});
    call_method!(env, {void setContextClassLoader(), thread, {system_class_loader}});
    env
}

@wenym1 I finished the refactoring, PTAL again.


More Diag functionality

This adds: Diag+Diag, Diag+Dense, Diag*Diag. Delta now uses Diag (but with vector storage, we still need to make a version of Diag where all of the elements are the same).

It looks like maybe something is messed up with the set up of Travis on Windows? Anyway, I will merge this.

We can take the windows build out of the .travis.yml file. It’s a new build type so maybe it’s buggy?


fix: restore node12 support (glob -> fast-glob)

fixes #659
Note that there are some edge-case differences between glob and fast-glob, but it's very unlikely that users of umzug will hit them.
See glob docs for an in-depth comparison: https://github.com/isaacs/node-glob?tab=readme-ov-file#comparison-to-other-javascript-glob-implementations
If there's anyone that needs the old glob behaviour, they can use glob themselves and pass an array of migrations to umzug. (see https://github.com/sequelize/umzug#direct-migrations-list)

CC @WikiRik any feedback on this change?

I don't have enough insights in the usages of umzug and the globs people use. So I can't say much on this.
It seems that there are well documented changes between the two libraries that people can refer to and the added benefit is that the indirect vulnerable dependency is resolved without dropping Node 12 support.

Agreed. I am working on a change to CI which will highlight that node 12/node 14 support is broken currently. I'll merge that first, which should make CI go red, then confirm that this change fixes it.


Authenticating with multiple subscriptions linked to your account

Hi,
When following this tutorial I encountered a problem with it. If you have access to multiple tenants and subscriptions it will try to generate create the config using ws = Workspace.from_config(). However it might be that it will select the wrong tenant and/or subscription which was the case for me, so instead of using this line of code I replaced this with the following which allows you to select your workspace id's by yourselves:
from azureml.core.authentication import InteractiveLoginAuthentication
interactive_auth = InteractiveLoginAuthentication(tenant_id="my-tenant-id")
ws = Workspace(subscription_id="my-subscription-id",
resource_group="my-ml-rg",
workspace_name="my-ml-workspace",
auth=interactive_auth)
I hope this can somehow get adjusted in the documentation to make it more clear for people with similar issues.
I found this solution at https://github.com/Azure/MachineLearningNotebooks/blob/master/how-to-use-azureml/manage-azureml-service/authentication-in-azureml/authentication-in-azureml.ipynb

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

ID: 23f9d033-9007-7b0a-d651-88d5d3846b2a
Version Independent ID: c6efa8cc-8594-698b-21eb-5e4d9ad24bbd
Content: Tutorial: Train your first Azure ML model in Python - Azure Machine Learning
Content Source: articles/machine-learning/tutorial-1st-experiment-sdk-train.md
Service: machine-learning
Sub-service: core
GitHub Login: @trevorbye
Microsoft Alias: trbye

@LorenzPensaert we will review your feedback and get back to you shortly. Thanks.

#assign:@trevorbye

@trevorbye can you please look into this feedback for doc enhancement? Thanks.

Hi @LorenzPensaert, this tutorial is a fairly basic getting-started experience, where we try not to include any extra info, or diverge down more-advanced paths. We have an article on authentication that goes into much more detail on these topics, including what you mentioned here about explicitly defined tenants. #please-close


Perform some file cleanup

@yurividal I went through and did some cleanup and abstracted the request block to an if at the end instead of it being duplicated in each if/else if.  Please test and then we can add this module to the core.

Thanks. Ill test this today and report back.

@krocheck just finished testing. Everything seems to be working just fine. I have updated the readme file too. feel free to merge module into core.


Add some form of CI for Windows

Just as the title says. There should be continuous testing on Windows x86_64 and arm64 so that releases don't just break with a BoringSSL bump, etc. Even just a smoke test would be extraordinarily helpful.

Thanks @gregcotten. I don't disagree with this in any particular way. Getting this CI setup is on the roadmap, but we're not immediately there. @FranzBusch is driving the CI adoption there.


Text input needs to be far more robust to different values

The text input needs error handling to stop errors and to flash up pointers to users on how to use the input text field.

Todays "Remote" push updated the input formatter to split based on any whitespace (includes tabs and spaces).
The nuclide input can be in any format and the formatter will pick up text as element. Still needs some testing before closing out.

Added error handling to the text parser. Should not throw user back to input screen with extra error messages if this fails.
This may well not be robust to incorrect nuclides. Will close this out in isse #21


Support optional count query param

https://github.com/cnguy/kayn/blob/4e6d0fa3a4385d2a00ccd87d00f5a158d160c84b/lib/Endpoints/TournamentStubEndpoint.js#L39
Tournament Stub endpoint allows you to create a number of tournament codes (upto 1000 Max)
This is also the same with the main tournament API
https://github.com/cnguy/kayn/blob/4e6d0fa3a4385d2a00ccd87d00f5a158d160c84b/lib/Endpoints/TournamentEndpoint.js#L41

Just realised this is probably support via .query({ count: <Int> })?

yes, tournament id is the only one mandatory so that is why it is already in the code @OllieJennings

Cheers


No way to show invisibles

My Atom "Show Invisibles" setting isn't respected with language-babel. Switching back to JavaScript makes invisibles work properly. I also can't seem to override the setting in my local styles.less.
I tried to dig into the source here a bit to figure out how this could be getting overridden, but didn't have much luck (and ran out of time), so I figured I'd file an issue in case it's a quick fix.
JavaScript

Babel ES6 JavaScript

Packages don't control invisibles and I have no issue with showing them. What does  Atom's config.cson look like?

Wow, I totally had an override buried in my config.cson for some reason. I have no idea how it got there. Possibly Nuclide related.
".js.jsx.source":
  editor:
    showInvisibles: false

Thanks for pointing me at that!


[RESTEASY-1310, JBEAP-2446] - ResponseObjectTest failing when using P…

…roxyBuilder
JIRA: https://issues.jboss.org/browse/RESTEASY-1310
JBEAP: https://issues.jboss.org/browse/JBEAP-2446

I've added a comment on  https://issues.jboss.org/browse/RESTEASY-1310.

Thanks, Martin. This is ready to merge.


Avatars only visible by self

Not sure if I'm doing it wrong™ but I only see the generic avatar of other users in a group chat, and I've been told by others that they do not see my avatar either.  Is this expected behavior, or is there a setting I missed, or is this a limitation of ejabberd somehow?

I did not set my own avatar in Kaiwa, it was fetched from prosody at login but I'm also having problem seeing avatars others then my own. We're running prosody with the mod_profile activated that has support for both PEP avatars and vcard.

Second time I logged in the avatars was there though.


question about make cert

Hello, there is an issue with the command line make cert. I am using a Mac, and the details of the problem are as follows. Could you please assist me with this issue? Thank you.
(base) % make
[  0%] Built target boringssl_prefix_symbols
[ 18%] Built target fipsmodule
[ 70%] Built target crypto
[ 78%] Built target ssl
[ 81%] Built target decrepit
[ 82%] Built target picotls-core
[ 82%] Built target picotls-openssl
[ 88%] Built target picotls-minicrypto
[ 96%] Built target picoquic-core
[ 97%] Built target picoquic-log
[ 98%] Built target spdlog
[100%] Built target quicr
[100%] Built target qclient
[100%] Built target qserver
(base) % make cert
make: *** No rule to make target `cert'.  Stop.
(base) % pwd
/Users/mac/Code/c/libquicr/libquicr/cmd/examples

@linsheng1 try running make cert from the root of the libquicr repo rather than `/cmd/examples/.

Thank you for your response, but it seems there are still some issues.
(base)  libquicr % make cert
make: *** No rule to make target `cert'.  Stop.
(base)  libquicr % pwd
/Users/mac/Code/c/libquicr/libquicr
(base)  libquicr % cd ..
(base)  libquicr % make cert
make: *** No rule to make target `cert'.  Stop.
(base)  libquicr % pwd
/Users/mac/Code/c/libquicr

I'm a bit confused by your double libquicr/libquicr folder structure. I've uploaded myself generating a build and certificate from scratch here if that's any clearer: https://asciinema.org/a/V8PgbN2dFRGrVxPZ5uT1Djs2r
You can also just run the command that's in the Makefile itself if that's easier:
openssl req -nodes -x509 -newkey rsa:2048 -days 365 \
        -subj "/C=US/ST=CA/L=San Jose/O=Cisco/CN=test.m10x.org" \
        -keyout <your_build_directory>/cmd/examples/server-key.pem -out <your_build_directory>/cmd/examples/server-cert.pem

@linsheng1 Oh, I've think I've just realized what it is. Have you done an in-source generation of the CMake build? That will overwrite our Makefile with the generated Makefile of the build, which that won't have our custom targets in (like cert). If that's the case, I would advise against ever doing in-source builds with CMake, they will pollute the source tree with build artifacts. For what it's worth, the CMake docs make the same reccomendation: https://cmake.org/cmake/help/book/mastering-cmake/chapter/Getting Started.html#directory-structure

This is the problem I encountered, and it has been resolved using the method you suggested. Thank you!


Bump grpc & netty version

What changes are proposed in this pull request?
Bump grpc & netty version
Why are the changes needed?

We want to try netty io_uring feature
The new aws client we are going to introduce relies on newer version of netty
Current grpc & netty versions do not match. (https://github.com/grpc/grpc-java/blob/master/SECURITY.md)

Does this PR introduce any user facing changes?
N/A

alluxio-bot, merge this please.

alluxio-bot, merge this please.


Initial S3 Setup

Adding packages, configuration, and notes about uploading files to S3 instead of local.

Add boto3 and django-storages packages (I copied the pyproject.toml from the docker-compose PR)
Serving static files and manage.py collectstatic are functional
Uploading individual media files with a django form, and then serving that file (such as the profile photo) is functional.

This PR does NOT cover:

Uploading, editing, and displaying files in unpublished projects
Moving files when a project is published
Serving and access control for published project files

No need to merge this in as the settings obviously conflict with the Dockerized config.

We decided to go with GCS, changes are implemented here: https://github.com/upsidelab/physionet-build/pull/11


*: rename NewApiStorageLessResource to NewApiResourceWithStorage

NewApiStoragelessResource actually allows you to pass in a custom storage instead of using the standard impl.
It is better to name it correctly. Also we can use the same func for API Resource with custom REST storage when needed.
/cc @pwittrock

I am going to add REST option for api resource, and change the template to use withStorage when REST option is given.

@xiang90 SGTM.  Thanks!


Update Podspec sources to different platforms

Just updated podfile to drop unnecessary source files for particular platform

@AYastrebov Looks nice! Thanks.


Add missing scopes to security.md

Coverage remained the same at 90.917% when pulling cb57e5c310cdd7c505c2551d0bcc895d33386693 on nmaslarski:patch-1 into e2c9a1f8416b72fd81b4ba421b7173a24db32e53 on cloudfoundry-incubator:master.


Signal users to create a PR in owid-content when creating explorers on staging servers

Context
Until recently, staging servers used the master branch of owid-content, and this means any explorer work there was done directly on prod.
However, as of yesterday, each staging server uses its own branch of owid-content, matched to the ETL branch name. Edits in the admin, or running export steps, will commit to this branch.
Problem
With the new behaviour, you need to remember to make a PR and merge your owid-content changes after you merge your ETL branch. However, you might forget and then your explorer changes will not go live.
Note that these branches are not garbage-collected, so your work will not be accidentally deleted.
Proposal
Idea by @lucasrodes from https://github.com/owid/etl/pull/3307

I was wondering if we should signal somewhere more prominently that the user needs to manually create a PR & merge their branch in owid-content. Maybe @owidbot?

We could also programmatically create the PR in owid-content.

Could also ping the user in Slack?

See ETL docs how to work with explorers on staging.

From discussion, we think it would be plenty to add a comment post-merge when there were changes outstanding, with a one-click link to create a PR on owid-content against the right targets.


Release drafter doesn't release properly

It's something to do with permissions.  Maybe make a separate job to test / retest.. and permissions that the pypi upload job doesn't have.. (all in build_wheels.yml workflow).

huh.. seems to have worked for an actual release


Improvise CONTRIBUTING.md

In the spirit of getting more contributors for this project during the Hacktoberfest, detailed steps to contribute for beginners written in CONTRIBUTING.md file would be highly appreciated and would prove to be a good guide for new comers to open source.

Hey @sdmg15, may I work on this issue?

hey there may i work on the issue

Hi , can I work on given issue?

Yes you can, it should be noted inside that only one link or max two per pull requests.

Hi, I would like to contribute.


change norman type from dnsLabel to hostname for RequestedHostname

https://github.com/rancher/rancher/issues/14571

@carolynhu could you explain the reason why we do it, for the future reference.

Here we extend the restriction for RequestedHostName field from dnsLabel to hostname. Since dnsLabel does not allow .(dot) to be set in RequestedHostName. If we change it to hostname, we make it possible to update labels, custom name for node with hostname contains . (dot), such as ip address. Then we only need to make sure we validate dnsLabel restriction on hostname when node is created.

LGTM


Export HID++ library

libratbag currently has its own HID++ handling code, which supports both the 1.0 and 2.0 versions.
We have similar code in UPower to query battery status and solar charging support:
http://cgit.freedesktop.org/upower/tree/src/linux/hidpp-device.h
And we would want to integrate unifying receiver support in GNOME directly, to allow pairing and unpairing devices:
https://bugzilla.gnome.org/show_bug.cgi?id=756574
To avoid adding external dependencies, the new library could be a cut'n'paste library, or a git submodule. The API needed would be:

enumerate
get sysfs path (to match with udev, eventually), whether device is a touchpad, keyboard, mouse, etc.
start pairing
unpair a device
get battery percentage, whether charging or not
when supported, the luminosity hitting the solar charger

There is actually two slightly different problems here:

the receiver handling for enumerating devices, pairing and unpairing
accessing devices capabilities and dealing with them

Both functionalities are relying on the HID++ protocol from Logitech. But I am not sure we want to provide a common API for both.
The problem I see is that if you enumerate devices from the receiver, you can not go from the returned device to an actual hidpp_device with the current code. And I am not sure you want to do that given the 2 use cases.
After brainstorming with @whot yesterday, we think will provide 2 sub-libraries, one for accessing the receiver, and one for accessing the functionalities you need for both hid++ 1.0 and 2.0 devices.
How does that sound?

https://github.com/whot/libratbag/tree/wip/receiver-library
is the first attempt, let me know what you think.
This is a library for accessing Logitech Unified Receivers, and provides the basic functionality you asked for (minus the sysfs bits). The caller opens a hidraw node, constructs a receiver device, then does enumerate/open/close/disconnect on the device. The devices give you name, pid/vid and type.
Only a header file and a basic tool to show how it would work, no code to back it up yet.
If that's what you want, I can start working on that though.

That interface looks good to me, I'd just be missing a couple of things:

something to filter unifying receivers with (a .rules files with a particular tag, or a function that tells me whether the usb device with this vid/pid is a unifying receiver). I can probably piggy-back on UPower's tags for that (UPOWER_BATTERY_TYPE=unifying)
something to tell me whether a particular device is connected (it's at the receiver level, right?)
a way to match connected devices in the library to connected devices in udev (if the device serial were exposed in both the kernel and the library, or the index were exposed in the kernel). This is to solve the edge case of 2 devices with the same name.

Thanks!

added lur_is_receiver(int vid, int pid)
there's an activity counter we could expose, otherwise the receiver knows which device is connected, even if the device isn't active right now. that's what the enumerate() will give you, but if you need something specific let me know so we can be sure it's doable.
the serial is easy enough to expose from liblur. I wonder if that's something a udev callout could set on the udev device? @bentiss, do yo have a better suggestion re:kernel maybe?

the serial is easy enough to expose from liblur. I wonder if that's something a udev callout could set on the udev device? @bentiss, do yo have a better suggestion re:kernel maybe?

Yes, it's easy to export the serial when we have the handle on the receiver.
The easiest way to know if a udev device is connected to a particular receiver when you have its udev handle is simply to look at the udev tree: the receiver attaches the input devices that are connected to itself below it's own device. For example when I have only one device attached to the receiver:
P: /devices/.../1-1:1.2/0003:046D:C52B.005C/0003:046D:4041.005D
P: /devices/.../1-1:1.2/0003:046D:C52B.005C/0003:046D:4041.005D/hidraw/hidraw2
P: /devices/.../1-1:1.2/0003:046D:C52B.005C/0003:046D:4041.005D/input/input133
P: /devices/.../1-1:1.2/0003:046D:C52B.005C/0003:046D:4041.005D/input/input133/event11
P: /devices/.../1-1:1.2/0003:046D:C52B.005C/0003:046D:4041.005D/input/input133/mouse3
P: /devices/.../1-1:1.2/0003:046D:C52B.005C/hidraw/hidraw1

The receiver is the hidraw1 node, and the device is the hidraw2/event11 node (with the wireless PID 0x4041).

https://github.com/whot/libratbag/tree/wip/receiver-library
updated the branch with an actual implementation. the lur-command tool works, at least in my testing here. lur-command list /dev/hidraw2 will list all devices paired with that receiver (even if they're turned off). Is that enough or do you want some activity check too?
HID++ has an activity counter but it'd require polling the device regularly to see if the counter increases. Alternatively, we could probably ping the device with some other request and check if it responds - if not, it's paired but not turned on or suspended.

@whot I would just need the serials exported in udev/sysfs, that way, I can find whether the device is active on the local machine or not (which is the same as connected/disconnected for 99% of cases), which would be good enough for me.

The serial fetching doesn't seem to work (both are connected, I'm using them):
$ lur-command list /dev/hidraw0
0: T650 (touchpad) serial 0
1: K750 (keyboard) serial 0

This fixes it:
diff --git a/src/liblur.c b/src/liblur.c
index cbe1c64..65fd0dd 100644
--- a/src/liblur.c
+++ b/src/liblur.c
@@ -228,6 +228,7 @@ lur_receiver_enumerate(struct lur_receiver *lur,
                        dev->vid = VENDOR_ID_LOGITECH;
                        dev->pid = wpid;
                        dev->type = type;
+                       dev->serial = serial;
                        dev->hidppidx = i;
                        dev->present = true;
                        list_insert(&lur->devices, &dev->node);

And the lur-command is really unforgiving:
$ lur-command --help
lur-command: lur-command.c:106: main: Assertion `argc > 2' failed.
Aborted (core dumped)

squashed in, thanks. And yes, it's just running aborts right now for anything it can't handle, was a lot easier than proper error handling. But looks like we generally agree on this half of the library, so I can start adding those bits to it.

alright, updated, with better handling now. afaict what is missing now is the kernel exporting the serial (@bentiss) and having this as a copy/paste library. atm it's a .so.

Merged in pull #53


multicast with default impl?

I'm experimenting with your library and I like the design. But I've come across an error in my editor implementation and I'm not sure how to handle that.
Basically I have a basic object in which I can add mixins. And I have multicast messages for different purposes (showing gui stuff for example).  Should I add a "default" mixin with empty implementations, is there something similar to default-implementation for multicast messages or should it be allowed to send multicast messages to empty objects?

workaround for now: check if the object implements the feature, but I would have expected that multicast messages are more like pub-sub, so sending a multicast message to an object should be successful, even if the object doesn't listen to the multicast message (yet). Am I wrong with my assumption?

Currently what you observe is the designed behavior. If the object doesn't implement a message, even if it's a multicast, calling it throws bad_message_call. You've spotted one way to circumvent this and it's by checking with implements. You anticipated the other way, even though it's poorly documented. Messages can have default implementations. You can check out the messages tutorial to see how that works (the example is towards the end).
Of course you can also change your version of the library to suit this need. In message_macros.hpp you can comment out lines 126, 127, 171, and 172 and multicasts will silently do nothing when there is no mixin implementing them.
I don't think making this the default library behavior is the way to go, but it seems it's a good idea to make it optional, and I'll do so for the next release

okay, thanks for the answer (and thanks for open sourcing this library! look really neat and to the point for gamedev stuff)


Question about DuplicateMapError w/benellis3/pymarl2

Hello, firstly, thank you for the contribution about this whole simulation environment.
I wanted to integrate SMACv2 with PyMARL, and luckly found the SMACv2 version of PyMARL, PyMARL2
https://github.com/benellis3/pymarl2 (@benellis3)

I tried using MAC OX but faced following error

File "/anaconda3/envs/smac/lib/python3.9/site-packages/pysc2/maps/lib.py", line 122, in get_maps
raise DuplicateMapError("Duplicate map found: " + map_name)
pysc2.maps.lib.DuplicateMapError: Duplicate map found: 3m

After I delete 2 lines as below, it seems working, but is it a good solution for this error?
https://github.com/deepmind/pysc2/blob/0df53d38c153972f1e368572ba65b1442a0fd41f/pysc2/maps/lib.py#L121
https://github.com/deepmind/pysc2/blob/0df53d38c153972f1e368572ba65b1442a0fd41f/pysc2/maps/lib.py#L122
Thank you.

here

https://github.com/benellis3/pymarl2 (@benellis3)

Hi there! I did check that out, but my doubt remains because when I try to run pymarl2 which should run on smac2, it tries to pull StarCraftCapabilityEnvWrapper from smac but it obviously not there since that it is smacv2.

@andresgr96 My apologies, I believe this error happens because I renamed the imports to smacv2, but forgot to change the pymarl2 code to import smacv2. I will test this and get a fix going for it.
@euikk I believe you should be able to fix that error by deleting any duplicate maps from your maps directory. Those lines of code shouldn't need to be deleted.

Thanks for your response, will be looking forward to it. Been meaning to use SMACv2 for my bachelor's thesis but for now I'm using SMACv1 because of this issue, thanks for your hard work!


Use Kotlin DSL?

As a kotlin linter plugin, would anyone be opposed to converting the gradle build script for this project to using the kotlin dsl?
I'm willing to do the work, just want the "blessing" of others involved.

Done now, forgot to close!


Updating the Home page with cool features.

I would like to work on the home page by creating some interactive things for increasing the dwell time.
Also, I would like to add a header and footer section dynamically for optimisation. Please allow me to do this.

I want to work on this issue. I will add animation and font styles,SVG and canvas to make really amazing.

@frantic32211  this issue is assigned to you

@Aakansha99 you can assign this issue to @Mitali-laroia  I am unable to work on it for now due to hardware problem.

@Aakansha99 you can assign this issue to @Mitali-laroia as I have some hardware issue


No window shadow on linux.

(app found in issues page)
I'm considering to use egui for my app (GPL) and that flaw is a no. Is it a feature or a bug.
It looks terribly out of place. I don't know how to enable native window frame if thats possible
6.6.5-arch1-1

https://github.com/PolyMeilex/sctk-adwaita/pull/43 was merged recently, providing window shadows support
but this is still not included in winit for now, so probably it won t be available until winit 0.30 (or later)

how to enable native window frame

...try to run with the x11 backend
note that wayland (or at least wayland on gnome) doesn t provide any native window frame.  it relies on the client to draw shadows and frames themselves.

Is there a development branch I can use or something ? I don't need stability it's full nightly and stuff

i dont think there is currently.. try forking and patching winit to upgrade sctk-adwaita to 0.8.

nice, it works https://github.com/planetoryd/egui_node_graph

but this is still not included in winit for now, so probably it won t be available until winit 0.30 (or later)

It should be possible to include it in a winit 0.29 patch release.
This egui issue can be closed, as this has nothing to do with egui.


add feature to avoid cross-az traffic

Cross-AZ traffic will cause extra costs on iaas providers like AWS, therefore prefer traffic to local AZs.
This will be implemented via backup server, which will get called only if the remaining regular servers are down

Can you make this default to being off? Otherwise, looks good

@geofffranks done


How to modify 02_filter_colmap_data.py for original 3DGS colmap format data

How to modify 02_filter_colmap_data.py for original 3DGS style input data as it is using colmap format data i.e. data includes folders like distorted, images (undistorted), sparse, stero. And not NeRFStudio's nsprocess generated json data?

Unfortunately, I don't think the same concept really applies to 3DGS (neither Inria's nor Nerfstudio's Splatfacto).
NeRFs (in Nerfstudio) only take the images and camera positions as input, as given by transforms.json.
3DGS, on the other hand, takes the images, positions (I think) and sparse point cloud as input.
While it might be possible to modify the COLMAP generated database file directly to remove the least sharp images, it will still use the full point cloud as input.
This might not be a problem, in fact, it might be beneficial, but it's a lot more complex than removing a few lines from a json file as we currently do with transforms.json.
I might try to make it work with Nerfstudio's Splatfacto in the future, if I have time (no promises though), but I have little interest in making it work with Inria's version, as I don't use it, and Inria's license makes it a very unattractive solution compared to Nerfstudio.
I'm open to contributions though.
I'm also looking into other methods that could completely remove the need for this project (and for COLMAP in general), but I don't know when, if ever, that will bear fruit yet.


Improve performance when using gometalinter

Before this change when I opened a big project that had 6000+ warnings/errors it took ages to get the actual warnings/errors and it caused my CPU to be busy for quite some time. The call to gometalinter alone took about 24 seconds, but after that vim was struggling as well.
After this change the gometalinter call just takes 2 seconds and nothing noticeable happens with the CPU and/or vim.

Cheers! :beers:


The update command's target version does not exist

Problem:
When running npm run update the following error occurs.
vue-sqlite-app-starter|main ⇒ npm run update

><EMAIL_ADDRESS>update
> npm install --save-dev @capacitor-community/sqlite@last --save vue-sqlite-hook@last

npm ERR! code ETARGET
npm ERR! notarget No matching version found for vue-sqlite-hook@last.
npm ERR! notarget In most cases you or one of your dependencies are requesting
npm ERR! notarget a package version that doesn't exist.

npm ERR!      /Users/[...]/.npm/cache/_logs/2023-01-14T11_41_06_660Z-debug-0.log

Solution:
Changing version from last to latest removes the error and hopefully fetches the latest version.

@RHellenes Thanks for your contribution to enhance the documentation.


Update ranker and sorting

Resolves #153

Unsubscribe
Sent from my iPhone

On Sep 14, 2020, at 12:37 PM, Angelos Chalaris<EMAIL_ADDRESS>wrote:

Resolves #153
You can view, comment on, or merge this pull request online at:
https://github.com/30-seconds/30-seconds-web/pull/172
Commit Summary
Deprecate updateCount from snippets
Update ranking formula
Update ranking engine keyword scores
Rename 'Popularity' to 'Recommended'
Export firstSeen from compileSnippet
Deprecate Expertise and Alphabetical ranking for main listing
Introduce freshness sorting for Blog
Hide sorter if it contains only one order
Add redirects for deprecated listing pages
File Changes
M src/build/listing/index.js (25)
M src/build/snippet/compileSnippet.js (1)
M src/build/snippet/snippetData.js (5)
M src/components/molecules/sorter/_index.scss (8)
M src/components/molecules/sorter/index.jsx (2)
M src/components/molecules/sorter/index.test.jsx (16)
M src/components/organisms/snippetList/_index.scss (2)
M src/components/organisms/snippetList/index.jsx (4)
M src/components/organisms/snippetList/index.test.jsx (23)
M src/config/rankingEngine.json (24)
M src/engines/rankingEngine.js (21)
M src/fixtures/requirables.js (4)
M src/fixtures/snippets.js (2)
M src/lang/en/listing/index.js (3)
M src/spec/listing.spec.js (54)
M static/_redirects (61)
Patch Links:
https://github.com/30-seconds/30-seconds-web/pull/172.patch
https://github.com/30-seconds/30-seconds-web/pull/172.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


d3d11 memleak. RGBA4 / BGRA4 conversion code is not functional in D3D11::update

https://github.com/bkaradzic/bgfx/blob/a855f1cf2dde73dc26346768b032530ca200c1b3/src/renderer_d3d11.cpp#L4818
@bwrsandman
There's a few issues going on with recent changes to TextureD3D11::update
Here's code for reference, with some interleaved comments
		{
			uint8_t* src = data;
			// (jamesf) this loop is going to happen for every call to this function, regardless of format. I think this is too much pointless work that could be avoided. I think this loop should be in an if (R5G6B5/RGBA4/RGB5A1) check
			for (uint32_t yy = 0, height = _rect.m_height; yy < height; yy += blockHeight)
			{
				switch (m_textureFormat)
				{
				case TextureFormat::R5G6B5:
					temp = (uint8_t*)BX_ALLOC(g_allocator, rectpitch); // allocating here every loop..
					bimg::imageConvert(temp, 16, bx::packB5G6R5, src, bx::unpackR5G6B5, rectpitch);
					data = temp;
					break;
				case TextureFormat::RGBA4:
					temp = (uint8_t*)BX_ALLOC(g_allocator, rectpitch);
					bimg::imageConvert(temp, 16, bx::packBgra4, src, bx::unpackRgba4, rectpitch);
					data = temp;
					break;
				case TextureFormat::RGB5A1:
					temp = (uint8_t*)BX_ALLOC(g_allocator, rectpitch);
					bimg::imageConvert(temp, 16, bx::packBgr5a1, src, bx::unpackRgb5a1, rectpitch);
					data = temp;
					break;
				}
				src += srcpitch;
			}
			// (jamesf) at end of loop, 'data' is essentially pointing at last row of image
		}

		deviceCtx->UpdateSubresource(
			  m_ptr
			, subres
			, depth ? NULL : &box
			, data
			, srcpitch
			, TextureD3D11::Texture3D == m_type ? slicepitch : 0
			);

		if (NULL != temp)
		{
			BX_FREE(g_allocator, temp); // (jamesf) freeing once at the end, leaking "loop count - 1"
		}

`

might be worth adding these formats to 08-update example

Oh wow, that code is quite idiotic...
I deleted it: https://github.com/bkaradzic/bgfx/commit/27602120e9af88ac36913e94824ed5ba8ec84864

That works. Adding an assert or something might be nice. Now if someone went down this codepath with an RGBA4 it would upload swizzled, but if they provide the mem in createTexture2D it will be correct. See nearly matching bimg::imageConvert calls in TextureD3D11::create.
I am actually using RGBA4 so it was weird to sync latest and see those textures breaking. The fix for me was to just swap to new BGRA4 enum since all my use-cases are greyscale. Seems like that's how the backend actually wants it anyway. Feel free to close this issue if you'd like.

This just need to be re-implemented and tested.

Similar conversion wasdone in D3D9, there could be an issue there too


🐛 BUG: API/unstable_dev race condition when running several unstable_dev commands at the same time

What version of Wrangler are you using?
wrangler@beta
What operating system are you using?
Mac
Describe the Bug
In CI, running tests with unstable_dev fails sometimes. Investigating why lead to the following discoveries:
This blows up with TypeError: fetch failed:
    workers.push(
            await wrangler.unstable_dev("src/basicModule.ts"),
            await wrangler.unstable_dev("src/basicModule.ts")
        );

This works:
        workers.push(
            await wrangler.unstable_dev("src/basicModule.ts", { port: 9001 }),
            await wrangler.unstable_dev("src/basicModule.ts", { port: 9002 })
        );

This fails with a jest timeout:
        workers = await Promise.all([
            wrangler.unstable_dev("src/basicModule.ts", { port: 9001 }) as Worker,
            wrangler.unstable_dev("src/basicModule.ts", { port: 9002 }) as Worker,
        ]);

Implementing a fix to this in this PR: https://github.com/cloudflare/wrangler2/pull/1538


invalid memory address or nil pointer dereference - lua pcall. (#693)

Description
lua pcall over but luastate is Closed caused by (match stop with call queue is full)
Steps to Reproduce
pcall(nk.storage_write,xxx) in match_loop
a lot of presences join match to trigger call queue is full
Actual Result
server crash

This pull request looks like fix my issue :
https://github.com/heroiclabs/nakama-unity/issues/114
https://github.com/heroiclabs/nakama/issues/636
@kavenGw can you tell me how to build a docker file from this pull request? Thanks in advance

This pull request looks like fix my issue : heroiclabs/nakama-unity#114 #636
@kavenGw can you tell me how to build a docker file from this pull request? Thanks in advance

I just read your issue, but I don't think we have the same reason. I'm using Nakama Lua runtime and my issue is caused by one of my matches being stopped by the call queue is full. But there is a DB read in match_loop, the DB read will return failed to the Lua runtime(which the Lua state is already disposed when the match stopped).
Anyway, if you want to try my code. Just use the default docker file. I just change 2 lines.

@kavenGw do you have more info or a stacktrace you could provide on the panic you've observed?

Addressed cleanly in 6d8195b99d4c59247e0aa98b92f1c8aa806be19d, stack should only have been nil when all match function executions ended.


Docs update: execution_engine/shared

Add or update existing rust docs in a shared rust module in EE:

[x] shared/account
[x] shared/logging
[x] shared/newtypes
[x] shared/system_config
[x] shared/*

#2110 merged


Generate better ID name than "check-account-is-associated-with-any" for "Check if a GitHub account is associated with any Marketplace listing"

endpoint scope: Apps
endpoint name: Check if a GitHub account is associated with any Marketplace listing
route: `GET /marketplace_listing/accounts/:account_id
documentation URL: https://developer.github.com/v3/apps/marketplace/#check-if-a-github-account-is-associated-with-any-marketplace-listing

Current specification
https://github.com/octokit/routes/blob/268049721ab5db75eb04a87ff5d41ff44aa4b0d1/routes/api.github.com/apps/check-account-is-associated-with-any.json#L1-L100
I expected
Maybe something like check-if-account-is-associated-with-any-marketplace-listing? Or check-if-account-has-marketplace-listings?
I think "check" is not the right word here. Other endpoints such as "Check if you are starring a repository" follow the convention that they either return 404 or 204 (no body). In this case the response is useful, so the title should maybe rather be something like "Get Marketplace listing for GitHub account"
This should be discussed with GitHub’s Documentation team before we create a workaround

that's fixed now


use datetime values for x-axis in line-chart

Hi, loving this package for graphs in flutter!
Just 1 thing though: if my x-axis values ('domain') are date-stamps I get an error that they must be type num. Could it be possible to use DateTime values there instead?

Yeah, we can replace the time-data with something like seconds-since-epoch - so it's now a numeric value and the graph works - but this makes the labels on the x-axis kinda meaningless (who knows how to translate that long number into the original datestamp that was in the data?). If the original data has timestamps which are meaningful and readable. I had a look at the source-code and can see where you've used num. It just means that line-charts are really not useful for time-based data, I guess. Thx anyway.

Yes,  for datetime value you must use base package,  charts_flutter.
You can see the example from Google Chart Gallery which this package is
discontinued
Pada tanggal Sel, 1 Nov 2022 02.59, Richard Shepherd <
@.***> menulis:

Yeah, we can replace the time-data with something like seconds-since-epoch

so it's now a numeric value and the graph works - but this makes the
labels on the x-axis kinda meaningless (who knows how to translate that
long number into the original datestamp that was in the data?). If the
original data has timestamps which are meaningful and readable. I had a
look at the source-code and can see where you've used num. It just means
that line-charts are really not useful for time-based data, I guess. Thx
anyway.

—
Reply to this email directly, view it on GitHub
https://github.com/indratrisnar/d_chart/issues/6#issuecomment-1297607249,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APTLYD74G2E3T2AEAIXVECLWGAQKZANCNFSM6AAAAAARSGQMRA
.
You are receiving this because you commented.Message ID:
@.***>

updated on d_chart: 2.6.2
you can use DChart Time with code T. like DChartLineT, DChartBarT, DChartComboT, DChartScatterT


Add WOT (Web of Trust) Icon

Request to add icon for WOT (Web of Trust).

Hi,
I am closing all icon requests in an effort to keep this project clean. We unfortunately don't have the time to implement all the icon ourselves and it would be misleading to keep these issues open.
The good news is that I have also made icon contribution much easier (aka possible). There are the documented steps that you can follow if you would really like to see your icon included: https://github.com/samcollins/css-social-buttons#how-to-contribute . Please keep in mind that this icon set is for social icons.
Mistakes happen so if you feel that I have unjustly closed this issue please let me know.
Cheers,
zimbatm


Allow piping through all configuration and configuring all ServiceMonitors

As in the title.
@kakkoyun

I think a nice follow up would be to have each component have the .manifests field, which can then be recursively built, so we don't need to leak the configuration into rendering like we do here.

I think a nice follow up would be to have each component have the .manifests field, which can then be recursively built, so we don't need to leak the configuration into rendering like we do here.

I love the idea. We do it for major building blocks like Thanos, Loki. It'd be best to have it for each component.


Move policy decisions into pshtt

This PR fixes Issue #29, adding three new fields to the tool output:

Domain Supports HTTPS: A domain 'Supports HTTPS' when it doesn't downgrade and has valid HTTPS, or when it doesn't downgrade and has a bad chain but not a bad hostname. This is an attempt to capture whether the domain owner intentionally supports HTTPS: a bad chain shows an intent (though a misconfiguration), while a bad hostname often means support is not intended.
Domain Enforces HTTPS: A domain that 'Enforces HTTPS' must 'Support HTTPS' and default to HTTPS. 'Redirect domains' must also strictly enforce HTTPS.
Domain Uses Strong HSTS: A domain 'Uses Strong HSTS' when max-age ≥ 31536000.

These definitions closely follow the contours of M-15-13 compliance, and also match what Pulse calls "Uses HTTPS", "Enforces HTTPS", and "Strict Transport Security (HSTS)".

I added some changes in https://github.com/dhs-ncats/pshtt/pull/42/files/bb3461dd61a4388c581898f5a4f82e8f9dafc4c0..c9fc2e3acbbb3095241cc4f85121d502673b2fb7 that fix the flake8 errors.
It was mostly indentation, but I also changed some [condition] != True lines to not [condition]. This is consistent with the other conditional statements in the tool, and meets flake8. @h-m-f-t I didn't test out those changes, but could you 👁 and test them the same as you've been testing this code generally before merging?

Looks good!