[WFCORE-705] Upgrading to JBoss MSC 1.2.6.Final

Core - Full Integration Build 36 is now running using a merge of 722bd79995f0b7a9e4eb8cffe4b27e3005223e71

Windows Build 36 is now running using a merge of 722bd79995f0b7a9e4eb8cffe4b27e3005223e71

Linux Build 42 is now running using a merge of 722bd79995f0b7a9e4eb8cffe4b27e3005223e71

Core - Full Integration Build 36 outcome was FAILURE using a merge of 722bd79995f0b7a9e4eb8cffe4b27e3005223e71
Summary: Exit code 1 Build time: 0:01:28
Build problems:
Process exited with code 1

Windows Build 36 outcome was SUCCESS using a merge of 722bd79995f0b7a9e4eb8cffe4b27e3005223e71
Summary: Tests passed: 3103, ignored: 66 Build time: 0:20:18

Linux Build 42 outcome was SUCCESS using a merge of 722bd79995f0b7a9e4eb8cffe4b27e3005223e71
Summary: Tests passed: 3103, ignored: 66 Build time: 0:23:17

retest this please

Core - Full Integration Build 39 is now running using a merge of 722bd79995f0b7a9e4eb8cffe4b27e3005223e71

Windows Build 39 is now running using a merge of 722bd79995f0b7a9e4eb8cffe4b27e3005223e71

Linux Build 45 is now running using a merge of 722bd79995f0b7a9e4eb8cffe4b27e3005223e71

Core - Full Integration Build 39 outcome was FAILURE using a merge of 722bd79995f0b7a9e4eb8cffe4b27e3005223e71
Summary: Exit code 1 Build time: 0:01:18
Build problems:
Process exited with code 1

Windows Build 39 outcome was FAILURE using a merge of 722bd79995f0b7a9e4eb8cffe4b27e3005223e71
Summary: Tests failed: 7 (6 new), passed: 3097, ignored: 66 Build time: 0:21:15
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.manualmode.management.cli.CustomVaultInCLITestCase.testWrongVaultPassword: <no details avaliable>

org.jboss.as.test.manualmode.management.cli.CustomVaultInCLITestCase.testNonExistingVaultPassword: <no details avaliable>

org.jboss.as.test.manualmode.management.cli.CustomVaultInCLITestCase.testRightVaultPassword: <no details avaliable>

org.jboss.as.test.manualmode.vault.VaultPasswordsInCLITestCase.testWrongVaultPassword: <no details avaliable>

org.jboss.as.test.manualmode.vault.VaultPasswordsInCLITestCase.testRightVaultPassword: <no details avaliable>

org.wildfly.core.test.standalone.mgmt.HTTPSConnectionWithCLITestCase.testTrustedCLICertificate: <no details avaliable>

org.jboss.as.test.manualmode.vault.CustomVaultInModuleTestCase: <no details avaliable>

Linux Build 45 outcome was FAILURE using a merge of 722bd79995f0b7a9e4eb8cffe4b27e3005223e71
Summary: Execution timeout (new); tests passed: 2928, ignored: 51; org.wildfly.core:wildfly-core-testsuite-domain Build time: 0:45:10
Build problems:
Execution timeout
Process exited with code 137

retest this please

Core - Full Integration Build 43 is now running using a merge of 722bd79995f0b7a9e4eb8cffe4b27e3005223e71

Windows Build 42 is now running using a merge of 722bd79995f0b7a9e4eb8cffe4b27e3005223e71

Linux Build 49 is now running using a merge of 722bd79995f0b7a9e4eb8cffe4b27e3005223e71

Windows Build 42 outcome was SUCCESS using a merge of 722bd79995f0b7a9e4eb8cffe4b27e3005223e71
Summary: Tests passed: 3104, ignored: 66 Build time: 0:21:57

Linux Build 49 outcome was SUCCESS using a merge of 722bd79995f0b7a9e4eb8cffe4b27e3005223e71
Summary: Tests passed: 3104, ignored: 66 Build time: 0:26:19

Core - Full Integration Build 43 outcome was FAILURE using a merge of 722bd79995f0b7a9e4eb8cffe4b27e3005223e71
Summary: Tests failed: 1 (1 new), passed: 2866, ignored: 373, muted: 1 Build time: 0:56:52
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.messaging.mgmt.JMSQueueManagementTestCase.removeJMSQueueRemovesAllMessages: <no details avaliable>


allow entries globs for multipage apps

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

linter issue:
/home/runner/work/aio-lib-web/aio-lib-web/src/bundle.js
Error:   15:7  error  'fs' is assigned a value but never used  no-unused-vars


adServerDomain setup in creatives appears to be useless

In at least some of our docs we ask publishers to set up PUC-GAM creatives with adServerDomain:
<script>
  var ucTagData = {};
  ucTagData.adServerDomain = "";

PUC includes it as part of the message payload for Prebid, but the latter does not use it: https://github.com/prebid/prebid-universal-creative/blob/68d81fc8eba26c903149b9ce5592ea832fc003a2/src/renderingManager.js#L144-L151
From my testing, including or removing it makes no difference. I believe it should be removed together with the code around it, and the docs updated.

I think it used to be used on the Prebid.js side of things to help handle the postMessage request.  But it seems that a while ago the logic was changed on the Prebid.js side through this PR:
https://github.com/prebid/Prebid.js/pull/5285
It's likely it was just not cleaned up on the PUC side, and now (as you inferred) doesn't do much and can probably be removed.

Thanks for the detective work! That PR was merged in 3.22; is that old enough to be OK to break with a PUC update?

I'm not sure what you mean by break in this instance?  Do you mean people that would still be using latest PUC (as that's how the creatives are setup) while also using a version of 3.x Prebid.js; as that would stop supplying the adserverDomain from updated PUC to that old version of Prebid.js?

Yes - with how we distribute PUC, we are implicitly promising unlimited backwards compatibility.

I think we need to review this with the larger group.

Agreed. Brainstorming ideas on how to escape this trap, we could try to tie breaking change to Prebid major versions; the transition will take forever, but we could for example ask pubs to set up
<script src = "https://cdn.jsdelivr.net/npm/prebid-universal-creative@%%PATTERN:hb_pb_majorversion%%/dist/creative.js" />

and start a new PUC branch (and release tag) for each new Prebid major version.


bots: Encode URLs from user input

See #5095 for more detail.
Basically sanitize data when processing user inputs.

@zulipbot label "help wanted" "bite size" "area: bots"

moving to new repo:
https://github.com/zulip/python-zulip-api/issues/6


Syntax error in file goes undetected

Running phantomjs on a file with a syntax error in it will leave phantomjs running without doing or outputting anything. I would expect it to raise a syntax error and then to terminate.
Example script:
var page = require('webpage').create()))

Duplicate. Please see #12973

I actually don't think that it's the same. That was inside a callback and the throwing of an error. This is just a syntax error in the main file.
console.log())) // This bug

throw new Error('x') // Works as expected

page.open('...', function () {
  throw new Error('x') // Bug #12973
})

Btw, did you ever post your solution somewhere? I would really really like to have it :+1:


Embassy examples not working

I'm trying to use embassy with esp-hal, however, the examples do not compile anymore since embassy_executor::Executor does not exist anymore.
It would be nice for examples to be compiled during CI tests to check for these kinds of issues.
Edit: It appears examples do run during CI tests, will investigate more about what might be wrong.

I've missed that https://github.com/esp-rs/esp-hal/blob/main/esp32c3-hal/Cargo.toml#L41 were dev-dependencies and were added when checking tests. Adding these features to my project fixed the issue!


CPU leak

I am observing rapidly increasing response times around a no-op route when using CLS.  See http://stackoverflow.com/questions/42345624/addition-of-continuation-local-storage-causing-cpu-leak and https://github.com/myles-mcdonnell/cls-express-cpu-leak.
Must be something I'm doing wrong in my usage of CLS but can't figure out what.

ok, got it..only one namespace require, not one per request...now using cls-middleware.


Misleading plugin settings if section’s selected author gets deleted

If a user gets deleted who was previously selected as the default author within the Guest Entries plugin settings, when you revisit the plugin settings, the section will continue to be enabled, and it will look like a default author is selected – but it won’t be the (now deleted) previously-selected user. And it won’t be clear that there is an issue.

Fixed in Guest Entries 2.3.0.


toptal.com broke your script!

The js minimizer, for example, is now here: https://www.toptal.com/developers/javascript-minifier/
But posting to .../raw doesn't work.  They have a "raw" button that seemed to work.  Haven't deciphered it yet...
Thanks,
--Dan

Hi @jodosha, I've just updated to the new URL.
Thank you!


Add thumbnail endpoints

This adds thumbnail model and endpoints, fixes a couple bugs with MongoDB collection names, and includes a beginning stub for dev-friendly docs about it.

Checked this one out, and so far here's what I see.
All the tests pass. I can also add a thumbnail to a dataset or save a thumbnail through the API routes. I don't see the preview on the front end, just a space where the thumbnail should be.
If that's going to be a separate issue, then I can mark this one approved.


Add AJP protocol (and test notification on dev list)

Test Notification with a simple PR

Test comment notification now.

A new comment. Notification on dev list for comment seems don't works.

Let's see if this triggers an email to the dev list.

Close PR. Notifications works on dev list now.


Add copy method to list object

Wait, list already has a copy method. You added it to set here, added tests for a list and test it in enumerate tests file.

@adrian17 You are right. Sorry for this mess. I move test to set.py

Codecov Report

Merging #473 into master will decrease coverage by 0.47%.
The diff coverage is 36.36%.

@@            Coverage Diff             @@
##           master     #473      +/-   ##
==========================================
- Coverage   58.55%   58.07%   -0.48%
==========================================
  Files          67       67
  Lines       12080    12161      +81
  Branches     2787     2821      +34
==========================================
- Hits         7073     7063      -10
- Misses       3233     3321      +88
- Partials     1774     1777       +3

Impacted Files
Coverage Δ

vm/src/obj/objset.rs
52.89% <36.36%> (-17.68%)
:arrow_down:

Continue to review full report at Codecov.

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


Status page

I would like to suggest a /status endpoint and move the info from / to it ?

Done as purposed: 2011bdea6e446e7b3eb55ca0d868ed808d86a852


Fix error handling when subscribing to unknown publication

