Do not understand the logic of transfering msg.value

Hi,
I'm trying to learn solidity. I think I misunderstood something.
Here you said that :
https://github.com/dabit3/polygon-ethereum-nextjs-marketplace/blob/c5a35262a430a6da1f9cd57cc731f227b5469716/contracts/NFTMarketplace.sol#L113
but just before you said that the seller is address(0) :
https://github.com/dabit3/polygon-ethereum-nextjs-marketplace/blob/c5a35262a430a6da1f9cd57cc731f227b5469716/contracts/NFTMarketplace.sol#L109
So what I understand is that instead of transfering the msg.value to the old owner, you transfer it to address(0). Do you want to do that or I didn't understand it correctly ?
Thank you,
Inaki

I think that's wrong when i buy nft the ethereum value dissapear into thin air becasue of seller address becoming address(0) but after moving
idToMarketItem[tokenId].seller = payable(address(0));
after
payable(idToMarketItem[tokenId].seller).transfer(msg.value);
its working perfectly.

Ok thank you !
How do you check that with the HardHat test?
I tried to had these 2 lines but I get "0" :
    const buyerBalance = await nftMarketplace.balanceOf(buyerAddress.address);
    console.log('buyerBalance: ', buyerBalance.toString())

So I think that `balanceOf()' is not the right function.
Do you have a suggestion?

i have no clue about that i am learning myself and playing with it using truffle >.>


Ops: Add worker health to status page

We should add our worker to our public status page.
Challenge
Our worker is not accessible publicly, i.e cannot offer a health endpoint directly.
Solution

Setup a scheduled task, e.g each minute, for the worker to store some checks in Redis.
Add a /healthz endpoint to our public API checking the above and reporting 200 or 500 accordingly

Discovered this:

https://arq-docs.helpmanual.io/#health-checks
https://github.com/python-arq/arq/blob/3914e48c530f0e5acf02cb340b1f8f3e201ba895/arq/worker.py#L891-L919

ARQ already writes its own healthcheck key in Redis; so maybe we just need to query for that key in our health endpoint.

Hi @birkjernstrom, I've created a PR with the changes mentioned in the discussions. Could you please check it?


Update CoreDNS to version 1.8.3.

This also required a run of go generate.

Proposed Changes

Types of Changes

Updates CoreDNS to 1.8.3.
Verification

Linked Issues

#3001
Further Comments

It looks like the CoreDNS RBAC needs to be updated:
[[36mPARALLEL[m] E0409 20:45:48.534248       1 reflector.go:138] pkg/mod/k8s.io/client-go@v0.20.2/tools/cache/reflector.go:167: Failed to watch *v1beta1.EndpointSlice: failed to list *v1beta1.EndpointSlice: endpointslices.discovery.k8s.io is forbidden: User “system:serviceaccount:kube-system:coredns” cannot list resource “endpointslices” in API group “discovery.k8s.io” at the cluster scope
[[36mPARALLEL[m] E0409 20:46:38.910482       1 reflector.go:138] pkg/mod/k8s.io/client-go@v0.20.2/tools/cache/reflector.go:167: Failed to watch *v1beta1.EndpointSlice: failed to list *v1beta1.EndpointSlice: endpointslices.discovery.k8s.io is forbidden: User “system:serviceaccount:kube-system:coredns” cannot list resource “endpointslices” in API group “discovery.k8s.io” at the cluster scope


Fix mismatched types in trait based on features

This was preventing version publication, so with this merged we'll try and re-publish version v0.7.1
Resolved #230

Thank you for fixing this @slawlor!


Fixed bit string support in postgres when using prepared statements

Previously bit types were being considered as booleans and would throw an exception when prepared statements were enabled.
Fixes https://github.com/jruby/activerecord-jdbc-adapter/issues/826

Did jruby:rails-5 get deleted? Seems to be missing?

the rails-5 branch is now the master branch


Fix exception about parameter already been declared & Change default c++ version to 17

1- Make the default C++ version to 17 since rclcpp uses features that were introduced in that version https://github.com/ros2/rclcpp/search?q=variant
2- Use auto_declare for multi_interface_forward_command_controller and forward_command_controller to fix the following exception Exception thrown during init stage with message: parameter 'joints' has already been declared

Same here, thanks for fixing this Jafar!


spelling: WebAseembly -> WebAssembly

An identical typo is also visible in the recently publicized wasm-core-2 TR. I haven't been able to find the template from which that was generated, but I assume it's replicated from this one.

Nice catch!


fix ElektronPattern implementation virtual methods, fix SDDrive read

Is the constructor (bool _init = true)  required?
Initialisation looks to be setting static type parameters
In other words, in what scenario are these anything but the constructor values ?
maxSteps = 64;
maxParams = 24;
maxTracks = 16;
maxLocks = 64;

Also, are we certain about clearPattern() originally calling base class {}
I have some concerns that the lock arrays need to be correctly initialized.
654 void MDSeqTrack::merge_from_md(uint8_t track_number, MDPattern *pattern) {
655   DEBUG_PRINT_FN();
656   for (int i = 0; i < 24; i++) {
657     if (!IS_BIT_SET32(pattern->lockPatterns[track_number], i)) {
658       continue;
659     }
660     int8_t idx = pattern->paramLocks[track_number][i];
661     if (idx < 0) {
662       continue;
663     }
664     for (int s = 0; s < 64; s++) {
665       int8_t lockval = pattern->locks[idx][s];
666       if (lockval >= 0 &&
667           IS_BIT_SET64(pattern->trigPatterns[track_number], s)) {
668         set_track_locks(s, i, lockval);
669       }
670     }
671   }

Checks for lookval >= 0

File was renamed from MDPattern.hh -> MDPattern.h, so git history only goes back to Augusst.
https://github.com/jmamma/MIDICtrl20_MegaCommand/blob/2.70/avr/cores/megacommand/MD/MDPattern.hh

https://onlinegdb.com/rye27Hsj2v
^
According to this behaviour the original implementation called MDPattern::clearPattern() not base class virtual clearPattern.

Also, are we certain about clearPattern() originally calling base class {}

It is called in the base method init()

RIght. Doesn't explain how lock decoding was working then ?

You are right.. The derived method IS virtual even if it's not declared with the virtual keyword.

https://www.onlinegdb.com/fork/rye27Hsj2v

Guess I could never be a C++ pro (

Yeah, i had to double check out of curiosity.
https://onlinegdb.com/H17muoonD
Type of question they throw on a CS exam.

If it was multiple choice you'd have a 25% chance of being a C++ pro.

If it was multiple choice you'd have a 25% chance of being a C++ pro.

Coin flippin pro
Actually the whole XPattern : ElektronPattern thing surprise me as the base defines concrete data fields (locks etc.)

	uint8_t maxParams;
	uint8_t maxTracks;
	uint8_t maxSteps;
	uint8_t maxLocks;

Initialisation looks to be setting static parameters related to the type.

Depends on how generic we want them to be. Setting static members means we cannot get the correct value in a "virtualized" routine.
http://www.cplusplus.com/forum/general/11196/#msg52877
Of course this will only be useful when we are able to decode MNMPattern and A4Pattern.

Still seems to be a problem here...
SDDrive restore doesnt work properly.. causes MD tempo to glitch to 247.7

ClearPattern() is no longer being called.. but the restored pattern data is still broken.

Try soft reset first?

Try soft reset first?

I've been doing factory resets and soft resets all day.
Are you able to sort this out?
Blocker for 3.0. I need to get it released tonight.

The other option is that I drop SD Drive from the release.

Probably a good idea -- make it back when the sampling feature is ready.

KitData is empty when transmitting...

  //  Kits
  progress_max = 64;
  for (int i = 0; i < 64; ++i) {
    progress_i = i;
    mcl_gui.draw_progress("Loading kit", i, 64);
    if (!mcl_sd.read_data_v<MDKit>(&MD.kit, &file)) {
      goto load_error;
    }
    MD.setSysexRecPos(4, i);
    delay(20);
    DEBUG_PRINTLN(MD.kit.name); <---- Prints empty string
    MD.kit.toSysex();

I made this change
   template <class T> bool read_data_v(T *data, FatFile *filep) {
     auto ret = read_data(data, sizeof(T), filep);
-    ::new(data)T();
+    ::new(data)T;
     return ret;
   }

and it seems to be fixed?

Kit and Pattern data is working now.
Globals still broken..

MDGlobal had a constructor whereas MDKit didn't.

-    ::new(data)T();
+    ::new(data)T;

I don't quite understand this (0o0)
Both seem to be placement-new.
Wait, it wasn't here before in 2.7 was it?
The whole placement-new thing is introduced when we were working on the generic seq track classes.

The one thing  I noticed was that MDPattern and MDKit did not have constructors..
I compares with deviceTrack implementation.. and it didn't use ()

MDGlobal problem seems different. It could be with sysex encoding. Can you take a look?

Hmm, i try MDGlobal.from and MDGlobal.to and it works okay from global 0 -> to global 1

How did you do that? Isn't the target slot governed by     MD.setSysexRecPos(2, i); ?

The one thing I noticed was that MDPattern and MDKit did not have constructors..

No constructor = compiler will auto create "trivial constructor"

  MD.getBlockingGlobal(i);

    MD.setSysexRecPos(2, i + 1);
    ElektronDataToSysexEncoder encoder(&MidiUart);
      delay(20);
      MD.global.toSysex(&encoder);

I think the global tempo decoding is broken.
     if (!MD.getBlockingGlobal(i)) {
      error_is_md = true;
      DEBUG_PRINTLN("error global");
      goto save_error;
    }
    DEBUG_PRINTLN("global");
    DEBUG_PRINTLN(MD.global.origPosition);
    DEBUG_PRINTLN(MD.global.tempo);

21:48:41.303 -> global
21:48:41.303 -> 0  <- orig position
21:48:41.303 -> 11832 <- tempo * 24

MD.global.tempo) should == 3000 for 125bpm

any idea what might have changed?

git blame shows I refactored it in Sep.

100   origPosition = midi->midiSysex.getByte(offset + 3);
101   ElektronSysexDecoder decoder(midi, offset + 4);
102   decoder.stop7Bit();
103   decoder.get(drumRouting, 16);
104
105   decoder.start7Bit();
106   decoder.get(keyMap, 128);
107   decoder.stop7Bit();
108
109   decoder.get8(&baseChannel);
110   decoder.get8(&unused);
111   decoder.get16(&tempo);
112   decoder.getb(&extendedMode);
113
114   uint8_t byte = 0;
115   decoder.get8(&byte);
116   clockIn = IS_BIT_SET(byte, 0);
117   transportIn = IS_BIT_SET(byte, 4);
118   clockOut = IS_BIT_SET(byte, 5);
119   transportOut = IS_BIT_SET(byte, 6);
120   decoder.getb(&localOn);
121
122   decoder.get(&drumLeft, 12);

111   decoder.get16(&tempo);

There it is

This thing should be broken from the very start (??)

Yeah, i suspect it was, but im not sure why  2.70 didnt glitch out the tempo on global transmission.

toSysex does it right:
  uint8_t tempo_lower = (uint8_t)(tempo & 0x7F);
  uint8_t tempo_upper = (uint8_t)(tempo >> 7);
  encoder->pack8(tempo_upper);
  encoder->pack8(tempo_lower);

can you submit a patch?
i need to eat some food.

Can do. But no access to MD so can't test it.

I'll think about this

we can always override tempo with a default value. tempo doesnt change when switching globals if I recall correctly.

Patched ^^^

ef5399b why 😂😂😂

C++ rule of thumb

Elegant stuff doesn't work
Working stuff ain't elegant

sorry i missed a file. forced pushed  eef9f12

machinedrum lockup on recv global :(

p = new T; // default init
p = new T(); // value init -- if no ctor is provided by user, then conducts zero init (zeros everything)

See:
https://en.cppreference.com/w/cpp/language/new#Placement_new
https://en.cppreference.com/w/cpp/language/value_initialization
https://en.cppreference.com/w/cpp/language/default_initialization

if no ctor is provided by user, then conducts zero init (zeros everything) ahaha.

machinedrum lockup on recv global :(

working now when i enabled debug.
let me try again for good luck.

I think we should add back MDGlobal()

This is the kind of dark corner in C++ -- if we don't provide ctor, but it's actually a derived class from ElektronSysexObject with virtual methods... the behavior is kinda undefined

Okay, can you add constructors  MDGlobal, MDPattern and MDKit ?

Processing. (You're doing declarative programming)

Just did an empty reset and snap restore. looks good.
I noticed from dumping Kit.name that the default MDKit's above 16 which appear as empty still had some data in them.

Thanks. one more factory reset, snap then restore.
then we merge this one.

then release 3.0.
then no more bug fixing for a long time.

then no more bug fixing for a long time.

Let's have some fun hacking new machines for MD instead!

I'd like to make some music, oneday.

Yep, one day, and then get back to work :p :p :p

restore was successful.

closing this one.


Write a Scala program to convert decimal to octal

Description
Write a Scala program to convert decimal to octal
Octal is a base-8 number system. It uses only eight digits: 0, 1, 2, 3, 4, 5, 6, and 7. The digits are called octits. An octit is the smallest unit of information in a computer. An octit can be used to represent a single value, such as 0, 1, 2, 3, 4, 5, 6, or 7, or it can be used to represent a single state, such as on or off.
To convert a decimal number to octal, we can use the following algorithm:

Divide the decimal number by 8.
Write down the remainder.
Divide the quotient by 8.
Write down the remainder.
Repeat steps 3 and 4 until the quotient is 0.
The octal number is the sequence of remainders written in reverse order.

Input  : 10
Output : 12

How to contribute

Save the solution in convertDecimalToOctal.scala file
Add convertDecimalToOctal.scala file in convert-decimal-to-octal folder

!assign

Hey @Chirag8023,
This issue has been assigned to you for more than 15 days.
We are unassigning you from this issue.
If you are still interested in contributing to this issue, please get auto-assigned on the issue by commenting !assign again.
Thanks for your contributions :)


Stop writeback when the disk is almost full

Currently, we don't limit the capacity used for writeback, which may fill all the free space, and other application may fail.
We should stop the writeback when the disk is almost full (the free ration is below half of specified free ratio), because writeback should have higher priority than read cache.

When the data is synchronized to the object storage, the write cache needs to be automatically cleaned up, otherwise the disk space will always be occupied

After uploaded, the data in local disk will be converted to read cache, which will be evicted automatically in an LRU-like strategy.


/etc/openvpn/ovpn_env.sh: No such file or directory

I updated all my system (Ubuntu 14.04.2 LTS) and docker (1.6.0) and my openvpn server won't start anymore, with the debug enabled I get this output :
$ docker run --volumes-from $OVPN_DATA -p 1194:1194/udp --privileged -e DEBUG=1 kylemanna/openvpn
+ set -e
+ source /etc/openvpn/ovpn_env.sh
/usr/local/bin/ovpn_run: line 13: /etc/openvpn/ovpn_env.sh: No such file or directory

It looks like that a path changed in the update process.
Thanks !

The ovpn_env.sh file is generated by the ovpn_genconfig script.  So either you never generated the config (probably not true if it worked before) or you deleted/changed the volume container.
Look at the $OVPN_DATA container.

I have the same issue and I ran the ovpn_genconfig before.

openvpn:
image: kylemanna/openvpn
container_name: "openvpn"
cap_add:
- NET_ADMIN
environment:
- OVPN_SERVER=udp://vpn.rutvik2611.com
volumes:
- "./persist/openvpn:/etc/openvpn"
restart: always
networks:
- backboneNetwork
labels:
- "traefik.enable=true"
- "traefik.tcp.routers.openvpn.entrypoints=openvpn"
- "traefik.tcp.routers.openvpn.rule=HostSNI(vpn.rDOMAIN.com)"
- "traefik.tcp.services.openvpn.loadbalancer.server.port=1194"
- "traefik.tcp.routers.openvpn.tls=true"
- "traefik.tcp.routers.openvpn.tls.certresolver=myresolver"
root@SOME:~# docker logs openvpn
/usr/local/bin/ovpn_run: line 55: /etc/openvpn/ovpn_env.sh: No such file or directory
/usr/local/bin/ovpn_run: line 55: /etc/openvpn/ovpn_env.sh: No such file or directory


transforms get pushed together upon conversion.

If I use the following SVG
<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 100 100">
  <g>
    <rect x="48" y="26" width="4" height="4" rx="2" ry="2" transform="translate(78 -22) rotate(90)" stroke="transparent"/>
    <rect x="49" y="65" width="2" height="14" rx="1" ry="1" transform="translate(122 22) rotate(90)" stroke="transparent"/>
  </g>
</svg>

it ends up converted to
<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 100 100" class="device-svg">
	<g stroke="transparent">
		<rect x="48" y="26" width="4" height="4" rx="2" ry="2" transform="rotate(90 50 28)"></rect>
		<rect x="49" y="65" width="2" height="14" rx="1" ry="1" transform="rotate(90 50 72)"></rect>
	</g>
</svg>

See how the transforms are all pushed into rotate instead of leaving them as they should.
Any fix for that?

I guess this is not an issue with this loader.
You could try disabling the apt (I'm not sure which one it is) svgo plugin that does this.

Yeah, I assumed it might have to do with SVGO, will try my luck going through their documentation.


Generating extension service assembly definitions & folder clean up

Overview
This PR adds the proper assembly definitions for the existing MRTK extension services.

New assembly definitions for the Extensions SceneTransitionService and LostTrackingService, including editor where appropriate.
Added folder structure to match UnityAR data provider extension service.

Changes

Fixes: #6562

Verification

This optional section is a place where you can detail the specific type of verification
you want from reviewers. For example, if you want reviewers to checkout the PR locally
and validate the functionality of specific scenarios, provide instructions
on the specific scenarios and what you want verified.
If there are specific areas of concern or question feel free to highlight them here so
that reviewers can watch out for those issues.
As a reviewer, it is possible to check out this change locally by using the following
commands (substituting {PR_ID} with the ID of this pull request):
git fetch origin pull/{PR_ID}/head:name_of_local_branch
git checkout name_of_local_branch

/azp run

/azp run


Add documentation to disable kube-prometheus-stack optional features

Documentation should be added to cover the issue described below.
Whenever the conditional dependency kube-prometheus-stack.enabled is disabled, it does not produce the expected behaviour of disabling all the Prometheus stack components. In fact, each of those should be disabled in the values.yaml, by adding the following:
kube-prometheus-stack:
  enabled: false
  prometheusOperator:
    enabled: false
  grafana:
    enabled: false
    adminPassword: e9JYtk83*4#PM8
  alertmanager:
    enabled: false
  prometheus:
    enabled: false

It should be noted that unless all the Prometheus components are disabled, the helmchart will attempt to install Prometheus CRDs. This might be an issue in cases when the service account used to deploy the cluster does not have enough permissions to install Custom Resources Definitions.

@MMirelli - would you mind opening a PR to document this in the README? Thanks!


Updates for v3.0.12

Description

Related issues/PRs

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

Deploy preview for calico ready!
Built with commit 9ecfcc79bace9f3286597572cb5330bec47615f9
https://deploy-preview-2808--calico.netlify.com


Bug in M2M authentication

(v3.3.0)
When attempting to follow the documentation for M2M authentication I get an error that after tons of digging seems to be the result of a bug.
Specifically, there's a bug in the ExternalAuthProvider class (which is used when attempting to authenticate via M2M following the instructions.
class ExternalAuthProvider(AuthProvider):
    def __init__(self, credentials_provider: CredentialsProvider) -> None:
        self._header_factory = credentials_provider()  # Returns a dict when instance of OAuthCredentialsProvider

    def add_headers(self, request_headers: Dict[str, str]):
        headers = self._header_factory()  # Attempts to call the dict resulting in TypeError
        for k, v in headers.items():
            request_headers[k] = v

If I had to guess, I would assume that either the OAuthCredentialsProvider should return a callable when called, or that assignment of the _header_factory attribute should not be a call to the instance. Then the headers dict would be correctly generated in add_headers

@jwhite3 Sorry, I misunderstood your issue. So, the example from the link you provided looks like this:
from databricks.sdk.core import Config, oauth_service_principal
from databricks import sql
import os

server_hostname = os.getenv("DATABRICKS_SERVER_HOSTNAME")

def credential_provider():
  config = Config(
    host          = f"https://{server_hostname}",
    client_id     = os.getenv("DATABRICKS_CLIENT_ID"),
    client_secret = os.getenv("DATABRICKS_CLIENT_SECRET")
  )

  return oauth_service_principal(config)                    # <-- !!!

with sql.connect(server_hostname      = server_hostname,
                 http_path            = os.getenv("DATABRICKS_HTTP_PATH"),
                 credentials_provider = credential_provider) as connection:

While it seems that there's a bug in ExternalAuthProvider - in fact, everything is okay there. credentials_provider has to be a factory - function that returns a function that returns a dict. The bug actually is in the line I highlighted - the oauth_service_principal(config) should be wrapped with a function. So the example should be fixed like this (note the additional wrapper function get_headers):
from databricks.sdk.core import Config, oauth_service_principal
from databricks import sql
import os

server_hostname = os.getenv("DATABRICKS_SERVER_HOSTNAME")

def credential_provider():
  config = Config(
    host          = f"https://{server_hostname}",
    client_id     = os.getenv("DATABRICKS_CLIENT_ID"),
    client_secret = os.getenv("DATABRICKS_CLIENT_SECRET")
  )

  def get_headers():
    return oauth_service_principal(config)

  return get_headers

with sql.connect(server_hostname      = server_hostname,
                 http_path            = os.getenv("DATABRICKS_HTTP_PATH"),
                 credentials_provider = credential_provider) as connection:

Indeed, we still have to fix this in docs, but at least not the library itself 🙂


GraphQL Tool

I am currently exploring the Agents API and could not find a GraphQL Tool. Are you planning to provide one?
(thanks for langchain!)

This would be interesting, I could actually take it up the development.  - If you think we should implement it @nfcampos.

This is partially solved in #596
But would be good to have a builtin tool that doesn't depend on an external api

🧟
Was anything ever merged in to support this feature? 596 hasn't moved in almost a year.  The latest docs still don't show a JS tool or agent for handling graphQL calls (AFAIK). Are there known workarounds beyond switching to the Python version?


Lock GitHub Actions runners image version

The GitHub Actions workflows use the latest Ubuntu images for action runners. This may cause instability when the images are updated to new OS releases. The workflows should lock the runners images to a specific OS version.

Changed 'runs-on' GitHub Actions runner image from 'ubuntu-latest' to 'ubuntu-22.04'.


Hey guys i'm currently checking out envoy with opa integration with http but i have an issue that the opa is throwing 404

Title:  Error 404 when envoy checks opa for authorization
Description:
Im currently using the envoy and opa container in my docker compose environment. I tested the routes without external authorization with opa and they worked. Now i added the opa with a config that allows all.
Envoy now tries to query the permissions without a request body and with the wrong path.
Here the opa logs
This is a curl request for the opa:

2024-09-13 09:00:53 {"client_addr":"<IP_ADDRESS>:40678","level":"info","msg":"Received request.","req_body":"{\r\n    "input": {\r\n        "attributes": {\r\n            "request": {\r\n                "http": {\r\n                    "method": "OPTIONS",\r\n                    "path": "/api"\r\n                }\r\n            }\r\n        }\r\n    }\r\n}","req_id":4,"req_method":"POST","req_params":{},"req_path":"/v1/data/envoy/authz","time":"2024-09-13T07:00:53Z"}`
2024-09-13 09:00:53 {"decision_id":"612d5490-0f45-47f5-9650-7f8af5e62588","input":{"attributes":{"request":{"http":{"method":"OPTIONS","path":"/api"}}}},"labels":{"id":"a446bfba-462f-4b39-9502-9b0a241b4fb7","version":"0.67.1-envoy-4"},"level":"info","metrics":{"counter_server_query_cache_hit":1,"timer_rego_external_resolve_ns":513,"timer_rego_input_parse_ns":77520,"timer_rego_query_eval_ns":218094,"timer_server_handler_ns":376393},"msg":"Decision Log","path":"envoy/authz","req_id":4,"requested_by":"<IP_ADDRESS>:40678","result":{"allow":true},"time":"2024-09-13T07:00:53Z","timestamp":"2024-09-13T07:00:53.125330015Z","type":"openpolicyagent.org/decision_logs"}
2024-09-13 09:00:53 {"client_addr":"<IP_ADDRESS>:40678","level":"info","msg":"Sent response.","req_id":4,"req_method":"POST","req_path":"/v1/data/envoy/authz","resp_body":"{"decision_id":"612d5490-0f45-47f5-9650-7f8af5e62588","result":{"allow":true}}\n","resp_bytes":79,"resp_duration":0.797659,"resp_status":200,"time":"2024-09-13T07:00:53Z"}

