About the response of entry scripts generated by the designer Train Model

[Enter feedback here]
https://docs.microsoft.com/en-us/azure/machine-learning/how-to-deploy-model-designer#download-the-entry-script-file-and-conda-dependencies-file
The entry script (score.py) in the steps above does not know what the order of the response results will be. For example, for the Multiclass Classification - Letter Recognition sample, the inference results are as follows:

Because the result is in a list format, it cannot be used as is. The entry script is as follows, but cannot be changed because the ScoreModelModule specification is unknown. Could you make the response dictionary-style by default, or publish how to change it?
import os
import json

from azureml.studio.core.io.model_directory import ModelDirectory
from pathlib import Path
from azureml.studio.modules.ml.score.score_generic_module.score_generic_module import ScoreModelModule
from azureml.designer.serving.dagengine.converter import create_dfd_from_dict
from collections import defaultdict
from azureml.designer.serving.dagengine.utils import decode_nan
from azureml.studio.common.datatable.data_table import DataTable

model_path = os.path.join(os.getenv('AZUREML_MODEL_DIR'), 'trained_model_outputs')
schema_file_path = Path(model_path) / '_schema.json'
with open(schema_file_path) as fp:
    schema_data = json.load(fp)

def init():
    global model
    model = ModelDirectory.load(model_path).model

def run(data):
    data = json.loads(data)
    input_entry = defaultdict(list)
    for row in data:
        for key, val in row.items():
            input_entry[key].append(decode_nan(val))

    data_frame_directory = create_dfd_from_dict(input_entry, schema_data)
    score_module = ScoreModelModule()
    result, = score_module.run(
        learner=model,
        test_data=DataTable.from_dfd(data_frame_directory),
        append_or_result_only=True)
    return json.dumps({"result": result.data_frame.values.tolist()})

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

ID: 87ec1212-ecfd-0c70-94c5-7f3be40a2262
Version Independent ID: ee729221-528a-a30c-cb85-8753506d86eb
Content: Use the studio to deploy models trained in the designer - Azure Machine Learning
Content Source: articles/machine-learning/how-to-deploy-model-designer.md
Service: machine-learning
Sub-service: core
GitHub Login: @likebupt
Microsoft Alias: keli19

@jpmlblog
Thanks for the question. We are investigating the issue and will update you shortly.

@jpmlblog  Thanks, you can return any data type as long as it is JSON-serializable.  Please follow the configure entry script .
https://docs.microsoft.com/en-us/azure/machine-learning/how-to-deploy-model-designer#configure-the-entry-script

@jpmlblog We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


do you want to update github.com/golang/protobuf to google.golang.org/protobuf ?

What version of V2Ray are you using?
github.com/v2fly/v2ray-core/v5 v5.7.1-0.20230812013354-508f0c9237f9

What's your scenario of using V2Ray?
Compile my own project with v2ray as a library

What problems have you encountered?
github.com/golang/protobuf Library shows as deprecated

What's your expectation?
For the official explanation of the difference between the two libraries
https://protobuf.dev/reference/go/faq/#versions
I wonder if there is any idea to replace this library in this project?

Sure, but the job can be a bit daunting.


Prevent privilege escalation by exec'ing into a container with more privileges than the user

Check if a user has at least the same permissions as the container before allowing to exec into it.

Already implemented by Darren in #2187

Tested with acorn version v0.9.1-16-g89212fea+89212fea

Deploy app with permissions

containers: foo: {
  image: "nginx"
  permissions: rules: [
     {
     scope: "account",
     apiGroup: "api.acorn.io",
     resource: "*",
     verb: "*"
     }
    ]
}

Share the project with another user  say user1 with Admin role.
User1 is not allowed exec into this app.

User gets a 403 forbidden: not authorized error.
acorn exec mytestnewprivilege
  ✗  ERROR:  websocket: bad handshake: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"containerreplicas.api.acorn.io \"mytestnewprivilege.foo-5848588779-6vxs9\" is forbidden: not authorized: [{\"serviceName\":\"foo\",\"rules\":[{\"verbs\":[\"*\"],\"apiGroups\":[\"api.acorn.io\"],\"resources\":[\"*\"],\"scopes\":[\"account\"]},{\"verbs\":[\"*\"],\"apiGroups\":[\"api.acorn.io\"],\"resources\":[\"*\"],\"scopes\":[\"account\"]}]}]","reason":"Forbidden","details":{"name":"mytestnewprivilege.foo-5848588779-6vxs9","group":"api.acorn.io","kind":"containerreplicas"},"code":403}


Adding the new parameter of the compactor for the ResLevel[12]DownsampleRange or the function of the query minSourceResolutionMillis

Is your proposal related to a problem?

(Write your answer here.)
I'm using thanos compactor, query, store with the grafana.
The compactor
        - compact
        - --log.level=debug
        - --delete-delay=24h
        - --wait
        - --wait-interval=2h
        - --objstore.config-file=/etc/objstore-config/bucketconfig.yaml
        - --retention.resolution-1h=0
        - --retention.resolution-5m=180d
        - --retention.resolution-raw=30d

The query
        - query
        - --log.level=debug
        - --query.auto-downsampling
        - --query.instant.default.max_source_resolution=1h
        - --query.replica-label=prometheus_replica
        - --query.replica-label=thanos_ruler_replica
        - --store=dnssrv+_grpc._tcp.thanos-sidecar.sre-mtr.svc.cluster.local
        - --store=dnssrv+_grpc._tcp.thanos-store.sre-mtr.svc.cluster.local
        - --store=dnssrv+_grpc._tcp.thanos-ruler.sre-mtr.svc.cluster.local

The store
        - store
        - --chunk-pool-size=64GB
        - --log.level=debug
        - --objstore.config-file=/etc/objstore-config/bucketconfig.yaml
        - --ignore-deletion-marks-delay=1d
        - --grpc-address=<IP_ADDRESS>:10901
        - --http-address=<IP_ADDRESS>:10902
        - --data-dir=/var/thanos/store

According to above, my raw, 5m and 1h data would be stored 30days, 180days, forever respectively.
Since I want to show the metric before 30days, I set the custom query parameter in the grafana datasource of the thanos query as max_source_resolution=1h
And in some grafana panel, I used the rate function with vector range 2m about the network.  (2m-rate-function below)
If I change the time range in the dashboard from 30m to 7days, the panel with the 2m-rate-function shows the 40hours data only. (exactly the data before 40h are seen a little bit sad...)

If I change the vector range from 2m to 10m (to above 5m and below 1h) it works only for the data between 40h and now.

If I change the vector range from 10m to 2h (to above 1h) it works for it works for every data. But the graph is too rough.
I want to get the data as detail as possible within any time range.
Describe the solution you'd like

(Describe your proposed solution here.)
So I think there could be 2 options.

add the new cli parameter of the compactor
add the new function of the query

Describe alternatives you've considered

add the new cli parameter of the compactor
As I researched, at the compactor there are two const ResLevel1DownsampleRange, ResLevel2DownsampleRange.
Both above are hard-coded as  40hours and 10days respectively at the thanos/pkg/compact/downsample/downsample.go#45
How about get both time range from the user?
If then, I could set the compactor make 5m, 1h data for the data after 30d, 180d respectively.
Then the query would find raw, 5m data only within 30d, 180d respectively.

add the new function of the query
As I inspected, there was maxSourceResolutionMillis function at the thanos/cmd/thanos/query.go
I think this runction return the max resolution (0, 300000, 3600000 for raw, 5m, 1h)
How about the minSourceResolutionMills? It returns exactly the reverse value of the maxSourceResolutionMillis.
Let assume below    (A,B,C are specific period and A is the most recent)
raw:   A
5m:  A,B
1h:  A,B,C
If I set min_source_resolution as 1h and query for the A,B and C timeline, then the query would return raw, 5m and 1h respectively.
For the A period, there are raw,5m and 1h data, so query returns raw data.
For the B period, there are 5m and 1h data, so query returns 5m data.
For the A period, there are 1h data, so query returns 1h data.

(Write your answer here.)
Additional context

(Write your answer here.)

Have you tried setting $__rate_interval in Grafana? https://grafana.com/docs/grafana/latest/datasources/prometheus/template-variables/#:~:text=We recommend using %24__rate_interval,avoid problems specific to Prometheus.
This could adjust the interval dynamically based on the your time range. It might not work perfectly for you but should be better than now.

@wotkddl21 Do you use query frontend? Query frontend has a feature to request downsampled data. You can try it by enabling --query-range.request-downsampled.

@yeya24
I tried query frontend with --query-range.request-downsamples.
But it is same as without the frontend.

        app: thanos-query-frontend
    spec:
      containers:
      - args:
        - query-frontend
        - --log.level=debug
        - --query-frontend.downstream-url=http://thanos-query.sre-mtr.svc.cluster.local:10902
        - --request.logging-config-file=/etc/log/config
        - --query-range.request-downsampled

My bad, I think the query frontend doesn't work well for your case because of the range is not big enough. To get it working you need to zoom in the graph.
I don't know if there is an easy way to get it working for your setup without any code changes. There is a workaround to add template variable for the range like rate(xxx[$range]) but it is still kind manual.  https://community.grafana.com/t/tweaking-the-interval-variable/84218/5
I don't think we will expose the downsampling configuration of compactor. It doesn't solve the root cause of the problem. Your grafana dashboard can break at some point (or break for other users) even if you change the downsampling time range.
And I don't think we can add a new query function as query engine shouldn't be aware of downsampling.

@yeya24 OK
Then I have to split the dashboard for long and short term.
I will close this issue. Thanks

@yeya24 You mean the custom datasource for the grafana?

@wotkddl21, no. I am thinking that the custom max_source_resolution=1h is not flexible at the datasource level because it applies on every query.
We should drop that. Instead we have a feature in Thanos QFE to inspect your query and set max_source_resolution based on the input query string.


[BUG] debian package 3.33 not supported on bullseye yet (no zstd support)

The debian package 3.33 uses zstd which is supported on ubuntu but only extremely recently on debian.
Please see
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=892664#129
Note that previous package v3.32 is ok (this is my turnaround for now).
Today, my bullseye says:

dpkg-deb --help|grep 'types'
Allowed types: gzip, xz, none.
=> I miss zstd yet, and it might take some time until it is supported in bullseye I suppose.

Nicolas

Hi @ngaullier

Today, my bullseye says:
dpkg-deb --help|grep 'types'
Allowed types: gzip, xz, none.

Mine says the same and the TSDuck package tsduck_3.32-2983.debian11_amd64.deb (and -dev) installs flawlessly. This is a standard Debian 11 bullseye with up-to-date packages.
I do not know what you mean by "debian package 3.32 uses zstd". What is your exact problem?
I have 10 different virtual machines with 10 different Linux distros (plus 2 Windows VM and 4 BSD VM, all distinct flavors). I cannot master all the gory details of each of them. They are "standard" installations with all prerequisites to build TSDuck (see script install-prerequisites.sh) and, most of the time, up-to-date packages. That's all.
I do my best to provide pre-built binaries for official releases on the latest version of the 4 major distros (Ubuntu, Debian, Fedora, Redhat). For any other version or distro, you need to rebuild.
In case of problem, I recommend to rebuild from the repo. The .deb or .rpm packages (depending on the distro) are built using make installer. For other distros (neither .deb nor .rpm), you need to make install, directly.

