How to get the PSK client's hint in an incoming request/middleware

Hey folks,
I'm trying to identify the CoAP client later during an incoming request by checking their DTLS identification. I saw that there is an example on the repo to do that using DTLS + PKI, where we register a onNewClientConn method and we can put the client certificate on the request context.
I'm trying do to kind of the same thing but with DTLS + PSK, so I need to access the Client's Hint on the incoming handler, because right now seems like the only place that I can access the PSK Client Hint is on the PSKCallback when configuring the DTLS Server. So what I'm trying to do would be more or less like this:
func handleA(w mux.ResponseWriter, r *mux.Message) {
	clientHint := r.Context.Value("client-hint").([]byte)
	log.Println("Client's Hint:", string(clientHint))

	log.Printf("got message in handleA:  %+v from %v\n", r, w.Client().RemoteAddr())
	err := w.SetResponse(codes.GET, message.TextPlain, bytes.NewReader([]byte("A hello world")))
	if err != nil {
		log.Printf("cannot set response: %v", err)
	}
}

func onNewClientConn(cc *client.ClientConn, dtlsConn *piondtls.Conn) {
	clientHint := dtlsConn.ClientHint() // <-- Missing part, How to get the client PSK hint here ?
	cc.SetContextValue("client-hint", clientHint)
	cc.AddOnClose(func() {
		log.Println("closed connection")
	})
}

dtlsConfig := &piondtls.Config{
	PSK: func(hint []byte) ([]byte, error) {
                // This seems to be the only place available to get the PSK Client Hint
		g.logger.Infof("Client's identifier: %s", hint)
		ctx := context.Background()
		identity := string(hint)
		psk, err := g.deviceAuthService.GetKeyByTypeIdentity(ctx, identity, credentials.CredentialTypes.PreSharedKey)
		if err != nil {
			g.logger.Errorf("identifier lookup err: %v", err)
			return nil, err
		}
		return []byte(psk), nil
	},
	CipherSuites: []piondtls.CipherSuiteID{
		piondtls.TLS_PSK_WITH_AES_128_GCM_SHA256,
		piondtls.TLS_PSK_WITH_AES_128_CCM_8,
		piondtls.TLS_PSK_WITH_AES_128_CCM,
	},
}

But by checking the project repo, I could not figure out a way to expose a ClientHint method on ClientConn or DTLSConn, so I'm creating this issue to see if I can get some guidance.
Thanks in advance and will be happy to collaborate if needed to make this work ( if it makes sense of course )

I want to suggest to contribute the feature to the pion/dtls and extend ConnectionState about member PeerHint []byte or something similar.

I want to suggest to contribute the feature to the pion/dtls and extend ConnectionState about member PeerHint []byte or something similar.

Thanks for the suggestion @jkralik. I created a PR on pion/dtls to add a IdentityHint field to the State struct. If you don't mind, would be awesome if you take a quick look on the PR to see if that is the correct way of doing.
https://github.com/pion/dtls/pull/335

Thanks for the suggestion @jkralik. I created a PR on pion/dtls to add a IdentityHint field to the State struct. If you don't mind, would be awesome if you take a quick look on the PR to see if that is the correct way of doing.
https://github.com/pion/dtls/pull/335

Gonna close this issue because this was solved with the https://github.com/pion/dtls/pull/335. Thanks for the help @jkralik.

Gonna close this issue because this was solved with the https://github.com/pion/dtls/pull/335. Thanks for the help @jkralik.


Can link original field with custom one

Hi~ I use FileMeta to adjust order of picture metadata field shown in explorer.
But I found, even not add any new fields, the metadata is totally been taken over by FileMeta, so, when copy the file to other computer, the default metadata is empty.

File Meta never deletes any property values. So this is what I think must be happening.

On computer A, you add the File Meta property handler and write values to some of the well-known properties of the file. Because the File Meta property handler is in use, the values are written to the alternate stream associated with the file, and not the body of the file itself.

You copy the file to computer B. If the computer has an NTFS drive, the alternate stream is copied with the file. But if you do not add the File Meta property handler on computer B, the meta data will be invisible. If the drive on computer B is not NTFS, or you copied it via a medium that is not NTFS, such as email, or a USB stick formatted as FAT, then the alternate stream will not be copied with the file, and the meta data is not present on computer B.

Does this make sense?
Dijji

Thanks. I understand now.
But in this case - "not add any new fields", "adjust order of picture metadata field"
Can synchronize/map property from alternate stream to original one.
So, when copy, at least, the property would been keep.

The catch is the copy step. What you suggest would require copying meta data properties from the alternate stream of the original to the main body of the copied file. However, this would require knowledge of the format of the file being copied, which File Meta does not have, as well as a hook to intercept the copy operation, which is far as I know, is not provided by Windows.
I am afraid that it is a characteristic of File Meta that if you write properties to the alternate stream, then you must install File Meta on all the computers where you wish to read it.

Eh...
I think what I mean is not the same as you said above.
Not synchronize when copy file, but when edit property.
And for case - property value is the same, only different in organization order in alternate stream and original one.
So, I thought they could be mapped.
User could write property from windows explorer, without the knowledge of format of file.
So, is there any interface could be wrapped? That would be enough to use, and perfect to control by script.


Prometheus Instrumentation

Similar to https://github.com/hyperledger/firefly/issues/189, it'd be great if FFTM had a Prometheus metrics manager and use https://gitlab.com/hfuss/mux-prometheus in the API server middleware so that all the APIs are automatically instrumented.
Then with a metrics manager, implementations of FFTM like EVMConnect can add their own custom metrics.

See https://github.com/hyperledger/firefly/blob/0b6c289ffee87330f578796d5ece8268216e133f/internal/metrics/prometheus.go for the firefly implementation of http and custom metrics.

Resolved in https://github.com/hyperledger/firefly-transaction-manager/pull/41


fix(alert & report): make to fix the issue when recreate report

SUMMARY
[Explore] Page crashes when user deletes report and then tries to create it again
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
BEFORE:
https://media.app.shortcut.com/api/attachments/files/clubhouse-assets/5d8baaab-e1a8-4512-a9c7-12979c8cd1c9/624343bb-1f3f-43be-a418-a2a76d0be05c/Screen Recording 2022-03-29 at 19.33.42.mov
AFTER:
https://user-images.githubusercontent.com/47900232/160701144-d3bbb2cd-bc01-4289-80b1-1e9ce5492f4b.mov
TESTING INSTRUCTIONS
How to reproduce issues

Enable feature flag ALERT_REPORTS
Open a chart
Create a report (with a calendar icon next to chart header)
Delete that report
Create report again
Crash

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[x] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

@rusackas
Would you like to merge this PR into master?


Review

Please review, @sebastiansuer

In this project, two python functions are provided to calculate/visualize the glucose level in the human body using the Sympy library. In the code there are two functions to calculate the glucose level in mg/dl in the human body, the first one returns the value at the input time in minutes and the second one has an additional threshold modeled in that also plots a graph from t=0 to the input time in hours.
The repository includes a pytest workflow to test the validity of the function's outputs. In addition, a readme file, a .gitignore file, and a license file are also present in the repository. There is also a .txt file with the requirements.
The ruff workflow checks the compatibility with pep8 coding style.
Tests

[x] License exists
[x] Create a fork and start a pull-request
[x] Pytests fails as expected
[x] Ruff linter fails as expected

Submission of second homework assignment, @danielhuppmann


Physics Bug Fixing

Delta rotation should be conjugated, otherwise the body will keep on
constantly flipping (if deltaRotation is set).
cannonJS collision of joined objects improved.
Added Conjugate function to the quaternion class.

#1000 milestone reached! yey champagne!

That's a lot!

1000 ftw!
Should have commited more than a bug fix :smile:

That's pretty awesome!


fix typo preventing some packages from being imported. fixes #1816 #task

No good feature goes unpunished by a new bug, eh?  Thanks for resolving so quickly @justinedelson!! 😄

nice find :D

Pull Request Test Coverage Report for Build 4111

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 51.179%

Totals

Change from base Build 4110:
0.0%

Covered Lines:
12766

Relevant Lines:
24944

💛  - Coveralls


[OPENJDK-2093] explicit tzdata-java dependency in container

This is a temporary measure to mitigate a regression in the OpenJDK 11 RPMs for the 2023-07 CPU update (which accidentally dropped the dependency on tzdata-java).
https://issues.redhat.com/browse/OPENJDK-2093
This means we can ship the next container update independently of this issue being addressed in the RPMs.

Looks good to me.


Initial prereq tree creation

This parser bit might be in your area of expertise. Hopefully code makes sense.

not gonna merge, my genius will go unappreciated


Readme + cleanup for umi_hello example

@petergrossmann21 good idea - just added that

noticed a typo in the new test file name - just fixed it


Empty cart flyout: should not display subtotal and "View Cart" button

Reproduce:

Make sure to have no items in cart
Mouse over cart icon to trigger flyout

Issue: The empty cart flyout doesn't differ much from the flyout of a cart with items as it displays a subtotal of 0 and a "View cart" button. This is not very common, the "View Cart" button doesn't make sense. I recommend to adjust the content of the overlay for empty carts, possibly by using a message saying "You have no items in your cart".

An empty cart should not have fly-out. It already displays all the information. There is also no need to go to the cart.

Issue not fixed for poster v2.1.

Continuing this issue with issue #313


client: drain body to reuse connections

google/go-github#317 for reference

LGTM :angel:
/cc @runcom @calavera

not sure why 512 bytes only but LGTM :smile:

all green, merging


Support multi-machine parallel training in a common network

@DavidGOrtega suggested that we could also support parallel training by launching many instances on a single private network, where one of them (leader) would run the CML job and the others (followers) would run the job parts sent by the machine learning framework.
@DavidGOrtega, feel free to edit the main issue description and extend/overwrite it as you want.

This is something that should fix the network itself.
Ideally the scenario would be setting the runner and its peers (non runners) in the same network being the users ML library the one that manages that.
However there are some questions:

align ami vs docker usage

Closed with #237 unless proven otherwise; leader election is not implemented yet.


refactor: change a file name

that's all it does not impact anything but my work

This change is

ok


(TWILL-254) Update to use ContainerId.fromString

This moves away from the deprecated ConverterUitls.toContainerId and updates the build to use Hadoop 2.7.2 to avoid folks grabbing master and having a failure trying to run on clusters with code newer than November 18th, 2014 (release of Hadoop 2.6.0).

I added a few omissions and corrected the parent  pom.xml from my original submission. Now the new code is actually being used and I have updated the default target to Hadoop 2.6 and removed the generic hadoop.version as suggested.

Thank you for the super fast code review @serranom! I think all comments have been addressed and the code should be complete now. Please let me know if you see anything else necessary. It is great working with Twill! Hopefully I can provide some other ideas as I work with it.

The change looks mostly ok to me, except for the change in the pom.xml profile.

Hi @chtyim, thank you for the review. Could you please expand on what you see concerning in the pom.xml? For reference, I need to provide a 2.6 target as it is the first version to provide the ContainerId.fromString() API.