This is the envoy request:

2024-09-13 09:02:12 {"client_addr":"<IP_ADDRESS>:44880","level":"info","msg":"Received request.","req_body":"","req_id":5,"req_method":"OPTIONS","req_params":{},"req_path":"/api","time":"2024-09-13T07:02:12Z"}
2024-09-13 09:02:12 {"client_addr":"<IP_ADDRESS>:44880","level":"info","msg":"Sent response.","req_id":5,"req_method":"OPTIONS","req_path":"/api","resp_body":"404 page not found\n","resp_bytes":19,"resp_duration":0.554975,"resp_status":404,"time":"2024-09-13T07:02:12Z"}

My envoy config for the opa:
static_resources:
  listeners:
    - name: listener_0
      address:
        socket_address:
          address: <IP_ADDRESS>
          port_value: 10000
      filter_chains:
        - filters:
          - name: envoy.filters.network.http_connection_manager
            typed_config:
              "@type": type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
              http_filters:
              - name: envoy.filters.http.ext_authz
                typed_config:
                  "@type": type.googleapis.com/envoy.extensions.filters.http.ext_authz.v3.ExtAuthz
                  http_service:
                    server_uri:
                      uri: http://opa:8181
                      cluster: opa_cluster
                      timeout: 5s
...
...
  clusters:
    - name: opa_cluster
      connect_timeout: 0.25s
      type: LOGICAL_DNS
      lb_policy: round_robin
      load_assignment:
        cluster_name: opa_cluster
        endpoints:
          - lb_endpoints:
              - endpoint:
                  address:
                    socket_address:
                      address: opa
                      port_value: 8181

