RHIROS-565 | Handle records with no operating_system data

Changes:

Added conditional init for 'os' in system detail and systems api
Added relevant test for system detail endpoint

@saltgen on frontend I have a check that if the value is null it will show N/A so in case other fields in the backend sending null as values (IIRC provider, instance type returns null in case of waiting for data state) then you can update this to send null for consistency. Let me know your thoughts!

@saltgen I would suggest the same to represent empty string or none value similar to how it is done for say, cloud_provider field and frontend can recognize and set it to N/A at it's end.

@saltgen I would suggest the same to represent empty string or none value similar to how it is done for say, cloud_provider field and frontend can recognize and set it to N/A at it's end.

@apuntamb Okay, lets stay consistent, I have also removed respective else blocks and added another test.
Change pushed!

@kgaikwad @saltgen sorry somehow the query got lost, my question was when os_data = None what will be the API response, will it be null or empty string for OS value.

@kgaikwad @saltgen sorry somehow the query got lost, my question was when os_data = None what will be the API response, will it be null or empty string for OS value.

@PreetiW -
{
    "meta": {
        "count": 2,
        "limit": 10,
        "offset": 0
    },
    "links": {
        "first": "/api/ros/v1/systems?limit=10&offset=0",
        "last": "/api/ros/v1/systems?limit=10&offset=0",
        "next": null,
        "previous": null
    },
    "data": [
        {
            "fqdn": "ip-172-31-28-69.ec2.internal",
            "display_name": "ip-172-31-28-69.ec2.internal",
            "inventory_id": "2e966f13-e300-4024-9f4a-dd2219a40efd",
            "account": "6089719",
            "number_of_suggestions": 1,
            "state": "Under pressure",
            "performance_utilization": {
                "cpu": 10,
                "memory": 12,
                "max_io": 0.314,
                "io_all": {
                    "xvda": 0.314
                }
            },
            "cloud_provider": "aws",
            "instance_type": "t2.micro",
            "idling_time": "179.70",
            "os": "RHEL 8.4"
        },
        {
            "fqdn": "iqe-ros-a84cec78-1abf-40b4-bbbf-9ddd183786bb",
            "display_name": "iqe-ros-a84cec78-1abf-40b4-bbbf-9ddd183786bb",
            "inventory_id": "8f6c2b49-0136-450a-b103-95dc6404662e",
            "account": "6089719",
            "number_of_suggestions": 1,
            "state": "Idling",
            "performance_utilization": {
                "cpu": 0,
                "memory": 0,
                "max_io": 0.314,
                "io_all": {
                    "xvda": 0.314
                }
            },
            "cloud_provider": "aws",
            "instance_type": "t2.micro",
            "idling_time": "199.70",
            "os": null
        }
    ]
}

Thanks for sharing @kgaikwad, no more queries from my side. 👍🏻

Thank you @saltgen!


Update definition of BootSplash hide on Non-Native platforms

Details
Before, we had a definition of hide that didn't support it returning a promise. Let's update it to include this so that people can use it without it breaking on non-native platforms. See issue for more on how this breaks stuff.
Fixed Issues
Fixes https://github.com/Expensify/Expensify.cash/issues/2618
Tests
Navigate to the sign-in page while logged out. Verify that it renders correctly.
QA Steps
Same as above tests.
Tested On

[x] Web
[x] Mobile Web
[x] Desktop
[ ] iOS
[ ] Android

