failed to fetch metadata token: metadata server responeded with code=500

Hi,
I have deployed two services in the same project to europe-west4 region. One of them is the 'curl' service coming from this repository and it has public access.
Second service is a very basic nodejs application that just returns static HTML. Service name is 'internaldocs'. If I make it publicly accessible, I'm able to get 200 response.
However, when I make the 'internaldocs' service allow internal traffic and traffic from Cloud Load Balancing only and trying to access it from 'curl' service by proving the 'http://internaldocs' url I get the following error;
$ curl -sSLNv --http2 http://internaldocs

*   Trying ::1:80...
* Connected to internaldocs (::1) port 80 (#0)
 GET / HTTP/1.1
 Host: internaldocs
 User-Agent: curl/7.76.1
 Accept: */*
 Connection: Upgrade, HTTP2-Settings
 Upgrade: h2c
 HTTP2-Settings: AAMAAABkAAQCAAAAAAIAAAAA

* Mark bundle as not supporting multiuse
< HTTP/1.1 101 Switching Protocols
< Connection: Upgrade
< Upgrade: h2c
* Received 101
* Using HTTP2, server supports multi-use
* Connection state changed (HTTP/2 confirmed)
* Copying HTTP/2 data in stream buffer to connection buffer after upgrade: len=0
* Connection state changed (MAX_CONCURRENT_STREAMS == 250)!
< HTTP/2 500
< date: Sun, 25 Apr 2021 18:22:07 GMT
<
{ [98 bytes data]
failed to fetch metadata token: metadata server responeded with code=500 500 Internal Server Error* Connection #0 to host internaldocs left intact

I believe I gave the right permissions to the services by following this document https://cloud.google.com/run/docs/authenticating/service-to-service#gcloud but obviously I'm missing something.
You think it might be a bug or misconfiguration?
Have a nice one!

What's your QPS? Is it like <1 or >30? I wonder if the metadata service is throttling you somehow.

It's less than 1

I assume you use VPC connector with this target service, correct?

Does it repro if you don't use a VPC connector but just make service internal by not giving allUsers access on IAM?

What happens when you put both services in the same VPC (assuming answer to 1 is yes)? Maybe "internal only and Load Balancer" requires both services to be in the same VPC.

Nothing else particularly come to mind. Maybe we can increase logging level in your deployment (see Troubleshooting) to get some verbosity.

Actually, there is no VPC in this project at all. It was not clear to me from documentation if we need to setup a VPC connector to achieve internal service-to-service communication. Is it mandatory?
Apart from that I will try the 2nd option you suggested and update here.
Thanks for the help, appreciated a lot 🙏

Okay problem has been solved and I learned two things;

Service Accounts can be disabled. We are using default compute SA of the project and it was somehow disabled. When I enabled it I was able to access 'internaldocs' from the 'curl' service
There was no VPC in the project. So if you want to allow internal traffic and traffic from Cloud Load Balancing you need to have VPC and VPC Connector attached to the Cloud Run services. I believe Google Cloud Run documentation especially service-to-service communication document can be improved by mentioning this use case.

Thanks again for the help.

If you want to make a service internal, you don't need VPC Connector. Just IAM.
VPC connector is for other resources that are in vpc and load balancers.


ipfix: add ipfix_report_interval field to NWI

The field specifies minimum per-flow IPFIX reporting interval in
seconds. If the field is set to 0 or ~0, the default value of 5
seconds is used. It's possible to set the value both through the
binapi and CLI.
This PR is stacked on top of #255. This PR must be merged first.

closing, created by mistake


fix trainer bugs

have you checked the thrift trainer code?

@Arefsa78
I know and it does not need to support gym thrift,
I just want to be sure that we don't have the same problems with thrift rpc codes

It's working now. You can check it too with this PR:
https://github.com/CLSFramework/sample-playmaker-server-python-thrift/pull/1


ES Lint rules

What rules are out there?

@sscotth I made a couple changes
, and I await your approval.

Greg and Chad also are breaking a few of my expected style rules here: https://github.com/ccates23/minitwit/blob/delete/app/delete_feature/delete.test.js
Maybe we can add some additional rules to watch

@sscotth 'Ello gov'na!

@sscotth I'm looking for a rule for formatting callback spacing.  Haven't found it yet, but I'm thinking about this...
Thoughts?

Just noticed this comment. I tend to not look at closed PRs. Can you give some examples on what you are looking for?


[Advice] Set Email Verification as Necessary

Thanks for 52poke wiki!
Advice:

Set Email address as a required item in the reg form.

Set "Email Verification" as Necessary.

The goal is to reduce junk users or attackers. By the way, this is helpful in case that some users may forget their password.

Another suggestion:
Forbid new users, who have registered no longer than 48 hours (or longer time), editting any pages.


Node page: sources next to a connected object should be the edge sources, not the node sources

Also applies to node labels.

"an edge between two labels has a source if there is an edge with that source between two nodes that have those labels"

This code is currently KgNodeLabelLink, which is used by KgNodeContextGrid and KgNodeContextTable. Nodes and node labels connect to node labels, thus KgNodeLabelLink.

Implemented the front end part of this, showing top edge sourceIds rather than the object nodeLabel sourceIds.


Query from multiple models

It would be useful to have the ability to use different models in one prompt when different holes have different complexity requirements

Maybe a from syntax that is compatible with the ast parser could be
from
    with "openai/davinici-003" as smart:
        decoder = beam
        temperature = 0.8
    with "openai/ada-001" as fast:
        decoder = argmax

Which could be used in the prompt string with something like
query
    "{easy_question}? [fast_answer]"
    "{hard_question}? [smart_answer]"

One idea for handling this is to add a new qstrings template variable like
@dataclass
class ModelTemplateVariable:
    model: str
    name: str

And then the compiler could be modified to sandwich query calls with set_model and set_decoder calls in the _compiled.py file. The decoder parameters could switch back to whichever configuration you assigned the name "default" so that you can still just do [completion] if you are using one model predominantly

This overlaps in parts with https://github.com/eth-sri/lmql/issues/88, but I think keeping multi-model suport separate makes sense. Another option for multi-model definition, would be to add a simple lmql.model function, allowing to specify model parameters like this:
from
      lmql.model("openai/davinci-003", decoder="beam", temperature=0.8) as smart
      lmql.model("openai/ada-001", decoder="argmax") as fast

This could also work with #88, where a model may also be defined implicitly via e.g. lmql.set_default_model.
As in your screenshot on Discord, we should then also think about how users define what model to use per variable. The proposal there was the following:
"Some prompt [easy VAR1] and something [hard VAR2]"

given the definitions of easy and hard above.
I think this is an interesting and concise way to express this, and more generally, it may make sense to think about a syntactic construct, extending simple template variable syntax like [VAR1] by some annotation like easy here. This can be used to specify the model, but other annotations could also be interesting to look at. For instance, one may want to mark a variable as [stream VAR1], to indicate to an output writer that the value of VAR1 should be streamed to the user.


vendor: Switch from Godeps tool to govendor

Remove the Godeps directory
https://github.com/kardianos/govendor

Why govendor? I am willing to throw in the towel on godep since there seems to be so much sentiment against it, but could we maybe attempt to standardise on a replacement across CoreOS projects? So far the leading contender seems to be glide.. particularly since some of the other big projects we integrate with (like k8s) are trending that way.

govendor has worked well for me in the past, no other reason. I'd favor us reviewing govendor, glide, etc. and standardizing across CoreOS. I don't have immediate plans to switch yet since it isn't high priority and needs investigation.

Created coreos/docs#775

Several CoreOS projects have switched to using Glide. I plan to switch this project as well.


How do you get the min- and max-value for clipping?

Great work!
Would you provide some information about how you get the magic numbers of the min- and max-values for clipping as  [-2.1179, -2.0357, -1.8044] and [2.2489, 2.4286, 2.6400]?
Thanks!
def clip(self, img, img_min=None, img_max=None):
    if img_min is None:
        img_min = torch.tensor([-2.1179, -2.0357, -1.8044]).view(1,3,1,1).cuda()

    if img_max is None:
        img_max = torch.tensor([2.2489, 2.4286, 2.6400]).view(3,1,1).cuda()

    img = torch.clip(img, min=img_min, max=img_max)

    return img

Hi @wuzhongwulidong , thank you for showing interest in our work. These values are the max and min pixel values normalized using the mean and std of ImageNet.

Thank you very much. Got it.


Add redirect from old gcp hosted site

Is your feature request related to a problem? Please describe.
A few users are still using the old gcp site, which is no longer maintained. We should host a redirect notice with plans to retire it completely in a month or so.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
We can just retire the site without a redirect notice
Additional context
Old site hosted on gcp: https://neural-dynamics-dev.uw.r.appspot.com/
Currently maintained site hosted on AWS Amplify: https://metadata-entry.allenneuraldynamics.org/

Let's just shut down the gcp hosted site

This was shut down. That url returns a 404 not found response.


[Button] Issue - No options to enable Cursor Pointer

Hi @Radix Team,
Recently i have started using radix ui to one of my project. I have used buttons in many places of my web page. I have received feedback to enable the hand cursor for buttons. When i checked Radix ui documentation, am not able to see any options to enable hand cursor.
Can you help me? How to enable hand cursor for buttons?
Thanks for producing this beautiful  radix/ui framework.

Sakthi.

Check this out: https://github.com/radix-ui/themes/issues/123#issuecomment-1778862252
You can do something like the following.
.radix-themes {
  --cursor-button: pointer;
}

Yep, you’d do it like that with a CSS variable override ☝ . We need to improve the docs on customizing Radix Themes to make this a bit more apparent. Here’s a common configuration that handles pointer cursors consistently:
.radix-themes {
  --cursor-button: pointer;
  --cursor-disabled: default;
  --cursor-link: pointer;
  --cursor-menu-item: pointer;
  --cursor-slider-thumb: pointer;
  --cursor-slider-thumb-active: pointer;
  --cursor-switch: pointer;
}

Check this out: #123 (comment)
You can do something like the following.
.radix-themes {
  --cursor-button: pointer;
}

Thank you friend for quick response. This solution is works for me.

Yep, you’d do it like that with a CSS variable override ☝ . We need to improve the docs on customizing Radix Themes to make this a bit more apparent. Here’s a common configuration that handles pointer cursors consistently:
.radix-themes {
  --cursor-button: pointer;
  --cursor-disabled: default;
  --cursor-link: pointer;
  --cursor-menu-item: pointer;
  --cursor-slider-thumb: pointer;
  --cursor-slider-thumb-active: pointer;
  --cursor-switch: pointer;
}

Great!!! Thank you so much for sharing additional options as well. I'll use it for other components as well.

didn't work for me, needed to add !important
.radix-themes {
--default-font-family: var("--font-inter") !important;
--cursor-button: pointer !important;
}


Spurious validation step when restarting with a checkpoint when max_steps is set in the trainer

Bug description
When restarting from an existing checkpoint with a trainer that has a max_steps value set, the trainer does a single validation step before actually restarting the training epoch (even if val_sanity_checks_step is set to 0)
This is an issue because that one validation step is considered a full validation round (callbacks are applied, which means the Tensorboard logger logs those spurious data points).
Note that this only applies when max_steps is set, and not max_epochs so there might be something going on with the management of the global step.
Also, this problem has been reported in this discussion:
https://github.com/Lightning-AI/lightning/discussions/18110
What version are you seeing the problem on?
v2.0
How to reproduce the bug
import os
from torchvision.datasets import MNIST
import os
from torch import optim, nn, utils, Tensor
from torchvision.datasets import MNIST
from torchvision.transforms import ToTensor
import pytorch_lightning as pl
from torch.utils.data import DataLoader
import tempfile

# define any number of nn.Modules (or use your current ones)
encoder = nn.Sequential(nn.Linear(28 * 28, 64), nn.ReLU(), nn.Linear(64, 3))
decoder = nn.Sequential(nn.Linear(3, 64), nn.ReLU(), nn.Linear(64, 28 * 28))

# define the LightningModule
class LitAutoEncoder(pl.LightningModule):
    def __init__(self, encoder, decoder):
        super().__init__()
        self.encoder = encoder
        self.decoder = decoder

    def training_step(self, batch, batch_idx):
        # training_step defines the train loop.
        # it is independent of forward
        x, y = batch
        x = x.view(x.size(0), -1)
        z = self.encoder(x)
        x_hat = self.decoder(z)
        loss = nn.functional.mse_loss(x_hat, x)
        # Logging to TensorBoard (if installed) by default
        self.log("train_loss", loss)
        return loss

    def validation_step(self, batch, batch_idx):
        x, y = batch
        x = x.view(x.size(0), -1)
        z = self.encoder(x)
        x_hat = self.decoder(z)
        loss = nn.functional.mse_loss(x_hat, x)
        # Logging to TensorBoard (if installed) by default
        self.log("val_loss", loss)
        return loss

    def on_validation_end(self) -> None:
        print('Ending Validation')
        return super().on_validation_end()

    def configure_optimizers(self):
        optimizer = optim.Adam(self.parameters(), lr=1e-3)
        return optimizer

def setup_and_train(ckpt_path):
  # init the autoencoder
  autoencoder = LitAutoEncoder(encoder, decoder)

  train_dataset = MNIST(os.getcwd(), download=True, transform=ToTensor())
  test_dataset = MNIST(os.getcwd(),train=False, download=True, transform=ToTensor())
  train_loader = utils.data.DataLoader(train_dataset)
  test_loader = utils.data.DataLoader(test_dataset)

  ckpt_dir = '/content/ckpt_dir'

  checkpoint_callback = pl.callbacks.ModelCheckpoint(
          dirpath=ckpt_dir,
          save_weights_only=False,
          save_top_k=0,
          # monitor='EM',
          # mode='max',
          save_last=True,
      )
  # Initial training
  trainer = pl.Trainer(limit_train_batches=2000, max_steps=100000, num_sanity_val_steps=0, val_check_interval=1.0, callbacks=checkpoint_callback)
  trainer.fit(model=autoencoder, train_dataloaders=train_loader, val_dataloaders=test_loader, ckpt_path=ckpt_path)

I used this script changing the trainer configuration and stopping the run manually in a notebook. You can see that when restarting the run from a checkpoint with max_steps set, there's one val step run before resuming the training.

### Error messages and logs

Error messages and logs here please

### Environment

- toml:              0.10.2
	- tomli:             2.0.1
	- toolz:             0.12.0
	- torch:             2.0.1+cu118
	- torchaudio:        2.0.2+cu118
	- torchdata:         0.6.1
	- torchmetrics:      1.2.0
	- torchsummary:      1.5.1
	- torchtext:         0.15.2
	- torchvision:       0.15.2+cu118
	- tornado:           6.3.2
	- tqdm:              4.66.1
	- traitlets:         5.7.1
	- traittypes:        0.2.1
	- transformers:      4.33.2
	- triton:            2.0.0
	- tweepy:            4.13.0
	- typer:             0.9.0
	- types-setuptools:  <IP_ADDRESS>
	- typing-extensions: 4.5.0
	- tzlocal:           5.0.1
	- uc-micro-py:       1.0.2
	- uritemplate:       4.1.1
	- urllib3:           2.0.4
	- vega-datasets:     0.9.0
	- wadllib:           1.3.6
	- wasabi:            1.1.2
	- wcwidth:           0.2.6
	- webcolors:         1.13
	- webencodings:      0.5.1
	- websocket-client:  1.6.2
	- werkzeug:          2.3.7
	- wheel:             0.41.2
	- widgetsnbextension: 3.6.5
	- wordcloud:         1.9.2
	- wrapt:             1.15.0
	- xarray:            2023.7.0
	- xarray-einstats:   0.6.0
	- xgboost:           1.7.6
	- xlrd:              2.0.1
	- xxhash:            3.3.0
	- xyzservices:       2023.7.0
	- yarl:              1.9.2
	- yellowbrick:       1.5
	- yfinance:          0.2.28
	- zict:              3.0.0
	- zipp:              3.16.2
* System:
	- OS:                Linux
	- architecture:
		- 64bit
		- ELF
	- processor:         x86_64
	- python:            3.10.12
	- release:           5.15.120+
	- version:           #1 SMP Wed Aug 30 11:19:59 UTC 2023

### More info

_No response_

cc @carmocca @justusschock

I'm experiencing the same issue.


Scale, canvas and textboxes

Scale, canvas and textbox functionality is tested and stable. Only issue is the middle button issues #212  in firefox.

Great to see this stuff, I wont be able to look at this until January so hopefully someone else can peer/code review and provide suggestions!

Hey guys, anything to stop this being merged?

Merged through other means


format_doc_comments is invoked even when set to false

This is similar to #3209, but a bit worse IMO: currently setting either normalize_doc_attributes or wrap_comments to true implies format_doc_comments as well and reformats docs, even if format_doc_comments = false is explicitly set in the config.

What does format_doc_comments mean exactly? Isn't wrapping a doc comment some sort of formatting the doc comment?
And is setting wrap_comments to false but format_doc_comments to true meaning do formatting other than wrapping to the doc comments?

The description is somewhat vague, but from the included example (https://github.com/rust-lang/rustfmt/blob/master/Configurations.md#format_doc_comments) and own experience, I've assumed it means just formatting code snippets within doc comments, and should work independently from wrapping comment text.

Reproduce with two snippets:
wrap_comments implies format_doc_comments:
// rustfmt-wrap_comments: true

/// Foo
///
/// # Example
/// ```
/// # #![cfg_attr(not(dox), feature(cfg_target_feature, target_feature, stdsimd))]
/// # #![cfg_attr(not(dox), no_std)]
/// fn foo() {  }
/// ```
///
fn foo() {}

/// A long commment for wrapping
/// This is a long long long long long long long long long long long long long long long long long long long long sentence.
fn bar() {}

normalize_doc_attributes implies format_doc_comments:
// rustfmt-normalize_doc_attributes: true

/// Foo
///
/// # Example
/// ```
/// # #![cfg_attr(not(dox), feature(cfg_target_feature, target_feature, stdsimd))]
/// # #![cfg_attr(not(dox), no_std)]
/// fn foo() {  }
/// ```
///
fn foo() {}

#[doc = "Bar documents"]
fn bar() {}

@RReverser You are correct and that is what it should do. Maybe we can review the description of this option too.

@RReverser, @scampi: Perhaps change format_doc_comment to format_code_in_doc_comment?

@xiongmao86 That sounds reasonable to me.


support for go_field_suffix

r? @patrickxb
Tested it by recompiling all of keybase via npm link. We should longer-term have a regtest suite locally. Sigh.

LGTM


Add missing autogenerated comment to header files

Client: erl

Add missing autogenerated comment to Erlang's header files

[ ] Did you create an Apache Jira ticket?  (not required for trivial changes)
[ ] Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?  (not required for trivial changes)
[ ] Did you squash your changes to a single commit?
[ ] Did you do your best to avoid breaking changes?  If one was needed, did you label the Jira ticket with "Breaking-Change"

erl tests passed despite the CI issues


syntax wrong on first code box on readme

= should be : I believee

Sorry I need more details. Can you paste the code?


Remaining Issue

Describe the bug
I spent my tokens resolving remaining issue raher than developing
i'm really disappointed about this
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-svhft5tz
Steps to reproduce
each time i click on Attempts fix
Expected behavior
Displaying the web page
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 131
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 292778649,
  "usedJSHeapSize": 262710201,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username =<EMAIL_ADDRESS>Chat ID = 9972a40cc19f

Additional context
No response

"[vite] Internal Server Error Failed to resolve import "./components/Header" from "src/features/layout/index.ts". Does the file exist? at TransformPluginContext._formatError (file:///home/project/node_modules/vite/dist/node/chunks/dep-CDnG8rE7.js:49242:41) at TransformPluginContext.error (file:///home/project/node_modules/vite/dist/node/chunks/dep-CDnG8rE7.js:49237:16) at normalizeUrl (file:///home/project/node_modules/vite/dist/node/chunks/dep-CDnG8rE7.js:64033:23) at async eval (file:///home/project/node_modules/vite/dist/node/chunks/dep-CDnG8rE7.js:64165:39) at async TransformPluginContext.transform (file:///home/project/node_modules/vite/dist/node/chunks/dep-CDnG8rE7.js:64092:7) at async PluginContainer.transform (file:///home/project/node_modules/vite/dist/node/chunks/dep-CDnG8rE7.js:49083:18) at async loadAndTransform (file:///home/project/node_modules/vite/dist/node/chunks/dep-CDnG8rE7.js:51916:27) at async viteTransformMiddleware (file:///home/project/node_modules/vite/dist/node/chunks/dep-CDnG8rE7.js:61873:24)"
at handleMessage (/@vite/client:698:19)
at _0x122afa. (/@vite/client:546:5)
at _0x122afa.dispatchEvent (/.localservice@runtime.c8c182a3.js:26:126607)
at _0x122afa._handleMessage (/.localservice@runtime.c8c182a3.js:26:110218)
at MessagePort._0x36418a (/.localservice@runtime.c8c182a3.js:26:106117)


[logstash] Ability to change the livenessProbe and readinessProbe

Describe the feature:
The httpGet fails for both the liveness and readinessProbes in our openshift v3.11 k8 v1.11 cluster
Readiness probe failed: Get http://<IP_ADDRESS>:9600/: dial tcp <IP_ADDRESS>:9600: connect: connection refused
Solution:
Change the logstash stateful set to allow the user to add their own probes other than httpGet.
Statefulset change:
        {{- if .Values.livenessProbe }}
        livenessProbe:
{{ toYaml .Values.livenessProbe | indent 10 }}
        {{- end }}
        {{- if .Values.readinessProbe }}
        readinessProbe:
{{ toYaml .Values.readinessProbe | indent 10 }}
        {{- end }}

values.yaml change:
# By default there will not be a livelenessProbe. You may uncomment this or use another probe.
livenessProbe: {}
  # httpGet:
  #   path: /
  #   port: http
  # initialDelaySeconds: 300
  # periodSeconds: 10
  # timeoutSeconds: 5
  # failureThreshold: 3
  # successThreshold: 1

# By default there will not be a readinessProbe. You may uncomment this or use another probe.
readinessProbe: {}
  # httpGet:
  #   path: /
  #   port: http
  # initialDelaySeconds: 60
  # periodSeconds: 10
  # timeoutSeconds: 5
  # failureThreshold: 3
  # successThreshold: 3

Describe a specific use case for the feature:
As the chart/value file are currently written you can specify only httpGets.  If an exec is probe is added in a custom values file and applied to the install helm combines them instead of replacing them resulting in a failed deployment.
the way they are written you cannot replace them with
readinessProbe:
  exec:
    command:
      - curl
      - localhost:9600

because when helm appends out custom-values.yaml on top of the built in values.yaml results in:
readinessProbe:
  exec:
    command:
      - curl
      - localhost:9600
  httpGet:
    path: /
    port: http

https://github.com/elastic/helm-charts/pull/469

Bump +1

Same problem here.
I can't even see how the  chart can run with the current probes as by default Logstash is listening on <IP_ADDRESS> for the API.
My fix for now is to listen on <IP_ADDRESS> (be aware of security risks that it might imply):
logstashConfig:
  logstash.yml: |
    http.host: <IP_ADDRESS>

Hi,
Indeed having http.host: <IP_ADDRESS> is mandatory for livenessProbe and readinessProbe.
This is the default config in Logstash Docker images (see logstash.yml) which is used with chart default values (logstashConfig: {}).
However when you override logstashConfig, this replace Docker image default config so you need to re-include http.host: <IP_ADDRESS>).
We will see how we can improve that so you don't need to add explicitely http.host: <IP_ADDRESS> when overriding config or at least document it.
In the mean time we will investigate why probes can't be overrided if you want to change them but the default values should still include these probes.

As mentionned in my previous comment http.host: <IP_ADDRESS> is the default values in Logstash Docker images and is mandatory for default probes, if you add some custom logstashConfig you should always include http.host: <IP_ADDRESS> to use default probes.
If you need to use http.host: <IP_ADDRESS> which is not compatible with default probes, you can disable or override probes using one of the following yaml syntaxes:
# disable livenessProbe
livenessProbe: null

# replace httpGet default readinessProbe by some exec probe
readinessProbe:
  httpGet: null
  exec:
    command:
      - curl
      - localhost:9600

We will update Logstash README.md to document it soon.

Can we close this issue or do you have any questions?


feat: support response objects in InputFile

Previously, it would not be a type error on Deno to do new InputFile(response) because Response is assignable to URLLike.
However, this would lead to unexpected behaviour because it would make grammY fetch the URL of the Response object again when sending the file, hence ignoring the response object's data.
This PQ fixes that by adding explicit support for Response objects and catching this case.
Close #426.

URLLike was a mistake. It should be removed or changed to Pick<URL, "href"> in 2.0.

URLLike was a mistake.

Why is that?

Because it forced Response support, and { href: fileUrl } still doesn't work: https://github.com/grammyjs/grammY/blob/b78c72ebec625ddf2d23eb27a9eed895710ac3c5/src/types.node.ts#L111-L116


Upgrade to Elasticsearch 7.0.0

#179

@duxet I am trying your PR but compilation fails for me. Here is my log:
$ git fetch origin pull/180/head:r7.0.0
remote: Enumerating objects: 5, done.
remote: Counting objects: 100% (5/5), done.
remote: Compressing objects: 100% (5/5), done.
remote: Total 5 (delta 0), reused 1 (delta 0), pack-reused 0
Unpacking objects: 100% (5/5), done.
From https://github.com/vvanholl/elasticsearch-prometheus-exporter
 * [new ref]         refs/pull/180/head -> r7.0.0
 * [new tag]         <IP_ADDRESS>           -> <IP_ADDRESS>
 * [new tag]         <IP_ADDRESS>            -> <IP_ADDRESS>

$ git checkout r7.0.0
Switched to branch 'r7.0.0'

$ git log --oneline -n 5
558cbd7 (HEAD -> r7.0.0) Upgrade to Elasticsearch 7.0.0
d52c619 (origin/master, origin/HEAD, master) Prepare next release
e7c69ed (tag: <IP_ADDRESS>) Release version <IP_ADDRESS>
76c0aab Merge pull request #177 from vvanholl/es671
47a1e6f Temporary use compuscene/gradle:5.3.1-jdk12 waiting official Gradle image

$ java --version
openjdk 12 2019-03-19
OpenJDK Runtime Environment (build 12+33)
OpenJDK 64-Bit Server VM (build 12+33, mixed mode, sharing)

$ gradle --version

------------------------------------------------------------
Gradle 5.3.1
------------------------------------------------------------

Build time:   2019-03-28 09:09:23 UTC
Revision:     f2fae6ba563cfb772c8bc35d31e43c59a5b620c3

Kotlin:       1.3.21
Groovy:       2.5.4
Ant:          Apache Ant(TM) version 1.9.13 compiled on July 10 2018
JVM:          12 (Oracle Corporation 12+33)
OS:           Mac OS X 10.14.3 x86_64

$ gradle clean check
Starting a Gradle Daemon (subsequent builds will be faster)

> Configure project :
=======================================
Elasticsearch Build Hamster says Hello!
  Gradle Version        : 5.3.1
  OS Info               : Mac OS X 10.14.3 (x86_64)
  JDK Version           : 12 (Oracle Corporation 12 [OpenJDK 64-Bit Server VM 12+33])
  JAVA_HOME             : /Library/Java/JavaVirtualMachines/openjdk-12.jdk/Contents/Home
  Random Testing Seed   : AC407754182B6F15
=======================================
Host: MacBook-Pro/<IP_ADDRESS>
Gradle: 5.3.1 JVM: 12 (Oracle Corporation 12+33) Groovy: 2.5.4
Build: group: 'org.elasticsearch.plugin.prometheus', name: 'prometheus-exporter', version: '<IP_ADDRESS>-SNAPSHOT'
Timestamp: 2019-04-12T09:29:56.235583+02:00[Europe/Prague]

> Task :compileJava
/Users/lvlcek/projects/elasticsearch-prometheus-exporter/src/main/java/org/elasticsearch/action/NodePrometheusMetricsAction.java:25: error: wrong number of type arguments; required 1
public class NodePrometheusMetricsAction extends Action<NodePrometheusMetricsRequest, NodePrometheusMetricsResponse,
                                                       ^
/Users/lvlcek/projects/elasticsearch-prometheus-exporter/src/main/java/org/elasticsearch/action/TransportNodePrometheusMetricsAction.java:49: error: TransportNodePrometheusMetricsAction is not abstract and does not override abstract method doExecute(Task,NodePrometheusMetricsRequest,ActionListener<NodePrometheusMetricsResponse>) in TransportAction
public class TransportNodePrometheusMetricsAction extends HandledTransportAction<NodePrometheusMetricsRequest,
       ^
/Users/lvlcek/projects/elasticsearch-prometheus-exporter/src/main/java/org/elasticsearch/action/TransportNodePrometheusMetricsAction.java:61: error: no suitable constructor found for HandledTransportAction(Settings,String,ThreadPool,TransportService,ActionFilters,IndexNameExpressionResolver,NodePromet[...]::new)
        super(settings, NodePrometheusMetricsAction.NAME, threadPool, transportService, actionFilters,
        ^
    constructor HandledTransportAction.HandledTransportAction(String,TransportService,ActionFilters,Supplier<NodePrometheusMetricsRequest>) is not applicable
      (actual and formal argument lists differ in length)
    constructor HandledTransportAction.HandledTransportAction(String,TransportService,ActionFilters,Reader<NodePrometheusMetricsRequest>) is not applicable
      (actual and formal argument lists differ in length)
    constructor HandledTransportAction.HandledTransportAction(String,TransportService,ActionFilters,Reader<NodePrometheusMetricsRequest>,String) is not applicable
      (actual and formal argument lists differ in length)
    constructor HandledTransportAction.HandledTransportAction(String,boolean,TransportService,ActionFilters,Supplier<NodePrometheusMetricsRequest>) is not applicable
      (actual and formal argument lists differ in length)
    constructor HandledTransportAction.HandledTransportAction(String,boolean,TransportService,ActionFilters,Reader<NodePrometheusMetricsRequest>) is not applicable
      (actual and formal argument lists differ in length)
    constructor HandledTransportAction.HandledTransportAction(String,boolean,TransportService,ActionFilters,Reader<NodePrometheusMetricsRequest>,String) is not applicable
      (actual and formal argument lists differ in length)
/Users/lvlcek/projects/elasticsearch-prometheus-exporter/src/main/java/org/elasticsearch/action/TransportNodePrometheusMetricsAction.java:69: error: method does not override or implement a method from a supertype
    @Override
    ^
/Users/lvlcek/projects/elasticsearch-prometheus-exporter/src/main/java/org/elasticsearch/action/TransportNodePrometheusMetricsAction.java:204: warning: [deprecation] logger in TransportAction has been deprecated
            if (logger.isTraceEnabled()) {
                ^
/Users/lvlcek/projects/elasticsearch-prometheus-exporter/src/main/java/org/elasticsearch/action/TransportNodePrometheusMetricsAction.java:205: warning: [deprecation] logger in TransportAction has been deprecated
                logger.trace("Return response: [{}]", response);
                ^
/Users/lvlcek/projects/elasticsearch-prometheus-exporter/src/main/java/org/elasticsearch/plugin/prometheus/PrometheusExporterPlugin.java:56: error: cannot infer type arguments for ActionHandler<>
                new ActionHandler<>(NodePrometheusMetricsAction.INSTANCE, TransportNodePrometheusMetricsAction.class)
                ^
  reason: cannot infer type-variable(s) Request,Response
    (argument mismatch; NodePrometheusMetricsAction cannot be converted to Action<Response>)
  where Request,Response are type-variables:
    Request extends ActionRequest declared in class ActionHandler
    Response extends ActionResponse declared in class ActionHandler
/Users/lvlcek/projects/elasticsearch-prometheus-exporter/src/main/java/org/elasticsearch/action/NodePrometheusRequestBuilder.java:29: error: incompatible types: NodePrometheusMetricsAction cannot be converted to Action<NodePrometheusMetricsResponse>
        super(client, action, new NodePrometheusMetricsRequest().local(true));
                      ^
/Users/lvlcek/projects/elasticsearch-prometheus-exporter/src/main/java/org/elasticsearch/action/NodePrometheusMetricsAction.java:34: error: method does not override or implement a method from a supertype
    @Override
    ^
/Users/lvlcek/projects/elasticsearch-prometheus-exporter/src/main/java/org/elasticsearch/action/NodePrometheusMetricsAction.java:40: error: method does not override or implement a method from a supertype
    @Override
    ^
/Users/lvlcek/projects/elasticsearch-prometheus-exporter/src/main/java/org/elasticsearch/action/NodePrometheusMetricsAction.java:45: error: method does not override or implement a method from a supertype
    @Override
    ^
Note: Some messages have been simplified; recompile with -Xdiags:verbose to get full output
9 errors
2 warnings

> Task :compileJava FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':compileJava'.
> Compilation failed; see the compiler error output for details.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 21s
2 actionable tasks: 2 executed

Are you able to run tests via gradle check?

I have fixed all build errors and warnings, however i haven't checked yet manually if the plugin works correctly.

There is a set of integration tests that are run when gradle check is run. It starts ES cluster with 2 nodes, deploy Prometheus plugin to all the nodes and them performs all the tests found under src/test/resources/rest-api-spec/test/resthandler folder (quite a bit of tests). If these tests are passing there is a good change the plugin works as expected (these are real integration tests). You should see some confirmation that all the tests passed.
I will do code review during the weekend. Thanks for the PR!

Done :)

@vvanholl I am handing it over to you :-)

@vvanholl ping?

Hi @lukas-vlcek @davidkarlsen I am working on it today

Thank you @duxet and @lukas-vlcek for supporting ES 7
It's time now to release !


Remove Ruby 1.9.3 support

It's time to remove support for Ruby 1.9.x.
This version of Ruby was initially released in 2008 (if my memory serves) and the Ruby maintainers have EOL'd Ruby from Feb 23rd, 2015. It's time we did the same.
This will be the only commit differentiating 0.9.5 and 1.0.0.
I'm choosing to release this as 1.0.0 to ensure that backwards compatibility with older versions of Rails is kept.

@radar Nudge, nudge. Is v1.0.0 going to be reflected on https://github.com/svenfuchs/i18n/releases page?


incorrect warning about index routes having child routes

If you have a file system that looks like this:
routes/
  _index/
    route.tsx
    blocks/
      index.tsx

remix build throws an error:

Error: Child routes are not allowed in index routes. Please remove child routes of _index

It does the same if you use blocks/route.tsx as well.
The code shouldn't be digging that deep in the first place to even register that there might a route definition. It should only be looking for routes at: routes/*.tsx and routes/*/route.tsx.
There's no reason for it to be digging deeper, you should be able to name things whatever you want once you're in a route folder aside from folder/route.tsx and folder/index.tsx.

fixed by the improvements made in 1.15 https://github.com/remix-run/remix/pull/5634


prepend licenses to npm-published bundles

I created two scripts to generate the umd and System bundles as well, so that SystemJS bundles could be included in global build.

Merged with 1ff3b89. Thanks @jeffbcross


For VS2013, downgrade to .Net 4.5, and upgrade the compilers.

LGTM

Out of curiosity, why was this necessary for VS 2013 to work?
.NET 4.5 reached end of life and no longer supported
https://blogs.msdn.microsoft.com/dotnet/2015/12/09/support-ending-for-the-net-framework-4-4-5-and-4-5-1/


Fixed long response times for requests made to reddit

Reddit servers seem to throw a fit if you don't specify both a UserAgent and an AcceptLanguage in the requests. For me, this results in very long responses that took 5-10 seconds for a request.
Using Fiddler, I was able to determine that the timeout was taking place between ServerGotRequest and ServerBeginResponse, so that's when I tried modifying the header until I pinpointed what was causing the long delays.
If others are having similar delays as me, this change makes a world of difference.

This repository is no longer being maintained by the owner. I'm maintaining a hard fork over at https://github.com/CrustyJew/RedditSharp


[Event Request] Codeunit 70 Purch.-Calc.Discount - procedure CalculateInvoiceDiscount - OnCalculateInvoiceDiscountOnBeforeFindForCalcVATAmountLines

Please add an Event OnCalculateInvoiceDiscountOnBeforeFindForCalcVATAmountLines in procedure CalculateInvoiceDiscount in Codeunit 70 Purch.-Calc.Discount
[Event Request] Codeunit 70 Purch.-Calc.Discount - procedure CalculateInvoiceDiscount
    var
        TempVATAmountLine: Record "VAT Amount Line" temporary;
        PurchSetup: Record "Purchases & Payables Setup";
        TempServiceChargeLine: Record "Purchase Line" temporary;
        PurchCalcDiscByType: Codeunit "Purch - Calc Disc. By Type";
        DiscountNotificationMgt: Codeunit "Discount Notification Mgt.";
        IsHandled: Boolean;
    begin
        PurchSetup.Get();

        IsHandled := false;
        OnBeforeCalcPurchaseDiscount(PurchHeader, IsHandled, PurchLine2, UpdateHeader);
        if IsHandled then
            exit;

        with PurchLine do begin
            LockTable();
            PurchHeader.TestField("Vendor Posting Group");
            VendPostingGr.Get(PurchHeader."Vendor Posting Group");

            PurchLine2.Reset();
            PurchLine2.SetRange("Document Type", "Document Type");
            PurchLine2.SetRange("Document No.", "Document No.");
            PurchLine2.SetFilter(Type, '<>0');
            PurchLine2.SetRange("System-Created Entry", true);
            PurchLine2.SetRange(Type, PurchLine2.Type::"G/L Account");
            PurchLine2.SetRange("No.", VendPostingGr."Service Charge Acc.");
            if PurchLine2.FindSet(true, false) then
                repeat
                    PurchLine2."Direct Unit Cost" := 0;
                    PurchLine2.Modify();
                    TempServiceChargeLine := PurchLine2;
                    TempServiceChargeLine.Insert();
                until PurchLine2.Next() = 0;

            PurchLine2.Reset();
            PurchLine2.SetRange("Document Type", "Document Type");
            PurchLine2.SetRange("Document No.", "Document No.");
            PurchLine2.SetFilter(Type, '<>0');

            OnCalculateInvoiceDiscountOnBeforeFindForCalcVATAmountLines(PurchHeader, PurchLine2, UpdateHeader); // <-- NEW EVENT

            if PurchLine2.Find('-') then;
            PurchLine2.CalcVATAmountLines(0, PurchHeader, PurchLine2, TempVATAmountLine);
            ...

[IntegrationEvent(false, false)]
local procedure OnCalculateInvoiceDiscountOnBeforeFindForCalcVATAmountLines(PurchHeader: Record "Purchase Header"; var PurchLine2: Record "Purchase Line"; UpdateHeader: Boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 46955.


#1099 Integrate cart UI with new APIs

Summary

Integrate cart UI with new APIs
Remove unused old cart files
Update cart automation tests

Screenshots

Hi @mochacr0, do we have automation test for below case? If not let's create one 🙂.

Add to cart

Remove cart item

Automation tests build succeeded

Hi @mochacr0, do we have automation test for below case? If not let's create one 🙂.

Add to cart

Remove cart item

Automation tests build succeeded

Hi @duylv27, we already have automation tests for this case. The screenshots are from manual testing, but I also ran the automation tests, and the test build result is included in the comment above. Could you take a look?

Hi @mochacr0, do we have automation test for below case? If not let's create one 🙂.

Add to cart

Remove cart item

Automation tests build succeeded

Hi @duylv27, we already have automation tests for this case. The screenshots are from manual testing, but I also ran the automation tests, and the test build result is included in the comment. Could you take a look?

I see, thank you!


vim mode: support gf command

Check for existing issues

[X] Completed

Describe the feature
The "goto file" command in vim allows to navigate to the file whose name is under or after the cursor.
This command is very handy to navigate to header files in C/C++.
If applicable, add mockups / screenshots to help present your vision of the feature
No response

This is already present?


Contact JS

When looking at the contact section, then I can see a main div with "get in touch" message
When looking at the contact section then I can see an email form which takes an email address, name and message.
When submitting the form, then the details are emailed to me.
When looking at the 'links' section then I can see links to my github and my linkedIn
When I get to the bottom of the page, then I can see 4 font awesome icons with links to github, codepen, linkedIn, stackoverflow

remaining:

functionality on form
apply the links to the icons

only need form functionality, send email through form required

Need to create a backend function to handle the form and email submission/sending, Research needed for this.


Fix for #377

For now, I've only add the unit test showing the issue #377

Current coverage is 100% (diff: 100%)

Merging #378 into master will not change coverage

@@           master   #378   diff @@
====================================
  Files           1      1
  Lines         844    846     +2
  Methods         0      0
  Messages        0      0
  Branches        0      0
====================================
+ Hits          844    846     +2
  Misses          0      0
  Partials        0      0

Powered by Codecov. Last update c7b1169...9d8e895


Table.Column not updating label prop when it's updated.

Bug report 🐞
Version & Environment

Chrome latest
Version of zeit-ui/react: ^1.8.0-canary.0

Expection
When using a table like this:
() => {
  const data = [
    { property: 'type', description: 'Content type', type: 'secondary | warning', default: '-' },
    { property: 'Component', description: 'DOM element to use', type: 'string', default: '-' },
    { property: 'bold', description: 'Bold style', type: 'boolean', default: 'true' },
  ]
  const [state, setState] = React.useState(0)
  React.useEffect(() => {
    const interval = setInterval(() => {
      setState((old) => old + 1)
    }, 1000)
    return () => {
      clearInterval(interval)
    }
  }, [])
  return (

      <Table.Column prop="description" label={`description: ${state}`} />

  )
}

The description label prop is not being updated. This is a "minimal" reproducable example, I'm not doing exactly this in my application.
Actual results (or Errors)
The description label prop isn't being updated, even though the variable state is being updated.

@merisbahti The pre-release version v1.8.0-canary.9 is out now, it should solve this issue.

Interesting, thanks. In a way, it does solve the issue, however it's not super obvious to me.
For example, this still does not work:
() => {
  const data = [
    { property: 'type', description: 'Content type', type: 'secondary | warning', default: '-' },
    { property: 'Component', description: 'DOM element to use', type: 'string', default: '-' },
    { property: 'bold', description: 'Bold style', type: 'boolean', default: 'true' },
  ]
  const [state, setState] = React.useState(0)
  React.useEffect(() => {
    const interval = setInterval(() => {
      setState((old) => old + 1)
    }, 1000)
    return () => {
      clearInterval(interval)
    }
  }, [])
  return (

        Description: {state}

  )
}

However, if you stringify the state in the label, like this:

        Description: {state}

it updates as expected. Is this expected?

Sorry, this is my negligence.
I created an online example using version 1.8.0-canary.10, the table children should be sync correctly now.


Resulting StubTypes return may return warnings

The example created type
if (-not ("Microsoft.Dism.Commands.LogLevel" -as [Type])) {
    Add-Type '
    namespace Microsoft.Dism.Commands
    {
        public enum LogLevel : int
        {
            Errors = 0,
            Warnings = 1,
            WarningsInfo = 2
        }
    }
    '
}

may return warnings saying "the generated type does not define public methods or properties" (translated to english). This may disturb pester results.
 if (-not ("Microsoft.Dism.Commands.LogLevel" -as [Type])) {
    Add-Type '
    namespace Microsoft.Dism.Commands
    {
        public enum LogLevel : int
        {
            Errors = 0,
            Warnings = 1,
            WarningsInfo = 2
        }
    }
    ' -IgnoreWarnings
}

Does not return this warning.

Thank you, I'll add it.


Multiple instances of vite-plugin-checker (one for the frontend and one for the backend)

Is your feature request related to a problem? Please describe.
We have a monorepo with a /frontend and /backend folder. Of course, vite-plugin-checker is installed and setup within /frontend.
I'd like to have multiple instances of vue checker to have both my frontend and backend lint errors to show when opening my web browser. This would be extremely valuable for my project development approach, which is fullstack
Describe the solution you'd like
Either:

Support multiple instances (and therefore multiple badges) of vite-plugin-checker
or
Support multiple eslint as an argument that each point to a different folder (ie: backend and frontend)

Describe alternatives you've considered
I've tried using the concurrent package to run both eslint commands as well as the && operator (to run eslint command twice, one for each folder) but it fails for some obscure reason. I feel like the eslint argument is parsed in a way that doesn't allow to put any kind of command in the "lintCommand" option.
Additional context
No response
Validations

[X] Read the docs.
[X] Read the Contributing Guidelines.
[X] Check that there isn't already an issue that asks for the same feature to avoid creating a duplicate.

I have a similar situation.
Here's the structure:
apps/
`-- example
    |-- eslint.config.js
    |-- package.json
    |-- src
    |-- tsconfig.json
    `-- vite.config.ts
libs/
`-- name-libs
    |-- eslint.config.js
    |-- package.json
    |-- src
    |-- tsconfig.json
    `-- vite.config.ts

Development mode starts from apps/example. But at the same time, I would like ESLint to show errors in the library because they are being developed simultaneously.
Please advise on what I should look into, if this is possible.


Can @jitclass support numpy.array([string])?

like
@jitclass( 'name', types.byte)
class a:
def init(self):
self.name=np.arraty(['a','b'])
will raise Error

Thanks for the report. The answer is "sort of", IIRC there's no support for creating np.ndarray with strings inside Numba jitted functions, but Numba can understand such arrays if they are passed in, e.g.:
from numba import njit, jitclass, types
import numpy as np

spec = {'name': types.Array(types.UnicodeCharSeq(1), 1, 'C')}

@jitclass(spec)
class MyClass(object):
    def __init__(self, data):
        self.name = data

my_data = np.array(['a','b'])
inst = MyClass(my_data)

print(inst.name)

Is there also a workaround for variable length strings? I have a array of strings from Xarray that gets typed as (e.g.) '<U12'  -- where Xarray (or netcdf) sets the size of the dtype on load. I have tried declaring ("cut", string[:]) and passing my strings in as cut=patterns.cut.values.astype(str) get:
 Can only insert {i8*, i64, i32, i32, i64, i8*, i8*}* at [4] in {i8*, i8*, i64, i64, {i8*, i64, i32, i32, i64, i8*, i8*}*, [1 x i64], [1 x i64]}: got [12 x i32]*
... ?

Is there also a workaround for variable length strings? I have a array of strings from Xarray that gets typed as (e.g.) '<U12' -- where Xarray (or netcdf) sets the size of the dtype on load. I have tried declaring ("cut", string[:]) and passing my strings in as cut=patterns.cut.values.astype(str) get:
 Can only insert {i8*, i64, i32, i32, i64, i8*, i8*}* at [4] in {i8*, i8*, i64, i64, {i8*, i64, i32, i32, i64, i8*, i8*}*, [1 x i64], [1 x i64]}: got [12 x i32]*
... ?

@shaunc I think you'd need to dynamically create a jitclass based on the type appearing from Xarray. e.g.
from numba import njit, types
from numba.experimental import jitclass
import numpy as np
import string

def mock_xarray_load():
    return np.array([string.ascii_letters], dtype='<U12')

my_data = mock_xarray_load()

# specialise type on data from Xarray
char_size = np.dtype('<U1').itemsize
uty = types.UnicodeCharSeq(my_data.dtype.itemsize // char_size)
spec = {'name': types.Array(uty, 1, 'C')}

@jitclass(spec)
class MyClass(object):
    def __init__(self, data):
        self.name = data

inst = MyClass(my_data)

print(inst.name)

Thanks! ... hmm... now I have to figure out how to keep mypy happy with my dynamic MyClass.

Closing this issue as the original questions appear to be answered.


How to select the checkbox and the node name separately?

Thanks for implementing such a good component, it's so good.
I recently find how to add a tri-state checkbox according to the official document,

but I want to select the checkbox and the node name separately.
currently when I click the node name, the checkbox state is changed, that's not I want.  what I want is: when I click the checkbox, the checkbox state changed, when I click the node name, the checkbox state keeps unchanged.
I tried to use the wrapper template and the full node template, but failed, not sure how to customized my requirement.
Thanks a lot!

Hi,
you should probably save your own set of checkedNodeIds instead of using the activeNodeIds - and set the checked value accordingly.
Hope it helps

Thanks, I resolved the issue by myself.
This is because the demo from the official document overrides the mouse click event in the IActionMapping.
I removed that override and then my requirement is met.
Thanks again for your great component!

@dannyzhan awesome


Visual Studio 2019 Preview 1.1 and ASP.NET Core 3.0 Not working with Libman 1.0.163

I'm using Visual Studio 2019 Preview 1.1 Version 16.0.0
My project is ASP.NET Core 3.0 preview.18572.1
My LibMan version is 1.0.163+g45474d37ed
In Visual Studio, the context Add / Client-Side Library, I add some library:
Here is the resulting Libman.json:
{
  "version": "1.0",
  "defaultProvider": "cdnjs",
  "libraries": [
    {
      "ProviderId": "cdnjs",
      "DestinationPath": "wwwroot/lib/animate.css/",
      "IsUsingDefaultProvider": true,
      "IsUsingDefaultDestination": false,
      "Name": "animate.css",
      "Version": "3.7.0"
    },
    {
      "ProviderId": "cdnjs",
      "DestinationPath": "wwwroot/lib/kendo-ui-core/",
      "IsUsingDefaultProvider": true,
      "IsUsingDefaultDestination": false,
      "Name": "kendo-ui-core",
      "Version": "2014.1.416"
    },
    {
      "ProviderId": "cdnjs",
      "DestinationPath": "wwwroot/lib/lightbox2/",
      "IsUsingDefaultProvider": true,
      "IsUsingDefaultDestination": false,
      "Name": "lightbox2",
      "Version": "2.10.0"
    },
    {
      "ProviderId": "cdnjs",
      "DestinationPath": "wwwroot/lib/tinymce/",
      "IsUsingDefaultProvider": true,
      "IsUsingDefaultDestination": false,
      "Name": "tinymce",
      "Version": "4.9.1"
    }
  ]
}

When I execute Restore Client-Side Librairies, I Get the message:
Restore operation started...
Restoring libraries for project Windfall
**The library id is undefined**
Restore operation completed with errors
Time Elapsed: 00:00:00.0160558
========== Finished ==========

I expected Libman to restore these libraries in my wwwroot/lib folder and none of these libraries gets restored.

I created a new ASP.NET Core 2.2 project (Latest stable version). In Visual Studio, with the contextual menu Add / Client-Side Library, I added the same libraries and Libman restored my libraries without any problem. The problem was not that Visual Studio was a preview version.
The resulting Libman.json is now very different:
{
  "version": "1.0",
  "defaultProvider": "cdnjs",
  "libraries": [
    {
      "library"<EMAIL_ADDRESS>      "destination": "wwwroot/lib/animate.css/"
    },
    {
      "library"<EMAIL_ADDRESS>      "destination": "wwwroot/lib/kendo-ui-core/"
    },
    {
      "library"<EMAIL_ADDRESS>      "destination": "wwwroot/lib/lightbox2/"
    },
    {
      "library"<EMAIL_ADDRESS>      "destination": "wwwroot/lib/tinymce/"
    }
  ]
}


Data preprocessing

I cannot find in the training logs how the data is being preprocessed for training and testing datasets. Does this mean that normalization only has been applied for both?

Please check. at https://github.com/chenyaofo/image-classification-codebase/blob/6bce2f0224552e1666ae254c362e6f03a38c0f8a/codebase/data/cifar.py#L13


Does this tool work correctly with high/low DPI?

Need to check this, and implement fixes if necessary.

No, it does not.
The bugs are:

icon looks badly scaled
whole app looks like bitmap scaling is applied (at 150%)
listbox rows are not tall enough (at 125%)
"Choose Number" dialog buttons are too large (at 125%)

Adding GetWindowsScreenScalingFactor referenced from https://stackoverflow.com/questions/5977445/how-to-get-windows-display-settings/5978597#5978597 allows a new MultiplyByDPIRatio() function to be used to correct values for High DPI in Windows 10.
For Windows 7, GetWindowsScreenScalingFactor(false) will always return 1, but it seems that the values I wanted to scale, were already scaled correctly before I call MultiplyByDPIRatio().
FormOperations.ScaleSize was changed to work as intended at all DPI settings tested.
The above resolves the remaining High DPI issues in Windows 7 and 10.


Errror when setup

I've clone your project and when I start meteor I've got error:
 While processing files with juliancwirko:s-grid (for target web.browser):
My env: windows 10, nodejs 6.3 meteor 1.3

Hi, try to update s-grid by meteor update juliancwirko:s-grid

thanks


Issue with bindings + commands that enter insert mode

If I bind something that includes a comment.nvim binding, e.g one that adds a comment:
gcATEST you'd expect that TEST would be added to the comment, but it seems that when TEST is input Vim is not yet in insert mode.
gcAaTEST<ESC> tries to account for this, but then I'm left in insert mode at the end.
gcAaTEST yields a comment string TESTa, hinting that you're using a to get into insert mode.
It seems that the command is not done synchronously and this makes is to there's a bit of a race between the function behind gcA and the rest of my mapping.

I am aware of this issue. I am using vim.api.nvim_feedkeys which might be the cause of the issue. It's just I don't know how to fix this or if this is even fixable.

Interesting. The docs say that it should be blocking and that nvim_input is async.
I can repro the issue with this binding: "<cmd>lua vim.api.nvim_feedkeys('jjja', 'n', true)<CR>jjj<ESC>" which should append jjj to three lines down, but instead it goes six lines down.
This doesn't seem correct neovim behaviour.

This could be a bug indeed. Maybe you can report this behavior to neovim.

Filed neovim/neovim#16571 to see whether it's a neovim issue.

@Grazfather Great, thanks! I am also looking forward to the explanation.

Looks like it's expected behaviour, but you can fix by changing the mode from "n" to "ni".
https://github.com/neovim/neovim/issues/16571#issuecomment-988890572

Care to test and make a PR ;)

Done


feat: Localization feature was added #45

Description
Localization feature was added. Please, let me know if I need to change something into my PR.
Maybe tests should be added.
Also, maybe this feature must to change current npm version of tui.image-editor.

Thank you for your contribution to TOAST UI product. 🎉 😘 ✨

Can one of the admins verify this patch?

@junghwan-park
@jungeun-cho
Anyone, give me feedback, please!

The documentation issue is on #110 .
Thank you @anton-marchenko . Really great helpful!.


Add State method that automatically infers state type from grain.

@hazzik Looks great! Thanks for the PR.

Thanks


Installed on M1 Mac: Could not initialize NNPACK! Reason: Unsupported hardware.

Thanks for your error report and we appreciate it a lot.
Checklist

I have searched related issues but cannot get the expected help.
I have read the FAQ documentation but cannot get the expected help.
The bug has not been fixed in the latest version.

Describe the bug
I have followed the installation instructions on this page:
https://mmdetection.readthedocs.io/en/latest/get_started.html
And performed the following Python test as you suggested on that page:
from mmdet.apis import init_detector, inference_detector

config_file = 'rtmdet_tiny_8xb32-300e_coco.py'
checkpoint_file = 'rtmdet_tiny_8xb32-300e_coco_20220902_112414-78e30dcc.pth'
model = init_detector(config_file, checkpoint_file, device='cpu')  # or device='cuda:0'
inference_detector(model, 'demo/demo.jpg')

But I get the following output and error:
Loads checkpoint by local backend from path: rtmdet_tiny_8xb32-300e_coco_20220902_112414-78e30dcc.pth
The model and loaded state dict do not match exactly

unexpected key in source state_dict: data_preprocessor.mean, data_preprocessor.std

[W NNPACK.cpp:64] Could not initialize NNPACK! Reason: Unsupported hardware.

Reproduction

What command or script did you run?

I executed the suggested python script mentioned above.

Did you make any modifications on the code or config? Did you understand what you have modified?

No

What dataset did you use?

The suggested one on the installation page mentioned above.
Environment

Please run python mmdet/utils/collect_env.py to collect necessary environment information and paste it here.

sys.platform: darwin
Python: 3.10.10 (main, Mar 21 2023, 13:41:39) [Clang 14.0.6 ]
CUDA available: False
numpy_random_seed:<PHONE_NUMBER>
GCC: Apple clang version 14.0.3 (clang-14<IP_ADDRESS>.1)
PyTorch: 2.1.0.dev20230406
PyTorch compiling details: PyTorch built with:
  - GCC 4.2
  - C++ Version: 201703
  - clang 13.1.6
  - Intel(R) oneAPI Math Kernel Library Version 2021.4-Product Build 20210904 for Intel(R) 64 architecture applications
  - Intel(R) MKL-DNN v2.7.3 (Git Hash 6dbeffbae1f23cbbeae17adb7b5b13f1f37c080e)
  - OpenMP 201811
  - LAPACK is enabled (usually provided by MKL)
  - NNPACK is enabled
  - CPU capability usage: NO AVX
  - Build settings: BLAS_INFO=mkl, BUILD_TYPE=Release, CXX_COMPILER=/Applications/Xcode_13.3.1.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang++, CXX_FLAGS= -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -DNDEBUG -DUSE_KINETO -DLIBKINETO_NOCUPTI -DLIBKINETO_NOROCTRACER -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DUSE_PYTORCH_METAL_EXPORT -DSYMBOLICATE_MOBILE_DEBUG_HANDLE -DUSE_COREML_DELEGATE -O2 -fPIC -Wall -Wextra -Werror=return-type -Werror=non-virtual-dtor -Werror=braced-scalar-init -Werror=range-loop-construct -Werror=bool-operation -Wnarrowing -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-unused-parameter -Wno-unused-function -Wno-unused-result -Wno-strict-overflow -Wno-strict-aliasing -Wvla-extension -Wno-range-loop-analysis -Wno-pass-failed -Wsuggest-override -Wno-error=pedantic -Wno-error=old-style-cast -Wconstant-conversion -Wno-invalid-partial-specialization -Wno-unused-private-field -Wno-missing-braces -Wunused-lambda-capture -Qunused-arguments -fcolor-diagnostics -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -fno-math-errno -fno-trapping-math -Werror=format -Werror=cast-function-type -DUSE_MPS -fno-objc-arc -Wno-unguarded-availability-new -Wno-unused-private-field -Wno-missing-braces, LAPACK_INFO=mkl, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, TORCH_DISABLE_GPU_ASSERTS=OFF, TORCH_VERSION=2.1.0, USE_CUDA=0, USE_CUDNN=OFF, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=OFF, USE_NCCL=OFF, USE_NNPACK=ON, USE_OPENMP=ON, USE_ROCM=OFF,

TorchVision: 0.16.0.dev20230406
OpenCV: 4.7.0
MMEngine: 0.7.2
MMDetection: 3.0.0+

You may add addition that may be helpful for locating the problem, such as

I installed pytorch in a conda enviroment as you suggested in the installation page mentioned above.

As a summary, here are the steps I have taken s explained on the installation page:
conda create --name openmmlab python=3.8 -y
conda activate openmmlab

conda install pytorch torchvision cpuonly -c pytorch

pip install -U openmim
mim install mmengine
mim install "mmcv>=2.0.0"

mim install mmdet

Then to verify the installation:
mim download mmdet --config rtmdet_tiny_8xb32-300e_coco --dest .

Then created a python file with inside:
from mmdet.apis import init_detector, inference_detector

config_file = 'rtmdet_tiny_8xb32-300e_coco.py'
checkpoint_file = 'rtmdet_tiny_8xb32-300e_coco_20220902_112414-78e30dcc.pth'
model = init_detector(config_file, checkpoint_file, device='cpu')  # or device='cuda:0'
inference_detector(model, 'demo/demo.jpg')

And when I run it, I get the error shown above.
Thanks in advance for any help.

Ok, I figured out the problem. in the Get Started guide:
https://mmdetection.readthedocs.io/en/latest/get_started.html
Instead of using "mim install mmdet" just use the developer option instead:
git clone https://github.com/open-mmlab/mmdetection.git
cd mmdetection
pip install -v -e .
# "-v" means verbose, or more output
# "-e" means installing a project in editable mode,
# thus any local modifications made to the code will take effect without reinstallation.

That worked for me.


Serverless Function runtime provided does not support more than 4KB for environment variables

Hello,
We're getting this error on deployment build of the rust runtime:
Serverless Function runtime provided does not support more than 4KB for environment variables

Reading through Vercel's limitation docs:
https://vercel.com/docs/projects/environment-variables#environment-variable-size
It seems that the limit should be 64kb.
In any case, we didn't get this error for our node.js serverless functions. What could be wrong here?

Hi, nodejs has a large env support which allows it up to 64Kib. The Rust runtime does not support this and is capped at 4Kib. You may want to unselect the option 'automatically expose system env variables'. Hope this helps!

@dglsparsons Hi, is there another way we could tackle this? This options appears to be global and not runtime specific, we don't want our next.js environment to be affected by this


Startup of desktop takes really long

Hi,
the startup of the desktop takes more than 3 minutes for me now:
The to console logs that are before and after about a 3 minute break are those:
INFO  meteorApp:  injecting isDesktop
INFO  meteorApp:  injected successfully

I forked your project and added some more logging.
I narrowed it down to be a caused by this function call:
https://github.com/wojtkowiak/meteor-desktop/blob/2b2a8e71026b5e5b8ebc35433adbef5dfe9ad8d3/lib/meteorApp.js#L781
INFO  meteorApp:  injecting isDesktop info 150249 /packages/underscore.js?hash=cde485f60699ff9aced3305f70189e39c665183c
INFO  meteorApp:  injecting isDesktop info   readFileSync  0 sec
INFO  meteorApp:  injecting isDesktop info   processFileContents 0.002 sec
INFO  meteorApp:  injecting isDesktop info ->   0.003 sec
INFO  meteorApp:  injecting isDesktop info 123128 /packages/meteor.js?hash=b09c234116cd2599026c6a5d872d2896d8ab5885
INFO  meteorApp:  injecting isDesktop info   readFileSync  0 sec
INFO  meteorApp:  injecting isDesktop info   processFileContents 0.001 sec
INFO  meteorApp:  injecting isDesktop info ->   0.001 sec
INFO  meteorApp:  injecting isDesktop info 720 /packages/meteor-base.js?hash=a4d07a6b394e56bbe6ccc773c95e7cdb3434960d
INFO  meteorApp:  injecting isDesktop info   readFileSync  0 sec
INFO  meteorApp:  injecting isDesktop info   processFileContents 0 sec
INFO  meteorApp:  injecting isDesktop info ->   0 sec
INFO  meteorApp:  injecting isDesktop info 726 /packages/mobile-experience.js?hash=8ded3e69a3e367f321ab9a2b52e3ecdd2661a365
INFO  meteorApp:  injecting isDesktop info   readFileSync  0 sec
INFO  meteorApp:  injecting isDesktop info   processFileContents 0.001 sec
INFO  meteorApp:  injecting isDesktop info ->   0.001 sec
INFO  meteorApp:  injecting isDesktop info 42886 /packages/modules-runtime.js?hash=2b888cb7873bacf38cf32f41d44ac3d4fd600b06
INFO  meteorApp:  injecting isDesktop info   readFileSync  0 sec
INFO  meteorApp:  injecting isDesktop info   processFileContents 0 sec
INFO  meteorApp:  injecting isDesktop info ->   0 sec
INFO  meteorApp:  injecting isDesktop info 10694182 /packages/modules.js?hash=7d5b770deb95fb1b99b42ea1b83ddaf49daf3a23
INFO  meteorApp:  injecting isDesktop info   readFileSync  0.041 sec
INFO  meteorApp:  injecting isDesktop info   processFileContents 190.942 sec
INFO  meteorApp:  injecting isDesktop info ->   191.061 sec
INFO  meteorApp:  injecting isDesktop info 346492 /packages/es5-shim.js?hash=adc3c6270d5697523fe2a72e73428390b7eba83a
INFO  meteorApp:  injecting isDesktop info   readFileSync  0.003 sec
INFO  meteorApp:  injecting isDesktop info   processFileContents 0.006 sec
INFO  meteorApp:  injecting isDesktop info ->   0.008 sec
INFO  meteorApp:  injecting isDesktop info 65779 /packages/promise.js?hash=d841a35ac4fe2f1bac3417ccd63d9ed0388b7b38
INFO  meteorApp:  injecting isDesktop info   readFileSync  0.003 sec
INFO  meteorApp:  injecting isDesktop info   processFileContents 0.004 sec
INFO  meteorApp:  injecting isDesktop info ->   0.004 sec

Any idea on how to speed this up?

Hey,
seems that you have a pretty big project. Can you attach your package.json? I will make myself a big project and try to profile that to a much smaller time.

sure
{
  "name": "meteor-starter-kit",
  "private": true,
  "scripts": {
    "start": "meteor run",
    "desktop": "meteor-desktop"
  },
  "dependencies": {
    "apollo-client": "^1.0.2",
    "bcrypt": "^0.8.7",
    "body-parser": "^1.15.2",
    "classnames": "^2.2.5",
    "express": "^4.14.0",
    "graphql": "^0.9.0",
    "graphql-server-express": "^0.6.0",
    "graphql-tag": "^1.3.1",
    "graphql-tools": "^0.11.0",
    "hedron": "^0.6.0",
    "invariant": "^2.2.1",
    "isomorphic-fetch": "^2.2.1",
    "meteor-node-stubs": "~0.2.0",
    "onsenui": "^2.2.1",
    "raf": "^3.3.0",
    "react": "^15.3.1",
    "react-addons-pure-render-mixin": "^15.3.1",
    "react-apollo": "^0.13.2",
    "react-dom": "^15.3.1",
    "react-fontawesome": "^1.5.0",
    "react-ga": "^2.1.2",
    "react-howler": "^3.5.0",
    "react-komposer": "^1.13.1",
    "react-modal": "^1.7.3",
    "react-onsenui": "^1.2.0",
    "react-redux": "^5.0.3",
    "redux": "^3.6.0",
    "redux-logger": "^3.0.1",
    "redux-persist": "^4.6.0",
    "request": "^2.81.0",
    "styled-components": "^1.4.4"
  },
  "devDependencies": {
    "meteor-desktop": "file:../../le3.packages/vilango-meteor-desktop"
  }
}

Ok, my modules.js went to 27mb after adding those 🙂 I need to tweak some regexes. I will release a fix for  that within few hours.

Cool the startup is now super fast again! Thanks!

@wojtkowiak quick question: when do you plan to release 0.6.2 to npmjs?

0.6.2 released

Works like a charm! Thx!


CI builds show the wrong version number

Title.

I've had a (belated) look over this and GitHub tags don't seem to propagate correctly onto the repo's git metadata (namely their timestamps are all over the place).
What we can try is to have the next tag created locally and then pushed to GitHub - hopefully that will push the right timestamp metadata along with it, at least for the tag in question, and should fix versioning on future dev builds, regardless of them being offline or CI.


fix build by exporting eh_personality

This bumps MSRV to 1.75, causing nix-ld-rs to no longer compile with rustc 1.73 (comes with NixOS 23.11)

Ok. Maybe use an old version for now. There are no important fixes in master beyond crate bumps.

The package is also in nixpkgs now, so in the next release this won't happen again.

The package is also in nixpkgs now, so in the next release this won't happen again.

The package is also in nixpkgs now, so in the next release this won't happen again.

Good to know, thanks!


Unity Errors/Crash on Play

[REQUIRED] Please fill in the following fields:

Unity editor version: 2020.1.7f1
Firebase Unity SDK version: 7.0.2
Source you installed the SDK:  Unity Package Manager
Problematic Firebase Component:  Database
Other Firebase Components in use: EDM, auth and Core installed
Additional SDKs you are using: None
Platform you are using the Unity editor on:  Windows
Platform you are targeting:  Android
Scripting Runtime:  IL2CPP

[REQUIRED] Please describe the issue here:
From the very beginning of using Unity and Firebase SDK, I have always gotten the following 2 warnings.
Assembly 'Packages/com.google.external-dependency-manager/ExternalDependencyManager/Editor/Google.IOSResolver_v1.2.163.dll' will not be loaded due to errors: Unable to resolve reference 'UnityEditor.iOS.Extensions.Xcode'. Is the assembly missing or incompatible with the current platform? Reference validation can be disabled in the Plugin Inspector.
And
Assembly 'Packages/com.google.firebase.app/Firebase/Editor/Firebase.Editor.dll' will not be loaded due to errors: Unable to resolve reference 'UnityEditor.iOS.Extensions.Xcode'. Is the assembly missing or incompatible with the current platform? Reference validation can be disabled in the Plugin Inspector.
I have always ignored them as they never kept me from working, they were just very annoying. However, now they are stopping me.
Trying to follow the recommendation to simply disable these plugins as found here: https://docs.unity3d.com/Manual/PluginInspector.html
Doesn't work because the plugins never are "enabled" as to be able to change their settings.
I've even tried updating the external-dependency-manager_version-1.2.163_manifest file to not include the offending IOS plugin which didn't matter.
And I've deleted the IOS plugins as well, and while that got rid of one error...
Assembly 'Packages/com.google.firebase.app/Firebase/Editor/Firebase.Editor.dll' will not be loaded due to errors: Unable to resolve reference 'Google.IOSResolver'. Is the assembly missing or incompatible with the current platform?
...remained.
I've also tried to modify registries via Assets/EDM/Package Manager Resolver/Modify Entries but nothing happens.
Here is my Start method if it matters, taken right from the docs
 private void Start()
    {
        Firebase.FirebaseApp.CheckAndFixDependenciesAsync().ContinueWith(task =>
        {
            var dependencyStatus = task.Result;
            if (dependencyStatus == Firebase.DependencyStatus.Available)
            {
                // Create and hold a reference to your FirebaseApp,
                // where app is a Firebase.FirebaseApp property of your application class.
                app = Firebase.FirebaseApp.DefaultInstance;

                // Set a flag here to indicate whether Firebase is ready to use by your app.
                OnFirebaseInit.Invoke();
            }
            else
            {
                UnityEngine.Debug.LogError(System.String.Format(
                  "Could not resolve all Firebase dependencies: {0}", dependencyStatus));
                // Firebase Unity SDK is not safe to use here.
            }
        });
}

None of this code runs before the above errors, and the Invoke() never gets called, or the LogError
Thanks in advance for any help.

So an update, the code I was using actually was causing the issue I was having.
Switching to this
Firebase.FirebaseApp.CheckAndFixDependenciesAsync().ContinueWithOnMainThread(task =>
        {
            if (task.Exception != null)
            {
                Debug.LogError(System.String.Format(
                   "Could not resolve all Firebase dependencies: {0}", task.Exception));
                return;
            }
            OnFirebaseInit.Invoke();
        });

Did resolve my issue.
However, I would like to know how to get rid of these error messages.
I'm crazy like that, but I don't like simply ignoring errors. Warnings sure, but not errors.

So an update, the code I was using actually was causing the issue I was having.
Switching to this
Firebase.FirebaseApp.CheckAndFixDependenciesAsync().ContinueWithOnMainThread(task =>
        {
            if (task.Exception != null)
            {
                Debug.LogError(System.String.Format(
                   "Could not resolve all Firebase dependencies: {0}", task.Exception));
                return;
            }
            OnFirebaseInit.Invoke();
        });

Did resolve my issue.
However, I would like to know how to get rid of these error messages.
I'm crazy like that, but I don't like simply ignoring errors. Warnings sure, but not errors.


Debugger silently crashes on open up Scripts window

Debugger silently crashes on trying to open up Plugin -> ODbgScript -> Scripts window

Unfortunately, the scripting functionality is not yet completed :( 😢

OK, thanks for prompt reply!
Then I suggest to remove this plugin until it becomes GA?...

Thank you for reporting the bug, and I have already responded to you on GitHub. Please continue to provide feedback on GitHub if you have any further questions or issues. Thank you for your support!
http://www.Ollydbg.com/  <http://www.Ollydbg.com/ >
Cpudbg x96全栈调试器 <http://www.Ollydbg.com/ >
穿山甲信息技术有限公司 <http://www.Ollydbg.com/ >
发件人：Alex @.>
发送时间：2023年7月29日(星期六) 19:23
收件人：CpuDbg/CpuDbg @.>
抄　送：Subscribed @.>
主　题：[CpuDbg/CpuDbg] Debugger silently crashes on open up Scripts window (Issue #4)
Debugger silently crashes on trying to open up Plugin -> ODbgScript -> Scripts window
—
Reply to this email directly, view it on GitHub <https://github.com/CpuDbg/CpuDbg/issues/4 >, or unsubscribe <https://github.com/notifications/unsubscribe-auth/BADMOR3HFHIFPKFNSLX72F3XSTXCBANCNFSM6AAAAAA24OD2FA >.
You are receiving this because you are subscribed to this thread.Message ID: @.>

I guess we need to have a new build, mainly about fix for issues with MBCS

OK, thanks for prompt reply!
Then I suggest to remove this plugin until it becomes GA?...

I will remove all unfinished plugins in the next version, and add them back only when all functionalities are completed. Thank you for your feedback.

I guess we need to have a new build, mainly about fix for issues with MBCS

My entire project currently uses the MBCS character set, and converting it to UNICODE would be a significant undertaking. It is likely that the replacement of MBCS with UNICODE will happen in a future version, such as v0.7.5 or v0.7.6, or possibly even later versions.

The mbcs problem has been fixed in cpudbg v0.7.5 release.😉


Shell: broken highlighting

sr.ht uses pygment to highlight source code.
Example of issue:
https://git.sr.ht/~sircmpwn/shit/tree/master/item/common.sh
Apparently it got confused by '<<'

Thanks for the report. Annoying case, we'd have to recognize Bash's $(()) construct...


nixos/smartd: allow extra cli options for daemon

Motivation for this change
This enables further customization of smartd.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg eval

Forgot to rebase 😞. Will fix.

I think the failure above may have been a race condition of sorts when I pushed twice in quick succession. Is there any way to re-run the checks?

@GrahamcOfBorg eval

Looks good to me. Applied to master (65fb15aaf8b4de22b846bb6de5c5d34c110ece84). Thanks!


Data Sidebar Fields Spacing, List Treatment, Design

FOR P2 - Part 2 of #775 (See #816 for Part 1)

All multi-select fields (eg. General Issues, Specific Topics, Purpose, etc.) to be listed on separate lines.

Get rid of all text in brackets, for example "Governance & Political Institutions (e.g. Constitutions, Legal Systems, Electoral Systems)" would become "Governance & Political Institutions".

Indent any 'overflow text' ie. anything that doesn't fit on one line. For example:

Governance & Political
--> Institutions
Law Enforcement, Criminal
--> Justice & Corrections

Location data does not need to display coordinates.

All values should have the same spelling/grammar/ordering as the data forms - see this document: https://docs.google.com/document/d/1sovR-RCilKwWN2XKxQH7xVo8XDEnh6BSgz7JfNqFy4g/edit#

'True' should be 'Yes', 'False should be 'No'

Amber approved items 1 and 3 above, I'll provide a mockup with spacing specs soon.
2 - if its easy for dethe to make this happen, ok. if not, maybe not worth it.
4 - sounds good.
5 - yes, and admins will be able to edit grammar etc soon! yay!
6 - sounds good.

@jesicarson cool beans! yeah the bracket thing isn't a necessity - your work on 1 and 3 might make things easy enough to read without getting rid of the brackets. I'm just worried about some of the method values which have really long explainers in brackets...
RE 5: I'm actually excited for this!! it's just going to be grammar though, right? not adding missing values?

Good question.. pretty sure you will be able to add values tho. Working on the ui and will confirm with Dethe what is possible and keep u posted. Would be best of admins can also add/ delete values.

@jesicarson there have also been some problems around selecting/deselecting values that were ported from .net (see participedia/usersnaps#207 participedia/usersnaps#203)

Saw those snaps, are you saying they need to be prioritized?
Also regarding values with really long explainers in brackets - BEST course of action would be to shorten them as much as possible.

no, they don't need to be prioritized, I was just wondering if that would be possible for admins to do since they are values.
RE long explainers: we did, any shorter and I don't think people would understand. I think I'm probably overthinking things - let's just see what happens when 1 and 3 are completed

ai file here
most up to date design for cases - sidebar indentation shown.


Crash when duplicating an editor internal node

Tested versions

v4.3.dev.custom_build.4a0160241

System information
Linux
Issue description
Trying to duplicate a node that class isn't exposed by ClassDB results in a segfault.
This happens because the node created by Node::duplicate() is instantiated using ClassDB:
https://github.com/godotengine/godot/blob/4a0160241fd0c1e874e297f6b08676cf0761e5e8/scene/main/node.cpp#L2656-L2661
Backtrace:
================================================================
handle_crash: Program crashed with signal 11
Engine version: Godot Engine v4.3.dev.custom_build (4a0160241fd0c1e874e297f6b08676cf0761e5e8)
Dumping the backtrace. Please include this when reporting the bug to the project developer.

-- END OF BACKTRACE --
================================================================

Steps to reproduce
Search for a node named "SceneTreeEditor" in the editor scene tree using a @tool script and duplicate it:

var q = [get_tree().root]
var scene_tree_editor: Node

while not q.is_empty():
	var node: Node = q.pop_front()

	if node.name == "SceneTreeEditor":
		print(node)
		scene_tree_editor = node.duplicate()
		break

	q.append_array(node.get_children())

Minimal reproduction project (MRP)
MRP: scene_tree_editor_crash.zip (open node.tscn in the editor to reproduce the crash)

According to the stack trace, it doesnt even happen inside _duplicate.

According to the stack trace, it doesnt even happen inside _duplicate.
The segfault happens here:
https://github.com/godotengine/godot/blob/4a0160241fd0c1e874e297f6b08676cf0761e5e8/scene/main/node.cpp#L2903-L2905

because p_copy is null and p_copy's value came from dupe:
https://github.com/godotengine/godot/blob/4a0160241fd0c1e874e297f6b08676cf0761e5e8/scene/main/node.cpp#L2815-L2826

I'll definitely take a look at this.

Thanks!


Using models other than the ones provided gives errors

Hi,
I would like to know the following things:
can you mention the training methodology for all the models that you have provided in this repo(object detection, segmentation and style transfer). Or if these are pretrained models, can you give me the link to the documentation that mentions their training methodology
The models that i  have trained are not working with the code provided. Hence would like to know the training strategy so that i can train the models on my own dataset.
Thanks

Hi,
These are pre-trained models:

Object Detection is taken from Google's MediaPipe project, you can use their model to do "transfer learning" and it should work (I did that at my workplace).
Segmentation is Google's DeppLab model
Style Transfer was contributed by someone, I am not sure how it was trained.

Closing this issue but feel free to ask questions if you have.


Update GHA configurations

This PR adds:

Explicit scoping for token usage in our existing workflows.
Updates close-stale-issues to only run once a day to save worker resources.
Adds CodeQL to run static analysis over code base.

Codecov Report
Patch coverage has no change and project coverage change: -0.03 :warning:

Comparison is base (104226d) 93.36% compared to head (28bd007) 93.33%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2989      +/-   ##
===========================================
- Coverage    93.36%   93.33%   -0.03%
===========================================
  Files           65       65
  Lines        13841    13841
===========================================
- Hits         12923    12919       -4
- Misses         918      922       +4

see 3 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Timeout's not properly caught by poller

raises the following traceback:
Traceback (most recent call last):
  File "/opt/hourglass/.virtualenvs/hourglass/lib/python3.4/site-packages/hourglass_naglitejs/cache.py", line 100, in get
    response = yield from session.get(url)
  File "/opt/hourglass/.virtualenvs/hourglass/lib/python3.4/site-packages/aiohttp/client.py", line 529, in __iter__
    resp = yield from self._coro
  File "/opt/hourglass/.virtualenvs/hourglass/lib/python3.4/site-packages/aiohttp/client.py", line 183, in _request
    conn = yield from self._connector.connect(req)
  File "/opt/hourglass/.virtualenvs/hourglass/lib/python3.4/site-packages/aiohttp/connector.py", line 310, in connect
    transport, proto = yield from self._create_connection(req)
  File "/opt/hourglass/.virtualenvs/hourglass/lib/python3.4/site-packages/aiohttp/connector.py", line 581, in _create_connection
    local_addr=self._local_addr)
  File "/usr/lib/python3.4/asyncio/base_events.py", line 524, in create_connection
    yield from tasks.wait(fs, loop=self)
  File "/usr/lib/python3.4/asyncio/tasks.py", line 331, in wait
    return (yield from _wait(fs, timeout, return_when, loop))
  File "/usr/lib/python3.4/asyncio/tasks.py", line 410, in _wait
    yield from waiter
  File "/usr/lib/python3.4/asyncio/futures.py", line 388, in __iter__
    yield self  # This tells Task to wait for completion.
  File "/usr/lib/python3.4/asyncio/tasks.py", line 286, in _wakeup
    value = future.result()
  File "/usr/lib/python3.4/asyncio/futures.py", line 269, in result
    raise CancelledError
concurrent.futures._base.CancelledError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "contrib/uwsgi/scheduler.py", line 10, in <module>
    scheduler.run_tasks()
  File "./hourglass/scheduler.py", line 46, in run_tasks
    results = self.loop.run_until_complete(asyncio.gather(*tasks))
  File "/usr/lib/python3.4/asyncio/base_events.py", line 276, in run_until_complete
    return future.result()
  File "/usr/lib/python3.4/asyncio/futures.py", line 277, in result
    raise self._exception
  File "/usr/lib/python3.4/asyncio/tasks.py", line 233, in _step
    result = coro.throw(exc)
  File "/opt/hourglass/.virtualenvs/hourglass/lib/python3.4/site-packages/hourglass_naglitejs/cache.py", line 111, in update_objects
    results = yield from self.get(session, url)
  File "/opt/hourglass/.virtualenvs/hourglass/lib/python3.4/site-packages/hourglass_naglitejs/cache.py", line 101, in get
    return (yield from response.json())
  File "/opt/hourglass/.virtualenvs/hourglass/lib/python3.4/site-packages/aiohttp/helpers.py", line 488, in __exit__
    raise asyncio.TimeoutError
concurrent.futures._base.TimeoutError

FWIW, this was not experienced on the latest version, so I am unsure if this issue has been addressed already or not

This has been fixed. Closing this out.


Refactor components to use k8s service discovery

Once we have all of the Deis v2 services defined, all of the components should begin using service discovery (via env vars) to find out where other components are.

done!


Live migration with Affinity

Is your feature request related to a problem? Please describe:
Right now, live migration can not select node for target launcher pod. I think add Affinity for target launcher pod is not difficult. Do we need this feature?
Describe the solution you'd like:
Create VirtualMachineInstanceMigration like this :
apiVersion: kubevirt.io/v1
kind: VirtualMachineInstanceMigration
metadata:
  name: migration-job
spec:
  vmiName: vmi-fedora
  Affinity: XXXXXXX

will add Affinity on target launcher pod after all Affinity like NodeAffinityForHostModelCpuModel has been setted.
Describe alternatives you've considered:
A clear and concise description of any alternative solutions or features you've considered.
Additional context:
Add any other context or screenshots about the feature request here.

I would like to be assigned to this issue.

Hi @Longchuanzheng
Would https://github.com/kubevirt/kubevirt/pull/10086 be enough for your use case?

Hi @Longchuanzheng Would #10086 be enough for your use case?

Looks good, we can set Affinity on vm before migrate.

Although #10086 can set Affinity for a migration, vm or vmi save affinity used in a migration is a little bit subtle. In my opinion, affinity which is used in a migration should only give to the new launcher pod we create in that migration.

I also agree to include affinity in migration. Unlike setting affinity in a virtual machine, it is a subset of setting affinity in a virtual machine.
The migration operation is a one-time action, After migration, it will not affect the affinity of virtual machine settings and there is no need to modify virtual machine configurations.
Alternatively, it can be placed in the annotations or labels of VirtualMachineInstanceMigration to set affinity for the migration target pod without modifying the VM configuration.


Should write type with queued child indentation

See how this code does not indent properly:
sourceFile.addTypeAlias({
    isExported: true,
    name: "Test",
    type: writer => writer.write("string").newLine().write("| number")
});

Marking this as breaking change because some people might be relying on this bug. I'll need to evaluate all the other places in the printers to see where it's not correctly using queued child indentation.

Fixed in #461.


FastReport.Data.MsSql vulnerability CVE-2024-0056 in System.Data.SqlClient

Hello,
Our security scans show that you are using a vulnerable version of the package System.Data.SqlClient v4.8.5, even in the latest version 2024.2.11.
Can you please update the usages of System.Data.SqlClient and publish a new version?
CVE:
https://nvd.nist.gov/vuln/detail/CVE-2024-0056
which is fixed in version 4.8.6 https://github.com/advisories/GHSA-98g6-xh36-x2p7
The CVE has a high score of 8.7 and the hotfix of System.Data.SqlClient was already released 5 months ago

Hello!
The current FastReport.Core.Data.MsSql connection uses the System.Data.SqlClient package version 4.8.6
You should be able to update your FastReport Community Edition to see the changes
Best regards,
Bogdan


Question: DataGrid?

Is there any planned support for DataGrid? I assume it's a heavy beast!!

Hey @bradphelan!
The problem with DataGrid is that it's not maintained actively (or at least it wasn't when I checked about a month ago). If I remember correctly the Avalonia Team considered replacing it, because it currently has no maintainer.
If this changed bindings would be nice! Personally I currently have no plans to add them, but it shouldn't be that complicated (but could be a lot of work) because you can add them gradually.
Let me know if you have further questions :)

Thanks for the feedback. I guess at the moment I don't really need a datagrid. Sizeable columns would be nice. What I'm really missing is a header row for the ListBox. I notice that there is a HeaderedItemsControl class but the ListBox doesn't implement it.
At the moment I'm fudging it by putting in a stackpanel at the top and manually padding the margin to make it line up.

Any suggestions?

Hey @bradphelan,
you should be able to use a HeaderedItemsControl (with data or view items) BUT you need to provide your own template because the default theme does not include one.
Let me know if you need some help with this.

@bradphelan can I close this now ? :)

Yes
On Tue, 17 Mar 2020, 18:47 Josua Jäger<EMAIL_ADDRESS>wrote:

@bradphelan https://github.com/bradphelan can I close this now ? :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AvaloniaCommunity/Avalonia.FuncUI/issues/85#issuecomment-600209755,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAEJ4WIZSBVT4ZA5Y4OPIDRH6ZUZANCNFSM4LMVZUDQ
.


eo-math: release 0.1.3

I made changes to eo-math accordingly to new 0.23.18 eo release.
Should merge this PR before this one

@maxonfjvipon thanks!


Make >> operator by-name like Cats's version is

I added an eager version of the same operation(*>) to support both evaluation types


Create context_processors.py

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

Project coverage is 98.18%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #10   +/-   ##
=======================================
  Coverage   98.18%   98.18%
=======================================
  Files           6        6
  Lines         110      110
=======================================
  Hits          108      108
  Misses          2        2

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


chore(main): release 0.5.1

:robot: I have created a release beep boop
0.5.1 (2023-06-29)
Code Refactoring

update values in Surge configuration files (4e3309b)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/cloverdefa/Rule-Sets/releases/tag/v0.5.1 :sunflower:


String literal type for use in single-property mapped types

Search Terms

string literal indexer mapped type
string literal
mapped type

Suggestion
Add a new type (stringliteral in the example below) that would be a super-type of all string literals, but not unions.
Use Cases
Mapped types are great when building an object out of another object, or when starting with object literals with constant property names. There currently doesn't seem to be an option to build out an object typed with mapped types with variable property names.
Examples
We can build an algebra of base object construction and composition functions.
Note: this example composes plain objects, but the actual motivation is to have more complex structures that internally maintain a typed object and the algebra allows composing them in various ways.
const singlePropertyObject = <N extends string, V>(n: N, v: V): { [P in N]: V } =>
  ({ [n]: v } as { [P in N]: V }); // note: cast required
const compose = <O1 extends object, O2 extends object>(o1: O1, o2: O2) =>
  Object.assign({}, o1, o2);

const oa = singlePropertyObject('a', 'v1'); // { a: string; }
const ob = singlePropertyObject('b', 'v1'); // { b: string; }
const oab = compose(oa, ob);

But this is unsafe - singlePropertyObject can be used to create arbitrary objects with non-optional fields but without value without type errors:
const oaUnsafe = singlePropertyObject<'a' | 'b', string>('a', 'v1'); // { a: string; b: string; }

The current behavior is of course correct, in that { [n]: V } can be implicitly cast only to { [P in N]?: V } - if N is a union, the other members of the union don't have a value set.
What could help is to narrow the generic argument N to a single string literal to prevent unions:
const singlePropertyObject = <N extends stringliteral, V>(n: N, v: V): { [P in N]: V } =>
  ({ [n]: v }); // note: no cast required anymore

Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript / JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. new expression-level syntax)

The type used in the OP { [P in N]: V } is not correct. it is more like:
type Unionize<T extends string, V> = { [P in T]: { [Q in P]: V } }[T];

See https://github.com/Microsoft/TypeScript/issues/18155 for more details.