LangChain integration

Pretty low hanging fruit with the wrapper we have, would be great to create a custom LangChain LLM wrapper for llama.cpp.
Then we could use it in the API and do all sorts of cool things with Serge.

Hi, I have been planning to work on this and was wondering if there was a way to just run the API server.

Hey! Happy to hear you want to tackle this task.
Currently you can't run just the API server. This used to be possible but now that the API & the web server are behind nginx, if you start nginx without the web server it will fail the health check and refuse to start. Shouldn't be too hard to fix though hopefully, I'll have a look at it. In the meanwhile you can still access the API at http://localhost:8008/api/docs.
But regarding the LangChain integration I was thinking that it would also be interesting to make a custom LLM that is a wrapper calling this generate method here.
https://github.com/nsarrazin/serge/blob/b5ff9d154142ca918347604e0fd89dd3b003fab0/api/utils/generate.py#L16
The custom LLM would only be working inside of the api container (as it depends on the llama binary) but this would still allow us to do cool stuff on the front-end of this project.
For interfacing with other projects indeed you will need to run the API server.

Did you looked at this repository @nsarrazin ?
Seems good.
https://github.com/linonetwo/langchain-alpaca


[Fixed: #6873 The product should not be out of stock if the quantity …

[Fixed: #6873 The product should not be out of stock if the quantity is available in the product inventory.]

@vivek-webkul
This is working fine.


bug: missing highlights.scm

Did you check existing issues?

[ ] I have read all the tree-sitter docs if it relates to using the parser
[X] I have searched the existing issues

Tree-Sitter CLI Version, if relevant (output of tree-sitter --version)
No response
Describe the bug
All the highlights.scm files don't exist, I guess they were moved: https://github.com/tree-sitter-grammars/tree-sitter-csv/blob/7eb7297823605392d2bbcc4c09b1cd18d6fa9529/package.json#L36-L70
Steps To Reproduce/Bad Parse Tree

Expected Behavior/Parse Tree

Repro
No response

According to the documentation, path is used to prefix where to look for src, but it is not used to prefix the queries.
As an example, the typescript grammar has a path but the queries use a path relative to the repository root, not relative to path: https://github.com/tree-sitter/tree-sitter-typescript/blob/198d03553f43a45b92ac5d0ee167db3fec6a6fd6/package.json#L91


feat: yearn confirm

Description
Please describe your changes
Notice
Before submitting a pull request, please make sure you have answered the following:

[ ] Have you followed the guidelines in our Contributing guide?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[ ] Do all new and existing tests pass? Does the linter pass?

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[x] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)
If applicable, please link to the github issue and put closes #XXXX in your comment to auto-close the issue that your PR fixes.
Testing
Please outline all testing steps

Pull branch locally and run yarn to install new deps
etc...

Screenshots (if applicable)

2 are required changes, 1 is stylistic/preference, otherwise looks good.
why is it that we went with the approach of putting this provider in web rather than a liquifier package in lib?


Change config/settings.py to to store downloaded data from the data source into data folder in the project directory instead of using home directory

@ramkumarkoppu I think this hasn't worked as intended. If I am in directory PyTrendFollow and run ./download.py quandl --concurrent the data end up in the ../prices_data directory.
Instead of
hdf_path = os.path.join("../", "price_data/quotes")

which puts price_data in the parent directory of where download is run, I think you need:
hdf_path = os.path.join(os.path.abspath(__file__), "../price_data/quotes")

which puts price_data as a subdirectory of PyTrendFollow.

Just a reminder that this kind of hardcoding of filepaths involving forward slashes will probably break on windows.


bugfix fix topn be inoperative because Field == Null always return true

Proposed changes
Issue Number: close #xxx
if (!new_top.is_null() && new_top != old_top) is always false since old_top is Null when init and Field == Null always return true.
We add old_top.is_null() check first to avoid the problem and then issue more carefull discussion about Field == Null semantics.
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 48.33 seconds
stream load tsv:          532 seconds loaded<PHONE_NUMBER>9 Bytes, about 134 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          28.6 seconds inserted 10000000 Rows, about 349K ops/s
storage size:<PHONE_NUMBER>0 Bytes


[Feature Request]: Custom button for DateRangePicker trigger node.

Description
just like:

Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate requests

Hello!
Thank you for the request. Can you provide some more context as to the use case for a custom button compared to the current pattern?


Adding compactMap to built_in

Related to issue #2478.

Please add entry to changelog! :-)

I added
(swift) Add compactMap to keywords as built_in (#2478) Omid Golparvar
to CHANGES.md. 😊

@omidgolparvar Thank you so much for contributing!


Glamorous Toolkit Book: broken shortcuts view

The view is broken due to a returned ValueHolder instead of an element.

Add and use new method called GtSourceCoderEditorElementExamples>>#editorForSnippetCoder2:


table Exporter button

i am vary fan of your datatable but can it's possible add table exporter button like as Excel, csv, pdf

hey @karim-007 thank you for your kind words. Exporting functionality is currently not supported out of the box in an effort to keep the library small. however you have access to the data, so you should be able to use any 3rd party libraries to generate your desired file.
closing.


Communication efficiency optimization

The messages are automatically transformed according to their type (e.g., int, float, str) before being fed into the gRPC communicator, which reduces 20%-50% of the message sizes when running toy example in distributed mode. In the previous version, messages are transformed to json_str first and then fed into the gRPC communicator.  For more details please refer to:

A new proto file gRPC_comm_manager.proto;
The transform function and parse function in message.py;

Minor fix for running with distributed mode.

It looks good to me. (Maybe we can add a unit test for the distributed mode?)

It looks good to me. (Maybe we can add a unit test for the distributed mode?)


AQL: cannot use shrink() to increase block

My Environment

ArangoDB Version:        3.7.3
Storage Engine:             RocksDB
Deployment Mode:       Single Server
Deployment Strategy:  Manual Start in Docker
Infrastructure:               own
Operating System:        Alpine Linux v3.11
Total RAM in your machine:        16GB
Disks in use:        HDD
Used Package:              Docker - official Docker library

Component, Query & Data
Affected feature:
Foxx
AQL query (if applicable):
ArangoError: AQL: cannot use shrink() to increase block (exception location: /work/ArangoDB/arangod/Aql/AqlItemBlock.cpp:372). Please report this error to arangodb.com (while executing)
    at ArangoStatement.execute (/usr/share/arangodb3/js/server/modules/@arangodb/arango-statement.js:84:16)
    at ArangoDatabase._query (/usr/share/arangodb3/js/server/modules/@arangodb/arango-database.js:102:45)
    at runQuery (/var/lib/arangodb3-apps/_db/CMDB/QDSL/APP/scripts/getNodeList.js:35:22)
    at /var/lib/arangodb3-apps/_db/CMDB/QDSL/APP/scripts/getNodeList.js:52:15
    at Array.forEach (<anonymous>)
    at executeNodeSearch (/var/lib/arangodb3-apps/_db/CMDB/QDSL/APP/scripts/getNodeList.js:49:11)
    at /var/lib/arangodb3-apps/_db/CMDB/QDSL/APP/scripts/getNodeList.js:51:15
    at Array.forEach (<anonymous>)
    at executeNodeSearch (/var/lib/arangodb3-apps/_db/CMDB/QDSL/APP/scripts/getNodeList.js:49:11)
    at Object.executeQDSL (/var/lib/arangodb3-apps/_db/CMDB/QDSL/APP/scripts/getNodeList.js:80:19)

Problem:
Can't understand the problem. I've upgraded from arangodb v3.4.7 to v3.7.3 and our foxx service log this errors...
Expected result:

@abrekhov : this is an internal error that is not supposed to happen.
Is it possible to share the query that triggers this error with us?
That would be very helpful!
Thanks!

@jsteemann :
here is the AQL from our foxx microservice:
for bObject, bLink, path in 0..10
outbound 'groups/opa'
links, geometricLinks, topologyLinks, storageLinks
filter (
        for iv in 0..(length(path.vertices) - 1)
            let r = path.vertices[iv]._meta.restrict
            return iv < 0 || !r ||
                (for e in slice(path.edges, iv)
                    return parse_identifier(e).collection) all in r
    ) all == true
filter path.edges[*].type ALL == 'path-downlink'

            let object = bObject
            let link = bLink
filter object
let fullPath = path
            for lvl in 0..(length(fullPath.vertices) - 1)
                let rPath = {
                    edges: slice(fullPath.edges, 0, lvl),
                    vertices: slice(fullPath.vertices, 0, lvl + 1)
                }
                let rObject = fullPath.vertices[lvl]
                let rLink = lvl > 0 ? fullPath.edges[lvl - 1] : {}
                return distinct {
id: rObject._id,
    mo: rObject._key,
    group: is_same_collection('groups', rObject._id),
    edges: rPath.edges[*].name,
    has_children:
            length((for o, l in 1 outbound rObject links, geometricLinks, topologyLinks, storageLinks
                filter l.type == 'path-downlink'
                let extPath = {
                    vertices: push(path.vertices, o),
                    edges: push(path.edges, l)
                }
                filter (
        for iv in 0..(length(extPath.vertices) - 1)
            let r = extPath.vertices[iv]._meta.restrict
            return iv < 0 || !r ||
                (for e in slice(extPath.edges, iv)
                    return parse_identifier(e).collection) all in r
    ) all == true
                limit 1
                return l)
            ) != 0
}

as discussed on https://github.com/arangodb/arangodb/issues/13124#issuecomment-739312809 - splice-subqueries is not the origin of this very issue.

@dothebart : I'm sorry but the author of https://github.com/arangodb/arangodb/issues/13124 issue said that turning off the splice-subquiries feature helped. Maybe I didn't get it?

oh, sorry - then I misread your post - I read it as "didn't fix it for me? "

ok, reading the later post by @goedderz in this other ticket, its now working for you. So we can confirm that this issue is most probably a duplicate of #13124 ?

@dothebart yeah, you're right! It's duplicate. Hope this will be fixed in next versions :)

thanks for confirming.

Apparently the same root cause as #13099

Hi @abrekhov,
v3.7.7 with a fix for the problem is now available. Please let us know if the issue is fully resolved.
Thanks

Hi @abrekhov,
Did you have a chance to upgrade and test the fix?

@maxkernbach Hi!
I am sorry for the late reply!
Yes, everything works fine.
Thanks team for the fix!


Question on the parameter count presented in Table 1

