Not all icons available in Lumina panels

Some icons are shown correctly, others are not. See panel screenshot.

When checking /usr/local/share/applications it makes a difference whether I am running Lumina-FM as root or as a normal user. When run as the latter the exact same icons that display correctly in the panel are the only ones shown as images in Lumina-FM. When run as root, all icons are displayed in Lumina-FM.
Lumina-FM run by normal user:

Lumina-FM run by root:

There is also difference in the directory icons in the directory tree for root/user, as you can see.

This may be completely unrelated, but I had an issue similar to this after the last update. I was previously testing the qt5ct theme engine and, after updating, Lumina continued to load/point to that instead of the Lumina theme engine as the new default. Have you made any adjustments/customisation in theming that could be conflicting with what Lumina is trying to do by default?
Regardless, thanks for the report and I'll try and see if I can reproduce!

This issue turned up when I switched to UNSTABLE end of August. I don't think that Lumina is looking in the wrong directory. All icons are there, Lumina just cannot see them all. Seems to be related to the Lumina-FM normal user/root issue described above. I already ran the desktop settings as root and added a program to the panel which icon only shows correctly in root-Lumina-FM but that did not solve it.

What are the file owner/group and permissions on the icons?

Hm, I don't see why e.g. the Darktable icon is displayed correctly and the Firefox icon is not.
ls -l /usr/local/share/applications total 162 -rw-r--r--  1 root  wheel    233 25 Aug. 17:13 appcafe.desktop -rw-r--r--  1 root  wheel    938 17 Sep. 17:35 audacity.desktop -rw-r--r--  1 root  wheel   8971 17 Sep. 03:27 bonobo-browser.desktop -rw-r--r--  1 root  wheel    323 17 Sep. 01:33 chromium-browser.desktop -rw-r--r--  1 root  wheel    216 17 Sep. 04:21 compton.desktop -r--r--r--  1 root  wheel   2069 16 Sep. 23:46 cups.desktop -rw-r--r--  1 root  wheel   3468 17 Sep. 19:10 darktable.desktop -rw-r--r--  1 root  wheel   1049 17 Sep. 21:06 FBReader.desktop -rw-r--r--  1 root  wheel   6545 17 Sep. 02:16 firefox.desktop -rw-r--r--  1 root  wheel   5550 17 Sep. 02:02 gcr-prompter.desktop -rw-r--r--  1 root  wheel    690 17 Sep. 02:02 gcr-viewer.desktop -rw-r--r--  1 root  wheel   2085 17 Sep. 16:49 geeqie.desktop -rw-r--r--  1 root  wheel    216 16 Sep. 23:54 geoclue-demo-agent.desktop -rw-r--r--  1 root  wheel    329 16 Sep. 23:54 geoclue-where-am-i.desktop -rw-r--r--  1 root  wheel  11456 17 Sep. 04:58 gimp.desktop -rw-r--r--  1 root  wheel    318 17 Sep. 18:42 gpodder-url-handler.desktop -rw-r--r--  1 root  wheel   4122 17 Sep. 18:42 gpodder.desktop -rw-r--r--  1 root  wheel    421 18 Sep. 05:40 gscan2pdf.desktop -rw-r--r--  1 root  wheel    201 16 Sep. 23:59 gtk3-demo.desktop -rw-r--r--  1 root  wheel    221 16 Sep. 23:59 gtk3-icon-browser.desktop -rw-r--r--  1 root  wheel    246 16 Sep. 23:59 gtk3-widget-factory.desktop -rw-r--r--  1 root  wheel   3623 17 Sep. 10:08 gvim.desktop -rw-r--r--  1 root  wheel   1681 17 Sep. 09:42 handbook.desktop -rw-r--r--  1 root  wheel    347 17 Sep. 14:51 hplip.desktop

Well there is the reason why, those icons are owned by root.  A regular user is unable to access them.  You need to resolve the screwed up permissions issues in your system.  This will continue to cause issues on your system until these are set correctly.  The ones that are working may be due to there being another copy of the icon available in your system that is not owned by root.
I would recommend re-installing and then migrating your user directory over to the new install.
What you are experiencing is not a bug... it's proper user restrictions working correctly.

I am sure I did not willingly change any permissions. And changing the owner did not solve this.
I checked with a Virtual Box installation of UNSTABLE and there I found the exact same permissions. I also checked with an old PCBSD 10.3 system: also owner root.
Did you try to set up a secondary panel with e.g. a Firefox application starter?
However, I will perform a fresh reinstall as soon as GELI full disk encryption works again. Maybe something else is broken with my system.


Triangulated fill outlines

feat: Fill layer to generate outlines composed of triangle primitives

Created a new addition to the fill generator functions to generate this new fill+outline variant
The Fill bucket generates fill outlines with triangle primitives, compatible with the Line shader
The Fill bucket uses gfx::generateFillAndOutineBuffers to generate its buffers
Added an option to enable/disable this feature at compile time with the MLN_TRIANGULATE_FILL_OUTLINES #define in fill_bucket.hpp

Refs: #1768

WIP

Comparison of fill outlines
Legacy renderer:

Drawable renderer:

Fill pattern outline drawn with the legacy renderer:

And with the Metal renderer (no outline):

performance impact versus main@a73e88fee28593a0e66be1e67d37ae444b0bfc6c

new measurements after commenting out [commandBuffer waitUntilCompleted];

New timing measured by averaging 3 runs on iPad 5th gen.
In LineBasicShader v1 the interpolation UBO is deleted.
In LineBasicShader v2 the blur, gap width, offset UBO fields are removed; permutation and expression UBOs are also removed on Metal.

Looks like reasonable numbers.
One more useful measurement might be lines with the single pixel Metal implementation (e.g. what Metal provides).  Just to see what lines are costing us.

Looks like reasonable numbers. One more useful measurement might be lines with the single pixel Metal implementation (e.g. what Metal provides). Just to see what lines are costing us.

This is what metal on main has, 2 pixel wide lines (which render as 1 pixel)...

Hmmm, still some cost to the simpler version.

Staring at the numbers a bit more.... I assume they'll be slightly different on more recent hardware, but they're unlikely to break the pattern we've seen before.
I think we want to go with the v2 shader and we may also want to provide an option to turn that off and use Metal lines.
But after we've got the v2 shader merged in, we should see how it fares in a Metal frame analysis.  Maybe those tools can see something we can't.

@louwers : I added that flag following Steve's recommendation to "provide an option to turn that off and use Metal lines"...
CC @sjg-wdw

@louwers : I added that flag following Steve's recommendation to "provide an option to turn that off and use Metal lines"... CC @sjg-wdw

Nice!

With #define MLN_TRIANGULATE_FILL_OUTLINES 1 render test render-tests/fill-outline-color/property-function is drawing white background, before it was drawing ok but failing because of line width.
Before:

After:

While investigating the failed render tests I realized that we also need shader variations for the triangulated lines in case of outline-color and opacity per-vertex, which is the case with some expressions.
render-tests/fill-outline-color/zoom-and-property-function
render-tests/fill-outline-color/property-function
ref #1855


Add note about code of conduct

Copied from Shapeless's README.

Current coverage is 67.05%

Merging #32 into master will not affect coverage as of 1d6e787

Powered by Codecov. Updated on successful CI builds.


NSI reservation request between same STP successful.

Was able to successfully reserve a connection on the same STP which should be rejected with a "00710 – NO_TRANSPORTPLANE_PATH_FOUND" error:
urn:ogf:network:netlab.es.net:2013::netlab-mx960-rt1:xe-11_2_0:+?vlan=2009
urn:ogf:network:netlab.es.net:2013::netlab-mx960-rt1:xe-11_2_0:+?vlan=2009
Also was able to reserve a connection on two different STP sharing the same root port:
urn:ogf:network:netlab.es.net:2013::netlab-mx960-rt1:xe-11_2_0:+?vlan=2009
urn:ogf:network:netlab.es.net:2013::netlab-mx960-rt1:xe-11_2_0:+?vlan=2010
Should be rejected with the same "00710 – NO_TRANSPORTPLANE_PATH_FOUND" error.

disallowed in 1.0.10


Integration with CircleCI

I wanted to try @circleci but it doesn't seem to want to work with @gogitservice hosted on @digitalocean. Am I being dim?
@gogitservice replied with "@andyjeffries @circleci @digitalocean I think it's matter of webhook format? Report a feature request on @github so we can count into plan!" -- https://twitter.com/gogitservice/status/558261002634928128

Thanks your feedback!

Note: http://circleci.com/docs/2.0/


How do I include postgresql.conf using 'Database containers launched via JDBC URL scheme'

Hi, is it possible to give postgresql testcontainer a custom postgresql.conf file? Or add custom settings to postgres?
I have included maven dependency
<dependency>
            <groupId>org.testcontainers</groupId>
            <artifactId>postgresql</artifactId>
            <version>1.10.6</version>
</dependency>

And using 'Database containers launched via JDBC URL scheme' for DB url As such have the setting in my Spring Boot app as:
datasource:
    url: jdbc:tc:postgresql:10-alpine:///databasename
    driver-class-name: org.testcontainers.jdbc.ContainerDatabaseDriver

I need to have a custom setting in postgresql.conf. Is there a way of pushing postgresql.conf to the docker container started by testcontainers?
Have asked the same question here:
https://stackoverflow.com/questions/54766846/how-do-you-include-postgresql-conf-on-docker-container-when-using-org-testcontai
Thank you in advance
Melissa

Just ran into the same problem. I need to have a custom setting in postgresql.conf for the postgres docker container.
Any updates on this?

@ameenhere no updates from myself, still having the same problem.

I also needed this, I ended up building my own docker containers with postgresql in order to be able to control this, it's quite easy to do, you can look here to see the source of my containers: https://github.com/pgjdbc/pgadba/tree/master/src/test/resources
And then I used them like this:
  public static PostgreSQLContainer getNewWithTls() {
    PostgreSQLContainer container = new PostgreSQLContainer("capitol/postgresql-tls:debian-stretch-postgresql10");
    container.withTmpFs(singletonMap("/var/lib/postgresql/data", "rw"));
    container.start();

    return container;
  }

@alexanderkjall  How did you manage to customize the container that is picked up when using "Database containers launched via JDBC URL scheme" ?