Changes LGTM.


Update create-procedure-transact-sql.md

added optional [NULL] qualifier for procedure parameter

Learn Build status updates of commit 4db4e8e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/t-sql/statements/create-procedure-transact-sql.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

#sign-off


Errors in agent logs

Hello, I am trying to use the collector on an Openshift 4.11 cluster to forward logs to Splunk.  I am using chart version 0.70. Using a pretty basic configuration file.
splunk:
  clusterName: "<REDACTED>"
  splunkPlatform:
    endpoint: "<REDACTED>"
    token: "<REDACTED>"
    index: "<REDACTED>"
    metricsIndex: "<REDACTED>"
    tracesIndex: "<REDACTED>"
    logsEnabled: true
    metricsEnabled: true
    tracesEnabled: false
  logsEngine: otel
  distribution: "openshift"
  environment: <REDACTED>

  agent:
    enabled: true
    resources:
      limits:
        #cpu: 1000m
        memory: 4Gi

  clusterReceiver:
    enabled: true
    resources:
      limits:
        #cpu: 1000m
        memory: 8Gi

  logsCollection:
    containers:
      enabled: true
      containerRuntime: "cri-o"

The agents get the following error messages in their logging. I am wondering if this is an Openshift thing or if I have something misconfigured.
2023-03-16T15:53:22.447Z error prometheusexporter/prometheus.go:139 Could not get prometheus metrics {"kind": "receiver", "name": "receiver_creator", "pipeline": "metrics", "name": "smartagent/kubernetes-scheduler/receiver_creator{endpoint=\"xxx.xxx.xxx.xxx\"}/k8s_observer/ebe4c92e-8b5d-4be3-9945-cd439c5825c8", "monitorID": "smartagentkubernetesschedulerreceiver_creatorendpoint2071306682k8s_observerebe4c92e8b5d4be39945cd439c5825c8", "error": "Get \"http://xxx.xxx.xxx.xxx:10251/metrics\": dial tcp xxx.xxx.xxx.xxx:10251: connect: connection refused", "monitorType": "kubernetes-scheduler"}

2023-03-16T15:53:22.550Z error prometheusexporter/prometheus.go:139 Could not get prometheus metrics {"kind": "receiver", "name": "receiver_creator", "pipeline": "metrics", "name": "smartagent/kubernetes-proxy/receiver_creator{endpoint=\"xxx.xxx.xxx.xxx\"}/k8s_observer/d742b91e-685e-4c35-8197-0b56ebc88e39", "monitorID": "smartagentkubernetesproxyreceiver_creatorendpoint2071306682k8s_observerd742b91e685e4c3581970b56ebc88e39", "monitorType": "kubernetes-proxy", "error": "Get \"http://xxx.xxx.xxx.xxx:29101/metrics\": dial tcp xxx.xxx.xxx.xxx:29101: connect: connection refused"}

I just tested that configuration. It did get rid of the errors. Thanks.

