Add condition option to read_table

This adds the possibility to use tables read_where method using a numexpr expression to only load rows fulfilling condition.
Very handy for example to load data from only one telescope from a by_type table: condition="tel_id == 1" or only load reconstructed events: condition = "hillas_length > 0"

do you know if the start/stop/step apply before or after the filtering? I was going to implement a chunked iterator wrapping read_table, so that may have an effect.

do you know if the start/stop/step apply before or after the filtering? I was going to implement a chunked iterator wrapping read_table, so that may have an effect.

It's before, the rows are first selected based on the slice(start, stop, step) and then filtered by condition:
In [13]: f.root.test.read()
Out[13]:
array([(0, -1.07400584), (1, -0.90504077), (2,  1.22623558),
       (3, -0.3141724 ), (4, -0.26661846), (5, -1.7715964 ),
       (6, -0.61327828), (7,  0.98540159), (8, -0.89481293),
       (9, -0.27684014)], dtype=[('idx', '<i8'), ('value', '<f8')])

In [14]: f.root.test.read_where(step=2, condition='value > 0')
Out[14]: array([(2, 1.22623558)], dtype=[('idx', '<i8'), ('value', '<f8')])


Setup gh actions

Out of curiosity, what's the difference between Jekyll and NPM for this build process?  Since Jekyll is on NPM, can't we use NPM to build it more "standard"-ly.  Just mentioning since I noticed that we didn't use our reusable workflow here.

Jekyll is a static site generator written in Ruby, which uses Ruby bundler for most of its lifecycle operations. NPM is a JS package manager written in Node.
This repo is built upon an unsupported version of ruby (2.6). To move to the next supported version (3.1) would require us to bump the version of "jekyll-open-sdg-plugins", which then would require a complete refactor in how data sources are imported into the site. Specifying an alternate version in a .ruby-version file complicates matters because Pages.cloud.gov respects that paradigm and then will fail any new builds since 2.6 is unsupported.
Given there is no momentum to implement the scope of change required to bring this repo into our reusable flow, I implemented a custom one here. This allows us to a) get off CircleCI and b) continue building with Pages.cloud.gov if changes are needed. This seemed like the best possible approach.


Improve the pre validation step of ReCaptcha validation flow.

Issue Description
When the Captcha is enabled for SSO login and the failed limit count is exceeded, the users from the secondary userstore are able to log in when the provided username does not have the userstore domain prepended to it.

Login username:  secondary/test > captcha verification works as expected.
Login username: test > users can proceed to log in without captcha verification.

Analysis
When  “Enable captcha after the max failed attempts”  feature  is enabled and maximum failed attempts count is configured, the number of failed login attempts of each user is captured and stored. This information is stored in the user claim http://wso2.org/claims/identity/failedLoginAttempts.
Before the user is getting authenticated this count is evaluated as a part of the recaptcha pre-validation step. In there, it is accessing the user store to get claim value. Since the user is not authenticated at the time domain information of the user is not available. Passing only the username instead of domain qualified user name results to get the user store evaluated to the primary user store. Therefore, for secondary user store users, the user not found exception is thrown and the method is not returning the correct values.
As a fix to the existing flow, to find the required user claim value, the whole chain of userstores has to be iterated to find the userstore of the user[1].
Improvement suggestion
Ths initial fix for this issue has been done via [1]. This will resolve the initial issue, but this will have more complications.

Performance impact.
Unable to handle the same username of users in multiple userstores.

The improvement should be not to store the failed attempts in the userstore. The ideal approach should be to add them to a separate identity database. With this approach, the pre listeners/handlers/ valves will be able to get the failed attempt counts without accessing the userstore. Also ,we need to implement a cache to improve the performance.
[1] - https://github.com/wso2/product-is/issues/13644

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Scaffolding

I don't understand how scaffolding works on this new version... do you have a sample?

It's not currently available. I had it working with my own tool on core 1, but after tooling update in asp net core 2 I could not get it working in any reasonable amount of hours I tried, so i just gave up for now.
I'm revisiting this problem from time to time but I only got it working on netcoreapp platform.

Ok them, i will study how can we make this works on aspnetcore2 and let you know if I find something


The framework should caution you against nesting scroll views that go in the same axis direction

For example, the following code should trip some kind of assert:
return ListView.builder(
  children: <Widget>[
     SingleChildScrollView(child: SizedBox(height: 200)),
     SizedBox(height: 200),
  ],
);

Although it would be fine if the SingleChildScrollView specified a direction different than the ListView.

Your example could probably really be asserted against. However, having a scroll view inside other scroll view is generally a valid layout.
For example
ListView(
  children: <Widget>[
    SizedBox(
      height: 500,
      child: SingleChildScrollView(
        child: Container(height: 2000, color: Colors.blue),
      ),
    ),
    Container(height: 200, color: Colors.red),
  ],
);

See also https://github.com/flutter/flutter/issues/50946

That's the kind of example I'd want to avoid - how is that good for a users? :) However, it does help me think of a valid example, e.g. if that large container was a text area that you wanted to allow the user to scroll while still scrolling the surrounding UI. I think it's probably still tricky for users, but could be valid.
Perhaps this would be good as a lint - @pq is that something that is lintable, checking whether a scrollable ends up being nested in another scrollable?

checking whether a scrollable ends up being nested in another scrollable?

Cases where the widget hierarchy is setup statically should be lintable for sure.   Specifically the issue is nested ScrollViews?

Nested scroll views in the same axis direction.


chore: Cleanup Oracle Linux (OL9) References since it's not supported

What this PR does / why we need it:
We initially introduced RHEL9 support in https://github.com/replicatedhq/kURL/pull/4175 and thought we can get Oracle Linux 9 support incidentally. However, OL9 does not work and requires dedicated effort to get it completely working. As a result, this PR removes references/support for OL9.
Which issue(s) this PR fixes:

Fixes sc-50954
Special notes for your reviewer:

Steps to reproduce

Does this PR introduce a user-facing change?

NONE

Does this PR require documentation?

NONE

Remarks on the two failing checks in this PR:
This test is expected to fail since containerd 1.6.4 is incompatible with weave CNI: https://github.com/replicatedhq/kURL/actions/runs/4609074232/jobs/8145708550?pr=4328
FOSSA scan failure is due to a timeout issue: https://github.com/replicatedhq/kURL/actions/runs/4609074204/jobs/8146755751?pr=4328
The FOSSA endpoint reported an error:

          Sorry, we've encountered an unexpected error. We've recorded it. If you continue to see this, contact support at<EMAIL_ADDRESS>and include the unique reference code: 9eceec8e-322c-4fbc-bd03-487d6491787e

      Error UUID from API:

          9eceec8e-322c-4fbc-bd03-487d6491787e

      If you believe this to be a defect, please report a bug to FOSSA support at https://support.fossa.com/
      Please include the error UUID in your request.


Stuck in an infinite loop if DF crashes.

Pretty much what it says. Unity stops responding to anything, and has to be killed.

79c22f9 addresses this somewhat.


add titles with width/height to scale buttons in ExportDialog

closes #2183

Hello. Thanks for the PR. It seems the width/height isn't quite correct when exporting with watermark (I haven't thoroughly checked other cases). Alas, you don't need to do any separate calculations to get the dimensions, just read the width/height props from the export dialog's canvas element.