Hi,
First, I have to thank you for all your work, including indeed all the available pre-built binaries, this is a great effort, very rarely seen elsewhere.
I think this issue is minor in that it may be fixed in the coming months (without any action from you), but it may be of interest for users encountering the same problem. I could also have build from the sources (I did not have because of missing disk space on my systems). So, you can close this ticket, for sure, this is so much you already have done.
So just for information, and to dig into the details:

ar x ./tsduck_3.32-2983.debian11_amd64.deb
extracts 'control.tar.xz' and, as the file extension presumes:
file ./control.tar.xz
./control.tar.xz: XZ compressed data

Whereas with v3.33:

ar x ./tsduck_3.33-3139.ubuntu22_amd64.deb
extracts 'control.tar.zst' and:
file ./control.tar.zst
./control.tar.zst: Zstandard compressed data (v0.8+), Dictionary ID: None

When I run 'sudo apt-get install ./tsduck_3.33-3139.ubuntu22_amd64.deb', I get this:
dpkg-deb: error: archive '/home/xxx/tsduck_3.33-3139.ubuntu22_amd64.deb' uses unknown compression for member 'control.tar.zst', giving up
And here it is:

dpkg-deb --version
Debian 'dpkg-deb' package archive backend version 1.20.12 (amd64).

It seems zstd was added very recently on debian on version 1.21.18:
https://github.com/guillemj/dpkg/commit/ebc2c3def335ac391b010323ab65a302648f3ec7
I am definitely not a distribution expert, so I don't catch how it is possible to get in such a situation...
Anyway, it is good to know about it.
Thank you again for your great tool
Nicolas

Is your Debian 11 "up to date"?
My Debian VM is also a Debian 11 bullseye and I do not see the same problem.  Do you have the opportunity to update your system (sudo apt update; sudo apt upgrade)?
Moreover, your initial post mentions that 3.32 uses zstd and does not work while 3.31 works. Now your latest post (after I just released 3.33 yesterday) mentions that 3.32 uses xz (and 3.33 uses zstd and does not work). Could you please clarify about 3.32?

I could also have build from the sources (I did not have because of missing disk space on my systems).

It is true that the history of the tsduck repo is quite big. This is due to my original mistake of having included Microsoft Office files as regular git files instead of using git-lfs for them.
If you have at least 200 MB free disk, use git clone --depth 1 to grab the latest state only, skipping the history. The repo size is then about 50 MB only. The complete build process should not use more than 150 MB, probably less.

I am very sorry, it seems I confused the debian and ubuntu packages since I can't reproduce the problem anymore!
I have re-downloaded/installed the latest v33 debian on both my debian VMs with no problem.
Thank you for the git clue, I am still interested in compiling anyway.
Really sorry for having having wasted your time.
I will just try to close the ticket properly now.


DFBUGS-937: [release-4.16] CVE css-tools: Improper Input Validation causes Denial of Service via Regular Expression

Issue: https://issues.redhat.com/browse/DFBUGS-937

/jira refresh

/approve

/approve

/override ci/prow/odf-console-e2e-aws

/override ci/prow/images


Drop the bundle and OCM deployment tests

These are currently broken, and as discussed in today's development
meeting, should be dropped. They will be restored once bundle
construction is fixed (as part of the planned Operator SDK upgrade).
Fixes: #717
Signed-off-by: Stephen Kitt<EMAIL_ADDRESS>

Backported as #750.


set the teacher.model_path under generate

This should fix 'ilab data generate' failing on not finding /root/.cache/instructlab/models/merlinite-7b-lab-Q4_K_M.gguf (default model) in case it was initialized with another model
Issue resolved by this Pull Request:
Resolves # RHELAI-625
Checklist:

[x] Commit Message Formatting: Commit titles and messages follow guidelines in the
conventional commits.
[x] Changelog updated with breaking and/or notable changes for the next minor release.
[x] Documentation has been updated, if necessary.
[x] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.

@mergifyio rebase


Remove call to qr in orthogonalize! so it works with qns

Would this be a good use case for factorize? Because then later we could just upgrade factorize for QN ITensors to use QR internally and then this code will automatically speed up from that.

Nice to see Travis is working again too for some reason

Codecov Report

Merging #254 into master will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #254      +/-   ##
==========================================
+ Coverage   79.83%   79.94%   +0.10%
==========================================
  Files          50       50
  Lines        4805     4806       +1
==========================================
+ Hits         3836     3842       +6
+ Misses        969      964       -5

Impacted Files
Coverage Δ

src/mps/mpo.jl
99.39% <100.00%> (ø)

src/Tensors/dense.jl
82.00% <0.00%> (-0.15%)
:arrow_down:

src/physics/site_types/spinhalf.jl
100.00% <0.00%> (ø)

src/mps/mps.jl
99.27% <0.00%> (+<0.01%)
:arrow_up:

src/Tensors/blocksparse/blocksparsetensor.jl
88.28% <0.00%> (+0.28%)
:arrow_up:

src/itensor.jl
76.65% <0.00%> (+0.38%)
:arrow_up:

src/Tensors/blocksparse/diagblocksparse.jl
33.54% <0.00%> (+0.64%)
:arrow_up:

src/qnitensor.jl
72.72% <0.00%> (+4.54%)
:arrow_up:

Continue to review full report at Codecov.

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

Yes that's a good point, let's do that.
I was wondering if you did something to get it back, but I guess it was just taking a break.

The test failed because it ran out of time on the mac build (not sure why it took so long...). Besides that, it is passing, I'll merge this.


[Question]: Follow the readme, i can not start program by 'bash docker/entrypoint.sh'

Describe your problem
Here are all the console outputs:

i had set <IP_ADDRESS>   es01 mysql minio redis to /ect/hosts.   <IP_ADDRESS> is my virtual machine address。
i had update  entrypoint.sh

        ......
        #/usr/sbin/nginx
        #export LD_LIBRARY_PATH=/usr/lib/x86_64-linux-gnu/
        export PYTHONPATH=$(pwd)
        export HF_ENDPOINT=https://hf-mirror.com
        ......

environment : M1 macbook

code 137.
It's been killed for lack of memory, may be.

code 137. It's been killed for lack of memory, may be.

The memory is enough, is there a way to display the detailed log

M1 MacBook cpu is arm. Which ragflow image are you using?

M1 MacBook cpu is arm. Which ragflow image are you using?

Launch the RAGFlow Backend Service From Source
i use this  docker-compose-base.yml deploy  es，redis......  on my virtual machine.  centos9
services:
  es01:
    container_name: ragflow-es-01
    image: elasticsearch:8.11.3
    volumes:
      - esdata01:/usr/share/elasticsearch/data
    ports:
      - 9200:9200
    environment:
      - node.name=es01
      - ELASTIC_PASSWORD=infini_rag_flow
      - bootstrap.memory_lock=false
      - discovery.type=single-node
      - xpack.security.enabled=true
      - xpack.security.http.ssl.enabled=false
      - xpack.security.transport.ssl.enabled=false
      - TZ='Asia/Shanghai'
    mem_limit:<PHONE_NUMBER>
    ulimits:
      memlock:
        soft: -1
        hard: -1
    healthcheck:
      test: ["CMD-SHELL", "curl http://localhost:9200"]
      interval: 10s
      timeout: 10s
      retries: 120
    networks:
      - ragflow
    restart: always

  mysql:
    # mysql:5.7 linux/arm64 image is unavailable.
    image: mysql:8.0.39
    container_name: ragflow-mysql
    environment:
      - MYSQL_ROOT_PASSWORD=infini_rag_flow
      - TZ='Asia/Shanghai'
    command:
      --max_connections=1000
      --character-set-server=utf8mb4
      --collation-server=utf8mb4_general_ci
      --default-authentication-plugin=mysql_native_password
      --tls_version="TLSv1.2,TLSv1.3"
      --init-file /data/application/init.sql
    ports:
      - 3306:3306
    volumes:
      - mysql_data:/var/lib/mysql
      - ./init.sql:/data/application/init.sql
    networks:
      - ragflow
    healthcheck:
      test: ["CMD", "mysqladmin" ,"ping", "-uroot", "-pinfini_rag_flow"]
      interval: 10s
      timeout: 10s
      retries: 3
    restart: always

  minio:
    image: quay.io/minio/minio:RELEASE.2023-12-20T01-00-02Z
    container_name: ragflow-minio
    command: server --console-address ":9001" /data
    ports:
      - 9000:9000
      - 9001:9001
    environment:
      - MINIO_ROOT_USER=rag_flow
      - MINIO_ROOT_PASSWORD=infini_rag_flow
      - TZ='Asia/Shanghai'
    volumes:
      - minio_data:/data
    networks:
      - ragflow
    restart: always

  redis:
    image: redis:7.2.4
    container_name: ragflow-redis
    command: redis-server --requirepass infini_rag_flow --maxmemory 128mb --maxmemory-policy allkeys-lru
    ports:
      - 6379:6379
    volumes:
      - redis_data:/data
    networks:
      - ragflow
    restart: always

volumes:
  esdata01:
    driver: local
  mysql_data:
    driver: local
  minio_data:
    driver: local
  redis_data:
    driver: local

networks:
  ragflow:
    driver: bridge

And i follow  https://ragflow.io/docs/dev/launch_ragflow_from_source
run by bash docker/entrypoint.sh

Show your docker/.env

Where did you run bash docker/entrypoint.sh, on M1 macbook, or the linux vm <IP_ADDRESS>?
We don't have M1 macbook. However I confirm that this procedure work on linux/arm64.

Where did you run bash docker/entrypoint.sh, on M1 macbook, or the linux vm <IP_ADDRESS>? We don't have M1 macbook. However I confirm that this procedure work on linux/amd64 and linux/arm64.

on M1 macbook

When I debug the breakpoints, it seems to be the code spec.loader.exec_module(page) that fails when conversation_app.py  is loaded。

What can be ruled out is the inability to access the virtual machine, and this is my service_conf.yaml file：

ragflow:
  host: <IP_ADDRESS>
  http_port: 9380
mysql:
  name: 'rag_flow'
  user: 'root'
  password: 'infini_rag_flow'
  host: '<IP_ADDRESS>'
  port: 3306
  max_connections: 100
  stale_timeout: 30
minio:
  user: 'rag_flow'
  password: 'infini_rag_flow'
  host: '<IP_ADDRESS>:9000'
es:
  hosts: 'http://<IP_ADDRESS>:9200'
  username: 'elastic'
  password: 'infini_rag_flow'
redis:
  db: 1
  password: 'infini_rag_flow'
  host: '<IP_ADDRESS>:6379'

How frequent does the issue happen? There could be some clue inside the system log on why the process be killed.

I removed all the environments and code, and re-pulled the code and configured the environment to run successfully.


3rd party use cases for consumption of your API and documentation requests

Howdy folks;
I'll have a few use cases for consuming your API in the immediate future.
a) automatically export backer email addresses into our projects marketing CRM (ActiveCampaign)
b) automatically do things when someone donates for the first time (via Zapier integration please)
c) when viewing a github issue (or pull-request) determine if someone is a financial supporter of the open-source project and display indicators to the maintainers as such. This enables scenarios such as

