Code coverage

Hi there,
Is it possible to obtain a code coverage report for Nextflow scripts using nf-test? If not, would you have any alternative suggestions?
Many thanks in advance!

https://github.com/pcolby/tap-summary
https://github.com/nf-core/methylseq/blob/5893122512add5350e4d9e3235b8e2fb500e5679/.github/workflows/ci.yml#L70-L76

After some deeper thinking, me, @aaron-fishman-achillestx and @byb121 came to a conclusion that implementing a code coverage tool should involve discussion on the paradigm and level of coverage measurement first. Here are a few questions that we think that would help to develop the code coverage feature:

What level of the granularity should be considered the lowest when measuring the coverage using nf-test? Would that be a workflow, process or function?
How do we want to measure the coverage? Will that be a simple measurement of executed lines or would we want to measure some of the branch coverage as well? If this would be a measurement of executed lines, then how should we approach shell blocks containing code from a different programming language?
How useful would that actually be to know the code coverage of Nextflow unit tests? Would it be the % value of test_files/ whole NF codebase? From my understanding, most of the times your process will execute one command that has specific purpose, without additional branch logic, so de facto existence of your test will guarantee 100% code coverage of the file - will code coverage by useful at that point?

I believe, that aproper scope should be defined on what users would want to have out of this feature to be able to progress with code coverage. I would also assume that this could be implemented as a plugin or a core functionality @lukfor ?

I was thinking this would make sense as a plugin.

how should we approach shell blocks containing code from a different programming language
I think just an if statement in a script: directive would be nice to know if that was covered or not.

How useful would that actually be to know the code coverage of Nextflow unit tests?
I think this is a good point. Probably not a huge priority IMO, but at a minimum telling you how many .nf files are covered would be nice just to see if you're missing anything.

+1 for this.

I'm also interested in this. My interpretation of the request (to clarify in relation to this comment, is as follows.
Given a project that implements a set of X functions, Y processes, and Z workflows, it would be nice to run something like:
nf-test test coverage

And see a report that A/X functions, B/Y processes, and C/Z workflows are run by tests, together with a summary of their pass/fail status.

I'll add a +1 for the coverage feature. Knowing whether or not we'll have the equivalent of coverage run -m pytest was one of the first questions my team had when I started talking up nf-test. I think using the coverage tools from the python ecosystem could be a good model for how we should think about implementing coverage here. I also think that it's worth thinking about this from the perspective of what is the minimum viable product (MVP) that will begin giving value to our users.

What level of the granularity should be considered the lowest when measuring the coverage using nf-test? Would that be a workflow, process or function?

Coverage should be measured at multiple levels so that CI can fail a PR for reducing coverage at the file and overall level. For an MVP, we should measure coverage at the level of each *.nf file and for the whole repo.

How do we want to measure the coverage? Will that be a simple measurement of executed lines or would we want to measure some of the branch coverage as well?

An MVP that only measures lines executed would already be very valuable. A fast-follow addition that adds branch coverage would also be very valuable but that could wait.

If this would be a measurement of executed lines, then how should we approach shell blocks containing code from a different programming language?

For an MVP we should focus solely on code written in nextflow/groovy. The only other language that is potentially worth considering in my view is bash, but my cursory search didn't reveal a mature bash test coverage that we could tap into.

How useful would that actually be to know the code coverage of Nextflow unit tests? Would it be the % value of test_files/ whole NF codebase? From my understanding, most of the times your process will execute one command that has specific purpose, without additional branch logic, so de facto existence of your test will guarantee 100% code coverage of the file - will code coverage by useful at that point?

I think this is an extremely useful feature. Although the simplest use case of a nextflow process is very straightforward as you describe, mature nextflow pipelines can grow substantially in complexity. For example nf-core/rnaseq supports multiple aligners and there is sometimes branching logic needed in downstream processes based on which upstream process was used.
All in all, I think this is extremely valuable feature and one that is likely to drive a greater adoption of nf-test by developers and organizations by providing a clear measurable metric by which they can assess the quality of their testing.

We’d be very interested in this at Genomics England as we have minimum coverage thresholds which we must adhere to in order to use the code in production, since our bioinformatics workflow is an accredited medical device. Any suggestions on how we can calculate coverage in the meantime? @edmundmiller I spotted your link in the second comment above but I wasn’t sure what to make of it

@lukfor I noticed in your nf-test pre-print (awesome read by the way) that there is a discussion of calculating test coverage. I couldn't find any documentation or details about how that was done. Is there perhaps some way already in place to accomplish that?


fix: add DAPS token scope and validation rule

What this PR changes/adds
This PR adds a possibility to provide a TokenDecorator that allows to set the scope property on request egress. If no TokenDecorator is registered, a no-op decorator is used.
The oauth2-daps module introduces a DapsTokenDecorator that sets a (configurable) scope, that defaults to idsc:IDS_CONNECTOR_ATTRIBUTES_ALL.
Further, a DapsValidationRule is added to the oauth2-daps module, that validates the referringConnector claim upon request ingress.
Why it does that
DAPS rejects any token request without the scope claim. The class that did that previously was deleted from the code base.
Further notes

Linked Issue(s)
Closes #3094
Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.

Codecov Report
Patch coverage: 32.43% and project coverage change: -0.05 :warning:

Comparison is base (905ef20) 64.41% compared to head (ed16d7e) 64.36%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3100      +/-   ##
==========================================
- Coverage   64.41%   64.36%   -0.05%
==========================================
  Files         939      941       +2
  Lines       19186    19216      +30
  Branches     1032     1035       +3
==========================================
+ Hits        12359    12369      +10
- Misses       6438     6458      +20
  Partials      389      389

Impacted Files
Coverage Δ

...org/eclipse/edc/iam/oauth2/daps/DapsExtension.java
0.00% <0.00%> (ø)

...clipse/edc/iam/oauth2/daps/DapsValidationRule.java
0.00% <0.00%> (ø)

...eclipse/edc/protocol/dsp/DspHttpCoreExtension.java
71.42% <100.00%> (+71.42%)
:arrow_up:

...dispatcher/DspHttpRemoteMessageDispatcherImpl.java
100.00% <100.00%> (ø)

...clipse/edc/iam/oauth2/daps/DapsTokenDecorator.java
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Energy can not be inserted into database with 2023.3

Describe the bug
A clear and concise description of what the bug is.
The energy data is not visbile in the "energy" tab. in logfile there are error eintries.
Log Entry:
2023-03-03 15:12:18.561 ERROR (MainThread) [homeassistant.helpers.entity] Update for sensor.energy_central_heating fails
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 541, in async_update_ha_state
await self.async_device_update()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 739, in async_device_update
raise exc
File "/config/custom_components/bosch/sensor/energy.py", line 89, in async_update
await self._insert_statistics()
File "/config/custom_components/bosch/sensor/energy.py", line 151, in _insert_statistics
if end_time and day_dt.date() <= end_time.date():
AttributeError: 'float' object has no attribute 'date'
2023-03-03 15:12:18.570 ERROR (MainThread) [homeassistant.helpers.entity] Update for sensor.energy_hot_water fails
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 541, in async_update_ha_state
await self.async_device_update()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 739, in async_device_update
raise exc
File "/config/custom_components/bosch/sensor/energy.py", line 89, in async_update
await self._insert_statistics()
File "/config/custom_components/bosch/sensor/energy.py", line 151, in _insert_statistics
if end_time and day_dt.date() <= end_time.date():
AttributeError: 'float' object has no attribute 'date'
To Reproduce
Update home assistant docker image from 2023.2.5 to 2023.3
Then start it.
Expected behavior
After a day, the last day data is visible in the energy tab. (No error entries in log )
Version

HA version? 20233
HA Bosch component version? v0.20.2
Python Bosch HTTP client version? where to find it?

** Debug SCAN **
** IMPORTANT **
Go to Developer tools in Home Assistant, choose Service tab and choose bosch.debug_scan
Download file to your computer and upload it somewhere eg. https://jsonblob.com/
this service not executes in my instaes.
Log: websocket_api script: Error executing script. Error for call_service at pos 1: Error rendering data template: Result is not a Dictionary
Additional context
Add any other context about the problem here.
home-assistant_bosch_2023-03-03T14-12-45.551Z.log

in the file /custom_components/bosch/sensor/energy.py  replace line to
end_time = datetime.datetime.utcfromtimestamp(last_stats_row["end"])
look for last_stats_row["end"]

Code proposed by @MG-Sky doesn't work for me - leads to "TypeError: can't subtract offset-naive and offset-aware datetimes" as utcfromtimestamp method creates an datetime object without timezone information while today object has timezone info.
Below code works for me:
end_time = datetime.datetime.fromtimestamp(last_stats_row["end"], tz=datetime.timezone.utc)
It is necessary to update both /custom_components/bosch/sensor/energy.py as well as /custom_components/bosch/sensor/recording.py files.

thanks I haven't check to recording as my energy stats are written   daily - 8ll change recorder too thanks!

Thanks for the solutions, it looks good again

yes, but no data in database.

I have restarted both the server and my app on phone , and the data for the last two days are visible. I always had a 1 day delay in the data. I do not checked the database itself, but I think it
Is fair to assume they had been read out form the database.  I have changed both of the files simultaneously, and also made a restart of ha server after that.

after some try&error , i replace:
end_time = last_stats_row["end"]
with:
end_time = datetime.datetime.fromtimestamp(last_stats_row["end"], tz=datetime.timezone.utc)
but i never see gas usage on "energy".
maybe i change the wrong line?

Fixed in v0.21.0 https://github.com/bosch-thermostat/home-assistant-bosch-custom-component/releases/tag/v0.21.0
Let me know if there are any problems.

@erreesse enable new API (follow wiki) and then add energy: statistic to your graph

@erreesse enable new API (follow wiki) and then add energy: statistic to your graph

