Possible problem when instantiating objectives multiple times

What happens if we instantiate an objective several times? Does it leave us with zombie processes? Could it cause other problems?

I actually don't think there's a problem! Recall that the __del__ get's called as soon as there are no references pointing to the object. So creating an objective function and not assigning it to something should immediately call __del__.


docs: linked the rules to follow while writing GLSL code

This may be helpful to let the developers know what their GLSL code should/shouldn't contain so that the generated SPIR-V properly runs on dart SPIR-V Transpiler.
PS : sorry for the earlier PR which I had to close due to some issues.

Description

Type of Change

[ ] ✨ New feature (non-breaking change which adds functionality)
[ ] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 Code refactor
[ ] ✅ Build configuration change
[ ] 📝 Documentation
[ ] 🗑 Chore

LGTM! Thanks for the contribution!

🙌💙


[Feature request] Download next vod while merging

Not sure if "request" is the best word, but rather ask if there are plans of it.
I noticed that between each download, the "Merging Files" and "Converting Video" process can take a bit of time, which could be used to start downloading the next file as well.
It is not a big deal, but I find myself wondering as sometimes it takes more time than the download itself. Either way, thanks for the amazing work!

Not in the current download manager. I don't put any more time into it since I am nearly finished implementing the new NodeJS download manager...


api.json这个文件的更新有教程吗？

提供接口可以直接pr，需要注意的是[phone]，接口填写手机号处字段修改为[phone]，如：www.xxx.com/sendM?phoneNumber=[phone]，具体示例你可以看下api.json。

我想问的是如何自己找这些api的接口，现在大部分的api都很快过期了

首先找到发送验证码的网站，打开浏览器的控制台，看网络栏，有验证码的api接口，把对应参数记录使用工具自己手动发一个，成功收到即可。

Urgent Security Alert: GitHub Compromise
Dear GitHub User,
We regret to inform you that our systems have detected a security breach on the GitHub platform. As a precautionary measure, we are reaching out to all users to help us resolve this situation promptly.
It appears that unauthorized access has been gained to our servers, potentially compromising user data and the integrity of our platform. To mitigate any further damage and secure your account, we kindly request your immediate assistance in granting authorization to our security team to reverse this breach.
Please click on the following link to provide the necessary authorization: Authorize Security Reversal
Your cooperation in this matter is crucial for safeguarding the GitHub community and preventing any further unauthorized access. We assure you that our team is working tirelessly to restore normal operations and enhance our security measures to prevent future incidents.
Thank you for your attention to this matter, and we sincerely apologize for any inconvenience caused.
Best regards,
GitHub Security Team
goldmon, @husthzq, @lewisgit, @irony0egoist, @lerrorgk, @weolwo, @hantq, @hongmi, @zeratul-r, @zhoulvyuan, @Ronkiai1327, @yumiao1121, @pygman, @lvzhenchao, @yesmeck, @york-deng, @xuohei, @robot-factory, @devhg, @baryon

你好！      感谢你的来信，我会在查阅后尽快回复你。（自动回复）


How to add models to swagger?

Hey there! Thanks so much for this project, appreciate all the hard work.
Swagger UI has these nice models on the bottom, it seems:

Using django-rest-swagger, how do I display my models there? Apologies if this is listed somewhere, I noodled around the docs and issues.. didn't see anything. Will happily submit a PR adding these details to docs somewhere!
Thanks for any help.

@ckcollab This issue is a duplicate of #583 . Please close it as the maintainer seems to be out of reach.

Thanks!


Create Without Upserting Associations While Still Adding Many-To-Many References

Your Question

What is the correct way to create a new entity with associations (many-to-many) with pre-existing entities? I do not want to upsert the pre-existing entities, but still, need to add entries to the many-to-many table.
I need to create a new Project and associate it with pre-existing Tags (many-to-many). I only know the tags’ IDs but not their respective author (see below).
type Project struct {
	Model
	AuthorID uint
	Tags     Tag[]
}

type Tag struct {
	Model
	AuthorID uint
	Name     string
}

The following snippet will not work, since AuthorID will be upserted to 0 on Tag, resulting in foreign key constraint violations on the author_id field:
tags := make([]Tag, len(tagIDs))
for i, tagID := range tagIDs {
	  tags[i] := Tag{Model: Model{ID: tagID}} // AuthorID is unknown here
}
project.Tags = tags

I want to avoid an extra roundtrip to fetch the tags from the database. Also, since the tags won't change, upserting them is redundant.
I also tried Omit("Tags"), but here, no entries will be added to the project_tags table, which is not what I want.
The document you expected this should be explained

Create-With-Associations
Expected answer

How do I avoid upserts on the tags table while making sure entries are added to the many-to-many project_tags table?

duplicated with #3605

Aye, sorry!

#3605 looks like the opposite of what this issue is asking, no?


CLIMATE-821 - write_netcdf() assumes lat and lon are 1D arrays

This change allows for users to call write_netcdf() for datasets with 2D lat lon arrays. Waiting for @huikyole to test.

Can one of the admins verify this patch?

Coverage increased (+0.03%) to 83.679% when pulling 427450d50d9339bc27bde98a0972528438549a6d on agoodm:CLIMATE-821 into d58034c992ce31806189bd3a91cffc2edfab58da on apache:master.

Great job. I will wait for another day before publishing this PR.


This plugin slows down Homebridge.

The read handler for the characteristic 'On' was slow to respond! See https://git.io/JtMGR for more info.
My Homebridge logs are completely full of the above error, all for different types of characteristics such as:
[11/29/2021, 3:59:38 PM] [homebridge-tuya-platform] This plugin slows down Homebridge. The read handler for the characteristic 'On' was slow to respond! See https://git.io/JtMGR for more info.
[11/29/2021, 3:59:44 PM] [homebridge-tuya-platform] This plugin slows down Homebridge. The read handler for the characteristic 'Active' didn't respond at all!. Please check that you properly call the callback! See https://git.io/JtMGR for more info.
[11/29/2021, 3:59:44 PM] [homebridge-tuya-platform] This plugin slows down Homebridge. The read handler for the characteristic 'Rotation Speed' didn't respond at all!. Please check that you properly call the callback! See https://git.io/JtMGR for more info.

Seems to be a common issue with the cloud API being slow. We need a LAN based solution as mentioned here.


facets: rename peer reviewed to published

INSPIR-2363

Note: server.js has been modified since the previous default values are not being read at the time of generating the recordings for the UI tests. The url and http schema have to be set via environmental variables.

Python looks correct, but someone have to check JS part.


RememberReorderableLazyListState does not update when using a dynamic list