Did i set up envoy wrong, or what could the problem be?
I tested it in multiple envoy version (1.23, .1.28, 1.31)

with_request_body in the ExtAuthz config would probably solve one of the stated problems.
https://www.openpolicyagent.org/docs/v0.22.0/envoy-authorization/ (which I realize is outdated but I couldn't find an equivalent up to date page) suggests that opa is a grpc service, so you should probably be using grpc_service rather than http_service in the config. That tutorial also shows how to do with_request_body.

cc @tyxia in case more familiarity with ext_authz would add something helpful.

Thanks for the reply, i found this here https://www.openpolicyagent.org/docs/latest/envoy-tutorial-standalone-envoy/#deploying-an-application-with-envoy-and-opa-sidecars
Do i really need to run the sidecar container for opa to run or is there a way to not have sidecars container running because it's add a new layer of complexity in my deployments?

Never mind was a layer 8 problem on my side because it used http:// in the hostname in the cluster object because i was still thinking about the http communication even in grpc so it works now


Inference is broken for object generics with return function

Bug Report
🔎 Search Terms
inference, generics, return type
🕗 Version & Regression Information
I'm not sure when this bug "appeared", but it seems to behave this way for every version, even the nightly

This is the behavior in every version I tried, and I reviewed the FAQ for entries about object property inference with regards to function returns

⏯ Playground Link
https://www.typescriptlang.org/play?ts=4.5.2#code/GYVwdgxgLglg9mABCAzgUwIojQJwJ4A8AKlrngNJp4A0iRAIgIZSMB8AFAPpwAOsCKAFyIA3gChEiAI7Z8lPMJKyKVANwTpygGJhh7CAihoAHlAD8wkZrLzFpOVUQBfAJSIAvKzpMW6yQgBlEAgINBQhRHYAE2ZGRR9GN09EADc4GCj1V1EnMTEDMBQoawc8D0QAbQAiIyKq2gBGWgAmWisAZmEoHGxnAF08gHpBxDA4RAKjU1HxiHwxVExldnFJGRsqag11-B09F2EwEABbACNcDy8mjUDg0PC9GJYkryfGADooOC0YYzQo9guLaudRiYaIRgoRAoOAICFQgCSyHQEygxlob0QMChY0QABsEABzC5wYCIKB4HhoUYnc44RDAOD024hMIoBboex4FbbZTyLZrbS6SLQYwHGlnC7Ja6ScEAdQAFmVsYgAAZvVWIUnkynU1WMMB4VVmG5gIKsh6RN4vRBvT7fX7-QHAlzqIA
💻 Code

function useQuery<TQueryKey, TData>(_options: {
  queryKey: TQueryKey;
  queryFn: (context?: { queryKey: TQueryKey }) => TData;
  onSuccess: (data: TData) => void;
}) {}

const queryKey = ["test", 1, 2, { 3: true }]

// no context no cry
useQuery({
  queryKey,
  queryFn: (): number => 1,
  onSuccess: (data) => data.toFixed(),
});

// as soon as I use ctx, data is no longer of type number for onSuccess
useQuery({
  queryKey,
  queryFn: (ctx): number => 1,
  // Why is `data` of type `any`?
  onSuccess: (data) => data.toFixed(),
});

🙁 Actual behavior
In the second example the data argument on the onSuccess function is typed as any.
🙂 Expected behavior
In the second example the data argument should be typed as the return value of the queryFn property, or more specifically, number

cc @tkdodo

Pretty sure it's the same as #43371.

Thanks @MartinJohns. I'll track it there.


feat: consolidate MonadThrow and derivations into Monad

derivations.ts: removed
monad_throw.ts: removed
monad.ts: inherits createMonad and MonadThrow
types.ts: updated to reference new MonadThrow location
testing: moved derivations.test.ts to monad.test.ts

Fixes #80
Fixes #101

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

69 of 69 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.003%) to 92.611%

Totals

Change from base Build<PHONE_NUMBER>:
0.003%

Covered Lines:
5049

Relevant Lines:
5458

💛  - Coveralls


Please make .OTF generation reproducible.

Whilst working on the Reproducible Builds effort, we noticed that ufo2ft generates .otf files that are not reproducible.
For example, here is showotf output of the league-spartan font:
│ │ │ │ │  HEAD table (at 188)
│ │ │ │ │  	Version=1
│ │ │ │ │  	fontRevision=2
│ │ │ │ │ -	checksumAdj=e32dbbe4
│ │ │ │ │ +	checksumAdj=e3309724
│ │ │ │ │  	magicNumber=5f0f3cf5 (0x5f0f3cf5, diff=0)
│ │ │ │ │  	flags=3 baseline_at_0 lsb_at_0
│ │ │ │ │  	unitsPerEm=1250
│ │ │ │ │  	create[0]=0
│ │ │ │ │ -	 create[1]=d580d7b0
│ │ │ │ │ -	File created: Tue Jul  4 05:27:12 2017
│ │ │ │ │ +	 create[1]=d57f6a10
│ │ │ │ │ +	File created: Mon Jul  3 03:27:12 2017

… which shows that it varies on the local timezone.
This has also been filed in Debian as #890280

Python 2 failing.. can't recall the incantation there! :)

I don't think there's a one-line way to do this in Python 2. Easiest way may be to implement timezone.utc locally, as in the example here: https://docs.python.org/2/library/datetime.html#tzinfo-objects (under "Example tzinfo classes:").

@jamesgk Thanks for that. I have updated the PR to match and now I read:

All checks have passed

:)

@anthrotype and/or @moyogo, want to review before merging? And maybe we should add a unit test.

Thanks!