i had to disable "use new statistic API" because , if "enabled" i got negative value in sensor.energy_central_heating
with  "use new statistic API" enabled, i hadn't "energy:" entity !
maybe i miss someting, i followed the "new api wiki" ( https://github.com/bosch-thermostat/home-assistant-bosch-custom-component/wiki/New-API-support ) , but in "configure" i had only

i can't : "Enable Recording or Energy (depends of device you have) sensor you want to gather statistic."

@erreesse that's how it is suppose to look. -17 it means it put data directly into statistics table in database.

@pszafer ok. but in "energy dashboard" i see 0 in "gas" ,  i had changed the sensor, from "sensor.energy_central_heating" to "sensor:echenergy_central_heatingexternal"  , bit still 0 in the graph.
i don't have "energy:" or "energy."  to use


Releases far behind masterr

As I understand it, go.mod will default to getting the latest released version on github, not master.  Since the last github release was in 2016, the code go mod grabs is very far behind what is on godoc.
Please consider releasing the current version.

too

go get github.com/tebeka/selenium@master


⚠ SceneTime has degraded performance

In bf7c59c, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 11186 ms

Resolved: SceneTime performance has improved in 62180c9.


chore(sqla): Add explicit bidirectional performant relationships for SQLA model

SUMMARY
In conjunction with https://github.com/dpgaspar/Flask-AppBuilder/pull/1961, this PR defines the explicit bidirectional relationship mapping between the SqlaTable model (parent) and the corresponding SqlMetric and TableColumn models (children) to improve the performance of the GET /api/v1/dataset/{pk} endpoint. This is a rehash of https://github.com/apache/superset/pull/22332 after it was discovered there were numerous shortcomings.
The motivation was driven by a specific (and somewhat atypical) virtual dataset at Airbnb which is comprised of over 7,000 columns and 30,000 metrics where previously the /api/v1/dataset/{pk} endpoint would timeout due to:

FAB was using Joined Eager Loading which "eagerly" loads the children alongside the parent, however this results in a slew of LEFT OUTER JOINs which will rapidly result into a expansive interim result set—from a multiplicative combinatorial sense—given that the SqlaTable model has potentially multiple high cardinality relationships (owners, columns, metrics). Furthermore the result set—which is repetitive in nature—requires further overhead by SQLAlchemy to de-duplicate the rows against the actual model record.
Superset was using lazy loading (see this Slack thread) when back referencing the TableColumn.table field when evaluating the columns.type_generic field which resulted in a SELECT statement per column, i.e., for each column it would execute a SQL statement to fetch the corresponding table ID.

The first issue is partially addressed in https://github.com/dpgaspar/Flask-AppBuilder/pull/1961 in conjunction with this change—as FAB defers the relationship loading logic to Superset—which meant that the response time (excluding downloading of the payload) went from timing out to ~ 20 seconds. The second issue was addressed by defining eager loading of the back (parent) references which are pre-defined whilst fetching the SqlaTable model. This further reduced the response time to ~ 5 seconds. Note this PR is safe to  merge prior to the FAB change as in the context of FAB the selectin—a more extensible form of select which can be used when fetching multiple SqlaTable columns—relationship loading technique is ignored given that FAB is using an explicit joinedload which is akin to lazy="joined".
Note to further improve the performance of the GET /api/v1/dataset/{pk} endpoint all requests should likely provide a filter which includes only the subset of the columns required. This should both reduce the backend complexity as well as reduce the size of the response payload.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS
CI. Tested locally.
 BTW I'm not sure that the two queries (which include the FAB change) are complete as they don't include the additional SELECT statements for fetching the columns and metrics and thus I'm also not sure if your timings include those facets.
The TL;DR—regarding performance—from you analysis is:

FAB
Superset (FWD)
Superset (BWD)
Endpoint
Response Time
Difference

joined
joined*
select
/api/v1/dashboard/
~ 27 ms
N/A

joined
joined *
select
/api/v1/dashboard/1
~ 2,300 ms
N/A

default
select
select
/api/v1/dashboard/
~ 61 ms
+ 34 ms (+ 126%)

default
select
select
/api/v1/dashboard/1
~ 875 ms
- 1,425 ms (- 62%)

default
selectin
joined
/api/v1/dashboard/
~ 61 ms
+ 34 ms (+ 126%)

default
selectin
joined
/api/v1/dashboard/1
~ 610 ms
- 1,690 ms (- 73.5%)

* By enforcement via FAB.
which indicates that there is a performance hit (albeit relatively small from an absolute perspective) for the /api/v1/dashboard/ endpoint, however there is a signification performance improvement (from both an absolute and relative perspective) for the /api/v1/dashboard/1 endpoint, i.e., the results seem quite favorable.
Regarding the get list concern—generating N + 1 queries—I'm not sure if this will materialize given we're using the selectin` eager loading which mentions that,

In most cases, selectin loading is the most simple and efficient way to eagerly load collections of objects.

i.e., though multiple SELECT statements will exists, the number is determined via the number of relationships as opposed to the number of items in the list/collection.
Additionally with regards to https://github.com/apache/superset/pull/22332, which added the /api/v1/dataset/<id>/column/ endpoint, per https://github.com/apache/superset/pull/22332#issuecomment-1344949577, I realized that this wasn't suffice given what is required by the frontent.

Thanks @dpgaspar for the detailed analysis. BTW I'm not sure that the two queries which include the FAB change (and their associated timings) are complete as they don't include the additional SELECT statements for fetching the columns and metrics.
The TL;DR—regarding performance—from you analysis is:
FAB	Superset (FWD)	Superset (BWD)	Endpoint	Response Time	Difference
joined joined*	select	/api/v1/dashboard/	~ 27 ms	N/A
joined joined *	select	/api/v1/dashboard/1	~ 2,300 ms	N/A
default select select	/api/v1/dashboard/	~ 61 ms	+ 34 ms (+ 126%)
default select select	/api/v1/dashboard/1	~ 875 ms	- 1,425 ms (- 62%)
default selectin joined	/api/v1/dashboard/	~ 61 ms	+ 34 ms (+ 126%)
default selectin joined	/api/v1/dashboard/1	~ 610 ms	- 1,690 ms (- 73.5%)

By enforcement via FAB.

which indicates that there is a performance hit (albeit relatively small from an absolute perspective) for the /api/v1/dashboard/ endpoint, however there is a signification performance improvement (from both an absolute and relative perspective) for the /api/v1/dashboard/1 endpoint, i.e., the results seem quite favorable.
Regarding the get list concern—generating N + 1 queries—I'm not sure if this will materialize given we're using the selectin` eager loading which mentions that,

In most cases, selectin loading is the most simple and efficient way to eagerly load collections of objects.

i.e., though multiple SELECT statements will exists, the number is determined via the number of relationships as opposed to the number of items in the list/collection.
Additionally with regards to #22332, which added the /api/v1/dataset/<id>/column/ endpoint, per #22332 (comment), I realized that this wasn't suffice given what is required by the frontent.

Ok, I've added a small change on FAB on top of your PR there, now we can have the best from both worlds: https://github.com/dpgaspar/Flask-AppBuilder/pull/1971
This is included on 4.2.0
So use:
show_outer_default_load = True RestModelApi attr when needed to improve performance /api/v1/dataset/<id>

Note I've discovered a performance regression with the api/v1/explore/?slice_id={pk} RESTful API endpoint which is likley associated with change which I'm currently investigating.


Unable to filter to a specific circuit.

Describe the bug
Unable to filter to a specific circuit.
To Reproduce

Visit http://ergast.com/api/f1/circuits/monza.json
Notice Ergast returns details about Monza.
Visit the same endpoint in Jolpica: http://api.jolpi.ca/ergast/f1/circuits/monza
Notice Monza details are not returned.

Expected behavior
Jolpica should return the same information for circuit related filtering.

As noted by @jolpica in Slack, http://api.jolpi.ca/ergast/f1/circuits/monza.json does work. Just dropping this here for our future selves, should we run into similar issues.


Add ColorScheme environment

Was temporarily blocked by #135.

Fails

:no_entry_sign:

Sources/TokamakDOM/DOMRenderer.swift#L62 - Line should be 100 characters or less: currently 127 characters (line_length)

  Generated by :no_entry_sign: Danger Swift against 34545ed97c6f41bb21a5f15b52f052d2ea692700

Yes, the listener works, but environment changes aren't propagated properly yet, I'm looking into that now.

Is this unblocked since #170 ?

I'm still seeing some issues with environment propagation (at least in this PR), working on that right now...

If you’re going to add support for dark mode it should also add the CSS property color-scheme: light dark to the html or body element. Not sure if there’s a way to forcibly enable or disable dark mode support in swiftui, but it should set it to light only or dark only if so.

I believe you would do that with environment(\.colorScheme, .light)

Good point. It still doesn't work because there's apparently a bug somewhere in JSClosure implementation in JavaScriptKit which makes the color scheme observer overwrite the scene phase observer, both can't exist at the same time. I'm trying to isolate it now and this is what led me to the attempt at reviving our test suite in #229. Now I see the test suite can't run on WASI, I'm back to looking into the test suite of JavaScriptKit...

I've fixed the issue with observers overwriting each other's closures. Turns out we have to be very careful with the lifetime of JSClosure. If an instance of it is not retained by the time next JSClosure instance is created, their ObjectIdentifier values can become the same, which will lead to hard to diagnose issues. PR to JavaScriptKit is coming soon.
I'm still seeing what seems like yet another environment propagation issue, but it's only reproducible with the environment not going as far as EnvironmentDemo, views closer to the root get colorScheme appropriately when the system scheme changes.

The last known environment update issue is fixed and it's ready for review. You can see the dark scheme environment text representation updated in EnvironmentDemo. As it has been open for a while now, I suggest adding default dark mode styles in a separate PR, I've created #237 as a reminder for that.

I gave it a test and it seems like clicking the dark mode toggle in Safari Developer Tools does not work unless you click it before the first render — subsequent changes are not reflected in the view.

I was testing with the globally set dark mode in System Preferences, do you see the same behavior with that one?

Yes, I see the same behavior there

Hm, I had this same issue caused by the environment bug, which was fixed in c397eb6c13ec3945fd1618c07de87f4a633589dd. Maybe you're not on the latest commit of this branch?

It's working for me in Chrome. Maybe a bug in Safari?


Forwardport: Login fails for users with escaped comma in CN when using Active Directory LDAP authentication

Forward port: https://github.com/rancher/rancher/issues/22192

The feature is validated in rancher\rancher:master-head version d6a9d4509.
Enable authentication with Active Directory.
Connect to AD management server via Microsoft Remote Desktop
Add users with escaped comma.
Login as user with comma in dn.
Lock back in as admin and check that a "User" account is created on the Global->Users page for this user with comma in dn.
While on the Global->Users page click on the "Refresh Group Memberships" button.
Check Rancher's log to make sure there are no errors.


Add tests for status pkg

Description of change

Add testings for status pkg

This change is

Aim for 100% test coverage


 Delete the others, including these Instructions.
{{cask_file}} represents the cask file you’re submitting/editing (if applicable).
If there’s a checkbox you can’t complete for any reason, that’s OK. Just explain in detail why you weren’t able to do so.

Changes to a cask
Editing an existing cask

[x] Commit message includes cask’s name (and new version, if applicable).
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

Adding a new cask

[x] Checked there aren’t open pull requests for the same cask.
[x] Checked there aren’t closed issues where that cask was already refused.
[x] When naming the cask, followed the token reference.
[x] Commit message includes cask’s name.
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] brew cask install {{cask_file}} worked successfully.
[x] brew cask uninstall {{cask_file}} worked successfully.

