Notify users when they're working offline

The software has no way, as far as I have seen, for users to tell whether they're working offline or online. This could cause users to not notice if a server goes down or if a network connection is severed, potentially for a long time. Also it could lead to data loss if browser data is cleared either by mistake or if a user thinks it's synced with the server but is not, or obviously if a laptop is lost or HDD failure.
I suggest the frontend should display a bar at the top if it's working offline that says something like "OFFLINE: Unable to connect to server to save data. Erasing browser history will result in the loss of unsaved changes." or some other device to convey the same information.

if (navigator.onLine) {

} else {

}

you can also subscribe to this event like that
   document.body.addEventListener("online", function () {          console.log("online")        }, false);
   document.body.addEventListener("offline", function () {          console.log("online")        }, false);
https://caniuse.com/#feat=online-status
96 % global usage seems pretty good what do you think?

I'm looking for a first ticket and I could pick this up based on RevIn9s' https://caniuse.com/#feat=online-status but what does it actually look like? Text was mentioned, but where would that go? Someone mentioned a cloud icon.

I'd like to work on this issue

I'd like to work on this.

sounds good, @roy-stewart!

Hi, I can help with this issue, I've implemented the same feature at work.

Of course @alessioprestileo!


Dependency Abstraction and Multi-Module Repository

What was changed

opentelemetry/ was moved to contrib/opentelemetry/ and made its own Go module
opentracing/ was moved to contrib/opentracing/ and made its own Go module
client.MetricsScope and all uses of metrics throughout the codebase were abstracted to use a client.MetricsHandler instead of having Tally hardcoded everywhere
A contrib/tally/ Go module was created with a Tally implementation of client.MetricsHandler
test/ was made its own Go module

Why?

By moving OpenTelemetry, OpenTracing, and Tally to their own Go modules, now users must opt-in to using those third party libraries. Today we force them on people and we have seen with both Tally and OpenTelemetry the cost of their incompatibilities.
This mimics the abstraction we did with logging
While this change is backwards incompatible (e.g. the client.Options.MetricsScope option is now client.Options.MetricsHandler with a different type), so is the interceptors change in the same way in master that is pending release
Putting the integration tests in their own Go module means that future needs of the integration tests don't infect the rest of the SDK

This sets the stage for use of an embedded temporalite-type server
This sets the stage for import of an external SDK features-like project for centralized e2e/integration testing without circular dependency concerns

People wanting to use, say, https://github.com/prometheus/client_golang can now do so without the Tally abstraction middleman

Checklist

Need to check whether everyone is ok with this contrib/ folder living here or if we want some kind of github.com/temporalio/sdk-go-contrib repo. I personally like it here because:

I don't expect to accept every integration under the sun, just these few that we used to hardcode
The versions are kept compatible
This mimics what Java SDK does with OpenTracing and TypeScript SDK does with OpenTelemetry where they put the integrations directly in the repo

Will need samples-go and other repo updates of course (part of normal release anyways)
May need to change our vanity URL repo (I haven't investigated whether the vanity URL base we use for go.temporal.io/sdk automatically works with nested modules)

I know this PR looks big, but it's really not that big, it just takes a lot of little changes to rip out Tally from where we hardcoded it everywhere.

By request I am making a server draft PR to show what would change.

A draft PR for server side has been created: https://github.com/temporalio/temporal/pull/2243


Ensureloggedin

Updated auth in App, newitemform, server, login, logout, reducer login. Added new routes for login, logout, register - this is a before lunch PR to be continued with more specificity after lunch on UX and correct auth pages views based of client req

 Add login, logout, register api route

 include redirect on index.js

 react route auth vs unauth

Looks good to me! Merge It!


any Path Component only Matches a Single Character

The any component only matches a single character, so v, 1, *, c, and anything else 1 character long will work, but v1 gets a 404.
For example, if I have a route:
router.get("hello", any, "world") {req in //... }

This path matches:
/hello/o/world

But this doesn't match:
/hello/oo/world

Failing test case would be appreciated if possible. @proggeramlug you said you'd take a look? If not, I can.

@tanner0101 I will by tomorrow! Unless your bored right now ;)

@calebkleveter @tanner0101
The specific project in which this case happened had the following routes structure:
router.grouped("v1")....
router.get(any, "health")

so the following routes did work:
/v1/whatever (goes into the group)
/whatever/health (goes to the health-get)

What did not work was:
/v1/health
The problem was that the "v1" is parsed before the "any" and so the "health" is never reached, though the group in v1 may not have a matching route.
The fix is really just for the app, to replace v1 in the group with any. After I did that it worked as expected.
I don't think we need any fixes for the module here but we may want to write something in the docs about it, in which order things are parsed.


mustache tags need to be double-quoted

missed this in my last PR but important otherwise the pipelines won't be able to deploy

Looks like this was already updated by someone else here: https://github.com/ibm-developer/generator-ibm-cloud-enablement/blob/development/generators/deployment/templates/toolchain_master.yml
Closing this pr.


fix addStep method type #793

add type form shepherd package and remove any

Is any thing I must do to merge this pull request ?


[SPARK-35910][Core][Shuffle] Update remoteBlockBytes based on merged  block info to reduce task time

What changes were proposed in this pull request?

Currently, we calculate the remoteBlockBytes based on the original block info list. It's not efficient.
If the original reducer size is big but the actual reducer size is small due to automatically partition coalescing of AQE, the reducer will take more time to calculate remoteBlockBytes.
We can reduce this cost via remote requests which contain merged block info lists.
Why are the changes needed?

improve task performance
Does this PR introduce any user-facing change?

no
How was this patch tested?

existing unit tests and verified manually.

BTW, is there no statistic change in terms of remoteBlockBytes value?

Hi @dongjoon-hyun, I add a new test to improve the coverage.

also cc @cloud-fan @maropu @Ngone51 thanks

I've checked the code and the change looks reasonable to me.

thanks @maropu

I'll leave this to those who are familiar with this part. @cloud-fan @Ngone51

Merged to master. Thank you, @yaooqinn and all!


Merging to Main

@lahari104 Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

NA


Variant of renderMustache which collects errors

(Originally from https://github.com/phadej/microstache/issues/3.)
Currently, renderMustache does not give any indication that substitution errors have occurred (other than a return value of "", but I dislike relying on stringly typed programming practices). This has caused confusion at least once: see https://github.com/bos/criterion/issues/127. It would be convenient to have a variation on renderMustache that also gives error messages, similarly to how checkedSubstitute from the mustache library does this.

The microstache version of this is renderMustacheW.

@mrkkrp what do you think?

That's a good idea, we should probably add it in the next version (probably 1.0.0, pending Megaparsec 6).

@mrkkrp, let's have discussion about the issue here.
Porting the microstache patches should be trivial, as rendering part is essentially the same.

Yep, I intend to port the feature from phadej/microstache#4 as soon as I have the time.


[FEATURE REQ] Batch.TryAdd could respect 100 items limit when used with SendVia

Library name
Azure.Messaging.ServiceBus
Please describe the feature.
Related https://github.com/Azure/azure-sdk-for-net/issues/21451 and https://github.com/Azure/azure-sdk-for-net/pull/19125
Currently, when transactions across entities are used, the TryAdd method does allow adding more items to the batch than the service can handle.
The current service restriction is 100 items per transaction.
I realize this might be very tricky to improve, and potentially not even possible without actually calling the service and the service returning the maximum number of items per transaction. I also haven't done any investigations if it is even possible to discover on the client-side the number of transaction enlistments from potentially multiple batches.
Yet, I want to raise this as a discussion item to explore if there is anything that could be done, similar to whether it is possible to improve the maximum number of items that can ever be sent within a batch depending on the tier.

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Thanks for filing this @danielmarbach. You are correct that we would need some sort of service support to add validation here as we try to avoid hard-coding limits on the client. I think you have linked the correct issue - https://github.com/Azure/azure-sdk-for-net/issues/21451, as the setting for the max messages in a transaction could be included as part of the same set of Link Settings as the setting for overall max messages. I am going to mark this issue as Service Attention and also add a comment to the linked issue.

We have assigned an engineer to look further into this issue. We will update this issue when we have more information.

Thank you for your feedback. We have brought this item in our current planning, and should be able to provide an update in the coming months.


Persistent help / feedback button

We could add a persistent help button in the current style to the site like:

If we did so, where would it link to? One idea is to link to the current Google form for the Migration Questionnaire
A second option is to develop  a second Google form with questions like:

I think adding a standard feedback option like this is a good one, though I think the migration questionnaire is perhaps not a good thing to use. I like your other suggestion of a more generic form. Perhaps the easiest thing would be for me to create another form along these lines as a start and we can go from there.

From @scottcain https://docs.google.com/forms/d/e/1FAIpQLSd8f5hrd1-ECgPUbS5dL9njoU1nvCSN5ukykKk9mF6WAyTh6A/viewform?
@frano-m, we can use this as the link.


Django's update_or_create doesnt work with automatic routing

I will open a PR for this, we noticed it while porting over some functionality from our non-sharded environment to our pre-production system.

If you can open a PR with a broken test I can take a look.

I actually already fixed it and just need to back port it to master when I have a chance. Will open up the PR later today.

I actually already fixed it and just need to back port it to master when I have a chance. Will open up the PR later today.

Ditto for this.


Cs/semantic release

Description
Semantic release (semantic-release) is used for automatically tagging and creating GitHub releases with change logs from commit messages. It uses the SemVer convention and the Conventional Commits specification by describing the features, fixes, and breaking changes made in commit messages.
Context

Type of changes

[ ] Refactoring (non-breaking change)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Bug fix (non-breaking change which fixes an issue)

Checklist

[ ] I am familiar with the contributing guidelines
[x] I have followed the code style of the project
[ ] I have added tests to cover my changes
[x] I have updated the documentation accordingly
[ ] This PR is a result of pair or mob programming

Sensitive Information Declaration
To ensure the utmost confidentiality and protect your and others privacy, we kindly ask you to NOT including PII (Personal Identifiable Information) / PID (Personal Identifiable Data) or any other sensitive data in this PR (Pull Request) and the codebase changes. We will remove any PR that do contain any sensitive information. We really appreciate your cooperation in this matter.

[ ] I confirm that neither PII/PID nor sensitive data are included in this PR and the codebase changes.

We could protect the main branch with "require PR's" and use Git Apps to add a semantic PR plugin to make sure they contain a SemVer convention. However I don't believe that branch protection rules are copied over when a template is used, so we might need to think about adding something like commitlint to the pipeline (either pre-commit if we want to enforce the convention on all commits, or just at the PR merge stage)
Finally, I think we might need to think about our preferred branching strategy (or even if we want to go that far) and decide if we want to configure semantic-release to work with release, alpha, next branches, etc

This is a very good-looking patch.  I want us to understand a couple of things, though:
Is SemVer appropriate to enforce across the board?
My default stance here is that it's ok as a starting point if you haven't already got a useful version policy in place, but there can be better options depending on the thing that's being versioned.  SemVer is particularly good where it's used on individual libraries consumed by other people who have a choice about the version they consume, and where releases to more than one lineage need to be made.  If any of those things aren't true - you're versioning more than one thing, or an app, or it's only for self-consumption, or the consumers can't choose whether or not to upgrade, or you're only ever releasing the tip, its usefulness drops off dramatically and you're better served by something that carries information other than inter-version compatibility.
What does this do for repositories that don't publish release artefacts?
semantic-release seems to want you to be publishing release artefacts via a mechanism it recognises and can control.  I'm not familiar with it to know how deep this rabbit-hole is (if at all), but this seems like a strong assumption.  We will want to support not publishing anything as a github or npm release in some cases, and we will want to avoid publishing every version  in others.  What problems is this likely to cause?  Can we head them off?  Am I worrying about nothing?

@regularfry following our discussion I've removed the previous versioning mechanism from the workflows, and now the version is pulled from the tag created by the SemVer step


When creating a new request the focus is now placed on the address bar not the request name

Expected Behavior
When creating a new request the cursor should be placed on the name of the request so you can name it.
Actual Behavior
When creating a new request the cursor is placed into the address bar.
Reproduction Steps

Go to the add button
Click "HTTP Request"
Cursor is placed directly into the address bar

Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
It's now three more clicks to name a request and go back to editing vs the previous way of naming first.
This is more of a feature request but it would be even better if after renaming a request you have just created pressing Enter saves the name and focuses the address bar. This would mean pasting in a curl would be CMD/Ctrl+N, Name the request, Enter, CMD/Ctrl+v
Insomnia Version
v2022.6.0
What operating system are you using?
macOS
Operating System Version
macOS Monterey 12.6
Installation method
Homebrew
Last Known Working Insomnia version
Before 22nd June 2022

Hi @OllieCrook thanks for reporting this.
For now this specific behavior is by design.
There was a similar issue that might have affected you that we've since fixed, https://github.com/Kong/insomnia/issues/5274, and it should be released in our upcoming beta release iteration (beta.4)

Yes that other issue affected me too but this is definitely one that is affecting me on every new request I make.
If fixing that issue caused this one isn't that an unintended side effect? I don't see any acknowledgement of this in the fix PR.


Z-Fighting on 2012-2015 macbooks

Issue details
Z-fighting occurs between polygons at z-depths that should be more than sufficiently widely separated to avoid conflicts- problem only appears (to my knowledge) on macbooks from around 2012-2015.  Spotted this while developing my citybuilder.

Reproduction steps/code
I've included a small sample project that should reproduce the problem on the relevant platform and hardware- just run TestZFight.java .
zfight.zip

Version of LibGDX and/or relevant dependencies
Tested with LibGDX 1.10, but was also present on 1.9.3.
Stacktrace
No stack trace per se- purely a visual error.
Please select the affected platforms

[ ] MacOS

I think I have a fix now- my fragment shader was calling gl_FragDepth = gl_FragCoord.z, which apparently was wreaking havoc on depth-testing on that specific range of macbooks!  When I took it out the z-fighting disappeared.
Apparently I should be avoiding gl_FragDepth calls in general:
https://jvm-gaming.org/t/libgdx-depth-testing-is-horrendously-slow/57707/2
Anyway, that's sorted.  Thanks for all the tips/advice.


Use JavaCPP Preset for ONNX

Addresses https://github.com/feiwang3311/Lantern/issues/42

Thanks for your contribution, cheers! We'll certainly take a look soon.

Merged! Thanks.


Kernel crashes when importing tensorflow version 1.14

My understanding is that the deep-symbolic-optimization repo requires that you have tensorflow version 1.14, and the code in the deep-symbolic-optimization repo does not work if you use any version of tensorflow 2.
I know this because when I used tensorflow version 2.something (I forgot the exact version but I think it was the latest version of the tensorflow) on Google Colab and I executed "pip install -e ./dso", I got some errors like tensorflow's "set_random_seed" method does not exist. Which probably happened because tensorflow 1.14 has "set_random_seed" method but tensorflow 2.something (the tensorflow version I was using) does not have a "set_random_seed" method but tensorflow 1.14 does have a "set_random_seed" method.
By the way, which place in the directory are we supposed to execute "pip install -e ./dso" from? When the current working directory is inside the outer-most "deep-symbolic-optimization" folder, right (This is where I have executing "pip install -e ./dso" from).
Not when the current working directory is inside the outer-most "dso" folder (the outer-most "dso" folder itself contains a "dso" folder, so "./dso" would be a valid path, which is why I am confused).
I tried Google Colab initially, but I got that issue about "set_random_seed" method of tensorflow not found, and Google Colab only allows tensorflow 2.something now, so I gave up on Google Colab.
Then I tried out Jupyter Notebook, which led to the issue of my kernel crashing when I try to import tensorflow.
Steps that I did that led to my issue:
First, on the terminal on my Macbook, I made a virtual environment (using Anaconda) and installed tensorflow (version 1.14) onto that virtual environment.
Then, I opened up that virtual environment on the terminal and then from the terminal, I ran "jupyter notebook".
Then, on the localhost website (Jupyter Notebook's interface is a localhost website) which shows the local files, I created a ipynb file.
Then, in this ipynb file, I did "!pip install tensorflow==1.14", which worked.
Then, I did "import tensorflow as tf" which caused the kernel to crash. I think because I am trying to import tensorflow version 1.14, and perhaps that version is somehow not compatible with some other software (like jupyter notebook).

Yes, DSO currently only works up to TF1.14, but we will add support for Pytorch soon. We know this is a current limitation as many systems don't work on TF1 anymore.
pip install -e ./dso should be run from above the outermost dso directory. Just inside deep-symbolic-optimization.
Lastly, I'd suggest installing in a terminal, not in a jupyter notebook. I have also only tested with pip, not Anaconda, following the README. After installing that way, notebooks on top should work fine.

Hey, thanks for the suggestions and advice.
When I was on the terminal on an Anaconda virtual environment and I ran "python -m dso.run ../BasicExampleConfigFile.json", I got this error:
"/opt/anaconda3/envs/dso_env/bin/python: No module named dso.run"

Meanwhile for when I did the exact commands as the Github README, I got the same error as when I used Anaconda to create a virtual environment (see below):
When I was on the terminal on a Python virtual environment ("python3 -m venv venv3 " followed by "source venv3/bin/activate ") and I was inside the "deep-symbolic-optimization" folder, and I ran "pip install -e ./dso", I got this error message. Any idea how to fix this error message or if this error message is the reason why I do not have a module named "dso.run"?
"distutils.errors.DistutilsError: Command '['/Users/[USERNAME]/Documents/[Folder Name]/deep-symbolic-optimization/venv3/bin/python', '-m', 'pip', '--disable-pip-version-check', 'wheel', '--no-deps', '-w', '/var/folders/hk/7vbk1d0x74dcjw_wpf8mp38h0000gn/T/tmp6dg6__h1', '--quiet', 'numpy']' returned non-zero exit status 1.
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error
× Getting requirements to build editable did not run successfully.
│ exit code: 1
╰─> See above for output.
note: This error originates from a subprocess, and is likely not a problem with pip.
"

Were you able to get this to work? I haven't used DSO with Anaconda, just regular Pip.


Remove web-app related files from hardware repo

Description
This PR removes all of the webapp-related files from the hardware repo.
This closes #1 .
Types of Changes
Put an x in the boxes that apply

[ ] Feature (non-breaking change which adds functionality)
[ ] Bug Fix (non-breaking change that fixes an issue)
[x] Breaking Change (feature/fix that causes existing features to not work as expected)
[ ] Documentation

Checklist

[x] I have read the [contribute]contributing doc
[x] Classes, scripts, and environment variables follow existing naming convention
[x] Lint and Unit tests pass locally
[x] New features on hardware have been tested on a local Raspberry Pi
[x] Mention new programs/binaries if any must be installed along with this change
[x] Mention new environment variables if any have been added to hardware/env file
[x] Please make sure test coverage does not drop. If it does, please explain the reasons.
[x] Any new required python modules are added to the requirements.txt

I noticed that it's not running any status checks, maintainers should probably enable branch protection on master. As it currently stands anyone with write access can merge whatever they want into master.

@ab7289 this is a good point. we should enforce the requirement. @MichaelLally has the necessary admin access to set this rule. Hopefully we will hear back from him when he sees this message.

@ab7289 thanks for removing the web app files! As part of this removal of web app files, could you also remove the docs that are no longer relevant? They are still in the repo: https://github.com/ab7289/mercury-hardware/tree/master/docs

Hey @jackxujh, thanks for the suggestion. I have updated the PR to remove the docs.


Reset gc for old snapshots 4.0

Cherry pick https://github.com/microsoft/FluidFramework/pull/14906 fix to 4.0

Prefix [Port v2int 4.0] to the description


How to make sure different instances of Oidc.UserManager work with the same item in local storage?

Most of my client app is a SPA, but I need access to the logged in user in some other pages. I can't use redirects or silent redirects to ID server in those pages since I want to avoid any latency in the page loading. I have decided to use local storage however I'm not sure, I know exactly how the storage item key is constructed. To play it safe, I create the UserManager instances on the different pages with the exact same configuration each time, except than on some pages I don't use silent refresh. I've noticed that the token is stored in local storage and the key includes authorization authority, client ID and some other settings but I'd really like to be sure that all instances work with the same storage item, so which are the configurations which affect this?
Also, in addition to the token record, my local storage gets 'flooded' with records like this:

By 'flooded' I mean hundreds for a couple of days (I'm testing a lot:)). I get the feeling that something is not removed when it should be. What might it be that I'm doing wrong, so those records get piled up?
Here is the configuration I use on pages without silent redirect
var oidc = new Oidc.UserManager({
            authority: config.idAuthority,
            client_id: config.clientId,
            redirect_uri: config.redirectUri,
            response_type: 'id_token token',
            scope: 'openid profile api1',
            post_logout_redirect_uri: config.postLogoutRedirectUri,
            userStore: new Oidc.WebStorageStateStore({
                prefix: 'api1',
                store: window.localStorage
            })
        });

and with silent redirect
var oidc = new Oidc.UserManager({
            authority: config.idAuthority,
            client_id: config.clientId,
            redirect_uri: config.redirectUri,
            response_type: 'id_token token',
            scope: 'openid profile api1',
            post_logout_redirect_uri: config.postLogoutRedirectUri,
            automaticSilentRenew: true,
           silent_redirect_uri: config.silentRedirectUri,
           accessTokenExpiringNotificationTime: 60,
           userStore: new Oidc.WebStorageStateStore({
                prefix: 'api1',
                store: window.localStorage
            })
        });

The values in the config variable are the same for all pages.
Thanks.

1: To insure each instance uses the same key you need to make sure each instance uses the same authority and client_id: https://github.com/IdentityModel/oidc-client-js/blob/dev/src/UserManager.js#L420
2: To cleanup orphaned state, use clearStaleState on the UserManager.

Thanks a lot!

2: To cleanup orphaned state, use clearStaleState on the UserManager.

I guess when logging out is a nice place to do this.

I guess when logging out is a nice place to do this.

Yes, or even just in each page load.


Support for archive formats PDF/A-1 and A-3

These formats are important to business use particularly the A3 which is a signed PDF with embedded XML data.  Will become mandatory in EU, starting with Germany.  This could take this library from being a fine achievement to being a necessity for Goloang developers in enterprises and by inference a great way to prompt developers of other languages to move to Go (guerilla marketing some would call it).

Thanks for the comments and confidence in the project, @26labs. I was not even aware of the PDF/A archival standard. The big task will be to support digital signing (see issue 115); beyond that, it looks like it is a matter of assuring that certain features like transparency, linked fonts, etc are not used.

It is not THAT difficult for simple documents: always embed fonts (don't use the standard 14), have an XMP metadata stream, and an outputintent. Here's a very simple PDF/a-1b file.
PDFA.pdf


Modularity by wildcard

Many Laravels developers implement modularity in their apps using packages like https://github.com/nWidart/laravel-modules or manually by special folders structure (like me).
Assuming structure like this:
app/Modules/ModuleName/*
I would like to have a dedicated folder for Filament:
app/Modules/ModuleName/Filament/*
It is possible to achieve this in Laravel Nova using a provider Nova::resourcesIn(app_path('Modules/*/Nova')).
Filament allows only manual registering resource classes using the register config.
This PR provides auto wildcard registering:
'resources' => [
    'namespace' => 'App\\Modules\\*\\Filament\\Resources',
    'path' => app_path('Modules/*/Filament/Resources'),
    'register' => [ ],
],

Implementing a wildcard is tricky only for namespace because allFiles method under a hood uses the native glob so no tweaking is needed.

Thanks, just refactored a bit, please double check it still works fine

@mgralikowski could you check this again for me please?
@ziming @torosegon care to share why you 👎 this solution? Is there something I missed?

@danharrin sorry for the late answer. I tested Your changes on my existing app and my module is still auto-discovered correctly in the admin panel.

No worries, thank you very much!


cause panic when concurrent use Msg

PoC (run about 1min on macOS 10.15.2):
package main

import (
	"fmt"
	"strings"
	"time"

	"github.com/miekg/dns"
)

func main() {
	msg := &dns.Msg{}
	msg.Question = []dns.Question{
		dns.Question{"example.com.", dns.TypeA, dns.ClassINET},
		dns.Question{"example2.com.", dns.TypeA, dns.ClassINET},
	}

	msg2 := &dns.Msg{}
	msg2.Question = make([]dns.Question, len(msg.Question))

	trimFunc := func() {
		for {
			time.Sleep(time.Millisecond)
			s := strings.TrimSuffix(msg2.Question[0].Name, ".")
			s = fmt.Sprintf("s %s", s)
		}
	}

	for i := 0; i < 20; i++ {
		go trimFunc()
	}

	for {
		time.Sleep(time.Millisecond * 10)
		msg.CopyTo(msg2)
	}
}

Possible root case:
https://github.com/miekg/dns/blob/f515aa579d28efa1af67d9a62cc57f2dfe59da76/msg.go#L1056-L1057
Inadequate Patch:
questions := make([]Question, len(dns.Question))
copy(questions, dns.Question)
r1.Question = questions

The race condition isn't in the CopyTo method but in your example code. CopyTo changes the msg2 you pass into it which is a race condition. You can't read from msg2 (in trimFunc) and write to it (in CopyTo) at the same time without locks.
I'm not sure what your original problem actually is, but either stop calling CopyTo while trimFunc is running or move msg2 and the copy into trimFunc entirely. (In your minimal example above, there's no need to create a copy at all).
I'm going to close this as it's not a bug in the library.


fix: log dialog bugs

:tada: This PR is included in version 0.323.6 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


At least this bot now works as expected.
 Not practical for people who can't program. The JSON structure makes it easy to add new GIFs without having to program.
Your changes add the alias option but are not practical for the use case for which it was programmed.

Proven that you are unable to understand basic Python.
You've added different types of the same library (one of them not even supported anymore)
You wrote code with an absolutely outdated not even supported library.
The whole pull request does not change any functionality of the Bot, it only brings it to correct library and Python standards.
JSON Structure is untouched and still implemented.
Also you hadn't understand the way how the async branch of discord.py works.. Your Bot breaks after a few seconds of API response problems and your hole "on_message" event is totally blocking!


Action should immediately fail when it fails to PUT a tag

Issue mentioned here --> https://github.com/devcontainers/spec/issues/61#issuecomment-1292804969
As the corresponding publish action showed a ✅  , the user would not realize that it failed to publish some tag. Need to fail the action as soon as CLI fails to PUT a tag.
// cc @eitsupi

I ran into this issue when I also found a possible ERR_FR_MAX_BODY_LENGTH_EXCEEDED  bug in the CLI here https://github.com/devcontainers/cli/issues/270

@joshspicer looks like https://github.com/devcontainers/action/pull/105 should fix this issue?

oh wait maybe not! The CLI still needs to be changed to return a non-zero exit code

Closing as fixed with https://github.com/devcontainers/cli/pull/296


feat: add "Unsubscribe from repository" button

Context
Add a "Unsubscribe from repository" button.

It uses the  "Set a repository subscription" REST API,
Discussion
Two things:

At first I was going to start implementing the "Mark repository as done" button (as it had already been discussed in #706, PR coming soon 😉), but having miss-aligned icons made me look into adding this feature first...
But I feel like it's really "heavy" on the UI, it's not so beautiful and it makes it harder to distinguish if the row is a Notification or a Repository...
What do you think?

Following the "Unsubscribe from notification" implementation, I made it so that clicking on  Repository unsubscribe marks all of the repository's notifications as read. But this behaviour looks a bit odd to me; why automatically discard all those notifications? And why mark them as read instead of done? Perhaps it is better not to discard those notifications at all? If so, we would have to find a way to tell the user that the unsubscribe action has been successful...

TBH even though this feature could still be a nice to have; I'm not sure it's required.
Maybe the solution to the misalignment of the icons between Repository and NotificationRow could just be a space between the Mark repository as done and Mark Repository as read buttons:

Please tell me what you think so I can adjust my PRs :P

I don't currently have mental space to wrap my head around everything in the description, but two thoughts that come to mind on a first pass:

There's an added twist for this one: it could be taken to mean "unsubscribe from this repository," which isn't what this is doing.
To solve the visual complexity of all these icons, we could show the icons only on hover. That would be discoverable (you naturally mouse over the items to open them).

There's an added twist for this one: it could be taken to mean "unsubscribe from this repository," which isn't what this is doing.

Yup, that's why I'm really not sure that it's a good idea to implement this 😬

To solve the visual complexity of all these icons, we could show the icons only on hover. That would be discoverable (you naturally mouse over the items to open them).

I thought about this too! If others agree with this we can try it?

I like the idea of exploring showing the 3-notification buttons on mouse-over the row, which is similar to how GitHub behaves.
Re: unsubscribe from repository, i'm not sure this is needed.  GitHub when viewing notifications grouped-by repo only has the mark as read button

I'm dropping this PR, I've been running a main build for a few days and the repository "done" button looks enough


Add GDLauncher image to showcase

It would be perfect if you could add an image below our project, you can use the one we have on our repo readme. Thank you :)

Would a PR be welcomed for this?


ValueErrors on some workitems

On a doing list the following error was returned due to missing milliseconds in the time data string:
   ValueError: time data '2021-07-13T08:56:40Z' does not match format '%Y-%m-%dT%H:%M:%S.%fZ'
This is caused by line 139 in _list.py:
    item_datetime = datetime.datetime.strptime(item_datetime, "%Y-%m-%dT%H:%M:%S.%fZ
When testing with the .%f removed from that line, a similar error occurred on another work item:
    ValueError: time data '2021-06-08T08:15:17.547Z' does not match format '%Y-%m-%dT%H:%M:%SZ'
So apparently some Work Items are returned with and some without milliseconds....
The following code fix solved the issue:
        try:
            item_datetime = datetime.datetime.strptime(item_datetime, "%Y-%m-%dT%H:%M:%S.%fZ")
        except ValueError:
            item_datetime = datetime.datetime.strptime(item_datetime, "%Y-%m-%dT%H:%M:%SZ")

Nice catch! Thanks for reporting. Would you like to provide a PR?

Of course, I have a branch ready to pushed once authorized.

No need to authorize, you can create a fork , commit and push there, and create then a pull request (guide).


Daily Motivation

Is your feature request related to a problem? Please describe.
Users often seek motivation and inspiration throughout their daily activities, including browsing the web.
Describe the solution you'd like
Develop a Chrome extension called "Daily Motivation" that delivers a wide selection of inspirational quotes directly within the browser. This extension aims to uplift users' spirits, spark creativity, and foster motivation during their online interactions.
Describe alternatives you've considered
One alternative could involve integrating the motivational quotes feature into existing productivity or mental wellness apps that users already use regularly. Additionally, users could have the option to customize their experience by selecting specific themes or categories for the quotes displayed.
Additional context
The goal of this feature is to enhance the user experience by providing a convenient source of daily motivation and positivity directly within the web browser. Completion of this feature can be evaluated based on user engagement metrics, such as the frequency of interactions with the extension and feedback on the effectiveness of the quotes in inspiring and uplifting users. Additionally, monitoring any improvements in users' mood or productivity while using the extension can help assess its impact on their overall well-being.

@Sulagna-Dutta-Roy  PLEASE ASSIGN ME THIS ISSUE

Already added this to my repository #633. closing this


An Interactive And UI Friendly Pop-Up when clicked on Connect with a reader section

Is your feature request related to a problem? Please describe.
⇾ Problem : I'm, As a viewer, it is completely not visually appealing that such a nice UI website gives a very unaesthetic pop-up which degrades the website.

Describe the solution you'd like
I would make a completely Aesthetic Pop-up for this section and various other sections that enhances these websites for users.
Describe alternatives you've considered
As an Alternative, we can remove the Connect with the reader button when the user clicks on it when submitting and then display "Thank you for your interest".

Please Assign to me under GSSoC'24

Issue already created!


[TOPSORT] added topsort events destination docs

[!NOTE]
READY FOR REVIEW

Proposed changes
This is the docs entry for the new Topsort Destination. Any comment, syntax or anything I missed please let me know.
Merge timing
Not urgent at the moment. The sooner the better though.

Ready! Thanks so much for the reviews @tcgilbert @forstisabella

@agustinespildora thanks for the changes. We have everything on our side to make the integration live.
If you have tested your destination and confirmed it is working as expected, we can get this live on the next docs deployment next week.

Hi team! Do you know when is the next deploy?

hi @agustinespildora - I'll merge this PR today, and the changes will be live tomorrow after 11AM Pacific time. Thanks!


gdrive slides in 3bot

Expose these openresty endpoints:

[x] http://$IPADDR/gdrive/slide/$gpresentation_guid/$gslide_slide_guid.png
[ ] http://$IPADDR/gdrive/slide/$gpresentation_guid/$slidename.png
where the slidename is in the notes section of the slide deck as name = <slidename>

The gedis actor should:

get the presentation slide if older than 5 minutes.


Add dozens of new integration tests to the GDScript test suite

This also makes all scripts use tab indentation (except the ones specifically designed to test space indentation or mixed tab/space indentation).
Reviewing tip: Use the file filter to hide .out files in the Files changed tab:

dictionary_consecutive_commas (test that currently crashes)
func test():
	# Dictionaries with consecutive commas are not allowed.
	var dictionary = { "hello": "world",, }

	dictionary = { hello = "world",, }

GDTEST_PARSER_ERROR
Expected expression as dictionary key.

Seems like static checks occasionally cause CI errors with *.out files, I think those files should be ignored from static checks eventually. I remedied this problem by stripping and appending the newline in the original PR, but seems like it's not enough to eliminate those errors completely.

Seems like static checks occasionally cause CI errors with *.out files, I think those files should be ignored from static checks eventually. I remedied this problem by stripping and appending the newline for each generated .out file in the original PR, but seems like it's not enough to eliminate those errors completely.

This should be fixed now.

Built and tested.
tests/scripts/parser/errors/dictionary-consecutive-commas.gd currently crashes the GDScript parser.

Built and tested.
tests/scripts/parser/errors/dictionary-consecutive-commas.gd currently crashes the GDScript parser.

Fixed :slightly_smiling_face:

@Calinou Whats the state of this? :)

@Calinou Whats the state of this? :)

It should be mergeable now. We can add more tests later as needed.

CI disagrees :P

I've added many tests related to heavy nesting in parser/features.

Thanks!


[New game]: Number Guesser

🎮 Game Request
Game logic and basic description
user gets to guess number and points will be decided on how many tries he takes and ranking will be displayed based on other users
Point down the features
Game points
As he guesses on the very first attempt he gets 100 and it will reduce as per tries
Select program in which you are contributing
Gssoc'24
No response
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @cxde-Sujal !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hey @cxde-Sujal!
We are already having a similar game request in #3217 👀
Make sure you come up with a cool unique idea 😀
Waiting for your new game idea 💗.

Hey @cxde-Sujal ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


About the input resolution

Hello, thanks for releasing the source code. Can I ask a question?
I found from the script that the image you used to train the network is with the size of 368368 (--train_resolution 368 368), I really want to ask why since the references are with size 320320. Did you also try to train with 320*320? If so, how about the result?
Thanks a lot.

Great question!
First, let me start with this: During test time we will have to perform reconstructions on the whole volume without cropping in order to have a correct forward model. Only after reconstruction will we be able to do cropping. Now ideally, we want our measurement simulations during training to resemble those during test time as closely as possible. However, due to the fact that all volumes have vastly different sizes and some of them are very large, working on full volumes during training would be computationally restrictive.
Therefore, we train on smaller patches. We know that by doing this, training and test data will not come from the same distribution. Empirically, however, we've had good results already with the baseline Recurrent Inference Machine (RIM) by training on smaller patches and transferring to larger patches. But, we have evidence that training on larger patches benefits performance nevertheless (see Lønning et al., 2019).
So the chosen patch size is a trade-off between computational overhead and performance. In the model we trained, the largest receptive field size of a residual block is 24. Now if you add that to the all edges we have 320 + 48 = 368. Working with this slightly larger context than the target is aimed to reduce edge effects.
We have not tried training on 320x320.
Hope this helps!
Lønning, K., Putzky, P., Sonke, J. J., Reneman, L., Caan, M. W., & Welling, M. (2019). Recurrent inference machines for reconstructing heterogeneous MRI data. Medical image analysis, 53, 64-78.


Is it possible to use 'on_publish' hook in Lua script?

Environment

Docker: vernemq/vernemq:latest

Expected behavior
My goal is to make VerneMQ send published data to Redis. And looks like using hooks and Lua scripting is a right way to go.
Actual behaviour
I've registered following Lua script:
function on_publish(pub)
    redis.cmd(pool, "RPUSH clients:events test_event")
    return true
end

pool = "auth_redis"
config = {
    pool_id = pool
}

redis.ensure_pool(config)
redis.cmd(pool, "RPUSH clients:events lua_script_loaded")
hooks = {
    on_publish = on_publish,
    on_publish_m5 = on_publish,
}

I can see that script was loaded and connection to redis made. It successfully executes redis.cmd(pool, "RPUSH clients:events lua_script_loaded"), but it looks like on_publish function is never called when i publish.
I didn't find any information in official documentation saying that some hooks are not available for Lua scripts, but for Erlang plugins only.
Could you tell me please if my scenario is possible or i have to look for other options?
Thanks.

Test example here: https://github.com/vernemq/vernemq/blob/4f0bbab8de80bb3b55497471c88d59d4e7de5eb0/apps/vmq_diversity/test/plugin_test.lua#L88
on_publish is called for side effect. don't return true.
Maybe this helps too: https://github.com/vernemq/vernemq/issues/1497

@ioolkos that helped, thanks a lot.

How to execute the script in docker vernemq ?

@kalyanijivrakh general Docker principles apply. You want to load a custom script, so you have to do that from a dir mounted into Docker.
You can either point the full script directory to the mount (using the vmq_diversity.script_dir setting) and copy all the scripts including your plugin there; or you can have an additional mount for your diversity plugin, and then point to the script file directly (vmq_diversity.my_plugin.file)

:point_right: Thank you for supporting VerneMQ: https://github.com/sponsors/vernemq
:point_right: Using the binary VerneMQ packages commercially (.deb/.rpm/Docker) requires a paid subscription.


FEATURE migrate classes from CoreTools

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@8f13500). Click here to learn what that means.
The diff coverage is 62.5%.

@@           Coverage Diff            @@
##             master      #1   +/-   ##
========================================
  Coverage          ?   62.5%
  Complexity        ?      43
========================================
  Files             ?       4
  Lines             ?     120
  Branches          ?       0
========================================
  Hits              ?      75
  Misses            ?      45
  Partials          ?       0

Flag
Coverage Δ
Complexity Δ

#php
62.5% <62.5%> (?)
43 <43> (?)

Impacted Files
Coverage Δ
Complexity Δ

src/Extension/DataobjectPermissionExtension.php
100% <100%> (ø)
4 <4> (?)

src/Model/HeaderImage.php
13.15% <13.15%> (ø)
13 <13> (?)

src/Extension/PreviewExtension.php
82.69% <82.69%> (ø)
18 <18> (?)

src/Extension/HeaderImageExtension.php
86.36% <86.36%> (ø)
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 8f13500...f16678b. Read the comment docs.


question: Why I cannot use dark mode?

I cannot use dark mode. For example, dark:bg-gray-800 doesn't work. Actually, dark: doesn't work anywhere.
My project is sites-list. So what is happening inside? I've used the inspector and found dark classes are generated. But they just don't work.

The final css generated by dark: is .dark .dark\:color-white {...}, you can try adding class dark to the html tag.
Can refer to:

https://github.com/antfu/vitesse/blob/main/index.html#L10-L15
https://github.com/antfu/vitesse/blob/main/src/composables/dark.ts

Or you can check this option.
https://github.com/unocss/unocss/blob/8cf56ab3dfbcea44cb154e20159de8b1da149372/packages/preset-mini/src/index.ts#L31-L36

OK I understand that. Many thanks for @Hongbusi and @zyyv !


Please post OpenVX feedback here

The OpenVX working group would like to ask the community for feedback and contributions to make OpenVX more accessible to developers. If you have any requests or suggestions on topics to cover or resources to be developed, please ask them here. If you have a resource to suggest please add an issue or pull request here. We look forward to hearing from you!
The OpenVX Working Group

Unable to build the OpenCV after enabling the WITH_OPENVX.

OpenVX version 1.3 is used


[Fine tuning] Various minor fixes

test passed internally, merging

https://ci.app-svc-perf.corp.redhat.com/job/ExternalTeams/job/RHODS/job/topsail/906/artifact/run/f23-h33-000-6018r.rdu2.scalelab.redhat.com/000_test_ci/


Added support for Hue Flourish Pendant Light

Thanks @kingofkobjer for your contribution, will add this to next release.


Critical Miss (Fumble)

Describe the new feature
The option to automatically roll on a custom RollTable for any critical miss (Natural 1) made by PC or NPCs for the attack roll.
Expected behaviour
When a certain actor gets a critical miss (Natural 1) on a d20 when rolling for the attack (should work with midi-qol merged card), it should trigger a roll on a Fumble RollTable by itself or maybe ask a player to roll via dialogue window.
Additional Information
I mostly want to use the Fumble RollTable from this book (pg. 49):
https://www.dmsguild.com/product/228484/Xanathars-Lost-Notes-to-Everything-Else
It would be really convenient this module could automate the rolling process, or at least bring up the dialogue window to click “Roll” button.

Hi, thanks for the suggestion. I could definitely make this work, however to keep it within the bounds of the module it would be an optional feature that would still rely on the PC having the Wild Magic Surge feat.
Would that still work for you or are you after any player rolling a Nat 1?

If this is the only way to add this kind of functionality to the module, it's obviously better than constantly making throws manually, so I'm all for it.


Add Radio Silence.app

Similar to PR #1802, an extra stanza is required so the uninstall does not fail. Both of these apps are from the same developer and using the same kext, hence the similar behavior.
Also, similar to PR #14369, the uninstall task will fail if the app is running. I'm not sure of a good way to fix that

Merged as faf1a76edab0205d3c707fed2380ff53303a62f8.


Fix Issue #2 (PGN encoding)

Added "errors='replace'" parameter for "open" functions, so that non-ascii PGN files don't make the parser crash.

Thanks for this 👍🏻 I'll merge it on 1st October so it'll count towards Hacktoberfest :)

I guess the hacktoberfest-accepted label is needed as well.

Thanks for this 👍🏻 I'll merge it on 1st October so it'll count towards Hacktoberfest :)

According to the Hacktoberfest rules, the Pull Request must also have been created in October, so I guess I was too fast and this one won't count anyway. ^^

Cool, thanks for this :)


Update story API format

Per Allen's request, change API based on below:

Get Stories /v1/story/[franchiseId]
Get Chapter /v1/story/[franchiseId]/[storyId]
Get Chapter Content /v1/story/[franchiseId]/[storyId]/[chapterId]

and all of these can add the following params to it:
limit=100
offset=200

Also change num -> id

Change Get Chapter Content response to:
{
  title: "XXXX",
  chapterId: "1",
  coverImage:
  content: [

  ]
}

change franchise id to franchise slug for urls. Use - to replace space

Done


[CGPO] Mixture of judges

What does this PR do?
This PR adds the Mixture of judges part of the CGPO paper (https://arxiv.org/pdf/2409.20370). The judges are described in section 4.1.4 and the mixture of judges simply labels a generation as “violated” (0) if it fails any one of the constraint judgments and “satisfied” (1) otherwise.
Related to #2156
Before submitting

[X] Did you read the contributor guideline,
Pull Request section?
[X] Was this discussed/approved via a GitHub issue? Please add a link
to it if that's the case.
[X] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines.
[X] Did you write any new necessary tests?

Who can review?
@kashif @lewtun

Still a draft PR

Thanks a lot @gaetanlop. Added some suggestion and open questions

Hey @qgallouedec, thanks for the review. I have added a gold_answers parameter to the judge function for both the base and MoJs class. Also, I have added a safety_judge and factuality_judge as described in the CGPO paper. They also have rule based judges, but in my opinion they are a little bit too tailored to specific tasks (coding/maths) to be added to the trl library. If you think the factuality and safety judges are also too specific to be in the lib I can remove them from the PR.
For the naming of the judges, let's do BaseBinaryJudge for the base class and AllTrueJudge for the moj following your suggestions?

Hey @qgallouedec, thanks for the review. I have added a gold_answers parameter to the judge function for both the base and MoJs class. Also, I have added a safety_judge and factuality_judge as described in the CGPO paper. They also have rule based judges, but in my opinion they are a little bit too tailored to specific tasks (coding/maths) to be added to the trl library. If you think the factuality and safety judges are also too specific to be in the lib I can remove them from the PR.

Thanks a lot! Nice work!

For the naming of the judges, let's do BaseBinaryJudge for the base class and AllTrueJudge for the moj following your suggestions?

LGTM.

WDYT of having generic classes in trl.judges (AllTrueJudge, BinaryJudge etc.) and subclass them in trl.trainer.cgpo_trainer to get SafetyConstraintJudge and FacultyConstraintJudge? If in the future we need these classes elsewhere, we can still move them in trl.judges

Ok, I have removed the FacultyConstraintJudge  and the SafetyConstraintJudge from the PR and made the required renamings. Thanks @qgallouedec for the feedback.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

LGTM, thanks @gaetanlop

We're having the issue
FAILED tests/test_nash_md_trainer.py::TestNashMDTrainer::test_nash_md_trainer_judge_training_1_conversational_prompt_only - ValueError: Cannot find pytorch_model.bin or model.safetensors in C:\Users\runneradmin\.cache\huggingface\hub\llm-blender\PairRM

that is mainly due to the fact that PairRM is requested for download simultaneously for different tests. This happens quite randomly, and is not related to any actual bug in the code base. I'll ignore it and merge.


request help: Apisix ingress controller config.yaml couldn't read os environment variable

Issue description
I want to set the default_cluster_admin_key from env variable. I have such config.yaml file and set the ADMIN_PASSWORD as env. And I mount the config.yaml as configMap for my apisix-ingress-controller pods.  but the config.yaml couldn't read the ADMIN_PASSWORD from os environment.
apisix-ingress-controller config.yaml
      apisix:
        default_cluster_base_url: "http://apisix-admin.ingress-apisix:9180/apisix/admin"
        default_cluster_admin_key: "${ADMIN_PASSWORD}"
      log_level: "debug"
      log_output: "stderr"
      http_listen: ":8080"
      enable_profiling: true
      kubernetes:
        kubeconfig: ""
        resync_interval: "30s"
        app_namespaces:
        - "*"
        ingress_class: "apisix"
        ingress_version: "networking/v1"
        apisix_route_version: "apisix.apache.org/v2beta1"

env config
          env:
          - name: ADMIN_PASSWORD
            valueFrom:
              secretKeyRef:
                key: password
                name: apisix-admin-secret

But the interesting thing is: my apisix config.yaml file could correctly read ADMIN_PASSWORD from environment.
apisix config.yaml
        apisix:
          node_listen: 9080             # APISIX listening port
          enable_heartbeat: true
          enable_admin: true
          enable_admin_cors: true
          enable_debug: false
          enable_dev_mode: false          # Sets nginx worker_processes to 1 if set to true
          enable_reuseport: true          # Enable nginx SO_REUSEPORT switch if set to true.
          enable_ipv6: true
          config_center: etcd             # etcd: use etcd to store the config value

          allow_admin:                  # http://nginx.org/en/docs/http/ngx_http_access_module.html#allow
            - <IP_ADDRESS>/0
          port_admin: 9180

          admin_key:
            # admin: can everything for configuration data
            - name: "${ADMIN_USERNAME}"
              key: ${ADMIN_PASSWORD}
              role: admin
            # viewer: only can view configuration data
            - name: "${VIEWER_USERNAME}"
              key: ${VIEWER_PASSWORD}
              role: viewer

Environment

your apisix-ingress-controller version (output of apisix-ingress-controller version --long); 1.2.0
your Kubernetes cluster version (output of kubectl version);   v1.20.11
if you run apisix-ingress-controller in Bare-metal environment, also show your OS version (uname -a). Linux apisix-ingress-controller-6d46bd8c5c-fwzjq 5.4.0-45-generic #49~18.04.2-Ubuntu SMP Wed Aug 26 16:29:02 UTC 2020 x86_64 Linux

This is normal, because apisix and apisix-ingress-controller are two independent software. The former supports the use of environment variables when parsing configuration files( code link ), and the latter has not been implemented.
Of course this is a nice feature, we can consider supporting it.

This is normal, because apisix and apisix-ingress-controller are two independent software. The former supports the use of environment variables when parsing configuration files( code link ), and the latter has not been implemented. Of course this is a nice feature, we can consider supporting it.

Thanks for quick response! Looking forward to this feature.
And could I just use the k8s command flags to config the apisix-ingress-controller. Is this recommend practice before this feature?
```
command=[
    "/ingress-apisix/apisix-ingress-controller",
    "ingress",
    "--log-level",
    "debug",
    "--log-output",
    "stderr",
    "--http-listen",
    ":8080",
    // namespaces that controller will watch for resources
    "--app-namespace",
    "*",
    "--apisix-route-version",
    "apisix.apache.org/v2beta1",
    "--default-apisix-cluster-base-url",
    "http://apisix-admin.ingress-apisix:9180/apisix/admin",
    "--default-apisix-cluster-admin-key",
    "$(ADMIN_PASSWORD)",
  ],

Using command is OK, but it will make the deployment yaml a bit more complicated. What we use in the helm chart is configmap for configuration file, I don't know why you didn't use it?

Using command is OK, but it will make the deployment yaml a bit more complicated. What we use in the helm chart is configmap for configuration file, I don't know why you didn't use it?

As for helm chart, I used it when I did experimental. However, as for production use, our company prefer to use deployment yaml directly.
As for configMap, actaully my first choice is configMap. However, just as I said in the begainning, when I mount the configMap as config.yaml, the config.yaml couldn't set the ${ADMIN_PASSWORD} from environment variable. And for production use, considering security, we definitely don't set the default_cluster_admin_key directly in configMap. Therefore I have to use command. It allow me to use ${ADMIN_PASSWORD} from environment variable

And as for why I couldn't use ${environment variable} in the apisix-ingress-controoler config.yaml, I suspend this is because the apisix-ingress-controller doesn't support bash.

And as for why I couldn't use ${environment variable} in the apisix-ingress-controoler config.yaml, I suspend this is because the apisix-ingress-controller doesn't support bash.

The configuration file is parsed directly through the golang code, without bash.

Using command is OK, but it will make the deployment yaml a bit more complicated. What we use in the helm chart is configmap for configuration file, I don't know why you didn't use it?

As for helm chart, I used it when I did poc of apisix. However, as for production use, our company prefer to use deployment yaml directly.
As for configMap, actaully my first choice is configMap. However, just as I said in the begainning, when I mount the configMap as config.yaml, the config.yaml couldn't set the ${ADMIN_PASSWORD} from environment variable. And for production use, considering security, we definitely don't set the default_cluster_admin_key directly in configMap. Therefore I have to use command. It allow me to use ${ADMIN_PASSWORD} from environment variable

OK, then I think it can be specified temporarily by command. Another way is to build a docker image (FROM apisix-ingress-controller), add a startup script, and use the envsubst tool in the script to replace the environment variables in the configuration file.

This is a very good suggestion! Thanks a lot!

#745  has been merged. This feature has been implemented, I will close this issue. Feel free to reopen it, if you have any questions.


WP8.1 - suddenly image-banners are very small...

Out of the blue, the "image" banners are very small on Windows Phone (see attachments).
Banners with text are rendered correctly, but "pure" image banners are barely visible.
It happened some days ago, and I didn't change anything of the code / neither did I update the app - it worked well before (they were stretched tot the width of the screen, with the right dimensions).
Same app on Android is doing fine (with exact the same source-code).

Seems it is viewport / -ms-viewport issue

Hello,
I too having this issue, ads banner suddenly getting very small, with the same size as it is in in screeenshots provided by @dMOTION.
@softexe, I don't understand what do you mean by  -ms-viewport, I mean what to do with it, I applied meta tag like this:

in my index.html, but it does nothing.
Please help me, if you already had solution for this issue.
Thanks

My header for WP build with correct banner size:

</head>

@softexe thanks, I also see some similar solution to fix the wp8 viewport issue in follow articles:
https://coderwall.com/p/zk_2la/responsive-design-in-ie-10-on-windows-phone-8
http://stackoverflow.com/questions/24007577/windows-phone-8-viewport-issue
Someone says this error only happen to IEMobile 10.0, and suggest "This code specifically targets version IE Mobile 10.0 (I've also added an extra condition to ensure it only happens in IE), so future, fixed versions will be untouched."
(function() {
    if ("-ms-user-select" in document.documentElement.style && navigator.userAgent.match(/IEMobile\/10\.0/)) {
        var msViewportStyle = document.createElement("style");
        msViewportStyle.appendChild(
            document.createTextNode("@-ms-viewport{width:auto!important}")
        );
        document.getElementsByTagName("head")[0].appendChild(msViewportStyle);
    }
})();

As this is not plugin issue but IEMobile issue, I will close this tracker.


erron in request: 404

Hi,
since this afternoon I´m getting the follwing error message (erron in request: 404), when I try
to start a song via my VIS control panel. Authorizations are fine (see screenshot).
Do you have any idea why thsi don´t work?
BR Hail0

What do you mean excatly with "start a song via my VIS control panel"? with state id?
Can you please change the log-level to debug and post the log with the error again?


Not downloading certain url

Checklist

[x] I'm reporting a broken site support issue
[x] I've verified that I'm running youtube-dl version 2019.06.27
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log

youtube-dl --verbose [youtube] 71Cs1pddOO4: Downloading js player vflxACNZ2
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', '[youtube]', '71Cs1pddOO4:', 'Downloading', 'js', 'player', 'vflxACNZ2']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.05.20
[debug] Python version 3.7.3 (CPython) - Darwin-18.6.0-x86_64-i386-64bit
[debug] exe versions: none
[debug] Proxy map: {}
ERROR: '[youtube]' is not a valid URL. Set --default-search "ytsearch" (or run  youtube-dl "ytsearch:[youtube]" ) to search YouTube
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 796, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 529, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/generic.py", line 2230, in _real_extract
    % (url, url), expected=True)
youtube_dl.utils.ExtractorError: '[youtube]' is not a valid URL. Set --default-search "ytsearch" (or run  youtube-dl "ytsearch:[youtube]" ) to search YouTube

Description

WRITE DESCRIPTION HERE
I was using youtube-dl and it was working just fine until I tried downloading from this url
https://www.youtube.com/watch?v=71Cs1pddOO4
I've updated youtube-dl and tried typing the youtube-dl command by putting comments around the url

your command is wrong, pass the URL directly to youtube-dl.


combine get inputs and type_parse function in onnx [fixed]

my bad I did run model benchmark locally and it passed but made a small adjustment after thinking it would be fine.
problem was this:

it looks like dim_value despite being Optional can be set to 0 when it's a variable dim.

cool, we also need to fix variable dim at some point

cool, we also need to fix variable dim at some point

yeah to do that we'll need to do more parsing at the beginning of get_run_onnx. I had plans for this already :D
will post PRs to share ideas to see if it's acceptable.


Use on_fetch for Python Worker handler.

We cannot use fetch for the handler name because it conflicts with the fetch function imported from js. This works out fine in JS, but not in Python where you end up calling the fetch handler recursively. Renaming this to on_fetch solves this problem.

Actually, let's update all the samples pyodide-xxx too.

hmm, I wonder: should we change test to on_test too?

Thanks @dom96!

I'm going to remove myself as a reviewer on this as I don't have enough context on the python stuff to do an adequate review.

hmm, I wonder: should we change test to on_test too?

Do we use test for Python workers at the moment? Assuming it's the same semantics as in JavaScript workers, I think test is a much better descriptor than on_test.


Delete Snippet

Is your feature request related to a problem? Please describe.
Add a delete button to delete any submitted snippet
Describe the solution you'd like
Making a DELETE request
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.

Are you gonna work on the mysnippet section on myprofile page ui or this issue?

MY Profile Page UI is done and I was thinking of adding it there only the user can delete only the snippets submitted by them I would give then a delete icon on the top right corner of the snippet from where they may send a delete request and I will seperate the rendering of bookmarked snippets and Submitted snippets .
I already made a PR for the UI changes #71
I pushed the new changes in there only reverted the changes I made in the route and added styling If you want anything more do let me know.
@notsoocool

Okay then i am assigning you this issue

@notsoocool DO checkout the PR .
I had worked on it till late to finish it on the same day and It has been 2 days since I made the PR.


Added the new features in that were added on March 13th to Mixer.

Fixes #85
NOTE: Textboxes don't work, but they won't crash the library either.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: funniray sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Not Working on Android 10

I have installed Sample App on my Oneplus 7 Running Android 10.
After clicking on Background button nothing happens notification is not visible as well.

766 / 5000
Résultats de traduction
Same problem with the "cordova-plugin-background-mode" plugin. (For mqtt I don't know because I don't have it in my project).
With the following permissions in the AndroidManifest:

FOREGROUND_SERVICE
SYSTEM_ALERT_WINDOW
==> User action ==> True
REQUEST_IGNORE_BATTERY_OPTIMIZATIONS
==> User action ==> Authorize

The pugin works on all phones.
However, when I use the plugin's "excludeFromTaskList ()" function, this is different.
When the application goes to the background, the application is excluded from the task list, but when I open another application, it stops systematically. This is valid for versions 10 and + of Android.
Does anyone have a solution?

Use this plugin ::
https://bitbucket.org/TheBosZ/cordova-plugin-run-in-background/src/master/

Yes I installed this plugin which is very useful with its "isIgnoringBatteryOptimizations" function and also for these two additional parameters in the "setDefault" function (channelName && channelDescription).
But unfortunately it did not solve my problem regarding the "excludeFromTaskList" function. The result is similar, on android 10 and above, as soon as I launch another application, mine in the background is automatically killed 👎🏻
So I gave up the idea of excluding it from this list and just

@GhoshWorld Thank you. That plugin works !
I also had to update to cordova-android@9, install cordova-plugin-androidx-adapterand add this to my config.xml <preference name="AndroidXEnabled" value="true" />

Yes I installed this plugin which is very useful with its "isIgnoringBatteryOptimizations" function and also for these two additional parameters in the "setDefault" function (channelName && channelDescription).
But unfortunately it did not solve my problem regarding the "excludeFromTaskList" function. The result is similar, on android 10 and above, as soon as I launch another application, mine in the background is automatically killed 👎🏻
So I gave up the idea of excluding it from this list and I content myself with the classic behavior.

I can confirm exactly the same beaviour. Any fix or workaround?


CHECK_CD is not saved into the CHECK_LOG table

Context
CHECK_CD is not saved into the CHECK_LOG table
Schemas: CAMDECMPSWKS, CAMDECMPSAUX
Steps to reproduce
SELECT chk_session_id, begin_date, mon_loc_id, source_table, check_result, check_cd, severity_cd FROM camdecmpsaux.check_log where extract (year from begin_date) = 2023 and check_cd is null
Private Zenhub Image

@ergjustin  will follow up with @djw4erg

Additional Information:
For ECMPS 1.0, the CHECK_LOG table on the Host included the CHECK_CD column, but the Client CHECK_LOG table did not.  The Client including the CHECK_CD in the submitted XML and that value was loaded into the Host table.
The code was not used in ECMPS 1.0 processing and Kyle indicated that it is not used in ECMPS 2.0.  So unless CAMD wants the value populated, I do not believe this ticket is necessary, and this seems to be Kyle's conclusion.

Closing since no changes are needed according to Kyle's comment and previous information on the ticket.


FR: Ensure batch error responses can be matched to their causing message

If I'm not mistaken it is currently not possible to match the single responses of a BatchResponse to the very specific messages of the call to sendAll in FirebaseMessagingClientImpl.
This is due to the fact that responses could be redirected/retried and then the order of the responses will be mixed up in the list of the BatchResponse resulting in a potential wrong matching to their original message.
I would therefore appreciate, if it would be possible to somehow match the potential error response of a single message to their root causing message.
A potential reason is, to figure out a specific device token that might be outdated and therefore resulting in a failed request within the batch request.
Feel free to correct me, if I got anything wrong.
Thanks in advance.

The order of the responses is guaranteed to match the order of the input messages. From the documentation:

The responses list obtained by calling getResponses() on the return value corresponds to the order of input messages.

Retries doesn't work at individual message level. It's all or nothing.

@hiranya911: Ok I was just confused due to the following facts:
If I check the implementation of the BatchUnparsedResponse:
https://github.com/googleapis/google-api-java-client/blob/0f6f940b70b4858b205424a2291820230b19d6e4/google-api-client/src/main/java/com/google/api/client/googleapis/batch/BatchUnparsedResponse.java#L190-L204
The retryAllowed property is true due to the outer batch request being allowed to be retried. (Even though the childRequestFactory doesn't support retries.)
This in turn will set the retrySupported to true.
As said before the childRequestFactory doesn't support retries so unsuccessfulResponseHandler is null.
This results in a call to handleRedirect.
I was now unsure whether this call could result for some of the responses in a "true" value, potentially resulting in those requests being added to the unsuccessfulRequestInfos list.
This of course won't happen if it is ensured that single responses never contain a result that may trigger that behaviour. And I guess this is the case, right?
Feel free to correct me, if I got anything wrong here.

Thanks @GFriedrich for the detailed and thorough analysis. We certainly don't want BatchUnparsedResponse to retry individual messages. To make sure this is the case I debugged through the following test case:
https://github.com/firebase/firebase-admin-java/blob/408b85f7d0072e8fb3b462aa2317843d683e7ad7/src/test/java/com/google/firebase/messaging/FirebaseMessagingClientImplTest.java#L372-L382
In BatchUnparsedResponse I observed that:

unsuccessfulResponseHandler is null on child requests. Therefore errorHandled never gets set to true.
FCM backend API doesn't send redirect status codes so redirectRequest doesn't get set to true.

These in turns cause the following condition to get skipped:
if (retrySupported && (errorHandled || redirectRequest)) {
    unsuccessfulRequestInfos.add(requestInfo);
}

However, now that I've taken a close look at the source, I'd like to make this behavior more explicit. Easiest would be to do the following on the child requests:
request.setUnsuccessfulResponseHandler(null);
request.setFollowRedirects(false);

I can create a PR to that effect sometime in the next few days.

FCM backend API doesn't send redirect status codes

Perfect, this is the info I was missing. 👍
Thank you for taking a look into this ticket again. Appreciate it.

@hiranya911 does the new sendEachForMulticast have the same order guarantee in the response as well?


Exception Unhandled in DiskItem.cs and WindowContent.xaml.cs during compilation

Running:

Microsoft Windows [Version 10.0.17763.316]
.NET Core 3.0.0 Preview 3
https://github.com/startnine/start9-ui/commit/d639cf41db028d8641c4d0528bb29645a1968bba
ribbonfilemanager/https://github.com/startnine/ribbonfilemanager/commit/575e4efa1964e3e92462203f36158fc40e6a049f
https://github.com/startnine/windows-sharp/commit/a7fc09a0075125af8768d0c26c6c8159b9ea3b76

During compilation, these errors are thrown:
line 47, DiskItem.cs

line 151. WindowContent.xaml.cs

A very terrible solution is to comment out line 165 of MainWindow.xaml.cs. However, that's what actually starts the initial tab when the program is starting.
Other solutions currently tried:

uncommenting line 212 in DiskItem.cs

206        public DiskItem(String path)
207        {
208            ItemPath = Environment.ExpandEnvironmentVariables(path);
209            if (Directory.Exists(ItemPath))
210                ItemCategory = DiskItemCategory.Directory;
211
212            SetFSInfo(ItemPath); // this line
213
214            //WatchFileSystem();
215        }

On my end, it looks like the DriveInfo.GetDrives() API in .NET was returning a drive D:\ even though I didn't have one, which tripped up DiskItem. I've fixed this issue on the end of this project, fix coming later.

Fixed in https://github.com/startnine/ribbonfilemanager/commit/c18c2ed85d8a299b0c88346fdf35f68c25fb0658 (I'd have to merge the branch to get it to auto-close so just gonna close it here)


Add text 'Dynamic programming is both a mathematical optimization met…

…hod and a computer programming method. The method was developed by Richard Bellman in the 1950s and has found applications in numerous fields, from aerospace engineering to economics. ' to the article.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Dynamic programming is both a mathematical optimization method and a computer programming method. The method was developed by Richard Bellman in the 1950s and has found applications in numerous fields, from aerospace engineering to economics.


Support for a new primitive type FStar.ImmutableArray

In response to a request from: @jaylorch
A new type, FStar.ImmutableArray.t is implemented primitively in OCaml as an array. Its operations reduce natively from within the F* normalizer and NBE engines, providing O(1) indexing into a sequence.
Supporting this involved also doing some surgery in the embeddings so that we pass the universe instantiations to the primitive steps. This accounts for a large part of the diff.
See tests/micro-benchmarks/TestImmutableArray.fst for an illustration of its use.
(Along the way, and totally unrelated, I fixed the behavior of interface interleaving when used in MLish mode so that any preceding opens and module abbreviations in the interface are included when checking the implementation, otherwise I keep hitting weirdness in emacs when trying to hack on the compiler, where some interfaces fail to check, unless I already have checked files for everything. That's a change in ToSyntax.Interleave)

This is just what we needed, thanks!

Hi Nik. This all looks good, no real comments on my part. It's interesting that the lazy embedding helps with SMT encoding (and I suppose extraction too). I was expecting just a completely opaque object, but this makes more sense.
You mentioned something in the F* call about plugins and interoperation. The problem is that, in the compiler, an immutable array is an array of terms, but a plugin will just natively use the ImmutableArray primitives and have, say, an array of integers or whatever. Am I on the right track? Not sure how to solve that though...

Thanks for taking a look, Guido. I'm merging this.

is an array of terms, but a plugin will just natively use the ImmutableArray primitives and have, say, an array of integers or
whatever. Am I on the right track? Not sure how to solve that though...

Yes, that is indeed the problem. It's orthogonal to this PR, but one thing we discussed in the past was to keep a tag to record what kind of native object it is, e.g., distinguishing a (native) array of integers from an (normalizer) array of terms from (nbe) an array of nbe.t


save kitti viewer

Hi,
I see the old kitti viewer version can save the visual result as *.jpg. but it's deprecated,
File "./kittiviewer/viewer.py", line 37, in
from second.core.point_cloud.point_cloud_ops import points_to_voxel
ModuleNotFoundError: No module named 'second.core.point_cloud'
so, with new kitti viewer web how to save all the result?
Thanks in advance.

The save result is implemented by directly get opengl buffer. the web version is not implemented yet. I will add a button to do this.
"save all result" has a problem that different point cloud may need different best camera position. so I have no plan to implement this.

yes, it's complex to get all result. I just want to save result one by one as former version. Thanks.


if (and) with indented condition list

This gives a ParseError saying it expected a ::= where I have undefined:
        return if (and)
            upper.test exactOrAny.toString()
            lower.test exactOrAny.toString()
        then exactOrAny
        else undefined

If I oblige and change the last line to else ::= undefined, the result is not remotely what I wanted out of this expression (and in fact not valid TS):
        return (((a,b) => a&&b)?(
            upper.test(exactOrAny.toString()),
            lower.test(exactOrAny.toString())):void 0)
        then(exactOrAny)
        type else = undefined

I was hoping to get this:
return (upper.test(exactOrAny.toString()) && lower.test(exactOrAny.toString())) ? exactOrAny : undefined;

Currently we forbid indented function application in an if condition, to avoid confusion with the usual "then" block. For example:
if foo
  bar
↓↓↓
if (foo) {
  bar
}
// NOT
if foo(
  bar
)

Perhaps we could make an exception when there's an explicit then. Meanwhile, you can wrap your condition in parentheses...

Just ran into this in a different context:
return false unless (and)
  ...

Ah, adding support for indented function application to postfix if/then should be easier to add. (...unless I'm missing some ambiguity...)

I just attempted to implement postfix if (and) [indented call], but failed.
I can see how to allow postfix conditions to use indented function application.
The problem is that there's already a parse for something like this:
x = if (and)
  foo
---
x = ((a,b) => a&&b) ? foo : undefined

So in particular if we use return instead of x = this parse takes priority, preventing the postfix clause from activating.

Maybe we need to solve the root problem of how to use (and) [indented call] in a condition. I wonder if it would work (and be unambiguous) to allow an indented condition to use (further) indented function calls:
// if/then/else
if
  (and)
    x
    y
  console.log 'yes'
else
  console.log 'no'

// postfix unless
return unless
  (and)
    x
    y


change import back to import type * as azdataType in create project from db dialog

I forgot about getAzdataApi() and why the import was originally import type * as azdataType when I was adding connection api calls in #19129 - turns out it was because the vscode extension doesn't load and throws an error about azdata not being found, even though this dialog isn't actually used in vscode.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 3 (66.67%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.005%) to 42.209%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/sql-database-projects/src/dialogs/createProjectFromDatabaseDialog.ts
2
3
66.67%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/book/bookTreeView.ts
2
38.21%

Totals

Change from base Build<PHONE_NUMBER>:
-0.005%

Covered Lines:
27884

Relevant Lines:
61780

💛  - Coveralls


No manga found

❯ manga-cli Search manga: One Piece EOF EOF Enter Number:
(zathura:250973): Gtk-WARNING **: 22:09:06.574: Theme parsing error: gtk.css:3213:16: Unit is missing. info: Opening plain database via sqlite backend. warning: sqlite database already exists. Set your database backend to sqlite error: could not open document
Currently using
  OS : Arch Linux x86_64
 Kernel : 6.6.28-1-lts

Check if you have dependencies installed?


Why the dag state in the ui website will disappear ?

I met a issue. I write  a dag using PythonOperator && TimeSensor && S3KeySensor . I add it into the dag by "airflow scheduler -d ********* -sd //***/test.py" . It run very well at first , but I find it turns to be the following :

I set up the airflow in the ubuntu ,add the scheduler and then quit from the terminal . it happened to be this .Are there any ways to add the dag into scheduler permanently ?

Did you hit the refresh button on the right side of the red rectangle in your screenshot?
If the link is not clickable it means the scheduler knows about the DAG but the web server hasn't found it yet.

Yes, I click the refresh button,but it still be not useful .  Any solution for this ?

Any way that I can use to add the dag into scheduler and show in the web permanently ? I need it to run every day .

Leave the scheduler running, maybe restart the web server? Make sure they both point to the same dags_folder.

OK. I run the scheduler in the remote ubuntu server . it seems this issue happened when I quit the remote ubuntu from terminal .


feat: Streamline dashboard view, fix some minor issues, add links

fixes #32
fixes #46

We should probably rename the file overview.ts and the url /overview too, if we opt for "info".
Same for the translation keys (maybe structure these into different views + common)


[typescript][enhancement]: Update typescript definitions to match the latest SDK v3.0.2

Hacktoberfest is underway right now and we would love for you to contribute  🚀  . More information on #hacktoberfest is here
Develop-with-Intuit for Hacktoberfest
Problem
Update typescript definitions file to match the latest SDK v3.0.2
Ref : index.d.ts file
Fix Needed
Add the missing API typings ( interfaces/classes ) into the definition file ( index.d.ts )
You can compare using the latest SDK version v3.0.2
Instructions to use the sample app
In order to test the typings locally from the sample project, you'll need to link the intuit-oauth package with yarn or npm.
# Link local intuit-oauth branch with typings support to sample typescript project
yarn link
cd sample/typescript
yarn link intuit-oauth

PR Guidelines
Kindly follow the below instructions to submit your PR :

Fork and clone the repository (typescript branch).
Run npm install for dependencies.
Test all your changes.
Run npm test to execute all specs.

Once ready, please submit a PR to the typescript branch here.
Note : PR's will be merged to the master branch once all the testing is completed.

Hi @abisalehalliprasan I would like to take a shot at this

@shlok-kothari : Awesome. Go for it. 👍

since this was merged, I suggest closing this issue.
By the way, since release 4.0 happened, have the typescript definitions been updated since then? @abisalehalliprasan


FocalLength

I have tried to add the focal length (210 mm) to exif file using:
exif_dict["0th"][piexif.FocalLength] = (210,100)
but I got the error:
AttributeError: module 'piexif' has no attribute 'FocalLength'
How can I solve the issue?
Thanks

May be you can use: piexif.ExifIFD.FocalLength


Autosquash bot

As a contributor it is frustrating to having to keep pressing the "Merge with master" button on my PRs. Also, the Cirq mergebot is flaky / rarely up.
What do you think about https://github.com/marketplace/actions/autosquash?

I don't see a drawback except I would miss CirqBot's cute little face.
There is no reason we couldn't enable it and test it out for a while while keeping old autobot around.

I'm going to close this - the decision for now is to keep running Cirq automerge bot.


Assessments should be editable before the due date and 'addable' before hard limit date

Update rules to ensure assessments are editable before the due date and can be added before hard limit date

@warrensearle is this done?

@Rebecca-mcknight1991 no this is not done


NetCDF4 dependency is missing in requirements file.

(fls_sat_verif) bcrezee@tsa-ln003:/scratch/bcrezee/scripts/support-c2e/src/fls_sat_verif(support-c2e)$ fls_sat_verif --calc_fractions --wd /scratch/bcrezee/wd_fls_sat_verif/ --start 22010100 --e
nd 22010200 --interval 6 --exp_model_dir /store/s83/bcrezee/EXP_TST/560/ --exp 560 --model c2e -vvv

-------------------------------
Working directory: /scratch/bcrezee/wd_fls_sat_verif/
-------------------------------

INFO:root:Your working directories:
INFO:root:   /scratch/bcrezee/wd_fls_sat_verif/sat
INFO:root:   /scratch/bcrezee/wd_fls_sat_verif/tqc
INFO:root:   /scratch/bcrezee/wd_fls_sat_verif/fls
INFO:root:   /scratch/bcrezee/wd_fls_sat_verif/plots
INFO:root:Calculating FLS fractions
INFO:root:   for Jan 01, 2022, 00 UTC to Jan 03, 2022, 00 UTC.
WARNING:root:Created new obs dataframe:
WARNING:root:  /scratch/bcrezee/wd_fls_sat_verif/fls/obs.p
WARNING:root:Created new fcst dataframe:
WARNING:root:  /scratch/bcrezee/wd_fls_sat_verif/fls/fcst_560.p
DEBUG:root:SAT timestamp:<PHONE_NUMBER>
WARNING:root:SAT file: /scratch/bcrezee/wd_fls_sat_verif/sat/MSG_lscl-cosmo1eqc3km_2112312345_c2e.nc
Traceback (most recent call last):
  File "/scratch/bcrezee/miniconda3/envs/fls_sat_verif/bin/fls_sat_verif", line 33, in <module>
    sys.exit(load_entry_point('fls-sat-verif', 'console_scripts', 'fls_sat_verif')())
  File "/scratch/bcrezee/miniconda3/envs/fls_sat_verif/lib/python3.8/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/scratch/bcrezee/miniconda3/envs/fls_sat_verif/lib/python3.8/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/scratch/bcrezee/miniconda3/envs/fls_sat_verif/lib/python3.8/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/scratch/bcrezee/miniconda3/envs/fls_sat_verif/lib/python3.8/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/scratch-shared/meteoswiss/scratch/bcrezee/scripts/support-c2e/src/fls_sat_verif/cli.py", line 189, in main
    obs, fcst = calc_fls_fractions(
  File "/scratch-shared/meteoswiss/scratch/bcrezee/scripts/support-c2e/src/fls_sat_verif/utils.py", line 363, in calc_fls_fractions
    ds = xr.open_dataset(obs_file).squeeze()
  File "/scratch/bcrezee/miniconda3/envs/fls_sat_verif/lib/python3.8/site-packages/xarray/backends/api.py", line 479, in open_dataset
    engine = plugins.guess_engine(filename_or_obj)
  File "/scratch/bcrezee/miniconda3/envs/fls_sat_verif/lib/python3.8/site-packages/xarray/backends/plugins.py", line 148, in guess_engine
    raise ValueError(error_msg)
ValueError: found the following matches with the input file in xarray's IO backends: ['netcdf4', 'h5netcdf', 'scipy']. But their dependencies may not be installed, see:
https://docs.xarray.dev/en/stable/user-guide/io.html
https://docs.xarray.dev/en/stable/getting-started-guide/installing.html

@swester I am looking into this, let me know if you encountered this problem already and know the solution.

Quick solution: Install netCDF4 by hand:   conda install -c conda-forge netCDF4
Will be added to requirements and the fresh installation should be tested.

Has been fixed in #7


refactor(esl-share): merge list and group attributes

:tada: This PR is included in version 4.5.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.6.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fixed citydb_pkg.set_enabled_geom_fkeys

This PR tries to fix the issue reported here #40.
The function set_enabled_geom_fkeys could also somehow be simplified by removing the database join with the pg_namespace table

:+1:


Ruby base module highlighted differently than rest of class/module is

(Original issue 1137 created by jbodah on 2015-08-08T16:45:22.855276+00:00)
See http://pygments.org/demo/2352761/
In this example we see that Jobs is highlighted differently than Search and DeepSearch. Ideally these would all be highlighted the same as they are all part of the class/module's name.
I ran into this when customizing my Jekyll highlighting and noticed that self and the base class/module name use the same highlighting (see http://pygments.org/demo/2352763/). In this example Jobs and self have the same highlighting. Ideally, Jobs, Search, DeepSearch, etc would have one highlight and self would have another

(Original comment by tshatch on 2016-06-01T20:50:19.356773+00:00)
The pastes are gone now, but maybe some Ruby expert can guess what syntax this was using.

Closing due to lack of reproducer :(


dynatrace_application_detection_rule_v2 resource errors right after dynatrace_web_application creation

Error: Value 'APPLICATION-XXX' could not be validated as no datasource options are available in the given context.
dynatrace_application_detection_rule_v2 resource errors after a dynatrace_web_application creation, subsequent apply succeeds - likely a timing issue.

Related to: https://github.com/dynatrace-oss/terraform-provider-dynatrace/issues/378
After giving some additional thought, we've decided to leave things as is since the current solution with DYNATRACE_CREATE_CONFIRM_WEB_APPLICATION seems to work and we don't any concrete alternatives that could improve the challenge.


RSI line seems to 'jump'

Steps to reproduce:

RSI must be immediately activated.
proportionOfDataToDisplayByDefault set to 1
Load the page and observe the RSI line 'jump'

bandicam 2016-02-29 10-23-18-520.zip

Waiting for next d3fc release


Editor - can't include a textarea on a page

Niko (@nmiran) has been trying to create some pages that include <textarea> tags. Although they preview initially ok they don't display correctly in the editor. Anything after the closing textarea tag is lost.
This must be because the editor itself uses a textarea tag and therefore doesn't want another textarea nested inside it. He's been using the embed content page to type his HTML into (although every page has this same issue)
For the moment I've given him a work around of writing the HTML in a separate file and showing it in an iframe but I wasn't sure if there was a way of getting this to work directly. I'm not sure there will be to be honest but thought I'd ask!

My own solution was to create the textarea at runtime using jquery. I think we are all noticing people doing more and more advanced stuff with html / css / js and running into a few issues. I'm wondering how we can make this more robust, though escaping code to store as xml params is tricky.

That's a much better work around - thanks