Working examples?

I tried following README to setup a project , bug only get error: expected Fn, Struct, Var, Let, Const, Alias or Override, but found Ident  on the include line.
Need some help, thx

Also struggling to get includes working.

Thanks for your interest in this crate!
So far I've just been adding features as I needed them in my project, but if there's interest in using this library in other projects then it makes sense for me to spend the next few days improving documentation and error messages. Hopefully it'll be easier to start using after then.

I've spent some time improving error messages, import resolution logic and documentation - there's now 3 working examples and I've fixed the 'getting started' snippet. Let me know if you're still having any issues getting started and if you are, what you're trying, so that I know what to document further :)


alerts: best or worst performing in period is not aware of non-judgemental measures like DOACS

The current logic will only show those in the "lowest" decile, but we want to show DOACS if it's in either the "lowest" or "highest".

Yes, this has been implemented for a while. This month, there were 30 alerts where a non-judgemental measure was the most outliery in the section and it was in the lowest decile, and 63 alerts where it was in the highest.

Example of the former (email, measure in question)
Example of the latter  (email, measure in question)

Please close this issue if I understood your concern correctly and it's OK!

Great, job done


Error with start undefined

From @ggshihai on March 27, 2017 2:9
Ionic version:  (check one with "x")
[ ] 1.x
[x] 2.x
I'm submitting a ...  (check one with "x")
[x ] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:

D:\ionic>ionic start myApp tabs
Expected behavior:

Steps to reproduce:

Related code:
insert any relevant code here

Other information:

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
Your system information:
ordova CLI: 6.5.0
Ionic CLI Version: 2.2.1
Ionic App Lib Version: 2.2.0
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Windows 7
Node Version: v6.10.1
Xcode version: Not installed
insert the output from ionic info here

D:\ionic>ionic start myApp tabs
Creating an Ionic app in D:\ionic\myApp based on the tabs template.

Downloading: https://github.com/driftyco/ionic-app-base/archive/master.zip
Downloading: https://github.com/driftyco/ionic-starter-tabs/archive/master.zip
Installing npm packages (may take a minute or two)...
Error with start undefined
Error Initializing app: There was an error with the spawned command: npminstall
There was an error with the spawned command: npminstall

_Copied from original issue: driftyco/ionic#10926_

From @realfreebird on March 27, 2017 6:59
Hi,
I had the same error.
Since I have a local npm repository, I solved it by running the following command:
npm config set @ionic-native:registry https://registry.npmjs.org/

thks

@jgw96
Hi, I got the same issue since a few hours ago, cannot be resolved by your command:
npm config set @ionic-native:registry https://registry.npmjs.org/
Any suggestions?

This issue has been automatically closed because it had the v2 label. Please install the new CLI v3 (npm install -g ionic@latest). See CHANGELOG.md#Upgrading from CLI v2 for details.
If this issue is still a problem in CLI v3, please create a new issue. Thank you!


does pretrained model not include space character?

hello everyone,
i use TPS-ResNet-BiLSTM-Attn-case-sensitive.pth pretrained model, is this pretrained model include space character?

@akbarwijayaa
Hi
Did you get this error too while using : TPS-ResNet-BiLSTM-Attn-case-sensitive.pth
"""
raise RuntimeError('Error(s) in loading state_dict for {}:\n\t{}'.format(
RuntimeError: Error(s) in loading state_dict for DataParallel:
size mismatch for module.Prediction.attention_cell.rnn.weight_ih: copying a param with shape torch.Size([1024, 352]) from checkpoint, the shape in current model is torch.Size([1024, 294]).
size mismatch for module.Prediction.generator.weight: copying a param with shape torch.Size([96, 256]) from checkpoint, the shape in current model is torch.Size([38, 256]).
size mismatch for module.Prediction.generator.bias: copying a param with shape torch.Size([96]) from checkpoint, the shape in current model is torch.Size([38])."""
if yes, can you please share with me the solution?
Thanks in advance

@akbarwijayaa Hi Did you get this error too while using : TPS-ResNet-BiLSTM-Attn-case-sensitive.pth """ raise RuntimeError('Error(s) in loading state_dict for {}:\n\t{}'.format( RuntimeError: Error(s) in loading state_dict for DataParallel: size mismatch for module.Prediction.attention_cell.rnn.weight_ih: copying a param with shape torch.Size([1024, 352]) from checkpoint, the shape in current model is torch.Size([1024, 294]). size mismatch for module.Prediction.generator.weight: copying a param with shape torch.Size([96, 256]) from checkpoint, the shape in current model is torch.Size([38, 256]). size mismatch for module.Prediction.generator.bias: copying a param with shape torch.Size([96]) from checkpoint, the shape in current model is torch.Size([38])."""
if yes, can you please share with me the solution?
Thanks in advance

Please help me if you find a solution for this issue?


popover: allow specifying default popoverTrigger via $tooltipProvider

To support both devices with and without touch, something like popover-trigger="mouseenter click" has to be used.
It would be really nice to provide this as a default via $tooltipProvider.

Per the docs, you can already configure the triggers globally.
If you are asking about supporting dual triggers, we do not support that as documented in #4007.

I have read the doc on $uibTooltipProvider, but as far as I understand, it's not possible to provide default value for popover-trigger via options. Can you elaborate a bit on what you mean?
Basically, I want to be able to configure $uibTooltipProvider so that every popover works like it had popover-trigger="mouseenter click" attribute.

@hoxu, there's no bookmark link that I can explicitly point you to, but if you read the docs for popover, you will see:
The popover directive also supports various default configurations
through the $tooltipProvider. See the tooltip section for
more information.

I'm not really sure how to make it more clear than that.

Yes, and $uibTooltipProvider section says:
options(obj): Provide a set of defaults for certain tooltip and popover attributes.
Currently supports 'placement', 'animation', 'popupDelay', and appendToBody.

popoverTrigger is not one of the supported properties (yet), right?

@hoxu, popover is build on top of tooltip so as a matter of fact, it is.

    $uibTooltipProvider.options({
        'trigger': "mouseenter click"
    });


Geo Replication Topic Policy

Is your feature request related to a problem? Please describe.
This feature request is not related to any problem, at least not that I am familiar with.
Describe the solution you'd like
I think the current architecture and way of configuring pulsar it will be nice to introduce geo-replication policy on topic level. Unfortunately, I dont know the details of the internal implementation of pulsar and if you have considered such feature before and if you did what would be the impact of it.
Describe alternatives you've considered
There is a way of making the so-called selective replication which ideally mimics the functionality that one geo-replication topic level policy would, but I think its rather odd making publishers in the clients to configure which clusters they should use. This should really be ops part since this is something you shouldn't (in fact maybe never) really change that often after you've configured it once in your environment.
Ofc there is another alternative - creating a dedicated namespace for this topic but this will only prove the fact that such configuration is needed on topic level. (imho)
Bottom line, I like where Pulsar is heading in terms of configuration and I think it should make sense to have one of its key functionalities able to be configured as flexible as possible, also what I said, I really think geo-replication should remain in the broker/server/ops domain of pulsar and developers (like me) shouldn't worry about this in the code. Not to mention that having to know which clusters you are connecting to seems like an abstraction leak to me.

The issue had no activity for 30 days, mark with Stale label.

Closed as stale and no one works on it. Please create a new issue if it's still relevant to the maintained versions.


fix(FocusTrapZone): Do not force focus inside focus trap zone on outside focus

Warnings

:warning:
There are no updates provided to CHANGELOG. Ensure there are no publicly visible changes introduced by this PR.

  Generated by :no_entry_sign: dangerJS

Keeping some context for the future here:
This behavior changed between different versions of (Fabric) FocusTrapZone. So reverting to the previous behavior as it breaks some of the scenarios in Stardust.
To use the default Fabric behavior, we can still use
trapFocus={ {
  forceFocusInsideTrapOnOutsideFocus: true
} }

We should probably use false for popup by default and true for dialog (?) but it requires some testing.


New release?

In master you have the --exit-zero flag available, and other new features, but no release for over a year. Is it possible to just release an up to date version?
Thanks!

This looks to me to be a duplicate of #611.
(ps. I am also interested in a new release but the maintainers should perhaps not be bothered by a lot of duplicate issues. ds.).

1.6.3 is out on PyPI. Thanks!


Appstudio update test-component-pac-egia

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-egia-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
16 seconds
init

✅ Succeeded
18 seconds
clone-repository

✅ Succeeded
43 seconds
build-container

✅ Succeeded
20 seconds
deprecated-base-image-check

✅ Succeeded
19 seconds
sanity-inspect-image

✅ Succeeded
3 minutes
clair-scan

✅ Succeeded
51 seconds
clamav-scan

✅ Succeeded
16 seconds
sbom-json-check

✅ Succeeded
14 seconds
sanity-label-check

✅ Succeeded
14 seconds
sanity-optional-label-check

✅ Succeeded
7 seconds
show-summary

Pipelines as Code CI/test-component-pac-egia-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
21 seconds
clone-repository

✅ Succeeded
29 seconds
build-container

✅ Succeeded
8 seconds
deprecated-base-image-check

✅ Succeeded
9 seconds
sanity-inspect-image

✅ Succeeded
35 seconds
clamav-scan

✅ Succeeded
11 seconds
clair-scan

✅ Succeeded
9 seconds
sbom-json-check

✅ Succeeded
13 seconds
sanity-optional-label-check

✅ Succeeded
13 seconds
sanity-label-check

✅ Succeeded
7 seconds
show-summary


Assertion error messages have local file paths

An assertion message has a local file path instead of variables.
expected:
assert(!!database_ === true);
       |||         |
       ||undefined false
       |true
       false

actual:
assert(!!dataD:/<dir>/<file>_ === true)
       |||         |
       ||undefined false
       |true
       false
...
... /power-assert.js:6093)

@falsandtru power-assert does not modify original expression, only canonicalize it in one line format.
So I think it is not by power-assert but some other npm module.
The output assert(!!dataD:/<dir>/<file>_ === true) seems that some module has replaced the word base with your project dir (database_ => dataD:/<dir>/<file>_)

I use karma-espower-preprocessor. Is this a cause?

I use karma-espower-preprocessor. Is this a cause?

I don't think so. In your output,
assert(!!dataD:/<dir>/<file>_ === true)
       |||         |
       ||undefined false
       |true
       false

vertical bar for === is out of alignment.
This means that the replacement (database_ => dataD:/<dir>/<file>_) has occurred after the power-assert processing.
If power-assert and related modules are wrong, the vertical bar will match.
So some reporter module for karma would be the criminal.

Now I can reproduce this problem in my repository(however, this is complex). Can you test that repository?

Thanks for repro case. However I don't have enough time now so I review it in a few days.

Thanks. This is details.
assert(!database_ === true);

            assert(assert._expr(assert._capt(assert._capt(!assert._capt(database_, 'arguments/0/left/argument'), 'arguments/0/left') === true, 'arguments/0'), {
                content: 'assert(!database_ === true)',
                filepath: 'D:/<path>',
                line: 40
            }));

Strange to say, I got correct assertion messages from browser, but I got wrong assertion messages from terminal.
I will provide repro in January ASAP.
Windows7 x64

@falsandtru Any advance on this issue?

Sorry, I close this issue because I overwrote that code. I think this problem probably caused by terminal. Thanks for the response.

@falsandtru Okay, thanks :)


Default symbol path for Microsoft Symbol Server should go to the secure site