Thanks @lamby for this PR! (hi @jamesgk! :hand:)
Yeah, this is a problem with the UFO spec, which doesn't make it clear whether dates and times should be interpreted as UTC or local time:
https://github.com/unified-font-object/ufo-spec/issues/54
Both you and I would interpret such "naive" UFO datetimes (without explicit timezone info) as UTC, however the author of ufo2fdk (from which ufo2ft was forked) seems to have interpreted them as local times. I only infer this from the fact that the code is using time.mktime, and the latter expects a time tuple that is expressed as local time (it's the inverse of time.localtime).
With your patch, you made this datetime object "aware" by giving it an explicit UTC tzinfo.
But if we agree that all dates and times in UFO (lacking an explicit +HHMM offset) should be understood as UTC, I think an even simpler solution is to avoid using time.mktime altogether and instead use calendar.timegm to convert the time struct to a timestamp. This is the inverse of time.gmtime (for historic reason it happens to be in the calendar module), it takes a time struct or tuple expressed as UTC, and converts it to a UNIX timestamp (integer):
https://docs.python.org/3.6/library/calendar.html#calendar.timegm
I'll send a PR for that.


commands are responding twice

commands are responding twice and need a help wih that 👎

Does this occur in all kinds of chats?

yes


[upstream-with-swift] Fix warnings building Swift with swift-llvm with MSVC

MSVC warns: '>=' unsafe use of type 'bool' in operation.

@jrose-apple here's the upstream-with-swift copy. Quick question: does this mean all commits need to be duplicated? I'm not sure whether I have anything else to contribute to swift-llvm right now but just to know

Generally yes, for LLVM and Clang. Formally, commits go to upstream-with-swift first, then get cherry-picked to swift-x.y-branch (if they're going into the current release) or stable (if they're going into a subsequent release).

@jrose-apple  understood, thanks for letting me know

@swift-ci please test

AFAIK "please test" isn't set up for the upstream-with-swift branches yet.


Version, Ice 1.3.1

Contains fixes for
PHP Fatal error:  Declaration of Ice\Arr::offsetExists(string $offset): bool must be compatible with ArrayAccess::offsetExists($offset) in Unknown on line 0
PHP Fatal error:  Declaration of Ice\Arr::offsetGet(string $offset) must be compatible with ArrayAccess::offsetGet($offset) in Unknown on line 0
PHP Fatal error:  Declaration of Ice\Arr::offsetSet(string $offset, $value) must be compatible with ArrayAccess::offsetSet($offset, $value) in Unknown on line 0
PHP Fatal error:  Declaration of Ice\Arr::offsetUnset(string $offset) must be compatible with ArrayAccess::offsetUnset($offset) in Unknown on line 0
PHP Fatal error:  Declaration of Ice\Mvc\Model::unserialize(string $data) must be compatible with Serializable::unserialize($serialized) in Unknown on line 0
PHP Fatal error:  Declaration of Ice\Auth\Driver\File::login(string $username, string $password, ?bool $remember = NULL, ?bool $force = NULL): ?bool must be compatible with Ice\Auth\Driver\DriverInterface::login($username, string $password, ?bool $remember = NULL, ?bool $force = NULL) in Unknown on line 0
``

There will be minor changes, so we need v1.4.0


chore(ln-dlc-node): Remove superseded onboard_from_lnd test

This test got superseded by tests-e2e / app_can_be_funded_with_lnd_faucet() test.
Along with the test, some dead code could be removed from our library.

bors r+


add missing settings translations

5 languages huh? You're either quite the linguist, or proficient with translate.google.com

LOL i speak french the other ones were missing and i just worked with google translation.
btw my native language is arabic and like i saw in translation file almost all is not right (just a google translation) So you need to thank me instead for adding this.


Goto Definition not working - Cannot read property getSourceFiles of undefined

Issue Type: Bug
Hello! Last week my ctrl+click for goto definition has stopped working. I get the following message from the Typescript output window (on verbose setting):

[Trace  - 09:40:00] Response received: documentHighlights (4). Request took 545 ms. Success: false . Message: Error processing request. Cannot read property 'getSourceFiles' of undefined
TypeError: Cannot read property 'getSourceFiles' of undefined
at ConfiguredProject.Project.getScriptInfos (c:\Program Files\Microsoft Visual Studio Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:126095:44)
at ProjectService.sendProjectTelemetry (c:\Program Files\Microsoft Visual Studio Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:128117:66)
at ConfiguredProject.updateGraph (c:\Program Files\Microsoft Visual Studio Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:126726:37)
at Object.updateProjectIfDirty (c:\Program Files\Microsoft Visual Studio Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:127081:45)
at ConfiguredProject.Project.getLanguageService (c:\Program Files\Microsoft Visual Studio Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:125915:28)
at IOSession.Session.getDocumentHighlights (c:\Program Files\Microsoft Visual Studio Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:130772:50)
at Session.handlers.ts.createMapFromTemplate._a.(anonymous function) (c:\Program Files\Microsoft Visual Studio Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:130173:61)
at c:\Program Files\Microsoft Visual Studio Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:131637:88
at IOSession.Session.executeWithRequestId (c:\Program Files\Microsoft Visual Studio Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:131628:28)
at IOSession.Session.executeCommand (c:\Program Files\Microsoft Visual Studio Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:131637:33)
at IOSession.Session.onMessage (c:\Program Files\Microsoft Visual Studio Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:131659:35)
at Interface. (c:\Program Files\Microsoft Visual Studio Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:132950:27)
at Interface.emit (events.js:182:13)
at Interface._onLine (readline.js:290:10)
at Interface._normalWrite (readline.js:433:12)
at Socket.ondata (readline.js:149:10)
at Socket.emit (events.js:182:13)
at addChunk (_stream_readable.js:279:12)
at readableAddChunk (_stream_readable.js:264:11)
at Socket.Readable.push (_stream_readable.js:219:10)
at Pipe.onread (net.js:636:20)

I've tried the following:

Restarting typescript server
Disabling the inbuilt features extension
Restarting the machine

What I've not yet tried:

Upgrading to Insiders
Downgrading
Cleaning the TS Server cache (does such a thing exist?)
Copying all files over to a new folder

I'd rather not go to the hassle of changing edition unless an issue has actually been identified in this version.
Thanks!
 I've also tried troubleshooting from this issue.
#75430 - Followed instructions here to provide the error output above. I'm also missing other JS features like hovertext.
#17221 - Haven't tried these 2016 instructions yet, are they still relevant? Issue was supposedly fixed in 2017. Below is my jsconfig.json:

{
  "compilerOptions": {
    "baseUrl": "./app",
    "paths": {
      "components/*": ["./components/*"],
      "features/*": ["./features/*"],
      "modules/*": ["./modules/*"],
      "pages/*": ["./pages/*"]
    }
  },
  "exclude": ["build", "node_modules"]
}

Possibly duplicate of https://github.com/microsoft/TypeScript/issues/32073
Can you please try upgrading your workspace to use typescript@next by installing this extension. Using the latest VS Code insiders build, does that fix the issue?


Jilll soft lock room 116

Version
3.2.1
Seed
R712-QNUT-J4HH40M5Z9Y00030ZZZZZZZ5ZZZZZZZZZZZZZZZ20HZZZZZZZ
Scenario
RE 1 (Jill)
Player Character
JIll
What happened?
Seed: R712-QNUT-J4HH40M5Z9Y00030ZZZZZZZ5ZZZZZZZZZZZZZZZ20HZZZZZZZ
Unzip save file and load the game. Proceed to room 116. There is a green herb instead of shotgun on the wall. Pick it up and and proceed to room 115 to trigger cut scene. Barry won't come to save you. Only choice now is come back to room 116. But you can't place green herb back to the wall and door to room 115 now locked and you will be soft locked in room 116 now.
P.S.
I already reported similar issue with Chris few moths ago, but this problem still present. The room 116 should not contain any items except normal shotgun or broken one, because only they could be placed on wall back to unlock door to room 115 in case if cut scene has triggered there. I understand what this is require big code change but soft lock still present.
Jill_07_2024-01-06.zip

Why did you try and take the item if you didn't have the broken shotgun?


Service Connection & Schema Name - Test Auth

Test that auth providers still work and that they can be configured from the ingestion JSONs and in the airflow lineage

Tackle this after https://github.com/open-metadata/OpenMetadata/issues/3850


Wait for deployments during the orchestration time.

Additionally to other checks, we should also consider the orchestration
done once that the expected pods are running.
feature#deployment-stability

Still not working locally, removing review requests and adding WIP.

This has succeeded three times in a row in CI: http://jenkins.caasp.suse.net/blue/organizations/jenkins/salt/activity?branch=PR-461
(ignoring http://jenkins.caasp.suse.net/blue/organizations/jenkins/salt/detail/PR-461/10/pipeline, that has nothing to do with this change)


Accessing the index graph

Using the python library is it possible to access the underlying graph of the index or just have it returned in some simple format?

Sorry, it's not possible it. Closing as it's a duplicate of #386

Thanks for the quick response but I don't quite understand. Since the index  can be written to disc one should be able to interpret the resulting file (even if it takes some effort). Is there a description of the file format.

Should be doesn't mean somebody implemented it, sorry. There is no description, one needs to read source code. It is probably easier to add Python-API function to export the graph.

OK, thanks. I may try.
On Tue, Feb 16, 2021 at 12:08 PM Leonid Boytsov<EMAIL_ADDRESS>wrote:

Should be doesn't mean somebody implemented it, sorry. There is no
description, one needs to read source code. It is probably easier to add
Python-API function to export the graph.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nmslib/nmslib/issues/473#issuecomment-779980187, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACMSGDAJ3BMXJAVRH6OZYKLS7KRA3ANCNFSM4XWV64JA
.

--
Maria Chikina
University of Pittsburgh
Computational and Systems Biology
Phone:<PHONE_NUMBER>

It may actually be easier to just requery every vertex. If you are interested in just a k-NN graph, it's definitely a better solution, b/c the stored graph is pruned. Also, please, note that you likely need only ground-layer graph, higher layers can be ignored.

I actually really do want to inspect the different layers. Just to
elaborate, I think HNSW may be useful for reducing the number of SGD steps
in an optimization problem that depends on pairwise distances and I wanted
to use nmslib as a quick shortcut. Thanks for the quick and helpful
responses.
On Tue, Feb 16, 2021 at 12:33 PM Leonid Boytsov<EMAIL_ADDRESS>wrote:

It may actually be easier to just requery every vertex. If you are
interested in just a k-NN graph, it's definitely a better solution, b/c the
stored graph is pruned. Also, please, note that you likely need only
ground-layer graph, higher layers can be ignored.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nmslib/nmslib/issues/473#issuecomment-779998727, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACMSGDBJFWUQBHW6YBNQ6TLS7KT7FANCNFSM4XWV64JA
.

--
Maria Chikina
University of Pittsburgh
Computational and Systems Biology
Phone:<PHONE_NUMBER>

Maybe, it's also easier to do in HNSWLIB, it could have a cleaner implementation.


Fix bundle install running twice when you change your Gemfile

This stems from having to generate webpacker packs for our test environment which means our webpacker image needs the complete Rails app available since the tests themselves end up running in the webpacker container. That's why it runs twice. Once for that and then again for the regular web image.
I know this kind of stinks because running bundle install takes a long time with Docker since if you change 1 dependency all dependencies need to be re-installed since that's how Docker layers work, and with this current set up we get dinged twice.
If we don't use a multi-stage build then our final Docker image is going to be over double in size since it'll include the entire Node runtime + all of the package.json libraries which isn't good.
Also we would still get hit with other delays, because if everything is in 1 image we have to individually run yarn install and bundle install, which means if you did yarn first it would then invalidate Docker's cache for bundle which means every time we change our package.json file our gems would get re-installed too. So moving to 1 image isn't ideal either.
I'm open for ideas on how to solve this.

How about copying bundler dependencies from webpacker like you do for public assets?
something like this would work? I mean once its copied, running bundle install again should be pretty fast.
COPY --chown=ruby:ruby --from=webpacker /app/vendor/bundle /vendor

Hey, that's a good idea.
I don't think we would even need to run it again if we copy over /usr/local/bundle. Do you see a reason why we would have to?
I tested it here and the COPY step took a second to copy the files and afterwards the project was able to be upped successfully and the tests are passing.
I ran both a run bundle:install and run bundle:update. Also manually upped the project and ran run cmd bundle install which finished very quickly. In all cases the correct thing happened and it was fast.

Thats great. On our project though (not using docker yet), when I restore the bundler dependencies in semaphoreci, it doesn't seem to work without running the command again, must be some other issue.

I wonder if that's because they're using a different CPU architecture or distro where it's not as simple as copying files over and new files need to be created from scratch but most of it works so it's still fast?
I just pushed it (without running it twice), let's see what happens. If it turns out to be broken in some case it can always be patched to add in the 2nd run.


List conflicts stability and e2e tests improvements

[x] Remove LiveList key duplication between children and list
[x] Fix Local only key collision edge case
[ ] e2e tests infrastructure to reproduce conflict consistently
[ ] LiveList.set conflicts
[ ] LiveList.clear updates

@ofoucherot @GuillaumeSalles There were quite some merge conflicts between this branch and my latest refactorings from main — sorry for the trouble! I spent some time resolving conflicts (see 5175728c67add969f7c1f4b9e5332a542effe4f9) and fixing resulting type issues for places that were anys before but now correctly point out problems (see e545ac364a5c758ed0ae25a3cbcc99f95e82422f).


Creating partial for whitelist

We should create a partial snippet of code for the whitelist, so that we do not have to make whitelist code changes in 3 places but rather only in the one place.
Similar partials we currently use:
https://github.com/grctest/Gridcoin-Site/tree/master/source/layouts/partials

working on implementation

I've removed the whitelist from the index page and the guides, it remains solely within the dedicated whitelist page. We could still make a partial so that it can be reused in the future though.

Created a partial for the whitelist:
https://github.com/grctest/Gridcoin-Site/blob/master/source/layouts/partials/_whitelist.erb


Expand World creation process + Add WorldProperties

This PR expands the process for creating worlds.
Notable new classes:

WorldBuilder This is a builder which can produce instances of WorldCreationSettings or Worlds directly.
WorldCreationSettings Holds information which is used to define a new world for creation.
WorldProperties defines numerous properties about the world (essentially a representation of the data with the level.dat file).
GeneratorType This is a representation of what you may also think of as a world type, it holds a generator and some settings. Examples are flatland or amplified.

Looks great.
@dumptruckman Can you review this PR and make sure it provides everything you need for MultiVerse?

Oh this also appears to close #479

In order to support the separation of creating a world and loading it I'd like to create a representation of a World which is not loaded (similar to how User works). Which I called WorldData (needs a better name) and is now extended by World with several of the data related methods moved over.
Thoughts? @bloodmc @dumptruckman


Adding Status and other Role models

Added initial Status model info with properties.

A few general questions here:

Should the build-in event methods be included in docs such as beforeSave, beforeValidate...etc?  How about getters and setters and validation?  Looking at the Member example that was there before I started, I would assume events would be simply listed, but get/set would be included?
Are the class names documented correctly?
Some properties are required when creating a model, so I listed them under "Required" but I wonder if this might confuse people when getting a model.  Thoughts?
I'm fairly stuck on understanding the custom field models.
In all the examples I include a validate step.  Should this stay in?
Should services be referenced?  For example, with Log models, there is a legacy service to add logs that works pretty well and has less code.  In these docs, should that be noted as a recommended way to add logs?
In some models when I was testing examples, the Site ID was required, but wasn't always listed as required in the model file.  Should this be noted in docs or filed as bugs?  If there is only one site, should Site ID always be included?

To elaborate on custom fields, I think having simple example for relationships, grid and fluid would be helpful.  However I couldn't quite figure this out.


User login

Value Proposition
As the site owner, I want only authenticated users to be able to create tutorials to limit usage.
Description

Acceptance Criteria

[ ] the home page displays a login button
[ ] once clicked, the user gets the possibility to log in with a Github account
[ ] the creation of a tutorial is only possible after logging in
[ ] trying to reach the create page while not logged in shows a message Please log in

Tasks

[ ] create a new branch
[ ] create new page home
[ ] create navItem for new page and add to navigation
[ ] add headline Tutorial maker
[ ] add two buttons for Login and Library
[ ] create routing for Library button to show library page
[ ] install nextAuth
[ ] setup nextAuth
[ ] only allow writing to database after user authentification
[ ] write user's email address into database when creating a new tutorial (author<EMAIL_ADDRESS>

Looks great and easy to understand!


Add options arbitrary options argument

Currently there are options available in the scryfall api like "unique" which allows you to get different versions of cards. If I'm understanding correctly, it seems there is no way to add in these extra options.
I'd like to add an extra paramater accepting an object of arbitrary parameters that will be merged with the request object. For example, instead of search sending just
{q: search} it would send {q: search, ...otherOptions}
Would you accept this PR?

Sure, I didn't even realise that was a thing. I'm assuming it's all the stuff on this page? I have a bit of free time, I can implement it today probably. Unless you've already started a PR

I haven't started, that would be great.
--
John Paul Penaloza
https://johnpyp.net
978-242-4651
On Wed, May 30, 2018 at 7:21 PM Mackenzie McClane<EMAIL_ADDRESS>wrote:

Sure, I didn't even realise that was a thing. I'm assuming it's all the
stuff on this page https://scryfall.com/docs/api/cards/search? I have a
bit of free time, I can implement it today probably. Unless you've already
started a PR
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Yuudaari/scryfall-sdk/issues/3#issuecomment-393350484,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATq45Mbpy6mQqm9yhSuycP_gY0pCdNdvks5t3ymCgaJpZM4UTwrU
.

Implemented as of v1.2.0

Works awesome, thanks a lot.
This is a very clean API, makes working with Scryfall a lot easier.
I want to try to make a general purpose, opinionated library for working with magic card data in general, using the scryfall API.
It would include adapters for database/memory caching abstracted, and easy price lookups, that sort of thing, in an attempt to make it even easier to interface with magic cards. I imagine it being used as a pain-free way of bootstrapping collection management, deck statistics, and trading sites without having to manually implement a lot of the methods you would need.
I'm thinking things like
mtg.getPrices(searchQuery | set | listOfCards),
mtg.priceHistory(card),
mtg.ev(set | product)
and various utilities like a string parser and better json support for searching e.g.
mtg.find('blue with cmc 3+') // q: 'c:u cmc>=3'
mtg.find({color: 'blue' || 'u', cmc: '3+' || x => x >= a3
and utility parsers/helpers like
mtg.collection.remaining(myCards, targetDeck) // {priceRemainingUsd: 100, cardsRemaining: ListOfCards, ...etc}
mtg.genHtml(cardObject || mtg.find('Karn, Scion of Urza'))
// Karn, Scion of Urza + a tooltip for art
Is this a project you would be interested in collaborating in?

Probably not, I actually don't even really play magic anymore, I just maintain this sdk because I want to have something well-maintained for my portfolio, lol...
You can feel free to send me a link or ask advice or whatever, I just probably won't have time to commit to another project


issue "Preparing submission... this may take a few moments.." never run just in submitting stage

how to solve this issue?

it keeps on submitting status and never changes ,even the log file is empty

I have not published it yet , I am testing it before publishing it

I see that the default queue is behaving correctly on our end, other competitions can run submissions without problems. I also ran a submission on my own test competition and everything worked correctly.
Maybe the problem comes from the way your competition is configured ? Are you using the default docker image for competition or a custom one ?

It still same issue , there are no logs at all and  the status is submitting

Can you add me as an organizer ? I'll try to do some submissions to see if mine works.
My username is obada

sure

I did ant it works

I ran your predictions.zip and it also got past the Submitted status. This looks like a stranger issue...
What browser are you using ?

chrome

I tried submitting on Chrome, and it also worked fine. Maybe you can try using Firefox, but it shouldn't change much since you can submit on my Iris test competition...
I can't add it to the leaderboard because the submission Failed (time limit exceeded), but it did finish.
As Organizer, you can check the status of all submissions on your competition by clicking on the Submission button under the competition title:

You will my submissions attempts and status, as well as have some actions (like putting them on the leaderboard, rerunning it etc...)

I reran yours again, but it is still the same. Anyway, if others are able to run it, then no problem. I have increased the time; can you resubmit to ensure the scoring file will run? Because up until this point, it has not run yet

I submitted it again.
I see that you wrote about the scoring.py earlier. Is that supposed to be in the predictions.zip file ? The one I downloaded only contains a .csv file inside the .zip.

is there a specific format for that file? can you double check it ?

You can find examples here : https://github.com/codalab/competition-examples/tree/master/codabench
The wiki also has some guides to help you out : https://github.com/codalab/codabench/wiki#2-organizers
I will bring your issue with the team on our next meeting, hopefully I will have more information for you afterward.
Concerning your current file, the worker log might give you more information as to what is missing :

I tried to fix them but i am having this issue now .Is there any constraints to name the files ?  or does the submitted file  pass with its name ?

It seems that the submitted file does not trigger.

It seems like the problem comes from your scoring program.
I suggest taking a look at the Iris scoring program : https://github.com/codalab/competition-examples/blob/master/codabench/iris/bundle/scoring_program/score.py
You can submit results for this competition (example : https://github.com/codalab/competition-examples/blob/master/codabench/iris/sample_result_submission.zip) , which is the same thing you are trying to do.
Maybe the path in which your scoring program is trying the find the data is wrong ?

yes this is the issue. How should I deal with paths? like the submission file after it submitted how to refer to it in my [ingestion.py] file ? and after that how to pass the result to the scoring .. I tried to follow the examples but it is sill the same issue

@SondosBsharat
Did you change path in the scoring and/or ingestion programs?
Please check the examples pointed earlier in the discussion and see if your path are different (inside scoring.py, metadata, etc.)
Another possibility is that the problem comes from zipping with directory structure.
Basically, when zipping submissions, programs, or data, you should zip the files directly, without directory.
Can you try this?
You can update your programs following this Wiki page: https://github.com/codalab/codabench/wiki/Resource-Management

These are the links in my scoring file. Are they correct? Previously, I included an ingestion file, but then I removed it since I only need the participants to submit their prediction results. However, after removing it, the submission stays in the running state until the time exceeds, or it continues running if I increase the time limit.:  f name == "main":
try:
# Set up directories
input_dir = sys.argv[1]
output_dir = sys.argv[2]
reference_dir = os.path.join(input_dir, "ref")
prediction_dir = os.path.join(input_dir, "res")
    prediction_file = os.path.join(prediction_dir, "predictions.csv")
    ground_truth_file = os.path.join(reference_dir, "ground_truth.csv")

    # Logging setup
    logging.basicConfig(
        filename=os.path.join(output_dir, "scoring.log"),
        level=logging.INFO,
        format='%(asctime)s - %(levelname)s - %(message)s'
    )

    # Validate files
    if not os.path.exists(prediction_file):
        raise FileNotFoundError(f"Prediction file not found: {prediction_file}")
    if not os.path.exists(ground_truth_file):
        raise FileNotFoundError(f"Ground truth file not found: {ground_truth_file}")

    # Evaluate predictions
    results = evaluate_predictions(prediction_file, ground_truth_file)

    # Write results
    write_results(results, output_dir)

except Exception as e:
    error_message = f"Error occurred: {str(e)}"
    logging.exception(error_message)
    with open(os.path.join(output_dir, "error.log"), 'w') as error_file:
        error_file.write(error_message)

It seems there is an issue because even if I try to include a print statement at the top of the scoring file, it does not print anything, so it is not being accessed


Consider patch release branch: Gherkin glitches

Fixes for inconsistencies in hook and configuration behaviour with Gherkin Syntax

@ashleyfrieze I will merge this, but there's a small conflict with master now. Sorry I let this PR sit out here way too long!


TFLiteConverter StridedSlice Error for Transformer Example Notebook

System information

Linux Ubuntu 18.04.2
Installed from binary
TensorFlow version 2.0.0

Provide the text output from tflite_convert
---------------------------------------------------------------------------
ConverterError                            Traceback (most recent call last)
<ipython-input-82-0b556551beac> in <module>()
      3
      4 converter = tf.lite.TFLiteConverter.from_concrete_functions([to_save])
----> 5 tflite_model = converter.convert()

~/anaconda3/envs/main/lib/python3.6/site-packages/tensorflow_core/lite/python/lite.py in convert(self)
    444         input_tensors=input_tensors,
    445         output_tensors=output_tensors,
--> 446         **converter_kwargs)
    447
    448     if self._is_calibration_quantize():

~/anaconda3/envs/main/lib/python3.6/site-packages/tensorflow_core/lite/python/convert.py in toco_convert_impl(input_data, input_tensors, output_tensors, enable_mlir_converter, *args, **kwargs)
    447       input_data.SerializeToString(),
    448       debug_info_str=debug_info_str,
--> 449       enable_mlir_converter=enable_mlir_converter)
    450   return data
    451

~/anaconda3/envs/main/lib/python3.6/site-packages/tensorflow_core/lite/python/convert.py in toco_convert_protos(model_flags_str, toco_flags_str, input_data_str, debug_info_str, enable_mlir_converter)
    198       stdout = _try_convert_to_unicode(stdout)
    199       stderr = _try_convert_to_unicode(stderr)
--> 200       raise ConverterError("See console for info.\n%s\n%s\n" % (stdout, stderr))
    201   finally:
    202     # Must manually cleanup files.

ConverterError: See console for info.
2019-10-22 12:59:50.304931: I tensorflow/lite/toco/graph_transformations/graph_transformations.cc:39] Before Removing unused ops: 915 operators, 1487 arrays (0 quantized)
2019-10-22 12:59:50.337611: I tensorflow/lite/toco/graph_transformations/graph_transformations.cc:39] Before general graph transformations: 915 operators, 1487 arrays (0 quantized)
2019-10-22 12:59:50.397450: F tensorflow/lite/toco/graph_transformations/resolve_strided_slice_attributes.cc:95] Check failed: start_indices_size <= num_input_axes (4 vs. 2)StridedSlice op requires no more than 2 start indices
Fatal Python error: Aborted