Summary of the changes / Why this improves CrateDB
An exception must be raised instead of treating the creation as successful and create a non-working subscription.
closes https://github.com/crate/crate/issues/12489
Checklist

[x] Added an entry in CHANGES.txt for user facing changes
[x] Updated documentation & sql_features table for user facing changes
[x] Touched code is covered by tests
[x] CLA is signed
[x] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

@jayeff

These subscriptions started working automatically once you create the publication on the publisher.

Ah thanks for point this out!
I've pushed a fixup with a modified CHANGES entry.

I still think this new behaviour with raising an error is better as a small typo in publication name can lead to confusing non-working subscription. And use case create subscription before creating the publication is probably very untypical 😅

Yes as long as a subscription is enabled by default, I think throwing an error is the better and expected behaviour.
Once we support to disable a subscription on creation, we could change that back based on the disable flag.
This change is a bit of a grey area between being a fix or change. Anyhow I'd prefer to call it a fix and push it out with the next hotfix release. @jordi wdyt?


Allow for private datasets

It should be possible to have private, hidden datasets.
These should not show up in the Clearinghouse UI, but probably also not in the API unless requested by an authorised user. All data related to a private dataset needs to be hidden.
As nearly all requests are dataset scoped this should be feasable without a major refactoring.

@thomasstjerne do you have concerns/input from the UI perspective on this, especially if the API would behave differently depending on the logged in user? E.g. dataset searches would return different results?

I guess this could be transparent for the client.
The UI already do authorised requests to all endpoints, so I guess only the amount of data returned would differ.
But a flag private: true would be useful for the UI to show it is private with a small icon or some  coloring.


Use indexes to lookup related resources more efficiently

I notice that trivy-operator has quite a few internal use cases for looking up related resources, where owners/dependents as the most common one. I suggest to use controller-runtime indexes to make these lookups more efficient and precise. That should also some existing TODOs in the code base. Example: https://github.com/aquasecurity/trivy-operator/blob/6916d7b1476cc50638a7d3f66822cfa96ad92b4e/pkg/vulnerabilityreport/controller.go#L214
An example for how this might look like code-wise: https://book.kubebuilder.io/cronjob-tutorial/controller-implementation.html#setup

This suggestion makes a lot of sense. I can help review if needed. Thanks for looking into this!


Fix tab completion for unlocalized about* topics

Fixes tab completion for help topics on systems without localized help by adding en-US as a fallback.
Get-Help About<Tab>
PR Context
Get-Help will automatically fall back to en-US help topics so the completion for it should do the same.
PR Checklist

[X] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[X] Summarized changes
[X] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[X] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[X] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[X] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[X] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[X] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

@MartinGC94 Thank you for your contribution!


[Event Hubs] Change trigger checkpointing behavior

Summary
The focus of these changes is to change the behavior of the Event Hubs trigger to not checkpoint when the listener is shutting down. This is necessary to prevent potential data loss from occurring when shutting down Function retries.  Because the trigger cannot know if the Function host would have retried a failure if it were not shutting down, we cannot assume that it is safe to checkpoint.  This change ensures that the batch of events being processed when shut down was requested will be retried by another instance or the next time the Function app is run.  Though this may cause duplicate processing, it is consistent with the Event Hubs at-least-once guarantee.

/azp run net - eventhub - functions - tests

API change check
API changes are not detected in this pull request.


Update docs for kubelet credentials provider GA

Signed-off-by: Narang Dixita Sohanlal<EMAIL_ADDRESS>Adding doc changes for kep-2133 for KubeletCredentialProviders GA.
https://github.com/kubernetes/enhancements/issues/2133

/lgtm

/lgtm cancel
These changes are useful, but they are not the whole set we need.
/retitle Update docs for kubelet credentials provider GA
/sig node

@sftim I have made the recommended changes PTAL. What other changes are needed to make this as complete?

I noticed another thing. The file content/en/docs/tasks/kubelet-credential-provider/kubelet-credential-provider.md should really be content/en/docs/tasks/kubelet-credential-provider.md and we should configure a redirect (see static/_redirects) for the old URL.

@sftim I don't see any readmes directly in tasks folder. Everything has a dedicated folder. Is my understanding correct?

I don't see any readmes directly in tasks folder. Everything has a dedicated folder. Is my understanding correct?

Ah, good point. How about putting this task inside https://kubernetes.io/docs/tasks/administer-cluster/?

@sftim @andrewsykim do we also need content/en/docs/reference/config-api/kubelet-credentialprovider.v1beta1.md for v1?
There are references to v1alpha1 config in here
https://kubernetes.io/docs/tasks/kubelet-credential-provider/kubelet-credential-provider/#what-s-next

@ndixita I think we should update all references to v1 only

@ndixita  is this PR https://github.com/kubernetes/website/pull/38006 valid for this release 1.26?

We often use tabs to document that an old API is also still supported (showing the stable API by default). Is the v1 API different from v1beta1 for kubelet image pull credential plugins?

@ndixita Please could you review the suggestions changes? We need to close this PR before the 1.26 release.
cc: @leonardpahlke @reylejano

/label tide/merge-method-squash

the technical content is lgtm for me for sig-node.
/lgtm

/lgtm

@tengqm, are you happy with how this previews now?

@ndixita: when you rebase this, would you be willing to squash it down to 1 or 2 commits as well?

/lgtm

/lgtm
/approve
Thanks.


Create .codeclimate.yml

Abby from Code Climate here! Refer to my email for more info but this:

Adds Go engines
Adds ratings

Thanks!

@abbyarmada Thank you!


[feature request] URL to latest release of repo

For stability and to avoid rebuilding images with each commit, I usually make Binder URLs that point to the latest release/tag of my repo, rather than master. The trouble is, I have to keep updating that URL. It would be great if there was a single URL pattern that always matches "latest release".
If the dates of tags are not available for some reason (for determining which one is "latest"), then perhaps the last in semantic version order? My release/tag names are all unadorned semantic versions (i.e. no project name, not even a "v" before the three numbers, separated by dots). A particularly slick implementation would support semantic version inequalities, like ">=1.2.3" to mean "1.2.3 or later, but not prereleases like 1.3.0rc1".
This would benefit a common use-case of Binder for demonstrating software, where you want a permanent link to the latest stable version. Documentation usually has this, with a "latest" or "stable" tag in the URL pattern to go to the latest version of the documentation—it would be great if Binder had the same thing.

You can get the list of tags in reverse order of creation from the GitHub API e.g. https://api.github.com/repos/IDR/idr-notebooks/tags so the information is available.
Alternatively you could solve the problem now by creating your own GitHub branch, e.g. latest, and keeping that in sync with the latest tag. This avoids problems where you're maintaining multiple incompatible releases (e.g. 1.x, 2.x) so most recently created tag might not be the the latest version, or dealing with a myriad of versioning systems.

Alternatively you could solve the problem now by creating your own GitHub branch, e.g. latest, and keeping that in sync with the latest tag.

Oh, that's right. Thanks for the suggestion! (I'll think about how to automate a latest branch in CI...)

Several people also use a latest tag that they move as they make new releases/updates and use that in the mybinder.org link.
If you find a way that works for you it might be worth adding this as a FAQ entry for people from the future.

Don't you have to delete and recreate a tag to move it? I would have thought that branches are easier to maintain for that reason. (And "easy" only if they're going forward—I'm not sure what surgery is required if latest needs to roll back changes.)
Anyway, this is getting more into git than Binder. It would be nice to see this as part of a "best practices" suggestion (or maybe I missed it). One thing that could still be a useful Binder feature would be selecting based on a semantic version rule (e.g. ">=1.2.3") but this is a casual wish, since I could go and try to implement the latest branch myself for the most common functionality.

Hi @jpivarski, how did you end up solving this?

I didn't. Whenever I use Binder, I manually set the URL tag to the latest tag, even if that means swimming it to a year that doesn't exist yet (in order for the tagged release to contain eventually-correct URLs).

Hi :wave: This is an old issue which doesn't sound like a BinderHub bug so I'm closing it. If you have more questions please use the Jupyter community forum https://discourse.jupyter.org/.


Add missing double-precision flag for Vector4 & Projection in encode_variant

Applies the same double-precision fixes found in #58205 to Vector4 & Projection, which were added after this fix was initially implemented

Thanks!


Create an "oracle" for testing pose control

In order to create a good test for pose control, I would like to create a node that asks the pose controller to move to a location, and then uses the gazebo position data to verify if it made it there in simulation.

Finished in the pose control branch, can be closed once pose control merges into master


Allow to set timeout in UniCreate.future

In some cases it is impossible to control passed future object and the executor thread may be blocked forever, so i think that timeout can be useful to avoid it and at least fail with TimeoutException 🙂

I agree; it could be useful.

@jponge, @cescoffier thank you for review! I've added fixes according to suggestions

