Using 'produce' in selectors

I'm not sure if this is a issue with immer, which conceptually I love, or I'm using it wrong so I'd be interested to hear thoughts on this.
I'm trying to plug immer into a relatively small React/Redux project I have on the go. In reducers it all seems to work exactly as expected as the inputs are essentially a merge of the old state and an action payload.
However I then started looking at some of my selectors and thought it would be applicable there since the output from a selector is frequently a mutated version of a portion of the state, with lookups and/or references plugged in or one-time evaluations executed and saved. Again, that more or less seemed to work until I got to the part where a reference to a different part of the state tree was being attached to my new state. Then I got the following error,
Uncaught TypeError: Cannot assign to read only property '_id' of object '#<Object>'
In my case the objects are from a Mongo DB and so _id is the first field in the object.
To express this in more familiar terms, given an example of users and todos where each todo belongs to a user, my selector might be called getUserAndTodos and would return both the original user record and include a list of their todos taken from the general todo list in the global state, as well as maybe an 'allDone' flag to indicate whether that user has completed all their todos.
Given that I am essentially modifying (by addition) the user field from state, I thought this was a good place to use 'produce'. If 'produce' returns just the user record unmodified, then it's fine. If it additionally adds the 'allDone' field calculated from the users list of todos then it's also fine, but if I filter the todo list from state and add a list of the users todos to the user object I get the error above. Note, that if I turn off autofreeze then I don't see the error.
My guess is that because I am plugging in already frozen objects into a member of the 'new state' that somehow it's getting itself confused and trying to copy them over, rather than noticing it can just use a reference? More so because I am just referencing them and they aren't a direct argument to 'produce' itself. Or perhaps more like I've entirely misunderstood how this is supposed to work.
If necessary I can try and get a sandbox up that shows the error but I thought I would ask first in case I'm just doing it all wrong.
The partial stack trace of the error is below if it's of any use.
Thanks
TypeError: Cannot assign to read only property '_id' of object '#<Object>'
finalize
node_modules/immer/dist/immer.js:226
  223 |     var proto = Object.getPrototypeOf(base);
  224 |     if (proto === null || proto === Object.prototype) {
  225 |         for (var key in base) {
> 226 |             base[key] = finalize(base[key]);
  227 |         }return freeze(base);
  228 |     }
  229 | }
View compiled
finalize
node_modules/immer/dist/immer.js:226
  223 |     var proto = Object.getPrototypeOf(base);
  224 |     if (proto === null || proto === Object.prototype) {
  225 |         for (var key in base) {
> 226 |             base[key] = finalize(base[key]);
  227 |         }return freeze(base);
  228 |     }
  229 | }
View compiled
finalizeObject
node_modules/immer/dist/immer.js:237
  234 |     var copy = state.copy;
  235 |     var base = state.base;
  236 |     for (var prop in copy) {
> 237 |         if (copy[prop] !== base[prop]) copy[prop] = finalize(copy[prop]);
  238 |     }
  239 |     return freeze(copy);
  240 | }
View compiled
finalize
node_modules/immer/dist/immer.js:212
  209 |         if (state.finalized === true) return state.copy;
  210 |         state.finalized = true;
  211 |         if (Array.isArray(state.base)) return finalizeArray(state);