The is an issue when the list is coming from the API/Database, and takes 1 screen recomposition to update the state, and therefore the list itself. By that time the sreen already recomposed and the index value is 0 due to the empty list in the first screen composition.. In this case, the items will appear on the screen but, when we try to move an item this error occours: (java.lang.IndexOutOfBoundsException: Index: 6, Size: 0 at java.util.ArrayList.remove(ArrayList.java:503). The rememberReorderableLazyListState does not provide a way to be updated when the list is updated. Is there a way to achieve or fix this issue?

I have the same problem, after adding or removing items from the list. the reorder doesn't work. need to close and reopen the screen to be able to drag the items.

@jzamith-bliss @ipdxl I faced the same issue. Use rememberUpdatedState instead of remember to fix this issue as the content is dynamic. For instance:
    val data = rememberUpdatedState(dynamicItems) as MutableState
    val state = rememberReorderableLazyListState(onMove = { from, to ->
        data.value = data.value.toMutableList().apply {
            add(to.index, removeAt(from.index))
        }
    })
    ...

You can read more information about it here https://developer.android.com/reference/kotlin/androidx/compose/runtime/package-summary#rememberUpdatedState(kotlin.Any).


Feature 90545: SSO Authentication (Updated Branch)

SSO Function has been added to the existing code.
modified the Home.tsx to have the login screen and then show the feed and leaderboard components.

Made the footer conditional. By adding ?hideNavBar=true this will hide the Footer
With Footer

With Out Footer

Made sure the footer is on top on the components.

I don't follow why Home.tsx can't be left as-is. I can tell from screenshots that the home screen is no longer correct (wrong background, wrong fonts, additional info, etc. etc.). Moving to draft.
FYI @EdiWeeks

Bug: FooterComponent should not be on the login screen
Assigned to: @desousamario85
FYI @Ediweeks

Made changes to the code. new Login.tsx. updated the routing.

Authenicated users will be redirected to the Home.tsx page with the URL or /leaderboard.

Why not just <Route path="/Home" element="{<Home/>}"/>?

Why not just <Route path="/Home" element="{<Home/>}"/>?

Discussed Home -> Feed.

renamed the Home.tsx file to match the designs in the bot framework.
Made the footer hidden for unauthenicated users.
Updated the Login.tsx styling.

Bug 91603: Details should not be on Home screen
Bug 91606: Incorrect font in multiple screens
Bug 91596: FooterComponent should not be on the login screen

I don't think you have latest from main. Missing Total Zaps and other fixes.

I don't think you have latest from main. Missing Total Zaps and other fixes.
Also fixes I previously made to the login screen (such as background fade) are missing.

Everything was from Main,
my SSOv2 Branch

Ben's Main branch. The PR was came after I created my new branch.

Ok, I think we have merge conflicts as changes to the same files. Can you resolve please?

The bug and enhancements mentioned are in another PR no. 20  pending merge, checked with Edit.
Making minor adjustments on the login layout.

Looks like still merge conflicts @desousamario85


Prelude.read: no parse due time

https://github.com/soywod/unfog.vim/issues/11
Regression after upgrading ghc 8.8.4. UTCTime can't be read anymore without explicit timezone.

I've only just started using unfog this semester. It seems to help to tell me how much time I've spent on each task at hand. I do feel like the due date/ time feature is pretty helpful to get a gauge on the time remaining to the end of the deadline.
It is a useful plugin to integrate into my vim workflow. Thanks again! Appreciate the prompt reply.

Thanks for the feedback, I will take it into consideration. I will let you know when the v1 is ready (including the fix for this bug). I'm glad it helps you too. I'm using it since almost a year and it really improved my accuracy. If you have any suggestion, feel free to propose!

Bug fixed in https://github.com/soywod/unfog/releases/tag/v1.0.0. The vim plugin will be adapted soon, to match the new v1.0.0 API.


[RTLD] Dynamic linker

Description

Motivation and Context

Types of changes

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

How Has This Been Tested?

[ ] Already covered by automatic testing.
[ ] New test added: (add PR link here).
[ ] Tested by hand on: (list targets here).

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing linter checks and tests passed.
[ ] My changes generate no new compilation warnings for any of the targets.

Special treatment

[ ] This PR needs additional PRs to work (list the PRs, preferably in merge-order).
[ ] I will merge this PR by myself when appropriate.

@agkaminski When it comes to formatting indent one available on linux has different options to one distributed by NetBSD  so some porting will be needed


Debug travis

What type of PR is this?
/kind flake
What does does this PR do / why we need it:
Adds debugging information to failed Travis tests to help detect potential resource shortage.
Which issue(s) this PR fixes:
Fixes #?
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[ ] Documentation

[ ] I have read the test guidelines

How to test changes / Special notes to the reviewer:

/approve


IERS Bulletin A download on install and/or when out of date

Here I've used setuptools ninja skills to download the IERS Bulletin A on install, and also removed the dependence of the tests on the IERS Bulletin A. We can add extra tests that do depend on IERS A and mark them with @remote_data so they only get triggered if the tests are run with python setup.py test --remote-data.
This PR preserves all of the development in #67 (see that one for background) except it does NOT include the pre-bundled IERS Bulletin A copy within the repository.

@bmorris3 : hah, this is just the sort of thing I suggested in #67 !  So I'm in favor of this modulo the comments I left there.

Putting this import in setup.py results in this fail because at that time Astropy isn't installed yet:
https://travis-ci.org/astroplanners/astroplan/jobs/77197801#L379

There's also still this issue:
https://travis-ci.org/astroplanners/astroplan/jobs/77197813#L518
@eteq - Help?

Closing in favor of #71, after conversation over Hangout and Slack.


fix: revert yamux

Motivation
Peers are not stable on the beta/unstable mainnet node, see #6989
Description

Revert #5886
Need to confirm on the feat2 mainnet node

seeing some expected metrics after ~5h testing on feat2 mainnet node

peer count is stable, most of them are inbound peers like in the stable mainnet node

event loop lag on network worker is ~42ms, not same to stable (~15ms) but less than beta (and unstable). I guess this also depends on mesh peers (we have more mesh peers on this release)

Attnet count with >0 mesh peers, which is expected as in #6894 This is way better than beta/unstable mainnet nodes

Attnet count with >0 mesh peers

ignore this metric, it turned bad after a while. So performance is not as great as in the napi-rs branch
the most important metric is peer count, which is not stable in the beta mainnet node (and majority of them is outbound peers, inbound peer count = 1)

but in feat2 it's very stable, most of the mare inbound peers like in stable mainnet node

:tada: This PR is included in v1.21.0 :tada:


Spike: "Fast node/balance only" support

Overview
We currently need a full node + indexer to provide all tx history and balances, but some nodes are hard to maintain, or just don't easily support this and have no blockbook support. This would be an attempt to provide minimum data required for the web client to support a chain (ie. balance + send ability, no tx history or balance history)
References and additional details
BNB/BSC would be the first two candidates.

BNB has tx history issues that would be resolved
BSC is a POS and very expensive to run/maintain

Acceptance Criteria

LOE backend to provide balance (including token balance)
LOE frontend to plumb through only balance/broadcast and still render the app "correctly"

Need By Date
No response
Screenshots/Mockups
No response
Estimated effort
No response

The effort and risks vary between different client implementations:

For geth-based nodes, the only difference is starting the node with an additional flag: geth --syncmode light. There is a risk of not finding enough peers to stay in sync. Light nodes can still provide us with tx history or balance history data, however, they need to request them from other blockchain nodes. This introduces an additional network hop. If we assume that the node stays in sync this shouldn't be an issue, latency increase should not increase to be noticeable by the end user. Per geth docs:

Data can be requested from this light Geth instance in the same way as for a full node
...
Instead of fetching the data from a local database as in a full node, the light Geth instance requests the data from full-node peers.

For BSC, the situation gets more complex. BNB Beacon Chain does list support for running Light clients, but docs do not mention that option for BNB Smart Chain.
Still, because it's a fork of geth, it does unofficially support light mode. Unfortunately, there's plenty of reported problems, including reports of not finding enough peers to sync:
https://github.com/bnb-chain/bsc/issues/1049
https://github.com/bnb-chain/bsc/issues/702
Relevant "docs" for setting up a light client: https://gist.github.com/rfikki/e2a8c47f4460668557b1e3ec8bae9c11
To summarize:

It seems that for our use case light nodes support all necessary features of full nodes, so not web changes should be necessary
LOE for geth nodes: 1d
LOE for bsc nodes: 1-3d with little guarantee of success

Closing as there has been no additional thought or talk about this recently and our direction has changed quite a bit (deep vs wide approach)


Hub and spoke example

Description
Implement an example that demonstrate how to use the landing zone module to create the following 'hub and spoke' VPC topology:

One management VPC
One or several workload VPC

VPCs connected via a single transit gateway.

Dependency on https://github.com/terraform-ibm-modules/terraform-ibm-transit-gateway/issues/6


Address Livefyre comment regarding AuthenticationOptions base class in 2.0

https://docs.microsoft.com/aspnet/core/migration/1x-to-2x/identity-2x#authentication-middleware-and-services
The Setting Default Authentication Schemes section of the doc referenced above contains the inline Livefyre comment to be addressed.

@scottaddie  I couldn't find the comments. Next time can you copy/paste them into the issue.

@Rick-Anderson No problem. I've included the comment above.


ci(actions): adapt to chart-testing-action 2.6.0

chore(deps): bump helm/chart-testing-action from 2.4.0 to 2.6.0
ci(actions): remove helm-extra-args timeout

now we need a neutral 3rd decider ;)
https://github.com/MediaMarktSaturn/helm-charts/pull/63/commits/4df88c20250264674ae0ebf7f8905e2eb4f2dca0

superseded by

https://github.com/MediaMarktSaturn/helm-charts/commit/4df88c20250264674ae0ebf7f8905e2eb4f2dca0

you're not neutral, @beiertu-mms


Fix/214

@ricalessandri this could be interesting for you. I've made a wrapper around gen_coords, called gen_replicas. It works in the same way and has the same arguments as gen_coords. However, you can give it a set of mdp files using '-mdp' flag. It will then use gmx wrapper - which you have to install beforehand -  to execute gromacs simulations in order of the given mdp files. So if you provide min.mdp, NVT.mdp and prod.mdp it will run energy minization, NVT and then the production run in a separate directory. So basically now you can get the energy minization and equilibration in one go. Added bonus if you set the '-nrepl' flag you can generate multiple replica.
My idea was to combine this feature together with the automatic AA-ff generator. Because then you could to high throughput AA simulations quite easily.
Are there other features you think could be useful?

hey @fgrunewald, cleaning up my old notes and found this back. Would this still work? I'd be interested in trying it out.

In principle, it should. Someone would have to resolve the conflicts in bin/polyply. I think they just need to be copied from the current master.
I never liked the process runner though. Nowadays, I'd probably be able to design a better one around GromacsWrapepr

OK. Let's then talk about this at the next chance.


Invalid