Current thread 0x00007f2d4bb1e740 (most recent call first):
  File "/home/mattc/anaconda3/envs/main/lib/python3.6/site-packages/tensorflow_core/lite/toco/python/toco_from_protos.py", line 52 in execute
  File "/home/mattc/anaconda3/envs/main/lib/python3.6/site-packages/absl/app.py", line 251 in _run_main
  File "/home/mattc/anaconda3/envs/main/lib/python3.6/site-packages/absl/app.py", line 300 in run
  File "/home/mattc/anaconda3/envs/main/lib/python3.6/site-packages/tensorflow_core/python/platform/app.py", line 40 in run
  File "/home/mattc/anaconda3/envs/main/lib/python3.6/site-packages/tensorflow_core/lite/toco/python/toco_from_protos.py", line 89 in main
  File "/home/mattc/anaconda3/envs/main/bin/toco_from_protos", line 10 in <module>
Aborted (core dumped)

Any other info / logs
I am following this notebook: https://www.tensorflow.org/tutorials/text/transformer
I am interested in doing NMT on an Android device with TFLite. The NMT Attention notebook uses an LSTM which is not supported yet for TF Lite conversion so I am looking into this approach that only uses Attention, Embeddings, and Dense layers in hope that it is convertible.
I haven't edited the code from that notebook at all, but created a new inference concrete function and am trying to convert it. I am getting this vague error about the StridedSlice op.
Looking for some guidance on what I can do from here.
tf.random.set_seed(1234)