Changes to the core

[ ] Followed hacking.md.

Hmmm.. Everything passed when I tested. I'll fix.

Thank you for the contribution. It needed some fixes, so they were made in 147f0b8311d768b170d9938e29862507df6f1711. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


[BUG] - Grafana Dashboard Link from the Rancher UI broken even after applying PR [harvester-installer/pull/666]

Describe the bug
Grafana dashboard broken even after applying PR.
To Reproduce
Steps to reproduce the behavior:

Update the grafana-nginx-proxy-config configmap in the cattle-monitoring-system on the harvester cluster with data from the map $http_referer $final_appSubUrl block mentioned in the following PR.

      map $http_referer $final_appSubUrl {
        ~.*/k8s/clusters/(c-m-.+)/api/v1/namespaces/cattle-monitoring-system/.*    '"appSubUrl":"/k8s/clusters/$1/api/v1/namespaces/cattle-monitoring-system/services/http:rancher-monitoring-grafana:80/proxy"';
        ~.*/dashboard/harvester/c/(c-m-.+)/kubevirt.io.virtualmachine/.*/.*    '"appSubUrl":"/k8s/clusters/$1/api/v1/namespaces/cattle-monitoring-system/services/http:rancher-monitoring-grafana:80/proxy"';
        ~.*/dashboard/harvester/c/(c-m-.+)/harvesterhci.io.addon/cattle-monitoring-system/.*    '"appSubUrl":"/k8s/clusters/$1/api/v1/namespaces/cattle-monitoring-system/services/http:rancher-monitoring-grafana:80/proxy"';
        ~.*/dashboard/harvester/c/(c-m-.+)/.*      '"appSubUrl":"/k8s/clusters/$1/api/v1/namespaces/cattle-monitoring-system/services/http:rancher-monitoring-grafana:80/proxy"';
        ~.*/dashboard/c/(c-m-.+)/.*     '"appSubUrl":"/k8s/clusters/$1/api/v1/namespaces/cattle-monitoring-system/services/http:rancher-monitoring-grafana:80/proxy"';
        ~.*/api/v1/namespaces/cattle-monitoring-system/services/.*    '"appSubUrl":"/api/v1/namespaces/cattle-monitoring-system/services/http:rancher-monitoring-grafana:80/proxy"';
        default    '"appSubUrl":"/k8s/clusters/local/api/v1/namespaces/cattle-monitoring-system/services/http:rancher-monitoring-grafana:80/proxy"';
      }

Login to Rancher --> Click the Hamburger Menu --> Virtualization Management --> Manage button of the Harvester cluster -->  Explore button --> Workloads --> Pods --> Delete the rancher-monitoring-grafana pod from the cattle-monitoring-system namespace to spin up a new pod.

Logout and Login to Rancher --> Click the Hamburger Menu --> Virtualization Management --> Manage button of the Harvester cluster --> Explore button --> Click on Grafana button.

Clicking on the link opens a new tab to Grafana with broken dashboard.

Click on home dashboard link and then Sign In button throws the following error message:

{
  "kind": "Status",
  "apiVersion": "v1",
  "metadata": {},
  "status": "Failure",
  "message": "services \"rancher-monitoring-grafana\" not found",
  "reason": "NotFound",
  "details": {
    "name": "rancher-monitoring-grafana",
    "kind": "services"
  },
  "code": 404
}

--
Expected behavior

Clicking on the Grafana page should not display a broken dashboard but instead pull metrics from the harvester cluster and display them.

Support bundle
harvester-supportbundle_1ea678f1-4959-40f8-b4b0-98313a5360f3_2024-03-18T11-14-17Z.zip
Environment

Harvester ISO version: v1.2.1
Rancher 2.7.6
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630): Baremetal with Equinix

Additional context

Before clicking on the Grafana link, if you open browser Inspect option and search for noreferrer string in the Element tab.
You would see something similar to this:

Grafana

When you hover your mouse on top of this link, it should highlight the Grafana button.
Remove the noreferrer string from the URL in the Elements page and then click on the Grafana link and the dashboard would work fine.

Before

After

@torchiaf
For the Rancher UI, the link to Grafana needs to remove the noreferrer as well, just like the Harvester dashboard Grafana link.  Could you help check is this an Rancher UI bug or Harvester UI bug, thanks.
Rancher --> Click the Hamburger Menu --> Virtualization Management --> Manage button of the Harvester cluster --> Explore button --> Click on Grafana button

@torchiaf I moved the milestone to v1.4.0 and created backport issues for tracking purposes, feel free to close backport issues if not needed.

@torchiaf - The Rancher version is indeed v2.7.6. Made a typo and have corrected that. Will test the fix with Rancher v2.7.7 and harvester v1.2.1 and will share the updates.

@torchiaf - I was able to test the fix with v2.7.7 and it works. Thanks!

@stalin4suse @bk201 should this is issue and its back-ports be closed ?
https://github.com/harvester/harvester/issues/5422
https://github.com/harvester/harvester/issues/5423


x/website:

What is the URL of the page with the issue?

What is your user agent?
[Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36](https://explore.whatismybrowser.com/useragents/parse/?analyse-my-user-agent=yes)
Screenshot

What did you expect to see?
When in dark mode and attempting to change the theme, I observed that although the icon associated with the theme updates immediately, the actual theme itself does not reflect the change until I perform an additional click. This behavior results in a discrepancy between the displayed theme icon and the applied theme.
What did you see instead?
it should the change the theme when i click on the change theme icon

It's a 3 state button: light, dark, system.
I believe this is working as intended.


Minor spelling updates

Please see the diff for details:

update CamelCase spelling for CloudOps and GitHub
update Canda > Canada

Sorry, I didn't see this PR and I just opened another PR (#26) with the same changes plus a close bracket.

Thanks for catching the spelling mistakes! I'll merge this request.


Creation of kuksa-common - way forward

Background
We have said that we want to split this repo into multiple repos in eclipse-kuksa. As part of https://github.com/eclipse/kuksa.val/wiki/Proposal-for-splitting-of-KUKSA.val it is proposed to create a repo kuksa-common where data typically needed by multiple applications are stored. This issue intends to propose a way forward
Step 1: Copy files to new repo

VSS files to be copied to new repo, folder called vss-json
Databroker JWT copied to new repo, folder called jwt. This is based on the assumption that in the long run (> 3 years) we will either not have KUKSA Server any longer, or it should have migrated to the tokens currently used by Databroker
TLS related data in kuksa_certificates copied to new folder tls
Server JWT (except scripts) copied to folder jwt/server
JWT scripts copied to folder jwt
proto/kuksa copied to new folder proto/kuksa
kuksa_databroker/proto/sdv copied to new folder proto/sdv
kuksa-val-server/protos will not be moved/copied

Step 2: Remove files from kuksa.val and add kuksa-common as submodule

Remove all files copied
Add kuksa-common as submodule on top level.
Adapt build scripts to either use files from the submodule or alternatively use other mechanism like separate clone
Acceptance criteria: Tools works as before

Discussion points

We have previously discussed removing certs/tokens from default installations, but that seems like a lengthy process so maybe best to first just migrate everything
An alternative approach would be to just copy, not remove, and wait with migration until the corresponding components are migrated, but that will make migration somewhat more complex, then there is a risk that certs are just copied.
PR #676 mist first be merged/closed

FYI @SebastianSchildt - I think we should get the migration going, this seems like a reasonable first step but should better be discussed.

Kuksa-common created, no files removed yet, Proto files still here. I think this one can be closed


On the domain users view role selector not possible to select GUEST role

@kbeyro one other thing to note is that the user search on this view also looks for usernames but it should only match against the email address

Ready for re-test


Support monitor.isDragging bool

Fixing a bug where monitor.isDragging() is always false for the KeyboardBackend because it doesn't provide the proper options and call to publish the drag source

@faultyserver Tagging for visibility if this repo still has active support

Sorry for the major delay. This is a great addition I didn't even realize was missing before, thank you.


Configurator/demo not load.

I have followed the steps to run the configurator/demo locally, and I get only a loading state. Seems a CORS issue.
I'm trying this because the same issue is affecting my app where I use playground-elements.
https://user-images.githubusercontent.com/8403993/118699479-f7580000-b7e7-11eb-81ff-7c137839fb5d.mov

I've just released v0.9.4 with https://github.com/PolymerLabs/playground-elements/pull/158 that I'm optimistic will fix this issue. Could you upgrade and confirm?

Marking this fixed, but please re-open if it is still an issue.


Integration - Setup of foi-request (Pre-requiste)

Repository to use - > https://github.com/bcgov/foi-requests


Feature request: add Condition attributeContainsValue (java.lang.String attributeName, java.lang.String matchedExpectedAttributeValue)

Feature request
The problem
Condition attribute requires exact text of attribute value.
Sample: $("#mydiv").shouldHave(attribute("fileId", "12345"));
It would be great to have Condition to verify attribute by matched text:
Example:
attribute "field" can contains value: "Job12345", "12345 job", etc. Where "12345" - is matched text.
$("#mydiv").shouldHave(attributeContainsValue("fileId", "12345"));
My environment

**Selenide Version:5.3.1
**Chrome\Firefox\IE Version: 77.0.3865.120\69.0.2
**Browser Driver Version:
**Selenium Version:3.141.59
**OS Version:Ubuntu 18.04.3 LTS

Code To Reproduce Issue [ Good To Have ]
Please remember that with sample code it's easier to reproduce the bug.
It's not good example, but:
http://the-internet.herokuapp.com/dynamic_content?with_content=static
$("content img").shouldHave(attributeContainsValue("src", "/img/avatars/Original-Facebook-Geek-Profile-Avatar-"));

@sergeyrybakin Thanks for a report!
In selenide we trying to hold only few basic conditions.
But! You can easily implement this condition by your own - just need to implement Condition.
Also you can find examples on our wiki page.
Is this message helps you?
Write back please with answer and extend our wiki with you condition implementation.
Feel free to close this issue if no question left.
Thanks!

@rosolko What about overloaded condition attribute with regexp parameter?
Something like this:
$("#mydiv").shouldHave(attribute(Pattern.compile(".*12345.*")));

// or

$("#mydiv").shouldHave(attributeMathching(".*12345.*"));


Rails 6 support

Hi,
any chance to relax dependencies to support Rails 6?
s.add_dependency 'activerecord', '>= 5.0', '< 6.0'
s.add_dependency 'activesupport', '>= 5.0', '< 6.0'

Released v0.5.0!
https://github.com/glebm/order_query/blob/v0.5.0/CHANGES.md


Warning thrown when a callback has multiple outputs and multiple inputs and the number of inputs != number of outputs

The following results in a warning after trying to add the callback:
library(dash)
app <- Dash$new()
app$layout(dashHtmlComponents::htmlDiv())
app$callback(
  list(
    output("out1", "children"),
    output("out2", "children")
  ),
  params = list(
    input("in1", "value"),
    input("in2", "value"),
    input("in3", "value")
  ),
  function(...) {}
)

After some testing, I think it might only happen when there are multiple inputs and multiple outputs and the number is not equal. If you remove one output, it works (1 vs 3). If you remove one input it also works (2 vs 2). If you add an output, it works (3 vs 3).

The problem seems to come from this validation check: https://github.com/plotly/dashR/blob/5c80285b90c24cf15375e6763b56425e9dc530ce/R/utils.R#L458-L460
It also seems that the error happens when:

of outputs > 1

of inputs > 1

of inputs is not divisible by # of outputs


2.1.0-4U_ctrtransfer_o3ds name error in Dcrypt9

Hello!! I have a problem when Im tryng to downgrade a 11.0 OLD 3ds usa.
1- I put all the necesary in the SD
2 - Start the hombrew using SOUNDHAX
3 - Enter in the decrypt9
4 - go to "SysNAND Options/CTRNAND transfer/CTRNAND Transfer"
5 - put the characters and displayed a lost of no sense character.
Correct displayed:
Selected: [Auto CTRNAND Transfer]
A9LH not detected, checking FIRM0...
Select CTRNAND transfer image
Use arrow keys and  to choose a file
2.1.0-4U_ctrtransfer_o3ds.bin
non sense character displayed:
Selected: [Auto CTRNAND Transfer]
A9LH not detected, checking FIRM0...
Select CTRNAND transfer image
Use arrow keys and  to choose a file
}╦╠░¢ Decryption success!
Im tryning a lot of diferent memories a lot of ctrnand transfer file mirrors.... I dont know what to do...

