Can't access database in SQL lab

Bug description
I have authorized the permissions of sample.main to this user, but this account still cannot be queried in sql_lab
How to reproduce the bug

Create a role testRole, and grant schema access on [examples].[main]
Create an account and give roles Gamma, sql_lab, testRole
Use the new account to query sql SELECT * FROM main.bart_lines
An error occurred

Screenshots/recordings

Superset version
3.1.3
Python version
3.9
Node version
16
Browser
Chrome
Additional context
No response
Checklist

[X] I have searched Superset docs and Slack and didn't find a solution to my problem.
[ ] I have searched the GitHub issue tracker and didn't find a similar bug report.
[ ] I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.

I've tried your steps but still can't solve my problem @dosu.

It sounds like you're encountering a permissions issue or a problem with database connectivity or table existence. Here are some targeted steps to troubleshoot and hopefully resolve the issue:

Verify Role and Permissions: Ensure the testRole you created and assigned to the user indeed has the necessary permissions for SQL Lab and the examples.main schema. Specifically, check for permissions related to database and datasource access, such as DATABASE_SECURITY_ACCESS_ERROR or DATASOURCE_SECURITY_ACCESS_ERROR.
Check Database Permissions: Confirm that the database access is explicitly granted to the user's role. This is crucial for non-Admin users and involves adding the database to the list of permitted databases for the role.
Review Custom Roles: If you've created custom roles, make sure they include the correct permissions for SQL Lab and database access. It's recommended to create new roles with the necessary permissions rather than altering base roles.
Database Connectivity: Ensure there's no issue with the database connection itself. Superset uses SQLAlchemy engines for database connections, and issues here could lead to the errors you're experiencing.
Table Existence: Verify that the table main.part_lines actually exists and that the user has permissions to access it. The error message suggests there might be an issue with accessing this specific table.

If these steps don't resolve the issue, providing the exact error message from the Superset logs could offer more insight into the problem.
Details
To continue the conversation, mention @dosu.

DB engine Error
Failed to execute query 'SELECT * FROM main.bart_l:ines ': can not access the query
@dosu.

DB engine Error
Failed to execute query 'SELECT * FROM main.bart_l:ines ': can not access the query

I encountered the same problem and tried to print some logs to investigate the issue. The code is as follows:
superset/sql_parse.py
class ParsedQuery:
def init(
self,
sql_statement: str,
strip_comments: bool = False,
engine: str | None = None,
):
if strip_comments:
sql_statement = sqlparse.format(sql_statement, strip_comments=True)
    self.sql: str = sql_statement
    self._dialect = SQLGLOT_DIALECTS.get(engine) if engine else None
    self._tables: set[Table] = set()
    self._alias_names: set[str] = set()
    self._limit: int | None = None

    logger.debug("Parsing with sqlparse statement: %s", self.sql)
    self._parsed = sqlparse.parse(self.stripped())
    for statement in self._parsed:
        self._limit = _extract_limit_from_query(statement)