I got the dimensions from the canvas like you suggested, and have a working version on my branch called dimensions-from-canvas.
Pulling width/height from canvas directly makes sense and feels like a very clean solution. However, it has some incorrectness as well. The dimensions are off by a small amount (1 - 2 px) when exporting the following file's scene:
{
  "type": "excalidraw",
  "version": 2,
  "source": "http://localhost:3002",
  "elements": [
    {
      "type": "arrow",
      "version": 457,
      "versionNonce":<PHONE_NUMBER>,
      "isDeleted": false,
      "id": "qrItPdYIud37Vk-E6HMpn",
      "fillStyle": "hachure",
      "strokeWidth": 1,
      "strokeStyle": "solid",
      "roughness": 1,
      "opacity": 100,
      "angle": 0,
      "x":<PHONE_NUMBER>75,
      "y": 254.625,
      "strokeColor": "#000000",
      "backgroundColor": "transparent",
      "width":<PHONE_NUMBER>5,
      "height":<PHONE_NUMBER>,
      "seed":<PHONE_NUMBER>,
      "groupIds": [],
      "strokeSharpness": "round",
      "boundElementIds": [],
      "startBinding": null,
      "endBinding": null,
      "points": [
        [
          0,
          0
        ],
        [
<PHONE_NUMBER>5,
          -134.79296875
        ],
        [
<PHONE_NUMBER>5,
          22.48828125
        ],
        [
<PHONE_NUMBER>5,
          -197.73828125
        ]
      ],
      "lastCommittedPoint": null
    }
  ],
  "appState": {
    "viewBackgroundColor": "#ffffff",
    "gridSize": null
  }
}

Using the approach in the branch I linked above, these are the dimensions that get passed to the scale buttons when exporting that file without a watermark:

Scale
1
2
3

Dimensions (actual)
665x240
1331x481
1997x721

Dimensions (calculated)
665x240
1130x480
1995x720

I suspect there are inconsistencies because the canvas returned by exportToCanvas has non-integer dimensions that get rounded down or truncated to 665 and 240. (The height and width props of HTMLCanvasElement are always integers.)
My first solution (getExportSize method, the one in this PR) gives the correct dimensions to the scale buttons when exporting that file in all 3 scales, with and without the watermark. With all due respect, I'm curious to see what the errors you're seeing look like!
If there's a more compact/convenient way for everyone to see my attempted fix / test data, please let me know.

With all due respect, I'm curious to see what the errors you're seeing look like!

If you make a small image, exporting with watermark doesn't increase reported width:

or this one:

I suspect there are inconsistencies because the canvas returned by exportToCanvas has non-integer dimensions that get rounded down or truncated to 665 and 240.

I wonder if this 1px difference is enough to warrant the added code complexity.

1-2px off is OK I would say, but on the other hand we should be able to calculate the actual size. I don't mind the complexity since it's just one new function that does one thing and does it well!

Good catch! I hadn't considered what would happen if adding the watermark made the canvas wider. Just pushed a fix that takes care of it, and perhaps some other errors too:
First case

Second case

As for the second case (3 rectangles), I wasn't able to reproduce the 5px disparity. Could you share the exported .excalidraw JSON?

Can't repro any issues, let's merge. Thanks @dawneraq!


Can the delete function be made asynchronous?

Can the delete function be made asynchronous?

?

?

https://codesandbox.io/p/sandbox/vue3-ts-forked-m3zj7z?file=%2Fsrc%2FApp.vue
toast.clearAll()
Not as expected, the later added TOAST is also removed

@xinchen37

@xinchen37

https://codesandbox.io/p/sandbox/vue3-ts-forked-m3zj7z?file=%2Fsrc%2FApp.vue%3A17%2C12&layout=%257B%2522sidebarPanel%2522%253A%2522EXPLORER%2522%252C%2522rootPanelGroup%2522%253A%257B%2522direction%2522%253A%2522horizontal%2522%252C%2522contentType%2522%253A%2522UNKNOWN%2522%252C%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522id%2522%253A%2522ROOT_LAYOUT%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522UNKNOWN%2522%252C%2522direction%2522%253A%2522vertical%2522%252C%2522id%2522%253A%2522clwrhzvlh00073b75rmfvc9c2%2522%252C%2522sizes%2522%253A%255B100%252C0%255D%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522EDITOR%2522%252C%2522direction%2522%253A%2522horizontal%2522%252C%2522id%2522%253A%2522EDITOR%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522EDITOR%2522%252C%2522id%2522%253A%2522clwrhzvlh00033b75df6utm4b%2522%257D%255D%257D%252C%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522SHELLS%2522%252C%2522direction%2522%253A%2522horizontal%2522%252C%2522id%2522%253A%2522SHELLS%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522SHELLS%2522%252C%2522id%2522%253A%2522clwrhzvlh00043b75stae08cc%2522%257D%255D%252C%2522sizes%2522%253A%255B100%255D%257D%255D%257D%252C%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522DEVTOOLS%2522%252C%2522direction%2522%253A%2522vertical%2522%252C%2522id%2522%253A%2522DEVTOOLS%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522DEVTOOLS%2522%252C%2522id%2522%253A%2522clwrhzvlh00063b75n5z4u3pf%2522%257D%255D%252C%2522sizes%2522%253A%255B100%255D%257D%255D%252C%2522sizes%2522%253A%255B50%252C50%255D%257D%252C%2522tabbedPanels%2522%253A%257B%2522clwrhzvlh00033b75df6utm4b%2522%253A%257B%2522tabs%2522%253A%255B%257B%2522id%2522%253A%2522clwrhzvlh00023b75w9m9oo13%2522%252C%2522mode%2522%253A%2522permanent%2522%252C%2522type%2522%253A%2522FILE%2522%252C%2522filepath%2522%253A%2522%252Fsrc%252FApp.vue%2522%252C%2522state%2522%253A%2522IDLE%2522%252C%2522initialSelections%2522%253A%255B%257B%2522startLineNumber%2522%253A17%252C%2522startColumn%2522%253A12%252C%2522endLineNumber%2522%253A17%252C%2522endColumn%2522%253A12%257D%255D%257D%255D%252C%2522id%2522%253A%2522clwrhzvlh00033b75df6utm4b%2522%252C%2522activeTabId%2522%253A%2522clwrhzvlh00023b75w9m9oo13%2522%257D%252C%2522clwrhzvlh00063b75n5z4u3pf%2522%253A%257B%2522id%2522%253A%2522clwrhzvlh00063b75n5z4u3pf%2522%252C%2522tabs%2522%253A%255B%257B%2522type%2522%253A%2522UNASSIGNED_PORT%2522%252C%2522port%2522%253A0%252C%2522id%2522%253A%2522clwsvtv0u001c3b773cih3vn8%2522%252C%2522mode%2522%253A%2522permanent%2522%252C%2522path%2522%253A%2522%252F%2522%257D%255D%252C%2522activeTabId%2522%253A%2522clwsvtv0u001c3b773cih3vn8%2522%257D%252C%2522clwrhzvlh00043b75stae08cc%2522%253A%257B%2522tabs%2522%253A%255B%255D%252C%2522id%2522%253A%2522clwrhzvlh00043b75stae08cc%2522%257D%257D%252C%2522showDevtools%2522%253Atrue%252C%2522showShells%2522%253Afalse%252C%2522showSidebar%2522%253Atrue%252C%2522sidebarPanelSize%2522%253A15%257D

update url
https://codesandbox.io/p/sandbox/vue3-ts-forked-m3zj7z?file=%2Fsrc%2FApp.vue

Same problem is here. Please resolve it. Thank you. @jerrywu001

v0.2.2

...I updated to the latest version v0.2.2. But the issue is the same.

Hi @jerrywu001 I tried the latest version. It looks like the bug is still there, If it calls the clearAll or remove first, the next toast will disappear quickly.

