Staking actions inconsistent with the on-chain states on Kusama

We launched a Kusama stakedrop. We read Staking::eraStakers to determine the staked amounts for nominators and validators. However we found an inconsistent case.
Nominator (controller & stash): CgagebHPFckribAKitWgMPEhZCc72nPNCZWUaSPrYcCyVSy
This account has nominated 4 validator candidates in his last transaction:
https://polkascan.io/kusama/transaction/0xf16bbe0bfaea3b6f7370b0e2ecdd55d4eaab06e2d22bea0dbad04e64906ec408
However, if I check eraStakers in the chain state with the four candidates, none of the query result shows the nominator. I did that by simply checking "chain state" tab in polkadot.js apps. It looks like indeed this account hasn't received any reward from nomination. Why is the historical extrinsic not consistent with the on-chain storage? Does it mean the nominator is currently not nominating any candidate?

Quotes from Jam: I haven't actually checked, other than looking at who exactly they are nominating, but:
they only nominated 3 extremely popular validators, and 1 waiting
there's a limit of 64 nominations per validator
I'm guessing the popular validators were capped out, so their nomination did not go into effect with them, the waiting validator did not successfully get into the set

I also checked the nominated validators. All of them have 8 or fewer nominators. So I don't think the max nomination is the cause of the problem.

It turned out this account had already unbounded the token before nominating 4 validators. So the active bound is zero. It explained why it doesn't have any reward.


Git not remembered in workspace

When I re-enter my workspace
docker-compose exec workspace bash
I have to set git config each time. Is there a way I can have this be remembered?

I use VSCode with the new remote development features. When opening a project it copies my hosts .gitconfig into the workspace container.


Negative values in Table BarChart renderer.

Table Barchart renderer works incorrect with negative values.

Indeed, this renderer only makes sense in the context of positive values....

I have some ideas how to improver renderer to work with negative values too.
I'll prepare pull request.

OK, great, looking forward to it!

I added pull-request. Could you review it?

Thanks for the PR! I'll take a look when I have some time, yes :)

+1

Fixed with v2.15.0!


Compute worker - docker pull image failure logs

@ mention of reviewers
@Didayolo
A brief description of the purpose of the changes contained in this PR.
Now if docker image pull fails, users can see logs live and in ingestion std_err
Live logs:

Submission logs:

Issues this PR resolves

#902 -> Show logs when compute worker crashes because of invalid docker image

Checklist

[x] Code review by me
[x] Hand tested by me
[x] I'm proud of my work
[ ] Code review by reviewer
[ ] Hand tested by reviewer
[ ] CircleCi tests are passing
[ ] Ready to merge

Handled in #1161


Can't connect to server but no error is thrown

Hi,
I'm running NodeJS 8.11, MySQL 5.5 and mysqljs 2.15.
I'm doing connections to an old machine running MySQL 5.5 on a Windows XP. When the machine is frozen (needs to press the power button to restart it but it responds to ping), when I'm trying to connect to it the connection gets stuck. I can't even see any debug traces.
When I try to connect to that machine using MySQL Workbench I get the following error:
Your connection attempt failed for user 'monitoring' from your host to server at E0081T102:3306:
  Lost connection to MySQL server at 'waiting for initial communication packet', system error: 60

I've seen that when the CPU of the host is at 100% it can lead to some problems ( #1616 ), but that's not my case. I'll be happy if I'd get a timeout and could continue.
The code I'm using to test this is:
const mysql = require('mysql');

var connection = mysql.createConnection({
    host     : 'host',
    user     : 'user',
    password : 'password',
    database : 'database',
    acquireTimeout: 2000,
    connectTimeout: 2000,
    timeout: 2000,
    debug:true
  });

  connection.connect(function(error){
      console.log('error: ' + error);
  });

  connection.query('SELECT 1 + 1 AS solution', function (error, results, fields) {
    if (error) throw error;
    console.log('The solution is: ', results[0].solution);
  });

  connection.end();

Yea, it should definitely timeout, it sounds like. Is there a good way I can reproduce the issue to take a look?

Actually I just noticed that you're passing in a bunch of pool options into createConnection instead of createPool which tricked me :) if you're not using the pool, then as long as it can establish a TCP connection to the server (which is what connectTimeout sets), then since you're manually calling .connect you need to pass that a timeout. Can you try changing
connection.connect(function(error){

To the following and see if that times out for your condition?
connection.connect({timeout: 2000}, function(error){

Thanks, I'll make the change if I get the error again. The problem is that we're not able to reproduce this locally and the machines on production get rebooted. I'll keep you updated. Thanks!

use pool.query() instead of pool.getConnection(). more info see#1668


About the dropout

Hi fchollet,
I want to ask whether the dropout layer perform the dropout on batch-wise or sample-wise? In the keras.io, it says dropout is conducted in each update, so presumably samples in one minibatch share the same dropout mask rather than each sample has its own dropout mask?
Cheers

The dropout mask is constructed on each update, but it is independent across samples. The relevant line is here:
https://github.com/fchollet/keras/blob/master/keras/layers/core.py#L154

Yes, for all intents and purposes you can consider that Dropout is per-sample. The Dropout mask is a tensor that is random over every dimension.


fix: double addr in resource population

Proposed Changes

Resource population would erroneously try to put the same address in the account array twice if that address was used for an assetHolding. This would result in a false error regarding not having enough space to populate resources

FYI there seems to be degredation in GitHub PRs right now. The branch is up to date but this PR is not so its missing the docs (which I squashed on the branch as seen here https://github.com/algorandfoundation/algokit-utils-ts/compare/main...fix/resource_pop_double_addr)

Weird, I tried closing and reopening and it won't let me reopen


opt Make auto_profile_threshold_ms works on BE

This is needed for task list https://github.com/apache/doris/issues/33744
We will discard profile of query that costs less than auto_profile_threshold_ms(50ms default).
Backend will not report profile if query runs less than auto_profile_threshold_ms.

run buildall


Sinking Superset to Datahub; 'NoneType' object has no attribute and SSLError caused

Describe the bug
When sinking superset information to datahub (0.10.5 and 0.11.0 both), below error is occurred and some data cannot emitted to datahub.
The data not emitted is changed every time I tried.
And after sink, dataset is not shown in Chart Lineage although it is already in datahub.
Below is error while Sinking
ERROR:root:'NoneType' object has no attribute 'startswith'
Traceback (most recent call last):
  File "/mnt/c/Users/<username>/Desktop/scripts/datahub_test.py", line 46, in <module>
    datahub_ingest_yaml(datahub_superset_ingest_yaml)
  File "/mnt/c/Users/<username>/Desktop/scripts/datahub_test.py", line 15, in datahub_ingest_yaml
    pipeline.run()
  File "/home/red/.local/lib/python3.10/site-packages/datahub/ingestion/run/pipeline.py", line 367, in run
    for wu in itertools.islice(
  File "/home/red/.local/lib/python3.10/site-packages/datahub/ingestion/api/source_helpers.py", line 119, in auto_stale_entity_removal
    for wu in stream:
  File "/home/red/.local/lib/python3.10/site-packages/datahub/ingestion/api/source_helpers.py", line 143, in auto_workunit_reporter
    for wu in stream:
  File "/home/red/.local/lib/python3.10/site-packages/datahub/ingestion/api/source_helpers.py", line 208, in auto_browse_path_v2
    for urn, batch in _batch_workunits_by_urn(stream):
  File "/home/red/.local/lib/python3.10/site-packages/datahub/ingestion/api/source_helpers.py", line 346, in _batch_workunits_by_urn
    for wu in stream:
  File "/home/red/.local/lib/python3.10/site-packages/datahub/ingestion/api/source_helpers.py", line 156, in auto_materialize_referenced_tags
    for wu in stream:
  File "/home/red/.local/lib/python3.10/site-packages/datahub/ingestion/api/source_helpers.py", line 70, in auto_status_aspect
    for wu in stream:
  File "/home/red/.local/lib/python3.10/site-packages/datahub/ingestion/source/superset.py", line 403, in get_workunits_internal
    yield from self.emit_chart_mces()
  File "/home/red/.local/lib/python3.10/site-packages/datahub/ingestion/source/superset.py", line 396, in emit_chart_mces
    chart_snapshot = self.construct_chart_from_chart_data(chart_data)
  File "/home/red/.local/lib/python3.10/site-packages/datahub/ingestion/source/superset.py", line 326, in construct_chart_from_chart_data
    datasource_urn = self.get_datasource_urn_from_id(datasource_id)
  File "/home/red/.local/lib/python3.10/site-packages/datahub/ingestion/source/superset.py", line 219, in get_datasource_urn_from_id
    platform = self.get_platform_from_database_id(database_id)
  File "/home/red/.local/lib/python3.10/site-packages/datahub/ingestion/source/superset.py", line 203, in get_platform_from_database_id
    return sql_common.get_platform_from_sqlalchemy_uri(sqlalchemy_uri)
  File "/home/red/.local/lib/python3.10/site-packages/datahub/ingestion/source/sql/sql_common.py", line 147, in get_platform_from_sqlalchemy_uri
    if tester(sqlalchemy_uri):
  File "/home/red/.local/lib/python3.10/site-packages/datahub/ingestion/source/sql/sql_common.py", line 109, in <lambda>
    return platform, lambda x: x.startswith(
AttributeError: 'NoneType' object has no attribute 'startswith'

DEBUG:datahub.ingestion.run.pipeline: sink wrote workunit urn:li:dashboard:(superset,28)
DEBUG:datahub.ingestion.run.pipeline: sink wrote workunit urn:li:dashboard:(superset,14)
DEBUG:datahub.ingestion.run.pipeline: sink wrote workunit urn:li:dashboard:(superset,11)
DEBUG:datahub.ingestion.run.pipeline: sink wrote workunit urn:li:dashboard:(superset,2)
DEBUG:datahub.ingestion.run.pipeline: sink wrote workunit urn:li:dashboard:(superset,9)
DEBUG:datahub.ingestion.run.pipeline: sink wrote workunit urn:li:dashboard:(superset,23)
DEBUG:datahub.ingestion.run.pipeline: sink wrote workunit urn:li:dashboard:(superset,21)
DEBUG:datahub.ingestion.run.pipeline: sink wrote workunit urn:li:dashboard:(superset,8)
DEBUG:datahub.ingestion.run.pipeline: sink wrote workunit urn:li:dashboard:(superset,25)
ERROR:datahub.ingestion.run.pipeline: failed to write record with workunit urn:li:dashboard:(superset,16) with ('Unable to emit metadata to DataHub GMS', {'message': "HTTPSConnectionPool(host='<hostname>', port=443): Max retries exceeded with url: /api/gms/entities?action=ingest (Caused by SSLError(SSLError(1, '[SSL: WRONG_VERSION_NUMBER] wrong version number (_ssl.c:997)')))", 'id': 'urn:li:dashboard:(superset,16)'}) and info {'message': "HTTPSConnectionPool(host='<hostname>', port=443): Max retries exceeded with url: /api/gms/entities?action=ingest (Caused by SSLError(SSLError(1, '[SSL: WRONG_VERSION_NUMBER] wrong version number (_ssl.c:997)')))", 'id': 'urn:li:dashboard:(superset,16)'}
ERROR:datahub.ingestion.run.pipeline: failed to write record with workunit urn:li:dashboard:(superset,15) with ('Unable to emit metadata to DataHub GMS', {'message': "HTTPSConnectionPool(host='<hostname>', port=443): Max retries exceeded with url: /api/gms/entities?action=ingest (Caused by SSLError(SSLError(1, '[SSL: WRONG_VERSION_NUMBER] wrong version number (_ssl.c:997)')))", 'id': 'urn:li:dashboard:(superset,15)'}) and info {'message': "HTTPSConnectionPool(host='<hostname>', port=443): Max retries exceeded with url: /api/gms/entities?action=ingest (Caused by SSLError(SSLError(1, '[SSL: WRONG_VERSION_NUMBER] wrong version number (_ssl.c:997)')))", 'id': 'urn:li:dashboard:(superset,15)'}
ERROR:datahub.ingestion.run.pipeline: failed to write record with workunit urn:li:dashboard:(superset,12) with ('Unable to emit metadata to DataHub GMS', {'message': "HTTPSConnectionPool(host='<hostname>', port=443): Max retries exceeded with url: /api/gms/entities?action=ingest (Caused by SSLError(SSLError(1, '[SSL: WRONG_VERSION_NUMBER] wrong version number (_ssl.c:997)')))", 'id': 'urn:li:dashboard:(superset,12)'}) and info {'message': "HTTPSConnectionPool(host='<hostname>', port=443): Max retries exceeded with url: /api/gms/entities?action=ingest (Caused by SSLError(SSLError(1, '[SSL: WRONG_VERSION_NUMBER] wrong version number (_ssl.c:997)')))", 'id': 'urn:li:dashboard:(superset,12)'}

Below is ingest yaml code

def datahub_ingest_yaml(ingest_yaml: str):
    pipeline: Pipeline = Pipeline.create(yaml.full_load(ingest_yaml))

    pipeline.run()
    pipeline.pretty_print_summary()
    pipeline.raise_from_status()

datahub_superset_ingest_yaml = """
source:
  type: superset
  config:
    connect_uri: <superset hostname>
    display_uri: <superset hostname>
    provider: "db"
    username: "admin"
    password: <password>
    stateful_ingestion:
        enabled: True
sink:
  type: datahub-rest
  config:
    server: <hostname>/api/gms
    token: <token>
    retry_max_times: 10
pipeline_name: superset_pipeline
"""

try:
    datahub_ingest_yaml(datahub_superset_ingest_yaml)
except Exception as e:
    logging.exception(e)
    pass

To Reproduce
Steps to reproduce the behavior:

Install Datahub 0.10.5 or 0.11.0 and Superset 3.0.0rc2
Execute above python code

Expected behavior

Working fine without exceeding max retries
Datasource is shown in chart lineage when succeeded sometimes.

Screenshots
Desktop (please complete the following information):

OS: Windows
Browser: Chrome
Version: Chrome 118.0.5993, Windows 10, python 3.10.6

Additional context
Superset and Datahub both are running in Kubernetes by Helm chart.

@realthered Looks like two different issues
Re superset - what db is superset connected to?
Re SSL errors - it doesn't really make sense that some metadata was written to GMS successfully and others failed - do you have a proxy / load balancer / something else sitting in the middle?

@realthered Looks like two different issues
Re superset - what db is superset connected to?
Re SSL errors - it doesn't really make sense that some metadata was written to GMS successfully and others failed - do you have a proxy / load balancer / something else sitting in the middle?

Superset: is connected to snowflake
SSL errors: We're using kubernetes so to access datahub pod from local, there's a network load balancer which is L4 LB so I don't think LB or other kubernetes things affects this.
In case of dbt or snowflake source, datahub sinking works well.
Is there any idea that this issue is related to any superset source structure changed?

update
after update to 0.12.0 image, SSL issue is not occured. But still superset snowflake data source is not shown in chart lineage.

I found the root cause.
Superset Database name was different to its original snowflake database name.
After I found this, it worked well.
Sorry for interruption 😭


Add libdbus-1-dev, libgconf2-dev, libnss3-dev to ubuntu-precise

Fix https://github.com/travis-ci/travis-ci/issues/4222.

Hello. Thank you for taking time to open a PR.
We are in the process of automating package request process. A part of it is running tests automatically. As this PR does not come with tests, we are not able to accept it.
Thank you for your understanding.


fix zookeeper path watches

Watch channel closes after each event, so we need to use GetW each time.
Sorry about closing the previous one.
@maniksurtani

lgtm


Missing namespace for ServiceAccount, Role and RoleBinding

Hello guys !
I little suggestion here :)
Is your feature request related to a problem? Please describe.
When I'm deploying the helm chart using a custom namespace, I expect resources to be created in that namespace
Describe the solution you'd like
I would like the SA, Role and RoleBinding in neo4j-standalone/templates/neo4j-service-account.yaml to register in the proper namespace
Does it make sense to you ?
Thanks !

Completed as part of https://github.com/neo4j/helm-charts/pull/109 and https://github.com/neo4j/helm-charts/pull/108
Will be part of the next release


When i add package by your guide .... but it is show error

Hello,
I add package to my project with your guide
https://akbv.dev/blog/skype-web-api-for-php
But it is show
Fatal error: Uncaught Error: Call to undefined method Akbv\PhpSkype\Connection::getContacts()
Pls help me
Thanks

Don't guide with this guide :-) Use example in src.


Promote globe tools API to public

Promote globe view tools to public. Add NextVersion promotion entries for these.

I added thematic to #1096 before you submitted this.

I added thematic to #1096 before you submitted this.
I don't think it's necessary to enumerate every type.

Thanks - I'll remove this from this PR.

I added thematic to #1096 before you submitted this.
I don't think it's necessary to enumerate every type.

Thanks - I'll remove this from this PR.

@pmconne removed.

Thanks - I'll remove this from this PR.

@kabentley seems to want people to add to NextVersion.md on #1096. I'm not sure if that's better or worse than having each PR author resolve conflicts in that file themselves - it will mean his PR can't build/complete until everybody else's PRs complete. Maybe move your NextVersion.md changes over to there?

Thanks - I'll remove this from this PR.

@kabentley seems to want people to add to NextVersion.md on #1096. I'm not sure if that's better or worse than having each PR author resolve conflicts in that file themselves - it will mean his PR can't build/complete until everybody else's PRs complete. Maybe move your NextVersion.md changes over to there?

@pmconne All right.

Thanks - I'll remove this from this PR.

@kabentley seems to want people to add to NextVersion.md on #1096. I'm not sure if that's better or worse than having each PR author resolve conflicts in that file themselves - it will mean his PR can't build/complete until everybody else's PRs complete. Maybe move your NextVersion.md changes over to there?

@pmconne All right.

@pmconne I removed NextVersion.md entry and will put it on Keith's PR.


No OperationId's in the openapi.json

So I'm generating client stubs with openapi generator from the openapi.json...
But when doing that I discovered there are no operationId's in the file.
This means my methods that are generated are quite clumsily named.
Is this file generated? if so can the operationId's be included? If not, can I create a pull request to add these operationId's? It is a lot of work to add these operation id's though... But it only needs to be done once

@jcourt562 -- please weigh in here on what is best to be done. I know you are looking at this same issue.

So I noticed operation Ids were never included in the output from ACA-py but never raised it because I thought it was intentional.
For my part I went the way of writing a simple TS utility that allows the merging of OperationIds into the openapi.json file prior to doing the codegen step if people wanted to go that way, the Typescript SDK we release provides this utility.
I think if you provide operationIds in the raw  OpenAPI ACA-py you will never make everyone happy so it could well be best to let people annotate it themselves.
In the sample REACT UI we publish I have just used the existing "clumsy" OpenAPI generated method names and isolated them in a models layer.  However this is more a case of I didn't want to manually name things whilst there is still a lot of flux in APIs (i.e. until ACA-py V1.0 is released).

So I should also comment on the generated question.  The openapi.json file is generated by the scripts/generate-open-api-spec utility.  However this is not automatically done on each build.  This was originally created so that differences in APIs could be seen before people committed changes or performed releases.  At that point there were issues around random ordering of parameters etc that were going undetected and a comparison of this output showed that up so it could be detected.
It seems to me you are wanting to move beyond this and like me trying to generate wrappers.  My suggestion would be that atm ACA-py as a product isn't focused on supporting that and its kind of a byproduct that it works.  If you want to do this I would suggest looking at scripts/generate-open-api-spec as an example and start from the raw ACA-py output it pulls from a running ACA-py.  The reason is that tbh there is a lot of nasty corners in each of the Swagger/OpenAPI generators that you will want to play around with before settling on one.  I am currently in the same process on deciding which generator to move to for 0.7.0.   If you want more details on how to generate take look at https://github.com/sudoplatform-labs/sudo-di-cloud-agent-js. or https://docs.sudoplatform.com/sudo-labs/decentralized-identity/cloud-agent-sdk

@jcourt562 -- there is good content here, but I'm thinking it will get lost in this issue.  How about adding a "UsingOpenAPI.md" (or something like that) document is this, where you capture the ideas in here and experience you are having, to help others?

I can do that.  Let me get the 0.7.0 generation off my plate and I will open a PR.

Thanks all...
@jcourt562 I am generating python bindings via a new openapi generator - when I saw the "weird" methods that were being generated is when I noticed the operationId's are not present.
Best case is that these operationId's can be added to the code so that they're embedded and are thus not generated and don't have to be manually added - though I suspect aiohttp-apispec would need to support this and a cursory scan of the the docs thereof did not find support for operationIds. I was hoping the route name would be rendered as operationId (I added some route names and it wasn't)...
Okay - for now I'm just going to add these operationId's manually on my side (at least they can be python friendly then) before generating my client. It's not a big deal as I don't expect functionality to change "en masse" and when new methods are added I can just compare with old and add the new operation Ids.
That UsingOpenAPI.md would be wonderful. Glad I found out how it's generated as I want to generate one that includes the multitenancy endpoints.
Happy for this issue to be closed.

@michaelwiles I would still suggest you automate this to some extent.  Take a look at the merge Typescript utility at https://github.com/sudoplatform-labs/sudo-di-cloud-agent-js/blob/main/src/operationids-merge.ts and https://github.com/sudoplatform-labs/sudo-di-cloud-agent-js/blob/main/src/operationids-merge.ts
Hopefully you would be able to use those as is if you have node installed or create a python equivalent since manually going through an openapi.json is painful and hard to track if it changes.

@michaelwiles there is now a "UsingOpenAPI.md" in the repo.  If this is satisfactory could you close this issue as I don't have permissions to do so but the issue creator would have.


CLOUDSTACK-10031: change default configuration for router.aggregation…

….command.each.timeout from 3 to 600 seconds
https://issues.apache.org/jira/browse/CLOUDSTACK-10031

LGTM

LGTM.

 Will merge this.


Animate edges

Please test this with big topos first.

Fixes #157

As I understand it, #473 includes this PR, so once that's merged, this can be closed.


Unexpected keyword argument : 'use_wheel'

This is the error that showed up when I was running peep.sh to install the requirements for kitsune in the virtual env. What do you think might be the issue ?
Running on Fedora 22.
peep: (2, 4, 0)
python: '2.7.10 (default, Jul  5 2015, 14:15:43) \n[GCC 5.1.1 20150618 (Red Hat 5.1.1-4)]'
pip: '7.1.2'
Command line:  ['./scripts/peep.py', 'install', '-r', 'requirements/default.txt']
Traceback (most recent call last):
  File "./scripts/peep.py", line 902, in <module>
    exit(main())
  File "./scripts/peep.py", line 874, in main
    return commands[argv[1]](argv[2:])
  File "./scripts/peep.py", line 834, in peep_install
    for path in req_paths))
  File "./scripts/peep.py", line 834, in <genexpr>
    for path in req_paths))
  File "./scripts/peep.py", line 795, in downloaded_reqs_from_path
    finder = package_finder(argv)
  File "./scripts/peep.py", line 335, in package_finder
    return PackageFinder(index_urls=index_urls, **kwargs)
TypeError: __init__() got an unexpected keyword argument 'use_wheel'

Two things:
First, if you look at the kitsune docs, they say to run it like this:
$ ./peep.sh install -r requirements/default.txt

http://kitsune.readthedocs.org/en/latest/hacking_howto.html#python-packages
Second, peep doesn't work with versions of pip past 6. this is covered by a bunch of issues and pr #94
Best to use an older version of pip.

@willkg Yes, this is regarding kitsune and that is how I ran it, yet running into these issues.
I think I'll try doing the same using an older version of pip.

@mythmon is on PTO and i'm traveling, but usually there are people in #sumodev on irc.mozilla.org that can help with issues like the ones you're having. There's also the mailing list.
http://kitsune.readthedocs.org/en/latest/contactus.html


Feature/add files and items

Feature: Implement Item Class and Console App for Item Management
Summary
This PR introduces the Item class with all properties defined and set up in the constructor method, including custom setter methods for 1-to-many relationships. It implements methods as per the provided diagram, including can_be_archived? and move_to_archive. Additionally, a console app entry-point, main.rb, has been created for user interaction, allowing users to perform various tasks related to item management.
Changes Made
Item Class

Created the Item class with all properties defined and set up in the constructor.
Implemented custom setter methods for 1-to-many relationships to ensure proper property assignment.
Added the can_be_archived? method, which returns true if published_date is older than 10 years; otherwise, it returns false.
Introduced the move_to_archive method, which reuses the can_be_archived? method to set the archived property to true if the item can be archived.

Console App Entry-Point (main.rb)

Created main.rb to serve as the entry point for the console app.
Implemented startup actions, including:

Presenting users with a list of options for item management.
Allowing users to choose an option and providing necessary parameters when required.
Including a graceful way to quit the app.

Testing
Comprehensive unit tests have been added to validate the functionality of the Item class methods, including can_be_archived? and move_to_archive. Testing ensures that these methods behave as expected.
Reviewer's Notes
I invite the code reviewer to thoroughly examine the codebase for the Item class and the console app entry-point. Please review the implementation of the can_be_archived? and move_to_archive methods and the user interaction flow. Your feedback on code quality, design, and overall functionality is highly appreciated. Thank you for taking the time to review this PR.

Thank you @Rysth


Incorrect variable used while iterating through sessions which makes the loop stuck

Should fix #2043
Now there may be another reason for it being stuck elsewhere, but I just did the thing which should had been done a long time ago, my bad

@AMajor-C
can you please check this fix, it is likely addresses the issue you reported

Running this fix on my vpn server which actually got stuck with the same problem sometimes before, without problems so far for now

oops, I accidently released without merging of this PR.
ok, today will be two releases

released: https://github.com/SoftEtherVPN/SoftEtherVPN/releases/tag/5.02.5187


Makes mandatory some fields when creating issues

After some real tests I've found some mandatory fields when creating issues
Please @juanpedromoreno could you review? Thanks!

Current coverage is 85.82% (diff: 100%)

Merging #51 into master will not change coverage

@@             master        #51   diff @@
==========================================
  Files            27         27
  Lines           402        402
  Methods         398        398
  Messages          0          0
  Branches          3          3
==========================================
  Hits            345        345
  Misses           57         57
  Partials          0          0

Powered by Codecov. Last update 045ffd7...27daf35


dummy_onnx_run.py is missing

Thank you very much for your work, it is really amazing. I followed the instructions in the readme file under the robot folder, but it shows that the file dummy_onnx_run.py is missing from the robot/nodes/ . Could you please provide the complete project package? Thank you.

dummy_onnx_run.py was referenced only from build files (CMakeLists.txt and setup.py), it was not actually being used. Removed references to it.


Wrong gen-driftignore --help example

Description
Currently when we use the --help flag:
This command will generate a new .driftignore file containing your current drifts and send output to /dev/stdout

Example: driftctl scan -o json://stdout | driftctl gen-driftignore > .driftignore

Usage: driftctl gen-driftignore [flags]

FLAGS:
      --exclude-changed     Exclude resources that changed on cloud provider
      --exclude-missing     Exclude missing resources
      --exclude-unmanaged   Exclude resources not managed by IaC
  -i, --input string        Input where the JSON should be parsed from

INHERITED FLAGS:
      --disable-telemetry   Disable telemetry
  -h, --help                Display help for command
      --no-version-check    Disable the version check
      --send-crash-report   Enable error reporting. Crash data will be sent to us via Sentry.
                            WARNING: may leak sensitive data (please read the documentation for more details)
                            This flag should be used only if an error occurs during execution

The example is wrong it misses the -i flag:
Example: driftctl scan -o json://stdout | driftctl gen-driftignore -i /dev/stdin > .driftignore

I would like to take this one.


update examples in index.rst in python docs

Changes made in #361

add examples for scroll, retrieve, upload_collection / upload_records (the latter two might not be in index.rst)


ML 8.2 : LSTM(A)

Description
📌  Issues for Week 8
Welcome to 'ML' Team, good to see you here
:arrow_forward: This issue will helps readers in acquiring all the knowledge that one needs to know about ----
LSTM.
:red_circle: To get assigned to this issue, add your Batch Numbers mentioned in the spreadsheet of "Machine Learning", the approach one would follow and choice you prefer (Documentation, Audio, Video). You can go with all three or any number of options you're interested to work on.
✔ Domain : Machine Learning
:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.

"Issue Title" and "PR Title should be the same. Include issue number along with it.

Changes should be made inside the Machine Learning/Machine_Learning/ML_for_Seqential_Data_&_Visual_Data Branch.

Follow Contributing Guidelines & Code of Conduct before start Contributing.

This issue is only for 'GWOC' contributors of 'Machine Learning' domain.

:white_check_mark: To be Mentioned while taking the issue :

Full name
Batch Number
GitHub Profile Link
Which type of Contribution you want to make :

[ ] Audio

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎
Domain
Machine Learning
Type of Contribution
Audio
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

Name: Jahnvi Shah
Batch: 5
Github Link: https://github.com/jahnvishah
Type: Documentation
I would like to contribute to this issue.

Name: Hrugved Kolhe
Batch: 3
GitHub Profile
[x]  Documentation
Interested

What's your progress here @jahnvishah

What's your Progress here @jahnvishah

Name: Shivangi Kochrekar
Batch: 3
Github Link: https://github.com/Shivangik01
Type: Audio
I would like to contribute to this issue if Jahnvi is busy.

@Shivangik01 This issue is already assigned to @jahnvishah

What's the progress here @jahnvishah

Due some unforseen circumstances I won't be able to generate the PR.

@Shivangik01 would you like to take up this issue?

Hello, I am Deepthi M with serial number:198, Batch-2. I would like to do audio on this issue.my approach: a detailed explanation

@Shivangik01 would you like to take up this issue?

Yes

@Shivangik01 issue assigned to you, complete this within 3 days.


OSR2+ pitch movement at bottom despite static pitch value

Dear Fain,
You cannot believe how glad I am for your work (and pointing me to platform.io). Thank you very much. I have learned a lot thanks to your input.
When I set SR6 mode to false is the code then optimized for OSR2+?
The reason I ask is that despite the pitch value being static I have a pitch movement at the bottom/ end of down movement. (More than for just keeping the up down axis aligned)?
TcodeESP32 0.258b set to Tcode V1
Multifunplayer <IP_ADDRESS> (but same on joyfunplayer)
OSR2+ https://discuss.eroscripts.com/t/osr2-work-in-progress/40027/15
ESP8266 DOIT DEV Kit V1
I have designed the arms myself to make them compatible to ball head joints (yes a possible failure) and would like to go further with my improved design. Could this be a geometry failure. They should be as close as possible to the original.
In case I can help with documentation or CAD, 3D Print you are very welcome to reach out to me. Any creative ideas I am your man to contact. :-)
Some more details/documentation to the user settings (in a separate PDF to reduce sketch size) would be a huge help for newcomers.
Best regards,
Michael