The Symbol path for Microsoft Symbol server by default is set to SRVC:\Users\Ilias\AppData\Local\Temp\symbolshttp://msdl.microsoft.com/download/symbols
It probably needs to be changed to SRVC:\Users\Ilias\AppData\Local\Temp\symbolshttps://msdl.microsoft.com/download/symbols
i.e. go the secure site. The default wasnt working for me and then when I updated the link, it was able to download the symbols.

@leculver, @noahfalk, do you know if we should be defaulting to the SSL version of msdl.microsoft.com for symbols?

We should be defaulting to https, yes

I am not aware of any reason not to use the SSL option.

Sounds good.  Thanks all for the confirmation.

Brain, can we close this issue?

Fixed via #1125.


[Question] Will Qna support upload from file?

Will Qna support upload from file? What is the best way to add a chitchat qna from composer?

chitchat content can be imported from https://github.com/Microsoft/BotBuilder-PersonalityChat/tree/master/CSharp/Datasets .

Thanks!


[BUG] CausalLanguageModeling do not mask last input item

Bug description

The clm masking for last item only do not mask last item in input.
It will cause using the embedding of the label instead of mask.
I think following code needs to be fixed.
https://github.com/NVIDIA-Merlin/Transformers4Rec/blob/348c9636399535c566d20e8ebff2b7aa0775f136/transformers4rec/torch/masking.py#L298
Steps/Code to reproduce bug

import torch
from transformers4rec.torch import masking

item_ids = torch.tensor([[1, 2, 0], ])
mask = masking.CausalLanguageModeling(hidden_size=10, train_on_last_item_seq_only=True)
masking_info = mask.compute_masked_targets(item_ids, training=True)
print(masking_info)

MaskingInfo(schema=tensor([[ True,  True, False]]), targets=tensor([[2, 0, 0]]))

Expected behavior

MaskingInfo(schema=tensor([[ True,  False, False]]), targets=tensor([[2, 0, 0]]))

Environment details

Transformers4Rec version: 23.08.00

Additional context

@sungho-ham thanks. would you like to create a PR for this?

I think this line of code need to be removed:
https://github.com/NVIDIA-Merlin/Transformers4Rec/blob/d0cce61b988a1e923545f94ac22499fb57928d18/transformers4rec/torch/masking.py#L298
As solution just use the mask_label from predict_all() above.
And I think the reason why current code somehow works is because of this part:
https://github.com/NVIDIA-Merlin/Transformers4Rec/blob/d0cce61b988a1e923545f94ac22499fb57928d18/transformers4rec/torch/masking.py#L318-L337
Given input sequence without padding [1,2,3], the mask schema generated by current code during evaluation will be [True, True, True], which exposes the last item. However the apply_mask_to_inputs will replace the last item with 0 embedding. And since the  schema are all True, no mask embedding will be applied on input. I think in this case 0 embedding sort of plays a role as mask.
However, when input has padding like [1,2,3,0,0], the current mask schema will be [True, True, True, False, False]. And because the last item is a padding item, the apply_mask_to_inputs  basically replaces the padding with 0 embedding. Then the mask schema comes in, masks the last 2 padding items, keeping the 1,2,3 visible to transformer.
I think thats why people encounter issues testing clm. If there are always paddings in input data, the evaluation metrics would be unrealistically high.

I also noticed this bug. Should be an easy fix

Any further updates?


Installation/Update progress indicator

https://user-images.githubusercontent.com/97352673/215037747-19a379f5-ad39-489a-939b-033e91d62d2a.mp4
Currently this is how an installation looks.

Pleass change the deactivated install button to a cancel download button.
If the download (and later the installation) is completed, show an pupup with: "Sucessfully downloaded (installed)".
Add an installation progress tracker after the user has confirmed the installation. (If it's not possible to get the installation progress, atleast show an indicator that something is happening).
1-3 should also be displayed if it the update was started manuelly.

Not necessarily a notification (although some would probably prefer it), but rather something like this:

https://user-images.githubusercontent.com/97352673/215072286-63ecf14b-3228-4e13-a961-7e21722b9fc7.mp4

It is not necessary to display the progress. Just an indication of whether something is happening at all would be enough.

@xIUPITERx These may all be good suggestions, but I'd ask that you create separate feature requests for each rather than putting them all in one issue so they can be discussed individually. In light of that, I'm going to close this issue, but feel free to create separate requests for each feature and/or mention them in the Matrix room so they can be tracked and discussed individually.


target.className is null

https://github.com/mzgoddard/preact-render-spy/blob/28423c6cd28e0673c915bfe54f649aa1be3e9732/src/is-where.js#L26-L27
Upon a couple of context.render(<Component />) calls, when calling context.find('.class') I get an exception thrown since target.className is null and typeof null === 'object'.
I am tempted to say that the flow is expected to return false in the next if and for that to happen, previous check needs to exclude null values before checking typeof.
https://github.com/mzgoddard/preact-render-spy/blob/28423c6cd28e0673c915bfe54f649aa1be3e9732/src/is-where.js#L48-L50
But I have to say that I don't know why target.className is null in the first place.

@tomascasas I think my changes in #49 might also fix this problem... Could you try using npm install git://github.com/mzgoddard/preact-render-spy#nodes and/or could you possibly write a failing test in our shared-render test suite?

@gnarf The issue still persists. The error triggers on is-where.js#72 cause target.className is null, so Object.keys(null) throws an exception:
attr = Object.keys(attr).filter(key => attr[key]);

@gnarf I just found that I have className set in the component to undefined and as a result preact might be initializing target.className to null

Alright, let me get that other one landed first then I will get this working.  Any chance you could write a minimal failing test scenario to make this easier on me?

I think that you test case could include rendering something like this:

test

This test passes:
  it(`${name}: find by class works with null and undefined class and className`, () => {
    const context = func(
test
);
    expect(() => context.find('.test')).not.toThrow();

    context.render(
test
);
    expect(() => context.find('.test')).not.toThrow();

    context.render(
test
);
    expect(() => context.find('.test')).not.toThrow();

    context.render(
test
);
    expect(() => context.find('.test')).not.toThrow();
  });

I pushed up this test to a branch class-undefined - could you help me figure out how to make this fail the same way yours does?

Also can confirm that it passed on node 6, 7, and 8 on travis: https://travis-ci.org/mzgoddard/preact-render-spy/builds/287678078

@gnarf I am committing a failing test, that shares the same structure as my actual test and that fails, perhaps you can help me figure out if the way I am using your module needs some tuning.
Thank you very for following up, and sorry for this delayed response. Invested a long time trying to reproduce the issue, and finally did so.

@gnarf could not commit, these are my lines at the end of src/shared-render.test.js:
class NullClassName extends Component {
  constructor(props) {
    super(props);
    this.state = {className: null};
  }

  render({other}, {className}) {
    return (
      <DivChildren other={other}>

        <DivChildren className={'children'}>
          <ClickCount className={'counter'} />
        </DivChildren>
      </DivChildren>
    );
  }
}

describe('test multiple component re-renders', () => {
  const context = deep(<NullClassName />);
  context.render(<NullClassName other="attempt1"/>);
  context.render(<NullClassName other="attempt2"/>);
  expect(() => context.find('.children').find('.counter').simulate('click')).not.toThrow();
  context.render(<NullClassName other="attempt3"/>);
  expect(() => context.find('.children').find('.counter').simulate('click')).not.toThrow();
});

Alright - check out the class-undefined branch again, I think I have your test case passing.
Will send in a PR for it after we land #49

@gnarf Problem fixed. Thank you so much!
PS: Sorry for the late response, was out this weekend.


[Basic] Implement FileOutputByteStream for Local and in-mem fs

This adds an interface to FileSystem protocol which creates file stream
for a file which can be written to using OutputByteStream methods.
Currently file is actually written to when flush() or close() is called.

Looks good in general, though I have the question about fwrite() in the code.  We will want to rework the output and input streams a bit at some point so that they're zero-copy streams and directly use non-buffering file system operations, since otherwise there's double-buffering (given that stdio already buffers).  But those can be future operations.

@swift-ci please test

@ddunbar Updated, re-review please

My recommendation is to split out a separate PR for the initial changes needed to make OutputByteStream subclassable (and find a reasonable name for the subclass which just buffers the entire string).

Also, for all of the places where I mention it would be nice to avoid a buffer copy -- I expect those to manifest as real changes in a performance test which hit that code path. It would be cool to try and make sure we have performance tests that do in fact reflect the benefit of making those changes (even if they are disabled in trunk).

Closing this in favor of #512


Help With Naming Collision

Can't figure out how to fix this, since most naming collisions you google address a pre RN0.18 version regarding FBJS dependency. Please help!
Others having same issue too: https://github.com/oliviertassinari/react-swipeable-views/issues/207
(but there solution not helpful)
Failed to build DependencyGraph: @providesModule naming collision:
Duplicate module name: react-native-vector-icons
Paths: .../node_modules/react-native/local-cli/rnpm/core/fixtures/files/package.json collides with .../node_modules/react-native/Libraries/Animated/release/package.json
This error is caused by a @providesModule declaration with the same name across two different files.
Error: @providesModule naming collision:
Duplicate module name: react-native-vector-icons
Paths: .../node_modules/react-native/local-cli/rnpm/core/fixtures/files/package.json collides with .../node_modules/react-native/Libraries/Animated/release/package.json
This error is caused by a @providesModule declaration with the same name across two different files.
at HasteMap._updateHasteMap (.../node_modules/react-native/packager/react-packager/src/node-haste/DependencyGraph/HasteMap.js:155:13)
at p.getName.then.name (.../node_modules/react-native/packager/react-packager/src/node-haste/DependencyGraph/HasteMap.js:130:31)

@oblador a fresh project solved it, thank you sooo much for the generous help! i'm new to coding, been coding for less than 4 months now, and I'm always impressed with how eager people are to help... it means a lot, thank you!


Where is the custom folder located after install from the package

I just followed the instruction to install gogs from the package.io but I have no idea where the custom folder should be. There is a folder /etc/gogs but is seem not accessible.

I guess it should be /etc/gogs/conf?
Please ask questions on https://discuss.gogs.io/


Fix clang-14 build (problem with chrono header)

Ubuntu clang-14 builder fails with
 Error: /usr/bin/../lib/gcc/x86_64-linux-gnu/13/../../../../include/c++/13/chrono:2320:48: error: call to consteval function 'std::chrono::hh_mm_ss::_S_fractional_width' is not a constant expression
        static constexpr unsigned fractional_width = {_S_fractional_width()};

See https://github.com/actions/runner-images/issues/8659
I have disabled the buildder for now, but this needs fixing and/or a different/newer clang builder needs to be adopted.

I think it just needs replacing with a clang-15 or newer builder. So many problems with this specific version for C++20.

The Ubuntu builders we use now have a sufficient clang version, plus we are not currently testing on clang anyway.
Closing.


Redraw SubViewportContainer on SubViewport size change

Fixes #86292
When setting the SubViewport's size, the parent SubViewportContainer would not redraw, causing it to display a stretched invalid image.
https://github.com/godotengine/godot/assets/12436824/03012549-d768-408a-9059-ca5fcdeee86d

Thanks!

Cherry-picked for 4.3.1.


Is it possible to view old logs?

I notice that whenever I restart the app, it looks like the old logs are deleted. Is it possible to view old logs? E.g. if the app recovers from a crash, it would be handy to view the logs leading up to that crash.
We're using LoggerStore.shared and the default ConsoleView to view the logs, so I am not sure if this is a config entry I am missing.

NVM, I just saw the Sessions button in the UI.


I can't download the VGG_ILSVRC_16_layers_fc_reduced.h5 file from the google drive in china

Are you so kind enought to send to to my email<EMAIL_ADDRESS>,or upload to somewhere that I can get it in China!
Thank you very much!!!

I wasn't aware that Google Drive isn't accessible in China. Could you maybe try a VPN? It seems as if a fair amount of the people using this implementation are Chinese - how are they doing it?
What cloud storage services do work reliably in China? I read that Dropbox doesn't work in China, either.
I'm busy and only willing to upload the trained weight files to a different service if it's worth the effort, i.e. if (1) the current situation makes this repository unusable for many users and (2) there is a sustainable solution. Since the Chinese government seems to be generally critical towards cloud storage services, is it likely that they will gradually block all major cloud storage providers?


Support downloading of Dash manifests that do not have inline PSSH value

Have you read the FAQ and checked for duplicate open issues?:
Yes
What version of Shaka Player are you using?:
2.3.x
Can you reproduce the issue with our latest release version?:
Yes
Can you reproduce the issue with the latest code from master?:
Yes
Are you using the demo app or your own custom app?:
Demo
If custom app, can you reproduce the issue using our demo app?:
N/A
What browser and OS are you using?:
Electron with Widevine CDM enabled:
https://github.com/castlabs/electron-releases
What did you expect to happen?
Ability to download Dash content (ondemand) with DRM (widevine) that does not have inline PSSH data in the manifest
What actually happened?

Downloading Dash with no inline PSSH value in the manifest throws a "NO_INIT_DATA_FOR_OFFLINE" error on download initiation:
https://github.com/google/shaka-player/blob/05b071d7fa66a23a91ba44c34667bbe0d0e81644/lib/offline/storage.js#L753
Downloading Dash with inline PSSH value in the manifest downloads & plays successful
Both Dash manifest types (inline vs no inline PSSH) do play successful in an online environment

The inline PSSH property should be optional in the manifest specification, but mandatory to be available in the binary fragments. Shaka seem to be correctly supporting this for online playback, but for the offline download initiation it seems to be depending solely on the inline PSSH value.
We are trying to understand if this is a bug or if there is a limitation that prevents shaka from using the PSSH value in the binary fragments while downloading content.

This is a limitation of Shaka Player.  If you will have internet connection when you play later, I think you can store it with usePersistentLicense=false and we won't store the licences.  But it requires internet access to play the content:  storage.configure({usePersistentLicense: false})
When we store content, we need to create an EME offline session so we can play the encrypted content later.  To create the session we need the init data, which is usually PSSH data.  If it is in the manifest, we can just pass it to EME to create the offline session.  But if we don't have it, we need to get it from the media.  In a streaming situation, we pass the media to the browser and it gives us an encrypted event with the PSSH data in it.  But when we are storing content, we can't give it to the browser since we don't want to create a <video> element and have to deal with MSE.
We could parse the segments while we store them and look for the PSSH data.  We don't do this since it would really complicate the storing logic and doesn't work for other container formats.  It might be easier for MP4 since we have an Mp4Parser type to make parsing easier, but trying to support WebM would be much harder since we don't have a simple parser for it.

Thank you for the detailed answer.
Understanding the limitations and complications, would it not be possible to create a small utility method as a fallback that would be able to capture the PSSH from the downloading (mp4) segments (without overhauling the logic to much and/or introducing complex parsers).
The below might be an interesting reference:
https://github.com/youtube/js_mse_eme/blob/586255b3261dd63b951246b2ef01085f9bc26ef5/js/lib/eme/encryptedMediaPortability.js#L109

Great! We have been working on a solution and have been able to make this work by using the shaka mp4 atom parser on the initial segment to extract the pssh. We will share a PR for review once ready

Excellent!  A PR is very welcome.

@joeyparrish , i think that it's necessary reopen the issue because the change was reverted in https://github.com/google/shaka-player/commit/625938e2f96f18bb40045a24f73b646615c16c41

Yes, good catch.  Thanks!


Dev/pickle properties

Adds optional pickling of properties.
namespace PropertyPickleOptions {
  const unsigned int Default = 0; // no data pickled
  const unsigned int AllProps = 0xFFFFFFFF; // all data pickled
  const unsigned int MolProps = 0x1; // only public non computed properties
  const unsigned int AtomProps = 0x10;
  const unsigned int BondProps = 0x100;
  const unsigned int PrivateProps = 0x10000;
  const unsigned int ComputedProps = 0x100000;
}

  static void pickleMol(const ROMol *mol, std::ostream &ss,
                        unsigned int propertyFlags=PropertyPickleOptions::Default);

Not yet exposed to the python API yet.
Need to see what happens when pickled properties are unpickled from older rdkits, but I think its ok (they are ignored).

@greglandrum The last commit 20ce5ea adds a bit of a hack that fixes the reaction binary.
This allows both the atom and query to be pickled.  This flag is holistic to a molecule (it was the only place the flag has space.)  It is not backwards compatible so it may not be acceptable in the current form.

@greglandrum reverted the reaction pickler fix, working on a backwards compatible version

@greglandrum it doesn't look like I can easily make a backwards compatible version easily as tags cannot be readily ignored (the size isn't stored after the tag, so it can't be skipped).
I suppose that this means if we write one of the new tags, then the version should be bumped but if we don't, the version should be the older version?  Would this work?

I answered the question above elsewhere, but just to capture it here: we decided to just bump the version all the time.

I'll think about that.  Should I put props on conformers while we're at it?

I'd save the conformer properties for a separate PR

sorry, I spaced on doing this merge.


dynamodb GetItemOutput is not nice to work with

resp, err := dynamoClient.GetItem(req)

resp.Item["Foo"] ///blows up with invalid operation: resp.Item["Foo"] (type *map[string]*dynamodb.AttributeValue does not support indexing

(*resp.Item)["Foo"]  //works

This is related to #114. We have plans to remove the pointer from maps based on feedback in that thread. That should solve this issue. I agree that it's ugly!
/cc @jasdel

:+1:

I just pushed a changed which converts the *map... to just map. This should improve the usability of maps throughout the SDK.


Infinite loop when navigating back

Reproduce: open fff, navigate to directory, navigate back --> infinite loop, 100% CPU.
macOS, iterm2
fff version of a few minutes ago

Fixed.


Better styling defaults

Our default styling options were largely determined very early on when I fiddled with the matplotlib defaults a little bit and did not go through any detailed discussion. With matplotlib 2.0 (currently in beta) a lot of the default styling will change and I think that's a good time to also rethink some of our defaults. While matplotlib will be our main focus in this redesign, perhaps it's also worth thinking about whether we want our styling to be a bit more consistent between backends. I don't have any detailed suggestions yet but was hoping to kick off some discussion here, so please report what has always annoyed you about our defaults. I know the default matplotlib background color is something I definitely want to change.

I agree matplotlib 2.0 is a great excuse for us to also improve our defaults. :-)