Hi @jerrywu001 I tried the latest version. It looks like the bug is still there, If it calls the clearAll or remove first, the next toast will disappear quickly.

can you provide your code, please?

Hi, @jerrywu001 , thanks for you replying. You can try this one: https://codesandbox.io/p/sandbox/vue3-ts-forked-m3zj7z?file=%2Fsrc%2FApp.vue%3A15%2C28
It is the same as the @xinchen37 used. It is already using 0.2.2.
when you click the test button, the second toast disappears very fast, it is not expected. The test2 is correct, but we have to sleep 2 seconds.


Malformed address display

Start a new tx.
Paste destination address.

Result: In summary, destination address has only first 2 hex chars. e.g. 0x42...5b3b
Expected: display canonical short-form string for address. e.g. `0x42b5...5b3b

fixed as part of #495


Release the package to Conda Forge

This task involves creating a necessary step in .circleci/config.yml, and creating an account on conda-forge.

I'l be looking at it.

Opened PR #12 in order to satisfy requirements for the conda-forge packaging.

Package is available on conda forge: https://anaconda.org/conda-forge/pandas-stubs


Define a default build, install, test-result, an log base folders for colcon builds

I suggest defining default base folders for build, install, test-result, and log colcon targets, that is, use the defaults.yaml feature. I think it is interesting because we could run colcon [build | test | test-result] from any folder without messing with our environment. I just want to warn that there is a bug with test-result mapped in this issue

The issue mentioned is solved, so this issue isn't blocked anymore


Add more details to the usage instructions

Add some more details about how to use bazel-deps.

It took me a while to figure out how to run the tool correctly. I finally figured it out when I stumbled across the last section the bazel-buildfarm README.


question

Dataset viewer issue for 'name of the dataset'
Link: link to the dataset viewer page
short description of the issue
Am I the one who added this dataset ? Yes-No

Hi ! What's your question ?


Add consul-template support

https://github.com/banzaicloud/bank-vaults/issues/403

@bonifaido Yeah, I have plans to do a CR and ConfigMap example. Maybe in it's own directory where all the current examples are.
Before we merge.... I need to make it all up and watch it work. :) Give me a few days. Early next week!

No problem, take your time, and we are here/on Slack to help :)


Core Setting: Title bar = hidden causes UI issues

With Atom 1.53.0 on Windows 10, when I set the Core Setting 'Title Bar' to 'hidden', some UI elements do not work as expected. None of the buttons in the search panel (Ctrl+F) works (they are not even highlighted on mouse hover) and I cannot scroll in the output pane of the script package anymore.
I am aware that the 'Title Bar' setting is labeled 'Experimental', but thought that this report might still be useful.

@mkunzBGC Thanks for reporting this.


Fedex - The city of the job "Operation Support Agent-Associate " is "Judetul Cluj" on solr.peviitor.ro

ENVIRONMENT: production
URL: https://solr.peviitor.ro
Browser: Microsoft Edge
Device: Laptop
OS: Windows 10
STEPS TO REPRODUCE:

Open the URL in the browser
Execute Query for "Fedex" Company
Check the city for the job "Operation Support Agent-Associate"
Open https://careers.fedex.com/fedex/jobs
Check the locations for the job "Operation Support Agent-Associate"

EXPECTED RESULTS:
The location of the job is Apahida commune in Cluj county as on the company's website
ACTUAL RESULTS:
The city of the jobs is "Judetul Cluj" on solr.peviitor.ro
On the scrapers.peviitor.ro the message "Judetul Cluj is not a city in Romania" is displayed

@lalalaurentiu in API la orasa nu apare apahida. apahida apare doar in descrierea jobului.

@AdrianaCionca te rog verifica daca apare Cluj Napoca. daca trebuie apahida, va trebui sa revenim asupra codului cu laurentiu

@davidmondoc Apare doar Apahida care e comuna in Judetul Cluj.

Fixed!


Update Blazor integration guidance

Fixes #33216
Thanks @Rod-at-DOH! 🚀 ...

I've re-organized the sections. The one-off section that only applies to non-routable component integration with the Component Tag Helper is now first, and it's clarified in the section and in the lead-in remarks that it only applies to the narrow case it covers.
I moved the section that included layout/styles to the full Blazor integration section (it's an H3 now in that H2 section), and I modified it to show how to set a layout and where to get Blazor project template resources from the product unit repo.
I made remarks on the dev possibly having to add an @using statement to the _Imports file and to not sweat setting up stylesheet references for CSS isolation stylesheets.

Internal previews

📄 File
🔗 Preview link

aspnetcore/blazor/components/integration.md
Integrate ASP.NET Core Razor components into ASP.NET Core apps

Thank you, @guardrex, for your work! I'll start over again reading the article. And I thank you for the change to not having to set up the CSS stylesheets. The original document wasn't clear on that point.


The problem when running the baseline FIM

When running the baseline FIM on A100(40GB), the training time is too long (over 15hrs can not finish an epoch).
I try to maximize the bachsize, but the situation is not solved. Moreover, the volatile GPU util is 20%.
What's the problem?
How much time is needed, and how much computing resource is needed?

This is because Pytorch selects the slow 3D-CNN operator. To solve this problem, please modify the code in config.py (line 108 and 109):
from
torch.backends.cudnn.benchmark = False   torch.backends.cudnn.deterministic = True
to
torch.backends.cudnn.benchmark = True   torch.backends.cudnn.deterministic = False
I set the cudnn.benchmark and cudnn.deterministic to make all the experiment results reproducible. For FIM, you can modify the above options.

The problem is solved. Thanks for replying.


[SUGGESTİON] Replace Final object

Hello, great repo! What do you think about replacing your custom Final class with this
Maybe there is a reason why you didn't choose it?

Hello, you mean typing using Final? It is only okay for type checkers as:
There is no runtime checking of these properties.
What we need is actual runtime check.
The idea behind this Final class is to ensure that there is no 2 level subclassing.
You can inherit from ReadOnce, say you have DBPassword class, but you can not inherit from DBPassword it will rise a TypeError, ensuring that DBPassword will not be exposed further.

Thanks for explanation, I forgot about runtime checking.


Integrated terminal (PowerShell) cannot launch with cwd containing square brackets

Issue Type: Bug

Create a directory D:\folderNameHas[SquareBrackets]
Open the above folder with VSCode
Toggle integrated terminal
See cwd is C:\Windows\System32\WindowsPowerShell\v1.0> rather than D:\...

VS Code version: Code 1.32.1 (05f146c7a8f7f78e80261aa3b2a2e642586f9eb3, 2019-03-08T00:51:49.278Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-6200U CPU @ 2.30GHz (4 x 2400)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
7.90GB (3.21GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (19)

Extension
Author (truncated)
Version

spellright
ban
3.0.28

vscode-custom-css
be5
3.0.3

log-wrapper-for-vscode
chr
1.0.2

vsc-material-theme
Equ
2.7.0

vsc-material-theme-icons
equ
0.9.8

vscode-commands
fab
1.2.5

vscode-github-notifications-bell
fab
2.1.5

vscode-highlight
fab
1.3.4

vscode-open-in-application
fab
1.0.5

latex-workshop
Jam
6.2.1

rainbow-csv
mec
1.0.0

git-graph
mhu
1.4.2

python
ms-
2019.2.5558

vscode-sort-json
ric
1.13.0

code-settings-sync
Sha
3.2.7

dictionary-completion
yzh
0.8.0

find-word-at-cursor
yzh
0.1.1

markdown-all-in-one
yzh
2.1.1

vscode-jupyter
yzh
1.2.3

I believe that this is a problem with node-pty.

This is a PowerShell bug. See PowerShell/PowerShell#5752 and UserVoice post.

Thanks @the-ress, I verified this works with pwsh 6.


shacl2ShEx/shaclConvert Not Working

Hello,
tried with Firefox and Chrome, but:

All "Data format" drop downs are empty
I can't paste or write in "by Input" text boxes

(And shouldn't there be a "Visualize SHACL schema" option?)
Thank you for your work!

Yes, we are having some issues with the server where the application is located, we will try to solve it ASAP.
Thanks four your report!! ❤

Thanks for the report. You can try it again and I think it works again as expected.

Hi, the dropdowns have data but the text boxes are without dunction and converting doesn't start.
Do I have to convert SHACL to ShEx to use the visualization online?

Sorry for the delay in answering. In principle, the goal is that it also works and visualized SHACL shapes, but the SHACL 2 UML visualization is not finished yet as described here. If you are planning to use it in the near future we could give it more priority.
Thanks for using the tool. We really appreciate your input.

Whether or not we are going to use the service depends on the results we get from it...
So don't do it for me, but do it anyways? :-)

Shacl 2 ShEx has been working for a long time now. If problems arise using it, new issues may be opened.
Additionally, the input/output formats of the conversion tools have been made more clear.
Closing this issue.


chore: export vault version enumeration

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Consider splitting the JSON source file into per-taxonomy-level

We currently load a large (4.5mb uncompressed) JSON file but by default we display only a small subset of it (countries).
We could split out other levels and load them lazily.

Fixed in https://github.com/zbraniecki/covid19-timeseries/commit/3db321f99fe7abf3e6657c9edd225bba1f60594e


remove highlight.js

Code is already highlighted through "rouge", so
enabling highlight.js only resulted in code
being parsed/highlighted twice.
This patch removes highlight.js and updates
the "github" stylesheet to be compatible
with rouge.
/cc @joaofnfernandes @mstanleyjones @johndmulhausen

Datacenter looks good. @sanscontext where's that page with all possible markdown permutations you've created? Is that page OK?

@joaofnfernandes https://deploy-preview-1021--psychologist-duck-16578.netlify.com/test/#/some-code-block-samples
Looks pretty good, but looks like YAML changed how it's broken, and it might not have dockerfile?

It seems that Rouge has support for Docker files in their latest version (2.0.7), but
we're importing Rouge through github-pages, which in turn imports Rouge 1.11.1.
https://github.com/jneen/rouge/blob/master/lib/rouge/lexers/docker.rb

While I'd love to do this, Jekyll does not support the Rouge version where Dockerfile support was added (which was 2.0.4). As you see here, it's dependency is set to ~>1.7: https://rubygems.org/gems/jekyll
$ bundle update
Fetching gem metadata from https://rubygems.org/...........
Fetching version metadata from https://rubygems.org/..
Fetching dependency metadata from https://rubygems.org/.
Resolving dependencies...
Bundler could not find compatible versions for gem "rouge":
  In Gemfile:
    rouge (~> 2.0)

    jekyll (~> 3.3) was resolved to 3.3.0, which depends on
      rouge (~> 1.7)

It's not a GitHub Pages thing; no idea when Jekyll will get their act together.

Trying this Gemfile:
source "https://rubygems.org"

gem "rouge","2.0.7"
gem "jekyll-seo-tag"
gem "kramdown"
gem "jekyll-redirect-from"
gem "jekyll","3.3.1"

You get the incompatibility error when running bundle update
Trying this:
source "https://rubygems.org"

gem "rouge","2.0.7"
gem "jekyll-seo-tag"
gem "kramdown"
gem "jekyll-redirect-from"
gem "jekyll"

Succeeds but bundler reverts way back to Jekyll 2.5.3, probably because that was before it had dependency checking for Rouge: https://rubygems.org/gems/jekyll/versions/2.5.3

In the mean time we can all watch this PR: https://github.com/jekyll/jekyll/issues/5556

Closing in favor of #2757. We can use conf instead of native Dockerfile support for now.


App takes too long to start

Describe the bug
Same as title
To Reproduce
Steps to reproduce the behavior:

Open the app and see how it slow is.

Expected behavior
Fast app loading
Screenshots (optional)
Nope
Version

Windows:
FluentHub:

Additional context
Making the principal view another one away from the profile wich needs to load all the information may fix this issue

I realised that that is taking too long is the sign in. We need to optimize it or open the app without signing in, and have a notification in-app telling you that you are getting signed in

this app cannot use without signing in.
notification page and home page use signed in user profile.

I know. What we can do it's optimize the logging or doing a Custom Splash
Screen telling that you are getting signed and later open the app. Like
charcter map uwp
El lun., 21 feb. 2022 23:57, NullPointerException @.***>
escribió:

this app cannot use without signing in.
notification page and home page use signed in user profile.
—
Reply to this email directly, view it on GitHub
https://github.com/fluenthub-uwp/FluentHub/issues/67#issuecomment-1047281051,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOMFYS73ZWXGDLA3L6ZO4ELU4K7MPANCNFSM5O7NP3WQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

ok i love that

Ok, nice!

I'll try to code a little these days. They're terrible because of school

ohok
And I now am publishing v0.7.1!

OH yes babyyy !!

I woll anounce this in multiple discord servers to get more people!

And maybe i know some friends to make us a logo. But probably we need to pay to him

ohh. I wanna pay.... but I maybe cannot. so if you are needed to pay, you dont have to. really.

release draft is seen in https://github.com/fluenthub-uwp/FluentHub/releases

Ok, nice!

All right. I thought about this.
Adding a custom splash means extending the display of the splash screen (Microsoft Docs).
So, adding a splash causes further delay.

However, the delay in startup time is due to await in OnLaunch. I don't use await and don't wait for a call to query the user's login name.

fixed in #74


fix(metrics): skip empty dimension values for addDimension & addDimensions

Summary
When adding a dimension to my metrics, I should get a warning if the dimension value is invalid aka an empty string ("") or undefined/null, and the dimension should not be added to the EMF blobs emitted by the utility.
Changes

Check for empty, undefined & null values for dimension and if found log warning
Skip invalid value dimensions to EMF, do this for addDimension & addDimensions
Unit test the fix

Issue number: #3302

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

It's ok, i have updated the warning message.


darwintrace: Support arbitrary path lengths

darwintrace.dylib did occasionally crash in __darwintrace_is_in_sandbox due to stack canary violations, because the reimplementation of realpath and splitting into path components assumed that paths will not be longer than MAXPATHLEN. In practice, this assumption was wrong, and many build systems will invoke file system operations with paths longer than that.
Switch to dynamically allocated strings and arrays and resize the buffers as necessary before using them. Since this could actually move the location of normPath around in memory, switch from the char *dst that pointed into this normPath buffer to a size_t dstoffset that can be used relative to a moved normPath.
With these changes, quassel builds successfully in trace mode – without them, the build abort(3)s.

I'll fix this with a bigger PR that also adds test coverage later on.

I'll fix this with a bigger PR that also adds test coverage later on.


Ability to replace image from local file path

Currently, batch_replace_shape_with_image function requires an http link. It would be great if you can also allow local file path.

Hi there @ericmanyc, thanks for sharing this use case, I'll take a look as soon as I can.
Meanwhile, I think you can use a combination of google drive library and Google Slides to achieve what you are looking for.
I've uploaded a notebook with some steps that you can adapt to achieve the result.
If you find a propper way to implement that within the "replace_shape_with_image" function itself, feel free to open a pull request. If not, I'll try to give it some time to implement it.
As said, here you have a quick workaround: https://github.com/vperezb/google-api-support/blob/master/docs/examples/upload_local_file_and_replace.ipynb
Hope it helps!

Thank you very much for looking into this. I am away this week but will take a look soon. Great project by the way!

@vperezb Works great for me! When are you planning to push another release ver? I am using hex tech for my environment and they require a release version.

I was testing this function
drive.delete_file(file_id=upload_response['file_id'])
and it is returning an error.
However
`upload_response = drive.upload_image_to_drive('graph', r'./chart_screenshot.png', folder_id)
slides.replace_shape_with_image(upload_response['image_url'], PRESENTATION_ID, 'image_1')`
works.
I granted my service account email with content manager access to the folder in my drive.

Good afternoon,
Do you mean that after you used the upload function the following "delete" does not work because the file is not found?
Is there a chance that the issue is related to the "not-refreshing" Google Drive client? Here's an example:
Imagine you've uploaded the file and then deleted it successfully, you may still see the image in your folder in the drive web. Then you try to delete the image because visually is there but in reality, is because is not refreshed. When trying to delete the image the request will prompt the error.
I had some issues with that and sometimes happens.

I dont believe so, when I upload_image_to_drive, I can see the file in my drive. However when I run drive.delete_file, it is stating "File not found: ***********************.".

Understood, I'll try to copy the lanes you pasted to see if I can reproduce it.
Or if you can copy the full snippet or error trace to have more info to see if I can identify something 👍
Thanks!

This is the entire log

HttpError                                 Traceback (most recent call last)
/tmp/ipykernel_1/1679185493.py in <cell line: 4>()
      2 _hex_pks.kernel_execution.scope_watcher.mark_dirty_scope_items(args=_hex_types.MarkDirtyScopeItemsArgs.from_dict({**_hex_json.loads("{\"dirty_scope_items\":[\"drive\",\"upload_response\"]}")}), app_session_token=_hex_APP_SESSION_TOKEN, python_kernel_init_status=_hex_python_kernel_init_status, hex_timezone=_hex_kernel.variable_or_none("hex_timezone", scope_getter=lambda: globals()))
      3
----> 4 drive.delete_file(file_id=upload_response['file_id'])

~/.cache/pypoetry/virtualenvs/python-kernel-OtKFaj5M-py3.9/lib/python3.9/site-packages/GoogleApiSupport/drive.py in delete_file(file_id)
    116 def delete_file(file_id):
    117     service = auth.get_service("drive")
--> 118     response = service.files().delete(fileId=file_id).execute()
    119     print(f"Deleted file: {file_id}")
    120     return response

~/.cache/pypoetry/virtualenvs/python-kernel-OtKFaj5M-py3.9/lib/python3.9/site-packages/googleapiclient/_helpers.py in positional_wrapper(*args, **kwargs)
    128                 elif positional_parameters_enforcement == POSITIONAL_WARNING:
    129                     logger.warning(message)
--> 130             return wrapped(*args, **kwargs)
    131         return positional_wrapper
    132

~/.cache/pypoetry/virtualenvs/python-kernel-OtKFaj5M-py3.9/lib/python3.9/site-packages/googleapiclient/http.py in execute(self, http, num_retries)
    838       callback(resp)
    839     if resp.status >= 300:
--> 840       raise HttpError(resp, content, uri=self.uri)
    841     return self.postproc(resp, content)
    842

HttpError: <HttpError 404 when requesting https://www.googleapis.com/drive/v3/files/*******************************? returned "File not found: *******************************.">


Does not see source.organizeImports code action from typescript-language-server

I'm using https://github.com/typescript-language-server/typescript-language-server v0.9.6
Their README has this part:

Server announces support for the following code action kinds:
...
source.organizeImports.ts - organizes and removes unused imports
...
Use like so:

"codeActionsOnSave": {
    "source.organizeImports.ts": true,
    // or just
    "source.organizeImports": true,
}

However kak-lsp doesn't seem to detect this code action, unless I'm doing something wrong.
Debug logs from kak-lsp:
Feb 17 18:03:14.132 DEBG From server: {"jsonrpc":"2.0","id":0,"result":{"capabilities":{"textDocumentSync":2,"completionProvider":{"triggerCharacters":[".","\"","'","/","@","<"],"resolveProvider":true},"codeActionProvider":{"codeActionKinds":["source.fixAll.ts","source.removeUnused.ts","source.addMissingImports.ts","source.organizeImports.ts"]},"definitionProvider":true,"documentFormattingProvider":true,"documentRangeFormattingProvider":true,"documentHighlightProvider":true,"documentSymbolProvider":true,"executeCommandProvider":{"commands":["_typescript.applyWorkspaceEdit","_typescript.applyCodeAction","_typescript.applyRefactoring","_typescript.organizeImports","_typescript.applyRenameFile"]},"hoverProvider":true,"renameProvider":true,"referencesProvider":true,"signatureHelpProvider":{"triggerCharacters":["(",",","<"]},"workspaceSymbolProvider":true,"implementationProvider":true,"typeDefinitionProvider":true,"foldingRangeProvider":true,"semanticTokensProvider":{"documentSelector":null,"legend":{"tokenTypes":["class","enum","interface","namespace","typeParameter","type","parameter","variable","enumMember","property","function","member"],"tokenModifiers":["declaration","static","async","readonly","defaultLibrary","local"]},"full":true,"range":true},"callsProvider":true}}}, module: kak_lsp::language_server_transport:151

Relevant part seems to only have source.organizeImports.ts, not the one without .ts suffix:
      "codeActionProvider": {
        "codeActionKinds": [
          "source.fixAll.ts",
          "source.removeUnused.ts",
          "source.addMissingImports.ts",
          "source.organizeImports.ts"
        ]
      },

At first I thought this is the root cause of the bug, and a quick grep in kak-lsp found only one occurrence:
https://github.com/kak-lsp/kak-lsp/blob/88f52f0c6271df372942b8344ea2bbca43c58b25/src/general.rs#L206
I tried to append "source.organizeImports.ts" to the list, but it didn't seem to help, so I'm wondering if the bug is elsewhere.
I also saw that maybe we can make a workaround command to send this command manually?
https://www.reddit.com/r/neovim/comments/lwz8l7/how_to_use_tsservers_organize_imports_with_nvim/gpkueno/?context=3
What do you think is truly going on? 🙂

Right, typescript-language-server (actually this behavior is inherited from tsserver) uses a special LSP command _typescript.organizeImports. Kind of defeats the point of LSP but OK.

I also saw that maybe we can make a workaround command to send this command manually?
https://www.reddit.com/r/neovim/comments/lwz8l7/how_to_use_tsservers_organize_imports_with_nvim/gpkueno/?context=3

That was very helpful; this command should be equivalent
define-command tsserver-organize-imports -docstring "Ask the typescript language server to organize imports in the buffer" %{
    lsp-execute-command _typescript.organizeImports """[\""%val{buffile}\""]"""
}

We should probably add this to rc/lsp.kak.
Due to a bug in kak-lsp, a version with lsp-execute-command-sync won't work yet.

I don't understand why they don't just use the generic textDocument/codeAction.
gopls behaves better here, they just use the generic textDocument/codeAction.
Technically, code actions are dependent on the cursor position, but gopls simply sends "Organize imports" for any cursor position.
Interestingly, typescript-language-server does send generic code actions for removing unused imports (and probably others too), but only if the cursor is on the import statement.
I tried with
echo "import { x } from './lib';
" > test.ts
echo "export declare const x = 1;" > lib.ts

with the cursor on the first line, there is a code action for removing the import, but not with the cursor on the second line.

So this feature is not exposed as code action but only as "workspace command".
I didn't check how the codeActionsOnSave is interpreted by VSCode's typescript support, but probably they special case the organizeImports code action and send the workspace command instead.

Thanks for looking into this! What you described makes sense.
I confirm the command works 👍
I don't know much about typescript but I think I read in some vscode issue that typescript-language-server was invented before LSP was a thing, so they had their own protocol and everything, and are still in progress of migrating to be a proper LSP now that it's a standard. I suspect they would be happy for issues describing exactly what they are missing or doing wrong or bad compared to other LSP implementations..


crictl exec failed connection using <IP_ADDRESS> in IPv6-only Kubernetes

What happened:
I'm running a crictl command in a IPv6-only K8s Cluster and get this error
bash-5.0# crictl -debug -r unix:///host/container-runtime.sock exec 4b3f4647627f37cf62bba7b7c305da087e87d0f5b8c85c81e9ecd43fcf0406d5 ls
DEBU[0000] get runtime connection
DEBU[0000] connect using endpoint 'unix:///host/container-runtime.sock' with '2s' timeout
DEBU[0000] connected successfully using endpoint: unix:///host/container-runtime.sock
DEBU[0000] ExecRequest: &ExecRequest{ContainerId:4b3f4647627f37cf62bba7b7c305da087e87d0f5b8c85c81e9ecd43fcf0406d5,Cmd:[ls],Tty:false,Stdin:false,Stdout:true,Stderr:true,}
DEBU[0000] ExecResponse: &ExecResponse{Url:http://<IP_ADDRESS>:34955/exec/gU1PMC4l,}
DEBU[0000] Exec URL: http://<IP_ADDRESS>:34955/exec/gU1PMC4l
DEBU[0000] StreamOptions: {<nil> 0xc00000e018 0xc00000e020 false <nil>}
FATA[0000] execing command in container: error sending request: Post "http://<IP_ADDRESS>:34955/exec/gU1PMC4l": dial tcp <IP_ADDRESS>:34955: connect: connection refused

The command is executed inside a container. It is to execute some command (ls in this case) in another container with container ID 4b3f4647627f37cf62bba7b7c305da087e87d0f5b8c85c81e9ecd43fcf0406d5 on the same node. The container runtime is containerd.
Is this error because crictl exec is using <IP_ADDRESS> instead of ::1 ? How can I let crictl use ::1?
What you expected to happen:
The crictl exec command can succeed.
How to reproduce it (as minimally and precisely as possible):
In an IPv6-only Kubernetes, execute the command in a Container for another Container on the same node.
Anything else we need to know?:
Environment:

Container runtime or hardware configuration: containerd
OS (e.g: cat /etc/os-release): alpine linux
Kernel (e.g. uname -a):  SMP Debian 5.10.162-1 (2023-01-21) x86_64 Linux
Others:
This is the /etc/hosts file viewed inside the container, not sure it matters.

bash-5.0# cat /etc/hosts
# Kubernetes-managed hosts file.
<IP_ADDRESS>       localhost
::1     localhost ip6-localhost ip6-loopback
fe00::0 ip6-localnet
fe00::0 ip6-mcastprefix
fe00::1 ip6-allnodes
fe00::2 ip6-allrouters

Looks like the host is defined there and not configurable right now:
https://github.com/kubernetes-sigs/cri-tools/blob/4daea44918cc34b63617c6c9e9923f43d47ddfdd/cmd/crictl/exec.go#L39

You may want to set StreamServerAddress in config.toml.
It seems containerd respects that config and returns that address to cri client.
https://github.com/containerd/containerd/blob/main/docs/cri/config.md#full-configuration


Change the color of under-bottom line of textboxes

Version
latest
Reproduction link
https://www.creative-tim.com/product/material-kit-pro-react
Operating System
Windows
Device
Windows
Browser & Version
Chrome (latest)
Steps to reproduce
Hi,
I am working on LOG-IN page and there is a CustomInput component, and I would like to know how to change the color of under-line of the textbox (which is an Input materil-ui componet)?
I have tried with ThemeProvider wraping the CustomInput but with no luck
const theme = createMuiTheme({
palette: {
  primary: {
    main: green[500],
  },
  secondary: {
    main:  green[500],
  },
},
});

.....
</ThemeProvider theme={theme}>
                    <CustomInput
                      id="first"
                      formControlProps={{
                        fullWidth: true
                      }}
                      inputProps={{
                        onChange: e => change1(e),
                        placeholder: "First Name...",
                        type: "text",
                        startAdornment: (
                          <InputAdornment position="start">
                            <Face className={classes.inputIconsColor} />
                          </InputAdornment>
                        )
                      }}
                      InputLabelProps={{
  className: classes.cssLabels
}}
                    />
</ThemeProvider>

I have couple of questions:
1- How to change the under-line color of textboxes in LOG-IN page or across whole pages and website
2- How to change the color of primary and secondary palettes across whole pages?
Regards,
Mahdi
What is expected?
Changing the color of textboxes in whole pages
What is actually happening?
I have tried several ways but with no luck

Solution
Additional comments

Hello there, @Bazarganigilani ,
I do not know why, but Github never notified me about your issue.
Do you still need help with this?
Best,
Manu


Autocomplete without automating inserting of default arguments

Hi!
Today, when autocompleting a function waiting for arguments, the autocomplete automatically inserts default arguments according to the function signature. But, in a lot of cases (when dealing with currying) I don't want them to be inserted. Moreover, if this behavior is specifically desired, there's the Insert Default Arguments features.
Is there any way to deactivate it?

Hi @ghivert, you can go to the Settings view to disable that.

To be honest, I can't find the option, neither in the Core Settings View nor the autocomplete-plus package settings... 🙁

It's an Elmjutsu setting :)


ArnoldDisplacement node

This adds support for assigning displacement shaders in Arnold, by combining an input displacement map (any Arnold shader) with attributes such as disp_height, disp_padding. That can then be assigned in the usual way using a regular ShaderAssignment node.
Arnold doesn't support changing displacement or subdivision in interactive renders, but this could be dealt with by deleting and recreating the polymesh node. It was my plan to implement that for this PR, but since SG#9168 seems more urgent, I thought it best to get the already-done displacement functionality merged first, and then come back later to add the bells and whistles for the interactive rendering. Let me know if that doesn't sound right...

After getting utterly fed up with Codacy's whining about additional complexity when doing things as simple as adding a single new unit test without even a single control structure in it, I've turned off the Codacy PR status. I've left on the "comment on PR's" option as an experiment though, to see if it seems helpful that way, and if it makes meaningless comments about complexity or not. First two comments seem helpful, so I'll address them now. Let me know if you have any thoughts...

Rebased to fix the conflict with master, and address the two Codacy comments.

Any chance of some movement on this one? I've got two other feature branches that I've based off this because they need use the Shader::attributes() method introduced here, so getting it merged would be a big help...


Provide an option for rendering custom component

Could you provide an optional option for rendering custom component.
use(container, {
  className: string, // default to "remark-container",
  containerTag: string // default to "div"
  render?: (type:string, title: string) => JSX.Element,    // <----- something like that (actually type corresponds with the className I suppose)
})

The below component (currently in the package) could be the default if any render function does not provided, of course.

    Custom Title

  Container Body

Wanted feature:  define a custom render function in the options in order to render the custom component like attachment as an example.
render?: (type:string, title: string) => <Callout type={type} title={title}>{children}</Callout>

The type could be 'warning' | "caution" | 'danger' | 'info' | 'note' | 'tip'  etc.
I mean this package could be more dynamic in the next versions. Thank you for initiating this project.

Hi @talatkuyuk, thank you for reporting the issue!
Looks interesting. However, I think we need to consider that this plugin is a one of a transform plugin, so it might not be better to convert given AST into JSX notation immediately.
If we provide an option like that, it may be like https://github.com/Nevenall/remark-containers#options?

Okey, agree. I am going to close the issue; but will create another one about extracting the title.


Exception from wallet

when i run equality farm summary, I get the following:
Exception from 'wallet' {'error': "'>' not supported between instances of 'NoneType' and 'uint32'", 'success': False}
Farming status: Farming
Total equality farmed: Unknown
User transaction fees: Unknown
Block rewards: Unknown
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Ubuntu 20.04
OS Version/Flavor: [e.g. CentOS 7.2]
CPU: Ryzen 9 and 7

Additional context
Add any other context about the problem here.

I have the same problem 'Exception from 'wallet' {'error': "'NoneType' object has no attribute 'tx_store'", 'success': False}'


How to make a note get not deleted when a different note is pressed

What is your question?
Hi,
sorry for that shitty title.
I managed to implement the deletion of the notes on the other grid after ShadowMario's advice, but one problem still exists.
The problem is that when 2 same notes (for example. 2 up arrows), one on the second grid and the other on the third grid, are placed, and then trying to delete the note on the third grid, leads to the note on the second grid to be deleted.
Here's a gif for demonstration purposes:

Here's the code (sorry if it's trash)

previous issue #2479

** MAYBE THIS IS INCORRECT **
Looks like its something to do with noteDataToCheck += 4
try to delete that if statement.
Explanation:
FNF handles notes using integers, so left is 0
down is 1
up is 2
right is 3
and they use + 4 to delete same note, because right side in charter is left side but + 4.
And if you delete statement with +=4 it won't check for other side.

Hi it didn't work but I came up with a different solution. I just made an if statement that checks if a specific key is pressed and if it is pressed you can delete the note on the second or third grid


[Bug]: With Nx, not working as expected

Summary
Publishing NX apps, I dont have single toml file, instead, each app has its own toml file. This was working on v3. It needs to be separated as these apps deployed to different domains. Once updated to v4, it doesnt work anymore, it requires single toml file on the root.
Is there any way to have multiple toml files, or any suggestion how to work around this?
I am not using the UI settings, but the CLI method.
Currently, moved everything back to v3. I wanted the v4 because of the ISR. Thanks.
Steps to reproduce

Apps built on NX 13
Nextjs 12
Apps: /app/web1/netlify.toml and /app/web2/netlify.toml

A link to a reproduction repository
No response
Plugin version
4.1.1
More information about your build

[X] I am building using the CLI
[X] I am building using file-based configuration (netlify.toml)

What OS are you using?
No response
Your netlify.toml file
No response
Your public/_redirects file
No response
Your next.config.js file
No response
Builds logs (or link to your logs)
No response
Function logs
No response
.next JSON files
No response

Hello! Is this still an issue?

Yes

Closing due to inactivity. If this is still and issue, feel free to reopen and we'll take another look at this.


Update for SBT 1.0, Scala 1.12

I don't know how far this project is still active, but I hope that it is possible to update it to current versions of Scala and SBT

@pfn Could you please comment? Thanks!

This pull request has been merged : https://github.com/pfn/kotlin-plugin/pull/35
Waiting for a v2.0.0 release which will have these changes!

@kpritam That's nice. Do you have published artifact for the snapshot of the plugin by any chance ?

For time being we are publishing it to our bintray account:
resolvers += Resolver.bintrayRepo("twtmt", "sbt-plugins")
addSbtPlugin("com.github.tmtsoftware" % "kotlin-plugin" % "2.0.1-RC1")


Torchvision or Pytorch Lightning?

Describe the bug
The documentation states that "DeepForest is built on the object detection module from the torchvision package". However, it appears that it is a Pytorch Lightning object. Has DeepForest moved from torchvision to Lightning? Or are these not mutually exclusive? I believe it was implemented in Tensorflow in v1.14
To Reproduce
The deepforest class in main.py inherits from pl.LightningModule.
Also
Describe the bug
In the CropModel .html, it says that "Beginning in version 1.4.0, the CropModel class can be used in conjunction with predict_tile and predict_image methods".
It looks like I'm using version 1.3.3, even after implementing the changes in #745. However, the CropModel class is still in the model.py file and the predict_tile function in the deepforest class contains cropModel as an argument.
I don't think it's particularly important as I have all the functionality of the latest version, but I'm not sure why it says it's v1.3.3
To Reproduce
pip show DeepForest
Name: deepforest
Version: 1.3.3

from deepforest.model import CropModel
model = CropModel()

Thanks!

Hey @camappel can you tell me a bit more about your project and yourself in terms of your needs.
Torchvision and pytorch lightning are both built on top of pytorch. Pytorch lightning in particular is just a pytorch wrapper, it should be considered as literally pytorch. DeepForest hasn't been in tensorflow for several years, since 1.0. Where did you see that? Perhaps a very old paper? I'd want to hunt down those references to prevent confusion. So, no there is no conflict at all between torchvision and pytorch lightning, its a very common combination of packages, for example see torchgeo.
Your second question on the CropModel is the difference between the docs which are generated on the main branch and the  pip version which is behind at 1.3.3. @ethanwhite and @henrykironde this is the second user to note this, its probably time to talk about deploying docs off the release? Instead of on main. @henrykironde can you make an issue here and add [Discussion] tag. @camappel if you want to the bleeding edge, install from the git repo, either clone the repo or pip install from git. I sense that after my collaborators and I discuss, 1.4.0 will be up on pip next week.
Please close unless there are further needs.

Thanks @bw4sz, We are working on that, in fact we activated 1.3.3. I am working on clarifying those doc versions.

Existing docs issue is here for reference: https://github.com/weecology/DeepForest/issues/746

Great, I think this can be closed.

Thanks very much for the explanation. TensorFlow being used in 1.14 was a guess, as I knew it had been used previously but wasn't sure which version.
For context, I'm working on another Environmental Data Science notebook for DeepForest, since the last notebook is from 2022 and didn't include the CropMode.
Closing this issue - thank you!


quotes fixes for username password

Proposed changes
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request.
Fixes
If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce? (A breaking change is a fix or feature that would cause existing functionality and APIs to not work as expected.)
Put an x in the box that applies

[ ] Non-breaking fix (non-breaking change which fixes an issue)
[ ] Breaking fix (breaking change which fixes an issue)
[ ] Non-breaking feature (non-breaking change which adds functionality)
[ ] Breaking feature (breaking change which adds functionality)
[ ] Refactor (non-breaking change which changes implementation)
[ ] Messy (mixture of the above - requires explanation!)

Checklist
Put an x in the boxes that apply.

[ ] I have read the CONTRIBUTING doc
[ ] I am making a pull request against the main branch (left side). Also you should start your branch off our main.
[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have locally run services that could be impacted and they do not present failures derived from my changes

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

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

Comparison is base (04519ba) 93.49% compared to head (3782c52) 93.49%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #90   +/-   ##
=======================================
  Coverage   93.49%   93.49%
=======================================
  Files           7        7
  Lines         446      446
=======================================
  Hits          417      417
  Misses         29       29

Flag
Coverage Δ

unittests
93.49% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


[BUG] Lower tiers fluid conduits can't replace higher tiers while being used up.

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
The player can place a lower tier fluid conduit on a pressurized or ender conduit without replacing it BUT it still consume it. The targeted block must not be the conduit itself but the block under it.

Expected Behavior
It should be possible to replace a conduit to a lower tier unless there's a liquid in it. However, I believe it would annoy players, so just make sure to prevent the player to replace a conduit into its lower counterpart.
Steps To Reproduce
No response
Environment
I am currently using the version 6.0.17-alpha of the mod.

Anything else?
No response

I'm not sure I completely follow what the requested change here is. Do you want the ability to downgrade a conduit from a higher tier to a lower tier? Or is this a bug where you cannot right-click the conduit itself? Thanks!

Can not reproduce, please make a new issue if this is still happens.


Add Basic Station Time Synchronization

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Summary

It seems like the current Basic Station implementation does not include support for the LNS @timesync@ message type, so it would be great if it could be included.
This is what happens currently:
time="2020-09-21T04:21:24Z" level=debug msg="backend/basicstation: message received" gateway_id=d4adbdfffe9e2890 message="{\"msgtype\":\"timesync\",\"txtime\":4035744838}"
time="2020-09-21T04:21:24Z" level=info msg="backend/basicstation: raw packet-forwarder event received" gateway_id=d4adbdfffe9e2890 raw_id=d145516d-901f-412b-b65c-57bfa0a8d6bb
time="2020-09-21T04:21:24Z" level=info msg="integration/mqtt: publishing event" event=raw qos=1 raw_id=d145516d-901f-412b-b65c-57bfa0a8d6bb topic=gateway/d4adbdfffe9e2890/au/event/raw

What is the use-case?

The most recent Cisco IXM gateway comes with Basic Station support, using GPS and @timesync@ messages to provide timestamps. Whenever possible I prefer to have timestamps applied at the gateway as they more accurately tell when a payload has been received.
Implementation description

The LNS Protocol indicates how this is implemented.
Can you implement this by yourself and make a pull request?
I can give it a try.

If you would like to create a PR for this, that would be much appreciated :+1:
Where the LNS Protocol docs mention that the LNS must answer, I think it makes sense to let the ChirpStack Gateway Bridge respond with the gps timestamp.
The github.com/brocaar/lorawan/gps package provides an utility to convert from a time.Time to GPS time (time.Duration<EMAIL_ADDRESS>E.g.
gpsTime  := gps.Time(time.Now()).TimeSinceGPSEpoch()

Thanks @brocaar, I'll give it a try when a get a moment.


(Feature) #798 image editor should have trash icon instead of x

Description
Change the icon of delete avatar from x to trash.
Remove user name from edit avatar screen.
Add done button at the bottom of edit avatar screen which redirects user back to Edit Profile screen.
About #798
How Has This Been Tested?
Register/login to the app.
Go to profile from side menu.
Go to edit profile.
Click on the photo button(edit avatar) under the avatar.
Add a new photo if not exists.
See the updated page according to design.
Checklist:

[x] PR title matches follow: (Feature|Bug|Chore) Task Name
[x] My code follows the style guidelines of this project
[x] I have followed all the instructions described in the initial task (check Definitions of Done)
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have added reference to a related issue in the repository
[x] I have added a detailed description of the changes proposed in the pull request. I am as descriptive as possible, assisting reviewers as much as possible.
[x] I have added screenshots related to my pull request ( for frontend tasks)
[ ] I have pasted a gif showing the feature.
[x] @mentions of the person or team responsible for reviewing proposed changes

Screenshot:

@yaroslav-fedyshyn-nordwhale It seems that the trash icon is smaller than the camera icon (not in hight but in line width).

@yaroslav-fedyshyn-nordwhale It seems that the trash icon is smaller than the camera icon (not in hight but in line width).

@LiavGut I have double-checked the sizes of icons. The sizes are the same for both.

@yaroslav-fedyshyn-nordwhale It seems that the trash icon is smaller than the camera icon (not in hight but in line width).

@LiavGut I have double-checked the sizes of icons. The sizes are the same for both.

@yaroslavfedyshyn Sorry, but there's a reason why my friends call me eagle eye ;-)

@yaroslav-fedyshyn-nordwhale It seems that the trash icon is smaller than the camera icon (not in hight but in line width).

@LiavGut I have double-checked the sizes of icons. The sizes are the same for both.

@yaroslavfedyshyn Sorry, but there's a reason why my friends call me eagle eye ;-)

@LiavGut you want me to stretch the trash icon to the same size as photo icon in width? Do you realize how it will be looking? Or I don't understand what you are trying to tell me. :)

@yaroslav-fedyshyn-nordwhale It seems that the trash icon is smaller than the camera icon (not in hight but in line width).

@LiavGut I have double-checked the sizes of icons. The sizes are the same for both.

@yaroslavfedyshyn Sorry, but there's a reason why my friends call me eagle eye ;-)

@LiavGut you want me to stretch the trash icon to the same size as photo icon in width? Do you realize how it will be looking? Or I don't understand what you are trying to tell me. :)

No, of course not. I just want you to enlarge a bit (proportionally) the trash can icon)

@yaroslav-fedyshyn-nordwhale It seems that the trash icon is smaller than the camera icon (not in hight but in line width).

@LiavGut I have double-checked the sizes of icons. The sizes are the same for both.

@yaroslavfedyshyn Sorry, but there's a reason why my friends call me eagle eye ;-)

@LiavGut you want me to stretch the trash icon to the same size as photo icon in width? Do you realize how it will be looking? Or I don't understand what you are trying to tell me. :)

No, of course not. I just want you to enlarge a bit (proportionally) the trash can icon)

done

this is how it looks on the preview branch
@LiavGut @yaroslav-fedyshyn-nordwhale

actually when looking at the preview, and inspecting the elements I the camera is 22px font size
and trash is 21px.
Why are you guessing the icon size? first 20, then 21, quick look at shows its 22px.
give it the same styles as CameraButton

done

this is how it looks on the preview branch
@LiavGut @yaroslav-fedyshyn-nordwhale
actually the icons dont look centered

I have checked if the icons are centered with margins: it seems like they are. See screenshots:


2.11 version of the spec is on a branch

As per the proposed release process, we will make a branch when a version of the spec is released.

LGTM

LGTM

@pmorie your LGTM wasn't picked up as it was a review comment, not a regular comment.

@shalako @zrob @vaikas-google @fmui - can y'all take a look at this PR. One more approval is needed.

LGTM


Internal vs. Inline JS and CSS

I don't think 'inline' is an accurate term for what's described on QUICK TIP #001—INLINE MINIFIED CSS and QUICK TIP #002—INLINE MINIFIED JAVASCRIPT.
An internal stylesheet is where you don't have an external CSS file,
but instead place your CSS inside a <style> element, contained inside the HTML head.

Inline styles are CSS declarations that affect one element only,
contained within a style attribute:

REF: https://developer.mozilla.org/en-US/docs/Learn/CSS/Introduction_to_CSS/How_CSS_works#How_to_apply_your_CSS_to_your_HTML
I think for JS as well, inline implies something like a click handler on an element:
<button onclick="console.log('clicked');">Click me</button>

That's correct, css in a style element is called Internal CSS.
JS in a script element is called Internal JS.

I think this convention is commonly used when referring to critical CSS, there are a few examples here: https://web.dev/extract-critical-css/


feat: initial pama imaging service [WIP]

Adds a new service endpoint for testing the PAMA Imaging scenarios for the 2019 Connectathon in Atlanta.
TODO(barabo):

[x] Advertise a new service in the discovery endpoint
[x] Create a skeletal service which does nothing
[x] Create a skeletal unit test for the service
[ ] Scenario 1

[ ] Return appropriate for all valid combos
[ ] Unit test coverage

[ ] Add more tests