eval_step_signature = [
    tf.TensorSpec(shape=(BATCH_SIZE, 64), dtype=tf.int64),
    tf.TensorSpec(shape=(BATCH_SIZE, 26), dtype=tf.int64),
]

@tf.function(input_signature=eval_step_signature)
def eval_step(inp, tar):

    enc_padding_mask, combined_mask, dec_padding_mask = create_masks(inp, tar)

    predictions, _ = transformer(inp, tar,
                                 True,
                                 enc_padding_mask,
                                 combined_mask,
                                 dec_padding_mask)
    return predictions

ckpt.f = eval_step
to_save = ckpt.f.get_concrete_function()

converter = tf.lite.TFLiteConverter.from_concrete_functions([to_save])
tflite_model = converter.convert()

Thank you!

I have also encountered the same problem. Looks like it is from create_padding_mask function.

Hi @mattc-eostar ,
I could resolve the issue with:
ckpt.f = eval_step
to_save = ckpt.f.get_concrete_function()

converter = tf.lite.TFLiteConverter.from_concrete_functions([to_save])
converter.target_spec.supported_ops = [tf.lite.OpsSet.TFLITE_BUILTINS,
                                       tf.lite.OpsSet.SELECT_TF_OPS]
tflite_model = converter.convert()

Thank you.

@mattc-eostar
Can you please check and confirm whether your issue was resolved by adding below 2 lines of code before converting
converter = tf.lite.TFLiteConverter.from_concrete_functions([to_save])
converter.target_spec.supported_ops = [tf.lite.OpsSet.TFLITE_BUILTINS,
                                       tf.lite.OpsSet.SELECT_TF_OPS]

Please, close this thread if your issue was resolved.Thanks!


Update evaluate-model.md

@chjinche : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Fix #26

Replace y = forward(x) with y = implicit(x) in the chain rules to allow higher-order derivatives

maybe add a test for higher order derivatives using ForwardDiff and the linear solver I defined in the issue?


The ""Rancher logging target setting validated" log message is not seen in fluentd server after clicking test

What kind of request is this:
bug
Steps to reproduce:
This is only happening for me in master-head.
v2.4-head (07/16/2020) e28427e4a I get the message properly in the fluentd server.

Create cluster either RKE or import k3s
Navigate to Tools -> Logging -> Fluentd
Add the required settings in the fields and hit TEST

Result:
The button will be turn Green with Settings validated message,
But if we look at the logs in the fluentd server there won't be settings validated message.
Like:
rancher: {"message":"Rancher logging target setting validated"}
Clicking then Save after some time I see the logs flowing to the target server. So the settings worked.

Moving this to 2.4.7, shouldn't be in 2.5 as this is a 2.4.x bug.
Is this similar to https://github.com/rancher/rancher/issues/26543 ?

This is different because the message is in the fluentd target server logs.
In the dry run issue the fluentd target server returns a 500 iirc but the Rancher UI shows "settings are ok"

Fluentd target configuration
fluentd:v1.7.4-debian-2.0
<source>
  @type forward
  port 9881
  bind <IP_ADDRESS>
</source>
<match **>
  @type stdout
</match>

Logging V1 is no longer supported


Added getDeviation(), getFolderDeviations() and getGalleryFolders() methods

These 3 methods allows to browse through an user's gallery
I tried to "npm install" then "npm run test" but it gives me :

./node_modules/mocha/bin/mocha
Client

"before all" hook

Client generation
Client Credentials
✓ should return a Client instance (354ms)
Implicit
2) should return a Client instance
1 passing (5s)
2 failing

Client "before all" hook:
Uncaught TypeError: Cannot read property '1' of null
at Request._callback (index.js:63:77)
at Request.self.callback (node_modules/request/request.js:187:22)
at Request. (node_modules/request/request.js:1044:10)
at IncomingMessage. (node_modules/request/request.js:965:12)
at endReadableNT (_stream_readable.js:913:12)
at _combinedTickCallback (internal/process/next_tick.js:74:11)
at process._tickCallback (internal/process/next_tick.js:98:9)

Client generation Implicit should return a Client instance:
Uncaught TypeError: Cannot read property '1' of null
at Request._callback (index.js:63:77)
at Request.self.callback (node_modules/request/request.js:187:22)
at Request. (node_modules/request/request.js:1044:10)
at IncomingMessage. (node_modules/request/request.js:965:12)
at endReadableNT (_stream_readable.js:913:12)
at _combinedTickCallback (internal/process/next_tick.js:74:11)
at process._tickCallback (internal/process/next_tick.js:98:9)

I would add tests for these methods if you can help me with this :)