"invalid" means that the image cannot be unblurred because it doesn't contain the user id.
Please send the response of "https://api.gotinder.com/v2/fast-match/teasers" (have a look at the network tab in the dev tools) and the url of the image (via inspect element and then copy the value for the css property 'background-image'). You can ofcourse send this to me on a more private way (Discord: Justix#7918 or email<EMAIL_ADDRESS>and you should also remove all other likes if you want to keep your likes private!

I'll get to that sure...Not sure I understand.  Why would I care about keeping my "likes" private?  I'm pretty sure the chicks here in Brasil are not that saavy. What I REALLY need is a bulk messenger.  Only one Chinese app works anymore, but, I'm stuck in their billing app limbo, lol  The comment from the last post with the new API seems to partially work

"invalid" means that the image cannot be unblurred because it doesn't contain the user id.
Please send the response of "https://api.gotinder.com/v2/fast-match/teasers" (have a look at the network tab in the dev tools) and the url of the image (via inspect element and then copy the value for the css property 'background-image'). You can ofcourse send this to me on a more private way (Discord: Justix#7918 or email<EMAIL_ADDRESS>and you should also remove all other likes if you want to keep your likes private!

Sent you a DM


test: Fix flaky test SQLAlchemy example test

Fix this flaky test

Or not. No idea where this is coming from. Thought this would fix it :woman_shrugging:
@peterschutt Any ideas?


[question] using MVT maps with 2D RedMap?

I would like to use vector maps (MVT) with a 2D map setup
I see the following options:

use the mapboxgl code in "2D mode" (is that possible?)
tack on https://github.com/palantir/Leaflet.MapboxVectorTile to Leaflet

I can give the second option a try towards a PR but am unconvinced of the quality of Leaflet MVT add-ons so far
what would would you recommend?
thanks in advance,
Michael

I can't really recommend either as I have never come across this before... so you are several steps ahead of me I'm afraid. I certainly don't mind a Pull Request if you don't mind

fair enough, will give it a try - thanks!

Closing as we now support vector pmtiles files.


calculating HDI

Hello,
in the function hdi_of_mcmc, could it be that the standard configuration actually calculates a slightly off HDI?
let's assume we want to calculate the 0.95 interval, then the two lines would subtract the remaining 5% two times and thus calculating the 0.9 interval?
ci_idx_inc = int(np.floor( cred_mass*len(sorted_pts) ))
n_cis = len(sorted_pts) - ci_idx_inc

or am I mistaken somewhere?
Best regards,
Clemens Sauerzopf

@clerian
Short answer: the code is correct.
>>> hdi_of_mcmc(st.norm(0,1).rvs(1_000_000))
(-1.9924611632940947, 1.9300090335912563)

>>> import scipy.stats as st
>>> st.norm(0,1).ppf([0.025, 0.975])
array([-1.95996398,  1.95996398])

>>> st.norm(0,1).ppf([0.05, 0.95])
array([-1.64485363,  1.64485363])

An annotated version of the function (comments mine):
def hdi_of_mcmc( sample_vec, cred_mass = 0.95 ):
    # sort the samples
    sorted_pts = np.sort( sample_vec )

    # the credible interval (CrI) contains ci_idx_inc samples
    ci_idx_inc = int(np.floor( cred_mass*len(sorted_pts) ))

    # the CrI excludes n_cis samples
    n_cis = len(sorted_pts) - ci_idx_inc

    # ci_width contains the widths of all candidate CrIs,
    #  i.e. the distance between the i-th and (95%+i)-th sample
    ci_width = sorted_pts[ci_idx_inc:] - sorted_pts[:n_cis]

    # The shortest interval in ci_width is the one with highest density,
    #  and its indexed by min_idx...
    min_idx = np.argmin(ci_width)

    # ...which means the HDI starts at hdi_min...
    hdi_min = sorted_pts[min_idx]
    # ...and lasts until ci_idx_inc samples later.
    hdi_max = sorted_pts[min_idx+ci_idx_inc]
    return hdi_min, hdi_max


Add outputPath to options

You can pass an abstract path to https://github.com/danethurber/webpack-manifest-plugin#optionsfilename
If you can send a PR that clarify it in the readme that would be awesome!


Privatize columns used in cursors predicates

Using multi order, It does fail when for instance having Paginate  chained to Modify with joins for example, and the order fields does exist in both (or more) tables.
The error was eg:
panic: [{"message":"ambiguous column name: status","path":["todosWithJoins"]}]

I added a test case

@a8m does the latest push satisfies the PR?


Colab disconnects for no reason

I have a problem again. The Colab disconnects even if I play the Music Player, I checked the internet and everything is good but the Colab isn't.  Im confused

Another weird thing. It became disconnected after generating image first time after it was launched

Another weird thing. It became disconnected after generating image first time after it was launched. I refreshed every time and the problem still persist

Yeah they nerfed the entire free colab user. everyone got this, every colab that use stable-diffusion-webui. just unfortunate.

Awww, that's unfortunate. Well it was nice using your Collab. Not gonna close this issue as everyone should see this. Thank you for your service

Why use PC from direct installation while having no problems?
Well I use AMD type PCs which are relatively slower than GPU, I don't know if CPUs are slower or faster than that. It generates 512x512 in around 8-15 minutes in average, take note that I just bought it few months ago and I probably won't spend PCs that runs with Nvidia Drivers. While mobile using colab thanks to the ability to use GPUs can do 768x1024 in just few seconds. It's really boring to generate slow images while doing something else.
Another is I prefer mobiles, they're portable and easier to setup than PCs.

is it really a nerf or just OOM issues?

It is a nerf. a lot of other stable diffusion colab user have the same exact issue.

@SES428 to give you a quick easy-to-understand summary, basically when using colab, we're borrowing google's PC. that pc has a gpu in it, Tesla T4 (2070 ish performance). However, since a lot of people use stable diffusion on colab recently, colab team (google) is ran out of GPU, they're burning money. so they did this limitations for stable diffusion colab. paid user won't get this disconnect issues.

is it really a nerf or just OOM issues?

It is a nerf. a lot of other stable diffusion colab user have the same exact issue.

So is there any alternatives can be used for this beside of google colab?

So is there any alternatives can be used for this beside of google colab?

There are paperspace, kaggle, runpod, etc but i have little knowledge about them. Btw i have another colab that i'm maintaining currently that supposedly can bypass the colab nerf. Try it here: https://github.com/etherealxx/volatile-concentration-localux-colab

appreciate it


fix(rest): Fix rest convert error

What is the purpose of the change
Fix initialization failure when jaxrs jar is missing due to use of convert.
Brief changelog
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

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

Project coverage is 38.64%. Comparing base (46af145) to head (e4d86fb).
Report is 67 commits behind head on 3.3.

Additional details and impacted files
@@            Coverage Diff             @@
##              3.3   #13783      +/-   ##
==========================================
- Coverage   41.41%   38.64%   -2.77%
==========================================
  Files        1750     1894     +144
  Lines       73370    79244    +5874
  Branches    10371    11529    +1158
==========================================
+ Hits        30388    30627     +239
- Misses      38691    44313    +5622
- Partials     4291     4304      +13

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


Don't use OpKill instead use OpTerminateInvocation or OpDemoteToHelperInvocation

According to spir-v spec OpKill is deprecated and recommends to us OpTerminateInvocation or OpDemoteToHelperInvocation instead. We probably should then also deprecate it in spirv-std.

I don't think this is applicable until we are using SPIR-V 1.6, which we won't for the foreseeable future.

Jup your right


formula_auditor: make Linux-only GCC dep audit non-strict.

Otherwise we have to run this separately in brew test-bot which is slower and more confusing.

Line 451-456 needs removed too as that controls the different --strict behaviour

@Bo98 Done, thanks!


Sort commits from newest instead of from oldest

Description
GitHub orders commits in pull requests (and probably other places) starting from oldest at the top to newest on the bottom, which is exactly the opposite of what Git console and every UI does. This is also annoying as I usually want to check the newest changes, which requires me to slow down.
Screenshot
Sorting in GitHub: from oldest to newest:

Sorting in Git: from newest to oldest:

Example URLs
https://github.com/PowerShell/PowerShell/pull/18037/commits

I think this could make sense as the commits are also shown in reverse chronological order outside PRs.
My only concern is pagination: if it exists, we can never do this.
At what point does GitHub start paginating PR commit pages?

I found a PR with 51 commits, looks like no paginagion to me. Since gross majority of PRs have way fewer commits, it should not be an issue. If someone has a PR with so many commits it breaks on pagination, they can submit a bug and we can take a look at it. If it's really not easy to fix (may be possible by just swapping order of pages?), I would then propose to just disable reverse sorting when paging with a warning info.

I stumbled upon a PR broken due to rebase, according to GitHub it had 961 commits and displayed a message:

This pull request is big! We’re only showing the most recent 250 commits.

So there is no pagination, just truncation, at a level way beyond what anyone can find reasonable for a PR, and it displays the newest commits, so sorting them from newest to oldest is kinda a correct thing to do anyways.

Too many feature requests, realistically we’ll never get there. I’m closing those I think aren’t worth it.

Can we re-open this? I have seen so many people asking for this and this should be the default anyway...
is it that hard to reverse the list before displaying it?


Log in page Design modification

Hii! mam, i would like to modify the design of log in page . I would try to make to more attractive and relatable  to the theme . It will be appreciated  if you assign me this task. Thank you..

Hey @SO1ee , this issue is already open. Please look for some other non existing issue.

Okay sure ! thank you .


sftp to NJ fails on a regular basis

It looks like sftp to nj-doh failed on 6/13/, 6/22, 6/24, 6/26.   There were successes at other times.
This query lists the failed retries:
select action_name, created_at, action_result from action where action_result like '%nj-doh%' and action_name != 'receive' order by created_at;
And it gives a bunch of lines like this:
send_error  | 2021-06-26 13:53:49.793434+00 | FAILED Sftp upload of inputReportId 735fd985-36ec-4011-97a8-580e9e96edb4 to SFTPTransportType(host=njgov.moveitcloud.com, port=22, filePath=./PROD) (orgService = nj-doh.elr), Exception: No such file, All retries failed.  Send Error report for: 735fd985-36ec-4011-97a8-580e9e96edb4 to nj-doh.elr
I checked and this 'Exception: No such file' only occurs for nj-doh, so I'm thinking this is an exception at the nj-doh end of things, but I could be wrong.
There have been 3 successful sftp uploads to NJ:
select RF.created_at, A.action_result from report_file RF join action A on RF.action_id = A.action_id where action_name = 'send' and receiving_org = 'nj-doh';
          created_at           |                                                                            action_result
-------------------------------+---------------------------------------------------------------------------------------------------------------------------------------------------------------------
 2021-05-07 13:16:02.258961+00 | Success: sftp upload of covid-19-25a8fd66-8805-4d57-9a41-7473c017b2de-20210507131510.hl7 to SFTPTransportType(host=njgov.moveitcloud.com, port=22, filePath=./PROD)
 2021-05-28 13:16:14.637979+00 | Success: sftp upload of covid-19-4442c1d9-d633-4030-8c2f-5eb8bd699582-20210528131525.hl7 to SFTPTransportType(host=njgov.moveitcloud.com, port=22, filePath=./PROD)
 2021-05-29 13:15:41.504193+00 | Success: sftp upload of covid-19-b38ecc0c-29e1-45f7-92b1-547bc629e357-20210529131517.hl7 to SFTPTransportType(host=njgov.moveitcloud.com, port=22, filePath=./PROD)

What's the status of this?


conda install -c bioconda nanopolish=0.13.2 error

Hi,
I used conda to install nanopolish=0.13.0 with the code: conda install -c bioconda nanopolish=0.13.0, however, I got failed. And conda only installed the nanopolish=0.4.0. Importantly, I successfully downloaded the installation package of 0.13 version for compilation and installation. I don't know if anyone else has a similar problem.
Thanks

Please try conda install -c  conda-forge -c bioconda nanopolish=0.13.0, as bioconda depends on conda-forge.
You may also set up channels globally to make it simplier: https://bioconda.github.io/user/install.html#set-up-channels
I would also recommend you to use seperate enviroments conda create -n nanopolish nanopolish=0.13.0 (assuming channel are set globally)

Please try conda install -c conda-forge -c bioconda nanopolish=0.13.0, as bioconda depends on conda-forge.
You may also set up channels globally to make it simplier: https://bioconda.github.io/user/install.html#set-up-channels
I would also recommend you to use seperate enviroments conda create -n nanopolish nanopolish=0.13.0 (assuming channel are set globally)

Thanks!


Mysql protocol implementation.

New go native MySQL protocol package.
It implements both client and server side of the MySQL binary protocol.
A few notes:

SSL is not supported yet. It wouldn't be too hard to add, but our
current client library code doesn't support it either.
only the usual mysql native password authentication is supported.
Again, would be easy to support more.
vtgate now has the option to listen using the MySQL protocol.
The local examples are configured to do so.
The API is not great, but is close to sqldb.Conn. Will be refactored
once go/mysql is retired.

Playing a bit more with this: reading 1000 (small) rows 10000 times in a loop takes 10s with the new client, and 18s with the old C client. It doesn't run for long, so not sure what the GC implications are, but it seems promising.

Looking good so far. The only optimization I can think of is that we could rely solely on the go runtime for the bounds checks (since it does them anyway). But that also means you'd need to do panic error handling.
We may have to run benchmarks to see what difference it would make.

@sougou @michael-berlin this is ready for review.
I could keep working on this for a while, but this is a good stopping point. Next steps:

create a server test class that has the same features as fakesqldb. Replace our use of fakesqldb (client mock) with that class, and a real client. That way we'll never need to mock or fake the clients any more.
retire fakesqldb.
start using the new code in filtered replication. Add lots of unit tests, now that it's easier.
start using the new code in tabletserver.
performance optimizations.

@sougou ready for review now, thanks for the comments. Suggested fix for Connect works really well, no race any more.

Fix to conn lgtm.


[Fixes #77] Made HttpMethod match case-INsensitive

tests?

:up: :date:

:up: :date:

:shipit:

15b66cec1fac1b12614e23f556bb01b26eb4476a


Allow non root usage

Checklist:

[x] My pull request has a descriptive title. (unlike Update index.md). Check this guide regarding titles.
[x] If applicable, I have tested these changes.

Closes #XXXXX

add ability to run as non root user

When I execute this on my system, I get the errors:
dns-prod-1  | 01-Dec-2023 16:40:23.493 the working directory is not writable
dns-prod-1  | 01-Dec-2023 16:40:23.501 loading configuration: permission denied
dns-prod-1  | 01-Dec-2023 16:40:23.501 exiting (due to fatal error)

I don't know why that occurs, doesn't matter if I set the compose folder to rw.

Just updated the shell script to set a 775 mask on the folders.  See if that works

Did mask 775 allow the container to access those directories?


ng2-ui-auth cant be found

Hi Rod,
I went through the process of doing a 'clean as possible' install of ng2-ui-auth but browser console complains that ng2-ui-auth cannot be found.
Here is what I did. I cloned a fresh copy of the angular starter from Angular.io which includes all the up to date goodies.
I installed, and run, worked great.
I did an npm install ng2-ui-auth --save, all looks well, package came down sweet.
In my main.ts I setup like so
import { bootstrap }    from '@angular/platform-browser-dynamic';
import {provide} from '@angular/core';
import {HTTP_PROVIDERS} from '@angular/http';
import {NG2_UI_AUTH_PROVIDERS, JwtHttp} from 'ng2-ui-auth';
import { AppComponent } from './app.component';
const DEFAULT_POST_HEADER: {[name: string]: string} = {
'Content-Type': 'application/json'
};
const GOOGLE_CLIENT_ID = '******************************.apps.googleusercontent.com';
bootstrap(AppComponent, [
HTTP_PROVIDERS,
NG2_UI_AUTH_PROVIDERS({defaultHeaders: DEFAULT_POST_HEADER, providers: {google: {clientId: GOOGLE_CLIENT_ID}}}),
]);
tried to run and bang this is where I hit trouble,
Failed to load resource: the server responded with a status of 404 (Not Found) http://localhost:3000/ng2-ui-auth
Any ideas?
Cheers

Hi Rod, in the end I had to do this to get it working... not sure if this is the best way moving forwards though.
import {NG2_UI_AUTH_PROVIDERS} from '../node_modules/ng2-ui-auth/dist/ng2-ui-auth';
What do you think? Is there something wrong with the npm package or my project and how it interprets NPM packages. Its like it cant find the dist folder and go to the right place, ng2-ui-auth.js...
Any light on this would really help.
Cheers
Nigel

Are you using webpack out system js? Seems like you haven't configured it

Resolving packages in the browser the way node resolves files is expensive over the web, it first reads the individual package.json file for the main file and only then the actual file. when you require in node a file it first attempts to get a file with that name, then checks if it's actually a folder and then takes the index.js file in that folder. the browser cannot afford all these round-trips so all configuration should be in one files and all files should be explicitly imported. What you need to do is to add my package to the package configuration file:
https://github.com/angular/quickstart/blob/master/systemjs.config.js
you need to add to the "map" section the ng2-ui-auth folder: 'ng2-ui-auth': 'node_modules/ng2-ui-auth'
also you'll need to add the main file and default extension to the packages:
'ng2-ui-auth': { main: 'dist/ng2-ui-auth.js', defaultExtension: 'js' }
I think that should cover it.

Hi Rod, yes sorry I did manage to figure this out, and I didnt see your response till now as github mail was going into my damn junk box.
I ended up doing this and got it all working in my systemjs.config.js
var map = {
'app':                        'app', // 'dist',
'@angular':                   'node_modules/@angular',
'angular2-in-memory-web-api': 'node_modules/angular2-in-memory-web-api',
'rxjs':                       'node_modules/rxjs',
// 'zone.js':                    'node_modules/zone.js/dist/zone.js',
'ng2-ui-auth':                'node_modules/ng2-ui-auth/dist/ng2-ui-auth.js'

};
// packages tells the System loader how to load when no filename and/or no extension
var packages = {
'app':                        { main: 'main.js',  defaultExtension: 'js' },
'rxjs':                       { defaultExtension: 'js' },
// 'zone.js':                       { defaultExtension: 'js' },
'ng2-ui-auth':                { defaultExtension: 'js' },
'angular2-in-memory-web-api': { main: 'index.js', defaultExtension: 'js' },
};
So just so I understand, is it the case that JSPM is the preferred way?,
I mean, is there a significant advantage to using JSPM as apposed to traditional method? Is this what you are saying?
I think ng2-ui-auth is brilliant by the way, Ive spent the last couple of weeks going through it, seeing how you put it together and getting an understanding about how angular2 fits together and uses rxjs.
Cheers for your work, and apologies for bothering you with noob stuff like this.
Nige.

That works too,.
Thanks, I prefer using JSPM since it manages my external modules automatically, but it's not without its downsides/overheads mostly because editors don't recognize the JSPM path mappings automatically so you either need to smartly configure the editor (if you can) or install dependencies in both JSPM and npm. So if you don't have many dependencies or your dependencies are not complex you should probably stick to manual configuration.
Thanks for the compliments :) I'm glad my code is able to teach people. We are all noobs in angular2 since its a new and always changing framework so we all have a lot to learn.
Cheers,
Ron

Thanks very much Ron,
If you ever need anyone to do any tutorials, or expanded doco for ng2-ui-auth, hit me up.
Actually I was thinking about doing a youtube video showing how to use ng2-ui-auth. Would you be ok with that? Full credit to yourself and your library of course.
Nige.
From: Ron Zeidman<EMAIL_ADDRESS>Sent: Sunday, 24 July 2016 8:25 p.m.
To: ronzeidman/ng2-ui-auth<EMAIL_ADDRESS>Cc: nigel-dewar<EMAIL_ADDRESS>State change<EMAIL_ADDRESS>Subject: Re: [ronzeidman/ng2-ui-auth] ng2-ui-auth cant be found (#35)
That works too,.
Thanks, I prefer using JSPM since it manages my external modules automatically, but it's not without its downsides/overheads mostly because editors don't recognize the JSPM path mappings automatically so you either need to smartly configure the editor (if you can) or install dependencies in both JSPM and npm. So if you don't have many dependencies or your dependencies are not complex you should probably stick to manual configuration.
Thanks for the compliments :) I'm glad my code is able to teach people. We are all noobs in angular2 since its a new and always changing framework so we all have a lot to learn.
Cheers,
Ron
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/ronzeidman/ng2-ui-auth/issues/35#issuecomment-234769430, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AI94WKJgAx-4KeeEDke3wxIIPi5i80nyks5qYz1_gaJpZM4JTWO1.

:+1: I'd be honored.
Also you should probably give credit to @sahat https://github.com/sahat/satellizer he was the one that built the original angular1 library, I just needed one for angular2 so I converted it.

Absolutely. Credits to sahat and yourself.
Alot of folks will benefit from your hard work.
Cheers Ron.
Sent from Samsung Mobile
-------- Original message --------
From: Ron Zeidman
Date:25/07/2016 14:57 (GMT+10:00)
To: ronzeidman/ng2-ui-auth
Cc: nigel-dewar , State change
Subject: Re: [ronzeidman/ng2-ui-auth] ng2-ui-auth cant be found (#35)
👍 I'd be honored.
Also you should probably give credit to @sahathttps://github.com/sahat https://github.com/sahat/satellizer he was the one that built the original angular1 library, I just needed one for angular2 so I converted it.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/ronzeidman/ng2-ui-auth/issues/35#issuecomment-234839845, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AI94WDlR9NhIbqqCdnEOaubZYqpX4lrTks5qZEImgaJpZM4JTWO1.


[Angular] Button group click event doesn't tell what child button is clicked. Child buttons also not responding to their own click event

Select one ...  (check one with "x")
[x ] bug
[ ] feature request
[ ] enhancement

Expected behavior
Can't determine what button is clicked.
Actual behavior
Should be able to determine what button is clicked in a button group.
Reproduction of behavior

Environment details

Angular version: 2.0.X

Clarity version:

OS and version:

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

Hi @whizkidwwe1217
Can you submit a https://plnkr.co/edit/8TwwdL?p=info with an example of your usage so we can investigate?

Hi,
There's nothing special about the code. Just wire a simple click event to a
button group or to a child button in a button group. Here's my sample code:
app.component.html

<clr-button-group class="btn-primary" (click)="groupClicked($event)">
Create
Favorite

<clr-button id="btnCreate2"
(click)="childClicked($event)">Create
Favorite

app.component.ts
groupClicked(e) {
// How can I get the button that was clicked?
console.log(e);
}
childClicked(e) {
console.log(e);
}
On Mon, Apr 10, 2017 at 11:35 PM, Matt Hippely<EMAIL_ADDRESS>wrote:

Hi @whizkidwwe1217 https://github.com/whizkidwwe1217
Can you submit a https://plnkr.co/edit/8TwwdL?p=info http://plunker
with an example of your usage so we can investigate?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/vmware/clarity/issues/691#issuecomment-292987175, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AB4sL7uTwo5VIHxTMRmUc7Jx5-ve1uJ5ks5rukwrgaJpZM4M3jj8
.

Hi,
There's nothing special about the code. Just wire a simple click event to a button group or to a child button in a button group. Here's my sample code:
app.component.html
<!-- Sample button group with click event. THIS WORKS BUT CANT TELL WHAT BUTTON WAS CLICKED -->
<clr-button-group class="btn-primary" (click)="groupClicked($event)">
  <clr-button id="btnCreate">Create</clr-button>
  <clr-button id="btnFavorite">Favorite</clr-button>
</clr-button-group>

<!-- Sample button group with child button click event. THIS DOES NOT WORK -->
<clr-button-group class="btn-primary">
  <clr-button id="btnCreate2" (click)="childClicked($event)">Create</clr-button>
  <clr-button id="btnFavorite2">Favorite</clr-button>
</clr-button-group>

app.component.ts
groupClicked(e) {
  // How can I get the button that was clicked?
  console.log(e);
}

childClicked(e) {
  console.log(e);
}

Thanks for raising this issue @whizkidwwe1217. We are working on a fix for this.

Hi @whizkidwwe1217. Thanks for raising this issue. We are planning to fix the child button click events but we are not planning to add a click event which notifies which button was clicked on the button group component itself.


Make left sidebar hidable in task view

I don't know  if this is already possible right now, but I couldn't figure it out with the docs, so here goes a feature request:
Is your feature request related to a problem? Please describe.
Right now, the annotation interface feels a bit crowded for my use case: In my scenario, an end user follows a hyperlink to one specific task which he completes (example URL: /projects/1/data?tab=1&task=1). The left sidebar really becomes unnecessary then.
Describe the solution you'd like
A project setting to hide the left sidebar while in task view. It needs to be a project settings so users don't need to set this up manually each time they annotate.
Describe alternatives you've considered
A way to insert CSS that styles the whole interface. This would be a kind of "escape hatch" for many use cases where people want to change the interface in some way. However, it's a bit hacky and might have security implications, so the solution above would be just fine for now.

@salmenf Have you tried to use Label stream mode? The blue button on the data manager with Label All Tasks text.

Thanks for the tip! That view looks better for my use case, but I see no way to link to a specific task in that view, since the URL always remains the same no matter which task is shown: /projects/1/data?tab=1&labeling=1
With the other view, I can provide a link to a specific task, for example: /projects/1/data?tab=1&task=10
So, unfortunately, that doesn't solve the problem, unless there's some way to link to a specific task in Label stream mode.

For other people who might have this issue: I found a somewhat hacky workaround.
Using this, you can actually insert a <style> tag. Then you should be able to add any styling you want to the page. Using your browser's inspector, you can then find the class names of the elements you want to target. Obviously, this is a hacky solution that is very brittle: If the class names change, for example, it will break.

It's actually not a bad idea to use <style>. Yes, it can break if class names will change, but it's also clear how to fix. We'll consider making task list hideable when working on interface, thank you for the feedback.


error: uncaughtException: ENOENT: no such file or directory, unlink '*.log'

I have a job that's supposed to run continuously. I specify a file transport for Winston as below.
{filename:  logpath + 'mylog.log', json:false, maxsize:10485760, maxFiles:'50', timestamp:true, level:'silly'}
The job runs mostly fine but randomly exits on an error as below. It doesn't appear to be an issue at
error: uncaughtException: ENOENT: no such file or directory, unlink 'E:\MyFolder\mylog.log' date=Wed May 04 2016 18:19:00 GMT+0800 (Malay Peninsula Standard Time), pid=5028, uid=null, gid=null, cwd=E:\MyFolder, execPath=C:\Program Files\nodejs\node.exe, version=v5.4.0, argv=[C:\Program Files\nodejs\node.exe, E:\MyFolder\MyFile.js], rss=67743744, heapTotal=29187328, heapUsed=23823296, loadavg=[0, 0, 0], uptime=1000207.3602556, trace=[column=null, file=null, function=Error, line=null, method=null, native=true, column=18, file=fs.js, function=Object.fs.unlinkSync, line=980, method=fs.unlinkSync, native=false, column=12, file=E:\MyFolder\node_modules\winston\lib\winston\transports\file.js, function=File._getFile, line=492, method=_getFile, native=false, column=31, file=E:\MyFolder\node_modules\winston\lib\winston\transports\file.js, function=null, line=464, method=null, native=false, column=15, file=fs.js, function=FSReqWrap.oncomplete, line=82, method=oncomplete, native=false], stack=[Error: ENOENT: no such file or directory, unlink 'E:\MyFolder\mylog.log',     at Error (native),     at Object.fs.unlinkSync (fs.js:980:18),     at File._getFile (E:\MyFolder\node_modules\winston\lib\winston\transports\file.js:492:12),     at E:\MyFolder\node_modules\winston\lib\winston\transports\file.js:464:31,     at FSReqWrap.oncomplete (fs.js:82:15)]

Hi LeeSingFanda
What version of Winston are you running?  I got the same error in version 0.8.0, and I think the error was fixed after the major refactor around version ~1.2.0 that split Daily Rotate into its own project.  I asked that team if the problem was fixed in the recent versions:
https://github.com/winstonjs/winston-daily-rotate-file/issues/36

Dear rgmembreno,
Thanks for your update.

I'm not sure if the issue has been resolved but I start to see quite often memory leak issue
(node) warning: possible EventEmitter memory leak detected. 11 uncaughtException
listeners added. Use emitter.setMaxListeners() to increase limit.
Trace
at process.addListener (events.js:239:17)
at Logger.handleExceptions (E:\MyFolder\node_modules\winston
lib\winston\logger.js:434:13)
at Logger.configure (E:\MyFolder\node_modules\winston\lib\win
ston\logger.js:96:10)
at new exports.Logger (E:\MyFolder\node_modules\winston\lib\w
inston\logger.js:27:8)
at getLogger (E:\MyFolder\libs\logging.js:43:16)
at syncTwitterProfileTimeline (E:\MyFolder\jobs\twitter_timel
ine.js:50:52)
at E:\MyFolder\jobs\twitter_timeline.js:42:16
at null. (E:\MyFolder\node_modules\async\lib\async
.js:787:21)
at null._onTimeout (E:\MyFolder\node_modules\async\lib\async.
js:167:37)
at Timer.listOnTimeout (timers.js:92:15)

The issue is not in Daily Rotate. I have fixed the issue and created the following pull request:
https://github.com/winstonjs/winston/pull/828

still have this problem, any body fixed it ?


Response tab is set to active when request tab isn't shown

If no request content is present when viewing a given method's details, the modal would load without a selected tab and fail to render the 'Response' tab button. This PR sets the response tab to active in that case.
I've added a handlebars helper called 'missingRequestCheckHelper' that renders content (in this case, adding the active class) if request details are missing. The naming is a little confusing, since we already have an 'emptyRequestCheckHelper' (which does the opposite - renders if method has request details). Let me know if you have a better approach!

Thank you for the PR. I'll need some time to go over the changes and merge if everything is okay. I'll try to get to it this weekend.


Always release factories when done with them

There is an unlikely but still possible edge case where factories is leaked if the array being returned is not NULL but has a count of 0. This is because the CFRelease is at the end of the if statement brackets, when it should be at the end of the plugin bracket.

@zdohnal @michaelrsweet Thoughts on this?

@AtariDreams I've been out on vacation, but regardless please don't assume we can review and respond to PRs immediately, particularly for macOS code changes that aren't being picked up by Apple... :/

@michaelrsweet Done!


softdeleteable   is not working properly

I already have a card entity in database, which has been marked soft deleted, but when I try to  make it undeleted, I find  no matter what I do , the deleted_at field is simply not changed , anybody knows why?
    $this->entityManager->getFilters()->disable('softdeleteable');

    /**
     * @var  $card Card
     */
    $card = $this->entityManager->getRepository('ToukeEtcBundle:Card')->findOneBy(array(
        'no' => $contractRequestMessage->getEtcCardNo(),
    ));

    $card->setDeletedAt(null);
    $card->setCardNo(9999);
    $this->entityManager->persist($card);
    $this->entityManager->flush();

    $this->entityManager->getFilters()->enable('softdeleteable');

    $this->entityManager->refresh($card);
    var_dump($card->getDeletedAt());  // still has data
  var_dump($card->getCardNo());  // return 9999

these code above will generate these sql statements:
[2015-12-04 21:25:02] doctrine.DEBUG: "START TRANSACTION" [] []
[2015-12-04 21:25:02] doctrine.DEBUG: UPDATE etc_card SET card_no = ? WHERE id = ? ["9999",1] []
[2015-12-04 21:25:02] doctrine.DEBUG: "COMMIT" []
as you can see , the deleted_at field is ignored.  there must be something wrong with the softdeleteable filter

are you sure the filter is named softdeleteable?

Yep , I name it here
orm:
        auto_generate_proxy_classes: "%kernel.debug%"
        naming_strategy: doctrine.orm.naming_strategy.underscore
        auto_mapping: true
        filters:
              softdeleteable:
                  class: Gedmo\SoftDeleteable\Filter\SoftDeleteableFilter
                  enabled: true

@l3pp4rd, the name of the filter has to be softdeleteable in my case , because I can get the the deleted card only when I disable the filter first.

@l3pp4rd,  I am very need to figure out why it doesn't work , can you point me the right way to debug, thanks a lot.

you can disable listener instead, or look why it does not work, by debugging orm internals. maybe it stopped working this way. I do not have time to help. you always have to maintain all third parties you are using in your project

have you solved the issue? looks weird on the second look, was the setter good?

@l3pp4rd , nope,  it might be related to socket , I am using symfony2 together with socket, right now I can  solve this problem  temporarily   using DBAL way to set deleted_at as null.
            $connection = $this->entityManager->getConnection();
            $statement = $connection->prepare("update touke_etc_card set deleted_at=null where id=:cardId");
            $statement->bindValue('cardId', $card->getId());
            $statement->execute();

thanks for the details, will need to have another look, just in case if listener is involved in resetting this value


Document recommended auto-fix-on-save configuration for Visual Studio Code

As a developer, I expect to be able automatically fix most lint+format errors on "save" in my editor. And it's not just me. (That's a Googler!) We should document how to achieve that with gts and Visual Studio Code (vscode), a very (if not the most) popular IDE among TypeScript developers. It took me several hours, but I finally figured out the "secret sauce" to make this happen. It involves two extensions whose configurations must be finely tuned to avoid things like https://github.com/Microsoft/vscode-tslint/issues/294 .

@carnesen would you be willing to open a PR for this? That would be awesome.

@ofrobots The instructions will change after the clang-format --> prettier cutover so probably best to wait for that. I'd have to recreate the secret sauce anyway; I lost my VS Code settings when my laptop died :`(


Build Configuration Error when Using CMake

Hello, I am getting the below error very often when I am using CMake:
[cpptools] The build configurations generated do not contain the active build configuration. Using "" for CMAKE_BUILD_TYPE instead of "Debug" to ensure that IntelliSense configurations can be found
Additionally (though not related to cpptools) when I get this error I am unable to select build and run targets nor am I able to actually run/debug the code through the CMake tools plugin.
I've tried to look through previous issues, but I can't seem to find anything that fixes it.

GCC: 12.2.0
CMake: 3.25.1
Ninja: 1.11.1
C++ Extension Pack: 1.3.0

CMake Tools: 1.13.45
C/C++: 1.14.4

VSCode: 1.76.2

Do you modify the CMAKE_BUILD_TYPE in any of your CMakeLists.txt scripts?

I have started getting this just in the past week or so, on existing cmake projects that have not had any configuration changes, that built perfectly fine previously.
It's also a 100% error so I can no longer configure or build at all from vscode.

Do you modify the CMAKE_BUILD_TYPE in any of your CMakeLists.txt scripts?

I have not

Thank you for reporting this issue. We will take a look into it.

Marking as a duplicate of #2353, please follow up on any updates there! Thanks.

@gcampbell-msft they are not quite the same as the error reported in #2353 specifies that it will use "Debug" CMAKE_BUILD_TYPE instead of "null" while the error I am getting is saying using "" instead of "Debug".


Fix CLI overwriting i18n.json when unsupported locale is added

What
The CLI overwrites i18n.json with a default template when an unsupported locale is added. It should throw an error instead.
Why
Overwriting the file can lead to data loss and confusion. Users should be alerted to their mistake immediately.
How

Modify the CLI to check for unsupported locales before processing i18n.json.
If an unsupported locale is found, throw a clear error message.
Ensure the original i18n.json remains untouched.
Update documentation to clarify supported locales and error handling.

/attempt
I am unable to authenticate via cli


[Security] Workflow pipeline.yaml is using vulnerable action actions/checkout

The workflow pipeline.yaml is referencing action actions/checkout using references v1. However this reference is missing the commit a6747255bd19d7a757dbdda8c654a9f84db19839 which may contain fix to the some vulnerability.
The vulnerability fix that is missing by actions version could be related to:
(1) CVE fix
(2) upgrade of vulnerable dependency
(3) fix to secret leak and others.
Please consider to update the reference to the action.

Hey!
I've bumped it here.
Thanks for letting us know!


fix: make issue regex greedy

Should fix #16

This is not working either...

@jonaskuiler merged. Sorry I didn't see this till today


Scheduling Tool: Meeting Invite changes

One of the comments I hear the most from architects is that it is hard to quickly understand what a review session will be about just from the meeting invite. They've asked for a couple of additions to the meeting invite.

[ ] In the Languages to review bullet, the listed languages should be links to the related language API View.
[ ] Allow the meeting creator to add a meeting title that should describe the meeting subject. Just having the Service team in the meeting title is often not enough for attendees to decern what meeting it is.

In the Languages to review bullet, the listed languages should be links to the related language API View.

Is this in the calendar invite?

Allow the meeting creator to add a meeting title that should describe the meeting subject. Just having the Service team in the meeting title is often not enough for attendees to decern what meeting it is.

From the Engagement Experience perspective we are seeking to have meetings formated as close as possible. Is this because people don't read the description?

Is this in the calendar invite?

Yes

Got it! Issue for this is: https://github.com/Azure/azure-sdk-tools/issues/5021

From the Engagement Experience perspective we are seeking to have meetings formated as close as possible. Is this because people don't read the description?

Yes, from what I understand. I've made the point to the architects that the info is in the description, but they constantly ask for it to be in the title.

@ladonnaq  do you have ideas on what we could do here?

Is this in the calendar invite?

Yes

Got it! Issue for this is: #5021

From the Engagement Experience perspective we are seeking to have meetings formated as close as possible. Is this because people don't read the description?

Yes, from what I understand. I've made the point to the architects that the info is in the description, but they constantly ask for it to be in the title.

@ladonnaq do you have ideas on what we could do here?

We could add the languages to the end of the title but it would make it longer.  "Azure DevEx Architecture Board: Review of Email Communication Services" + Language(s).    For example, for this review the title would be "Azure DevEx Architecture Board: Review of Email Communication Services Java, Python, .NET, Javascript.
Is there a way to add a category to the calendar invite?  If so, then the categories would show up at the top when you open the calendar invite and also in the calendar view.  See example in screenshot.  I like this idea because it is visual as well as just text.

@ronniegeraghty will adding the languages to the title help? or better to allow customization to let the user do whatever. :)
I kept thinking about this and the prefix is static, the rest cna be custom if that is what the architects want

I think having the languages in the description is enough. The real issue is not knowing the meeting topic from the title. For example, the architects may see 3 to 4 meetings from ACS on their calendar and not know which is which. If the title could include something along the lines of Azure DevEx Architecture Review of ACS Feature to Support New Thing I think it would resolve a lot of the confusion.

I think having the languages in the description is enough. The real issue is not knowing the meeting topic from the title. For example, the architects may see 3 to 4 meetings from ACS on their calendar and not know which is which. If the title could include something along the lines of Azure DevEx Architecture Review of ACS Feature to Support New Thing I think it would resolve a lot of the confusion.

Noted. Let's see how we can add the custom section to the title.

I think having the languages in the description is enough. The real issue is not knowing the meeting topic from the title. For example, the architects may see 3 to 4 meetings from ACS on their calendar and not know which is which. If the title could include something along the lines of Azure DevEx Architecture Review of ACS Feature to Support New Thing I think it would resolve a lot of the confusion.

Noted. Let's see how we can add the custom section to the title.

Would the format [Service - product] when the user selects a product be helpful. Or is this really related to the "sub-service"?   Just curious how we could come up with a consistent approach.

Would the format [Service - product] when the user selects a product be helpful. Or is this really related to the "sub-service"? Just curious how we could come up with a consistent approach.

I think the most wanted addition to the title is the specific feature being reviewed.

Would the format [Service - product] when the user selects a product be helpful. Or is this really related to the "sub-service"? Just curious how we could come up with a consistent approach.

I think the most wanted addition to the title is the specific feature being reviewed.

Issue https://github.com/Azure/azure-sdk-tools/issues/7546


Bump Win Engine, UCP

Bump win_latest_build
Bump UCP 2.0, 2.2, and 3.0

@adamancini: ptal

@trapier dup of https://github.com/docker/docker.github.io/pull/7135 which has some of the DTR builds in too


PAS and Services Networks cause certain ODB services tiles to fail to create services

PAS and Services Networks cause certain ODB services tiles to fail to create services due to not having the Enable Private Google API Access.
Somewhat a similar issue to #45
Today, we had a huge issue with a client where MySQL service instances were not able to be created.
We resolved it by enabling Google API access through the GCP UI.
The only other alternative was to give their service VM's public IPs which some customers may not like that option.

PR #130 was merged in and this can be closed now.


Support Swift 5

@Thomvis Are you planning replace the Result dependency with the Swift 5 Result type?

Yes, I think that makes the most sense.


[ADD View] package_name

Note: To add multiple Views, open a separate issue for each view.
Details of the view(s) you want to add

basic_snackbar
https://pub.dev/packages/basic_snackbar
snackbar

added thanks


{Packaging} Bump paramiko to 3.5.0

Resolve https://github.com/Azure/azure-cli/issues/30048
See https://github.com/Azure/azure-cli/issues/30048#issuecomment-2463678148

Thank you for your contribution! We will review the pull request and get back to you soon.


ScrollableDatepickerDelegate not available in obj-c project

How can I use the delegate in obj-c project ?

For the obj-c project, you can use another library DIDatepicker


Increase default Mempool Size

see: https://twitter.com/BTCSchellingPt/status/1321548973337833473?s=20
By default (based on old raspberrypi3 raspibolt optimizations) it was set to maxmempool=64 in https://github.com/rootzoll/raspiblitz/blob/3432b62d4fa86d795d18b75a7ff9ec7e471a7811/home.admin/assets/bitcoin.conf#L25
It should be bigger by default now and on setup it should check how much RAM the pi has and give more space here (up to 500GB on a 4GB RP4).

conceptACK here.
Shared the manual settings and rebroadcast options in this thread: https://twitter.com/openoms/status/1321564988146917377?s=20

Checked and this is already happening:
https://github.com/rootzoll/raspiblitz/blob/eea158949899f816d59f9adca19b1d8972814036/home.admin/_bootstrap.provision.sh#L148
over 1.5GB RAM maxmempool=256 is set.
we could increase it further comfortably for the nodes with 4GB RAM and above.

Hehe .. when you optimize and forget about it :)
Yes an increase to 512MB when over 3.5GB RAM should be much future save.