Thoughts? Please leverage something like https://zapier.com/developer/builder/ as writing glue code between opencollective and other services (such as twitter/github) to automate things is not compelling (time setting up and maintaining it would be best spent on maintaining the open-source project)

a) I fixed the API. See the wiki: https://github.com/opencollective/opencollective/wiki/API#get-list-of-collective-users-and-their-emails
b) yes we should totally support that. I'll look into Zapier.
c) would love to see that happening. We should ask people to connect their GitHub account to their backer account on open collective to support that. But imagine we have a simple call to verify the status of a GitHub user, what would you do on Github side?

c) I have an ongoing convo with Nadia @ Github about badges for users. This can be a nice step towards it.

I was thinking of writing a chrome plugin that queried the OC API when viewing GitHub issues. Would love to have features like this inlined into GitHub.

https://www.theverge.com/2017/10/25/16525586/patreon-api-app-directory-wordpress-zapier-slack-discourse

That includes a connection with automation service Zapier, which lets creators do things like instantly add new patrons to mailing lists.

This this this this 🚢

Any #hacktober contributor experienced with Zapier that would love to take on this?

c) when viewing a github issue (or pull-request) determine if someone is a financial supporter of the open-source project and display indicators to the maintainers as such. This enables scenarios such as

Someone could create a probot plugin for this: https://probot.github.io/
(It's an officially-unofficial github project, as far as a I can tell -- no big launch yet, but employees working on it.)
could prob do it by creating a small heroku app that uses this method to find emails (sample), then matching against some opencollective api endpoint that perhaps Boolean checks whether an email represents a backer of a project? (This would require a new endpoint, and I haven't thought through whether that endpoint would be open up new security/privacy concerns...)

Zapier and probot are the two most compelling use cases at the moment.
Personally I would use both as follows
a) probot used to automatically triage GitHub issues and add labels ("flare") to signify that someone is a financial sponsor.
b) Zapier to automatically add rows to a Google spreadsheet, publish tweets on Twitter, announce in Slack, send a personalized thank-you email.

Let's review the use cases and make sure it's covered in upcoming API.

I could not figure a clear expectation on how to move that issue forward.
Let's extract clear issues out of this and close it.

GitHub Actions beta is on. They're looking for people doing interesting things. Someone with OC could prob submit on behalf of the GitHub org, then tweet to bump up in the queue. Details:
https://blog.github.com/2018-10-17-action-demos/

I wanted to express that I'm still interested in seeing this (since stale bot has tagged it). I unfortunately don't think I'm up to adding this feature myself.

@nemchik there are multiple things mentioned here, what are you specifically interested in?

B and C from the original post. For B it'd be nice if it were more than just over event that could trigger some kind of action.

b) is being worked on, will be ready in a few days / weeks (let's stay in touch to try it first!)
c) might be solved by the https://github.com/opencollective/opencollective-bot

Thanks!
I've subscribed to the issues related to B. I had the bot setup on GitHub with permissions to my repo but did not make the yml file needed. I'm going to commit that now and see how it goes. If i have any trouble I'll open an issue on that repo for support.

The configuration of the Open Collective Bot should be easier in the future with https://github.com/opencollective/opencollective/issues/1876

I'm closing this issue. Some of the feature requests are covered by the Open Collective Bot and the Open Collective Zapier integration.
If something is not covered by that or if you want something more specific, please create a new issue!


Authorization header missing