I would say that using volumns via "Database containers launched via JDBC URL scheme" (to customise a container) has not been implemented yet?
Is anyone able to verfy this?
Using standard docker commands you can supply your own Postgres server configuration via 'Use a custom config file' for example:
docker run -d --name postgres -v "$PWD/my-postgres.conf":/etc/postgresql/postgresql.conf -e POSTGRES_PASSWORD=postgres  postgres:10-alpine -c 'config_file=/etc/postgresql/postgresql.conf'
However testcontainers 'Database containers launched via JDBC URL scheme' currently only allows for TC_INITSCRIPT or TC_INITFUNCTION.
Looking at @alexanderkjall example he's use the
org.testcontainers.containers.GenericContainer.withTmpFs function
Further investigation shows me that
org.testcontainers.containers.MariaDBContainer might be using a way to pass in custom config to container via;
optionallyMapResourceParameterAsVolume(MY_CNF_CONFIG_OVERRIDE_PARAM_NAME, "/etc/mysql/conf.d", "mariadb-default-conf");
optionallyMapResourceParameterAsVolume is a function of abstract class JdbcDatabaseContainer which PostgreSQLContainer is also extending.
PostgreSQLContainer does not use optionallyMapResourceParameterAsVolume from what I can see.
From what I can see its the org.testcontainers.jdbc.ContainerDatabaseDriver which drives which DB container is started for
'Database containers launched via JDBC URL scheme'
And to sort out: possibly an additional db url query param such as TC_INITSCRIPT or TC_INITFUNCTION would be needed.
Which can then do the volume mapping for us

@melissapalmer  I assume you are correct. But I have found a workaround to my problem. We were using Database container objects for our integration tests and wanted to customize a postgres config (namely max_locks_per_transaction). Eventually ended up extending some of the testcontainer classes and modifying the command from "postgres" to "postgres -c config=value -c config2=value2 ..." as required. Here is the source code of what I did, https://github.com/dhis2/dhis2-core/blob/master/dhis-2/dhis-support/dhis-support-test/src/main/java/org/hisp/dhis/container/DhisPostgreSQLContainer.java
I feel you may also be able to extend org.testcontainers.jdbc.ContainerDatabaseDriver into using such customized Database container objects?

@kiview
Does the underlying problem I have on this issue relate to #1239 possibly?

@ameenhere Based on your comment this worked for me:
    @Rule
    public PostgreSQLContainer container = new PostgreSQLContainer<>()
            .withCommand("postgres -c max_prepared_transactions=10");

No need for a custom subclass.


how to use tensorrt_llm backend in tritonserver

System Info
None
Who can help?
@juney-nvidia
Information

[ ] The official example scripts
[ ] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction
Please tell me how to use tensorrt_llm in tritonserver, do you have docs for this issue? FYI
Expected behavior
None
actual behavior
None
additional notes
None

Hi @AndreWanga , please refer to this doc: https://github.com/triton-inference-server/tensorrtllm_backend/blob/main/docs/llama.md


Question about the Off-Canvas Navigation

Hi James,
I want to use the off-canvas navigation on mobile, but have the nav visible on desktop, what's the best way of doing so?
I reckon that I can either; duplicate the nav and use the hidden and visible classes…or…override the width and visibility of .navigation for desktop only using the media queries within my css.
The duplicate may be the most straightforward way, especially if the layout of the nav on desktop would be considerably different from the mobile, but toggling the visibility may work best if the navigation is to be left or right aligned.
Just wanted to check in case I'm being daft and missing an easy way of setting it so the off-canvas nav only kicks in on certain layouts?
Alternatively, would I be best to use the previous method for creating navigation using the dropdown functionality when the layouts don't fit?
Just trying to be mindful of the aria-hidden attribute that is applied by the navigation.
Cheers,
Iain

To be honest I'm not sure. This iteration of the navigation menu is an all or nothing approach.
Duplication probably isn't wise since the presence of duplicate elements could confuse screen readers so I would recommend using media queries and the $.support.currentGrid() method to let you know when to toggle the aria hidden attribute.
It might however, be simpler to take the basis of my code and use that for whatever you need.
I did make a start to making the navigation toggleable but haven't got far since I don't know if I can cater for what developers would need. I would like to have another crack at it though when I can.

Okay, I'll see what I can do. Thanks for getting back to me!

No worries :smile:  I'll start experimenting again as soon as I can. It's all hectic here at the moment though what with the big move and all.

I would imagine. Hope it all goes well! :)

Cheers!


OSMNX geometries deprecation

osm_online_loader.py:93: UserWarning: The `geometries` module and `geometries_from_X` functions have been renamed the `features` module and `features_from_X` functions. Use these instead. The `geometries` module and function names are deprecated and will be removed in a future release.
  geometries = ox.geometries_from_polygon(polygon, {key: value})

Closed with #341


Updating Office Blog

OPS Build status updates of commit ce5c83f:
:clock10: Preparing: average preparing time is 50 sec(s)

OPS Build status updates of commit ce5c83f:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/open_specs_blog, working directory: W:\31mw-s, target branch: live, target branch commit: f6e98a20b6cb0aeae9c28cbaa207ac2687869baf, source branch: master, source branch commit: .

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


⚠ Bofrost.be has degraded performance

In c0ab198, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1994 ms

Resolved: Bofrost.be performance has improved in 4875b20 after 6 minutes.


Help to build

you can describe how i build this project?

Hi ! Because it is build as a web app with a php backend you can't actually build it. Instead, you can run in in the background in a docker container.
Just download/clone the repo on your machine and run the docker container in the background (docker-compose up -d)

Oh yes, it's just that I would like to deploy on a shared server that doesn't have Docker in order to run some tests.

Just run a basic php server then ! Docker is just running a barebones apache php server under the hood (Dockerfile here) with php -S localhost:[port] -t public

ohhh ok


Release binaries for mac os and windows

Description
Release binaries for mac os and Windows
Test Plan
Running via my branch

Do we know approximately how much this will cost us running on windows-latest and macos-latest? Last I remember they weren't cheap.

This workflow should only run a few times a week, at most. So, not sure how much that would cost us more. @brad-mysten  do you know?


coreml deleagate not support ResizeBilinear layer

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
2.10 or 2.11
Custom code
Yes
OS platform and distribution
No response
Mobile device
No response
Python version
No response
Bazel version
no
GCC/compiler version
no
CUDA/cuDNN version
no
GPU model and memory
111
Current behavior?
i run coreml delegate on iphone 12. The tflite model has ResizeBilinear layer ( align_coreners == false , half_pixel_center ==true),
but I find the tflite code following  (coreml_delegate.mm):
// For most ops, only version 1 is supported.
if (registration->version > 1) {
return false;
}
ResizeBilinear layer is not supported by ANE because of version >1.
But if ResizeBilinear layer ( align_coreners == false , half_pixel_center ==false) , it is supported by ANE engine.So the performance will be improved
Standalone code to reproduce the issue
my question is ,
ResizeBilinear layer ( align_coreners == false , half_pixel_center ==true) can be supported by ANE?
this format is common during training model
thank you!

Relevant log output
No response

@mengran1234 As per the documentation here in coreml_delegate, the ResizeBilinear is a supported ops. Could you please let us know if this is what you are looking for?
Thank you!

@mengran1234 As per the documentation here in coreml_delegate, the ResizeBilinear is a supported ops. Could you please let us know if this is what you are looking for? Thank you!

yes
but the documentation is very simple。

align_corners = false , half_pixel_centers = true
this  configuration parameter is common， how coreml can support it ?

Hi @mengran1234, the current version is https://github.com/tensorflow/tensorflow/blob/master/tensorflow/lite/delegates/coreml/coreml_delegate.mm, which seems different than your code snippet. Can you try with the latest release and let us know if it resolves your issue? Also I am having trouble understanding your issue. Does your code work? Does it work but your performance doesn't match what you expected? If so, please tell us how. Thanks for any additional information you can provide.

My code can run normally.
But  performance doesn't match what i expected because of ResizeBilinear layer.
the code is:
// For most ops, only version 1 is supported.
if (registration->version > 1) {
switch (registration->builtin_code) {
case kTfLiteBuiltinDepthwiseConv2d:
if (registration->version > 2) return false;
break;
// FullyConnected without bias is supported starting from version 6.
case kTfLiteBuiltinFullyConnected:
if (registration->version > 6) return false;
break;
default:
return false;
}
}
If ResizeBilinear (align_corners = false , half_pixel_centers = true) ,
above code will run to "
default:
return false;
"
so coreml  do not support this layer (meet the conditions registration->version > 1)

Ok thanks for the information, we'll consider this a feature request then, @yishuangP can you please take a look?

do you know
// For most ops, only version 1 is supported.
if (registration->version > 1) {
above where is the value of registration->version from? how it defined

Hi, @mengran1234
Thanks for raising this issue. Are you aware of the migration to LiteRT? This transition is aimed at enhancing our project's capabilities and providing improved support and focus for our users. As we believe this issue is still relevant to LiteRT we are moving your issue there. Please follow progress here: https://github.com/google-ai-edge/LiteRT/issues/158
Let us know if you have any questions. Thanks.


Swap getUpSQL and getDownSQL

I have an existing db, generated schema.xml, then added new column to the db and run propel diff
In a generated migration I see getUpSQL dropping the field and getDownSQL adding new field
I think getUpSQL and getDownSQL should be swapped here https://github.com/propelorm/Propel2/blob/431b8d059b660fc5afc674cb90c15b5e23af3089/src/Propel/Generator/Command/MigrationDiffCommand.php#L218-L219
Or I just use Propel in a wrong way)

@melnikaite you should modify schema.xml and run propel diff.
if you can modify your DB, you already have the right DDL so no need for propel diff.
http://propelorm.org/documentation/09-migrations.html#migration-workflow

The workflow we have (same issue) is that we first modify DB in our dev system directly, and then want to roll it out to our various production environments.
The advantage of this approach is that it is quite simple: We use GUI tools like phpmyadmin to edit the DB structure, and then make sure the application runs with the new structure. In this approach, there is no need to learn the Propel-specific XML format for DB structures.

That's not an advantage, it's sloppy.  Use the schema and diff set to manage the changes in your database.  Review how ORMs work and why you should manage chnagesets and not modify your database directly.  This is an ORM toolset.


static method on js objects behave differently

While looking at my google_maps package to migrate it from dart:js to package:js I faced a difference between dart2js and dev_compiler with the following code:
@JS()
class Event {
  external static Event get CLICK;
  static Event get click => CLICK;  // static method with body on js type
}

void main() {
  print(Event.CLICK);
  print(Event.click);
}

// js
function Event(){}
Event.CLICK = 'click';

I get the following output:
// with dart2js (webdev build)
click
click
// with dev_compiler (webdev serve)
click
null

The behaviour of dart2js is really better and it would be great to fix dev_compiler.
$ webdev --version
2.5.7
$ dart --version
Dart VM version: 2.7.2 (Mon Mar 23 22:11:27 2020 +0100) on "linux_x64"

/cc @vsmenon @sigmundch


[BACKLOG-31925] - refactoring browse for more flexible calls