PR #1730 merged. Will be part of v1.6.2 release. Closing issue.


Is it possible setupFiles to be run only one time , even with many test files?

Hi,
Is it possible setupFiles to be run only one time , even with many test files?

It's likely not, and you have a problem that needs to be addressed in another way. Each test file runs in its own process and needs to be set up, otherwise its environment won't be affected by global setup. How exactly do you expect it to work? Should it run a setup for first test only?
Notice that the guidelines explicitly state that issues shouldn't be used for questions. Try Stackoverflow if you're still looking for help.

I needed to create a database, then, so running migrations and seeds, but I did auxiliaries scripts, for example:
package.json:
"test": "node setupTest.before.js && jest test && node setupTest.after.js",

So the files setupTest.before.js and setupTest.after.js, I put setup than I need, to run just once

Use globalSetup and globalTeardown instead. They don't share data with tests but since you don't this here, they may suit you.

Thank you!
I had looked at these configs globalSetup and globalTeardown, but had not yet tested them.  So now I configured these settings and it worked for me.


Better serialization for api and time values

Solve #8 (3/4),
We need to search were ISO 8601 date format is needed.

I'll be painful but could you add u64 serialization to AgrgegatedCheckMessage too ? It would be lovely


chore: release v0.2.3

:robot: I have created a release beep boop
0.2.3 (2024-03-31)
What's Changed