@nryanov Ah we have a rule that commits need to be signed (see https://docs.github.com/articles/about-gpg/). Would you mind redoing your commits with a signature?

@jponge done (squashed into a single signed commit) 👍


move larch into package

@Raynos @jcorbin
Larch is in its own repo and published. This removes larch from Hyperbahn and uses Larch out of its package.

lgtm.


Transcriptions still wrapping

Not really a good way to do this without using white-space: pre, which isn't an appropriate solution.


refactor uuid function into more readable form

According to the following statement from the typescript migration pull request

I think a future PR should refactor the uuid function inside uuid.ts. Code golf isn’t very readable, and it isn’t friendly with typings or linters either.

this could be a first approach to create a more readable form of the function.
Implementation is realized by creating an array of the 5 groups of a UUID, using their specified individual lengths (8-4-4-4-12) and setting the two particular digits to their specified values (or range, respectively).
The array creation of each group could be realized more compact using Array.from, but that would make the transpiler angry, as it's an ES2015 feature and not available atm.

Besides that, the linter warns about using any in the groups array's map functor. Any info about how to handle that better would be appreciated (I was somehow hesitant to declare an Interface or sth like that just for that single use).

so the reason I left this function as-is:

this function was not my work.
I gave credit to the author and pointed to the git Gist
reading the gist linked, in lieu of refactoring the function, is superior in my opinion.  since I (and likely many others) do not know the finer points of the v4 uuid specification.

Given that explanation I'd in no way be offended would this pr be closed. I did not want to create unnecessary noise.

no problem at all, I appreciate you contributing!


Add ability to support branch coverage statistics ruby >= 2.5

Show statistics of branch coverage result.

You can put this merge on pending until they merge simplecov gem branch that generates needed statistics. Anyway, these modifications are not harmful.

Ugh all the CI failing is my/our fault... need to fix that up. Sorry.

@PragTob I updated the branch, hope it will fix CI, else I will try to have a look this evening!

Ugh all the CI failing is my/our fault... need to fix that up. Sorry.
.travis.yml updated according to the release https://github.com/colszowka/simplecov/releases/tag/v0.17.0

Ah wanted to fix that up on a separate branch, small changes, fix it for everyone and not tie it to this branch and such... but thank you still, I'll do further fixing after this landed which will hopefully be soon :D

🎉


Network Policies are actually don't block anything

Hi Team,
As a customer I wan't to have an option to restrict access to Gdata-VaaS service inside Kubernetes cluster.
Unfortunately I have noticed, that with current helm chart implementation network policies does not block anything. Ingress rules does not have From: field, and Egress rules on the other hand does not have To: fields.
I have tried to add custom networkPolicy outside of the Helm Chart, but there is no way how I can differentiate gateway and gdscan pods, because they have completely the same labels (usually app.kubernetes.io/instance is using for this purpose).
Could you maybe fix it? Good example how it could be done is actually in the Redis helm chart here.
Thank you in advance and best regards.

Hi behoof4mind,
thank you for your feedback. We will look into

Explicit from and to Ingress/Egress rules
Allow selecting pods in custom networkPolicies
Example for custom networkPolicy

A solution will be ready at the end of the week.
Cheers!

Updated in chart version 2.1.0:

app.kubernetes.io/name is now different for gateway and gdscan pods
default network policy is more restrictive where possible


Ordered entities

In some cases, it could be useful to store and display data in timed order. Like messages in the chat.
I tried to implement the receiving of new messages via custom redux action, but we know only ids of entities in updaters, so it is not possible to identify where to store entity.
export const conversationUpdated = (conversationUpdate: ConversationByOperator) => ({
  type: RECEIVE_MUTATE_TYPE,
  payload: conversationUpdate,
  meta: {
    schema: ConversationByOperatorResource.asSchema(),
    updaters: {
      [ConversationByOperatorResource.listShape().getFetchKey({})]: (
        newConvId: string,
        prevResults: { conversations: string[] | undefined },
      ) => ({
        conversations: [...(prevResults?.conversations ?? []), newConvId],
      }),
    },
    date: new Date(),
    expiresAt:<PHONE_NUMBER>12939889,
  },
});

Do you know how this can be done except manually sorting everything in react components?

Thanks for the report!
Let me see if I understand your scenario correctly: Is there a member of the conversation that has it's date, that you want to use to determine ordering rather than the order in which it was processed?

@ntucker Exactly! For example, we need to display a list of messages within conversation and receive new message via websocket. But it is possible for some case to get an older message first and then get a new one. If you just append ( or prepend ) ordering could be wrong.
I will be glad to help you with this feature if you do not have time if you give me some guidance.

Ah interesting, ya we did start seeing use cases like this. We'll probably need to make some sort of backwards incompatible change to support this - though we're in 5.0 beta development now so that's not a huge deal.
I guess the most obvious thing to do would be to actually pass in what the denormalized response would be (something resembling what is returned from useResource or useCache) and then you return a new denormalized response as well. Then the return value would be normalized again to just grab the results to update. This would technically be a bit more computation, which is why I didn't initially do it, but I think it is less confusing and allows cases like this, so probably makes sense. Computation on a write like this is probably fine considering there will likely be more responses updating as a result.
One open question would be what if people alter the entity themeslves in the updater function? Should this be allowed? It probably has to. I guess that is fine?

Now that I think a little more, this works well with my plans to create a universal memoization of denormalized forms as those will end up simply being lookups into the table.

RFC to fix this and others in https://github.com/coinbase/rest-hooks/issues/760


chore: Delete .DS_Store file

What type of PR is this?

Check the PR title.

[x] This PR title match the format: <type>(optional scope): <description>
[ ] The description of this PR title is user-oriented and clear enough for others to understand.
[ ] Attach the PR updating the user documentation if the current PR requires user awareness at the usage level. User docs repo

(Optional) Translate the PR title into Chinese.
(Optional) More detailed description for this PR(en: English/zh: Chinese).

en:
zh(optional):
(Optional) Which issue(s) this PR fixes:

(optional) The PR that updates user documentation:

why??

why??

not something big, just the convention to ignore the MacOS's .DS_Store.
feel free to close it 😂

o

why??

not something big, just the convention to ignore the MacOS's .DS_Store. feel free to close it 😂

oh, I misunderstood. It's an useful PR. Thx! @haoxins


What's the correct way to use this package?

It looks like

tmp = Template("./my_template.html")
init = Dict("name" => "blah")
return html(tmp(init))

doesn't work anymore (will not find the function)
I have to now do

tmp = Template("./my_template.html")
init = Dict("name" => "blah")
return html(tmp(; tmp_init=init))

There was a lack of explanation. As I create template code, you have to use not init but jl_init or tmp_init.

You still have some inline example code pointing to the first way. That's fine. I'll mark this not as a bug.


Made changes to adding item to shopping list

Adjusted the approach when adding a new item. Now an array is presented and refined as the search is filled in. At any point the user can use the "tick" to add the desired item. "Clearing" will re-enable the search.


bash: 5.2p26 -> 5.2p32; add upstream fix for pop_var_context error

Description of changes
Tested with the first‐built Bash in stdenv; it applies, builds, and runs. I haven’t tried injecting it into derivations to check that the error doesn’t appear any more because I don’t really know what the best way to do that without rebuilding all of stdenv is and I’m not entirely sure how to arrange the circumstances that cause the error.
Closes: #214822
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

It might be a good idea to backport just the https://github.com/NixOS/nixpkgs/issues/214822 fix back to 24.05 since it confuses people so often.

Tested this on an intentionally broken package, and the error is in fact gone (and everything else works normally). Going to try building up to something more interesting now, like Mesa.

Built up to Mesa on x86_64-linux, I think we can be reasonably confident this doesn't break bash too hard.


Running in a container - issue with display updating / blitting / flicker

So nice with harlequin running anywhere! I wanted to run it in a container. I tried building a container image with harlequin (also in a web-enabled variant running under tty2web) and running it I noted some issues when running the query "select 1;".
Sometimes results appear but often not. Not sure what causes this... occasionally it works. Could it be some kind of textual rendering issue (it looks like there is a quick flicker but the display is not properly updated)?
To replicate (use either "docker" or "podman"):
docker run --rm -it ghcr.io/mskyttner/harlequin harlequin
# then enter query "select 1" and press either "run query button" or press ctrl + j

docker run --rm -it -p "80:1294" ghcr.io/mskyttner/harlequin
# then open browser at http://localhost

Screenshots:
Running in container shell often the result of running the query doesn't appear:

Running in browser (under tty2web) shows same behavior:

What terminal and shell is this? I think of you SSH into the container with a modern terminal this should work fine; last I tried the default Docker terminal isn't great and I think launches sh

See also: https://harlequin.sh/docs/troubleshooting

The web tty is provided via xterm.ts, resizing and such seem fine. I could use some other shell in the non-web container if that would alleviate the issues. Any recommendations? I tested the non-web variant of the harlequin container using terminator and also whatever the default terminal app is in Linux Mint.

Strangely it often seems to work better if I use another query than "select 1;"

oops sorry, I by mistake attempted to merge a branch from my fork to the wrong repo by accident... please remove or ignore that one...
It looks that setting the COLORTERM  to truecolor makes a difference for the better though.
podman run -p "1294:1294" --rm -it harlequin
# and open browser at http://localhost:1294

Gives:

@mskyttner I noticed you like ending your queries with ; ... I actually just fixed this in my branch, but if your cursor is right after the semicolon, the query won't execute (since we think you're trying to execute the query where the cursor is, which in this case is just blank). Could that be the cause?

This seems to be the case! Knowing about this now, it works quite smoothly also in the containerized web-enabled variant... just a few keybinding collisions (Ctrl+Enter doesn't come through, but Ctrl+J works) so mostly both mouse and shortcuts works well even in the browser. Now I just have to figure out how to configure a nice webfont. Even when trying to force a custom webfont in tty2web I seem to be getting this Courier like font. I wonder if harlequin/textual has its own css that overrides this setting...
Thanks for the help!


Multiple ISS - Not Pulling Data from Primary

The problem
Hi
I have 2 devices linked to my weatherlink account a vantage vue on ID5 and a Pro 2 on ID8. I have switched my primary device to my Pro 2 ID8 on weatherlink, have deleted my HA integration and re-added with API 2 info but it is still pulling data from the Vantage Vue not the new Pro2 ISS.
How can I force it to use the Pro 2 ID 8?
Thanks
Alex

Version of Weatherlink integration having the issue?
2024.1.1
Version of Home Assistant Core having the issue?
2024.5.3
Anything in the logs that might be useful for us?
No response
Diagnostic data file
No response
Additional information
No response

The information of which unit is assigned ar pimary unit is not available in the API AFAIC. Therefore the integration always uses the ISS unit with the lowest ID as primary. In your case ID5. There is currently no support in the integration to select which (if more than one) to use as primary. The workaround for you is to set your VP2 to ID1-ID4. I will put the issue on my to-do list.

The information of which unit is assigned ar pimary unit is not available in the API AFAIC. Therefore the integration always uses the ISS unit with the lowest ID as primary. In your case ID5. There is currently no support in the integration to select which (if more than one) to use as primary. The workaround for you is to set your VP2 to ID1-ID4. I will put the issue on my to-do list.

Ok, thank you


Default to number of usable CPUs if possible

multiprocessing.cpu_count() returns the number of CPUs, usable or not. While in most situations this is good enough it fails when not all CPUs are usable. An example using docker:

Start a container: docker run -it --rm --cpuset-cpus="0" ros:melodic-perception-bionic python3
Run:

import multiprocessing
multiprocessing.cpu_count()

It will return the number of CPUs of the whole system instead of 1, the number of CPUs that are available.
Consequently, when using catkin_make it tries to use more CPUs than the ones available and the build ends up failing.
Expected behavior is:
import os
len(os.sched_getaffinity(0))

Returns 1.

Thanks for the contribution!


feat(player): send a beforesourceset event before a new source is set

Description
Adding the beforesourceset event, allows to perform operations before the new source is loaded.
Changes made

override the src function in the video.js Player class to trigger a beforesourceset event
add test cases for src and loadMedia functions

I'm closing this PR for the time being, even though this event may address certain use cases. It can be reopened if necessary.


Added Jackson support

[x] Currency serialization
[x] Currency deserialization
[x] Currency pair serialization
[x] Currency pair deserialization
[x] Currency pair deserialization (custom separator)
[x] Currency pair deserialization (without separator)

Moved some of the tasks into a separate issue: #3


Idea: For the raspberry pi image a function for GPIO so we can turn on and off the printer from all over the world.

For the raspberry pi image a function for GPIO so we can turn on and off the printer from all over the world.

+1, I hate leaving my high wattage power supply on with my printer not doing anything

That is already possible using scripts called from menu or as host command to be inserted in the stop script. Please see our forum for workign examples.


xxd: make -R configurable by environment

It's a bit of a fine line whether this is a feature request or a bug report.
xxd had recently introduced colorized output which appears to be defaulting to 'auto', presumably decided by whether or not the output is a tty.
May I please ask for a way to set -R by an environment variable? If there is some sort of an established (semi-)standard or convention for this sort of thing (which there does not seem to be, AFAICT), following that would be nice. If not, whatever sensible environment variable works.
The point here is to be able to disable color output "globally" in one fell swoop without having to resort to aliases or wrappers or functions or whatnot.

Indeed it does, thank you very much.
Dropping a note in the manual would be nice though :).