I agree as well; mpl 2.0 is the time, and improving consistency with Bokeh would also be great (as Bokeh is about to reach 1.0).  I'd personally be happy if all of our example notebooks came out reasonably similar between the two backends, after these updates.
For that to be meaningful I think we need to update our examples to use non-square aspects a lot more often, so that we notice behavior for long or tall plots better.

The matplotlib team have just uploaded the default styling changes for 2.0. You can see them here.

I've assigned this to 1.8 and repurposed this issue to cover bokeh style changes as well.
My first proposed bokeh style change: make errorbars thicker my default:

I also think we should start adding concrete suggestions for style changes to this issue. E.g making the default colormap 'fire'?

I think we should update to mpl 2.0 defaults in general; they look better to me, and they are what new users will get with mpl by default outside of hv.

My first proposed bokeh style change: make errorbars thicker by default:

I agree.  I also favor removing the grid lines by default, and making the default sizes comparable between the two backends.

I think we've addressed this for the most part. I'm sure there will be more tweaks we want to do so I'll reassign to 2.0.

I think we've addressed this for the most part.

I agree! I just want to mention that #1610 was merged as well.


Allow to properly disconnect and reconnect the iOS KeyboardAutoManagerScroll at Runtime.

If you would want do disonnect and reconnect at runtime:
Maui.FixesAndWorkarounds.KeyboardAutoManagerScroll.Disconnect();
//and then later:
Maui.FixesAndWorkarounds.KeyboardAutoManagerScroll.Connect();

That would not be possible, even though the Connect and Disconnect methods are public. When you invoke the Disconnect, but then try to Connect again, the Connect logic will stop you since TextFieldToken is still not null even though the Notification Center observers have been removed.

Good find @busec0  !!!


amd64-gpu image is out of date

The image snowzach/doods2:amd64-gpu is out of date and I believe isn't compatible with cuda 12.2
When running docker run -it -p 8080:8080 --gpu all snowzach/doods2:amd64-gpu I got the following error:
Traceback (most recent call last):
  File "main.py", line 8, in <module>
    from doods import Doods
  File "/opt/doods/doods.py", line 20, in <module>
    from detectors.pytorch import PyTorch
  File "/opt/doods/detectors/pytorch.py", line 7, in <module>
    import torch
  File "/usr/local/lib/python3.8/dist-packages/torch/__init__.py", line 229, in <module>
    from torch._C import *  # noqa: F403
ImportError: /usr/local/lib/python3.8/dist-packages/torch/lib/libtorch_cuda.so: undefined symbol: cudaGraphDebugDotPrint, version libcudart.so.11.0

The image snowzach/doods2:amd64 worked fine on the same machine.
This is a new installation of ubuntu server 22.04, with docker-engine installed via the instructions on the Docker website (i.e. not the ubuntu docker snap, since the snap is not compatible with gpu acceleration of containers).
I ran the following from within a container using the base image snowzach/doods2:amd64:
$ apt update
$ apt upgrade
# At this point, I was still getting the same error when i ran python3 main.py
$ pip install --upgrade pip
$ pip install --upgrade torch torchvision
# The following were to fix the unresolved dependency error that last command gave me
$ pip install --upgrade numpy
$ pip install --upgrade ultralytics
$ python3 main.py

At this point, I tested it and it ran much better and faster, presumably indicating it successfully used the GPU.
I assume the image would work if the image build process were run again, but I was unable to find any instructions on the build process. I'd also appreciate instructions on building doods2 images locally.
nvidia-smi output:
Wed Dec 13 00:57:04 2023
+---------------------------------------------------------------------------------------+
| NVIDIA-SMI 535.129.03             Driver Version: 535.129.03   CUDA Version: 12.2     |
|-----------------------------------------+----------------------+----------------------+
| GPU  Name                 Persistence-M | Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp   Perf          Pwr:Usage/Cap |         Memory-Usage | GPU-Util  Compute M. |
|                                         |                      |               MIG M. |
|=========================================+======================+======================|
|   0  NVIDIA GeForce GTX 1080        Off | 00000000:01:00.0 Off |                  N/A |
| 28%   35C    P8              11W / 180W |      2MiB /  8192MiB |      0%      Default |
|                                         |                      |                  N/A |
+-----------------------------------------+----------------------+----------------------+

+---------------------------------------------------------------------------------------+
| Processes:                                                                            |
|  GPU   GI   CI        PID   Type   Process name                            GPU Memory |
|        ID   ID                                                             Usage      |
|=======================================================================================|
|  No running processes found                                                           |
+---------------------------------------------------------------------------------------+