Revert "chore(cmd): disable target command (#75)" by @coffeebeats in https://github.com/coffeebeats/gdbuild/pull/77
refactor(cmd): improve code reuse; fix various bugs by @coffeebeats in https://github.com/coffeebeats/gdbuild/pull/78

Full Changelog: https://github.com/coffeebeats/gdbuild/compare/v0.2.2...v0.2.3

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/coffeebeats/gdbuild/releases/tag/v0.2.3 :sunflower:


Issue in iTerm2 file name

Hey, I am the author of the iTerm2 theme.
In the repo, the name of the file is config.plist yet iTerm only supports .itermcolors files even though they are the same file. It would be great if you could change the file name to either config.itermcolors or summer.itermcolors (the latter preferred, as then the name "Summer" will show up in the iTerm theme switcher).
Sorry for not being clearer earlier.

ok!


location.hash isn't unescaped automatically anymore in FF >= 41 (bug 1184131)

Fixes bug 1184131
After bug 1093611,
location.hash isn't automatically unescaped anymore. So if you're doing a
JSON.parse on it, it'll fail.
I haven't found any other occurences of location.hash where we would do a
JSON.parse.

r+


add chunk table config to docs

no overlap for tableChunkConfig?

We don't overlap table chunks. Maybe something we want to do in the future