Current getting error:
404 bad request due to no Authorization header.
Here is my request header:
Accept:application/json, text/javascript, */*; q=0.01
Accept-Encoding:gzip, deflate, sdch
Accept-Language:en-US,en;q=0.8,cs;q=0.6
Cache-Control:max-age=0
Connection:keep-alive
Host:localhost:8181
Origin:http://localhost:4200
Referer:http://localhost:4200/dashboard

Here is my env
    'simple-auth': {
      store               : 'simple-auth-session-store:local-storage',
      crossOriginWhitelist: [process.env.API_SERVER_URL],
      authorizer          : 'simple-auth-authorizer:token'
    },
    'simple-auth-token': {
      serverTokenEndpoint:        process.env.API_SERVER_URL + '/' + process.env.API_SERVER_VERSION + '/' + 'local/auth/',
      serverTokenRefreshEndpoint: process.env.API_SERVER_URL + '/' + process.env.API_SERVER_VERSION + '/' + 'local/auth/refresh/',

      refreshAccessTokens: true,
      timeFactor         : 1000,
      refreshLeeway      : 300 // Refresh the token 5 minutes (300s) before it expires.
    },

I should mention that's with simple-auth 0.8.0

False call. It was a version mismatch!


Update bank inforamtion for India

Dear Team,
Could you please help to update the public article for : https://support.office.com/en-us/article/Pay-for-your-Office-365-for-business-subscription-734f4aab-df2d-4e9b-8cb1-691910bde216?ui=en-US&rs=en-US&ad=US
When we select India as the country, there is no payment bank info displayed. Please help to update bank info for India customer for better customer service experience. Thanks very much.

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

ID: 682c512f-ad8a-2513-d7cd-87db2c0bd9fb
Version Independent ID: 72eacd6a-5b78-1802-3941-8d6419e907fa
Content: Payment information for India
Content Source: Office365-Admin/subscriptions-and-billing/pay/india.md
Service: o365-administration
GitHub Login: @cmcatee-MSFT
Microsoft Alias: cmcatee

@officedocsbot assign @mypil

@LydiaDong - Thank you for submitting feedback.
I will get this issue over to the O365ITPro writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

@LydiaDong - Thank you for submitting feedback.
I think the best way forward is if you can open a service ticket in your tenant so this can get resolved ASAP. Based on outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:  https://docs.microsoft.com/office365/admin/contact-support-for-business-products
If you don't mind please keep us posted here on the resolution and I really want to get this information you are discovering back into the docs.

@officedocsbot close

Dear Joyce and Team,
Thanks for your support.
This link https://docs.microsoft.com/office365/admin/contact-support-for-business-productshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Foffice365%2Fadmin%2Fcontact-support-for-business-products&data=02|01|v-lyddo%40microsoft.com|e6316dd7e175455a738e08d6f0e301a2|72f988bf86f141af91ab2d7cd011db47|1|0|636961255107088948&sdata=djGt6GAa5WjE1ZMCvdvL3DYPx%2FAQsm7%2B03h1R0fGoWU%3D&reserved=0 redirected to Officehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Foffice365%2Fadmin%2Fcontact-support-for-business-products&data=02|01|v-lyddo%40microsoft.com|e6316dd7e175455a738e08d6f0e301a2|72f988bf86f141af91ab2d7cd011db47|1|0|636961255107088948&sdata=djGt6GAa5WjE1ZMCvdvL3DYPx%2FAQsm7%2B03h1R0fGoWU%3D&reserved=0 365 support. Actually here we are Office 365 billing support. We escalate the request for our customer to get better experience for our public article. Could you please help on this request or help to involve related team? Really appreciate.
Best Regards
Lydia Dong
From: Joyce Y<EMAIL_ADDRESS>Sent: Saturday, June 15, 2019 12:12 AM
To: MicrosoftDocs/OfficeDocs-O365ITPro<EMAIL_ADDRESS>Cc: Lydia Dong (Wicresoft North America Ltd<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/OfficeDocs-O365ITPro] Update bank inforamtion for India (#562)
@LydiaDonghttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FLydiaDong&data=02|01|v-lyddo%40microsoft.com|e6316dd7e175455a738e08d6f0e301a2|72f988bf86f141af91ab2d7cd011db47|1|0|636961255107078952&sdata=o3XkqKZjZC5EG0%2F5f5pYcjG0FxvFsis805qmXFfZwQQ%3D&reserved=0 - Thank you for submitting feedback.
I think the best way forward is if you can open a service ticket in your tenant so this can get resolved ASAP. Based on outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products: https://docs.microsoft.com/office365/admin/contact-support-for-business-productshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Foffice365%2Fadmin%2Fcontact-support-for-business-products&data=02|01|v-lyddo%40microsoft.com|e6316dd7e175455a738e08d6f0e301a2|72f988bf86f141af91ab2d7cd011db47|1|0|636961255107088948&sdata=djGt6GAa5WjE1ZMCvdvL3DYPx%2FAQsm7%2B03h1R0fGoWU%3D&reserved=0
If you don't mind please keep us posted here on the resolution and I really want to get this information you are discovering back into the docs.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FOfficeDocs-O365ITPro%2Fissues%2F562%3Femail_source%3Dnotifications%26email_token%3DAMLIFY4IMMLPHRVYATVRNK3P2O7MLA5CNFSM4HYDIAV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXXH6PA%23issuecomment-502169404&data=02|01|v-lyddo%40microsoft.com|e6316dd7e175455a738e08d6f0e301a2|72f988bf86f141af91ab2d7cd011db47|1|0|636961255107088948&sdata=JvUMYWFFTTZd7laIwXc9JbRoJOeHoAxssgm6qwPRyaE%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAMLIFY2F3GO2CCAMXZ44RO3P2O7MLANCNFSM4HYDIAVQ&data=02|01|v-lyddo%40microsoft.com|e6316dd7e175455a738e08d6f0e301a2|72f988bf86f141af91ab2d7cd011db47|1|0|636961255107098940&sdata=E6rpZjIaba8lhjMiPPBGVNwgzN4SLygyBOpffKgdBZ4%3D&reserved=0.

@officedocsbot reopen

@LydiaDong - Thank you for this additional feedback. I reopened this issue and will get help from the doc owner.
@cmcatee-MSFT - Can you please advise on how to proceed to resolve this issue?
Thank you.

@mypil I am investigating.

@LydiaDong As I understand it, there are 2 types of invoices issued to customers in India, depending on what products they have. We don't want customers accidentally submitting payment to the wrong account, so we therefore do not include the addresses on this page, and instead instruct customers to send payment to the address listed on their invoice.


Quantization and extra args

Thanks to @DylanAlloy for original PR
Image ID you can use for testing: ghcr.io/substratusai/vllm:pr-5
Testing it out in K8s
Helm install:
helm repo add substratusai https://substratusai.github.io/helm
# Note by default the resource limit is set to 1 GPU
helm upgrade --install mistral-7b-awq substratusai/vllm  -f - << EOF
model: TheBloke/Mistral-7B-Instruct-v0.1-AWQ
image:
  tag: pr-5
env:
- name: QUANTIZATION
  value: awq
- name: DTYPE
  value: half
EOF

I am getting OOM error on a 24GB L4 GPU which makes no sense since it's only a 7B model. I can use vLLM with this model without quantization on the same GPU just fine:
  File "/usr/local/lib/python3.10/dist-packages/vllm/model_executor/layers/quantized_linear/awq.py", line 55, in apply_weights
    out = quantization_ops.awq_gemm(reshaped_x, self.qweight, self.scales,
torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 1.75 GiB (GPU 0; 21.99 GiB total capacity; 18.87 GiB already allocated; 1.64 GiB free; 19.56 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

It looks like vLLM quantization feature might not be mature enough yet. Were you able to get quantization to work? @DylanAlloy
Some others seem to struggle too: https://github.com/vllm-project/vllm/issues/1236#issuecomment-1743285360
I will poke around and see if I can change to another model or tweak params to get it to work.

Got it working. The missing parameter was max-model-len , when that's set too high it will run out of memory when using awq quantization. I had to decrease it to 4096.

Works great now that I'm able to test it. Thanks for such quick attention for something you don't use yourself.

Don't forget to star the repo and you're welcome on our discord if you have any issues or questions related to LLMs


Uses up lots of AppVeyor time

All rust-lang projects are sharing a single AppVeyor build slot, and Miri is regularly waiting for >1h to get one, so I am trying to figure out where all that time is going. ;)
It seems to me cargo is one offender here, because currently AppVeyor runs three times on every PR: it runs in the PR, then it runs on the auto  branch, and then when bors pushes the final thing to master it runs again. Is that really necessary? Do you think you could reduce this?
In Miri we decided to basically have AppVeyor only on the auto and try branches, that reduces it to once-per-PR. Maybe that is not appropriate for Cargo though since you want to see AppVeyor state in the PR itself; then maybe you could find a way to not also run AppVeyor when bors lands the change on master? That run is entirely redundant. (The travis config does branch != master OR type = pull_request to avoid that, but I don't know the AppVeyor equivalent of this.)

master builds are already skipped: https://github.com/rust-lang/cargo/blob/a3b46ff42fa9e4db34a142d9482e75f970a801fe/appveyor.yml#L7

Oh they are, sorry! I was looking for some kind of conditional in the control part of the file, similar to Travis.

I'm going to close this since I don't think there are any actionable changes we can make here.  Running the test suite on PRs is very useful, and Cargo is only using 1 job which runs for about 18 minutes, and our PR traffic is pretty low.  Although I'm always happy to find ways to cut down on CI time, I can't think of anything we can easily do for now.
I would recommend reviewing all the other projects, though.  For example, I notice clippy runs on master when it is probably not necessary (since they use bors).  They should probably add a manual exit the same way cargo's is defined.  There is no way AFAIK to completely skip the job, but it should only run for a few seconds.
I don't have a complete list of rust-lang-libs projects, it might be good to find out who all shares that account, and if they have multiple jobs if they can cut PR's down to 1 job like how rust-lang/rust worked (and have only the bors auto branch do all jobs).

I'm going to close this since I don't think there are any actionable changes we can make here. Running the test suite on PRs is very useful, and Cargo is only using 1 job which runs for about 18 minutes, and our PR traffic is pretty low. Although I'm always happy to find ways to cut down on CI time, I can't think of anything we can easily do for now.

Fair.

I would recommend reviewing all the other projects, though.

Yeah, I am on that. Unfortunately I cannot view the queue, so whenever Miri times out (because we didn't get a slot in 2h) I am stabbing in the dark to guess which project is causing the load.

it might be good to find out who all shares that account, and if they have multiple jobs if they can cut PR's down to 1 job like how rust-lang/rust worked (and have only the bors auto branch do all jobs).

Miri has two jobs, because it is very sensitive to 64bit vs 32bit differences, and I don't know how to cross-compile things on Windows. But it's auto- (and try-) only, not in PRs, which works for us.


Join multiple spans (grpc)

I created a small sample application to try out dapr and decided to go the grpc route, while it seems a bit more unpolished it works great, so great work and thank you!
The issue I have is that while all my spans correctly shows up in zipkin they always just show one interaction. Like from Client -> Server or Server -> Access and not the full call chain which is Client -> Server -> Access. However it shows up correctly in the dependencies tab.
So my question is, how to I join the spans together?

Seems you can propagate the traceparent header Dapr sends in through the reactor context (kotlin snip below, should be close to Java) - you'd put this on your middle service Server making the call to Access.
However! I haven't found a way to get this value  from a GRPC service endpoint, only through HTTP rest endpoint headers. 😞
This contextWrite impacts the lower level call of the SDK client regardless of HTTP or GRPC from the looks of it - I just can't find out how to get the original traceparent value from an incoming GRPC invocation...
        return cli.invokeMethod(
            "charlie",
            "api/pulse-charlie",
            "",
            HttpExtension.POST,
            null,
            Date::class.java
        ).contextWrite {
            it.put("Traceparent", hdr["Traceparent"].toString())
        }.awaitFirst()

I started to look into the http implementation today and it seems like that this issue also exists for http as you cant access the headers of the response.
How can I solve this? I am trying do do 1. in this artical:
https://docs.dapr.io/developing-applications/building-blocks/observability/w3c-tracing/w3c-tracing-overview/#you-need-to-propagate-or-generate-trace-context-between-services


Synced account address broken in hamburger (three dot) menu

Sanity checks (must complete)

[x] I have read and followed the installation instructions in the README
[x] I have not modified the userChrome.css file
[x] I have tested the latest release for my Firefox version, or commit on master/beta branch (beta is for Firefox Beta only) Note that I'm using Firefox Developer Edition

Describe the bug
First of all I'd like to thank you for making such an amazing project.
Bug:

Synced account name isn't completely displayed
"Sign in to sync" is shown (in menu bar) even when I'm already syncing. Clicking it does nothing

To Reproduce
Steps to reproduce the behavior:

Enable sync
Enable MaterialFox
Click on Three-dot menu
Sync account name is only half shown ----- (1)
Click the account name
'Sign in to sync' is still shown ----- (2)

Screenshots
Problem 1:

Problem 2:

I'm already syncing:

System info

OS: macOS 10.15.6
Firefox version: 80.0b2

Okay so both issues fixed themselves, idk how...

The issue is back. This seems weird

Are you sure this is a MaterialFox issue? Have you tried with the default theme? I can't reproduce on my end.

Are you sure this is a MaterialFox issue? Have you tried with the default theme? I can't reproduce on my end.

Ok so this seems like a Firefox Issue... I've noticed that in the first window opened, the synced address is okay, the second window it breaks. Closed as reproducible without MaterialFox

Fixed in 80.0b4


In Voice Over mode on macOS in chrome when timer is running clock updates are said every second

Environment details

Operation system: macOS 10.12.6
Extension version: Version (1.11.1)

Expected behavior
Toggle Timer extension not a make Voice Over impossible to use in macOS due to continually
vocalizing timers seconds update.
Actual behavior
Timer updates vocalized every second when in Voice Over mode in macOS
Steps to reproduce the behavior

start timer in chrome
Enable Voice Over on mac
navigate between tabs in chrome
Here voice over continually say "1 2" "2 3" "3 4" "4 5" "5 6" "6 7" and on and on and on.

Hey, @madhaze, thank you for reporting this. I followed the steps you provided but couldn't reproduce the problem. Could you please give us more information?

Hello @madhaze, thanks for finding it out. However, it seems like a Chrome issue, constantly reading updating tab title even if it's not active.
As a current workaround you can disable Show running time on the title bar in your profile preferences, you can read more here.
Please reach us out in case of any further questions.

I'm closing this issue based on the advice in the above comment, please let us know if there are further related problems.


User input

If it's possible to reduce the number of parameters the user has to enter, always do so. For example it's unnecessary to have the user enter "n", the number of data points, as that can be derived from the dimensions of X. If you think there might be edge cases where n != X.shape[0], consider setting n as a named parameter with sensible defaults and allowing the user to overwrite those defaults when necessary.

Thanks. This issue has been taken care of in the latest release.


NotImplementedError: auto_pad=VALID functionality not implemented.

When using latest onnx2pytorch.ConvertModel to import onnx model, it raise this error:
NotImplementedError: auto_pad=VALID functionality not implemented.

I'm having a similar issue with NotImplementedError: auto_pad=SAME_UPPER functionality not implemented

looks like from https://github.com/onnx/onnx/blob/master/docs/Operators.md that VALID is padding=0, and SAME_UPPER and SAME_LOWER are padding = ciel(kernel_size/2)

looks like from https://github.com/onnx/onnx/blob/master/docs/Operators.md that VALID is padding=0, and SAME_UPPER and SAME_LOWER are padding = ciel(kernel_size/2)

So, what can I do to solve this issue? I just have a model, no code. Must I wait the author to solve this issue?

Currently it's not implemented, so you're probably out of luck without code. But auto_pad is getting deprecated across ONNX, so if you're able to get access to a newer model, you'll be less likely to have this issue.

Adding to the issue, we can't do a SAME_UPPER too, and a manual workaround is to add a pass in the else clause, though that of course doesn't replicate the ONNX file into PyTorch 1:1, so that's a huge con.


fixnormalize column name before parseMvColumnToSql

Proposed changes
Issue Number: close #xxx

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

run buildall

run buildall

run buildall

run buildall

run buildall


build.zig not compiling with zig 0.14.0-dev.1820+ea527f7a8

Looks like the dev zig version has moved forward with build system changes which breaks the ziglua build:
C:\Users\janne\dev\ziglua>zig build test
C:\Users\janne\dev\ziglua\build.zig:81:14: error: no field or member function named 'addIncludeDir' in 'Build.Step.TranslateC'
    c_headers.addIncludeDir(b.getInstallPath(install_lib.h_dir.?, ""));
    ~~~~~~~~~^~~~~~~~~~~~~~
C:\Users\janne\scoop\apps\zig-dev\0.14.0-dev.1820\lib\std\Build\Step\TranslateC.zig:1:1: note: struct declared here
const std = @import("std");
^~~~~
C:\Users\janne\dev\ziglua\build.zig:81:14: note: method invocation only supports up to one level of implicit pointer dereferencing
C:\Users\janne\dev\ziglua\build.zig:81:14: note: use '.*' to dereference pointer

I tried to fix this by making the following change (blindly as I almost never know what I'm doing when I'm in build.zig):
diff --git a/build.zig b/build.zig
index 9313cf0..aa9678b 100644
--- a/build.zig
+++ b/build.zig
@@ -78,7 +78,7 @@ pub fn build(b: *Build) void {
         .target = target,
         .optimize = optimize,
     });
-    c_headers.addIncludeDir(b.getInstallPath(install_lib.h_dir.?, ""));
+    c_headers.addIncludePath(b.path(b.getInstallPath(install_lib.h_dir.?, "")));
     c_headers.step.dependOn(&install_lib.step);
     ziglua.addImport("c", c_headers.createModule());

But this doesn't build as apparently these paths are not supposed to be absolute:
C:\Users\janne\dev\ziglua>zig build test
thread 17512 panic: sub_path is expected to be relative to the build root, but was this absolute path: 'C:\Users\janne\dev\ziglua\zig-out\include'. It is best avoid absolute paths, but if you must, it is supported by LazyPath.cwd_relative
C:\Users\janne\scoop\apps\zig-dev\0.14.0-dev.1820\lib\std\Build.zig:1793:24: 0x2b7d5f in path (build.exe.obj)
        std.debug.panic("sub_path is expected to be relative to the build root, but was this absolute path: '{s}'. It is best avoid absolute paths, but if you must, it is supported by LazyPath.cwd_relative", .{
                       ^
C:\Users\janne\dev\ziglua\build.zig:81:36: 0x272d67 in build (build.exe.obj)
    c_headers.addIncludePath(b.path(b.getInstallPath(install_lib.h_dir.?, "")));
                                   ^
C:\Users\janne\scoop\apps\zig-dev\0.14.0-dev.1820\lib\std\Build.zig:2294:33: 0x25b455 in runBuild__anon_4739 (build.exe.obj)
        .void => build_zig.build(b),
                                ^
C:\Users\janne\scoop\apps\zig-dev\0.14.0-dev.1820\lib\compiler\build_runner.zig:335:29: 0x25653e in main (build.exe.obj)
        try builder.runBuild(root);
                            ^
C:\Users\janne\scoop\apps\zig-dev\0.14.0-dev.1820\lib\std\start.zig:439:53: 0x25dbc7 in WinStartup (build.exe.obj)
    std.os.windows.ntdll.RtlExitUserProcess(callMain());
                                                    ^
???:?:?: 0x7ffb73227373 in ??? (KERNEL32.DLL)
???:?:?: 0x7ffb7515cc90 in ??? (ntdll.dll)
error: unable to read results of configure phase from 'C:\Users\janne\dev\ziglua\.zig-cache\tmp\b0dbf6324abaeb01': FileNotFound

Hmm. I wonder if there is a more correct way to write the line and Zig 0.14.0 is just being more strict about correctness.. I'll look into this!
c_headers.addIncludeDir(b.getInstallPath(install_lib.h_dir.?, ""));

I was able to get the interpreter example building with this:
C:\Users\janne\dev\ziglua>git diff
diff --git a/build.zig b/build.zig
index 9313cf0..f0558e2 100644
--- a/build.zig
+++ b/build.zig
@@ -78,7 +78,7 @@ pub fn build(b: *Build) void {
         .target = target,
         .optimize = optimize,
     });
-    c_headers.addIncludeDir(b.getInstallPath(install_lib.h_dir.?, ""));
+    c_headers.addIncludePath(lib.getEmittedIncludeTree());
     c_headers.step.dependOn(&install_lib.step);
     ziglua.addImport("c", c_headers.createModule());

it builds and runs.
But running zig build test leads to a lot of errors (which I imagine are not related to my change):
C:\Users\janne\dev\ziglua>zig build test
test
└─ run test
   └─ zig test Debug native 21 errors
src\define.zig:101:34: error: no field named 'Struct' in union 'builtin.Type'
        inline for (@typeInfo(T).Struct.fields) |field| {
                                 ^~~~~~
C:\Users\janne\scoop\apps\zig-dev\0.14.0-dev.1820\lib\std\builtin.zig:268:18: note: union declared here
pub const Type = union(enum) {
                 ^~~~~
src\lib.zig:3055:14: error: no field named 'Pointer' in enum '@typeInfo(builtin.Type).@"union".tag_type.?'
            .Pointer => |info| {
            ~^~~~~~~
C:\Users\janne\scoop\apps\zig-dev\0.14.0-dev.1820\lib\std\builtin.zig:268:18: note: enum declared here
pub const Type = union(enum) {
                 ^~~~~
src\lib.zig:3055:14: error: no field named 'Pointer' in enum '@typeInfo(builtin.Type).@"union".tag_type.?'
            .Pointer => |info| {
            ~^~~~~~~


langchain: add Jina Reranker in retrievers module

Description: Adapt JinaEmbeddings to run with the new Jina AI Rerank API

Twitter handle: https://twitter.com/JinaAI_

[ ] Add tests and docs: If you're adding a new integration, please include

a test for the integration, preferably unit tests that do not rely on network access,
an example notebook showing its use. It lives in docs/docs/integrations directory.

[ ] Lint and test: Run make format, make lint and make test from the root of the package(s) you've modified. See contribution guidelines for more: https://python.langchain.com/docs/contributing/

@JoanFM feel free to ping when it's ready for review!

@JoanFM feel free to ping when it's ready for review!

Hey @baskaryan ,
This is now ready for review.

Hey team, is there any timeline for the review of this PR?
FYI @baskaryan

Hey @baskaryan ,
is there anything missing for this to be merged?


sometimes loss is negative during training model

I use Tensorflow to do the work.And my loss:
self.g_loss = -tf.reduce_mean(d_logits_fake)
self.d_loss = tf.reduce_mean(d_logits_fake) - tf.reduce_mean(d_logits_real)+GP
but during traning,I find sometimes g_loss is negative.I don't know why i will get it.Who will explain it

I think that's when training generator, we want it produce fake data to fool the discriminator, so the d_logits_fake will be quite large like the real data. So d_logits_fake will be a positive val.
when training discriminator, d_logits_fake is quite small instead. That's mean it is  a negtive value


Spawn in launcher

#23 has the concept of a per-provider TsunamiBuilder. It turns out this is not very useful and in fact sometimes annoying to have to create multiple TsunamiBuilders. Get rid of it, and replace with a default implementation of spawn on the Launcher trait.
The methods on TsunamiBuilder become crate-level helper methods.

cc @znewman01

IMO great change, thanks :)
will take a look probably tomorrow

Thanks for the comments!

I also have one more nitpick, which is that I find the terminate-resources-in-Drop to be a bit unwieldy, especially once there's async stuff involved (the implementation is kind of a hack, and there's no better way to do this right now.

Yeah, it definitely sucks, and when writing #31 today I didn't really have a good solution for it. The closure idea (or, in async tsunami, a closure that returns a future) might work, but I agree I dislike the nested calls to connect_all.

Closing to fold into #23.


Nesting check that requires all children to fail

This pull request proposes a way of overriding a set of child checks with a parent check if all the children fail.
The proposed interface to this is a ValidatorChain named all, which accepts a list of ValidatorChains and checks that they all failed. If they all did it removes them from the requests _validationErrors list and adds it's own error in their place.
Example:
req.check('name', 'Please enter your name').all([
    req.check('firstName', 'Enter your first name').notEmpty(),
    req.check('lastName', 'Enter your last name').notEmpty()
]);

Background
This comes from the approach to errors that I've previously used on GDS services where we want to provide error messages that are as human readable as possible. For example:

Providing no input at all gives: Please enter your date of birth

Only filling in some of the fields gives errors for each field

This is a simple example but they can get quite complex. The aim is always to allow the message to the user to be as simple and understandable as possible.

Uncertain who's lead maintainer so poking owner and most active: @ctavan @rustybailey.

Sorry, been super busy. Thanks for adding tests, but since this is a new feature, can you also add documentation to the README?

Also, can you fix the style issues that are causing your build to fail?

@michaeldfallen, do you care to finish this PR by adding what Rusty commented?

Yes, will do. Sorry, been busy here myself and we ended up using a different approach for the checks we wanted this feature to help with.
As long as you find it useful (since I won't be using it day to day) I'll finish this up and get it clean.
Closing for now until I get time to fix this.


How to use Aggregated trades

There is a problem I am facing with. When I use aggTrades without fromid and only with start and end time it's gives only 4 minutes before now and right now so its not working well and when I use aggTrade with fromid it gives wrong time let's say my trade id is wrong how am I gonna get trade id from the time I want and can u guys give me an working example for this function

Here's example using the Binance REST API directly. But the logic is the same when you swap in this library.
https://github.com/tgcandido/binance-data-fetcher/blob/master/binance_trades_fetcher.py


fix(build): correct deployment errors

PR summary
A fix to the build to correct problems with our maven central deployment.
PR Checklist
Please make sure that your PR fulfills the following requirements:

[ ] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Current vs new behavior

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

:tada: This PR is included in version 0.18.10 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Date filter slection in history is not working as expected from the label

Describe the bug
The history filtering is not working as expected. Say I ran several commands yesterday, then ran clear, closed Wave, shut down machine. I opened Wave today again, and ran some more commands (in same workspace, in same tab). Now when I go the history section, there is a filter named 'From'. I expect that I get list of all commands that I ran from the selected date. However it seems that the commands I get are upto the selected date.
To Reproduce

create a workspace + tab (or use existing)
run some commands, run clear (not sure if clearing plays a part or not, but I did run it), close wave and do something else for rest of the day
the next day run some more commands in the same workspace+tab.
go to history section, and select current day in the calendar selection.
similarly select the previous day in calendar selection, and several days back in the selection

Expected behavior

When selected current date, I expect to get commands run only today ; however I also see commands I ran ysterday
When selected yesterday's date, I see only the command I ran yesterday, but I expect to see those and today's commands
When selected a date before I installed Wave, I see 'No history Item found' ; I expect to see all commands I ran till now

Desktop (please complete the following information):

OS: Linux x64 Pop OS
Version v0.5.1 20231206-015352

Additional context
It seems that the 'From' in date selection is actually used as 'Upto' internally? It ok if that is the intention, but the label seems to create confusion.

Thanks for submitting.  I understand your point about the label.  The history search UI needs to be improved and at that point we'll probably add a real date range picker.  My thought when building that control was that since the commands are sorted in reverse time order, the only date filter that makes sense would be to start searching "from" a date in the past.


QuantumLeap API Query doesn't find records

We have Timeseries data of NGSI Devices from Dec. 2019 to now stored in CrateDB. So when I query Crate
select * from etdevice
where entity_id = 'urn:ngsi-ld:Device:evn-joulie:FFFF644214006784:64002D0204BC'
  and time_index between '2020-04-04T12:00:00' and '2020-04-04T13:00:00'
order by time_index desc;

I get 13 records (5 Minute steps) for the given timespan. But when I query QuantumLeap with the same parameters
/v2/entities/urn:ngsi-ld:Device:evn-joulie:FFFF644214006784:64002D0204BC/attrs/value?fromDate=2020-04-04T12:00:00&toDate=2020-04-04T13:00:00
I get
{
"description": "No records were found for such query.",
"error": "Not Found"
}

When I just change the dates to 2019-12-xx, I get 13 records either on CrateDB directly but also from QuantumLeap. The datasets certainly only differ in time_index and value, other fields have the same content in every row.
Versions:
CrateDB: 4.1.7
QL: 0.8.0
P.S. Although we use 'ngsi-ld' in our ID fields, this data isn't a NGSI-LD model...we are just preparing to fit the standard as much as possible.

@pinkerltm thanks for reporting this, we'll look into it.

use 'ngsi-ld' in our ID fields

it should make no difference to QL how you choose your entity ID values, any string should do. so it's most likely a genuine bug we've bumped into :-)

@pinkerltm can you try without from and to? i suspect it may be related to that and last changes we made

@pinkerltm can you try without from and to? i suspect it may be related to that and last changes we made

That works, but it only delivers values until a certain limit is reached (which is all in 2019, which also works with from and to anyway).

until a certain limit is reached

results sets get capped at 10,000 records unless you explicitly specify a limit query param, in which case the least of the two is used. #319/#330 made the result set cap configurable to something else than 10,000.

@pinkerltm we are going to check generated queries, something is clearly broken.

until a certain limit is reached

results sets get capped at 10,000 records unless you explicitly specify a limit query param, in which case the least of the two is used. #319/#330 made the result set cap configurable to something else than 10,000.

Sorry, I wasn't clear... it returns only 534 rows although the database table holds 62.524 for this specific device.

@pinkerltm can you export data and ship them to us?

@chicco785 mhm...only via another channel as I am not allowed to publish it here. I'll send you a message where to get the timeseries of this device as CSV File, ok?

@chicco785 Did you receive my testdata file and were you able to import it?

@pinkerltm we got the data, but we hadn't time to work on it yet

@chicco785 are there any news on this issue? We (@pinkerltm) using QL API for a big data energy project and rely on that functionality to work. Can we somehow help?

@mclucky @pinkerltm it's a bad timing, with holidays and projects for customers, not sure when we will have time to luck into it

@chicco785 I´ve spinned up the code locally, debugging the generated SQL Statement to sort things out. What I didn't know yet: QL inserts and fiware_servicepath = '' to the WHERE clause, so you expect the fiware_servicepath to be an empty string.
Our data rows whic aren#t returned have all fiware_servicepath set to NULL, as we don't use the servicepath and data got restored from a CSV File, so fiware-servicepath was never set.
To understand this and prevent future issues: Is this an NGSI requirement that fiware_servicepath is NOT NULL? (and therefore have to be set to an empty string)?
If we know how to prevent this, I think we are able to close this issue immediately

ql should use the fiware_servicepath of the orginal entity in orion, is any path set in orion? and do you use any header? it could be that the query is not correct when fiware_servicepath is not set in the header

Ok, thanks for clarifying this... as said, these rows were restored from CSV manually and therefore fiware_servicepath was not set. So it was my fault and I was able to solve this now by setting fiware_servicepath via an UPDATE Statement.

@pinkerltm great!


Modify RunONNXModel.py for pip install

The goal is to create a python package so that we can run an inference like onnxruntime in python script. Since the RunONNXModel.py provides most of the necessary functionality, I added extra support in that script. The pip package related code will be checked later. I tested with a local pip install. We can upload the package later.
Major changes:

Add the function interface to call the main in RunONNXModel.py with the parameters similar to onnxruntime:
Accept list of arrays as input.
Return the outputs

Example:
import numpy as np
import onnxmlirrun

a = np.zeros((3,4,5), dtype=np.float32)
b = a + 4
r = onnxmlirrun.RunONNXModel.onnxmlirrun(compiled_so="/Users/chentong/Projects/onnx-mlir/build/test_add.so", inputs=[a,b])
print(r)
r = onnxmlirrun.RunONNXModel.onnxmlirrun(onnx_model="/Users/chentong/Projects/onnx-mlir/build/test_add.onnx", inputs=[a,b])
print(r)

We may have different name for the functions and the package. Comments are welcome.

@chentong319 Can you add a reference to the ORT interface that this PR is imitating.
In your first example
r = onnxmlirrun.RunONNXModel.onnxmlirrun(compiled_so="/Users/chentong/Projects/onnx-mlir/build/test_add.so", inputs=[a,b])

was the  onnx_model parameter omitted on purpose? Just for me to understand better the interface.

@chentong319 Can you add a reference to the ORT interface that this PR is imitating.
In your first example
r = onnxmlirrun.RunONNXModel.onnxmlirrun(compiled_so="/Users/chentong/Projects/onnx-mlir/build/test_add.so", inputs=[a,b])

was the onnx_model parameter omitted on purpose? Just for me to understand better the interface.

Yes, onnx_model is not needed if the compiled .so is provided. By the way, the two onnxmilrrun.RunONNXModel.onnxmlirrun commands are two independent ways to do inference.

Got it, so we can provide a onnx file or a pre-compiled binary, smart, thanks.

I changed the init.py for the package to simplify onnxmlirrun.RunONNXModel.onnxmlirrun to onnxmlir.onnxmlir.
I am thinking to add the code for package into onnx-mlir in future.

Can you copy/link a description of the ORT runtime interface and update ours with the new names, so that we can see how they relate? Thanks

The Ort code is like
import onnxruntime as ort
import numpy as np
x, y = test_data[0][0], test_data[0][1]
ort_sess = ort.InferenceSession('fashion_mnist_model.onnx')
outputs = ort_sess.run(None, {'input': x.numpy()})

Since the current RunModelPython does not separate the session creation and run, our code will be like:
import onnxmlir
import numpy as np
x, y = test_data[0][0], test_data[0][1]
outputs = onnxmlir.run(onnx_model='fashion_mnist_model.onnx', inputs = [a, b])

It is doable to totally mimic ORT in separating the session creation and run, and using dictionary for input.

Ideally
import onnxmlir as ort
import numpy as np
x, y = test_data[0][0], test_data[0][1]
ort_sess = ort.InferenceSession('fashion_mnist_model.onnx')
outputs = ort_sess.run(None, {'input': x.numpy()})

with additional parameters ok
import onnxruntime as ort
import numpy as np
x, y = test_data[0][0], test_data[0][1]
ort_sess = ort.InferenceSession('fashion_mnist_model.onnx', compile_flags="-O3")
outputs = ort_sess.run(None, {'input': x.numpy()})

reusing flags that already exists, if any.

Check error message if file does not exist.

onnx-mlir will report the error if file does not exist. Leave this to the TODO list to catch all the errors gracefully.

@jenkins-droid test it!

@jenkins-droid test this please

Jenkins Linux ppc64le Build #14467 [push] Modify RunONNXModel.py f... started at 11:56

Jenkins Linux amd64 Build #15437 [push] Modify RunONNXModel.py f... started at 10:46

Jenkins Linux s390x Build #15444 [push] Modify RunONNXModel.py f... started at 11:46

Jenkins Linux amd64 Build #15437 [push] Modify RunONNXModel.py f... passed after 1 hr 7 min

Jenkins Linux s390x Build #15444 [push] Modify RunONNXModel.py f... passed after 1 hr 29 min

Jenkins Linux ppc64le Build #14467 [push] Modify RunONNXModel.py f... passed after 2 hr 1 min


"SLOBOZIA" is listed in the drop-down menu

Description:
When user writes "SLOBOZIA" the location appears in the drop-down menu next to the comune and the county which is a part according to the law
Precondition:
The website is up an running.

Step 1
Write in the search bar "SLOBOZIA"

Expected results
The location appears in the drop-down menu as "Municipiul SLOBOZIA, IALOMIȚA "


is there a way to determine the previous route's name?

is there a way to determine the previous route's name?
 i know that

this.router.history.previousLocation
 gives me the url for the previous route. means the one you go to when you do

this.router.navigateBack();
 but i need to know the routes name, the one o would need for doing:

this.router.navigateToRoute(previouseRouteName);
 any ideas?is there a way to determine the previus route's name?

 i know that

this.router.history.previousLocation
 gives me the url for the previous route. means the one you go to when you do

this.router.navigateBack();
 but i need to know the routes name?  --  the one o would need for doing:

this.router.navigateToRoute(previouseRouteName);

Try router.currentInstruction. You should be able to get the information from that.

Hi, I have tried the suggestion and cannot find a way to get the previous route name or path.
I am using<EMAIL_ADDRESS>has this been fixed in the latest release? I am not quite ready to update everything yet.
This is what i have found:
this.theRouter.currentInstruction.previousInstruction is always null
this.theRouter.currentInstruction.parentInstruction is always null
i also tried:
this.theRouter.history.previousLocation
however the location returned is not the previous one, its the current one.
There really needs to be a property for the previous route name rather than the location so that the router can be controlled as it uses route names not locations.
Despite the previousInstruction and previousLocation not being available, the function this.theRouter.navigateBack() does work, i assume this is because its just controlling the browser back button?
The use case for needing the previous location is to allow the navigateBack() to be conditional depending on the previous route name. Also i need to account for when there is no previous route, if a page was launched directly from a link, the  navigateBack() function does nothing, so i need to know if this is the case and behave accordingly.
Btw. I am using the correct methods for navigating, either this.theRouter.navigateToRoute( or in the html using route-href
Thanks!

@bryanrsmith This is related to the other issue I tagged you on this morning. I believe we have/had a way to do this. Can you investigate? There may have been a regression here.

7fa6e0b97370f08a4e3b1c08bad2ea1eded330f4 removed the previousInstruction at the end of the activation cycle to plug a memory leak. You will need to read the name of the previous route during activation and store it for later use. I believe this happened around version 0.13. I'm not sure why it would have worked with any beta version.


Can't working on OSX

dnscrypt-proxy 1.6.0

Try https://github.com/alterstep/dnscrypt-osxclient !


Emoji's color gets inverted in the dark mode

Reproduction
git clone https://github.com/yihui/hugo-paged.git
cd hugo-paged
git reset --hard cc8e306
echo '🙈' > exampleSite/content/_index.md # see-no-evil \u{1F648}
hugo server --source exampleSite --themesDir ../..

Dark mode (ugly):

Light mode (normal):

Emojis can't be selected via CSS selectors, so I'm afraid I can't do anything about them.

filter: invert(1) is hacky. If you are content with current colors in the dark mode, maybe you can use color picker software to get color values.

It is a hack indeed. And yes, I can get the color values for the dark mode, but I don't have time or interest...

I can get the color values for the dark mode, but I don't have time or interest...

Me neither. I'll let you decide whether to close this issue.


All the Litmus tests from readme return 404

Thanks for reporting this @v-karbovnichy. All the Litmus links are fixed now.


Need to update TodoTextInput.js to avoid blank screen at Step 5

This has been documented at https://github.com/FountainJS/generator-fountain-webapp/issues/190
The single quotes around the word edit in the rendering section (line 40 in Atom text editor) need to be removed in order to comply with new formatting requirements.

@zckrs any chance you want to update this part of the codelab?

It has still this problem. Redux TodoMVC does not have it.
I can not find yo use which repository that has this problem.
By the way, to solve it simply remove quotes of 'edit' in line 40 of ./src/app/components/TodoTextInput.js.

Thank you, editing 'edit'  worked for me.


startpos switcher + smart startpos implementation

the code is kinda bad but i tried

Man you're just crazy. Big respect to you


Tweego binary does not have "executable" attribute in the release ZIP

$ wget -O tweego.zip https://github.com/tmedwards/tweego/releases/download/v2.1.1/tweego-2.1.1-linux-x64.zip
$ unzip tweego.zip
$ ls -l                                                                                 ✔  6698  14:45:38 
total 7732
-rw-r--r--  1 username username    1341 Feb 24  2020 LICENSE
drwxr-xr-x  2 username username     140 Dec 23  2019 licenses
drwxr-xr-x 11 username username     220 Dec 23  2019 storyformats
-rw-r--r--  1 username username 4506940 Feb 25  2020 tweego
-rw-r--r--  1 username username 3107046 Feb 25  2020 tweego.zip

Have to apply chmod +x tweego

In general this is a limitation of the zip format as it stores extremely limited attributes about the original file (as it was originally only designed for MS-DOS file attributes). That said some zip programs allow storing some POSIX/Unix-style permissions in the optional extended attributes that can be saved in zip archives, however, how to set these and how compatible they are across zip clients is undefined (and so may not be compatible across different zip clients).
I am assuming @ezsh is using infozip. Using infozip one can query these extended attributes via unzip -Z tweego.zip (-Z causes infozip unzip to act like infozip zipinfo). This can be limited to specify files within the archive. As an example (and to keep from spamming lots of data here), executing unzip -Z tweego.zip tweego to examine these extended attributes for only the tweego executable within the released zip archive, I get:
-rw-a--     6.3 fat  4506940 bx defN 20-Feb-25 01:09 tweego

It does not appear as if the released archives are storing such extended attribute information--or at least not in a format that infozip seems to understand.
If I use the infozip tools to unzip the archive, set the executable permission bit on the executable file and rezip the files, effectively recreating the zip archive, and then execute the same command, I now get:
-rwxr-xr-x  3.0 unx  4506940 bx defN 20-Feb-25 01:09 tweego

For reference if it matters, I am using infozip tools from/on Ubuntu focal in WSL2 on Windows 10.
If this is a concern for the tweego release process, I recommend switching the POSIX/Unix binary releases to different archive formats (e.g., a compressed tar archive, etc.).


API missplaced keys [BUG]

API data bug
When you hit the endpoint /cases/deaths
"age" and "gender" keys do not have the
corresponding values.
"age" has a string value that belong to "gender"
and "gender" has a integer value that belong to "age"
To Reproduce
Steps to reproduce the behavior:

Go to to this end piont
Check each object of the JSON file
You will see the error

Expected behavior
A clear and concise description of what you expected to happen.
Swipe the key values of gender and age

@Vutlhari please check

I will take a look 😊

This issue was fixed in a previous PR, it was just not deployed for some reason. I have deployed the latest changes. I have tested the deployed version and the mapping now seems alright. Please verify @henkhodryza


SARIF result not uploaded

Hi,
I'm trying to setup a basic code analysis in my CI on my project.
I setup CodeQL for Ubuntu, worked first time.
I tried to set up the same for Windows using this :
https://devblogs.microsoft.com/cppblog/microsoft-cpp-code-analysis-with-github-actions/
The workflow started and completed fine, except this :

The result of the MSCV check is not displayed in the Code Scanning Alert in the Security tab of my project, the Ubuntu one is.
I can get the SARIF file as an artifact, just not get it to display properly on the project page.
How can I solve this ?
Thanks in advance
The yml code is the basic one :
name: Microsoft C++ Code Analysis

on:
  push:
    branches: [ main, dev, Basic_Protections ]
  pull_request:
    branches: [ main ]

env:
  # Path to the CMake build directory.
  build: '${{ github.workspace }}/build'

permissions:
  contents: read

jobs:
  analyze:
    permissions:
      contents: read # for actions/checkout to fetch code
      security-events: write # for github/codeql-action/upload-sarif to upload SARIF results
    name: Analyze
    runs-on: windows-latest

    steps:
      - name: Checkout repository
        uses: actions/checkout@v3

      - name: Configure CMake
        run: cmake -B ${{ env.build }}

      # Build is not required unless generated source files are used
      # - name: Build CMake
      #   run: cmake --build ${{ env.build }}

      - name: Initialize MSVC Code Analysis
        uses: microsoft/msvc-code-analysis-action@04825f6d9e00f87422d6bf04e1a38b1f3ed60d99
        # Provide a unique ID to access the sarif output path
        id: run-analysis
        with:
          cmakeBuildDirectory: ${{ env.build }}
          # Ruleset file that will determine what checks will be run
          ruleset: NativeRecommendedRules.ruleset

      # Upload SARIF file to GitHub Code Scanning Alerts
      - name: Upload SARIF to GitHub
        uses: github/codeql-action/upload-sarif@v2
        with:
          sarif_file: ${{ steps.run-analysis.outputs.sarif }}

      # Upload SARIF file as an Artifact to download and view
      # - name: Upload SARIF as an Artifact
      #   uses: actions/upload-artifact@v3
      #   with:
      #     name: sarif-file
      #     path: ${{ steps.run-analysis.outputs.sarif }}

The problem happens ath the Upload

Thank you for raising this issue. The upload is failing because the file includes too many SARIF runs. Looking at how the microsoft/msvc-code-analysis-action is implemented, each compile target is converted into a single run.
I can think of two workarounds:

Slice your analysis into multiple runs, each with a smaller number of targets. In this case you need to specify a different category for each slice.
Post process the SARIF file and combine multiple runs into a single run.

Forgive my ignorance, I just started using yml CI's, but how do I slice the analysis ? I do not see any args in the msvc-code-analysis repo to do so.
I just need to concatenate the data on the SARIF file and upload it then ?

Honestly, I'm not 100% sure either. But it would be something like this. Since this is cmake, instead of calling the root target, call each of its child targets separately and upload separately. However, I don't know for certain if this would work since I'm not sure how your make file is split up.
And now that I'm thinking about it, merging SARIF runs won't be straight forward either. Let me chat with the code scanning team to see if there are any suggestions they have.
I see that your repo just had a successful upload. Did something change there? And is it working now?

Sadly no :
The workflow always runs and completes, it's the upload that's unsuccessful. The errors are mostly me messing around trying to get the upload to work, and show up.
As previously stated, the analysis completes and I can get the file via artifact, just not to display :'(

Only the ubuntu run is displayed even with the success

I've asked the code scanning team for some help with this.

There is no easy way around the limitation of 15 runs per upload. After some discussion, the best we can suggest is that you work with the maintainers of microsoft/msvc-code-analysis-action and create fewer runs. Since all of these runs were created through a single invocation of the tool, conceptually, they should be all combined into a single run. However, I do not know enough about the tool to suggest how to do this.

Thanks, I will write an issue tomorrow on their repo

Did you file an issue?
Fwiw, you can probably just hack through things like this using jq. It isn't terribly painful to have it merge objects together.

Hello,
I did not file an issue as I fell ill while working on this and forgot. I don't think I plan on continuing this project, but for curiosity's sake, what is that jq that you mention ?

No worries. I hope you're recovering/recovered.
https://stedolan.github.io/jq/

sarif-multitool can be used for doing that.  It has an undocumented "--merge-runs" argument.


Feedback on Tutorial: Configure SAP HANA, express edition Security

Please update the tutorial:

The instruction in section "Change the SSFS Master Keys" for step 4. "Re-encrypt the system PKI SSFS" is incorrect. the variables DATAPATH and KEYPATH must be set to the proper directory - as stated in the PDF tutorial, but there the "export" statement is missing.
export RSEC_SSFS_DATAPATH=/usr/sap/HXE/SYS/global/security/rsecssfs/data
export RSEC_SSFS_KEYPATH=/usr/sap/HXE/SYS/global/security/rsecssfs/key

Please do not change the Root Keys for HANA Express VMs with DB+Apps (hxexsa).
"You should NOT generate a new root key for the DPAPI service as there is a PEM certificate in the database encrypted with the current key",
Change the rootkey only if following views are empty in the SystemDB: "SYS"."P_DPAPI_KEY_" and "PUBLIC"."CREDENTIALS".
Explanation is here: https://answers.sap.com/questions/27365/uaa-security-is-not-up-on-hana-express.html

Thanks for your feedback. This tutorial was updated with the release of SAP HANA, express edition 2.0:

Configure SAP HANA 2.0, express edition Security


Updated formatting of features in readme

🚀 🚀 Pull Request
Checklist:

[ ]  My code follows the style guidelines of this project and the Contributing document
[ ]  I have commented my code, particularly in hard-to-understand areas
[ ]  I have kept the coverage-rate up
[ ]  I have performed a self-review of my own code and resolved any problems
[ ]  I have checked to ensure there aren't any other open Pull Requests for the same change
[ ]  I have described and made corresponding changes to the relevant documentation
[ ]  New and existing unit tests pass locally with my changes

Changes

agree with the problem, but solution uses the space sub-optimally

[ ] instead of using ### Storage Agnostic maybe give a try just Storage Agnostic or #### ...
[ ] Since you have single bullet point then no longer need using bullet point under section, maybe instead you can set the bullet to the * Storage Agnostic

also maybe worth leaving the feature text after the visualization, it would give discount on the full blown text content to be shown. (unless there is no way to shorten the text or the list of features)

agree with the problem, but solution uses the space sub-optimally

[ ]  instead of using ### Storage Agnostic maybe give a try just Storage Agnostic or #### ...
[ ]  Since you have single bullet point then no longer need using bullet point under section, maybe instead you can set the bullet to the * Storage Agnostic

@davidbuniat I tried both. #### is too small. It looks ok, but when you skim, the features just don't pop out. The single-bullet also has the same problem. That's how it was in the old readme, and the feature does not stand out enough when it's immediately followed by text.
Do you think the formatting in the old readme is an issues at all? Maybe I'm trying to solve a problem that doesn't exist.

also maybe worth leaving the feature text after the visualization, it would give discount on the full blown text content to be shown. (unless there is no way to shorten the text or the list of features)

@davidbuniat I don't love this, but am down to try. @mikayelh what do you think?


RDS issue

need a minor change for RDS:

2023-10-25 15:16:56 UTC+1000
DatabaseCluster
CREATE_FAILED
Resource handler returned message: "Cannot find version 5.7.mysql_aurora.2.10.1 for aurora-mysql (Service: Rds, Status Code: 400, Request ID: c5356dd6-cdfd-488a-81f2-e18049469773)" (RequestToken: 8a2a2636-49ed-3918-edc7-c0126f52150d, HandlerErrorCode: InvalidRequest)

I'm seeing the same issue.  Can the stack RDS property "EngineVersion: 5.7.mysql_aurora.2.10.1" be presented as a template parameter?


Fixed #11 by saving the names to db

Saved the Names from the fetchNames function on !explore
Proof:

Ensured, that the case insensitivity still works


Dutch translations

Thanks for contributing to Postleaf! 💙💚
Contributor License Agreement
We want Postleaf to remain free and open for everyone to use forever, so please don’t submit any code or content that is licensed or copyrighted without first getting written consent.
Before we accept your pull request, we ask that you agree to the following terms to ensure there are no conflicts with Postleaf’s license now or in the future.

I certify that the code I am contributing to this project is completely original or free of any known license or copyright.

I agree to allow the developer of Postleaf, A Beautiful Site, LLC, license or relicense my work at their discretion. By submitting this pull request, I agree to forfeit all copyrights, intellectual property rights, and royalties pertaining to my contribution.

By entering my name in the space provided below, I agree to all of the terms mentioned above.

Signed: KevIT
Date: 16-08-2016

@claviska Made a new pull request with the new Dutch translations.

Thanks. I fixed a missing quote on line 157 so make sure you pull the latest.


Corrected pgoapi reference in requirements.txt

Short Description:
https://github.com/PokemonGoF/PokemonGo-Bot/commit/730534daf96c9b9bbf7989ef5b39da080b5c0839#diff-b4ef698db8ca845e5845c4618278f29aL1
Introduced a syntax error in requirements.txt which would yield the error:
# pip install -r requirements.txt
Invalid requirement: 'pip install'
Traceback (most recent call last):
  File "/root/pokemon/GoF-PGo/local/lib/python2.7/site-packages/pip/req/req_install.py", line 78, in __init__
    req = Requirement(req)
  File "/root/pokemon/GoF-PGo/local/lib/python2.7/site-packages/pip/_vendor/packaging/requirements.py", line 96, in __init__
    requirement_string[e.loc:e.loc + 8]))
InvalidRequirement: Invalid requirement, parse error at "u'install'"

Fixes:

Havn't made an issue for it.

my bad for the previous commit merge.


Grok Pattern for European Timestamp

Hi,
i want to mention that if you have an European Timestamp like "6.11.2015 13:01:29 1274 PACKET...." the GROK-Pattern wont match. You have to Change WINDNS_TIME to "(?:0?[1-9]|1[0-2]).-.-{1,2}\s(?!<[0-9])(?:2[0123]|[01]?[0-9]):(?:[0-5][0-9]):(?:(?:[0-5]?[0-9]|60)(?:[:.,][0-9]+)?)" (without Quotes).
And don't forget to add the Static Field: "SourceModuleName: dns" to the DNS-Server GELF-Input if you dont use the Content-Pack generated input.
Kind Regards,
Rene

Can you please try updating the WinDNS_Time GROK pattern to the following and let me know if it continues to match the EU Timestamp?
WINDNS_TIME:
((?:0?[1-9]|1[0-2])/-/-{1,2}\s(?!<[0-9])(?:2[0123]|[01]?[0-9]):(?:[0-5][0-9]):(?:(?:[0-5]?[0-9]|60)(?:[:.,][0-9]+)?)\s(A|P)M|(?:0?[1-9]|1[0-2]).-.-{1,2}\s(?!<[0-9])(?:2[0123]|[01]?[0-9]):(?:[0-5][0-9]):(?:(?:[0-5]?[0-9]|60)(?:[:.,][0-9]+)?))

Sry for the late Answer, but no it doesn't work with the pattern above.

@renapu
Give this a try:
WINDNS_TIME:
((?:0?[1-9]|1[0-2])/-/-{1,2}\s(?!<[0-9])(?:2[0123]|[01]?[0-9]):(?:[0-5][0-9]):(?:(?:[0-5]?[0-9]|60)(?:[:.,][0-9]+)?)\s(A|P)M)|((?:(?:0[1-9])|(?:[12][0-9])|(?:3[01])|[1-9])..{1,2}\s(?!<[0-9])(?:2[0123]|[01]?[0-9]):(?:[0-5][0-9]):(?:(?:[0-5]?[0-9]|60)(?:[:.,][0-9]+)?))

Nice it's working now! Thank you for fixing this

Hello i also have a problem with the time i think:
2016.03.02. 21:26:49 08B8 PACKET<PHONE_NUMBER>DF9A10 UDP Rcv <IP_ADDRESS>     c6e3   Q [0001   D   NOERROR] A      (9)PC-737-11(7)hivatal(13)erzsebetvaros(2)hu(0)
I tried all of the above windns time patterns
thanks in advance


Get metrics working on VAE code

These are results with the highest magnitude cut:
{'precision': tensor(0.7104), 'recall': tensor(0.5854), 'f1': tensor(0.6419), 'class_acc': tensor(0.6567), 'conf_matrix': tensor([[ 40,  80],
        [  0, 113]])}

After fixing the VAE in #442, recall and precision for magnitude cut 20 are about the same as AE now
{'precision': tensor(0.4856),
 'recall': tensor(0.9303),
 'f1': tensor(0.6381),
 'class_acc': tensor(0.9819),
 'conf_matrix': tensor([[81,  3],
         [ 0, 82]])}


disk cache: avoid leaking cache files in onEvict function

If there's an ongoing blob upload when that blob is evicted, then the
item was deleted from the LRU index, but the files on disk were left
behind and no longer included in the cache size accounting.
When we evict a blob that's being uploaded, we might need to remove:

Both the temp file and the regular cache file.
Just the regular cache file.

Note that if you're encountering this situation, it might be a sign
that your cache size is too small- when blob uploads start the item
is moved to the most-recently used end of the index, and the item
has moved all the way to the least-recently used end of the index
before the upload has completed.
Fixes #176.

Great catch!

These two fixes seem to be working (stress test includes a small disk cache, large blobs, and proxy backend):
https://github.com/buchgr/bazel-remote/issues/176#issuecomment-582826035

I think we should land these two fixes soon.
Filed #184 to figure out what the status of windows support is, that's probably a larger task.


Get extent of view

@wakirin:
Is your feature request related to a problem? Please describe.
Sometimes it is necessary to know what view the calendar shows as a whole (range from start to end).
Describe the solution you'd like
A function that returns the extent of the current calendar view,
a range/period or list of ranges (for split-view) the calendar currently shows.
Describe alternatives you've considered
Currently, I have to calculate it myself (using the number of months option).
This is much harder with split-view calendars.

onRender event should help here.
onRender: function(element) {
    console.log('picker element', element);
}

element is main container of Litepicker.

onRender event should help here.
onRender: function(element) {
    console.log('picker element', element);
}

element is main container of Litepicker.

@wakirin: But I would still have to query for all the day DOM elements and then use the first and last one for finding out the actual range?

@wakirin: But I would still have to query for all the day DOM elements and then use the first and last one for finding out the actual range?

Maybe I don't understand very well, but if you use something like that:
onRender: function(element) {
    [...element.querySelectorAll('.is-start-date, .is-in-range, .is-end-date')].forEach(el => {
        // here some action for each day in the range
    });
}

I do not plan to include this specific action in the package.
Could you show your current solution and result ?
Thanks.

Maybe I don't understand very well, but if you use something like that:
onRender: function(element) {
    [...element.querySelectorAll('.is-start-date, .is-in-range, .is-end-date')].forEach(el => {
        // here some action for each day in the range
    });
}

I do not plan to include this specific action in the package.
Could you show your current solution and result ?
Thanks.

// get the extent of calendar view
function getCalendarViewExtent( pickerUiElem ) {
    let dayElems     = pickerUiElem.querySelectorAll('.day-item');
                                    // Note: if the calendar adds days of previous/next month to the month days, modify this line to filter these out!
    let firstDayElem = dayElems[ 0 ];
    let lastDayElem  = dayElems[ dayElems.length - 1 ];

    let firstDayDate = new Date( Number( firstDayElem.dataset.time ) );
    let lastDayDate  = new Date( Number(  lastDayElem.dataset.time ) );

    return { // [date-fns interval]
        'start' : firstDayDate,
        'end'   : lastDayDate,
    };
}

// get the extent of calendar view
function getCalendarViewExtent( pickerUiElem ) {
    let dayElems     = pickerUiElem.querySelectorAll('.day-item');
                                    // Note: if the calendar adds days of previous/next month to the month days, modify this line to filter these out!
    let firstDayElem = dayElems[ 0 ];
    let lastDayElem  = dayElems[ dayElems.length - 1 ];

    let firstDayDate = new Date( Number( firstDayElem.dataset.time ) );
    let lastDayDate  = new Date( Number(  lastDayElem.dataset.time ) );

    return { // [date-fns interval]
        'start' : firstDayDate,
        'end'   : lastDayDate,
    };
}

I am assuming you have autoApply: false in your configuration and you want do some action on preselect dates ?
If so, in v2 will be added two events: onSelectStart(startDate) and onSelectEnd(startDate, endDate).

I am assuming you have autoApply: false in your configuration and you want do some action on preselect dates ?
If so, in v2 will be added two events: onSelectStart(startDate) and onSelectEnd(startDate, endDate).

@wakirin: Ah, in this case I want to know the displayed range of the calendar so only that corresponding data can be requested from the server. For example, when the range 01/01/2021 to 01/01/2022 is displayed, the data for that range should be requested from the server and then applied. Currently I have to check the calendar DOM to find out the actual displayed range, including overlapping days (from previous months) that are also shown in the calendar (usually greyed out).

@wakirin: Ah, in this case I want to know the displayed range of the calendar so only that corresponding data can be requested from the server. For example, when the range 01/01/2021 to 01/01/2022 is displayed, the data for that range should be requested from the server and then applied. Currently I have to check the calendar DOM to find out the actual displayed range, including overlapping days (from previous months) that are also shown in the calendar (usually greyed out).


Fixes #6659 - Swagger schemes selection default to page's protocol

Fixes #6659
Makes it so the Schemes selection on the swagger page (e.g. http://localhost:3000/api/swagger) doesn't have to be set if the website uses https. Before would always just default to HTTP since it was the first item in the v1_json.tmpl file. Uses pure javascript to move the protocol of the page's url to the top of the list by getting the swagger json file first, manimpultes json.schemes, and then calls SwaggerUIBundle() with that JSON object rather than a URL.
Fix taken and modified from: https://gist.github.com/miguelmota/289643541cf55825f14d2ea66598bf64
Can see it work with https at https://bg.door43.org/api/swagger

Codecov Report

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

@@           Coverage Diff           @@
##           master    #6660   +/-   ##
=======================================
  Coverage   40.54%   40.54%
=======================================
  Files         406      406
  Lines       54501    54501
=======================================
  Hits        22095    22095
  Misses      29370    29370
  Partials     3036     3036

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 450fc9a...72c5573. Read the comment docs.


Docs: Dataset write takes reader by owner now

Copypasta'ing the docs is currently a compile error

windows CI failure is expected due to vcpkg permissions issue


HTTP responses are cached in the browser

Description
I tried testing the recent changes from https://github.com/onflow/fcl-dev-wallet/pull/155 with the updated flow-cli version (v0.41.1), but noticed that my local browser cached the fcl-dev-wallet HTTP file responses, which prevented the new changes from being used (notice the "from memory cache" label next to the response status code).

See https://github.com/onflowser/flowser/issues/134
Solution
I think we will need to add the appropriate HTTP cache control headers to the static responses.

I found this solution, which should work: https://github.com/zenazn/goji/blob/master/web/middleware/nocache.go


Issue 113

This is a rebased version of #115 submitted from a branch in the main repo for collaboration. Some fixes for the build have been added on top of #115.

TODO: Fix the versions of ga4gh-consent-codes and ADA-M dependencies.

What does “Fix” mean?
Tony
From: Miro Cupak<EMAIL_ADDRESS>Sent: 19 December 2017 14:04
To: ga4gh/beacon-team<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: Re: [ga4gh/beacon-team] Issue 113 (#120)
TODO: Fix the versions of ga4gh-consent-codes and ADA-M dependencies.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/ga4gh/beacon-team/pull/120#issuecomment-352761131, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AI_EVNo39YEUv_05BCQGPFGp84ECf_S2ks5tB8JZgaJpZM4RGutW.

Sorry about the ambiguity. It means to refer to a fixed, stable tag (to be created) instead of a moving branch. Otherwise we risk our spec gets broken as a result of a change in its dependencies.

Hi @mcupak . Do you think this PR can be merged and closed? Or maybe just closed without merging as it's outdated.

This is now out of date. Closing.


Turned off fail-fast

#1237  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @CatChen and the rest of your teammates on  Graphite


[TRIVIAL] Fix NuDB build error

High Level Overview of Change
Fix a build error that seems to be limited to non-unity builds on Windows (Visual Studio) due to a missing #include in NuDB.
Context of Change
I think the reordering of #includes in the physical redesign PR (#4997) removed a silent dependency from one of the headers in NuDB. The dependency version of NuDB didn't change, as far as I can tell. CI didn't catch the issue because the Windows build uses unity to save time.
The error is:
C:\Users\passp\.conan\data\nudb\2.0.8\_\_\package\5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9\include\nudb/detail/stream.hpp(64,20): error C2039: 'logic_error': is not a member of 'std' [C:\Dev\rippled\upstream\build\cmake\msvc19.OFF\rippled.vcxproj]

std::logic_error requires <stdexcept>, but nudb/detail/stream.hpp does not include it.
My assumption is some other header that had previously been ordered before stream.hpp and included <stdexcept> is now ordered after stream.hpp. I don't think it's worth figuring out which file it is, because it's obviously a problem with NuDB.
Type of Change

[X] Bug fix (non-breaking change which fixes an issue)
[X] Build

Future Tasks
See also: https://github.com/cppalliance/NuDB/pull/100

Rewrote this to patch the source via Conan.