@brainwater Just thought I'd say I saw your post. I'd been having the same problem. Seeing your solution made me want to try again. This time starting with a fresh pull it worked immediately, something I'd never seen before.  Running under WSL2. Just a bit of fiddling with WSL2 to get the port working beyond the local machine. No other changes necessary. So I can't say the issue is closed as I don't see any updates in the repository, but it's definitely working for me now.

I did actually just rebuild the image and pushed it. It may have picked up some new stuff from the base image. I meant to post here but I forgot.

Actually it seems I was mistaken when I said it was working. I neglected to add --gpus all to the docker run command initially. So it was only operating in CPU mode. When I added it the YOLOv5 startup lists the GPU instead of the CPU but exits without an error.
YOLOv5 🚀 2024-1-1 Python-3.8.10 torch-2.1.2+cu121 CUDA:0 (NVIDIA GeForce GTX 970, 4096MiB)
So mine is probably a different issue at this point but I'd love to see it working with the current CUDA. Not sure how to proceed with troubleshooting.

Apologies if this is a newbie question, but what is the minimum required compute capability for running this? I didn't see anything listed. My test case is currently 5.2. If it needs significantly higher I may need to rethink my ideas. I'm using it in the context of Home Assistant and I'm not sure what would satisfy the requirements.

@keyboarderror honestly, I don't know what it requires... I don't do much with the Nvidia GPU side of things. I wouldn't think it requires very high as the model it uses is old but it's good.I really can't tell you to be sure.

This is the version the container has currently:
root@7ebde0b3c926:/opt/doods# nvcc --version
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2022 NVIDIA Corporation
Built on Wed_Sep_21_10:33:58_PDT_2022
Cuda compilation tools, release 11.8, V11.8.89
Build cuda_11.8.r11.8/compiler.31833905_0

Okay, I tried building with updated tensorflow quite a few times but something was wrong with Docker hub. I just finally tried again and it took. Maybe try now. I believe this will have updated Cuda.

OK. It fails with or without enabling the GPU, but at least there's an error. It's the same trying to run on CPU. Doesn't appear to be a Cuda problem.
sudo docker run --gpus all -it -p 8080:8080 snowzach/doods2:amd64-gpu
Traceback (most recent call last):   File "/opt/doods/main.py", line 5, in <module>     from api import API   File "/opt/doods/api.py", line 8, in <module>     from fastapi import status, FastAPI, WebSocket, WebSocketDisconnect   File "/usr/local/lib/python3.11/dist-packages/fastapi/__init__.py", line 7, in <module>     from .applications import FastAPI as FastAPI   File "/usr/local/lib/python3.11/dist-packages/fastapi/applications.py", line 3, in <module>     from fastapi import routing   File "/usr/local/lib/python3.11/dist-packages/fastapi/routing.py", line 22, in <module>     from fastapi.dependencies.models import Dependant   File "/usr/local/lib/python3.11/dist-packages/fastapi/dependencies/models.py", line 3, in <module>     from fastapi.security.base import SecurityBase   File "/usr/local/lib/python3.11/dist-packages/fastapi/security/__init__.py", line 1, in <module>     from .api_key import APIKeyCookie as APIKeyCookie   File "/usr/local/lib/python3.11/dist-packages/fastapi/security/api_key.py", line 3, in <module>     from fastapi.openapi.models import APIKey, APIKeyIn   File "/usr/local/lib/python3.11/dist-packages/fastapi/openapi/models.py", line 103, in <module>     class Schema(BaseModel):   File "/usr/local/lib/python3.11/dist-packages/pydantic/main.py", line 369, in __new__     cls.__signature__ = ClassAttribute('__signature__', generate_model_signature(cls.__init__, fields, config))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/usr/local/lib/python3.11/dist-packages/pydantic/utils.py", line 231, in generate_model_signature     merged_params[param_name] = Parameter(                                 ^^^^^^^^^^   File "/usr/lib/python3.11/inspect.py", line 2715, in __init__     raise ValueError('{!r} is not a valid parameter name'.format(name)) ValueError: 'not' is not a valid parameter name