:white_check_mark: Build finished in 10m 45s
Build command:
mvn clean verify -B -e -Daudit -amd -pl ui

:ok_hand: All tests passed!
Tests run: 1100, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message


Harness - Truncation

[ ] Alternate truncation

I'm about to write an issue on their GitHub.

https://github.com/EleutherAI/lm-evaluation-harness/issues/1426

We implemented alternate version of truncation (truncating instruction instead of few-shot samples). This doesn't seem to be helpful on propaganda datasets (7/13 tasks got worse result for CSMPT-100k). So we keep the original truncation for now.
However this might be still an issue for instruction tuned models.


TableModel.set data() should check the input and gracefully handle exceptions

Detailed description

Describe in detail the issue you're having.

I'm seeing ERROR TypeError: Cannot read property 'length' of undefined at TableModel.set [as data] (table-model.class.js:82) in the console when I execute the following code:
ngOnInit() {
  this.myModel = new TableModel();
  this.myModel.data = [];
  this.myModel.header = [
    new TableHeaderItem({ data: 'My column name' }),
  ];
}

I traced down the problem and it looks like the data is not checked for null, undefined or [] values and it fails with an Exception.

What did you expect to happen? What happened instead? What would you like to see changed?

I think it should gracefully convert any falsy input as to no data is provided.

What browser are you working in?

Chrome

What version of the Carbon Design System are you using?

    "carbon-components": "9.8.5",
    "carbon-components-angular": "^1.6.0",

For null and undefined it could make sense to initialize data to empty table [[]].
For [] I'd argue that it can introduce problems, but adding a better error message might be useful.

For me null and undefined are the same as never calling model.data = xxxx and the component would use its defaults in that case.
I understand [] as empty data (no data) as that is literally passing an empty object with no rows. Passing [[]] looks weird and it really means "one row with no data in it" type of a scenario which is essentially the same as no data.


QnAMaker API v4.0 generateAnswer missing?

Can anyone confirm if the generateAnswer method for QnaMaker API v4.0 is not available anymore or is it move to somewhere else. Please help

generateAnswer exists on the QnA maker instance, but not on the QnA Maker api. Please see #127 for more information.


Boolean comparision

"true" and true are not found as differences

Fixed! Thank you, if you find more Bugs, please send me. -> 7b063363d3b4a8ba6e3f2a01b326db39a55b86c8


Add OutputType property to the BrowseObject so that the application prop page can open

The application property page was getting the OutputType of the application through an interface that vslangproj implements and CPS doesn't. Instead, we will get the property from the BrowseObject. I've duplicated the XAML definitions for the BrowseObject and made it override the one in the MSBuild directory so that we can add to it. What we do with the MSBuild one is yet to be figured out.
OutputType property in MSBuild maps to OutputTypeEx in vslangproj and needs to be integers. OutputType itself is a smaller list and CPS doesn't like Enums with repeated values in the XAML file. So I've implemented a IDynamicEnumProvider to do the mapping from OutputTypeEx to OutputType.
Tagging @dotnet/roslyn-analysis

LGTM

:+1: The general approach looks sound, just would like to understand the DisplayName handling.


Dump: Add parser for Symbol

Addressing #1008. The root issue is that dump.parse didn't have a parser defined for object's of type symbol and thus it didn't return a string. This fixes that and adds a test to catch regressions.
Here are some examples of the fix in action. The first test is essentially Symbol("QUnit") === Symbol("QUnit") and the test shows Symbol("QUnit1") === Symbol("QUnit2").

This is great, thanks!


soc: nxp: add flashing configuration

Added a flash runner configuration for rw, mcx, lpc, kinetis and imxrt, used for sysbuild multi-image projects like MCUBoot.
Solved the mass erase issue.
The sysbuild project "west flash --erase" command caused the mass_erase->flash_img1->reset->mass_erase->flash_img2->reset sequence. It was fixed to the mass_erase->flash_img1-> flash_img2->reset sequence.

@butok is there a github issue number for the "solves the mass erase issue"?

@butok is there a github issue number for the "solves the mass erase issue"?

I have not found a reported issue, so no number.
The problem is described here https://docs.zephyrproject.org/latest/build/flashing/configuration.html

I think this is a must for all platforms that need support mcuboot, am I right?

not sure whether it is my board issue, but in frdm_k64f I still get below error
2024-07-03 11:49:54,390 - twister - DEBUG - Flash command: ['west', 'flash', '--skip-rebuild', '-d', '/home/ubuntu/nxp/frdm_k64f/temp/frdm_k64f_twister_test_only/twister-out/frdm_k64f/tests/boot/test_mcuboot/bootloader.mcuboot', '--runner', 'pyocd', '--', '--dev-id', '0240000047784e4500229003d917001be561000097969900']
2024-07-03 11:49:54,392 - twister - DEBUG - DEVICE: frame: IDE 0x0 RTR 0x0 ID 0x1 DLC 0x8
2024-07-03 11:49:54,393 - twister - DEBUG - DEVICE: [00:00:04.001,000] <inf> net_socket_can_sample: Data
2024-07-03 11:49:54,395 - twister - DEBUG - DEVICE: f0 f1 f2 f3 f4 f5 f6 f7                          |........
2024-07-03 11:49:54,396 - twister - DEBUG - DEVICE: [00:00:04.001,000] <inf> net_socket_can_sample: [4] CAN frame: IDE 0x0 RTR 0x<DAPLink:Overflow>
2024-07-03 11:49:54,396 - twister - DEBUG - Launched 8 jobs
2024-07-03 11:49:58,257 - twister - DEBUG - DEVICE: *** Booting MCUboot v2.1.0-rc1-4-g898a1ca64a75 ***
2024-07-03 11:49:58,262 - twister - DEBUG - DEVICE: *** Using Zephyr OS build v3.7.0-rc2-5-g270c99f7c936 ***
2024-07-03 11:49:58,264 - twister - DEBUG - DEVICE: I: Starting bootloader
2024-07-03 11:49:58,264 - twister - DEBUG - HARNESS:Console:EXPECTED(1/5):'I: Starting bootloader'
2024-07-03 11:49:58,270 - twister - DEBUG - DEVICE: I: Primary image: magic=unset, swap_type=0x1, copy_done=0x3, image_ok=0x3
2024-07-03 11:49:58,276 - twister - DEBUG - DEVICE: I: Secondary image: magic=bad, swap_type=0x0, copy_done=0x2, image_ok=0x2
2024-07-03 11:49:58,278 - twister - DEBUG - DEVICE: I: Boot source: none
2024-07-03 11:49:58,282 - twister - DEBUG - DEVICE: W: Failed reading image headers; Image=0
2024-07-03 11:49:58,285 - twister - DEBUG - DEVICE: E: Unable to find bootable image
2024-07-03 11:50:00,805 - twister - DEBUG - DEVICE: *** Booting MCUboot v2.1.0-rc1-4-g898a1ca64a75 ***
2024-07-03 11:50:00,810 - twister - DEBUG - DEVICE: *** Using Zephyr OS build v3.7.0-rc2-5-g270c99f7c936 ***
2024-07-03 11:50:00,812 - twister - DEBUG - DEVICE: I: Starting bootloader
2024-07-03 11:50:00,819 - twister - DEBUG - DEVICE: I: Primary image: magic=unset, swap_type=0x1, copy_done=0x3, image_ok=0x3
2024-07-03 11:50:00,825 - twister - DEBUG - DEVICE: I: Secondary image: magic=bad, swap_type=0x0, copy_done=0x2, image_ok=0x2
2024-07-03 11:50:00,827 - twister - DEBUG - DEVICE: I: Boot source: none
2024-07-03 11:50:00,831 - twister - DEBUG - DEVICE: W: Failed reading image headers; Image=0
2024-07-03 11:50:00,834 - twister - DEBUG - DEVICE: E: Unable to find bootable image
2024-07-03 11:50:03,632 - twister - DEBUG - DEVICE: *** Booting MCUboot v2.1.0-rc1-4-g898a1ca64a75 ***
2024-07-03 11:50:03,637 - twister - DEBUG - DEVICE: *** Using Zephyr OS build v3.7.0-rc2-5-g270c99f7c936 ***
2024-07-03 11:50:03,639 - twister - DEBUG - DEVICE: I: Starting bootloader
2024-07-03 11:50:03,645 - twister - DEBUG - DEVICE: I: Primary image: magic=unset, swap_type=0x1, copy_done=0x3, image_ok=0x3
2024-07-03 11:50:03,652 - twister - DEBUG - DEVICE: I: Secondary image: magic=bad, swap_type=0x0, copy_done=0x2, image_ok=0x2
2024-07-03 11:50:03,653 - twister - DEBUG - DEVICE: I: Boot source: none
2024-07-03 11:50:03,656 - twister - DEBUG - DEVICE: I: Image index: 0, Swap type: none
2024-07-03 11:50:03,764 - twister - DEBUG - -- west flash: using runner pyocd
-- runners.pyocd: Flashing file: /home/ubuntu/nxp/frdm_k64f/temp/frdm_k64f_twister_test_only/twister-out/frdm_k64f/tests/boot/test_mcuboot/bootloader.mcuboot/mcuboot/zephyr/zephyr.bin
[---|---|---|---|---|---|---|---|---|----]
[========================================]
-- west flash: using runner pyocd
-- runners.pyocd: Flashing file: /home/ubuntu/nxp/frdm_k64f/temp/frdm_k64f_twister_test_only/twister-out/frdm_k64f/tests/boot/test_mcuboot/bootloader.mcuboot/swapped_app/zephyr/zephyr.signed.hex
[---|---|---|---|---|---|---|---|---|----]
[========================================]
-- west flash: using runner pyocd
-- runners.pyocd: Flashing file: /home/ubuntu/nxp/frdm_k64f/temp/frdm_k64f_twister_test_only/twister-out/frdm_k64f/tests/boot/test_mcuboot/bootloader.mcuboot/test_mcuboot/zephyr/zephyr.signed.bin
[---|---|---|---|---|---|---|---|---|----]
[========================================]

I think this is a must for all platforms that need support mcuboot, am I right?

Yes. I have added it for all NXP platforms with the MCUBoot support.

not sure whether it is my board issue, but in frdm_k64f I still get below error

Based on the log "Secondary image: magic=bad", you need to do a mass erase.
I have tested this way:

west build -p always -b frdm_mcxn947/mcxn947/cpu0 --sysbuild tests/boot/test_mcuboot
west flash --erase --runner jlink

I think this is a must for all platforms that need support mcuboot, am I right?