I do not understand.


Fix to keep the compatibility with Python 2

The project doesn't support old Python for a while and I'm not sure trying to get that support back will end with the satisfying result for anyone 🙂

The project doesn't support old Python for a while and I'm not sure trying to get that support back will end with the satisfying result for anyone 🙂

Coverage remained the same at 0.0% when pulling a1c2597ed7d4d74bcb0a05de580991a2c65d8720 on AndreAngelucci:bugfix/graph_models_python_27 into fe74542d51e5f1d780bba01de80f6bdca9437135 on django-extensions:master.

Coverage remained the same at 0.0% when pulling a1c2597ed7d4d74bcb0a05de580991a2c65d8720 on AndreAngelucci:bugfix/graph_models_python_27 into fe74542d51e5f1d780bba01de80f6bdca9437135 on django-extensions:master.

Got it, thanks for the feedback!

Got it, thanks for the feedback!


Upgrade file-type

Link to reproduction
https://github.com/payloadcms/payload/blob/08fdbcacc02c1768770516c2dd2e22df7bc48843/packages/payload/package.json#L97
Environment Info
beta.100

Describe the Bug
payload 3 currently uses file-type 17 while the latest is file-type 19. The newer version is smaller and would remove a couple of transitive dependencies
Reproduction Steps
See https://github.com/payloadcms/payload/blob/08fdbcacc02c1768770516c2dd2e22df7bc48843/packages/payload/package.json#L97
Adapters and Plugins
No response

I'll upgrade this to file-type 19.3.0. Cannot upgrade to the latest (file-type 19.4.0 and higher) because of the following:

they added 3 new dependencies through https://github.com/sindresorhus/file-type/pull/649 (total: 9). 19.3.0 seems to have the least amount of dependencies (total: 6). 17.1.6 had 11.
they decided to go full combat mode against typescript. https://github.com/sindresorhus/file-type/commit/5034fc1f7025554b1df81ed6cb1d3b32ce239eb2 broke file-type when used in combination with moduleResolution: Bundler or jest (jests esm support sucks, it's probably falling into the same trap). They're trying to get typescript to fix it and keep the package broken for these configurations until that happens: https://github.com/sindresorhus/file-type/issues/652. I respect that, but I'd also like to stay as far away from this as possible until it's resolved

that's awesome!! thank you!

Damn I wish there was a way to add comments to package.json's to document these decisions

there is, but it's really hacky...
"comment": "we use version 19.3 of file-type because..."

Fixed by https://github.com/payloadcms/payload/pull/8195

there is, but it's really hacky...
"comment": "we use version 19.3 of file-type because..."

Yeea that might be a bit too hacky - I'm hoping for jsonc support or something like that to be added in the future!


MCProtocolLib data usage

How much RAM usage and network usage does mcpl typically use per player? (Doubt CPU stats could be given reliably)

That hugely depends on what you plan to do with your players. Usually only a keepalive packet is sent when no actions are taken, thus network traffic is next to nothing and the same could be said about the cpu.
If you plan to do complex stuff this of course increases. Your question isn't really complete imo.

I don't really have any statistics on this, but I don't think it should be too memory intensive other than whatever amount of memory Netty might use.


Rebaseline failing performance tests

Because they're tiny and flaky.

@bot-gradle test and merge

OK, I've already triggered a build for you.


it's the current year

https://github.com/honestica/angular-hot-reloader#its-2016-how-about-angular2-
wave goodbye to 2016 my friends

Ok, right. It's 2017 now


Remove Styled Components peer dependency

It would be a relatively large overhaul to remove styled-components but I would do it if there is interest.. Thoughts anybody?

Done!


Change Java path

Hi,
I cannot install the new Java version and has downloaded the portable jdk 8; not sure how to point dbeaver to use the portable version rather than installed one.
thanks.
!SESSION 2018-08-29 22:36:03.063 -----------------------------------------------
eclipse.buildId=unknown
java.version=1.7.0_79
java.vendor=Oracle Corporation
BootLoader constants: OS=win32, ARCH=x86_64, WS=win32, NL=en_NZ
Command-line arguments:  -os win32 -ws win32 -arch x86_64

issue resolved:
https://superuser.com/questions/1219494/how-to-point-to-jre-home-in-dbeaver-ini


Added ColorCode Converter project files

Description
A color code converter that converts between RGB, Hex, CMYK & Hsl.

Conversion from any of the supported formats: RGB, Hex, CMYK, & Hsl.
User input is validated using Regex.

Fixes #288

Type of change

[x] New feature (non-breaking change which adds functionality)

Explain the Testing instructions
Test Configuration:

Operating system: Linux, Windows
Version: Ubuntu, 10
Text-editors used: Sublime

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

changes req

implement it naive language colors like i can give input as red,green ,BLUE,YeLLow
hex conversion is not working properly chak it again

@arpit456jain can you give me the test case you tested the Hex part? I tried multiple combinations and it worked perfectly from my side.

@arpit456jain Added colors name support.

@arpit456jain Any updates about the review?

@arpit456jain I cannot find the project on the deployed link. Can you help?

@VidhiBhatt01
https://deploy-preview-318--amazing-js-projects.netlify.app/color_code_converter/

its good now


Unable to build netop from /pkg/network/netop.go

Describe what happened:
I am trying a test to get  network communication stats using datadog-agent using netop.go file but it is cautiously complaining
# github.com/DataDog/datadog-agent/pkg/ebpf/compiler
In file included from compiler.cpp:1:0:
compiler.h:5:38: fatal error: clang/Driver/Compilation.h: No such file or directory
compilation terminated.

clang --version
Ubuntu clang version 11.1.0-++20210204122740+1fdec59bffc1-1~exp1~20210203233413.161
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/bin

cmake --version
cmake version 3.17.3
CMake suite maintained and supported by Kitware (kitware.com/cmake).

When I am trying to run without the tag I get following error
BPF-based system probe not implemented on non-linux systems

I am running it on ubuntu 4.15.0-1080-gcp kernel
Describe what you expected:
should be able to execute and see traffic summary
Steps to reproduce the issue:
trying to run it using go run -tags linux_bpf netop.go
Additional environment details (Operating System, Cloud provider, etc):

Hi @chiragtayal! Apologies for the delay in responding. This error is caused because it cannot find our custom build of clang with only the eBPF backend. This is currently built in our CI.
If you follow those same instructions and extract the tarball to /opt/datadog-agent/embedded, you should be able to resolve this compilation error.


少了一个括号"}"

@keob commented on Sat Oct 26 2019
代码的402行

@srvbpigh commented on Sat Oct 26 2019
Hello, @keob
Thank you for your feedback.
We are actively reviewing your comments and will get back to you soon.
Kind regards,
Microsoft DOCS International Team

@keob commented on Sat Oct 26 2019
Can you tell me how to modify the csharo code in markdown?

@imba-tjd commented on Sat Oct 26 2019

Can you tell me how to modify the csharo code in markdown?

https://github.com/aspnet/AspNetCore.Docs.zh-cn/blob/live/CONTRIBUTING.md#文件夹结构约定
https://github.com/aspnet/AspNetCore.Docs/blob/live/aspnetcore/tutorials/first-web-api/samples/3.0/TodoApi/Controllers/TodoItemsController.cs#L12

@keob commented on Sat Oct 26 2019
@imba-tjd thank you

Hi team,
I moved this here because it refers to the code
best regards

@CeciAc thanks.
@Rick-Anderson I believe this is a doc issue. Should we just transfer it to the docs repo?


Insomnia reporting: oas3-valid-oas-content-example 'example' property should match exactly one schema in oneOf

Expected Behavior
No warning message
Actual Behavior
oas3-valid-oas-content-example 'example' property should match exactly one schema in oneOf warning appears
Reproduction Steps

use the following spec in Insomnia's document design tab

