add a method of passing params

like make accepts KEY=value i could add this too, that would override env

c8670ab90382af45217cc5623b6f58e863d74943


add diff/same accepted param to duplicate API

see https://github.com/Sp2000/colplus-frontend/issues/324

@thomasstjerne API GET parameter also renamed from parentDifferent to acceptedDifferent


 What appears to have happened is that the code now skips this block that ensures a pd.MultiIndex. So obs in this test, while functionally identical to demand, has a different index class and thus fails assert_qty_equal().
This fix is probably upstream, this is just to track.

@glatterf42 my preferred temporarily mitigation here would be to install genno<1.23 in the CI workflows.

We currently have two failed tests, but when all tests first started failing, there was only one. This was on Saturday and I found these version changes:

dask 2024.1.0 -> 2024.1.1
genno 1.22.0 -> 1.23.0
coverage 7.4.0 -> 7.4.1

On Sunday, the second test failure was introduced when some additional dependency updates came along:

overrides 7.6.0 -> 7.7.0
pytest 7.4.4 -> 8.0.0

The relevant changes in pytest might be detailed here: https://docs.pytest.org/en/stable/changelog.html#old-deprecations-are-now-errors

Partially/temporarily addressed by #789.

This should be addressed in genno 1.23.1, just released.


Ethical Engineering

Our increasingly connected world, along with the promises of Big Data and Cloud Computing, offers us multitudes of opportunities to model the world and build powerful technology to improve not only the companies we work for but the lives of many. Yet far too often we, as engineers, focus on providing a solution to the problem at hand and overlook the problems that the solution itself might cause. In this talk, Christopher examines some of the issues our world throws up, and asks what it takes to engineer ethically.

Not sure if this is wholly clear that it's a talk submission, but it is ;)

Hi @chrishoult, I'd be excited to hear this talk, and it could work well in our September event (https://github.com/jsoxford/jsoxford.github.com/issues/97).

Hey @spikeheap I'm running PHP Berkshire that evening, I'm afraid - are you always third Wednesday? Or does it vary? Probably should have checked with @benfoxall the other day :)

Hey @chrishoult, no worries! We haven't pencilled in events post-September yet, but they vary, normally towards the middle of the month.

Hi @chrishoult, sorry for the looooong wait. We're putting together our October event for Wednesday 14th October. Are you free that evening, and are you still keen to give this talk? If it clashes with PHP Berkshire again let's coordinate and we can shift the date of our November or January talks to make things easier.

@spikeheap Thanks for getting back to me - unfortunately I've booked myself out to be in a show that night! Can we look at November or January? They're relatively free for me now - PHP Berkshire is the third Wednesday of every month.

Hi @chrishoult, no worries – we'll make it happen soon!

@choult thanks for this! It was great to get you along finally. #phptakeover


Mobile/compact menu does not behave like other ‘hamburger menus’

Describe the bug
Compact menu does not behave like a typical compact menu
To Reproduce
Steps to reproduce the behavior:

https://openfl.readthedocs.io/en/latest/index.html
Resize window to a narrow enough width to get the mobile/compact layout
Use keyboard, mouse and screenreader to interact with the 'hamburger menu' element

Notice:
a.	Functions as an expand/collapse for the mouse user
b.	Menu button isn’t keyboard accessible, doesn’t support aria-expanded
c.	Items in the sidebar are visually hidden but not programmatically hidden; keyboard user can tab to hidden / collapsed items
Expected behavior
Hamburger menu icon should present a menu that contains same menu items as are seen in desktop OR element should be represented as a simple show/hide for a collapsed sidebar, and when the sidebar is collapsed its elements shouldn't be able to take keyboard focus because they're not visible
Screenshots

Desktop:
This issue is not OS or browser specific, but for tracking purposes, testing was done on Win11, in Edge Version 114.0.1823.58 (Official build) (64-bit), and the screenreader in use was the common 3rd party screenreader NVDA (http://www.nvaccess.org).
Smartphone:
This issue is not device, OS, or browser specific.
Additional context
This is an accessibility issue, please annotate this bug with the 'accessibility' label.

Closing. As mentioned by @fstrr, this is an issue with Read The Docs platform.


Export Resource Metadata Labels

Background
My company has an internal monitoring/alerting solution that serves both our on-premise
and cloud needs; it's backed by Prometheus and Grafana.  We get Cloud Monitoring (née
Stackdriver) metrics into our internal solution using this exporter.
I'm currently building some Grafana graphs for Dataflow jobs.

Metric types under the dataflow.googleapis.com
namespace can be filtered down to a specific Dataflow job using the job_name label
since this label's value includes the Dataflow job name.  (Technically, the job_name
label is attached to the dataflow_job
monitored resource type).
Metric types under the compute.googleapis.com
namespace can be filtered down to a specific Dataflow job using the instance_name
label since this label's value includes the Dataflow job name.
Metric types under the agent.googleapis.com
namespace CANNOT be filtered down to a specific Dataflow job.  None of the labels
include the Dataflow job name.  The closest label that gives me what I need is the
instance_id label that comes attached to the gce_instance monitored resource type.
instance_id is numeric and there's no way to know what Dataflow job it belongs to.

Because I can't filter down to a specific Dataflow job with the agent.googleapis.com
metric types, I can't build a complete Grafana dashboard that visualizes all the metrics
I care about.  Because of this, I still have to go back to using a combination of
Dataflow's Job Metrics UI and Cloud Monitoring's Metrics Explorer UI just to visualize
those metrics alongside others.  I'm taking on the overhead of context-switching between
3 different monitoring tools.  This slows down any analysis work that I want to do.
Request
As I understand it, in addition to the labels that are attached to metric types and
monitored resource types, Cloud Monitoring also attaches
resource metadata labels
to the monitored resource types.  Specifically, the metadata.system_labels.name label
includes the GCE instance name which also includes the Dataflow job name.  I would be
able to use this label to filter the agent.googleapis.com metric types down to a
specific Dataflow job.
The Stackdriver exporter doesn't seem to export these resource metadata labels though.
Is there a way we can start exporting these labels by default?  Or, add a configuration
that allows users to enable collection of these extra labels?
References:

LOC where this feature might need to be implemented
These issues look related, but not exactly the same (let me know if I'm wrong though): https://github.com/prometheus-community/stackdriver_exporter/issues/78, https://github.com/grafana/grafana/issues/18750, https://github.com/grafana/grafana/issues/18887

Happy to be corrected on anything stated here.

It would be nice to have something similar to aws_resource_info from the cloudwatch-exporter. That has allowed me to create custom tags on the aws resources themselves, i.e. owner, so that I can come up with a generic alerting rule and pass it along to alertmanager for proper routing.
From what I have seen currently, in order to do something like that with stackdriver-exporter, one would need to create multiple alerting rules for each resource, that hardcodes the needed labels?


Set Classification Type on GeoJSON Entities

Currently, there's no way to set the classificationType on entities created from GeoJSONs. The reason you'd want this is if you're using GeoJSONs to classify your 3D Tiles.
For example, this example uses the experimental 3D Tiles Vector format:
https://cesiumjs.org/Cesium/Build/Apps/Sandcastle/?src=3D Tiles Terrain Classification.html&label=All
But since no tools exist that can output this, you can't actually do something like this in your own applications in CesiumJS. But you can totally do it if expose an option on GeoJSONDataSource to set the classification type.

@OmarShehata I don't think this is something we would add to the GeoJsonDataSource.  Instead, the user can set the classificationType attribute on the entities after they are created, similarly to the Custom Styling part of this sandcastle example: https://cesiumjs.org/Cesium/Build/Apps/Sandcastle/?src=GeoJSON and TopoJSON.html&label=DataSources
You would just need to set entity.polygon.classificationType = Cesium.ClassificationType.CESIUM_3D_TILE
If that's not a solution that works for your use case, let me know and we can reopen this issue.

I can see that it's possible that way. I think it's just easier for the user to have this abstraction, since I can imagine it'd be a common use case to use vector data to mark tilesets like this.
For example, if my GeoJSON contains more than just polygons, would I have to figure out what the type of each entity is, so I can say entity.polygon.classificationType vs entity.rectangle.classificationType  etc ? In my case, I grabbed the GeoJSON of road data from the Australian government, and I knew I wanted to use it to classify my photogrammetry, but I didn't know what all the feature types in the GeoJSON were.

I understand what you are going for, but I don't think extending the GeoJsonDataSource to handle Cesium specific features are the way to go, as @hpinkos mentioned, that's what the Entity API is for.
GeoJson only has points/lines/polygons, and (unless I'm wrong) points and polylines aren't eligible for classification, so having a rectangle is impossible.

yep, @mramato is right.  GeoJsonDataSource will only make entity.billboard, entity.polyline and entity.polygon.

That's fine then! That means there's only one type to worry about (polygons). I think I saw some confusion on the forum about the difference between datasources and entities (the users weren't aware that a datasource created entities) but it sounds like I can fix that by explaining in the doc/tutorials. That Sandcastle is definitely a good resource.

I just realized this was already suggested by @lilleyse , in this issue which is still open: https://github.com/AnalyticalGraphicsInc/cesium/issues/6634

I tried,but it didn't work @hpinkos, can you help me check it.the code is here:
var tileset = new Cesium.Cesium3DTileset({
url: '../../../data/20181102/tileset.json',
maximumScreenSpaceError: 2,
maximumNumberOfLoadedTiles: 1000
});
viewer.scene.primitives.add(tileset);
var buildingsPromise = Cesium.GeoJsonDataSource.load('../../../data/test.json');
buildingsPromise.then(function (dataSource) {
viewer.dataSources.add(dataSource);
var buildings = dataSource.entities;
    var buildingsEntities = dataSource.entities.values;
    for (var i = 0;i < buildingsEntities.length;i++) {
        var entity = buildingsEntities[i];

        if(Cesium.defined(entity.polygon)) {
            entity.name = entity.properties.Id;
            entity.polygon.material = Cesium.Color.fromRandom({
                red : 0.2,
                minimumGreen : 0.5,
                maximumBlue : 0.5,
                alpha : 0.6
            });
            entity.polygon.outline = false;
            entity.polygon.classificationType = Cesium.ClassificationType.CESIUM_3D_TILE
        };
    }
});


MINOR: removing codedungeon/phpunit-result-printer. Closes #29

Hey @nesk
getting started with doing something, even it's minor tidy up.
Cheers,
Peter

Thank you!


Add game websites to learn some tech stacks

Heya @debrajhyper
I wanted to add some websites where people can learn tech stacks like JS, Python, SQL, etc. as a game.
Should I add them to the Documentation section?

Hey, can I add on some wesites to this as part of hactoberfest? Kindly assign me this issue.

You are all good to go.


ValueError: Failed to start Kaleido subprocess. Error stream: [NONE]

Running Python 3.8.17 in a conda environment on Windows 10. I have created a dedicated virtual environment and installed plotly and python-kaleido. I have then executed the following script:
import plotly

fig = plotly.graph_objects.Figure()

fig.write_image("plot.png")

Output is as follows:
Traceback (most recent call last):
  File "write_image.py", line 5, in <module>
    fig.write_image("plot.png")
  File "C:\Users\marianoju\anaconda3\envs\plotly\lib\site-packages\plotly\basedatatypes.py", line 3840, in write_image
    return pio.write_image(self, *args, **kwargs)
  File "C:\Users\marianoju\anaconda3\envs\plotly\lib\site-packages\plotly\io\_kaleido.py", line 267, in write_image
    img_data = to_image(
  File "C:\Users\marianoju\anaconda3\envs\plotly\lib\site-packages\plotly\io\_kaleido.py", line 144, in to_image
    img_bytes = scope.transform(
  File "C:\Users\marianoju\anaconda3\envs\plotly\lib\site-packages\kaleido\scopes\plotly.py", line 153, in transform
    response = self._perform_transform(
  File "C:\Users\marianoju\anaconda3\envs\plotly\lib\site-packages\kaleido\scopes\base.py", line 293, in _perform_transform
    self._ensure_kaleido()
  File "C:\Users\marianoju\anaconda3\envs\plotly\lib\site-packages\kaleido\scopes\base.py", line 198, in _ensure_kaleido
    raise ValueError(message)
ValueError: Failed to start Kaleido subprocess. Error stream:

The empty Error stream makes this error particularly difficult to debug. I have tried installing

plotly 5.9.0 from defaults and python-kaleido 0.2.1 from conda-forge
plotly 5.11.0 and python-kaleido 0.2.1 from conda-forge
plotly 5.15.0 and python-kaleido 0.0.3 from plotly
plotly 5.15.0 from plotly and python-kaleido 0.2.1 from conda-forge
plotly 5.15.0 and python-kaleido 0.2.1 from conda-forge

In short, I have not found a way to fix the error. I would be very grateful for any help.

my issue with windows 10 in conda was the spaces in the Program Files path throwing it off.
reverting to this older version worked from me for my windows 10 env
pip install --upgrade "kaleido==0.1.*"

Same issue, fixed by reverting to version 0.1.0.post1
pip install "kaleido==0.1.0.post1"

Issue was fixed with version 0.1.0.post1 on Windows.
Now this simple code :

Thanks for your interest in Kaleido. We are currently working on an overhaul that might address your issue - we hope to have news in a few weeks and will post an update then. Thanks - @gvwilson

Hi there, I am having the same error and downgrading to 0.1.0.post1 didn't solve my issue. I'm working with windows 10, I wasn't getting this bug last week on the same laptop. Are there any updates on a fix? Thanks

same here...

Hi @rhiannaleslie1 an @konosubakonoakua - there should be a new version of kaleido out today. It needs chromium installed separately.
If you'd like help, please post an issue with your output, and its better if you set env var KALEIDO_DEBUG=true before running it so I can see more output.
Some people are pinning backwards, but we can't support those older versions. Thanks so much!

Hi @rhiannaleslie1 an @konosubakonoakua - there should be a new version of kaleido out today. It needs chromium installed separately.
If you'd like help, please post an issue with your output, and its better if you set env var KALEIDO_DEBUG=true before running it so I can see more output.
Some people are pinning backwards, but we can't support those older versions. Thanks so much!

Hi @ayjayt , Can I use edge?
@konosubakonoakua
Other users have had success with edge if they set the env var BROWSER_PATH to its path


JdbcTpchQueryForAllDatabasesTest failing in master

JdbcTpchQueryForAllDatabasesTest has been changed sometime ago in MASTER to only run tests on redshift with query22.
Please refer to these two links: link1, link2.
I have found that current master fails with other unit tests if I revert it back locally to test on all databases:
testTpch[impala_tpch_13](org.verdictdb.jdbc41.JdbcTpchQueryForAllDatabasesTest)  Time elapsed: 5.81 sec  <<< FAILURE!
java.lang.AssertionError: expected:<4> but was:<1>
	at org.junit.Assert.fail(Assert.java:88)
	at org.junit.Assert.failNotEquals(Assert.java:834)
	at org.junit.Assert.assertEquals(Assert.java:118)
	at org.junit.Assert.assertEquals(Assert.java:144)
	at org.verdictdb.jdbc41.JdbcTpchQueryForAllDatabasesTest.testTpch(JdbcTpchQueryForAllDatabasesTest.java:277)

testTpch[impala_tpch_14](org.verdictdb.jdbc41.JdbcTpchQueryForAllDatabasesTest)  Time elapsed: 4.745 sec  <<< FAILURE!
java.lang.AssertionError: expected:<true> but was:<false>
	at org.junit.Assert.fail(Assert.java:88)
	at org.junit.Assert.failNotEquals(Assert.java:834)
	at org.junit.Assert.assertEquals(Assert.java:118)
	at org.junit.Assert.assertEquals(Assert.java:144)
	at org.verdictdb.jdbc41.JdbcTpchQueryForAllDatabasesTest.testTpch(JdbcTpchQueryForAllDatabasesTest.java:277)

testTpch[impala_tpch_16](org.verdictdb.jdbc41.JdbcTpchQueryForAllDatabasesTest)  Time elapsed: 7.023 sec  <<< FAILURE!
org.junit.ComparisonFailure: expected:<Brand#[35]  > but was:<Brand#[42]  >
	at org.junit.Assert.assertEquals(Assert.java:115)
	at org.junit.Assert.assertEquals(Assert.java:144)
	at org.verdictdb.jdbc41.JdbcTpchQueryForAllDatabasesTest.testTpch(JdbcTpchQueryForAllDatabasesTest.java:277)

These errors do not occur in my local branch that fixes redshift errors (which was branched quite a while ago so I cannot use it to simply overwrite master), so it must be one of the changes applied in the last 5 days (or even further back in the past), which is bad since there have been lots of changes merged into master...
It looks like ORDER BY clause in the original query is not applied correctly in VerdictDB's rewritten query. This may or may not be specific to Impala.

Order-by and limit were removed indeed and didn’t put it back. I will work
on them tonight.
On Tue, Jul 31, 2018 at 6:35 PM Dong Young Yoon<EMAIL_ADDRESS>wrote:

JdbcTpchQueryForAllDatabasesTest has been changed sometime ago in MASTER
to only run tests on redshift with query22.
Please refer to these two links: link1
https://github.com/mozafari/verdictdb/blame/master/src/test/java/org/verdictdb/jdbc41/JdbcTpchQueryForAllDatabasesTest.java#L53,
link2
https://github.com/mozafari/verdictdb/blame/master/src/test/java/org/verdictdb/jdbc41/JdbcTpchQueryForAllDatabasesTest.java#L174
.
I have found that current master fails with other unit tests if I revert
it back locally to test on all databases:
testTpchimpala_tpch_13  Time elapsed: 5.81 sec  <<< FAILURE!
java.lang.AssertionError: expected:<4> but was:<1>
at org.junit.Assert.fail(Assert.java:88)
at org.junit.Assert.failNotEquals(Assert.java:834)
at org.junit.Assert.assertEquals(Assert.java:118)
at org.junit.Assert.assertEquals(Assert.java:144)
at org.verdictdb.jdbc41.JdbcTpchQueryForAllDatabasesTest.testTpch(JdbcTpchQueryForAllDatabasesTest.java:277)
testTpchimpala_tpch_14  Time elapsed: 4.745 sec  <<< FAILURE!
java.lang.AssertionError: expected: but was:
at org.junit.Assert.fail(Assert.java:88)
at org.junit.Assert.failNotEquals(Assert.java:834)
at org.junit.Assert.assertEquals(Assert.java:118)
at org.junit.Assert.assertEquals(Assert.java:144)
at org.verdictdb.jdbc41.JdbcTpchQueryForAllDatabasesTest.testTpch(JdbcTpchQueryForAllDatabasesTest.java:277)
testTpchimpala_tpch_16  Time elapsed: 7.023 sec  <<< FAILURE!
org.junit.ComparisonFailure: expected:<Brand#[35]  > but was:<Brand#[42]  >
at org.junit.Assert.assertEquals(Assert.java:115)
at org.junit.Assert.assertEquals(Assert.java:144)
at org.verdictdb.jdbc41.JdbcTpchQueryForAllDatabasesTest.testTpch(JdbcTpchQueryForAllDatabasesTest.java:277)
These errors do not occur in my local branch that fixes redshift errors
(which was branched quite a while ago so I cannot use it to simply
overwrite master), so it must be one of the changes applied in the last 5
days, which is bad since there have been lots of changes merged into
master...
It looks like ORDER BY clause in the original query is not applied
correctly in VerdictDB's rewritten query. This may or may not be specific
to Impala.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mozafari/verdictdb/issues/187, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABQYCrOrsy12tbuO-fpAwlfyt4K5aU8mks5uMNuPgaJpZM4VphFE
.

--
Yongjoo Park, Ph.D.
Research Fellow
Computer Science and Engineering
University of Michigan
2260 Hayward St.
Ann Arbor, MI 48109-2121
Office: 4957 Beyster
Phone:<PHONE_NUMBER>
Website: yongjoopark.com


Set channel to stable in rust-toolchain.toml

atm stable is at 1.76.0, and current hardcoded version 1.70 prevents installing newer crates.   This will allow PR https://github.com/SergioRibera/virtual_joystick/pull/20 to build.

Hey thank you very much for your contribution


[Task]: Investigate Resource not found on app fork

Is there an existing issue for this?

[X] I have searched the existing issues

SubTasks
While forking an app, Resource Not Found is getting thrown.

https://github.com/appsmithorg/appsmith/blob/release/app/client/src/pages/Editor/EditorHeader.tsx#L531 has stale workspaceId which then makes a call to fetch all of the members in the workspace here: https://github.com/appsmithorg/appsmith/blob/release/app/client/src/pages/Editor/EditorHeader.tsx#L321
When the fork button is pressed then the workspaceId should be updated in the redux store.

FORK_APPLICATION_SUCCESS should be saving the workspace id to redux.

    case ReduxActionTypes.FORK_APPLICATION_SUCCESS:
    case ReduxActionTypes.CREATE_APPLICATION_SUCCESS: {
      const application: ApplicationPayload = action.payload.application;
      const { pages } = application;
      appParams = {
        applicationId: application.id,
        applicationSlug: application.slug,
        applicationVersion: application.applicationVersion,
      };
      pageParams = pages.map((page) => ({
        pageSlug: page.slug,
        pageId: page.id,
        customSlug: page.customSlug,
      }));
      break;
    }

https://github.com/appsmithorg/appsmith/blob/release/app/client/src/RouteParamsMiddleware.ts#L37-L52


ESP32 build: Fix function prototype warning

@advaitjain PTAL.
Moved this warning fix commit from https://github.com/tensorflow/tensorflow/pull/44580


Polygon strokeColor and fillColor not working

if i do this stroke color and fillcolor, dont apply.
<Polygon
            draggable
            editable
            strokeColor="#13a168"
            strokeOpacity={0.8}
            strokeWeight={2}
            fillColor="#13a168"
            fillOpacity={2}
            paths={this.state.polygon}
          />

See #125

thanks


[BUG] Blocking HA startup

Describe the bug
This component blocks HA from starting if mining is not active (even though Quick Hash / Excavator is running)
To Reproduce
Steps to reproduce the behavior:

Stop mining, but leave Quick Hash / Excavator running
Restart HA
Watch it block startup
Now disable integration or uninstall
Restart HA
Notice no issues

Expected behavior
Component needs to better handle if mining is not active.
Screenshots
If applicable, add screenshots to help explain your problem.
Home Assistant (please complete the following information):

Version 2022.2.9
Installation docker

Additional context
HA has a fail-safe built in when components are blocking proper startup, so it eventually ignores the components and starts after about 600 seconds
Web UI error:
Starting nicehash_excavator, not everything will be available until it is finished.

Log snippets:
2022-02-22 08:30:36 WARNING (MainThread) [homeassistant.bootstrap] Waiting on integrations to complete setup: nicehash_excavator
2022-02-22 08:31:36 WARNING (MainThread) [homeassistant.bootstrap] Waiting on integrations to complete setup: nicehash_excavator
2022-02-22 08:32:36 WARNING (MainThread) [homeassistant.bootstrap] Waiting on integrations to complete setup: nicehash_excavator
2022-02-22 08:33:36 WARNING (MainThread) [homeassistant.bootstrap] Waiting on integrations to complete setup: nicehash_excavator
2022-02-22 08:34:19 WARNING (MainThread) [homeassistant.bootstrap] Setup timed out for stage 2 - moving forward
2022-02-22 08:39:19 WARNING (MainThread) [homeassistant.bootstrap] Setup timed out for bootstrap - moving forward
2022-02-22 08:39:19 INFO (MainThread) [homeassistant.bootstrap] Home Assistant initialized in 600.86s

Nevermind the details above... The issue is happening if mining is running also

Another log snippet (this is repeated many times)
2022-02-22 08:52:53 WARNING (MainThread) [custom_components.nicehash_excavator.excavator] Error while getting data from http://<IP_ADDRESS>:18001/api?command={"id":1,"method":"algorithm.list","params":[]}

Your last log snippet is caused by home assistant not reaching a specific endpoint on your excavator. Normally this might occur a couple of times a day which would be normal since networks drop some packets occasionally. If it occurs a lot more frequently you might have a network-related issue.
I could not recreate your initial bug on my development system, I tried restarting while mining, while having mining disabled, and with the mining software closed. There are some error messages when the mining is not running because the miner could not be found but home assistant started for all scenarios without any issues multiple times. If you can recreate the issue reliably please provide more information on how to do it.


Added urlencoding for file paths.

Thank you for the great package!
While working with it, I noticed that it was not able to download files that had special characters in the name (i.e. "&", "#"). I know that this is not really common to have them in the path, but as long as Seafile allows to use them, such paths can appear.
My guess, that such characters corrupt the request URI. And while when downloading files it is no real harm, it can be dangerous when removing files, as when there is "&" in the directory name the rest of the path will be left out. And if there is the file with resulted path it will be removed instead.
So I wrote the method that urlescapes the path and added it to all File methods that forms such URIs.
I'm not an expert in guzzle, so I'm not sure it won't double encode something, so take a look, please. As far as tested everything is OK.
And one more thing. While testing methods, I noticed that File::rename() method causes an error with files, that reside in folders. It renames file OK, but after it yells that it cannot find the file with the old filename. I didn't have time to investigate this today, so maybe you could take a look?
Thanks one more time for the package. It helped me a lot.

Thanks!


Running Scenario by line number from within Scenario doesn't work

Summary
I used to be able to run cucumber feature_file_name.feature:XX with XX being a line anywhere in the scenario (usually the line that failed last time, to easily re-run a failure) but now it only works if XX is the number of the "Scenario" heading line.
Expected Behavior
cucumber feature_file_name.feature:XX should run the Scenario, from the beginning, that contains line XX.
Current Behavior
The Scenario does not run, and the following output is produced.
0 scenarios
0 steps
0m0.000s

Steps to Reproduce (for bugs)

Run any Scenario (with cucumber feature_file_name.feature:XX) from the command line, tagging the line number of any line within the scenario other than the "Scenario" heading

Your Environment
ruby 2.7.1
MacOS 10.15.6
cucumber-rails 2.1

Is this ruby, javascript or java?

Is this ruby, javascript or java? What cucumber version?
@aslakhellesoy

Updated to include environment

Thanks for adding the cucumber-rails version. What cucumber version do you have?

Thanks for adding the cucumber-rails version. What cucumber version do you have?

Updated again -- cucumber 4.1.0

I'm unable to reproduce this with cucumber version 4.1.0. I tried with https://github.com/cucumber-ltd/shouty.rb (changed Gemspec to 4.1.0) and ran bundle exec cucumber features/hear_shout.feature:5 which runs a single scenario.
It also works fine with 5.1.0.
Is this a bug that only manifests itself with cucumber-rails? (sounds unlikely). Can you provide a Minimal, Reproducible Example

Oh, I see what you mean now - it no longer works if I specify line 6 instead of 5. Yep, that's definitely a regression. Never mind the Minimal, Reproducible Example.


Ability to suppress notice WARNING

I am using rpi-update in an automated build script to prepare new raspberry images. The automated functionality is now broken because there is a warning in the NOTICE.md file that requires user input:
#############################################################
WARNING: This update bumps to rpi-4.0.y linux tree
Be aware there could be compatibility issues with some drivers
Discussion here:
https://www.raspberrypi.org/forums/viewtopic.php?f=29&t=113753
##############################################################
Would you like to proceed? (y/N)

Generated by function:
function show_notice {
local NOTICE=`curl -Lfs https://raw.githubusercontent.com/hexxeh/rpi-firmware/${FW_REV}/NOTICE.md`
if [ -z "$NOTICE" ]; then
return
fi
echo "$NOTICE"
if ! echo "$NOTICE" | grep -q WARNING; then
return
fi
read -p "Would you like to proceed? (y/N)" -n 1 -r -s
echo ""
if ! [[ $REPLY =~ ^[Yy]$ ]]; then
exit 1;
fi
}

Could we add a variable or a command line switch to suppress such notices?
Thanks

Can you try now with:
sudo SKIP_WARNING=1 rpi-update

Now it works EXACTLY as I needed.
No prompt when variable set:
#############################################################
WARNING: This update bumps to rpi-4.0.y linux tree
Be aware there could be compatibility issues with some drivers
Discussion here:
https://www.raspberrypi.org/forums/viewtopic.php?f=29&t=113753
##############################################################
 *** Downloading specific firmware revision (this will take a few minutes)

Thanks!


Better typed fetch

Context, work, and explanation by @danielroe: https://github.com/unjs/nitro/pull/1532
We are thinking of improving (auto-generated) typed fetch. This requires adopting a new standard/reusable type utility for fetch, which @danielroe is working on. By using the new layer, we can improve performance and consistency and adopt it in several places instead of directly being in nitro core (benefiting nuxt, and in next steps ofetch and h3 typed fetch support.)

Improving fetch types would be an immense QOL improvement. I for one am very excited about this and am looking forward to more info as y'all push forward. Will there be a relevant issue or PR in ofetch for this effort?

Will this also type readBody to match the json serialization similiar to https://github.com/unjs/nitro/pull/1002?

Is this still planned? To my knowledge, it's one of the things gating Nuxt 4


Add Google Analytics

Pull Request Description

Add Google Analytics
Add cross-domain linking between website homepage and cloud dashboard
Highlight buttons on authentication flows on hover
Save logged in state as logged_in cookie
Remove saved access token from disk when signing out
Support redirect_to parameter on /register page

Important Notes
None
Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[x] The documentation has been updated, if necessary.
[x] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[x] All code follows the
Scala,
Java,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
All code has been tested:

[x] Unit tests have been written where possible.
[x] If GUI codebase was changed, the GUI was tested when built using ./run ide build.

... on second thought i should probably have created an issue for this instead of putting it here. oh well


Add Google SSO

This pull request adds Google SSO support. There are three main changes:

Add the Google auth view resource and update the Angular app.js to
use it
Change the "login with example" button to "login with google"
Make passwords optional when creating a new user. This allows an admin
to create a user without a password so that they can only login via Google.

I'm guessing you'll have some suggestions to make these changes more generally applicable.

This looks good, I would ask that you remove the google specific login/client-id from the javascript until I can figure out a better method of doing this dynamically. That being said if you are up for attempting to make those changes in this or another pull request that would great as well.

Updated to remove the Google changes. I plan on doing another pull request for the server side stuff to support the idea of active providers.

Ended up just pushing to this same pull request. Added a providers endpoint to return the configuration details for active providers.

There are a couple of TODOs in the commit that adds the providers endpoint. I'm not sure what the appropriate values for those keys would be.


Add thin-edge.io v1 releases

[x] create recipes for 1.0.0
[ ] run the test suite for 1.0.0

Version 1.0.0 and above implemented, test suite will be covered by #57


ANGULAR vs PROD Distance matrix parsing seems to have a glitch in PROD

When you load a distance matrix in PROD that has a text value in cell A1 (a header), it thinks of it as a node and creates artifacts, and the network looks weird. When that header cell is deleted the network is correct (David's email)
Angular is able to recognize that cell A1 contains random text and does not count it as a node. It process the matrix correctly JPEG files are below
https://cdc.sharepoint.com/:f:/r/teams/nchhstp-dhap-lb-microbetrace/Shared Documents/General/Files for bug reports?csf=1&web=1&e=huTWro

I've found the issue.  In the prod code there is this extra condition with the comment of "triangle matrix" and this essenitlly adds on an empty node at the beginning of the nodes list proir to pruning.  The nodes list prunes off the first node since that is the top left corner of the matrix and should be avoided, but since there is an empty node now input at the beggining, that gets pruned off instead of the node/1st cell.  That is why we are seeing it.  I am not usre as to what this triangle functionality is for, as it's not in Angular and not sure what the need for it is.  Maybe we can ask tony.
Private Zenhub Image


Fix AnonymousMigrations parsing for } beyond the class declaration

In the AnonymousMigrations task the closing ; will appear in the wrong location if } appears after the class (in a comment section for example), as reported in issue #10 .
In this PR, we parse the class to determine where the declaration ends, then insert a ;
This is similar to what happens in the DownMigration task.

An new migration fixture is created specifically to address the issue, but the comment section could simply be added to the "simple.php" migration if wanted

Sorry for the delay, if you can fix the lint issue, I'll get this merged and tagged.

Locally, phplint itself was ok but pint fixed a couple whitespaces. I enabled GH actions on my fork and pushed to main to trigger them and got all green. Should be ok now.


Intermittent Response Error when trying to play videos

Hi there,
Quick question, I seem to be getting some intermittent errors when trying to play videos on the site. The data all gets fetched from the plugin correctly but every so often the site will display a bunch of net::ERR_HTTP2_PROTOCOL_ERROR messages which seems to be an issue with the response headers.
I was just wondering if you have had any similar issues with this?

I'm using the source URLs that get returned in the files object which would be the same URLs that I would have used if we were copying and pasting the Vimeo file links from the Vimeo site.

Any info would be super appreciated.
Thanks,
Jack

Hi Jack,
here's an example of what my data looks like from the Sanity database with the plugin

And in my frontend I do something like this, for example when I want to use the 1080p file:
const highResFile = vimeo?.files?.find((file) =>
  file?.rendition?.includes("1080p")
);
videoEl.setAttribute("playsinline", "playsinline");
videoEl.src = highResFile?.link;

Does that help?

Hi there,
Apologies for the late reply.
Unfortunately, it doesn't seem to be an issue with how I'm fetching the data but more of an issue on Vimeo's side, I reached out to their support and they responded with the following: "This error is commonly caused by restricted networks or computers (e.g in an office or school), and in some cases, browser extensions or content filter applications. These systems are known to block some of our playback URLs, which can lead to errors in our player."
I'm not 100% sold on the issue related to restricted networks as my office or home and the clients don't have any specific restrictions. I'm waiting to hear back from support but I was hoping someone else may have had a similar issue.
just for context these are the console errors i get on the front end when a video won't load.
(safari error)

(brave error)

Cheers,
Jack

No worries! I hope you can find a solution for that edge case -- haven't encountered that one yet.
I will close the issue for now.


[5.10] Fix regression where DocC no longer emits warnings for unresolved links.

Cherry pick of #774
Explanation: Fixes regression where DocC no longer emits warnings for unresolved links.
Scope: Warnings for unresolved links in article files will now be registered and emitted.
Issue: rdar://119603319
Risk: Low.
Testing: Added unit test, ran the existing tests, and manually tested in different scenarios.
Reviewer: @d-ronnqvist

@swift-ci please test

@swift-ci please test


Integrate with must-gather

We need to make sure that the operator logs are picked up by must-gather

ACM must gather https://github.com/stolostron/must-gather


It is better to translate 'Id' to instead of 'ID' in Manage Access Policies dialog

Storage Explorer Version: 1.11.0
Build: 20191219.2
Branch: master
Language: Chinese(zh-TW)
Platform/OS: Windows 10/ CentOS 7.6.1810 /macOS High Sierra
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Launch Storage Explorer.
Open 'Settings' -> Application (Regional Settings) -> Select '繁體中文' -> Restart Storage Explorer.
Expand a storage account -> Blob Containers.
Select one blob container and right click it ->Click '管理存取原則...' -> Click '新增'.
Check the localized Id in Manage Access Policies dialog.

Expect Experience:
Id is translated to 識別碼.
Actual Experience:
Id is translated to ID.

More Info:

For one message ID in queue.
In zh-TW ID is localized as '識別碼'.

ENU screenshot:

This issue also reproduces for one file share/ queue/ table.

ENU screenshot for Manage Access Policies dialog.

On review.
0; "Properties.DocumentDB.Id"
0; "TableView.IDHeader"

It's better to translate 'ID'/'Id' to '識別碼'

This issue still reproduces on the latest build 20200108.3.

Fixed above location


[Bug]: ConnectionString parsing for AppInsights - DAB CLI

What happened?
A bug happened!
Invalid connection string results in uncaught exception.
When starting using "dab start" the error message returned when connection string is invalid is not user-friendly. See stack trace. I originally thought the error was for db connection string until i saw stack trace originated from "Microsoft.ApplicationInsights"
Version
.11rc
What database are you using?
Azure SQL
What hosting model are you using?
Local (including CLI)
Which API approach are you accessing DAB through?
REST
Relevant log output

crit: Microsoft.AspNetCore.Hosting.Diagnostics[6]
      Application startup exception
      System.ArgumentException: There was an error parsing the Connection String: Input contains invalid delimiters and cannot be parsed. Expected example: 'key1=value1;key2=value2;key3=value3'.
       ---> System.ArgumentException: Input contains invalid delimiters and cannot be parsed. Expected example: 'key1=value1;key2=value2;key3=value3'.
         at Microsoft.ApplicationInsights.Extensibility.Implementation.ConfigString.ConfigStringParser.Parse(String configString)

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@Sean, Can you please add details on what was the connection string you provided for AppInsights. Also the expected behavior.


test-proxy restore should emit local paths

Currently, test-proxy restore emits something like:
Running proxy version is Azure.Sdk.Tools.TestProxy 20230627.1
git --version
git checkout *
git clean -xdf
git remote -v
git remote set-url origin https://github.com/Azure/azure-sdk-assets
git fetch origin refs/tags/net/cognitivelanguage/Azure.AI.Language.Conversations_76d76224ff:refs/tags/net/cognitivelanguage/Azure.AI.Language.Conversations_76d76224ff
git sparse-checkout set --no-cone net/sdk/cognitivelanguage/Azure.AI.Language.Conversations
git -c advice.detachedHead=false checkout net/cognitivelanguage/Azure.AI.Language.Conversations_76d76224ff
git remote -v
git remote set-url origin https://github.com/Azure/azure-sdk-assets

For diagnostics this is fine, but what's missing is probably something most devs will need: the local mapping paths under <root>/.assets. I know this is in <root>/.assets/.breadcrumb, but it seems extraneous to have to go look here when you want to look at your files.
It's not uncommon to have to do that, and certainly easier to pull down what you need and use VSCode or whatever to browse it. Alternatively, a test-proxy view -a assets.json could open a browser right to your files on the right branch on github.com.

Alternatively, a test-proxy view -a assets.json could open a browser right to your files on the right branch on github.com.

This is definitely planned. config locate -a <path-to-assets> also works. Making it output by default is totally doable and will be done.

Don't be afraid to add a little color. 😉 Helps info pop. Console.ForegroundColor (save it, then restore it in a try...finally; Console.ResetColor is finicky) works without having to detect if you're running in a TTY.


Naming conventions for functions and docs

Hi,
while starting to work with willow-rs I noted some things where the codebase differs from the  rust naming conventions and  documentation conventions:

In function docs willow-rs uses imperative ("Create a new path") whereas the rust conventions use third person ("Creates a new path")
A few methods are prefixed with get_, which is discouraged in the conventions. So Path::component_count instead of Path::get_component_count

If there aren't strong reasons for deviating from the convention, I found it to be quite helpful in my coding experience to just stick with it for general consistency and bikeshed avoidance.

Those are on me, I think. I didn't know about those conventions, and these are how I do them usually (exactly for general consistency =D). But I personally am happy to defer to the wider conventions of the rust ecosystem on these points.


[Assembler] LDC also support "symbolic references"

According to JVMS 6.5.ldc:

Otherwise, if the run-time constant pool entry is a symbolic reference to a class or interface, then the named class or interface is resolved (§<IP_ADDRESS>) and value, a reference to the Class object representing that class or interface, is pushed onto the operand stack.

Otherwise, the run-time constant pool entry is a symbolic reference to a method type, a method handle, or a dynamically-computed constant. The symbolic reference is resolved (§<IP_ADDRESS>, §<IP_ADDRESS>) and value, the result of resolution, is pushed onto the operand stack.

Oh, and I would suggest adding a label beside the Save button in Assembler window, to show something like "2 errors". I spent a lot of time finding out why I just can't click Save even if there were absolutely nothing wrong with my changes. Then I finally realized its because Recaf can't parse its own assembly output LDC Lorg/bukkit/craftbukkit/Main; even without me changing anything. And the validator won't run before the first modification, so I didn't notice the warnings immediately.

By the way, I also suggest changing the category display name for CFR to something like "CFR decompiler". Because why would the user know that you're using CFR for decompilation and/or that CFR settings mean decompilation settings without... reading... documentation. Well I supposed that "when in doubt, read the docs" is always good practice but I don't think the majority of the users actually read the docs word-by-word? There're not many modern GUI software that you CAN'T use without reading the manuals first anyway.

只要我尝试使用换行符，代码就会自动转换

You left the comment at the wrong issue...
你评论错地方了。

Missing support for symbolic references

Done!

Label beside save to show "2 errors"

Also done!

changing the category display name for CFR to something like "CFR decompiler"

Done x3


MH-13083: Unify modal navigation

(Copied from Jira)
The detail modals in the Admin UI are not consistent considering...

Title of modal window
Navigation between individual objects

The goal of this story is to make the detail modals consistent considering the user experience.

H  @gregorydlogan & @lkiesow
It seems that Travis is in trouble:

[ERROR] Failed to execute goal com.github.eirslett:frontend-maven-plugin:1.5:install-node-and-npm (install node and npm) on project opencast-admin-ui: Could not extract the Node archive: Could not extract archive: '/home/travis/.m2/repository/com/github/eirslett/node/8.9.4/node-8.9.4-linux-x64.tar.gz': EOFException -> [Help 1]

I've manually restarted the build but the same error occurred again.
Could you please have a look at why Travis fails?

@KatrinIhler & @lkiesow Seems the ball is currently in your hands... Is there anything left that needs to be addressed?

@staubesv, no idea until I take another look at this…


Cleanup CodeFactor issues with empty strings

PR Summary
Please fast review to avoid merge conflicts because of many files changed.
Please review commit by commit - every commit contains only single kind of change so you can do very fast review.
This corrects about 1% of the CodeFactor issues.
The PR replace "" with string.Empty.
All changes is made in VS Code with Regex patterns - one pattern by commit:
'= "";' -> '= string.Empty;'

'return "";' -> 'return string.Empty;'

': "";' -> ': string.Empty;'

'\?\? "";' -> '?? string.Empty;'

'\(""\)' -> '(string.Empty)'

' ""\)' -> ' string.Empty)'

'\("",' -> '(string.Empty,'

', "",' -> ', string.Empty,'

',""\)' -> ', string.Empty)'

'\? "" :' -> '? string.Empty :'

'\?\? ""' -> '?? string.Empty'

' = "",' -> ' = string.Empty,'

': "",' -> ': string.Empty,'

'@""\)' -> 'string.Empty)'

PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Change is not breaking
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

User-facing changes

[x] Not Applicable
OR
[ ] User-facing Documentation needed

[ ] Issue filed - Issue link:

Testing - New and feature

[ ] Not Applicable or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

[x] Add [feature] if the change is significant or affects feature tests

@markekraus Many thanks for fast review!

I don't see the point. Why not just disable the warning and skip the code churn?

It is readability issue. If we use advanced editor with code highlighting we haven't problems to see "". If we use a simple editor the string.Empty visibility is much better.
Also see my https://github.com/PowerShell/PowerShell/pull/6949#issuecomment-393030657

@SteveL-MSFT Can we merge?

@iSazonov What does indicate as the reasoning for this change?  Is this a readability or programmatic warning?
My understanding of "" is that it is interned and you will have one instance per assembly.  In all other regards, String.Empty is synonymous with "".

@dantraMSFT I think the purpose is to enable the style checking about empty string (favor string.Emtpy over ""). That rule cannot be turned on today due to too much noise.
The intern string should not be a concern, now it's one instance per assembly, but after replacing with string.Empty it will be one instance per process.
@iSazonov Thanks for spending efforts on driving the style consistency. I ran check-stringreplacementchange-ps1 with this PR and the results look good except for the three comments I left.
I quickly go through the changes and didn't notice any replacements in PowerShell scripts in .cs files, that's good because string.Emtpy will definitely fail in powershell script. However, since I went through so quickly, I could miss something, so please double check that you are not replacing anything in powershell script (script strings). After double checking that, the PR should be good.

@dantraMSFT The PR has a purpose to turn CodeFactor into a more useful tool. Currently we see over 100000 issues most of which are style issues.


feat: Add fail-fast behavior to Ginkgo in test-e2e

in regards to the follwing issue:
https://github.com/ionos-cloud/cluster-api-provider-proxmox/issues/194
What was changed: Describe the specific change:
Added --fail-fast to the Ginkgo command in the test-e2e target of the Makefile.
Ensures the e2e test suite halts immediately upon encountering a failure.

Why the change was made:
Saves compute resources and time by stopping further tests when a failure is detected.

Further information about the --fail-fast flag can be found here: https://onsi.github.io/ginkgo/

test and e2e pass was discarded by rebase, merging.

Thanks!


feat: stop zero meaning all for meat

Relates to https://kolmafia.us/threads/csend-with-commas-acts-unexpectedly-when-sending-meat.27447/
"0 meat" now parses as no meat, instead of all your meat. * continues to be all. -X continues to be all but X.

I think any test would go against one of the commands.
1,000 meat is an error. Parses as "1" (error) and "000 meat" (0 meat).


Transparent background PNG shown as black

This can be tested on the poetry book from Pizarnik. On this books this EPublibre logo comes with transparent background:

@cgreening is there a way to detect when there is transparent background and render it as white instead of black?
I remember EPDiy had a function called
441 | static void draw_rotated_transparent_image(EpdRect image_area, const uint8_t *image_buffer, uint8_t *framebuffer, uint8_t *transparent_color)
But I really don’t have any idea about PNG since I never worked with this image format. Only thing I know is that supports transparent backgrounds just as gif.

Hmm, let me check the png library - should be possible.

Possible - we'd need to read the current value of the pixel so it could be blended with the png pixel before being drawn.


Quantile LightGBM - inconsistent deciles

Hi, I'm doing Quantile LightGBM, defined as follows:
QuantileEstimator(lgb.LGBMRegressor(n_jobs=-1,
                             seed=1234,
                             learning_rate=0.1,
                             reg_sqrt=True,
                             objective = 'quantile',
                             n_estimators=100))

Where QuantileEstimator is just:
class QuantileEstimator(BaseEstimator, RegressorMixin):

    def __init__(self, model):
        """
        """
        self.alphas = [round(x, 1) for x in np.arange(0.1, 1, 0.1)]
        self.model_factory = []
        self.model = model
        super().__init__()

    def fit(self, X, y=None):

        for a in self.alphas:
            model_i = clone(self.model)
            model_i = model_i.set_params(**{'alpha': a})
            model_fitted = model_i.fit(X, y)
            self.model_factory.append(model_fitted)
        return self

    def predict(self, X):
        predictions = pd.DataFrame()
        for a, m in zip(self.alphas, self.model_factory):
            predictions["y_pred_" + str(a)] = m.predict(X)
        return predictions

Sometimes it happens that following decile for individual predictiions has lower value than the previous one.

Any idea why that's happening? I understand that those are different models, but still as I set the same seed I would expect that the results for different deciles should be monotonic.

Duplicate of #3447.


RA cannot be shared between threads safely

When I install on Ubuntu using curl https://getsubstrate.io -sSf | bash at 214b687d, I get the following compilation failure:
[...]
info: downloading installer
info: syncing channel updates for 'stable-x86_64-unknown-linux-gnu'
info: latest update on 2018-11-08, rust version 1.30.1 (1433507eb 2018-11-07)
info: downloading component 'rustc'
info: downloading component 'rust-std'
info: downloading component 'cargo'
info: downloading component 'rust-docs'
info: installing component 'rustc'
info: installing component 'rust-std'
info: installing component 'cargo'
info: installing component 'rust-docs'
info: default toolchain set to 'stable'

  stable installed - rustc 1.30.1 (1433507eb 2018-11-07)

Rust is installed now. Great!

[...]
   Compiling parity-crypto v0.1.0
   Compiling libp2p-tcp-transport v0.1.0 (https://github.com/libp2p/rust-libp2p?rev=d961e656a74d1bab5366d371a06f9e10d5f4a6c5#d961e656)
   Compiling libp2p-dns v0.1.0 (https://github.com/libp2p/rust-libp2p?rev=d961e656a74d1bab5366d371a06f9e10d5f4a6c5#d961e656)
   Compiling jsonrpc-ws-server v9.0.0 (https://github.com/paritytech/jsonrpc.git#a39139f9)
   Compiling jsonrpc-http-server v9.0.0 (https://github.com/paritytech/jsonrpc.git#a39139f9)
   Compiling libp2p-kad v0.1.0 (https://github.com/libp2p/rust-libp2p?rev=d961e656a74d1bab5366d371a06f9e10d5f4a6c5#d961e656)
   Compiling substrate-state-machine v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling srml-metadata v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling substrate-keyring v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling substrate-state-db v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling libp2p-websocket v0.1.0 (https://github.com/libp2p/rust-libp2p?rev=d961e656a74d1bab5366d371a06f9e10d5f4a6c5#d961e656)
   Compiling substrate-keystore v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling libp2p v0.1.0 (https://github.com/libp2p/rust-libp2p?rev=d961e656a74d1bab5366d371a06f9e10d5f4a6c5#d961e656)
   Compiling sr-primitives v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling substrate-network-libp2p v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling srml-support v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling sr-version v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling substrate-transaction-graph v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling node-primitives v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling srml-system v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling substrate-consensus-common v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling substrate-executor v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling srml-consensus v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling srml-balances v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling srml-executive v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling substrate-client v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling srml-timestamp v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling srml-upgrade-key v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling srml-democracy v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling srml-contract v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling srml-treasury v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling srml-session v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling substrate-test-runtime v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling substrate-transaction-pool v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling substrate-client-db v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling substrate-fg-primitives v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling srml-council v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling srml-staking v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling substrate-test-client v0.1.0 (https://github.com/paritytech/substrate#214b687d)
   Compiling substrate-rpc v0.1.0 (https://github.com/paritytech/substrate#214b687d)
error[E0277]: `RA` cannot be sent between threads safely
   --> /root/.cargo/git/checkouts/substrate-7e08433d4c370a21/214b687/core/rpc/src/state/mod.rs:117:23
    |
117 | impl<B, E, Block, RA> StateApi<Block::Hash> for State<B, E, Block, RA> where
    |                       ^^^^^^^^^^^^^^^^^^^^^ `RA` cannot be sent between threads safely
    |
    = help: within `client::Client<B, E, Block, RA>`, the trait `std::marker::Send` is not implemented for `RA`
    = help: consider adding a `where RA: std::marker::Send` bound
    = note: required because it appears within the type `std::marker::PhantomData<RA>`
    = note: required because it appears within the type `client::Client<B, E, Block, RA>`
    = note: required because of the requirements on the impl of `std::marker::Sync` for `std::sync::Arc<client::Client<B, E, Block, RA>>`
    = note: required because it appears within the type `state::State<B, E, Block, RA>`

error[E0277]: `RA` cannot be shared between threads safely
   --> /root/.cargo/git/checkouts/substrate-7e08433d4c370a21/214b687/core/rpc/src/state/mod.rs:117:23
    |
117 | impl<B, E, Block, RA> StateApi<Block::Hash> for State<B, E, Block, RA> where
    |                       ^^^^^^^^^^^^^^^^^^^^^ `RA` cannot be shared between threads safely
    |
    = help: within `client::Client<B, E, Block, RA>`, the trait `std::marker::Sync` is not implemented for `RA`
    = help: consider adding a `where RA: std::marker::Sync` bound
    = note: required because it appears within the type `std::marker::PhantomData<RA>`
    = note: required because it appears within the type `client::Client<B, E, Block, RA>`
    = note: required because of the requirements on the impl of `std::marker::Sync` for `std::sync::Arc<client::Client<B, E, Block, RA>>`
    = note: required because it appears within the type `state::State<B, E, Block, RA>`

error: aborting due to 2 previous errors

For more information about this error, try `rustc --explain E0277`.
error: Could not compile `substrate-rpc`.
warning: build failed, waiting for other jobs to finish...
error: failed to compile `substrate v0.1.0 (https://github.com/paritytech/substrate#214b687d)`, intermediate artifacts can be found at `/tmp/cargo-installNstJcN`

Caused by:
  build failed

Will be fixed by: https://github.com/paritytech/substrate/pull/1121

Pull request is merged into master.


Add two labels on left and right of menu view as shown in second screenshot.

u got the answer for it ?

plese guide me that how to add two labels as shown in second screenShot

By changing contentRatio of  _calendarMenuView:
_calendarMenuView.contentRatio = .5;


[Feature Request] Utilize encoding.TextMarshaller and encoding.TextUnmarshaller

If encoding.TextMarshaller is used for types found with clover.NewDocumentOf and encoding.TextUnmarshaller with clover.Document.Unmarshal, then standard library and custom types can be used for fields that aren't currently able to marshal/unmarshal correctly like a custom UUID type.

Hi, @duckbrain, thanks for the suggestion. However, using a textual encoding, such as encoding.TextMarshaller, would have a severe impact on query performance, considering that, at query time, much time is wasted in document deserialization.
This is why, I switched from json to msgpack, which is a binary encoding and is thus more performant.
Supporting custom types is definetely a good idea, but I think this feature should not rely on some assumptions made about the underlying encoding.

In that case, encoding.Binary*?

encoding/binary only allows you to encode integer types. Anyway, the problem here is not about finding a better encoding for documents, since msgpack is already doing its job. Supporting custom types should not be tight to a particular encoding scheme, since it would be impossible to change it in the future. The way encoding libraries (among these, json and msgpack itself) allow the user to include custom types is by letting the user define custom Marshal() and Unmarshal() methods on the custom object, which serialize the custom type to byte slices.

I was referring to the encoding.BinaryMarshaller and encoding.BinaryUnmarshaller interfaces instead of the text-based interfaces in the same package.
They serve as a standard for encoding and decoding custom types into a binary format, not an encoding scheme, but instead can be used by encoding formats like gob.
Standard library types also implement these standards, so it could remove some special handling for certain types.

Oh, sorry about that, we were basically saying the same thing. Would you like to work on this feature?

After looking into it. msgpack is already using encoding.BinaryMarshaller and encoding.BinaryUnmarshaller, but document.Document is normalizing everything into map[string]interface{} and that's what is failing to handle the binary encoding.
It looks like internal.Convert is using json to copy values from one object to another. In general, an normal lookup into an object goes from binary encoded msgpack -> map[string]interface{} -> json -> your object. To fix this Document would need to be able to use an object and reflect directly on it.
I don't see a way to do this without rewriting large swaths of the internals, and I'm not sure backwards compatibility can be preserved with the API.

After looking into it. msgpack is already using encoding.BinaryMarshaller and encoding.BinaryUnmarshaller, but document.Document is normalizing everything into map[string]interface{} and that's what is failing to handle the binary encoding.

I think it should be enough to leave values implementing the binary.BinaryMarshaller interface untouched when normilizing to map[string]interface{} in order to preserve custom types.

I think it should be enough to leave values implementing the binary.BinaryMarshaller interface untouched when normilizing to map[string]interface{} in order to preserve custom types.

Right you are.  PR is up.

Anyway, when documents are loaded from disk, they are simply unmarshalled from the msgpack encoding.

I see how that works and the need for the Document type for building the indexes and having a way to pass the values around, but don't personally see when I would get a document out of the database and not immediately convert it into a type-safe object.

I would like to optimize this in the future, by removing the json step, which is actually a way to simply convert between types

The "simple" way to handle that of course is custom reflection or something like structmap.
On the idea of optimizing for what I see as the common case. I messed around with the idea deferring the unmarshalling until requested or needed to use Get and the like. That's is not completely functional, nor suggested, but you're welcome to peruse/use.
https://github.com/duckbrain/clover/tree/feature/delay-decoding

Thank you for the PR, which I just merged in the v2 branch.

The "simple" way to handle that of course is custom reflection or something like structmap.

On the idea of optimizing for what I see as the common case. I messed around with the idea deferring the unmarshalling until requested or needed to use Get and the like. That's is not completely functional, nor suggested, but you're welcome to peruse/use.
https://github.com/duckbrain/clover/tree/feature/delay-decoding
These are interesting ideas, which I think should be handled in a separate issue.
I think this one is solved, so I'm closing it


Automatically show inline values if an inline values provider is registered

If a user does not have an explicit "debug.inlineValues" setting, we should automatically enable inline values if an inline values provider is registered.
see https://github.com/microsoft/vscode/issues/105690#issuecomment-804710058

Yes this makes sense, for this we can change the debug.inlineValue to be true | 'false' and add a new default value 'auto'.
'auto' would mean it is off unless there is a value provider registered.
That way we will still give the option for the user to turn it off even if a provider is registered.

I like the idea of adding a new default value auto.

I just pushed this change as we agreed upon, debug.inlineValue can now be true | false | "auto". "auto" is the default
@testforstephen would be great if you can verify this with vscode insiders starting from Friday (once we release a new Insiders build). Thanks!

Tried 04-02 insider version, it works well. Thanks.
The new auto will enable the inline values by default if Java debugger uses the API to provide inline values.
And true and false will also turn on/off the inline values.

@testforstephen great, thanks for trying, adding verified label.


Wait for feature element

Should fix CI.

Interesting: it seems to fail for a different reason now.

Interesting indeed ... perhaps react is mounting div and then child components on different ticks -- which would make sense for performance reasons, and we're grabbing it too soon now (10ms vs 100ms).

https://github.com/facebookincubator/create-react-app/pull/1274 causes the annoying errors in console. There is a bug in sockjs-client that was fixed in the release we downgraded from.

Mb dispatch event at setState callback? Seems the result is same but less files are changed.
fixtures/kitchensink/integration/initDOM.js:
setFeature(feature) {
  this.setState({ feature }, () => window.dispatchEvent( new Event('fixture')));
}

@tuchk4 I'm not sure if that works because the component that is set to the state sets its own state on mount.

Not everything ends up firing the event -- not sure if this is by code design or jsdom flaw... Need to investigate more, what do you think, @EnoahNetzach?

yesterday I was fiddling around with the events idea:

class BuiltEmitter extends React.Component {
  componentDidMount() {
    document.dispatchEvent(new Event('ReactFeatureDidMount'));
  }

  render() {
    return
{this.props.children}

  }
}

// ...

class App extends React.Component {
  // ...
  render() {
    const Feature = this.state.feature;
    return Feature ? <BuiltEmitter><Feature /></BuiltEmitter> : null;
  }
}

but I wasn't able to produce a consistent output between the E2E_FILE and E2E_URL versions.
Today I'm dedicated to investigate this, but if this PR's solution is sound, I'd say to accept this.

@Timer here the results EnoahNetzach/e2e-jsdom-fix.
TL;DR; it works.
It's not completely clear to me why in E2E_FILE I have to listen to window's load event and ReactFeatureDidMount is not even dispatched; while in E2E_URL the ReactFeatureDidMount event is correctly dispatched.

...aaaand I spoke too early.
Just to validation, using the componentDidMount should assure all children are mounted, ergo the
 is present, isn't it?

no, ok, now the problem (I don't have while running the suite locally) appears to be that promises (await/async) are resolved too late.
It makes sense.

It's not completely clear to me why in E2E_FILE I have to listen to window's load event and ReactFeatureDidMount is not even dispatched; while in E2E_URL the ReactFeatureDidMount event is correctly dispatched.

I'm running into the same problem ... puzzling. 🤔

#1470


Create documentation listing papers with stim files attached

2021

arXiv:2103.02202 → Ancillary files of "Stim: a fast stabilizer circuit simulator"
arXiv:2108.10457 → Ancillary files of "A Fault-Tolerant Honeycomb Memory"

2022

arXiv:2202.11845 → Data for "Benchmarking the Planar Honeycomb Code"
arXiv:2204.13834 → Data for "Stability Experiments: The Overlooked Dual of Memory Experiments"
arXiv:2206.12780 → Data for "A Pair Measurement Surface Code on Pentagons"
arXiv:2207.06431 → Data for "Suppressing quantum errors by scaling a surface code logical qubit"
arXiv:2209.08552 → Parallel window decoding enables scalable fault tolerant quantum computation

2023

arXiv:2302.02192 → Data for "Relaxing Hardware Requirements for Surface Code Circuits using Time-dynamics"
arXiv:2302.07395 → Data for "Inplace Access to the Surface Code Y Basis"
arXiv:2302.12292 → Data for "Cleaner magic states with hook injection"
arXiv:2305.12046 → Data for "Less Bacon More Threshold"
arXiv:2307.10147 → Stim circuits for "Tangling schedules eases hardware connectivity requirements for quantum error correction" manuscript
arXiv:2308.03750 → Ancillary data for the paper "Constructions and performance of hyperbolic and semi-hyperbolic Floquet codes"
arXiv:2312.04522 → Data for "Yoked Surface Codes"
arXiv:2312.08813 → Data for "New circuits and an open source decoder for the colorcode"
arXiv:2312.11605 → Stim circuits and collected data for "Error-corrected Hadamard gate simulated at the circuit level" manuscript

2024

arXiv:2405.15854 → Stim circuits for 'Accommodating Fabrication Defects on Floquet Codes with Minimal Hardware Requirements' manuscript

arXiv:2405.15854 → Stim circuits for 'Accommodating Fabrication Defects on Floquet Codes with Minimal Hardware Requirements' manuscript

arXiv:2209.08552→ Parallel window decoding enables scalable fault tolerant quantum computation

arXiv:2408.00758 --> Stim circuits for ``To reset, or not to reset -- that is the question" manuscript

arXiv:2407.13826 →  Stim circuits for 'Designing fault-tolerant circuits using detector error models'

arXiv:2408.11894 → Stim circuits for 'Automated Synthesis of Fault-Tolerant State Preparation Circuits for Quantum Error Correction Codes'


[Map] MH-AttackTest_00

Not a playable map - as this name suggests, the map indicates how to trigger an attack by a monster.  The map consists of a visible trigger and an Ice Skaarj out of sight - approach the trigger and the Skaarj will appear and attack the player - so the map is best placed in the Demo/Test category.

URL: https://unrealarchive.org/maps/unreal-tournament/monster-hunt/A/mh-attacktest_00_accd139d.html
Hash: accd139dfddae9355066df351747afde92d38eb7
Current name: MH-AttackTest_00

Moved to tests


Make parallax tab header

I cant figure out how make parallex tab header with native base.
Anyone knows how do it?
I needs something like this: https://snack.expo.io/@satya164/collapsible-header-with-tabview.
Thanks in advance

This query is more suitable to be asked on SO or NativeBase slack

Not exactly what you asked, but quite similar https://github.com/pranit123/Twitter-Clone


udp_proxy: support proxy access logging

Signed-off-by: giantcroc<EMAIL_ADDRESS>Commit Message:
Some stats like no_route and idle_timeout can't be printed by session access log,
so we need proxy-level access logging to log global stats.
Additional Description:
Risk Level: Low
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue] #23241
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

Sorry, misoperation, change to another PR.


Error when when I try to start bot

Every time I start the bot, it gives me this error:
An error occurred while initializing the packages: bptf-listings (HTTP error).
I tried the old "delete and re-install" thing but didn't work.
Any ideas?

Have you checked your account / config file? Something is probably wrong with your bptf apikey or bptf token

Closing because inactive. If you need help then just reopen the issue.


Add throttling to login form to prevent brute force password attacks

Prevent user from logging in for 10 minutes if they fail to enter correct password after 20 tries.

Jordan, this is yours to fix. If you need to have Jacob change some server configurations, do so; I'm not exactly sure what it will take to resolve this. It shouldn't be too hard.

Done! :D


suggestion

it would be cool to have GPU accelerated rendering of web apps that run in your terminal
https://github.com/unconed/TermKit#readme
though it might be a better idea to have an optional dependency that adds support for these web technologies

my whole idea here is my terminal emulator and window manager don't have to be different things just have them be different UI's you can switch between of course I know turning this into a full fledge window manager is out of scope but adding support for this so people can easily convert this into one doesn't seem like anything to ridiculous

though pls don't use there UI it's ugly I still want my stuff to look terminally but I also want pictures and videos in my terminal

I don't think this is within the scope of Alacritty, or a terminal emulator in general.
There are options for rendering raw image data to the terminal, the primary one which is already standardized and supported by XTerm would be libsixel (see #910). Other terminal emulators like Kitty (and I believe Terminator?) have their own escape sequences which are supported by tools written just for these terminal emulators.
If you want to support having non-terminal content inside a terminal emulator, using an escape sequence like libsixel is the way to go. Just embedding arbitrary content doesn't really fall in line with the way terminal emulators work. So I recommend subscribing to the libsixel issue, that's probably the most likely path to get what you're looking for working right now.

Notty is a good choice, but it is currently stagnant.

then how about extratermhttps://github.com/sedwards2009/extraterm then?

This isn't going to happen. We are interested in extending functionality with notty protocol at some point, though.


Vector search fails after merging with a table

I receive a OSError: Io error: Execution error: Schema error: field summary does not exist in the RecordBatch when doing a vector search after merging a table with an extra column (this is essential), that is, a column that is neither used for merging nor in the vector search.
The behavior described below has been tested (amongst others) with pylance 0.9.12 and python 3.11.7 on Ubuntu 22.04.
Here is a minimal example.
import random
import shutil
import string

import polars as pl
import pyarrow as pa

import lance

shutil.rmtree("testlance", ignore_errors=True)

def get_string_seq(n: int, length: int = 10) -> list[str]:
    random.seed(a=111)
    return ["".join(random.choices(string.ascii_letters, k=length)) for _ in range(n)]

def get_vec_seq(n: int, length: int = 20) -> list[list[float]]:
    random.seed(a=111)
    return [[random.gauss(0.0, 1.0) for _ in range(length)] for _ in range(n)]

df = pl.DataFrame(
    {
        "text": get_string_seq(1000),
    }
).with_row_index("id_")

vec = pl.DataFrame(
    {
        "summary": get_string_seq(1000),  # this seems to cause the issue
        "vector": get_vec_seq(1000),
    },
    schema_overrides={"vector": pl.Array(pl.Float32, 20)},
).with_row_index("id_")

dataset = lance.write_dataset(df.to_arrow(), "testlance")
dataset.merge(vec.to_arrow(), left_on="id_")
dataset.to_table(
    nearest={
        "column": "vector",
        "q": pa.array([0.0 for _ in range(20)], type=pa.float32()),
        "k": 5,
    }
)

The final to_table generates the error. If one

drops the summary column from vec or
removes the nearest argument form the to_table,
then the issue goes away.

Hi @christian-hnz. Thanks for reporting this. I can reproduce this on 0.9.12 (the latest release as of now).
Good new is this is already fixed by #1901. I'll add a unit test for this specific case in #1468 just to be sure it keeps working, though. We'll make a release around Monday that will include this fix.

Wow, that was fast. Thanks! Looking forward to the new release :+1:


Analytics Expression Language.

Analytics Expression Language - a intuitive language for performing analytics.

If you are interested, this is how I do things:
https://gist.github.com/daleroberts/75cb0f0d5a77bedc28cf

Thanks Dale I'll take a look at it.
Closing this.
First version of the AEL is located here https://github.com/data-cube/agdc-v2/tree/develop/datacube/ndexpr
It provides the ground work for for feedback and further improvements.


For type script

Is type script file available for this frame work? As for now I use d.ts files in my project.

Hi,
there is Typescript File https://github.com/allipierre/DefinitelyTyped/tree/master/types/confirmdialog


Update this repo

waow

waow x2

What

Agario is dead no point

lol never knew you'd reply. i was bored
On 16 February 2018 at 17:08, Logan<EMAIL_ADDRESS>wrote:

Agario is dead no point
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Logan7432/MyServer/issues/2#issuecomment-366296975,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWjIVJwkZCsh9RoSLnvTQDPNmp2mpDObks5tVbX7gaJpZM4O1bGv
.

Lol I get the emails


Error:(23, 44) error: cannot access zzbej

Hi Ayoub,
After I add implementation 'com.github.ayoubfletcher.GDPR-Admob-Android:consentsdk:0.1.6'
it shows these 2 Errors below
Error:(23, 44) error: cannot access zzbej
class file for com.google.android.gms.internal.zzbej not found
Error:Execution failed for task ':app:compileDebugJavaWithJavac'.

Compilation failed; see the compiler error output for details.

and then it takes me to MyFirebaseMessagingService.java
here some screenshots of my build.gradle and the error and MyFirebaseMessagingService.java page
https://imgur.com/a/Q2LP4X6
Can you please help

The error class "com.google.android.gms.internal.zzbej not found" has nothing to do with the library the error is triggered because of firebase missing in your project
"https://stackoverflow.com/questions/38073706/class-file-for-com-google-android-gms-internal-zzaja-not-found"

I tried so many solutions but the error presents only when I add your library

I can confirm for you the library is working and have nothing to do with firebase, I've tested it myself and others. I've already integrated the library into my apps and I can confirm for you that's nothing wrong with. It's working just fine without any error. You can download the demo app and try it yourself.

Hello,
i faced The same Issue only when i add the library

I debugged the problem and it happens when you use an older version of firebase because of the current library use 'com.google.android.gms:play-services-ads:15.0.1'  to get the "AdRequest" and when you use an older version of firebase such as 'com.google.firebase:firebase-core:10.0.1' the error happens so please upgrade the version of firebase


remote + default sort + default filter - many onTableChange calls

Is there a way to avoid making multiple requests in onTableChange on table init?
Example of the problem:
https://stackblitz.com/edit/react-spttgg?file=index.js
An attempt to solve with blockRequests flag:
https://stackblitz.com/edit/react-circw1?file=index.js

It works like that now:
sort  Object { sortOrder: "asc", sortField: "id", filters: {}, page: 1, sizePerPage: 25, searchText: undefined }
filter  Object { sortOrder: "asc", sortField: "id", filters: {…}, page: 1, sizePerPage: 25, searchText: "" }
table wrapper mounted

I was trying to add a flag to state like blockRequests, but it doesn't work since state is not updated with default filter after mount
  componentDidMount() {
    console.log('table wrapper mounted');
    // table prop: onTableChange={this.props.handleTableChange}

    // this.props.handleTableChange('init', {});  // <-- doesn't work
    setTimeout(() => this.props.handleTableChange('init', {}), 1);  // <-- ok, since state is updated
  };

@python273 so far, we dont have solution, but i will keep to find whether we can enhance it, thanks

@python273 did you manage so sort this issue out, or maybe found a decent workaround?

@Uznick there's hacky solution in:
https://stackblitz.com/edit/react-circw1?file=index.js
In componentDidMount, you need to comment
this.handleTableChange('init', {});
and uncomment
setTimeout(() => this.handleTableChange('init', {}), 1)
I ended up making my own table component, since I needed only simple features

@python273 thank you :)


BackgroundColor of pinview indicator rectangle!

When filling in the pin code, the indicator will not be round but square.

Hi,
I have added two lines to drawing function like below and problem solved!
`private func drawing(isNeedClear: Bool, tag: Int? = nil) { // Fill or cancel fill for indicators
let results = pinIndicators.filter { $0.isNeedClear == isNeedClear }
let pinView = isNeedClear ? results.last : results.first
pinView?.isNeedClear = !isNeedClear
    pinView?.layer.cornerRadius = 0.5 * (pinView?.bounds.size.width)! // just add those two lines
    pinView?.clipsToBounds = true

    UIView.animate(withDuration: ALConstants.duration, animations: {
        pinView?.backgroundColor = isNeedClear ? .clear : .white
    }) { _ in
        isNeedClear ? self.pin = String(self.pin.dropLast()) : self.pincodeChecker(tag ?? 0)
    }
}`

Hi,
I have added two lines to drawing function like below and problem solved!
`private func drawing(isNeedClear: Bool, tag: Int? = nil) { // Fill or cancel fill for indicators
let results = pinIndicators.filter { $0.isNeedClear == isNeedClear }
let pinView = isNeedClear ? results.last : results.first
pinView?.isNeedClear = !isNeedClear
    pinView?.layer.cornerRadius = 0.5 * (pinView?.bounds.size.width)! // just add those two lines
    pinView?.clipsToBounds = true

    UIView.animate(withDuration: ALConstants.duration, animations: {
        pinView?.backgroundColor = isNeedClear ? .clear : .white
    }) { _ in
        isNeedClear ? self.pin = String(self.pin.dropLast()) : self.pincodeChecker(tag ?? 0)
    }
}`


dry the code using traits

You may rename the traits names to "update" and "created" if you are not happy with the current names

The use statement should be on top of the class like the official doc also it's in PSR-5 draft iirc

@imanghafoori1 names look good for now can you kindly make the changes suggested by @ludo237

Align the comment
Move use to top

Thanks for drying up the code 👍
@ludo237 idea of merging them in single Traits looks good but some modules are not using both createdby / updatedby

I can perform much more refactors using my own package if you do not mind an extra composer package dependency in your project.
@ajitbohra

@imanghafoori1 not for now but will go through http://github.com/imanghafoori1/laravel-widgetize and let you know if plan to use that.

@ajitbohra Added newlines


Ensure detect-daemon-osarch works for docker-1.6

Description of problem:
Docker version output varies between 1.6 and 1.9.
How reproducible:
Running make - runs the generic docker file rather than Dockerfile.ppc64le
Steps to Reproduce:

$ make

Actual Results:
docker build  -t "docker-dev:master" -f "Dockerfile" .
Sending build context to Docker daemon 115.3 MB
Sending build context to Docker daemon
Expected Results:
make
docker build  -t "docker-dev:master" -f "Dockerfile.ppc64le" .
....
 You can check out any of the 1.6.x tags and use their make scripts.

I could be misunderstanding this however I was trying to build a latest docker with only a 1.6 instance installed. I'd assume checking out 1.6.x would only build 1.6.x (which I have).

I see. Nothing against changing it, but I would not try to build with a year old version, tbh. I'll leave merging this up to others.

@tianon the floor is yours :)

ya we don't need this, the string in 1.6.x is different that is why

do you have a usecase for this, this is an internal script used for testing the current checkout, we need not support older versions, it kinda makes no sense and adds complexity for us, closing, but if your reasoning is good enough we can reopen, thanks

The hack/make/.detect-daemon-osarch script is used to perform a build (Makefile line 4) and not just testing. Without detection of DOCKER_OSARCH by this script, DOCKER_OSARCH will be blank and use the Dockerfile without the right architecture extension. All arm, arm64, linxu/ppc64le, linux/s390 and windows/amd64 architectures can't upgrade without this patch. I have only a Docker-1.6 version on ppc64le (the latest available 15.10 Ubuntu package). Without supporting the latest available "old" version of docker for various architectures there is no upgrade path for them.

Have you seen master.dockerproject.org :)
On Wednesday, February 3, 2016, Daniel Black<EMAIL_ADDRESS>wrote:

The hack/make/.detect-daemon-osarch script is used to perform a build
(Makefile line 4) and not just testing. Without detection of DOCKER_OSARCH
by this script, DOCKER_OSARCH will be blank and use the Dockerfile without
the right architecture extension. All arm, arm64, linxu/ppc64le, linux/s390
and windows/amd64 architectures can't upgrade without this patch. I have
only a Docker-1.6 version on ppc64le (the latest available 15.10 Ubuntu
package). Without supporting the latest available "old" version of docker
for various architectures there is no upgrade path for them.
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/19825#issuecomment-179577265.

--
Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC  511E 18F3 685C 0022 BFF3
pgp.mit.edu http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3


Fluid Layout : End line too much spacing

When we have a FluidLayout with severals items and we are on the edge of placing the last widget of a line to the next one, the spacing here is too wide.
It's should be like on the left side.
Here is a screen with the problem shown:

Fixed in PhotonUI 1.4.1


Too Slow

I need to wait like a minute or more so I can see the changes in the explorer in any view (group/sort, etc.)

Unfortunately, that is how MS Explorer works, or, rather, how it's caching works. I will check If I can implement re-caching trigger or some kind of update, but not quite sure this is something that would fit the app how I see it, as it might require cracking the work of a system app to force those mechanisms.
However, if there's a ready-to-be-used windows API for that, I'll add support for that.


Update ReadTheDocs

Moves ReadTheDocs to the new configuration file 2 format.
TODO:

[ ] Unit tests and/or doctests in docstrings
[ ] tox -e py310 passes locally
[ ] Docstrings and API docs for any new/modified user-facing classes and functions
[ ] Changes documented in docs/release.rst
[ ] tox -e docs passes locally
[ ] GitHub Actions CI passes
[ ] Test coverage to 100% (Coveralls passes)

@joshmoore could you please review this one?

Thanks Josh! 🙏
Yeah this is one of the things we did in zarr-python. Another was moving to pyproject.toml, which I have a PR ( https://github.com/zarr-developers/numcodecs/pull/382 ) for.


Can we expect integration with Spring Security?

Can we implement some sort of integration with spring security? I think it might look like special GraphQL mutation, witch will be handled by library. The problem is that I cannot configure app using existing '..ConfigurerAdapter's since they rely on filters and with GraphQL we always have one endpoint.
I'd like to avoid manual authentication on home grown mutation.

can you explain a bit more in detail what kind of support you would expect?
thanks

closing because no response

Can we reopen this ?
I wanted to use webflux but cannot seem to be able to accomplish the easiness of spring object mapping that graphql-spring-boot-starter 5.0.2 provided plus the fact that I can't seem to be able to configure the security on the graphql endpoint as well.
For instance, a pretty basic use case would be to only allow authenticated user to the graphql endpoint. Right now, I use a normal rest endpoint to control my authentication flow (login, logout, roles access, etc) and only once he is logged in would I want the user to access graphql queries and mutations.
If it can be achieved with spring security, maybe a note or wiki page on how to do it would help


Modernize and fix #83

Hey @robertleeplummerjr, thanks for the PR!
Will try to find some time over the next week or so to check out these changes.

Hi @robertleeplummerjr, I finally got a chance to look through your changes a bit.  You've been busy!
There's about 25 failures when I run grunt test now.  Is that expected?
I don't see any new or updated tests for this work, which would make it difficult to ensure your changes continue to work as expected.  How have you been testing your changes?

I was testing using manual because of how in-depth the tests were.  We diverged enough from the original codebase that we started this: https://github.com/SaramaJS/sarama.js and are testing using a simplified method, mocha and should.js with acorn and comparing side by side a python string to it's JavaScript equivalent.  By using mocha, we have better ide compatibility for debugging in real time.  This had made development time quite a bit faster, but we didn't want to superimpose our own ideology on your project.
One of the things we descided on for scalability was to make the parser do as little as possible in terms of rewriting for compatibility to run in node or web. This way we can focus just on parsing and not environment.  The environment "casting" (from pythons ast to one acceptable by node) would be done in a different project, one focused on simple rewrites to different parts of the ast.
By abstracting the two portions of the project development time for this project will drop, befause the project is responsible for less, and we'd gain more thorough unit testing.
That was the aim anyway.  We'd be happy to drop the project and merge these into your project if you wanted the changed, but we just didn't want to step on toes.

Here is the branch the work has been done on.  Unit tests currently pass. https://github.com/SaramaJS/sarama.js/pull/11


Simplify RuboCop::AST::Traversal

For all nodes with children, simply iterate over all child nodes and call the appropriate on_#{type} method. Some children are not Node instances, and must necessarily be skipped - these children appear to be either nil or instances of Symbol.
TODO: See if performance is significantly better or worse.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[ ] Added tests.
[ ] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

Any updates here?

I'm closing this PR due to no recent activity. Feel free to re-open it if you ever come back to it.


Add a way to query IP address through a specific interface

Describe the bug
I have two uplinks in my system: wan0 and wan1. wan1 is wired, but is behind a CGNAT. wan0 is a backup link to a sattelite, but I get a dynamic IP address to use with this link that's not behind a CGNAT. The routes are set up such that by default packets are routed through (the CGNATed) wan1.
I would like godns to query the IP address through wan0 link specifically, so that it obtains the IP address through which the machine is actually reachable.
The frequency of this issue

[ ] This issue is easy to reproduce.
[x] This issue is not easy to reproduce. -- requires multiple uplinks

Steps to Reproduce
Obtain two interfaces with a public internet access. Set up the routes such that they read like this:
$ ip route
default dev wan1 proto static scope link metric 1000
default dev wan0 proto static scope link metric 1001
...

Your GoDNS configuration
{
  "ip_url": "https://api.ipify.org/",
  "ip_type": "IPv4",
  "debug_info": true,
  "interval": 3600
}

Expected behavior
It should be possible to ask godns to make a request through wan0 specifically (this can be done by binding the TCP socket I believe) so that the machine making the request is actually reachable over the obtained IP address.
Screenshots
N/A
Environment (please complete the following information):

DNS Provider: N/A
OS: Linux

Additional context
Add any other context about the problem here.

Thanks for your feedback, I think it's possible.
Let me figure out the solution.


enhanced-switch - clearfix

@mpal9000 What was the use case where this was needed?

Hi! Nothing important, I was playing with backgrounds with nested css selectors and this way you can have two full width backgrounds (one inside the other). Also I prefer to wrap floating elements with clearfixs :smiley:

...also you can have the same layout by replacing some float: left with float: right without using calc().
Maybe it's overkill without a serious issue, because of the real elements instead of pseudo.

Thanks @mpal9000


Fix uv readthedocs

Fixed!
It looks like the rtd builds have actually been breaking since I added the notebook examples, I just couldn't notice it then! I've solved both the uv issue and the example notebooks issue.
It did require lowering the rtd fail_on_warning setting, but given the number of other issues we have with the docs at the moment, I think that's acceptable until we get them to a stable state.
I've also enabled rtd build on pull requests so we can catch this before merging next time.
I identified an issue with the directivity_example.ipynb notebook pointing to bugs in the directivity.py code. Opened #32

Documentation configuration updates:

.readthedocs.yaml: Updated the Read the Docs configuration to use a new version format, adjusted the build commands to use uv via a script, and modified the fail_on_warning setting.

README updates:

README.md: Added a Read the Docs badge to the README file.

Example references updates:

docs/source/directivity_example.nblink: Renamed directivity_example.ipynb notebook to avoid a naming conflict with directivity.rst.
docs/source/index.rst: Updated the example list to replace directivity with directivity_example.

perfect ! Good job !


Prevent log message builder from exiting early

Addresses: https://code.ornl.gov/sns-hfir-scse/reflectometry/reflectometry/-/issues/257
To test, Use "Open Sum" and select the two runs specified in the issue, 38749 and 38753.

The story mentions messages for Slit 1, Slit 2, Slit 3, Slit 4, DANGLE and SANGLE, it seems to be missing Slit 4:

I could be wrong, but I can't find any log value corresponding to slit 4. Like if you print all the log values out, there is no slit 4, nor can I find it anywhere in the Das logs tab.

@gecage952 probably worth bringing up at standup or asking the story owner just to be sure.  I'll merge this now and if there is in fact a slit 4 you can make a follow up pr

Apparently, there is no slit4 for this instrument, so we should be good.


ci: conditionally run benchmarks

Documentation PRs unnecessarily waste valuable time in running tests and benchmarks to make it better!
Don't run tests and micro-benchmarks in PRs that only have .md file changes.

/bounty 50$

Kindly assign would like to work!

might i give it a shot?

I'd like to attempt this one
/attempt #860


github pages broke the css

god damnit

fix found: if you are having issues with CSS, press ctrl + shift + r (MacOS: cmd + shift + r)


Fixes gravity calculation for kinematic bodies in Bullet

Resolves #15554, ensures that gravity calculations are done for kinematic rigid bodies and is reflected via get_total_gravity()

This can be merged

Thanks!
@AndreaCatania Do you think this can be cherry-picked for 3.0.x or does it break compat?


Detect dominant language type

This detects language of source code repository at runtime.
Fixes #1

Yeah, I might have a look sometimes, how tokei or loc do their analysis and maybe extract the relevant code, so it is not as big dependency. The reason I decided to go with this, it tokei had quite nice library api next to their command line utility, so I used that.


Update main CI

Trying to resolve https://github.com/coverallsapp/github-action/issues/26
Also a couple other updates.

⚠ This issue has been locked due to age. If you have encountered a recent
problem that seems to be covered by this issue, please open a new issue.
Please include a minimal reproducible example
when opening a new issue.


Add optional destination chain and verbose opts for CLI query channels

Closes: #1132
Description

Add optional destination chain and verbose opts for CLI query channels. If a --destination-chain/-d is specified we only output those channels whose counterparty is the destination chain. If the --verbose/-v flag is set, the output has the same structure as the ChannelEnds described in #1130.

For contributor use:

[ ] Added a changelog entry, using unclog.
[ ] If applicable: Unit tests written, added test to CI.
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Updated relevant documentation (docs/) and code comments.
[ ] Re-reviewed Files changed in the Github PR explorer.

There are some opportunities for code refactor/reuse/deduplication here. For example ->

The impl Runnable for QueryChannelEndsCmd and QueryChannelsCmd are both using a lot of the same code i.e. getting the counterparty chain from a channel.
Also the channel_connection_client() is using much of the same code although it takes a impl ChainHandle as input instead of impl Chain.
I think we can separate out common trait methods in the Chain and ChainHandle traits into a super trait, e.g.

pub(crate) trait Queryable {
    fn query_commitment_prefix(&self) -> Result<CommitmentPrefix, Error>;

    fn query_compatible_versions(&self) -> Result<Vec<Version>, Error>;

    fn query_latest_height(&self) -> Result<ICSHeight, Error>;

    // etc.
}

pub trait Chain: Sized + Queryable + Provable {}

This should allow us to reuse more code in the future.

@soareschen could you please help with reviewing and testing this PR?


Create branch global-tests.

see issue title. This branch will be used for every global tests.

completed.