I'm not sure what you mean with this but the pitch acting weird like this (I think) can be the firmware is running SR6 mode on OSR hardware.


IOS, Home Pod and rebooting devices

I have a Home Pod running the IOS beta software. It is attached to a Matter Device. Now I reboot the Matter device. After the Matter device reboots it just sits there displaying this...
E (726576) chip[IN]: Data received on an unknown session (LSID=41282). Dropping it!
E (729956) chip[IN]: Data received on an unknown session (LSID=41282). Dropping it!
E (732106) chip[IN]: Data received on an unknown session (LSID=41282). Dropping it!
E (735596) chip[IN]: Data received on an unknown session (LSID=41282). Dropping it!

I think what is going on here is that the Home Pod  can not tell the Matter device has been rebooted, so it does not know to establish a new session.   As long as the Matter device silently discards messages from this old session the Home Pod just assumes that it is dead.
Shouldn't the Matter device send a NAK of some kind back on that old session to give the Home Pod a clue to try again with a new session?

Per spec, this is correct behavior.  There is no way to send a NAK back on an encrypted session if there are no encryption keys....
Eventually the homepod side will detect that it failed to deliver a message, mark the session as defunct, and create a new session the next time it needs one (though not all of this may be in the version you are using right now).

If the user is tapping on the device in the Home app, the Home app should be aggressive about quickly detecting a reboot.

The fastest that you can detect failure to deliver messages is when all the MRP sends fail, no?

The device could send an unencrypted advisory NAK packet back. Normally just discard this packet. But if the the Home app is in a retry loop you could look for it.  That keeps it from being used for DOS attack.

The device could send an unencrypted advisory NAK packet back

This was discussed during the spec discussions and rejected, fwiw, precisely because the DoS risks were too high.  In particular, since the packet can't be encrypted, anyone on the LAN can send it and prevent communication between devices.

It can only DOS you if you are looking for it. Don't look for except when you are retrying failed sends, in that case it is useful.

And it is not a general NAK, the source address of the packet has to match the destination address of the failed node you are retrying before you do anything with it.

Use case I am thinking about --- user frustrated with some part of the setup procedure who then keeps power cycling the device.

the source address of the packet has to match the destination address of the failed node you are retrying

Source addresses can be spoofed.

You're over thinking this. I can DOS the whole LAN just by flooding it with broadcast packets, not need to mess with a Matter device.

I'm just telling you what the spec discussions looked like.  These are not my personal opinions.


MNT: Re-render the feedstock

Hi! This is the friendly conda-forge-admin automated user.
I've re-rendered this feedstock with the latest version of conda-smithy (0.10.3) and noticed some changes.
If the changes look good, then please go ahead and merge this PR.
If you have any questions about the changes though, please feel free to ping the 'conda-forge/core' team (using the @ notation in a comment).
Remember, for any changes to the recipe you would normally need to increment the version or the build number of the package.
Since this is an infrastructural change, we don't actually need/want a new version to be uploaded to anaconda.org/conda-forge, so the version and build/number are left unchanged and the CI has been skipped.
Thanks!

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

The top level meta keys are in an unexpected order. Expecting ['package', 'source', 'build', 'requirements', 'test', 'about', 'extra'].
When defining a source/url please add a sha256, sha1 or md5 checksum (sha256 preferably).

The failure is just the linter. While that should be addressed, there is already an issue to track that ( https://github.com/conda-forge/cycler-feedstock/issues/4 ). Plus we need to get this feedstock updated as it pretty out-of-date (e.g. brew cleanup was added here). Hope that is ok guys.


Arch

Hi, I love your Rice, is it possible to modify it for an Arch installation?

Hi, I love your Rice, is it possible to modify it for an Arch installation?

Here you go,
https://github.com/VaughnValle/elektropunk
this is similar rice for arch distro


Disable rustc incremental compilation to avoid ICEs

However, I'd also pin rust to 1.56.1 to match the one used in circleci. I would expect this to fix the problem, as well as add to the build consistency. If we'd like to have a separate pipeline for highlighting upcoming rustc compatibility issues, we'd better make it stand out (give it a descriptive name etc). However, you don't have to do any of these in this patch.

I like the idea of pinning, but would want to do that when getting another bump on CircleCI.
It feels like having one less CI system to deal with might simplify our work on that front though?

It feels like having one less CI system to deal with might simplify our work on that front though?

Agreed. As far as I know, @LizardWizzard would also prefer to have a single CI system, but I haven't heard of any concrete plans on that matter. That said, I'd find myself in a great trouble If I were to choose between GH actions and CircleCI, since they both have so many nuances and shortcomings that I'm unable to tell which is worse.


sha 256 isn't working

I think there might be an issue with Lambda.
Sha256 hashes don't just work!
Thanks for your tool. :-)

Thanks for reporting.
Bug will be solved within an hour.

[Fixed]


Clarification on using the store in React components

Great plugin, I have been testing it for some days and it works great.
Just a clarification on which store to use with a collected React component (I am a bit confused on this).
As I understood from the documentation a component "reading" from the store should be collected and use the store that the HOC provides in the props.
My question is when this component makes changes to the store (that will affect other components), should it make them to the store from props, the one imported from the plugin or it doesn't matter?
A follow up question is if the component only modifies the store but doesn't read from it, should it be collected? And if so, the first question again, which store to use?
Thanks!

@garciaalvaro FYI in 4.0.0 there is now an error if you try to read from the 'wrong' store.
When making changes, it doesn't matter which version of the store. The only rule is "when reading inside a collected component, use the store passed to that component as a prop".


Doc Viewer shows "unable to fetch documents" error during initial load

Bug report
Description
On opening the Document Viewer in sandbox, the available document list is fetched from the server. But while we wait for that to load, we see error message "Unable to fetch documents..." until the request completes. This is obviously misleading. Instead we should probably see a loading indicator, and an error message only if the request failed or returned with an empty document list.

The problem is exacerbated because the component is unmounted on close, meaning a request is made every time the viewer is opened. We might therefore consider lifting the "available documents" state into a parent component, and passing it in as a prop. Currently the App component is the parent of the viewer, so the state would need to live there, and probably be fetched when Sandbox is first opened.
Alternatively, we could send the available documents list in the level data response. Level data are fetched from MainComponent rather than App, so the function to open the document viewer would need to be moved into there. This would be the more efficient solution.
Update
The "alternative" solution above will be implemented for reasons of efficiency.
Reproduction steps
Steps to reproduce the behaviour:

Open the Sandbox level
Click on View Documents in the left side panel
Message is visible: "Unable to fetch documents ..."
After a short pause, the above message is replaced by the first document.

Expected behaviour
Error message does not display on initial fetch; instead, a loading indicator is displayed if the docs request is in progress.
Acceptance criteria
GIVEN I am on Sandbox level
WHEN I open the Document Viewer
THEN the first document loads immediately

Deployed to remote, and it now works as expected 👍


Add support for DAE and STL meshes

Hi Robin,
Thank you for creating this awesome visualiser! We've been thinking of using it for a while and are currently integrating it with Pinocchio and Exotica.
This PR adds support for loading DAE and STL meshes by using the three.js loaders for these types.
Note, I've copied them into src/ the same way you added the OBJLoader2 but I am not sure that is even necessary - and there are likely better ways to do it (in webpack?) - very open for suggestions here :-).
I'll open the corresponding meshcat-python PR shortly.
Thank you,
Wolfgang

This looks great, thank you! Copying the loaders into the source tree is fine: it would also be possible to wrap them up in node modules, but I don't think it's necessary or worth the effort.
One thing to note: the generated file dist/main.min.js is intentionally checked into this repo, so I'm going to merge this PR and then make one more commit on master to update that generated file.

Thank you, Robin! I'll wait for your commit with the dist/main.min.js to update the meshcat-python PR in the morning - and will add two meshes as examples (I tested with locally available ones).

Ok, done: https://github.com/rdeits/meshcat/commit/fc7e50d232a866129c8dd82b4217335158f3dcf6


Add the start/draft of the new installer/bootstrap

Here's the start of an installer I'm making to automagically install everything required, save python, for hypatia. because gosh pygame is excessive.

@ejmr add to which commit message? how'd you recommend going about that?

Enough changes incurred to require a new pull requests. New features have been added, etc.

@lillian-lemmer For a branch with a lot of patches I personally tend to add any Reviewed-by, Tested-by, etc., metadata to the actual merge commit when integrating the branch.


other field: (i) change to JSON? (ii) update docs

Thoughts
1. Change to JSON?
I feel like embedding JSON would be useful because it is a known standard. I'm not sure if there might be any issues though including a large amount of JSON inside of a single cell.
2. Update docs
Currently says:

Pipe separated list of key value pairs for properties not part of the SSSOM spec. Can be used to encode additional provenance data.

I propose (changes in bold):

Pipe separated list of equals-sign delimited key value pairs for properties not part of the SSSOM spec. Can be used to encode additional provenance data.
Example: myKey1=myVal1|myKey2=myVal2

Additional info
Current docs page: https://mapping-commons.github.io/sssom/other/

I moved this issue here @joeflack4 because this is a question for the spec. Spec changes must be very well motivated, so I feel like embedding JSON would be useful all by itself is not enough. I was like 90% we have had a lengthy discussion about the question of wether the other field should permit arbitrary JSON. I remember a lot of back and forth on the issue during our workshop (see notes here: https://mapping-commons.github.io/sssom/mc2021/), so I know others like @ShahimEssaid share your thoughts. For those that would like to open the "other" field natively to arbitrary json, please provide a set of arguments. Remember that SSSOM will treat arbitrary json as a string. There wont be any serialisation at all to RDF, other than an AP with a literal, so technically speaking, you can already just write this in the other columns:
json: "{ .. some json..}"

(i.e. json blob is a string). If you want LinkML validation (I don't even know if that is possible, requires a complex datatype "json"), then yes, you would have to push your suggestion here forward.
(As an aside, dear horrified Semantic Web people: there is a clear reason: we need some way to be able to transport additional metadata not current allowed in the spec.) @udp also made some suggestions on how to do this better with QNames, see ticket above.
At this point, I think we need some clear set of arguments before making any spec changes. Once all the arguments are together, we call a vote.

Sounds good. Thanks for the feedback.
I am on vacation this upcoming Monday, so I think our next 1:1 is on the 21st. I'll be taking half-day vacations during that week as well, but I think it will be no problem for me to meet. Fingers crossed though. It's possible that my mom might have some appointment for me to take her to; she frequently has me help w/ that when I'm visiting. If that happens and I need to reschedule, I'll let you know.

very cool. i think this is a great idea. still will want to have a general way to include other information if no applicable CURIE for the field exists.

You can always invent a property! joe:comment!


Disable host key checking in non-interactive sessions

IP reuse is frequent in our GCE test clusters.

This change is

Ah, hmm, maybe this message doesn't actually cause problem except that it gets displayed. (I was running a grep across as 12 node cluster).

Yeah, looks like it. I assume we're not going to merge this then.

Actually the warning message without this is pretty distracting.


Android fails with Error: uses-sdk:minSdkVersion 16 cannot be smaller than version 17 declared in library [:flutter_inappbrowser]

After adding the plugin to my project (using Flutter 0.11.3) Android builds fail with:
AndroidManifest.xml Error: uses-sdk:minSdkVersion 16 cannot be smaller than version 17 declared in library [:flutter_inappbrowser] ...
.. as the library might be using APIs not available in 16
Suggestion: use a compatible library with a minSdk of at most 16,
or increase this project's minSdk version to at least 17,
or use tools:overrideLibrary="com.pichillilorenzo.flutter_inappbrowser" to force usage (may lead to runtime failures)

The minSdkVersion is already setted to at least 17! Just see the build.gradle file.

@pichillilorenzo many thanks for your quick reply - I'm sure it is, however that was the error I got when adding the package and then trying to run it in my Android VM running Pie - I didn't do anything else and I haven't made any manual changes to the Android build files etc. Any ideas on how to fix this? Many thanks (super keen to get this plugin working as looks like it does exactly what we need 👍)

Oh hang on - I need to update my AndroidManifest file don't I - so that has the higher version as requested by this plugin? Let me check...

Yep, sorry - that was the problem. Thanks. Any chance the iOS build issue as a similar easy issue?

@alsocalledchris
I am not very experienced with Android development yet.
Could you please explain what needs to be changed in AndroidManifest to set the higher version of the sdk library?
Thanks

@velsa try change file android/app/build.gradle minSdkVersion config
sorry! I can not speak English

Oh hang on - I need to update my AndroidManifest file don't I - so that has the higher version as requested by this plugin? Let me check...

Can you please elaborate what you did?

Hi! The library description says that minSdk = 17, but the library manifest specifies 20. What is the current minimum version of the Sdk for the library?


Update to SQL Server on Linux Security Limitations - Authentication Mode

Adding information to document on SQL Server authentication for SQL Server on Linux.

@clayton006 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off
@VanMSFT  can you please help merge this change to the live doc

@amvin87: I'm sorry - only the author of this article, @VanMSFT, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@amvin87 I can merge this with your approval. Thanks for the review.


First Issue

I think you need actual issues for collaborators to work on if you want to add the collaborate tag to your repository, not empty ones.

Hey LyubomirT,
Thanks for the suggestion! :purple_heart:


Enable configurable solc --allow-paths to accomodate non-standard imports

The --truffle mode is unpleasant to work with, as it creates a huge amount of output. Source mappings are currently also much better when first applied to solidity files.
A more flexible option would be to create a config file like .mythrilrc, which would allow me to specify path mappings, similar to solc --allow-paths.

As a workaround, you can use the --solc-args argument to pass additional arguments to solc.

oh nice.

How would  I structure the command if I need to remap two paths?
ie. in my env I've set two path re-mappings named ZEPPELIN and TF.
Solc handles this fine: $ solc $ZEPPELIN $TF Sale.sol
But if I need to import two path re-mappings, this gives Input file not found:
myth -x --solc-args $ZEPPELIN $TF Sale.sol

Does it work if you quote the solc-args?
myth -x --solc-args "$ZEPPELIN $TF" Sale.sol

Yes it does. 🌮

Assuming there is still interest in this being used as part of the dev process, rather than just auditing, I think a config file would be much better. That would simplify IDE integrations, and align with tools like linters, truffle, and scripting via NPM.

For developers, mythril-cli will be superseded by mythjs, so I'll make this part of the mythjs specs.


Define U gate before parsing

This adds a bogus gate definition for U, which is required to be built in. This appears as the first statement in the ASG.
This is perhaps not the best way to handle the built in gate. But it doesn't touch much code. It can be changed to something better without too much trouble.

Since U is required by the language spec, I am thinking it may be better to treat this specially rather than always sticking it in the ASG.
This would require an explicit check in a gate call. Also an explicit check every time a symbol is bound (If I recall correctly you are not allowed to shadow U).

If I recall correctly you are not allowed to shadow U

I think the scoping rules are no different from any other global-scope identifier (if I remember right), in that you can shadow it with a variable in an inner scope, but not in the global scope.  But equally, if it's easier to treat it as reserved for the time being, I don't think that's an onerous request for people.

My mention of shadowing is a bit of a red herring. In fact, we can treat U as reserved (with special handling for checking semantics of calls); and at the same time allow it to be shadowed where the spec allows.

Closing this in favor of #68

#68 is cleaner, smaller, and more efficient.


Make SmartTypes and NSException (WinObjC) public for middleware

Fixes #2317.
Changes:

SmartTypes.h and TypeTraits.h are public in the language package.    These were public in the old zip file but, are missing in the new packages.  I made them available in the language package.   I'm not sure this is the correct place.    I feel  CF* should be in foundation.core.  And, while AutoId could be in language, it does have dependency on Foundation (for nil....).  I spent some time trying to make this happen, but, it just wasn't worth the effort.
In the end, I took the easiest approach and just made SmartTypes.h public in language.

The NSException category in FoundationErrorHandling has been moved to NSException.  wil\result.h, which is a public header references FoundationErrorHandling.h.  So, we have to make FoundationErrorHandling.h public.  I decided to move all the category methods into NSException(winobjc).  Again, not sure if this is the best place.

[updated pull request title to avoid truncation/improve clarity]

I don't like exposing the _ things, but it's more ABI-breaking to fix that.

The original PR title was supposed to be a teaser to make people click on it.  Just kidding.
thanks.

In reply to: 291669860 [](ancestors = 291669860)

Isn't the  only other thing we'd have to update is libobjc2?
I'm fine updating libobjc2 to remove the _.

In reply to: 291669943 [](ancestors = 291669943)

@pradipd We'd need to rev the projections libraries again, as well as all modules using ErrorHandling (since they contain static copies of the error handling functions.) 😦 It would probably temporarily damage incremental build for anyone who pulled the changes down.

@pradipd And thanks. I'm piloting talking to people about commit message guidelines, and I wanted to passive-aggressively try them on for size before I did. ;)


Add ability to delete NDP entry

@jim-p Thanks for your feedback, the command is only executed if the IP is deemed valid by is_ipaddrv6() (which rejects emptry strings, null and other dummy values).
I've added escaping to the parameter when sent to the command, let me know if there is additional validation to be added.

Looks OK now, tested/works.


jsmall no longer opens

If you try opening jsmall (I am using the most recent VDC version on little_glade) you get the following error:
Failure to open data set of type "vdc"
Failed to initialize horizontal coordinates
Failed to initialize derived coord variable
Invalid variable "Cs_r"

On OSX.

This issue has been partially fixed by PR #2277. However, the fix only works for VDC files generated after #2277. Older VDC files will still not open.


Change to parse composite types

This change makes it possible to parse composite types (e.g. list<record<key: string>>, record<key: list<int>>)

This looks good to me but my only question is will it still parse if it's on different lines. For instance, this is one of the tests.
def test [
  x: record<key: list<int>>
]: nothing -> record<key: list<int>> {}

but what about
def test [x: record<key: list<int>>
]: nothing -> record<key: list<int>> {}

or
def test [x: record<key: list<int>>]: nothing -> record<key: list<int>> {
}

or all on one line
def test [x: record<key: list<int>>]: nothing -> record<key: list<int>> {}

do all these work too?

Yes, all of those works.
I added tests for those cases.

Thank you! and thanks for your continued work here!


Greater Teleport doesn't take riders along

GTPing a horse/skeleton horse/donkey/probably others while riding them results in the player getting kicked off them and only the mob teleporting.
hex to reproduce, while riding a horse:
Mind's Reflection
Compass' Purification II
Numerical Reflection: 1
Zone Distillation: Animal
Flock's Disintegration
Vector Reflection: +Y
Greater Teleport
Forge 43.2.21
Kotlin For Forge 3.12.0
Patchouli 1.19.2-77
JEI <IP_ADDRESS>6
PAUCAL 0.5.0
Hex Casting 0.10.3

Related: #402
I think f6cf4af59b72ff741b41137d893e7743d0f46230 fixes this?

Do you know if this bug is still present in the 1.20.1 prereleases?

Do you know if this bug is still present in the 1.20.1 prereleases?

where can i find those?

1.20 prerelease builds:
https://ci.blamejared.com/job/petrakat/job/Hexcasting/job/main/
https://ci.blamejared.com/job/petrakat/job/PAUCAL/job/main/

The horse can be teleported alone, but if I'm riding it it drops my staff and says "Greater Teleport: cannot alter Horse". i'm guessing this isn't the intended behavior either.

I think this is intended, actually, though the error message isn't great. Does it work with a pig?

Same thing happens with pigs and boats


Build error (vercel)

Describe the bug
Vercel gives an error while building this project!
To Reproduce

Connect Vercel with this repo
Start building now
The error will show up

Expected behaviour
A successful build
Additional context
Here's the build log:
TypeError: Cannot read property 'content' of undefined
12:24:08.959 | at get (file:///vercel/path0/.svelte-kit/output/server/entries/endpoints/_slug_/index.json.js:11:38)
12:24:08.959 | at render_endpoint (file:///vercel/path0/.svelte-kit/output/server/app.js:150:26)
12:24:08.959 | at resolve (file:///vercel/path0/.svelte-kit/output/server/app.js:1819:59)
12:24:08.959 | at async Object.handle (file:///vercel/path0/.svelte-kit/output/server/app.js:1935:20)
12:24:08.960 | at async respond (file:///vercel/path0/.svelte-kit/output/server/app.js:1770:22)
12:24:08.960 | at async fetch (file:///vercel/path0/.svelte-kit/output/server/app.js:1206:22)
12:24:08.960 | at async load (file:///vercel/path0/.svelte-kit/output/server/entries/pages/_slug_/index.svelte.js:150:15)
12:24:08.961 | at async load_node (file:///vercel/path0/.svelte-kit/output/server/app.js:1286:14)
12:24:08.961 | at async render_page (file:///vercel/path0/.svelte-kit/output/server/app.js:1651:20)
12:24:08.964 | TypeError: Cannot destructure property 'datePublished' of 'post' as it is undefined.
12:24:08.964 | at file:///vercel/path0/.svelte-kit/output/server/entries/pages/_slug_/index.svelte.js:80:11
12:24:08.964 | at Object.$render (file:///vercel/path0/.svelte-kit/output/server/chunks/index-13c0de55.js:90:18)
12:24:08.964 | at file:///vercel/path0/.svelte-kit/output/server/entries/pages/_slug_/index.svelte.js:172:54
12:24:08.964 | at Object.$render (file:///vercel/path0/.svelte-kit/output/server/chunks/index-13c0de55.js:90:18)
12:24:08.965 | at Object.default (file:///vercel/path0/.svelte-kit/output/server/app.js:61:96)
12:24:08.965 | at file:///vercel/path0/.svelte-kit/output/server/entries/pages/_slug_/__layout.svelte.js:4:42
12:24:08.965 | at Object.$render (file:///vercel/path0/.svelte-kit/output/server/chunks/index-13c0de55.js:90:18)
12:24:08.965 | at Object.default (file:///vercel/path0/.svelte-kit/output/server/app.js:55:111)
12:24:08.965 | at file:///vercel/path0/.svelte-kit/output/server/entries/pages/__layout.svelte.js:176:82
12:24:08.965 | at Object.$render (file:///vercel/path0/.svelte-kit/output/server/chunks/index-13c0de55.js:90:18)
12:24:08.967 | > 500 /undefined (linked from /)
12:24:08.968 | at file:///vercel/path0/node_modules/@sveltejs/kit/dist/chunks/index5.js:408:11
12:24:08.968 | at visit (file:///vercel/path0/node_modules/@sveltejs/kit/dist/chunks/index5.js:585:4)
12:24:08.968 | at processTicksAndRejections (internal/process/task_queues.js:95:5)
12:24:09.009 | error Command failed with exit code 1.
12:24:09.009

Hi Amr, thanks for reaching out.  Sorry to see this isn't working for you.  Just wanted to check you have replaced the static adapter with the Vercel one before deploying (I should add this to the README as a step)?

Sorry for the late reply :(
I added vercel adapter but got another error while building!
Generating manifest.json
--
19:48:34.403 | > Cannot find module '@sveltejs/app-utils/renderer'
19:48:34.403 | Require stack:
19:48:34.403 | -<EMAIL_ADDRESS>19:48:34.403 | Require stack:
19:48:34.404 | -<EMAIL_ADDRESS>19:48:34.404 | at Function.Module._resolveFilename (internal/modules/cjs/loader.js:902:15)
19:48:34.404 | at Function.Module._load (internal/modules/cjs/loader.js:746:27)
19:48:34.404 | at Module.require (internal/modules/cjs/loader.js:974:19)
19:48:34.404 | at require (internal/modules/cjs/helpers.js:93:18)
19:48:34.404 | at Object.<anonymous> (/vercel/path0/node_modules/@sveltejs/adapter-vercel/index.js:7:16)
19:48:34.404 | at Module._compile (internal/modules/cjs/loader.js:1085:14)
19:48:34.405 | at Object.Module._extensions..js (internal/modules/cjs/loader.js:1114:10)
19:48:34.405 | at Module.load (internal/modules/cjs/loader.js:950:32)
19:48:34.405 | at Function.Module._load (internal/modules/cjs/loader.js:790:12)
19:48:34.405 | at ModuleWrap.<anonymous> (internal/modules/esm/translators.js:199:29)
19:48:34.416 | error Command failed with exit code 1.

aha, at least we're making progress 😀.
I should have said you need to get the @next version of the adapter so if you're using pnpm:
pnpm add -D @sveltejs/adapter-vercel@next

hope that helps.

Thanks for the hint, the build went successfully ✌🏽
Which makes me ask why it didn't in the first time? why adding @next solved it? is it the adapter problem, pnpm's or my lack of knowledge ?
Now we're clearly making progress, but now the webapp has a runtime error when visiting the homepage.
2022-02-16T14:16:38.643Z	556050fe-ea0f-49e7-bf62-de5d1fb5b5a2	ERROR	Error: ENOENT: no such file or directory, scandir '/var/task/src/content/blog'
    at Object.readdirSync (fs.js:1047:3)
    at getPostsContent (/var/task/index.js:57961:45)
    at get (/var/task/index.js:58024:24)
    at load_shadow_data (/var/task/index.js:59577:28)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
    at async load_node (/var/task/index.js:59341:28)
    at async respond$1 (/var/task/index.js:59742:22)
    at async render_page (/var/task/index.js:59872:20)
    at async resolve (/var/task/index.js:60059:105)
    at async Object.handle (/var/task/index.js:60173:20)

Sorry this dropped off my radar... did you get it working?  There have been a couple of breaking changes in SvelteKit so it might just need updating

No problem, I rewrote the whole thing and used vite's glob import while using vercel adapter.

No problem, I rewrote the whole thing and used vite's glob import while using vercel adapter.

Can you share?


Fix postinstall and auto link in iOS

Description
After upgrading to React Native 0.64.2, react-native-ui-lib is no longer auto linked in iOS and throws the following error:

TypeError: null is not an object (evaluating '_reactNative.nativeModules.KeyboardTrackingViewTempManager.KeyboardTrackingScrollBehaviourNone')

Related:
https://github.com/wix/react-native-ui-lib/issues/968
https://github.com/wix/react-native-ui-lib/issues/592
https://github.com/wix/react-native-ui-lib/issues/592#issuecomment-571083027

 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.2 out of 3 committers have signed the CLA.:white_check_mark: glinda93:white_check_mark: diegolmello:x: ethansharYou have signed the CLA already but the status is still pending? Let us recheck it.


vue-query add reactive support

What are the steps to reproduce this issue?

Generate using vue-query template option
Generated query functions are not typed to support reactive values. For example doesn't use UnwrapRef Refs or MaybeRef
Generated query functions won't unwrap reactive path params. eg: /the-path/something-reactive-not-unwrapped
Generated query keys won't unwrap paths

Vue JS supports reactive values - This is also supported by https://tanstack.com/query/latest/docs/vue/reference/useQuery. We should be able to pass in a reactive value and expect it will be unwrapped and typed appropriately where necessary.
VueJS composition API has a series of utility helpers to assist in unwrapping reactive values. You can unref any arbitrary value (reactive or not) using unref(theValue). This is also possible with supplied typescript utilities UnwrapRef. And for anybody picking this up, please ensure vue 2.7 and vue 3 are both supported.
https://vuejs.org/api/reactivity-core.html#ref
The problem:

I'm running into similar issues...
Basically what I'm trying to do is very simple - when user clicks on collection (folder) - fetch that folder items.
I believe that I have problems with reactivity here, because if I manually setup watchEffect() and do refetch() there when activeCollectionPK changes - everything works.
ChatGPT suggested similar approaches, like invalidating query on change, or manually fetching and setting query data. However, all that seems to kinda defeat the purpose of Vue Query...
I saw your example with computed prop from PR and it does seem very similar to what I'm trying to achieve. I've tried that approach and it didn't work for me (I built and linked your fork). The only difference is that activeCollectionPK  is coming from pinia, and I converted it to ref so it should be reactive... And it does work fine in the
 - updates as soon as I click. But doesn't update in the query.
Perhaps I'll have to do more investigation and try to isolate the problem, but let me know if you have any pointers, thank you so much for reporting this and for the PR!
export default defineComponent({
  name: "LibraryList",
  setup() {
    const library = useLibraryStore();
    const { activeCollectionPK } = storeToRefs(library);
    const activeCollectionDetailsQuery = useMonitoringRequestsGETDetailedMonitoringRequests(activeCollectionPK);
    const allCollectionsQuery = useMonitoringRequestCollectionGETAll();

    return {
      library,
      allCollectionsQuery,
      activeCollectionDetailsQuery,
      activeCollectionPK,
    };
  },
  methods: {
    renderItems() {
      return unref(this.activeCollectionDetailsQuery.data)?.data.map((item) =>
{item.name}
);
    },
    renderCollections() {
      return this.allCollectionsQuery.data.value?.data.map((collection) => (

 {
            this.library.setActiveCollectionPK(collection.pk);
          }}
        >
          {collection.name}

      ));
    },
  },
  render() {
    return (

Selected: {this.activeCollectionPK}

        {this.renderCollections()}

        {this.renderItems()}

    );
  },
});

@Maxim-Mazurok
Are you able to share detail about the endpoint. The Orval generated output? It's hard to infer how the endpoint is called. For example with query parameters or request path.
Are you using a custom fetch client? Thanks

useMonitoringRequestsGETDetailedMonitoringRequests

It's in the path:
export const monitoringRequestsGETDetailedMonitoringRequests = (
  collectionPK: string,
  options?: AxiosRequestConfig
): Promise<AxiosResponse<ResponseMonitoringRequestModel[]>> => {
  return axios.default.get(`/api/v1/monitoring-requests/${unref(collectionPK)}/detailed`, options);
};

export const getMonitoringRequestsGETDetailedMonitoringRequestsQueryKey = (collectionPK: MaybeRef<string>) =>
  [`/api/v1/monitoring-requests/${unref(collectionPK)}/detailed`] as const;

export const getMonitoringRequestsGETDetailedMonitoringRequestsQueryOptions = <
  TData = Awaited<ReturnType<typeof monitoringRequestsGETDetailedMonitoringRequests>>,
  TError = AxiosError<unknown>
>(
  collectionPK: MaybeRef<string>,
  options?: {
    query?: UseQueryOptions<Awaited<ReturnType<typeof monitoringRequestsGETDetailedMonitoringRequests>>, TError, TData>;
    axios?: AxiosRequestConfig;
  }
): UseQueryOptions<Awaited<ReturnType<typeof monitoringRequestsGETDetailedMonitoringRequests>>, TError, TData> & {
  queryKey: QueryKey;
} => {
  const { query: queryOptions, axios: axiosOptions } = options ?? {};

  const queryKey = getMonitoringRequestsGETDetailedMonitoringRequestsQueryKey(collectionPK);

  const queryFn: QueryFunction<Awaited<ReturnType<typeof monitoringRequestsGETDetailedMonitoringRequests>>> = ({
    signal,
  }) => monitoringRequestsGETDetailedMonitoringRequests(collectionPK, { signal, ...axiosOptions });

  return { queryKey, queryFn, enabled: !!collectionPK, ...queryOptions };
};

export type MonitoringRequestsGETDetailedMonitoringRequestsQueryResult = NonNullable<
  Awaited<ReturnType<typeof monitoringRequestsGETDetailedMonitoringRequests>>
>;
export type MonitoringRequestsGETDetailedMonitoringRequestsQueryError = AxiosError<unknown>;

export const useMonitoringRequestsGETDetailedMonitoringRequests = <
  TData = Awaited<ReturnType<typeof monitoringRequestsGETDetailedMonitoringRequests>>,
  TError = AxiosError<unknown>
>(
  collectionPK: MaybeRef<string>,
  options?: {
    query?: UseQueryOptions<Awaited<ReturnType<typeof monitoringRequestsGETDetailedMonitoringRequests>>, TError, TData>;
    axios?: AxiosRequestConfig;
  }
): UseQueryReturnType<TData, TError> & { queryKey: QueryKey } => {
  const queryOptions = getMonitoringRequestsGETDetailedMonitoringRequestsQueryOptions(collectionPK, options);

  const query = useQuery(queryOptions) as UseQueryReturnType<TData, TError> & { queryKey: QueryKey };

  query.queryKey = queryOptions.queryKey;

  return query;
};

I don't think I use a custom fetch client, I use axios with a few interceptors that just show loading status.
This is my orval output config:
{
  target: "query",
  client: "vue-query",
  schemas: "model",
  workspace: "./src/api",
  mode: "split",
  mock: false, // maybe use MSW for testing later on? https://mswjs.io/
  prettier: true,
  query: {
    queryOptions: {
      refetchInterval: 0,
      refetchOnWindowFocus: false,
    },
  },
}

However looks like I specified queryOptions in a wrong way as they don't appear to be applied.

So I've just tried sample from your linked PR and it has the same problem with reactivity as the one I've described.
When you click to load a certain pet - it loads it, but then when you click to load another pet - the selected ID changed, but the name stays the same (and no request to fetch that new pet in devtools). I've recorded a screencast with voice to demonstrate the issue:
https://user-images.githubusercontent.com/7756211/237001025-8aa20dcc-b7b6-478e-af3b-cfb94e9f0251.mp4

@anymaniax @Marcel-G @Maxim-Mazurok
In version 6.18 this change, there seems to be an issue that's affecting reactivity in Vue. I suspect that Vue Query might require the ref key to maintain reactivity.
I've prepared a CodeSandbox example to illustrate the problem. I modify the generated file to remove the unref from vueQueryKey function, One version uses ref on the key, while the other removes it. This demonstrates the difference in reactivity.
Link to CodeSandbox

In version 6.18 this change, there seems to be an issue that's affecting reactivity in Vue. I suspect that Vue Query might require the ref key to maintain reactivity.

Thanks for reporting, you are absolutely right the change did unfortunately effect reactivity in the vue client. Is the issue you observe the same described here:  https://github.com/anymaniax/orval/pull/931#issuecomment-1752355686?

@anymaniax @Marcel-G @Maxim-Mazurok
In version 6.18 this change, there seems to be an issue that's affecting reactivity in Vue. I suspect that Vue Query might require the ref key to maintain reactivity.
I've prepared a CodeSandbox example to illustrate the problem. I modify the generated file to remove the unref from vueQueryKey function, One version uses ref on the key, while the other removes it. This demonstrates the difference in reactivity.
Link to CodeSandbox

This has been fixed in #931 (not released yet)
I've generated endpoint using master for your example and it's working fine now: https://codesandbox.io/p/devbox/ecstatic-microservice-8z4rzt
Also the original issue was resolved in #852 but looks like wasn't properly linked to the PR, so closing it now as done


Remove small redundant code

This PR removes some old redundant code as a result of #166.
Functionality doesn't change because even with this code it will just do nothing but removing for tidiness

Codecov Report

Merging #168 (d89bd2e) into master (0d37c97) will increase coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #168      +/-   ##
============================================
+ Coverage     69.08%   69.10%   +0.01%
+ Complexity      792      791       -1
============================================
  Files           118      118
  Lines          2410     2405       -5
  Branches        270      268       -2
============================================
- Hits           1665     1662       -3
+ Misses          701      700       -1
+ Partials         44       43       -1

Impacted Files
Coverage Δ

.../seedu/contax/logic/commands/ImportCsvCommand.java
93.75% <ø> (+4.56%)
:arrow_up:

Continue to review full report at Codecov.

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


Fix #259 - deal with Bracket's error strings in addition to Filer err objects.

This fixes the case that higher-level Brackets filesystem failures happen in addition to low-level Filer errors.  In the latter case, we only get an error string, which is all we need for Brackets to know what to do and show the correct error UI.  I've tested with the rename case, where you try to rename to an existing filename, and also with creating a new file and using an existing name.  Both do what you'd expect.

r+


Quest : Add ability to capture logs

Add the ability to track and add logs for the below listed items :

[ ] Log system interactions by the user
[ ] Log system logins and logouts
[ ] Log all activities performed by the user
[ ] Log access to views of individual client records
[ ] Log exchange of data with other systems
[ ] Log all data and system errors

This would be interesting to have, at this level of detail I'd describe this more as "audit trails" or "instrumentation" than logs.
Looks like there are lots of libraries out there for this, we'd need to eval and choose one, we can consult w/other android devs too.

Notes - Feb 21st 2021

We  can use FHIRBase logging
Alternative can be to use the AuditEvent resource - http://hl7.org/fhir/auditevent.html - This resource may take up a lot of storage and we may need a policy to control when to clear the logs  - May need some more grooming/scoping

Do you mean firebase? Or do you have a references for fhirbase?

On Feb 21, 2022, at 02:41, Francis Odhiambo Otieno @.***> wrote:

Notes - Feb 21st 2021
We can use FHIRBase logging
Alternative can be to use the AuditEvent resource - http://hl7.org/fhir/auditevent.html - This resource may take up a lot of storage and we may need a policy to control when to clear the logs - May need some more grooming/scoping
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you commented.

Do you mean firebase? Or do you have a references for fhirbase?
…
On Feb 21, 2022, at 02:41, Francis Odhiambo Otieno @.***> wrote:  Notes - Feb 21st 2021 We can use FHIRBase logging Alternative can be to use the AuditEvent resource - http://hl7.org/fhir/auditevent.html - This resource may take up a lot of storage and we may need a policy to control when to clear the logs - May need some more grooming/scoping — Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android. You are receiving this because you commented.

Typo - Firebase


[bug] Media currently downloading are shown as failed until they are fully downloaded in download menu

Fintunes 2.0.1 on iOS 15.7.1
Also downloads in the queue don't show up

Fintunes 2.0.1 on iOS 16.2
Actually can't seem to get anything to download locally. Just set up my own jellyfin server, it's possible I've done something wrong, though streaming works just fine. I can download audio tracks from the browser.
Fintunes definitely updates the icon (from a cloud to a cloud with arrow) when I select download track, but the downloads tab remains empty.


[BUG] SwProductSelect doesn't render itself based on product variants

Describe the bug
When I add a new product variants, the SwProductSelect component doesn't render these variants (SwProductDetails.vue component). Also computed getters getAllProductOptions, hasChildren and getAllProductOptionsTypes are empty. What is interesting, is that the default shopware6 shop has these variants to choose (screenshot below in the Additional context section). I suppose that they should render itself by default.
Steps to reproduce

Create a new product in the shopware6 admin dashboard
Create few variants of the main product
Go to that product page (PWA shopware app) and check the product object and computed properties (SwProductDetails.vue component)
Go to the default shopware6 shop, find that product and check the variants (they should be visible as on screenshot below)

Additional context
Screenshot of computed properties (PWA shopware)

Screenshot of default shopware6 shop (not PWA)

Shopware6 PWA version: ^0.4.2
Shopware6 Dashboard version: <IP_ADDRESS>

hi @Radek-Wawrzyk, thanks for reporting the issue :+1:
I have created a couple of variants for a specific product, the result is here: https://shopware-pwa.storefrontcloud.io/detail/fb14584a96864e1ba607e04d41625a84
shopware: <IP_ADDRESS>
pwa: latest stable
so maybe there are missing parts within the request itself :thinking:
in order to reproduce the entire env please paste the request you are sending while you are entering the product details page (CURL format preferred) :muscle: and the response maybe?
we will figure it out :)

CURL request
curl 'http://admin.britishstandard.test:8108/store-api/v3/pwa/page' \
  -H 'Connection: keep-alive' \
  -H 'Accept: application/json, text/plain, */*' \
  -H 'sw-access-key: SWSCRTZCDM5TEWE0EDLSZMRVTQ' \
  -H 'sw-context-token: T0N0mFSLuo7xZW7g2HNL5CHgWIoC5xyL' \
  -H 'sw-language-id: 2fbb5fe2e29a4d70aa5854ce7ce3e20b' \
  -H 'User-Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/85.0.4183.102 Safari/537.36' \
  -H 'Content-Type: application/json;charset=UTF-8' \
  -H 'Origin: http://britishstandard.test:8108' \
  -H 'Referer: http://britishstandard.test:8108/' \
  -H 'Accept-Language: pl,en-GB;q=0.9,en;q=0.8,pl-PL;q=0.7,en-US;q=0.6,la;q=0.5' \
  --data-binary '{"path":"detail/1aac798959784c769b73d5758df0b7a8","limit":10,"associations":{"media":{},"properties":{"associations":{"group":{}}},"productReviews":{},"children":{"associations":{"options":{"associations":{"group":{}}}}},"options":{"associations":{"group":{}}}},"includes":{"cms_page_slot":["id","type","slot","blockId","config","data","backgroundMediaMode","backgroundMedia"],"cms_page_block":["slots","type","id","backgroundColor","backgroundMedia","sectionPosition","backgroundColor"],"cms_page_section":["id","backgroundMedia","blocks","type","sizingMode"],"cms_page":["id","name","sections","type","config"],"product":["manufacturer","id","name","properties","deliveryTime","stock","media","productNumber","children","calculatedListingPrice","translated","options","properties","manufacturer"],"product_media":["media"],"media":["thumbnails","width","height","url"],"calculated_price":["unitPrice","quantity"],"product_group_option":["name","id","group","translated"],"product_group":["id","name","options","translated"],"property_group":["id","translated","options"],"property_group_option":["name","translated","id","colorHexCode","media","group"]}}' \
  --compressed \
  --insecure

shopware-pwa.config.js
module.exports = {
  apiDefaults: {
    useCms: {
      includes: {
        product: [
          "manufacturer", "id", "name", "properties", "deliveryTime",
          "stock", "media", "productNumber", "children", "calculatedListingPrice"
        ],
        property_group: ['id', 'translated', 'options'],
        property_group_option: [
          'name',
          'translated',
          'id',
          'colorHexCode',
          'media',
          'group',
        ],
      },
      associations: [
        { name: "media" },
      ]
    },
  },
};

missing variants may be caused by lack of associations/includes params within the request (loadAssociations). will be fixed in milestone #v0.5.1

@Radek-Wawrzyk have you checked it on v0.5.1?

it has been fixed in 0.5.2


Support mixing standard js module wrapping along side less-conforming module systems

You get common.js wrapping by default on all your locally defined script files, and only get an option to turn it off globally for all files (modules -> wrapper = false). There should be a mechanism in the config to turn off or re-configure module wrapping for certain sets of files. This could be done as a separate concat, or as an orthogonal option that is applied to all files while doing the concat step (as another setting under modules?) - or maybe some other way I haven't thought of :)
This would be useful for setups that define a mix of angular 1.0 code and more general purpose javascript code. A work-around is to include your more general purpose code in a separate repot with its own build, but this might not be as convenient.

I'm not sure how often this sort of scenario crops up, so can't comment on the priority :) I'm just running into this when learning angular for the first time, and trying to use a from-scratch brunch setup to build it... I don't know how much non-angular code I'm going to make, so for me, I can probably just turn it off globally. As a newbie to angular, this seems a bit janky to me right now, is all.

Interesting idea. Let's take a look at possibilities / +1s.

@kavika13 you can use your non-angualr/general purpose JS code via modules, too, can't you?
I mean, if it's some sort of helper/utility functions, should be fine to export them
module.exports = myGeneralPurposeFunction;

and require it when used
var myGeneralPurposeFunction = require('my-general-purpose-function-module-name');

that should be fine in most of the cases. In fact, that would make your code clear as to where things come from and such. We do it too
Otherwise, putting your non-moduled code into vendor or extending the conventions.vendor could prevent these from getting wrapped: see this on how to extend the default conventions.

I've been using brunch for a couple years now. A new project of mine has led me to a brunch/npm angular setup and this would be ideal imo.


feat: improve stream filter builder pattern

Thank you for this!

This item belongs to payment request #D2C5C8 on OnlyDust:

from fracek to MartianGreed
1 item included
$63 for 1 hour of work

This item belongs to payment request #D2C5C8 on OnlyDust:

from fracek to MartianGreed
1 item included
$63 for 1 hour of work

Payment request #D2C5C8 has been processed and payment is complete. See you around on OnlyDust.


Not able to optimise config cache

If the following line is in the config file:
'get_avatar_url' => fn ($user) => $user->getAvatarUrl(),
When running:
php artisan optimize
A Your configuration files are not serializable. error is produced.
This is caused by using a closure in the config file.

Thanks a ton for the PR! I merged it and released in version 2.0.1. :) Good luck!


Umm

Is it just me or does this feel super incomplete?

It's still in development. If you are unhappy with the progress how about you pitch in and contribute?

It’s good but feels weird

...ok?


Carbon\Exceptions\InvalidTimeZoneException

Carbon\Exceptions\InvalidTimeZoneException
Unknown or bad timezone (Error: Could not connect to database.)
at vendor\nesbot\carbon\src\Carbon\CarbonTimeZone.php:109
105▕             return new static($tz->getName());
106▕         }
107▕
108▕         if (Carbon::isStrictModeEnabled()) {
➜ 109▕             throw new InvalidTimeZoneException('Unknown or bad timezone ('.($objectDump ?: $object).')');
110▕         }
111▕
112▕         return false;
113▕     }
1   vendor\nesbot\carbon\src\Carbon\Traits\Date.php:626
Carbon\CarbonTimeZone::instance()
2   vendor\nesbot\carbon\src\Carbon\Traits\Date.php:1631
Carbon\Carbon::safeCreateDateTimeZone()

Thank you for bringing this issue to our attention, and I apologize for any inconvenience you've experienced. I understand that you're encountering a problem similar to the one mentioned in the provided GitHub issue: https://github.com/abdelmjid-saber/SpladePanel/issues/1
Quick solution

Delete the .env file.
Try running composer install.

If the issue remains unresolved, update the project. The problem has been resolved in the latest update.


how to run the TonKeeper  Web and debugger

I build and success , it is nothing when I click the index

Hello @yoga1314
In case you would like to run web version locally,
please run this commands:
yarn build:web - from the root of project, it would build units packages
cd apps/web - navigate to web app
REACT_APP_LOCALES=en yarn start - run app on localhost


How does pricing works for Windows 8?

If I build a eindows 8 from a vhd then how does thr pricing works in this case??

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

ID: 82cde811-05df-c34c-6268-90e7c71e9c53
Version Independent ID: b7b97f26-733f-95d2-d62c-8fe6c68e0560
Content: Use Windows client images in Azure
Content Source: articles/virtual-machines/windows/client-images.md
Service: virtual-machines-windows
GitHub Login: @cynthn
Microsoft Alias: cynthn

Thanks for the question! We are investigating and will update you shortly.

@MicahMcKittrick-MSFT Hi micah, thanks for the response, much appreciate it.

@Anmolgan81 if you want to upload your own VHD then you will need to have your own license for the product. Assuming you have that, then when you upload to Azure then you will only pay for the resources you consume. Such as CPU and Storage.


Warning C4834 docs should clarify it is enabled in C++14 mode

The docs for this warning correctly state it was introduced in C++17, but we (and all other major compiler vendors) actually respect this attribute and will emit warnings related to it in C++14 mode too. The docs could use a clarifying note here.

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

ID: f7edfac9-2178-0e6a-d534-51ab9d3503ef
Version Independent ID: 8993c7ff-6dd3-ddc3-4739-eb9a7594a713
Content: Compiler warning (Level 1) C4834
Content Source: docs/error-messages/compiler-warnings/c4834.md
Product: visual-cpp
Technology: cpp-diagnostics
GitHub Login: @TylerMSFT
Microsoft Alias: twhitney

Done. It'll go live with a batch of other warning work that I'll have published by Monday.


add root build info to interprojectalert

Unexpanded

Expanded

@gchomatas

JS looks okay.

I don't feel that is a good idea to mix the top-level (root) build(s) within the immediate upstream builds. Even adding the extra info in parenthesis to designate the root build it is confusing.
I would imaging something like that:
Top level (or root) Builds that triggered this build:

build 1
buil 2

Immediate Upstream Builds:

build 3
build 4

Immediate Downstream Builds:

build 5
build 6

The wording in the titles is just indicative and to make clear what I mean, we can use any other more appropriate / more terse wording
/cc @markhazlewood

^ Updated

Update with new styling

@gchomatas I was looking at a different layout/design to better highlight the root in the IPB details that might address your concerns:

In the case where you are ON the repo build page that triggered the IPB, the text would just read
ModuleName [buildNumber]
Rather than
Repo::ModuleName [buildNumber]

🚸


Update url dependency

This updates the dev-dependency url to the current version v2.5 and revises the url parsing in the examples client and http3-client.

Merged, thanks @gierens!


frontend: Derive output column names

In PG
postgres=# select abs(1);
 abs
-----
   1
(1 row)

postgres=# select count(1);
 count
-------
     1
(1 row)

In risingwave
dev=> select abs(1);
 ?column?
----------
        1
(1 row)

dev=> select count(1);
 ?column?
----------
        1
(1 row)

related but seems different: https://github.com/singularity-data/risingwave/issues/3690

To clarify the difference from #3690: the field names mentioned in that issue is internal, while the output column name shown in this issue is part of SQL semantic. For historical context on how they were decoupled and derived independently, check #2465 and #1433

This affects the result comparison in regress tests (e.g. #5452)
PostgreSQL:
https://github.com/postgres/postgres/blob/86a4dc1e6f29d1992a2afa3fac1a0b0a6e84568c/src/backend/parser/parse_target.c#L1729
RisingWave:
https://github.com/risingwavelabs/risingwave/blob/4998ba94da6f85f1568fa36a399e21a5797b05b9/src/frontend/src/binder/select.rs#L163-L180


Consider replacing jquery with D3 for all graphics templates

http://blog.webkid.io/replacing-jquery-with-d3/

This is super easy for the graphic templates, but complicated by the awkward system we're using to load the parent.html JS dependencies. Need to refactor this before factoring out jquery.


New version: Bluebeam.Revu.21 version 21.1.0

Checklist for Pull Requests

[ ] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

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

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

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Installation failed with exit code -1978335226
2024-04-09 16:02:24.324 [CLI ] ShellExecute installer failed: 1603

(Automated response - build 876.)

@stephengillie this might need manual verification?

@wingetbot waivers Add Validation-Shell-Execute


Variable Substitution documentation does not explain how it works

The main doc page has a title "Understanding Tekton Pipelines" and under that, there is a link to Variable Substitutions.
I expect that link to explain how variable substitutions work.
From discussions on Slack, I now know that there is also:

arrays
star-operator to expand arrays (explained in specifying params)

Also

how it works within the script: field is not documented. It should be noted that variable substitution can be used within the middle of a script and is executed before (?)

that it can be expanded in steps.args list is not documented

It would also be good to describe how I can debug variable substitution? e.g. how I quickly can see to what values was inserted. (or maybe this is obvious?)

I expect those things to be explained on the docs, when I follow the doc-flow described initially.

Also "variable substitution" works as script-injection, I can make something like

and it works. But not everywhere. It would be also cool to find description for that in doc.

/remove-lifecycle stale


Fix #250: Ensure consistent documentation sidebar widths.

Summary
The documentation .content is currently set to flex-grow: 1, which interferes with the predefined sidebar widths. Removing this, and tweaking some overflow settings should resolve #250.
What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[x] Docs
[ ] Build-related changes
[ ] Other, please describe:

If changing the UI/CSS related code, please provide the before/after screenshot:
Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)

You have tested in the following browsers: (Providing a detailed version will be better.)

[ ] Chrome
[x] Firefox
[ ] Safari
[ ] Edge
[ ] IE

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature
[ ] Related documents have been updated
[ ] Related tests have been updated

To avoid wasting your time, it's best to open a feature request issue first and wait for approval before working on it.
Other information:

maybe just hide the rightbar when max-width: 1024px

Looks like there's quite a mess in the specification about overflowing tables, and you need a wrapper to do them properly. It would be nice to auto hide the right bar only when .content overflows, but not sure if there is a way to do that without JavaScript.


Remove separate release notes

Lets get this merged! I added the Auto CA identity role attributes capability and need to add it to this new file!

Lets get this merged! I added the Auto CA identity role attributes capability and need to add it to this new file!

New file is already in release-0.13, so you should be good to go. This just removes all the other files.


Elemental ui extension 1.10 should not be installable in rancher 2.7.1

As discussed yesterday with @aalves08, I was able to install elemental-ui extension 1.1.0 in rancher 2.7.1:
helm upgrade --install rancher rancher-latest/rancher --namespace cattle-system --create-namespace --set replicas=1 --set hostname=rancher.<IP_ADDRESS>.sslip.io --set bootstrapPassword=password --version 2.7.1

image: rancher/ui-plugin-operator:v0.1.0

And things like elemental-icon are broken:

What I can do is limit the rancher version for this extension, with the side effect that it won't even show on the extensions screen that there's a 1.1.0 available for upgrade. Shall I do that?

The mechanism of https://github.com/rancher/dashboard/issues/9132 should help with this

Closing in favor of https://github.com/rancher/elemental-ui/issues/225


[FromForm] attributed model is not showed in swagger.json.components.schemas

I need to read properties from components.schemas, but   [FromForm]FormExample form is not recognized and not showed in schema.
V 5.5.0

Form schema's are defined inline (as opposed to being a reference to shared schema in components.schemas) and are therefore never written to components.schema.

@domaindrivendev we also noticed that Document Filters are ignored for schemas that are defined inline. Is there a way to modify inline schemas?

@domaindrivendev any updates?

Due to the way in which form fields are surfaced by ApiExplorer (the API metadata layer that ships with ASP.NET Core), form Schema's need to be treated as a special case in Swashbuckle, and are generated in a slightly different way to to regular payload schemas.
As a result, they are a) always generated inline within the relevent Operation definition rather components.schemas and b) never passed through the schema filters.
If you need to inspect those schemas, look in the relevent Operation definition instead. If you need to modify, use an IOperationFilter instead of an ISchemaFilter.