def stripped(self) -> str:
return self.sql.strip(" \t\r\n;")
I try to  print log:
logger-info (self.sql)
The output is
Template (body=[Output (nodes=[TemplateData (data='SELECT * from xxxx. xxxx limit 100; ')])
The code will not continue to execute until this point
return self.sql.strip(" \t\r\n;")

Awesome! @leoguo1024 fixed my problem as well. I was experiencing the same issue.

Hopefully we can merge that PR soon and close out this issue. I think the PR has a couple of comments/requests that need to be addressed by @leoguo1024, and then we'll be all good!

@rusackas @john-bodley  This is still an issue for me. Is there anything I can do to help?


[LIBCLOUD-951] Add possibility to define hostname to UpCloud driver

UpCloud driver to support defining hostname
Description
Added ex_hostname to UpcloudDriver create_node method to define the hostname.
Status

done, ready for review

Checklist (tick everything that applies)

[x] Code linting (required, can be done after the PR checks)
[x] Documentation
[x] Tests
[x] ICLA (required for bigger changes)

Codecov Report

Merging #1123 into trunk will decrease coverage by <.01%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           trunk   #1123      +/-   ##
========================================
- Coverage   85.4%   85.4%   -0.01%
========================================
  Files        346     346
  Lines      66191   66200       +9
  Branches    5892    5892
========================================
+ Hits       56528   56535       +7
- Misses      7265    7267       +2
  Partials    2398    2398

Impacted Files
Coverage Δ

libcloud/compute/drivers/upcloud.py
95.95% <ø> (ø)
:arrow_up:

libcloud/test/common/test_upcloud.py
100% <100%> (ø)
:arrow_up:

libcloud/common/upcloud.py
90.47% <100%> (ø)
:arrow_up:

libcloud/test/compute/test_upcloud.py
92.19% <100%> (ø)
:arrow_up:

libcloud/test/compute/test_ec2.py
97.74% <0%> (-0.17%)
:arrow_down:

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 ce5d3be...ba34061. Read the comment docs.

Close & reopen to rebuild travis

It looks like there's a flaky test in test_ec2.py that makes the coverage go down here, but that's not your fault. Thank you @mlackman, merged in trunk! 👍

Thanks. By the way should I close the JIRA issue. It seems that the accepting PR does not close the JIRA issue.

Yes, please close the JIRA issue as I can't do this myself.


Applying on EEG Motor Movement/Imagery Dataset

I am trying to use the network to classify Motor Movement but this network works very well on the author's example data but doesn't work on new data.
Actually, I am using the "EEG Motor Movement/Imagery Dataset" (https://www.physionet.org/content/eegmmidb/1.0.0/) and it's a popular dataset, other people arrived at ~80 % accuracy and I got this accuracy with brain decode (CNN - https://tntlfreiburg.github.io/braindecode/). Now, I am trying to classify this data using EEGLearn (RNN-CNN https://github.com/YangWangsky/tf_EEGLearn) but I'm still on the chance level. Do you have any suggestions?

The feature EEGLearn used here is PSD. It is a frequency domain feature, which I think is not suitable for MI. It seems something like CSP works well on MI. It means space information is better for MI. So, maybe you can try other features which work for MI


Merge aps-iot-extensions code into this repo

Currently this sample app uses another GitHub repo (https://github.com/autodesk-platform-services/aps-iot-extensions) which provides a set of wrappers around the DataViz extension functionality. Let's move that code into this repository.

Done in https://github.com/autodesk-platform-services/aps-iot-extensions-demo/commit/187e01d2e7be0b7d196c0c8749d30b1ac2fd47fe.


Migrate this to a LR plugin

LR plugins are written in Lua: http://wwwimages.adobe.com/content/dam/Adobe/en/devnet/photoshoplightroom/pdfs/lr6/lightroom-sdk-guide.pdf
The basic strategy would be to write a plugin that would watch for recently imported pictures, ideally RAF files, that would then run exiftool on said pictures and set the rating according to the rating found in RAF.

we could use this library: https://github.com/EvandroLG/lua-watcher to monitor

We can also use LrTasks to monitor a specified directory and continuously poll

@avdeveloper I think a good solution could be if it was possible to apply the plugin during import, since the problem we are trying to solve is that the in-camera fuji ratings are not recognised directly out of camera. And fixing those ratings could be a one time "fix" upon import, after which the ratings would be saved in the commonly used standard.
To fix the already existing images, there could maybe be an option to "fix the ratings" for the whole library, but then i can imagine running into conflicts if ratings had been added later in Lightroom. Something to think about.
I haven't worked with Lua, or created plugins for LR for that matter, so at this point I'm not sure what's possible. This is something I'm planning to investigate now and will come back to this once it's a bit more clear how the plugin could be made and with how much effort.

Right. It would be nice if we didn't have to monitor the directories by
continuously polling. I haven't coded in Lua either but the syntax doesn't
seem to have a steep learning curve. I think Adobe marks photos with their
own metadata so we could probably use that to find out which photos need to
be scanned for ratings.
On Sat, Feb 18, 2017, 11:11 AM aldis-ameriks<EMAIL_ADDRESS>wrote:

@avdeveloper https://github.com/avdeveloper I think a good solution
could be if it was possible to apply the plugin during import, since the
problem we are trying to solve is that the in-camera fuji ratings are not
recognised directly out of camera. And fixing those ratings could be a one
time "fix" upon import, after which the ratings would be saved in the
commonly used standard.
To fix the already existing images, there could maybe be an option to "fix
the ratings" for the whole library, but then i can imagine running into
conflicts if ratings had been added later in Lightroom. Something to think
about.
I haven't worked with Lua, or created plugins for LR for that matter, so
at this point I'm not sure what's possible. This is something I'm planning
to investigate now and will come back to this once it's a bit more clear
how the plugin could be made and with how much effort.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aldis-ameriks/IncameraFujiRatings/issues/1#issuecomment-280855297,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAYyOmWTjZY8uXCI20kSkBF_QHeLsDzHks5rdxgtgaJpZM4MFFJC
.

--
Albert Villaroman
Front-End Developer at The Social Edge http://thesocialedge.co/
http://goog_1479753109/
<EMAIL_ADDRESS>albertvillaroman http://www.linkedin.com/in/albertvillaroman
@arian_santi https://twitter.com/arian_santi
@ http://instagram.com/arian_santiarian_santi
http://instagram.com/arian_santi

for reference: http://u88.n24.queensu.ca/exiftool/forum/index.php/topic,5509.msg26696.html#msg26696

Here is a short overview of current progress with the plugin development and things that still need to be done:

Plugin currently works only on Mac, due to the nature how ratings are read from files (nested bash statements obviously won't work on Windows). The reason why it's necessary to nest the command, is because I'm unable to capture the output of the command inside the script, i.e. I cannot get the actual value of rating from the executed command inside the plugin.
After spending good time to investigate how this could be done, I've concluded that the only way is to write the output of the exiftool to a temp file and then reading the value from that temp file inside the script. (How nasty).

The path of exiftool should be configurable, currently I've hardcoded separate values for Mac and Windows, but it would be nice if i could specify the path in the Plug-in Manager inside Lightroom. It's possible to create that kind of settings panel.

Currently the plugin understands only ".JPG", ".jpg", ".RAF", ".raf" files. Not sure why anyone would have their extensions in a different format, but it would be nice to make the logic case in-sensitive.

When Fuji RAW file is present, the metadata is written in a sidecar file ".xmp". Should first check if there is an already existing sidecar file with extension in different letter case, e.g. ".XMP" and write to that file. Checking that existing sidecar file is related to point 3.

Show some kind of progress or notification. Currently everything is happening silently and there will be no notification if there are errors or when the plugin has finished executing.

Honestly, I have no idea if anyone actually needs this plugin, but at this point I'm just doing it because.
My current plan is also to go through the point in the order I've just described, starting on fixing it to work on Windows.

Thanks so much for working on this plugin. I'll fork this and tackle point #1 and see if there is a way.

#5 LrProgressScope

The plugin is now working on Windows too. I've tested it and seems to be working same as on Mac.

@avdeveloper I'm closing this issue because the first version of the plugin is done. I'll create separate issues for making the improvements.


Variable length digit recognition with CNN?

I'd like to use Keras for recognizing handwritten digit sequences with CNN but I cannot find any clue how to approach the problem. The segmentation of the image by individual digits is misleading as there is no reliable way to do that (sometimes the bounding boxes are overlapping, sometimes multiple digits are written by one continuous line etc.). I'm considering to feed the whole sequence as a single image to a CNN but I don't know what would be the last layer here, this not being a classification with fix outputs. Neither seq-to-seq seems to be a good idea, having no real "temporal" sequence here. So I'm clueless a bit...

You may want to look at this article on Connectionist Temporal Classification and a nice post on the subject.


Codeunit 5766 new event on procedure "Code"()

Hi,
We need a new event on codeunit 5766 procedure  "Code"() :
....
            PostedWhseRcptHeader.SetRange("Whse. Receipt No.", "No.");
            PostedWhseRcptHeader.SetRange("Location Code", "Location Code");
            PostedWhseRcptHeader.FindLast;
            WarehouseDocumentPrint.PrintPostedRcptHeader(PostedWhseRcptHeader);
            Message(Text001);

            Clear(WhsePostReceipt);

            // > New event
            OnAfterCode(WhseReceiptLine);
            // < New event

....
[IntegrationEvent(false, false)]
    local procedure OnAfterCode(var WarehouseReceiptLine: Record "Warehouse Receipt Line")
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Referencing a keyvault secret in a bicep template will result in an attempted delete of that secret on unmanage

Describe the bug
Given a template were a keyVault secret is referenced via keyVault.getSecret('x')
When trying to create a new deployment where the secret is no longer referenced with --action-on-unmanage "deleteResources"
The deployment stack will attempt to delete the referenced secrets (and fail if the principal does not have delete permissions)
To Reproduce
Given the following template:
Using https://github.com/Azure-Samples/azd-starter-bicep as a base

in the root create a bicep file with the following contents

param keyvaultName string
param secretName string

resource keyVault 'Microsoft.KeyVault/vaults@2023-07-01' existing = {
  name: keyvaultName
}

module sqlServer './infra/core/database/sqlserver/sqlserver.bicep' = {
  name: '${deployment().name}-sqlServer'
  params: {
    location: location.name
    tags: tags
    name: 'dps-kv-reference-bug-test'
    databaseName: 'testdb'
    keyVaultName: keyVault.name
    appUserPassword: keyVault.getSecret('sql-user-password')
    sqlAdminPassword: keyVault.getSecret('sql-admin-password')
  }
}

Deploy the bicep in a deployment stack
comment out the sql module
Redeploy bicep with --action-on-unmanage "deleteResources"

The deployment stack will attempt to delete the referenced secrets (and fail if the principal does not have delete permissions)
Expected behavior
When doing the second deployment I would expect that the deployment stack would not try to delete referenced secrets
Screenshots
If applicable, add screenshots to help explain your problem.
Repro Environment
Host OS: Linux Ubuntu 22.04 (azure devops hosted agent)
Azure CLI Version: 2.65.0
Server Debugging Information
Correlation ID: 40fc9386-27e8-4668-a005-743231e42f03
Tenant ID:
Timestamp of issue (please include time zone): 2024-10-21T09:56:23Z
Data Center (eg, West Central US, West Europe): West Europe
Additional context
Add any other context about the problem here.

Hi @azcloudfarmer I think that this issue is distinct from #142. I'm not creating secrets in my stack.
I'm simply raising a reference to a secret.
Just like the fact that a resource with the existing keyword does not get into a deployment stack,
for example
resource keyVault 'Microsoft.KeyVault/vaults@2023-07-01' existing = {
  name: keyvaultName
}

I would expect that a keyVault.getSecret('sql-user-password') would als not add the secret to the stack. This would translate to the following in arm:
"appUserPassword": {
    "reference": {
        "keyVault": {
            "id": "[resourceId('Microsoft.KeyVault/vaults', 'sql-user-password')]"
        },
        "secretName": "permissions-sql-user-password"
    }
}

So I think I should rephrase the bug:
When referencing a exsiting keyvault secret, it gets mistakenly added to the tracked resources.

Nevermind, I feel really dumb right now. I see now that the template I've been using creates a new secret (which it shouldn't).
Thanks for looking into it. I will close this issue.


How do I perform a custom validation against an input?

Thanks for the great library!
Can you please provide me with an example on how to validate an input of type email with a regular expression when the form is submitted?
I noticed that a field's options can contain an error attribute whose value is a function. However, it seems the function is called on a change event and that the context of the function isn't bound to anything in particular.
I'm not sure how to validate a specific field against a regular expression only when the form is submitted or ideally, when the field triggers a blur event.

Hi, thanks.

Can you please provide me with an example on how to validate an input of type email with a regular expression

// use tcomb's type combinators (https://github.com/gcanti/tcomb)
var Email = t.subtype(t.Str, function (s) {
  return /^foo/.test(s);
});

var Type = t.struct({
  email: Email
});

<Form type={Type} />

I noticed that a field's options can contain an error attribute whose value is a function

The error option can be a string, a JSX or a function
var Email = t.subtype(t.Str, function (s) {
  return /^foo/.test(s);
});

var Type = t.struct({
  email: Email
});

var options = {
  fields: {
    email: {
      //error: 'my custom error message', // <-- fixed custom error message
      error: function (value) { // triggered when the field value changes
        return 'my custom error message: ' + value; // <-- custom error message based on current field value
      }
    }
  }
};

<Form type={Type} options={options} />

Thanks for taking the time to explain this. It's extremely helpful to someone who is new to your libraries and it might make good material for the docs.
I'm not sure I would want the error function to fire onChange. I think in most of our use cases, we would want to fire it on form submit or perhaps on blur. However, if someone is tackling that other issue addressing other event binding, this probably won't be an issue.

@gcanti I'm struggling with something related to this..
If the user submits, the form hasErrors is true and the message is displayed. The thing is that while user fixes the email the error function is called each time the input is changed. Once the email is correct and error function doesn't return 'Invalid email' the input is still marked as error and shown with a red border which might be confusing (the email is valid, no error message but still in red).
How can I trigger field validation again? Am I doing something wrong?

I fixed the order of onChange and setState in all the components (thanks for pointing out).
Now you can call safely getValue in the onChange handler:
var Email = t.subtype(t.Str, function (s) {
  return /^foo/.test(s);
});

var Type = t.struct({
  email: Email
});

var options = {
  fields: {
    email: {
      error: 'Invalid email' // <- fixed message
    }
  }
};

var App = React.createClass({
  getValue: function () {
    var value = this.refs.form.getValue();
    if (value) {
      console.log(value);
    }
  },
  onChange: function (value) {
    this.refs.form.getValue(); // <- validate on every change
  },
  render: function() {
    return (
      <div>
        <Form ref="form"
          type={Type}
          options={options}
          onChange={this.onChange} />
        <button className="btn btn-primary" onClick={this.getValue}>getValue</button>
      </div>
    );
  }
});

React.render(<App />, document.getElementById('app'));

published v0.4.3 with the fix

Awesome. Works great.
Thanks @gcanti have a beer on me @ChangeTip.

Hi @gcanti, @AlanVerbner sent you a Bitcoin tip worth a beer (12,740 bits/$3.50), and I'm here to deliver it ➔ collect your tip.
Learn more about ChangeTip

Thanks @AlanVerbner, I appreciate ;-)


Support string assertions resulting in non-empty-string

Not sure if this is the best solution tbh., let me know if you have any other ideas.
A few string assertions are still left like e.g. contains or startsWith but they should have dedicated resolvers IMO to check if the needle is a non-empty-string or not. I'd look at them in a dedicated PR afterwards.

Thank you! :)

Oh, I'm pretty sure those are not supported via nullOr* or all* assertions.. I'll fix that later


fix: set revision based on PUPPETEER_PRODUCT

Whoops, this should have been part of #5640

@jackfranklin PTAL since this touches CI config

LGTM, thanks!


inspect all build files simultaneously with vifm

Inspect the possibility of using of vifm (or similar tools) as an alternative to standard editors to speed up PKGBUILDs review.
This is implemented in aurutils by @AladW.

How does he use it? I've looked his code but it is not clear for me. The file manager ranger could also be useful depending on your idea about this feature.

Not sure what's not clear exactly but it just uses find to get a list of build directories and the files they contain, then passes that to vifm on stdin. The reason for vifm is that it's the only curses file manager that supports stdin. You can do something similar with ranger or mc, but you'd either have create symlinks to all current build files in some temporary directory, or be cluttered with files unrelated to the current operation in AURDEST.

After having a closer look, I'm kinda skeptical about this feature. On one hand, it's incompatible with the diff viewing feature, on the other hand it's only useful for the vifm utility. I'm still interested in speeding up the review process, but I don't see vifm very relevant here.
Closing as wontfix.

FWIW, you can view diffs in vifm by saving them in a temporary folder (at least that's what I do). I can't think of a much better way to not prompt 100 times, but ping me if you do. ;P

FWIW, you can view diffs in vifm by saving them in a temporary folder (at least that's what I do).

This sounds ugly to me. The reason I waited AUR 4 before implement the diff feature was exactly for avoiding temp dir and using git directly.

I can't think of a much better way to not prompt 100 times, but ping me if you do. ;P

Hopefully doesn't happen that often in practice. I can only think of git submodule as an alternative but that also sounds overengineered.


Elevator music for non-subscribers

When a 30 second sample stops playing (for a non unlimited subscriber), start looping the 30 second version of http://www.rdio.com/artist/The_Offspring/album/Ixnay_On_The_Hombre_2/track/Intermission/ until the next track starts.
Better than silence!

And add some indication about why it is like that and try to up-sell.


Refactor

Codecov Report

Merging #614 (897b492) into main (e296c4f) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #614   +/-   ##
=======================================
  Coverage   84.82%   84.82%
=======================================
  Files         302      302
  Lines       26943    26943
=======================================
  Hits        22855    22855
  Misses       4088     4088

Impacted Files
Coverage Δ

lib/ffi-macro/src/function.rs
94.33% <100.00%> (ø)

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 e296c4f...897b492. Read the comment docs.


[Feature Request] Improve ESLint Warning Box Visibility for Blue Themes

Thank you for the excellent work migrating this theme to Zed. I've noticed that in some cases, the background color changes to #0090FF, making the text difficult to read. I tried setting [conflict] to null, but the issue persists. Your attention to resolving this would be appreciated.

Thanks for raising this! I will try and get to it soon.

I have submitted a PR with updates that should fix this issue. I think for your case you want to override warning.background but this should be fixed in v0.1.1 of the theme. If/when it is updated and all is fixed could you close this issue?
Thanks again!

Thanks a lot. This works. I found it's related to the hint.background params.


languages are collected from wrong object

The description on how to use language links seems wrong to me. As far as I can tell, it would need to refer to $site->kirby()->languages() instead of $site->languages() (the latter returns null in my installation).

AFAIK this was a change from Kirby 2 to Kirby 3 and is probably a leftover in the docs. This should be kirby()->languages().


Add  Better Snippets

HNY! I've just made a new productivity extension.
Name of the extension you are adding
Better Snippets
Why do you think this extension is awesome?
Why not? It let's you create more powerful snippets, and I think even more powerful than live templates. I don't expect from other extensions web support. I'll probably add tests if I receive more feedback
You can also look at example usage
Make sure that:

[ ] Screenshot/GIF included (to demonstrate the plugin functionality) Not sure what screenshot I should include, everybody knows how suggestion widget looks like.

[X] ToC updated

Other changes are from VSC formatter. I don't it's worth to add introduce them as separate PR, so leaving them just here.

You can look at my twitter for more gifs demonstrating how cool this extension is!

@viatsko thoughts?

We are trying to have more widely adopted extensions on the list, 35 installations wouldn't be enough.


GUI: Save (yes/no) still confusing.

Do not prompt the user to type confirmation in the GUI. Buttons exist in a GUI

This was already finished, just wasn't fully updated.


[cherry-pick][BugFix] Fix multi nest cte inline error (#9467)

What type of PR is this：

[ ] bug
[ ] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

run starrocks_fe_unittest


Interaction of Create/VS/Clockwork and JSG

Hi,
I'm experiencing a crash when using JustStargateMod (JSG) mods together with the Valkyrien Skies and Clockwork. Specifically, the game crashes when I try to assemble Stargate on a platform that has its own physics enabled by these mods.
Here are the details:

Valkyrien Skies version: 2.3.0-beta.5
Clockwork version: 1.20.1-0.1.16-forge-b3b22e39fe
Architectury: 9.2.14
Kotlin Fro Forge: 4.5.0
JustStargateMod version: <IP_ADDRESS>-Beta
Minecraft version: 1.20.1

Crash log snippet:
java.lang.ArrayIndexOutOfBoundsException: Index 4194596 out of bounds for length 315
at net.minecraft.world.level.levelgen.Aquifer$NoiseBasedAquifer.m_188445_(Aquifer.java:497)
The crash seems to happen when the physics for the platform are activated and I attempt to assemble the Stargate.
I guess when the physics engine is activated (the platform becomes mobile), the JSG mod cannot correctly determine the coordinates of the blocks or manage the objects.
crash-2024-07-21_13.16.26-server.txt
Thank you for your help!
Best regards,
Alex

Well its actualy right - because address system is based on x y z of baseblock if stargate - when you move it without unmarge gate - gate will have more addreses

Okay, my next question might seem stupid (it really is, I think), but even if we assume that the assembled ship from VS can freely move in space, wouldn't each of its points have x y z coordinates, albeit with a few decimal places?
I mean, even if we assume that the gates on the assembled ship are also assembled (let's take gate_1 as the gate on the ship and gate_2 as the gate on the stationary base), and as the ship moves through space the address of gate_1 can constantly change, while gate_2 remains static, wouldn't that make traveling from gate_1 to gate_2 impossible? (and from gate_2 to gate_1 without updating the address of gate_1 indeed not possible)
And even if we assume that building gates on the assembled ship would be possible, how difficult would it be to implement dynamic coordinate changes for the gates?
Thank you in advance for your attention to my amateur questions :D

We already tried that with different mod on 1.12.2 but it caused more cons then pros. Basicly gate states werent updated. So its hard. Also from some research of VS - this mod doesnt work well on servers as well... I know that it will be great to have gate here but its not possible - at least not now or nearest future.

I have experimented with this setup in the past.

The crash has something to do with the World generation. Valkyrien Skies places those Ships somewhere way outside the World. When the Gate calculates the Temperature, it checks the blocks around itself. Somewhere in this process, something doesn't work correctly and it's trying to generate some chunks there which crashes the World generation. This can be (temporarily) fixed by using a different world generation Algorithm. Superflat worlds work and some world gen Mods also work.
The DHD doesn't correctly recognize its position and rotation. It gets pleased wrong, The back (for changing the upgrades) changes and you can't select the Symbols.
Dialing the Gate often fails, it works rarely with the DHD but flawlessly with the Admin tool.

All in all, technical problems should be easily fixable. I know it is not a Priority, but the idea of having working Stargates on flying Ships is exciting.

I have experimented with this setup in the past.

The crash has something to do with the World generation. Valkyrien Skies places those Ships somewhere way outside the World. When the Gate calculates the Temperature, it checks the blocks around itself. Somewhere in this process, something doesn't work correctly and it's trying to generate some chunks there which crashes the World generation. This can be (temporarily) fixed by using a different world generation Algorithm. Superflat worlds work and some world gen Mods also work.
The DHD doesn't correctly recognize its position and rotation. It gets pleased wrong, The back (for changing the upgrades) changes and you can't select the Symbols.
Dialing the Gate often fails, it works rarely with the DHD but flawlessly with the Admin tool.

All in all, technical problems should be easily fixable. I know it is not a Priority, but the idea of having working Stargates on flying Ships is exciting.

Hello,
It is looking that you still not understand how JSG works - so again address of gate is based on base block position in world - ON CORDINATED X Y Z so everytime you **move merged ** gate (or even just placed base block) this gate get NEW address!
We already tried to solve this issue and make compatibility with space mod that have same mechanic on 1.12.2 - without success. It only caused bugs like not closing gates, changing address of gates after every update...
And before you say anything - I just test test this mod - you dont just move in world forward, backward, left, right up and down, but you can also move diagonally.... and all this movement is in world - with CHANGING X Y Z coordinates!
So that will cause not just one problem but thousand of problems...
I can say that VS cant be compatible with JSG.

I know how JSG  handles coordinates, but I think you have a misconception about how VS handles coordinates. The Blocks used in the Ships are stored a couple of million blocks away in the Shipyard. The Mod then Projekts the Ship into the World. that also means when the Ship is moved the coordinates from the Stargate don't change. You can see that if you lock at the block with F3 while moving the Ship. The only real problems occur when you build the Gate in the world and turn it into a Ship or if you build it onto a Ship and place it back in the world.


Adds logic to load graphql file on the basis of the "source" attribute in the  tag

#796
This is a first implementation.
When interpreting the  custom block in a Vue component it now also checks for an attribute called "source" and if that exists it will load the content of that file and use that as the GraphQL query.
I think there still can be some improvements:

there might be a better mechanism to get the file content using webpack
currently when there is a "source" attribute and there would be content for the  tag the logic would priotize the attribute
error handling when the file cannot be found
I would prefer to have an e2e test using that to ensure this functionality

I had some problems executing the e2e tests.
Is there something I need to be aware of if I run  npm run test:e2e ?

Any news on this pull request? When is it going to be merged?


Fixed commenting by using issue comment instead of review comment

Current dependencies on/for this PR:

main

PR #1589   👈

This comment was auto-generated by Graphite.


Add init_resource_with_descriptor

What problem does this solve or what need does it fill?
I'm making a scripting language and I would like to be able define resources from it dynamically. I can already do this with components by calling init_component_with_descriptor with a ComponentDescriptor::<Box<dyn PartialReflect>> but there is no equivalent for resources.
What solution would you like?
Add init_component_with_descriptor

takes in ComponentDescriptor
a unique ComponentId should be returned for each call

What alternative(s) have you considered?
I have a Wrapper(Box<dyn PartialReflect>) and I call let id = init_resource::<Wrapper>(); remove_resource_by_id(id) but that doesn't allow for multiple resources with the same underlying type.

Am I misunderstanding something here? Can't you just use register_component_with_descriptor in conjunction with insert_resource_by_id to achieve this already? Shouldn't that allow for multiple resources with the same type?
I saw this test when working on a PR for this, which demonstrates what I mean.
I'm not super familiar with ECS internals, so apologies in advance 😅.

Am I misunderstanding something here?

Nope, I just didn't look hard enough. I thought that because there were some differences between component/resource registration that there were separate id namespaces for resources and components...


feat(ENG-2494): adds tenant members and invitations functions

Description

adds tenant members and invitations functions

Testing required outside of automated testing?

[x] Not Applicable

Screenshots (if appropriate):

[x] Not Applicable

Rollback / Rollforward Procedure

[x] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

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

v1.27.0
GitHub release

Your semantic-release bot :package::rocket:


Visual script : exported release game crash on array append/push front/push back

Godot version:
v3.1.dev.calinou.e522baf
OS/device including version:
windows 10 x64
Issue description:
In visual script: when append any value (ex: vector2D/bool..) to an array variable , the debug (or debug export) doesn't show any error ,but when export as release it crashes.
also tested with push back / push front with the same release crash.
https://drive.google.com/open?id=1JC9xjtnqLmt7KYMFoxwK8OSqZ1rrVD9B
Steps to produce:
create visual script >> create function >> create array variable >> drag and drop Array.Append node >> create custom variable (vector2d/bool..) >> link nodes so when call the function it append the vector2d to the array >>export to debug : no error | export to release : it crash

array_run.zip
Export release will crash


Select field is empty if there are some pre-selected Items in the component

Reproduction link
https://stackblitz.com/edit/angular-fxhfwu?file=src/app/app.component.ts
Steps to reproduce

Run the demo app from docs: "Hide already selected options in the dropdown"
Initialize selected items array with some values: listOfSelectedValue: string[] = ['Apples', 'Bananas'];

What is expected?
Pre-selected Items should be displayed as tags in the Select field (i.e. 'Apples', 'Bananas').
What is actually happening?
The Select field is empty if there is pre-selected Items in the component.
This might be the case if we are using "nz-select" as a form field and we want to patch form with some previously saved data during Form initialization.

Environment
Info

ng-zorro-antd
8.2.0

Browser
Any browser

Same question.
I have a nz-select in a nz-dropdownMenu.When close dropdown and reopen it.Pre-selected Items are missing.

Same question.
I have a nz-select in a nz-dropdownMenu.When close dropdown and reopen it.Pre-selected Items are missing.

plz check https://github.com/NG-ZORRO/ng-zorro-antd/issues/1455
it is the bug of angular

@shohrukh92 plz check this https://github.com/NG-ZORRO/ng-zorro-antd/issues/4000#issue-480195237

@vthinkxie I've checked but it seems like #4000 is related to something different. From what I understand the tags in the Select field is generated based on items in the dropdown list itself. So when we want to hide selected items in the list, this issue is happen.

@shohrukh92 it should be https://github.com/NG-ZORRO/ng-zorro-antd/issues/1455

Hi @shohrukh92
we will provide nzHide in next version, use it instead of *ngIf, everything will work fine


Add two new features to the Sensor peripheral

#1:
A new lua-accessible function 'sonicScanTarget(int,int,int)', which is simmilar to 'sonicScan()' except it only returns data for a single block.
#2:
The data returned by 'sonicScan()' and 'sonicScanTarget(int,int,int)' now includes map-color information about blocks. The color is provided as a single hexadecimal character as listed on the CC Wiki.

Looks good, I will merge tomorrow.

The new commits I've added to this pull request depend on OpenModsLib#63.
Please let me know if you would like me to make any more changes.


chore: Updating dependencies

Why?
There's the need to update the fraction package to version 3.0.0 (latest).
What?
I have updated the pubspec.yaml file and added a new entry in the CHANGELOG.ms.
Types of Changes

Dependencies update

Checklist

[x] I have provided a description of the proposed changes.
[x] I added unit tests for all relevant code.
[x] I added golden tests for all UI changes.
[x] I added documentation for all relevant code.

Codecov Report

Merging #15 (9b9f806) into master (dc6a6c5) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #15   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           44        44
  Lines         1600      1600
=========================================
  Hits          1600      1600

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 dc6a6c5...9b9f806. Read the comment docs.


0x6b051dcd9ca5aa3AC5C066302a45cfeE2E2f596d

hello you still have nothing on the "commit" submitting a logo on the comments does not allow us to merge anything. Please correct
A correct PR looks like this: https://github.com/traderjoe-xyz/joe-tokenlists/pull/1367

Closing due to inactivity and incorrectness


Add inbuilt option for tapered flange beams, & tapered flange channels

Describe the solution you'd like
Add a feature to generate properties for tapered flange beams and tapered flange channels. Often these are older sections, which are harder to source printed section properties for, so would seem like a useful addition to the capabilities of your programme.
Older beams often seem to be defined in the following manner:-
root radii
flange radii
flange angle
flange thickness at midpoint

or similarly for channels:-

taken from
Dorman Long 1906 Handbook.pdf
Though I note the US AISC S Shapes and tapered channel shapes are defined slightly differently (and not too clear to me what the intended flange angle and radii are!):-

Actions:

Implement TFB and TFC as per the British definition

Implemented in 8e35ec5.


Stored plans on GCP not having unique IDs

It appears that stored plans don't have unique IDs (see attached screenshot)

Raised by @fleroux514

If run digger plan , which succeeds and  stores the plan file on GCP bucket, than append changes to the PR with errors and the plan now fails, is the original plan file deleted from the bucket?

... that would become another blocker ..  since digger apply by mistake would apply that original plan file which is not up to date

More from @fleroux514 :

Don't think having unique IDs is the issue. When digger is in the process of running a plan, it should start by deleting the existing one..  Think that should do it and be a quick fix.

I've been doing a lot of debugging/fixing on my own time lately and would appreciate if you could confirm the issue and fix it before I get to testing the latest release again. Think its a basic issue that would affect anyone, and not something specific for my needs.. Adding the PR number to the planfile name would probably be a good idea also..


Fixing squid S1217  Thread.run() and Runnable.run() should not be called directly

This pull request is focused on resolving occurrences of Sonar rule
squid:S1217 - “ Thread.run() and Runnable.run() should not be called directly”.
You can find more information about the issue here:
https://dev.eclipse.org/sonar/rules/show/squid:S1217
Please let me know if you have any questions.
Fevzi Ozgul

I don't think blindly "fixing" things reported by static analysis is very helpful. This particular just breaks things.


Rework almost everything

Breaking changes!!
See commit comments for details

@alexanderjanke Thanks again for all the help!


Can't use Material3 Card

I can see Card, ElevatedCard and OutlinedCard under CardKt.class in the material3 library but I can't access any of them even though they're public.
I'm using Compose 1.1.0 and the material3 library version is 1.0.0-alpha05

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Add mps-cpu support

Summary
・Adapted stablediffusion to work on M1 mac(mps).
・ Created an automatic device detection system for #147
Feature
・Support mps for txt2img.py, img2img.py, gradio and streamlit
・Add mac config(v2-inference-v-mac.yaml)
Of course, it can also run on cuda

Hi,
nice work..
Can share perf on M1 CPU and GPU like https://github.com/Stability-AI/stablediffusion/pull/147#issuecomment-1402153477  for recent Intel IPEX pull request?

This is the result in my environment for txt2img.py!

CPU
Memory

Apple M1
16GB

Batch size
Model name
device
argument
Avg gen time
How fast compared to cpu

1
SD2-v (768px)
CPU
--precision full
273s
N/A

1
SD2-v (768px)
MPS

100s
273%

4
SD2-v (768px)
CPU
--precision full
1010s
N/A

4
SD2-v (768px)
MPS

577s
175%

1
SD2-base (512px)
CPU
--precision full
271s
N/A

1
SD2-base (512px)
MPS

98s
276%

4
SD2-base (512px)
CPU
--precision full
1014s
N/A

4
SD2-base (512px)
MPS

600s
169%

Thanks for testing! Hope can be merged soon!

If there is a problem, I'll revert 90d4c71

Can confirm this works more or less on M2 w/ 16GB (although I'm not sure how you generated 768x768, that crashes for me in allocating a buf)


26 - Add frontend guides

What this PR does / why we need it:
This issue adds a basic Frontend Guide. Which will be extended in future issues to add different sections in more detail.
Which issue(s) this PR closes:

Closes #26

Special notes for your reviewer:
This issue was done by Matthew. He's no longer in the project, but I think we can review the changes and apply the required changes to merge this PR
Suggestions on how to test this:
Visual inspection of the changed files from the README
Visit the README page to see the changes https://github.com/IQSS/dataverse-frontend/tree/docs/26-add-frontend-guides
Does this PR introduce a user interface change? If mockups are available, please link/include them here:
No
Is there a release notes update needed for this change?:
New Frontend Guide
Additional documentation:

Moving this back to in progress because there are a number of TODOs

coverage: 97.425% (+0.2%) from 97.179%
when pulling 98ba0bb285ee28d2f7649c91c9e32aa969e34b30 on docs/26-add-frontend-guides
into a9d62daa50800bb866a5884136ef075cd2183d46 on develop.

@ekraffmiller thanks for your comments. I addressed the changes!


home page responsive problem

on laptop with small screen, search bar is hidden by the "Featured projects"

This issue still exists in one of my configurations: laptop with separate display (1920x1080), linux mint 21 and firefox 117. On the other hand, the display is fine if I use Chrome. On my two other configurations (both desktop, linux mint, firefox), the display is fine... I therefore think that this bug should remain documented, but that it only affects a very small number of people and that there's no point in spending more time fixing it.


Add option to insert soft breaks

Added softBreakBefore to TextOptions it in src/core-interfaces.ts  and types/index.d.ts along with simple logic in genXmlTextBody to insert "<a:br/>" if it is set.
Implements #805

Thanks @memorsolutions !


fixed phantom.args for phantomjs 2.x

@n1k0 this works for me for the phantomjs 2.0 version.
I'm not sure, if it would work for the 1.x version, so that's why I checked it that early only to set the phantom.args

Ah, my bad I was unclear; you need to add a new entry for PhantomJS 2 to the Travis build matrix (basically after this line), so we can ensure that everything is fine.

there are no binary files for linux packed for PhantomJS 2

Wat.  So forget about it, I guess. Grmbl, snap.
In other news, the build is green, which makes me confident we can safely merge that. We can always revert later if people complain ;)

Great :)

Please update the same for beta so one can install with brew.

I just install from brew, met the problem, and solved by overwrite the file : /usr/local/Cellar/casperjs/1.1-beta3/libexec/bin/bootstrap.js, thanks a lot :+1:

Can you please release a new beta or update the latest one ? so we can move forward with the error ?
@goggelz thanks for the fix !


Ask node red status (read data from node red)

Hi everyone,
Is it possible to read the values present on the node red?
I have a DHT11 (temperature and humidity sensor), on my P0 raspb with node red.
I need to ask Alexa:

what are the temperature and humidity of my room, or
know if a switch is ON or OFF.

Thanks

Check out https://flows.nodered.org/node/node-red-contrib-virtual-smart-home
You can create a virtual smart thermostat and keep it updated with your DHT11.
You can create a virtual smart switch and do the same with the switch.
This assumes you already have a way to read the sensor/switch into Node Red
There may be better ways but I use this way to read and control my really old wifi thermostat through alexa.


Assignment 5 Cats and Dogs & Assignment 4 Wine

completed part I and part II

Looks good, thanks!


All functions in tests and examples should be explicitly imported

When writing tutorials / examples it is difficult for a new user to understand what parts are defined within a repository and what parts are external. Thus we should conform to the standard that all examples and tests are explicit in the use of functions that require external packages.

The current examples and tests to do not conform to this


Formatting in description

Allow for formatting in description:  bold, italic, lists, url links.

This can be done with the label using setText(...)
label.setText("
Header

This is a bold paragraph.
")

Html text can be retrieved from markdown using the markdown package.


How navigate textInput with arrow keys?

Hello,
I would like to be able to go from one text input to the other with the arrow keys or enter key. Could somebody show me how to do it with an example?
Thank you!

I found a not so good workaround because I had to edit an internal file from this library. I hope somebody can give me a better solution.
I did the following:
Added the following line to TextInput.svelte that comes with the carbon-components-svelte package.
<svelte:options accessors="{true}" />

And this is what I have in my code:
<script>
    ...
    refs[index].ref.focus();
    ...
</script>

<TextInput
    bind:this={refs[0]}
    size="xl"
    placeholder="Enter user name..." />


added the ability to filter columns with array values

Allows the ability to filter based on array values in a single cell.  Addresses Issue #546.
Please let me know if there is anything additional I should do or change.  This is my first time creating a PR and contributing to open source! :smile:
Example pictures below.

Coverage decreased (-0.1%) to 71.816% when pulling d662e33c9b5b1a70862f78049479aa1410688446 on DevelopmentByDavid:issue-546 into 247b6703015edb404730ef39a9c83a760a3225b8 on gregnb:master.

hey @DevelopmentByDavid just returning from a long vacation. Need some time to review. This PR from a quick glance is also asking to make a big change to the way the package accepts data by allowing: strings, numbers, and arrays?
What would happen if a user places an array but does not use a customBodyRender? how does that display?
I think it's a good idea, but I need some time to think about this major change of allows Array data

Hi @gregnb  welcome back! Hope your vacation was good.
I believe that the way in which I modified the code only effects filtering arrays of values in a single cell.  Shown below is the filtering done without the customBodyRender.
If a person has an array of values in all cells of a column, then they can specify all possible filterOptions (from the docs).  However, my problem was that I had no way to filter that column (the one with arrays as values) appropriately.  I wanted the row to display if it contained any of the tags in that particular cell.
Let me know if I need to add better comments or do something else! Happy and willing to help if I can.

Hey @gregnb any update on this PR? I'm able to update this more if needed.

Hey @DevelopmentByDavid can you add some unit tests for this enhancement? Test a case with arrays and without

Hey @gregnb quick question before I code up the tests.
Would you rather me modify the data here (link below), which results in me having to modify other tests, or would you rather me build the 2 tests you mentioned with their own set of data?
I think if I modify the original data below it makes sure the changes integrate well with everything else. However, there are a decent number of tests where I have to change what the expected value is.
Let me know what you prefer/think is better and I will do it.  Thanks!
https://github.com/gregnb/mui-datatables/blob/b972ad44d5d57c77668017db65eb19d07e788e3c/test/MUIDataTable.test.js#L25-L67,

@DevelopmentByDavid I would prefer new tests that way it will be easy to track the new enhancement

Hey @gregnb
So, I changed a few things from my original PR code-wise to be more consistent with how the table already works.  I also added one more enhancement so that the table just works with arrays of data.  Here's the breakdown of everything I added.

New example  in array-value-columns/index
Filtering based on arrays of data is now more consistent with how the table already works (I'll clarify below)
filterData is extracted from arrays properly, now you don't need to specify filterOptions if there are arrays of data in the table -- nice for arrays just working.
Wrote 4 new test: array filtering, array filter option extraction, non-array filtering, non-array filter option extraction

From bullet 2, I realized that all data in the filterList is logically AND'd  and that with array data, I was essentially doing logical OR's -- array data is now logically AND'd.
Please let me know if I need to explain anything further, add more tests, cleanup/modify code, etc.  Thanks!  :smile:

Great! Thanks for making the changes. Merged!


feat(core-flex-grid): add responsive props

This adds the responsive components xs, sm, md, lg and xl to the col component of flex-grid. It also deprecates the span prop in this component.

@ryanoglesby08 Working on getting those examples updated now.

Just committed some docs updates, let me know what you guys think and if there's anything I can improve/change!


Backport bugfixes to 4.9.2

[x] #460
[x] #452
[x] https://github.com/KyoriPowered/adventure/commit/a92511ca9a49a12b0768556b678d5b486ba3b5dd

https://github.com/KyoriPowered/adventure/compare/959ea8fedabd^...98bcf720aebc


chore: add fs to release config

fs instrumentation release is skipped in #929 because it is missing from the config

I thought we didn't need to add those manually anymore with v3?

no need anymore to update .release-please-manifest.json.

I thought we didn't need to add those manually anymore with v3?

config is required, manifest is not.


Typos

Typo fixes are always welcome :-)
Can you please sign the CLA?

Signed

👍
/cc @njuettner @linki

Thanks for fixing those typos :) ! 👍


Validation On Transaction Required Sector

When updating a transaction on activity GB-COH-01926828-TZ21 the transaction validation required that I enter a Sector code on the transaction. However, sector is not mandatory for a transaction unless it is not present at activity level (there is a sector code present at activity level) and then must be present for all transactions - http://iatistandard.org/201/activity-standard/iati-activities/iati-activity/transaction/sector/

Validation of sector code of required has been removed. However, if sector vocabulary is selected then sector code is required.

In addition to what @RajbanshiNeha has said, AidStream has made the sector element required at the activity level and optional at the transaction level. We are hoping with this, the system becomes easier to use for its users.


revert incorrect change to use key_owner_type instead of object_type

To fix the broken build...
/ping @chef/lob

:+1:

Already merged, github just hasn't figured it out yet.


Merge CB-2.61.0 to master

from cherry-picked change for already on master 1fe5f24

do not create merge prs directly from release branch


Add infinite as an option for max uses

Currently for the scale value that we use to track the uses of Barbarian rage we enter an arbitrarily large number for the 20th level (999 at the moment) to represent unlimited uses. It would be great if the max uses field on character sheets recognized "Infinity", or "Unlimited", or "∞" as valid max uses and would disable max use tracking in that case.

@arbron This might be a bit too hacky, but unlimited uses can already be achieved by making the max uses field blank, i.e. ""  (this is the default value for items). Could we just have the scale.barbarian.rage-uses value progress as "2" "3" "4" "5" "6" ""  ? Also, Foundry appears to ignore the remaining uses field when the max uses field is blank, so we don't need to change how that works either.
Upside: Actually zero codebase changes (besides changing "999" to "" in the packs)
Downside: It might not be intuitive, but we could set a hint below the max uses field saying "blank max uses = unlimited uses"?


Does detox support pure native apps? Meaning non react native mobile apps?

Description
Basically as the title says, can we use detox like we use appium, irrespective of what the app was built with? Can we use it to test regular native apps?
Your environment
Detox version:
React Native version:
Node version:
Device model:
OS:
Test-runner (select one): jest / other

by support i don't mean low level stuff, but similar functionality if not better than appium? basically is this a replacement for appium, or only for react native apps?

actually it support native app. for iOS, you would use Xcode to build and launch App test with Detox. For Android, you may need to put your android native code into a RN project and install node_modules with Detox, configure DetoxTest.java to build App https://github.com/wix/Detox/blob/master/docs/guide/android-dev-env.md


bls12_381_plus Dependency duplication

did-key at ^0.2.0 has bumped bls12_381_plus to ^0.7
signature_bls current is v0.35.0 which only brings bls12_381_plus at ^0.5
Need to raise with signature_bls (last 7 months ago) upstream to bump it and then ask bump at did-key on signature_bls
warning[B004]: found 2 duplicate entries for crate 'bls12_381_plus'
   ┌─ /home/foobar/pp/fission-cli/Cargo.lock:16:1
   │
16 │ ╭ bls12_381_plus 0.5.2 registry+https://github.com/rust-lang/crates.io-index
17 │ │ bls12_381_plus 0.6.0 registry+https://github.com/rust-lang/crates.io-index
   │ ╰──────────────────────────────────────────────────────────────────────────^ lock entries
   │
   = bls12_381_plus v0.5.2
     └── signature_bls v0.30.0
         └── did-key v0.1.1
             └── fission v0.1.0
   = bls12_381_plus v0.6.0
     └── did-key v0.1.1
         └── fission v0.1.0

Ok looks like ockam has re-organised this stuff - crate repository points here:
https://github.com/build-trust/ockam/tree/develop/implementations/rust/ockam/signature_bls
But this gives:
The 'build-trust/ockam' repository doesn't contain the 'implementations/rust/ockam/signature_bls' path in 'develop'.
I suspect the crate name has changed too... let's see at
https://github.com/build-trust/ockam/tree/develop/implementations/rust/ockam

Raised at upstream: https://github.com/build-trust/ockam/issues/3864 to locate the new crate / repo.
Might need to change a crate around on this to migrate off from signature_bls

Hi 👋 from the Ockam team. We're not using bls anymore and didn't realize did:key was using it directly :worried:. If someone would like to maintain it, we'd be happy to help get that started in whatever way we can.

Thanks - I've circled back at upstream: https://github.com/decentralized-identity/did-key.rs/issues/38#issuecomment-1327989962


ci: Python release fails without protoc

https://github.com/lancedb/lance/actions/runs/9749071233/job/26905457885

It's just a temporary Yum failure:
https://github.com/lancedb/lance/actions/runs/9749071233/job/26905457885#step:5:218
We can just re-run the job.


docs(user): move examples and recipes to Python files

Fixes #44.
This is a work-in-progress, just sharing a preview in order to discuss the structure.

nice work!

Thanks!


Tables in pdf not getting saved into csv file

HI,
In my pdf file, I have 4 tables [4 regions] for listing the holidays for a year. the tables has columns, Sr.No, Date, Day and Festival. The title on the table is Region Name Holiday List 2024. However, when i execute this line, there is no csv file being created nor the pdfdocs.jsonl file is created. it is just creating the data.jsonl file.
def parsing_the_pdfs():
t0 = time.time()
# Create a Library
LLMWareConfig().set_active_db("sqlite")
lib = Library().create_new_library("pdfdocs")
#parse and extract all of the contents from these documents
# Add file to the library
parsing_output = lib.add_files(input_folder_path=input_data)

print("Update: parsing time :", time.time() -t0)
print("Update: parsing output :", parsing_output)
#export all of the content of the library into jsonl files with metadata
output1 = lib.export_library_to_jsonl_file(output_data, "data.jsonl")
# export all of the tables
output2 = Query(lib).export_all_tables(query="Holiday", output_fp=output_data)

return 0

p= parsing_the_pdfs()
This is the output when I execute the code:
Update: parsing time : 0.0057866573333740234
Update: parsing output : {'docs_added': 0, 'blocks_added': 0, 'images_added': 0, 'pages_added': 0, 'tables_added': 0, 'rejected_files': []}

assign me this issue

@vijayproxima you can fix this in 3 ways
1.import tabula
Path to your PDF file
pdf_file = 'your_pdf_file.pdf'
Extract tables from the PDF (all pages)
tables = tabula.read_pdf(pdf_file, pages='all', multiple_tables=True)
Check how many tables were extracted
print(f'Total tables extracted: {len(tables)}')
Export the tables to CSV files
for i, table in enumerate(tables):
output_csv = f'output_table_{i}.csv'
table.to_csv(output_csv, index=False)
print(f'Table {i} saved to {output_csv}')

import camelot
Path to your PDF file
pdf_file = 'your_pdf_file.pdf'
Extract tables from all pages of the PDF
tables = camelot.read_pdf(pdf_file, pages='all')
Check how many tables were extracted
print(f'Total tables extracted: {len(tables)}')
Export each extracted table to a separate CSV file
for i, table in enumerate(tables):
output_csv = f'output_table_{i}.csv'
table.to_csv(output_csv)
print(f'Table {i} saved to {output_csv}')

3.import pdfplumber
Path to your PDF file
pdf_file = 'your_pdf_file.pdf'
Open the PDF with pdfplumber
with pdfplumber.open(pdf_file) as pdf:
for page_number, page in enumerate(pdf.pages):
# Extract tables from the page
tables = page.extract_tables()
    # Check if any tables were found
    if tables:
        for i, table in enumerate(tables):
            # Save each table as a CSV
            output_csv = f'output_table_page_{page_number}_table_{i}.csv'

            # Writing table to CSV
            with open(output_csv, 'w') as f:
                for row in table:
                    f.write(','.join(str(cell) for cell in row) + '\n')

            print(f'Table {i} from page {page_number} saved to {output_csv}')
    else:
        print(f'No tables found on page {page_number}')

please let me if any of this is helpfull for your repositries


Add script that sets the arango server.pem file on db.multinet.app

This is the script that will run on the letsencrypt deploy hook. It combines the fullchain and privkey into one file in the home directory, sets permissions, and then restarts the arango service so it's using the updated cert.
I modified this slightly for the db-testing server by using /etc/letsencrypt/live/db-testing.multinet.app/... as the folder to grab the certificate files from

It seems like this should be hooked into our ansible provisioning (Ansible has ways to send files over to the provisioned server, and methods for restarting services).

I agree, since our ansible provisioning (at multinet-deploy) is where we setup letsencrypt anyway, so we could install this script as a task after the letsencrypt tasks run. It would also allow us to encode these actions into ansible, making them more concrete/safe imo.

It seems like this should be hooked into our ansible provisioning (Ansible has ways to send files over to the provisioned server, and methods for restarting services).

I agree, since our ansible provisioning (at multinet-deploy) is where we setup letsencrypt anyway, so we could install this script as a task after the letsencrypt tasks run. It would also allow us to encode these actions into ansible, making them more concrete/safe imo.

@AlmightyYakob, @waxlamp I've updated this branch to use ansible for the provisioning of this script. I tested it successfully on db-testing.multinet.app and had no issue.
I moved the ansible files to their own folder, updated the README with the correct instructions, and removed all the vagrant references (since vagrant will no longer work with this setup).
Let me know your thoughts.

@AlmightyYakob, @waxlamp I've updated this branch to use ansible for the provisioning of this script. I tested it successfully on db-testing.multinet.app and had no issue.
I moved the ansible files to their own folder, updated the README with the correct instructions, and removed all the vagrant references (since vagrant will no longer work with this setup).
Let me know your thoughts.


--embed-thumbnail option in python implementation?

Checklist

[X] I'm asking a question and not reporting a bug/feature request
[X] I've looked through the README
[X] I've read the guidelines for opening an issue
[X] I've searched the bugtracker for similar questions including closed ones

Question
I'm in the process of transitioning my script to using the python implementation of yt-dlp. The only options I can't seem to find in the documentation are the equivalent sof --embed-thumbnail and --embed-metadata
Are there equivalent options in the python version?

They are postprocessors. See the various classes defined in yt_dlp/postprocessors. The embedding example in readme shows how to add a PP to the options with embed-metadata as the example

I got metadata to work, but I feel I'm missing another key or value to get the thumbnails PP to work. Here's my yt-opts:
ydl_opts = {   'download_archive': 'data/archive.txt',   'paths': { 'home': '/downloads/test', 'temp': '/data' },   'outtmpl': '%(uploader)s - %(title)s.%(ext)s',   'merge_output_format': 'mkv',   'noplaylist': 'true',   'writesubtitles': 'true',   'subtitleslangs': 'en',   'merge_output_format': 'mkv',   'postprocessors': [{     # Embed metadata in video using ffmpeg.     'key': 'FFmpegMetadata',     'add_metadata': True, }, {     # Embed thumbnail in file     'key': 'EmbedThumbnail',     'already_have_thumbnail': False,   }], }

Solved. For anybody interested:
ydl_opts = {   'download_archive': 'data/archive.txt',   'paths': { 'home': '/downloads/test', 'temp': '/data' },   'outtmpl': '%(uploader)s - %(title)s.%(ext)s',   'merge_output_format': 'mkv',   'noplaylist': 'true',   'writesubtitles': 'true',   'subtitleslangs': 'en',   'merge_output_format': 'mkv',   'writethumbnail': 'true', # Required to embed thumbnail   'postprocessors': [{     # Embed metadata in video using ffmpeg.     'key': 'FFmpegMetadata',     'add_metadata': True, }, {     # Embed thumbnail in file     'key': 'EmbedThumbnail',     'already_have_thumbnail': False,   }], }


Transcription

https://cloud.google.com/speech-to-text/docs/async-time-offsets#speech_transcribe_async_word_time_offsets_gcs-protocol
implement the above workflow to code, will get resulting json file. use json file and convert to subtitles in pymovie

Yep this is our next step after MVP!


ProductionPanel breaks when the "Civil Engineering" tech is modded out

There's a line in productionpanel.lua:
if(row.OuterDefenseHitPoints and pPlayer:GetCulture():HasCivic(GameInfo.Civics["CIVIC_CIVIL_ENGINEERING"].Index)) then
It assumes that CIVIC_CIVIL_ENGINEERING is present in DB and this may not always be true. My mod actually removes this Civic and thus Panel breaks. Could you pls add an additional condition, so the line would be:
if(row.OuterDefenseHitPoints and GameInfo.Civics["CIVIC_CIVIL_ENGINEERING"] ~= nil and pPlayer:GetCulture():HasCivic(GameInfo.Civics["CIVIC_CIVIL_ENGINEERING"].Index)) then
This fixes the situation.

Hi, there. I've noticed that a new version of CQUI has been posted.
May I ask why this simple fix has not been included? It's like 1 minute to make this change...

@Infixo 1 programmer, many requests. I promise there's nothing more than a simple lack of time that's prevented this change from getting implemented. I appreciate the gentle nudge and will work towards having this out by the next workshop update for you

Thanks, appreciate it!

Hi, I've checked the files and the update you've made is still going to generate an error. If a civic is missing, then GameInfo.Civics["CIVIC_CIVIL_ENGINEERING"] will return nil. You cannot do .Index at this time. That's why I used ~= nil in my code.

@Infixo Ah, apologies, I'll just lop off the .Index then


Investigate eBPF for interposing exec to force all processes to load libscope

Looked at a few approaches:

Netlink
https://bewareofgeek.livejournal.com/2945.html

Kernel Probes
https://github.com/brendangregg/perf-tools/blob/master/execsnoop
https://docs.kernel.org/trace/kprobetrace.html

inotify
https://stackoverflow.com/questions/6075013/how-to-detect-the-launching-of-programs-on-linux/8255487#8255487

These 3 approaches work. The use of netlink and kernel probes requires root access. That may be a show stopper for libscope. It may work well for Edge to recognize that a new app started and then attach in response.
The use of inotify is interesting in that root access does not appear to be required. It's a bit wonky in that getting notifications from /proc doesn't work. A notification from access to the loader is a good indication. However, there is not notification when a static app is started.
A fourth approach may have potential. The entry in /proc/loadavg contains the number of running processes as well as the pid of the most recently started process. It could be worth considering keeping state of the previous read compared with a current read to determine if new procs have started. This has real potential in that it doesn't require root priv and it would apply to any exec, including statics. The need to keep state is painful.
A fundamental point in all of the above is that they all represent responsive behavior. That is, we recognize that a new app has started after it is started. This implies that we would then need to attach to the new process. An attach operation requires root priv. So, does that imply that a netlink is a reasonable approach, since attach would need root priv anyway? It would be interesting to determine that a new exec is happening and be able to enable preload before the exec. That needs further research.


Icons

Hi Igor,
I have a problem with my icons, it appears with icons unknown. In below, I leave the code I have of the file configuration.py.
`# Plugins configuration settings. These settings are used by various plugins that the user may have installed.
PLUGINS_CONFIG = {
    'nextbox_ui_plugin': {
        'layers_sort_order': (
#            ADD YOUR SETTINGS HERE
#            layer_sort_order is a tuple
             'Firewall',
             'Switch Core',
             'Switch Access',
             'VoIP Phone',
             'UPS',
        ),
#        'icon_model_map': {
#            ADD YOUR SETTINGS HERE
#            icon_model_map is a dict
#             'PA220': 'firewall',
#             'PA820': 'firewall',
#             'PA850': 'firewall',
#        },
#        'icon_role_map': {
#            ADD YOUR SETTINGS HERE
#            icon_role_map is a dict
#             'Switch Access': 'switch',
#             'Swicth Core': 'switch',
#             'Firewall': 'firewall',
#             'Wireless Access-Point': 'accesspoint',
#        }
       'undisplayed_device_role_slugs': (
#            ADD YOUR SETTINGS HERE
#            undisplayed_device_role_slugs value is a list or a tuple
#            Listed device role slugs are hidden on initial view load,
#            you may then hide/display any layer with a control button.
             'Organizador Escova',
        ),
        'DISPLAY_PASSIVE_DEVICES': False,
        'DISPLAY_LOGICAL_MULTICABLE_LINKS': True,
        'DISPLAY_UNCONNECTED': False,
        'INITIAL_LAYOUT': 'auto'
    }
}`

I hope that you can help me.
Thanks.

Hi. Are icon map config parameters commented out in your actual config? Please note, once set, custom 'icon_role_map' or 'icon_model_map' values rewrite the default mapping. Also keys in those mappings should match corresponding slug values, not display names.

I also struggled with the keys in those mappings and used the display names first. Maybe you @iDebugAll can add in the documentation that we have to use the slug values.


Windows notification - log not being parsed

Got a couple of notifications yesterday that the log wasn't being parsed
The effect is that my collection is short of some cards, vault process is less than the progress in the latest log and the last booster I opened is not appearing in boosters
I am running 1.5.1. and have tried Manual resync, re-reading old logs and a full reset

The issue continues - opened 2 packs, and won 6 ICR but it isn't in progress, boosters or profile
Thanks

I'm running 1.5.2 i got some issue

Can you send me a zipped log on<EMAIL_ADDRESS>? I'll try to find out what's wing with it

Got another of those notifications yesterday - would be handy if it said what line in the log file had tripped things up

I have just had it start working again - did all the fixup steps without result when my log file was 70+mb but Arena must have archived most of that and the log is down to 11 Mb. The reset worked this time. Whatever couldn't be parsed in the primary log must have gone.
The 70mb log is with Razvier, who might be able to isolate the log issue


Undefined functions, types, etc.

Describe the bug
It seems that the plugin is not able to find the definitions. I get all kind of undefined ... errors.
However the Intelephense plugin for VS Code works just fine. See screenshots.
To Reproduce
Steps to reproduce the behavior:

Install Neovim and use my config and my coc-settings
Install a regular Laravel Project and install and run barryvdh/laravel-ide-helper
(optional) Install some other Packages (GraphQL or whatever you want)
Use some stuff from the vendor folder, or even use some laravel functions like abort or app
See the undefined messages

Expected behavior
I'd expect the functions and classes to be found ^^
Screenshots

Desktop

OS: macOS 10.15.3 / Linux Manjaro 18
NVIM v0.4.2

Let me check. Thanks for your feedback

Hi, please try call "CocCommand intelephense.index.workspace" command
Can you solve your problem?

@marlonfan
unfortunately it did not help.
All errors came back after it said "[coc.nvim] Indexed php files, times: 2338ms"

Same issue:

Sorry, i solve my last commented issue. The class WebTestCase is under a Test folder. In your package.json is

But i think i can override with :CocConfig and writing somethig like

I have the same problem on neovim for termux

Yeah, it would be nice if someone could fix this.
For now I disabled all the broken stuff.
"intelephense.diagnostics": {
    "undefinedTypes": false,
    "undefinedFunctions": false,
    "undefinedConstants": true,
    "undefinedClassConstants": true,
    "undefinedMethods": false,
    "undefinedProperties": false,
    "undefinedVariables": false
},

I am also having issues with this. I am using Laravel, and all of the model classes are undefined.
Don't get me wrong, I expect the php integration will have a lot of issues. Even intellij struggles working with all of Laravels magic methods. But there are just too many errors to be able to comfortably work.
I think in theory it should at least be able to give the same errors as the vscode intelephense lsp.

I actually retract what I said before. I have vim rooter, and startify installed. I thought vim rooter was setting the correct root, but startify was overriding that. When the correct project root was set, these errors went away. I apologize for incorrectly reporting this as a phpls error.

I'm also working on a Laravel project and it is unable to find the Str class specifically in the Controllers namespace. When working on an Resource, it did detect and import the Str class. But now it is impossible. I really wish it would be fixed soon.

Apparently you need to initialize the project with git in order for the extension to work as expected (At least in laravel projects) . I had the same problem but once I did a git init on the root of the project now I don't get Undefined types, functions, etc. I discovered this because I installed another similar extension coc-intelephense and it worked on one project but not the other, then realized that the project in which that extension worked was a git repo.

All of my projects are git projects from the very beginning and I also had coc-intelephense installed. This is/was not the issue.
I'm not using coc anymore .. so maybe something changed.


there is no dancetrack/yolox_x.py?

find?

you need to download here :https://github.com/Megvii-BaseDetection/YOLOX

@ZHuangyeei gave the right answer. Sorry for the late reply. I am closing this issue as it has been solved.


adds advice to run restic check regularly

First shot at partly fixing #1137...
Ideas / comments?

Codecov Report

Merging #1259 into master will decrease coverage by 5.8%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1259      +/-   ##
==========================================
- Coverage   52.78%   46.97%   -5.81%
==========================================
  Files         131      131
  Lines       12432    12432
==========================================
- Hits         6562     5840     -722
- Misses       5105     5883     +778
+ Partials      765      709      -56

Impacted Files
Coverage Δ

internal/backend/b2/b2.go
0% <0%> (-79.93%)
:arrow_down:

internal/backend/azure/azure.go
0% <0%> (-75.35%)
:arrow_down:

internal/backend/swift/swift.go
0% <0%> (-73.9%)
:arrow_down:

internal/backend/gs/gs.go
0% <0%> (-70.82%)
:arrow_down:

internal/backend/swift/config.go
34.37% <0%> (-56.25%)
:arrow_down:

internal/archiver/archiver.go
65.2% <0%> (-0.17%)
:arrow_down:

internal/backend/test/tests.go
59.2% <0%> (+0.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 c40b3d3...6f0f2fa. Read the comment docs.

Argh, will fix the typos ...

Petty? Just a tad ;-) So - no problem, it's your project so you set the guidelines.
My PRs will be about documentation, should I prefix them with doc:?

Good idea, for commits that only change documentation. Thanks!


[kmac] main FSM hangs after kmac_app completion

Hi Eunchan,
I am running into a FSM bug when running some transactions through the application interface ports, the waves are below:

At the marker, kmac_app module has received the absorbed_i input from SHA3 logic, so raises the app_o[0].done signal for the KeyMgr interface port and transitions into Idle state, outputting CmdDone to top level KMAC module.
Meanwhile, the top level FSM sees the same sha3_absorbed signal from SHA3 logic and transitions from KmacMsgFeed into KmacDigest state.
However on the same cycle sha3_done goes high since CmdDone is seen from kmac_app logic, so we skip over the KmacDigest -> KmacIdle transition and fail to properly reset the entropy state for upcoming transactions.

Ah.. Thanks for finding the bug. I will fix the main FSM to move to Idle state directly in case of App requests.


0.10.0 has no Flow type support

Updating to 0.10.0 causes Flow errors as this version contains no type definitions for Flow. 😢
We use Flow type instead of TS in all our projects and we don't plan to switch to TS in near future.

Hey! Flow support for graphql-js was dropped and to support v16 here the Flow support was dropped.

So, no Flow types will be ever provided?

If you wish to help maintain can add them to https://github.com/flow-typed/flow-typed Providing part of this project is also a possibility

Ok, got it. Thanks for the lightning-fast reply!


treeGridAttributes is not used

Expected behavior and actual behavior.
The treeGridAttributes of the args object passed to canvasDatagrid are not used to create treeGrids.
When creating a treeGrid, self.args.treeGridAttributes is used, but this is never set. I believe self.treeGridAttributes is what should be used.
Specifications like the version of the project, operating system, or hardware.
tested in Firefox and Chrome
http://jsfiddle.net/jswolf19/kcr9Lje2/

Workaround:
var grid = canvasDatagrid(args);
grid.args.treeGridAttributes = {
    ...
};


Add formatting support for more syntaxes

Purpose

This will add formatting support for string template literal, type guard
and ternary expressions.

Goals
Related to https://github.com/ballerina-platform/ballerina-lang/issues/13280

Codecov Report

Merging #13828 into master will increase coverage by <.01%.
The diff coverage is 74.66%.

@@             Coverage Diff              @@
##             master   #13828      +/-   ##
============================================
+ Coverage     64.48%   64.48%   +<.01%
  Complexity      426      426
============================================
  Files          2154     2154
  Lines         95329    95396      +67
  Branches      12682    12706      +24
============================================
+ Hits          61472    61519      +47
- Misses        28863    28865       +2
- Partials       4994     5012      +18

Impacted Files
Coverage Δ
Complexity Δ

...lang/langserver/formatting/FormattingNodeTree.java
71.5% <74.66%> (+0.2%)
0 <0> (ø)
:arrow_down:

...allerinalang/test/util/client/ResponseHandler.java
65.38% <0%> (-15.39%)
0% <0%> (ø)

...org/ballerinalang/bre/bvm/WaitCallbackHandler.java
81.08% <0%> (-3.61%)
0% <0%> (ø)

...et/http/actions/httpclient/AbstractHTTPAction.java
78.29% <0%> (-0.86%)
0% <0%> (ø)

...server/compiler/format/TextDocumentFormatUtil.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...va/org/ballerinalang/channels/ChannelRegistry.java
100% <0%> (+5.4%)
0% <0%> (ø)
:arrow_down:

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 867840a...dc38536. Read the comment docs.


Envoy crashes when processing a TLS certificate with no SANs or a CN

Envoy crashes when processing a TLS certificate that does not have SubjectAltNames or a CN field in the Subject.
This looks to be intentional: https://github.com/envoyproxy/envoy/blob/v1.11.2/source/extensions/transport_sockets/tls/context_impl.cc#L838
While there is no valid use case for having a certificate with no SANs or CN it should not be possible to cause a crash by providing a bad certificate. Perhaps this could be an exception instead like when there are no certificates provided:
https://github.com/envoyproxy/envoy/blob/v1.11.2/source/extensions/transport_sockets/tls/context_impl.cc#L744
?

I'm looking at this. I think the right approach is to throw, and ServerContextImpl already does this in some cases. There's also some refactoring needed for other release assertions that should not assert.
Please assign to me?

Not stale, see related PRs.


Discrepancy between EventItems and PDF version of agenda

The July 18 meeting of the Finance, Budget, and Audit Committee shows a discrepancy between the posted agenda and the EventItems (shown as related board reports in Councilmatic): https://boardagendas.metro.net/event/finance-budget-and-audit-committee-e911cf6f4bcb/
Granicus told Metro that this type of error can happen due to duplicate data in their system (i.e., line items in the database can be numbered the same, so the data looks like a duplicate and the system ignores it). They did not provide an explanation for why this can happen.
Metro now runs a script, which checks for and solves this discrepancy, before publishing an agenda.
In addition, DataMade may be able to alert Metro of these discrepancies using the following strategies:

finding a clue in the API for the abovementioned duplicates - @shrayshray can you tell me specifically which field showed the duplicate? or is it not visible in the Legistar API? (I do not see duplicate EventItemAgendaSequence in the API, so I might be overlooking something: http://webapi.legistar.com/v1/metro/events/1366/eventitems)
parsing the PDF for event items and comparing those results to the items scrapes from Legistar. DataMade will show the results (if "red") on the forthcoming integrity dashboard.

The second option would require significant time. Let's determine if the first option is possible, first.

@reginafcompton an item number was duplicated, so EventItemAgendaSequence makes sense as the API field where this would be stored ... is it possible you're not seeing the duplicate now because it has since been corrected in Legistar?

That does seem possible - I did not realize that Granicus corrected the problem. Are you be able to contact Granicus and ask specifically about the name of duplicate field (and if it is, indeed, EventItemAgendaSequence)?

@reginafcompton I'm still waiting on a response from Granicus for confirmation on the name of the field, will let you know as soon as I hear back.

@shrayshray can you clarify - is it the EventItemAgendaSequence that provides the line number? Or am I misreading your comment?

@reginafcompton sorry! I copied and pasted from Granicus's response and the field name didn't show up. Yes, EventItemAgendaSequence provides the agenda line number.

Great - we could implement my first suggestion (see above).
It seems like the best place to catch these duplicates is at the scraper level, specifically, when we iterate over the sorted_items list. Then, we could (1) raise an error or warning, and email Metro. Or (2) we could add a note in the event extras field, import that note to Councilmatic, and then render a warning on the forthcoming report dashboard.
The second option would require a little extra work, but would insure a fully automated process.

@reginafcompton with option 2, could there also be an email alert sent to Metro? With either option, would the raising of an error or a note in the event extras field preclude the flagged item or the event/Agenda from posting on the website ... until it's reviewed/approved, or the like?

@shrayshray I implemented solution (1), wherein the scraper fails – since this seems like a significant (albeit rare) data error, and we want to insure that duplicates do not appear on the site.
In sum, the scrapers look for EventAgendaItems with duplicate Sequence numbers. If a duplicate exists, the scrape fails and raises an error, instructing DataMade to contact Metro.
I also updated issue #257 with a suggestion about integrating such errors into the "data integrity" report.
I'll close this, but let me know if you have questions @shrayshray !


Style based on an external css framework

I'm creating a style guide built on top of bootstrap (bootstrap-material to be exact). Two things I'm struggling with.

Including the bootstrap style messes up the fabricator styles for the menu etc. (everything gets really small).
Next to the aggregates css and js, I would also like to offer these files without the external bootstrap.css/js included. Ideally I'd offer even more css/scss files: one for colors (that can be used by apps that don't use bootstrap), one for bootstrap apps that already include bootstrap and one monolithic css (as it is now).

Any ideas on how to approach this? Thanks!

Check this project out
https://github.com/mohamedzougam/fabricator-bootstrap.

I'm working on a refactor of fabricator, part of which is to address issues like this. Just need to find the time to knock it out!


Jv rox 12402 support packages should have the module version for master

Description
Support packages will include the MODULE_VERSION in master.
Checklist

[ ] Investigated and inspected CI test results
[ ] Updated documentation accordingly

If any of these don't apply, please comment below.
Testing Performed
TODO(replace-me)
Use this space to explain how you tested your PR, or, if you didn't test it, why you did not do so. (Valid reasons include "CI is sufficient" or "No testable changes")
In addition to reviewing your code, reviewers must also review your testing instructions, and make sure they are sufficient.
For more details, ref the Confluence page about this section.

/retest


Line Chart Cant hide x and y axis @version 2

Hi,
I am trying to hide the x and y axis referencing the documentation's example.
But I cant seem to make it work, does anyone know what am i doing wrong?
http://plnkr.co/edit/BpPmJSZ9ugT3otE3Pdc6?p=preview
Thanks in advance!

It can, you only have a small issue with the way how you declared the yAxis, here fix:
http://plnkr.co/edit/Y79YoAPzaXKTl9Zowx3F?p=preview

@AdamChlupacek
Wow thanks!


Add 'wordBreak' option to prevent cutting words

Ideally, this would be set as an attribute so that it can be controlled on a per-instance basis, but it exceeds what I can do now and this seems good enough for now.

I agree with the idea, but why not just add the ability to pass in what word-wrap you want
options = {...options, textOverflow: 'ellipsis', wordBreak: 'normal'};
 word-break: ${options.wordBreak};

@AndyDyer I'm okay with whatever method allows me to achieve word-break: break-word.

Thanks both for your contributions!
At the moment we are not setting any word-break by default... What stops you from selecting the element and applying the css yourself? I don't believe word-break goes with the line-break idea, neither its vue-line-clamp responsibility to set it.

@Frondor  Totally could set it by ourselves, but the fact you are setting word-break to its non default value leads me to believe that their is an inherent link between word-break and line-break.
In fact the use case that made me do this was that I needed word-break to be its default value and was surprised that you were changing it

@Frondor vue-line-clamp does set word-break.

Why is word-break: break-all on by default?

For some reason I missed that line, and we are indeed setting word-break to a hardcoded default.
I'm closing this in favor of #28 for an extended config.
Thank you both, and  sorry for the delay.


Material Editor preview improvements

From @grizeldi 's wishlist:
"Preview could really do with being able to turn the preview object around. Since it’s currently static, it’s rather useless for anything that involves lighting or normal manipulation. While I’m writing down my wishlist, a way to import a custom model for preview would be awesome as well."
Loading a custom model may be useful in some cases, but the user could just use the scene viewer instead. Same goes for moving around.
I'm going to take some liberties with this request and suggest implementing being able to select a more "complex" (predefined) model in addition to the primitives. This would go as an extra button beneath the others.

While not exactly what I had in mind, at least the famous teapot or Suzanne would be a good start for previewing some general more complex effects.
I primarily meant it as a way to preview materials that only make sense on a certain 3D model, but as you pointed out, scene composer can get the job done just fine for that. I'd have to check if shader refreshing works properly there though, it's been a while and I vaguely remember there being a reason why I didn't end up using scene composer for this.

At least material refreshing works, but I don't think matdefs would trigger the update since it's tied to the material editor atm. That's part of the other ticket.


not compatible with current spheres/gambit/ffi

This package tries to import "(spheres/gambit/ffi ffi-simple)" which does not exist!

Thank you @snifel0!
I really have to fix this. I'm currently AFK for some days and I'll get back to the issue ASAP.

Done! Sorry for the delay replying!

Thank you @alvatar!


Add tests for Docker

We're shipping Docker containers which differ with our current test environment (for eg, test environment runs on ubuntu-latest but the Docker container runs on PYTHON image which is based on Debian).
Due to divergence like these, we need to have separate tests for Dockerfile.

Hi, I'm interested in working on this issue. Is it still open and is it okay if I take a stab at it? If so, could you please provide some guidance on how I should proceed? Thanks!


update to ver 0.2.2: add conv data and make mask modules

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender; the new update passed the bot check. It is waiting for your approval.


Adding save/load weights functionality to model weights

This PR closes #59. The test is currently a script but will be turned into a function as soon as we start working on the test suite.

For people coming here from Google looking for a way to serialize a Keras model: How can I save a Keras model?


Vuejs improvements

I was having problems with .vue files, has it has several syntax in one.
When trying to split / join an object literal between script tags, it would detect it as a css definition (as the plugin uses the first split/join function that matches.
I have extracted the css function definitions for vue file so that they are only executed between style tags.
I also had a few problems with split/join for one life ifs in javascript, fixed it too.

Thanks for the fixes. I don't use vue.js myself, so I appreciate you putting the work to improve the plugin.
I've made some minor (mostly) stylistic changes, and I'll go ahead and merge this. If I've accidentally broken something, please let me know.


Fix/nacos cluster

What this PR does:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Codecov Report

Merging #440 (b1c205a) into develop (0a54226) will decrease coverage by 0.41%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           develop     #440      +/-   ##
===========================================
- Coverage    37.62%   37.20%   -0.42%
===========================================
  Files           55       55
  Lines         3748     3790      +42
===========================================
  Hits          1410     1410
- Misses        2180     2220      +40
- Partials       158      160       +2

Impacted Files
Coverage Δ

pkg/router/route.go
45.73% <0.00%> (-13.80%)
:arrow_down:

pkg/server/cluster_manager.go
37.41% <0.00%> (-1.05%)
:arrow_down:

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 0a54226...b1c205a. Read the comment docs.


Expand the widget bigger than 4x4

The widget is currently only able to be a maximum of 4x4.  Is there anyway this can be increased?

Hi Chris, I wanted to make it bigger, but according to the documentation, 4x4 is the maximum supported size: https://learn.microsoft.com/en-us/azure/devops/extend/develop/add-dashboard-widget?toc=%2Fazure%2Fdevops%2Fmarketplace-extensibility%2Ftoc.json&view=azure-devops


Interview coding challenge  - Long Nguyen

Closes #60 .
[ISSUE-60] Refactor the signing functions in sign.js to reduce boilerplate code.
Motivation
Reduce the amount of boilerplate code inside the handleSign<Coin> functions.
Overview Of Changes

Sorry I forgot to mention this before, but you're going to need to sign these commits (something we require being a crypto + fintech company focused on security).
Some notes:
https://help.github.com/en/github/authenticating-to-github/about-commit-signature-verification
https://help.github.com/en/github/authenticating-to-github/signing-commits

Looks good, though see my comment w.r.t commit signatures

Commits are signed.