> 212 |         return finalizeObject(state);
  213 |     } else return state.base;
  214 | } else if (base !== null && (typeof base === "undefined" ? "undefined" : _typeof(base)) === "object") {
  215 |     // If finalize is called on an object that was not a proxy, it means that it is an object that was not there in the original
View compiled
produce
node_modules/immer/dist/immer.js:258
  255 | //values either than undefined will trigger warning;
  256 | !Object.is(maybeVoidReturn, undefined) && console.warn("Immer callback expects no return value. However " + (typeof maybeVoidReturn === "undefined" ? "undefined" : _typeof(maybeVoidReturn)) + " was returned");
  257 | // and finalize the modified proxy
> 258 | var res = finalize(rootClone);
  259 | // revoke all proxies
  260 | revocableProxies.forEach(function (p) {
  261 |     return p.revoke();
View compiled

I think Im facing a similar issue


Randomize service and service account names in tests

Please describe the Improvement and/or Feature Request
There was a serious bug that went uncaught in testing because all e2e tests used the same name for the service and corresponding service account for the pod backing the service.
The issue was caught during manual testing, and fixed by https://github.com/openservicemesh/osm/pull/2950
This issue proposes to randomize the service and service account names used in e2e tests to catch such problems.
Scope (please mark with X where applicable)

New Functionality          [ ]
Install                    [ ]
SMI Traffic Access Policy  [ ]
SMI Traffic Specs Policy   [ ]
SMI Traffic Split Policy   [ ]
Permissive Traffic Policy  [ ]
Ingress                    [ ]
Egress                     [ ]
Envoy Control Plane        [ ]
CLI Tool                   [ ]
Metrics                    [ ]
Certificate Management     [ ]
Sidecar Injection          [ ]
Logging                    [ ]
Debugging                  [ ]
Tests                      [X]
CI System                  [ ]
Demo                       [ ]
Project Release            [ ]

Possible use cases
To detect incorrect assumptions in the controller.

I'd propose we randomize:

names of Services
names of Service Account
names of deployments
Namespace
Labels on Services (and Deployments)


RustAnalayzer mistakenly gives error on a.max(b)

It seems to beleive max in a.max(b) is about Iterator::max, whereas it's about Ord::max, and as a consequence gives the error: expected 0 arguments, found 1 (and also when hovering with the mouse, doc points to core::iter::traits::iterator::Iterator). Using std::cmp::max(a, b) makes the error disappear.
Note that the feature that writes inferred types in light gray mentions &{unknown} instead of &Something (despite the type I'm iterating on being explicitly Vec<Something>), I suspect that is linked.

It would be helpful to see a more complete snippet. Are you using any local imports(as in use statements that are inside of functions or expressions)? Because those are currently being ignored by RA which could lead to this problem.

Are you using any local imports(as in use statements that are inside of functions or expressions)?

I'm not using use statements inside the function, however the Something I'm iterating on is declared in the function, just above, as it's a local structure used only to deserialize the result of a specific database query made there. Could that also be it?

Because those are currently being ignored by RA which could lead to this problem.

Why is that? :o

Why is that? :o

Nvm, I found the answer.
https://github.com/rust-analyzer/rust-analyzer/issues/1165#issuecomment-696560892
Looks like this issue is the long-standing-much-duplicated #1165 / #1559.

Hmmm that being said, it looks like we could still improve the method inference when the type is {unknown}, by at least picking functions that have the proper number arguments/the right return type.

I don't know about that, but we should avoid giving follow-up errors when we have unknown types.

Closing in favour of #6975.


chore: create module directory for discovery

Closes #19
The structure is in place but need some assistance with filling out the finer details, meaning about 80% of the readme 😄

cool, looks good to me!


feat: add "first" scope to get_player_scores

Describe your changes
Adds the "first" scope for first-place scores to the API v1, as the API v2 does not contain any implementation for this kind of stuff yet.
Checklist

[x] I've manually tested my code

ignoring the fact i see this becoming SHIT slow on maps with many scores, this needs consideration/ignoration for restricted users as their first places should not count

wait didnt I already add that to the query

i still think this would be a nice feature to get in, however i'm still concerned about performance - we should be using an intermediary table that is inserted to on score submission if score.rank == 1 with a script to back-fill this table. this is going to be quite involved as there's events which might trigger changes to these - right now i can only think of (un)restriction. @minisbett would you be happy to do something along these lines?


Best

Best practices do not include Azure role-based access control for keys, certificates, and secrets. At the moment, access control can be done in units of keys, certificates, and secrets. Instead of splitting the Key Vault Container for each application, you can think of it on a security perimeter basis.
This document needs to be updated.

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

ID: 199fbc30-6751-5ee6-4fbb-f23f2f5137cc
Version Independent ID: 00bcc33c-c14b-fa5f-74ab-f1bd635dc043
Content: Best practices for using Azure Key Vault
Content Source: articles/key-vault/general/best-practices.md
Service: key-vault
Sub-service: general
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@takekazuomi Thank you for your feedback . We are engaging the content team on the above request .

@takekazuomi best practices remain the same for both permission models. Individual object level should only be used when object is shared with multiple application. You can find more here:
https://docs.microsoft.com/en-us/azure/key-vault/general/rbac-guide?tabs=azure-cli#best-practices-for-individual-keys-secrets-and-certificates #please-close

@takekazuomi - I fully agree with your assessment and brought up the same topic back in September 2021.
I ended up getting some reputation points deducted because of my responses.


Simple file Download

In addition to Ookla tests, may be add a possibility to grab file over http, and display results.
could be very nice addition! thank you for the work so far.

What are you thinking, like I have X file (say it's 200MB) and it took me Y long to download it?

yeah, more "simple" download test, sometimes speedtests are being Mocked (rerouted by ISPs), and such test can give another aspect

I'm planning on adding iPerf, would that help alleviate the mock concerns?
Rolling my own is probably more complex

Its good, yeah, sometime could be harsh finding reliable iPerf server in the public.

What if you setup your own? My plan was to show how to configure one on a vps provider

yeah, again, its nice addon definitely.

Love the idea of adding iPerf support!


V2: Format toggling via keyboard is not being reflected in the toolbar

After upgrading to v2, the toolbar doesn't reflect formats being toggled via the keyboard until the user starts typing.
Steps for Reproduction

Visit quilljs.com
Clear all text in the demo editor
Press CMD+B (or Ctrl+B?) to enable bold formatting without typing anything

Expected behavior:
The toolbar does not reflect the new format being enabled. It's only after you start typing that the "bold" button in the toolbar changes to its "enabled" state.
Actual behavior:
The toolbar should reflect the enabled format right away, giving feedback to the user and informing them what format will be applies when they write.
This is how pretty much all editors I have used work, and its also how Quill used to work before v2. You can even try the steps for reproduction in https://v1.quilljs.com/ and see that it works differently.
Platforms:
I have observed this in macos using Chrome, Safari and Firefox. Would be surprised if it worked differently in other platforms.
Version:
2.0.2
Worked will in v1.3.7

FWIW I am currently working around this by overriding the default keybindings, copying their behavior and manually updating the toolbar (which isn't great of course). Something like this:
new Quill(this._quillHost, {
  // ...
  keyboard: {
    bindings: {
      bold: {
        key: 'b',
        shortKey: true,
        handler: function (range, context) {
          this.quill.format('bold', !context.format['bold'], Quill.sources.USER);
          this.quill.getModule("toolbar").update(range);
        }
      }
    }
  }
);


HADOOP-19018. Release Hadoop 3.4.0.

JIRA: HADOOP-19018. Release Hadoop 3.4.0.
Prepare for Hadoop-3.4.0-RC3, Update validation.

@steveloughran Can you help review this PR? Thank you very much!


Captions on gallery thumbnails

Is there a way to add captions inside thumbnails in whole gallery view?

This isn’t Fresco, maybe you’re using some other script that builds on top of it, you’d have to add captions there.

I see, thanks and sorry for bothering with such a minor issue.
Pozdrawiam
Grzegorz Dudek
+48 *693 686 901 * |  grzegorzdudek.com
pt., 21 maj 2021 o 00:26 Nick Stakenburg @.***>
napisał(a):

This isn’t Fresco, maybe you’re using some other script that builds on top
of it, you’d have to add captions there.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/staaky/fresco/issues/12#issuecomment-845520910, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ARMC7GG572WUWMQ4LQTXVNLTOWEBPANCNFSM45HX64VA
.


[swift-3.0-branch] stdlib and runtime: a bunch of fixes for AnyHashable and id-as-Any

This PR is a set of fixes related to AnyHashable and id-as-Any work:

StdlibUnittest: change Minimal~Class types to actually be classes (these types were added for testing AnyHashable).

Added tests for AnyHashable.base.

Changed _SwiftNativeNSError to use the Hashable conformance, if the boxed value conforms to Hashable.  (rdar://problem/27574348)

Changed SwiftValue to use the Hashable conformance, if the boxed value conforms to Hashable.  (rdar://problem/27574348)

Changed AnyHashable to eagerly unbox SwiftValue.  Previously AnyHashable would consider SwiftValue to be a subclass of NSObject (which it is in practice) and return false when trying to compare an AnyHashable of a SwiftValue box to an AnyHashable of the unboxed value.

This PR is a cherry-pick of #4124.
Reviewed by: @jckarter.

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

A smoke test on macOS does the following:

Builds the compiler incrementally.
Builds the standard library only for macOS. Simulator standard libraries and
device standard libraries are not built.
lldb is not built.
The test and validation-test targets are run only for macOS. The optimized
version of these tests are not run.

A smoke test on Linux does the following:

Builds the compiler incrementally.
Builds the standard library incrementally.
lldb is built incrementally.
The swift test and validation-test targets are run. The optimized version of these
tests are not run.
lldb is tested.

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

@swift-ci Please test

macOS and all simulator testing succeed, but the CI failed later because it could not find JUnit XML files.
Trying Linux locally.

Tests pass on Linux.


[Sprint: 50] XD-3141 Avoid ClasspathResource for module config files

When the module configuration files (config/.xml, config/.properties) are loaded,
they are added as ClasspathResource. This doesn't work in case if the module was already
loaded once and the module definition is overridden by module upload command. This
requires server restart to re-initialize the classloader.
This PR suggests using UrlResource for all the ClasspathResource sources and thereby
any overridden module definitions are always re-loaded.

We should get rid of classpath: module loading entirely anyway, as currently, because

$xd.home/config is on the classpath
we support classpath:/modules/ as a registry root

then module option override directories (such as $xd.home/config/modules/sink/rabbit) are mistaken as module definitions. I a user were to upload a custom 'foo' module and put defaults in $xd.home/config/modules/sink/foo, then he can't --force update it.
See https://jira.spring.io/browse/XD-2988

@ericbottard Since module upload --force will require restart of admin/container when the changes are in ClassPathResource,  I believe we need this fix at least for GA as we wouldn't plan to get rid of classpath loading entirely at least for GA. What do you think?

I think we should consider dropping class path resource, given that it is causing the problem I described AND the one this PR is trying to cure. The only reason we have it still is for YARN if I'm not mistaken, but maybe @trisberg can come up with a different approach

Why would they put custom properties for foo in $xd.home/config/modules/sink/foo? Shouldn't that be packaged in the foo.jar?
For YARN we are now using hdfs for xd.customModule.home
and for module config: xd.module.config.location: "file:./modules-config.zip/"

$xd.home/config/modules/sink/foo is the location where the module user can put defaults (as opposed to the module author). The reporter of https://jira.spring.io/browse/XD-2988 tried exactly that (even though he actually played the two roles)

I see, makes sense now. Any chance we could add hdfs support for xd.module.config.location?

For the module configurations, I don't think we need to drawback support for classpath resource. In some way, that makes sense. The reason is that, we don't want to have conflicts when there are existing modules that use the over-ridding configurations. Since the only place where we allow override is module upload by the option --force, I believe this PR would fix the underlying issue.

Removed the unused ResourceConfigureModule changes.

The rootCause for the problem is that the underlying module classloader has a weak reference to the jar that was loaded before. Anytime the module classloader is created just for loading the properties, then it needs to be closed. In other cases, the module undeploy would take care of closing this.

Pushed the change to fix the root cause.

I created https://github.com/spring-projects/spring-xd/pull/1696 which IMHO addresses the real problem.
From a pure API contract point of view, we can't close the classloader as we don't know how the resource has been obtained and it might be used still.

From a pure API contract point of view, we can't close the classloader as we don't know how the resource has been obtained and it might be used still.

At least in this case, we know how this resource is obtained and all we needed is the properties. Moreover, the classloader we create here is a throwaway one and the resources will always be re-loaded if it is a ClassPathResource.
Your suggestion on the other PR uses URL connection for the resource was similar to the original fix to return URLResource if the instance is of ClassPathResource though now it uses URLConnection for all the resources. If we choose this way, then we don't need to set the classloader for the resource when loading properties as we make URLConnection all the time. and, the caching there is never used as it only caches if the resource uses JNLP file.
I am fine with both ways, but requires some testing on the new PR :-)

Your suggestion on the other PR uses URL connection for the resource was similar to the original fix to return URLResource

not entirely. The URLConnection is a JDK API used anytime we access streams, while URLResource is our resource abstraction over a plain URL. URLConnection is used in all cases actually

@ericbottard similar != equal ;) what I meant there is that we retrieve the resource explicitly by URL rather than downloading from Classpath. So, we don't depend on the classloader

Closing for now, as this PR moved more on the side of ClassLoader closing. Need to address this more globally, created https://jira.spring.io/browse/XD-3159 to that effect


911 PSAP GIS update (3 of 3)

Sprint 5

[ ] Layton
[ ] Price
[ ] Weber
[ ] upgrade for Uintah and Box Elder
[ ] troubleshooting TOC


Check if itlwm is loaded at startup

This PR checks at the app startup if itlwm is loaded and responding to ioctl calls. Prints an error or the itlwm version and interface.
Useful for debugging and error reporting.

Ready for review


Provide the parent pom in the Adobe Repo as well

Currently only the modules are available in https://repo.adobe.com/nexus/content/groups/public/com/adobe/cq/sample/ but not the parent com.adobe.cq.sample:we.retail:2.0.2. Can you publish the latter as well?

The parent com.adobe.cq.sample:we.retail:2.0.2 has been published on repo.adobe.com as well. Thanks for the hands up.


Transition selection toggle for ATEM mixers

Currently the ATEM module allows you to select which elements will be subject to the next transition. For a particular element (background, key{1-4}), you can set that selection to on or off.
I would like to see the option to toggle the current setting rather than explicitly setting the next transition state to on or off. This would replicate the behavior of the transition selection button on the ATEM software interface or physical control surface.
The toggle feature in companion is not a good solution here, as if a particular element is the only remaining selection, it cannot be deselected, and the state of the button and the on/off state of the companion button toggle will get out of sync.

There's some related discussion in #39 on this topic.

Duplicate of #39
This isnt forgotten, but I dont feel I had a good idea for a reliable solution yet


Custom config.js that loads windows.globalConfig

Hacky way of fixing  https://github.com/nemgrouplimited/symbol-explorer/issues/683
Atm, globalConfig is injected with vue at build time. This doesn't allow custom network configurations.
This fix uses a config.js file loaded in the index.html before the generated app.js starts and uses it.
Ideally, we should use ajax to load the config before/while Vue is bootstrapping. But the current code has the static globalConfig references. The globalConfig would be undefined before Vue can load it. In this case, the config.js is loaded before the whole app.

Hi @AnthonyLaw, you are totally right.
Using Axios to load the config.json when Vue is booting is the way to go. I hack my way in by loading a public config.js file before the application. I'm in a rush to get explorer up and running in bootstrap and I didn't have time to do it properly.
Atm, the globalConfig is used statically, the explorer needs a little bit of refactoring so the different components load the config object from the Vue store or it's passed as a param.
This PR does the job and it would be nice to have images configurable now, but if we can do it the right way, let's merge a better one.

Hi @AnthonyLaw, you are totally right.
Using Axios to load the config.json when Vue is booting is the way to go. I hack my way in by loading a public config.js file before the application. I'm in a rush to get explorer up and running in bootstrap and I didn't have time to do it properly.
Atm, the globalConfig is used statically, the explorer needs a little bit of refactoring so the different components load the config object from the Vue store or it's passed as a param.
This PR does the job and it would be nice to have these configurable images automatically when merging to main. But if we can do it the right way, let's merge a better one.

No problem, let discuss in the upcoming short call and decide later 👍


pacx auth create examples?

It would be nice to have some examples for commands. I am trying to authenticate with my environment using 'pacx auth create' but encounter several errors.
Examples:

Test with AuthType Office365: pacx auth create --name 'name' --conn<EMAIL_ADDRESS>Result: "AuthType is invalid.  Please see Microsoft.PowerPlatform.Dataverse.Client.AuthenticationType for supported authentication types. (Parameter 'AuthType')" - probably related to Deprecation of Office365 authentication type?
Test with OAuth: pacx auth create --name 'name' --conn<EMAIL_ADDRESS>Result: " Failed to connect to Dataverse
Only loopback redirect uri is supported, but app://<GUID>was found. Configure http://localhost or http://localhost:port both during app registration and when you create the PublicClientApplication object. See https://aka.ms/msal-net-os-browser for details"

The 'normal' pac auth create (not pacx) command without parameters displays the login pop-up box. Could this be achieved for pacx as well?

Hi kevin,
thanks for yous ask.
pacx auth create command accepts any valid dataverse connection strings. Dataverse connection string format is fully documented here. I don't think that I will replicate the contents here, to avoid misalignment, but I could add a pointer to the official MS document in order to streamline usage of the tool.
If you want to open the standard login popup box just type a connection string like this one:
AuthType=OAuth;Url=https://<myorg>.crm4.dynamics.com;RedirectUri=http://localhost

But I haven't found a way (yet) to save the credentials you provide in that login windows as pac or XrmToolBox does... Any pull request in that direction is appreciated!
Let me know if this fits your needs.

I have a similar issue: thought I would test against my dev tenant:
 pacx auth create --name "Daniel 2" --conn<EMAIL_ADDRESS>Object reference not set to an instance of an object.

Hi @westerdaled,
yes please, any info would be helpful.

Ok , can you message me on<EMAIL_ADDRESS>and I will resend the actual error messages .

Here is the current error in the latest version
pacx auth create --name "Daniel  2" --conn AuthType=OAuth;Url=https://org041XXXX.crm11.dynamics.com;RedirectUri=http://localhost

Version 1.2024.6.138
Online documentation: https://github.com/neronotte/Greg.Xrm.Command/wiki
Feedback, Suggestions, Issues: https://github.com/neronotte/Greg.Xrm.Command/discussions

Object reference not set to an instance of an object.

Url=https://org041dXXXXX.crm11.dynamics.com: The term 'Url=https://org041XXXX.crm11.dynamics.com' is not recognized as a name of a cmdlet, function, script file, or executable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
RedirectUri=http://localhost: The term 'RedirectUri=http://localhost' is not recognized as a name of a cmdlet, function, script file, or executable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

Ok  I thought I would use the alternate syntax.  I used  pac admin create-service-principal - to create my service principal to connect my personal environment
pacx auth create --name "Daniel Personal 2" --conn AuthType=ClientSecret;Url=https://org041XXXXX.crm11.dynamics.co
m;ClientId=myAppId;ClientSecret=mySecretValue

Version 1.2024.6.139
Online documentation: https://github.com/neronotte/Greg.Xrm.Command/wiki
Feedback, Suggestions, Issues: https://github.com/neronotte/Greg.Xrm.Command/discussions
Object reference not set to an instance of an object.
Url=https://orgXXXX.crm11.dynamics.com: The term 'Url=https://orgXXXX.crm11.dynamics.com' is not recognized as a name of a cmdlet, function, script file, or executable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
ClientId=????: The term 'ClientId=????' is not recognized as a name of a cmdlet, function, script file, or executable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

Yes, it seems that the semicolon breaks the flow for the terminal, thus everything after is assumed to be another instruction.
I'm currently investigating the "Object reference.." error, it should not be thrown in any case.
I will also add a note in the docs to clarify the importance of double quotes in the connection string


Add Python 3.11 to test matrix

Codecov Report
Base: 95.21% // Head: 95.21% // No change to project coverage :thumbsup:

Coverage data is based on head (0e1866f) compared to base (4bfba7e).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #18   +/-   ##
=======================================
  Coverage   95.21%   95.21%
=======================================
  Files          10       10
  Lines         439      439
=======================================
  Hits          418      418
  Misses         21       21

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Feat: support export nervosdao csv

resolve: https://github.com/Magickbase/ckb-explorer-public-issues/issues/592

https://ckb-explorer-frontend-in-magickbase-repo-q0n2e3iyp-magickbase.vercel.app/nervosdao?page=1

https://ckb-explorer-frontend-in-magickbase-repo-q0n2e3iyp-magickbase.vercel.app/nervosdao/transaction/export?end-date=2024-05-27&start-date=2024-05-26&tab=date
exported-nervosdao-txs-2024-05-26-2024-05-27 (3).csv

Export data according to the time range. It is expected that 0xaa6d0fad0e0943c60e748e4b9b8fdf86fc6349539ac0c782fc86299e6f0ca2c9 is the last data hash, but it is actually 0xdb6c74ff1416082bc7c5022b0fdf17b1b1c53 fb5251af0ea540d51212f640621.

https://ckb-explorer-frontend-in-magickbase-repo-q0n2e3iyp-magickbase.vercel.app/nervosdao?page=1
https://ckb-explorer-frontend-in-magickbase-repo-q0n2e3iyp-magickbase.vercel.app/nervosdao/transaction/export?end-date=2024-05-27&start-date=2024-05-26&tab=date exported-nervosdao-txs-2024-05-26-2024-05-27 (3).csv  1、Export data according to the time range. It is expected that 0xaa6d0fad0e0943c60e748e4b9b8fdf86fc6349539ac0c782fc86299e6f0ca2c9 is the last data hash, but it is actually 0xdb6c74ff1416082bc7c5022b0fdf17b1b1c53 fb5251af0ea540d51212f640621.
https://ckb-explorer-frontend-in-magickbase-repo-q0n2e3iyp-magi

https://ckb-explorer-frontend-in-magickbase-repo-q0n2e3iyp-magickbase.vercel.app/nervosdao/transaction/export?end-date=2024-05-29&start-date=2024-05-28&tab=date
2、The data after the decimal point of Amount data needs to be displayed accurately except 0.
https://ckb-explorer-frontend-in-magickbase-repo-q0n2e3iyp-magickbase.vercel.app/zh/nervosdao?page=3

3、Compare `0x13cb480446a7a0309032dc6a306b66e7cb1ffce92ae21b829be1aaa0b81f55d1` data, `0x1b03163475d35fd090176ff8f336dc1a03908da142299b18fe6ca71 c599addc5` and `0x549c78c4f3ec4951551438e33a98b647b2b6f48cec24bb07cac99ae7b754dcf9` are also `Deposit` methods, but they will have an `Amount` of 0.00000001 exception information.
https://ckb-explorer-frontend-in-magickbase-repo-q0n2e3iyp-magickbase.vercel.app/nervosdao/transaction/export?end-date=2024-05-27&start-date=2022-07-05&tab=date exported-nervosdao-txs-2022-07-05-2024-05-27.csv
4、The actual upper limit of exported data is greater than the 5000 in the prompt message.
https://ckb-explorer-frontend-in-magickbase-repo-q0n2e3iyp-magickbase.vercel.app/zh/nervosdao/depositor/export?end-date=2024-05-28&start-date=2010-01-01&tab=date exported-nervosdao-depositors-2010-01-01-2024-05-28.csv
5、The exported file is blank. At the same time, the table format must be in accordance with the prd.
https://ckb-explorer-frontend-in-magickbase-repo-q0n2e3iyp-magickbase.vercel.app/zh/nervosdao/depositor/export?end-date=2024-05-28&from-height=13397400&start-date=2020-01-01&tab=height&to-height=13397400
6、Tip 500.

Currently the table data is provided by the back end, @rabbitz  can you help with that?

Tip 500

The request api is:
https://ckb-explorer-api-staging.magickbase.com/api/v1/dao_depositors/download_csv?start_date=1577808000000&end_date=1717054130645&start_number=13397400&end_number=13397400
The date and block number parameters appear here together，The request is abnormal.

fixed the issues 1, 2, and 3 . PR: https://github.com/nervosnetwork/ckb-explorer/pull/1901

1、Export data according to the time range. It is expected that 0xaa6d0fad0e0943c60e748e4b9b8fdf86fc6349539ac0c782fc86299e6f0ca2c9 is the last data hash, but it is actually 0xdb6c74ff1416082bc7c5022b0fdf17b1b1c53 fb5251af0ea540d51212f640621.

1、There is a problem with the data submitted by the front end. The normal range should be [2024-5-26 00:00:00,2024-5-27 23:59:59].

https://ckb-explorer-frontend-in-magickbase-repo-q0n2e3iyp-magickbase.vercel.app/nervosdao/transaction/export?end-date=2024-05-27&start-date=2022-07-05&tab=date
exported-nervosdao-txs-2022-07-05-2024-05-27.csv

4、The actual upper limit of exported data is greater than the 5000 in the prompt message.

2、The number of 5,000 here refers to the 5,000 transactions, but how many DAO records may be included in a transaction, so the prompt information needs to be modified to The earliest 5,000 transactions within the selected range will be exported, the number of records may exceed 5,000 because a single transaction can include multiple records.

3、An exception occurred in the request to get data after switching input information between `date` and `block nubmer`.
@PainterPuppets

fixed

https://ckb-explorer-frontend-in-magickbase-repo-amp0tefjo-magickbase.vercel.app/en/nervosdao/depositor/export?end-date=2024-06-03&from-height=1&start-date=2020-01-01&tab=date&to-height=5000

Api prompt 504.

https://ckb-explorer-frontend-in-magickbase-repo-amp0tefjo-magickbase.vercel.app/en/nervosdao/depositor/export?end-date=2024-06-03&from-height=1&start-date=2020-01-01&tab=date&to-height=5000  Api prompt 504.

I've got the same thing here, all other api seem to be fine, @rabbitz can you help look at it?

https://ckb-explorer-frontend-in-magickbase-repo-amp0tefjo-magickbase.vercel.app/en/nervosdao/depositor/export?end-date=2024-06-03&from-height=1&start-date=2020-01-01&tab=date&to-height=5000  Api prompt 504.

I've got the same thing here, all other api seem to be fine, @rabbitz can you help look at it?

The efficiency of the SQL query has some issues, and it is currently being optimized.

1、Export data according to the time range. It is expected that 0xaa6d0fad0e0943c60e748e4b9b8fdf86fc6349539ac0c782fc86299e6f0ca2c9 is the last data hash, but it is actually 0xdb6c74ff1416082bc7c5022b0fdf17b1b1c53 fb5251af0ea540d51212f640621.

1、There is a problem with the data submitted by the front end. The normal range should be [2024-5-26 00:00:00,2024-5-27 23:59:59].

https://ckb-explorer-frontend-in-magickbase-repo-q0n2e3iyp-magickbase.vercel.app/nervosdao/transaction/export?end-date=2024-05-27&start-date=2022-07-05&tab=date
exported-nervosdao-txs-2022-07-05-2024-05-27.csv

4、The actual upper limit of exported data is greater than the 5000 in the prompt message.

2、The number of 5,000 here refers to the 5,000 transactions, but how many DAO records may be included in a transaction, so the prompt information needs to be modified to The earliest 5,000 transactions within the selected range will be exported, the number of records may exceed 5,000 because a single transaction can include multiple records.

3、An exception occurred in the request to get data after switching input information between date and block nubmer.
@PainterPuppets

fixed

https://ckb-explorer-frontend-in-magickbase-repo-amp0tefjo-magickbase.vercel.app/nervosdao/transaction/export?end-date=2024-05-27&from-height=1&start-date=2024-05-26&tab=date&to-height=5000
https://ckb-explorer-frontend-in-magickbase-repo-amp0tefjo-magickbase.vercel.app/en/nervosdao/depositor/export?end-date=2024-06-03&from-height=5001&start-date=2020-01-01&tab=date&to-height=10000
After using Block Number to export the data, switch to date and find that the url also contains to-height information. This information should only appear when Block Number is used.  @PainterPuppets

should

fixed

1、Export data according to the time range. It is expected that 0xaa6d0fad0e0943c60e748e4b9b8fdf86fc6349539ac0c782fc86299e6f0ca2c9 is the last data hash, but it is actually 0xdb6c74ff1416082bc7c5022b0fdf17b1b1c53 fb5251af0ea540d51212f640621.

1、There is a problem with the data submitted by the front end. The normal range should be [2024-5-26 00:00:00,2024-5-27 23:59:59].

https://ckb-explorer-frontend-in-magickbase-repo-q0n2e3iyp-magickbase.vercel.app/nervosdao/transaction/export?end-date=2024-05-27&start-date=2022-07-05&tab=date
exported-nervosdao-txs-2022-07-05-2024-05-27.csv

4、The actual upper limit of exported data is greater than the 5000 in the prompt message.

2、The number of 5,000 here refers to the 5,000 transactions, but how many DAO records may be included in a transaction, so the prompt information needs to be modified to The earliest 5,000 transactions within the selected range will be exported, the number of records may exceed 5,000 because a single transaction can include multiple records.

3、An exception occurred in the request to get data after switching input information between date and block nubmer.
@PainterPuppets

fixed

https://ckb-explorer-frontend-in-magickbase-repo-amp0tefjo-magickbase.vercel.app/nervosdao/transaction/export?end-date=2024-05-27&from-height=1&start-date=2024-05-26&tab=date&to-height=5000
https://ckb-explorer-frontend-in-magickbase-repo-amp0tefjo-magickbase.vercel.app/en/nervosdao/depositor/export?end-date=2024-06-03&from-height=5001&start-date=2020-01-01&tab=date&to-height=10000
After using Block Number to export the data, switch to date and find that the url also contains to-height information. This information should only appear when Block Number is used. @PainterPuppets

fixed

https://ckb-explorer-frontend-in-magickbase-repo-amp0tefjo-magickbase.vercel.app/en/nervosdao/depositor/export?end-date=2024-06-03&from-height=1&start-date=2020-01-01&tab=date&to-height=5000  Api prompt 504.

I've got the same thing here, all other api seem to be fine, @rabbitz can you help look at it?

The efficiency of the SQL query has some issues, and it is currently being optimized.

https://ckb-explorer-frontend-in-magickbase-repo-8j78rfifq-magickbase.vercel.app/en/nervosdao/depositor/export?end-date=2024-05-27&start-date=2024-05-26&tab=date
Depositors data export interface problem still exists.
At the same time, has the data display logic in /Withdraw request/Withdraw Finalization exported data by Depositors been confirmed?  @Sven-TBD

ckb-explorer-frontend-in-magickbase-repo-amp0tefjo-magickbase.vercel.app/en/nervosdao/depositor/export?end-date=2024-06-03&from-height=1&start-date=2020-01-01&tab=date&to-height=5000  Api prompt 504.

I've got the same thing here, all other api seem to be fine, @rabbitz can you help look at it?

The efficiency of the SQL query has some issues, and it is currently being optimized.

ckb-explorer-frontend-in-magickbase-repo-8j78rfifq-magickbase.vercel.app/en/nervosdao/depositor/export?end-date=2024-05-27&start-date=2024-05-26&tab=date
Depositors data export interface problem still exists.
At the same time, has the data display logic in /Withdraw request/Withdraw Finalization exported data by Depositors been confirmed? @Sven-TBD

Please check

Request URL:
https://ckb-explorer-api-staging.magickbase.com/api/v1/dao_depositors/download_csv?start_date=1719763200000&end_date=1719849599999

Request URL:
https://ckb-explorer-api-staging.magickbase.com/api/v1/dao_depositors/download_csv?start_number=13757249&end_number=13761121

The api prompts an exception.

ckb-explorer-frontend-in-magickbase-repo-amp0tefjo-magickbase.vercel.app/en/nervosdao/depositor/export?end-date=2024-06-03&from-height=1&start-date=2020-01-01&tab=date&to-height=5000  Api prompt 504.

I've got the same thing here, all other api seem to be fine, @rabbitz can you help look at it?

The efficiency of the SQL query has some issues, and it is currently being optimized.

ckb-explorer-frontend-in-magickbase-repo-8j78rfifq-magickbase.vercel.app/en/nervosdao/depositor/export?end-date=2024-05-27&start-date=2024-05-26&tab=date
Depositors data export interface problem still exists.
At the same time, has the data display logic in /Withdraw request/Withdraw Finalization exported data by Depositors been confirmed? @Sven-TBD

Please check @Sven-TBD

@FrederLu Confirmed with Chen, I have no question on the export function and we can put it on the Depositors Tab. If you have any questions , you can still ask me about it. When this feature pass the test, @PainterPuppets could merge it.

refactor pr:

Request URL: https://ckb-explorer-api-staging.magickbase.com/api/v1/dao_depositors/download_csv?start_date=1719763200000&end_date=1719849599999
Request URL: https://ckb-explorer-api-staging.magickbase.com/api/v1/dao_depositors/download_csv?start_number=13757249&end_number=13761121
The api prompts an exception.

refactor pr: https://github.com/nervosnetwork/ckb-explorer/pull/2037

refactor pr: nervosnetwork/ckb-explorer#2037 @FrederLu

https://ckb-explorer-frontend-in-magickbase-repo-njk190r6m-magickbase.vercel.app/en/nervosdao/depositor/export?from-height=13838820&tab=height&to-height=13839319

According to this export, these two data are incorrect, the calculation is 103749 and 535.


Environment Variable OLLAMA_API_BASE Not Taking Effect When Starting R2R Service using Docker

Description:
I already installed latest version of r2r. When using the r2r serve --docker --config-path=my_config.toml command to start the R2R service, I attempted to configure the remote OLLAMA service by setting the environment variable OLLAMA_API_BASE. However, I found that this environment variable did not take effect.
Reproduction Steps:
1.Set the environment variable OLLAMA_API_BASE:
export OLLAMA_API_BASE=http://example.com:11434

2.Start the R2R service:
r2r serve --docker --config-path=my_config.toml

Expected Behavior:
The environment variable OLLAMA_API_BASE should take effect when the R2R service starts and configure the remote OLLAMA service.
Actual Behavior:
The environment variable OLLAMA_API_BASE does not seem to be read or take effect, resulting in the R2R service being unable to connect correctly to the remote OLLAMA service.

fixed


Set selected item programmatically

Hello and thank you for this library!
I'm trying to implement a fullscreen image viewer when tapping the slideshow and so far I;ve got it working, but I need to set the selected item in the slideshow programmatically, so when the user changes the page in my fullscreen image view, the slide show changes as well. Is this possible?
Thank you

Hello and thank you for this library!
I'm trying to implement a fullscreen image viewer when tapping the slideshow and so far I;ve got it working, but I need to set the selected item in the slideshow programmatically, so when the user changes the page in my fullscreen image view, the slide show changes as well. Is this possible?
Thank you

Hi , updated , 1.3.1 and use         sliderView.setCurrentPagePosition(int);
method to change current page programmatically, happy coding.

Perfect, thank you very much!


Apache Kafka Guide for Spring Boot

I would like to contribute for Apache Kafka messaging service for Spring Boot

It would be a sample application and explain the broker, producer, consumer and topic

@MihSawant Are you working on this? I would also love to contribute here, let me know we we can do it toghether.

Closing as a duplicate of #72


Do not send message from CONTENT_SCRIPT to CONTENT_SCRIPT

Changes done in the PR:

Does not send message from CONTENT_SCRIPT to CONTENT_SCRIPT anymore. This is because, the variable safari.application.activeBrowserWindow.tabs is not defined when the context of the script is the CONTENT_SCRIPT.

@smcgregor Should the messaging to CONTENT_SCRIPT from a CONTENT_SCRIPT be supported? If so, I will update the PR accordingly.

I don't see a reason to prohibit it if an application asks to send it.

@smcgregor Changes made currently:

The getContextName() function has been fixed now.
From the PRIVLY_APPLICATION pages, message cannot be sent to the CONTENT_SCRIPT. This is because from the PRIVLY_APPLICATION pages context, only the safari variables available are, safari.extension.baseURI and safari.self.tab, none of which allow to send message to the CONTENT_SCRIPT. This can be seen,

Then how does the iframe resize? Iframe resizing is typically a message from the privly application to the content script.

@smcgregor Can you point me to the code in privly-applications where the iframe resize message is sent? That will help me update this PR.

https://github.com/privly/privly-applications/blob/master/shared/javascripts/host_page_integration.js#L25

@smcgregor Thanks for that. I have updated the code and renamed the PR.
Final changes made in the PR:

For sending message from PRIVLY_APPLICATION to CONTENT_SCRIPT, use parent.postMessage
For sending message to CONTENT_SCRIPT from other context, use safari.application.activeBrowserWindow.tabs
Fixed the getContextName() function to identify PRIVLY_APPLICATION appropriately

Does parent.postMessage get captured by the content script's scripting environment or the host page's scripting environment? Is it visible to both scripting environments?

The parent.postMessage is visible to both the scripting environments.

OK, that is fine since it matches with our threat model for content scripts, but this message interface can't be used for any privileged information.

Sounds good then. I think the PR can be merged now.

@smcgregor Updated the PR.


Requiring Upload CGM toggle could exclude some users

The Service derives from a remote data service class which is designed to upload data to a remote locations. Such as the nightscout service, which also derives from that class. However, services of that class won't receive updates from CGMs if the upload toggle in the CGM setting is "off". This will become a bottle neck for those users that

want to use the garmin service but
don't want to upload their CGM data (e.g. to nightscout).

Solution is yet unclear. Potentially, all remote services could choose which data to upload.
How often this case exists is unclear.

Updated the instructions.
https://github.com/janvv/GarminService/commit/8918e2a7e00480b7e0a049b914be335575a7b9df

https://github.com/janvv/GarminService/commit/8918e2a7e00480b7e0a049b914be335575a7b9df


REQ: support alpine linux

Hi, when running MariaDB4j on a alpine container, we met error:

./mysqld: /usr/lib/libstdc++.so.6: no version information available (required by ./mysqld)

If we must depend gnu libstdc++, could we build a alpine version (based on musl libc) mysqld?

@gzm55 from what I understand, this is more of request for the core MariaDB binaries than the MariaDB4j Java wrapper (this project) - would you agree? But if you would to make a contribution to core MariaDB which works on Alpine Linux and then makea  JAR of that for use with MariaDB4j, I'm all for it... ;-) Contributions welcome - I'd certainly look at and merge anything like this. (But I myself wouldn't have the time to do this for you).

OK, you are right. MariaDB has alpine version: https://pkgs.alpinelinux.org/package/edge/main/x86_64/mariadb

@gzm55 OK, then I think what we're really saying here is that you would like to contribute new binaries packaging (in the DB/ directory; like a mariaDB4j-db-alpine64-10.1.24-r0) to MariaDB4j?
Or, alternatively, why don't you just pre-install that Alpine MariaDB native package into your container? If you wanted to, you could still use MariaDB4j just to start and stop isolated fresh DBs from your Java tests code... this isn't how most typically use this library today - but you absolutely could, probably with a little bit of tweaks - I'd love contributions on that front, see https://github.com/vorburger/MariaDB4j/issues/59.

@vorburger
solution A: add binary in DB/, will result more predictable action on the machine, so most cases we prefer this one. but it is difficult to distinguish between linux and alpine at run-time when generate binary path.
solution B: described in #59 , could be very convenient in developing phase.
solution C: such as #59, maybe we can further fallback to local tcp port mode, which can be useful when the docker connecting the service from other container.

@BenDave to carry over from your issue #80, how about you just install the MariaDB binary Alpine package which @gzm55 mentioned above in your container, and then use the MariaDB4j API mentioned in #59 (which FYI @gzm55 actually already works...) to control it - would that work for you?
[Standard Footer] PS: Shameless plug, consider https://www.patreon.com/vorburger to support MariaDB4j.


Dark mode

Hello :)
Dark mode is not supported. So we don't see the toast. Just the text.
What would be the solution?
Thanks you

By using the darkGrey color the result is close to android and looks good in dark and light.
But I think it would be nice if by default it were supported :)
var style = ToastStyle()
style.backgroundColor = .darkGray


Lethal Level Loader 1.2.3 Patchnote callout

I can't try it just yet, but lethal level loader's newest patch stated:
Changed ExtendedDungeonFlow.dungeonEvents to ExtendedDungeonFlow.DungeonEvents (Sorry to PoolRooms which likely needs an update).
Given it's a direct callout to poolrooms, I thought I'd open an issue to at least call attention to it.

Thanks for the heads up, new version pushed :)


Release/5.6.0

@5.6.0
Features

[#348] Add Makefile to build commands by @tyrro in https://github.com/nimblehq/rails-templates/pull/359
[gh384] Add PostCSS autoprefixer by @sanG-github in https://github.com/nimblehq/rails-templates/pull/403
[gh385] Minify the CSS and JS for production build by @sanG-github in https://github.com/nimblehq/rails-templates/pull/405
[#409] Upgrate Postges to 15.2 by @sanG-github in https://github.com/nimblehq/rails-templates/pull/412

Bugs

[#404] Fix: Loaderror: cannot load such file -- sassc when generating the project from the template by @sanG-github in https://github.com/nimblehq/rails-templates/pull/406

New Contributors

@tyrro made their first contribution in https://github.com/nimblehq/rails-templates/pull/359

Full Changelog: https://github.com/nimblehq/rails-templates/compare/5.5.0...5.6.0

The PR title should be Release - 5.6.0 🙏


Paint of the Grid with Tabsets

Hellow everyone,
i try to display a site in tabs and tabsets.
I have one or more grids on every tab and the grids simply show nothing.
Know, when i move the window (resize it), or open the developertools (Chrome), the content of the grid appears. When i change the tab, the game starts again:
nothing is there -> resize -> data appears -> tab change -> nothing is there ...
I made a little plunker to show the problem:
http://plnkr.co/edit/nlyA9z?p=preview
I hope I am just to dump to do it properly.
That is a minimalistic version, the cellTemplate and the structure of the html is because of the project I am working on.

You need to set a width/height on the grids
Working plunker: http://plnkr.co/edit/0Nljj5zbKjUQPv80uKcj
More info on the issue http://ui-grid.info/docs/#/tutorial/108_hidden_grids

That solved my problem, thank you really much!


Feature/338

Service reference save to elastic search test success.
Entry service save to elastic search test success.

Coverage increased (+0.2%) to 17.851% when pulling 82f79a734d4f4da7813337c87a19504e0ccc2642 on feature/338 into 9559c75c580ac180102001fe9599b09795a9c62e on master.

Coverage increased (+0.2%) to 17.907% when pulling eb72cb04bab47e65113b53c7d212d786081d3e28 on feature/338 into 9559c75c580ac180102001fe9599b09795a9c62e on master.

Coverage increased (+0.2%) to 17.92% when pulling a7c11a1baf9a2bd0e25295d12cedac872bd2a0d0 on feature/338 into 9559c75c580ac180102001fe9599b09795a9c62e on master.


411 Setup documentation seems outdated for crontab section

Hi,
In the initial setup guide, the cronjob configuration seems outdated.
I've followed the guide today, and got the system up and running, but cronjob didn't work
# Add the following line into your crontab.
$ crontab -u USER -e
 /var/www/411/bin/cron.php > /dev/null 2>&1 && /var/www/411/bin/worker.php > /dev/null 2>&1

Running the CRON itself produces the following error
[+] Scheduler:<PHONE_NUMBER>
except [NONE] Errno 2: "pcntl_exec(): Error has occurred: (errno 13) Permission denied" at [/var/www/411/phplib/Scheduler.php:44] 0:[pcntl_exec() called at [/var/www/411/phplib/Scheduler.php:44]] 1:[FOO\Scheduler->process() called at [/var/www/411/bin/cron.php:47]]
err [411_Scheduler] Scheduler error site:[1] ret:[256]

I had to change to this, which seems to make the cron functionality work
* * * * * php /var/www/411/bin/cron.php --backfill --date=/$(date +\%Y-\%m-\%d) --site=1
* * * * * php /var/www/411/bin/worker.php --site=1**

am i completely off, or is this correct?

Still seems to break after added +X
root@ubuntu-template:/var/www/411/bin# chmod +x cron.php
root@ubuntu-template:/var/www/411/bin# chmod +x worker.php
root@ubuntu-template:/var/www/411/bin# php cron.php

[+] Scheduler:<PHONE_NUMBER>
except [NONE] Errno 2: "pcntl_exec(): Error has occurred: (errno 2) No such file or directory" at [/var/www/411/phplib/Scheduler.php:44] 0:[pcntl_exec() called at [/var/www/411/phplib/Scheduler.php:44]] 1:[FOO\Scheduler->process() called at [/var/www/411/bin/cron.php:47]]
err [411_Scheduler] Scheduler error site:[1] ret:[256]

root@ubuntu-template:/var/www/411/bin# ls -lh
total 36K
-rw-r--r-- 1 www-data www-data  597 Sep 28 12:44 backfill.php
-rw-r--r-- 1 www-data www-data  881 Sep 28 12:44 create_site.php
-rw-r--r-- 1 www-data www-data  879 Sep 28 12:44 create_user.php
-rwxr-xr-x 1 www-data www-data 1.1K Sep 28 12:44 cron.php
-rw-r--r-- 1 www-data www-data  698 Sep 28 12:44 db_cleanup.php
-rw-r--r-- 1 www-data www-data  769 Sep 28 12:44 es_sync.php
-rw-r--r-- 1 www-data www-data 1.3K Sep 28 12:44 migration.php
-rw-r--r-- 1 www-data www-data  281 Sep 28 12:44 shell.php
-rwxr-xr-x 1 www-data www-data  776 Sep 28 12:44 worker.php
root@ubuntu-template:/var/www/411/bin#

My own --date parameter did not work, cron.php will default if --date is not provided. I removed it

Hm, does the user the cron is running as have read access to that directory?

Running as root, and i just run chmod 777 recursively on /var/www/411/
Errormessages has changed

root@ubuntu-template:/var/www/411# php bin/cron.php
the time:<PHONE_NUMBER>[+] Scheduler:<PHONE_NUMBER>
except [NONE] Errno 2: "pcntl_exec(): Error has occurred: (errno 2) No such file or directory" at [/var/www/411/phplib/Scheduler.php:44] 0:[pcntl_exec() called at [/var/www/411/phplib/Scheduler.php:44]] 1:[FOO\Scheduler->process() called at [/var/www/411/bin/cron.php:48]]
err [411_Scheduler] Scheduler error site:[1] ret:[256]

Scheduler.php:39 defines a variable called $cmd. Can you check whether this contains the full path of the cron.php script?

           // This is the child. Launch the scheduler for this site.
            if($pid === 0) {
                $cmd = sprintf('%s/bin/cron.php', BASE_DIR);
                echo "----".$cmd."----";
                $args = ["--date=$date", "--site={$site['id']}"];
                if($backfill) {
                    $args[] = '--backfill';

This was the output. Looks good to me
----/var/www/411/bin/cron.php----

I'm out of ideas at this point... As a workaround, you could try invoking cron.php --site=1 instead of just cron.php.

Yep, that seems to work... lets leave it at that.


Invite status feedback

FC clicks the button, invite request is pending an ESI response

ESI has accepted the invite request - the invite is sent to the pilot:

Note: while the invite is sent to the user at this point it can still fail because:

The pilot is in a fleet (immediate failure)
The invite times out
The pilot triggers a session change
Each of these failures is not reported via ESI so the waitlist state cannot change; however, a small pop error similar to natural phenomena displays on the FCs screen

ESI has rejected the invite because: the user is offline, has a CSPA charge, has blocked the fleet boss or the fleet member limit (fleet cap) has been reached.

Currently, FCs are given limited feedback about the status of their invites. An FC is required to try and remember who they've invited. If the invite fails they are given a generic error message but it doesn't state which invites failed. The purpose of this PR is to provide a visual indicator to FCs which fits they've tried to invite, which fits have received an invite and which invites were rejected by ESI.
Please use this draft PR to provide feedback or suggest other ways this could be implemented.


Installing cupy 7.8.0 from PyPi using pip throws a RuntimeError: Missing file: cupy/cuda/cutensor.cpp

I am trying to install cupy from PyPi using pip install cupy and getting an error.

Installation log: https://dpaste.com/EJ8BKNM3X
/proc/cpuinfo: https://dpaste.com/44Z7M7JH5
CUDA deviceQuery: https://dpaste.com/HEZTLAP26

I have all necessary libraries and tools:
      cuda      : Yes (version 11000)
      cusolver  : Yes
      cudnn     : Yes (version 8002)
      nccl      : Yes (version 2703)
      nvtx      : Yes
      thrust    : Yes
      cutensor  : Yes (version 1200)
      cub       : Yes

... and Cython v0.29.21.
$ cython -V
Cython version 0.29.21
$ python -V
Python 3.8.5
$ pip -V
pip 20.1.1 from /usr/lib/python3.8/site-packages/pip (python 3.8)
$ python -c "import setuptools as st; print(st.__version__)"
50.3.0

For the time being you can install the cupy-cuda110 wheel, downloadable from https://github.com/cupy/cupy/releases/tag/v7.8.0.

We've released a post-release for this issue.
https://pypi.org/project/cupy/7.8.0.post1/
Thanks for reporting!


Large the JSON input text size

The DAB JSON input is too small to input.

The edit can't adaptive height, We can only define a right height.But this edit supports contract, can alleviate height problem


Travis CI: Add Python 3.9 and 3.10-dev to the testing

Thank you! Would you mind also adding 3.9-dev as an allowed failure? I don't want to fail builds on pre-release python versions

https://travis-ci.org/github/jsocol/pystatsd/pull_requests


Bring back separate lint/build

Move "fast" (sub-30s) linters to their own "lint" workflow again
Make the "rust" workflow run after "lint" completes (i.e. don't do 6m tests if rustfmt failed in 16s).
Move the discord notifier to a "notify" workflow that runs when "lint" or "rust" completes, but only notifies discord if they fail.

Current dependencies on/for this PR:

main

PR #94

PR #95

PR #96

PR #97

PR #98   👈

This comment was auto-generated by Graphite.


never rest type not assignable to complex rest type

TypeScript Version:  3.5.1

Search Terms:
Code
function foo<T extends any[]>() {
    const fn1: (x: string, ...rest: T) => unknown = (x, ..._) => x;
    const fn2: (...args: never) => unknown = fn1 // error
}

Expected behavior:
Assignment of fn1 to fn2 should pass.
Actual behavior:
Assignment of fn1 to fn2 should fails.
Playground Link: link
Related Issues:
#33457

The goal here would be to make all signatures structurally assignable to (...args: never) => unknown.

@ahejlsberg this seems reasonable to me. Thoughts?

I did some experiments in the playground.  This seems to break only when the rest parameter is generic.  Use a concrete type and all is well:

Playground link

Interestingly, [ string, ...T ] (where T is generic) isn't even legal and produces the error A rest element type must be an array type.  This seems like a pretty big clue: the assignment of fn1 to fn2 may only be failing because it's unable to construct a tuple type for the parameter list of (x: string, ...rest: T) => unknown.

This is where it fails in checking:
const sourceCount = getParameterCount(source);
const sourceRestType = getNonArrayRestType(source);
const targetRestType = getNonArrayRestType(target);
if (sourceRestType && targetRestType && sourceCount !== targetCount) {
    // We're not able to relate misaligned complex rest parameters
    return Ternary.False;
}

@jack-williams That doesn't seem to explain why it succeeds for the non-generic case.  What is the exact definition of "complex" here?

What is the exact definition of "complex" here?

A rest type where you don't know the bounds, or whether it is unbounded---I would guess.

FWIW. I think the fix is basically:
function getNonArrayRestType(signature: Signature) {
    const restType = getEffectiveRestType(signature);
    return restType && !isArrayType(restType) && !isTypeAny(restType) && !isTypeNever(restType) ? restType : undefined;
}

+ isTypeNever.

Repro taken from TS' source code,
https://github.com/microsoft/TypeScript/blob/c447ebc59c49a1b071c099ce6055e94517973eab/src/compiler/core.ts#L1460-L1464
/**
 * Safer version of `Function` which should not be called.
 * Every function should be assignable to this, but this should not be assignable to every function.
 */
export type AnyFunction = (...args: never[]) => void;

function foo<T extends any[]>() {
    const fn1: (...rest: T) => void = (..._) => {};
    const fn2: AnyFunction = fn1 // error
}

Playground


Add topic to cloud event envelope for pub sub

Description
This is a PoC for adding the published topic to Cloud Event.  This change also changes the properties of the NewCloudEventsEnvelope in dapr/components-contrib (https://github.com/dapr/components-contrib/pull/396).
Issue reference
Please reference the issue this PR will close: #1786
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[x] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

@allymparker the PR in contrib is now merged.
Can you run go get -u github.com/dapr/components-contrib@master and push the changes?

Can you run go get -u github.com/dapr/components-contrib@master and push the changes?

Done.

/ok-to-test


Adding in FSDP Docs

What does this PR do?
Adding FSDP docs to composer
Before submitting

[ ] Have you read the contributor guidelines?
[ ] Is this change a documentation change or typo fix? If so, skip the rest of this checklist.
[ ] Was this change discussed/approved in a GitHub issue first? It is much more likely to be merged if so.
[ ] Did you update any related docs and document your change?
[ ] Did you update any related tests and add any new tests related to your change? (see testing)
[ ] Did you run the tests locally to make sure they pass?
[ ] Did you run pre-commit on your change? (see the pre-commit section of prerequisites)

@bcui19 here's a pseudocode snippet that can be used as an example for FSDP:
import torch
from composer import Trainer

class Block (nn.Module):
    ...

class Model(nn.Module):
    def __init__(self, n_layers):
        super().__init__()
        self.blocks = nn.ModuleList([
            Block(...) for _ in range(n_layers)
        ]),
        self.head = torch.nn.Linear(...)

    def forward(self, inputs):
        ...

    # FSDP Wrap Function
    def fsdp_wrap_fn(self, module):
        return isinstance(module, Block)

    # Activation Checkpointing Function
    def activation_checkpointing_fn(self, module):
        return isinstance(module, Block)

class MyComposerModel(ComposerModel):

    def __init__(self, n_layers):
        super().__init__()
        self.model = Model(n_layers)
        ...

    def forward(self, batch):
        ...

    def eval_forward(self, batch, outputs=None):
        ...

    def loss(self, outputs, batch):
        ...

    ...

composer_model = MyComposerModel(n_layers=3)

fsdp_config = {
    'sharding_strategy': 'FULL_SHARD',
    'min_params': 1e8,
    'cpu_offload': False, # Not supported yet
    'mixed_precision': 'DEFAULT',
    'backward_prefetch': 'BACKWARD_POST',
    'activation_checkpointing': False,
    'activation_cpu_offload': False,
    'verbose': True
}

trainer = Trainer(
    model=composer_model,
    fsdp_config=fsdp_config,
    ...
)

trainer.fit()


Error when using action()

Hello.
I've updated L5 Modular in my project from V1 to V2.
I've set the config file to load the V1 modules, but now I'm getting this message:
Target class [App\Modules\MyModule\Controllers\App\Modules\MyModule\Controllers\MyController] does not exist.
I think that this is happening because some of the project developers have used the action function with the full path to the controller method [Ex: action('\App\Modules\MyModule\Controllers\MyController@index');] and L5 Modular makes the concatenation beetween the module path and with the action parameter.
Can someone help me with this?
Thank's!

Hi @lparede,
I‘ve never used the action helper. I‘ll dig into it.
For now you could try to refactor the parts to something like action('MyModule\Controllers\MyController@index'); while whithin the module.`.
But to be honest, I think your example should work as long as it starts with a backslash.
Could you please paste the real code, which causes the exception?

Hi @lparede,
I‘ve never used the action helper. I‘ll dig into it.
For now you could try to refactor the parts to something like action('MyModule\Controllers\MyController@index'); while whithin the module.`.
But to be honest, I think your example should work as long as it starts with a backslash.
Could you please paste the real code, which causes the exception?

@Artem-Schander ,
Thank you for your reply.
Apparently, the problem was related to the namespace of Route::group(). Some developers have defined the namespace without the initial backslash. Ex: 'namespace' => 'App\Modules\MyModule\Controllers' instead of 'namespace' => '\App\Modules\Schedules\Controllers'.
Now everything seems to be ok. :)

@Artem-Schander ,
Thank you for your reply.
Apparently, the problem was related to the namespace of Route::group(). Some developers have defined the namespace without the initial backslash. Ex: 'namespace' => 'App\Modules\MyModule\Controllers' instead of 'namespace' => '\App\Modules\Schedules\Controllers'.
Now everything seems to be ok. :)

Cool, glad it's working now :)

Cool, glad it's working now :)


bump io.mongock:mongock-bom from 5.4.2 to 5.4.3

Fix #10038

@pascalgrimaud @murdos not sure if it is the best solution, WDYT?

I did the same thing on the generator https://github.com/jhipster/generator-jhipster/pull/26409 and it was merged

Just if you can find the doc to know what this property is used for

@pascalgrimaud https://docs.mongock.io/v5/features/transactions/index.html

FYI The breaking change will be reverted in an upcoming 5.4.4 version.


memory leak

I wrote a test case and found that the memory has been rising, can you help me to see if it is a problem with my use or a memory leak, thank you.
But when I increase the number of tasks to 10000 level, this phenomenon will not appear.
system: MacOS 12.5 (m1)
go: 1.16
cron: github.com/robfig/cron/v3 v3.0.1
code:
func TestCron(t *testing.T) {
	c := cron.New(cron.WithSeconds())
	c.Schedule(cron.Every(time.Second), cron.FuncJob(func() {}))
	c.Start()

	go func() {
		for {
			var ms runtime.MemStats
			runtime.ReadMemStats(&ms)
			memory := int64(float64(ms.Alloc))
			routines := runtime.NumGoroutine()
			fmt.Printf(">> runtime: routines: %d, memory: %d\n", routines, memory)
			time.Sleep(time.Second)
		}
	}()

	sig := make(chan os.Signal, 1)
	signal.Notify(sig, syscall.SIGINT, syscall.SIGTERM)
	<-sig

	// Output:
	// >> runtime: routines: 4, memory: 156304
	// >> runtime: routines: 5, memory: 166616
	// >> runtime: routines: 5, memory: 167528
	// >> runtime: routines: 5, memory: 167968
	// >> runtime: routines: 5, memory: 168808
	// >> runtime: routines: 5, memory: 169632
	// ...
}

There is a timer var timer *time.Timer in function func (c *Cron) run(), I think maybe because of using time.NewTimer on each time around and not closing it.

Alloc is the same as HeapAlloc.
documentation info
HeapAlloc is bytes of allocated heap objects. "Allocated" heap objects include all reachable objects, as well as unreachable objects that the garbage collector has not yet free. Specifically, HeapAlloc increases as heap objects are allocated and decreases as the heap is swept and unreachable objects are freed. Sweeping occurs incrementally between GC cycles, so these two processes occur simultaneously, and as a result HeapAlloc tends to change smoothly (in contrast with the sawtooth that is typical of stop-the-world garbage collectors).

When I delete the relevant cron code, there will still be this phenomenon, I understand that it may not be the problem of the library


Allow for skipping of salmon alevin process and proceed directly to alevin-fry quantification

Closes #41. This PR, adds in flexibility to the workflow to skip the first step of running salmon alevin in cases where the RAD file already exists and a new parameter, rad_skip is set to true.
Here we are specifically only skipping the first step in the af-rna.nf module as this is the step that is the most costly in both time and memory. This update will enable us to re-run samples that have already been processed and incorporate changes in the later parts of the workflow, such as the filtering steps, or future changes to the QC report, without having to re-run the mapping step.
To do this, we first add the paths to the rad directory to the meta object. Then we create two channels within the map_quant_rna workflow. The first channel includes libraries that are missing the rad directory and then becomes the input to the alevin_rad process. The second channel contains libraries that already have a rad directory, and becomes the input to the next step, fry_quant_rna. We then combine these two channels, pull out the barcode_file to create a tuple of [barcode_file, rad_dir, meta] to be used as input to fry_quant_rna.
A few things to note. The first is that in order to properly combine the output channel from alevin_rad with the rna_rad_ch, I had to rename the output of the process as rad_dir to match with how we had named it in the meta. I could rename everything here, run_dir, but I thought rad_dir was a better fit for everything related to looking for the rad files and then I am still using run_dir for the downstream fry_quant_rna process.
I also am not quite sure why, but nextflow didn't like .map{it + [it[0].barcode_file]} to pull out the barcode file, but preferred .map{[it[0].barcode_file] + it}. So I changed it to have the barcode file as the first item in the tuple and it stopped complaining.
I tested the workflow and everything worked as plan, skipping the RAD step and taking only 7 minutes to process a single RNA sample!
We also added here the parameter, rad_skip which I set to true. By having this parameter here, we can force the rad step to still run and set it to false at the command line by using --rad_skip false. I tested this out and when set to false it no longer skips the alevin_rad step as planned.
Also when adding in this parameter, I moved all of the other remaining parameters to the main configuration file rather than keeping some of them within main.nf.

👏🏼 👏🏼 👏🏼 !!!
I will review this tomorrow, but I wanted to say congrats on getting it done!
I also had a quick thought about why it + [x] pattern didn't work, which I thought was worth mentioning now. I'm not sure now whether we will want to do anything about it, but I have an idea of how we would if we wanted to.
My explanation is that the elements in a channel are stored by nextflow as a tuple, and tuples are immutable. So we can't add to a tuple, but we can add a tuple to a list (which is mutable) we create with [], which explains why the other direction works. A potential option if we want to keep the order would be to do it.toList() + [x]*. Again, not sure whether that is worth it, but it might be nice to keep the metadata first for consistency.
* I'm actually not sure that the square brackets are needed; when I was looking this up and testing in the console, it seemed to work as just it.toList() + x, but I feel like I must have done it before for a reason? Probably because I was putting things first for sorting reasons, so the first element had to be a list. There is also apparently the option to use it.toList() << x which is a groovy-specific way to add to a list. I don't love it.

Thanks for the helpful tips on why adding to the tuple wasn't working @jashapiro, and I like the use of branch here as well! I went ahead and implemented the changes you suggested and tested them. I just changed the name of the branch to make_rad instead of do_rad. It was minor but something about it wasn't sitting well with me.
And then when testing, there was some issues with using .map{it.toList() + file(it[0].barcode_file)} and nextflow yelled at me saying that the input tuple didn't match the input set cardinality declared by process. When removing file it worked as expected.

I did have a couple of minor additional comments below (and I tested with the << version to be sure it worked), but I'm excited to have this optimization in, and I might consider this worthy of a version bump to 0.2

I would agree with this! I think before we process the next set of projects it would be great to have this release as an updated version. That would be especially great for the projects where we were planning to process the new modalities and now we could process the whole project instead of just the new modalities and incorporate all of the updates for single-cell/nuclei samples too that have happened downstream of alevin-fry.


Namespace list refactor

The pr uses the the new v3/namespaces endpoint to populate the namespaces list view using the following environment:
https://github.com/pulp/pulp_ansible/pull/1437 (galaxy_ng)
https://github.com/ansible/galaxy_ng/pull/1705 (pulp_ansible)
The MyNamespaceAPI  my-namespace.ts has been removed and replaced with calling the NamespaceAPI with a { my_permssions : ansible.change_ansiblenamespace} param.
This is in draft form. Thank you for your review.

Looks like the API PR timed out, closing this one too.
The changes still look good, but will need a rebase once relevant again :)


supervise.sh running in a loop

Hi,
Today I found out that the two scripts      /opt/bin/supervise.sh /opt/bin/bootstrap  and /opt/bin/supervise.sh /opt/bin/setup  were constantly running in a loop on all our worker nodes. So constantly we have apt update / upgrade / etc. running.
Part of  /var/log/syslog, this is flowing in a loop:
systemd[1]: Finished Update APT News.
systemd[1]: esm-cache.service: Succeeded.
systemd[1]: Finished Update the local ESM caches.
supervise.sh[3482641]: Fetched 222 kB in 1s (206 kB/s)
supervise.sh[3482641]: Reading package lists...
supervise.sh[3481021]: + apt-get install -y --allow-downgrades 'containerd.io=1.6*'
supervise.sh[3483659]: Reading package lists...
supervise.sh[3483659]: Building dependency tree...
supervise.sh[3483659]: Reading state information...
supervise.sh[3483659]: The following held packages will be changed:
supervise.sh[3483659]:   containerd.io
supervise.sh[3483659]: The following packages will be upgraded:
supervise.sh[3483659]:   containerd.io
supervise.sh[3483659]: 1 upgraded, 0 newly installed, 0 to remove and 94 not upgraded.
supervise.sh[3483659]: E: Held packages were changed and -y was used without --allow-change-held-packages.
supervise.sh[3277]: + sleep 1
supervise.sh[3277]: + /opt/bin/setup
supervise.sh[3483816]: + systemctl is-active ufw
supervise.sh[3483817]: inactive
supervise.sh[3483816]: + systemctl mask ufw
systemd[1]: Reloading.
supervise.sh[3483816]: + systemctl restart systemd-modules-load.service
systemd[1]: systemd-modules-load.service: Succeeded.
systemd[1]: Stopped Load Kernel Modules.
systemd[1]: Stopping Load Kernel Modules...
systemd[1]: Starting Load Kernel Modules...
systemd[1]: Finished Load Kernel Modules.
supervise.sh[3483816]: + sysctl --system
supervise.sh[3483850]: * Applying /etc/sysctl.d/10-console-messages.conf ...
supervise.sh[3483850]: kernel.printk = 4 4 1 7
supervise.sh[3483850]: * Applying /etc/sysctl.d/10-ipv6-privacy.conf ...
supervise.sh[3483850]: net.ipv6.conf.all.use_tempaddr = 2
supervise.sh[3483850]: net.ipv6.conf.default.use_tempaddr = 2
supervise.sh[3483850]: * Applying /etc/sysctl.d/10-kernel-hardening.conf ...
supervise.sh[3483850]: kernel.kptr_restrict = 1
supervise.sh[3483850]: * Applying /etc/sysctl.d/10-link-restrictions.conf ...
supervise.sh[3483850]: fs.protected_hardlinks = 1
supervise.sh[3483850]: fs.protected_symlinks = 1
supervise.sh[3483850]: * Applying /etc/sysctl.d/10-magic-sysrq.conf ...
supervise.sh[3483850]: kernel.sysrq = 176
supervise.sh[3483850]: * Applying /etc/sysctl.d/10-network-security.conf ...
supervise.sh[3483850]: net.ipv4.conf.default.rp_filter = 2
supervise.sh[3483850]: net.ipv4.conf.all.rp_filter = 2
supervise.sh[3483850]: * Applying /etc/sysctl.d/10-ptrace.conf ...
supervise.sh[3483850]: kernel.yama.ptrace_scope = 1
supervise.sh[3483850]: * Applying /etc/sysctl.d/10-zeropage.conf ...
supervise.sh[3483850]: vm.mmap_min_addr = 65536
supervise.sh[3483850]: * Applying /usr/lib/sysctl.d/50-default.conf ...
supervise.sh[3483850]: net.ipv4.conf.default.promote_secondaries = 1
supervise.sh[3483850]: sysctl: setting key "net.ipv4.conf.all.promote_secondaries": Invalid argument
supervise.sh[3483850]: net.ipv4.ping_group_range = 0<PHONE_NUMBER>
supervise.sh[3483850]: net.core.default_qdisc = fq_codel
supervise.sh[3483850]: fs.protected_regular = 1
supervise.sh[3483850]: fs.protected_fifos = 1
supervise.sh[3483850]: * Applying /usr/lib/sysctl.d/50-pid-max.conf ...
supervise.sh[3483850]: kernel.pid_max = 4194304
supervise.sh[3483850]: * Applying /etc/sysctl.d/99-cloudimg-ipv6.conf ...
supervise.sh[3483850]: net.ipv6.conf.all.use_tempaddr = 0
supervise.sh[3483850]: net.ipv6.conf.default.use_tempaddr = 0
supervise.sh[3483850]: * Applying /etc/sysctl.d/99-sysctl.conf ...
supervise.sh[3483850]: * Applying /etc/sysctl.d/k8s.conf ...
supervise.sh[3483850]: kernel.panic_on_oops = 1
supervise.sh[3483850]: kernel.panic = 10
supervise.sh[3483850]: net.ipv4.ip_forward = 1
supervise.sh[3483850]: vm.overcommit_memory = 1
supervise.sh[3483850]: fs.inotify.max_user_watches = 1048576
supervise.sh[3483850]: fs.inotify.max_user_instances = 8192
supervise.sh[3483850]: * Applying /usr/lib/sysctl.d/protect-links.conf ...
supervise.sh[3483850]: fs.protected_fifos = 1
supervise.sh[3483850]: fs.protected_hardlinks = 1
supervise.sh[3483850]: fs.protected_regular = 2
supervise.sh[3483850]: fs.protected_symlinks = 1
supervise.sh[3483850]: * Applying /etc/sysctl.conf ...
supervise.sh[3483851]: ++ command -v hostnamectl
supervise.sh[3483816]: + '[' -x /usr/bin/hostnamectl ']'
supervise.sh[3483816]: + '[' -s /etc/machine-name ']'
supervise.sh[3483852]: ++ cat /etc/machine-name
supervise.sh[3483816]: + machine_name=ston-k1d-worker-965b66456-ljlkt
supervise.sh[3483816]: + hostnamectl set-hostname ston-k1d-worker-965b66456-ljlkt
supervise.sh[3483816]: + apt-get update
systemd[1]: Starting Update APT News...
systemd[1]: Starting Update the local ESM caches...
supervise.sh[3483865]: Get:1 http://security.ubuntu.com/ubuntu focal-security InRelease [114 kB]
supervise.sh[3483865]: Hit:2 http://az-1.clouds.archive.ubuntu.com/ubuntu focal InRelease
supervise.sh[3483865]: Hit:3 http://az-1.clouds.archive.ubuntu.com/ubuntu focal-updates InRelease
supervise.sh[3483865]: Hit:4 https://download.docker.com/linux/ubuntu focal InRelease
supervise.sh[3483865]: Get:5 http://az-1.clouds.archive.ubuntu.com/ubuntu focal-backports InRelease [108 kB]
systemd[1]: apt-news.service: Succeeded.
systemd[1]: Finished Update APT News.
systemd[1]: esm-cache.service: Succeeded.
systemd[1]: Finished Update the local ESM caches.
supervise.sh[3483865]: Fetched 222 kB in 1s (227 kB/s)
supervise.sh[3483865]: Reading package lists...
supervise.sh[3483816]: + DEBIAN_FRONTEND=noninteractive
supervise.sh[3483816]: + apt-get -o Dpkg::Options::=--force-confdef -o Dpkg::Options::=--force-confold install -y curl ca-certificates ceph-common cifs-utils conntrack e2fsprogs ebtables ethtool glusterfs-client iptables jq kmod openssh-client nfs-common socat util-linux ipvsadm
supervise.sh[3484443]: Reading package lists...
supervise.sh[3484443]: Building dependency tree...
supervise.sh[3484443]: Reading state information...
supervise.sh[3484443]: conntrack is already the newest version (1:1.4.5-2).
supervise.sh[3484443]: ebtables is already the newest version (2.0.11-3build1).
supervise.sh[3484443]: ethtool is already the newest version (1:5.4-1).
supervise.sh[3484443]: ipvsadm is already the newest version (1:1.31-1).
supervise.sh[3484443]: socat is already the newest version (<IP_ADDRESS>-2).
supervise.sh[3484443]: glusterfs-client is already the newest version (7.2-2build1).
supervise.sh[3484443]: ca-certificates is already the newest version (20230311ubuntu<IP_ADDRESS>).
supervise.sh[3484443]: ceph-common is already the newest version (15.2.17-0ubuntu<IP_ADDRESS>).
supervise.sh[3484443]: cifs-utils is already the newest version (2:6.9-1ubuntu0.2).
supervise.sh[3484443]: curl is already the newest version (7.68.0-1ubuntu2.18).
supervise.sh[3484443]: e2fsprogs is already the newest version (1.45.5-2ubuntu1.1).
supervise.sh[3484443]: iptables is already the newest version (1.8.4-3ubuntu2.1).
supervise.sh[3484443]: kmod is already the newest version (27-1ubuntu2.1).
supervise.sh[3484443]: nfs-common is already the newest version (1:1.3.4-2.5ubuntu3.4).
supervise.sh[3484443]: openssh-client is already the newest version (1:8.2p1-4ubuntu0.7).
supervise.sh[3484443]: util-linux is already the newest version (2.34-0.1ubuntu9.4).
supervise.sh[3484443]: jq is already the newest version (1.6-1ubuntu<IP_ADDRESS>).
supervise.sh[3484443]: 0 upgraded, 0 newly installed, 0 to remove and 95 not upgraded.
supervise.sh[3483816]: + apt-get update
systemd[1]: Starting Update APT News...
systemd[1]: Starting Update the local ESM caches...
supervise.sh[3484446]: Hit:1 http://az-1.clouds.archive.ubuntu.com/ubuntu focal InRelease
supervise.sh[3484446]: Hit:2 http://az-1.clouds.archive.ubuntu.com/ubuntu focal-updates InRelease
supervise.sh[3484446]: Get:3 http://security.ubuntu.com/ubuntu focal-security InRelease [114 kB]
supervise.sh[3484446]: Get:4 http://az-1.clouds.archive.ubuntu.com/ubuntu focal-backports InRelease [108 kB]
supervise.sh[3484446]: Hit:5 https://download.docker.com/linux/ubuntu focal InRelease
systemd[1]: apt-news.service: Succeeded.
systemd[1]: Finished Update APT News.
systemd[1]: esm-cache.service: Succeeded.
systemd[1]: Finished Update the local ESM caches.
supervise.sh[3484446]: Fetched 222 kB in 1s (213 kB/s)
supervise.sh[3484446]: Reading package lists...
supervise.sh[3483816]: + apt-get install -y apt-transport-https ca-certificates curl software-properties-common lsb-release
supervise.sh[3485253]: Reading package lists...
supervise.sh[3485253]: Building dependency tree...
supervise.sh[3485253]: Reading state information...
supervise.sh[3485253]: lsb-release is already the newest version (11.1.0ubuntu2).
supervise.sh[3485253]: ca-certificates is already the newest version (20230311ubuntu<IP_ADDRESS>).
supervise.sh[3485253]: curl is already the newest version (7.68.0-1ubuntu2.18).
supervise.sh[3485253]: software-properties-common is already the newest version (<IP_ADDRESS>).
supervise.sh[3485253]: apt-transport-https is already the newest version (2.0.9).
supervise.sh[3485253]: 0 upgraded, 0 newly installed, 0 to remove and 95 not upgraded.
supervise.sh[3485263]: + curl -fsSL https://download.docker.com/linux/ubuntu/gpg
supervise.sh[3485264]: + apt-key add -
supervise.sh[3485264]: Warning: apt-key output should not be parsed (stdout is not a terminal)
supervise.sh[3485264]: OK
supervise.sh[3485409]: ++ lsb_release -cs
supervise.sh[3483816]: + add-apt-repository 'deb https://download.docker.com/linux/ubuntu focal stable'
systemd[1]: Starting Update APT News...
systemd[1]: Starting Update the local ESM caches...

I guess this issue is very similar to https://github.com/kubermatic/machine-controller/issues/1049
Maybe the option  --allow-change-held-packages should be added, as in https://github.com/kubermatic/machine-controller/pull/1073/files
We are using Ubuntu focal workers with machine-controler v1.54.4
Thanks!

Hi @Farfaday, apologies for the late reply. Is this still an issue for you? Have you found any solution? Judging by the log output, is it possible that your image already had containerd installed in an older version?

Hi @embik,
That could be possible. In the meantime, we changed the image used by our workers and also let them be replaced/rotated by the machine deployment, so I unfortunately cannot reproduce that issue.
Thank you!


Documentation improvements in publishing chapter

Rework chapter structure and document suppress*Warnings for publication warnings.

Messed up command line merge so GitHub believes it was closed but I confirm this got merged!


What string should be entered when we wanna open file in remote custom git host?

Hey!
First thanks for the great package.
I have a question about opening the custom git host website.
I follow the Custom Remotes Settings and add a custom remote in the workspace settings.json.
When I select a file and want to Open file to remote, it shows this dialog:

Does anyone teach me what should I input? I tried any string I can think of but it didn't work.
Thanks!

@ctxhou Are you still experiencing this issue? If so, can you try with the GitLens 9 beta release to see if you still have the issue?

@ctxhou GitLens v9 is out hopefully this will resolve any issues. If not please re-open this issue.


Light version

Hi,
The minified JS is quite big, is there a way to create a light version?
Looks like it includes Framework7? There's a Lite version of it.

https://github.com/nolimits4web/swiper/discussions/4228


Callback of the UUID even when the profile does not exist.

I have a unique problem with the project. Within my plugin DDoSDefender, my goal is to verify player accounts before  the main class of the server by utilizing the UUID from the LoginStart packet to check whether the player's profile exists.
Currently, I’m encountering an issue: when bots send a UUID that should not correspond to any existing profile, the .fetch(playerUUID); method consistently returns the same UUID I sent instead of providing an indication that the profile does not exist (Like a null value or 0)  (I also checked many Bot UUID's against NameMC and they do
not exist).
I’m unsure whether this is a problem with my code or if it’s an intended feature of the wrapper. Despite attempting several solutions, I have not been able to resolve the issue.
If you’d like to discuss this further, I would be happy to provide more details on Discord. You can reach me at the username Koskolonium.

Unfortunately, not much can be done about this. As long as the UUID you provided is a valid UUID, NameMC's API endpoint will give a result regardless of whether there is a Minecraft account with that UUID or not (empty array in case of non-existing profiles). It does not work like their frontend (that returns 404 if Minecraft account with a specific UUID was not found) and as such simply using the available API does not give the ability to determine the existence of a Minecraft account (or at the very least one saved on the NameMC's side).
Server fetching works in a similar way, preventing me from being able to verify whether the provided address is actually valid.
For this, NameMC would have to update their API endpoints or you would need to use a different library to verify the UUID.
Alternatively, I could add custom UUID verification myself with a way to bypass that check if you are sure the Minecraft account with that UUID exists.

Unfortunately, not much can be done about this. As long as the UUID you provided is a valid UUID, NameMC's API endpoint will give a result regardless of whether there is a Minecraft account with that UUID or not (empty array in case of non-existing profiles). It does not work like their frontend (that returns 404 if Minecraft account with a specific UUID was not found) and as such simply using the available API does not give the ability to determine the existence of a Minecraft account (or at the very least one saved on the NameMC's side).
Server fetching works in a similar way, preventing me from being able to verify whether the provided address is actually valid.
For this, NameMC would have to update their API endpoints or you would need to use a different library to verify the UUID.
Alternatively, I could add custom UUID verification myself with a way to bypass that check if you are sure the Minecraft account with that UUID exists.

i see that is unfortunate, if you do want to add custom UUID Verification that would be very useful but i wont pressure you, i will look into other sources.

It would be better to look into a different solution for you what you are trying to achieve. In the end, it is not really in scope of such a library (wrapper for NameMC API endpoints) to offer something like that.


install yarn on travis

this is some CI madness this week.
looks like Travis changed the way we run builds today and now the default preinstalled version of yarn is 0.17

Maybe we can just pour the latest?

@thymikee yea! the first one wasn't even the right command :)
updated with the latest version install.sh script

Codecov Report

Merging #4155 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4155   +/-   ##
=======================================
  Coverage   60.55%   60.55%
=======================================
  Files         196      196
  Lines        6777     6777
  Branches        6        6
=======================================
  Hits         4104     4104
  Misses       2670     2670
  Partials        3        3

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 bd2bd8c...8ddedea. Read the comment docs.


Extra whitespace characters in the annotation file breaks

The program breaks, when there are extra whitespace characters in the annotation file e.g.
bs 2        5       1
bs 6        8       2
bs 9        12      3
bs 13       16      4

The extra white space chars makes the annotation file more readable and editable with column selection mode. Therefore, I think, it will be nice to be ready for such an extra whitespace.
If You think it's an issue, I can handle it.

Hmm indeed. Thanks for bringing this up! I think this is a bug. I think i can easily fix it without much effort. So, no worries from your side. Ill update this thread when im done later!

#7
This should fix it now. Works on my end. Can you let me know whether this fixes it for you too?

Issue fixed.


Add an option to read notes from a file

HockeyApp's puck command has a handy option to read the release notes from a file. I've replicated this feature here.

Hi @philippelatulippe ,
Thanks for the PR! :)
That said, this seems quite redundant, as exposing a file as an env var is quite simple, see http://devcenter.bitrise.io/tips-and-tricks/expose-environment-variable/ - envman add --key MY_RELEASE_NOTE --valuefile ./some/file/path
I'd rather not duplicate functionality provided by the bitrise CLI tools, but I also see the utility to have this as an input directly..
WDYT @philippelatulippe ? Did you know about the envman option? Now that you do, do you think we should still merge this PR?

I mistakenly thought the steps were limited to exposing variables defined in their yaml file. It's been too long since I hacked on steps! I was bringing over a changelog script that wrote them to file, but it's much easier to just adapt the script to use envman. Since it's so easy to fix the script, I agree that the slight benefit of my patch isn't worth the cost of maintaining duplicate functionality.
Thank you for considering my PR and for your gracious response.

No problem at all and thanks again for the PR! :)


Update Debian built to include intel_iommu change

https://salsa.debian.org/cloud-team/debian-cloud-images/-/tree/732707480b4c6a5174066a9d4d5a7b0c05ab0fb1

/lgtm


Segmentation error

Hi, thank you for the work.
I've met the segmentation error from d2f6e71e5898fd62399fc37929d23683a0054b0f.
I tried 2 methods of building from source: python setup.py develop and python setup.py build_ext --inplace; python setup.py bdist_wheel; pip install dist/*.whl
>>> import tvdcn
tvdcn.>>> tvdcn.__version__
'0.4.0'
>>> tvdcn.has_ops()
True
>>> tvdcn.with_cuda()
True
>>>
(base) van-tien.pham@siim-3:~/tvdcn/tests$ python test_grad.py
DeformConvTestArgs(spatial_dim=2
- conv_params:
        in_channels     = 4
        out_channels    = 2
        kernel_size     = (2, 3)
        stride          = (1, 2)
        padding         = (0, 1)
        dilation        = (1, 2)
        weight_groups   = 1
        offset_groups   = 2
        mask_groups     = 1
        transposed      = False
- input_params:
        batch_size      = 1
        in_size         = (5, 6)
        out_size        = [4, 2]
- input_shapes:
        input           = torch.Size([1, 4, 5, 6])
        weight          = torch.Size([2, 4, 2, 3])
        offset          = torch.Size([1, 24, 4, 2])
        mask            = torch.Size([1, 6, 4, 2])
        bias            = torch.Size([2])
- output_shapes:
        expected_cols   = torch.Size([4, 12])
        expected_output = torch.Size([1, 2, 4, 3])
- factory_params:
        device          = cuda
        dtype           = torch.float64
        contiguous      = True
)
Erreur de segmentation

This also happens with ecn.
I'm not sure if this is just because of my machine/installation. Can you please verify?
During building, I notice that at some moments the process consumes 100% CPU, which didn't happen with the previous version.
Thanks in advance for your help!

Fixed with f1872bdc9114d12093131994f4dc2e12600ba207.
Regarding the high CPU consumption, it is because I enabled ninja-build as it helps saving a lot of time. It used to be disabled because torch.utils.cpp_extension.BuildExtension caused an error on Windows in earlier versions of torch. They seem to have sorted that out but I didn't notice until recently. You can disable ninja by modifying the setup.py before compiling with:
def setup_package():
    setup(
        version=get_version(),
        ext_modules=get_extensions(),
        cmdclass={
            'build_ext': torch.utils.cpp_extension.BuildExtension.with_options(use_ninja=False)
        },
    )


Update imported CCI2 binaries, compile CCDoc

This fixes issue #169
CCDoc now compiles with solution, binary output is still the ImportedCCI folder so deploy should not be affected.

LGTM. @sharwell: what do you think? I know that we keep updating binaries in the repo, but it seems that this will allows us to fix few very nasty issues...

Looks good.
@SergeyTeplyakov @mike-barnett  Is there any long term plan here? Some major parts of Code Contracts depend on CCI so maybe it could be moved to github under .NET Foundation also?

I applied for it to move to the.NET Foundation two months ago. If they don't accept out by the time I get back (December 1), then you can just set up a github site, and I'll close down the codeplex one.
Mike
Sent from my Windows Phone

From: Hubert<EMAIL_ADDRESS>Sent: 10/25/2015 12:57 PM
To<EMAIL_ADDRESS>Cc: Mike<EMAIL_ADDRESS>Subject: Re: [CodeContracts] Update imported CCI2 binaries, compile CCDoc (#276)
Looks good.
@SergeyTeplyakovhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FSergeyTeplyakov&data=01|01|Michael.Barnett%40microsoft.com|8dd82b67663e495a240708d2dd767898|72f988bf86f141af91ab2d7cd011db47|1&sdata=hHVDyl%2F5k7fIHmyK3tzhuXcMycColDgpN79g74VV3Qs%3D @mike-barnetthttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmike-barnett&data=01|01|Michael.Barnett%40microsoft.com|8dd82b67663e495a240708d2dd767898|72f988bf86f141af91ab2d7cd011db47|1&sdata=JSialnaqpqYqPqhc6xj2og3ZO%2FJQN5YlPK2aCafe6G4%3D Is there any long term plan here? Some major parts of Code Contracts depend on CCI so maybe it could be moved to github under .NET Foundation also?
—
Reply to this email directly or view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2FCodeContracts%2Fpull%2F276%23issuecomment-150963706&data=01|01|Michael.Barnett%40microsoft.com|8dd82b67663e495a240708d2dd767898|72f988bf86f141af91ab2d7cd011db47|1&sdata=u2WZkMTMWnYL4Gmnrnh8WlhRu5gdXMOgbvHQPAPkGUg%3D.