I have done so now. Thanks.


Logging out Automatically

Hi Vassim,
I have replaced the assets file as you mentioned in earlier issues however, I am not able to download the videos, as it logging out automatically. please refer the following screen shot.

HI Vassim,
Issue is resolved as my subscription in expired and will be able to download after resubsciption

Good to hear you solved the issue. Btw I was the one who fixed the assets.py file, in case there are other issues ;-)

I m again facing issue
INFO     Logging in ...
INFO     Analyzing ...
INFO     Downloading files ...
INFO     Logged Out!
ERROR    Unknown Exception
ERROR    list index out of range
Traceback (most recent call last):
File "build/bdist.linux-x86_64/egg/linuxacademy_dl/cli.py", line 206, in main
la.download()
File "build/bdist.linux-x86_64/egg/linuxacademy_dl/linux_academy.py", line 155, in download
self._course.download()
File "build/bdist.linux-x86_64/egg/linuxacademy_dl/course.py", line 102, in download
asset.download()
File "build/bdist.linux-x86_64/egg/linuxacademy_dl/assets.py", line 140, in download
self.analyze()
File "build/bdist.linux-x86_64/egg/linuxacademy_dl/assets.py", line 136, in analyze
self._resources = self.get_resources()
File "build/bdist.linux-x86_64/egg/linuxacademy_dl/assets.py", line 133, in get_resources
return processor()
File "build/bdist.linux-x86_64/egg/linuxacademy_dl/assets.py", line 108, in call
return self.process()
File "build/bdist.linux-x86_64/egg/linuxacademy_dl/assets.py", line 71, in process
play_list_url = self.get_m3u8_playlist_url(lesson_page_source)
File "build/bdist.linux-x86_64/egg/linuxacademy_dl/assets.py", line 64, in get_m3u8_playlist_url
return LessonProcessor.m3u8_grabber.findall(lesson_page_source)[0]
IndexError: list index out of range
INFO     Please report this issue on https://github.com/vassim/linuxacademy-dl/issues. Make sure you are using the latest version. Be sure to call linuxacademy-dl with the --debug flag and include its complete output.

Seems like the old code (with errors) is active again. The command "return LessonProcessor.m3u8_grabber.findall(lesson_page_source)[0]" is not part of the fixed code I posted.


Implement Multiples Tracks on the Schedule

Today in our backend we already have the Tracks implemented, but in the Frontend didint.
This is directly related with the Schedule page.
By default we already have a single track, this issue asks for a interface to implement:

Using a modal to create new tracks
On the Create Schedule, allow to drag proposals to this tracks
On the Schedule Detail show all the tracks side-by-side

i'd like to try to take a look at this. i'm sure i'll have questions this seems a little complicated.

is this what you had in mind? the default track isn't displayed in a panel currently because there is no title for it.

Hi @sunnySinghD, yeah it something like that, do you want to work on this?

yes if you dont mind.

I would love to have someone working on this. Join me on http://gitter.im/luanfonceca anytime you have any questions


Feature / Batch Requests