Screenshots
This only modifies an index.js file (and not its corresponding index.native.js file, so no native screenshots are needed.
Web

Mobile Web

Desktop

LGTM! Merging now so that it can go out on the next deploy.

🚀 Deployed to staging in version: 1.0.33-1🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


opt js load and preset variables

add preset variables:

current version
versions file location

:tada: This issue has been resolved in version 0.0.9 :tada:
The release is available on:

v0.0.9
GitHub release

Your semantic-release bot :package::rocket:


Show HierarchicalResourceQuota status by kubectl get

What type of PR is this?:
/kind feature
What this PR does / why we need it:
We have added kubectl-hns hrq sub-command to show HierarchicalResourceQuota status in easy way.(#283)
However, it's not yet possible to achive that using the Kubernetes standard way kubectl get hrq.
$ kubectl get hrq -n test
NAME       AGE
test-hrq   80s

So in this PR, I made it possible to show HierarchicalResourceQuota status by using kubectl get hrq.
$ kubectl get hrq -n test
NAME         REQUEST                                              LIMIT
test-hrq     requests.cpu: 500m/1, requests.memory: 100Mi/200Mi   limits.cpu: 500m/2, limits.memory: 100Mi/500Mi
test-hrq-2   requests.memory: 100Mi/300Mi                         limits.cpu: 500m/1

$ kubectl get hrq --all-namespaces
NAMESPACE   NAME         REQUEST                                              LIMIT
test-2      test-hrq-3   requests.cpu: 0/4, requests.memory: 0/500Mi          limits.cpu: 0/8, limits.memory: 0/750Mi
test        test-hrq     requests.cpu: 500m/1, requests.memory: 100Mi/200Mi   limits.cpu: 500m/2, limits.memory: 100Mi/500Mi
test        test-hrq-2   requests.memory: 100Mi/300Mi                         limits.cpu: 500m/1

Tested:
Unit and E2E tests has passed.
I also added test case to HRQ E2E test which is now Pending.
Related:
#274
#283
#289
Special notes for your reviewer:
HierarchicalResourceQuota documentation is proceeding by @zfrhv and @adrianludwin.(#289)
If this PR will be merged to v1.1, we should add this feature to it.

/assign @adrianludwin

@adrianludwin
OK, I've fixed it.

/ok-to-test
This lgtm but I'll let @adrianludwin have final say.

I've fixed conflict.
Could anyone please add /lgtm again?


useRouter access from next-i18next

Describe the bug
I'm not able to import useRouter from next while it's supposed to be exposes by next-i18next.
Occurs in next-i18next version
Here my dependencies
{
  "express": "^4.16.4",
  "i18next": "^14.0.1",
  "next": "^9.0.3",
  "next-i18next": 1.1.2",
  "react": "16.9.0",
  "react-dom": "16.9.0"
}

Steps to reproduce
Here a codesandbox with minimal configuration to reproduce the issue.
Expected behaviour
As described from nextjs documentation, useRouter is a hooks version of Router class.

If you want to access the router object inside any component in your app, you can use the useRouter hook

As described from next-i18next documentation, next-i18next should shares the same API as the native Next Router.

The exported Router shares the same API as the native Next Router.

I expected to have access to useRouter hooks from i18n instance created by next-i18next.
import { useRouter } from "../i18n"; // undefined
Additional context
Add any other context about the problem here.

Is this a duplicate of #316? I'd expect that work to be done at the same time. Contributions welcome.

Any news on this one?

I think an implementation idea would be to do the exact thing that nextjs is doing, so basically.
Start by creating a context for the router instance: https://github.com/zeit/next.js/blob/canary/packages/next/next-server/lib/router-context.ts
Wrap the app with this context, we can do that inside the appWithTranslation, we already have the router available there, we can call our wrapper there and pass it to the context provider.
https://github.com/zeit/next.js/blob/canary/packages/next/client/index.js#L309
withRouter will simply use the context and pass the router as a prop to the composed component:
https://github.com/zeit/next.js/blob/canary/packages/next/client/with-router.tsx
useRouter just returns the result from useContext
https://github.com/zeit/next.js/blob/canary/packages/next/client/router.ts#L115
Doesn't seem too difficult, I don't have the time to open a PR right now though, let's discuss if this sounds like a good implementation plan, ping @capellini @isaachinman

Going to close as a duplicate of #316. If anyone is interested in doing this work, please do let me know.


can not build client ressources anymore: bootstrap-container definitions maybe need "!default" apendation fix, since design-system-styles v 6.4.2

Hy Post-Design-System-Team!
I’m struggling with updating the design-system-styles component above version 6.4.1. (6.4.1. still works, but 6.4.2 or up to current 7.1.0 does not work anymore.)
It has to do with a change/override at the container component of bootstrap.
Context of our setup:
We use following components successfully:
    "@swisspost/design-system-components": "7.0.0",
    "@swisspost/design-system-icons": "7.0.0",
    "@swisspost/design-system-styles": "6.4.1",
    "@swisspost/internet-header": "1.9.1",

If we want to update the “internet-header”, (and thus by new dependencies/versioning all other packages), it has a new internal dependency to “design-system-styles” 7.1.0.
To make the design-system-styles 7.1.0 running, we also had to add manually the dependency
"@stencil/core": "4.18.3",
Because it wasn’t available/resolvable within the design-system-styles package itself. (at least, that was, what our webpack-builder was noting)
We also add our own bootstrap package in addition to your design-system-packages. (We use your styles, but also have our own overridden definitions, based, on the original bootstrap (5.3.3.) definitions.)
The following error makes it impossible for us to build our definitions anymore. The “container” change after design-system-styles 6.4.1 makes it impossible to override the container definitions again.

[./node_modules/<EMAIL_ADDRESS>2.51 KiB
[./node_modules/@stencil/core/internal/client/](./node_modules/@stencil/core/internal/client/) lazy ^\.\/.*\.entry\.js.*
$ include...(truncated) 160 bytes Module build From line 30, column 13 of node_modules\bootstrap\scss\_containers.scss:
╷30 │             @extend %responsive-container-#{$breakpoint};   │             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
╵You may not @extend selectors across media queries.

I was wondering, if you could define your container related overrides with a !default apendation, so they could be overridden again at a later compiling-stage, e.g. by the original bootstrap or our own definitions.
Example for form-switch definitions, we had to override with !default definitions, so the original bootstrap behaviour was still working. But this way, on special circumstances, we could still override the default behaviour later:
$form-switch-width: 2em !default;
$form-switch-height: 1em !default;
$form-switch-padding-start: $form-switch-width + .5em !default;
$form-switch-border-radius: $form-switch-width !default;
$form-switch-transition: background-position .15s ease-in-out !default;
$form-switch-unchecked-bg-position: left center !default;
$form-switch-checked-bg-position: right center !default;

Is it somehow possible to use !default definitions on the container definitions/breakpoint definitions/overrides, so we can still override them again, if needed?
Not sure, if this would be a solution, but it would be nice, if we could still override your definitions with our own bootstrap-based definitions, otherwise, we would have to change so much on our several web-portals, if we only want to use your design-system, without using our own bootstrap-based controls. (we need native bootstrap functionality for other things too.)
Greetings
Adrian

Hey @aedu-hostettler, thanks for submitting this issue. Out of interest, what are you overriding the container styles with? What are your requiremenets for containers? We'll have a look at the request and update as soon as possible.

@aedu-hostettler can you once try to update our packages to 7.4.2 and the internet-header to 1.14.3?
Do you have the same issue then?


hiddenbar: init at 1.9

Description of changes

macOS utility that helps hide menu bar icons https://github.com/dwarvesf/hidden
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Oh, it was done in #302093, sorry...


Fix documentation search

Weirdly enough, the problem is still the same as discussed here: https://github.com/sphinx-doc/sphinx/issues/5460
This is a quick and dirty fix that copies the Sphinx RTD theme layout and adds the missing import.  This is bad because when updating the RTD theme layout package, you have to re-copy the layout and re-add the import (at least until they fix the bug).
There's probably a more maintainable way to add the missing import;  I'll do a little poking around Sphinx and the RTD theme to figure it out.

closed because #17 is a better way to fix search

closed because #17 is a better way to fix search


Chrome does not render new app

After running reapp new myapp and then reapp-run, when I visit http://localhost:3010/ in Firefox, I see the basic app, but visiting in Chrome I see nothing.
Developer tools is showing the following error:
Uncaught TypeError: Object.getPrototypeOf called on non-object

Could you paste where it reports the line of the error?

I have updated reapp and it works now .. highly likely it was some fail of mine :panda_face:


Fix sonatype issues

This is an attempt to fix as many easily correctable issues discovered by sonatype as possible.

I'm not sure how to fix the rest of the reported issues.

Thanks. It may be easy to get wrong XML from attacked mirror site. We need to protect ourselves.


bonus: vi vs emacs warz!

Just copy and paste some old references, since bonus points don't expect it inside our docs xD

https://twobithistory.org/2018/08/05/where-vim-came-from.html

https://www.youtube.com/watch?v=ayc_qpB-93o&t=4m58

https://www.youtube.com/watch?v=leBUlnNQ8OM


Add region prompt for next iteration

While going through a major refactor for DAB support, I think it would not only be best to move "region" to a global/general setting, but I think it's prudent to prompt the user during load to choose between "North America" and "World" one time.
"North America" should default to allowing FM, HD Radio, and Weather Radio channels and disable DAB.
"World" should default to allowing FM and DAB and disable HD Radio and Weather Radio.
The user must remain free to enable/disable other modulations at their discretion, and nothing in the channels database should be removed based on this prompt.
This is coming up as a result of refactoring the "add channel" operation for DAB, which in turn affects Weather Radio. Moving forward, the user will need to select a named channel for these modulations and will not be able to enter a frequency anymore. The frequency selection dialog will need to know in advance what the region is to limit key entry of analog FM for "North America" and HD Radio to those same key entry restrictions.
DAB and Weather Radio are being converted into a selection dialog based on a new static database table that defines the available frequencies for these modulations.

Implemented via https://github.com/djp952/pvr.rtlradio/commit/9cda17066feca70c0b0ccf0c434766c939a2caed


Install Jbang automatically

currently, it requires that JBang is available on command line.

Any tips how to make it?

Some examples of VS Code extensions downloading binaries automatically:

VS Code Kubernetes https://github.com/vscode-kubernetes-tools/vscode-kubernetes-tools/blob/master/src/components/installer/installer.ts
VS Code Camel K (not a very good example but the one I know the most). https://github.com/camel-tooling/vscode-camelk/blob/main/src/installer.ts
VS Code OpenShift connector

Thanks @apupier


Item ID for the SPT32 Grad - Overwatch Unique Sniper?

I looked all over the Item Hash List but couldn't find it (Overwatch sniper). What is it called, and how do I add it to my inventory? Please help, thanks.

does it not have an item hash?

Overwatch is Panam's weapon. The item name is Preset_Grad_Panam. Add with Game.AddToInventory("Items.Preset_Grad_Panam", 1)
Opening an issue here may not be an appropriate place to ask about specific item codes, as inventory hacking is not the primary focus of this mod. Item lists are freely available from this wiki, or from other resources such as https://www.nexusmods.com/cyberpunk2077/mods/186


[native_assets_builder] Release 0.6.0

coverage: 92.009%. remained the same
when pulling 17757e3d6667b59cddc6a3f8ea596138fcc6bdbc on nab-0.6.0
into b59a55d688a8ebdaa9c1234f3d6a4c7ac310dd3f on main.


Not functioning as of 1.19.3

When I attempt to use the following code
import { InstantConnectProxy } from 'prismarine-proxy'
const proxy = new InstantConnectProxy({
  loginHandler: (client) => {
    console.log(client.username)
    return {
      username: 'ice',
    };
  },
  clientOptions: {
    host: 'localhost',
    version: '1.19.3'
  },
  serverOptions: {
    version: '1.19.3',
    port: 25566
  }
})

proxy.on('incoming', (data, meta, toClient, toServer) => {
  toClient.write(meta.name, data)
})

proxy.on('outgoing', (data, meta, toClient, toServer) => {
  toServer.write(meta.name, data)
})

I fails with the error
Disconnecting client because error TypeError [ERR_INVALID_ARG_TYPE]: Serialization error for play.toClient : SizeOf error for undefined : The "string" argument must be of type string or an instance of Buffer or ArrayBuffer. Received undefined
    at new NodeError (node:internal/errors:387:5)
    at Function.byteLength (node:buffer:740:11)
    at Object.string (eval at compile (C:\Users\dihydromonoxide\Desktop\Node Projects\minecraft-bot\minecraft-bot\node_modules\protodef\src\compiler.js:259:12), <anonymous>:45:25)
    at Object.packet_respawn (eval at compile (C:\Users\dihydromonoxide\Desktop\Node Projects\minecraft-bot\minecraft-bot\node_modules\protodef\src\compiler.js:259:12), <anonymous>:2269:27)
    at eval (eval at compile (C:\Users\dihydromonoxide\Desktop\Node Projects\minecraft-bot\minecraft-bot\node_modules\protodef\src\compiler.js:259:12), <anonymous>:3083:54)
    at packet (eval at compile (C:\Users\dihydromonoxide\Desktop\Node Projects\minecraft-bot\minecraft-bot\node_modules\protodef\src\compiler.js:259:12), <anonymous>:3131:9)
    at CompiledProtodef.sizeOf (C:\Users\dihydromonoxide\Desktop\Node Projects\minecraft-bot\minecraft-bot\node_modules\protodef\src\compiler.js:89:14)
    at e.message (C:\Users\dihydromonoxide\Desktop\Node Projects\minecraft-bot\minecraft-bot\node_modules\protodef\src\compiler.js:96:40)
    at tryCatch (C:\Users\dihydromonoxide\Desktop\Node Projects\minecraft-bot\minecraft-bot\node_modules\protodef\src\utils.js:50:16)
    at CompiledProtodef.createPacketBuffer (C:\Users\dihydromonoxide\Desktop\Node Projects\minecraft-bot\minecraft-bot\node_modules\protodef\src\compiler.js:96:20) {
  code: 'ERR_INVALID_ARG_TYPE',
  field: 'play.toClient'
}

I am using a lan world with the port 25565, this was also tested with a server

It appears as though this has to do with the respawn packets and commenting out the following lines in instant_connection_proxy.js seems to have fixed it
//toClient.write('respawn', { ...data, dimension })
//toClient.write('respawn', data)


Javadoc GETTER/SETTER sections not working on Android Studio

I have followed the documentation here:
https://projectlombok.org/features/GetterSetter.html
Example:
/**
 * Age of the person.
 *
 * -- GETTER --
 * Returns the age of this person.
 *
 * @return The current value of this person's age.
 *
 * -- SETTER --
 * Changes the age of this person.
 *
 * @param age the new age for this person.
 */
@Getter @Setter
private int age;

And this is the result:

Am I doing something wrong? Is this issue related to Lombok itself and not the plugin?

Probably the lombok plugin just does not handle javadoc yet.

But shouldn't Lombok itself generate the individual getter/setter methods and copy/paste each Javadoc section to each of the methods? This way, the plugin wouldn't need to do a thing because IntelliJ would simply pick the documentation for each method.
Or am I misunderstanding something?

Intellij does not execute any annotation processor (like lombok) when displaying javadoc or creating errors / warnings / etc.


User Story for download statistics

As a service manager of ScholarSphere, I need to know how often items have been downloaded in the past week / month / year, so I can show impact as part of promoting the service.

Is the google analytic report for file downloads enough? https://analytics.google.com/analytics/web/#report/content-event-events/a33252017w60487619p61852754/%3F_u.date00%3D20160501%26_u.date01%3D20160710%26explorer-table.secSegmentId%3Danalytics.eventAction%26explorer-table.plotKeys%3D[]/


RC.8 fails on ng build --prod

Version info
Angular: 6.0.1
Firebase: 5.0.2
AngularFire: 5.0.0-rc.8.0
How to reproduce these conditions

Follow AngularFire2 setup instructions
Run ng build --prod

Debug output
ERROR in Error during template compile of 'AppModule'
Function calls are not supported in decorators but 'AngularFireModule' was called.
Expected behavior
Build should run successfully.
Actual behavior
Build fails only in production (AOT)
Problem
I don't think the Angular compiler can analyze code dynamically in initializeApp. These lines are the source of the problem
https://github.com/angular/angularfire2/blob/a8414f5d62ec073db760e0b7e9346e4d946bd3a3/src/core/firebase.app.module.ts#L50-L51
Possible Solution?
Require the second arg to initializeApp to be a FirebaseAppConfig object and remove the string union type.

Just noticed this is a duplicate. Will close for #1635


Text Generation Web tool (website)

Project Name
Text Generation Web tool (website)
Project Description
Given a text input this tool should recognize the context and create a paragraph. You can create your own models and use them to create a web based interactive tool or you can even use APIs from hugging face or other open-source models.
Project Type:

NLP, Text processing

Target Folder
GenAI-Projects/Text-Generation/<your_project_folder>

Mention the technology stack, and how your are going to go through with this project.

My Tech Stack -
HTML , CSS , JS , React
GAN, streamlit , Langchain ,  Used openai , palm, gemini api
using prompting we can do this

Go head with your project I have assigned it to you

Thank you so much.
I'll try my best👍🏻👍🏻

Create your own branch and give a unique folder name and add your project folder in the
GenAI-Projects/Text-To-Speech/<your_project_folder>

Since other people will be creating similar projects try to give a unique name to the folder.
Feel free to ask anything, we can discuss on discord.

Sure..
From tomorrow I'll start working on it
sanketshinde1497
Here is my discord

Sure.. From tomorrow I'll start working on it
sanketshinde1497
Here is my discord

Just say hi on this project channel in the SWOC server, so there will take all related discussions for this repo

hi, sorry for inactivity .

Yes @sanketshinde3001 please do. Otherwise I'll  have to reassign

Completed it. Sent link of it on discord . Plz check it

Great work @sanketshinde3001  ✨✨ It's a great tool.
Make a PR for this. Make sure to mention the isse in the PR and a ss of the site

I will add exceptional handling code in it till tomorrow afternoon.
And Make PR asap..
Thank you for giving me time to make this😄


Deploy in Xilinx FPGA

Dear,
I am trying to deploy these models for Xilinx FPGA using the FINN framework. Do you have in your knowledge any incompatibility? Currently, I am having many issues while converting to the ONNX format.
Thanks

I am sorry but we don't have knowledge in ONNX export or FPGA deployment :( Maybe you can have a look at this framework daBNN? I saw an Bi-Real-Net ResNet deployment example there.


Bogus column and row at edge?

There seems to be an empty column and row at the edge of aggregated arrays:
import pandas as pd
import numpy as np
import datashader as ds
import datashader.transfer_functions as tf

df = pd.DataFrame(dict(x=np.random.normal(0,10,10000),y=np.random.normal(0,10,10000)))

canvas = ds.Canvas(plot_width=8, plot_height=8, x_range=(-4,4), y_range=(-4,4))
agg = canvas.points(df, 'x', 'y')
agg.values

array([[21, 13, 21, 26, 17, 18, 20,  0],
       [22, 16, 28, 23, 17, 16, 23,  0],
       [17, 20, 22, 20, 20, 19, 26,  0],
       [16, 14, 28, 14, 12, 16, 22,  0],
       [16, 17, 16, 25, 19, 18, 18,  0],
       [ 9, 24, 24, 24, 17, 14, 17,  0],
       [19, 24, 17, 19, 13, 12, 21,  0],
       [ 0,  0,  0,  0,  0,  0,  0,  0]], dtype=int32)

The extra row and column ends up being transparent, at least for counts, so it has little effect, but it still seems like we should be returning a fully filled-out array.

After looking at the code, it looks to me like the handling of the last column has multiple issues.  For example:
import pandas as pd
import numpy as np
import datashader as ds
import datashader.transfer_functions as tf

num=10000

df = pd.DataFrame(dict(x=[0.1, 0.5, 1.1, 1.5, 2.2, 2.9],
                       y=[0.1, 0.1, 0.1, 0.1, 0.1, 0.1]))

canvas = ds.Canvas(plot_width=3, plot_height=3,
x_range=(0,3), y_range=(0,3))
agg = canvas.points(df,'x','y')

Here there are six data points, all within the specified range, and I would expect them to fit evenly into the pixel array as follows:
  0  1  2  3
0 _____________
  |. .|. .|. .|
1 |---+---+---|
  |   |   |   |
2 |---+---+---|
  |___|___|___|
3

Instead the current code yields:
array([[3, 3, 0],
       [0, 0, 0],
       [0, 0, 0]], dtype=int32)

I.e., all six datapoints are being squeezed into only two of the pixels, which is difficult to justify from their coordinates.  The correct answer can be obtained by removing a bogus "-1":
diff --git a/datashader/core.py b/datashader/core.py
index c1c3fcf..574aa9f 100644
@@ -61,7 +62,7 @@ class Axis(object):

         """
         start, end = map(self.mapper, range)
-        s = (n - 1)/(end - start)
+        s = n/(end - start)
         t = -start * s
         return s, t

yielding:
array([[2, 2, 2],
       [0, 0, 0],
       [0, 0, 0]], dtype=int32)

Great! However, there are then problems with the upper bound.  According to the docstring for ds.core.Axis, the range is meant to be inclusive on both ends, and indeed if the point (3.0,0.1) were to be included in the dataframe above, the original code will count it in the last cell, yielding [3,3,1].  But with the change from n-1 to n, this new datapoint ends up very dangerously overwriting the next cell in memory:
array([[2, 2, 2],
       [1, 0, 0],
       [0, 0, 0]], dtype=int32)

Obviously, the result is then not just incorrect, but if that point is at the end of the array then segfaults are likely.  To fix this, we can define the upper bound to be exclusive:
diff --git a/datashader/core.py b/datashader/core.py
index c1c3fcf..574aa9f 100644
--- a/datashader/core.py
+++ b/datashader/core.py
@@ -49,8 +49,9 @@ class Axis(object):
         Parameters
         ----------
         range : tuple
-            A tuple representing the boundary inclusive range ``[min, max]``
-            along the axis, in data space.
+            A tuple representing the range ``[min, max]`` along the axis, in data space.
+            min is inclusive and max is exclusive.
+
         n : int
             The number of bins along the axis.

@@ -61,7 +62,7 @@ class Axis(object):

         """
         start, end = map(self.mapper, range)
-        s = (n - 1)/(end - start)
+        s = n/(end - start)
         t = -start * s
         return s, t

diff --git a/datashader/glyphs.py b/datashader/glyphs.py
index a61e66b..e765fa1 100644
--- a/datashader/glyphs.py
+++ b/datashader/glyphs.py
@@ -57,7 +57,7 @@ class Point(_PointLike):
             for i in range(xs.shape[0]):
                 x = xs[i]
                 y = ys[i]
-                if (xmin <= x <= xmax) and (ymin <= y <= ymax):
+                if (xmin <= x < xmax) and (ymin <= y < ymax):
                     append(i,
                            int(x_mapper(x) * sx + tx),
                            int(y_mapper(y) * sy + ty),

But this is a change in semantics, and it may be surprising to users that points on the upper boundary are not included in the counts.  Still, it seems like there would need to be quite a lot of extra code and checking to somehow ensure that points on the upper boundary are included if the mapping is done correctly, since no other cell boundary can act that way (or else points in interior cells would be counted twice).
See my HoloViews continuous coordinates tutorial for a detailed analysis of these issues; it's focusing on making a regular grid of samples from a continuous function, but basically the same approach applies to irregular samples from a continuous function as we have here.
For what it's worth, in the code used in HoloViews we decided many years ago that it was simpler and more consistent (both self-consistent and consistent with how arrays work in Python/Numpy) to make all upper boundaries exclusive.
If we do so here, we will then need to update the test data.


Error on unpack with unused Dataverse data source

[x] Please attach a sample .msapp to this Issue which demonstrates the issue you are experiencing. (Please do not attach any file which may contain sensitive personal or company data.)

Manage Your Groups.msapp.zip
Describe the bug
A clear and concise description of what the bug is.
Please mention if it is reproduced using PAC CLI or PASopa. (Please mention the version number, if possible)
I was able to identify the issue as I was writing the bug report.
Cause & Workaround
I had an unused Dataverse Data Source in the application. Once I removed the Data Source, I was able to successfully unpack the solution.
See working msapp file (with dataverse data source removed):
Manage Your Groups - Unused Data Source Removed.msapp.zip
Original Bug Report
Getting error when unpacking attached msapp file.
Encountered bug when running Power Platform Build Tools as part of ALM Accelerator for Power Platform
Task         : Power Platform Unpack Solution
Version      : 2.0.7
Then exported msapp file manually and was able to reproduce using pac
Microsoft PowerPlatform CLI
Version: 1.18.4+g1a00e63
Downloads pac canvas unpack --msapp "Manage Your Groups.msapp" --sources src
Warning: canvas unpack is in preview, and functionality is not guaranteed. Use extreme caution if using in a production environment. For more information see https://aka.ms/paccanvas
Unpack from /Users/lukask/Downloads/Manage Your Groups.msapp --> /Users/lukask/Downloads/src
  source format version: 0.24
Warning PA2001: Checksum mismatch. Checksum indicates that sources have been edited since they were unpacked. If this was intentional, ignore this warning.

You have found a bug; this is not a specific bug, rather an indicator that some bug has been encountered.
Please open an issue and log the entirety of this error log at https://github.com/microsoft/PowerApps-Language-Tooling

To Reproduce
Steps to reproduce the behavior:

Download attached msapp file
Run pac canvas unpack --msapp "Manage Your Groups.msapp" --sources src

Screenshots
If applicable, add screenshots of error messages to help explain your problem.
Additional context
Add any other context about the problem here.
Add any helpful context on the .msapp like created from teams, contains custom pcf components, etc.
Application contains code and canvas components from Creator Kit
Code components:

Fluent Command Bar (1.0.8)
Fluent Details List (1.0.8)
Fluent Elevation (1.0.8)

Library components:

Panel
SearchBox

Connectors:

Office365Users
Office365Groups
Dataverse (Accounts) -- This was the culprit.

Marking this as 'in next release' and will be completed once changes are in next release of the Power Platform CLI.

Closing this issue, since the fix is out in the latest releases.


Create ThreadUtils for getting threads by name

@bbende This is another way to get the thread.  If you are interested.

This change is

@ottobackwards thanks, I considered doing something like this, but I'm not sure we can rely on the name. Every HDFS processor will have a thread with the same name so we won't know which one is for the specific processor being stopped.

No problem, just a side thing to do on a Saturday morning :)


Question:  How is the "plugin" system standardized

    // Provides a nice, tested, standardized way of adding plugins to a `Bookshelf` instance,     // injecting the current instance into the plugin, which should be a module.exports.
at bookshelf.js:109, it says it's a standardized way, is that statement true in some way outside of the bookshelf library?  If so, what's the source for the idea?
If it's standardized for just Bookshelf, I think it's a great idea, I just wanted more info.

I'm almost certain it's standardized just for Bookshelf. Do you need info on how to create a plugin?

Nope, I saw how it works for bookshelf. I just wanted to get a source of where the "standard" came from.  And I'll make sure to look through the labels first next time.


Merge new_cluster into master

Changes:

Add confignode package
Add thrift-confignode
Add DeviceGroupHashExecutor interface

Coverage increased (+0.01%) to 67.897% when pulling 7cff204c008d6dffb70766b5324ce7ed65141238 on new_cluster into fd381934eba3d5e8ca04c8b830ef011416d5ff37 on master.


DNN-6562 Add cache buster to logo uri so that site management may add ca...

...ching headers to logos

I'm withdrawing this request because I think I have an improved solution.


Remove unnecessary value list in null-related filters

value: [] used to be required but then we relaxed the @filter definition to allow it to not be present.

Does any change need to happen with read-the-docs documentation as well?

Does any change need to happen with read-the-docs documentation as well?

I think this content also exists in docs/source/language_specification/query_directives.rst, so probably yes.

The CI passed but the webhook is broken, so feel free to override!

d'oh-- fixed!


New Romainian Named Entity Corpus (RONEC)

Hello,
Me and my colleague created a new named entity corpus for the Romanian language - RONEC. The paper (recently accepted at LREC2020) can be found at https://arxiv.org/abs/1909.01247 and the repository at https://github.com/dumitrescustefan/ronec.
Could you add our corpus along the others at the Romanian language?
Thank you,
Avram Andrei-Marius

I just added it.  Thanks for the recommendation!


Release pipeline

Trying out a new release pipeline that is built on standard circleci practices as opposed to goreleaser.

This turns out to be much more involved than expected. Goreleaser is likely a better choice unless we build and maintain our own tool.


InvisibleMan Not Connect

Its Not Connect To Any Location Please Update It

Why can't I connect for so long?


Allow inline bools

Right now, if you want to pass a 0 or 1 to most nodes, you need a logic bool node running to it.
It'd be nicer if you had the option to use an input widget in cases where you want to just set the value directly. I've only done image input because that's the one I use the most but should be pretty easy to implement this across all the logic nodes.

This isn't a complete PR since it only effects one node and wouldn't introduce confusion.

Yup - I thought I'd double check before making this change across all nodes. If I adjusted all of them to be widget-able, would that work?


my side of the story

Hi @sarawildenor ! Thanks for your submission.
Unfortunately it seems that there were changes in the master that conflict with yours.
Could you perform a merge as described in Tutorial 5 slides 84-96 and integrate those changes?
We do not want to lose any of them, so you can keep what others have written and add yours.
You're free to reorganize the content so that it still makes a story, but please do not modify
the content of what others have written.

Hi Florido!
Hope this is ok!


Adding explanation for metricsProvider.enabled

Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

Hello @Sagar2366,
Thanks you for your contribution !
Can I ask you to bump the version inside Chart.yaml, to add an entry in CHANGELOG.md and and to add a “signoff” entry in the commit message (git commit --amend --signoff should do the job) ?


electron-cash: ease protobuf dependency

Motivation for this change
We have protobuf 3.12 in the master so electron-cash is not building anymore. This change eases the dependency requirement and fixes the build.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

There is a new version 4.1.0 which does not suffer from this. Will do a version update PR instead.


Add support for sparse matrices.

The models should support sparse adata.X.

Using scvi-tools dataloader at the very least makes this trivially easy

https://github.com/theislab/scarches/pull/124


Various improvements

Improved Grid component API (for example, it can automatically determine rows and columns now)
Added 'rasters' (those gridlines that extend axis ticks)
Implemented clipping functionality from PR #177

Nice work! Three comments:

There was an empty unit test for the updates in the Grid component. Did you mean to add some tests there? I have removed it temporarily so the CI will pass.
The 'raster' component is a great addition! I'm not sure about its naming though since it deviates a bit from the most commonly used names for these (grid lines, guides) and might also be confusing to those coming from mapping as 'raster' has a specific connotation there. Shall we rename to guides perhaps?
Can you add a simple integration test for the new raster/guide component? I'd like to make sure we have at least a simple test for each main component to ensure things don't radically break. Might also be nice to add something to the sandbox for future dev (you probably already have something like it created from when you were implementing the feature?).


Incorrect magicnum and size in hex for block test

A block serialization bug was fixed in https://github.com/bitpay/bitcore/pull/938 and the tests here need to be updated also, this is needed before upgrading to the latest version of bitcore.

Fixed with https://github.com/bitpay/bitcore-p2p/pull/11


Fixes #1186 - Race condition due to fast zooming

As reported in #1186, zooming in/out too fast raises some exceptions, and makes some pages not render unless the user scrolls. These two issues are fixed here.
Basically I'm removing the item.rendering check and a bug with my recently introduced loadingIconDiv element. Removing that check from the queue doesn't seem to be a problem for most PDFs I've tried -- no duplicate drawing/rendering ensues.
PS: I really wish we were using a more organized MVC structure in viewer.js... debugging this was an exercise in following a web of window events and function calls. Perhaps when we have a new UI we can revisit the overall code structure? :)

TEST-UNEXPECTED-FAIL | eq issue840 | in firefox | rendering of page 1 != reference rendering

See #923


Adopt vscode-oniguruma

All measurements done with the setting:
"editor.maxTokenizationLineLength": 200000000

VS Code Desktop

File
Size
Lines
Before
After
Difference

checker.ts
2.2MB
38127
15990
5551
2.9x faster

sqlite3.c
7.7MB
228634
23241
12189
1.9x faster

bootstrap.min.css
152.1KB
7
1653
6900
4.1x slower

cpp.tmLanguage.json
477KB
16221
416
139
3x faster

VS Code Web (Chrome Version 81.0.4044.122)

File
Size
Lines
Before
After
Difference

checker.ts
2.2MB
38127
21255
5573
3.8x faster

sqlite3.c
7.7MB
228634
35981
11196
3.2x faster

bootstrap.min.css
152.1KB
7
--
5100
--

cpp.tmLanguage.json
477KB
16221
261
136
1.9x faster

@alexdima are these before/after measurements in milliseconds or something else?
Do you happen to have millisecond measurements, or info on how you measured? Curious how this approach compares to https://github.com/trishume/syntect :)

Also curious if you have any ideas why bootstrap.min.css became 4.1x slower on VS Code Desktop?

Came here from link in release notes.
I have massive respect for you @alexdima + vscode team's ever prioritization of performance. Really cool that this uses webassembly under the hood. Wow!

This is insane! Keep it up!!

@slimsag Yes, the measurements are in millis. I got them by running F1 > Developer: Force Retokenize which flushes any cached tokens and synchronously (without yielding) tokenizes the entire file top-bottom. It then prints the time taken to the developer tools... [Normally tokenization runs for some time, then yields to allow the event loop to process events, then tokenizes again, etc].
https://github.com/trishume/syntect looks like a great project. I'm not sure how times can be compared objectively. The current JS TM grammar that VS Code ships with is not written with speed in mind (it is basically the entire TS TM grammar with some minor things removed). Philosophically, it is also written more like a parser, which leads sometimes to some extraordinarily complex regular expressions --  e.g.). Matching such regular expressions is no breeze for any regex engine:

Regarding bootstrap.min.css, I honestly didn't have a lot of time to dig deep into it, since I was so happy with the wins for the regular files. Minified code is something very specific to the web ecosystem (CSS/JS/HTML), and VS Code often cheats by ignoring such long lines. There is a setting to forcefully enable parsing of those, but that is used rarely in practice, since most folks end up opening a minified file due to a breakpoint or to check something out, and rarely have the intention of really editing the minified code.


Error Downloading from command line

I've been using Raccoon by launching it in the command line for a month and had no problem, but, since yesterday, everytime I try to download an apk by using it, it just prompts me on the terminal
!fail.Target host is null!

have you solve this problem? i met this problem too.

example app?

I have received this error when I try to download an incompatible app onto an unsupported device. For example, trying to download TikTok (com.zhiliaoapp.musically) onto an x86 based device.

OK, TikTok is ARM only, so what do you expect to happen?

OK, TikTok is ARM only, so what do you expect to happen?

Sorry, I should have clarified that I was providing context to OPs issue. I'm aware that TikTok is ARM only. I was merely indicating that the error code mentioned in the post appears in such scenarios. I do not have any issue downloading from Raccoon at the moment.


Update default admin group for self managed install

NOTE: PR is opened against isv-and-4907 branch
Description

How Has This Been Tested?

Merge criteria:

[ ] The commits are squashed in a cohesive manner and have meaningful messages.
[ ] For commits that came from upstream, [UPSTREAM] has been prepended to the commit message.
[ ] JIRA link(s): https://issues.redhat.com/browse/RHODS-5394
[ ] The Jira story is acked.
[ ] Live build image:
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work.
[ ] QE contact acknowledges that this has been tested and is approved for merge.

/hold
Until https://github.com/red-hat-data-services/odh-deployer/pull/260 PR is merged

@VaishnaviHire Two small changes after your update

Thanks, updated the PR


[runtime] Fix lookup of NSObject : Hashable conformance.

@jckarter, any suggestions for a test that might exercise this path? The symbol name was wrong when it was first written so I assume it has never been tested.

(I added a call to getNSErrorConformanceToHashable() at a random place in the runtime to verify that the new lookup does find the symbol, but we can't put that into the test suite.)

I think this code path doesn't ever get exercised. We only call getHashableConformance() in order to implement the -isEqual: method on _SwiftNativeNSError by forwarding to the == operation in the underlying bridged Swift error value. If you're using an actual NSError, then you'd of course go through the default - [NSError isEqual:] implementation. We could probably drop this code to look up the Hashable conformance and instead assert that we only ever ask for the Hashable conformance of a _SwiftNativeNSError.

Thanks, Joe. I'll push this change as-is and file a followup to investigate whether we need this code at all.

@swift-ci please smoke test

@swift-ci please smoke test linux

@swift-ci please smoke test

@swift-ci please smoke test macosx


Faster integration tests

This PR introduces a number of changes in order to reduce the time needed for integration tests. Part are just simulating less when running tests and part are actual optimisations like opening just one connection to the DB, etc.
Fixes #743.

@orelgueta - should I review it or wait until you fixed the doc issue?

Fixed the doc issue now, never dealt with docstring issues, it's so sensitive...


Dliddell fix expand strided

Not to be checked in!
2nd attempt at sharing code changes after CLA failure

(you're gonna have to figure that out ;)
also, pushing to the main repo notifies any watchers - if you are pushing here then please use proper PR titles and descriptions - if you just need to share changes you can do that from your own fork

(you're gonna have to figure that out ;) also, pushing to the main repo notifies any watchers - if you are pushing here then please use proper PR titles and descriptions - if you just need to share changes you can do that from your own fork

Thanks, Ben!  Clearly, I don't know what I'm doing.  I thought I'd just share some changes with Nirvedh, so I opened it as a draft, with the intention of cleaning it up for check in later.  I guess I don't understand the point of a draft if it has the same heavyweight process as a non-draft PR.  Will close this one and open one on my fork.
Also, it looks like I also have a couple of CLA problems which I don't know how to fix.  I don't have these problems with torch-mlir or iree-amd-aie.

drafts are for not-yet-ready-for-review code that will eventually be turned into a real PR that you expect to land. if you're sharing with a broad group and going to request feedback from multiple people then here is fine though it's still best to do that on your own fork if you're not planning on landing it. one-off direct sharing is what forks are for - like sending a message to someone instead of broadcasting it in a public channel use your judgement :)


Variable buffer size

This PR adds a buffer resizing feature before and after the handshake if MBEDTLS_SSL_VARIABLE_BUFFER_LENGTH is defined. The work is based on https://github.com/ARMmbed/mbedtls/pull/3074.
All of the handshake tests now have buffer resizing on by default, so thanks to component_test_variable_ssl_in_out_buffer_len in all.sh are performed with this option turned on.
Darryl's changes to the I/O buffer lengths had to be split between ssl files, because of the changes on development .

I would also like to add two tests to this PR - CID + MFL + VB with max data lengths, for various MFL's, and a record splitting test which will fail without the two additional lines updating in and out msg pointers.

I have added a bunch of updates:

Added CID and CBC record splitting tests to all.sh;
Forced a record splitting ciphersuite with MFL handshake fragmentation tests (these were failing without the in and out msg pointer updates, so now we have coverage of it in unit tests);
Added comments;
Added sign-offs;
Added Piotr's memory acceptance test (passing now), and tweaked it to use VARIABLE_BUFFER_LENGTHS define;
Added tests in test suites integrating MFL + serialization + VB, MFL + renegotiation + VB (to cover the buffer expansion code too) ;
Merged the static function commit with the commit that actually used it.

Introduced requested changes and fixed one missing dependency.

CI failure is Mbed OS import of Mbed TLS, which looks for the recently removed apache-2.0.txt file. The pr-head job passes, so we know this PR works fine with Mbed OS. This means CI has passed.

Added the suggested static function and fixed spaces in Piotr's commit.

Also, I think it would be good to request a review from the OSS team.

One line fix (and a comment) pushed - changed the size of buffer that is being copied inside the new static function.

The only failures seem to be cp: cannot stat 'TARGET_IGNORE/mbedtls/apache-2.0.txt': No such file or directory.

The following tests fail:

The following tests fail:

DTLS client reconnect from same port: reconnect;
K64F due to the apache file fail.

@Patater - these are pre-existing and I would like to fix them in a separate PR, since they're from lower level, message queue tests.

Fixed here: https://github.com/ARMmbed/mbedtls/pull/3092

@Patater - these are pre-existing and I would like to fix them in a separate PR, since they're from lower level, message queue tests.

OK, we've merged #3092 now. Next time we run the merge tests with gcc 7, we should be good.

I've kicked off the merge CI job.

CI passed on merge job. Good for merging.


Minor cleanup of Java example code for AsyncFunctions

Use CompletableFuture correctly, via .supplyAsync() wrapping Future.get()

BTW, how do people check .md editing results before pushing? I've tried a few different plugins and command line utilities, but the mix of HTML and markdown has meant none of them render properly.

Hi @tedyu - not able to assign you as a reviewer, I guess I would need more Git fu to be able to do that.

lgtm

Thanks a lot!
Merging this in the next merge batch...


Accordion improvements - design and missing data

Widened accordion, added accident id and provider code, added date to its section

I suggest you check that with @galraij, as I understood from him we are leaning towards making the "accordion" thinner rather than expanding elements to occupy the extra white space.

Waiting for @galraij 's input/

@galraij  ??

@omerxx also need to rebase and resolve conflicts.

Sure, once this is approved I will

I would say as thin as possible since we are going to change many of the parameters to icons.

On Jul 20, 2015, at 02:12, OmerSchechter<EMAIL_ADDRESS>wrote:
I suggest you check that with @galraij https://github.com/galraij, as I understood from him we are leaning towards making the "accordion" thinner rather than expanding elements to occupy the extra white space.
—
Reply to this email directly or view it on GitHub https://github.com/hasadna/anyway/pull/397#issuecomment-122716464.

Why would that affect the position and size of lines inside the marker?
I changed it because UI wise this seems wrong to me..

But if we move the logo of the Lamas the line that says “source” and make it smaller we save a lot of space, not? The story can spread to more lines...

On Jul 26, 2015, at 12:35, omerxx<EMAIL_ADDRESS>wrote:
Why would that affect the position and size of lines inside the marker?
I changed it because UI wise this seems wrong to me..
—
Reply to this email directly or view it on GitHub https://github.com/hasadna/anyway/pull/397#issuecomment-124963859.

Of course. That's a decision to make - whether you want to shrink things up or widen the accordion.
Regarding the story - I think the current size is optimal, having it squeezed into more lines would not be comfortable for reading. We could reduce the font size though.
One way to go is making everything thiner or making everything wider, the offered solution here is for the current state - where the marker is still at its original size, and containing an accordion, they should complete each other (design-wise)..
We can continue the discussion tomorrow and make these decisions...


File upload support along with message

Please describe your feature request:

a feature where we can upload file and also add a message along with it would be great since it makes things simple and avoids splitting out data and all major providers such as slack , discord preview file

Describe the use case of this feature:

Very useful feature


don't accept/apply command until i hit enter

if i type half a command pipeline, switch windows, then switch back, it has accepted and applied my command line.

I can't repro this on the mac - does it still do it?

I can't repro it anymore either on my Mac.


Enhancements for OSM

User Story
Enhancements/Fixes for OSM
Acceptance Criteria

[x] Limiting the allowed cipher suites for kubelet in all default OSPs
[x] Use OverwriteCloudConfig if specified in Machine Deployment

Closed with https://github.com/kubermatic/operating-system-manager/pull/90


Using User as a related model

Hi there, I have a mode called Incident with a property called Resolver which needs to be a User model. This is how I defined the relationship:
public function getResolver()
{
    return $this->hasOne(User::className(), ['id' => 'resolver_id']);
}

I'm getting an error right there because User does not extend ActiveRecord and hence does not have the find() method.
Is there any other way to set a relationship with the User model?
Many thanks in advance

You probably using wrong model to define a relation. I mean if you have your own User model in the same namespace with the model you defing relation in, PHP will use it instead of yii2-user User model. It's all about namespaces, you should read about them. If you want to use yii2-user User model then you should use it as follows:
public function getResolver()
{
    return $this->hasOne(\dektrium\user\models\User::className(), ['id' => 'resolver_id']);
}

Hi,
That does not solve anything, since I´m already using that User model. The problem is the model \dektrium\user\models\User does not extends ActiveRecord class but \yii\base\Object and hence does not implement the find() method.
Sorry if my explanation was not enough accurate.

That's not truth, since User model of yii2-user extends ActiveRecord class.

Hi dmeroff,
You were right. I have another getCreator() that should represent the relationship with yii2-user model, but it was pointing to the built-in user model. Now it's working like a charm. Many thanks for your help!

Adding the "use" statement to my refering models solved the issue for me.  In this case, I had a model called "Orders" with an OrderedBy_User column with a referential index back to the users table.
`<?php
namespace app\models;
use Yii;
use dektrium\user\models\User;
/**

This is the model class for table "orders".`
Jim


Update license to current year

The LICENSE file in the root directory is out of date. Anyone can grab this.

Actual: 2022 - 2023
Expected: 2022 - 2024

done!


[SOLVED] AWS SSM ParameterNotFound error

When using confd with the ssm backend, String parameters are found while SecureString parameters are not.
TOML file:
[template]
src = "security.properties.tmpl"
dest = "/usr/share/conf/security.properties"
keys = [
"/mycompany.internal.encryption.key"
]
TMPL file (security.properties.tmpl):
mycompany.internal.encryption.key={{getv "/mycompany.internal.encryption.key"}}
SSM "SecureString" parameter:
/common/mycompany.internal.encryption.key (encrypted with "alias/ssm-micro-services" key)
KMS key "alias/ssm-micro-services":
Key Users => MicroServiceTaskRole
Docker container:
Alpine 3.6 with confd v0.14.0 running on ECS (Docker v17.03) with MicroServiceTaskRole
confd invocation:
confd -confdir /etc/confd/common -prefix /common -onetime -backend ssm
Result:
2017-10-18T20:38:40Z ab8660aa9fce confd[301]: ERROR ParameterNotFound: Parameter /common/mycompany.internal.encryption.key not found.
Alternative result:
When the parameter is switched to "String" (instead of SecureString), confd finds it just fine.

Finally identified the problem. Confd does not seem to acquire the IAM role with the policy that grants access to ssm GetParameter action. Only got it to work after adding credentials in the .aws directory.

So, it works on the base ECS instance without .aws credentials (by using the assigned IAM role), but not in an ECS task (with a comparable role). I think that would qualify as a bug.

@watsonta I'm using it with IAM role in AWS ECS just fine. Have you tried debugging the communication with AWS API from inside the container?

Thanks for responding. It was entirely my fault. I had embedded AWS in the example project while I worked toward using an IAM role, and the I forgot about them. As soon as I removed them, it began working.
Tom

On Nov 2, 2017, at 7:31 AM, Oleksandr Kushchenko<EMAIL_ADDRESS>wrote:
@watsonta I'm using it with IAM role in AWS ECS just fine. Have you tried debugging the communication with AWS API from inside the container?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Glad it worked out for you. I'm closing this, for now, feel free to reopen.


Unable to load Dynamic Link - Entry Point could not be located

.+Please check the FAQ (frequently-asked questions) first. If you have other questions or something to report, please address the following:
+## PHP Driver version or file name

php_sqlsrv_73_nts_x64 and php_pdo_sqlsrv_73_nts_x64
+## SQL Server version
15.0
+## Client operating system
+Windows 10
+## PHP version
php 7.4.8
+## Microsoft ODBC Driver version
?
+## Table schema
N/A
+## Problem description
When trying to start up PHP I get an error box that says:
The procedure entry point add_index_zval could not be located in the dynamic link library c:\php\ext\php_sqlsrv_73_nts_x64.dll
The procedure entry point add_index_zval could not be located in the dynamic link library c:\php\ext\php_pdo_sqlsrv_73_nts_x64.dll

This is the output in CMD
D:\User\Desktop\phptest>php -v
PHP Warning:  PHP Startup: Unable to load dynamic library 'php_sqlsrv_73_nts_x64' (tried: C:\php\ext\php_sqlsrv_73_nts_x64 (The specified procedure could not be found.), C:\php\ext\php_php_sqlsrv_73_nts_x64.dll (The specified module could not be found.)) in Unknown on line 0
Warning: PHP Startup: Unable to load dynamic library 'php_sqlsrv_73_nts_x64' (tried: C:\php\ext\php_sqlsrv_73_nts_x64 (The specified procedure could not be found.), C:\php\ext\php_php_sqlsrv_73_nts_x64.dll (The specified module could not be found.)) in Unknown on line 0
PHP Warning:  PHP Startup: Unable to load dynamic library 'php_pdo_sqlsrv_73_nts_x64' (tried: C:\php\ext\php_pdo_sqlsrv_73_nts_x64 (The specified procedure could not be found.), C:\php\ext\php_php_pdo_sqlsrv_73_nts_x64.dll (The specified module could not be found.)) in Unknown on line 0
Warning: PHP Startup: Unable to load dynamic library 'php_pdo_sqlsrv_73_nts_x64' (tried: C:\php\ext\php_pdo_sqlsrv_73_nts_x64 (The specified procedure could not be found.), C:\php\ext\php_php_pdo_sqlsrv_73_nts_x64.dll (The specified module could not be found.)) in Unknown on line 0
PHP 7.4.8 (cli) (built: Jul  9 2020 11:30:33) ( NTS Visual C++ 2017 x64 )
Copyright (c) The PHP Group
Zend Engine v3.4.0, Copyright (c) Zend Technologies
Not sure what's going on. I'm unable to find a 74 version of the driver as well.

hi @lukshan13 how or where did you download the sqlsrv or pdo_sqlsrv drivers? I suppose you used Web Platform Installer?
We started to support PHP 7.4 in version 5.8.0.
If you can't find v5.8 in WebPI, please download the latest version 5.1.

@lukshan13 Aren't you trying to load the PHP 7.3 extension in PHP 7.4.8? That is bound to fail.

Any update, @lukshan13 ?

Closing this due to inactivity


Fix code scanning alert - IAM policy should avoid use of wildcards and instead apply the principle of least privilege

Tracking issue for:

[ ] https://github.com/nebari-dev/nebari/security/code-scanning/29

Fixed in main on Oct 10


Feature: ssh configuration file support

@jafreck,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


repeat entries for subjects classified in multiple workflows

This might be a duplicate, but I couldn't find the ticket. If you have subjects classified in more than one workflow, you get  multiple entries in the subjects csv dump
subject_id,project_id,workflow_ids,subject_set_id,metadata,locations,classifications_by_workflow,retired_in_workflow
490089,17,[],73,"{""filename"":""APFT0000276a.png""}","{""0"":""https://panoptes-uploads.zooniverse.org/production/subject_location/deabab21-7bde-4d50-a6f9-9ca80d087cf4.png""}",null,"[16,2063]"
490089,17,[2063],4718,"{""filename"":""APFT0000276a.png""}","{""0"":""https://panoptes-uploads.zooniverse.org/production/subject_location/deabab21-7bde-4d50-a6f9-9ca80d087cf4.png""}","{""2063"":2}","[16,2063]"
Is it possible to only have on entry or create separate files for separate workflows?
It's not expected that there would be duplicate entries

yeah - this is a by product of the way we process the data right now (via the linkage model). If we process by the project subjects link (distinct via sets) then we should only process this once.

I think it would be helpful if you did it by subject links. It means that basically I have process the file and find the duplicates if I want to find out even how many subjects I have on the site. I'm not sure all research teams will be as capable.

This is by design. If we didn't, the other columns have to be JSON in order to provide their values on a per-workflow basis (retirement etc). For those less proficient in programming, I think that is the less of two evils: Excel is able to filter rows by making a column unique, but not able to decode and split out JSON, so the current setup allows more queries.


Bug in font color feature

If we have some font color to the selected text and then if we try to change the color , there will open a color dialog box and after canceling the dialog box it will remove the current font.
Fix this bug.

Completed on commit number :-> 6108bc9


Apply the new test lib to other exercises

This follows from #102
Clearly a very large PR, one exercise per commit.

@softmoth FYI. The a:cmp word has been added just in the pythagorean-triplet test.8th file.


Suggestion: Ignore all params after the required ones.

I constantly see in my chat users using commands with no params, who add some either way, that don't effect the command itself, but still prevent it from being run.
For example, if a use was to type !invite link the command would fail responding that they were incorrectly using the command, and provide the correct usage, being: !invite.
I know this can be fixed by adding the param [Remainder] string ignore = null to every command, but I feel it would be insanely redundant.
Another example would be something like: !chill 30 stop spamming guys, a user putting some text after the required params, but in consequence failing the command.
I feel it's unnecessary to fail the execution of a command simply because they added things after a command, and it would be nice if there could be a bool added in the CommandServiceConfig allowing you to ignore everything after the required params of a command.

I finally have a bit of free time to put some input here.
I totally agree that ignoring extra params would be useful, and I don't even think it would be that hard to implement. (Just break out of the loop here rather than return an error)


Script to run/test pre trained model on a single image

Hi @da03 ,
I'm trying to write a script to run/test the trained model you provided on a single image to get the output characters. Would it be possible to provide a sample script or list of commands if you already have one ? If not, it would be great if you can briefly mention a series of steps on what/how to do it.
Thanks in advance.

Hi @tumusudheer ,
Did you find any options for running the test for set of images using pretrained model?

https://github.com/emedvedev/attention-ocr/issues/145#issuecomment-523344851
Try this link and repo for testing.
Just put all the images in the directory and specify the path in the main.py script.


[SPARK-14038][SQL] enable native view by default

What changes were proposed in this pull request?
As we have completed the SQLBuilder, we can safely turn on native view by default.
How was this patch tested?
existing tests.

cc @yhuai @liancheng

Test build #53734 has started for PR 11872 at commit a3ec2d9.

Test build #53734 has finished for PR 11872 at commit a3ec2d9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53734/
Test PASSed.

Thanks - going to merge this!


Missing 2017 train/val annotations under the annotations folder of the cocoLRPapi-master

Thank you for your interest for our work. I have just uploaded the annotations and an example json output file.


User roles to cell

Issue #181 .
Description
Roles with links for add/delete to cell.
Checklist
Make sure that all steps a checked before merge

[ ] RSpec tests are passing on CI
[ ] Cucumber features are passing localy
[ ] bin/cop -a does not return any warnings
[ ] Tested manually

@ValentinAndreev ping


Bug Fix For High Saliency Pointing Game

Aligning values of ground truth bounding box with the correct x and y channels.
When iterating through a 2D array structure, the first loop will iterate through the y axis. The second loop will iterate through the x axis. Therefore, to correctly index to the ground truth bounding box, the following logic is needed:
gt_bbox[gt_bbox[1]:gt_bbox[3]+1, gt_bbox[0]:gt_bbox[2]+1]
instead of
gt_bbox[gt_bbox[0]:gt_bbox[2]+1, gt_bbox[1]:gt_bbox[3]+1]

Codecov Report

Merging #42 (2073b50) into main (657fe46) will decrease coverage by 0.78%.
The diff coverage is 55.55%.

:exclamation: Current head 2073b50 differs from pull request most recent head aa7c8d0. Consider uploading reports for the commit aa7c8d0 to get more accurate results

@@            Coverage Diff             @@
##             main      #42      +/-   ##
==========================================
- Coverage   81.72%   80.95%   -0.78%
==========================================
  Files           8        8
  Lines         394      399       +5
==========================================
+ Hits          322      323       +1
- Misses         72       76       +4

Flag
Coverage Δ

unittests
80.95% <55.55%> (-0.78%)
:arrow_down:

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

Impacted Files
Coverage Δ

...on_explanation_methods/evaluation/pointing_game.py
81.63% <55.55%> (-7.01%)
:arrow_down:


Backend: NodeJS , ExpressJS, MongoLab

Start to resource how this can be implemented.
Maybe use of http://www.clementinejs.com/ (just idea)
We need plan and resources and then break work in smaller issues

Hmm just some thoughts about breaking this down further:

[ ] create server.js file to serve index.html with Express
[ ] install mongo, and connect up with server.js
[ ] migrate data from firebase to mongo
[ ] add github oauth using Passport
From a front-end perspective this will mean that we need
[ ] a login view
[ ] a view to let a logged in user view and edit their own profile

I like it. Are we going for Mongo installation or MongoLab? https://mlab.com/

Isn't MongoLab just a hosted instance of MongoDB? I think we'd probably use whatever Heroku lets you have on the free tier

I can help with some of this stuff, clementine has a user profile and github passport already set up.

Heroku has mongolabs

mongolabs is just to avoid the headache to install MongoDB on local/server which is a pain in the ass.....but open to otherwise if you like the challenge

Sounds good. I think we should make it Github OAuth only rather than Username and Password. It should mean less complexity and more security I hope :)

totally GitHub 4 the best!

Hi, What is the current status of this issue. I'd like to contribute.

Can I contribute in any way possible


is honeywell CK65 supported?

Hello There is honeywell CK65 supported on this library if not what can ı do for make app to honeywell CK65

Hi, the model CK65 appears in the Supported devices, so it should be supported. That list is the official list published by Honeywell. Anyway you can update the DataCollection.aar library with the latest from Honeywell and use the plugin as it is., check this https://github.com/luis901101/honeywell_scanner/issues/8

I'm working with a CK65 and this package. So far it works perfectly fine :)!

The last days I did have a lot of jank in my app and the scanner reacted really slow, even on simple screens. Maybe @luis901101 : can you let me know if the same happens to you when trying the CK65 with this package?

The last days I did have a lot of jank in my app and the scanner reacted really slow, even on simple screens. Maybe @luis901101 : can you let me know if the same happens to you when trying the CK65 with this package?

I don't have a CK65 to test. I recommend you to test the example project directly in your device and compare.

@luis901101 sorry, I wanted to mention @sorosons for the test and mixed it up...
I checked the example project. It has the same jank when I'm adding a TextFormField and focus it. Do you have any idea how this could be solved or who I can ask for help?

@novas1r1  not sure what you mean by jank when adding TextFormField and focus it.
Are you using DataWedge instead of native scanner implementation? Note that for DataWedge there is no meed to use this plugin.
Check you are using flutter latest version and honeywell .aar latest.
Also test your code in profile/release mode because debug mode is slow.

I have try in CK65 and it works fine thanks all guys

@luis901101 I did run the application in profile mode, not debug mode. I'm not using DataWedge. I can't download the .aar file, since it doesn't work on mac. I will try to find someone who can download it for me and try it with this solution.
By jank I mean high red peaks in profile mode which lead to slow UI. Clicking buttons, any other touch gestures on the app etc are slow, it feels laggy. I did try it with the example project and added a TextFormField since I'm using forms in my application and wanted to check if it's my app or the same thing happens in the example. I guess it's more a device problem, than a plugin problem - so feel free to close this issue :)! Thanks for your help!

Yeah @novas1r1 it seems more like a flutter problem or device problem, because buttons and the rest of the UI are not affected by this plugin, so laggy behavior seems related to another issue.
Yes, downloading the .aar is cumbersome, windows required 🤦🏻♂.
Try app sample code in emulator, or another device no matter if it is a PDA or not, thing is to test laggy behavior, and make sure using latest flutter or different flutter versions to discard any Skia problem.
I won't close the issue now, maybe you will find some new info and can share it later.


Added Iterative deepening search in search.ipynb

Reference issue #863

@norvig, please review my pr.


Update me->tail with a single assignment operation.

This makes it more likely that updating the tail will be done atomically.

Actually, achieving thread safety is harder than I thought. cbufer_poll reads the tail variable (by calling cbuf_is_empty) so there's still a race condition there. I'm closing this issue.


single @ symbol

I'm not sure why @@ in there?

Ugh! I see the difference now.

(The difference between UDP and TCP is using @ instead of @@ as target descriptor.)


chore: Iterate gossipsub. Changes to gossipsub options.

Update gossipsub options in line with release notes.
The OrbitDB examples, tests and documentation will refer to the latest version of GossipSub. OrbitDB instances implementing older versions of GossipSub will not work with these examples.

LGTM 👍


⚠ SceneTime has degraded performance

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

HTTP code: 200
Response time: 7703 ms

Resolved: SceneTime performance has improved in 2e64367 after 25 minutes.


[Bug]: Puppeteer old Headless deprecation warning:

Steps to reproduce
`const puppeteer = require("puppeteer");
const fs = require("fs");
const botConfiguration = {
headless: false,
executablePath: "C:\Program Files\Google\Chrome\Application\chrome.exe",
userDataDir: "C:/Users/bot/AppData/Local/Google/Chrome/User Data/",
args: [
--profile-directory=Profile 28
]
};
async function runBot() {
const browser = await puppeteer.launch(botConfiguration);
await puppeteer.launch();
const Page = await browser.newPage();
await Page.goto("https://www.google.com/");
}
runBot();`
Expected results
want to see no issues in my terminal
Actual results
Puppeteer old Headless deprecation warning:
.............................
In the near future headless: true will default to the new Headless mode for Chrome instead of the old Headless implementation. For more information, please see https://developer.chrome.com/articles/new-headless/. Consider opting in early by passing headless: "new" to puppeteer.launch() If you encounter any bugs, please report them to https://github.com/puppeteer/puppeteer/issues/new/choose.

I don't want to use the new mode as it uses way more CPU. How do I set the "PUPPETEER_DISABLE_HEADLESS_WARNING"? I tried in both ways below and it didn't work:
        const browser = await puppeteer.launch({headless: true, PUPPETEER_DISABLE_HEADLESS_WARNING: true,             args: [                 '--no-sandbox',                 '--disable-setuid-sandbox',                 '--PUPPETEER_DISABLE_HEADLESS_WARNING'             ]});

I also am not able to disable the warning using PUPPETEER_DISABLE_HEADLESS_WARNING, any help appreciated.

see this:
you should set PUPPETEER_DISABLE_HEADLESS_WARNING to "true" in the environment variable or change the log level.


Error running app. If this keeps happening, please contact support.

I can not see the vizualisations

Hi, thank you for reporting the issue.

I can not see the vizualisations

You probably meant a few embedded apps in the documentation were down. The issue has been fixed and all embedded apps are back online. 🎈
The next time, please include more information in your issue, including what visualisations you were unable to see and on what pages.
Happy Streamlit-ing!


[Tree Details - Design] Edit Maintenance (Completed Maintenance, but wanted to add more to the same session)


Dependency tracking for DNX apps

This package provides request telemetry and exception telemetry. It would be good to also have a way to enable dependency telemetry via this package for projects targeting DNX, even if it's only for the full framework (dnx451) and not CoreCLR (dnxcore50).

@bojanrajkovic, thanks for the interest to Application Insights SDK. Monitoring of dependencies and performance counters is not a supported scenario today. However for dnx451 I just tried it and it seems to be working.
This is how I made it work:

Import packages into project.json. You should only import it for dnx451 target

"dnx451": {
  "dependencies": {
        "Microsoft.ApplicationInsights.PerfCounterCollector": "0.16.1-build01613",
        "Microsoft.ApplicationInsights.DependencyCollector": "0.16.1-build01613"
  }
}

In Configure method you'll need to get TelemetryConfiguration object from the services and initialize corresponding telemetry module:

public void Configure(IApplicationBuilder app, IHostingEnvironment env, ILoggerFactory loggerfactory, TelemetryConfiguration configuration)
        {
#if dnx451
            TelemetryConfiguration.Active.InstrumentationKey = configuration.InstrumentationKey;
            TelemetryConfiguration.Active.TelemetryChannel = configuration.TelemetryChannel;

            PerformanceCollectorModule module = new PerformanceCollectorModule();
            module.Initialize(configuration);
            configuration.TelemetryModules.Add(module);

            RemoteDependencyModule dependencyModule = new RemoteDependencyModule();
            dependencyModule.Initialize(configuration);
            configuration.TelemetryModules.Add(dependencyModule);
#endif

I was also needed to initialize TelemetryConfiguraiton.Active due to some issues in how these modules are implemented.
Dependencies monitoring will use EventSource-based monitoring, not profiling API-based so no additional configuration as mentioned here required
Please let us know how it is working for you and we will be happy to hear feedback on request/exceptions monitoring.

Cool, will test this out and let you know. Thanks @SergKanz!

@SergKanz when I delete the application insights extension for the Azure Web App -- SQL dependency tracking stops working. Can you confirm if it is needed or not?

@hfjallemark , sorry missed your comment. For the full framework we are using the same mechanism of collecting dependencies as for the regular ASP.NET applications. If you have framework 4.6 installed you'll see dependencies collected in development experience. No additional configuration required. We are using EventSource events reported by SQL code. For Azure WebSites you will not have 4.6 installed so you'll need to install azure WebSite extension. I've explained some basics on how we are doing it here: apmtips.com

@SergKanz I added the extension but dependencies are not being collected. If I add the code you suggested above, it does work. When will dependency tracking be supported out of the box for both full framework and CoreCLR (preferrably through an extension method)?

Update about dependency support in aspnet5?

@pherbel do you run aspnet5 on full framework or core framework?

cc: @kartang

I'm using ASP.NET5 RC1 with dnx4.5.1.
TelemetryConfiguration doesn't have TelemetryModules interface anymore(AI 2.0.0-beta4). How can I add DependencyCollector module? Any updates about this?

@kartang is working on enabling this functionality out of the box. For telemetry modules - you don't need to store them into configuration, just construct the module and call Initialize

@SergeyKanzhelev, you mean DependencyCollector is not working with AI(2.0.0-beta4) currently, right?
I've tried with new ASP.NET 5 application. Microsoft.ApplicationInsights.AspNet itself is working fine. When I add Microsoft.ApplicationInsights.DependencyCollector as dnx451 dependency, a TypeLoadException is occurred on this line:
public void ConfigureServices(IServiceCollection services)
        {
            services.AddApplicationInsightsTelemetry(Configuration);
//...

Could not load type 'Microsoft.ApplicationInsights.Extensibility.IContextInitializer' from assembly 'Microsoft.ApplicationInsights, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'.

@kartang just updated it to use beta4 https://github.com/Microsoft/ApplicationInsights-aspnet5/commit/4bee4894485d2fd9f59d538743074e6538354094 You can find a package at https://www.myget.org/F/applicationinsights-sdk-labs/api/v3/index.json It will be compatible with the latest DependencyCollector.
Also @kartang plans to turn on dependency collector by default for Full Framework applications

Dependency collection and performance counters collection was enabled by default for full framework


Update README.md

Looks like the status has changed on what is implemented in v0.2.

Samplers appear to be implemented
(un)mapping of Arrays appears to be implemented
Anything else that was reported as not implemented yet ?

Currently in the process of installing the latest anari-ospray and will test the above.

A small subset of samplers work (notably plain image2d) but still much functionally is missing (in/outTransform, attributes other than 0, wrap modes; primitive and images3d samplers) which is why I kept samplers in the "not implemented" section.
Similarly for mapping of arrays:
void *OSPRayDevice::mapArray(ANARIArray a)
{
  throw std::runtime_error("mapping arrays not yet supported");
}

README is updated wrt. samplers.


outgoing-webhooks.md: Pretty-print sample messages/fixtures.

This is a follow-up to #9462.
@timabbott: FYI :)

@timabbott: Just updated this! Also, which fields in the message dict do you think we should document? I documented recipient_id and rendered_content for now, I thought I'd ask you first before documenting others! Thanks! :)

@timabbott, @roberthoenig: Updated! :) Thanks for the feedback!

@timabbott, @roberthoenig: Just rebased this to make the build pass! :)

Just rebased this :)

@timabbott, @YagoGG: Just updated this! :)

Merged, thanks @eeshangarg!
There's a few things we should definitely do to improve this:

Test actually triggering a Slack format outgoing webhook (both in Slack and in Zulip) and make sure everything's in the right format both in what Zulip outputs and in our example fixture; @rheaparekh can you help with this?  E.g. timestamp is definitely wrong, and service_id should definitely not be None.
Add some example correct JSON-format responses to the Zulip format API, so that users aren't left guessing as to actually how things should look.  @eeshangarg can you take care of this piece?


luci-mod-admin-full: fix missing wifi_assoclist

With e5ba594d77eed77d31d4b9b8c0e86026eb5a5fac the list of the connected device broke up. This fix this problem by creating a proper request link.
@jow- bug.
Signed-off-by: Ansuel Smith<EMAIL_ADDRESS>

Indeed, I forgot to commit that change :)

Merged, thanks!


Internal error while flattening [QNT404] Name 'Listed' not found

Trying to verify the accompanying model results in error
Error: Internal error while flattening [QNT404] Name 'Listed' not found
    at resolveNamesOrThrow (/home/user/.npm-packages/lib/node_modules/@informalsystems/quint/dist/src/flattening/fullFlattener.js:91:15)
    at flattenModules (/home/user/.npm-packages/lib/node_modules/@informalsystems/quint/dist/src/flattening/fullFlattener.js:62:35)
    at compile (/home/user/.npm-packages/lib/node_modules/@informalsystems/quint/dist/src/cliCommands.js:493:104)
    at EitherConstructor.asyncChain (/home/user/.npm-packages/lib/node_modules/@informalsystems/quint/node_modules/@sweet-monads/either/cjs/index.js:125:16)
    at /home/user/.npm-packages/lib/node_modules/@informalsystems/quint/dist/src/cli.js:58:39

Steps to reproduc:

Install quint version 0.19.1
Save the following to Simple.qnt:

module T {
  type State =
    | Listed

  type File = {
    state : State,
  }

  def listed(files: int -> File) : Set[int] =
    files.keys().filter(x => files.get(x).state == Listed)
}

module Spec {
  import T.*

  const init_files: int -> File

  var files: int -> File

  action init = all {
    files' = init_files
  }

  action step = {
    nondet file_id = oneOf(listed(files))
    files' = files
  }
}

module Simple {
  import T.*
  pure val file1: File = {
    state  : Listed,
  }
  import Spec(init_files = Map(1->file1)).*
}

quint verify Simple.qnt
Witness the error

Expected results:

No error; possibly an explanation what I might be doing incorrectly, though it feels correct to me..

It works if I move the definition listed to module Spec. It also works if I incorporate the Simple module inside Spec.

Hi, thanks for reporting! Seems it might be related to the error I've hit in https://github.com/informalsystems/quint/issues/1353.
Unfortunately, Quint's flattening (which converts multiple modules into a single one) is probably our most unstable feature, specially in the presence of constants and instances. I'm glad you found a workaround!


[Q] How to log custom config parameters?

I am working with the Wandb integration with SB3 and I would like to log additional config parameters that I use for my environment.
When I open the table in my log, I can see the custom config parameters; however, they are hidden and cannot be shown.
This is a minimal example:
`
config = {
"policy_type": "MlpPolicy",
"total_timesteps": 10e6,
"env_name": "env-v0",
"num_workers": 21,
"env_time_limit": 7,
"local_log_dir": "/home/path/path/",
"observations_set": "state",
"model_save_freq": 10000,
}
run = wandb.init(
    project="sb3_test",
    config=config,
    sync_tensorboard=True,  # auto-upload sb3's tensorboard metrics
    monitor_gym=True,  # auto-upload the videos of agents playing the game
    save_code=True,  # optional
)

env = ****
model = PPO(policy_type, env, verbose=1, tensorboard_log=log_dir+f"runs/{run.id}")

model.learn(
    total_timesteps=total_timesteps,
    callback=WandbCallback(
        gradient_save_freq=1000,
        model_save_path=log_dir+f"models/{run.id}",
        verbose=2,
        model_save_freq=config["model_save_freq"],
    ),
)
run.finish()

`

Hi @JoseBarreiros-TRI, we will take a look at this for you. We will attempt to duplicate and get back to you. In the meantime, can you please provide a link to your workspace (We can access private projects)

It is fixed, not sure if this is the result of a recent PR since the last time I tested. I can see all the columns now. Thank you.

Hi @JoseBarreiros-TRI , thank-you for letting us know it's been resolved on your end.


[BUG] Server Variable Object missing in Schema Index

I've noticed that Server Variable Object is missing in index of main Schema definitions: https://github.com/asyncapi/asyncapi/blob/master/versions/2.0.0/asyncapi.md#schema

The same applies for:

Server Object

@char0n hey, sorry but could you specify what you mean? Servers is one of main definitions, then Server is one of Servers and Server Variable is part of Server

Ah ok I get it, so only top level objects are within Schema index. Would be nice to have all specification objects included in Schema  as for example OpenApi specification does.

It's easier to work with the spec that way.

you mean something more than just this table of contents?

Yep, sorry for being unclear. It would help tremendously navigating the specification if all Objects from the specification would be mentioned in this TOC (like Server Object, Security Requirement Object) not only the top level objects.

@char0n no worries, thanks for clearing this out. Makes sense what you write. Would you mind opening a PR for this?

Would you mind opening a PR for this?

Sure I'll open it directly against current master. Thanks

Attaching the PR. Along with improving TOC I've fixed some naming inconsistencies of using object names, which prevented for searching for them inside document.


bwavfile::raw_chunk_reader is a private module

I'm very new to Rust so I may be doing something wrong - I'm trying to write a function that returns an AudioFrameReader, this is the function:
fn from_wav_filename(wav_filename: &str) -> Result<(WaveFmt, AudioFrameReader<RawChunkReader<std::fs::File>>), ()> {
    if let mut Ok(r) = WaveReader::open(&wav_filename) {
        let format = r.format().unwrap();
        let mut frame_reader = r.audio_frame_reader().unwrap();
        Ok((format, frame_reader))
    } else {
        Err(())
    }
}

However, compilation fails because bwavfile::raw_chunk_reader is a private module:
error[E0603]: module `raw_chunk_reader` is private
   --> src/main.rs:2:15
    |
2   | use bwavfile::raw_chunk_reader::RawChunkReader;
    |               ^^^^^^^^^^^^^^^^ private module
    |

It seems like I'd run into the same issue if I tried adding AudioFrameReader to a custom struct type.

It's okay this is my first Rust project too!
I don't think you need to have that use statement in your main. Just remove it and it should compile fine, all you need is bwavefile::WaveReader.
Look at the integration tests and see its pattern of uses, it does this same operation in test_read() that you're doing and you can see how I do it there. Let me know if this helps!

Actually looking at this maybe I need to rewrite the AudioFrameReader if you want to be able to spell out in a return type. I don't want the RawChunkReader to be a part of the interface so it's a bug if you have to.
The OTHER thing you can try for now is write the function signature like this:
fn from_wav_filename<T: io::Read+io::Seek>(wav_filename: &str) -> Result<(WaveFmt, AudioFrameReader<T>), ()> {

}

Thanks for your help, btw this lib looks very good, it's already been useful for me.
I've tried adding the generic type, the compiler complains I think because in this case the return type can only be one thing i.e. since I'm returning the type returned from audio_frame_reader() it expects the RawChunkReader type
error[E0308]: mismatched types
  --> src/main.rs:79:21
   |
74 | fn from_wav_filename<T>(wav_filename: &str) -> Result<(WaveFmt, AudioFrameReader<T>), ()>
   |                      - this type parameter
...
79 |         Ok((format, frame_reader))
   |                     ^^^^^^^^^^^^ expected type parameter `T`, found struct `bwavfile::raw_chunk_reader::RawChunkReader`
   |
   = note: expected struct `bwavfile::AudioFrameReader<T>`
              found struct `bwavfile::AudioFrameReader<bwavfile::raw_chunk_reader::RawChunkReader<'_, std::fs::File>>`

error[E0282]: type annotations needed
  --> src/main.rs:88:41
   |
88 |     if let Ok((format, frame_reader)) = from_wav_filename(&wav_filename) {
   |                                         ^^^^^^^^^^^^^^^^^ cannot infer type for type parameter `T` declared on the function `from_wav_filename`

I guess instead of the AudioFrameReader I could return the WaveReader but then I'd lose the state i.e. where I am in the wav file. My use case is that I need a handle to the wav that I can use later, so I read the wav file in then read/seek around the wav using this handle object, which is what it seems the AudioFrameReader is for.

So in 09a9413ff on master I've changed the interface so this function would work more like...
    fn from_wav_filename(wav_filename: &str) -> Result<(WaveFmt, AudioFrameReader<std::fs::File>), ()> { }

However you will have to rejigger your function a bit because the AudioFrameReader contains the WaveReader by reference.

In the future I might revisit this and have AudioFrameReader  consume the WaveReader, in which case your function would work.

FYI this is implemented in master at<PHONE_NUMBER>317c00026453e20a2c5ed6e05b088a so your function should work now as you wrote it.
Thanks for the good note, I've been having trouble understanding why you unwrap inner fields on structs in Rust and this was a great example!


Limit order issue

Make an order like that, submit it, it'll just buy indiscriminately even though it shouldn't due to your cap on the price.

Hmm, it's not supposed to accept a cap without a price.

It should - that's a standard limit order

The price is when the order starts to be filled.  The cap is when the order stops being filled.  These aren't "real" limit orders -- they're just stop orders with a cap.

I don't see your point. Why can't I submit an order with a cap and that's it?  The backend supports it

All it is is - buy shares automatically at this price - this price being 0, highest price is my limit

Obsoleted by 59f966ae8b6d24f221fec1ec166e75a53216429e.


changed Longhorn chart location from longhorn.io to rancher.io

Update the Longhorn charts location to use charts.rancher.io instead of charts.longhorn.io. See our (internal, sorry) Jira ticket EDGE-552.
Note: The upstream PR has landed (https://github.com/rancher/charts/tree/dev-v2.9/charts/longhorn/104.2.0%2Bup1.7.1) but it's not yet available in the registry, so I wasn't able to actually test these changes yet.
This PR does not include:

Updates to the release notes. I added a comment to #376 that this is the version that will be available in that registry.
I didn't touch downstream-cluster-helm.adoc. There's an entry in there on line 243 that references the old location. I can update this if it's safe; I didn't want to touch the day 2 stuff in case that was in progress and/or needs more information to update properly. Let me know and I'll update that reference too.

I added the -crd chart and updated the air gap image list from https://raw.githubusercontent.com/longhorn/longhorn/v1.7.1/deploy/longhorn-images.txt

@dprodanov4 this is going to conflict with #376 because that one updates the versions but still pulls from the longhorn registry, whereas this changes the registry location too (and adds in the separate -crd chart).

@jdob thank you for the heads-up will update mine now.

@dprodanov4 this is going to conflict with #376 because that one updates the versions but still pulls from the longhorn registry, whereas this changes the registry location too (and adds in the separate -crd chart).

Sorry missed this and merged #376 - this PR lgtm other than now needing a rebase 👍

@hardys conflicts resolved when you get a second to approve

Hmm, GH still says conflicts - I would suggest dropping the merge commit, then doing git fetch -va && git rebase -i origin


kafka consumer does not work with SecurityProtocol.Ssl while producer does.

Description
Confluent kafka version: 1.8.1
Apache kafka version: 2.8.0
The producer and consumer is run on a Windows 10 machine
we have kafka cluster hosted in k8s with strimzi and certificate from lets encrypt.
We can connect to the cluster with a producer config:
var producerConfig = new ProducerConfig
{
    BootstrapServers = "bootstrap.kafka.company.io:443",
    SecurityProtocol = SecurityProtocol.Ssl,
};

But when we try the same with a consumer config it fails to validate handshake
var consumerConfig = new ConsumerConfig
{
    BootstrapServers = "bootstrap.kafka.company.io:443",
    SecurityProtocol = SecurityProtocol.Ssl,
    GroupId = "groupid",
};

Log message from client:
SSL handshake failed: error:14090086:SSL routines:ssl3_get_server_c
ertificate:certificate verify failed: broker certificate could not be verified, verify that ssl.ca.location is correctly configured or root CA certificates are installed (add broker's CA certificate to the Windows Root certific
ate store)

How to reproduce
kafka cluster with certificate and consumer running on windows.
Checklist
Please provide the following information:

[x] A complete (i.e. we can run it), minimal program demonstrating the problem. No need to supply a project file.
[x] Confluent.Kafka nuget version.
[x] Apache Kafka version.
[x] Client configuration.
[x] Operating system.
[x] Provide logs (with "debug" : "..." as necessary in configuration).
[x] Provide broker log excerpts.
[ ] Critical issue.

I tried manually point to the certificate and confirm that it is working.
var consumerConfig = new ConsumerConfig
{
    BootstrapServers = "bootstrap.kafka.company.io:443",
    SecurityProtocol = SecurityProtocol.Ssl,
    SslCaLocation = @"C:\Users\joseph\Downloads\ISRG Root X1.cer",
    GroupId = "instruments",
};

So I suspect that ProducerConfig loads the default "root" certificate store. While ConsumerConfig does not.
Though trying to set SslCaCertificateStores does not work.

Using kafkaflow were the transitive dependency was not updated to 1.8.*
Perhaps something to do with certificate chain after dst root ca x3 expired?


optimization mouse cursor saving while drawing

Adding the saving of the modified background pixel while drawing the mouse cursor.
So the function mouse_cursor_save_background is no more used.

You gave me idea and I optimized code even more, so now we do not need not only mouse_cursor_save_background but also mouse_cursor_restore_background and draw_mouse_cursor.


fix(bundler): Fix DMG script

What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] New Binding Issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes. Issue #___
[x] No

The PR fulfills these requirements:

[x] It's submitted to the dev branch and not the master branch
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix: #xxx[,#xxx], where "xxx" is the issue number)

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

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

Fix sips command
Copy seticon from templates folder
Only cp bg.png if it exists

This should fix the script for now, but I think it could become difficult to maintain in the future. We might want to look into using a package such as create-dmg or node-appdmg instead

Ah, I missed that. Will fix

On Jan 6, 2020, at 5:19 AM, nothingismagick<EMAIL_ADDRESS>wrote:

@nothingismagick commented on this pull request.
In cli/tauri-cli/src/bundle/templates/bundle_dmg:

@@ -128,6 +128,9 @@ hdiutil detach ${device}

hdiutil convert "${DMG_TEMP_NAME}" -format UDBZ -imagekey zlib-level=9 -o "${MACOS_APP_NAME}"
+wget -nc https://github.com/sindresorhus/create-dmg/raw/c9b232da8fc2d7068a85f227d3a85bd8210b5bcd/seticon
+chmod +x seticon
this is already available. its shipped in cli/tauri-cli/src/bundle/templates/seticon
and is wired up to be consumed by cli/tauri-cli/src/bundle/templates/bundle_dmg
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

well, its passing, so i am going to merge it. thanks @rajivshah3 !!!