Hello @romaindurand, sorry for the very late answer I just discovered your PR. :( I'll keep a closer eye on my notifications from now on. ;)
I haven't touched this project in a while and it seems the Travis configuration is broken, hence the red build. You code looks fine to me so I'll merge.
Thanks a lot for contributing!


Failed to do a feepayment

signer output:
INFO [05-12|07:40:43.035] checking if transaction exists           memo=bf45534a8319f433ef3dfa2db0b6f411e06148a1be7911ddbe359956b46dc905
INFO [05-12|07:40:43.035] scanning account                         account=GARQ6KUXUCKDPIGI7NPITDN55J23SVR5RJ5RFOOU3ZPLMRJYOQRNMOIJ
INFO [05-12|07:40:43.035] Start fetching stellar transactions      horizon=https://horizon.stellar.org/ account=GARQ6KUXUCKDPIGI7NPITDN55J23SVR5RJ5RFOOU3ZPLMRJYOQRNMOIJ cursor=198585569642311680
INFO [05-12|07:40:43.132] checking if transaction exists           memo=bf45534a8319f433ef3dfa2db0b6f411e06148a1be7911ddbe359956b46dc905
INFO [05-12|07:40:43.132] Transaction with this hash already executed, skipping now..

master output:
INFO [05-12|07:40:53.336] checking if transaction exists           memo=bf45534a8319f433ef3dfa2db0b6f411e06148a1be7911ddbe359956b46dc905
INFO [05-12|07:40:53.336] transaction not found
ERROR[05-12|07:40:53.436] failed to get signature from             err="failed to connect to host id '12D3KooWLEdJsPFe3utrnc3teHJvS9oN9MfuBWRW9hxmoEGZGVfm': failed to dial 12D3KooWLEdJsPFe3utrnc3teHJvS9oN9MfuBWRW9hxmoEGZGVfm:\n  * [/p2p/12D3KooWGttLPcYHMGLep8F9G2xAxM1wLT3JvraXhV2z9BeYmmok/p2p-circuit] dial backoff"
ERROR[05-12|07:40:53.436] failed to get signature from             err="transaction with hash already exists"
ERROR[05-12|07:40:53.537] failed to get signature from             err="transaction with hash already exists"
ERROR[05-12|07:40:53.537] error sending fee to the fee wallet      err="required number of signatures is not met"

963eb7fa21c73b59b30f5b0640f93d9853e8628e


GEOPY-358: fix start-app in dev environment

GEOPY-358 - use local editable geoh5py in geoapps dev environment

Tested - GTG


build of distmqtt without distkv?

Hi,
Did you mention in python-trio general about making a build of distmqtt that isn't built with distkv? Is that still in the works?
Thanks
Charles

That boils down to adding an option to the requirements in setup.py. I'll do that sometime soon, if nobody beats me to it with a three-line pull request. ;-)


add links to the gifs in README.md file

Description
I think these two gifs (see the below image) should contain a links to the related files because there are the chances that anyone can click on them and these are of no use without them.

Solution
I want to add links to these gifs so that if anyone click on them it will take them to the related web pages where these files are present.

@smaranjitghose sir should I make these changes??

Not relevant


All issues related to icons.

General issues: #35683 , #35783

APP (Desktop, taskbar, start or context menu issues):

[x] Pinning vscode to the taksbar changes it's colour from orange to blue 1.17 #35663

[x] Open with code in context menu still old icon #35747

[x] cmz and insiders have similar icon. https://github.com/Microsoft/vscode/issues/35683#issuecomment-334757991 (stable and insiders are two different colors)

[x] Providing different icon for the TouchBar #35641  (this isn't a logo/icon issue)

[x] Icon did not update in Windows search bar #33685 (caching problem)

[x] The lack of visual distinctness with Sublime is not great. As noted on https://github.com/Microsoft/vscode/issues/6607#issuecomment-334792687. ... https://github.com/Microsoft/vscode/issues/35683#issuecomment-334831978  (we're changing the icon from orange to blue, this should make it distinct again)

[x] Stable should be green, Insiders orange (swap the icon color?) #35674 (we will keep Insiders green and make Stable blue. this is how it was prior to the new icon design)

[x] inability to distinguish between stable and insiders https://github.com/Microsoft/vscode/issues/35672 (this should be much better with green and blue)

[ ] New macOS icon is too small (tracking here: https://github.com/Microsoft/vscode/issues/36692)

[ ] Allow to change Visual Studio Code icon #26839

[x] Missing Task Bar Icon #34106 (cache problem)

[ ] It is difficult to distinguish VSCode icon on Linux Mint task bar. https://github.com/Microsoft/vscode/issues/35683#issuecomment-334994102  (blue should help here, but keeping open for now to verify)

[x] If vscode is notified in the windows task bar, it looks strange because the colors are similar. https://github.com/Microsoft/vscode/issues/35783#issuecomment-335351345 (related to the fact that the icon on windows is of a single color.  This is a limitation of the guidelines, which we are working to fix, so that we can then fix the icon)

[x] new icon blends on an orange background (transparent parts of icon cause it). #36657  (this is a limitation of the single color requirement. we need to change the branding guidelines and then we can fix the icon for this case.)

Installer:

[x] Installer Icon on Windows #33603  (upstream (Inno Setup) issue, nothing we can do on our side, same problem existed with previous icon)

Files (associated with vscode, icons in "lists" (in settings / controll panel etc.)):

[x] Files associated with VSCode had old icon of VSCode https://github.com/Microsoft/vscode/issues/35683#issuecomment-334734135 (this is a caching issue, probably not an issue now.)

Website:

[x] [website] favicon is not updated #35708

That's a quite useful list for the team! :)

-- Sorry for the English, I used Google Translator --
@Mlocik97-issues I think you can relate this question ( #33685 ) as well. No?

Yes, this is a helpful issue. I'm editing the checklist above.

I updated the list.

We're changing the orange to blue, you can read more here: https://code.visualstudio.com/blogs/2017/10/24/theicon

Nooooooooooooo @chrisdias  :(
I LOVED IT. Let's make a petition of introducing a setting for controlling the icon. 😋

@chrisdias Could we not have yet another blue icon.  I think Microsoft doesn't need every icon to be blue or some derivative of blue.  There's what, Access (red) and Powerpoint (orange).  Everything else is either blue, purple, or green.

@chrisdias: please keep the orange icon, this way it is visually distinct from Visual Studio icon. When both icons are blue or violet with similar shape, they are very confusing if both programs are installed on the same machine.

For me VS orange-stripes logo is confusing with SublimeText logo ((

I was quite pleased with the introduction of the tilted icon on macOS builds, as it feels like it better adheres to the Human Interface Guidelines for application icons on macOS. You can see the section about "Perspective and Textures" mentions the tilt.
https://developer.apple.com/macos/human-interface-guidelines/icons-and-images/app-icon/
You can see it looks quite consistent in the example below. I hope you will consider keeping the tilt.

The only issue I have is there isn't as much contrast when the Dock is set to dark mode, but it's only a minor complaint.

Ah, the new logo works well with my theme though, anyway, the logo color doesn't really matter for me, thx for your contribution

biggest comment I have on the new icons (including oct release) is that it is so hard to spot or make out of a crowded taskbar.
The old one stood out, and was easy to spot and go for

The blue icon is a bit problematic on windows when it is asking what program you want to open a file with. The background color is the same blue as the logo

Is there any way to get the Windows icon on Ubuntu instead of the macOS one? The tilted icon looks great on macOS but clashes with everything else on Ubuntu and is too dark and detailed to work at smaller dock sizes, especially when using a dark theme (like the Ubuntu default). This has been the case for the Unity desktop before and is still the case with the new Gnome desktop in 17.10.

I'm guessing a poll on blue or orange is out of the question?
🔸 😉

First, this is my first time commenting on github, so I apologize if I screw this up.
Second, I love Visual Studio Code, recommend it to everyone, and has become my default text/code editor on all platforms (Window, Ubuntu, and OSX). It has made my life so much easier to have a non-java based, cross platform, seamless user experience, across all platforms. This allows me to navigate clients through administering P.O.S. application config changes.
Now the Issue, I will agree with everyone above I didn’t like the orange. Not because it was a change, but because it stands out almost as an eye sore (the neighbor bringing down everyone’s property values). Then with the new blue it is basically invisible on the Windows 7 Taskbar with the default Aero Theme and Background. I haven’t checked Windows 8 yet (I’m sorry I refuse to use Windows 10 until Microsoft figures out what they are doing with the Windows Embedded Platform). Most of the custom applications I deploy I can configure the Icon that is to be displayed with that application prior to build. I can either build it into the EXE or point it to a location. I haven’t looked at the source of Code to see how this is handled, but it could be dynamic setting within the application with a couple of options that would be fine. It might be simpler to point it to a file that could be changed for the desired color. I know it sounds lazy but when I deploy this application on hundreds of systems, I avoid altering Windows as much as possible from the default, it also helps people from irrationally thinking the system is somehow not the same windows they are used to using.
I have attached some screen shots.
Windows 7 Taskbar with the default Aero Theme and Background

Windows 7 Taskbar with the default Aero Theme and different Background

To help understand what I am talking about.
https://social.msdn.microsoft.com/Forums/vstudio/en-US/9e23a82c-8bed-4b96-8b9a-4c2b6136a622/changing-the-icon-of-exe-files-programatically?forum=csharpgeneral
https://msdn.microsoft.com/en-us/library/339stzf7.aspx


Remove keepalived

What are your changes?
Remove keepalived
What type of changes does your doc introduce?

[ ] Bug fix
[x] New feature

What is the urgency level of this change?

[ ] Blocker
[x] High
[ ] Medium

I have completed these items:

[x] I have tested all commands and code snippets.
[x] I have built locally and tested for broken links and formatting.
[x] I have made this change for all affected versions (e.g. 1.7, 1.8, and 1.9).
[x] My doc follows the style guide: https://github.com/dcos/dcos-docs#contributing.

Ping emanic, sascala, or joel-hamill for reviewing pull request changes.
If you included a comment with your commit, it appears here:

@sascala ready for merge


Language chooser cannot switch language for untranslated page

Language chooser uses page_language_url from menus.templatetags. The final except solution for this template tag is to provide no URL if no translation could be found. For untranslated django-cms pages, this blocks a visitor from switching his language. I would rather redirect those users to the translated homepage then to block them from switching.
The bug arises from here. I created a temporary workaround by changing that line to this:
# no localized path/slug.
url = "/" + lang + "/"

ojii solved this issue for me by working around the default templatetag. If django-cms would include his workaround, that is saving the page url to a variable, would allow me to create a custom template. See his answer on stackoverflow.

I just stumbled upon this problem after setting CMS_HIDE_UNTRANSLATED to True. According to the docs I would expect, that it redirects to the start page, see here http://docs.django-cms.org/en/latest/advanced/i18n.html#cms-hide-untranslated


fix: small typo error

@0xflotus thanks for the PR, much appreciated! Please ignore the build failure. Currently it fails for all PRs from forks.


Use default path of Toolchain

The installation instructions does not use the default installation path of the SDK which is /opt/pxc/2019.3

This would help getting started faster.

Yeah, it would just be great if you can just copy the commands and start. No need for looking for correct paths

Closing in favour of the more general issue at #7.


added note about keeping things folded

What do you think about this one @ranjitjhala ? Is this something you stress in class?
For some exercises, things get very messy if they don't do this.
I'm not sure my phrasing will be understandable to them though (like, the verbs "fold/unfold").

👍 This is  a great note to add -- it is something I should emphasize (have never really taught this stuff before in this "formal" way ...)
Thanks!


InvalidOperationException: ContainerControlledLifetimeManager can only be set once being thrown

Description
Unity.Container 5.11.[4|5] throws an InvalidOperationException resolving a Microsoft.Extensions.Logging.Logger<> which is mapped using the AddLogging extension method, and then registered in Unity using the Unity.Microsoft.DependencyInjection module.
This is working using Unity.Container 5.11.1 with a custom build of Unity.Microsoft.DependencyInjection that targets 5.11.1 with this fix (https://github.com/unitycontainer/microsoft-dependency-injection/pull/64)
System.InvalidOperationException: ContainerControlledLifetimeManager can only be set once
   at void Unity.Lifetime.ContainerControlledLifetimeManager.SetValue(object newValue, ILifetimeContainer container)+(object o, ILifetimeContainer c) => { }
   at void Unity.Lifetime.ContainerControlledLifetimeManager.SetValue(object newValue, ILifetimeContainer container)
   at void Unity.Strategies.LifetimeStrategy.PostBuildUp(ref BuilderContext context)
   at Unity.UnityContainer(UnityContainer parent)+(BuilderStrategy[] chain, ref BuilderContext context) => { } [3]
   at object Unity.Builder.BuilderContext.Resolve(Type type, string name, InternalRegistration registration)
   at object Unity.Builder.BuilderContext.Resolve(Type type, string name)
   at object Unity.Builder.BuilderContext.Resolve(ParameterInfo parameter, object value)
   at ResolveDelegate<BuilderContext> Unity.Processors.ConstructorProcessor.GetResolverDelegate(ConstructorInfo info, object resolvers)+(ref BuilderContext c) => { }
   at ResolveDelegate<BuilderContext> Unity.Processors.MemberProcessor<TMemberInfo, TData>.GetResolver(Type type, IPolicySet registration, ResolveDelegate<BuilderContext> seed)+(ref BuilderContext c) => { }
   at ResolveDelegate<BuilderContext> Unity.Processors.MemberProcessor<TMemberInfo, TData>.GetResolver(Type type, IPolicySet registration, ResolveDelegate<BuilderContext> seed)+(ref BuilderContext c) => { }
   at ResolveDelegate<BuilderContext> Unity.Processors.MemberProcessor<TMemberInfo, TData>.GetResolver(Type type, IPolicySet registration, ResolveDelegate<BuilderContext> seed)+(ref BuilderContext c) => { }
   at void Unity.Strategies.BuildPlanStrategy.PreBuildUp(ref BuilderContext context)
   at Unity.UnityContainer(UnityContainer parent)+(ref BuilderContext context) => { } [2]
   at object Unity.UnityContainer.Unity.IUnityContainer.Resolve(Type type, string name, ResolverOverride[] overrides)
   at T Unity.UnityContainerExtensions.Resolve<T>(IUnityContainer container, ResolverOverride[] overrides)
   at IConsumerScopeContext<TConsumer, T> MassTransit.UnityIntegration.UnityConsumerScopeProvider.GetScope<TConsumer, T>(ConsumeContext<T> context)
   at async Task MassTransit.Scoping.ScopeConsumerFactory<TConsumer>.Send<TMessage>(ConsumeContext<TMessage> context, IPipe<ConsumerConsumeContext<TConsumer, TMessage>> next)
   at async Task MassTransit.Pipeline.Filters.ConsumerMessageFilter<TConsumer, TMessage>.GreenPipes.IFilter<MassTransit.ConsumeContext<TMessage>>.Send(?)

To Reproduce
TBD. I have not managed to created an isolated test case that demonstrates the issue. This is still a work in progress. Might have to try and build up a small representative application.
Additional context
The ILogger<> open generic is being resolved from a child container that is being created by the MassTranist.Unity package, which uses the child container to create a scope around processing a message from the bus.
I also noticed while debugging that the container that the BuilderContext.Container was the child container while resolving all dependencies, until it got down to resolving the ILogger<> , at this point the BuilderContext was the root container. The debugger was showing something like Container[255], where for all higher level dependencies it was showing Container[255].Child[0]

Duplicate #210


CEHorizontalSwipeInteractionController popOnRightToLeft is within method

CEHorizontalSwipeInteractionController method's
wireToViewController:(UIViewController *)viewController forOperation:(CEInteractionOperation)operation
has popOnRightToLeft = YES within method. So, whatever I change back to YES. So, I need to change the initial code. I think it be better we remove this line.
Thanks

统一楼上的!


Utility to verify if build configuration is OK

Please add a utility that will verify if build configuration is OK. For example, if the user will specify cppstd=20 for gcc-7 the builder will run and then conan will complain with the following message:
conans.errors.ConanException: The specified 'cppstd=20' is not available for 'gcc 7'. Possible values are ['11', '14', '17', '98', 'gnu11', 'gnu14', 'gnu17', 'gnu98']'

It would be nice to have the possibility to manually check and filter invalid configurations.

Hi @mpusz !
Actually, CPT only generates a matrix with all default settings and options. Conan is who checks if settings and options are supported.
To check if some the configuration is correct, we need to run Conan API, which means we will need to add conan info
What is your idea with this feature? I've used CPT for a long time, but I never thought about verifying before to build.

Hi,
Let's imagine that we want to multiplex the current build matrix containing common builds with all valid values of cppstd:
    builder.add_common_builds(pure_c=False)
    new_builds = []
    for settings, options, env_vars, build_requires, reference in builder.items:
        for std in ["98", "gnu98", "11", "gnu11", "14", "gnu14", "17", "gnu17", "20", "gnu20"]:
            new_settings = copy.copy(settings)
            new_settings["cppstd"] = std
            if not is_valid_configuration(new settings): # check if current compiler and its version supports that cppstd
                continue
            new_builds.append([new_settings, options, env_vars, build_requires])
    builder.builds = new_builds
    builder.run()

Doing is_valid_configuration() manually in every build.py all over again would be counterproductive and error-prone. That is why I suggest having such a tool in a library, especially that such logic is implemented already.


orientDb unable to get outgoing edges.

OrientDB Version:
compile group: 'com.orientechnologies', name: 'orientdb-graphdb', version: '2.2.11'

Java Version:
1.8
OS:
Linux/Windows
Expected behavior
vertex1.getEdges(Direction.OUT, relationshipId) returns the given relationship
Actual behavior
vertex1.getEdges(Direction.OUT, relationshipId) returns empty
Steps to reproduce
Fill Database with some nodes and edges.
Take one note and execute the method on it and require the outgoing edge.
Returns empty list for me.

Hi @Raycoms
This is a very basic operation, it's quite unlikely that it doesn't work in general.
Can you please post a script to reproduce it? Which value are you passing as relationshipId?
Thanks
Luigi

I tested vertex2.getEdges(Direction.IN, relationshipId) and it worked great.
I took the relationship I received, searched for an outgoing node and found vertex1.
It just didn't work the other way around.
Whole code of the project regarding orientDB:
https://github.com/Raycoms/thesis/blob/master/src/main/java/main/java/com/bag/server/database/OrientDBDatabaseAccess.java
It would be great if there was a method "graph.getEdge(startVertex, endVertex, type)"

Hi @Raycoms
From your code, I see the following:
final String relationshipId = "class:" + relationshipStorage.getId();

For edges, you don't have to prefix class names with "class:" (it's needed for addVertex() though), probably this is the reason why it doesn't work. Just try to replace it with the following:
final String relationshipId = relationshipStorage.getId();

Please let me know if it fixes the problem
Thanks
Luigi

I already tried that. I even added both versions in it.
Strangely, I got the correct return for the "IN" case when prefixing "class:" .

Hi @Raycoms
I could not manage to find anything strange in the code. It would be great if you could isolate the problem in a small test case.
Thanks
Luigi

I am working on my thesis this week. I might be able to get something isolated in the next 2 weeks.

Hi @Raycoms
Did you have a chance to review this? Does this issue still make sense?
Thanks
Luigi

Still happening, we have build the work around I wrote.
Will have to publish a paper soon so I unfortunately don't have the time to
further investigate this.
2017-05-02 10:20 GMT-03:00 luigidellaquila<EMAIL_ADDRESS>
Hi @Raycoms https://github.com/Raycoms
Did you have a chance to review this? Does this issue still make sense?
Thanks
Luigi
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/orientechnologies/orientdb/issues/7292#issuecomment-298633203,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGI9y-t5whQO1hZmJQsfpNRIGNi_2ByMks5r1y2LgaJpZM4Mw-V0
.

Hi @Raycoms
Does this issue still happen in latest version? Do you have any updates?
Thanks
Luigi

Hi @Raycoms
I'm closing this issue (no response for a long time), please feel free to re-open if it's still relevant
Thanks
Luigi


www/nginx: #1374 - Fixes matching values in URLs using naxsi rules.

This PR fixes #1374, locally verified with my own rules.
Since the form already allowed to specify the "URL" zone, this PR is a bugfix only.

Hi Fabian, yes that would be nice from the changes in the other PRs, I'm not entirely sure what I should change. Thanks a lot in advance.

@jkellerer FYI: https://github.com/opnsense/plugins/commit/77b89b53806601df385087e14fc8c66e99b0d597


fixed watch history bug and lint.yml

PR Fixes:

fixed the watch history bug where video is not playing through watch history
also fixed lint.yml file

Resolves #1418
fixed bug video
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I assure there is no similar/duplicate pull request regarding same issue

@devsargam reduced the no. of files changed

Super sorry actually just had to ask you to create pr after pr

yeah sure no problem


Expressions.bnode().getQueryString() produces NullPointerException

The issue is caused by the fact that Expressions.bnode() calls Expressions#function(SparqlFunction, Operand...) with null as an operand (Expressions.function(BNODE, (Operand) null)). Calling getQueryString will lead to calling getQueryString for all the operands and we get NPE since the operand is null.
I guess this can be fixed by removing the null argument from Expressions.function(BNODE, (Operand) null) call and setting printBodyIfEmpty to true for all Function objects.
I can create a MR, however I'm not sure where exactly I should put the unit test for the issue. The sparql11spec package contains only the examples from the specification and there's no example involving BNODE() function call in the spec. Should I create a separate class ExpressionsTest for the test case?

Related issue: #1385

@RoKish thanks for reporting this, and a Pull Request would be most welcome!
The SparqlBuilder currently has poor unit test coverage, we're aware of this (see also #1385). Your help with making a start would be great. A new class ExpressionsTest in src/test/java/org/eclipse/rdf4j/sparqlbuilder/constraints sounds like a good approach.
Before you get started with your PR, please have a look at the contributor guidelines if you haven't yet, in particular the points on how to sign the Eclipse ECA. Let me know if you need any help with any of it.

Nice work, thanks @RoKish!


Change LevelStore to Memory in documentation

On documentation website for GeoStore (http://terraformer.io/geostore/), under "Using in Node.js", the documentation still reads: var LevelStore = require('terraformer-geostore-leveldb').LevelStore;  There is no longer a LevelStore module...
Shouldn't that line read var Memory = require('terraformer-geostore-memory').Memory; ?
Holdover from a previous version?

it'd definitely be good to remove the reference to terraformer-geostore-leveldb on the website, but the last time i tried to get the ruby environment set up locally i had nothing to show for it at the end but headaches.