Closes #198
My initial approach was to only expose BatchGQLRequest (and same for QueryBuilder / Response), but I could not find any specification for how streaming responses should be handled in batch request case (also it appears that you have plans to change streaming implementation anyway https://github.com/async-graphql/async-graphql/issues/146). It also would mean that return types would all be BatchQueryResponse which seemed like too big of a public API change - let me know your thoughts.
So all I did is to add batch support as an option, and add batch handlers to the integrations.

Thanks for hark work, I have reviewed your changes, and indeed, as you said, this is indeed a very big change.
In fact, I plan to delete streaming requests. After all, this feature has not yet entered the official specification, and the current architecture is very difficult to support these features.
So if you don't consider this function, can you extend GQLRequest instead of adding BatchGQLRequest type, so that async_graphql_warp::graphql_batch is no longer needed, async_graphql_warp::graphql can support batch requests and try to keep the API concise.
Thanks again!

Thanks for your kind words.
As for extending GQLRequest instead of adding BatchGQLRequest - I don't think this is possible, since we need the BatchGQLRequest enum to be able to deserialize both single and batch requests easily.
I can however make only BatchGQLRequest and BatchQueryResponse public and make them main input and output types of the library (meaning for example calling execute on schema will also return BatchQueryResponse, just with a Single variant).
This will mean updating a bunch of tests, but probably won't affect users much? They should automagically gain batch query support.
One awkward part I find with batch query support is the QueryBuilder, specifically support for batches - constructing it from request is nice and straightforward, but if somebody wants to construct query with a builder from the ground up, I can't find a nice way represent building either a single query or a batch query on a single struct. Although I still have an idea for a nice manual batch query-building experience I want to try out first.
So to summarize:

Are we OK with making Batch* types as main exposed types? (btw, my implementation is heavily inspired by juniper, so I may simply be missing other possible solutions)
If so, should I just remove all mentions of streaming support / replace with mention of it not being supported until a specification settles? (multipart requests for uploading files will still be supported both for single requests and batches)
I'll put some more work into making better API for BatchQueryBuilder if it's to become the main interface - main goal is that building a query should not require juggling both QueryBuilder and BatchQueryBuilder at the same time.

In the meantime I'm converting this to a draft :)

Is it feasible to replace the original GQLRequest with BatchGQLRequest and try to keep the original interface?
I'm going to open a branch to remove all the code for streaming requests.

Or add BatchRequest and BatchResponse as you said, which I think is also a good implementation.

I've removed all the code for streaming requests from the BatchRequest branch.

If so, should I just remove all mentions of streaming support / replace with mention of it not being supported until a specification settles? (multipart requests for uploading files will still be supported both for single requests and batches)
I'll put some more work into making better API for BatchQueryBuilder if it's to become the main interface - main goal is that building a query should not require juggling both QueryBuilder and BatchQueryBuilder at the same time.

I thought about it and thought it was a good idea. Let's move forward with this goal.
Thank you again for your efforts on this project. 😀

Keep GQLRequest, but GQLBatchRequest is an enumeration, can hold one or more GQLRequests, corresponding to GQLBatchResponse, my understanding is correct?

Yes

Sorry it's taking so long - I know you want to stabilize API :)
So this still requires a bit of polish, but the main things are implemented as we discussed above. Wanted to know your opinion before proceding to documentation/removing stream response code.
Streaming response support is removed (execute_stream etc), and by default all integrations are batch-enabled, without separate batch functions.
Just to test user-facing changes, I added a PR into examples that compiles - most changes are removing defer examples. As for other examples, changes are constrained to renaming types, so should not be a big deal.
For QueryBuilder I decided it was best to split it into QueryBuilder and QueryDefinition - first one is used to build queries, second can store ctx_data and execure itself on a given schema.
This way I was able to achieve nice-ish API for building the queries:
Example for single query:
let definition = QueryBuilder::new_single(&query_source)
.variables(vars)
.label(label)
.finish();

And it can be chained further:
let resp = QueryBuilder::new_single(&query_source)
.variables(vars)
.label(label)
.finish()
.execute(&schema)
.await;

Same for batches:
let resp = QueryBuilder::new_batch(&first_query_source)
.variables(vars_one)
.next(&second_query_source)
.variables(vars_two)
.finish()
.execute(&schema)
.await;

Thank you for your work, but this change is too big, I want to evaluate it. 😀

I deleted all the code about streaming requests.

I have a suggestion to rename BatchGQLRequest to GQLRequest and GQLRequest to GQLRequestPart. What do you think?

Sure, sounds good to me :smile:

Done with renames, also renamed QueryDefinition -> QueryDefinitionPart

API is still not easy to use, can you find ways to simplify?

Hi!
Terribly sorry for prolonged silence, got pulled into another project.
Regarding simplifying API - frankly, I've got no ideas. Request needs to be an ENUM to seamlessly handle both single and batch requests, and consequently response needs to be ENUM as well. I assume working with the resulting response enum is what you say isn't easy?
One change I can think of is that QueryDefinition.execute need not return BatchQueryResponse - that should eliminate at least some of the friction with this implementation. This way changes can be limited only to interactions with QueryDefinition, what do you think?

Let me think about it. In fact, I haven't thought of a good way. 😂

It's easier for me to think while writing code, so here's how the changes look if QueryDefinition.execute returns Result<QueryResponse> instead of BatchQueryResponse :smile:
Will see if I can come up with something else as well

Thank you for your efforts. I think it is now gradually moving in the right direction.
I think we should not expose the batch API to users who do not need to use them, only when they really need batch. For example, async_graphql_actix_web::BatchGQLRequest, this type confuses me. 😁
Thanks again, because this is a huge change, I hope to keep the api forward compatibility as much as possible.

Because there are many problems in the API design of the 1.x version, the code for batch queries is not easy to use. This is my problem. I apologize for that.
In version 2.0, we have redesigned a lot of APIs and added support for batch queries. Using a lot of code you have already done. Thank you very much for your contribution.
In version 2.0, you can use http::receive_batch to parse batch queries, and then use Schema::execute_batch to execute it. If you want, you can review our current implementation, your opinion is very important to us.
Thanks again! 🙂 @msdinit

@sunli829 Right now, batched operations are implemented
separately from everything else (e.g. BatchRequest and Request). It might be simpler to treat everything as a batch request, and have singular requests be a special case where the number of requests in the batch is 1.

@sunli829 Right now, batched operations are implemented
separately from everything else (e.g. BatchRequest and Request). It might be simpler to treat everything as a batch request, and have singular requests be a special case where the number of requests in the batch is 1.

Make Request to an internal type, BatchRequest instead of Request, right?

Actually nevermind I didn't read the code properly. It's all good, sorry for the inconvenience!

Actually nevermind I didn't read the code properly. It's all good, sorry for the inconvenience!

Actually I also considering whether to use BatchRequest instead of a Request. 😃


[Dropdown] Menus Should Open to the Left When Going Off-screen to the Right

Currently the horizontal positioning of the menus is not adjusted in any way, if the menu goes off-screen. This is extremely frustrating when building menus with multiple nested levels.
I know the direction of the menu can be set to open either to the left or to the right, but this is not ideal in a responsive website where the position of the menu items is not fixed and can change on any screen. Ideally the dropdown should know when a menu goes off-screen and open it in the opposite direction.
JSFiddle: http://jsfiddle.net/znhby35f/
Screenshot:

I'm not sure if this is already something confirmed or in progress, but I couldn't find a similar issue addressing this exact problem.

@andreiho after 2 years, do you remember how you solved it?

@mehi-shokri This was fixed a while ago: https://github.com/Semantic-Org/Semantic-UI/pull/4211


Custom Catalog no longer refreshes

On the last two versions, my custom catalog never refreshes. I'm running Mavericks and currently Quicksilver 1.2.0 400F. Files and folders created after a Catalog scan never appear in the search and clicking the refresh button in the Catalog preferences does nothing. The only way to get new files to show up is to remove my custom catalog entry and re-add it, waiting for another scan. Prior to the last two beta versions, this functioned perfectly.
This is a major part of my daily workflow as I use QS exclusively to search a folder structure of over 50,000 files. Please help. Thanks.

i would like to add a 'me too' where I'm noticing that I sometimes need to manually rescan individual catalog items to get them up to date. Rescanning the entire catalog does not update them. Here are two situations where I've seen the catalog not automatically update:
moving an application from one cataloged folder to another cataloged folder
ejecting and then remounting a disk image which has catalogued items in it
I have noticed this behavior only since I updated to version 1.2.2.

To update:  auto refresh does not occur on my large catalog, only manual
refresh does, even at program launch.
On Jan 9, 2015 8:53 AM, "louabill"<EMAIL_ADDRESS>wrote:

i would like to add a 'me too' where I'm noticing that I sometimes need to
manually rescan individual catalog items to get them up to date. Rescanning
the entire catalog does not update them. Here are two situations where I've
seen the catalog not automatically update:
moving an application from one cataloged folder to another cataloged folder
ejecting and then remounting a disk image which has catalogued items in it
I have noticed this behavior only since I updated to version 1.2.2.
—
Reply to this email directly or view it on GitHub
https://github.com/quicksilver/Quicksilver/issues/1960#issuecomment-69336448
.

My best guess is that Quicksilver doesn’t think it needs to rescan. That decision is made by this method:
https://github.com/quicksilver/Quicksilver/blob/master/Quicksilver/PlugIns-Main/QSCorePlugIn/Code/QSFileSystemObjectSource.m#L393
I can’t reproduce it, but if anyone can, and feels comfortable stepping through the code, that’s probably where the answer lies.

I'm having this issue too. Workaround I did is to create a custom trigger, using Event Triggers plugin, to "Execute Force Catalog Rescan" on "Computer Will Sleep" event, which works for laptop use. Doesn't remedy the problem though.

Guess I'm not alone in this, I'm too having the same issues where QS doesnt scan custom added file and folders, also I'm restrict to folder only too.

Same here, one year later: I have a Custom Catalog that reads my home (to depth 6). There is no automatic refresh ("every 10 minutes", normally).
I set the Task viewer do display any activity, and I see the progress bar stop at ~20 % : maybe QuickSilver fails to read something and aborts?
I'd be happy to investigate more and help nail this problem down.

I have a Custom Catalog that reads my home (to depth 6).

That sounds painful. 😃  Did you know you can access files deeper down by selecting a top-level folder and hitting → or /?
Having said that, it should be possible to do what you’re doing. Next time you see it stall during scanning, open Activity Monitor, select Quicksilver, and choose Sample Process from the gear menu. That will hopefully tell us where it gets stuck.

Yep, I do know about navigating inside folders in Quicksilver, thanks. But I am not sure how this can help with the rescan that fails?
I am attaching a Sample Process. Contacts are for instanced scanned quickly, but Quicksilver stops as soon as it starts scanning Home. I am attaching the
trace. It'd be great someone can figure out what happens, because I gather I am not the only one with this problem. :)

Yep, I do know about navigating inside folders in Quicksilver, thanks. But I am not sure how this can help with the rescan that fails?

It’s not related to the scan. Just general advice. You’re adding tens of thousands of things to the catalog that you’ll never access, which slows everything down.
Strange that contacts appear to get scanned quickly. Almost all of the activity in that sample comes from the Contacts plug-in. It appears to be scanning contacts multiple times simultaneously (which shouldn’t be possible). Could you try temporarily disabling the Contacts plug-in and see if it has any effect on scanning Home?

I did disable Contacts, but my Home is still not scanned during the automatic refresh. Other things are apparently scanned, and then when Home is hit (while the Task window still shows a largely incomplete progress bar), the tasks stops brutally (the Task window disappears).
I am attaching a process sample from before and during the refresh.
Let me know if you want me to take more samples, or better timed samples. There is obviously a big bug lurking somewhere.

Same problem


Remove Geuze

remove Geuze's spreading model from core
replace by ideal filling
opens more flexibility for other spreading models

Should have some internal discussion about this with @msch67 and @axtimhaus next week.

Geuze is clearly a model. We agreed, that the core should not contain models itself. This opens flexibility and avoids conflicts.
It opens the possibility of simulating the "most ideal" case. May be interesting for design routines.


Running in Tempdir, cleaning up generated files?

Hi,
is there a possibility to run my tested binary in a tempdir and also cleaning that up after a successful (or even after a failing) test?
My binary creates files, I would thus also need a way to check whether files where generated with the appropriate contents, and clean them up after the tests.
If there is no such feature, feel free to consider this a feature-request! :laughing:

We've not baked in running in a tempdir but leave the file system operations to https://github.com/assert-rs/assert_fs, with the assumption people will most likely want to additional things, like setup some files before running the command or verify the files after.
For a simple example (though without assert_cmd), see https://github.com/sunng87/cargo-release/blob/b5db1c3c2686f2dcb91c2bf3f303b9f6e99f5945/src/cargo.rs#L236


Fix icons color in 20170608

Fixes https://github.com/caneco/design-tips/issues/4

Following up the ideas from Steve it's always better to change the text color of the element than using opacity.
Hope you don't mind :)

Yeah, no worries. It’s a bit hacky but I did it that way so the hover worked (i.e. the svg changes color on hover on its container a). Which by the way does not work after your commit: hover:text-grey-black does not exist.

Again… thank you! 😅
Should be fine now. Dunno if the main idea from Steve it was to change the color of the svg on :hover, I might just leave it as this since the background already gives some UX information.


No Error Logged to Error Log on View Panel Failure

System information:

Operating system (distribution) and version: MacOS Catalina 10.15.4
DBeaver version: <IP_ADDRESS>004051509
Additional extensions: None

Connection specification:

Database name and version: N/A - Denodo 7.x
Driver name: N/A- Denodo 7.x
Do you use tunnels or proxies (SSH, SOCKS, etc)? No

Describe the problem you're observing:
I'm trying to troubleshoot what I assume is an out of spec JDBC Driver provided by Denodo aka www.denodo.com but I'm unable to confirm anything because the exception doesn't result in an error in the Error Log View. After expanding Views/my_view_name I get the following error but I really need the full stack trace to figure out exactly where the driver is failing and maybe what DBeaver is calling
. I'm using the "Generic" option with the https://files.denodo.com/support/releases/Denodo/7.0/driver/denodo-vdp-jdbcdriver.jar?fname=denodo-vdp-jdbcdriver-7.0-update-20200310.jar version of the JAR.
SQL Error: Error in method findColumn(). Field name not found.
Error in method findColumn(). Field name not found.
Error in method findColumn(). Field name not found.
To be clear the exception isn't the DBeaver Bug, the bug is that the exception isn't getting logged anywhere preventing me from troubleshooting furthur.
Steps to reproduce, if exist:
Include any warning/errors/backtraces from the logs

[dbeaver-debug.log](https://github.com/dbeaver/dbeaver/files/4580410/dbeaver-debug.log)

hi @shawnweeks ,
could you check dbeaver-debug.log? https://github.com/dbeaver/dbeaver/wiki/Log-files

I checked there as well, nothing.
dbeaver-debug.log

This is weird.
I don't see any logging in debug-log. Although there should be some logging about connection initiation.
I can't test this as I don't have test Denodo instance.
Anyhow, I've added extra logging of all items loading errors.
Please try in the new dbeaver version.

Is this fix in 7.0.5 the current release or do I need to wait? I can test this tomorrow when I get back home.

@shawnweeks you could test if the issue is fixed in EA version https://dbeaver.io/files/ea/

Tested in the EA version this morning and my exception is gone so apparently you fixed whatever edge case the driver was hitting.

Thank you for your help! Closed.


[Wiki] 32bit Warning

JMusicBot does not officially support 32bit java. It would probably be a good idea to add a blurb to either the main readme.md or the setup wiki page that says something along the lines of:

JMusicBot is designed to be run on 64bit systems.
While you can still run JMusicBot on 32bit systems (eg: a Raspberry Pi), it is not officially supported, and you may run into issues.

Such a disclaimer for all platforms/operating systems should also appear in the Installing Java page.


feat: improve local source code download behaviour

generate hash based on source dir and save it as the current version

Fixes #1926
What logic is applied:

in case of local sources (checked via the IsLocalSource func)
generate a hash of the local directory and use it as version for terragrunt

Currently in discussion state. If the maintainers are happy with the changes I will cleanup the print outputs ( for the developing they are useful to keep an eye on the generated hashes :) )

@rhoboat  did you have some time to recheck the changes so i can finish the PR to be mergable?

Hi,
once test printfs will be removed and fixed spacing for TestAlreadyHaveLatestCodeLocalFilePathWithHashChanged I think PR will be ready for merge

wrong spacing in the test method?

Not a big deal, but it is inconsistent with the rest of test functions

Hi, looks like recent changes broke unit tests...
cd cli
go test -run TestAlreadyHaveLatestCodeLocalFilePathWithHashChanged  -v

=== RUN   TestAlreadyHaveLatestCodeLocalFilePathWithHashChanged
=== PAUSE TestAlreadyHaveLatestCodeLocalFilePathWithHashChanged
=== CONT  TestAlreadyHaveLatestCodeLocalFilePathWithHashChanged
    download_source_test.go:355:
                Error Trace:    download_source_test.go:355
                                                        download_source_test.go:50
                Error:          Not equal:
                                expected: false
                                actual  : true
                Test:           TestAlreadyHaveLatestCodeLocalFilePathWithHashChanged
                Messages:       For terraform source TerraformSource{CanonicalSourceURL = file:///home/denis/projects/gruntwork/terragrunt/test/fixture-download-source/hello-world-local-hash, DownloadDir = /tmp/download-source-test846638434, WorkingDir = /tmp/download-source-test846638434, VersionFile = /tmp/download-source-test846638434/version-file.txt}
--- FAIL: TestAlreadyHaveLatestCodeLocalFilePathWithHashChanged (0.01s)
FAIL
exit status 1
FAIL    github.com/gruntwork-io/terragrunt/cli  0.053s

Hi, looks like recent changes broke unit tests...
cd cli
go test -run TestAlreadyHaveLatestCodeLocalFilePathWithHashChanged  -v

=== RUN   TestAlreadyHaveLatestCodeLocalFilePathWithHashChanged
=== PAUSE TestAlreadyHaveLatestCodeLocalFilePathWithHashChanged
=== CONT  TestAlreadyHaveLatestCodeLocalFilePathWithHashChanged
    download_source_test.go:355:
                Error Trace:    download_source_test.go:355
                                                        download_source_test.go:50
                Error:          Not equal:
                                expected: false
                                actual  : true
                Test:           TestAlreadyHaveLatestCodeLocalFilePathWithHashChanged
                Messages:       For terraform source TerraformSource{CanonicalSourceURL = file:///home/denis/projects/gruntwork/terragrunt/test/fixture-download-source/hello-world-local-hash, DownloadDir = /tmp/download-source-test846638434, WorkingDir = /tmp/download-source-test846638434, VersionFile = /tmp/download-source-test846638434/version-file.txt}
--- FAIL: TestAlreadyHaveLatestCodeLocalFilePathWithHashChanged (0.01s)
FAIL
exit status 1
FAIL    github.com/gruntwork-io/terragrunt/cli  0.053s

Hi, looks like recent changes broke unit tests...
cd cli
go test -run TestAlreadyHaveLatestCodeLocalFilePathWithHashChanged  -v

=== RUN   TestAlreadyHaveLatestCodeLocalFilePathWithHashChanged
=== PAUSE TestAlreadyHaveLatestCodeLocalFilePathWithHashChanged
=== CONT  TestAlreadyHaveLatestCodeLocalFilePathWithHashChanged
    download_source_test.go:355:
                Error Trace:    download_source_test.go:355
                                                        download_source_test.go:50
                Error:          Not equal:
                                expected: false
                                actual  : true
                Test:           TestAlreadyHaveLatestCodeLocalFilePathWithHashChanged
                Messages:       For terraform source TerraformSource{CanonicalSourceURL = file:///home/denis/projects/gruntwork/terragrunt/test/fixture-download-source/hello-world-local-hash, DownloadDir = /tmp/download-source-test846638434, WorkingDir = /tmp/download-source-test846638434, VersionFile = /tmp/download-source-test846638434/version-file.txt}
--- FAIL: TestAlreadyHaveLatestCodeLocalFilePathWithHashChanged (0.01s)
FAIL
exit status 1
FAIL    github.com/gruntwork-io/terragrunt/cli  0.053s

Found the error. Remove one output to the test version file which is necessary. Now the tests should pass again.

Just eyeballing the changelogs, I am suspecting that this change might be behind #2171 ....


updating BYOS commands to mark them as preview

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[x] I adhere to the Command Guidelines.

@yonzhan this change is to basically reflect that this command/feature is in Preview. We noticed that adding the _is_preview attribute to the command doesn't update the documentation (help) to reflect the same -> is this something that can be done automatically without needing an update to _help.py?

When you type -h to show help, make sure you can see a notification saying that it's in preview. I think it's enough.

@qwordy will the auto-gen doc page also get updated with preview information?
Specifically --> https://docs.microsoft.com/en-us/cli/azure/webapp/config/storage-account?view=azure-cli-latest
I didn't see this happening for other commands marked as preview.

Oh. I understand the problem now.
Doc page won't be updated automatically.
@Juliehzl Do you know how to mark a command as preview in _help.py?


Due to the use of netty component, the netty.controller.package Property is invalid, causing controller to fail

Describe the bug
Due to the use of netty component, the netty.controller.package Property is invalid, causing controller to fail.
Desktop (please complete the following information):

OS MAC
Browser chrome, safari,curl
Version 1.0.2

this issues is fix please select 1.0.2 version

this issues is fix please select 1.0.2 version


cudominer: add package

cryptocurrency miner from https://www.cudominer.com/cudo-miner-cryptocurrency-mining-software/
draft in https://github.com/milahu/nur-packages/commit/c1c57bea8288119ea66fe491ffae6aa8dcb3115d
status: fails to run
readme.txt
$ ./result/bin/cudominercli
Pkg: Error reading from file.

$ strace ./result/bin/cudominercli 2>&1 | grep open
...
openat(AT_FDCWD, "/sys/fs/cgroup/memory/memory.limit_in_bytes", O_RDONLY|O_CLOEXEC) = -1 ENOENT (No such file or directory)
...

$ ls /sys/fs/cgroup/memory.* | cat
/sys/fs/cgroup/memory.numa_stat
/sys/fs/cgroup/memory.pressure
/sys/fs/cgroup/memory.reclaim
/sys/fs/cgroup/memory.stat

# https://stackoverflow.com/questions/65646317/sys-fs-cgroup-memory-memory-limit-in-bytes-not-present-in-fedora-33

$ findmnt /sys/fs/cgroup
TARGET         SOURCE  FSTYPE  OPTIONS
/sys/fs/cgroup cgroup2 cgroup2 rw,nosuid,nodev,noexec,relatime,nsdelegate,memory_recursiveprot

The issue is related to new cgroup v2. To resolve this, revert cgroup to v1:

sudo sed -i '/^GRUB_CMDLINE_LINUX/ s/"$/ systemd.unified_cgroup_hierarchy=0"/' /etc/default/grub

and reboot.

After this /sys/fs/cgroup/memory/memory.limit_in_bytes will be present.

issue reported in discord chat https://discord.com/channels/541993453467336704/541993454180237333/1291413717568716862
upstream issue https://github.com/NixOS/nixpkgs/issues/333219 - ping @Nika03
the download url for install.sh seems to be stable for different users at cudominer.com

Hi, I just wanted to let you know that I'm only a user. I used this software on Windows but I haven't tried to use it on Linux since I use NixOS as my primary OS. CudoMiner was made primarily for Ubuntu as far as I'm aware. I haven't tried packaging software for Nix.
Here is a screenshot of the download page from CudoMiner for Ubuntu

Here is the line of bash as text:
sudo su -c "bash <(wget -qO- https://download.cudo.org/tenants/135790374f46b0107c516a5f5e13069b/5e5f800fdf87209fdf8f9b61441e53a1/linux/x64/stable/install.sh)" root
I don't think I can be of assistance. Sorry.

all good : )
i just wanted to share my progress
waiting for https://github.com/NixOS/nixpkgs/issues/333219 could take forever...

download page from CudoMiner

yepp, i have a user account at cudominer.com, and i get the same url for install.sh
a possible workaround for the cgroup2 error would be to downgrade to cgroup1
as described on stackoverflow
but i hope that cudominer will fix their code in the near future...


Subscription-based service for lifetime/expanded feature loot tracker or other useful things.

Hello, I saw that the loot tracker only tracks the last 1024 kills of the current session, and I'm going out on a limb here thinking that may be because of database limitations or some other limitation that increased funding may solve.
I and many others really appreciate the runelite platform, and I was wondering if it would be possible to do some sort of subscription service that we could subscribe to, for say, 2.99/month or 4.99/month that would be able to secure this additional funding for cool features like an expanded loot tracker or GE tracker or something along those lines.
I understand this may be a non-starter because of possibly issues with jagex by monetizing runelite in a way such as a subscription service, but I just wanted to provide this as possible food-for-thought or to spark any discussions.
Sorry if this question/request was already posed.
Thanks for all you guys do.

Not that it wouldn't be nice for the tracker to just do it, but FYI there's the Loot Logger plugin from Plugin Hub that extends the base Loot Tracker addon.  It only resets data if you tell it to.  It's locally stored, tho it shouldn't be hard to point it to a file server/cloud storage service if you wanted that.  It stores data at .runelite/loots/{email}/ if you wanna figure something out with that.
May be a necessary work around.


Allow local callbacks in addition to global onResponse function

From a recent support request at PullString:

I've looked into the javascript API, but some features of it won't work
with my code. I'm mixing Pullstring with socket.io and I can't have a
global onResponse call. I need to be able to redefine the different
onResponse with different sockets call.
If I could do something like this:
socket.on('new userroom', function(msg){
socket.join(msg.sessionid);
conversation.start( MY_PROJECT_ID, request, function( response ){
// custom on response code
io.sockets.in(msg.sessionid).emit('bot send', { sessionid : 1, msg
: output.text, user: 'bot', user: output.character, conversationId:
response.conversationId } );
}
saveClient(msg);
});

To support technologies like socket.io, we could easily allow both a global onResponse callback and a locally scoped one. While it seems clear that a local callback would take precedence over a global one, there is a design choice to make about whether the local callback would work in exclusion of the global one.

Hello,
I'm just wondering if any progress has been made on this issue, and if we have an ETA for the implementation.
If there is no ETA, what would you suggest as a workaround?
Cheers

Thanks for checking in about this. I haven't implemented it yet, but I should get to it in the next week.

sorry for the delay. this was just merged into master, commit 108fed3.

note that use local callbacks, method calls will now look something like:
conversation.sendText('my text', null, callback);

The callback is always the last parameter of functions that send input to the web api. This means also setting the preceding parameter to null if you are not also passing in a Request object.


Provide an option for chunking based on a signal to include empty chunks

For sequences that chunk based on a signal, the signal will be effectively ignored if no elements were received since the previous signal. Adding an option to pass an empty chunk on such a signal would be useful, particularly for use cases where you want to detect and report on the absence of something regularly happening.
An alternative approach would be to make this the default behaviour since there are other mechanisms that can be used to compose a sequence that only produces non-empty chunks.

I could like to work on this issue, Can you provide more context on how to start and which part you want me to enhance ? @FranzBusch


Implement unlink command

[kind/Feature]
Which functionality do you think we should add?
Provide odo unlink command for unlinking components from another component/services.
Subtask of https://github.com/redhat-developer/odo/issues/171
cc/ @geoand

Thanks @jorgemoralespou
I will fix this immediately after #856 is merged

I will start work on this soon, I admit I had forgot about it :)

Adding size/L since I'm also going to refactor link as part of this to include the new command organization pioneered in the service module


Visualization in Steamgraphchart showing incorrect results when single value gets edited

Repro steps:
Select example music datafile.
Select Steamgraph visualization and complete the axis thing.
... we get the visualization as seen on the upper left corner.
Then edit the data, alter a single value with a value of 1000 market share.
Redo the visualization.
It is observed that the data that can be seen ordered BEFORE in time than the value that we edited changed from 50 to almost 500.
It was expected that the values from years after the edited value, still got the same values as the ones from years after the edited value.

Not sure to have understood the problem.
In RAW, the visualization is scaled according the maximum value.
In the second visualization, data didn't change, it is scaled to fit the maximum value in the given visualization height.
If you want to keep the same scale, you should increase the visualization height by ~10 times.
If I misunderstood the question, please let me know.

The problem is not with the data that have been altered in order to test the feature. The problem seems to be with how the previous data are displayed.
Compare 8-track, Vinyl and CD in both images. Only altering the amount of vinyl to 1000 we get a 8-track display close to 500 (red square on bottom capture)

Not sure what do you mean with

close to 500 (red square on bottom capture)

If you are referring to the Y axis, you are misreading the visualization.
In the streamgraph the value (in your case, the market share) is not mapped on the y position, but on the thickness of the stream, see: https://en.wikipedia.org/wiki/Streamgraph


WIP scheduling stuff

Wont pass until we merge https://github.com/rancher/agent/pull/52, cut an agent release, and update this PR with the new release

@cjellick postgres has been merged so you can just rebase and it will catch schema issues.


chore(curriculam): Made the changes as requested in the issue

I have made the changes as requested i.e. https://github.com/freeCodeCamp/freeCodeCamp/issues/48273#issue-1425104016 to this line https://github.com/freeCodeCamp/freeCodeCamp/issues/48273#issuecomment-1294023664
Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #48273

Testing In Gitpod:

awesome work, sorry for misguiding you. you didn't need to use code snippet markdown, you can keep the previous update.

I committed again. You can check.


fix bad JSON, file is invalid currently!

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

note: some deletes need to happen as people have 2 tokens with the same address under the same chainId and other various things needed to make the file JSON valid (like remove image links [i picked the working image url always and the one under the main/assets dir vs another place if another url was provided]).
Here were the 2 address that i think i had to modify:
6SKogZxCWY9jKsKPMT3ChJUhQxAEeB6NjVidXQK6TEdW
DEhAasscXF4kEGxFgJ3bq4PpVGp5wyUxMRvn6TzGVHaw


Autocomplete input inside ScrollView

Not sure that is a library problem, but touch events of autocomplete options is not working when Autocomplete component placed inside ScrollView.
Maybe someone saw this behavior earlier or I did something wrong?

See #5. If you have still trouble feel free to reopen.


RC Test: Circuit Construction Kit: DC & DC "Virtual Lab" 1.2.0-rc.1

@arouinfar @kathy-phet @KatieWoe, circuit-construction-kit-dc/1.2.0-rc.1 and circuit-construction-kit-dc-virtual-lab/1.2.0-rc.1 are ready for RC testing.
@kathy-phet said:

Teachers are head-over-heals for this non-ohmic bulb and if we get it out soon, it would be used immediately. So this is a priority and I'd like to see it move to rc quickly if possible.
[...] This is higher priority than GAO.

Document issues in https://github.com/phetsims/circuit-construction-kit-common/issues and link to this
issue.
Prior dev testing completed in https://github.com/phetsims/QA/issues/587
The publication issue is https://github.com/phetsims/circuit-construction-kit-common/issues/616
Assigning to @kathy-phet or @KatieWoe for prioritization.

General RC Test

What to Test

Click every single button.
Test all possible forms of input.

Test all mouse/trackpad inputs.
Test all touchscreen inputs.

If there is sound, make sure it works.
Make sure you can't lose anything.
Play with the sim normally.
Try to break the sim.
Test all query parameters on all platforms. (See QA Book
for a list of query parameters.)
Download HTML on Chrome and iOS.
Make sure the iFrame version of the simulation is working as intended on all platforms.
Make sure the XHTML version of the simulation is working as intended on all platforms.
Complete the test matrix.
Don't forget to make sure the sim works with Legends of Learning.
Test the Game Up harness on at least one platform.
Check this LoL
spreadsheet and notify AR or AM if it not there.
If this is rc.1 please do a memory test.
When making an issue, check to see if it was in a previously published version
Try to include version numbers for browsers
If there is a console available, check for errors and include them in the Problem Description.
As an RC begins and ends, check the sim repo. If there is a maintenance issue, check it and notify developers if
there is a problem.

Focus and Special Instructions
[Provide further instructions here. If you have any further tests you want done or specific platforms you want tested,
list them here. Any behaviors you want QA to pay special attention to should be listed here.]

Issues to Verify

[x] Switching screens makes interacting with carousel difficult/impossible

These issues should have the "status:ready-for-qa" label. Check these issues off and close them if they are fixed.
Otherwise, post a comment in the issue saying that it wasn't fixed and link back to this issue. If the label is
"status:ready-for-review" or "status:fixed-pending-deploy" then assign back to the developer when done, even if fixed.

Link(s)
CCK DC

Simulation
iFrame
XHTML
Test Matrix
Legends of Learning Harness

CCK DC Virtual Lab (just needs a spot check, since it is 99% same as CCK DC)

Simulation
iFrame
XHTML
Test Matrix
Legends of Learning Harness

Accessibility (a11y) RC Test

What to Test

Specific instructions can be found below.
Make sure the a11y feature doesn't negatively affect the sim in any way.
You should be able to use the delete key to delete a selected circuit element
Keyboard navigation is not supported
Screen readers are not supported

Magnification
This sim supports magnification with pinch and drag gestures on touch screens, keyboard shortcuts, and mouse/wheel controls. Please test magnfication and make sure
it is working as intended and well with the use cases of the simulation. Due to the way screen readers handle user input, magnification is NOT expected to
work while using a screen reader so there is no need to test this case.
Final Requests

[ ] If this sim is in this list, make an
issue in website to ask if PhET research page links need updating. Please assign
to @terracoda and @emily-phet.

FAQs for QA Members

There are multiple tests in this issue... Which test should I do first?
Test in order! Test the first thing first, the second thing second, and so on.

How should I format my issue?

We typically assign the developer who opened the issue in the QA repository.

My question isn't in here... What should I do?
You should:

Consult the QA Book.
Google it.
Ask Katie.
Ask a developer.
Google it again.
Cry.

I set priority to "high" and moved it above "Gravity and Orbits" in the Project Board, please adjust as necessary.

@jessegreenberg said there is a joist bug that affects the screen.  We would likely invalidate this RC and take new SHAs once that's corrected.

Sorry @samreid, the regression identified in https://github.com/phetsims/joist/issues/658#issuecomment-772811567 made it to this RC. This reverting commit in joist should be cherry-picked into the joist code for this RC. https://github.com/phetsims/joist/commit/d0be22a46c325481ea2cb3bca6bb5d3c02f1a704

We would likely invalidate this RC and take new SHAs once that's corrected.

@KatieWoe if you have a chance to start testing before I publish new versions, I think that's OK.  Let's just consider https://github.com/phetsims/joist/commit/d0be22a46c325481ea2cb3bca6bb5d3c02f1a704 as something to be cherry-picked before publication.

Noting that this sim should be added to the LoL spreadsheet when it is ready.

These sims have already been delivered to LoL. Since the client uses the "latest" version of the sim, they won't need to manually update anything. The changes are not significant enough to warrant new icons/descriptions.

QA is done.

@samreid @KatieWoe CCK DC and CCK DC Virtual Lab 1.2 scenery SHAs received a patch for phetsims/phet-android-app#155.
If possible, you may wish to verify this sim works well in the PhET Android app as part of this test or the next one if there is one.

Thanks, I reviewed the opened issues and will continue there.  Nice discoveries and thanks!


[#390] New Interactive Button Task Completed

Has you assign #390 issue i have completed, Hope you like the work

@PoorviShetty Ok sir i'll checked it out

@PoorviShetty sir now check, i have resolved the conflict

There are no files changed
I guess we already merged the same PR.
closes #390


Add support for preprocessed CSS languages

styled-jsx supports plugins which allows for the use of multiple different languages (Stylus, Sass, PostCSS, etc.)
It would be cool to support multiple languages out of the box and be able to specify the language used in a project-specific configuration.

Agreed! styled-jsx plugins like styled-jsx-plugin-sass work well, but not if you want syntax highlighting. For now I have stopped using SCSS syntax because vscode-styled-jsx is more important 😄

While it would be awesome to have a project-specific configuration and use only this extension, I forked the project to support Stylus syntax highlighting.
See: https://github.com/samuelroy/vscode-styled-jsx-stylus

We use styled-jsx-plugin-sass with Next.js to import and reuse SCSS functions and SCSS variables from customized Bootstrap design systems. The system is used globally but this way we can create scoped CSS in line with the design system. We tried vscode-styled-jsx-languageserver but it can not handle how we break out styles as const variables to keep the component function clean. Issue thread with example code.


Password reset doesn't work and changing password almost never works

The password reset feature doesn't work and changing password almost never works. Currently neither the password I used before or the password I tried to change to works, for some reason. So it looks like I will be locked out of my account when I close my current Winds session.

Hey @anonsubmitter, I just tested using Chrome and it worked without any issues. Are you on the hosted version (https://winds.getstream.io) or your own fork? Thanks, Nick.

I'm on the hosted version.

What browser are you using?

I'm logged in in both Chrome and Firefox and I tried logging in in Pale Moon to see if the password change had gone through.

Can you clear your localStorage and do a hard reset on your cache, then give it another round? http://superuser.com/questions/220179/how-can-i-do-a-cache-refresh-in-google-chrome

I tried clearing localStorage in Chrome, changing the password and logging in in Pale Moon, but I still got "INVALID EMAIL OR PASSWORD".

What happens when you attempt to create an account?
On Tue, Nov 22, 2016 at 3:31 PM anonsubmitter<EMAIL_ADDRESS>wrote:

I tried clearing localStorage in Chrome, changing the password and logging
in in Pale Moon, but I still got "INVALID EMAIL OR PASSWORD".
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/GetStream/Winds/issues/54#issuecomment-262385669, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABRFBBVOhYYwCi68DtNgNnD9GZcth5X7ks5rA20vgaJpZM4K5_Pp
.

I could create a new account just fine in Chrome.

With the same email address?

No, I used another email address since I assumed that creating another account with the same email address would be impossible.

Can you attempt to create with the email that you were using for the password reset? It'll yell at you if there's an error. What is the email you're trying to create an account for? (happy to chat with you directly via<EMAIL_ADDRESS>

If I try to create an account with the same email address, will my feeds get wiped then? I spent about an hour adding 55 feeds and I'd like to avoid redoing that if possible. Is it possible to export the feeds to an importable backup file?
I'll email you the details.

No, they won't get wiped. Don't worry.
On Tue, Nov 22, 2016 at 3:44 PM anonsubmitter<EMAIL_ADDRESS>wrote:

If I try to create an account with the same email address, will my feeds
get wiped then? I spent about an hour adding 55 feeds and I'd like to avoid
redoing that if possible. Is it possible to export the feeds to an
importable backup file?
I'll email you the details.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/GetStream/Winds/issues/54#issuecomment-262388473, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABRFBGuvsVny_IXDxpw3OBvHLgNqp8Iqks5rA3BCgaJpZM4K5_Pp
.

Ok, I've tried signing up again with the same email address. So far I haven't gotten any confirmation email.
The email with my email address has been sent.

Thank you. Can you please check your email now for a password reset
confirmation? It should contain a temporary password.
On Tue, Nov 22, 2016 at 3:50 PM anonsubmitter<EMAIL_ADDRESS>wrote:

Ok, I've tried signing up again with the same email address. So far I
haven't gotten any confirmation email.
The email with my email address has been sent.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/GetStream/Winds/issues/54#issuecomment-262389702, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABRFBCBpyUNAEMFa3NhSUElcFF9bieNfks5rA3GqgaJpZM4K5_Pp
.

Yup, I got it. I'll log out from my account on all browsers and see if the password reset works.

👍

It worked. I'll try and change the password twice to see if the error is gone.

It didn't change to the password I tried to change it to.

No - it should happen right away. Does the temporary password still work? Also, do you see any errors in your console / network tab when attempting to reset?

The password reset seems to work now at least.
The only thing I saw in the console was a bunch of DOMPropertyOperations.js:142 The specified value "" is not a valid email address. lines where the characters between "" were the characters in my email address as I was typing it.
The temporary password has thus far worked with two login attempts.

Hey @anonsubmitter, this should be fixed in our latest release (0.2.0). Please let me know if the issue persists and we can re-open the case. Thanks for your help!


#2117 cleanup JSR166 package

Current coverage is 40.43% (diff: 62.50%)

Merging #2119 into master will increase coverage by 0.14%

@@             master      #2119   diff @@
==========================================
  Files          1584       1583     -1
  Lines         55385      54799   -586
  Methods       10920      10813   -107
  Messages          0          0
  Branches       7604       7485   -119
==========================================
- Hits          22314      22158   -156
+ Misses        33071      32641   -430
  Partials          0          0

Powered by Codecov. Last update 9a79560...f883dc2


[Feature][dolphinscheduler-api] access control of ExecutorService and schedulerService in project

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
[Feature][dolphinscheduler-api] access control of ExecutorService in project
Use case
[Feature][dolphinscheduler-api] access control of ExecutorService in project
Related issues
https://github.com/apache/dolphinscheduler/issues/7035
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

num
Interface
Interface method
reason

1
ExecutorService
execProcessInstance
processDefinitionMapper.queryByCode used in this method

2
ExecutorService
execute
findProcessInstanceDetailById used in this method

36
SchedulerService
createSchedule
processDefinitionMapper.queryByCode used in this method

37
SchedulerService
updateSchedule
processDefinitionMapper.queryByCode used in this method

38
SchedulerService
setScheduleState
processDefinitionMapper.queryByCode used in this method

39
SchedulerService
querySchedule
processDefinitionMapper.queryByCode used in this method

40
SchedulerService
updateScheduleByProcessDefinitionCode
processDefinitionMapper.queryByCode used in this method


Problems with lucky eggs and evolving

I've noticed several quirks with lucky eggs and evolving:

It will place a lucky egg in even if it doesn't have anything to evolve.
It cannot detect that a lucky egg is in use already (see below)
It cannot detect that a lucky egg is in use when you start the bot.

More on 2:
I use 3 different evolvepokemon tasks to get the desired behavior for pokemon that are useless but good for evolve XP vs the better pokemon I want to have a little more control over when to evolve them.     The problem is if I have more than 1 set for lucky egg, only the one that placed the egg knows it's in use and the others "Skipping evolve because has no lucky egg." because it was my last egg.

#3404 You can't both win :p

@accident12123
Explain what you want to do with your transfer/evolution and we might  be able to suggest a working configuration

@accident12123 ?

Haven't heard anything in 5 days. Closing.


Part 6: Build an Identity Perimeter video is wrongly linked

Part 6: Build an Identity Perimeter (13:57) on this page is referencing wrong video (Credentials Security instead of building identity perimeter)

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: a4226cb9-a80f-aff5-95e8-ef73bc4c7ddc
Version Independent ID: de7d413b-9f5c-dafc-10b1-a750e6820f91
Content: Chief Information Security Officer (CISO) Workshop, Module 3:Identity, and Access Management - Security documentation
Content Source: ciso-workshop/ciso-workshop-module-3.md
Service: o365-seccomp
GitHub Login: @Dansimp
Microsoft Alias: dansimp

@MicrosoftGuyJFlo , @MarkSimos - This issue is old but still true. Please fix if appropriate. Thanks!

@TerryLanfear Found the video and resolving in PR 1162 later today.
@MarekLani Sorry for how long it took to resolve this.

#please-close


Move sample annotations into separate table

As part of a moral crusade against nulls in databases.

Open design questions:

How do we annotate a range of samples? Do we want to?
Are overlapping annotations on samples a problem?
Should each sample be limited to one annotation, or can each sample have many?
Should include_in_limit_calculation also be moved out into another table?


Activation Contribution

As a consultant I'm helping some of my customer to use Metrics.NET.
Along with my customers we found needs for having dynamic activation for metrics report.
The idea is to set it in the configuration and bind it into the sending pipeline before the actual metrics reports.
This way the organization will be able to activate/deactivate metrics of specific profiles (for example tags) on demands, in order to reduce the traffic without having to limit the metric reports beforehand.
We intend to add this functionality for our use, and considering to contribute it into this repository.
Do you find it useful and like to get this kind of contribution?

So, if I understood correctly, the feature is to optionally filter which metrics get pushed based on the given configuration. Correct?


[ADD TOKEN] BPM

https://discord.com/channels/990271820101988362/1253772917326745702/1257123381292175370

I changed it.Hope it is right now


Authentication and authorization

Thinking of using Ueberauth and its oauth strategies for facebook, twitter, github etc

TODO:

[ ] Install Ueberauth with facebook and github strategies
[ ] add client ids and secrets for facebook and github apps
[ ] implement endpoints

gave up on Ueberauth, going with porting current hand-rolled auth from ruby api

auth done after 73b3bbf6f94361abeb750ee4972eb204b93a6803


Add CODEOWNERS and review request github action

For more context, see https://github.com/ForwardFinancing/docs/pull/51.

This repo is not active.


The return value of 'sscanf()' should be checked

https://github.com/IBM/kube-safe-scheduler/blob/a1d2a735fa03d92ab1b738ad1283b91c3a720791/pigeon_c/src/replay/parser.c#L91

Done

Added if clause around sscanf().


Pedersen commitment improvements

According to the suggestions by Adam Back and Blockstream team.
API changes are breaking, thus it can go only into v0.11

Oh wow... Different semantic IDs for the core RGB lib. Is this a pushback or a fast-forward?

Neither. It is a bugfix, meaning the existing way Pedersen commitments are done was non-standard and may contain inflation issues. I was seeking for Blockstream review of the way we use their API for Pedersen commitments and only now do we have it, thus we need this fix.

 Also, we need to get RGB and std libraries updated and ensure that all PSBT methods create properly balanced Pedersen commitments and tapret commitments. Thus please no releases before that is accomplished.

Understood. Will these changes invalidate existing assets?

@zoedberg thank you for the good point with tests. I have committed your doc fixes, fixed different lints and resolved conflicts with master branch as well.