I'd agree. Sort of odd that we don't have a more generic way to do this (since this wouldn't only affect NXP), but this seems like the recommended method.

I think this is a must for all platforms that need support mcuboot, am I right?

I'd agree. Sort of odd that we don't have a more generic way to do this (since this wouldn't only affect NXP), but this seems like the recommended method.

So far the flash configuration is used only by Nordic.
Here we ae adding NXP.
Agree, it should be global. It can take much more time to figure out how to achieve it and definitely not in this PR.


Fix #17: Support objects with cycles

Fix #17

Coverage remained the same at 100.0% when pulling 9f141123869b03674913a4a43e954eb15cd5700a on stereobooster:cycles-2 into a8e7172b6c411ec320d6045fd4afbd2abc1b4bde on epoberezkin:master.


Dartium useing querySelector('#any').remove() throw The null object does not have a method 'remove'.

first i append a div with id="any" using the following
document.querySelector('body').appendHtml('

');
then use the following to remove it
querySelector('#any').remove();
result of my test is
The null object does not have a method 'remove'.
I'm using webstorm and dart test package with dartium , what's wrong ?

Looks like the same issue you created 2 days ago #26272

GitHub issues are for bug reports and feature requests.
For support questions please use other channels like the ones listed in https://www.dartlang.org/support/

thx i'll close it


Streaming in and out data live from CARLA

Hi Team, Iam trying to feed in the position of the Ego vehicle live into CARLA and take out the target position data also live from CARLA. I have seen the recording functionality and retrieval of data from the logged file , but I would live to extract the data live during the simulation and sent to an external module. I t would be really helpful if anyone could provide any insights on this .
In order to feed the ego position data live into the scenario , I am planning on editing the manual_control.py script where in the movement data is taken from the keyboard at every world tick. Any suggestions and views on this would also be very helpful. Thanks a  lot in advance.
CARLA version:0.9.11
Platform/OS:Ubuntu 18.04

Hi Team, any insights or suggestions on this issue

Hi @fabianoboril Could you kindly share your insights on the same


[Dewhale] implements #7

[Dewhale] This PR implements #7, created by Dewhale.

[Dewhale]: Based on the image provided, I'll create a static React JSX prototype for a clean and elegant hospital management UI. The design will feature good spacing and a layout that includes a dashboard, a welcome message, statistics, a list of today's doctors, and a calendar for appointments.
This code snippet creates a hospital management UI with a header, main content area, and footer. The main content area is divided into two sections: the larger left section includes a welcome message and statistics, as well as a list of today's doctors, while the right section includes a calendar and a list of appointments. The UI is designed to be clean and elegant, with good spacing and a modern look.

put a different background color


Consider providing Prerequisites section for PicoSoC

(I never dealt with Verilog before, I'm a software dev)
At https://github.com/cliffordwolf/picorv32/tree/master/picosoc (README.md) I see

make hx8ksim
make hx8kprog

I get:

yosys-config: not found
iverilog: not found
yosys: Command not found

Consider providing the Prerequisites section to the README.md, something like
Prerequisites.
sudo apt-get install ..yosys.. ..verilog..

I installed yosys from apt-get, but in the cell_sim.v file I cannot find the SPRAM_256KA...

I installed yosys from apt-get, but in the cell_sim.v file I cannot find the SPRAM_256KA...

You should install yosys-dev also for yosys--config.


⚠ Turnos 24x7 has degraded performance

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

HTTP code: 200
Response time: 2562 ms

Resolved: Turnos 24x7 performance has improved in 1d9166c.


Typora can't be installed correctly

Brief description of your issue
After when I used "winget install Typora".I can't get the result of "winget list Typora".So I can't winget upgrade it after the next update of Typora.
Steps to reproduce
First I used 'winget install Typora.Typora -l "D:\Program Files\Typora"'command，I can use typora to edit md.
Second I used "winget list Typora",it display "can't find the installed package that matches the input criteria "
Expected behavior
I expect it can be list in "Add and Remove Program" or "remove program"of registry.
Actual behavior
it can't be list in "Add and Remove Program" or "remove program"of registry.
Environment
Windows Package Manager v1.1.12701
Windows: Windows.Desktop v10.0.19043.1288
程序包: Microsoft.DesktopAppInstaller v1.16.12701.0

Typora:lastet version of winget

@gorskii ，would you help me look at the problem about typora


Research: Document how to add an existing COVID pipeline HL7 Sender to the UP

Problem statement
The overall migration strategy involves migrating all senders from the CP to the UP.
In order to determine a reliable strategy for migrating COVID senders to the UP, we need to know the process and requirements for successfully adding an existing HL7 Sender to the UP.
What you need to know

Jessica and Dylan(?) should have some documentation on what has happened with HCI/Prophase

Scope

Existing and previous UP HL7 sender onboarding initiatives (for example, HCI/Prophase)

Acceptance criteria

[ ] Determine what is required from the RS team for adding a CP HL7 sender to the UP
[ ] Determine what is required from the CP HL7 sender perspective for being added to the UP
[ ] Determine potential timeline and constraints for adding an existing CP HL7 sender to the UP
[ ] Determine areas that may need further investigation
[ ] Findings documented
[ ] Reviewed with Product and Engineering for alignment

To do

[ ] ...

In process research doc


Okta Integration - Bring Okta OAuth as an available option

Edited package.json for react bug

https://github.com/ReactTraining/react-router/issues/6630

Pull Request Test Coverage Report for Build 423

2 of 63 (3.17%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.5%) to 12.519%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/handlers/auth.go
2
5
40.0%

internal/handlers/auth/okta.go
0
58
0.0%

Totals

Change from base Build 422:
-0.5%

Covered Lines:
169

Relevant Lines:
1350

💛  - Coveralls


Native libraries failing on Windows with UnsatisfiedLinkError [2 XMR]

Currently, native libraries are built for Windows, producing these files:

libmonero-cpp.dll
libmonero-cpp.dll.a
libmonero-java.dll
libmonero-java.dll.a

However, using the native libraries results in an error when using JNI functions: "UnsatisfiedLinkError".
To reproduce the failure in a test:

On Windows, download Visual Studio Code and install Java extensions.
Import the monero-java project into workspace.
Build or download the Windows native libraries to the ./build folder.
Open TestMoneroUtils.java and click the button to run testGetIntegratedAddresses().
The test should succeed, but instead fails with UnsatisfiedLinkError:

I'm trying to debug, it seems that libmonero-java.dll failed to load other dlls

I just put C:\msys64\mingw64\bin the Path environment variable and now it works:

How did you add it to the path?
I tried in Windows command prompt: set PATH=C:\msys64\mingw64\bin;%PATH%
But I'm still seeing the same error in VS Code.

Hm, I tried adding C:\msys64\mingw64\bin to the Path environment variable, but I'm still getting the UnsatisfiedLinkError running the test in VS Code.
Dependency walker looks much better after adding to Path, though:


Several unhandled Exceptions

There are several exceptions within the GuzzleClient which we are not handling at the moment.
It would be good to have at least a few more for different scenarios like the URL is malformed. # @

Addedd support for all know Guzzle exceptions.


Patch still needed? Where is it?

The patch linked in the README (https://review.typo3.org/#/c/40794) is no longer accessible. Is the patch still required? Could you provide the patch as a .diff file here instead?

The patch is not needed anymore


Command failed: iwgetid --raw

$ wifi-password
>>> Command failed: iwgetid --raw

Platform: Ubuntu 16.04

I got the same issue.

Linux Mint 18 (Ubuntu 16.04 based)
Node.js v6.5.0

I don't think this repo is maintained anymore.

@s0nskar, yes, it is. If anyone knows of another way to get the SSID name I'd be happy to fix this issue.

Going to close this in favour of #9.


Add minimal Sinatra example

It doesn't seem to be working if I follow the documentation, I suppose every file and directory has to be carefully named to work? I understand .rb file itself needs to be in an 'api' directory, does the 'config.ru' file also go there? Or does it have to be named 'index.ru' specifically?

As the ZEIT Now platform is focused on static frontends, we will not be adding a Sinatra example. As @m5o has shown, it is technically possible to do so but is not recommended.


Typo in a paragraph

Hello,
I think there is a typo in the following paragraph:
Alternatively, cloud role instance can be helpful for scenarios where a cloud role name tells you the problem is somewhere in your web front end. But you might be running multiple load-balanced servers across your web front end. Being able to drill in a layer deeper via Kusto queries and knowing if the issue is affecting all web front-end servers or instances or just one can be important. intelligent view A scenario when you might want to override the value for cloud role instance could be if your app is running in a containerized environment. In this case, just knowing the individual server might not be enough information to locate a specific issue.
The typo is at the start of:
intelligent view A scenario when you might want to override the value for cloud role instance could be if your app is running in a containerized environment. In this case, just knowing the individual server might not be enough information to locate a specific issue.
Thanks!
Bye
Wouter

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

ID: bfc7b703-b823-5719-36de-0eeaf096b3ca
Version Independent ID: d85431d2-7253-80e9-9036-b62e2e230cca
Content: Application Map in Azure Application Insights - Azure Monitor
Content Source: articles/azure-monitor/app/app-map.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @AaronMaxwell
Microsoft Alias: aaronmax

Thanks for bringing this to our attention. We will investigate it further and update you shortly.

Thanks again for bringing this to our attention. We have updated the Doc and it will be live once its merged.  We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Node recursivity scrape bug fix

Discovered an issue when running a recursive scrape via Node.js. The fix was to ensure that the $ variable is set from artoo.$ in the scrape function.
Provided a unit test to demonstrate fixed recursivity bug when run via Node. Example is taken verbatim from the artoo documentation on
recursivity.

@slashdotdash, you are right indeed.

@Yomguithereal done

Thanks @slashdotdash, merging right now.

@slashdotdash, do you need me to make a npm release quickly?

@Yomguithereal no great rush as I'm just using a local copy or artoo at the moment.


Fixed schema.yaml setting in email_from_field

Description

Checklist

[x] I have reviewed the contributing guidelines.
[ ] I have included unit tests for my changes or additions.
[x] I have successfully run make test-docker with my changes.
[ ] I have manually tested all relevant modes of the change in this PR.
[ ] I have updated the documentation.
[x] I have updated the changelog.

Questions or Comments

Right after I merged this in I realized this doesn't seem right. The value for the field email_from_field will only be a string. Not a list of strings. What the field supports, however, is an elasticvalue for the given field name, where the value can be a string or list of strings. I think this needs rolled back.


remove typo

Hi @sahilg06 thanks for the catch. I think you created a new branch on top of your question branch sahilg06:misc which has caused the commits of the question to be included in this PR. Please remove those changes and keep only the change in ml.yml so that they are kept separate.

Oh! I didn't realize that. Removed extra changes!


Fix failover peer name

Use "dhcp-failover" as failover peer name, if failover != ''

@saz thanks again for the contribution, but we haven't heard from you in a while so we're going to have to close this PR. If this is still something you're interested in can you address the comments and resubmit the PR.


Merge remote-tracking branch 'refs/remotes/spring-io/master'

111

@lazymangod Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.


Readme tip to add a default commodity to your ledger

Makes it way easier to enter amounts and is not necessarily known to all users.
See my confusion and wasted effort in #6
@hpdeifel

Thanks!


Event is not fully rendered

Describe the Bug

The default time period is the last 24 hours. When I change it to the current day and then change it to the last 24 hours, the event will only display the events of the current day.
Database
Umami Cloud
Relevant log output
No response
Which Umami version are you using? (if relevant)
No response
Which browser are you using? (if relevant)
No response
How are you deploying your application? (if relevant)
No response

I think your screenshot is showing 12PM-12PM which includes the last 24 hours. For example our demo.


feature: 新增openapi接口，快捷获取某个审批节点的审批结果

Codecov Report

Merging #1037 (8258866) into v2.6.x_develop (6f821ed) will decrease coverage by 0.05%.
The diff coverage is 11.53%.

@@                Coverage Diff                 @@
##           v2.6.x_develop    #1037      +/-   ##
==================================================
- Coverage           60.63%   60.58%   -0.05%
==================================================
  Files                 469      469
  Lines               30047    30073      +26
==================================================
+ Hits                18218    18220       +2
- Misses              11829    11853      +24

Impacted Files
Coverage Δ

itsm/openapi/ticket/views.py
41.78% <11.53%> (-1.97%)
:arrow_down:

itsm/workflow/validators.py
42.17% <0.00%> (-0.22%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


bug: cannot update ip_access list

I have a clickhouse_service with 2 CIDRs in the ip_access list. When I attempted to remove one from the list, I get the following results:
plan (looks ok):
Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
  ~ update in-place

Terraform will perform the following actions:

  # module.clickhouse.clickhouse_service.this will be updated in-place
  ~ resource "clickhouse_service" "this" {
      ~ iam_role             = "arn:aws:iam::<some_acct>:role/<some_IAM_Role>-Role" -> (known after apply)
        id                   = "caf76259-bcbb-4c20-93cf-dbdbd1a27c97"
      ~ ip_access            = [
          - {
              - description = "" -> null
              - source      = "<redacted>" -> null
            },
            # (1 unchanged element hidden)
        ]
      ~ last_updated         = "Sunday, 10-Dec-23 19:13:39 CST" -> (known after apply)
        name                 = "my-service-name"
        # (9 unchanged attributes hidden)
    }

Plan: 0 to add, 1 to change, 0 to destroy.

apply (fails):
Acquiring state lock. This may take a few moments...
module.clickhouse.clickhouse_service.this: Modifying... [id=caf76259-bcbb-4c20-93cf-dbdbd1a27c97]
╷
│ Error: Error Updating ClickHouse Service
│
│   with module.clickhouse.clickhouse_service.this,
│   on ../libs/clickhouse/main.tf line 8, in resource "clickhouse_service" "this":
│    8: resource "clickhouse_service" "this" {
│
│ Could not update service, unexpected error: status: 400, body:
│ {"requestId":"5cb84d86-2f04-429c-862a-9f5ae5741c1c","error":"BAD_REQUEST:
│ request body.ipAccessList.add[0]: [object]","status":400}
╵
Releasing state lock. This may take a few moments...

Using v0.0.4

Looks like same error found in https://github.com/ClickHouse/terraform-provider-clickhouse/issues/11 but different cause and affecting latest version.

As a workaround (for anyone else stuck on this) I was able to remove the IP access via the Clickhouse Cloud UI, and then a subsequent plan and apply worked without issue.
Obviously, this defeats the purpose of using terraform to define this and manage it as code, but gets me unblocked.

this should be addressed in #34

@Kinzeng is this fixed now? Can we close this issue?


Please sign your releases

Proposal
It is easier than ever to do in 2024 ! You can even do it fully-automated via Github Actions, Github OIDC and Sigstore "keyless" signing.

Useful references:
https://github.com/slsa-framework/slsa-github-generator/blob/main/internal/builders/generic/README.md#provenance-for-goreleaser
https://goreleaser.com/blog/slsa-generation-for-your-artifacts/#slsa-github-generator

Thanks but unfortunately, it's not something we can provide/guarantee given the current build process
I'll close this as a duplicate of the meta issue https://github.com/prometheus/prometheus/issues/11462
You can add your ideas there if you want.


Add Infinity Wallet Connector

We have added support for Infinity Wallet Connector. This allows users to connect with a DApp from within the Infinity Wallet, without needing to use a browser. This feature provides users with a wide range of benefits, such as the ability to have multiple DApps open, or the same DApp open across multiple chains at the same time. When visiting a DApp from an external browser, it will prompt the user via a Deeplink to open that DApp within the Infinity Wallet Browser-less DApps to connect.
A little about Infinity Wallet:
Infinity Wallet is the first desktop non-browser extension DeFi & Web3 wallet, supporting over 150,000+ users and securing over $400,000,000 of users funds. We also support the ability to connect via WalletConnect to any DApp or software when outside of Infinity Wallet Browser-less DApps.

Hello @chadyj, we are following up to see if there is anything else needed? Thanks

@InfinityWallet @arons1 Sorry for the delay, but happy to approve this.
We have been heads down on v2!
Hope you can check it out too.

With stable version 2.0.0 of Web3Modal now released, we are officially dropping support for version 1.x
Due to this this issue/pr was marked for closing. It is highly recommended to upgrade as 2.x will be receiving further updates that will enable functionality for some of our newer sdks like auth and push as well as support for WalletConnect v2 (See this post about WalletConnect v1 being deprecated https://medium.com/walletconnect/walletconnect-v1-0-sunset-notice-and-migration-schedule-8af9d3720d2e)
If you need to continue using Web3Modal 1.x and require this feature/fix implemented, we suggest adding it via forking V1 branch.

With stable version 2.0.0 of Web3Modal now released, we are officially dropping support for version 1.x Due to this this issue/pr was marked for closing. It is highly recommended to upgrade as 2.x will be receiving further updates that will enable functionality for some of our newer sdks like auth and push as well as support for WalletConnect v2 (See this post about WalletConnect v1 being deprecated https://medium.com/walletconnect/walletconnect-v1-0-sunset-notice-and-migration-schedule-8af9d3720d2e)
If you need to continue using Web3Modal 1.x and require this feature/fix implemented, we suggest adding it via forking V1 branch.

No problem, we have actually already submitted a PR for V2 now also https://github.com/WalletConnect/web3modal/pull/834, hopefully it will be accepted a bit quicker than this one

@arons1 awesome, it might be open for a bit, we are actually working on a way to add injected/extension browser management into https://cloud.walletconnect.com similar to mobile and desktop, once that is in I will leave instructions on your pr.


Properly X.509 encode RSA signatures imported from raw bytes

There was confusion about whether the bytes stored in the signature object were already ASN.1 BitString encoded or not. Closes #195.

@JesusMcCloud add a regression test to the branch please

This is also suspicious to me, so please also add a regression test for the parallel publickey encoding.

This is also suspicious to me, so please also add a regression test for the parallel publickey encoding.

Yes, this smells funky

ping me, once this one is ready


Add Code of Conduct

Changes

Create a CODE_OF_CONDUCT from https://www.contributor-covenant.org/version/2/1/code_of_conduct/

Ref

#20

Verify
Verify text online
Blocked

We need a maintainer contact. @benvanwerkhoven @bouweandela @isazi Decision moved to #49
#42 needs to be merged first

Thanks for the review

You forgot to add the contact in case of trouble...

#49 is the decision around the contact.


Rate limit reached only 60/min current 100/min

code: df['davinci'] = df.title.apply(lambda x: get_embedding(x, engine='text-similarity-davinci-001'))
RateLimitError: Rate limit reached for default-global-with-image-limits in organization org-1UIBeDGivAGB5s2IeBxupqGD on requests per min. Limit: 60.000000 / min. Current: 100.000000 / min. Contact<EMAIL_ADDRESS>if you continue to have issues.

I also would like to know whether engine='text-similarity-davinci-001' will provide the same vector size for every different length of words and sentences?

I recommend using text-embedding-ada-002 it's better and far cheaper than text-similarity-davinci-001. See more here: https://openai.com/blog/new-and-improved-embedding-model/
This guide gives advice on rate limits: https://github.com/openai/openai-cookbook/blob/main/examples/How_to_handle_rate_limits.ipynb


Need clarification on uploading multiple file attachments

Would you be so kind as to elaborate upon the process to upload multiple files from a form? I do not seem to have much success configuring the f.input to allow multiple file selections.
Or is this capability not present?

Question answered offline, unfortunately capability for multiple file selection uploads is not a built in feature at this time.


RuntimeError: Found dtype Long but expected Float

dependency versions:
simpletransformers (0.60.9)
torch (1.9.0)

when one tries to create a classification model with just one unique label, this happens. I'm assuming simpletransformers takes care of the tensor type differently when the no. of unique labels is 1 (and opp type, when it's greater than 1 -> usual scenario).
actual stack trace:
2021-07-28 22:24:07:851 slu [train.py:54] INFO Training started.
Epochs 0/1. Running Loss:    0.1531:   0%|                                                                                | 0/2 [00:00<?, ?it/s]
Epoch 1 of 1:   0%|                                                                                | 0/1 [00:00<?, ?it/s]
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/root/jaivarsan/to_be_deleted/yellow/slu/dev/cli.py", line 100, in main
    train_intent_classifier(config, file_format=file_format)
  File "/root/jaivarsan/to_be_deleted/yellow/slu/dev/train.py", line 55, in train_intent_classifier
    model.train_model(
  File "/root/.pyenv/versions/3.8.9/envs/dgblue/lib/python3.8/site-packages/simpletransformers/classification/classification_model.py", line 463, in train_model
    global_step, training_details = self.train(
  File "/root/.pyenv/versions/3.8.9/envs/dgblue/lib/python3.8/site-packages/simpletransformers/classification/classification_model.py", line 720, in train
    loss.backward()
  File "/root/.pyenv/versions/3.8.9/envs/dgblue/lib/python3.8/site-packages/torch/_tensor.py", line 255, in backward
    torch.autograd.backward(self, gradient, retain_graph, create_graph, inputs=inputs)
  File "/root/.pyenv/versions/3.8.9/envs/dgblue/lib/python3.8/site-packages/torch/autograd/__init__.py", line 147, in backward
    Variable._execution_engine.run_backward(
RuntimeError: Found dtype Long but expected Float

supposedly every label should have (strictly) greater than 1 observations too. else one can face this error.
we require dataset validators from our end and, say the reason why it can fail (with warnings maybe).


Apply conversation reference in TurnContext.UpdateActivity

Fixes #2388

Pull Request Test Coverage Report for Build 78891

0 of 0   changed or added relevant lines in 0 files are covered.
351 unchanged lines in 53 files lost coverage.
Overall coverage increased (+0.08%) to 78.571%

Files with Coverage Reduction
New Missed Lines
%

/libraries/integration/Microsoft.Bot.Builder.Integration.ApplicationInsights.Core/ApplicationBuilderExtensions.cs
1
71.43%

/libraries/Microsoft.Bot.Builder.AI.LUIS/LuisApplication.cs
1
95.45%

/libraries/Microsoft.Bot.Builder.Dialogs/Prompts/PromptValidatorContext.cs
1
82.35%

/libraries/Microsoft.Bot.Builder.Dialogs/WaterfallStepContext.cs
1
86.96%

/libraries/Microsoft.Bot.Builder/TelemetryLoggerMiddleware.cs
1
57.58%

/libraries/Microsoft.Bot.Builder.Testing/XUnit/TestDataObject.cs
1
82.35%

/libraries/Microsoft.Bot.Builder/TurnContext.cs
1
45.83%

/libraries/integration/Microsoft.Bot.Builder.Integration.AspNet.Core/HttpHelper.cs
2
90.91%

/libraries/integration/Microsoft.Bot.Builder.Integration.AspNet.Core/ServiceCollectionExtensions.cs
2
94.94%

/libraries/Microsoft.Bot.Builder/ActivityHandler.cs
2
8.33%

Totals

Change from base Build 78755:
0.08%

Covered Lines:
4290

Relevant Lines:
5460

💛  - Coveralls

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.3.1

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.3.1


Fix crash when update() is called inside _draw()

Fixes #55645
The _update_texture_*_changed() functions are called before _enter_canvas(). This causes the message queue to grow indefinitely when flushing if update() is called inside _draw().

_update_texture_*_changed() functions will call update(), inserting a draw call if not pending.
_enter_canvas() always force insert a draw call to the message queue.

This results in two draw calls in the message queue, which is unexpected.
CanvasItem::pending_update can prevent the first draw call from enqueueing another draw call. But it's set to false when the first draw call finished. So the second draw call is able to enqueue another one.
The message queue can't be modified when flushing, it only resets the queue size after consuming all the messages. So doing something that calls update() in _draw() causes the message queue to grow indefinitely.
Godot crashes when trying to report exhausted message queue with MessageQueue::statistics(). Because previous handled messages are already destructed.

Tested PR, it's indeed fixes the crash. LGTM 👍

Thanks!


Vulnerability CVE-2018-25031

Could this vulnerability be a concern?
https://nvd.nist.gov/vuln/detail/CVE-2018-25031

working link: https://nvd.nist.gov/vuln/detail/CVE-2018-25031
This is swagger UI and not NSwag?

Description
Swagger UI before 4.1.3 could allow a remote attacker to conduct spoofing attacks


Automation of icon optimization

Unfortunately, I can't do a PR right now, but I can offer an idea for automatic icon optimization
The first option:
I need a workflow that automatically optimizes the icon after merging a branch, but I doubt that this is possible
The second option:
After I push a commit to the main repository (not a fork) with icon changes, then image optimization is automatically performed using svgo, after which a commit is made on behalf of github-actions
And also to be able to activate the action manually (in order to optimize all the currently unoptimized icons)
I believe that the second option will be the most optimal, because there will be no unwanted optimization of images after each commit while working on PR, and it will also optimize icons that did not come from PR (for example, if the members of the organization added the icon to the main branch themselves)

As every merge of a PR resolves with a new squashed commit on the main branch, I don't see the difference between option 1 and 2. However, if we want to run the optimizer every time when a new commit lands on the main branch and we optimize the icons, we need to push the updated icons either with a separate commit to the branch or adjust the merged commit. Pushing an additional commit will spam the whole commit history with additional commits:
- Add file icon A
- Optimize icon
- Add file icon B
- Optimize icon
- Add file icon C
- Optimize icon

Personally, I don't want to see something like that.
Another possibility is adjusting the history every time, but this would require force pushing of the workflow every time to the history to override the commit with the updated SVG files. This can also have dangerous side affects and is a bit risky.
Another approach could be a pre commit hook that runs the optimizer on the icons locally (see #1526). Would be maybe the best approach, but I don't see the need at the moment of why the bulk optimization before a release isn't enough.


fix: increase polling interval if we're using Azure Key Vault

The default polling of 1000ms is too short and triggers the rate limiting.

Hi @hervelemeur. Thanks for your PR.
I'm waiting for a jenkins-x or todo member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign rawlingsj
You can assign the PR to them by writing /assign @rawlingsj in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/assign @rawlingsj

We could probably make this the default for all, lemme update your PR if you don''t mind...

/ok-to-test

/hold

We could probably make this the default for all
Need to take in account the env: declared below too, updating my pr

@hervelemeur: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

kube
d2e5590930728303e9fd4610a4516f505dc895a8
link
/test kube

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@hervelemeur: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

kube
d2e5590930728303e9fd4610a4516f505dc895a8
link
/test kube

gsm
d2e5590930728303e9fd4610a4516f505dc895a8
link
/test gsm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@hervelemeur: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

kube
d2e5590930728303e9fd4610a4516f505dc895a8
link
/test kube

gsm
d2e5590930728303e9fd4610a4516f505dc895a8
link
/test gsm

tls
d2e5590930728303e9fd4610a4516f505dc895a8
link
/test tls

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@hervelemeur: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

kube
d2e5590930728303e9fd4610a4516f505dc895a8
link
/test kube

gsm
d2e5590930728303e9fd4610a4516f505dc895a8
link
/test gsm

tls
d2e5590930728303e9fd4610a4516f505dc895a8
link
/test tls

vault
d2e5590930728303e9fd4610a4516f505dc895a8
link
/test vault

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

/hold cancel
The 2 first if concerning AWS could be regrouped as they only have their urls as difference, but not sure at the moment of the syntax of an if eq or () (), I've let them as they were.

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: rawlingsj
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [rawlingsj]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


ci: remove thread parallelism

Fixes https://github.com/nix-rust/nix/issues/529, also refer to
https://github.com/nix-rust/nix/issues/529 which first added this
for the old CI infrastructure.

You meantion the same issue twice. Do you want me to homu r+ this? Or do you just want to merge this stuff yourself?

r+ is fine.  It doesn't look like this is enough to resolve all issues on its own, so I'm still investigating.  I need to update this as well as it should be CARGO_TEST_THREADS

Your answer confuses me. But @homu r+

:pushpin: Commit d0a151b has been approved by fiveop

:hourglass: Testing commit d0a151b with merge 01ccd59...

@homu r-

:boom: Test timed out

@posborne can you double check the issue references in your PR / commit message?

:umbrella: The latest upstream changes (presumably c3d5eda) made this pull request unmergeable. Please resolve the merge conflicts.

Have we figured out if this is still necessary?

I think that PR #579 fixes the root problem in a better way.  At least, it does for now.

But it didn't change anything in CI, so I guess we're still trying to figure out the root of those errors, yes?

Correct.  I don't know why those tests are still failing in CI.

Then let's close this PR in favor of a separate issue to figure that out.


[Error] Bot crashes after network issue

2018-06-24 23:36:05 Error HTTPSConnectionPool(host='api.bitfinex.com', port=443): Max retries exceeded with url: /v1/offer/new (Caused by NewConnectionError('<requests.packages.urllib3.connection.VerifiedHTTPSConnection object at 0x7f3ee7aa0e50>: Failed to establish a new connection: [Errno 113] No route to host',)) Requesting https://api.bitfinex.com/v1/offer/new
--
2018-06-24 23:36:05 Error HTTPSConnectionPool(host='api.bitfinex.com', port=443): Max retries exceeded with url: /v1/offer/new (Caused by NewConnectionError('<requests.packages.urllib3.connection.VerifiedHTTPSConnection object at 0x7f3ee7aa0e50>: Failed to establish a new connection: [Errno 113] No route to host',)) payload:{'nonce': '152990129624939', 'direction': 'lend', 'request': '/v1/offer/new', 'rate': '23.0084466', 'period': 9, 'currency': u'EOS', 'amount': '13.97391293'}

2018-06-24 23:36:05 Error HTTPSConnectionPool(host='api.bitfinex.com', port=443): Max retries exceeded with url: /v1/offer/new (Caused by NewConnectionError('<requests.packages.urllib3.connection.VerifiedHTTPSConnection object at 0x7f3ee7aa0e50>: Failed to establish a new connection: [Errno 113] No route to host',)) payload:{'nonce': '152990129624939', 'direction': 'lend', 'request': '/v1/offer/new', 'rate': '23.0084466', 'period': 9, 'currency': u'EOS', 'amount': '13.97391293'}
2018-06-24 23:36:05 Error HTTPSConnectionPool(host='api.bitfinex.com', port=443): Max retries exceeded with url: /v1/offer/new (Caused by NewConnectionError('<requests.packages.urllib3.connection.VerifiedHTTPSConnection object at 0x7f3ee7aa0e50>: Failed to establish a new connection: [Errno 113] No route to host',)) Requesting https://api.bitfinex.com/v1/offer/new
Traceback (most recent call last):
  File "lendingbot.py", line 114, in <module>
    Lending.lend_all()
  File "/poloniexlendingbot/modules/Lending.py", line 252, in lend_all
    usable_currencies += lend_cur(cur, total_lent, lending_balances, ticker)
  File "/poloniexlendingbot/modules/Lending.py", line 475, in lend_cur
    raise msg
ConnectionError: HTTPSConnectionPool(host='api.bitfinex.com', port=443): Max retries exceeded with url: /v1/offer/new (Caused by NewConnectionError('<requests.packages.urllib3.connection.VerifiedHTTPSConnection object at 0x7f3ee7aa0e50>: Failed to establish a new connection: [Errno 113] No route to host',))

v326
 Unhandled error, please open a Github issue so we can fix it!
Error during new loans notification: HTTPSConnectionPool(host='api.bitfinex.com', port=443): Max retries exceeded with url: /v1/credits (Caused by NewConnectionError('<requests.packages.urllib3.connection.VerifiedHTTPSConnection object at 0x7f3ee7be1f50>: Failed to establish a new connection: [Errno 113] No route to host',)) Requesting https://api.bitfinex.com/v1/credits
Traceback (most recent call last):
  File "lendingbot.py", line 164, in <module>
    log.notify("{0}\n-------\n{1}".format(ex, traceback.format_exc()), notify_conf)
  File "/poloniexlendingbot/modules/Logger.py", line 168, in notify
    send_notification(msg, notify_conf)
  File "/poloniexlendingbot/modules/Notify.py", line 125, in send_notification
    post_to_slack(msg, nc['slack_channels'], nc['slack_token'], nc['slack_username'])
  File "/poloniexlendingbot/modules/Notify.py", line 46, in post_to_slack
    response = urllib2.urlopen(url, enc_post_data)
  File "/usr/lib/python2.7/urllib2.py", line 154, in urlopen
    return opener.open(url, data, timeout)
  File "/usr/lib/python2.7/urllib2.py", line 429, in open
    response = self._open(req, data)
  File "/usr/lib/python2.7/urllib2.py", line 447, in _open
    '_open', req)
  File "/usr/lib/python2.7/urllib2.py", line 407, in _call_chain
    result = func(*args)
  File "/usr/lib/python2.7/urllib2.py", line 1241, in https_open
    context=self._context)
  File "/usr/lib/python2.7/urllib2.py", line 1198, in do_open
    raise URLError(err)
urllib2.URLError: <urlopen error [Errno -3] Temporary failure in name resolution>

Caused by a temporary outage in your internet/DNS, all we would be able to do is sit there and wait for your internet to come back on, which might not be a bad idea.

Maybe I was not clear. When this happens, the application crashes. This is not a handled error so even if the network/internet comes back up, it won't matter. You need to manually terminate the execution and restart the program. So though we could talk about caching DNS results or what not, this error needs to be handled maybe with a minute delay and try again and avoid outright crashing.


Added missing return statement

Description
Added missing return statement
Type of change

[ ] Refactor
[ ] New feature
[x] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library

/lgtm

/approve


Add routerPath to context

I just add the path of the layer in the context ctx during the memo middleware. The last value set to ctx.routerPath is the route not calling next(), for my usage of koa-router it corresponds to the route I call the matching route.
Maybe this help for #444 ?

Could you remove package-lock.json from this PR?

This will also need some documentation about the information being exposed at ctx.routerPath. Please reopen when the PR is updated! Thanks.


Replacing Hover with Click to view pop-over below screen-size 768px

Hover is not suitable for Mobile view, On desktop it works perfect.
Use Case

[x] If Screen size is greater than 768 px , hovering on track should show pop-over.

[x] If Screen size is lesser than 768px means smaller screens, Clicking on track should Show pop-over.

@championswimmer @juslee Solved here https://github.com/fossasia/open-event-webapp/pull/315

Done !!!


Groups not switching to light mode

Context
Components dont seem to have a light mode version
Expected Behavior
Should change when in light mode
Current Behavior
Doesnt change when in light mode
Screenshots
https://github.com/codu-code/codu/assets/46611809/8f599b29-87ca-4566-9734-b0ff9c1e609c
Steps to reproduce
Please provide detailed steps for reproducing the issue

Go to http://localhost:3000/hub?tab=groups
Switch between light mode and dark mode

Additional info

This can prob be closed now I think since the above was merged @NiallJoeMaher

Thanks @realshaunoneill


[Issue-2205] Update extension to MV3

Related issue: #2205

🚀 Deployed on https://pr-2363--sw-web-runner.netlify.app


Support Blogs not in English

Some blogs are not in English! We had a submission of an indie blog all in Chinese: https://github.com/artlung/blogofthe.day/pull/27
In the spirit of sharing interesting blogs, doing it all English may be implied by a domain name in English, but scoping the blog of the day by language could be great.
so:

https://blogofthe.day/en
https://blogofthe.day/zh
https://blogofthe.day/fr
https://blogofthe.day/de

I believe we have a few blogs with some multilingual content. @alexture for example.
Thanks @capjamesg for offering to think through solutions to this!

I'll get us set up with separate feeds per region, and write some docs on how to add a region. @huhexian if you want to work on a translated index.html file for Chinese, that would be great! We should save these as index-{countrycode}.html for consistency.


webpack v4 - compatibility

thx

webpack >= v2.0.0 has 'native'/direct JSON support. Since webpack >= v4.0.0  it's a module.type  (no JS wrapper needed anymore). You can remove the json-loader from your setup. Otherwise please provide more info about what 'compatibility'  exactly means here :).
⚠  It's possible the json-loader causes issue with webpack >= v4.0.0, because it issues a JS Module (module.type = 'javascript/auto'), which isn't needed anymore/doesn't work with module.type = 'json' (JSONParser)

This (removing json-loader) fixed an issue I was having with webpack throwing errors about Unexpected token in package.jsons.

@zjr I will add a note to the README when I have a few minutes :)

Thanks!

@michael-ciniawsky removing the loader works. Thank you all.

Is there a possibility to override the webpack-builtin json-loader? I have my own and it looks like the output of mine is being fed into json-loader (which then errors out, because my loader returns JS code, not JSON code)

@retorquere https://github.com/webpack/webpack/issues/6572


Upgraded to Gradle 7 and other dependencies

Bumped major version given that we're updating Gradle to next major
Latest Gradle version 7
Latest Gradle plugin publish version
More current spock version that works with Gradle 7
Gradle wrapper files

@koral--, interested in joining the Shipkit team / getting GitHub permissions to merge PRs?


Pass the node to the supstance drag event listener

In my use case, when dragging a line, some other related lines also move. I use it to implement fibonacci trend, so the registered drag event listener needs to know which line is dragged. So add the node as a second argument.

Hi @sf-wind
Thanks again for the contribution.
This appears to have pull request #100 merged in already making a bit harder to review. I also won't be able to merge it in until #100 is merged in. Any chance those commits could be taken out? Otherwise this one will have to wait until then.
cheers
andre

Thanks. This is my first pull request and I did it on my master branch by mistake.
As for the d3 api, I'm not quite familiar with and I will definite take a look.


Grafana Dashboard

Hello,
thank you very much for maintaining this version of the rtorrent-exporter, it's a great addition to my setup. Since I'm not really a big user of grafana, it has been a struggle creating a nice looking dashboard. I've managed to do so, but I believe many users feel the same, therefore it would be great if you could share your config.
Thank you very much in advance :)

I'm not sure that I am exactly the best dashboard-er out there, but I added an example here: https://github.com/aauren/rtorrent-exporter/blob/master/resources/grafana-dashboard.json
I've also tried uploading it to grafana.com: https://grafana.com/grafana/dashboards/22581 however, my experience is that it can take several weeks to get them to actually show it.


presentationRequest protocol

Same as credentialOffer :
when the user does not have the requested credential, display the card with a link to get it

I need test URLs for this...
https://talao.co/wallet/test/presentationRequest
The above URL is not working...
And where will the required credentials be listed.. inside the credential manifest???
@ThierryThevenet @hawkbee1

you can test it here
https://talao.co/sandbox/presentationRequest
the protocole is here
https://sandbox-doc.readthedocs.io/en/latest/altme.html#altme-wallet-presentation-request-query-types

it is less complex that the imput descriptors with JsonPath

Size: S/M


2.3.3

2.3.3 (4 points)
Use that pattern in conjunction with re.sub and list comprehension to clean the employer name columns in each dataset. Save the new columns as name_clean in each. Then, use row subsetting to (1) subset to rows that changed names and (2) for:

debar_clean print the Name and name_clean columns

jobs print the EMPLOYER_NAME and name_clean columns

Make sure to use the uppercase versions of the variables

Do we overwrite the data to be only the cases where the names have changed, or just subset to show examples?

@sanhatahir - I don't think I understood your question completely - but the subseting is only as an example. However, the name_clean column is required in the next set of questions as well. Prof. @rebeccajohnson88 can you confirm this ?

sorry just saw this--- for @sanhatahir you:

keep all rows in the dataframe (so keep both the ones that didnt change and the ones that changed) but
print just one that changed for us to check


Fork regex issue

I am not entirely sure, so lets first settle it here if it is a bug or not.
This works
#[regex="~s[0-9]+(\\.\\.[0-9a-f\\.]+)"]
#[regex="~s[0-9]+"]
LiteralRelDate

But collapsing the above regex into single one
#regex=["~[0-9]+(\\.\\.[0-9a-f\\.]+)?"]

Gives unexpected error on things that would otherwise match into LiteralRelDate in the former case.

Definitely a bug, can you provide tests / examples for breakage?

Will send a PR soon; One question though, how do you work around
--> tests/tests/properties.rs:2:5
  |
1 | use logos::Logos;
  |     ------------ previous import of the macro `Logos` here
2 | use logos_derive::Logos;
  |     ^^^^^^^^^^^^^^^^^^^ `Logos` reimported here
  |
  = note: `Logos` must be defined only once in the macro namespace of this module
help: you can use `as` to change the binding name of the import
  |
2 | use logos_derive::Logos as OtherLogos;
  |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

or do you not see it? I am working from nigthly, and for every PR I have to comment it out
the use logos_derive for testing, and then turn it back on.


missing_data errors on hubble.load

When running salt-call hubble.load, there are missing_data errors for windows and debian:
missing_data:
    ----------
    /cis/debian-8-level-1-scored-v1.0.0.yaml:
        while parsing a block mapping
          in "/var/cache/salt/minion/files/base/hubblestack_nova/cis/debian-8-level-1-scored-v1.0.0.yaml", line 8, column 15
        expected <block end>, but found '<scalar>'
          in "/var/cache/salt/minion/files/base/hubblestack_nova/cis/debian-8-level-1-scored-v1.0.0.yaml", line 9, column 49
    /cis/windows-2008r2-level-1-scored-v1.yaml:
        while parsing a block mapping
          in "/var/cache/salt/minion/files/base/hubblestack_nova/cis/windows-2008r2-level-1-scored-v1.yaml", line 1313, column 3
        expected <block end>, but found '<block mapping start>'
          in "/var/cache/salt/minion/files/base/hubblestack_nova/cis/windows-2008r2-level-1-scored-v1.yaml", line 1838, column 4
    /cis/windows-2008r2-level-1-scored-v3.0.0.yaml:
        mapping values are not allowed here
          in "/var/cache/salt/minion/files/base/hubblestack_nova/cis/windows-2008r2-level-1-scored-v3.0.0.yaml", line 334, column 41
    /cis/windows-2012r2-level-1-scored-v2.0.0.yaml:
        mapping values are not allowed here
          in "/var/cache/salt/minion/files/base/hubblestack_nova/cis/windows-2012r2-level-1-scored-v2.0.0.yaml", line 334, column 41

Thanks for opening this.
I'll also assign myself, I'll look at the debian failures.

#200 fixes the debian profile. I'll leave the others to @madchills.

This should be fixed in my most recent round of bug fixes


Clustering supported in 0.3.x?

Hi,
I have read somewhere that clustering is experimentally supported in v0.3 - unfortunately, I could'nt find any hint how to turn it own.
Can you help me out? ;-)
Thanks for your great work,
uli

ran into the same issue.
why can't the cluster node have edges?
@ulilicht @niyue how have you handled?

@iampeter
I'm sorry I don't work with dagre any more as i'm on a different project now. Unfortunately I don't remember what the issue was ;-/  Good luck!

Has anyone managed to get this working? @ulilicht 's plunk doesn't work, gives the same error.
http://jsfiddle.net/snowystinger/fq0d7w5r/ (commented out the edges I want) I'm trying to do it with the plain version of Dagre, not sure if that makes a difference, both seem to give the same.

Would there be any downside to changing the Dagre project to allow a null check here...
return (label.rank = rank);

Playing around it looks like that function is being constantly called. Does anyone know why graph.node(3) returns null for clusters?

Subgraphs work for me.  Here is a demo: http://bitdivine.github.io/dotfile-editor/dotfile-editor.html#http://bitdivine.github.io/dotfile-editor/sample.dot

any solution?
thanks!


3xx status code that represent successful redirects should be invisible to client code

I discovered, at least with the 301 status code, that secret-agent returns a goto response's status code of 301
for (some/all) pages that redirect successfully via 301, rather than the expected 200 as seen with playwright.
However, if we wait for DomContentLoaded and then use the plugin to evaluate the dom, we do get the expected
behavior, so that it appears that with one or more 3xx status codes, we have to ignore them as if they were 200s.
The example url that always shows this behavior is:  https://httptoolkit.tech/blog/unblocking-node-with-unref

Here is a test snippet:
import { Agent, ConnectionFactory, ConnectionToCore, LocationStatus } from 'secret-agent';
import Path from 'path';
import Resource from '@secret-agent/client/lib/Resource';
import ResourceResponse from '@secret-agent/client/lib/ResourceResponse';

let sharedConnection: ConnectionToCore = null;

function getConnection (maxAgents: number): ConnectionToCore {
  if (sharedConnection !== null) return sharedConnection;
  sharedConnection = ConnectionFactory.createConnection({ maxConcurrency: maxAgents });
  return sharedConnection;
}

async function test(): Promise<void> {
  const agent: Agent = new Agent({
    connectionToCore: getConnection(1),
  });

  try {
    let pageResource: Resource;

    agent.use(Path.resolve(__dirname, 'simple-plugin'));
    pageResource = await agent.goto('https://httptoolkit.tech/blog/unblocking-node-with-unref');
    const pageResponse: ResourceResponse = pageResource.response;
    const status: number = await pageResponse.statusCode;
    console.log(`BUG: goto response status: ${status}, but expecting 200 as with playwright`);

    await agent.activeTab.waitForLoad(LocationStatus.DomContentLoaded);

    // @ts-ignore
    const html: string = await agent.evaluate(0, () => {
      // @ts-ignore
      const win: any = window;
      return win.document.documentElement.outerHTML;
    });

    // the idea is that if all the snippets are found in the eval result
    // it appears that secret-agent did what it supposed to, but IMO
    // it should return a status of 200, not 301 when the document body
    // is an actual 3xx document.
    //
    const snippets: string[] = [
      `It holds a queue of tasks to run`,
      `Occasionally you want to break out of that model`,
      `but it's very annoying to have to remember to carefully shut`,
      `This functions like a flag you can set on your timers`,
      `Although this can let you skip complicated cleanup processes`,
      `This can come with a small performance cost,`,
      `discover that your app is unexpectedly exiting half way through`,
    ];

    // ... and yes, all the flags are true, so this bug is an inconvenience
    // in comparison to playwright, not a showstopper
    const flags: boolean[] = snippets.map(snippet => html.indexOf(snippet) >= 0);
    console.log(`found flags for dom's html`, flags);

  } catch (e) {
    console.log('failure', e);
  } finally {
    await agent.close();
  }
}

(async () => {
  await test();
})();

and here is the simple-plugin.ts code:
import ClientPlugin from '@secret-agent/plugin-utils/lib/ClientPlugin';
import CorePlugin from '@secret-agent/plugin-utils/lib/CorePlugin';
import { ISendToCoreFn } from '@secret-agent/interfaces/IClientPlugin';
import { IOnClientCommandMeta } from '@secret-agent/interfaces/ICorePlugin';
import { IPuppetPage } from '@secret-agent/interfaces/IPuppetPage';
import { IPuppetFrame, ILifecycleEvents } from '@secret-agent/interfaces/IPuppetFrame';

export class MyClientPlugin extends ClientPlugin {

  static readonly id = 'my-evaluate-plugin';
  static coreDependencyIds = [MyClientPlugin.id];

  public onAgent(agent, sendToCore: ISendToCoreFn): void {

    agent.numFrames = (): Promise<number> => {
      return this.execFunc(sendToCore, 'numFrames');
    };

    agent.evaluate = (frameIndex: number, callback: Function, ...args: any[]): Promise<any> => {
      return this.execFunc(sendToCore, 'evaluate', frameIndex, callback, args);
    };
  }

  public onTab(tab, sendToCore: ISendToCoreFn): void {

    tab.numFrames = (): Promise<number> => {
      return this.execFunc(sendToCore, 'numFrames');
    };

    tab.evaluate = (frameIndex: number, callback: Function, ...args: any[]): Promise<any> => {
      return this.execFunc(sendToCore, 'evaluate', frameIndex, callback, args);
    };
  }

  private async execFunc(
    sendToCore: ISendToCoreFn,
    funcName: string,
    frameIndex?: number,
    callback?: Function,
    args?: any[]
  ): Promise<any> {
    const callbackStr: string = callback ? callback.toString() : undefined;
    return await sendToCore(MyClientPlugin.id, funcName, frameIndex, callbackStr, args);
  }
}

export class MyCorePlugin extends CorePlugin {

  static readonly id = 'my-evaluate-plugin';

  private hasWaited: boolean[] = [];

  private async getAwaitedFrame(page: IPuppetPage, frameIndex: number): Promise<IPuppetFrame | null> {
    if (frameIndex < 0) throw new Error('frameIndex is negative');
    const frames: IPuppetFrame[] = page.frames;
    if (frameIndex >= frames.length) return null;
    const frame: IPuppetFrame = frames[frameIndex];
    if (!frame) throw new Error('frameIndex in bounds but frame is ' + frame);
    if (!this.hasWaited[frameIndex]) {
      const key: keyof ILifecycleEvents = 'load';
      try {
        await frame.waitForLoad(key);
      } catch (e) {
        console.log('getAwaitedFrame error', e);
        throw e;
      }
      this.hasWaited[frameIndex] = true;
    }
    return frame;
  }

  public async onClientCommand(
    { puppetPage }: IOnClientCommandMeta,
    funcName: string,
    frameIndex: number,
    callback: string,         // this is the stringified value of the callback given to execFunc above
    args: any[]
  ): Promise<any> {
    const page: IPuppetPage = puppetPage;

    if (funcName === 'numFrames') {
      return page.frames.length;
    }

    if (funcName === 'evaluate') {
      const frame: IPuppetFrame = await this.getAwaitedFrame(page, frameIndex);
      if (!frame) return null;
      const expression: string = `((args) => {
        const fn = ${callback};
        const result = fn.apply(null, args);
        return result;
      })(${JSON.stringify(args)})`;
      try {
        return await frame.evaluate(expression, true);
      } catch (e) {
        console.log(`evaluate error: ${e.toString()}, frameIndex: ${frameIndex}`)
        throw e;
      }
    }

    throw new Error('invalid funcName: ' + funcName);
  }
}

Thanks for reporting this.


Can't write "e", "+" or "-" in Unstable_NumberInput BaseUI

Related page
https://mui.com/base-ui/react-number-input/
Kind of issue
Other
Issue description
Hi, in the component introduction, it's written :
Allowing certain non-numeric characters ('e', '+', '-', '.') and silently discarding others
But in the demo or using the component in my own project, it's impossible to use these characters.
Context
Writing non-numeric authorized character in number input
Search keywords: number input base-ui
Search keywords:

In the new Number Field in v1.0.0-alpha.1, we only allow - and ., and + if the formatting permits it.
I believe e isn't allowed at all, even if you've set scientific as a formatting option. However, I understand it's useful to use scientific notation as a shortcut (e.g. it's easier to input 1e9 than<PHONE_NUMBER>, and I do use this feature in Google Sheets), so we can look into supporting this.

Would be a cool feature, and above all in the component introduction you're talking about the fact that e is authorized.

And the . character is not allowed either.

The one on the live website underwent an API rewrite in our new alpha version (@base_ui/react@1.0.0-alpha.1), so it's been recently replaced.
You can take a look at the new one's docs here. It allows . (including , if the locale permits), and + if formatting permits, but not e yet as I mentioned above.


Fix querying of gda, fois and observations by adding urlencode.

Since owslib 0.22.0 the openURL method expects a string as data for GET requests (calls unquote())

@CarstenHollmann Is this change backward compatible with owslib versions < 0.22.0?

@MartinPontius It is backward compatible with owslib versions 0.22.0.


User with multiple sessions does not show in occupants list

The XMPP MUC specification allows a user to enter a room from different resources, using the same nickname. This is useful, for instance, when you want to be in the room from both your laptop as well as your mobile phone. See http://xmpp.org/extensions/xep-0045.html#enter-conflict
In its occupants list, Candy does not show users that are logged in with more than one device (using the same nickname).

I can confirm this is the case.


Useful API functions

I had need some functionality like getting Country  by isoCode and callingCode, when i  want to show country image or name that user has selected it before so I create  two functions.
let viewController = CountryPickerViewController()
let germany = viewController.getCountry(callingCode: "+49")
let turkey = viewController.getCountry(isoCode: "TR")

Hope this is not just my need :)

I think this is a very good idea. I feel like, ideally the finding methods would be in Country. SO something like Country.find(isoCode: "TR") and Country.find(callingCode: "+90"). This will allow you to use this info in model contexts without instantiating a view controller.
To do this, we could move the country creating, JSON parsing etc. from CountryDataSource to Country.
What do you think about that?
PS: I had an not pushed commit setting the  podspec platform to 9.0. So this is a conflict now, sorry about that. You can remove that change.

@kmikael
I think that would be a better aproach. Creating an instance of viewcontroller for getting country is redundant.

@okankocyigit Awesome, would you be interested in going forward with those changes? I can then merge them into master and make  new release.

@kmikael, sure i will when I have time.

@kmikael it seems to be ok for me. Could you please check if I did something wrong?. still a newbie 😄

Thanks, @okankocyigit! 👍🏻 I changed the way the locale works a little to make it less global and added a few test.