We noticed the same.
The helm chart hardcodes port 10251. However this port has been deprecated.
From: https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG/CHANGELOG-1.17.md#cluster-lifecycle-1
Kubeadm: enable the usage of the secure kube-scheduler and kube-controller-manager ports for health checks.
For kube-scheduler was 10251, becomes 10259.
For kube-controller-manager was 10252, becomes 10257.
(https://github.com/kubernetes/kubernetes/pull/85043, [@neolit123](https://github.com/neolit123))

@jvoravong do you have a timeline when this will be fixed?

We typically do not communicate timelines or commit to resolution times on Github if we can. Please make sure to open a support case if you are encountering this issue, so we can best help you.

Closing this as fixed.

@jvoravong @atoulme
The change implemented here https://github.com/signalfx/splunk-otel-collector-chart/pull/711/files fixes the kubernetes-scheduler issue however, it does not fix the kubernetes-proxy issue.
From original post:

2023-03-16T15:53:22.550Z error prometheusexporter/prometheus.go:139 Could not get prometheus metrics {"kind": "receiver", "name": "receiver_creator", "pipeline": "metrics", "name": "smartagent/kubernetes-proxy/receiver_creator{endpoint="xxx.xxx.xxx.xxx"}/k8s_observer/d742b91e-685e-4c35-8197-0b56ebc88e39", "monitorID": "smartagentkubernetesproxyreceiver_creatorendpoint2071306682k8s_observerd742b91e685e4c3581970b56ebc88e39", "monitorType": "kubernetes-proxy", "error": "Get "http://xxx.xxx.xxx.xxx:29101/metrics\": dial tcp xxx.xxx.xxx.xxx:29101: connect: connection refused"}

Moved to a separate issue, #758


Blog app - integration specs for views and fix n+1 problems

in this [PR] we (@sarazaToussaint, and @ibr5500 ) have:

Solved the N+1 problem, when fetching all posts and their comments, and likes for a user.
Use Capybara to write integration tests for the following pages (User index page, User show page, Post index page, and Post show page).
Update the README file.

Thank you @dasileker for the code review.

Thank @sja-thedude for the code review.


improve(black) : Added italic fontStyle to specific code in black theme

Added italic fontStyle to specific code in black theme #1

Thank you for contributing.


Treesitter support

Can someone please add tree-sitter support to this theme?
It would be awesome!

It is pretty much added, the majority of treesitter defaults are good, I changed what it needed to be fixed.

thanks, shadotheme's back to my go-to colourscheme


Resource resolution fails when specifying uuids in the schema and resource segments

assuming a resource constrained by a schema (other than unconstrained.json)
if the schema is identified by a generated uuid
if the resource is identified by a generated uuid
when trying to fetch the resource using: /v1/resources/{org}/{proj}/{schema_uuid}/{resource_uuid} it returns 404
when trying to fetch the resource using a wildcard for the schema: /v1/resources/{org}/{proj}/_/{resource_uuid} it returns the resource
when trying to fetch the schema using its uuid: /v1/resources/{org}/{proj}/_/{schema_uuid} it returns the schema

And we expect exactly /v1/resources/{org}/{proj}/{schema_uuid}/{resource_uuid} == /v1/resources/{org}/{proj}/_/{resource_uuid}


refactor(instructor module): create new files and modify patch.py

:rocket: This PR description was created by Ellipsis for commit 450c30c4a94e57ef0dee30596a77cfb66902fdc4.

Summary:
This PR refactors the instructor module by creating new files for handling responses, retrying functions, and utility functions, and modifies patch.py to use new classes and deprecates the apatch function.
Key points:

Refactored the instructor module.
Created response.py to handle response model type hint and process OpenAI response.
Created retry.py to implement retry logic if the response is not valid.
Created utils.py for utility functions.
Modified patch.py to use new InstructorOpenAI and InstructorAsyncOpenAI classes.
Deprecated apatch function in patch.py.

Generated with :heart: by ellipsis.dev

moved some stuff around, cause i figured merging your PR would have some fat conflicts

it looks like i deleted patch(create)

Do these pass as is? I have some changes locally I have to push up

you'll run into conflicts due to my new simple types stuff


Updating pipelineTask spec to accomodate for CustomRun and Run

Feature request
As mentioned in TEP-0114, we aim to transition
v1alpa1.Run to v1beta1.Custom. After a lot of discussion regarding implementation for this feature,
some of which can be found in this PR, the team is
moving forward adding the CustomRun CRD and deprecating v1alpha1.Run after some time.
This will be done by adding a field, runAPIVersion, in the taskRef spec to distinguish which run
is being used. If the field is populated with v1alpha1 then the Run CRD will be used, and on the other hand if
v1beta1 is used, then the CustomRun CRD will be used.
Things that will need updating:

taskRef struct
pipelineRun reconciler
tests and adding end-to-end tests for CustomRun

/assign

PipelineTask's TaskRef already has an APIVersion field - why do you need a new one?

we need to know which version of Run or CustomRun to create. the existing apiVersion filed refers to the apiversion of the custom task to use. I don't think we can just move to always creating a  CustomRun or a Run - this would mean a user would have to make sure all of their custom tasks have been upgraded to use a reconciler that supports CustomRun. With a field that allows setting the apiVersion of the Run/CustomRun to be created - a user can use custom tasks that use the older v1alpha1 Run as well as tasks that use CustomRun

Aaaaaah - good point. I forgot about that!

two downsides - 1. we are adding a custom task specific field into task ref and 2. having two apiVersions within the same taskRef is going to be confusing - open to other ideas here as well!

Can we write an example of usage here to see how it would look like ?
I am not sure if I completely follow the reasoning here, and I "fear" we are adding fields and complexifying things for very little gain. We also need to take into account that this is for an alpha API (both in its "usage" and its definition) — and with alpha api, we can pretty much.. switch on/off from one release to the other (if we communicate and announc it properly), so changing as well.
The problem we "are" facing here is that, the day we introduce v1beta1.CustomRun, some Custom Task may/will not work, because they only watch v1alpha1.Run objects. Do we agree on this ?
If I understand correctly, what is proposed here, is to let the user choose which object to use, which I feel is not the best user experience, nor a long-term solution. We are basically doing this for a couple of release and them removing it, right ? We are basically asking the users to set the correct "internal" object (and version) for a given time.
In my opinion, what we need to do, is to give a bit of time for Custom Task to support v1beta1.CustomRun objects. Let's take a "foo" custom task, that I am the owner. Let's also assume, for this example, that v1beta1.CustomRun is introduced and used in 0.42. With that Custom Task, I want to support multiple versions of Tekton Pipeline, let's say from v0.35.x to v0.42.x (and above). For me to support those versions, I need to be able to reconcile v1alpha1.Run and v1beta1.CustomRun, for a time being. And when I do not want to support Tekton Pipeline version below v0.42.x, I can just drop the v1alpha1.Run support.
I do not really see what we are asking the user to specify something, where it can be transparent for them. Of course, this means, all custom task "provider" will need to add support for v1beta1.CustomRun in a timely fashion (before v0.42.0) if they want to still work when this gets released. But I would rather put the burden of this transition on Custom Task authors than on pipeline's users.
This is very similar to what we face from time to time in our consumption of kubernetes APIs — e.g. autoscaling/v2beta1 HorizontalPodAutoscaler or PodSecurityPolicy that got deprecated in vX and remove in vY (possibly making tekton pipeline not work on given versions of k8s).

If I understand correctly, what is proposed here, is to let the user choose which object to use, which I feel is not the best user experience, nor a long-term solution. We are basically doing this for a couple of release and them removing it, right ? We are basically asking the users to set the correct "internal" object (and version) for a given time.

I think we can just set defaults here right? We default to v1alpha1.Run for x releases and then switch to v1beta1.CustomRun after. In case a user wants to use a custom task that has not upgraded to supporting v1beta1.CustomRun, they still have some way of using it.

I do not really see what we are asking the user to specify something, where it can be transparent for them. Of course, this means, all custom task "provider" will need to add support for v1beta1.CustomRun in a timely fashion (before v0.42.0) if they want to still work when this gets released. But I would rather put the burden of this transition on Custom Task authors than on pipeline's users.

I think this is a valid approach though I am unsure how realistic it is to expect all custom tasks to be upgraded to support both v1alpha1 Run and v1beta1 CustomRun. In any case, I think it is useful for the pipeline user to have the option to use a custom task that only supports v1alpha1 Run

I think we can just set defaults here right? We default to v1alpha1.Run for x releases and then switch to v1beta1.CustomRun after. In case a user wants to use a custom task that has not upgraded to supporting v1beta1.CustomRun, they still have some way of using it.

Yes, that would be a valid approach imho.

I think this is a valid approach though I am unsure how realistic it is to expect all custom tasks to be upgraded to support both v1alpha1 Run and v1beta1 CustomRun. In any case, I think it is useful for the pipeline user to have the option to use a custom task that only supports v1alpha1 Run

Indeed. Also, I am not sure how much Custom Task there is in the wild, but they depend on alpha api, it's in the contract that things might break 🙃


Test cases failing in test_add.py

As new features were added to add.py, these code changes were not accounted for in the old test cases. After successful update of the test cases, the test cases and workflows are running as expected.


Test dependencies

Allow additional dependencies when compiling tests.

I've been making some adjustments to our python converter. It's not perfect but it'll take a bulk of work out of you.


creating model framework

standardizing some of the framework between jack's bert model & my code from friday

Updates:

created custom datasets/dataloaders as well as training/functions etc to work across all models
pipeline seems to be working

What isn't working:

model structure is off for the ones I've made (can talk more about this) so we can divide up the models and use the dataloaders to experiment with which ones can work best.

@jgibson517 or anyone else - if this looks good to you, we can merge into main before we collab tonight!

@claireboyd  do we want to combine our ReviewDataset classes, they should be basically the same expect in the tokenizer and the label class, the BERT model takes [0,1] and [1,0] vs 0, 1.

@jgibson517  yeah sure! i need to pivot to other stuff until we meet but one of us can combine them tonight/tomorrow

also i should change it so it just takes it a built tokenizer and doesn't need to build it within the class - i went back and forth on that but especially if we are using the same across both then that design makes more sense


Weights at epoch 64 not found

I was running the script of run_demo_pytorch.py using the command:
python run_demo_pytorch.py --config_filename=data/model/pretrained/METR-LA/config.yaml
This is what I got:
Traceback (most recent call last):
File "run_demo_pytorch.py", line 33, in
run_dcrnn(args)
File "run_demo_pytorch.py", line 18, in run_dcrnn
supervisor = DCRNNSupervisor(adj_mx=adj_mx, **supervisor_config)
File "/home/cyd/DCRNN_PyTorch/model/pytorch/dcrnn_supervisor.py", line 50, in init
self.load_model()
File "/home/cyd/DCRNN_PyTorch/model/pytorch/dcrnn_supervisor.py", line 93, in load_model
assert os.path.exists('models/epo%d.tar' % self._epoch_num), 'Weights at epoch %d not found' % self._epoch_num
AssertionError: Weights at epoch 64 not found
Could you please upload the 'models/epo64.tar' to the repo? I hope to reproduce the MAE results demonstrated in README. Thx!

I meet the same problem, have you solved it?

@zhenhuascut @victorsoda
"'models/epo64.tar' " is not provided in this branch........

Thanks, I have solved it.

Still don't see epo64.tar ... It would be great if you could upload it. Many thanks!

Still don't see too... Would you upload it then I could reproduce your work? Thanks a lot.

I also got the same trouble, but at 51 epoch. Can you help me solve it? Thanks a lot.

Setting train/epoch to 0 in data/model/dcrnn_test_config.yaml solves the problem.

Setting train/epoch to 0 in data/model/dcrnn_test_config.yaml solves the problem.

Thanks! I tried successfully.

Hi everyone, I am sorry I have been unable to reply - @htn274 is correct - the epoch is intended to be from where to resume and I have since then shut down the server (I am a poor student 🥼) and haven't been able to retrieve the weights at epoch 64 - if you just set it to 0 and train, you will be able to reproduce the results

I don't believe setting train/epoch to 0 is a solution for this problem. As you train the model by the code, the trained models are saved as models/epo0.tar, epo1.tar,... etc.  So I ask you to add the best model (epoXX.tar) at the models/ for METR-LA, and PEMS-BAY then we can test with these models

@semink that is incorrect - the model only tries to load existing weights if epoch > 0, so by setting epoch=0 will do the job as @baosws helpfully pointed out.
I am closing this issue for now - the solution is to train it and once it has trained, set the correct epoch number in config.yml and that should work

@chnsh this didn't work for me. I still get the same error

@chnsh actually I ended up doing the change in data/model/pretrained/METR-LA/config.yaml and that did it


Default default_date gets selected even if it falls outside the accebtable date range

If I set the "max" to yesterday (for example,) today's date will still populate the select when the picker opens even though it's outside the acceptable range.
I imagine this is also a problem when default date is set by the user, but presumably in that case they'd make sure to set a valid default_date.

Would you be so kind to prepare demo on JsFiddle/JsBin?

http://jsfiddle.net/7fy1d25s/
When no default_date is set or an invalid date is set, it won't let you "select" the date, but it'll put the default in the input field.
as long as there's a valid default_date set there's no problem.

This bug should be addressed in a6f74a8: https://jsfiddle.net/7fy1d25s/2/
Could you try build from master branch before I release it (there was quite a lot changes)?


Stack block selection using tab in stack view messes up the UI

When a clr-stack-block is selected using Tab instead of Enter, the UI gets messed up.
Attaching the screenshot for the same:

Hello, for us to be able to review this issue we need an isolated reproduction of this issue to triage. Please provide a working Stackblitz demo that has the minimal elements needed to reproduce this issue.

Also, please tell us which version of Clarity are you using.
I'm almost sure we had a fix for this some time ago.

Yes, this must have been fixed with this commit:
https://github.com/vmware/clarity/commit/ee73df36e5ed356d7f9236042d61ecbd4d44af31
Please open a new issue with more details if you're using an up to date version of Clarity and still observing the problem.


MM-32556: s/Management/Collaboration

Summary
Let it be called, "Incident Collaboration." Note that this PR does not touch the plugin id, making it much easier to upgrade to this version.

Ticket Link
Fixes: https://mattermost.atlassian.net/browse/MM-32556

The repository rename is effected by https://git.internal.mattermost.com/infrastructure/github-management/-/merge_requests/30.

In test..


Allow inclusion in older projects

I set iOS and macOS back one version as fits my needs, but could be pushed further. This lets me use Flow within a project using @available checks for Flow usage.

Great addition! Thank you 🎉


Select buffer lines when clicking the gutter, not screen lines

Fixes https://github.com/atom/atom/issues/5738:

d21ae2c modifies the click handlers for the gutter to select buffer lines, rather than screen lines. Clicking the gutter for any screen line of a wrapped buffer line selects the whole buffer line (previously, only the clicked screen line would have been selected).

03a064b adds more tests for checking how click handlers for the gutter behave when soft wrap is turned on (I duplicated all the existing tests, turned on soft wrap, and then tweaked them to test the new functionality).

Before :tv: (notice screen lines being selected):

After :tv: (notice buffer lines being selected):

cc @nathansobo for :eyes:

Kicked CI again to see if the build failures were a fluke. The code looks good to me. Thanks for taking care of this @izuzak.


Use the service connection instead of PAT to kick off docs CI runs

The msdocs-apidrop-connection service connection will replace the azuresdk-apidrop-devops-queue-build-pat that was being used to kick off docs CI runs.
The docindex run was run against this refs/merge for both testing and also initial approval of the service connection.

I'm going to override check enforcer as the core - ci run doesn't test anything related to this change.

/check-enforcer override


Start Up Probe does not support Exec command

Please provide us with the following information:

This issue is a: (mark with an x)

[X] bug report -> please search issues before submitting
[ ] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

Issue description
Use Case: Java Apps , Spring Boot does not expose an endpoint for the StartUp Probe
see at https://learn.microsoft.com/en-us/azure/templates/microsoft.app/2022-03-01/containerapps?pivots=deployment-language-bicep#containerappprobe :
The only option is to set httpGet to check the Probe but no way to specify just a Command to exec like it is possible in Azure Spring Apps:
        probeAction: {
          type: 'ExecAction'
          command: [
            'ls'
            '-al'
            '/tmp/app/*.jar'
          ]
        }

resource AdminServerContainerApp 'Microsoft.App/containerApps@2022-03-01' = {
  name: adminServerContainerAppName
  location: location
  identity: {
    type: 'UserAssigned'
    userAssignedIdentities: {
      '${adminServerIdentity.id}': {}
    }
  }
  properties: {
    managedEnvironmentId: corpManagedEnvironment.id
    configuration: {
      activeRevisionsMode: 'Multiple'
      ingress: {
        allowInsecure: true
        external: true
        targetPort: 8888
        traffic: [
          {
            latestRevision: true
            weight: 100
          }
        ]
        transport: 'auto'
      }
      registries: [
        {
          server: ACR.properties.loginServer
          identity: adminServerIdentity.id
        }
      ]
      secrets: [
        {
          name: 'appinscon'
          value: appInsights.properties.ConnectionString
        }
      ]
    }
    template: {
      containers: [
        {
          command: [
            'java', '-javaagent:${applicationInsightsAgentJarFilePath}', 'org.springframework.boot.loader.JarLauncher', '--server.port=8888'
          ]
          env: [
            {
              name: 'SPRING_PROFILES_ACTIVE'
              value: 'docker,mysql'
            }
            {
              name: 'APPLICATIONINSIGHTS_CONNECTION_STRING'
              secretRef: 'appinscon'
            }
          ]
          image: imageNameAdminServer
          name: adminServerContainerAppName
          probes: [
            {
              failureThreshold: 5
              httpGet: {
                path: '/manage/health/liveness' /* /actuator */
                port: 8081
                scheme: 'HTTP'
              }
              initialDelaySeconds: 30
              periodSeconds: 60
              successThreshold: 1
              timeoutSeconds: 3
              type: 'Liveness'
            }
            {
              failureThreshold: 5
              httpGet: {
                path: '/manage/health/readiness' /* /actuator */
                port: 8081
                scheme: 'HTTP'
              }
              initialDelaySeconds: 30
              periodSeconds: 60
              successThreshold: 1
              timeoutSeconds: 3
              type: 'Readiness'
            }
            {
              failureThreshold: 5

              initialDelaySeconds: 30
              periodSeconds: 60
              successThreshold: 1
              timeoutSeconds: 3
              type: 'Startup'
            }
          ]

Steps to reproduce
Expected behavior [What you expected to happen.]
Bicep should offer an option to exec a command in any Probe instead of HTTP GET
Actual behavior [What actually happened.]
Deployment fails because there is no HTTP endpoint to call for the Start Up Probe
Screenshots
N/A
Additional context
IaC

@kendallroden , see also issue #118 #448

@anthonychu  can I get an update on this issue please ?
I have another use case : I am building WindUp on ACA to help customer Java migration and  RedHat had designed WindUp Probe using exec, see https://github.com/windup/windup-openshift/blob/master/templates/src/main/resources/web-template-empty-dir-executor.json#L322 :
                                "livenessProbe": {
                                    "exec": {
                                        "command": [
                                            "/bin/bash",
                                            "-c",
                                            "${JBOSS_HOME}/bin/jboss-cli.sh --connect --commands='/core-service=management:read-boot-errors()' | grep '\"result\" => \\[]' && ${JBOSS_HOME}/bin/jboss-cli.sh --connect --commands=ls | grep 'server-state=running'"
                                        ]
                                    },
                                    "failureThreshold": 3,
                                    "initialDelaySeconds": 120,
                                    "successThreshold": 1,
                                    "timeoutSeconds": 10
                                },

Closing this for now, if it's still a priority please reopen this.


Android support

This PR adds CMake compilation support (c-api) for Android. From my tests it does function correctly, no unexpected crashes, so no code modifications were made. It uses the ANDROID CMake variable to transparently detect when Android compilation is being done and sets flags and some other environment variables accordingly. The environment variables are necessary for some crates (as far as I could tell). BUILD_SHARED_LIBS might have to be off.
Some relevant issues are #3867, #3877, #3921.

I should note BUILD_BYPRODUCTS ${WASMTIME_BUILD_PRODUCT} was required to add support for ninja, ninja ignores ExternalProject_Add if no external product is specified. A little additional code was required to set that variable.

I only sort of barely understand cmake, but is this sort of configuration standard? I would expect all this toolchain configuration to be configured by whomever is building wasmtime-c-api rather than hardcoded into cmake files. I feel like this sort of configuration is also necessary for cross-compiled builds and I don't think we'd want to hardcode everything for that per se?
This is all sort of on the side and fine to add but I'm not sure if you have a particular end goal for the cmake support in mind or how many other platforms might be added.

I suppose the reason it's so excessive here is that rust doesn't fit nicely into the CMake ecosystem so I have to manually pass it the compiler, archiver and locations of some required shared libs, especially because as noted in the comments rust incorrectly handles some shared libraries for later versions of the NDK. This process would be a lot cleaner if the calling CMake project could modify the compiler and libraries used by wasmtime but with ExternalProject you can't do that.

Could this perhaps be generalized to avoid duplication? Could --target always be passed perhaps? Or could RUSTFLAGS-the-env var be set to avoid duplicating the build command?

My PC won't have internet access for a couple of days but I'll see about that change. I do think this effortlessly pulls wasmtime into Android projects that don't understand rust but if it's messy I understand

Made a slight change to deduplicate ExternalProject_Add and make the CMakeLists.txt cleaner.

Cool, thank you!


[BUG] Facebook app :)

I went to official https://connect.sensiolabs.com/
Profile => Your online Profile => Facebook => Link your facebook account
and this is what I got

:)

Facebook integration has been removed as of today.


[nrf noup] fixed check of reset addres for the net app

The primary slot for the network core is emulated in RAM.
Its flash_area hasn't got relevant boundaries.
Therefore need to override its boundaries for the check.
ref. NCSDK-13532

@hakonfam @sigvartmh
Is it possible to have right offset autogenerated?

This will require PM changes - Better to fix this in hacky way for the release.


[dashboard][color theme] chart color are changed when refreshing dashboard

chart color are changed when refreshing dashboard
How to reproduce the bug

Go to dashboard with multiple charts
Applied color theme to dashboard, for example supers color
Refresh dashboard
Observe color of each chart
See error

Expected results
Refresh dashboard did not change color of chart
Actual results
Refresh dashboard did not change color of chart
Screenshots
https://user-images.githubusercontent.com/81597121/135937535-016cd613-cbf5-48a4-9849-7b0010a68176.mov
Environment
(please complete the following information):

browser type and version: Chrome
superset version: master
python version: python --version
node.js version: node -v
any feature flags active:

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Add any other context about the problem here.

@geido Let's create a new project in Projects for the color consistency ticket in one place

Hello @amitmiran137 I can move this to the top of my list. No problem

@geido  thanks friend ✌

For us this happens again after migrating from 2.0.1 to 3.0.2. Can anyone else confirm?

For us this happens again after migrating from 2.0.1 to 3.0.2. Can anyone else confirm?

Thanks for the heads up. We are investigating the issue. Please try to disable AVOID_COLORS_COLLISION and see if that fixes it

Thanks for the reply @geido . I did test it and it seemed to have helped int that regard but this introduced a new issue: two different labels in the same chart could have the same color (which imho is even worse).
Seems to me like one has to decide with v3 between one of the two:

changing colors of labels when refreshing a dashboard page
or the possibility of two labels in one chart having the same color

I don't know how others handle it currently on their production environment but for us this is fairly annoying and we couldn't find any solution to it. I'm surprised that only few other people are reporting this issue

Happening with us also. @geido in SS 3.0.x

Actual for 3.0.3

Actual for 3.1.0
I will test disabling AVOID_COLORS_COLLISION to see if its a valid workaround, or if the @D3nn3 collateral issue happens

Thanks everyone for reporting these issues. Working on a longer term solution here https://github.com/apache/superset/pull/27439


Please consider using redis-py

By using your own hand-rolled connection to redis you cut off using unix domain sockets and taking advantage of other features of redis.  This cache looks nice, but there is no reason for you to be making your own interface to redis.

Yeah I wrote that as an exercise to be honest. With the new PR coming in (I just have to fix up some tests) you are able to specify any backend that implements an interface (therefore it will be a cache decorator for any backend, not just redis). You will be able to define the interface to use with redis-py as well.


Auto update mods

As the title says, maybe a feature that would automatically update mods would be nice.
If someone for example tries to host a game, with V2 of a certain mod, but V5 is the latest, the client should recognise V2 as hidden in the vault (maybe just a simple name-check, since that's how automatically hiding old versions works already?), and auto-download the latest version before the game launches.
This would reduce the hassle for mod users, while also preventing the old versions from circulating after they have been updated. (as they are no longer obtainable from the vault anyway)

This is a very complex topic with different layers of problems.

We have people with very limited internet connections (bandwith and total limits). Those people won't be happy about auto-updating larger mods (think about blackops for example, that is 300mb), especially if they just play against AI or friends.
Mods in game are handled completely different than mods in FAF client. Afaik the game just scrapes everything in the mod folder and lists it. From the games perspective there is no such thing as a version or a hidden flag (there is a blacklist, but that can only be updated with game patches and I don't know how it works exactly). The FAF client can query our API and knows about hidden stuff and newer version. The connection between client and game is the filename.

What we can control is which mods we show in the create game dialog in the FAF client. However what is shown in the game is based on what the user has on disk. So if the user starts a game without mods and then selects them in the ingame lobby we wouldn't notice.
The alternative would be to update all mods before launching the game, which is problematic because of point 1. Also this still doesn't solve what shows up ingame.
So the client would need to delete older mods from disk to not have them show up in the game anymore. But for replays you might still need these files. So deleting them is not an option.

Backwards compatibility: If a newer version is broken or incompatible with another mods, there would be no way to get it back, as it would be auto-updated. I don't know if this is a common issue in FAF.

So the question is what would be the best solution? I don't know.
Potential ideas:

Make available updates on mods appear in the notification screen.
Move outdated mods in a dedicated directory and mount them only in the init.lua on playing replays (not sure if this is possible and/or requires code changes in the game)

I like your last idea.
I think for point 1 we could add an option in the settings : "slow internet"
otherwise mod update is default.
And yeah "updates" would consist of FAF moving all outdated mods to folders that FA doesn't "scrape".
as for point 4 I guess the right choice would be to make the mod dependency chain outdated as a whole : the responsibility has to go back to the mod maintainers. FAF client can't be expected to be responsible for that.
I think with such a setup we would have be the best of all worlds.
It would however have the "downside" of requiring more rigor from the modders

I will only check on mods that are actually selected in the current game. Nobody should be playing outdated mods. If your connection is slow you should not play mods... Also mods normally do not change to frequently

Haha well you should check the recent mod uploads there are some hefty mods that get at least weekly updates

Well if you plays those mods that on you then I guess

Or the contributors need to change their behaviour.


feat(breaking)!: support API endpoint override

There are two things I'll need to be careful about with respect to this change:

Updating @google-cloud/common has broken our backoff when multiple agents with the same deployment are running. We don't have automated tests for this, and I'll need to test this manually.

It looks like this change is a breaking change for our agent (since setting baseApiUrl no longer has an effect; I know our probers that run against our test environment will be broken by this). Since we recently bumped out major version (for dropping Node 6), merging this definitely requires careful consideration for us. I don't want to need to bump our major version too frequently.

Added a "do not merge" label because I'd like to get a release of @google-cloud/profiler out before merging this change.


Fix of absolute header for page anchors do not work.

"btw when you navigate to this anchor link about cupids you won't see full paragraph due to "absolute header" :" (c) the greatest test engineer.

@fauvel please review.

So this will replace SK_Layout_Default.scrollTo()
Isn't the if ($anchor.length) still needed? Otherwise we'll always scroll to 0 if there's no anchor?

So this will replace SK_Layout_Default.scrollTo()

Yes.
@njam please review.

lgtm

So what was the problem? page was not the right one in this context?

It's hard to explain only in the context of CM =( Lets watch how it will behave.


can I use async await feature?

when I use async await feature I got some error
[0] WARNING in ./~/es6-promise/dist/es6-promise.js
[0] Module not found: Error: Cannot resolve module 'vertx' in /Longo/node_modules/es6-promise/dist
[0]  @ ./~/es6-promise/dist/es6-promise.js 132:20-30
[0]
[0] WARNING in ./~/require_optional/index.js
[0] Critical dependencies:
[0] 63:18-42 the request of a dependency is an expression
[0] 71:20-44 the request of a dependency is an expression
[0] 78:35-67 the request of a dependency is an expression
[0]  @ ./~/require_optional/index.js 63:18-42 71:20-44 78:35-67
[0]
[0] WARNING in ./~/require_optional/LICENSE
[0] Module parse failed: /Longo/node_modules/require_optional/LICENSE Unexpected token (1:40)
[0] You may need an appropriate loader to handle this file type.
[0] SyntaxError: Unexpected token (1:40)
[0]     at Parser.pp.raise (/Longo/node_modules/acorn/dist/acorn.js:923:13)
[0]     at Parser.pp.unexpected (/Longo/node_modules/acorn/dist/acorn.js:1490:8)
[0]     at Parser.pp.semicolon (/Longo/node_modules/acorn/dist/acorn.js:1469:73)
[0]     at Parser.pp.parseExpressionStatement (/Longo/node_modules/acorn/dist/acorn.js:1994:8)
[0]     at Parser.pp.parseStatement (/Longo/node_modules/acorn/dist/acorn.js:1772:188)
[0]     at Parser.pp.parseTopLevel (/Longo/node_modules/acorn/dist/acorn.js:1666:21)
[0]     at Parser.parse (/Longo/node_modules/acorn/dist/acorn.js:1632:17)
[0]     at Object.parse (/Longo/node_modules/acorn/dist/acorn.js:885:44)
[0]     at Parser.parse (/Longo/node_modules/webpack/lib/Parser.js:902:15)
[0]     at DependenciesBlock.<anonymous> (/Longo/node_modules/webpack/lib/NormalModule.js:104:16)
[0]  @ ./~/require_optional ^\.\/.*$
[0]
[0] WARNING in ./~/require_optional/README.md
[0] Module parse failed: /Longo/node_modules/require_optional/README.md Unexpected character '#' (1:0)
[0] You may need an appropriate loader to handle this file type.
[0] SyntaxError: Unexpected character '#' (1:0)
[0]     at Parser.pp.raise (/Longo/node_modules/acorn/dist/acorn.js:923:13)
[0]     at Parser.pp.getTokenFromCode (/Longo/node_modules/acorn/dist/acorn.js:2831:8)
[0]     at Parser.pp.readToken (/Longo/node_modules/acorn/dist/acorn.js:2526:15)
[0]     at Parser.pp.nextToken (/Longo/node_modules/acorn/dist/acorn.js:2518:71)
[0]     at Parser.parse (/Longo/node_modules/acorn/dist/acorn.js:1631:10)
[0]     at Object.parse (/Longo/node_modules/acorn/dist/acorn.js:885:44)
[0]     at Parser.parse (/Longo/node_modules/webpack/lib/Parser.js:902:15)
[0]     at DependenciesBlock.<anonymous> (/Longo/node_modules/webpack/lib/NormalModule.js:104:16)
[0]     at DependenciesBlock.onModuleBuild (/Longo/node_modules/webpack-core/lib/NormalModuleMixin.js:310:10)
[0]     at nextLoader (/Longo/node_modules/webpack-core/lib/NormalModuleMixin.js:275:25)
[0]  @ ./~/require_optional ^\.\/.*$

I found the reason  😳， thank you
https://github.com/chentsulin/electron-react-boilerplate#externals

@FoghostCn Can you expand on the solution? I see you suggest putting 3rd party libs in externals, but which ones?

@jefffriesen I just put mongodb to webpack.config.base.js
externals: [
    // put your node 3rd party libraries which can't be built with webpack here
    // (mysql, mongodb, and so on..)
    'mongodb'
  ]

I see. I thought you enabled async/await through externals. Did you get that working?

it works well ,no more third lib


dgTool error (failure opening sd files)

It's able to backup the firmware, but for so reason it can't downgrade due to being unable to read the sd files...apparently. It doesn't make sense when it uses the boot.nds to work in the first place. What can I do to resolve this issue?

Change another SD card....just copy and paste then test again

have you put the 11.1firm(or 11.0 frim) and 10.4firm in the dgTool?

Gonna put a correct firm also

Probably SD card corruption. Try fixing it or using another SD card.


Excel-to-JSON is using shape-json

@ansteh
Excel-to-JSON is using shape json, https://github.com/he-yang/Excel-to-JSON
Thank you for your work.

Thank you for your notice.
One of the greatest pleasures in life is to contribute in some way.

Great work folks had a few Excel files to convert to JSON and the addin really helped. it becomes very messy to read excel via OleDB


Real-time data

What is the proper way to update the table with streaming real-time data? Currently re-rendering the whole table just because a cell value changed is not practical and causing perf issues. Any suggestions?

Well, this is the "React" way of doing things.  The cells and rows are properly keyed to allow React to only re-render what has changed. If it is causing performance issues then you may need to consider a different package.
However, this question is probably better asked over on the #react-table Slack channel as the github issues are not monitored by a wide range of people for implementation questions.

Thanks for the quick reply.


#594 -- Fix travis-ci config

Codecov Report

Merging #595 into master will not change coverage by %.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master      #595   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines            8         8
=========================================
  Hits             8         8

Continue to review full report at Codecov.

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


Try out openjdk8

Try out openjdk8 for simpler staging setup

Coverage remained the same at 70.875% when pulling fb5feeeebb1a7c9eb17aa6172e68bd12c0cdb70f on feature/openjdk into d9a17c10c5fc8e9b3954f1a5dfcdc018a7bcadf1 on develop.

Note to self: test a few plugins and cromwell as well

Try this after 1.3, looks like Cromwell and wdl4s may work, but will likely have to be after the rush


support image/x-icon mime types

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Currently, there is no way to read image/x-icon, which is the standard mime type for favicon.ico files when passed to jimp as a Buffer.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Since these are typically just png or jpeg images, it would be great if jimp was able to read them, or at least try.
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 encountered the same issue as yours.
I solve it by using https://github.com/LinusU/ico-to-png to get the png buffer.


README.md tutorial updates

Type of issue: bug report

What is the current behavior?

Two things in the README need a refresh.
First is that the prerequisites should be updated to include yosys else a handful of things fail during sbt test.
Second is that when attempting to run a custom transform tutorial, we get told the following:
/utils/bin/firrtl -td regress -tn rocket --custom-transforms tutorial.lesson1.AnalyzeCircuit
------------------------------------------------------------------------------
Error: Option --top-name failed when given ''. Option '--top-name/-tn' was removed as part of the FIRRTL Stage refactor. Use an explicit input/output options instead.
This error will be removed in 1.3.
Try --help for more information.
------------------------------------------------------------------------------

Thanks for identifying and reporting these issues. :+1:
#1113 fixes these narrowly. However, with the advent of 3.2/1.2, this is likely the time for a broader README overhaul. Chisel is in much better shape than it was before after (Chisel#1106) and associated PRs. FIRRTL likely needs similar treatment. Additionally, I need to get the new Chisel website better integrated with this, notably, using tut to make sure that problems like the --custom-transforms example are tested.


dotnet release notes for the 2021-05 release

This is an automated pull request to gather the dotnet release notes for the 2021-05 release. See Producing Release Notes for details of the processing.

Hi @pakrym - please check the text for the packages you own. If everything looks OK, simply approve the PR. If you want to make changes, please add it as a suggested commit.

We should open these with "auto-merge" label. cc @weshaggard

We should open these with "auto-merge" label. cc @weshaggard

Two reasons to not want that -

It will cause cause a lot of churn with multiple PRs in a single day
It will cause merge with the first approval, while we want to wait a bit to give all package authors with updates a chance to take a look.


1.0.0 - API 3.0.0

Please Update API for 3.0.0-ALPHA1

It isn't released fully yet

Not 3.0, but 3.0.0-ALPHA1


How do i add this node to my workflow?

Hi. I have the custom node installed and loaded to  the UI. My CSV styles are all available in the dropdown menu which is great, however I cant figure out how which nodes to attach the positive and negative connections to.

You attach these to your conditioning nodes and sampler.


Added .NET Framework 4.7.1 info

Another topic missing from PR #3415
/cc @rpetrusha, @preetikr, @vivmishra

Thanks @rpetrusha for the review. See what you think now. It seems that the previous version was inaccurate since 4.7 was also supported on the anniversary edition, right?

Yes. Evidently this was written based on the initial 4.7 release, which was available only on Windows 10 Creators Update and was not available on downlevel systems. Then when 4.7 was released for general availability, we forgot to revise it.

@mairaw, LGTM.


Camera Controller doesn't work properly

Whenever I try to wire the CameraController, it gives me this error. (yep, this is with the pure gmod_wire_cameracontroller.lua) I double-checked everything, tried to get this working for days, and also spent hours trying to understand the lua code, and almost got it working, but it's still a mess. (If you want me to upload the "fixed" file, just ask.)
 (For this error I only wired the CamController's Active to an E2 chip output, but that only contained a value of 1.)
(PS: Looked the problem up on google, didn't find anything, and I didn't see any active issues here w/ this problem. Pls help, I think this is an important part of wiremod.)

It says this:
Wire error (Entity [90][gmod_wire_cameracontroller]):
addons/trunk/lua/entities/gmod_wire_cameracontroller.lua:399: bad argument #1 to 'WriteFloat' (number expected, got function)
stack traceback:
    [C]: in function 'WriteFloat'
    addons/trunk/lua/entities/gmod_wire_cameracontroller.lua:399: in function 'SendPositions'
    addons/trunk/lua/entities/gmod_wire_cameracontroller.lua:416: in function 'SyncSettings'
    addons/trunk/lua/entities/gmod_wire_cameracontroller.lua:664: in function 'EnableCam'
    addons/trunk/lua/entities/gmod_wire_cameracontroller.lua:773: in function <addons/trunk/lua/entities/gmod_wire_cameracontroller.lua:770>
    [C]: in function 'xpcall'
    addons/trunk/lua/wire/server/wirelib.lua:62: in function 'TriggerInput'
    addons/trunk/lua/wire/server/wirelib.lua:495: in function 'Wire_Link'
    addons/trunk/lua/wire/server/wirelib.lua:674: in function 'Link_End'
    addons/trunk/lua/weapons/gmod_tool/stools/wire_adv.lua:160: in function 'func'
    lua/includes/modules/net.lua:32: in function <lua/includes/modules/net.lua:17>

It seems the "Distance" variable on the cam controller is a function for some reason. It should be a number.
Write this in your console, while looking at the camera controller
lua_run_cl PrintTable(debug.getinfo(LocalPlayer():GetEyeTraceNoCursor().Entity.Distance))
this should tell us where this function is defined. Post the results here.

So, first of all, I do have the latest wiremod version, as I said before, I've had this problem for days - so I've set up TortoiseSVN, got the latest Wire from here, and when that didn't work, jumped into the lua code myself. (it's not that far jump for me)
Well, I did what you've asked for;
Wire error (Entity [556][gmod_wire_cameracontroller]):
addons/trunk/lua/entities/gmod_wire_cameracontroller.lua:391: bad argument #1 to 'WriteFloat' (number expected, got function)
stack traceback:
[C]: in function 'WriteFloat'
addons/trunk/lua/entities/gmod_wire_cameracontroller.lua:391: in function 'SendPositions'
addons/trunk/lua/entities/gmod_wire_cameracontroller.lua:412: in function 'SyncSettings'
addons/trunk/lua/entities/gmod_wire_cameracontroller.lua:659: in function 'EnableCam'
addons/trunk/lua/entities/gmod_wire_cameracontroller.lua:766: in function <addons/trunk/lua/entities/gmod_wire_cameracontroller.lua:763>
[C]: in function 'xpcall'
addons/trunk/lua/wire/server/wirelib.lua:62: in function 'TriggerInput'
addons/trunk/lua/wire/server/wirelib.lua:492: in function 'Wire_Link'
addons/trunk/lua/wire/server/wirelib.lua:671: in function 'Link_End'
addons/trunk/lua/weapons/gmod_tool/stools/wire_adv.lua:160: in function 'func'
lua/includes/modules/net.lua:32: in function <lua/includes/modules/net.lua:17>
] lua_run_cl PrintTable(debug.getinfo(LocalPlayer():GetEyeTraceNoCursor().Entity.Distance))
linedefined	=	156
currentline	=	-1
func	=	function: 0x3ea73028
isvararg	=	false
namewhat	=
lastlinedefined	=	161
source	=	@lua/autorun/slvbase/shared/slvbase_sh_meta.lua
nups	=	0
what	=	Lua
nparams	=	2
short_src	=	lua/autorun/slvbase/shared/slvbase_sh_meta.lua
Tough I don't think this tells too much. But: in the lua code, at the line 391, ( dist ) is an interesting thing, if set to ( distance ) then the new error is almost the same saying:
Wire error (Entity [556][gmod_wire_cameracontroller]):
addons/trunk/lua/entities/gmod_wire_cameracontroller.lua:391: bad argument #1 to 'WriteFloat' (number expected, got nil)
the rest is the same. As far as I remember, if distance is defined as (at line 23) " local distance = 1 " instead of " local distance = 0 " then the error vanishes, and a new one appears.
Hope this helps with the problem. ^^

The problem is you have an addon called "slvbase" that is overwriting the "Distance" variable of other entities.

Thanks, (last time I was a little bit tired) removing this was really simple, though I didn't even knew about the existence of that thing...

I get the same error.
So I hit the lua_run_cl PrintTable(debug.getinfo(LocalPlayer():GetEyeTraceNoCursor().Entity.Distance)) console
This is the result.
] lua_run_cl PrintTable(debug.getinfo(LocalPlayer():GetEyeTraceNoCursor().Entity.Distance))
currentline	=	-1
func	=	function: 0x5c7a6db0
isvararg	=	false
lastlinedefined	=	161
linedefined	=	156
namewhat	=
nparams	=	2
nups	=	0
short_src	=	lua/autorun/slvbase/shared/slvbase_sh_meta.lua
source	=	@lua/autorun/slvbase/shared/slvbase_sh_meta.lua
what	=	Lua
[Wiremod] lua/entities/gmod_wire_cameracontroller.lua:482: bad argument #2 to '__mul' (number expected, got function)

__mul - [C]:-1
UpdateOutputs - lua/entities/gmod_wire_cameracontroller.lua:482
3. unknown - lua/entities/gmod_wire_cameracontroller.lua:573

[Wiremod] lua/entities/gmod_wire_cameracontroller.lua:482: bad argument #2 to '__mul' (number expected, got function)

__mul - [C]:-1
UpdateOutputs - lua/entities/gmod_wire_cameracontroller.lua:482
3. unknown - lua/entities/gmod_wire_cameracontroller.lua:573

[Wiremod] lua/entities/gmod_wire_cameracontroller.lua:482: bad argument #2 to '__mul' (number expected, got function)

__mul - [C]:-1
UpdateOutputs - lua/entities/gmod_wire_cameracontroller.lua:482
3. unknown - lua/entities/gmod_wire_cameracontroller.lua:573

[Wiremod] lua/entities/gmod_wire_cameracontroller.lua:482: bad argument #2 to '__mul' (number expected, got function)

__mul - [C]:-1
UpdateOutputs - lua/entities/gmod_wire_cameracontroller.lua:482
3. unknown - lua/entities/gmod_wire_cameracontroller.lua:573

[Wiremod] lua/entities/gmod_wire_cameracontroller.lua:385: bad argument #1 to 'WriteFloat' (number expected, got function)

WriteFloat - [C]:-1
SendPositions - lua/entities/gmod_wire_cameracontroller.lua:385
3. SyncSettings - lua/entities/gmod_wire_cameracontroller.lua:407
4. DisableCam - lua/entities/gmod_wire_cameracontroller.lua:616
5. ClearEntities - lua/entities/gmod_wire_cameracontroller.lua:846
6. unknown - lua/entities/gmod_wire_cameracontroller.lua:601

What is the solution? I can not use the cam controller because of this error

What addon is causing this? I can't find any addon that would resemble, or initialize or has an author by the name of "slv". Only thing I can think of is Gbombs 5. Whats causing it?

Alright after some google research, I found what SLV Base is, and I dont have it. Any idea what is causing this? I have a few addons when I start a singleplayer, but aren't in my addon list or addons folder.

dunno
try doing an in-file search with an editor like notepad++ or sublime text to check for "slv" (or similar) in all files in your gmod folder.

According to Google, it is Silverfag's SLVBase addon (search for it in your addons and lua folders).

I dont have this addon, And I completely searched my steam folder entirely, then my gmod.

@Tyler13700 If you could, post whatever
lua_run_cl PrintTable(debug.getinfo(LocalPlayer():GetEyeTraceNoCursor().Entity.Distance))
prints out in console here

@bigdogmat doesn't it need to be run on the server

Sorry for bumping this issue, but I was having the same problem. After looking inside a few gma's, I found that the conflicting mod is Half Life Renaissance. Apparently, it includes the slvbase. So if anyone else is still having that problem, try unsubscribing that mod.

Can we make a big list of "ADDONS KNOWN TO BE INCOMPATIBLE WITH WIREMOD"? Although some authors are quick to make fixes, some don't care, but either way people keep pestering about it.

I was thinking about making a small thing in wiremod which detects when known conflicts are installed and notifies the user, but I've not gotten around to it because effort

There are a lot of places that could be overridden by other mods. Would this even be feasible?

I think the key word is  known conflicts, we can easily check for ones we know about.

I don't think there are that many mods conflicting with wiremod. I have lots of addons and everything works fine, except for the slvbase thing. As far as I know, mods conflict when there are functions, variables or files with the same names, or when someone does stupid things like making a function named "Distance", but if you keep your stuff properly organized, that shouldn't happen. Anyway, a list of known conflicts would be cool, even better if it's notified inside the game. I've seen quite many people complaining about this error, and HL Rennaissance is quite popular (even tho is really outdated). Fixing the conflict should be quite easy, but I think nobody is working on that addon anymore.

Having a function named distance is fine as long as it's only on your entities. It breaks because slvbase tries to add this function to EVERYONE'S entities


installs wget which is used by the wait-for script

Description
installs wget which is used by the wait-for script
Type of change
(Delete what does not apply)

Bug fix (non-breaking change which fixes an issue)

Checklist
(Delete what does not apply)

[X] Have you checked to there aren't other open Pull Requests for the same update/change?
[X] Have you added an explanation of what your changes do and why you'd like us to include them?

tests are failing with 502 ;(


(#116) Avoid FILE directive in gain plugin example and docs

Per the discussion in #116, I'm proposing here that we avoid __FILE__ as it seems to have non-standard behavior. Instead, I'm using a preprocessor directive for the GainPlugin example, and I've updated the docs to alert new users that they will need to fill in the correct path to their bundle file.

Looks sensible to me sir!

Looks reasonable to me too!


lara_cms.sql is incomplete

After loading lara_cms.sql file into the lara_cms database, an error showed about table products not having the tags fields. I added the tags fields as varchar(255), but the following error showed:
SQLSTATE[42S02]: Base table or view not found: 1146 Table 'lara_cms.orders' doesn't exist (SQL: SELECT YEAR(FROM_UNIXTIME(UNIX_TIMESTAMP(time_created))) as year, MONTH(FROM_UNIXTIME(UNIX_TIMESTAMP(time_created))) as month, COUNT(id) as num FROM orders GROUP BY YEAR(FROM_UNIXTIME(UNIX_TIMESTAMP(time_created))), MONTH(FROM_UNIXTIME(UNIX_TIMESTAMP(time_created))) ASC )
So far I found that the orders table, as well as the carousel and carousel_translations are none existent.

Sorry. I forget to update database file.... It is fixed with commit - https://github.com/kirilkirkov/Shopping-Cart-Solution-Laravel/commit/a1073cd304cde60908ac270b6c8144a7d6120458  @robernet


[rv_dm,dv] Make lots of changes to vseqs to allow reset during a run

This is tied in with changes I'm making locally to allow a reset to happen more easily in the middle of a run (for the stress_all_with_rand_reset testpoint). When the base vseq is configured to allow these resets to occur, the changes in this PR are needed to avoid things falling apart.
This PR has quite a few commits (sorry!). All of them are new except for the first one, which is from #24138 (which I hadn't been chasing reviews for: oops).

About the millions of separate commits: it was a consequence of me doing stuff in bits (fixing a vseq at a time). I hadn't realised quite how ridiculous things had got! I've squashed the relevant commits together now.


plugins/vim-illuminate: init + test

Add support for vim-illuminate.

Also, I would rather see this plugin named illuminate instead of vim-illuminate.

Also, I would rather see this plugin named illuminate instead of vim-illuminate.

What is the guideline for this? Just use name that you use in lua require?

What is the guideline for this? Just use name that you use in lua require?

Unfortunately we do not have a strong and deterministic guideline on the naming.
The current codebase already has some discrepancies in the naming of plugins.
If there is no clash or other exception, I tend to prefer sticking to the namespace name (i.e. telescope, harpoon, treesitter...).
So in this case, I would go for illuminate.

What is the guideline for this? Just use name that you use in lua require?

Unfortunately we do not have a strong and deterministic guideline on the naming. The current codebase already has some discrepancies in the naming of plugins.
If there is no clash or other exception, I tend to prefer sticking to the namespace name (i.e. telescope, harpoon, treesitter...). So in this case, I would go for illuminate.

Ok, makes sense, renamed :+1:


Istanbul ignore statements not being respected

Issue

/* istanbul ignore next */ statements are not being respected in files, even with removeComments: false set in tsconfig.json

Expected behavior

function* infiniteCounter() {
  let i = 0;
  /* istanbul ignore next */
  while(true) { // <-- coverage should be ignored here
    yield i;
    i++;
  }
}

Related:

https://github.com/facebook/jest/issues/4538
https://codecov.io/gh/dsifford/academic-bloggers-toolkit/src/master/src/js/dialogs/pubmed/index.tsx#L157
https://github.com/dsifford/academic-bloggers-toolkit

ts-jest doesn't handle coverage at all so I can't put a finger right away on what might be going wrong.
Can you create a minimal repo that demonstrates this issue? It'll help debug and, if the problem is in ts-jest, fix the issue.

@kulshekhar Sure thing. I'll see if I can get to that sometime tomorrow afternoon (USA EST)

@dsifford is this something jest handles correctly that we're breaking in ts-jest?

@kulshekhar Let's put this on hold.. Just made a minimal duplicate of my issue and for whatever reason I can't get it to error. So something fishy must be happening on my end.
I'll reopen if necessary once I figure out what the heck is going on.

Hmmm...I'm seeing something similar.

I am also experiencing the same issue with istanbul comments not being respected in coverage reports. removeComments in tsconfig.json didn't seem to have any effect.
I'm using jest 21.2.1 and ts-jest 21.1.2

Feel free to provide a minimal repo where it errors on typescript but not on regular jest.
As we don't handle coverage, I have a sneaking suspicion that it's not an issue on our end.

Changing the removeComments to false fixed it for me. All we need is for ts-jest to override that option to be true. I had a go at doing that but overriding it in the preprocessor doesn't seem to work.

Same problem here. Flipping my removeComments did not change anything with versions:
jest: 22.4.3
ts-jest: 22.4.2

This also happens to me.
ts-jest: 22.4.2
jest: 22.4.2

Same problem here. Flipping my removeComments did not change anything with versions:
ts-jest: 22.4.6
jest: 22.4.4

@kaskar2008 you should update both jest and ts-jest

Also happening to me
<EMAIL_ADDRESS><EMAIL_ADDRESS>

@jbccollins please open a new issue and provide a link to a minimal repo where we can reproduce your issue, so that we can fix it then ;-)

Same for me
"jest": "~24.7.1",
"ts-jest": "^24.0.2",

Can we reopen it?

"jest": "^24.7.1", "ts-jest": "^24.0.2",

Solution is
// istanbul ignore next

It appears because TS also makes transpilation according to compilerOptions:target.  So you need tsconfig.test.json with target: es6 for null coalesce and other stuff, tsconfig.build.json for project with target: es3. Until istanbul has no code block ignore we cannot ignore optional chaining

Another workaround specifically for while(true) is to use for (;;) { instead.

@tjenkinson Changing my code for a code-coverage tool is too much. Comments are already enough mess IMO.

Any solution to this now?

Use  coverageProvider :  'babel' instead of 'v8'  in jest.config.
https://jestjs.io/docs/cli#--coverageproviderprovider

It happens to me as well
jest: 28.1.1
babel-jest: 27.4.5


Mediated transfer: Target is not online

Happens in this PR: https://github.com/raiden-network/light-client/pull/392
Steps to Reproduce

Start raiden client (here: 0x30FcED14DbcbB91A6c2ED31C42C4240056974CEd)
Start 2nd raiden client (here: 0x1D3810ce695AbbD6Eb70FC4ccBbDde664667e071)
Start light client (here: 0x360611bB4562C2C0B65FB6C2D552ED2d1cBd55cc)
Create & fund channel between 0x360611bB4562C2C0B65FB6C2D552ED2d1cBd55cc and 0x30FcED14DbcbB91A6c2ED31C42C4240056974CEd
Create & fund channel between 0x30FcED14DbcbB91A6c2ED31C42C4240056974CEd and 0x1D3810ce695AbbD6Eb70FC4ccBbDde664667e071
In Light Client click TTT token => Pay
Try to pay 0x1D3810ce695AbbD6Eb70FC4ccBbDde664667e071

Expected Result

I get a route and can do a mediated transfer
There is a network request to the PFS in the network tab

Actual Result

No request to PFS in network tab
PathFindFailed:

"Error: PFS: target 0x1D3810ce695AbbD6Eb70FC4ccBbDde664667e071 not online
at MergeMapSubscriber.eval [as project] (webpack-internal:///../raiden-ts/dist/path/epics.js:66:19)
at MergeMapSubscriber._tryNext (webpack-internal:///../raiden-ts/node_modules/rxjs/_esm5/internal/operators/mergeMap.js:71:27)
at MergeMapSubscriber._next (webpack-internal:///../raiden-ts/node_modules/rxjs/_esm5/internal/operators/mergeMap.js:61:18)
at MergeMapSubscriber.Subscriber.next (webpack-internal:///../raiden-ts/node_modules/rxjs/_esm5/internal/Subscriber.js:63:18)
at ThrowIfEmptySubscriber._next (webpack-internal:///../raiden-ts/node_modules/rxjs/_esm5/internal/operators/throwIfEmpty.js:37:26)
at ThrowIfEmptySubscriber.Subscriber.next (webpack-internal:///../raiden-ts/node_modules/rxjs/_esm5/internal/Subscriber.js:63:18)
at TakeSubscriber._next (webpack-internal:///../raiden-ts/node_modules/rxjs/_esm5/internal/operators/take.js:46:30)
at TakeSubscriber.Subscriber.next (webpack-internal:///../raiden-ts/node_modules/rxjs/_esm5/internal/Subscriber.js:63:18)
at ReplaySubject._subscribe (webpack-internal:///../raiden-ts/node_modules/rxjs/_esm5/internal/ReplaySubject.js:76:28)
at ReplaySubject.Observable._trySubscribe (webpack-internal:///../raiden-ts/node_modules/rxjs/_esm5/internal/Observable.js:50:25)
at ReplaySubject.Subject._trySubscribe (webpack-internal:///../raiden-ts/node_modules/rxjs/_esm5/internal/Subject.js:100:51)
at ReplaySubject.Observable.subscribe (webpack-internal:///../raiden-ts/node_modules/rxjs/_esm5/internal/Observable.js:36:22)
at TakeOperator.call (webpack-internal:///../raiden-ts/node_modules/rxjs/_esm5/internal/operators/take.js:30:23)
at AnonymousSubject.Observable.subscribe (webpack-internal:///../raiden-ts/node_modules/rxjs/_esm5/internal/Observable.js:31:31)
at ThrowIfEmptyOperator.call (webpack-internal:///../raiden-ts/node_modules/rxjs/_esm5/internal/operators/throwIfEmpty.js:23:23)
at AnonymousSubject.Observable.subscribe (webpack-internal:///../raiden-ts/node_modules/rxjs/_esm5/internal/Observable.js:31:31)"

Additional Infos (Optional)

Matrix: All clients use https://transport04.raiden.network
PFS: All clients use https://pfs-goerli-with-fee.services-test.raiden.network
Raiden Client: v0.200.0-rc1

Issue was fixed within #392
We need to fetch the availability before fetching suitable routes


Anyone can give me compiled kisskiss Please

Sorry, it's not provided in that form. You'll need to compile it yourself.


filter with / erroneously switches panes on number input

Repro:

Have list of items open on pane 1
Filter / for text
Type "PROJ-2" - on pressing 2, it switches to the worklog list, despite being in filter mode.

Expected:

It should remain in the issue list view, since that's what I'm filtering on.

Thank you for your work. This util is fantastic.

Yes, I just grabbed 1.1.0 and can confirm it's fixed. Thank you. :) 💚


Sent UST from BSC to TERRA and has not received it yet

Hello, I sent UST from BSC to TERRA through Terra Bridge, but I have not received it.
Transaction:
https://bscscan.com/tx/0x14c2d256f3b362a12fb959e40e1b7a0541318986af57e8d1657f423097d4662c
My TERRA adress:
terra1v64ycvp7jqeya5w4ts6karqhk7qg95g0thvkr2
Thanks and please help me ASAP!

Hello @Paul7474 . A support ticket has been opened for you, with support ticket ID FLV9829. Talk to a Terra Dev on the Live support page so that you can get a response and transaction looked into. Thank you. CLICK HERE

Hello, I sent UST from BSC to TERRA through Terra Bridge, but I have not received it.
Transaction: https://bscscan.com/tx/0x14c2d256f3b362a12fb959e40e1b7a0541318986af57e8d1657f423097d4662c
My TERRA adress: terra1v64ycvp7jqeya5w4ts6karqhk7qg95g0thvkr2
Thanks and please help me ASAP!

Hello @Paul7474 . A support ticket has been opened for you, with support ticket ID FLV9829. Talk to a Terra Dev on the Live support page so that you can get a response and transaction looked into. Thank you. CLICK HERE


Remove Aim Hand?

smro81
Is it possible to just remove the right hand, but keep the mouse tracking?

This is actually extremely easy to implement, but via the original config I don't think it's possible. Though my big fix update will address these kinds of issues quite heavy.

Correction, via the original config it is possible. You can change the opacity of the hand by setting the alpha value to whatever you want. 0 for completely transparent, while 255 is opaque.


Asset detail page doesn't show all the asset owners

The Total Supply section and the sum of quantity in Asset Owners section doesn't match.
It seems that only the first page of the Asset Owners are loaded.

I really was just web surfing and this app popped up I really don't know about the bug 🤷🏽♂


pull request for e2e test at 11/10/2016 07:45:09

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fix PR #28 and improve positioning fpr Internet Explorer

This adds the changes I commited to dist/chartist-plugin-tooltip.js to src/scripts/chartist-plugin-tooltip.js.
Bonus: Tooltip doesn't jump around on IE :).

Thank you @hansmaad :)


fix(deps): update terraform-module

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/cbr/ibm (source)
module
patch
1.23.3 -> 1.23.5

terraform-ibm-modules/kms-all-inclusive/ibm (source)
module
patch
4.15.3 -> 4.15.4

Release Notes

terraform-ibm-modules/terraform-ibm-cbr (terraform-ibm-modules/cbr/ibm)
v1.23.5
Compare Source
Bug Fixes

Set enforcement mode as disabled for EN by default, as SMTP API does not support report mode.(#497) (e76f4b4)

v1.23.4
Compare Source
Bug Fixes

mqcloud is removed from default target service list and fscloud doc is updated (#495) (5af76ee)

terraform-ibm-modules/terraform-ibm-kms-all-inclusive (terraform-ibm-modules/kms-all-inclusive/ibm)
v4.15.4
Compare Source
Bug Fixes

deps: update terraform ibm to latest for the deployable architecture solution (#528) (2daf902)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

:tada: This issue has been resolved in version 1.9.6 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


empathy, anemonin

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 100
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[a,e]

/trunk merge


Hard-to-use histogram

Histogram needs many samples to give us a meaningful data.
However, current spec generates one data observation per scrape.

FIxed a program in #18.
I will fix the document in another PR.


More Linux distribution and their package manager

Refer to repology and all their package managers

1st level：Deepin/Ubuntu/AUR/Guix
2nd level：Solus/ChromeBrew/MacPorts/Slackware/Gentoo
3rd level：FreeBSD/OpenBSD/F-droid


Renamed latestRelease to latestReleaseTag

Current dependencies on/for this PR:

main

PR #83   👈

This comment was auto-generated by Graphite.


HTML inside data-error

Hi,
I'm trying to insert html inside the error messages, i insert it and i add data-html="true", but still doesn't work...any idea?
Thanks

You have to put that data attribute on the form element.
On Sunday, November 15, 2015, therayess<EMAIL_ADDRESS>wrote:

Hi,
I'm trying to insert html inside the error messages, i insert it and i add
data-html="true", but still doesn't work...any idea?
Thanks
—
Reply to this email directly or view it on GitHub
https://github.com/1000hz/bootstrap-validator/issues/237.

Yup thats what i'm doing, this is how it looks:

Yeah i did that, this is the form element:
<input type="email" name="email" class="form-control" placeholder="Email" required data-checkemail="email"  data-error="Invalid email" data-checkemail-error="Email already registered, login" data-valid="Looks great" data-html="true" />

ah nvm, gotcha, thanks alot :)

Wait a minute> i have the same problem. And by reading this thread I get no idea how you solved it.
My code is:-
<input type="password" class="form-control" id="create_admin_password" name="create_admin_password" style="padding: 6px 12px;" autocomplete="off"  											required pattern="/^[a-zA-Z0-9!@#\$%\^\&*_=+-]{8,12}$/g"                       data-html="true"                       data-error="
qqqqq
"                      />

What am I doing wrong?????


Unable to run

Can you pls break the code down a little. i don't understand the MySQL portion of the code I might be doing it all wrong. Also the "source' part of the code doesn't work on windows

Hi,
What issue are you running into? And what do you mean with the source code doesn't work on windows? That's pretty vague.

first i am unable to succefully execute the MySQL commands using MySQL workbench

I personally never used the MySQL workbench. Just running it through the terminal. Might have to seek external help for that then.

i use windows OS i guess u use somthing other than that.

I work only on Linux Ubuntu. Not windows. Python works on windows just as good, not sure about MSQL tbh.
It's recommend to run it on a GPS regardless, as mentioned in the readme.

I will close this issue for now. Reopen a new one if you're still running into issues.

#21


add watchdog for service health check

watchdog currently is a single instance(or 3 replica), deploy mode is different with node exporter, so make it as a separate pod
Report metrics:
service / k8s / node level metrics.
Output:
it will generate log file which could view from k8s watchdog container logs or ssh to this server /datastorage/prometheus/watchdog.log to view the history.
It also generate a .prom file which could be used by prometheus in the future

Coverage increased (+14.4%) to 62.513% when pulling 26f171cd68f13c3f59d31a39329f08d8bdda26e7 on yanjga/watchdog-new into cdafdacfd2168fb09cdc2d878d1470286a55e56b on master.

Coverage increased (+14.4%) to 62.513% when pulling 26f171cd68f13c3f59d31a39329f08d8bdda26e7 on yanjga/watchdog-new into cdafdacfd2168fb09cdc2d878d1470286a55e56b on master.

I once check this, its default config has a limitation for 10mb of the file. after 10mb will rotate. This doesn't affect current code logic. Only different is whether remove filelog handler. Watchdog also need logger to write logs to console. Other outputfile is in order to output metrics by prometheus and can't leverage k8s.

Hao, will remove .log file

please ensure the max memory usage

added at yaml file
resources:
limits:
memory: "1Gi"


appium.rb is broken on ruby 2.2.2

The process APIs might of changed. flaky is unable to reliably start and stop the appium process.

https://github.com/jarib/childprocess is probably better for managing processes in a cross platform way compared to posix-spawn

this is fixed now, it was a bug in the flaky code related to threading / posix-spawn and tracking @ready state.


Add Configuration Options

Add configuration options created to enable development of new features

Accidentally selected wrong destination branch. I want to add these to my main branch first, then continue development on player info messages while waiting for this to be merged.


Rename sr-only/sr-only-focusable

To be more representative of the fact that these are not necessarily "screen reader" specific, but actually apply to assistive technologies in general (and also things like Alexa/Siri/etc). So .v-hidden and .v-hidden-focusable.
Goes hand-in-hand with #31133

This is a breaking change, so no backporting. tried to strike a balance of brevity here with .v-hidden rather than .visually-hidden.
And yes, it's possibly silly/inconsequential, but the term sr is loaded with false assumptions, and would be good to transition away from it going forward...so doing it as a breaking change in a new version seems the most opportune time

I think this is a very good idea. Two things, though:

I'm afraid .v-hidden could be confusing, meaning visibility for some people.
maybe the file itself should be renamed? _screenreader.scss does not make much sense, does it?

@ffoodd i think even expanding this to .visually-hidden it will still be confusing/confused with visibility for some devs. Not sure there's a better way/naming around this. as for the second point, see #31133 (i filed that separately because even if we don't want to go through all this class renaming, the documentation should be clearer/more reflective of the reality anyway)

related:

reachUI uses "VisuallyHidden" https://reacttraining.com/reach-ui/visually-hidden/
a11yproject has .visually-hidden (so perhaps there's an argument to go for the longer/more verbose version?) https://a11yproject.com/posts/how-to-hide-content/

so are we happy/ok with .v-hidden (relatively short) or want to go for full-on .visually-hidden (perhaps clearer but more wordy)?

also, thoughts on #31133, and whether to backport or not?

TBH I don't know if v-hidden is better or not. I feel like it more ambiguous, but sr was also an abbreviation.

I'd be in favor of .visually-hidden.

If I saw v-hidden in some Bootstrap markup I would assume it was the utility for visibility: hidden;. I had to go reference the docs to actually check what the utility for visibility is (it's .visible and .invisible) because I rarely use it.
Using .visually-hidden seems like a good solution to that.

so despite the lengthier classname, it seemingly preferred to go with .visually-hidden . .visually-hidden-focusable for clarity (and at least for the former, precedent of https://a11yproject.com/posts/how-to-hide-content/). @mdo?

I'm good with the longer name. Honestly every time I see it I think "very hidden" more than I think "visually hidden" lol.

Joining camp longer name

right, renamed the classes/mixins to the longer but clearer variant. and on reflection, probably little use in backporting #31133 ... just concentrating on v5 for this.
in short, i think this is ready for merging, and after that #31133 would be good to merge as well. unless this needs to wait for #31122 @XhmikosR ?

sorry, a bit of back and forth, but i think the migration.md is now as it should be... sanity check from @XhmikosR ?

Just FYI that's not a rebase, not sure how you resolved the commits but in the future please rebase and squash your patches.

i followed the command line instructions given here by github. sorry i'm not a git expert...


Not replacing non ASCII character

What TMM version are you using?
2.9.10
release, pre-release, nightly, or directly from GitHub/branch?
release
What is the actual behaviour?

It should replace ł to l
Steps to reproduce:
Use in folder name ł and try to change name in dry run. Remember to check option in rename: Replace non ASCII
Additional
win

thanks for reporting..
looks like Java is not aware to normalize this character - I've included a workaround with https://github.com/tinyMediaManager/api-scraper/commit/2512f054eac0c3f05e60af583ba426dda0fce9dc
will be in the next version


Can't login anymore.

Checklist

Have you pulled and found the error with jc21/nginx-proxy-manager:latest docker image?

Yes

Are you sure you're not using someone else's docker image?

Yes

Have you searched for similar issues (both open and closed)?

Yes

Describe the bug
When pressing the login button, I am welcomed with this very satisfying error message:
SyntaxError: JSON.parse: unexpected character at line 1 column 1 of the JSON data
Nginx Proxy Manager Version

To Reproduce
Steps to reproduce the behavior:

Input your credentials
press log in

Expected behavior
It should log me in like it did before
Screenshots

Operating System
Ubuntu 22.04 LTS Pro
Additional context
Docker version: Docker version 24.0.5, build 24.0.5-0ubuntu1~22.04.1
Browser: Firefox 124.0.2 (64-bit) - Up To Date
Last time I logged in was 4 days ago I think.

This issue is somehow fixed and I didn't do nothing??

Have you checked the docker logs? There might be any hint as to why the backend service can't start properly. That's why this error appears most likely.

I noticed that nginx proxy manager requires port 3000, which was in use by another service, I changed the port of the other service and now it works.


Augment {Azure}OpenAI chat completion to support auto-invoke of functions passed in via the kernel, validating the pass-kernel-everywhere design.

Fixed by https://github.com/microsoft/semantic-kernel/pull/3754


add Uttaradit Rajabhat University

Web Site
Page  Facebook
Wikipedia

@TaxBoonyarit I have to close this request without merging, because you requested to add a non-existent domain to this repository: ;; connection timed out; no servers could be reached Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for understanding!

@TaxBoonyarit I have to close this request without merging, because you requested to add a non-existent domain to this repository: ;; connection timed out; no servers could be reached Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for understanding!


Deprecate in favour of docker/engine-api

A bunch of projects use this library, so we probably shouldn't abandon it, but it would perhaps be sensible to suggest people try using https://github.com/docker/engine-api instead.
Is there anything missing from docker/engine-api that would stop users from being able to use it instead of this library?
/cc @samalba @vieux

I would say event management, it's pretty rough in https://github.com/docker/engine-api

@vieux Specifically what about event management is lacking? I want to open an issue against docker/engine-api about this, but not sure exactly what to say. :)

@bfirsh all the complicated code here https://github.com/samalba/dockerclient/blob/master/dockerclient.go#L472 that parses events and return them via channel, etc...
last time I looked the only thing in engine-api was a GET /events but you have to do most of the rest by hand

/cc @vdemeester (we were talking about this a while back, re: events)

@bfirsh @vieux yeah, the events in engine-api is pretty bare-metal. I've created a library docker-events (that we use in libcompose) to make it a little useful and was wondering if we should have some code like that in engine-api.

@vieux @vdemeester Thanks, this is really helpful.
@vdemeester Agreed something like that should probably be in engine-api. I've opened https://github.com/docker/engine-api/issues/246 to keep track.


Convert hardcoded hotkeys into configurable ones

I find myself frustrated when I need to use a hotkey on my work machine (which I RDP into from my home desktop so as to utilize my multiple monitor setup) and it turns out that hotkey is hardcoded to do something in FreeRDP. The example I've found so far is Ctrl+Alt+Enter, which is used by VS Code for "replace all occurrences" but is the FreeRDP fullscreen toggle.
I would like to see a mechanism for configuring hotkeys, even if it is just a simple on-off functionality rather than an actual "change the keybind for this" function.
I am going to look into changing the keybind in VS Code in the meantime, but this does mean relying on a non-default keybind for the function in VS Code which is not ideal as my development machine is considered ephemeral and thus I could be assigned a new machine (with default keybinds) at any time.

added the option for a per user configuration file for sdl-client.


Add lu_factor and lu_solve to cupyx.scipy.linalg

This PR adds lu_factor and lu_solve to cupyx.scipy.linalg that correspond to scipy.linalg.lu_factor and scipy.linalg.lu_solve.
I want those functions to port https://github.com/locuslab/qpth/ from PyTorch to Chainer/CuPy.

Jenkins, test this please.

Jenkins CI test (for commit b609605caedc71cf1c9c6b0388e9344b4046a585, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Jenkins, test this please.

Jenkins CI test (for commit 9174231abf4d2c409c97399996ffeb7201e202f8, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Jenkins failure is irrelevant to this PR. LGTM.


CORS for Remoting does not work

Running the following snippet on ASP.NET under IIS works fine. When a remote client attempts to do Rpc to the server, the sitelet correctly sets the Preflight headers:
let Main =
    let origin = "http://foo.com"
    WebSharper.Web.Remoting.DisableCsrfProtection ()
    WebSharper.Web.Remoting.AddAllowedOrigin origin
    Application.SinglePage HomePage

Running the exact same example on Suave (exe), the OPTIONS request is ignored, and the sitelet instead treats it as a GET request and returns the actual sitelet.  Consequently the client hickups with CORS errors.
let wp = WebSharperAdapter.ToWebPart Main
startWebServer defaultConfig wp

Relevant version info:
FSharp.Core (<IP_ADDRESS>)
FSharp.Data (2.3.2)
Microsoft.Owin (3.0.1)
Owin (1.0)
Suave (1.1.3)
WebSharper (<IP_ADDRESS>)
WebSharper.Owin (<IP_ADDRESS>)
WebSharper.Suave (<IP_ADDRESS>)
WebSharper.UI.Next (<IP_ADDRESS>)

I suspect the problem might actually be in either WebSharper.Owin or Sauve.Owin, but I have not had an opportunity to do any serious debugging yet.