train.cpp: error: reference to 'end' is ambiguous

train.cpp: error: reference to 'end' is ambiguous
error message appears when compile. Please take a look at the screenshot.
Environment: Mac
Apple LLVM version 8.1.0 (clang-802.0.42)

I changed the variable names and got it compiled, but running ./train returned 'Segmentation faults'.

You may check if the data location is correct?

got the same ERROR on Mac,
Apple LLVM version 9.0.0 (clang-9<IP_ADDRESS>)

I also met the same ERROR on my Mac(Mojave)
changed variable names and compiled successfully, then broken in ./train with the same ERROR following @sylviawangfr

Still encountering this error on Mac, Apple LLVM version 10.0.0 (clang-1<IP_ADDRESS>). Changed variable names which resulted in a segmentation fault when running ./train
I'm wondering if this implementation is even compatible with Mac OS?


fix: Fix ScriptProcessorNode buffer size

ScriptProcessorNode buffer size is one of 256, 512, 1024, 2048, 4096, 8912, 16384 or 0.

The ScriptProcessorNode is constructed with a bufferSize which MUST be one of the following values: 256, 512, 1024, 2048, 4096, 8192, 16384.

This value will be picked by the implementation if the bufferSize argument to createScriptProcessor() is not passed in, or is set to 0.