Hello @jmliu206,
thank you very much for providing your interesting work.
Could you please explain in more detail how exactly you calculate the parameter number given in Table 1? According to your paper, your small model should have a parameter count of 44.96M, while I get about 76M when testing your code. I have created a colab to reproduce this result:
https://colab.research.google.com/drive/1KdwoC1i-TYMtc3akyuX83exipynKEE4v?usp=sharing
I have used the default setting with C=128 - probably I am just missing some details here...
I was also a bit surprised by the reported number of model parameters for SwinT-ChARM. According to Zhu et al., they have a total of 32.6M (Table 3), whereas you report 60.55M.
It would be great if you could provide further insights here.
Thanks in advance,
Nikolai

I apologize for the misunderstanding. In the code, N refers to 1/2 of C in the paper, as it represents the number of channels used to be input into the Tensformer and CNN networks respectively. Therefore, for the small model, N should be set as 64.
Regarding Swin-Charm, since the open-source code was not available when we finished our work, we reproduced the method based on the paper. The difference between our implementation and the open-source code may be the slice transform. Since the paper didn't say the output channels of the middle convolutional layers, we used the same convolutional layers of our method.
Sorry for any confusion caused. I'll update README to clarify both points.

Thanks a lot for your help. Now I get the following values with the Deepspeed Profiler/ get_model_profile():
N=64    - 441.38 G 215.32 GMACs 45.18 M (flops, macs, params)
N=96    - 865.73 G 425.09 GMACs 59.13 M (flops, macs, params)
N=128   - 1454.5 G 717.08 GMACs 76.57 M (flops, macs, params)

The number of parameters is now quite similar to the reported numbers. The Flops however are approximately twice as high compared to the reported numbers. Do you have any idea why? How are you profiling your model?
I also used a RTX 3090 GPU.
Thanks again!

For all methods in table 3, we use flops-counter.pytorch to caculate complexity.
Specifically, the Flops we report here should be MACs.
For most CV tasks, many papers identify FLOPs with MACs. Some versions of packages also  mixed up Flops and MACs .
I think the following reference might be helpful:
https://github.com/open-mmlab/mmcv/issues/785#issuecomment-766840386
https://github.com/sovrasov/flops-counter.pytorch/issues/16#issuecomment-802631732
https://github.com/sovrasov/flops-counter.pytorch/blob/1ad0ed1999620c0170e5854dde39805d30d9b6aa/sample.py#L36
https://github.com/Lyken17/pytorch-OpCounter/tree/160004dd1535323d71763c93482d2a8f5f260301

Thank you very much :)


Allow tasks in an included build to be executed directly from composite build command line

@bigdaz commented on Thu Jul 21 2016
For example, allow gradle my-included-build::task-name to be executed in the root of a composite build.

Will this not be implemented?

Related: Execute tasks of included build via TAPI (https://github.com/gradle/gradle/issues/14117)

This is currently being addressed in #14894

Closing as implemented.
The main difference between the proposal and the new implementation is that included builds can be references the same way as subprojects: gradle :included-build:sub:myTask.


add metric for total number of requests

currently only response errors are counted, while this it self is useful it would be more complete to also log the total number of requests. by comparing the ratio of errors and total requests a operator is able to determine the healthiness of their installations.
the concrete change needs to be made somewhere along these lines. i'm still unsure how to implement it "correctly", but would like to contribute to the project. 2 solutions that come to my mind would be to either.

drop the if clause and count all responses
add a new metric keycloak_responses_total similar to keycloak_response_errors that counts all responses

https://github.com/aerogear/keycloak-metrics-spi/blob/a9ede42d7a462c6670cdaa6772d27806c2500de9/src/main/java/org/jboss/aerogear/keycloak/metrics/MetricsFilter.java#L44-L48

@xinau That sounds reasonable. But if the goal is assessing the health, wouldn't a rate on the response errors also do the job?

@pb82 Using the error rate can be misleading. I.x. I do know when there is a larger response errors than expected, but this could also be the case when the overall traffic went up and shouldn't need to worry me in the case the error to total response ratio remains the same.
If it's okay with you I can provide a timely patch and tests.


Can't select specific fonts on Android

I've tried to update my android/app/build.gradle file to only use two fonts that I actually need:
/**
 * Load fonts from react-native-vector-icons
 */
project.ext.vectoricons = [
    iconFontNames: [ 'FontAwesome.ttf', 'Entypo.ttf' ]
]

apply from: "../../node_modules/react-native/react.gradle"
apply from: "../../node_modules/react-native-vector-icons/fonts.gradle"

but it still keeps loading all 10 fonts on react-native link. I'm sure it loads them all as I tried deleting those 10 fonts from android/app/src/main/assets/fonts/ before running react-native link. What do I do wrong?

I have the same problem

Hi, react-native link automatically copies the fonts, so refrain from using it. If you need to use it for another project, simply do react-native link the-module-name and it will only run for that module.

@oblador Thanks, got it. It's just that I wanted to slim down my repo by removing third-party assets, but I'm also using native-base in my project and they don't provide a .gradle script to copy assets on the fly, so I either need to run react-native link on setup (which caused the confusion in the first place) or store assets in the repo.
Now that I think about it, I could just run react-native link native-base on setup and it would probably solve my issue, but it's still kinda abusing react-native link... But that's another story :)

@oblador It should be configurable if u ask me, sometimes we run react-native link to re-link everything at once. Maybe there's a way to configure rnpm to run a custom script which would take a whitelist of icon names from a json file on project root if it exists?


Work with external groups to validate hub/spoke DNS

In order to [reason/outcome/goal], [someone or "we"] want [a specific change in product implementation/behavior]
In order to allow the creation of new hub and spoke environments to be routable from both within and external to cloud.gov, we need to work with other groups within GSA to rediscover how DNS and routing works with zscaler and other tools
Acceptance Criteria

[ ] Given a DNS call to the new prometheus hub endpoint, map the GSA teams and products that impact the DNS/host header flow.
[ ] Work with Electra and her team on the required documentation and collaboration needed to document the workflow

Security considerations
None of note, should be an exploratory ticket
Implementation sketch

[ ] Maps the current flow of DNS resolution for internal and public facing components for tooling
[ ] Map how the new hub/spoke DNS architecture will need to follow if different than tooling
[ ] Work with Electra and her team on filling out the document templates
[ ] Schedule subsequent meetings and followups with the additional GSA groups identified during exploration with Electra

Did discover on how our traffic flows from our laptops, over the ZScaler VPN, through the GSA Network and out GSA's Internet egress to AWSGovCloud.
Updated the cg-diagram for jumpbox flow to show these changes and use as example document with Electra and her team to coordinate work with the ZScaler and Internet proxy folks so we can follow the same flow for the new AWS accounts/VPCs
https://github.com/cloud-gov/cg-diagrams/pull/129

Updates and paperwork where sent on Thur 8/10 - key players on the other team where on annual leave last week.  Received an update that the CISS team is reviewing and will get back to us so moving this to blocked until we hear back on next steps.

Update from today - will be attending a work session headed by Jamil Castillo on 9/6 with the CISS working group

Had the working group meeting with the following status:

confirmed our theory that we are in fact a private island
have ZScaler contact info

We can close this ticket.


works with default models but when i dump the model into the model folder that i want to use and i am getting why is it trying to goto face hugger i have already downloaded the model

works with default models but when i dump the model into the model folder that i want to use and i am getting why is it trying to goto face hugger i have already downloaded the model
Loading flatbuffer at C:\ai\New folder\euler_step_1_512_512_vulkan_fp16.vmfb as a mmapped file
use_tuned? sharkify: False
_1_64_512_512_fp16_realisticVisionV40_v40VAE
Checkpoint already loaded at :  C:/ai/New folder/models/diffusers/realisticVisionV40_v40VAE
Error no file named config.json found in directory C:/ai/New folder/models/diffusers/realisticVisionV40_v40VAE.
Retrying with a different base model configuration
Error no file named config.json found in directory C:/ai/New folder/models/diffusers/realisticVisionV40_v40VAE.
Retrying with a different base model configuration
Error no file named config.json found in directory C:/ai/New folder/models/diffusers/realisticVisionV40_v40VAE.
Retrying with a different base model configuration
Error no file named config.json found in directory C:/ai/New folder/models/diffusers/realisticVisionV40_v40VAE.
Retrying with a different base model configuration
Error no file named config.json found in directory C:/ai/New folder/models/diffusers/realisticVisionV40_v40VAE.
Retrying with a different base model configuration
Traceback (most recent call last):
File "gradio\routes.py", line 439, in run_predict
File "gradio\blocks.py", line 1384, in process_api
File "gradio\blocks.py", line 1103, in call_function
File "gradio\utils.py", line 343, in async_iteration
File "gradio\utils.py", line 336, in anext
File "anyio\to_thread.py", line 33, in run_sync
File "anyio_backends_asyncio.py", line 877, in run_sync_in_worker_thread
File "anyio_backends_asyncio.py", line 807, in run
File "gradio\utils.py", line 319, in run_sync_iterator_async
File "gradio\utils.py", line 688, in gen_wrapper
File "ui\txt2img_ui.py", line 155, in txt2img_inf
File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_utils.py", line 389, in from_pretrained
File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_txt2img.py", line 51, in init
File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_utils.py", line 82, in init
File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_utils.py", line 114, in load_unet
File "apps\stable_diffusion\src\models\model_wrappers.py", line 826, in unet
File "apps\stable_diffusion\src\models\model_wrappers.py", line 821, in unet
File "apps\stable_diffusion\src\models\model_wrappers.py", line 63, in check_compilation
SystemExit: Could not compile Unet. Please create an issue with the detailed log at https://github.com/nod-ai/SHARK/issues
Found device AMD Radeon RX 6700 XT. Using target triple rdna2-unknown-windows.
Tuned models are currently not supported for this setting.
Traceback (most recent call last):
File "huggingface_hub\utils_errors.py", line 261, in hf_raise_for_status
File "requests\models.py", line 1021, in raise_for_status
requests.exceptions.HTTPError: 401 Client Error: Unauthorized for url: https://huggingface.co/4201/resolve/main/scheduler/scheduler_config.json
The above exception was the direct cause of the following exception:

