feat: move clearing logic to onDetachedFromWindow

Moved the logic of clearing views from onDropViewInstance, which is not the proper place for it since the view can be dropped, but still visible for during the transition (see https://github.com/software-mansion/react-native-screens/issues/797 and https://github.com/software-mansion/react-native-screens/issues/773). They have been moved to onDetachedFromWindow, which is the moment the view is not present in the window of the app. It required passing the requestManager to the FastImageViewWithUrl instance.

@WoLewicki  Are you sure this solution is OK? I have in app:
react-navigation v4
react-native-screens 2.16
and when I change screens plugged into bottomStackNavigator, then onDetachedFromWindow cleans images that are still on these screens.
Maybe we should plug in onDestroyView event here?

Can you post a reproduction here? I am not 100% sure if this solution is 100% correct.

I have created a repro here in Example on master: https://github.com/ku8ar/react-native-screens

I changed the implementation to handle loading/clearing of the images in onAttachedToWindow and onDetachedFromWindow. Can you check if it fixes the issues? Or maybe I am missing something here too?

When image is generated in onAttachedToWindow, it is not possible to dynamically change source. We need to recreate a new fast-image object. Right?

Yeah, we should probably keep the logic of loading in setting the source too. I will change the implementation to an easier one and I will ping you then.

any progress on this?

I added loading the image on setting the source, can you test if all is ok now @ku8ar @scvgoe?

@WoLewicki  checked. Looks ok. Thanks!

can you please merge it @DylanVann

bump, would be good to have this merged

@WoLewicki  Having flickering issues with this fork, here's some example code
`
import React from 'react';
import {NavigationContainer} from '@react-navigation/native';
import {createMaterialTopTabNavigator} from '@react-navigation/material-top-tabs';
import {ScrollView} from 'react-native';
import Image from 'react-native-fast-image';
const Tab1 = () => {
return null;
};
const Tab2 = () => {
return (
<ScrollView style={{margin: 10}}>
<Image
style={{
width: 600,
height: 600,
}}
source={{
uri: 'https://mht.wtf/post/content-aware-resize/sample-image.jpeg',
}}
/>
  <Image
    style={{
      width: 600,
      height: 600,
    }}
    source={{
      uri: 'https://encrypted-tbn0.gstatic.com/images?q=tbn:ANd9GcT_12mZYw9c0QfCb875croxaX5vufbLRdrXLRhudSwVmHtuSKYOPise1ZTKQgl-_QzGdJc&usqp=CAU',
    }}
  />
</ScrollView>

);
};
const Tab3 = () => {
return null;
};
const App = () => {
const Tab = createMaterialTopTabNavigator();
return (

<Tab.Navigator initialRouteName="Tab2">
<Tab.Screen name="Tab1" component={Tab1} />
<Tab.Screen name="Tab2" component={Tab2} />
<Tab.Screen name="Tab3" component={Tab3} />
</Tab.Navigator>

);
};
export default App;
`
if you test going back and forth from tab1 and tab2 you'll notice that the first image rendered always flickers

@moulie415 I tested the code and did not spot any flickering. Could you provide a video where it is clearly visible? Also, you are using createMaterialTopTabNavigator which does not use react-native-screens underneath, so there may be something connected to this package that does not work well with this PR.

@moulie415 I tested the code and did not spot any flickering. Could you provide a video where it is clearly visible? Also, you are using createMaterialTopTabNavigator which does not use react-native-screens underneath, so there may be something connected to this package that does not work well with this PR.

@WoLewicki  I've managed to repro with both bottom and material top tabs
https://user-images.githubusercontent.com/10909374/129879290-aef6f282-9855-431d-b16d-8a317bc077a0.mp4
https://user-images.githubusercontent.com/10909374/129879274-06fe04fa-a05f-4600-b6dd-0d85c3e587ef.mp4

Hmm it is probably caused by the fact that the whole tab is detached from the native view hierarchy when the tab is changed with react-native-screens enabled. Can you spot flickering when you don't use the react-native-screens in this navigator (set https://reactnavigation.org/docs/bottom-tab-navigator/#detachinactivescreens to false)?

@WoLewicki yeh that works however  "react-native-fast-image": "^8.3.7" is fine regardless of what you use.

Yeah, I don't know an easy way to fix the bug and make it work correctly. I am open to suggestions.

@WoLewicki
What about this? I know too little about how things work but this seemed to fix the original issue without breaking the other one
    @Override
    public void onDropViewInstance(final FastImageViewWithUrl view) {
        // This will cancel existing requests.
        if (requestManager != null) {
            runOnUiThread(
                new Runnable() {
                @Override
                public void run() {
                    requestManager.clear(view);
                }
            });
        }

      ....

essentially running the requestManager.clear() on UI thread

@chr4ss12 I also have not enough knowledge about this, but if it works, please make a PR with it and link it here so people can easily test it. Maybe it is a better solution to the problem.


DoubleCMD installation problem

Installing 'doublecmd' (0.9.8) [64bit]
The remote server returned an error: (403) Forbidden.
URL https://downloads.sourceforge.net/project/doublecmd/DC%20for%20Windows%2064%20bit/Double%20Commander%200.9.8%20beta/doublecmd-0.9.8.x86_64-win64.msi is not valid

Link is valid but have redirect.

seems like sourceforge links are serving up html pages.


Automatically use the protoc version downloaded by the maven plugin

Which issue does this PR close?
Closes #.
Rationale for this change
Simplify protoc installation and configuration.
What changes are included in this PR?
Try using the protoc version downloaded by protobuf-maven-plugin.
Are there any user-facing changes?
No

it's great, so we don't need to install protobuf manually in the future


SSL_OP_ almost no more bits

Every 32-bit value from 0x00000001 to 0x80000000 is taken by an SSL_OP value.
I don't have any good suggestions on how to handle this, short of naming any new options with OPX and return two new API's to set/clear the OPX values.  Ugh.

Not quite. 0x00000020U and 0x00000100U are still free.

3.0 would be the time to change the APIs to long longs if we're possibly going to use both remaining ones.
I'm not sure it's a good idea.

change the APIs to long longs

uint64_t would be better

It would be a minor API break though.

It would be a minor API break though.

Yes. But if we're going to do it then 3.0 would be the time.

Setting beta1 milestone for OTC to evaluate if we want this change.

Adding an OTC hold so it doesn't get missed.

If the OTC wants to do this, I'll make the change.  I would prefer to write things as 1<<X to make it more clear what the bits are. Suggestions for how to write that expression would be useful. But please wait until the OTC decides to move forward or not.

See #15184.  "and then there was one."

This was fixed by #15230. Closing.


replaced priv for env, added branch note, small edits

Added branch note to acknowledge why multiple branches are utilized and how + Small linguistic edits.
https://github.com/rsource-open-source/rsource-records/commit/c6aef142520671ca25b2be6c98cb4f26003f111c

fuck, i meant to merge to dev


docs(chips): add drag and drop sample

Closes #13215

I signed it!

@jelbourn Am I missing something about the ci:lint. I think that I've fixed the errors that appears in the build check?

@crisbeto Thanks for the detailed review. I have applied the changes. Hope that the sample is fine now.

Looks like CI has a legitimate failure


start upgrading selenium 3.0

Hi,
Yesterday selenium 3.0 was released and was stable how about start using it in protractor?
Thanks,
G. Anil

Thanks for the heads up! We are aware of the release, and are working hard to get Protractor upgraded to the latest Selenium version.


⚠ Avatar API Endpoint has degraded performance

In 832e46c, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 1052 ms

Resolved: Avatar API Endpoint performance has improved in 9f606c3 after 6 minutes.


Creating provisioningComposer.js

Creating as a separate file from .zip contents so that it can be source controlled/edited/etc.
Description

Task Item

Screenshots

Coverage remained the same at 55.944% when pulling a6a79382a9986fee9db43f429ebc3a2484e323dc on dmvtech-provisioningscripts into 24b070bf92467da7770ad9de94b3974c24cd139a on main.


Add Search in Rotated Sorted Array Question of Leetcode

Is your feature request related to a problem? Please describe.
I would like to add Search in Rotated Sorted Array of Leetcode(Medium)
Describe the solution you'd like
I would like to code the solution of this question in cpp

/assign


migrated to express. change to accept only cid, not urls

I believe the unit tests are not yet working


Record Versioning / Changing properties

Hi,
I defined a complex type as a Record with many fields and now I need to change just a couple of fields type. The desiderata would be to generate a "version 2" of the original complex type and then merge it with the "version 1".
I started from here:
const MyTypeV1 = Record({
    version: Literal(1),
    ....
    prop: SomeRuntype
})

const MyTypeV2 = Record({
    version: Literal(2),
    ....
    prop: AnotherRuntype
})

const MyType = Union(MyTypeV1, MyTypeV2)

Then I could call MyType.check(x) and pass, as x, a record "version 1" or a record "version 2".
Now since the MyTypeV1 is very complex, and I want to avoid Copy&Paste the unmodified fields, there is a way to generate MyTypeV2 from MyTypeV1 ?
const MyTypeV2 = MyTypeV1.Override({
    version: Literal(2),
    prop: AnotherRuntype
})

I'd wrap it in a function in such case:
const MyType = (version: 1 | 2) => Record({
  version: Literal(version),
  prop: version === 1 ? String : Number
})

But that way you can't use Static:
const MyTypeV1 = MyType(1)
type MyTypeV1 = Static<typeof MyTypeV1>
// `MyTypeV1` above is invalid:
// type MyTypeV1 = {
//     version: 1 | 2;
//     prop: string | number;
// }

So if you miss it, you need an ugly hack like this:
const MyType = <V extends 1 | 2>(version: V) =>
  Record({
    version: Literal(version),
    prop: (version === 1 ? String : Number) as V extends 1
      ? typeof String
      : typeof Number
  })
const MyTypeV1 = MyType(1)
type MyTypeV1 = Static<typeof MyTypeV1>
// `MyTypeV1` above works expectedly:
// type MyTypeV1 = {
//     version: 1;
//     prop: string;
// }

Another option without wrapping in a function is:
const MyTypeCommon = Record({
  value: Literal("common")
})
const MyTypeV1 = MyTypeCommon.And(Record({
  version: Literal(1),
  prop: String
}))
const MyTypeV2 = MyTypeCommon.And(Record({
  version: Literal(2),
  prop: Number
}))
const MyType = Union(MyTypeV1, MyTypeV2)

I don't know how much your model is complicated, so please reopen and let me know if above options does not suit your case.


[APM Onboarding] SSI bug fix for 7.52.0

What does this PR do?
Fixes the bug of NIL due to *sync.once not being initialized.
Motivation
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

DCA dev image https://gitlab.ddbuild.io/DataDog/datadog-agent/-/jobs/438235718
Deploy the following helm configuration on DA to the test cluster:

datadog:
  apiKeyExistingSecret: datadog-secret2
  clusterName: "apm-onboarding-injection"
  site: datad0g.com
  apm:
    enabled: false
    socketEnabled: true
    instrumentation:
      enabled: true
      enabledNamespaces:
        - rc-tests
clusterAgent:
  image:
    repository: datadog/cluster-agent-dev
    digest: "sha256:c7cc34bcd153f73c47a7e1ce4c0daa1b2a7a82dc7674514de658213955019521"
    pullPolicy: Always
    doNotCheckTag: true

If DA is installed from main of public helm-chart, DCA will have the following env variables installed:

    - name: DD_APM_INSTRUMENTATION_ENABLED
      value: "true"
    - name: DD_APM_INSTRUMENTATION_DISABLED_NAMESPACES
      value: '["default"]'

In https://github.com/DataDog/datadog-agent/pull/21673/files we re-defined SSI configuration options, but for those to take an effect, we also need to change helm-chart.
If DA is installed from a branch liliya.belaus/remove-datadog-ns-blacklisting-ssi (https://github.com/DataDog/helm-charts/pull/1318), DCA has the expected env variables installed.
    - name: DD_APM_INSTRUMENTATION_ENABLED
      value: "true"
    - name: DD_APM_INSTRUMENTATION_ENABLED_NAMESPACES
      value: '["rc-tests"]'

NIL pointer exception is gone.

/merge


Parabolas are weird to construct

Parabolas require a line on the board in order for them to properly be constructed:
http://jsxgraph.uni-bayreuth.de/docs/symbols/Parabola.html
Not sure how that will work for the user

Two points B and C are already created by the user... create a line:
var userLine = board.create('line', [B,C]);
Then use this new userLine to create the Parabola where A is the focus.
var parab = board.create('parabola',[A,userLine]);

Can now graph a parabola.


Adopt pipeline-as-yaml-plugin

Link to GitHub repository
https://github.com/jenkinsci/pipeline-as-yaml-plugin/
PR request I want to deliver : https://github.com/jenkinsci/pipeline-as-yaml-plugin/
Until delivered plugin is totally broken with recent core version
When modifying release permission
If you are modifying the release permission of your plugin or component, fill out the following checklist:

### Release permission checklist (for submitters)
- [x] The usernames of the users added to the "developers" section in the .yml file are the same the users use to log in to [accounts.jenkins.io](https://accounts.jenkins.io/).
- [x] All users added have logged in to [Artifactory](https://repo.jenkins-ci.org/) and [Jira](https://issues.jenkins.io/) once.
- [x] I have mentioned an [existing team member](https://github.com/orgs/jenkinsci/teams) of the plugin or component team to approve this request.

FYI @oleg-nenashev
When enabling automated releases (cd: true)
Follow the documentation to ensure, your pull request is set up properly. Don't merge it yet.
In case of changes requested by the hosting team, an open PR facilitates future reviews, without derailing work across multiple PRs.
Link to the PR enabling CD in your plugin

### CD checklist (for submitters)
- [ ] I have provided a link to the pull request in my plugin, which enables CD according to the documentation.

### Reviewer checklist
- [ ] Check that the `$pluginId Developers` team has `Admin` permissions while granting the access.
- [ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
- [ ] If security contacts are changed (this includes add/remove), ping the security officer (currently `@Wadeck`) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it.

Posted on dev list : https://groups.google.com/g/jenkinsci-dev/c/0p1dTD2JmV0

Feel free to enable CD in the same PR :)

cc @oleg-nenashev and @aytuncbeken FYI


Cannot create custom aliases (ie. put/get)

Cannot create custom aliases (ie. put/get)
The following configuration results in the error: "pcluster: error: argument command: invalid choice: 'put'"
[aliases]
ssh = ssh -i ~/key.pem {CFN_USER}@{MASTER_IP} {ARGS}
put = scp -i ~/key.pem {ARGS} {CFN_USER}@{MASTER_IP}:~/
get = scp -i ~/key.pem {CFN_USER}@{MASTER_IP}:~/{ARGS} .

Hi @RickoLacharity
This isn't currently supported. You can modify the ssh command but not the others... If you want to open a pull request to do this, you'd start by modifying the cli/pcluster/cfnconfig.py file. Follow the ssh example. Thanks!

This functionality did work and was broken in commit: fb23bf184b983a5af39315549d2ff100930dfac1 with a change from parse_args to parse_known_args in cli.py.

@RickoLacharity yep, I wrote that commit initially with the intent to support other use cases, such as what you've added with put and get. However, there's a catch 22 in this, one of the cli options, --config, needs to be parsed before reading from the config file. Since the extra commands (such as put and get) are included in the config file, we need to parse the config before parsing the cli, and we can't parse the cli with parsing the config.
A solution is to just add put and get as command line options. For example, put would look like:
pput = subparsers.add_parser(
        "put",
        help="Connect to the master instance and copy a file into the home directory.",
        description="Run scp command with the cluster username and IP address pre-populated. "
        "Any arguments provided are used as flags and local file location."
        "This command may be customized in the aliases "
        "section of the config file.",
        formatter_class=argparse.RawDescriptionHelpFormatter
    )
    pssh.add_argument("cluster_name", help="name of the cluster to connect to")
    pssh.add_argument("-d", "--dryrun", action="store_true", default=False, help="print command and exit")
    pssh.set_defaults(func=command)

I'd be happy to review a pull request with this functionality. Feel free to add this to cli.py


Add autocomplete feature. Based on, but heavily adapted from, leaflet-search implementation.

The feature is initially disabled. Enable by setting the L.control.GeoSearch option enableAutocomplete to true. Assumes that geosearch query will return a list of partial matches.
Feedback welcome.

@jasco, I am aware that this PR was requested a long time ago. Any chance that you could apply a rebase on the latest version?

Does enableAutocomplete works with Esri provider?

@smeijer I has been a very long time indeed. Yes, I'll take a look at rebasing.

@vlasvlasvlas it can be integrated with esri but has not yet been.  An interface file is required construct the query to the service and parse the results.

@vlasvlasvlas I think I was mistaken above. I have been away from this project for a number of years.  If your provider returns partial matches then autocomplete should work with the provider as is.  I use a custom provider so I did not realize/recall until I just looked more closely that the default esri provider is equivalent.

@smeijer the pull request has been rebased.  I hope it proves helpful.

txs!

It does not seem to work for me , I'll dig into it but I'd like to know if it is working for someone else on the latest version of leaflet and L.GeoSearch

It's not working for me too.

@MrSohey, @falqn.  Thanks for giving it a try.  It should be working now.

It's working great ! Thanks @jasco

I'm currently working on a big rewrite of this plugin. I'll take a look at this PR soon, to make it part of the (new)  core.
Thanks.

Yes, it's working now. Nice to hear about new core soon @smeijer. It will be great to include autocomplete function from @jasco PR. It will be very useful if autocomplete will suggest some results and user can choose which one is correct, ex. there are many places named 'Newton' in US.

@MrSohey, @falqn, great to hear it is working now.
@smeijer Thanks for the status update. I look forward to seeing the new
release.
When you dig into it you will notice that the AutoComplete class supports
overriding of the rendering of the HTML for the autocomplete data with
onMakeSuggestionHTML. I do not expose it currently but being able to
override the rendering might be of value to the users.
On Thu, Nov 3, 2016 at 1:37 AM, falqn<EMAIL_ADDRESS>wrote:

Yes, it's working now. Nice to hear about new core soon @smeijer
https://github.com/smeijer. It will be great to include autocomplete
function from @jasco https://github.com/jasco PR. It will be very
useful if autocomplete will suggest some results and user can choose which
one is correct, ex. there are many places named 'Newton' in US.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/smeijer/L.GeoSearch/pull/29#issuecomment-258086896,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AApr7GmcHl-GikiAcf7aHa1LSkwiT7t4ks5q6Z0ugaJpZM4A3fuG
.

Fixed this in the next branch. Release v2.0.0 be released before the end of this year. Currently usable trough:
npm install leaflet-geosearch@next --save

@jasco, Thanks so much for this contributions to this project. Both your contributions have been re-created into the next branch. Sorry for being able to "just merge your PR's".


Code highlighting | Namespace import

First of all, this plugins is amazing!
Thanks for all the work guys. This plugin really helps with developing in laravel.
I tested it in Netbeans 17 and it works great.
besides that is works great here are a few options that can be fixed in the next verisons.
Namespace use statements
When importing classes using the use the editor does not bind the classes you import. (No typehinting etc)
Also the editor does not register the imported class as being used.

shorthand if statement
When using the  ?  :  the highlighting does't work anymore.

Hi wimurk, thanks for the observation.
I agree, these are things which need to improve.
External language embedding can be a challenge when implementing custom plugins, but I think it can be doable in the future.
For the second issue ... there is a problem with Netbeans Ide freezing in some context when typing "::", so I added some harsh embedding limitations . I think this shorthand if statement is fixable, also;

When importing classes using the use the editor does not bind the classes you import.

Yes, but normally you shouldn't do that (good practice), you should use view composer, or use User::first() in the controller and send it to the view
Also you could simple do:
<?php
use App\Models\User;
?>

When importing classes using the use the editor does not bind the classes you import.

Yes, but normally you shouldn't do that (good practice), you should use view composer, or use User::first() in the controller and send it to the view
Also you could simple do:
<?php
use App\Models\User;
?>

Yes, this was an example. Most of the times i have services that i use in my blade to process some data information.
Have you guys already looked into the code formtting? Myself i use the psr12 provided by pint (default laravel). This sometimes gives weird markup in combination with <x- components. Might be an issue with netbeans itself tho.

I will close the ticket as the plugin has changed a lot.
Currently there is no Warning notice inside @php directive for unused Use Statement.


Change verses class to update in verse of the day  file to fix passage bug

The former verses array was wrong, the passage kept returning home so i went to the url, and checked the appropriate classe to this
const verses = $("a.text-text-light.w-full.no-underline");

this is the screenshot of the verses anchor tag class in you version verse of the day


[Feature Request]: Actions to automate messages

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Auto Comments on the issues & PR that are created by another contributors
Use Case
This will improve the overall engagement of the other contributors on your repository
Benefits
make your project look more professional & brilliant from another projects.
Priority
High
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

@mansiruhil13 I would like to work on this enhancement. can you assign me this issue 7 add labels to it.
hacktoberfest, hacktoberfest-accepted, gssoc-ext
also one more thing. can you connect with me on Linkedin to ask few questions. I can't find your project channel on discord.

This issue could be Level 2 right, @mansiruhil13


document Docker Registry SSL CA cert location on windows

Moved from https://github.com/moby/moby/issues/21189

I am trying to follow this guide https://docs.docker.com/engine/security/certificates/, but the paths are for linux. What would be the equivalent for Windows?

/cc @JimGalasyn

Closing this ticket due to its age, and the impending refactor. If you think this is in error, feel free to reopen. Thanks!

I think this is still relevant, and not documented

Also see https://github.com/moby/moby/issues/21189

@thaJeztah Not sure if this issue still relevant. Could you PTAL?


Leverage extensions in graph-node responses

This is an open issue to discuss the option to add some metadata in graph-node responses, using the extensions field of GraphQL response.

@lutter @azf20 what do you think about this?
We discussed this with several daaps developers, we think maybe the gateway/graph-node can have an option to return execution/cost/indexing information, and we can have an easy API/tooling on The Graph client to use those (for deciding on preferred indexer/source, collecting some metrics, finding performance bottlenecks and so on)

That makes sense to me - @Jannis what do you think?
There is some corresponding work in graph-node, and maybe also in Gateways to support? Of course would also need to decide what metadata is of interest etc.

This is available via includeHttpDetailsInExtensions flag. @dotansimha

Thanks @ardatan ! Closing ;)


Support InnerTimeJoin & LeftOutTimeJoin distribution planner

Codecov Report
Attention: 269 lines in your changes are missing coverage. Please review.

Comparison is base (31c65b3) 48.93% compared to head (a6693f3) 48.90%.
Report is 6 commits behind head on master.

Files
Patch %
Lines

...gine/plan/planner/distribution/SourceRewriter.java
41.12%
63 Missing :warning:

...e/plan/planner/distribution/ExchangeNodeAdder.java
0.00%
45 Missing :warning:

...nner/plan/node/process/join/InnerTimeJoinNode.java
26.22%
45 Missing :warning:

.../apache/iotdb/commons/partition/DataPartition.java
2.32%
42 Missing :warning:

...ngine/plan/planner/plan/node/PlanGraphPrinter.java
0.00%
29 Missing :warning:

...ueryengine/plan/planner/OperatorTreeGenerator.java
0.00%
22 Missing :warning:

...storageengine/dataregion/tsfile/TsFileManager.java
50.00%
5 Missing :warning:

...yengine/plan/optimization/LimitOffsetPushDown.java
0.00%
4 Missing :warning:

.../db/protocol/thrift/impl/ClientRPCServiceImpl.java
0.00%
3 Missing :warning:

...ne/execution/fragment/FragmentInstanceContext.java
25.00%
3 Missing :warning:

... and 4 more

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #11860      +/-   ##
============================================
- Coverage     48.93%   48.90%   -0.03%
- Complexity    25573    25606      +33
============================================
  Files          2908     2908
  Lines        181276   181633     +357
  Branches      21691    21750      +59
============================================
+ Hits          88707    88833     +126
- Misses        92569    92800     +231

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


@mzikherman =>  Hide followed artists module

Hides followed_artists module; this could be further optimized by not even including FOLLOWED_ARTISTS in the initial metaphysics query, but I think that might be more trouble than it's worth since that initial query is quite small and the real images/data are fetched later.

Yea, this is a simple fetch from Gravity indicating which rails to include, so the optimization would have to go up to Gravity: https://github.com/artsy/gravity/blob/163713676b84f452728aa6662e7ed56a7d7de25e/app/api/v1/me_modules_endpoint.rb#L18 (not worth it IMO).
And then client-side, it loops over all the modules that were fetched server-side, and actually requests their content. So I think your approach is totally fine.


Capistrano 3 integration should use fetch(:stage)

Hi there,
I'm trying out the appsignal gem, and while it works quite smoothly, I'm having an issue with capistrano deployments: appsignal always thinks I'm deploying to production, while I'm actually deploying to staging.
I think the culprit of this is at lib/appsignal/integrations/capistrano/appsignal.cap:3 :
  1 namespace :appsignal do
  2   task :deploy do
  3     env = fetch(:rails_env, fetch(:rack_env, 'production')) #### <=
  4     user = ENV['USER'] || ENV['USERNAME']
  5     revision = fetch(:appsignal_revision, fetch(:current_revision))

Appsignal is using :rails_env or :rack_env, while it's pretty much standard to use fetch(:stage). This will make appsignal use the right environment. Right now, if I capistrano staging deploy, a deploy appears in the production environment. By using fetch(:stage), staging is used for deployment tracking, as I think it should be. :)
Thanks a lot in advance!

Could I have some feedback on this, guys?

This makes a lot of sense, we'll make this change. I expect we can do this for the next 1.2 release.

Thanks a lot, @thijsc! :+1:


Github Collector is Not working for private repo

downloaded Hygieia Latest code
maven version :  3.2.1
Followed all step for configuration of private repo in Hygieia which are as follows
Step 1
Generation of key for private repo
java -jar core-2.0.5-SNAPSHOT.jar com.capitalone.dashboard.util.Encryption
output of this as follows
SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.
Your secret key is:
x0CDs8cQicRJp9Nby4qG6Wf7gFiivNzV
Sample encrypted string with the above key for 'thisIsMyPassword' is:
ZxBvfKWygjZLEQE5WMRzzTWaqrlw1Io7
Step 2
The highlighted Key added in api.properties file as follows
dbname=dashboarddb
dbhost=hostname
dbport=27017
dbhostport=hostname:27017
server.contextPath=/api
server.port=8888
key= x0CDs8cQicRJp9Nby4qG6Wf7gFiivNzV
Step 3
Same key is added in github.properties file as follows
The highlighted Key added in api.properties file as follows
dbname=dashboarddb
dbhost=hostname
dbport=27017
logging.file=./logs/github.log
github.corn=0 0/5 * * * *
github.host=github.com
github.key= x0CDs8cQicRJp9Nby4qG6Wf7gFiivNzV
while running collector after maven build getting error as follows
404 Not Found:http://api.github.com/repos///commits?sha=master&since=2017-04-25T23:40Z*
and also from UI while providing username and password it show  Error encrypting password

@ritusingh16  - refer https://github.com/capitalone/Hygieia/issues/1284 workaround helps. I will look into issue meanwhile.

@nireeshT  : I tried to modify ApiSettings.java as shown below
public void setKey(final String key) {
    this.key = "############";   (with my access token)
}

I then built and ran api and collector . I am still seeing 404 when hitting
https://api.github.com/repos/company/repo-name/commits?sha=master&since=2017-04-26T12:25Z.
Also seeing the same error as @ritusingh16 when trying to add username and password . Can someone take a look at it ?

@ritusingh16 @nireeshT  I updated queryUrl variable in DefaultGitHubClient.java  to include access token in the query parameter as below
String queryUrl = apiUrl.concat("/commits?access_token=MYTOKEN&sha=" + repo.getBranch()
+ "&since=" + thisMoment)
I am able to get the responce from github api . Hope it helps

@nireeshT
Thanks for the reply
I tried to modify but getting same 404 error

@Ramya126
I tried the same as you mentioned in comment,but still same error
Thanks for the reply

I tried with latest updated code base of hygieia to configure github private repo still getting 404 error,but this time error have some slight change
PrivateGithubRepoError.docx
Please find the attached document which contain screenshots of error and all configuration details

Any update ?

@ritusingh16 Paste the commands you are using to launch the api and github collector modules.

Commands as follows:
For api module
java -jar api.jar --spring.config.location=dashboard.properties
For Github collector
java -jar github-scm-collector-2.0.5-SNAPSHOT.jar

@nireeshT, @satishc1 -
It would seem that the requied key property is never built/created in dashboard.properties for the running API component to ingest. Do you concur?
Looking at properties-builder.sh file for the API component, it is and always has been missing a line to create a property named key, which would get built from an environment variable injected during docker run (or docker-compose up) and placed into dashboard.properties just before the container performs its command of
java -Djava.security.egd=file:/dev/./urandom -jar api.jar --spring.config.location=/hygieia/dashboard.properties

So if it's not established during docker run-time... how else should this runtime property get into the running spring boot app? Maybe there's some hidden or missing docs that explain this. If not, then the properties-builder.sh needs an update, to add the line:
KEY=${KEY:-}

Without this, in the UI, if someone tries to create a GitHub repo collector that is password protected, the rest call to /encryption/{message} will just return "ERROR" giving a message of Error encrypting password
If somehow the GitHubRepo object is created in MongoDB with an empty or null password, this logic
                 // decrypt password
		 String decryptedPassword = "";
		 if (repo.getPassword() != null && !repo.getPassword().isEmpty()) {
		 	try {
		 		decryptedPassword = Encryption.decryptString(
		 				repo.getPassword(), settings.getKey());
		 	} catch (EncryptionException e) {
		 		LOG.error(e.getMessage());
		 	}
		 }

Will continue to pass along that empty or null password, which will resulting in requests sent to getCommits from the repo failing.
Also the linked issue above suggest to look at these docs: https://github.com/capitalone/Hygieia/blob/master/UI/README.md#encryption-for-private-repos
But, it refers to api.properties when it seems that docker uses dashboard.properties

@ritusingh16, @nireeshT, @satishc1
I found a decent workaround for this. Spring and its property source precedence allows for it to pick up config properties from various places, including the environment.
So, while the key property is not declared anywhere as a placeholder in a properties file or in the properties-builder.sh  (and it should be), you can inject its value into the container as an environment variable, and spring will pick it up and use it.
Example snippet from docker-compose.override.yml:
hygieia-api:
  environment:
    - JASYPT_ENCRYPTOR_PASSWORD=hygieiasecret
    - AUTH_SECRET=********
    - KEY=********

Could really use some gap-filling in the docs to cover this option though, or to clarify the intended approach.

@ritusingh16 closing the issue due to inactivity. Please re open if this issue is still occuring

I get 401 unauthorized as the private gitlab branch is protected.. anyone knows how to solve this issue? help me..


chore: update ui packages to 18.14

SUMMARY
update contains these commits:
https://github.com/apache-superset/superset-ui/pull/1409
https://github.com/apache-superset/superset-ui/pull/1411
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

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

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

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

/testenv up


Linkedin social sharing

I want to add linkedin social sharing in the theme can you please suggest me how can i do this

Afaik Ghost does only support Facebook and Twitter natively: https://forum.ghost.org/t/additional-social-accounts-in-general-settings/750
You can add however your own links with icons.

Download https://github.com/eddiesigner/liebling/blob/master/src/sass/fonts/icomoon/icomoon.svg
Open https://icomoon.io/app/#/select
Click on Import Icons and select the downloaded icomoon.svg (Use this font's metrics and metadata when exporting fonts? click yes)
Select every icon from the base font (should be the upper row) and any additional icons you want in the pack (linkedin in your example)
In the bottom left click Generate Font and then download
Remember the id that is written under the new Icon (for instagram its e90c)

Download the .zip file and replace the font files in src/sass/fonts/icomoon/ with those in the .zip file fonts directory
Open src/sass/common/_icons.scss and add your new icon with his id

.icon-instagram:before {
  content: "\e90c";
}

so for linkedin:
.icon-linkedin:before {
  content: "\eYOUR_ID";
}

After that you can add the linkedin link in the footer for example in partials/footer.hbs after the twitter icon

{{#if @site.twitter}}

{{/if}}

Now you should see the linkedin icon in the footer
@eddiesigner I just noticed that i snuck in the instagram icon with 8eccc87723fa1ccfcb2b44cf1f99ceefe9673bbe :see_no_evil:
Regards,
D3473R

Thank you so much this is so helpful. I will let you know once it's done

@D3473R Cool! Thank you so much for this detailed explanation, I think I should add it to the Wiki for future reference 👌🏼
And don't worry about the Instagram icon 😆
@veshitala To keep the same style, please consider using this icon: https://github.com/FortAwesome/Font-Awesome/blob/master/svgs/brands/linkedin.svg or this one: https://github.com/FortAwesome/Font-Awesome/blob/master/svgs/brands/linkedin-in.svg

Since Linkedin is not supported right now by Ghost, I'll close this issue by now since this workaround helps ;)


Stubbing fixture element appears to cause next fixture load to fail

I have a fixture something like this
<test-fixture id="basic">
  <template>
    <pas-app unresolved></pas-app>
  </template>
</test-fixture>

The unresolved attribute is removed by pas-app element in the attached callback.
Various aspects of it are instantiated, then items replaced then stubbing occurs in nested describe, beforeEach pairs. In particular a session element is replaced by a mock
      beforeEach(function(done) {
        app=fixture('basic');
        stub('iron-ajax', {
          generateRequest: function() {
            //make a noop;
          }
        });
        replace('pas-session').with('test-session');
        flush(done);
      });

At one point in my test sequence I am trying to check that it calls importHref to lazily load an element so I stub it like this
       it('should try and load pas-pages', function(done) {
          var called = false;
          stub('pas-app', {
            importHref: function(href,onload,onerror,async) {
              called = href;
            }
          });
          session.user = {name: 'LoggonOnUser'};
          flush(function() {
            expect(called.slice(-14)).to.equal('pas-pages.html');
            done();
          });

        });

This test passes, but if I now put a breakpoint on the next time I hit the beforeEach function above,  before I call fixture('basic') the dom shows a fixture with a completely empty document fragment, and after I have called fixture('basic'), the pas-app element is there but remains un-resolved and its template is not stamped out
The test that I refer to above output some pretty heavy error messages to the console
Cannot read property 'parentNode' of undefined
                           Function.invoke at /bower_components/sinonjs/sinon.js:2621
  test-session.proxy [as instanceTemplate] (eval at createProxy at /bower_components/sinonjs/sinon.js:2526:17), <anonymous>:1
               test-session._stampTemplate at /bower_components/polymer/polymer-mini.html:25
                test-session._initFeatures at /bower_components/polymer/polymer.html:3914
              test-session.createdCallback at /bower_components/polymer/polymer-micro.html:190
                           Function.invoke at /bower_components/sinonjs/sinon.js:2621
  pas-app.proxy [as instanceTemplate] (eval at createProxy at /bower_components/sinonjs/sinon.js:2526:17), <anonymous>:1
                    pas-app._stampTemplate at /bower_components/polymer/polymer-mini.html:25

I refactored my tests so that I stub the top level element before calling replace. When I do that, all the indications are that replace hasn't done anything, although its not done so silently.  All that has happened is that some tests that passed when the replace did work, now act as though  the  replace hasn't happened.
I must say I am really puzzled by what replace is actually doing.  With chrome dev tools and breakpoints I am looking at the dom  after each step, including a call to replace, and I can't see any identifiable change with replace.  Certainly there is no reference to the test-element, all the dom shows is the original element.
Should I be calling replace just once, before I call fixture?

Now I understand how replace works (see #356) I think this is just a symptom of that misunderstanding


feat(popup): dynamic colors

auto update colors them changes, on theme mode changed

fix with vars detection https://github.com/pengx17/logseq-plugin-link-preview/issues/122

can this be merged? thank you

I am not actively developing this plugin. Merging anyway. Let me know if anything breaks


gpsvc failed  to sign in

I used this tool some time ago and now when I go to log in pc loads and will sit there for around 20 minutes then pop up

then it signs out and makes me re login and then it loads within minutes. Please tell me a fix its so annoying have to wait 20 minutes for your pc to login.
and no I didn't select something random. I followed one of his guides on how to use it and what you should enable and I did what he said.

Check Group Policy Client Service

Press Win + R to open the Run dialog box.
Type services.msc and press Enter.
Locate "Group Policy Client" in the list.
Right-click and select Properties.
Ensure the Startup type is set to Automatic.
Click on Start if the service is not running, then Apply and OK.

UUID
To chech the UUID run following script in PowerShell as Admin:
wmic csproduct get uuid

If it is empty or even invalid that might be the issue.

I've tried everything the internet, AI, people have said. I've tried even what you said before and also It is on automatic but I can not change the service status and yeah my uuid is good.

Hard to tell what might be causing the issues without knowig what you've done exactly.
It is also not clear if the issues are comming from a Tweak done by Winutil because of the delay that the issue started.
I would try out:
sfc /scannow

If that does not work:
DISM /Online /Cleanup-Image /RestoreHealth

If nothing worked:

Press Win + X and select Event Viewer.
Look through it and search for something that looks like the Issue you are having.

I've already tried all of that. it didn't work.

I ran the options the guy suggested on youtube. the real owner of it.


The numbers next to the filter selections don't accurately reflect the results count with the filter applied.

The numbers next to the filter options indicate to the user how many results match with that selection.  Yet, it's frequently the case that applying the filter (and that filter alone) results in more results than the filter label suggests.
003.zip

Verified on beta.  Closing as resolved.


TipsPagination: get lastTipId properly

introduced in #425

Does not seem to be fixing the issue 🤔

Does not seem to be fixing the issue with getting back from profile page 🤔

Did i connect it to this issue? No.


Call to undefined function StickyNotes\Auth\ldap_connect()

I am trying to setup LDAP and when my users try to login in, they are getting the error: Call to undefined function StickyNotes\Auth\ldap_connect()
I am not sure if this is due to a configuration error on my part or if the server is missing this.
Thanks.

You need to enable php_ldap.


Observables async advices

Since observables got quite popular, it might be necessary in some cases to use observable advices.
This issue intents discussion of them.

We can achieve the same result using after advices using meta.method.result.

I'd suggest reopening this issue. This is not always possible because, for instance, IndexedDB is async. So if you use IndexedDB as cache store, you can't prevent the original method from being executed if the data is already present in the cache.
My suggestion would be to not name them sync_advices, but just advices. Add some extra code in the advices that checks the result of each call (void, value, Promise, Observable). Handle it correctly and execute the next step.
I'm not sure if this is actually 100% possible though. What if the invoked method is synchronous, but the advices are asynchronous. This would result in the entire call becoming asynchronous, although actually the developer shouldn't care about it.
I think it's worth a discussion :).

I think this puts another layer of complexity. What I'm using here is:
@beforeMethod(...)
foo(meta: Metadata) {
   meta.method.result.map(bar => baz(bar))
     .do(performSideEffect);
}

Yes it definitely adds complexity here. But the use case of preventing the method to execute when data is stored in an async cache (WebSQL, IndexedDB) can't be covered at the moment.
One way to solve this would be to attach an invoke() method to meta.method.
@beforeMethod(...)
foo(meta: Metadata) {
    meta.method.proceed = false;

    meta.method.result = retrieveFromCache().then(result => {
        if (result) {
            return result;
        }

        return meta.method.invoke();
    })
}

But then again, the developer has to make sure that meta.method.result returns the same type of result (Promise, Observable, ...) which is indeed more complex. But adding the invoke() method makes it possible to at least allow these kind of use cases. It's up to the developer of the aspect that the result matches the type.

Yes, adding meta.method.invoke() makes sense.


Is it compatible with React ?

Is it possible to use your library with react ?
do you have any exemple ?

I just did a quick test and got this error:
<EMAIL_ADDRESS>5:16
Module parse failed: Unexpected token (5:16)
File was processed with these loaders:

./node_modules/babel-loader/lib/index.js
You may need an additional loader to handle the result of these loaders.
|
| class UncompressedSplatArray {

static OFFSET = {
|     X: 0,
|     Y: 1,

any help ?

Can you post the code you're using to load and run the viewer? I can try to debug from there.

Hi, i actually have the error as soon as i try to import:
import * as GaussianSplats3D from '@mkkellogg/gaussian-splats-3d';

the problem might come from my react app, i'm trying with a new one an will let you know

i'm almost there... but the processing splat keeps spining forver.
here is my code:
`import React, { Component } from 'react';
import * as GaussianSplats3D from '@mkkellogg/gaussian-splats-3d';
import {
Vector3,
Scene,
WebGLRenderer,
PerspectiveCamera,
Mesh,
MeshBasicMaterial,
BoxGeometry
} from 'three';
class GaussianSplatComp extends Component {
constructor(props) {
super(props);
  this.update = this.update.bind(this);

   this.state = {

  };

}
init(){
const {renderer, renderWidth, renderHeight} = this.setupRenderer();
this.renderer = renderer;
this.camera = this.setupCamera(renderWidth, renderHeight);
this.threeScene = this.setupThreeScene();
this.controls = this.setupControls(this.camera, this.renderer);

this.viewer = new GaussianSplats3D.DropInViewer();
this.viewer.addSplatScenes([

/*     {  //'path': 'assets/data/bonsai/bonsai_trimmed.ksplat',
'path': 'https://data2.gigascope.net/splats/bonsai_trimmed.ksplat',
'splatAlphaRemovalThreshold': 20,
},*/
{
'path': 'https://data2.gigascope.net/splats/bonsai_trimmed.ksplat',
'rotation': [-0.14724434, -0.0761755, 0.1410657, 0.976020],
'scale': [1.5, 1.5, 1.5],
'position': [-3, -2, -3.2],
'splatAlphaRemovalThreshold': 20,
}
], true);
this.threeScene.add(this.viewer);
requestAnimationFrame(this.update);

}
update() {

   // if (this.controls!=undefined){
    requestAnimationFrame(this.update);
    this.controls.update();
    this.renderer.render(this.threeScene, this.camera);
  //  }

}

setupRenderer() {
const renderWidth = 800;
const renderHeight = 600;
const rootElement = document.createElement('div');
rootElement.style.width = renderWidth + 'px';
rootElement.style.height = renderHeight + 'px';
rootElement.style.position = 'relative';

//  rootElement.style.left = '50%';
rootElement.style.top = '50%';
// rootElement.style.transform = 'translate(-50%, -50%)';
document.getElementById("GSContainer").appendChild( rootElement );

//   document.body.appendChild(rootElement);
const renderer = new WebGLRenderer({
    antialias: false
});
renderer.setSize(renderWidth, renderHeight);
rootElement.appendChild(renderer.domElement);

return {
  'renderer': renderer,
  'renderWidth': renderWidth,
  'renderHeight': renderHeight
}

}
setupCamera(renderWidth, renderHeight) {
const camera = new PerspectiveCamera(65, renderWidth / renderHeight, 0.1, 500);
camera.position.copy(new Vector3().fromArray([-1, -4, 6]));
camera.lookAt(new Vector3().fromArray([0, 4, -0]));
camera.up = new Vector3().fromArray([0, -1, -0.6]).normalize();
return camera;
}
setupThreeScene() {
const threeScene = new Scene();
const boxColor = 0xBBBBBB;
const boxGeometry = new BoxGeometry(2, 2, 2);
const boxMesh = new Mesh(boxGeometry, new MeshBasicMaterial({'color': boxColor}));
threeScene.add(boxMesh);
boxMesh.position.set(3, 2, 2);
return threeScene;
}
setupControls(camera, renderer) {
const controls = new GaussianSplats3D.OrbitControls(camera, renderer.domElement);
controls.rotateSpeed = 0.5;
controls.maxPolarAngle = Math.PI * .75;
controls.minPolarAngle = 0.1;
controls.enableDamping = true;
controls.dampingFactor = 0.05;
return controls;
}
componentDidMount() {
this.init();
window.addEventListener( 'resize', this.onWindowResize, false );

}
componentWillUpdate(nextProps) {
}
componentDidUpdate(){
}
render() {
return (

);

}
}
export default GaussianSplatComp;
`

Hi @mkkellogg i  hope you'll find some time to have a look at my issue, you library is great and it's very frustrating not being able to use it.

I hope I do too :) I will let you know when I have figured something out.

Are there any errors in the debug console? I have seen this situation before when dealing with CORS issues (there's more about that in the README). You can also quickly check if it's a CORS issue by setting the sharedMemoryForWorkers constructor parameter to false:
this.viewer = new GaussianSplats3D.DropInViewer({
    `sharedMemoryForWorkers`: false
});

indeed i had this error:
Uncaught (in promise) DOMException: Failed to execute 'postMessage' on 'DedicatedWorkerGlobalScope': SharedArrayBuffer transfer requires self.crossOriginIsolated.
at blob:http://localhost:3000/9df051eb-797e-4731-88fd-e2d957f8e513:134:16
with the modification you suggested it's working. can i leave this like this or should i solve the cors issue ?

It's really up to you, using shared memory for the sorting web worker should boost performance, so it might be worth it. What web server are you using?

i'm using firebase.
here is a link to test: www.preiss.fr
i have two other issues:
"optimizing splats" never stops
Left click to set the focal point doesn't work

I'm not familiar with firebase, so you'll have to consult their documentation to determine how to enable CORS functionality (if you want to use shared memory for the sorting web worker)
It looks like the "Optimizing splats" message isn't going away because another error is occurring during that phase... I can't tell what it is  because the code is minified.
As for the controls not working, that could be because you're using the drop-in version. If you want to use the standard Viewer, you should be able to pass your own DOM element as the root element:
const viewer = new GaussianSplats3D.Viewer({
    'rootElement': yourDOMElement
});

thank you so much for responding so quickly !
i'm almost there !
everything is working fine in local emulator, but once online the "optimizing splats" stays on and so the mouse click interaction doesn't work (it works fine in my local emulator)
might be related to the cors issue, i will investigate.
here is the new version with the garden scene:
https://www.preiss.fr/gs/garden

neanwhile is it possible to disable the "optimizing splats" ?

The "Optimizing splats" dialog isn't going away because there was an error during the building of the splat tree:

You could hide that dialog by passing 'showLoadingUI': false to the Viewer, but it doesn't solve the error, and the viewer won't work correctly.

but why is there an error, i'm using your garden.ksplat it should be working.
And as said it's working fine in local emulator

I'm not sure why there is an error, it probably has something to do with your build & bundling system. It's impossible to tell from the debug console because the code is minified.

it seems to come from the processSplatTreeNode funcion

What version of my code are you using? Are you using the latest version?

0.3.4
will update right now

v0.3.5 fixed the issue ;-)
thanks !!!!!

I don't want to revive this issue but it looks like the latest version (0.4.5) is having an issue with the SplatTreeWorker. When webpack compiles using babel-loader, the worker code is replaced with the babel compiled code, but does not have the babel imports so you get the following errors:

Related to this source code:

This causes the raytracing capabilities to not work as you pointed out above, any ideas on a fix? It seems it was fixed previously so not sure if this is a new issue or the same issue appearing again.

This is quite odd, I don't see this issue at all (on my machine that is). The issue doesn't exist in version v0.4.4?, If it still exists there, do you know the latest version that doesn't have the problem?

@mkkellogg the problem occurs when using webpack and babel to transpile the code. The issue is that because the webworker is being created by "function.toString" it will only grab the code within that function when creating the web worker. This means the imports at the top of the file from webpack + babel, are not included in the web worker, so the error occurs.
Personally I ended up switching to using vite for my builds instead of webpack + babel, but I think the solution would be how the web workers are created, if they were their own files, webpack could add them as a static resource, or bundle them separately.

That's a good point, I think I'll try to move them to being in their own files.


Add support for 256x64 4-bit greyscale SSD1322 device

This 3.2 inch OLED looks awesome, but it's a SSD1322. Can this be supported (easily)?

Datasheet: http://www.farnell.com/datasheets/303158.pdf

Looks relatively straightforward. This seems to be a more detailed data sheet though: http://www.newhavendisplay.com/app_notes/SSD1322.pdf

got one of these on order, https://www.aliexpress.com/item/1pcs-3-12-inch-16PIN-Yellow-OLED-Module-SSD1322-Drive-IC-256-64-SPI-8Bit-Parallel/32414142892.html

I ordered a white one earlier today!

Note to self: https://en.wikipedia.org/wiki/Grayscale#Luma_coding_in_video_systems

Package arrived today ... standby!

Arr! Mine hasn't yet but that's ok.. Would love to see a clip/picture.

Mine arrived as well, but there are no pins on it, and soldering looks hard, e.g. have to disassemble it? Thoughts @rm-hull?

Mine looks exactly like the one in https://www.aliexpress.com/item/1pcs-3-12-inch-16PIN-Yellow-OLED-Module-SSD1322-Drive-IC-256-64-SPI-8Bit-Parallel/32414142892.html - 2x8 holes down one side - there are no pins, so it will need some soldering. It doesn't look like it needs to be disassembled though, but must admit I haven't looked in too much detail.
 If the latter, then try and source some solderless headers like these: https://www.sparkfun.com/products/retired/10527

Another picture (with horrible quality, I don't get this phone) that shows the pins are in the center, unlink yours on the side. So this is not a recommended display to buy. Your solderless headers sound like a good idea but it looks like taking it apart temporarily shouldn't be too hard, I'll try that first.

Soldered it, hooked it up, but then realized there's no support yet, but yeah.

This is probably the next task.... I think it should be quite similar to the SSD1325 drivrt

@thijstriemstra Well I've drafted out a driver for this OLED, but am not getting any signs of life on the device at all ... if I push the WIP to a branch, can you try the code with your device?

@rm-hull yep sounds good

The ebay page also mentions 5V v.s. the 3.3V in our hardware doc.

@rm-hull seem results as you (with both 3.3v and 5v): nothing's happening. The ports I used: SCK, MOSI, RES, CS, DC, GND and VCC.

ok, well thats ~sort of~ encouraging, it is unlikely that both of them are broken, more likely the init sequence is wrong.
Also, on the back side, do you see any jumpers similar to BS1 and BS2 below with R18-R20 0-ohm resistors:

If so how are they jumpered?

Progres:

Bingo!

$ cat ssd1322.conf
--display=ssd1322
--interface=spi
--width=256
--height=64
--spi-bus-speed=16000000

@thijstriemstra:

My device has the following connections: one the left side there's the 'TF' section with MISO, SCK MOSI, CS and on the right 'SPI' side there are: RES, CS, DC, SCL, SDA, GROUND and VCC.

The 'TF' section probably refers to the unsoldered SD card slot on your device:

The pic is too fuzzy to make any other details out - you need a better 📷 !!

Do you have the same connections @rm-hull?

No, mine is two rows of 8 pins, and is documented like this, and can be configured in SPI or 8-bit parallel mode:

I guess I'll use connections from both sides if I look at the connection diagram in our docs.

You should only connect up the pins on the SPI side only, wiring exactly like the docs here: https://luma-oled.readthedocs.io/en/latest/hardware.html#spi
It should work with 3V3 or 5V (mine was 3V3)

New demo in examples repo to showcase colour and greyscale rendering:

Holy shit, wow! Awesome work @rm-hull, as always! Gonna test it now. Thanks for the detailed comments, that'll make good documentation as well.

I'm getting this traceback for the examples:
Traceback (most recent call last):
  File "/home/pi/projects/pi-test/luma.examples/examples/demo_opts.py", line 77, in <module>
    rotate=args.rotate)
  File "/home/pi/projects/pi-test/luma.oled/luma/oled/device.py", line 284, in __init__
    self.command(0xFD, 0x12)        # Unlock IC
  File "/home/pi/projects/pi-test/luma.oled/luma/oled/device.py", line 347, in command
    self._serial_interface.data(args)
  File "/home/pi/projects/pi-test/luma.core/luma/core/serial.py", line 152, in data
    write(data[i:i + 4096])
TypeError: write() argument 1 must be list, not tuple

Hmm

And here's a picture:

Re: traceback ... Haven't tried it on python 3 and there are no tests yet, so not trapped by tox/travis

Fix for python 3 in commit 63814ef

You should only connect up the pins on the SPI side only, wiring exactly like the docs here: https://luma-oled.readthedocs.io/en/latest/hardware.html#spi

@rm-hull but my spi side contains SDA and SCL pins and D0 and D1 are not even on there. :(

D0 = SCL (clock)
D1 = SDA (data)
Probably worth adding some wiring info for different devices

@thijstriemstra - did you get your SSD1322 working? I only tried it on a PiZero so far (both Py2.7 and 3.4) and seems to work fine, but it would get glitchy display if I raised the SPI bus speed over 16MHz (this might be something to do with the PiZero... it's not fast)

I'll test it later tonight.

@rm-hull unfortunately I didn't get it to work and kind of ready to throw it in the bin :-1: raspberrypi is wonderful but the amount of monkeypatching and trickery needed to get 3rdparty stuff working is disheartening.

@thijstriemstra That's a bit disappointing 🙁. It's possible that it is configured in parallel mode rather than SPI... I made a comment on this thread about the BS1 and BS2 jumpers -they control the config.
If you want to post it on to me, I'm happy to test it for you, document the wiring connections and then send it back to you.


XTTS build

Thanks for the great work! Three questions:

For the version build for 35.2.1 do I have to rebuild python3.9 (this is coqui requirement)? For JP5 what's the best way to go about it?
https://github.com/dusty-nv/jetson-containers/blob/aef3d09267c0b8564c6f3c18dc59873ce0a179d4/packages/audio/xtts/test_voices.py#L12C1-L12C57 - the path to the models have been changed (https://huggingface.co/coqui/XTTS-v2/tree/main)
If I understand the quantization happens during the setup?

@budzianowski I'm not sure that I ever got it working on JetPack 5 and only enable it in my stuff on JetPack 6, and even so found XTTS to be too slow in practice (even on AGX), so I changed to using Piper TTS.

Thanks for the heads up - I guess the <200ms latency claims were exaggerated. I will have a look at Piper.

Thanks for jetson-containers, it is very useful!
I would-like to run also xtts container  on my jetson-orin-AGX.
$ jetson-containers run $(autotag xtts)
Namespace(packages=['xtts'], prefer=['local', 'registry', 'build'], disable=[''], user='dustynv', output='/tmp/autotag', quiet=False, verbose=False)
-- L4T_VERSION=36.2.0  JETPACK_VERSION=6.0  CUDA_VERSION=122.0
-- Finding compatible container image for ['xtts']
Couldn't find a compatible container for xtts, would you like to build it? [y/N] y
-- Package onnxruntime:1.17 has missing dependencies, disabling...  ("couldn't find package:  cuda")
-- Package onnxruntime:1.17-builder has missing dependencies, disabling...  ("couldn't find package:  cuda")
-- Package tensorflow2 has missing dependencies, disabling...  ("couldn't find package:  cuda")
[..]
and more and more dependencies not found.
I need to use xtts_v2 from Hugging-Face, and I cannot substitute it with piper.
I'm new to jetson and docker. Can you help me to build this container or how to go on the right way?


[1.11-rc2] Container command %s could not be invoked.. (AppArmor)

Output of docker version:
Client:
 Version:      1.11.0-rc2
 API version:  1.23
 Go version:   go1.5.3
 Git commit:   388f544
 Built:        Fri Mar 25 19:58:22 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.11.0-rc2
 API version:  1.23
 Go version:   go1.5.3
 Git commit:   388f544
 Built:        Fri Mar 25 19:58:22 2016
 OS/Arch:      linux/amd64

Output of docker info:
Containers: 3
 Running: 0
 Paused: 0
 Stopped: 3
Images: 2
Server Version: 1.11.0-rc2
Storage Driver: overlay
 Backing Filesystem: extfs
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge null host
Kernel Version: 4.2.0-34-generic
Operating System: Ubuntu 15.10
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 15.6 GiB
Name: suda-PC2
ID: 42B5:4E7P:VILK:75VX:TKMU:7PC7:QYS3:CZHX:R6SR:3VWI:E75Y:G4L6
Docker Root Dir: /var/lib/docker
Debug mode (client): false
Debug mode (server): true
 File Descriptors: 12
 Goroutines: 31
 System Time: 2016-03-30T11:13:00.041578861+09:00
 EventsListeners: 0

Additional environment details (AWS, VirtualBox, physical, etc.):
physical
Steps to reproduce the issue:

Install Docker v1.11.0-rc2 (curl -fsSL https://test.docker.com/ | sh)
docker daemon -D -s overlay
docker run -it busybox (any image/any command is reproducible)

Describe the results you received:
docker: Error response from daemon: Container command 'sh' could not be invoked..
The underlying error is EACCES: https://github.com/docker/docker/blob/v1.11.0-rc2/daemon/start.go#L146
Note that the container works after running aa-disable /usr/bin/docker.
aa-status was as following before running aa-disable:
apparmor module is loaded.
66 profiles are loaded.
30 profiles are in enforce mode.
   /sbin/dhclient
   /usr/bin/evince
   /usr/bin/evince-previewer
   /usr/bin/evince-previewer//sanitized_helper
   /usr/bin/evince-thumbnailer
   /usr/bin/evince-thumbnailer//sanitized_helper
   /usr/bin/evince//sanitized_helper
   /usr/bin/lxc-start
   /usr/lib/NetworkManager/nm-dhcp-client.action
   /usr/lib/NetworkManager/nm-dhcp-helper
   /usr/lib/connman/scripts/dhclient-script
   /usr/lib/cups/backend/cups-pdf
   /usr/lib/libvirt/virt-aa-helper
   /usr/lib/lightdm/lightdm-guest-session
   /usr/lib/lightdm/lightdm-guest-session//chromium
   /usr/lib/telepathy/mission-control-5
   /usr/lib/telepathy/telepathy-*
   /usr/lib/telepathy/telepathy-*//pxgsettings
   /usr/lib/telepathy/telepathy-*//sanitized_helper
   /usr/lib/telepathy/telepathy-ofono
   /usr/sbin/cups-browsed
   /usr/sbin/cupsd
   /usr/sbin/cupsd//third_party
   /usr/sbin/ippusbxd
   /usr/sbin/libvirtd
   /usr/sbin/tcpdump
   docker-default
   lxc-container-default
   lxc-container-default-with-mounting
   lxc-container-default-with-nesting
36 profiles are in complain mode.
   /usr/bin/docker
   /usr/bin/docker///bin/cat
   /usr/bin/docker///bin/ps
   /usr/bin/docker///sbin/apparmor_parser
   /usr/bin/docker///sbin/auplink
   /usr/bin/docker///sbin/blkid
   /usr/bin/docker///sbin/iptables
   /usr/bin/docker///sbin/mke2fs
   /usr/bin/docker///sbin/modprobe
   /usr/bin/docker///sbin/tune2fs
   /usr/bin/docker///sbin/xtables-multi
   /usr/bin/docker///sbin/xtables-multi//null-2
   /usr/bin/docker///sbin/zfs
   /usr/bin/docker///usr/bin/xz
   /usr/bin/docker//null-1
   /usr/bin/docker//null-1//null-3
   /usr/bin/docker//null-1//null-3//null-4
   /usr/bin/docker//null-1//null-3//null-5
   /usr/bin/docker//null-1//null-6
   /usr/bin/docker//null-1//null-7
   /usr/bin/docker//null-1//null-7//null-8
   /usr/bin/docker//null-1//null-7//null-9
   /usr/bin/docker//null-1//null-a
   /usr/bin/docker//null-b
   /usr/bin/docker//null-b//null-10
   /usr/bin/docker//null-b//null-10//null-11
   /usr/bin/docker//null-b//null-10//null-12
   /usr/bin/docker//null-b//null-13
   /usr/bin/docker//null-b//null-14
   /usr/bin/docker//null-b//null-14//null-15
   /usr/bin/docker//null-b//null-14//null-16
   /usr/bin/docker//null-b//null-17
   /usr/bin/docker//null-b//null-c
   /usr/bin/docker//null-b//null-c//null-d
   /usr/bin/docker//null-b//null-c//null-e
   /usr/bin/docker//null-b//null-f
7 processes have profiles defined.
5 processes are in enforce mode.
   /sbin/dhclient (7739)
   /usr/lib/telepathy/mission-control-5 (16006)
   /usr/sbin/cups-browsed (799)
   /usr/sbin/cupsd (710)
   /usr/sbin/cupsd (793)
2 processes are in complain mode.
   /usr/bin/docker (21617)
   /usr/bin/docker//null-b (21625)
0 processes are unconfined but have a profile defined.

/etc/apparmod.d/docker
/etc/apparmord.d/docker-engine
Describe the results you expected:
The container should work with the default installation.
Additional information you deem important (e.g. issue happens only occasionally):
This is an upgrade installation from 1.10.3.

I tried to repro this issue on an upgrade from from 1.10.3 -> 1.11-rc2 as well as a fresh install of 1.11-rc2 on Ubuntu 15.10. Cannot repro it.
@AkihiroSuda : Do you see the issue on a fresh install of 1.11-rc2 as well?

@anusha-ragunathan Thank you for looking into this.
I didn't have the issue on a fresh install of 1.11-rc2.
$ sudo aa-status
apparmor module is loaded.
10 profiles are loaded.
10 profiles are in enforce mode.
   /sbin/dhclient
   /usr/bin/lxc-start
   /usr/lib/NetworkManager/nm-dhcp-client.action
   /usr/lib/NetworkManager/nm-dhcp-helper
   /usr/lib/connman/scripts/dhclient-script
   /usr/sbin/tcpdump
   docker-default
   lxc-container-default
   lxc-container-default-with-mounting
   lxc-container-default-with-nesting
0 profiles are in complain mode.
1 processes have profiles defined.
1 processes are in enforce mode.
   /sbin/dhclient (545)
0 processes are in complain mode.
0 processes are unconfined but have a profile defined.

$ sudo cat /etc/apparmor.d/docker

#include <tunables/global>

profile docker-default flags=(attach_disconnected,mediate_deleted) {

  #include <abstractions/base>

  network,
  capability,
  file,
  umount,

  deny @{PROC}/* w,   # deny write for all files directly in /proc (not in a subdir)
  # deny write to files not in /proc/<number>/** or /proc/sys/**
  deny @{PROC}/{[^1-9],[^1-9][^0-9],[^1-9s][^0-9y][^0-9s],[^1-9][^0-9][^0-9][^0-9]*}/** w,
  deny @{PROC}/sys/[^k]** w,  # deny /proc/sys except /proc/sys/k* (effectively /proc/sys/kernel)
  deny @{PROC}/sys/kernel/{?,??,[^s][^h][^m]**} w,  # deny everything except shm* in /proc/sys/kernel/
  deny @{PROC}/sysrq-trigger rwklx,
  deny @{PROC}/mem rwklx,
  deny @{PROC}/kmem rwklx,
  deny @{PROC}/kcore rwklx,

  deny mount,

  deny /sys/[^f]*/** wklx,
  deny /sys/f[^s]*/** wklx,
  deny /sys/fs/[^c]*/** wklx,
  deny /sys/fs/c[^g]*/** wklx,
  deny /sys/fs/cg[^r]*/** wklx,
  deny /sys/firmware/efi/efivars/** rwklx,
  deny /sys/kernel/security/** rwklx,

  # suppress ptrace denials when using 'docker ps' or using 'ps' inside a container
  ptrace (trace,read) peer=docker-default,

}

I didn't have /etc/apparmor.d/docker-engine (which seems to be the cause of the issue) on this fresh installation.

@AkihiroSuda Did you have docker 1.10.0-rc1 installed on that machine previously? There were issues with that apparmor profile, and due to a bug in apparmor on Ubuntu, the profile was always applied, even in complain mode; we removed that profile from 1.10.0-rc2, but if you had it on your system already, you had to manually remove it

@thaJeztah Yes, perhaps.
Thank you for the information!
I'm closing this because it won't affect production.

Thanks @AkihiroSuda and happy to hear it wasn't anything serious in our code base :sweat_smile:


Training and Inference Time

Thanks for sharing this excellent work!
As I have not found the description of training and inference time in the paper, I would like to ask how long it will take to train or fine-tune the model. And how long does it take to perform depth inference on 5 images (1 reference + 4 source)?
It would be so nice of you to present the training and inference time with detailed description of the settings, e.g. the type and the number of GPUs, resolution of input images, size of training set, the corresponding accuracy/completeness, etc. Many thanks.

As declared in our paper, our Unification representation and Unified Focal Loss are memory-free and computation-free modules. Therefore, the computational cost and run-time of Unification+UFL is the same as our baseline CasMVSNet, and you can refer  to the Figure 1 in their paper. And if you adopt the adaptive aggregation strategy, the training time for an epoch increases by about 20 minutes from our experiments with the input size 512x640 in a Tesla V100 GPU. The corresponding performance will also be improved to a certain extent. You can refer to our Table 3 for more detail.

Thanks for your reply. One more question: do the new task formulation and the loss have influences on the convergence speed of training? Currently the number of training epochs seems to remain the same as CasMVSNet.

In fact, training for 12 epochs can already get the similar performance, so we think the convergence speed will at least not be slower than our baseline.

@prstrive I wonder if you can clarify -- did you used Tesla GPUs only for training on DTU? But in order I want to finetune model on BlendedMVS according to the description in your paper -- "...we fine- tune our model on BlendedMVS for 10 epochs. We take 7 images as the input with the original size of 768 × 576." -- then two Teslas won't be enough, isn't it?
Also, fyi you provide dataset_low_res.zip for BlendedMVS but it lacks certain directories from training_list, i.e. dataset_low_res/58c4bb4f4a69c55606122be4: No such file or directory. However I believe this dataset is similar to BlendedMVS provide in off.repository.


PR: Limit the number on lines in the history according to the relevant setting.

Pull Request Checklist

[ ] Read and followed this repo's Contributing Guidelines
[x] Based your PR on the latest version of the correct branch (master or 3.x)
[x] Followed PEP8 for code style
[x] Ensured your pull request hasn't eliminated unrelated blank lines/spaces,
modified the spyder/defaults directory, or added new icons/assets
[ ] Wrote at least one-line docstrings for any new functions
[ ] Added at least one unit test covering the changes, if at all possible
[ ] Described your changes and the motivation for them below
[x] Noted what issue(s) this pull request resolves, creating one if needed
[x] Included a screenshot, if this PR makes any visible changes to the UI

Description of Changes

A better thing would be to read the history from ipython, but I am not sure that this would work in every cases.
Issue(s) Resolved

Fixes #7080

Hello @impact27! Thanks for submitting the PR.

In the file spyder/plugins/history.py, following are the PEP8 issues :

Line 238:48: E225 missing whitespace around operator

Thanks for doing this! I need to head to work now, but a few quick things:

Please briefly describe what your PR actually does in the title and the description, so we can follow it easily
Fix the PEP8 issue
The test failures are legitimate, and it looks like its due to a typo in your code (see comment). It's easy to make, but if you're using Spyder or had actually tested the change, it would have been easy to catch
Can you add a test for this?
Not sure why you deleted the PR template that specified several of these things in a checklist...

Things look good, I just left a minor comment.

@ccordoba12 Do we want a regression test for this to ensure the change is and remains effective?

A test would be nice, yes, but it doesn't seem easy to implement. Besides, the code looks good.


Segmentation fault running component_container.

The compiler warning gives a hint:
...
<***removed***>/ws_rmw_zenoh/src/rmw_zenoh/rmw_zenoh_cpp/src/detail/liveliness_utils.cpp: In function ‘std::string rmw_zenoh_cpp::liveliness::zid_to_str(const z_id_t&)’:
<***removed***>/ws_rmw_zenoh/src/rmw_zenoh/rmw_zenoh_cpp/src/detail/liveliness_utils.cpp:228:12: warning: comparison of unsigned expression in ‘>= 0’ is always true [-Wtype-limits]
  228 |   for (; i >= 0; i--) {
      |          ~~^~~~
---
...

The variable i is of type size_t and will wrap. It is then possible that we will eventually access an invalid memory location.
https://github.com/ros2/rmw_zenoh/blob/e6864931cccfc9c504b4bbc3ae7899334e70fdae/rmw_zenoh_cpp/src/detail/liveliness_utils.cpp#L228
Terminal 1:
$ ros2 run rmw_zenoh_cpp rmw_zenohd

Terminal 2:
$ export RMW_IMPLEMENTATION=rmw_zenoh_cpp
$ ros2 run rclcpp_components component_container
[ros2run]: Segmentation fault

$ gdb --args /opt/ros/jazzy/lib/rclcpp_components/component_container
...
(gdb) run
Starting program: /opt/ros/jazzy/lib/rclcpp_components/component_container
...
[New Thread 0x7ffff64c66c0 (LWP 1346392)]
[New Thread 0x7ffff5cc56c0 (LWP 1346393)]
[New Thread 0x7ffff383f6c0 (LWP 1346394)]
[New Thread 0x7ffff363e6c0 (LWP 1346395)]
[New Thread 0x7ffff343d6c0 (LWP 1346396)]
[New Thread 0x7ffff323c6c0 (LWP 1346397)]
[New Thread 0x7ffff303b6c0 (LWP 1346398)]
[New Thread 0x7ffff2e3a6c0 (LWP 1346399)]
[New Thread 0x7ffff2c396c0 (LWP 1346400)]

Thread 1 "component_conta" received signal SIGSEGV, Segmentation fault.
0x00007ffff543d952 in rmw_zenoh_cpp::liveliness::zid_to_str[abi:cxx11](z_id_t const&) (id=...) at <***removed***>/ws_rmw_zenoh/src/rmw_zenoh/rmw_zenoh_cpp/src/detail/liveliness_utils.cpp:229
229     ss << static_cast<int>(id.id[i]);
(gdb) bt
#0  0x00007ffff543d952 in rmw_zenoh_cpp::liveliness::zid_to_str[abi:cxx11](z_id_t const&) (id=...) at <***removed***>/ws_rmw_zenoh/src/rmw_zenoh/rmw_zenoh_cpp/src/detail/liveliness_utils.cpp:229
#1  0x00007ffff54148b8 in rmw_zenoh_cpp::GraphCache::GraphCache (this=0x5555555db750, zid=...) at <***removed***>/ws_rmw_zenoh/src/rmw_zenoh/rmw_zenoh_cpp/src/detail/graph_cache.cpp:72
#2  0x00007ffff546978a in std::make_unique<rmw_zenoh_cpp::GraphCache, z_id_t&> () at /usr/include/c++/13/bits/unique_ptr.h:1070
#3  0x00007ffff546717e in rmw_init (options=0x5555555a5798, context=0x5555555a56a0) at <***removed***>/ws_rmw_zenoh/src/rmw_zenoh/rmw_zenoh_cpp/src/rmw_init.cpp:181
#4  0x00007ffff692abe7 in rcl_init (argc=1, argv=0x7fffffffb478, options=0x55555556f420, context=0x5555555a5500) at <***removed***>/ws_rmw_zenoh/src/rcl/rcl/src/rcl/init.c:306
#5  0x00007ffff796104f in rclcpp::Context::init (this=0x5555555a5200, argc=1, argv=0x7fffffffb478, init_options=...) at <***removed***>/ws_rmw_zenoh/src/rclcpp/rclcpp/src/rclcpp/context.cpp:208
#6  0x00007ffff7c3d3a7 in rclcpp::init (argc=1, argv=0x7fffffffb478, init_options=..., signal_handler_options=rclcpp::SignalHandlerOptions::All) at <***removed***>/ws_rmw_zenoh/src/rclcpp/rclcpp/src/rclcpp/utilities.cpp:41
#7  0x0000555555556323 in main ()

rmw_zenoh: e6864931cccfc9c504b4bbc3ae7899334e70fdae
ROS release: jazzy

Thanks for the ticket. This is a regression from #190

It is working for me now. Thank you for the quick fix!


Adding multiple js-social instances on one page

I'm trying to add js-social multiple times on a page but only one instance shows. Is there a way to display more than one instances?

Sure, multiple instances are supported.
Please, provide more info how exactly you configure several jssocials.

Thanks for replying so quickly @tabalinas ! In a nut shell I have something like this - https://jsfiddle.net/2n23owj7/1/ I have three instances of js-social but for some reason only one instance is showing. Do you know why this is? Thanks again.

Hi, I ran into the same with my Joomla! plugin that utilizes the jssocials script.
What I did to solve it was make every occurrence of the buttons 'unique'
like this:


Unit test for CLI requires binary

Description

If the binary hasn't been rebuilt recently, then this can cause test errors. Two ideas:

Tag these tests as e2e and provide instructions for what to do before running e2e tests (make timestamp-cli)
Build the CLI while running the test - if we do this, we should build it in a temporary folder and clean up after the test finishes. I kind of prefer this, as I'd rather not have e2e tagged tests if we can avoid them

I would like to work on this.

Thanks, assigned you! I'd prefer to avoid tagging the tests if possible and go with the second approach. If you find a better way to do this also, great!

At this moment the make test https://github.com/sigstore/timestamp-authority/blob/06993cb6c1f322f048fab2558b167bdbe7c4ccb6/Makefile#L81 builds the CLI prior to running tests,

Build the CLI while running the test - if we do this, we should build it in a temporary folder and clean up after the test finishes. I kind of prefer this, as I'd rather not have e2e tagged tests if we can avoid them

Is there a reason for building CLI in a temporary folder?

I was thinking this was useful for running go test without make. The main issue is that once you run make test, it'll cache a binary that'll be used for subsequent runs of go test without rebuilding, so you might get inaccurate results (I ran into this).

Instead of building the CLI in a temporary folder, could we use a .PHONY target on timestamp-cli to solve this problem?

Would that force rebuilding? I think that's a good enough solution if so!

I have addressed this issue in this PR #159


NullPointerException always occur in SparkDl4jMultiLayer

This line always cause a null pointer exception because conf is always null in this constructor.
Can the following resolve this issue??
public SparkDl4jMultiLayer(SparkContext sparkContext, MultiLayerNetwork network) {
    ....
    this.conf = network.getLayerWiseConfigurations().clone();

Fixed in latest commit.


CRS checks

current checks would benefit from going via https://hypertidy.github.io/vapour/reference/vapour_srs_wkt.html
then error when "" is returned? maybe just warn?

sorted here: https://github.com/Permian-Global-Research/ezwarp/pull/18/commits/8982a80da4336844fbafbf06e6f2dc3b0b765db6


Double times played

Issue description
I've written completely my own webpage containing my mp3 collection. With the 3.x version there was no problem.
My script records how many times a track is played, or is requested.
In the new/latest version I have the problem that tracks are double counted...
how is this possible, and how can I circumvent this??
I use following code to record the times a track is played:
$sql->query("UPDATE track SET played = played + '1' WHERE trid = ".sqlesc($trid));
$sql->query("UPDATE stats SET tot_played = tot_played + '1'");
statsset("played=played+'1'");
}
$packet_interval = 0.2;
$bandlim         = 800;
$traffic_limit   = 300;
$file_cache      = false;
$packet_size = ceil($traffic_limit * 3.4133 * $bandlim);
session_write_close();
$sizet = filesize($filepath);
$fmime = get_mimetype($filepath);
header("Content-Type: ".$fmime."");
/*
if ($ext == 'mp3')  header("Content-Type: audio/mpeg");
if ($ext == 'flac') header("Content-Type: audio/flac");
if ($ext == 'ogg')  header("Content-Type: audio/ogg");
if ($ext == 'wma')  header("Content-Type: audio/x-ms-wma");
*/
header("Cache-Control: no-store, no-cache, must-revalidate, post-check=0, pre-check=0");
header("Pragma: no-cache");
header("Content-Disposition: attachment; filename=temp_mp3");
header("Content-Length: ".$sizet);
header("Content-Transfer-Encoding: binary\n\n");
$fh = fopen($filepath,'rb');
while (!feof($fh)) {
set_time_limit(0);
list($usec, $sec) = explode(' ', microtime());
$start            = ((float)$usec + (float)$sec);
print(fread($fh, $packet_size));
list($usec, $sec) = explode(' ', microtime());
$stop             = ((float)$usec + (float)$sec);
$time_diff        = $stop - $start;
if ($time_diff < (float)$packet_interval)
usleep((float)$packet_interval * 1000000 - (float)$time_diff * 1000000);
ob_flush();
};
Environment
firefox

Amplitude.js Version: 4.10

Steps to reproduce the issue

Please send me a mail so I can send you a link<EMAIL_ADDRESS>

What is expected?
tracks should be counted once
Link to where issue can be reproduced
I can send you a link to show whats happening exactly. I can't make these url's public...

Hi @StuffyNL Could you send a link to the email in my bio: https://github.com/danpastori. I don't know why this would be counting twice.  Amplitude should be pulling the audio once unless it's a live stream then it will reconnect which could trigger it.

you have mail ;)

Hi Dan,
Thank you for the mail, but somehow I can't find it back.... Can you put what you wrote here, so others that may have the same problem, know what they have to do to resolve this.
I remember something about a XHR request.
Is there an example of responding to a XHR request in php? Or is it a combo of JS and PHP?
For as long I'm bizzy with per. phyton,js, and so on and on, I never had anything to do with things like that..

Is there anybody out here who has some PHP code/JS code for reacting on a XHR request??

@StuffyNL I believe you have two click handlers bound to the element that send the XHR request or you initialized AmplitudeJS twice.

Hi Dan,
thank you for your answer.
I've been bizzy quite some while with this issue, but thanks to some debugging I have managed to resolve this issue.
The only part of that is making a XHR request is the part that makes the Waveform. After removing that part, and some other things that was related to it, I have no XHR requests anymore, nor double countings. And I'm not using the Waveforms anyway, so for me has this issue been solved.
I have a question (actually 2) to you Dan: Can you make the Waveforms switchable, by adding a config var?
Now the waveforms are created whether you use them or not, at least I think it does. By making it switchable that code can be skipped.
Second question: Can you make the Waveform part as a FX plugin for example?
amplitude.test.zip
Attached is the edited JS, that has been edited.
Thanks.

for me this issue has been solved.
therefore closed.


Cant update access of a blob

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az storage fs access set
Errors:
The command failed with an unexpected error. Here is the traceback:
At least one parameter should be set for set_access_control API
Traceback (most recent call last):
  File "/usr/local/Cellar/azure-cli/2.29.0/libexec/lib/python3.9/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/usr/local/Cellar/azure-cli/2.29.0/libexec/lib/python3.9/site-packages/azure/cli/core/commands/__init__.py", line 657, in execute
    raise ex
  File "/usr/local/Cellar/azure-cli/2.29.0/libexec/lib/python3.9/site-packages/azure/cli/core/commands/__init__.py", line 720, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/usr/local/Cellar/azure-cli/2.29.0/libexec/lib/python3.9/site-packages/azure/cli/core/commands/__init__.py", line 712, in _run_job
    return cmd_copy.exception_handler(ex)
  File "/usr/local/Cellar/azure-cli/2.29.0/libexec/lib/python3.9/site-packages/azure/cli/command_modules/storage/__init__.py", line 371, in new_handler
    first(ex)
  File "/usr/local/Cellar/azure-cli/2.29.0/libexec/lib/python3.9/site-packages/azure/cli/command_modules/storage/__init__.py", line 370, in new_handler
    raise ex
  File "/usr/local/Cellar/azure-cli/2.29.0/libexec/lib/python3.9/site-packages/azure/cli/core/commands/__init__.py", line 691, in _run_job
    result = cmd_copy(params)
  File "/usr/local/Cellar/azure-cli/2.29.0/libexec/lib/python3.9/site-packages/azure/cli/core/commands/__init__.py", line 328, in __call__
    return self.handler(*args, **kwargs)
  File "/usr/local/Cellar/azure-cli/2.29.0/libexec/lib/python3.9/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/usr/local/Cellar/azure-cli/2.29.0/libexec/lib/python3.9/site-packages/azure/multiapi/storagev2/filedatalake/v2020_02_10/_path_client.py", line 333, in set_access_control
    raise ValueError("At least one parameter should be set for set_access_control API")
ValueError: At least one parameter should be set for set_access_control API

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az storage fs access set --owner={} -f {} -p {} --account-name {} --auth-mode {} --account-key {}

Expected Behavior
Environment Summary
macOS-11.6-x86_64-i386-64bit, Darwin 20.6.0
Python 3.9.7
Installer: HOMEBREW

azure-cli 2.29.0 *

Extensions:
storage-preview 0.7.4

Additional Context

storage

@blcksrx Please upgrade to latest version azure cli 2.39.0


Proposal: Deprecate UKMod and make UMM manage BepInEx plugins

Introduction
Right now, UMM uses a custom framework for defining mods and a custom system for loading them (which I will refer to as "UKMod"). UKMod has several features that BepInEx does not:

Ability to disable mods at runtime
GUI for managing mods in-game
Attributes for mods to specify descriptions and Ultrakill-specific metadata (like disabling leaderboard submissions)

On the contrary, UKMod also has several drawbacks:

Added confusion for the user as there are two different folders to install mods in and only some mods appear in the GUI.
Technical debt; having a custom system opens many opportunities for bugs and oversights.

In this proposal I suggest that we deprecate UKMod and change the "manager" part of UMM to manage BepInEx plugins.
Deprecating UKMod
Deprecate the following types/members:
UMM.Loader.UltraModManager
UMM.UKMod
UMM.UKPlugin
UMM.UKDependency
UMM.ModInformation
UMM.UKAPI.AllModInfoClone
UMM.UKAPI.AllLoadedModInfoClone
Move UMM.Loader.UltraModManager.outdated, UMM.Loader.UltraModManager.devBuild, and UMM.Loader.UltraModManager.newLoaderVersion to UMM.VersionHandler.
Features we miss out on
By deprecating UKMod we seemingly miss out on its features. However, this is not necessarily the case.
Metadata (Display Names, Descriptions)
Since Thunderstore now supports Ultrakill mods, we can use the Thunderstore package specification (https://thunderstore.io/package/create/docs/) to allow mods to define descriptions and display names. As for other data such as disabling cybergrind submissions, we can use a umm.json file placed in the same directory as the Thunderstore manifest to store extra data:
{
    "disable_cybergrind_leaderboard_submissions" : true
}

Disabling mods at runtime
TMM/r2modman both support disabling mods before running the game. Although this is arguably not as good as being able to disable mods at runtime, I believe this is adequate. Ultrakill does not take much time at all to start up, meaning one can fairly easily exit the game, disable mods in their mod manager of choice, and start the game back up.
The new way of making mods
From a mod developer's perspective, the new way of making UMM mods is the exact same as a regular Thunderstore BepInEx plugin; with the mod DLL, manifest, readme, and icon. Additionally, an optional umm.json can be provided that grants the mod developer the ability to disable cybergrind score submissions with the disable_cybergrind_leaderboard_submissions property.
From a mod user's perspective, downloading mods is easy via r2modman, or they can download mods manually into the BepInEx plugins directory.
 Fell free to ask me any questions. Until then I'll be waiting for the all-clear to start implementing this.

Would you still be willing to do this? Because one addition I would make is maybe making UKAPI a separate DLL, that was people can choose to not have the GUI if they don't want it.
At the time, the reason that I didn't merge this is because I didn't want to get rid of UKMods (as I've put a lot of effort into them), however that was just me being selfish. I think as a whole, this proposal would greatly benefit the modding community.

Yeah I've decided to move ahead with this, if you don't have the time anymore I can do it, however if you also want to work on it go ahead.

Ok


Add documentation about how to modify NC_DISPATCH_VERSION

re: Issue https://github.com/Unidata/netcdf-c/issues/2582
Update the file netcdf-c/docs/dispatch.md to include an appendix that discusses how to properly update the NC_DISPATCH_VERSION for the dispatch table.

Ok, for whatever reason I cannot merge these manually, so I am copying the changes over by hand.  It's only 2 files, so not a big issue.


Support for package details sections in resource docs

Builds on pulumi/pulumi#4256 and https://github.com/pulumi/pulumi-terraform-bridge/pull/148.
Part of pulumi/home#734.

I'm going to go ahead and merge this since https://github.com/pulumi/pulumi/pull/4256 is merged now, and  generates the .package-details class in doc codegen, so I want to make sure this CSS is available in the docs site before anything ships with that.
@cnunciato Would appreciate your review and improvements to the CSS I added here though! :smile:

I'll merge this. Christian and I chatted about this PR. I would like to get this in before my PR.


unhandled promise rejection: Cannot read property 'split' of undefined

Hi since update to 1.6. i have an issue, so I goes back to 1.5.3 and all is working fine

stiebel-isg.0
2021-01-11 17:46:30.118
error
at processTicksAndRejections (internal/process/task_queues.js:97:5)

stiebel-isg.0
2021-01-11 17:46:30.118
error
at /opt/iobroker/node_modules/iobroker.js-controller/lib/adapter.js:8147:76

stiebel-isg.0
2021-01-11 17:46:30.118
error
at Object.ready (/opt/iobroker/node_modules/iobroker.stiebel-isg/main.js:67:59)

stiebel-isg.0
2021-01-11 17:46:30.118
error
(25467) TypeError: Cannot read property 'split' of undefined

stiebel-isg.0
2021-01-11 17:46:30.115
error
(25467) unhandled promise rejection: Cannot read property 'split' of undefined

Did you edit the fields on the expert tab of the adapter settings?
Maybe there is an error within these fields.

Did you edit the fields on the expert tab of the adapter settings?
Maybe there is an error within these fields.

no, I only updated to 1.6 and change nothing else.
I have to do this ?

no, I only updated to 1.6 and change nothing else.
I have to do this ?

Normally you don't have to. It should work properly without reconfiguration. But you can shorten the lines for sites you don't need.
Please try to restart or even reinstall the adapter. The datapoints will be there afterwards, so don't worry.

Normally you don't have to. It should work properly without reconfiguration. But you can shorten the lines for sites you don't need.
Please try to restart or even reinstall the adapter. The datapoints will be there afterwards, so don't worry.

ok after reupdate it looks better, thx. for support

ok after reupdate it looks better, thx. for support


"this drive can't handle this image" with Greaseweazle-produced SCPs

I read a set of Mac 800KB SCP images using the official Greaseweazle client, and tried decoding them using Fluxengine. It claims that it can't handle them, with the full message "Error: this drive can't handle this image, because the drive TPI doesn't divide neatly into the layout TPI". Using Fluxengine itself to directly read from the Greaseweazle works, however. I am able to decode these files other software, and in previous versions of Fluxengine I was able to decode GW-produced SCP images just fine.
Is this a bug in Fluxengine, or is it actually reading the disk differently when Fluxengine itself directly uses the GW?

It's not so much a FluxEngine bug as a misfeature. Try adding --tpi=135. This is all part of a system I put in where you'd specify the tpi of both the format and the drive, allowing the system to decide how to do track stepping. Unfortunately it doesn't map very well to what people actually do (e.g. flux images coming from other systems don't know the tpi...) and I need to rethink it.


Add missing changes in the generated buildRun CRD

Add missing spec.output declaration in the buildrun. I think
this was missing in https://github.com/redhat-developer/build/pull/233
Add missing timeout in spec.status.buildSpec, I think this was missed
somehow after https://github.com/redhat-developer/build/pull/158 was merged.

Also, this can be reproduced by doing operator-sdk generate crds

/assign @xiujuan95

/ok-to-test

/lgtm

/approve


No validation for Events?

I find no validation logic for event objects

@saad-ali

Did you see pkg/api/validation/events.go?

feel free to reopen if that doesn't do what you want.

Sorry - looked for it in the wrong place
On Tue, Jan 20, 2015 at 4:09 PM, Eric Tune<EMAIL_ADDRESS>wrote:

feel free to reopen if that doesn't do what you want.
--
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/3638#issuecomment-70760937
.


Refactor CLI to support subcommands

Description
Add support for subcommands for CLI:

install
uninstall
plugins
transtypes
deliverables
version

Subcommand conversion is implied and cannot be explicitly used.
$ dita install org.dita.eclipsehelp
$ dita uninstall org.dita.eclipsehelp
$ dita plugins
$ dita transtypes
$ dita deliverables --project=base.json
$ dita version

The old option style functionality is retained for backwards compatibility.
$ dita --install=org.dita.eclipsehelp
$ dita --uninstall org.dita.eclipsehelp
$ dita --plugins
$ dita --transtypes
$ dita --deliverables --project=base.json
$ dita --version

Each subcommand has its own help, accessed with help option
$ dita install --help

Motivation and Context
Refactoring the CLI to support subcommands clarifies the different roles the CLI has. New functionality will be easier to add as a new subcommand without making the CLI more complex.
How Has This Been Tested?
Added unit tests.
Type of Changes

New feature (non-breaking change which adds functionality)

Not sure if we should use dita deliverables --project=base.json or dita deliverables base.json`, i.e. prefer arguments over options.


Add a unit test suite for GDScript

This tests run-time script loading and is a complement to the existing integration tests.

Shouldn't this go in modules/gdscript/tests/ together with other GDScript tests? And then might need a more explicit name as test_gdscript.h already exists.

Shouldn't this go in modules/gdscript/tests/ together with other GDScript tests? And then might need a more explicit name as test_gdscript.h already exists.

Yes, but I'm not sure if this will work, but should likely work... Yet maybe Calinou has other motivations for doing so. But yeah, I'd prefer to have all GDScript related tests to be under modules/.

Shouldn't this go in modules/gdscript/tests/ together with other GDScript tests? And then might need a more explicit name as test_gdscript.h already exists.

Done. I ended up just moving the test case into the existing GDScript test suite.

Thanks!

Hm, this is actually crashing for me because the language has not been initialized.


Error in app.module.ts after on adding AngularMultiSelectModule

I'm using angular version 6 and am trying to integrate angular2-multiselect-dropdown in my app. On trying to run the app, after following the usage steps from this reference:
http://cuppalabs.github.io/components/multiselectDropdown/
https://www.npmjs.com/package/angular2-multiselect-dropdown
I get error as below:
Date: 2019-05-02T07:41:56.573Z
Hash: 8b7ca08bfd8ad516f706
Time: 27855ms
chunk {main} main.js, main.js.map (main) 4.99 MB [initial] [rendered]
chunk {polyfills} polyfills.js, polyfills.js.map (polyfills) 473 kB [initial] [rendered]
chunk {runtime} runtime.js, runtime.js.map (runtime) 5.22 kB [entry] [rendered]
chunk {scripts} scripts.js, scripts.js.map (scripts) 158 kB  [rendered]
chunk {styles} styles.js, styles.js.map (styles) 255 kB [initial] [rendered]
chunk {vendor} vendor.js, vendor.js.map (vendor) 7.43 MB [initial] [rendered]
ERROR in ./src/app/app.module.ts
Module not found: Error: Can't resolve 'angular2-multiselect-dropdown/angular2-multiselect-dropdown' in 'D:\Projects\Gosnow\src\gosnow_admin\src\app'
i ｢wdm｣: Failed to compile.
I tried with "angular2-multiselect-dropdown": "^4.2.2" and "angular2-multiselect-dropdown": "^4.2.0" both to get the same error.
May I please get some help on this?

I think the import statement in the documentation here http://cuppalabs.github.io/components/multiselectDropdown/#Usage is wrong.
import { AngularMultiSelectModule } from 'angular2-multiselect-dropdown/angular2-multiselect-dropdown';
should be
import { AngularMultiSelectModule } from 'angular2-multiselect-dropdown';

@fridoo you are right, I made the changes and its working now. Thanks for the help! :)

@CuppaLabs , can you please make the necessary changes to the documentation as mentioned by @fridoo  in his comment so as to avoid any such errors in future. Thanks!

Fixed.


Update service-base to version 0.1.1

Fix #6

Pull Request Test Coverage Report for Build 607

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

Totals

Change from base Build 604:
0.0%

Covered Lines:
1758

Relevant Lines:
2096

💛  - Coveralls