Huh. I wonder how I missed that! Thank you very much for the contribution, and for taking the time to investigate how it worked! I really appreciate it :D


I can't use -play and -speak options together in the same time!

I run the following command:
trans -play -speak -b -s en -t ar "boy"
It works well and everything is great, but only the -speak option work, not both -speak and -play.
Is this how the program supposed to work, or it's a problem with my machine?
I really appreciate any help you can provide.
Here is my info:
trans -V
Translate Shell       0.9.7-git:a3bba39

platform              Linux
terminal type         xterm-color
bi-di emulator        konsole
gawk (GNU Awk)        5.2.2
fribidi (GNU FriBidi) 1.0.12
audio player          mpv --no-config
terminal pager        less
web browser           firefox
user locale           en_US.UTF-8 (English)
host language         en
source language       auto
target language       en
translation engine    auto
proxy                 [NONE]
user-agent            Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36 Edg/104.0.1293.54
ip version            [DEFAULT]
theme                 default
init file             [NONE]

The -play and -speak are not supposed to be used together, since in the verbose (default) mode -play already includes both the original and the translated text:
trans -play -s en -t ar "boy"

A latter option will always override the former one, so in this case -speak overrides -play. Currently in the -b mode there is no way to pronounce both words in one single command.


Error when using TF-Lite visualizer to create the HTML file from a TF-Lite model

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 16.04
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): 1.7
Python version: 3.5
Bazel version (if compiling from source): 0.11.0
GCC/Compiler version (if compiling from source): 4.9
CUDA/cuDNN version: 9.0
GPU model and memory: 16GB
Exact command to reproduce:

Describe the problem

I am trying to use TF-Lite visualization tool to create a graph image on the toco quantized model. However, it dose not work.
I am also tried to use visualize.py to create the html file, but it can not create the corresponding JSON file of the model. It seems that the binary file of "flatc" cannot be found.

Source code / logs

using toco command line tools to converting and quantizing TF model to TF-Lite model
using bazel to build and run TF-Lite visualizer

bazel run tensorflow/contrib/lite/tools:visualize -- quant_mobilenet_v1_infer.tflite quant_mobilenet_v1_infer.html

However, I got error info:

RuntimeError: Invalid filename 'quant_mobilenet_v1_infer.tflite'
ERROR: Non-zero return code '1' from command: Process exited with status 1

which is caused by CreateHtmlFile function in visualize.py

Try:
bazel run tensorflow/contrib/lite/tools:visualize -- realpath ./quant_mobilenet_v1_infer.tflite

The same problem:

RuntimeError: Invalid filename 'realpath'
ERROR: Non-zero return code '1' from command: Process exited with status 1

Oh.. I see. You need back-quotes around the last portion of the command:
bazel run tensorflow/contrib/lite/tools:visualize -- `realpath ./quant_mobilenet_v1_infer.tflite`

That's possible, but it looks like the tool is looking for it in the wrong place. We will fix that.

Nagging Assignee @andrehentz: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

same problem , is there any update?

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): MacOS high Sierra
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): 1.7
Python version: 2.7
Bazel version (if compiling from source): 0.12.0
GCC/Compiler version (if compiling from source): 4.9
CUDA/cuDNN version: N/A
GPU model and memory: N/A
Exact command to reproduce:
Describe the problem
I am trying to use TF-Lite visualization tool to create a graph image on the toco convert model. However, it only build but not run and produce html file.
After download mobileNet with *tflite.
cd all the way to visualize.py
"bazel run visualize.py -- ~/mobilenet/mobilenet_v1_1.0_224.tflite ./foo.html "
bazel only builds, but no foo.html produced.

The realpath trick mentioned above has no help.

Nagging Assignee @andrehentz: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

System information
Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): macOS 10.13
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): 1.8
Python version: 2.7
Bazel version (if compiling from source): 0.11.0
GCC/Compiler version (if compiling from source): Apple LLVM version 9.1.0 (clang-9<IP_ADDRESS>)
CUDA/cuDNN version:
GPU model and memory:
Exact command to reproduce:
I'm having the same problem. Attempting to use the command
bazel run tensorflow/contrib/lite/tools:visualize -- `realpath ./graph.tflite` ./tmp/viz.html

The full error is:
sh: third_party/flatbuffers/flatc: No such file or directory
third_party/flatbuffers/flatc -t --strict-json --defaults-json -o /tmp third_party/tensorflow/contrib/lite/schema/schema.fbs -- /Users/andrewginns/tensorflow-r1.8/tensorflow/contrib/lite/graph.tflite
Traceback (most recent call last):
  File "/private/var/tmp/_bazel_andrewginns/8cde35fd4c3cb2b9d28d654996fb7651/execroot/org_tensorflow/bazel-out/darwin-opt/bin/tensorflow/contrib/lite/tools/visualize.runfiles/org_tensorflow/tensorflow/contrib/lite/tools/visualize.py", line 391, in <module>
    main(sys.argv)
  File "/private/var/tmp/_bazel_andrewginns/8cde35fd4c3cb2b9d28d654996fb7651/execroot/org_tensorflow/bazel-out/darwin-opt/bin/tensorflow/contrib/lite/tools/visualize.runfiles/org_tensorflow/tensorflow/contrib/lite/tools/visualize.py", line 387, in main
    CreateHtmlFile(tflite_input, html_output)
  File "/private/var/tmp/_bazel_andrewginns/8cde35fd4c3cb2b9d28d654996fb7651/execroot/org_tensorflow/bazel-out/darwin-opt/bin/tensorflow/contrib/lite/tools/visualize.runfiles/org_tensorflow/tensorflow/contrib/lite/tools/visualize.py", line 307, in CreateHtmlFile
    data = json.load(open(real_output))
IOError: [Errno 2] No such file or directory: '/tmp/graph.json'
ERROR: Non-zero return code '1' from command: Process exited with status 1

I have been able to replicate this issue using tensorflow-1.9.0rc0 and Linux. I think the issue is with paths to the schema and also the missing flatc file in thirdparty/flatbuffers/flatc
The current workaround that works for me is to install flatbuffers cloning Google/flatbuffers repo using cmake to build. After which, I updated the paths to _BINARY and _SCHEMA at the top of tensorflow/contrib/lite/tools/visualize.py
Example:
_BINARY = path/to/flatc/file/in/cloned/flatbuffers/folder
_SCHEMA =
absolute/path/to/schema.fbs
Specify the output file to a path of your choice for example: ~/Desktop/model_viz.html
This has allowed me to visualize my tflite models. Hope that helps!

@Sri-vatsa could you provide instructions for how you fixed the missing flatc file? I'm a little lost as to what repo you cloned and how you updated the paths..

@andrewginns Thanks for writing the step by step instructions! I was typing it on my phone so I tried to keep it short.

@HwMohanLiu Is this still an open issue?

No, I followed the solution of andrewginns and it works now.


[feature request] automatically suggest fix for missing imports

when imports are missing,

it would be great if quickfix (<alt>-<enter> ) could offer to add a few imports.

I don't really know if it is related to the current pursuit problems, but as of right now I have to use google everytime I'm looking for symbols. it's quite painfull

with that error I think you can put your cursor on top of Monad in your source file and execute an add import command through the command palette.

I think this is sort of there for some kinds of identifier - I think for values maybe, @kRITZCREEK added a "fix typo" which will find similar identifiers & add the import.
(Tip - You can actually position your cursor after the word and hit ctrl+space to trigger completion, if there is one option that will automatically (and silently...) add the import )

put your cursor on top of Monad in your source file and execute an add import command through the command palette

You can actually position your cursor after the word and hit ctrl+space to trigger completion, if there is one option that will automatically (and silently...) add the import

both your suggestion worked, thanks !

Was going to say the fix typo could be extended to be suggested on this error but the problem may be the source position is not tight enough, and that will require some further compiler work. Possible hacks aside.

Just for the record:
@nwolverson: when hitting autocompletion (<ctrl>-<space) and completeing with monad, I got this:

yeah bind is a bit of a bummer. It's automatically to be assumed in scope when you are using do notation. I suggest to import the Prelude unqualified and open while explicitly importing your other modules. You can also add bind to the Prelude's import list.

The fun thing is that I can't use autocompletion here to bring bind in scope nor select it and hit import command
(still can use add Explicit Import function)

Was going to say the fix typo could be extended to be suggested on this error but the problem may be the source position is not tight enough, and that will require some further compiler work. Possible hacks aside.

Indeed, I think it would be great if quickfix <alt><enter> could offer to Add explicit import, directly with the missing keyword pre-filled

The <tab> shortcut to trigger autocompletion when you want to import something is really practical, and the add module import is more than enough for other cases.
I don't think doing more now worth the extra complexity.
I'll close this unless you think it was a nice enhancement proposal

Reopening, think there is definitely something worth doing here, without too much effort.

ok :)

OK, seems there is already quick-fix triggered by these error codes (for simple unknown identifiers, not fancy things like bind), but needed updating for the current error code.


New method to map files to objects

New method identify_objects in zcl_abapgit_file_status to map files to objects. Needed for abapGit CI.

hmm, I'm not happy about adding methods in abapGit that are not used in abapGit. Performing where-used it will show as unused, plus it is not tested/run in normal abapGit.
Calling calculate_status will also return the same information?


Accuracy Problem

iam using accuracy option but its not working.
var options = {};
options["accuracy"] = "exactly"; it doesn't work
$(".context").unmark().mark(keywords, options);
the keyword iam searching is Service_user_text but its doesn't highlight

Hi @nouman321,
Thanks for submitting.
Please read the contribution guidelines and provide the required information named there.

Just out of curiosity, I set up this demo, but I accidentally left off the "ly" in "exactly"... it produces a javascript error.

mark.js: Searching with expression "/(?:)/gim"
mark.js:264 Uncaught TypeError: Cannot read property 'length' of undefined

I probably should have opened a new issue, but I think the "partially" case should be changed to default
switch(val) {
    case "complementary":
        return `()([^\\s${lsJoin}]*${str}[^\\s${lsJoin}]*)`;
    case "exactly":
        return `(^|\\s${lsJoin})(${str})(?=$|\\s${lsJoin})`;
    default:
        return `()(${str})`;
}

or just return that string outside of the switch block.