I didn't have this problem with that model (downloaded as pruned fp16 safetensors from civitAI.  It refused to tune without changing the VAE, but I'm not convinced this isn't some issue with Shark itself (i.e. the matrix mismatch reported by the first run causes it to fail to tune, but the second time you access the model in a later session it pulls the .vmfb files it generated the first time and runs those) .
A 401 error indicates that you're not logged in to huggingface, but that wouldn't have mattered since there's no repo at that address and I should have received a file not found error if mine was trying to download from there too, I'd assume.  I'd suggest trying a .safetensors version if you had just installed a model tree from GIT.


Angular template not available

I ran: dotnet new --install Microsoft.AspNetCore.SpaTemplates::*
But angular is not an option for available project types.
This was the output:
NuGet Config files used:
C:\Users{}\AppData\Roaming\NuGet\NuGet.Config
C:\Program Files (x86)\NuGet\Config\Microsoft.VisualStudio.Offline.config
Feeds used:
https://api.nuget.org/v3/index.json
C:\Program Files (x86)\Microsoft SDKs\NuGetPackages\
Installed:
112 package(s) to C:\Users{}.templateengine\dotnetcli\v1.0.4\scratch\restore.csproj
Templates                          Short Name      Language      Tags
Console Application                console         [C#], F#      Common/Console
Class library                      classlib        [C#], F#      Common/Library
Unit Test Project                  mstest          [C#], F#      Test/MSTest
xUnit Test Project                 xunit           [C#], F#      Test/xUnit
ASP.NET Core Empty                 web             [C#]          Web/Empty
ASP.NET Core Web App               mvc             [C#], F#      Web/MVC
ASP.NET Core with Aurelia          aurelia         [C#]          Web/MVC/SPA
ASP.NET Core with Knockout.js      knockout        [C#]          Web/MVC/SPA
ASP.NET Core with Vue.js           vue             [C#]          Web/MVC/SPA
ASP.NET Core Web API               webapi          [C#]          Web/WebAPI
Solution File                      sln                           Solution

Also getting this issue, No issue with 0.9.3, You can install using:
dotnet new --install Microsoft.AspNetCore.SpaTemplates::0.9.3

Same here. Was able to resolve by installing 0.9.3. I note that 1.0.0 was newly released today.

As of .NET Core 2.0, the Angular, React, and Redux templates are built into the SDK, so you get them automatically without having to install any additional package.
As of version 1.0.0, the Microsoft.AspNetCore.SpaTemplates package is an optional add-on that only contains the Aurelia, Knockout, and Vue templates.

Ok, that makes sense. But for someone who still has .NET Core 1.1 and installs the templates, they will find them missing. Isn't it possible to make the SpaTemplates package dependent on Core 2.0?

You could create a .Net Core 2.0 site with Angular from the template and then covert the MVC Core to 1,1 .


Fibaro FGS221 local button 1 press not updated to controller

Today i made the update from version 1.0.0 to version zwavejs2mqtt: 3.1.0 (zwave-js: 7.1.0) but now i`m seem to be having issues with Controller / Lifeline updates on at least Fibaro FGS221 and FGS222
Button 1 is giving me absolutly radio silence
Button 2 is working when i press the local button
2021-04-07 00:14:01.416 INFO ZWAVE: Node 4: value updated: 37-2-currentValue true => false
2021-04-07 00:14:03.437 INFO ZWAVE: Node 4: value updated: 37-2-currentValue false => true

The issue occurs on at least 3 modules (which are in my reach to test) as soon as i start version 1.0.0 the Button 1 notify to the controller is working again.

Please make a zwave-js log, loglevel debug or silly and attach it here as a file.

I have the same issue with a FGS222, here are some logs of the specific node when the button was pressed:
2021-04-09T12:15:15.793Z SERIAL « 0x01090004000e032503ff26                                            (11 bytes)
2021-04-09T12:15:15.796Z CNTRLR   [Node 014] [~] [Binary Switch] currentValue: false => true        [Endpoint 0]
2021-04-09T12:15:15.797Z CNTRLR   [Node 014] [+] [Binary Switch] targetValue: undefined             [Endpoint 0]
2021-04-09T12:15:15.798Z SERIAL » [ACK]                                                                   (0x06)
2021-04-09T12:15:15.801Z DRIVER « [Node 014] [REQ] [ApplicationCommand]
                                  └─[BinarySwitchCCReport]
                                      current value: true
2021-04-09T12:15:15.904Z SERIAL « 0x01090004000e032503ff26                                            (11 bytes)
2021-04-09T12:15:15.906Z CNTRLR   [Node 014] [~] [Binary Switch] currentValue: true => true         [Endpoint 0]
2021-04-09T12:15:15.907Z SERIAL » [ACK]                                                                   (0x06)
2021-04-09T12:15:15.910Z DRIVER « [Node 014] [REQ] [ApplicationCommand]
                                  └─[BinarySwitchCCReport]
                                      current value: true
2021-04-09T12:15:16.099Z SERIAL « 0x01160049840e10041001728670858e6025277a73ef256061                  (24 bytes)
2021-04-09T12:15:16.101Z SERIAL » [ACK]                                                                   (0x06)
2021-04-09T12:15:16.102Z DRIVER « [Node 014] [REQ] [ApplicationUpdateRequest]
                                    payload: 0x840e10041001728670858e6025277a73ef2560
2021-04-09T12:15:16.106Z CNTRLR « [Node 014] Received updated node info
2021-04-09T12:15:17.905Z SERIAL « 0x01090004000e03250300d9                                            (11 bytes)
2021-04-09T12:15:17.909Z CNTRLR   [Node 014] [~] [Binary Switch] currentValue: true => false        [Endpoint 0]
2021-04-09T12:15:17.910Z SERIAL » [ACK]                                                                   (0x06)
2021-04-09T12:15:17.915Z DRIVER « [Node 014] [REQ] [ApplicationCommand]
                                  └─[BinarySwitchCCReport]
                                      current value: false

In home assistant the entity didn't show a "Turned on" event.

Sorry, but I need a little more context. Please re-interview the device (you can do that in the zwavejs2mqtt panel under "advanced") and post a complete log of that.

@AlCalzone sorry, first time providing the log files..
Attached the logs of the re-interview and 2 button presses afterwards.
zwavejs_356.log
zwavejs2mqtt.log

I mean it's nice that you try to keep the noise down but there's often a lot of context elsewhere in the logs for me :)

Ok the problem for FGS-222 is that the reports from endpoint 1 don't tell us they are from endpoint 1. I've added this case to https://github.com/zwave-js/node-zwave-js/issues/2286
I'll still need a log for FGS-221, but I guess that's the same.

Upgraded to zwavejs2mqtt 3.2.1 (zwave-js 7.2.2) and all is working ok

Yeah, I've reverted the change that caused this for now.

The problem has returned


Azure Blob Module installation issue in azure Virtual machine

Describe the bug

Azure.storage.blob module 12.9.0 is installed in Python 2.7.13 but still its failing with no module found

Code Snippet

Error added in same issue .

Expected behavior

Python 2.7.13 (default, Jan 11 2017, 10:56:06) [GCC] on linux2
Type "help", "copyright", "credits" or "license" for more information.

import azure.storage.blob
Traceback (most recent call last):
File "", line 1, in
ImportError: No module named azure.storage.blob

Setup (please complete the following information):

Python Version: Python 2.7.13
SDK Version: azure-storage-blob==12.2.0

Commands we used -->
1-Created a SLES 12 sp4 gen2 VM, as your VM
2-Downloaded pip:
linux:/tmp # wget -c https://bootstrap.pypa.io/pip/2.7/get-pip.py
--2022-08-25 09:14:09--  https://bootstrap.pypa.io/pip/2.7/get-pip.py
3- -Installed pip, and got the same deprecation message, but pip installed successfully (the installed version is 20.3.4 , and the one that does not support Python 2.7 will be version 21.0);
linux:/tmp # python2 get-pip.py
4- linux:/tmp # pip --version
pip 20.3.4 from /usr/lib/python2.7/site-packages/pip (python 2.7)
Per the above, the running version of pip will still support python 2.7, only the upcoming pip version 21.0 will not
Trying to update pip will not upgrade from 20.3.4 to a newer version:
linux:/tmp # pip install --upgrade pip
5 - Installed same package
linux:/tmp # pip install azure-storage-blob
6 - Installed the missing dependency "requests" with version 2.18.4, as asked by azure storage-blob
linux:/tmp # pip install requests==2.18.4
7 - Uninstalled azure-storage-blob:
linux:/tmp # pip uninstall azure-storage-blob
8 - And installed azure-storage-blob again
linux:/tmp # pip install azure-storage-blob
9  - But yes after this still we are getting error like : no module found

Hi @rushikesh0900 , thanks for bringing this to our attention. We'll investigate and get back to you ASAP.
Thanks!

Hi @rushikesh0900 , thanks for bringing this to our attention. We'll investigate and get back to you ASAP.
Thanks!

Hi @rushikesh0900, module errors such as this typically indicate a setup issue. There are a couple of things that I suspect could be the culprit here:

You may have multiple Python's/pips installed on the system and where you are using pip install is not the same location where your Python code is running. We can see that your pip is coming from /usr/lib/python2.7/site-packages/pip (python 2.7). Can you please verify your Python script is running in the same envorment?

You can check where your currently executing Python executable from inside your script using something like below.
You may also want to check you path to see where things are pointing.

import os
import sys
print(os.path.dirname(sys.executable))

Using pip install azure-storage-blob I would expect would give you the latest version of blob (currently 12.13.1) which does not support Python 2.7 and should give you an install error. This may be causing an issue. We recommend using Python 3, since 2.7 is deprecated, and installing the latest version of blob but the last version of blob to support 2.7 should be 12.9.0.


test(e2e,playwright): 404.spec.ts

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Solves Pages-404 #51705
bazingaa

Thank you for submitting a pull request for Hacktoberfest. However, it appears that you have not followed the instructions provided in the Hacktoberfest issue thread or our other contributing guidelines.
Please review them and make the necessary changes.

@raisedadead I have made the necessary changes but I'm not able to figure out why some test cases are failing. Need help.

No worries, we are here to help: contributors chat room - for the specific issues you should check the details link in each of those errors:

Some common troubleshooting measures:

Run this in your IDE to check the linting issues.

npm run lint

Actually execute the tests in your IDE, before pushing changes and make sure they are working.

If you are stuck please let us know, and we will guide further.

in this case , i think, the meaning of title -  is what you see when
hovering over favicon of website (of what you can see at developer tool,
when inspection head>title tag)
On Mon, Oct 2, 2023 at 6:11 PM Subhrajit Dutta @.***>
wrote:

@.**** commented on this pull request.
In e2e/404.spec.ts
https://github.com/freeCodeCamp/freeCodeCamp/pull/51736#discussion_r1342819509
:

+import { test, expect, Page } from @.***/test';

+test.describe('404 Page', () => {

let page: Page;

test.beforeAll(async ({ browser }) => {
page = await browser.newPage();
await page.goto('/404');
});

test.afterAll(async () => {
await page.close();
});

test('Should have a relevant page title', async () => {
await expect(page).toHaveTitle('404 Page Not Found');

[image: image]
https://user-images.githubusercontent.com/98512995/272008840-90a49e3d-2299-4910-b041-ddccd577d222.png
This is the 404 page. Should the function contain "Page not found."? I
mean like this await expect(page).toHaveTitle('Page not found.');
—
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/pull/51736#discussion_r1342819509,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKGMHUOC4WCXVDN4Z2DGQDX5LKRDAVCNFSM6AAAAAA5OHGMCKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTMNJSHE2TMNZXGU
.
You are receiving this because you commented.Message ID:
@.***>

--
http://gs500coder.blogspot.com

Okay, I see. I'll change it to Page not found| freeCodeCamp. But I have one doubt: when I run npm run lint locally, it throws some errors in api folder. Here's the error message.
`Found 20 errors in 9 files.
Errors  Files
5  api/src/routes/challenge.test.ts:234
2  api/src/routes/challenge.ts:130
2  api/src/routes/helpers/challenge-helpers.test.ts:2
2  api/src/routes/helpers/challenge-helpers.ts:36
2  api/src/routes/user.test.ts:20
2  api/src/utils/common-challenge-functions.ts:1
1  api/src/utils/default-user.ts:4
1  api/src/utils/get-challenges.ts:6
3  api/src/utils/normalize.ts:3
ERROR: "lint:ts" exited with 2.`
However I have not modified any of these files. It is the same as the original repo. I only added the 404.spec.ts in the e2e folder. I think it's something with typescript.

i think you can run the lint only on your file by specify it as a
parameter...
something like npx tslint src/code/YOURFILE.ts --fix
On Mon, Oct 2, 2023, 18:33 Subhrajit Dutta @.***> wrote:

Okay, I see. I'll change it to Page not found| freeCodeCamp. But I have
one doubt: when I run npm run lint locally, it throws some errors in api
folder. Here's the error message.
`Found 20 errors in 9 files.
Errors Files
5 api/src/routes/challenge.test.ts:234
2 api/src/routes/challenge.ts:130
2 api/src/routes/helpers/challenge-helpers.test.ts:2
2 api/src/routes/helpers/challenge-helpers.ts:36
2 api/src/routes/user.test.ts:20
2 api/src/utils/common-challenge-functions.ts:1
1 api/src/utils/default-user.ts:4
1 api/src/utils/get-challenges.ts:6
3 api/src/utils/normalize.ts:3
ERROR: "lint:ts" exited with 2.`
However I have not modified any of these files. It is the same as the
original repo. I only added the 404.spec.ts in the e2e folder. I think it's
something with typescript.
—
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/pull/51736#issuecomment-1743243902,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKGMHQBMVK5KTO4XZXMMULX5LNFRAVCNFSM6AAAAAA5OHGMCKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONBTGI2DGOJQGI
.
You are receiving this because you commented.Message ID:
@.***>

All checks are passing now. Thanks for the guidance.

@Subhrajit-Dutta your tests cover only the page title and for the presence of 'Page not found' text. There are a few more elements on this page that can be covered, like the presence of the image, the explanation paragraph, the quote and the button:

Additionally it will be of help if the tests cover whether the button redirects users to the right page.
All these are important and if one does not work people working on the project should be notified.

@raditotev Okay, I'll also try to add tests for those elements.

Cool. If you want your tests to perform well and to take less time you might want to consider including all assertions related to the content in a single test. This is possible in this test since there is no interaction that might change page content. Only the button test will have to be in a separate test. Here is an example:
import { test, expect } from '@playwright/test';

test.describe('404 Page', () => {
  test.beforeEach(async ({ page }) => {
    await page.goto('/404');
  });

  test('Should render correctly', async ({ page }) => {
    await expect(page).toHaveTitle('Page not found| freeCodeCamp');
    await expect(page.getByText('Page not found.')).toBeVisible();
    // rest of assertions go below
  });

  test('Button should redirect users to correct page', async ({ page }) => {
    // test goes here
  });
});

Hey there,
Thank you for your interest in contributing to freeCodeCamp. We have merged PR #51934, which added more robust tests. While we are closing this PR, we've added the https://github.com/freeCodeCamp/freeCodeCamp/labels/hacktoberfest-accepted label so you still get credit for your efforts.
We look forward to seeing future contributions from you! :3


the system cannot find the path

Problem encountered on https://dotnet.microsoft.com/learn/aspnet/hello-world-tutorial/create
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

Can you provide more details about your issue? Just by the title, it's not clear what the problem is.


Fix missing tags in dataset cards

Fix missing tags in dataset cards.
This PR partially fixes the missing tags in dataset cards. Subsequent PRs will follow to complete this task.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Segmentation fault (core dumped)

Hi, thanks the opening source code! When I train my own datasets with VOC style, the process of training is nomal, but the eval process raise 'Segmentation fault (core dumped)'. It seems the val_dataloader problem.

The answer in #34 may help you. In short, change the num_workers to 0 for your val dataloader.
And it's better to provide us your error log.

Thank! It works in epoch 1 when I set the num_workers to 0, but a 'NoneType' error like this raised when start to train epoch 2.

@Joker316701882
It can be work when init prefetcher in 'before_epoch' function!


default configuration causes segfault on out-of-the-box systems

at ncui.c:44 we call fopen then immediately fclose the returned handler. The default location for the file is /etc/ly/ly.save. The /etc/ly directory doesn't exist unless it is manually created beforehand. If /etc/ly is not previously created, the call to fopen will return a null file handler, which causes a segfault in the subsequent fclose.
I propose either the default location for ly.save is changed to a location that exists by default, or instructions are added to the README to explicitly create /etc/ly beforehand or manually change that config option.

If i'm not mistaken ly runs as root when you starts up your machine.
Why not just create it then ?

That's another valid option.

Will not it be more correct to place the /etc/ly/ly.save in a /var/lib/ly/ly.save ?
Temporary files are best practic stored in the /var directory.

This is now fixed thanks to the above PR, closing


is there any specific reason for :method => 'post'?

Is there any specific reason for :method => 'post' while handling response with Rack::OpenID.build_header() ?
I'm using just this w/o devise database_authenticatable. After successful auth I'd like the user to be redirect right where they came from and Rails obviously complaining about "no such action X with method POST".

Sorry, I probably didn't explain that.
I'm using devise_openid_authenticatable w/o sign-in form. A custom sessions controller would redirect any request from sessions#new back to after_sign_in_path_for(:user) adding user[remember_me] and user[identity_url] parameters. My identity_url is something hardcoded (I sort of "trust" only once specific provider, it's an in-house app):
class SessionsController < Devise::SessionsController
  def new
    original_url = after_sign_in_path_for(:user)
    openid_params = "user[remember_me]=1&user[identity_url]=#{APP_CFG[:provider_url]}"
    url = [original_url, openid_params].join(original_url.index('?') ? '&' : '?')
    redirect_to url
  end
end

The return URL is to reauthenticate once the OpenID request has gone through, so it is back to the openid_authenticatable method itself (e.g. POST to /user/sign_in).  If you want to redirect the user somewhere after they have successfully authenticated, a better way to do it is using the usual Devise methodology, e.g.:
session[:user_return_to] = my_desired_return_url

If, after doing that, you're still getting that same error, please file an issue with instructions to reproduce the issue and I'll check into it.


FUSETOOLS2-405 - when creating a datasource preset all property value…

…s with an empty string
Signed-off-by: Lars Heinemann<EMAIL_ADDRESS>

When creating a datasource via the UI, we currently fill a set of predefined datasource properties into that datasource. Those properties have no pre-defined value. Due to the way the YAML library works it then only inserts the names of the properties to the YAML file but no the value tag. In our last dv tooling call @rareddy asked me to provide some dummy value for those datasource properties so the value tags are filled to the yaml. We decided to use the string <empty>. Now a user can open the YAML file directly and enter the values inside the YAML file without the need to first add the value tag. I know there is also the option to edit those values through the UI but that way we make it also more user friendly for people who prefer the direct editing of the YAML file.
Just to restate: This only applies to datasource properties. Environment variables are not handled that way.

if I understand well YAML specification, you can have the node visible for null values. You need to provide null or ~ for an unsetted value.
see https://yaml.org/spec/1.2/spec.html#id2786563 (or https://symfony.com/doc/current/components/yaml/yaml_format.html#nulls which is clearer)

Just to restate: This only applies to datasource properties. Environment variables are not handled that way.

description of https://issues.redhat.com/browse/FUSETOOLS2-405 needs to be updated then.

Just to restate: This only applies to datasource properties. Environment variables are not handled that way.

I was wrong about this one. Should be filled in both cases.

I agree that putting in <empty> might confuse users. I switched to an empty string which does the trick too.

ok to go with empty string.
I'm wondering if it would not be even better to support null or ~ for not set values but it would require also to update the "Edit property" which is not supporting it currently.

Edit Property has the same logic for setting an empty value. It defaults to empty string too, same as with using the create datasource. I think null is not really a nice preset as it looks like code rather than a value but I guess that's a matter of personal preference.


pistes:grooming

creating nordic slopes
adding pistes:grooming
bug: one preset is now "classic+skating" this does not fit do the common value separator in OSM. Correct style: "classic;skating"

This topic is mentioned on the wiki page, I think you'd need to start a discussion or proposal on the wiki if you want to deprecate a tag


Bumps version of uuid to latest

This mainly solves one problem that we face with jest. Please follow this discussion for more info

@ondrej-langr could you help us understand how you end up having a dependency on @azure/ms-rest-js?  This repo is not actively maintained any more.


[SPARK-31512]-3 we may not always need order by when using window function

we may not always need order by when using window function

Can one of the admins verify this patch?

Can one of the admins verify this patch?


trying to delegate std::ops::Add

Hi there! I'm sure this is me not understanding something, but I have a new type struct pub struct MyType(#[serde(serialize_with = "serialize_date_time")] pub NaiveDateTime); that manually implements the inner type's traits. For example:
 impl std::ops::Add<chrono::Duration> for MyType {
    type Output = MyType;

    fn add(self, rhs: chrono::Duration) -> MyType {
        MyType(self.0 + rhs)
    }
}

I tried to replace it with the following:
#[derive(Delegate, Serialize, Debug, Copy, Clone)]
#[delegate(std::ops::Add<X>, generics = "X")]
pub struct MyType(#[serde(serialize_with = "serialize_date_time")] pub NaiveDateTime);

but I get the following error:
Error[E0046]: not all trait items implemented, missing: `Output`, `add`
  --> XXXXXXXmod.rs:27:14
   |
27 |     #[derive(Delegate, Serialize, Debug, Copy, Clone)]
   |              ^^^^^^^^ missing `Output`, `add` in implementation
   |
   = note: this error originates in the derive macro `Delegate` (in Nightly builds, run with -Z macro-backtrace for more info)
   = help: implement the missing item: `type Output = Type;`
   = help: implement the missing item: `fn add(self, _: Rhs) -> <Self as Add<Rhs>>::Output { todo!() }`

For more information about this error, try `rustc --explain E0046`.
warning: `domain` (lib) generated 4 warnings
error: could not compile `domain` due to 2 previous errors; 4 warnings emitted

First, is what I am trying to achieve possible, and secondly, what am I doing wrong :-).
Thanks!
P.S> issue asked https://users.rust-lang.org/t/newtype-and-traits-of-the-inner-type/92664/2

I think this is most likely due to std::ops::Add being a remote trait that needs its definition to be duplicated together with #[delegatable_trait_remote].
For reference see the README here or this test: https://github.com/hobofan/ambassador/blob/a6703fed5e3c3672af0ec76e8767c78f23a0a218/ambassador/tests/run-pass/associated_types.rs
(might also been a something completely different; haven't used the crate in years myself 😬 )

Thanks!Sent from my iPhoneOn 17 Apr 2023, at 20:06, Maximilian Goisser @.***> wrote:
I think this is most likely due to std::ops::Add being a remote trait that needs its definition to be duplicated together with #[delegatable_trait_remote].
For reference see the README here or this test: https://github.com/hobofan/ambassador/blob/a6703fed5e3c3672af0ec76e8767c78f23a0a218/ambassador/tests/run-pass/associated_types.rs
(might also been a something completely different; haven't used the crate in years myself 😬 )
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


fix: bug fixes to AtKey.fromString static method and various toString instance methods

- What I did
fix: Fixed bug in AtKey.fromString() was stripping the leading '@' sign from the sharedBy when parsing any keys
fix: Fixed bugs in the various atKey.toString() instance methods which were all assuming presence of namespace and were therefore adding spurious '.null' when constructing string representation
fix: added validation : when setting sharedBy and sharedWith, if they are not null or empty, then they must start with '@'. Did this in AtKey class itself, and also in builders
test: made at_key_test tests stronger, added toString() assertions also
chore: updated pubspec.yaml in at_dump_atKeys so it picks up at_pkam locally
- How I did it
See code changes
I have also run unit tests in at_client against this branch, all of them pass
- How to verify it
Tests pass
- Description for the changelog
fix: bug fixes to AtKey.fromString static method and various toString instance methods

@murali-shris please can you review these changes also? I believe they are safe but I'm not entirely sure. Perhaps I should create branches / PRs in client and server repos with dependency overrides for at_commons to this PR's branch, and then see if the functional and e2e tests pass?

@murali-shris please can you review these changes also? I believe they are safe but I'm not entirely sure. Perhaps I should create branches / PRs in client and server repos with dependency overrides for at_commons to this PR's branch, and then see if the functional and e2e tests pass?

changes look good to me.

Thank you both


Feature: make tables sortable

Type of feature
🍕 Feature
Current behavior
We currently have three main different types of tables in the platform, but none of them can be sorted:
Contributor Table:

Repositories Table:

Contributor Table on the List creation flow:

Suggested solution
Make the tables sortable on the following columns:
A UI example of what it could look like:

Arrow up / down for column that is being sorted
Double chevron for columns that can be sorted

The values that should be sortable are:
Contributors Table

Activity
Repositories
Last Contributed
Time Zone
Contributions

Repositories Table

Activity
PR Velocity
Spam
Contributors (use number of contributors to sort)

Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[X] I agree to follow this project's Contribution Docs

Maybe we can make this issue a tracker, there is already one issue opened for the repositories table https://github.com/open-sauced/app/issues/1787


fix error for schema validation

@barrymay sorry that was my fucked up in schema validation errors, i think this will make type correct as well.

@barrymay thanks heaps for pointing out the root cause.


Login doesn't work when running a local instance without real mail server environment variables

It needs the real mail host username and password environment variables to work.


Adding grpcio to setup.py.

grpcio can only be installed with Python 2.7, so the py27 tox environment is the only one that doesn't need to run without mocks.
FYI @jgeewax the grpcio install fails on Python 2.6 and 3.4 (didn't try 3.5).

This won't run Py2.7 tests on AppVeyor with grpcio but will at least install it. I tried to tackle but can't write faithful powershell without a test machine
If this works, I'll conditionally add the Bigtable tests (in Py2.7) in attempt_system_tests.py

I like the idea of the extra:  users can make perfectly valid use of gcloud without grpc.

:+1: I like the idea. Then we'd have
pip install gcloud[grpc]

for people that want to use gRPC?

@jgeewax Please weigh in

Actually I think it's better for the extra to the the part of this library that requires extra dependencies. So IMO it should be pip install gcloud[bigtable].

But in the not-too-distant future datastore, pubsub, logging, others? will have direct gRPC support in this library.

Hmm. Fair point. I'm unsure. It seems that right now grpc doesn't have a good install story (unless that's recently changed).
One part of me is leaning on having grpc remain an "extra" (in one way or another) until the install story improves or the tipping point where most clients in this library use grpc.
@jgeewax what do you think?

pip install grpcio>=0.13.0 works fairly well now, but must be 2.7 and only targets a limited number of platforms (but Linux, OS X and Windows covers a lot).

That's fair, I suppose. Are any os-level dependencies needed other than just build-essential?

No idea. @nathanielmanistaatgoogle do you know who might be able to chime in?

I don't think even build-essential is needed.... It shouldn't as there's no compiling so gcc and make shouldn't be required for this...

Cool, SGTM.

I'm about to test it on bare GCE Debian and Ubuntu images.

Seems to work on Ubuntu:
$ sudo apt-get update
$ sudo apt-get -y upgrade
$ curl -O https://bootstrap.pypa.io/get-pip.py
$ sudo python get-pip.py
$ sudo -H pip install --upgrade pip
$ sudo -H pip install 'grpcio==0.13.0'
# FAILURE: https://gist.github.com/dhermes/f3cc291bb289ae69c86d
$ sudo -H pip install --upgrade six
$ sudo -H pip install 'grpcio==0.13.0'
$ # SUCCESS!
$ dpkg -s build-essential
dpkg-query: package 'build-essential' is not installed and no information is available
Use dpkg --info (= dpkg-deb --info) to examine archive files,
and dpkg --contents (= dpkg-deb --contents) to list their contents.

Confirmed it works (with the same funky VersionConflict when --upgrade isn't used) on Debian

@tseaver PTAL. Updated to make it an extra.

LGTM

grpc 0.13.0 is broken on UCS2 versions of python 2.7 (which you get when you build Python 2.x from source).
We should drop testing with it in tox until they make a release with the fix.

@tseaver can you file an issue? I'm AFK right now

#1572


app now shows different locations based on re-entry.

You can now enter any CO city and get the updated weather.

still delightful


Highlight content passed to Obsidian

When content is passed to Obsidian please also pass on the highlighting done in the app. Its good to get the highlights as notes, but it should also use ==markdown highlighting== wherever the content is highlighted in your app.

Now that this works, I get it as an html entity --> = is converted into = so no highlighting can be seen.


External Signers: MultiPath support + Fingerprint/PubKey verification

Task Description
Improve the verification(bip32Derv mfp/pubkey) and signing mechanism for External Signers in order to make them compatible for signing a Miniscript-based vault(multipath).
Note: External signers which are on the previous version of the app will require the app to be upgraded to the Miniscript-vault version in order to successfully sign a Miniscript-vault transaction, as they can’t currently parse the chain index(see internallySignVaultPSBT) in a way required to support the multipath approach of Miniscript vaults.

Addressed by: https://github.com/bithyve/bitcoin-keeper-private/pull/8

Closing as completed


Unable to get ACTIONS_ID_TOKEN_REQUEST_URL env variable

Run actions/deploy-pages@main
  with:
    token: ***
    emit_telemetry: false
    timeout: 600000
    error_count: 10
    reporting_interval: [5](https://github.com/AceKiron/portfolio/runs/6745490016?check_suite_focus=true#step:7:5)000
Error: Error: Error message: Unable to get ACTIONS_ID_TOKEN_REQUEST_URL env variable

That's all it says, running this action using
- name: Deploy artifact
  uses: actions/deploy-pages@main
  with:
    token: ${{ github.token }}

Repository for example: https://github.com/AceKiron/portfolio

Got that variable after setting permissions correctly (see https://github.com/glassechidna/ghaoidc/issues/1), but now I get 403 twice.
Run actions/deploy-pages@main
  with:
    token: ***
    emit_telemetry: false
    timeout: 600000
    error_count: 10
    reporting_interval: 5000
Actor: AceKiron
Action ID: [2](https://github.com/AceKiron/portfolio/runs/6745517291?check_suite_focus=true#step:7:2)44[3](https://github.com/AceKiron/portfolio/runs/6745517291?check_suite_focus=true#step:7:3)[4](https://github.com/AceKiron/portfolio/runs/6745517291?check_suite_focus=true#step:7:4)29149
Artifact URL: https://pipelines.actions.githubusercontent.com/jOLGc9JPBcAvEMZE0pB81t9ChTRNtIF3ANoYkTNjqEC[5](https://github.com/AceKiron/portfolio/runs/6745517291?check_suite_focus=true#step:7:5)z5XRzc/_apis/pipelines/workflows/2443429149/artifacts?api-version=[6](https://github.com/AceKiron/portfolio/runs/6745517291?check_suite_focus=true#step:7:6).0-preview
{"count":1,"value":[{"containerId":1[7](https://github.com/AceKiron/portfolio/runs/6745517291?check_suite_focus=true#step:7:7)755[8](https://github.com/AceKiron/portfolio/runs/6745517291?check_suite_focus=true#step:7:9),"size":20480,"signedContent":null,"fileContainerResourceUrl":"https://pipelines.actions.githubusercontent.com/jOLGc[9](https://github.com/AceKiron/portfolio/runs/6745517291?check_suite_focus=true#step:7:10)JPBcAvEMZE0pB81t9ChTRNtIF3ANoYkTNjqEC5z5XRzc/_apis/resources/Containers/177558","type":"actions_storage","name":"github-pages","url":"https://pipelines.actions.githubusercontent.com/jOLGc9JPBcAvEMZE0pB81t9ChTRNtIF3ANoYkTNjqEC5z5XRzc/_apis/pipelines/1/runs/2/artifacts?artifactName=github-pages","expiresOn":"2022-06-06T[14](https://github.com/AceKiron/portfolio/runs/6745517291?check_suite_focus=true#step:7:15):49:25.4263898Z","items":null}]}
Creating deployment with payload:
{
	"artifact_url": "https://pipelines.actions.githubusercontent.com/jOLGc9JPBcAvEMZE0pB81t9ChTRNtIF3ANoYkTNjqEC5z5XRzc/_apis/pipelines/1/runs/2/artifacts?artifactName=github-pages&%24expand=SignedContent",
	"pages_build_version": "cc257a9dc63c45[16](https://github.com/AceKiron/portfolio/runs/6745517291?check_suite_focus=true#step:7:17)85c38f5ff6d2a80354a09feb",
	"oidc_token": "***"
}
Failed to create deployment for cc257a9dc63c451685c38f5ff6d2a80354a09feb.
{"message":"Resource not accessible by integration","documentation_url":"https://docs.github.com/rest/reference/repos#create-a-github-pages-deployment"}
Error: Error: Request failed with status code 403
Error: Error: Request failed with status code 403

Got this fixed by using an entirely different action.


[BEAM-9547] Thorough testing for global aggregations

In the same spirit as #14850 I added some thorough parameterized tests for global aggregations, separated into their own test suite AggregationTests. Not surprisingly this knocked loose a few unhandled edge cases, also fixed in this PR:

We don't need a specialized implementation of nunique in DataFrame, it can be done with _agg_method. We can add a distributed implementation in Series though, based on drop_duplicates, and it will be used through DataFrame.agg.
Added logic to direct DataFrame.agg to specialized implementations for corr, cov, quantile and size.
Direct Series.agg to specialized implementations for corr, cov.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @rohdesamuel


rustfmt should value internal consistency in trait method signatures

Input
trait Sample {
    fn bytelen(width: u32, height: u32, samples: u32)
        -> ImageResult<usize>;

    fn from_bytes(bytes: &mut [u8], width: u32, height: u32, samples: u32)
        -> ImageResult<Vec<u8>>;

    fn from_ascii(reader: &mut dyn Read, width: u32, height: u32, samples: u32)
        -> ImageResult<Vec<u8>>;
}

Output
trait Sample {
    fn bytelen(width: u32, height: u32, samples: u32) -> ImageResult<usize>;

    fn from_bytes(mbytes: &mut [u8], width: u32, height: u32, samples: u32)
        -> ImageResult<Vec<u8>>;

    fn from_ascii(
        reader: &mut dyn Read,
        width: u32,
        height: u32,
        samples: u32,
    ) -> ImageResult<Vec<u8>>;
}

Expected output
None of the lines in the Input violate the line length requirements. Thus, I would expect that code is most legible if it has a consistent style and function parameters and return types are placed similarly. Instead, it changes the output to have all three different styles. Another acceptable output would be to change all or at least the second and third method to the fully expanded style.
trait Sample {
    fn bytelen(width: u32, height: u32, samples: u32)
        -> ImageResult<usize>;

    fn from_bytes(bytes: &mut [u8], width: u32, height: u32, samples: u32)
        -> ImageResult<Vec<u8>>;

    fn from_ascii(reader: &mut dyn Read, width: u32, height: u32, samples: u32)
        -> ImageResult<Vec<u8>>;
}

Meta

rustfmt version: 1.4.27-nightly (2020-11-16 580d826)
From where did you install rustfmt?: play.rust-lang.org
Similar to #3995, #3176 for which I would both have more samples but they have enough already.

Thank you for reaching out @HeroicKatora, however, this is working as designed. The formatting emitted by rustfmt is governed by the Rust Style Guide which prescribes specific formatting guidelines for all code constructs. All of the formatting rules are applied consistently to each node in the AST, and by design those rules are not conditionally applied/broken based on original formatting nor the formatted result of other/"related" elements.
The request here runs counter to the formatting approach of the style guide and thus rustfmt, so going to close this.


Adjust API error parsing to match new format

https://github.com/mesosphere/marathon/pull/3387
Quoting from issue:
As discussed in #3339, following changes have been applied:
replace [ ] with ( ) respectively
replace . by /
prepend / at the root of each path. This would also replace self with / in root-only errors
replace self with /
correct docs

Insights https://github.com/mesosphere/marathon/issues/3361#issuecomment-191781629


add dcos api token

Add dcos api token fixes #23

Thanks @mehtaphysical! I just made a few minor changes to get the automated checks passing again.


Lae ingest

Fixes #1193

Just a small thing and we should be good.

Coverage increased (+0.3%) to 99.678% when pulling 8f15ad85661c391267758206cb133528f9588d77 on lae-ingest into bae255704973d72addd95a92515fdd336003aba4 on master.

Coverage increased (+0.3%) to 99.678% when pulling 8f15ad85661c391267758206cb133528f9588d77 on lae-ingest into bae255704973d72addd95a92515fdd336003aba4 on master.


Editor crash on node hierarchy duplication

Operating system or device, Godot version, GPU Model and driver (if graphics related):
Win 7 64 bit, Godot 31f62f5
Issue description:
The minimal description I found is: when duplicating the instanced scene node with children in the main scene, one gets the editor crash.
Steps to reproduce:

Open the attached project:

select "DUPLICATE ME" node

duplicate it (CTRL+D)

crash

Link to minimal example project:
duplicate-crash.zip

cc @RandomShaper

Let's have a look!

I think the check for !current_node should be called earlier, i.e.:

	Node *current_node = node->get_node(get_path_to(N->get()));

	if (!current_node)
		continue;

	if (p_flags & DUPLICATE_SCRIPTS) {
		bool is_valid = false;
		Variant script = N->get()->get(script_property_name, &is_valid);
		if (is_valid) {
			current_node->set(script_property_name, script);
		}
	}

That would certainly fix the issue.
But we must delve deeper in order to understand why that can be null at that point in the first place. Otherwise we'll end up with lots of ad hoc logic no one really understands and no one dares to touch because it's actually dangerous.
Please have a look at #13288 to see more exactly what I mean.

@RandomShaper that makes perfect sense to me; thanks for the description in #13288


How to use?

Hello,
From the example in the read me it's not very clear how this library works. I may be missing something very crucial but I'm not sure.
So first off I copied the example method dynatable() into my controller and served it. However the route was a get and did not have the request input that dynatable needed like perPage, offset etc so it generated an error. So then I dug into the source and checked the resources/views/ folder to find the script and html blade templates.
I thought ok, I need to get these in my views because to use dynatable I need to setup the table first then use the sample code as an Json api of the table. But, the views has variable $dynatable and $options which the example does not bind or anything like that.
So my next step was to use the example with a get route but this time bind the $dynatable to the html view in the source code that I copied into my folder. Still didn't work.
Went deeper in the source to find this DynatableWidget class which seems like the ticket to figuring this thing out but it confused me still. It had the methods that called the two blade templates and returned a view and it also loaded the views on the register() method so I realized copying the views into my resources folder was a mistake. But I still don't know how to set this up.
How do I access the widget and setup the initial table so I can then copy the readme and set it as the route at which the table gets the data. Sorry for long post just trying to explain my process before asking such a simple question as "how do I use this?"

Hi !
First of all, do you know the dynatable jquery plugin ? (http://www.dynatable.com/)
Thanks for the detailed procedure.

Hello,
Yes I have looked at docs for the dynatable plugin already.
Where do the methods DynatableWidget->html() and script() get called? I think that's something that I'm missing...?

Hi,
Sorry for the lack of documentation, i am currently integrating dynatable into the white-frame framework (it is a work in progress). I updated some elements to the documentation, specially about the widget aspect, only available when used from white-frame.
So, if you are using white-frame/dynatable directly (without white-frame/white-frame) you should make your own html/javascript calling the laravel handler :
1 - Make a dynatable action to the controller of your choice (see example in documentation).
2 - Create a route to this action
3 - Create a new page with an html table and the dynatable javascript referencing on it, then configure the ajaxUrl param to the url of your router.
Then your dynatable javascript plugin should ask for the action controller to bring him some json datas and automatically fill the table.

Thank you for the quick responses.
I see what you are saying about not using the white-frame framework. It sounds like it would be better to just use the framework rather than use dynatable directly. If using directly, I lose out on the sorting and pagination that seems to be built into your framework.
I will try both methods (framework and direct) later today and get back to you tonight on what worked and didn't work.

Is there any chance you could finish the white frame framework usage for dynatable? I see that you started it in the readme. Would be extremely helpful 👍

I advise you to use the package directly as the white-frame/white-frame is a work in progress and have no solid documentation.
If you use white-frame/dynatable directly you will have all features including pagination, searching and more ! The only goal of white-frame/white-frame is to build automatically table with columns, automatically present datas using presenter etc ...
Yeah sure, i am still working on the whole white-frame project, but there is work to do before a stable release. As the code is already functionnal in a "big" project. I have to abstract all code to bring it here :)

Ok I'm back! Got it to work and it works flawlessly. I am very grateful for this repo.
One thing I am running into trouble with is this: I have two columns in my database, both of which are integers, and I want to combine them into one column on the front end table.
So lets call one int_one and the other int_two as columns. I want the output to be {{int_one}}/{{int_two}} so it looks like 3/4 remaining spots. But right now it just shows different columns int_one, int_two with the data associated with each column.
Is there a way to remove a column/combine a column with another?

For now, I combined these columns into a string field with the format above, but still curious if there is a method to do this with dynatables/ this repo.

Hi,
You can choose to not show int_one and int_two in the dynatable and use a new inexistant column instead. For example name it new_int where you can fill with the result of you division :
$dynatable->column('new_int', function($row) {
    return $row->int_one / $row->int_two;
});

Hello,
Thanks for the input I will change it to do that, seems much better. One last question, the Dynatable::of() requires a query builder for the first parameter, but what If I want to return all records if no search is input by the user yet?
I just want $cars = Cars::where('id', '>', 0) or something that returns all records where the WHERE has no effect. Is there a better way? Just to convert a model to a query builder or something.

Hi,
I am glad to hear thas is working well for you.
Instead of using Obj::all(), you can use Obj::query() for getting direct query builder without any conditions :)

All your questions are relevant, i added this response to the documentation too.
Thanks for your help :)

I have had good results with this after a few days. I'm going to close this as you have explained everything. Good work!

You welcome :-)

Just to notify, as the package is under heavy development and moved to the namespace WhiteFrame. If you composer upgrade to 2.2.1 (last stable) you will have to change Ifnot namespace to WhiteFrame namespace.


Update gems with bummr

Coverage remained the same at 86.158% when pulling 63a2159d4d9e0dd2a7371bd293c14e63922dbde9 on mb-update-gems into 74c24a530587e5cee0725b33380238aa91582f0f on master.

Coverage remained the same at 86.158% when pulling cb646792b89e13aaa3d902473994c1d0cc77c5a6 on mb-update-gems into 74c24a530587e5cee0725b33380238aa91582f0f on master.


File Transfer Protocol Implementation

Script Description -
File Transfer Protocol(FTP) is an application layer protocol that moves files between local and remote file systems. It runs on the top of TCP, like HTTP. To transfer a file, 2 TCP connections are used by FTP in parallel: control connection and data connection.
what will change -
This will add new project which is related to the Computer Networks (Application Layer)
Instructions

Create a new folder for your script and file/folder name should be appropriate.
Create a README.md in your folder for program Instructions
add requirements.txt if needed
add pytest/unittest if possible

Please add/delete options that are not relevant.

[x] Adding New Code
[x] Improving Documentation

Programming Language

[x] Python

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server  - https://discord.gg/FXyh2S3
Happy Coding,

@codePerfectPlus please accept the issue so that I can contribute FTP project for this hacktoberfest. Thanks

@Sushant-1999  interesting project.
Happy hacktoberfest


Need of contact name and available balance on send screen.

On send screen  there should be the name of the contact you are sending to and also need to show  available balance while sending.

There are lots of changes on Send screens after this big so this bug is invalid. Current UI we have Contact name and available balance also.


Update circuit drawing notebook for order changes again

At the last minute we decided to change the order back to what it was
before, and standardize on that for visualization. This commit adjusts
the visualizations in the notebook to that, by running with
Qiskit/qiskit-terra#1475 applied.

This depends on https://github.com/Qiskit/qiskit-terra/pull/1475 being merged, so we should wait for that before moving forward with this.

approving as it has a warning but for more changes lets put to terra branch. I am going to rebase that and then remove the warning.


Add ability to cancel a job [WIP]

What
This adds the ability to cancel a job, by making .cancel() method of a future actually do something.
Why

If you launch 1e4 lambdas and then think "oops!" this lets you bleed through the queue quickly -- the lambdas will each run for like 100ms to clear out the queue.
if you are using standalone mode for long-running tasks like deepnets and want to abort the remote work, this is the only supported way of saying "oops" and actually cancelling the deepnet process while it is running.

How
All it does is write a cancel key to s3; which our runner periodically polls for. We check for the cancel job id

at the very beginning of the job
every CANCEL_CHECK_EVERY_SECS while running the actual function

and then return an exception for the canceled functions.
Note that a function's "canceled" status isn't set on the client side but rather the client side writes the key and then relies on the remote to signal (via the exception) that it was canceled. Sometimes functions may be canceled but it's noticed remotely too late, or they are already done.
*Looking for feedback here. *
This once again exposed some of the weaknesses in our storage configuration abstraction; once again the future has to create a default storage handler inside itself.
associated with issue #187

The reason this was failing previously had to do with the difference in spelling of "cancelled" and "canceled" (British and American English). I went with "cancelled" because that seems to be what Python uses.
Another note is that if this is to be merged in with #281, there will have to be a bit more refactoring to make the wrenhandler changes work on Windows as well.


Deleting multiple entries at once from the Vi throws status 412 errors

possibly because the core cannot serve new skeys in rapid succession. could also be a problem with the Vi itself... not sure if this is the correct place or if the issue concerns both core and vi

this has to be handled differently by the vi so I am closing this issue and will open another one in the vi repo


Can't update database

Software Specs:

Ubuntu 16.04.4 LTS x86_64
Python 3.6.5
Bash Shell

Steps to Reproduce:

Clone this repository using git clone https://github.com/mubaris/motivate.git
cd motivate
Exec ./UPDATE

Expected Result:
Motivate quote database is updated.
Actual Result:
bash: ./UPDATE: No such file or directory
There is no "UPDATE" script inside the repository? So, how do I update the database?

Hey @mubaris ! Mind if I take this one?

I can not find Update file too


Setup Pylint, Pytest-Html, & CircleCI

Feature Request
Test that our app is live on servers upon every branch merge for Integration Testing
Deliverable

[ ] .circleci/config.yml
[ ] Ping the sign-in heroku webapp
[ ] Assert HTTP 200 responses

Helpful Links
https://youtu.be/kDWHPlrEgEY?t=118

If you want to setup circle ci with SHPE at UCI account, ping me for credentials.


Download PDF via button in PDF viewer

When viewing a PDF there are buttons to open the PDF in Preview or download. Opening a PDF in Preview works and the PDF is located in the Preview folder in my iCloud drive. Clicking the Save button does nothing. Is there a way to 'hook' the PDF(kit?) save button to initiate a download? Or should this work automatically?

Thanks for the report! I think this has worked previously, might be a new bug 🤔 This can happen when Fastmail changes download URL's


Remove "support" from incompatible images

Since it's possible to open and draw on files that you can't actually edit (e.g. SVG), there should either be:

a clear warning put up when you open such a file in the editor,
all the tools should be grayed out/locked,
or being able to open such files should be entirely removed.

I'd be happy to implement this myself if you're willing to bear with the time it will take me (as I'm new to Godot), just let me know which you prefer or if you have other ideas (also which filetypes that don't work).

Actually, it is possible to open them. Kinda. Just saves it as .png in the same folder when you try (thanks for reporting last time, that's what made me fix this)
At one point, I thought of implementing proper SVG editing support. Simple raster editors are good for pixelart but some people despise it, or just prefer vector after working in Flash (yes there are still people making games in Flash) Also would mean I get to not touch Inkscape again.
But that would take lots of time because of a completely different toolset needed, plus programming in the performant SVG rendering

Oh! Good to know, perhaps we should document this and / or add a note when in svg files?

Likely. I'll need to think how to better inform the user.

I just accidentally lost a file because the plugin overwrote a png with the same name when opening an svg.
Even though I didn't save it.  I think it would be nice to have something for file security.

I just accidentally lost a file because the plugin overwrote a png with the same name when opening an svg. Even though I didn't save it. I think it would be nice to have something for file security.

Yes, when you click on the Discard button, the plugin still saves the file, apparently.


Add add and remove buuton functions with dispact hook

In this milestone of the project, i have done the following;

I have displayed books, received from the Bookslice.
I have added the a button with a function to call the addBook reducer witrh the useDispact hook.
I have added the a button with a function to call the removeBook reducer witrh the useDispact hook.
I have added the a button with a function to call the checkstatus reducer witrh the useDispact hook.
i disabled the camel-case ruse so as to let me use the item_id as stated in the instruction.

Noted and thank you. I was confused about this.
Sent from Yahoo Mail on Android
On Wed, Mar 8, 2023 at 4:43 PM, Adedamola @.***> wrote:
@DammyShittu requested changes on this pull request.
Hi @Divinity-dev,
Highlights
✔ Books addition functionality
Good job so far!
There are some issues that you still need to work on to go to the next project but you are almost there!
Required Changes ♻
Check the comments under the review.
Cheers and Happy coding! 👏 👏 👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.
As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.
In src/Components/Book.js:

Action

{title}

{author}

comments

remove

     <button className="btn" type="button" onClick={() => dispatch(removeBook({ item_id }))}>remove</button>

     <button className="btn" type="button" onClick={() => dispatch(checkStatus())}>check status</button>

I think this checkStatus dispatch here is meant for the categories component. When I visit the categories component and click the check status button, you should dispatch an action to the store and fetch the "under construction" string from the store and display it on the page. Kindly do that.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Thank you for the feedback
On Wednesday, March 8, 2023 at 08:42:40 PM GMT+1, David Bassey @.***> wrote:
@Davidosky007 approved this pull request.
Hi,
Your project is complete! There is nothing else to say other than... it's time to merge it
Congratulations! 🎉
Highlights

No linter errors
Good looking PR description
Proper use of Gitflow

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will improve your application.
Cheers and Happy coding! 👏 👏 👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


⚠ Bofrost.de has degraded performance

In 3acee4f, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 778 ms

Resolved: Bofrost.de performance has improved in 19151b2 after 10 minutes.


Training error

When I run the script for my own classes I get an error. I made sure that the training key and images url are correct but could't solve the problem. I should also specify that the images are local
File "C:\Users\danbe\AppData\Local\Programs\Python\Python36\lib\site-packages\azure\cognitiveservices\vision\customvision\training\custom_vision_training_client.py", line 1018, in create_images_from_urls
raise HttpOperationError(self._deserialize, response)
msrest.exceptions.HttpOperationError: Operation returned an invalid status code 'Bad Request'

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

ID: 646cbb8a-041e-9d99-0385-272f31793552
Version Independent ID: b9ae4376-2e8f-f311-8b1b-d4e3120bf5b5
Content: Quickstart: Create an image classification project with the Custom Vision SDK for Python
Content Source: articles/cognitive-services/Custom-Vision-Service/python-tutorial.md
Service: cognitive-services
GitHub Login: @areddish
Microsoft Alias: areddish

@Tzuya14 Thanks for your feedback, I will investigate and get back to you soon.

Hi @Tzuya14, this error message usually indicates you've hit the project limit. If you navigate to https://customvision.ai and click on the gear icon in the upper right it should take you to a page that lists your limits.
Towards the bottom left under the training key you should see text of the form "XX projects created; YY remain"

@areddish Hi, I checked the limits and I have "1 projects created; 99 remain" and Unlimited predictions

@Tzuya14 sorry, I see now that the bad request was from create_images_from_urls so you were successful in creating the project. Were any of the images uploaded to the project in the portal? Or did it fail on the first one?
 Does that mean you are passing "file://" urls? Or how are you specifying the url?
For local files we there is create_images_from_files. I can provide you a sample for that if it helps.

@areddish When I check the web page the project is created but no pictures are uploaded. I just declared the url as "data/images/image.jpg". I was thinking maybe here is the problem

Ah yes, that will result in an invalid url to the service, hence the bad request. You want to use create_images_from_files. Something like this:
trainer = CustomVisionTrainingClient(training_key, endpoint=ENDPOINT)

# Create a new project
print("Creating project...")
project = trainer.create_project("Man_vs_Woman")

# Make two tags in the new project
man_tag = trainer.create_tag(project.id, "Man")
woman_tag = trainer.create_tag(project.id, "Woman")

man_images_list = os.listdir("data/train/man/")
woman_images_list = os.listdir("data/train/woman/")

print("Adding images...")
for image_num in man_images_list:
    file_name = "data/train/man/"+image_num
    with open(file_name, mode="rb") as image_contents:
        trainer.create_images_from_files(project.id, images=[ImageFileCreateEntry(name=file_name, contents=image_contents.read()], tag_ids=[man_tag.id])])

for image_num in woman_images_list:
    file_name = "data/train/woman/"+image_num
    with open(file_name, mode="rb") as image_contents:
    	trainer.create_images_from_files(project.id, images=[ImageFileCreateEntry(name=file_name, contents=image_contents.read()], tag_ids=[woman_tag.id])])

print("Training...")
iteration = trainer.train_project(project.id)
while (iteration.status != "Completed"):
    iteration = trainer.get_iteration(project.id, iteration.id)
    print("Training status: " + iteration.status)
    time.sleep(1)

# The iteration is now trained. Make it the default project endpoint
trainer.update_iteration(project.id, iteration.id, is_default=True)
print ("Done!")

@areddish I have another question, if I want to use an existing model to make some predictions, what values should I declare in results = predictor.predict_image()?
I tried the following but didn't work
predictor = CustomVisionPredictionClient(prediction_key, endpoint=ENDPOINT)

man_images_list = os.listdir("data/validation/man/")

print("Adding images...")
for image_num in man_images_list:
    file_name = "data/validation/man/"+image_num
    with open(file_name, mode="rb") as test_data:
        results = predictor.predict_image("manvswoman", test_data, "3")
        print(results)

@Tzuya14 when you train a model you get back an iteration. You can pass that iteration in the predict image call to target that specific model iteration. The above code will set the latest trained iteration to be the default which is what predict_image will use if you don't pass an explicit iteration.
If you want to re-use an iteration in a separate program (or without retraining) you just need to save off the project id and iteration id and then re-use them in the predict_image call:
predictor = CustomVisionPredictionClient(prediction_key, endpoint=ENDPOINT)

man_images_list = os.listdir("data/validation/man/")

print("Adding images...")
# Fill this in with your project id returned from create_project. Alternatively, you can create a
# CustomVisionTrainingClient and then call get_projects and filtered that by the project if
# only have the name.
my_project_id = "<your project id>"
# Fill this in with your iteration id returned from train_project or get_iteration. You can get a list of your
# iterations by calling trainer.get_iterations(my_project_id).
my_iteration_id = "<your iteration id>"
for image_num in man_images_list:
    file_name = "data/validation/man/"+image_num
    with open(file_name, mode="rb") as test_data:
        results = predictor.predict_image(my_project_id, test_data, iteration_id=my_iteration_id)
        print(results)


Fix string on payment recipt

There's a faulty string text that appears on our recipts. Back when we developed the payment feature, it was a walk around to handle hebrew, but not it's just a bug.
In order to debug it, we have a dummy credit card info and development configuration for the payment service, will be given when requested from Gal.
The wanted text is
תרומה חודשית לעמותת מעירים

yaad api documentation
https://yaadpay.docs.apiary.io/#introduction/soft-protocol-transaction-in-web-server/step-2-soft-transaction-data-transfer-method

seems like a bug on yaad side
they said they will fix it

contacted them again about this
seems to be low priority on their side
said they'll let me know when they get to it

another reminder email sent....

another reminder email sent....


add module-invalidate in Miscellaneous section

module-invalidate allows you to invalidate a given module (or all modules) and make it automatically reloaded on further access, no need to call require() again.

Needs to be older than 30 days

Ok, let's wait 23 days :)

Can you reopen please ?


Change RNG semantics to help Poseidon.

Change RNG semantics to not require 32 bits, only the number actually required, so that Poseidon can make RNGs reasonably

The latest benchmark check shows 10% increase on linux-cuda, not sure what to do with that.

Re the benchmark, is there a way I can just rerun. Currently, the code doesn't change anything related to the normal code path. For all the SHA based hals, it's effectively a no-op, so I strongly suspect it's just noise.

Its probably noise. Currently benchmarks don't trigger PR blocking are just for information. I will look into if we can get that noise floor down though.
If you wanna trigger a re-run of a workflow like the Benchmark Check you can open up the completed workflow in the actions view and then click rerun all jobs in the upper left.


Baseline doc for User Consent Management

This PR provides a first draft of the doc for user consent management. The aim is to agree on the ToC.

@jordonezlucena off-topic, just about the contribution process (see CONTRIBUTING.md): the official contribution process is "fork and pull" ... which means the preparation of a pull request should happen in a fork of the repo (it's just one click to create it on the top of the repo) and then the pull request should be done from there. Creating a branch within the main repo is an exception you can do as codeowner (with write access to the repo) but no other contributor. The advantage is that you have less risk to accidentially change something in the main repo.

Hi,
regarding the concept of purpose, we (Orange) would like to correctly understand the benefits to add the purposes to the OIDC standard. We already have a group component using the standard and there will be impacts in this case. Our understanding is that if purpose is studied/proposed, it means that the standard with scopes isn't sufficient for all the use case needed in the camara perimeter. However, we currently don't know which use case won't be covered by the standard. Or is it just in order to "provides a human-friendly description" ? We also wonder if adding the purpose would complicate the standard with another layer to define, administer, manage changes, agree on vocabulary etc.
thank you in advance for the clarifications

@sebdewet This is actually a really good point and is open for discussion. I've created a specific github issue (https://github.com/camaraproject/IdentityAndConsentManagement/issues/32) to get agreement on this. Because I think we can discuss it separately without interfering with this PR, which is just intended to merge a first baseline template document proposal that will need to be progressively edited and developed in the near future to document the complete final agreed solution.

Reviewers (@chrishowell @gmuratk @shilpa-padgaonkar @sebdewet @rartych @eric-murray) and all, do you agree (given the latest changes included in the template document proposal) to merge the document as a starting point for discussion?
As we said in the last call, everything in the document so far is set in stone and the document will evolve and everything can be discussed/changed. I don't see the point in being too strict about the content of the document, as it is only a draft template proposal.

@jpengar

everything in the document so far is set in stone
Guess you meant the opposite, right? 😃

@jpengar

everything in the document so far is set in stone

Guess you meant the opposite, right? 😃

Edited. XD. Exactly, just the opposite... My bad.

We should carefully check if this will fit for all scenarios (e.g. event subscriptions for a set of devices or were consent need to be granted upfront to avoid an interruption of the actual service). Another scenario could be that the API/capability consumer does not have a sufficient identifier beforehand, but only some data which will be resolved on operator side into an identifier ... read: a scenario were the consent can only be checked within the service logic and the API consumer might not need to get an identifier of the subject.

This is a really good point, and there is a specific issue (https://github.com/camaraproject/IdentityAndConsentManagement/issues/21) where all the necessary scenarios to cover should be discussed and agreed upon. Including the ones you mention. We can move the discussion there if you like.

@gmuratk Since you approved the content of this PR, should we close the baseline document feedback issues you opened? https://github.com/camaraproject/IdentityAndConsentManagement/issues/25, https://github.com/camaraproject/IdentityAndConsentManagement/issues/26, https://github.com/camaraproject/IdentityAndConsentManagement/issues/27 & https://github.com/camaraproject/IdentityAndConsentManagement/issues/28

I don't really see the problem here.

@jpengar let's postpone this discussion until we have more concrete flows on the table.

@gmuratk Since you approved the content of this PR, should we close the baseline document feedback issues you opened? #25, #26, #27 & #28

I closed these issues as they are addressed with the changes introduced in the recent commit.

Could you take a look and give your final approval? I think we should be able to merge the baseline document now...


Place MapRoots into Log by index

Now that PREORDERED_LOGs are a thing, we can insert map roots into the
log in a deterministic way, and remove the sword of Damocles that was
previously there due to non-deterministic log sequencing

LGTM but let's wait until it passes CT tests.

Codecov Report

Merging #988 into master will decrease coverage by 0.04%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master    #988      +/-   ##
=========================================
- Coverage   49.95%   49.9%   -0.05%
=========================================
  Files          29      29
  Lines        2028    2028
=========================================
- Hits         1013    1012       -1
- Misses        835     836       +1
  Partials      180     180

Impacted Files
Coverage Δ

core/adminserver/admin_server.go
56.75% <0%> (ø)
:arrow_up:

core/client/client.go
29.37% <0%> (-0.63%)
:arrow_down:

Continue to review full report at Codecov.

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


[crystal mode] Fix page title

Sorry :cry: . I forgot to fix this in copied and pasted index.html from Ruby's.

Hah. I was just about to push my own patch that fixed those. Merged yours as 6bc4c5a


Publish

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
Thank you for contributing to harp.gl!
Before requesting a pull request, please remember to check the following documents:

contribution guidelines
coding style

If you are adding new functionality we would highly appreciate if you can describe what is the capability you are adding and even better if you can add some examples. Please also remember to add tests for it.
CI Check
Our bots will check whether your PR can be directly integrated into the mainline. We have some internal integration tests running on the background, our bots will inform you of the next steps and someone from our team will take a look and help if needed!
And please do not forget to sign-off your commit! You can read more about DCO here. But, in short, you just need to use git commit -s or append --signoff when you are committing to the repo.
Happy contributing!

Codecov Report

Merging #1506 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1506   +/-   ##
=======================================
  Coverage   60.15%   60.15%
=======================================
  Files         262      262
  Lines       23855    23855
  Branches     5368     5368
=======================================
  Hits        14351    14351
  Misses       9504     9504

Continue to review full report at Codecov.

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