example OAS
{
  "openapi": "3.0.3",
  "info": {
    "title": "playground-api",
    "version": "0.1.0"
  },
  "paths": {
    "/todo": {
      "post": {
        "tags": [
          "todo"
        ],
        "summary": "Create new Todo to shared in-memory storage.",
        "description": "Create new Todo",
        "operationId": "create_todo",
        "requestBody": {
          "content": {
            "application/json": {
              "schema": {
                "$ref": "#/components/schemas/Todo"
              }
            }
          },
          "required": true
        },
        "responses": {
          "201": {
            "description": "Todo created successfully",
            "content": {
              "application/json": {
                "schema": {
                  "$ref": "#/components/schemas/Todo"
                }
              }
            }
          },
          "409": {
            "description": "Todo with id already exists",
            "content": {
              "application/json": {
                "schema": {
                  "$ref": "#/components/schemas/TodoErrorResponse"
                },
                "example": {
                  "Conflict": "id = 1"
                }
              }
            }
          }
        },
        "deprecated": false
      }
    }
  },
  "components": {
    "schemas": {
      "TodoErrorResponse": {
        "oneOf": [
          {
            "type": "object",
            "properties": {
              "NotFound": {
                "type": "string",
                "description": "When Todo is not found by search term."
              }
            }
          },
          {
            "type": "object",
            "properties": {
              "Conflict": {
                "type": "string",
                "description": "When there is a conflict storing a new todo."
              }
            }
          },
          {
            "type": "object",
            "properties": {
              "Unauhtorized": {
                "type": "string",
                "description": "When todo enpoint was called without correct credentials"
              }
            }
          }
        ],
        "description": "Todo endpoint error responses"
      },
      "Todo": {
        "type": "object",
        "required": [
          "id",
          "value",
          "checked"
        ],
        "properties": {
          "id": {
            "type": "integer",
            "format": "int32",
            "description": "Unique id for the todo item.",
            "example": 1
          },
          "checked": {
            "type": "boolean",
            "description": "Mark is the task done or not"
          },
          "value": {
            "type": "string",
            "description": "Description of the taks to do.",
            "example": "Remember to buy groceries"
          }
        },
        "description": "Task to do."
      }
    }
  },
  "tags": [
    {
      "name": "todo",
      "description": "Todo management endpoints."
    }
  ]
}

View warning related to line 43

Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
I am no expert at OpenAPI specs, however, from my limited understanding I see the example at line 43 is valid, if not please let me know what the correct formatting is.
Insomnia Version
2022.2.1
What operating system are you using?
Windows
Operating System Version
Windows 11 Pro 21H2
Installation method
Downloaded from insomnia.rest
Last Known Working Insomnia version
No response

We took a good look at this today on the Insomnia stream.  I'm stumped.  I will say, this is something we did notice is that:

the property is named examples (not example) from what I can tell
inside the example, you provide a few properties (see the spec) one of which is value.
you need to give the example a name (e.g. foo on the OAS docs lined in the above)
Insomnia does seem to be seeing the schema, but something's not right...