I think the fix is to update pydantic within requirements.txt
I'm getting the same issue, ValueError: 'not' is not a valid parameter name.
Here is a comment about it https://github.com/tiangolo/fastapi/issues/5048#issuecomment-1170204100
The issue looks like it's occurring on line 231 of pydantic/utils.py https://github.com/pydantic/pydantic/blob/v1.8.2/pydantic/utils.py#L231
Pydantic is pinned at an old (1.8.2) version within requirements.txt.
The problem was identified in pydantic as of April of 2022, and a fix was merged into pydantic in August of 2022. Pydantic v1.8.2 is from 3 years ago, it doesn't have that fix, so updating pydantic to a recent version should fix the issue.
$ sudo docker run -it -p 8080:8080 --gpus all snowzach/doods2:amd64-gpu
Traceback (most recent call last):
  File "/opt/doods/main.py", line 5, in <module>
    from api import API
  File "/opt/doods/api.py", line 8, in <module>
    from fastapi import status, FastAPI, WebSocket, WebSocketDisconnect
  File "/usr/local/lib/python3.11/dist-packages/fastapi/__init__.py", line 7, in <module>
    from .applications import FastAPI as FastAPI
  File "/usr/local/lib/python3.11/dist-packages/fastapi/applications.py", line 3, in <module>
    from fastapi import routing
  File "/usr/local/lib/python3.11/dist-packages/fastapi/routing.py", line 22, in <module>
    from fastapi.dependencies.models import Dependant
  File "/usr/local/lib/python3.11/dist-packages/fastapi/dependencies/models.py", line 3, in <module>
    from fastapi.security.base import SecurityBase
  File "/usr/local/lib/python3.11/dist-packages/fastapi/security/__init__.py", line 1, in <module>
    from .api_key import APIKeyCookie as APIKeyCookie
  File "/usr/local/lib/python3.11/dist-packages/fastapi/security/api_key.py", line 3, in <module>
    from fastapi.openapi.models import APIKey, APIKeyIn
  File "/usr/local/lib/python3.11/dist-packages/fastapi/openapi/models.py", line 103, in <module>
    class Schema(BaseModel):
  File "/usr/local/lib/python3.11/dist-packages/pydantic/main.py", line 369, in __new__
    cls.__signature__ = ClassAttribute('__signature__', generate_model_signature(cls.__init__, fields, config))
                                                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/pydantic/utils.py", line 231, in generate_model_signature
    merged_params[param_name] = Parameter(
                                ^^^^^^^^^^
  File "/usr/lib/python3.11/inspect.py", line 2715, in __init__
    raise ValueError('{!r} is not a valid parameter name'.format(name))
ValueError: 'not' is not a valid parameter name
$

It's back to exiting without any errors. CPU mode works.

I'm getting the same using the gpu
blake@srv-docker:~$ sudo docker pull snowzach/doods2:amd64-gpu
<snipped>
Digest: sha256:d439e0c4d43d50d023fae5e8f3056ad20c68e086ccdfd1d61d6201ee8df843fa
Status: Downloaded newer image for snowzach/doods2:amd64-gpu
docker.io/snowzach/doods2:amd64-gpu
blake@srv-docker:~$ sudo docker run -it -p 8080:8080 --gpus all snowzach/doods2:amd64-gpu
INFO: Created TensorFlow Lite XNNPACK delegate for CPU.
2024-01-30 16:28:01,935 - doods.doods - INFO - Registered detector type:tflite name:default
2024-01-30 16:28:03,518 - doods.doods - INFO - Registered detector type:tensorflow name:tensorflow
/usr/local/lib/python3.11/dist-packages/torch/hub.py:294: UserWarning: You are about to download and run code from an untrusted repository. In a future release, this won't be allowed. To add the repository to your trusted list, change the command to {calling_fn}(..., trust_repo=False) and a command prompt will appear asking for an explicit confirmation of trust, or load(..., trust_repo=True), which will assume that the prompt is to be answered with 'yes'. You can also use load(..., trust_repo='check') which will only prompt for confirmation if the repo is not already trusted. This will eventually be the default behaviour
  warnings.warn(
Downloading: "https://github.com/ultralytics/yolov5/zipball/master" to /root/.cache/torch/hub/master.zip
YOLOv5 🚀 2024-1-30 Python-3.11.0rc1 torch-2.1.2+cu121 CUDA:0 (NVIDIA GeForce GTX 1080, 8112MiB)

Downloading https://github.com/ultralytics/yolov5/releases/download/v7.0/yolov5s.pt to yolov5s.pt...
100%|█████████████████████████████████████████████████████████████████| 14.1M/14.1M [00:00<00:00, 62.9MB/s]

Fusing layers...
blake@srv-docker:~$

Tonight I'll see if I can debug it to get more details on exactly where it had an error.

The problem is due to out-of-date apt packages. I can work around the issue by running an apt update && apt upgrade -y within the container before running doods2.
There's 58 out-of-date apt packages, and 67 out-of-date pip packages.
blake@srv-docker:~$ sudo docker run --entrypoint=bash -it -p 8081:8080 --gpus all snowzach/doods2:amd64-gpu
<snipped>
root@e915b583aeee:/opt/doods# apt update
<snipped>
root@e915b583aeee:/opt/doods# apt upgrade
<snipped>
root@e915b583aeee:/opt/doods# python3 main.py api
<doods2 is now running>

Confirmed that fixes it here too. Excellent. And thanks @brainwater for the --entrypoint=bash switch. I'm still pretty new to docker and couldn't figure out how to get a persistent shell if the container didn't want to run. Now I can poke around.

Awesome! Thanks for tracking that down. I updated the Docker builds and pushed everything out. I even dug out my GTX970 and verified it runs now. Closing this issue., LMK if still problems.

It's working for me now. Thanks for your work @snowzach !

Yes, I pulled the update and it works immediately. Thank you very much @snowzach!


Fix GUI: current Tab/File transfer bug

Bug fixing: Backwards compatibility with Python 3.9.1. Currently the newer version of the Avatar with the new Tab and Transfer file integration crashes when folks try to install and run the Avatar GUI. What is causing it needs investigation, and it has to be fixed:
https://github.com/3C-SCSU/Avatar/issues/60
https://github.com/3C-SCSU/Avatar/pull/65
An implementation that fixes it will be awarded 1 bounty for improving it and 1 extra bounty if the implementation is voted the best version and approved into the main open source repo.
Record a short video showing the new version is running with no problem. Upload it on YouTube or some other online option, then submit the pull request with the link to the video demo included.
Have fun coding!
Note: after the best version is accepted and the pull request is merged into the main repo, further submission is closed.

Python 3.9.1 problem -- (note: quick fix update your Python installation - for Python 3.11 works)
pip show bcrypt
Name: bcrypt
Version: 4.1.2
Summary: Modern password hashing for your software and your servers
Home-page:
Author:
Author-email: The Python Cryptographic Authority developers<EMAIL_ADDRESS>License: Apache-2.0
Location: c:\users\matth\appdata\local\programs\python\python39\lib\site-packages
Requires:
Required-by: paramiko
PS C:\Users\matth\AppData\Roaming\SPB_Data\Avatar-Matt\brainwave-prediction-app>
Exception has occurred: ImportError
DLL load failed while importing _bcrypt: The specified procedure could not be found.
File "C:\Users\matth\AppData\Roaming\SPB_Data\Avatar-Matt\file-transfer\sftp.py", line 1, in
import pysftp
File "C:\Users\matth\AppData\Roaming\SPB_Data\Avatar-Matt\brainwave-prediction-app\gui_windows\transfer_files_window.py", line 6, in
from sftp import fileTransfer
File "C:\Users\matth\AppData\Roaming\SPB_Data\Avatar-Matt\brainwave-prediction-app\GUI.py", line 10, in
from gui_windows.transfer_files_window import TransferData
ImportError: DLL load failed while importing _bcrypt: The specified procedure could not be found.
Full error traceback:
Traceback (most recent call last):
File "c:\Users\matth\AppData\Roaming\SPB_Data\Avatar-Matt\brainwave-prediction-app\GUI.py", line 10, in
from gui_windows.transfer_files_window import TransferData
File "c:\Users\matth\AppData\Roaming\SPB_Data\Avatar-Matt\brainwave-prediction-app\gui_windows\transfer_files_window.py", line 6, in
from sftp import fileTransfer
File "C:\Users\matth\AppData\Roaming\SPB_Data\Avatar-Matt\brainwave-prediction-app\gui_windows....\file-transfer\sftp.py", line 1, in
import pysftp
File "C:\Users\matth\AppData\Local\Programs\Python\Python39\lib\site-packages\pysftp_init_.py", line 12, in
import paramiko
File "C:\Users\matth\AppData\Local\Programs\Python\Python39\lib\site-packages\paramiko_init_.py", line 22, in
from paramiko.transport import (
File "C:\Users\matth\AppData\Local\Programs\Python\Python39\lib\site-packages\paramiko\transport.py", line 93, in
from paramiko.dsskey import DSSKey
File "C:\Users\matth\AppData\Local\Programs\Python\Python39\lib\site-packages\paramiko\dsskey.py", line 37, in
from paramiko.pkey import PKey
File "C:\Users\matth\AppData\Local\Programs\Python\Python39\lib\site-packages\paramiko\pkey.py", line 32, in
import bcrypt
File "C:\Users\matth\AppData\Local\Programs\Python\Python39\lib\site-packages\bcrypt_init_.py", line 13, in
from ._bcrypt import (
ImportError: DLL load failed while importing _bcrypt: The specified procedure could not be found.

Is this an issue for multiple users? I switched to Python 3.9.1 (the version seemingly used in the error message), cloned it, and ran the GUI without issue

Okay, I was able to recreate the error by installing python 3.9.1 on Windows.

I confirmed that the TABS implementation did not cause this error. Pysftp has issues running on older versions of Python on Windows. If you are running Avatar on Windows with WSL, you need Python v3.9.2 or higher. Anything older than that will cause this error on Windows (it will work fine on Linux and probably WSL though).
You can confirm this for yourself by installing Python v3.9.1 and running the following program:
import pysftp
This will cause the same error as in this issue, which shows that this is not caused by the Avatar project.

I think it is an issue with the latest version of bcrypt (4.1.2). Downgrading to bcrypt version 4.1.1 solves this issue.

If that's the case then we can fix this in the future by completing this issue (https://github.com/3C-SCSU/Avatar/issues/69) and specifying that version number.
For now, users can type pip3 install bcrypt==4.1.1 to install the older version
However, it might be good to use more modern versions of Python instead to avoid more problems like this in the future.

Solution proposed by KINGTUT10101 accepted. Ticket closed.


Wrong ID in task view

The page of a task report the wrong ID in the page title. It shows the session ID instead of the task ID.

v0.10.3

fixed in #773

Found a pre-implemented tool that recycle unchanged rows in a table: https://material.angular.io/cdk/table/overview#recyclerows


Whitelist all the things

We have a few pages that are appearing in the wrong mainstream browse section.
We also have a lot of content tagged to highway-code-road-safety that don't have
the link expanded properly in the content store. We think this might be related to an
earlier incident affecting to that content (the browse page).
There are other problems with consistency of organisation tagging but I'm deferring
this until we get monitoring in place, because it could take a long time to debug
these.

👍


Feature: Sort by particulars

Closes Issue #86

Testing will be done in future, as per noted in #102


Switching to OpenRC

Uses systemd. There is no simple way to switch this to OpenRC.

Could you elaborate more on that?

So it's quite easy in general to switch between the systemd and OpenRC Gentoo base systems using eselect during install, but what I mean here is that there's no simple way (e.g. with a variable or command-line switch) to switch this playbook to OpenRC.
The reason for that is that I'd have to implement a whole bunch of extra conditional logic that would run differently depending on whether the user picked systemd or OpenRC. So, for example, systemd-timesyncd would need swapping out, and the network config would need rewriting as it currently uses systemd-networkd formatting. Similarly, anything that invokes systemctl would need replacing and some of the package-specific USE flags might need some work.
Basically, keeping a playbook that could comfortably flip between systemd and OpenRC easily would mean implementing most of the tasks twice.
If you can think of a smart way of managing it though, it's definitely possible. And this is a random hobby project after all...

Okey, so then, could you please list all systemd related things that I need to swap out with OpenRC variants?


ObjectSerializer SpinWaits forever when BuildSerializer fails

I've come across a situation where the BuildSerializer method on some unknown or odd type causes a failure in the expression compiler. This causes the serializer to remain in the _serializers dictionary but without being initialized. This by itself is not an issue.
However the problem comes if you later try to serialize the same type - instead of getting an error, the serializer essentially spinwaits forever, waiting indefinitely for _isInitialized to be set to true, which will never happen.
Several solutions:

Add the object to _serializers only after its successfully initialized. Downside is that concurrent calls will end up creating wasted extra serializers.

Add a catch block and remove the serializer from the dictionary if it couldn't be initialized. Also set some flag in the serializer so if any concurrent code is accessing it before it could get removed, the spinwait can check for this error flag and raise an exception - Requires changes in every place where the dictionary gets a serializer added to it - need to check for exceptions and remove the item.

Just add a timeout to the spinwait - easiest solution. If it spins for more than a second, raise an error.

I was able to reproduce this scenario by accidentally trying to serialize a BsonDocument (from MongoDB). I think its ok to throw an error on unknown types, but it shouldn't spinwait indefinitely.
I'll be happy to submit a pull request for any of these three methods - would like some feedback on which is the preferred approach?

Thanks for reporting.
The best way to deal with this would probably be to have some sort of InvalidTypeSerializer that is just a no-op serializer, and if the object serializer cannot be created, the no-op serializer could be added to the _serializers lookup instead.
That would be more explicit IMO, the type still has a serializer, everything works as normal, it just doesn't do anything.
Where do the actual exception occur? inside the generated read or write methods?
If it fails during the generation, then everything would fail so it has to be one of the two above, right?

Where do the actual exception occur? inside the generated read or write methods?

Yep, it occurs at DefaultCodeGenerator.GetFieldsWriter.

That would be more explicit IMO, the type still has a serializer, everything works as normal, it just doesn't do anything.

Maybe I didn't understand clearly but - isn't it better to fail noisily rather than silently no-op the serialization?
How would the caller know that something failed?

Fixed in #118


String typing does not work

String typing on query parameters does not work. bool and int do. If you have an integer with leading zeros, you would need to strigify it to prevent the leading zeros from being removed. this does not happen.

Can you put this in a plunk?  Thanks!

@christopherthielen this project has 400 issues open. I would just start closing issues (with labels if necessary) and reopen, let them reopen, or let them file new ones.


Publications should be opt-in

Publications should be opt-in and provide a method for opt-out on the profile page. This change should also be reflected on the member edit page accessible by the board, which currently states that this should only be changed in extreme situations.

This needs to be changed in the enrollment form and possibly in the member profile preference form. The default needs to be set to false, but it might be nice to add a message to encourage people to tick the box. Not only because we get money from sponsors for it, but also because the IO-vivat is a nice publication to get.

"gratis"/"free" should be incentive enough

Fixed by changing the corresponding Preference object in production.


Upgrade to latest release 7.8.1

Checklist

[X] Used a personal fork of the feedstock to propose changes
[X] Bumped the build number (if the version is unchanged)
[X] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[X] Ensured the license file is being packaged.

There's an issue in the osx build, I'm looking at it

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@looooo Can you please merge this PR? pythonocc-7.8.1 has been tagged and everything seems to be ok

:+1:


q: what should I do if just input help/version and show help/version info and exit 0

my question is...
I have a flag named config to specify config file path. However, when user input -h,-v,--version,--help and without --config, I want to just show help / version information and exit 0.
so what should I do for my question.

Hi @Turing-Chu, here is some functions that may help:

ShowAppHelp()
ShowAppHelpAndExit() Or:
ShowCommandHelp
ShowCommandHelpAndExit

I believe ShowAppHelpAndExit() is what you are looking for.

thanks for your help. Now I have another question: I define app in another package init function, and called before main function. but I can not get Arg. the code as follows:
project struct:
awesomeProject
├── config
│   └── init.go
├── go.mod
├── go.sum
└── main.go

main.go:
package main

import (
	_ "awesomeProject/config"
)

func main() {}

config/init.go:
package config

import (
	"fmt"
	"os"

	"github.com/urfave/cli/v2"
)

func init() {
	app := &cli.App{
		Name:  "grid",
		Usage: "a sample command line application",
		Flags: []cli.Flag{
			&cli.StringFlag{
				Name:    "config",
				Aliases: []string{"c"},
				Usage:   "specify config file to be used",
			},
			&cli.BoolFlag{
				Name:    "version",
				Aliases: []string{"v"},
				Usage:   "show version for this application and exit",
			},
		},
		Action: func(ctx *cli.Context) error {
			if ctx.NArg() == 0 {
				cli.ShowAppHelpAndExit(ctx, 0)
				return nil
			}
			fmt.Println(ctx.Args().Get(0), ctx.NArg())
			return nil
		},
	}

	if err := app.Run(os.Args); err != nil {
		fmt.Println(err)
		return
	}
}

when I run with go run main.go -c config/init.go, it exit at if ctx.NArg() == 0 {

So, you code should work just fine.
If you want to get the value of --config, you will need to do this:
fmt.Printf("config=%s\n", ctx.String("config"))

And remove the if ctx.NArg() unless you are expecting arguments in addition to flags.

and another question: how can I run go test success?
before urfave/cli, I used alecthomas/kingpin.
But alecthomas/kingpin panic with error: unknown short flag '-t', try --help when run go test.
Now I intend to change kingpin to urfave/cli to resolve this, ie. kingpin or urfave/cli can parse command line arguments and run go test success.
But now urfave/cli run go test with error Incorrect Usage. flag provided but not defined: -test.paniconexit0
So what should I do with urfave/cli or how can I run go test without flag error?

I'm not sure what testing framework (end-2-end vs unit tests) you use, or your test code. Check your test code for for undefined flags. Testing frameworks are usually very verbose on which test failed.

@Turing-Chu it sounds like you may be running something like app.Run(os.Args) in your tests. That will pass the arguments used internally by go test to your application, where presumably you're interested in passing a set of arguments particular to the test case, rather than the test runner.
Make sure that the flags your test uses are the flags you want to test. That should start with the name of your executable, followed by any flags you want to pass for the test case, e.g., []string{"my-app", "--name", "foo"}. There should probably not be a reference to os.Args in your test case or the code it calls, as a quick spot check.

yes, I used same to app.Run(os.Args), and I have some test cases which have nothing to app.Run(os.Args). By the way , app.Run(os.Args) was written at an init function. so, when run go test, some package contain test case  and import package with init function which contain app.Run(os.Args). then it exit with panic. Let me have a look at  what I can do about it.

Now for test my code that has nothing to parse arguments, I add the following code in init function first line😄 :
func init() {
	if strings.HasSuffix(os.Args[0], ".test") {
		return
	}
       // MY OTHER CODE
}


Extending the module, documentation about overriding

I understand that developers of the module are out of time, but I would really appreciate it anybody could add just a basic portion of documentation about overriding views, controllers and models to make it properly? I want to customize the module, add a few fields in the model, add a few actions to controller and change views.
PS: The module looks fantastic and very flexible. Thanks for your work.

The docs can be created - but the extending of classes SECTION and a few others would undergo changes as @derekisbusy is working on componentizing properties for better extensibility and avoid loading them all with module (using module level properties).
That's the reason the docs are being held up - as it would need to be changed and would impact developers to change their codebase later. We would try to get it in a shape sooner but will wait for @derekisbusy to comment.

@kartik-v
@derekisbusy
Any updates on this? It looks like @derekisbusy doesn't have time for github now.

Will try to put some time into this...

It's hard to extend the module components (properties) without the documentation. I also need it very much. As I can see @derekisbusy hasn't been contributed for more than a month. Are his previous changes on componentizing properties finished?
I would be really appreciated if @kartik-v or @derekisbusy would find some time for this task.


(Bug) xDai : last transaction of deployment is always skipped

Steps:

Set up xDai network in wallet
Create crowdsale with any valid parameters(doesn't matter Minted or Dutch)
Proceed deployment

Expected result: all deployment's transaction should be done
Actual result: last transaction 'Add proxy contract to registry' is skipped
.env
REACT_APP_ABSTRACT_STORAGE_ADDRESS='{"3":"0x0648b0645574966c32d37bc5be0208a979342677","77":"0x9d126f44151f70a4da03f53fcb485dea6c0b8d64", "42": "0x2b2d1b6867572ed42140b11b2dac7d454bbce73d", "4": "0xb5bfd08585b705c0eec3c9043249902cc9291ad8","99":"0xd70befce3cf1cc88119c8f4eb583ccd4c39d06e2", "100": "0x373edad6bf3c7bf0da2885dbd9b1988e6c4efe39"}'
REACT_APP_REGISTRY_IDX_ADDRESS='{"3":"0x417f7d0f10fc18c6e139c06e0db7634b5d5756f4","77":"0x76fafe6f7e2f25e40664cf9f22d6e18b709d0087", "42": "0x1be7505450bcd62b2001db8eefb9be1706db45c6", "4": "0xf9d9365d93a351ea4f9ee712a3b731e1d6440a59","99":"0xda7cd8385be8f293677574925f35fbeb38c35a85", "100": "0x4316f704989157162de90997ce1a05cff3118680"}'
REACT_APP_PROVIDER_ADDRESS='{"3":"0xc8bdff0860cc2d0c349208f79e0f3e086effe0cb","77":"0x663d2feb6af69867e0144aab82624edb7af1728b", "42": "0x11bd7d75c081ae7deb9244981a72ceeb340bcbc1", "4": "0x8ca673abb1123a3ffe4000bc813ae2c00d0c7c02","99":"0xef5879ad4ba3fa0312875426902dd384d8c63fcd", "100": "0xe18b28d9a77da833a13e47b1df3e4b8c22695164"}'
REACT_APP_REGISTRY_EXEC_ADDRESS='{"3":"0x1e2f3f5e8022cf28804dd69b81629927079f9951","77":"0x6865f64ecb1f0222a0febe5c1fc1b40c7a378da6", "42": "0xef4dc7adb5d13448597e4d4b471b66c4690568d2", "4": "0x13462c0292f3007ee6d0aaf600dbe1db2ebcf4be","99":"0xd7a4deeeb59569a9d00ba78a18d591bfe53807e0", "100": "0xda7cd8385be8f293677574925f35fbeb38c35a85"}'
REACT_APP_MINTED_CAPPED_IDX_ADDRESS='{"3":"0x21712e030328541ba8a1d044df0317047dd76bb9","77":"0xeeaefd3a69f7d10db92205bbaa54f7c5aa8edf3f", "42": "0xa32057752831f4cc0cc49e7f74bb3cd67237af00", "4": "0xd504e3fb278d36437023fbcdcfd9dd194c71c6c2","99":"0x6f1b623058ffe30b14a6071bbbecaa373cef20c8", "100": "0xa7f6303b14b582be568aefed0296ea61d02bd857"}'
REACT_APP_MINTED_CAPPED_CROWDSALE_ADDRESS='{"3":"0x9039d36603b92211d2532f7f4ecbeced912172bb","77":"0x1d9702eb87c47f90c8669f3d36322f8cf985f721", "42": "0x3397eef245bd419b9e8cc5bc20e896ff784e6d5d", "4": "0xef4dc7adb5d13448597e4d4b471b66c4690568d2","99":"0x39b586af1b947a31806406d8e7dca12240e149eb", "100": "0x502deffdbaa91dd4ee9ae4f11996481b7a0bd237"}'
REACT_APP_MINTED_CAPPED_CROWDSALE_MANAGER_ADDRESS='{"3":"0x10df208e0b76a4e9f8b54e436022f113d4bae5c3","77":"0x3ab54a53eed7e0d3c2f051e4992bb893e8434e5f", "42": "0xf9d9365d93a351ea4f9ee712a3b731e1d6440a59", "4": "0xefc62352f46c62b8aab3d1a802c0c54fc320130b","99":"0x3f099bd191c3f38d60088c287b18f91b1a84f972", "100": "0xd70befce3cf1cc88119c8f4eb583ccd4c39d06e2"}'
REACT_APP_MINTED_CAPPED_TOKEN_ADDRESS='{"3":"0x7b0ab844e92ce6ac36cc844e452766fa11a48c82","77":"0xbe1efbd9134bb5cd0ad3ee6cd2c246e56961ac5c", "42": "0xb5bfd08585b705c0eec3c9043249902cc9291ad8", "4": "0xf0e295c8c1947e834ade8fd3742f51ac44fccba9","99":"0xebd4f6ef12688a35406198ae2a39934fc70e3378", "100": "0x747f40c1fc4bcc96ea9e7b2826c1e32fcbdd04e0"}'
REACT_APP_MINTED_CAPPED_TOKEN_MANAGER_ADDRESS='{"3":"0x0a019b617a0e1745494e9f5dbefdd82445fd9035","77":"0x7e9f62b32538255065681c24efd7a5947f258351", "42": "0x8ca673abb1123a3ffe4000bc813ae2c00d0c7c02", "4": "0xa4fec065faa308a08dba10373e78a04ea5cc5117","99":"0x001aed057cb44ff7325d990db3ff69b0761cccab", "100": "0xdac34accee3c691231f0f6627a799e76a83b5637"}'
REACT_APP_DUTCH_IDX_ADDRESS='{"3":"0x6d7b8d8b819dfb3b03b1a9b1c376f409f48e11f4","77":"0xfbb8127eeb14be7305a6738aa9e43654a95ad908", "42": "0x13462c0292f3007ee6d0aaf600dbe1db2ebcf4be", "4": "0xd1ad7938b170fd9f79221497178b5abc68924d34","99":"0xcb5e9ded56c9fbf4a309f2d6f4e1bbbeb4e31bf7", "100": "0x001aed057cb44ff7325d990db3ff69b0761cccab"}'
REACT_APP_DUTCH_CROWDSALE_ADDRESS='{"3":"0x9c5f8263171b83c303c18b7b1977fa28bdd45116","77":"0x3f52577e2772f0adacd9d3e6a0fdb456701bf386", "42": "0x1c6d46dedd61ef982f0938e73975404b8de6d739", "4": "0xd37bfd3632f78608ad82d1a8c253e2b35908e35f","99":"0x729b907b2e4ff8d150bc774f400a3afe4553b468", "100": "0x69e8835c3b0d2fc98b326e7806b78cdae068e1e5"}'
REACT_APP_DUTCH_TOKEN_ADDRESS='{"3":"0xd1A26D4A869d66EEeA894c47b5d4F0B3A6FC6cAb","77":"0x2a5fbbb4ec592bace485d0f653b051d991eeffdc", "42": "0xf188df9b40f5a5598a1d7088021a679312e62d03", "4": "0x4b34e3df0b4803674db01b0eaef937e6022f4610","99":"0x9f057f47e10b1d58ce2531863597d9dc992df07a", "100": "0x3f099bd191c3f38d60088c287b18f91b1a84f972"}'
REACT_APP_TW_PROXIES_REGISTRY_ADDRESS='{"3":"0xf8141d3ef4ca940d0f485d0abbf2b11e6303c484","77":"0x16a3e07d364d410c6c99e82047177e604779eb02", "42": "0x7cbe84100ff284f87ecd4d5830ff788b9a7ce49e", "4": "0xd836f5716217640ab97bd6ee848881c42625a05d","99":"0xc6468767214c577013a904900ada0a0dd6653bc3", "100": "0xd6ba2ec6fb77f1b17a87446c74aa7d4dff29c64e"}'
REACT_APP_PROXY_PROVIDER_ADDRESS='{"3":"0x7ae09e9963c835959b06240ba69b00febdff681a","77":"0x7ae09e9963c835959b06240ba69b00febdff681a", "42": "0x7ae09e9963c835959b06240ba69b00febdff681a", "4": "0x7ae09e9963c835959b06240ba69b00febdff681a","99":"0x7ae09e9963c835959b06240ba69b00febdff681a", "100": "0x7ae09e9963c835959b06240ba69b00febdff681a"}'
REACT_APP_REGISTRY_EXEC_ID='{"3": "0xb10e2d527612073b26eecdfd717e6a320cf44b4afac2b0732d9fcbe2b7fa0cf6", "77": "0xb10e2d527612073b26eecdfd717e6a320cf44b4afac2b0732d9fcbe2b7fa0cf6", "42": "0xb10e2d527612073b26eecdfd717e6a320cf44b4afac2b0732d9fcbe2b7fa0cf6", "4": "0xb10e2d527612073b26eecdfd717e6a320cf44b4afac2b0732d9fcbe2b7fa0cf6","99":"0xb10e2d527612073b26eecdfd717e6a320cf44b4afac2b0732d9fcbe2b7fa0cf6", "100": "0xb10e2d527612073b26eecdfd717e6a320cf44b4afac2b0732d9fcbe2b7fa0cf6"}'
REACT_APP_MINTED_CAPPED_APP_NAME='MintedCappedCrowdsale'
REACT_APP_DUTCH_APP_NAME='DutchCrowdsale'
REACT_APP_MINTED_CAPPED_APP_NAME_HASH='0x4d696e74656443617070656443726f776473616c650000000000000000000000'
REACT_APP_DUTCH_APP_NAME_HASH='0x447574636843726f776473616c65000000000000000000000000000000000000'
REACT_APP_INFURA_TOKEN='kEpzZR9fIyO3a8gTqJcI'

If you are reporting a problem with Token Wizard, please include the following information:
Which network did you use? (Mainnet, Kovan, Rinkeby, etc.)
xDai
If you were able to create it, what is the URL of your crowdsale?
weren't able
Do you have screenshots showing the problem?

Do you see errors in the dev console? If yes, please include a screenshot
no

@dennis00010011b Can you check this issue again? thanks


[Feature] Selecting Set of Containers with Globbing

>>> lxc list
+--------+---------+------------+------+-----------+-----------+
|  NAME  |  STATE  |    IPV4    | IPV6 | EPHEMERAL | SNAPSHOTS |
+--------+---------+------------+------+-----------+-----------+
| bdr    | RUNNING | <IP_ADDRESS> |      | NO        |         7 |
| bdr0   | STOPPED |            |      | NO        |         0 |
| bdr1   | STOPPED |            |      | NO        |         0 |
| bdr2   | STOPPED |            |      | NO        |         0 |
| ssh    | RUNNING | <IP_ADDRESS>  |      | NO        |         0 |
| ubuntu | STOPPED |            |      | NO        |         0 |
+--------+---------+------------+------+-----------+-----------+
>>> lxc start 'bdr*'
error: not found

Would be great if that would work.

duplicate of #1045


Rust web frameworks

There are quite some server and client web frameworks implemented in Rust already: https://github.com/flosse/rust-web-framework-comparison#comparison . Would these fit into this list?

@fkromer  Sorry for the delay, maybe some they need to look like Python's Flask not be powerful

I'll close, because so long time here.


custom title components. fixes #79

currently ignores onClick that is passed to it though

@brukh that is fine. The catch is you need to call evt.stopPropagation() in this.delete and this.edit or the accordionItem will open as well as the action happening.

That makes sense. Initially I thought you were trying to override the handleClick event with the onClick event passed to the title node! Nice work 👍

Could this be published in NPM also?

I'll publish tonight unless someone beats me to it

@hannupekka published as 0.0.15


Clarify Accumulo API

This pull request attempts to clarify what the Accumulo API is. The existing API statement does not include all types that a user would realistically need to use.   A new API statement is proposed in 2f7d095.
This pull request also prunes (via deprecation) things from the API that were not intended to be in the API, but ended up there.  For example parts of the API that referenced non API types and as a result may not be stable were deprecated.
I am requesting this review on GH instead of ReviewBoard because I thought it would be much easier to review the commits in this PR individually rather than squasing them for RB.  For example the commit that deprecated and moved KeyExtent was very noisy and would make reviewing other changes difficult.
One thing that I was not able to deprecate move was o.a.a.c.security.crypto.  I would have liked to have moved this to o.a.a.c.crypto.  Its not really client API code, its more server side extension code. However the class names are possibly referenced by user config.  Also users may have extended the classes.  I added an exception for the package in the README's API stmt.  Does anyone have any ideas about making this less confusing w/o causing existing users any pain.

@ctubbsii I meant to do something about secrutiy.Credentials in this change, but forgot.  I was thinking of moving it to another package so that it would not be in public API.    I don't think its using any public API methods.  Need to do something about it because it will end up in public API.

@keith-turner Moving it to another package makes sense to me. It's not intended to be public API.

Pull looks good. Thanks for putting in the tedious work to clarify the API.

Overall this looks pretty good, but needs a little bit of touch up still. Nice work Keith and Christopher, this took some doing. Some general thoughts:

DelegationToken move is awkward to me. I think this should be in public API and we just hide the impl details.
Regarding what to do with the crypto package, can we get @ohshazbot attention maybe? Perhaps he can help give some insight. I think it was designed to be extendable by users (CryptoModule and SecretKeyEncryptionStrategy), but there seems to be a lot of other stuff in there that shouldn't be (or be in an impl). Maybe we can get this figured out for 1.7.0 so we can start the deprecation cycle to remove the bits which shouldn't be there?

Regarding what to do with the crypto package, can we get @ohshazbot attention maybe? Perhaps he can help give some insight. I think it was designed to be extendable by users (CryptoModule and SecretKeyEncryptionStrategy), but there seems to be a lot of other stuff in there that shouldn't be (or be in an impl). Maybe we can get this figured out for 1.7.0 so we can start the deprecation cycle to remove the bits which shouldn't be there?

I know of somebody working with the crypto stuff, I'll reach out and make sure this gets noticed.

@joshelser Regarding what to do about the crypto package... that stuff is still marked as experimental, and some of it is still incomplete for full on-disk encryption (ACCUMULO-981), so whatever we do, I think for now, it should not be moved into the public API, because it may require unstable API changes to finish that work.

If we do move it to the public API, it would need to be accompanied with a Beta annotation or something similar.

that stuff is still marked as experimental

That is a very good point. Putting something marked as experimental into public API would be precedent setting. I would agree with you that it should not be added until it is actually capable of performing complete encryption of Accumulo data.

that stuff is still marked as experimental,

That is a good point.   I will wait and see if there are any other comments on this.  Moving it and removing the exception from the README would be nice.

Said it in irc, but, for the record, I'm +1 with these changes + no crypto stuff in public API.

I'm +1 with these changes

Also +1

no crypto stuff in public API.

Same, at least for now, until it's 100% feature complete and no longer experimental, because until that time, we can't really make strong API guarantees about it. I also don't see the sense in moving it, because it'd unnecessarily disrupt those taking advantage of it today as an experimental feature.

I rebased on master and squished most of the commits.   After rebasing on master, APILyzer found some new problems.
  FIELD                org.apache.accumulo.core.client.mapred.AbstractInputFormat$AbstractRecordReader split                               org.apache.accumulo.core.client.mapreduce.impl.AccumuloInputSplit
  METHOD_PARAM         org.apache.accumulo.core.client.mapred.RangeInputSplit       updateSplit(...)                    org.apache.accumulo.core.client.mapreduce.impl.AccumuloInputSplit
  FIELD                org.apache.accumulo.core.client.mapreduce.AbstractInputFormat$AbstractRecordReader split                               org.apache.accumulo.core.client.mapreduce.impl.AccumuloInputSplit
  METHOD_PARAM         org.apache.accumulo.core.client.mapreduce.RangeInputSplit    updateSplit(...)                    org.apache.accumulo.core.client.mapreduce.impl.AccumuloInputSplit

Newest changes look ok too. A few are a little confusing (notably (re)introducing protected RangeInputSplitsplit), but I assume that they were potentially visible before.
Also, do we want all the members on AccumuloInputSplit to be private and not package-private?

Also, do we want all the members on AccumuloInputSplit to be private and not package-private?

Well I think some of the public members of AccumuloInputSplit used to be public methods if RangeInputSplit.  I think some public members of AccumuloInputSplit were not intened to be in public API, but end up there because RangeInputSplit extends AccumuloInputSplit.  I think public API types extending impl types is nothing but trouble.
I think another issue needs to be opened.  I think the map reduce public API is a mess.  Too much of the implementation is in the API, makes it very hard to add features.

These changes were merged


Web (Desktop)
Device Info
MacOS Sonoma 14.3.1 (23D60)
What version of the app are you using?
Chrome, Version 130.0.6723.70 (Official Build) (arm64)
Additional Information
No response

I am fixing it

This was fixed by #6302 which was included in 1.94, released yesterday.


Trim trailing whitespace in d.ts output

Here's a scenario in which trailing whitespace is not trimmed in d.ts output, specifically when adding jsdoc-style comments above each arg with line breaks in-between:
export class Point {
	constructor(
		/**
		 * The x coordinate.
		 */
		public x: number,
		/**
		 * The y coordinate.
		 */
		public y: number
	) {

	}
}

The generated output has spaces after each argument's comma that has another argument after it. In this case, only the x arg has the space after the comma:
define(["require", "exports"], function (require, exports) {
    var Point = (function () {
        function Point(
            /**
             * The x coordinate.
             */
            x, <-- trailing whitespace!
            /**
             * The y coordinate.
             */
            y) {
            this.x = x;
            this.y = y;
        }
        return Point;
    })();
    exports.Point = Point;
});

See the result on the TypeScript Playground.

A PR would be appreciated.

@yuit and I spent a good bit of time looking into alternative approaches, and while we got something that works decently, the gain seems relatively marginal compared to the hoops we had to jump through.
I think that overall this is easily amended by a pretty-printer or minifier.


rtree dependency error

Like other issues reported complaining about spyder-kernels error on startup I receive this message:
You have missing dependencies!

# Mandatory:
rtree >=0.8.3 : None (NOK)
spyder-kernels >=2.0.1,<2.1.0 : 2.0.1 (NOK)

Please install them to avoid this message.

Below the conda list command filtering these packages.
(myenv) PS C:\Users\EPEV> conda list | Select-String -Pattern "rtree|^spyder"

rtree                     0.9.7            py38h8b54edf_1    conda-forge
spyder                    5.0.0            py38haa244fe_0    conda-forge
spyder-kernels            2.0.1            py38haa244fe_0    conda-forge

I have a very similar error, where spyder is asking for spyder_kernels (with _ instead of -).

I also have the exact error. Looking forward for any support. Thank you!

same here... all dependencies fulfilled but spider 5 is complaining about missing the aforementioned dependencies

This is a bug in Python 3.8.8, and has nothing to do with Spyder. Try "import rtree" and see what happens.

... I'm not sure it has something to do with python 3.8.8 since I'm getting the same error in python 3.7.10
however, you are right, import rtree fails with the following error:
OSError: could not find or load spatialindex_c-64.dll

@raphaelquast you are on windows so the error is different from mine. on Linux if I roll back Python from 3.8.8 to 3.8.5 then everything is fine. Also reported the same thing to rtree. see https://github.com/Toblerity/rtree/issues/182.

I wonder if this is a compilation error in Conda-forge and not something directly related to rtree.
@sweettyler, what rtree version are you using in Python 3.8.5?

@ccordoba12 I have never upgrade or downgrade rtree. it is v0.9.4 as shown below:
Name                    Version                   Build  Channel
rtree                     0.9.4                    py38_1
The only difference is Python 3.8.5 or Python 3.8.8 and with the Python 3.8.8 I cannot import retree.

As @sweettyler mentioned, this is caused by an installation issue with Python 3.8.8. So for now the only advice we can give you is not use that version.
Other than that, there's nothing else we can do to help you with this problem, sorry.

@ccordoba12 Anaconda released another version python 3.8.8 two days ago and this new version fixed the rtree bug.

Good to know! Thanks for letting us know about it @sweettyler.

Thank you so much....I'm a newbie and I was going crazy trying to get Spyder working after upgrading to v5....which must have brought Python 3.8.8 along for the ride


Can I use this model to get embeddings of a paragraph of classical Chinese?

I also notice this.
If so, how to make it?
Many thanks!

Sorry but you cannot use SuPar-Kanbun models to get embeddings, since they are Biaffine models to get matrices for dependency-parsing.

Thank you! BTW: Do you know which kinds of LLM support generating embeddings?


remove ResponseNotReady unit test in vca

Fix: #887
Signed-off-by: Wenxing Hou<EMAIL_ADDRESS>

please dont change case index. Just delete the function and make it reserved.

I have change the code. Thanks.


Please make a new release/tag with latest fix for tip

I'm not sure I understand the goal here. Would it be alright if you provided more reasoning for what you're looking for? This package isn't using gopkg.in or something that uses releases and tags. Since this is an unversioned go-gettable package, don't new users always get the latest commit?

For context, no releases or tags exist currently.

That's not true: https://github.com/jtolds/gls/releases. Any package management tools tend to treat it as the latest release.

 I usually use branches so when I didn't see any I didn't think to check for tags.
Okay, thank you so much for filing this issue, sorry for my unreasonable push back and lack of understanding on my own project! I've created a new tag.

lol "For context, no releases or tags exist currently." Right before writing this I went to go check the Github branch dropdown, saw nothing besides master in the branch list, and moved on. :(


make area generation optional?

I took a rather light-weighted OSM extract (a province in Germany) and fired up a container. All going smooth until it reaches the area generation (I think, judging by other issues). It seems to take a very long time compared to populating the db and it's stuck on this line of the entrypoint: https://github.com/wiktorn/Overpass-API/blob/4fe130a36a9df9b496e8c5cfceac21edf5342130/<IP_ADDRESS>/docker-entrypoint.sh#L83
with the output being along the lines of:
After 0h40m0s: in "recurse", part 0, on line 255. Stack: 0 of 0 171594 of 0 0 of 0
After 0h40m15s: in "recurse", part 0, on line 255. Stack: 0 of 0 173042 of 0 0 of 0
After 0h40m30s: in "make-area", part 0, on line 257. Stack: 0 of 0 175297 of 0
After 0h40m45s: in "make-area", part 0, on line 257. Stack: 0 of 0 176717 of 0
After 0h41m0s: in "make-area", part 0, on line 257. Stack: 0 of 0 178671 of 0

Before researching, I figured overpass made some nominatim call when using place names :sweat_smile: It's really super convenient for a lot of people, but really not everyone.
Would you accept a contribution to opt out of area computation via env var? I think the biggest hurdle is to disable the supervisord program dispatcher_areas, maybe we could use its autostart config option with a boolen env var OVERPASS_USE_AREAS which gets exported in the entrypoint script?
BTW, it seems to me the script updating the areas is never used (maybe on purpose)? https://github.com/wiktorn/Overpass-API/blob/master/<IP_ADDRESS>/bin/rules_loop.sh my bad, I deleted a part of the supervisord config already..

I'm also not 100% why it's taking so long to generate areas for this small area: http://download.geofabrik.de/europe/germany/saarland.html.
I seem to remember geofabrik does include lots of irrelevant relations which are missing the corresponding ways, I'm not often working with relations. Could that be a problem for the area generator?

Hi,
Making area optional will be most welcome PR. What I think, would be good, is that it would be optional both at the startup as well as it should disable the background update of areas started through supervisord. I think, we could extend OVERPASS_RULES_LOAD values to allow 0 with the meaning, that areas will not be generated at all.
I have no idea why your small extract is taking so long to process. But as far as I know, missing ways in relations are only non-issue, in case of full planet dumps. In all partial extracts that I worked with there were some missing ways.

What I think, would be good, is that it would be optional both at the startup as well as it should disable the background update of areas started through supervisord

yeah I think so too. we could also use your suggestion of OVERPASS_RULES_LOAD, just a couple of thoughts why IMHO a new "boolean" OVERPASS_USE_AREAS (true/false) would be beneficial:

it's more explicit I find
we could use it directly in the supervisord config if we set a program's autostart=%(ENV_OVERPASS_USE_AREAS)s, so it won't run by itself, but could still be started if someone has a change of heart. Didn't test that yet, but will do today.

I'm fine with whatever you're happier with though.

+1
Also using an extract and DB setup finished hours ago, but still seeing
After 2h13m57s: in "recurse", part 0, on line 255. Stack: 0 of 0 636743 of 0 0 of 0
After 2h14m12s: in "make-area", part 0, on line 257. Stack: 0 of 0 638139 of 0

oh damn, totally forgot about this.. actually you can try #75 @larsschwarz , it was pretty much done IIRC. I used that branch for a setup which seems to be working


Scroll compilation output buffers

I noticed that the output buffers for compilation or tests fill up, but doesn't scroll when the output doesn't fit in the window.  I always find myself directly selecting the window and scrolling down.
Would automatically scrolling to the bottom be a potential feature of rustic or is that something that I, the user, have to configure myself?

It's a feature of emacs that's reused by rustic. Does https://github.com/brotzeit/rustic/pull/463 help you ?


Advanced Paste : remove locale part from URL

Description of the new feature / enhancement
Some document site's URL has locale string (en-us, ja-jp).
for example : https://learn.microsoft.com/en-us/windows/powertoys/install
Could you remove en-us when paste from clipboard.
copy string: https://learn.microsoft.com/en-us/windows/powertoys/install
paste string: https://learn.microsoft.com/windows/powertoys/install
Scenario when this would be used?
When a URL containing a locale such as en-us is provided, the browser will display the specified URL.
However, if a URL without a locale is given, like on the Microsoft Learn site, it will automatically redirect to the browser's default language.
I believe it would be very convenient if the advanced paste feature could optionally remove the locale part, allowing users to view URLs shared from other countries in their own language.
Supporting information
No response

This should already work with Paste with AI - see sample below. If you use this often, you can turn this into a Custom Action.
Please let us know if this fixes your issue! /needinfo

It's great. Thank you!
But I don't have OpenAI.com license 😢.


Question: How to best add layer with plugin…?

Hi,
This is just asking for some advice. I want to create a docker image that adds a plugin to this image. However, it seems the plugin directory naming isn't constant over versions. Like:
/usr/lib/plexmediaserver/Resources/Plug-ins-e0b7243
where the 8 chars after the last dash changes.
Would you just as part of the script in the Dockerfile do ls  /usr/lib/plexmediaserver/Resources/Plug-ins-* and use the result? Or is there some cleaner way?
Thanks for a great image!

Oh, it seems the Plugins directory is named as the Plex version ending on the deb file:
https://downloads.plex.tv/plex-media-server/<IP_ADDRESS>.1620-e0b7243/plexmediaserver-ros6-binaries_<IP_ADDRESS>.1620-e0b7243_amd64.deb
->
/usr/lib/plexmediaserver/Resources/Plug-ins-e0b7243

SUB_ZERO_ZIP=`find /usr/lib/plexmediaserver/Resources/ -name 'Plug-ins*'\`/Sub-Zero.bundle.zip"

Great ! Where is your dockerfile ?

I actually just modified your Dockerfile. I should make it a small Dockerfile that inherits this one instead, but this was quick and dirty:
https://github.com/hedefalk/docker-plex-media-server/blob/master/Dockerfile

Pretty neat. For most users I believe they'd like to install plugins on their own and that would just go under config directory. It'd also be consistent with their configuration.
What's your use case to include it in the image itself?

@hedefalk could be a great idea to build a docker file with plugins by passing env var to enable it.
But i see you force a version of SubZero (<IP_ADDRESS>9), that could be a problem.
@wernight yes in my server, i populate myself plugins on plex by config dir and git :)

@wernight Oh, I somehow didn't know I could put the plugin under config. I see you've written that on the readme now :)
I would probably just do that then actually.
Or I don't know. My use case is pretty much just that I want to have my whole wardrobe server pretty reproducible from source with as few steps as possible. In theory I would like, in the case of total hardware failure to just do:

build new hardware
git checkout totalConfig
apt-get install docker
docker-compose run all the things

I used to have lots of shell scripts and listings for setting up my environment but I'm trying to dockerize the shit out of everything.
But as config is on my "supposedly persistent and safe separate datastore under raid 1 and backups nas" anyways, I can just put the plugin there and consider it persistent data instead.

You can have different solutions for how you store you config (docker container, mounted file system, Kubernetes with cloud file system...). In all cases you'd want your config to limit work in case of failure.
Once you've that config (with your plugins if you so wish), just run the image and voilà!


Refactor custom node implementation, add visitor API and stable unqoted text.

Closes: #13
Closes: #5

This pr contain a lot of fixes, few of them refactor CustomNode to use ParseRecoverable and ToTokens instead of custom methods.

New crate rstml-controll-flow was added in order to show how to use CustomNode in real project.

Added visitor API and refactored html-to-string-macro to use it.

Added hack for stable to parse unquoted text correctly, this is hidden behind feature gate "rawtext-stable-hack"
Because hack is using source_text function, it cannot be used if template macro called from generated input.
Macro matcher module allows extracting html template from macro arguments.

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

Comparison is base (de04b27) 77.09% compared to head (8c683c1) 59.37%.
Report is 5 commits behind head on main.

Files
Patch %
Lines

src/visitor.rs
62.00%
57 Missing :warning:

rstml-controll-flow/src/escape.rs
0.00%
43 Missing :warning:

rstml-controll-flow/src/lib.rs
0.00%
41 Missing :warning:

src/parser/recoverable.rs
31.25%
33 Missing :warning:

src/rawtext_stable_hack.rs
0.00%
33 Missing :warning:

src/parser/mod.rs
15.15%
28 Missing :warning:

src/node/raw_text.rs
35.29%
22 Missing :warning:

src/node/mod.rs
57.57%
14 Missing :warning:

rstml-controll-flow/src/extendable.rs
0.00%
13 Missing :warning:

rstml-controll-flow/src/tags.rs
0.00%
13 Missing :warning:

... and 4 more

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##             main      #44       +/-   ##
===========================================
- Coverage   77.09%   59.37%   -17.73%
===========================================
  Files          12       18        +6
  Lines         751     1184      +433
===========================================
+ Hits          579      703      +124
- Misses        172      481      +309

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


TypeError: tx.wait is undefined

Not sure if this error is on zksync's end or ethers' end but the error TypeError: tx.wait is undefined occurs after invoking syncWallet.onchainAuthSigningKey()

the transaction goes through and everything however because of this thrown error I'm not able to get back the response object to retrieve tx info

Let me check.

Closing. It was an issue with the ethers Signer.


[Bug]: Syntaxt error on .closest(selector:has(+ selector))

Version
28.1.3
Steps to reproduce
const ancestor = myElement.closest('.thisClass:has(+ .thisClass)')
Expected behavior
No syntax error found.
Actual behavior
Jest is flagging a syntax error:
SyntaxError: '+.thisClass,.thisClass' is not a valid selector
Additional context
I'm basically finding a parent of myElement that has a class of .thisClass that has a sibling with the same class.
It perfectly works on execution of the script, I can see no JS documentation saying it is not a valid selector, but Jest says it is...
I tried to replace .closest() by .querySelector() and it is accepted.
I tried to remove the + sign, and it is accepted.
Environment
System:
  OS: MacOS 14.7
  CPU: M3 pro
Binaries:
  Node: 20.9.0
  npm: 10.1.0
npmPackages:
  jest: 28.1.3

Issue still happening.
Anyone for Triage?
Thanks.


Add zoom level to class names

When zooming, it would be nice to have a class name that reflected the zoom level.  This would allow me to use CSS to, for example, hide labels on points when the user zooms out, but when they zoom in we can make the labels visible.

Added on develop; CHANGELOG.md has very rough details


Implement date/time parsing for delimited reads

Reported by ansley b manke on 13 Jul 2016 17:47 UTC
A field may contain both date and time 04/09/13 02:20:00.  Currently the file would need pre-processing to break these into two fields before reading them with Ferret.
Add date/time and eurodate/time field types.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/2450

Comment by ansley.b.manke on 13 Jul 2016 22:28 UTC
This is done.  The list of types is now

(skipped field)
NUMERIC
TEXT
LATITUDE
LONGITUDE
DATE
EURODATE
TIME
DATIME
EDATIME

Because of the way string matching is done, these names can be abbreviated.  This meant that DATETIME was not a good choice - it's got the same 3- or 4-letter abbreviation as "DATE".

Modified by ansley.b.manke on 13 Jul 2016 22:28 UTC