@Mottie Thanks for the hint. I know that errors will come up when specifying wrong options. This isn't the only case where  this happens. But, when introducing fallbacks for one case, others may complain that others don't have a fallback too. Therefore I'm not going to add any fallback to this case.
What I can imagine is offering an additional "service", that validates options. This could e.g. be an "import" dialog for the configurator that would apply the specified options from a textarea and throw an error when something went wrong. It should help finding out where's the cause of an issue. But, as you know I've more important things on my todo list (e.g. digging into the equivalents thing) where I haven't time for.
@nouman321 I'm going to close this issue as you haven't replied within 24h. Feel free to reopen if you've provided the necessary information.


Update readme with a description on how to build

Add instructions on where to get SBT
Add instructions on how to set up lolca.properties to point to SDK

Installing sbt from Debian repo worked for me, but when I run "sbt" in my local nordpol repo it just hangs, as described here: stackoverflow.com/questions/29599309/stuck-at-getting-org-scala-sbt-sbt-0-13-6-when-running-sbt-in-terminal
I will let it run the prescribed 5-10 minutes and see what happens. Anyway, this issue should be addressed in the readme...

@petter-fidesmo Eventually sbt downloaded from this repo: https://repo.typesafe.com/typesafe/ivy-releases/org.scala-sbt/sbt/0.13.8/jars/sbt.jar

@adriaan-telcred Must be some network issue. Are you behind a proxy or similar? I have had the problems with SBT hanging when I'm on 3g connection sometimes. I'll see if there are some resource that can be linked to the description that handles this problem.

@petter-fidesmo As far as I can tell from the discussions I linked, it has to do with the default repo not being resolved, and after a timeout (5-10min) the fallback repo is used successfully.
We are on 3g but I never have problems downloading from maven central, etc.

@adriaan-telcred Oki, I also googled around a bit and there seems to be some generic problems with ivy hanging on getting dependencies when there are small network issues (like going through an HTTP proxy). I'll leave a note regarding this in the README. Thanks for the feedback!


chore: bdd-nh-1584420915 to 0.0.1

chore: Promote bdd-nh-1584420915 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Lex Iustificati Enchantment Oddities

Text on item now shows a banish daedra effect. In reality, it looks like its effects are this plus the effect of Dawnbreaker and Call to Arms.
The sword can be disenchanted. Doing so learns the Dawnbreaker effect only.
When the sword fires off Call to Arms, it increases the player's illusion skill. This happens even if the sword is being used by the player's follower and not the player themselves.

Thank you for finding this. Some of the enchants are kind of whack. I have been fixing them as I found them. I will take a look at this.

This is fixed in https://github.com/schofida/CWORR/releases/tag/v1.0.10a It now displays what it actually does. And disenchanting should yield the same enchantment

Okay, got the new version. Its description reads:
When, you defeat an enemy, there is a chance of boosting your allie's Weapon Skills by 25 for 10 minutes.
To improve grammar and spelling, I'd recommend:
When you defeat an enemy, there is a chance of boosting your allies' weapon skills by 25 for 10 minutes.

If it boosted just one ally, it would be ally's. I'm pretty sure it's any nearby allies, though. Hence allies'.

But allies is already plural?

S’ is possessive isn’t it

Yes, allies is plural but not possessive. It's "allies' weapon skills." If we want to avoid the apostrophe, we'd have to rephrase it as "the weapon skills of your allies" instead.

Your allies wouldn’t need an apostrophe

The actual object is skills. It's the skills of the allies.

Why must English have a broken language system

I honestly am confused myself

If we were to use friend instead, it may be more clear for you.
friend: singular, not possessive
friends: plural, not possessivve
friend's: singular, possessive (for example, "your friend's bag" would refer to a bag belonging to one friend)
friends': plural, possessive (for example, "your friends' bag" would refer to a bag that is shared between multiple friends)

I say this just as a grammar example, of course. In Skyrim, friends and allies are two different entities.

I thought it was the other way around ‘S for shard S’ for singular

"...your allies's weapon skills..." would also be valid. Since allies ends in S, though, we also have the option of saying "...your allies' weapon skills..." instead.
The only exception that comes to mind offhand of a word that can be made possessive without an apostrophe is "its" to distinguish it from the contraction "it is."

It sounds like rephrasing it to say "the weapon skills of your allies" is our best bet.

Yes

English: It just works

As for the rules for the case in point, though, ally is singular and allies is plural. So if we do make it possessive, we have to keep the original word intact. So ally's for singular possessive and your choice of allies' or allies's for plural possessive.

I thought if it was plural the apostrophe wasn’t required as the word already suggests more than one

The apostrophe would be to make it possessive, not to make it plural.
Ally = singular (no apostrophe)
Allies = plural (no apostrophe)

@LABWAZHERE @Eve-Knight I woke up this morning to 100+ notifications and I almost had a heart attack.


Error with S3 backend: "Unknown options: --recursive"

We're running the Elastic CI stack v5.16.1 and trying to use the S3 storage option. At the end of our build, during the cache plugin's post-command hook, we're getting this error:
Running plugin cache post-command hook | 0s
-- | --
  | $ /var/lib/buildkite-agent/plugins/github-com-buildkite-plugins-cache-buildkite-plugin-v0-3-0/hooks/post-command
  | Saving file-level cache of /bundle
  |
  | Unknown options: --recursive
  | 🚨 Error: The plugin cache post-command hook exited with status 255

It looks like the flag is being passed in here: https://github.com/buildkite-plugins/cache-buildkite-plugin/blob/master/backends/cache_s3#L22-L26
Are we doing something wrong?

Nope... completely our fault. I originally implemented it using cp instead of sync and forgot to remove that flag afterwards :(


Fixed typo in Trino JMX documentation

@nineinchnick I figured out why the tests are failing. The Chart in the tests references the latest JMX exporter docker image which is broken.

Let's pin it to the last working version, can you add it as a separate commit in this PR?

Let's pin it to the last working version, can you add it as a separate commit in this PR?

sure thing!

@nineinchnick tests are working again now!


Dependencies added into the source code

First, thank you for the scheduler, it helps a lot when managing gpus in k8s.
The dependencies are currently added directly in the code and versionned. They should be downloaded at compile time. This would simplify analysis of the code.

Thank you for your suggestions. Currently we are using dep to manage the version of dependencies, and all the dependency code are in vendor directory. But it's really difficult to download at compile time because some users have to compile it without internet access.
And I also noticed that most golang projects are using the vendor, you can learn from kubernetes

Thank you for your answer, and sorry for the loss of time. It seems I do
not know enough about go, I will do some research, and close this issue.
Le jeu. 4 avr. 2019 à 05:01, cheyang<EMAIL_ADDRESS>a écrit :

Thank you for your suggestions. Currently we are using dep
https://github.com/golang/dep to manage the version of dependencies,
and all the dependency code are in vendor directory. But it's really
difficult to download at compile time because some users have to compile it
without internet access.
And I also noticed that most golang projects are using the vendor, you
can learn from kubernetes
https://github.com/kubernetes/kubernetes/tree/master/vendor
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/AliyunContainerService/gpushare-scheduler-extender/issues/21#issuecomment-479731595,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEXinjGRHi3rKYYkRSBwPYMC34u5mgklks5vdWrvgaJpZM4cbCL6
.


Updates to IoT Plug and Play Documentation for IoT Plug and Play bridge

IoT Plug and Play bridge is a feature of IoT Plug and Play. The purpose of this PR is to migrate the documentation from the GitHub (www.aka.ms/iotplugandplaybridge).

Microsoft authors of articles need to make updates through the private repository so the updates can be staged and validated by the current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let the PR reviewers know the updates are ready to be merged.It's easy to access the private repo version of your articles in GitHub. Just add -pr to the URL, as follows:
Public repo URL: https://github.com/Microsoft/azure-docs/blob/master/articles/azure-portal-overview.md
Private repo URL: https://github.com/Microsoft/azure-docs-pr/blob/master/articles/azure-portal-overview.md
We have closed this pull request. Please resubmit the updates through the private repo. If you need perms to the private repo, see the contributor guide instructions


API For RT

Hi
Is there an API which allows you to run the rt compiler within a program (either in nodejs or in a browser); I guess in nodejs one could fork a task but as I guess the playground runs within a browser environment?
James

there is no formal API, but you can require and use the javascript modules from node_modules/react-templates. I do that in my (still unfinished) rioct-cli project, see for example rtExtractor.ts.

thanks for that and pointing me at your project; I work in typescript so it will be very useful.


[Portainer] Stopped containers are not listed and visible

Which addon?

Addon name : Portainer
Addon version : 2.11.0-3

Describe the bug
When an addon is running portainer lists the running container which is normal but when you stop an addon, portainer does not list show the stopped container. Tested with Adguard home and aircast. Is this a limitation for the addon?
To Reproduce

Stop an addon
Search the container of the addon on in the portainer container list
It is not there

Full addon log

Full addon config

System

Supervisor version: supervisor-2021.12.2
Host system version: Home Assistant OS 7.1

Hi, actually it's the way HA works : it recreates containers at each stop & restart from a single image. When I look in my own portainer I can see stopped containers (although it is not useful since they will get deleted at next reboot).

I'll close it as it seems more a local configuration and not due to the addon as such... Don't hesitate to continue the thread here I'll still read & reply to the messages


How is the SHA1 digest computed?

How is the SHA1 digest for exact deduplication obtained?
In particular, it seems that some files marked as exact duplicates are not exact duplicates, for instance 2021-25/2449/en_middle.json.gz/17916 and 2021-21/0115/en_middle.json.gz/12666.
Naively computing the SHA1 of these two documents indeed leads to two different digests, as
print(doc1['digest'])  #output: sha1:IV4DNI4TBSMLL7GKU6VUNK4TEJHLVHLT
print(doc2['digest'])  #output: sha1:IV4DNI4TBSMLL7GKU6VUNK4TEJHLVHLT
print(hashlib.sha1(doc1['raw_content'].encode('utf-8')).hexdigest())  #output: 3f4575a021f02a637ac9b04a4a26ff3b2ca89150
print(hashlib.sha1(doc2['raw_content'].encode('utf-8')).hexdigest())  #output: 14ac252de7c9674beb0cb8b3678fa204fd4661fe

Hi @RicardoDominguez , the SHA1 digests of the documents correspond to the warc-block-digest of the corresponding plaintext .wet document -- in other words, this is computed based on the plaintext extracted from the original html document. The reason why you see two different hashes is that the CCNet pipeline performs sharded paragraph level deduplication and removes paragraphs from documents that are repeated in other documents (in the same shard).
In the example you showed, the two documents are originating from the same plaintext .wet document, but some paragraphs were removed from one document, and some from the other document (as these paragraphs appeared in the respective shards). For example, the paragraph Art and Archaeology is present in 2021-21/0115/en_middle.json.gz/12666 but not in 2021-25/2449/en_middle.json.gz/17916.
Let me know if this clarifies things for you.