the best I could come up with was:
          "409": {
            "description": "Todo with id already exists",
            "content": {
              "application/json": {
                "schema": {
                  "$ref": "#/components/schemas/TodoErrorResponse"
                },
                "examples": {
                  "foo": {
                    "value": {
                      "NotFound": "id = 1"
                    }
                  }
                }
              }
            }
          }
          ```

          which appears to me to be what the OAS spec calls for.. Any thoughts?

The stream clip was actually fairly informative, just a couple things I could add
While originally searching this issue, I did notice that quite a few 'example' examples showed using the plural 'examples' in the way that you describe, I did find that 'example' where it is just the specific example as I have should still be valid (so says swagger)
As you mentioned in your stream this seems to be almost entirely just a linting error, and it doesn't really effect anything other then just having to see a warning.
I also believe that the issue is specifically related to using examples with a schema that utilizes oneOf (Though I could be wrong on this, and I haven't really tested if it effects any of the other enum style schema's,)

Seems like this might be a linting issue.  I'm going to close this issue for now.  We are rethinking how we do OpenAPI in general and we'll keep these linting issues in mind too.


Can I render/reset an environment based on a specific observation?

I am conducting an experiment where I test different starting points for the completion of the task and I want to visualize these starting points afterwards. Is there any way to visualize these points by using the observation?

Hi, sorry for the delay. The easiest way would be to render directly when the starting point are generated:
import gymnasium as gym
from PIL import Image

import panda_gym

env = gym.make("PandaPickAndPlace-v3", render_mode="rgb_array", renderer="OpenGL")

# Render the starting point
env.reset()
Image.fromarray(env.render()).save("starting_point.png")

# Interract
for _ in range(50):
    env.step(env.action_space.sample())

env.close()

If, for some reasons, you really want to do it afterward, you can use the save/restore state feature:
import gymnasium as gym
from PIL import Image

import panda_gym

env = gym.make("PandaPickAndPlace-v3", render_mode="rgb_array", renderer="OpenGL")

# Reset and save the initial state
env.reset()
state_id = env.save_state()

# Interract
for _ in range(50):
    env.step(env.action_space.sample())

# Restore initial state and render it
env.restore_state(state_id)
Image.fromarray(env.render()).save("starting_point.png")

env.close()


New package: NirSoft.CSVFileView version 2.64

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[x] Resolve #172277

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Manual validated in Windows Sandbox.

Verify:


Make frozendict picklable

Replaces frozendict implementation by existing stdlib thing which behaves more or less the same, but coops with pickle
I've tested like d = frozendict({'a': 4}) and d['a'] == 4 and d['a'] = 5 # raises and del d['a'] # raises and d['b'] = 7 # raises. Are there any other cases of interest?
(ftr I put no changelog here, this should have no outwardly visible effect)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

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

Project coverage is 75.00%. Comparing base (91b104a) to head (0a1c262).
Report is 3 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop      #69   +/-   ##
========================================
  Coverage    75.00%   75.00%
========================================
  Files            1        1
  Lines            4        4
========================================
  Hits             3        3
  Misses           1        1

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


DOC: SQLRecordManager documentation does not exist

Issue with current documentation:
Please write docs for SQLRecordManager
Idea or request for content:

Supported databases
List of methods
Description of method parameters

@jamescadd Hi, has the bot solved your issue? I found some usage examples in the index function in this documentation. This explains quite well of the SQL record manager. Also, most SQL Alchemy supported databases works, as in this link. If you wish to have a documentation created, could you guide me on where to add the documentation? I would be willing to do the documentation if needed!!

@PennlaineChu Thank you, I would love to see the documentation created!  Are you asking where in the langchain docs it belongs? You could use https://python.langchain.com/docs/modules/data_connection/sqlrecordmanager if it matches your taxonomy.  There are several points that documentation could help clarify, here are some examples - though this is by no means exhaustive:

Which databases are supported?  You say most SQL Alchemy supported databases work, but dosu-bot above in this issue says only SQL Lite is supported.  Which is correct?
What's the purpose of the namespace?  Can there be multiple namespaces in the same SQLRecordManager?  Please provide an example, such as use with multiple vector stores that each have different sets of data.
Can I retrieve the Documents from the SQLRecordManager?  If so how?
What are the indexes referenced in the above functions?
Can I create my own record manager?  Can you provide an example of another record manager?

@PennlaineChu hi, any updates on the docs you can share?

@PennlaineChu hey, checking in there's any update.

Hello -
I'd also be interested in the evolution of SQL Record Manager. I've been successfully using it with postgres, and I'm interested in bringing it to another database. (Spanner)
As I do that, I'm noting some oddities in the schema. The uuid column does not seem needed, and the indexes seem to be not thought through (in terms of what's needed). Spanner, for example, is easiest if we simply use a primary key of (key, namespace) as a way to have a uniqueness constraint. I'll need to investigate sqlite/postgres.
Thanks

Hello, any news about this ? I'd love to use this amazing feature :)
By the way, the "SQLRecordManager" and "index" API link (as seen above) of this documentation page is a 404 error.

I would also like the documentation of SQLRecordManager


add whitelist option for hosts/ports that shouldn't be MITM'd

maybe with automatic addition for failed handshakes (like in Burp)

also, this requires an additional "pass-through" implementation for such sessions


[video_player]add integration test for inverted aspect ratio bug on iOS

The original bug here: https://github.com/flutter/flutter/issues/109116
We need to add integration test to verify the bug fix. However, right now we are having a problem creating the video that can reproduce the bug (more info here).

I deprioritizing it since we are waiting for permission to use the video stream in our integration test.


Gradle update and fixes

The changes I made related to properties weren't caused by the update - not sure why it was only me getting compile errors on the build scripts.
eg

Make sure you're on the latest IntelliJ version. I believe that was an issue from IntelliJ their side

agh

wahoo.


Loading index.html doesn't work in the samples

Hello,
This is probably a windows problem with how windows resolves relative paths. this line
window.loadURL("file://" + Node.__dirname + "/../index.html");

doesn't work as I get an this error (I tried 3 samples helloworld, react-todomvc and material-ui, all three seem to have the this problem so I assume the rest have the same problem as well)

Workaround
Use url and path libraries from node as per electron-quick-start like this:
import the libraries
let url = importAll<obj> "url"
let path = importAll<obj> "path"

then replace window.loadURL("file://" + Node.__dirname + "/../index.html") with
    let formatOptions =
        createObj [
            "pathname" ==>  path?join(__SOURCE_DIRECTORY__ , "../app/index.html")
            "protocol" ==> "file:"
            "shashes" ==> true
        ]

    // Load the index.html of the app.
    window.loadURL(unbox<string> <| url?format(formatOptions));

Then it works (at least on my windows machine)

Hi @Zaid-Ajaj! Thanks for the report. Strange, in my machine (Windows 10) it's working even with the back slashes. I've updated the helloworld sample. Could you please give it a try? Before compiling, please pull latest changes from this repo and install latest Fable version with npm i -g fable-compiler. Among other changes I update that line to:
    window.loadURL("file://" + Node.path.join(Node.__dirname.Replace("\\", "/"), "/../index.html"));

If it works for you, I'll update the other samples as well.

Hello @alfonsogarciacaro, I was not able to compile properly
I have<EMAIL_ADDRESS>globally installed (that's what I see when I npm list -g --depth=0).
In fableconfig.json, the projFile: ["src/main.fsx", "src/renderer.fsx"] gives me a warning the a string was expected. (I though multiple project files were supported in the latest version)
When I run fable I get this:

The strange thing is that I get the log fable-compiler 0.6.15: Start Compilation... this is neither my globally installed fable-compiler nor the one defined in package.json
The other stange thing is that webpack could not resolve the compiled main.js and renderer.js (I tried appending .js to the files or removing the first ./ from the path but these didn't work either)
Again maybe it's just my windows that is a bit drunk. I will re-try this on my linux soon and see what happens.

I shouldn't trust npm global installations, they always give problems :grin:
I've changed again the helloworld sample so fable-compiler is installed locally. Sorry for the inconvenience, but can you please pull the latest changes and try the following?
npm install
npm run build
npm start

In fableconfig.json, the projFile: ["src/main.fsx", "src/renderer.fsx"] gives me a warning the a string was expected. (I though multiple project files were supported in the latest version)

This is because the fableconfig schema hasn't been updated yet in Ionide. We need to fix that.

@alfonsogarciacaro Compilation was succesful with the correct fable-compiler version (0.7.35), webpack worked as expected without changes. However, I still got the same problem of not being able to load the index.html file (same error as reported above). Using the workaround got it working so this line
 window.loadURL("file://" + Node.path.join(Node.__dirname.Replace("\\", "/"), "/../index.html"));

did not work for me

Hmm, strange. Ok, I've modified the code again following your indications. Can you give it another try please? Thanks a lot for your time!

@alfonsogarciacaro Confirmed, works like a charm without any changes :)


Release build in linux

let me know how is build jar file and run this one in linux from source code

We use gradle for the project build so running ./gradle bootRun allows you to run it from source. Creating the jar file would be ./gradle jar


Add a class to the body/html which represents each step

I need to know in my CSS which step i'm at, to hide certain elements and show others on each step. right now shepherd doesn't seem to do that. it only adds the shepherd-active.

Don't think it has to be built in as it should be fairly simple to achieve using some of the provided methods.

My guess to why the current step is not added as a class on the body or html element is that more than one tour could potentially run concurrently. In that situation, the current step is ambiguous. Like @woble, it should be possible to do this with the current API. One way could be to add classes on the elements you want to hide/show and hook into the tour events to show/hide the elements accordingly. Let me know if that still doesn't work for you and I'll be happy to help out.


Use local shuffle reader in containerized environments and some impro…

…vements.
Which issue does this PR close?

Closes #356.
Rationale for this change

support containerized environments
Modify Error type for scheduler retry task
Make local shuffle files read serially for better memory control

What changes are included in this PR?

Are there any user-facing changes?

@andygrove  @yahoNanJing  @mingmwang  Test with tpch-1g query1 in local mac, no error no performance downgrade.


Cycle comes from different places

The cycle comes from different places:

For the viewer it comes from the database. For the time slider in the main window it comes from the filename. Why do we do it this way? This leads to issues like #20080, but also can lead to issues whenever the filename cycle doesn't match the database cycle.
If you add print statements to the Silo Database everywhere that cycle comes up you will see that the plugin is being asked for the cycle in different ways.

The stuff related to handling of cycles and times from databases is massively complex and involves optimizations for whether a file is actually to be opened by a plugin or just the string of the filename itself is processed. So, the GUI does as little work as possible to get cycle information and will therefore always guess from a filename. The value displayed in the viewer is always better in that it is derived most frequently from data actually stored in the file. Why? Because by the time something is being displayed in the viewer, the file has to have already been opened.
Next, we tend to gather cycles (and times when/if available) for all states when we open any state. That means opening and getting data from each and every file in a time series (expensive) or opening only the current state (wherever the user decided to open) and guessing from a filename for everything else.

I see, that makes sense. Thanks for the detailed answer. Do we want to change anything or should we just close this?

Do we want to change anything or should we just close this?

I think the logic for all this could be substantially simplified and if that is what this ticket is about, I think it's worth keeping open. There may be other issues related to this too. The complexity I mentioned is partly my own doing and partly other developer's doing. But, I also think it is unnecessary.

Ok. Let's leave it open then and have the action item be to reduce complexity for determining the cycle.


Input parameters do not support lambda expressions

Hi there,
our team is having lots of unnecessary lambda expressions pre and post workers execution.
Most of which are checking for null values or exist checks increasing the size of the workflows drastically.
Our first goal, was to do something like this:
"inputParameters":{
        "pn": "${ ${w1.output.nb} == null ?  null : \"w1.output.nb\"},
        "http_request": {
                 "uri": "http://someEndPoint/user",
                 "method": "POST",
                 "accept": "application/json",
                 "body": "{ \"username\":  \"w1.output.username\" , \"pn\": $.pn}
        }
 },

Q:

Is there a way to do what I've attempted to do with pn without using lambda workers?
If what I am attempting to do is not supported is there a plan of adding feature similar to what I've described in the next release?
If this is something that you guys are interested in but don't find super essential. Would you like our team to take it on?

Thank you!

Is there a way to do what I've attempted to do with pn without using lambda workers?

Currently, there isn't a way to achieve this without using lambda tasks.

If what I am attempting to do is not supported is there a plan of adding feature similar to what I've described in the next release?

This is not supported currently, but we feel that this would be a great addition to the project. Unfortunately, we will not be able to work on adding this feature.

If this is something that you guys are interested in but don't find super essential. Would you like our team to take it on?

We would greatly appreciate it if your team can contribute this feature. I am sure this would benefit other users of Conductor as well.

Hi @apanicker-nflx,
that's awesome, our team is probably able to handle this!
We will add it to our backlog whenever we get more info from you.
Here's a list of questions the team had in mind for you.
Q:
1- What is the character sequence of choice to signal embedded JS code.
We were thinking ${ JS_code }
2-  Our proposed approach would be to add a regex matcher that check at every task in the workflow
for un-escaped ${ JS_CODE }  after variables declaration for that worker and before the worker
call kicks off.
a) What is your input on this approach?
if you agree with it, where in the current repo would this change needs to reside.
Otherwise, what other approach would you suggest?
Any other guidance you think off will really help us achieve this.

That is awesome, thanks for your contribution.

1- What is the character sequence of choice to signal embedded JS code.
We were thinking ${ JS_code }
2- Our proposed approach would be to add a regex matcher that check at every task in the workflow
for un-escaped ${ JS_CODE } after variables declaration for that worker and before the worker
call kicks off.
a) What is your input on this approach?

${ } is a good choice to signal embedded JS. Please note that this character sequence is currently being used to identify JSONPath expressions. Reference. So this can be extended to support JS code as well while keeping the workflow definition syntax the same.

if you agree with it, where in the current repo would this change needs to reside.

The class that evaluates parameters is ParametersUtils. This would be a good reference point for where you could look into adding your JS evaluation.


How to use raw ServiceAccount bearer token to authenticate out-of-cluster ?

There are two ways to authenticate:

In-cluster
out-of-cluster

But I want to be able to use a raw ServiceAccount token I have without using the "in-cluster" method to create a container.
The out-of-cluster method require me to specify a kubeconfig file.
Like kubectl have the option of --token to specify a raw token.
Is it possible ?

when constructing a client, you can build a rest.Config struct manually, specifying the bearer token there: https://github.com/kubernetes/client-go/blob/master/rest/config.go#L72
/close


could not support file operation in python

I use the code " with open (...) as file: "
codon could not support and report an error : " error: syntax error ,unexpected 'as'  "

@fzwwj95
Please test this code and let me know what errors you encounter:
with open('codon.test', 'w') as w:
    w.write('test')

with open('codon.test') as r:
    print(r.read())

I tested both pieces of code separately, and both gave the same error : " error: syntax error ,unexpected 'as' "

@fzwwj95 Did you test it in Python?

@fzwwj95 Duplicate of #317


What to set to increase number of rows in the grid?

Let say I want to show 100k of rows per page. What do I need to set in Laravel config file or php.ini etc? Or to change the server spec?

How if you try this:
        $grid->paginate(100);
        $grid->perPages([100, 200, 300]);

I try but it takes forever to load so I wonder what setting should I set in Laravel or the server (ubuntu nginx) or database to make it possible to show lots of data.
$grid->perPages([5000, 10000, 50000]);


Can we use Tuesday for Html Presentations? like RevealJS

Please let us do so.. It would just king everything.

Yes! You can use Tuesday JS to create presentations


Support For Xiaomi Robot Vacuum S20 EU -  xiaomi.vacuum.d106gl

Hi,
Is it possible to add support for the Xiaomi Robot Vacuum S20 EU (xiaomi.vacuum.d106gl)? I can provide additional information about the device. I was able to add the string {xiaomi.vacuum.d106gl} to config_flow.py, but I cannot retrieve any other data from the vacuum.

I have also enabled debug logging.
home-assistant_dreame_vacuum_2024-08-15T15-33-53.780Z.log
Thanks for your support and hard work.

I see, its now working using official HA Xiaomi integration. Thanks for your time. Have a nice day.

@Rakura-cloud did you get it to work? I could´t find it with the Xiaomi integration.

@powas
Hi,
I have a issues with that integration, but I used the Xiaomi Miot Auto integration in HACS from this link. It doesn't have all the features, but at least I can turn on the vacuum, stop it, and return it to the dock. The picture below shows the available entries. I hope my advice will be useful to you.

@Rakura-cloud Awesome, thank you!!!


Anomaly Detection and Recommendation

As I understand, PcaAnomaly detector is available in ML.Net 0.4. Can you please suggest when the Recommendation is planned to be a part of ML.Net? Thanks.

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

ID: bd15175c-5b7d-4310-ecf5-32ffc7bc588b
Version Independent ID: 38adc676-934e-0ff9-c868-527f075c96df
Content: Machine learning tasks
Content Source: docs/machine-learning/resources/tasks.md
Product: dotnet-ml
GitHub Login: @aditidugar
Microsoft Alias: johalex

@astmehalthakor, You can do recommendations using FieldAwareFactorizationMachineBinaryClassifier. There should be a new movie recommendation sample using this at https://github.com/dotnet/machinelearning-samples soon. You can see it in action here - starting at about 21:59.

Closing this as I believe its resolved. Please reopen if that is not the case. Thanks!


Related issues in the result output section

Hello,
I hope this message finds you well. I wanted to start by expressing my admiration for your outstanding work and thanking you for open-sourcing your project. While using your code, I encountered a question that I hope you could kindly assist me with.
In your paper's "Abstract" section, you mentioned: "More challengingly, how can we do this in a way that captures complex inter-sensor relationships, and detects and explains anomalies which deviate from these relationships?" After going through some parts of your project's source code, I'm curious about how I could adjust the result output section of the code to obtain multiple sets of sensor IDs that might exhibit abnormal relationships.
I'm looking forward to your response. Thank you very much for your time and consideration.

Hi, if you want to use custom data, you could adjust your input data format similar to the demo data in https://github.com/d-ailin/GDN/tree/main/data/msl. The sensor IDs might require you extract and process from the intermediate results, such as https://github.com/d-ailin/GDN/blob/9853899da860682669a134e4af315d036aab4eca/main.py#L158.

Hi, if you want to use custom data, you could adjust your input data format similar to the demo data in https://github.com/d-ailin/GDN/tree/main/data/msl. The sensor IDs might require you extract and process from the intermediate results, such as
https://github.com/d-ailin/GDN/blob/9853899da860682669a134e4af315d036aab4eca/main.py#L158
.

Thank you very much for your answer. I will try it first. Thank you again.


Could not find the fixed version ?

Please the latest version with fixed dependencies for the dll.
I get same error with the available files.

Hi,
Is yours a 64 bit?
Did you try  installing Microsoft Visual C++ 2013 run-time for 64 bit?If not please get it installed and then try again.

Yes I have Visual studio 2015,  I build the project.  Build was successful.  However while using Java client to connect to MSMQ on different host.  I receive that error.

Is it working when both Java client and MSMQ are on same machine?
Please paste the error message as well.

Did not try with MSMQ in same machine.  However the error seems to be different.  Please find the error msg.
Hello from the Java MSMQ Test Client.
[C]reate [O]pen [R]eceive [S]end [P]eek [X]close [D]elete [I]nfo or [Q]uit?
==> O
Name of Queue to open? MSGMGR
Name of Queue Server (ENTER for local)? nvm2192pdv
open (DIRECT=OS:nvm2192pdv\private$\MSGMGR)
Exception in thread "main" java.lang.UnsatisfiedLinkError: C:\Ravindra\MsmqJava-master\MsmqJava-master\Release\MsmqJava.dll: Can't find dependent libraries
at java.lang.ClassLoader$NativeLibrary.load(Native Method)
at java.lang.ClassLoader.loadLibrary1(ClassLoader.java:1965)
at java.lang.ClassLoader.loadLibrary0(ClassLoader.java:1890)
at java.lang.ClassLoader.loadLibrary(ClassLoader.java:1880)
at java.lang.Runtime.loadLibrary0(Runtime.java:849)
at java.lang.System.loadLibrary(System.java:1088)
at ionic.Msmq.Queue.(Queue.java:452)
at ionic.MsmqJava.TestClient.Open(TestClient.java:172)
at ionic.MsmqJava.TestClient.Run(TestClient.java:268)
at ionic.MsmqJava.TestClient.main(TestClient.java:53)

@Ravi1971  that simply means that the msmq.dll file is not in your windows PATH .


Array syntax

Currently such oddities as [0 0 0 0] or [0'0'0'0] "work" in the sense that they get parsed.
Ditto, things like (let x0).
Should we allow them? Or just restrict it to " comma ? Or what?
In general: should we require whitespace? Or not?

Ah I see. I'm experimenting with the tokenizer and parser right now, so I'll see if I can come up with a syntax that is more strict in some cases.

Good to know - I shall stay away from that then for now. I'm currently working on tidying up core_functions.c and related.

Nice job on those core functions by the way! I just made an issue about the minus sign which is related to this one, if you want you can assign tickets to yourself so that we don't do overlapping work. I'll assign ones I plan on doing to myself.

Thank you, and I shall.

 If so, should the blank values be undef, or what?


SAK-47454 Group manager: Inactive users are shown

I have changed the participant selector so that only active members on the site are shown.

Well done, this was my fault !!


Nuitka 1.1.6 with OpenCV 4.6.0 Recursion issue

Nuitka version, full Python version, flavor, OS, etc. as output by this
command.

python -m nuitka --version

1.1.6
Commercial: None
Python: 3.9.12 (main, Apr  5 2022, 06:56:58)
Flavor: Anaconda Python
Executable: /home/austin/miniconda3/bin/python
OS: Linux
Arch: x86_64
Distribution: Ubuntu (based on Debian) 22.04.1

Issue has been mentioned in
#1366

How did you install Nuitka and Python

pip

The specific PyPI names and versions

opencv-python
import cv2
cv2.__version__
'4.6.0'

Many times when you get an error from Nuitka, your setup may be special

The full code:
import cv2 (Named opencv_tester.py)
Command: python3 -m nuitka --follow-imports --static-libpython=no opencv_tester.py
Also the command
Command: python3 -m nuitka --include-data-file=/home/austin/miniconda3/lib/python3.9/site-packages/cv2/config.py  --follow-imports --static-libpython=no opencv_tester.py

Provide in your issue the Nuitka options used

--follow-imports
--include-data-file
--static-libpython=no

Error during ./opencv_tester.py

['/home/austin/Documents/tmp/ws/src/opencv_test', '/home/austin/miniconda3/lib/python3.9/site-packages/cv2', '/home/austin/Documents/tmp/ws/install/velodyne_msgs/lib/python3.10/site-packages', '/home/austin/Documents/tmp/ws/install/swift_pgm2/lib/python3.10/site-packages', '/home/austin/Documents/tmp/ws/install/phidgets_msgs/lib/python3.10/site-packages', '/home/austin/Documents/tmp/ws/install/optical_capture/lib/python3.10/site-packages', '/home/austin/Documents/tmp/ws/install/interfaces/lib/python3.10/site-packages', '/home/austin/Documents/tmp/ws/install/geojson_msgs/lib/python3.10/site-packages', '/home/austin/Documents/tmp/ws/install/commander/lib/python3.10/site-packages', '/home/austin/Documents/tmp/ws/install/breadcrumbs_msgs/lib/python3.10/site-packages', '/home/austin/Documents/tmp/ws/install/api_interfaces/lib/python3.10/site-packages', '/opt/ros/humble/lib/python3.10/site-packages', '/opt/ros/humble/local/lib/python3.10/dist-packages', '/home/austin/miniconda3/lib/python39.zip', '/home/austin/miniconda3/lib/python3.9', '/home/austin/miniconda3/lib/python3.9/lib-dynload', '/home/austin/miniconda3/lib/python3.9/site-packages']
Traceback (most recent call last):
  File "/home/austin/Documents/tmp/ws/src/opencv_test/opencv_tester.py", line 2, in <module>
    import cv2 as cv
  File "/home/austin/miniconda3/lib/python3.9/site-packages/cv2/__init__.py", line 181, in <module cv2>
    bootstrap()
  File "/home/austin/miniconda3/lib/python3.9/site-packages/cv2/__init__.py", line 153, in bootstrap
    native_module = importlib.import_module("cv2")
  File "/home/austin/miniconda3/lib/python3.9/site-packages/cv2/__init__.py", line 181, in <module cv2>
    bootstrap()
  File "/home/austin/miniconda3/lib/python3.9/site-packages/cv2/__init__.py", line 76, in bootstrap
    raise ImportError('ERROR: recursion is detected during loading of "cv2" binary extensions. Check OpenCV installation.')
ImportError: ERROR: recursion is detected during loading of "cv2" binary extensions. Check OpenCV installation.

It may only work in standalone mode, mixes compilation is known to work badly. This also should be some sort of options hint in the yaml file, that some things will not work to be included in accelerated mode.

Thanks for the quick reply @kayhayen, which yaml file are you talking about?

In https://github.com/Nuitka/Nuitka/blob/develop/nuitka/plugins/standard/standard.nuitka-package.config.yml we have a bunch of configuration for missing DLLs, code changes, etc. and also warnings to be given for options that are missing. You not using standalone should be warned against for packages like cv2, which do require things of Nuitka that are currently not implemented (loading bytecode and source from disk into compiled packages).

Thanks @kayhayen, I may not be able to come back to this for a few weeks. Please feel free to close the issue in you'd like.

Closing for lack of feedback.


Update modules without any change to the code

I updated all possible node_modules to bring some of the fixes to madge. Most notable is the update from detective module which is now also able to process hasbanged javascript files (previously threw up internally and didn’t process rest of the file).

Sweet! Tagged v0.5.1 now and published to NPM.


Fix isolated

Type of change

Bugfix

Description

Checklist

[ ] Update/write design documentation in ./documentation/design
[ ] Write tests and make sure they pass
[ ] Update documentation
[ ] Check RBAC rights for Kubernetes / OpenShift roles
[ ] Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally
[ ] Reference relevant issue(s) and close them after merging
[ ] Update CHANGELOG.md

@enmasse-ci run tests profile=isolated test_only

:heavy_exclamation_mark: Build Failed :heavy_exclamation_mark:


Added English (UK)

Also fixed a small error in US

Coverage remained the same at 77.261% when pulling a60720d08d6f04ebe141163e1813e762f6b1e186 on GreenTurtwig:englishUK into 17ecd25f212703111622d1bbdf75f76ddb66cb15 on MarshallOfSound:master.

Coverage remained the same at 77.261% when pulling 1ef25ca00481407828ec0dcd23d3ebca839aa1ca on GreenTurtwig:englishUK into 17ecd25f212703111622d1bbdf75f76ddb66cb15 on MarshallOfSound:master.

In general I don't like this much duplicate, pretty much all the strings are the same as en-US which makes adding new strings just that much more annoying.
I would suggest instead modifying TranslationProvider to accept js files aswell and simply use Object.assign() to override the few object properties you want to change for UK

This PR is stale, closing.


Getting "Invalid file name" error

I am getting invalid file name error while importing the conf file. I renamed manually from wg0-xxxx-xx.conf to wg0xx.conf to resolve.
On Huawei P30 latest firmware and latest wireguard app.

The same problem Xiaomi phone. I guess, the problem is in length of file name. Idk why


Balance and History not matching

Issue Impact: Minor Issue
Device Manufacturer: Samsung
Device Model: Galaxi S20+ 5G
Divice OS: Android
Device OS Version: 11
Application Version: 3.1
What Happend?
What Did You Expect To Happen: History and Balance would be the same.
Can't reproduce it. Closing and Restarting
and it's still the same.
More info in the video
Cheers,
BoldorXL
https://user-images.githubusercontent.com/73369734/120811153-f7782f80-c54b-11eb-83c7-d4c9bf9c61cc.mp4

Hi Boldor,
I’ve just seen your video, it looks like it may be in your mempool still as
it’s flashing.
Elias
On Fri, 4 Jun 2021 at 4:50 PM, BoldorXL @.***> wrote:

Issue Impact: Minor Issue
Device Manufacturer: Samsung
Device Model: Galaxi S20+ 5G
Divice OS: Android
Device OS Version: 11
Application Version: 3.1
What Happend?
What Did You Expect To Happen: History and Balance would be the same.
Can't reproduce it. Closing and Restarting
and it's still the same.
More info in the video
Cheers,
BoldorXL
https://user-images.githubusercontent.com/73369734/120811153-f7782f80-c54b-11eb-83c7-d4c9bf9c61cc.mp4
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/minima-global/Minima/issues/241, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIBLNWO2DJJVBSS2HMSYT23TRDKZZANCNFSM46C3BYTA
.

I think it something els bc it's still not correct and flashing ever since

I hope the log file will be uploaded now

Can someone tell me how to attach a file to a reply. I can select a file but it not showing

check the mempool too please

coin stuck in mempool, should be solved in 98.24

Cleared the mempool was the solution to solve the problem using flushmempool -hard. At first I didn't see any changes. I had to close the window en op the wallet again to see all was fine. I wonder if it is possible to see the outcome of such actions right away without closing and Re-starting the wallet. Is this programmable?
After some more reading I've seen the help text also mentioned the solution. But I think that it would be more user friendly if there is some kind of trouble shooting menu that is easy excusable and user friendly.... just a thought. But of course it's testing time so if all goes well it's something of the past already 🙂

Once the coins hit the mempool the transaction is registered by History and not after.  If the transaction is invalid and doesn't go through so the balance doesn't change.  This is currently being worked on an is going to be fixed in the next hard fork of the apk.

@eliasnemr was this fixed?


Beautify README

The goal is to make a first time reader feel like a guest in our home. Make it really easy to know why we exist, what problem we solve, and give them enough of a great first impression that they can't wait to pip install sqlmesh

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: sungchun12:x: Sung Won ChungSung Won Chung seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Using reporter with Azure Devops CI

I am using this reporter for my CI in Azure Devops. I created a pipeline in which I  specify the test plan. I have about 61 test cases that run, Until I had like 50 test cases the reporter worked fine. All results appeared in the  test plan. After having more then 50 test cases it does not work anymore. It only populates about 14 test cases with the results and no more. It gives me the error : "azure:pw:warn No test points found for test case [123456] associated with test plan 654321 for configurations [5]
Check, maybe testPlanId or assigned configurations per test case, what you specified, is incorrect. "
But the test is in the test plan and it has the correct configuration. All tests are configured the same, but only 14 get populated. Any ideas?

What is the value of publishTestResultsMode in your config? testResult or testRun ?
If you choosed the latter, you should consider test with the former as a temporary fix ;)
Maybe the AzDO API have a limitation for the number of results it can handle

@andreipintilei
can you read this comment and step by step through API request figure out in which place you miss your results

using testResult fixes the issue :) thanks


Docs: Change to rule acceptance

This is a proposal for changing our procedure for accepting rules and rule changes. The general feeling amongst reviewers is that we have too many rules, and we're also spending a lot of time evaluating and adding new rules and rule changes.
This proposes that:

We only accept new rules if they are absolutely necessary for ESLint to be complete. Otherwise, we should suggest people create custom rules.
If a rule is so important that we accept it, then the rule must be implemented by the person who suggested it or a designee of that person. It will not be done by an ESLint team member.
Rule changes follow the same process: it must be important, and if we accept it, then the person who suggested it or a designee must implement it. It will not be done by an ESLint team member.

That basically means any new rule or rule change that is accepted must be labeled with "help wanted" immediately.
Please review @eslint/eslint-team

YWIYBI might be a bit more agressive term then I would've liked. But other than that, LGTM.

It sounds like the goal here is to suggest "By default, new rules belong in plugins, but sometimes, if you write something almost everyone will use, we'll make an exception and bring it into core." I agree with @ilyavolodin - it comes off as saying the default state for a rule being rejected. I know it's just semantics because users are always free to write a plugin, but I think the messaging could be more encouraging. (In the spirit of you want it you build it, if you'd like, I can take a stab at some edits to this at some point over the weekend.)

@btmills Here are 2 thoughts that came to mind if you can rewrite:

The difference in my follow-through if I want it or if I need it.
The 200 rules in ESLint remind me of the 100 markers in Multi-Dictionary Formatter. The feedback from the front lines was consistent: There are too many 😱 I need another one 😠

Thinking at a different level than is usual for my Pooh Bear brain, I wonder if any of us have contacts who could become early adopters of this new program:

companies which are already active contributors to open-source software that they use
JS meetup groups which already have hack activities
student groups or inclusiveness organizations

A flurry of (positive) tweets and a few blog articles about the experience could give it some news value beyond our “press release.” So ordinary devs might say, I could do that and want to be a part of that.
Like the momentum behind first-time contributions from tweets by Kent C. Dodds (and others) and Healthy Open Source by Mikeal Rogers.

Not exactly thrilled about this, since my contributions are mostly in the rules space, but I understand the motivation.

How about simply encouraging people to create a plugin, share and test it out in the wild and come back with data and experience and say: this looks like a good rule to have in core because many people use it now and it is not something-specific.
There can of course be exceptions but they are, well, exceptions.
Now I know this is not really different from what you are saying but the tone as others mentioned will affect the perception and friendliness of the project. Encouraging to write the rule as a plugin and testing out is a "nicer" thing than saying "We don't want this unless really necessary".

If we want to encourage the creation and usage of custom rules we should probably do something to increase their discoverability, maybe something like the grunt plugins listing?

I'm all for this change and will try to dedicate time to write a blog post
or a screencast about writing custom rules. This is long overdue IMO
because it's the easiest way to scale the project if we can help out with
discoverability.
On Sat, Mar 19, 2016 at 2:52 AM alberto<EMAIL_ADDRESS>wrote:

If we want to encourage the creation and usage of custom rules we should
probably do something to increase their discoverability, maybe something
like the grunt plugins listing http://gruntjs.com/plugins?
—
You are receiving this because you are on a team that was mentioned.
Reply to this email directly or view it on GitHub
https://github.com/eslint/eslint.github.io/pull/227#issuecomment-198675676

@ilyavolodin I'm open to other wording, I just want to make it clear that we aren't going to be a rule factory anymore.
@btmills @byk that's not quite what this is saying. I don't want a stream of "please consider my plugin rules for inclusion." The message here is if your rule is so important that we agree it should be included, then you need to build it. If a rule exists in a plugin, it should stay there.
@alberto you can still work on rules or enhancements if you want, I just want to shift the discussion so people know that they are expected to implement their own suggestions. If something is accepted and you want to work on it, you still can if the submitter can't fit some reason (you can also submit your own).
Discoverability is something we need to work on. It's a pretty difficult problem, as we don't even have good discoverability for core rules today. I'd say we have to solve it for core rules before we can have a chance at solving it for plugin rules.
So to be clear: the goal here is to not accept rules into the core and encourage people to write plugins instead. The case where we accept a new core rule should be extremely rare. According an enhancement to a rule will be more common, but we still want to encourage submitters to make the change themselves.
We will still address rule bugs that submitters don't fix, and we can still implement truly important rule enhancements.

I'm not sure if this is gonna yield the results we are hoping that it would but I also don't think it is bad/dangerous to try it so I'm :+1:.

:+1: No problem as I still can write rules if I want to get it.

Okay, I've updated the text to try to be a bit nicer. :) Please take a moment to review.

Looks good to me.

LGTM

This looks good to me :+1:

I like the friendlier tone :)

This latest version looks awesome! :tada:

I've updated the text based the last bit of feedback that @alberto have. I'm merging now, as I don't think we should hold this up for other minor issues. We can deal with confusion or other changes as necessary and as they come up.


Updating CSRF token

Dropbox changed the CSRF login token.

Thanks @josemalonsom and @mariosk for help with this Dropbox change. I've combined your suggestions in the hope this makes the token extraction more stable now. See jakajancar/DropboxUploader#52 See jakajancar/DropboxUploader#53


Add 'record_bytes' field that contains raw data of each input record

Background
Original record bytes used for decoding its fields is useful for validation and audit, as well as writing filtered data back to EBCDIC format.
Feature
Add the ability to generate 'reaord_bytes' field that contains raw bytes of the decoded record.
Example
.option("generate_record_bytes", "true")

We are migrating an ETL process from DataStage to scala using your libraries. The file has fields that are handled as "Raw" and converted with the iconv function to hexadecimal. But we are losing bits and we cannot solve it. They are fields that have packaged bits inside.

We read them as follows
.withColumn("field", hex(col("field")))

Is this feature request relevant to the topic? https://github.com/AbsaOSS/cobrix/issues/624
Please, create a separate issue for the issue you are having and please provide more context.
What do you mean by losing bits. Could you please provide an example?

Sorry, We create the issue #627.


WebIDL integration tests

Right now, WebIDL tests are just asserting that the output of compiling some webidl exactly matches what we expect.
We should have integration tests and infrastructure where to write a test you supply:

a WebIDL source string
a JS polyfill for the things described in that WebIDL
a src/lib.rs that exports a #[wasm_bindgen] pub fn run() function that uses the things described in that WebIDL (via mod bindings;)

Then, the testing infrastructure will:

use wasm_bindgen_webidl in a build.rs to emit wasm-bindgen imports into a file in the build's out directory
add a src/bindings.rs module that does include!(...) of the file created in build.rs
add a test.ts that calls the user-defined run function

It shouldn't be too hard to extend the existing project()-based test infrastructure used in tests/all/* (and defined in tests/all/main.rs) to do this stuff.

Implemented in #264. It should support auto-generating test.ts files that will call #[wasm_bindgen] pub fn test(), however given that constructors aren't implemented at the moment, the basic example doesn't currently use it (although this example does demonstrate that you can still use a custom test.ts for webidl if needed for a test).

Fixed in #264.


Could not parse n transform function (File included)

Download is failing with the following error:
WARNING: Could not parse n transform function.
Please report this issue with the "1734021378158-base.js" file on https://github.com/distubejs/ytdl-core/issues.

I think the YT file changed an we need to update sig.js. I don't have enough knowledge to do it but I'll try.
In the mean time, I'm creating an issue with the file.
Debug File
1734021378158-base.js.zip
Environment

@distube/ytdl-core version: 4.15.1
Node.js version: 20.2.0
Operating system: Mac OS 15.1.1

I'm experiencing the same issue as well, but with the distube.js Discord bot. Weird thing is, I'm able to get the music playing without issue, even with all the filters as well
This is my package.json
   ...
    "@distube/deezer": "^2.0.1",
    "@distube/soundcloud": "^2.0.4",
    "@distube/spotify": "^2.0.2",
    "@distube/youtube": "^1.0.4",
    "distube": "^5.0.4",

@chemisax @earnestangel any luck with that?
i am getting the same issue.

Have you found a fix yet?

I have faced the same issue.
WARNING: Could not parse n transform function.
Please report this issue with the "1734172429595-base.js" file on https://github.com/distubejs/ytdl-core/issues.
But I can download the files.

do npm uninstall @distube/ytdl-core
after it finishes do npm add https://github.com/ToddyTheNoobDud/ytdl-core-toddys.git


Use jsunitsaucelabs to run our unit tests on Saucelabs

Things to do :

[ ] Make sure we return proper error levels
[x] Change sauce_browsers.yml to JSON file
[x] Use this JSON with jsunitsaucelabs
[x] Remove grunt and other grunt dependencies

Work in progress...
Fixes #23116
/CC @XhmikosR

I to check my last changes tomorrow because I did to many attempt on Sauce Labs 😄

@XhmikosR and @bardiharborow everything is done, I let you the final words and to review what we've done here :+1: