questions about evaluate_segmenters.py

Thank you very much for sharing the code.
I have many questions about evaluate_segmenters.py. Could you tell me more details about how to run the evaluate_segmenters.py? And I can't find the src/losses, so I import the voxelmorph package. However I find I cannot find the binary_dice. So how can I do?
And if I let the ds_key=mri-100unlabeled-test then will "IndexError: list index out of range"
If I let the ds_key=mri-100unlabeled then will "KeyError: 'n_test'"
Thanks for your help

evaluate_segmenters.py is a script used only to compare the different segmentation models described in the paper; it is not necessary for training an augmented segmentation model.
Searching for "dice" in the voxelmorph repo, I see there is now this sample: https://github.com/voxelmorph/voxelmorph/blob/dev/scripts/tf/test.py
You should select a ds_key that refers to your desired dataset defined in main.py. mri-100unlabeled-test is the one that I used, but depending on how many test volumes you have, it may be looking for more volumes than what is available (that is what the IndexError sounds like but it is impossible for me to say what the issue is without knowing where the error is coming from).

Sorry, I still don't know how to run the evaluate_segmenters.py. If I want use the mri-100unlabeled-test, how could I edit the code? Here is the errors

@xamyzhao
Sorry to trouble you. I waste much time on the script but still dont  know how to run it. Can you tell me more details about it?

It seems like here, evaluate_segmenters is looking for test data to evaluate your trained segmentation networks on, and is not finding any training or testing data. How many .npz files do you have for training/testing?

@xamyzhao
I use the dataset you provided to train and test. I know I have to find new data to run the test script, but I want to run the script firstly. I input the --data mri-100unlabeled for train script, and I let the ds_key as mri-100unlabeled-test in the test script.

The script is failing right now because it is not finding enough test volumes. If you look at https://github.com/xamyzhao/brainstorm/blob/4634c602881ebe5226c268a4d12488cc79fe2036/src/mri_loader.py#L124, the data loader is looking for 100 test volumes in a folder where you presumably have none.

@xamyzhao
Thank you very much. I modify the code you mentioned, and then I met this question.


let(:facts) gives Error: Operator '[]' is not applicable to an Undef Value.

Similar to previous issue https://github.com/rodjek/rspec-puppet/issues/344 I am having issues getting tests to run where $facts are used in the manifest. The values don't appear to be populated so they always fail.
I have setup a new CentOS 7 VM with a number of gems installed to try out rspec tests for some code I am refactoring. I followed some setup guides, and for the most part I have the tests running.
The problem is a fair bit of the code uses the new $facts hash for os values and $trusted for role information etc.
I saw in the README that I should be able to define the os facts, but I cannot get it to work. I even copied the line directly out of the README file.
I have the following manifest:
class profile::virtual::yum_repos {

  $el_version = $facts['os']['release']['major']

  # use el_version for yumrepo wrapper...
}

I have a spec file for it:
require 'spec_helper'

describe 'profile::virtual::yum_repos', :type => :class do
  let(:facts) { {:os => { :family => 'RedHat', :release => { :major => '7', :minor => '1', :full => '7.1.1503' } } } }
  it { is_expected.to compile }
  it { should contain_class('profile::virtual::yum_repos') }
end

When I try to run it, all I get is the same error message:
$ rake test

  1) profile::base should compile into a catalogue without dependency cycles
     Failure/Error: it { is_expected.to compile }
       error during compilation: Evaluation Error: Operator '[]' is not applicable to an Undef Value.

     Failure/Error: require profile::virtual::yum_repos

     Puppet::PreformattedError:
       Evaluation Error: Operator '[]' is not applicable to an Undef Value.

I don't know how else to try setting the facts for this to work. I tried different combinations of keys and values in the hash, but nothing worked.
As a workaround, I tried setting the defaults_facts which is also mentioned in the README file. And this works with the tests passing:
Spec.configure do |c|
  c.default_facts = {
    :os => {
      :release => { :major => '7', :minor => '3', :full => '7.3.1611' }
    }
  }
end

profile::virtual::yum_repos
  should compile into a catalogue without dependency cycles
  should contain Class[profile::virtual::yum_repos]

However we use a number of different OS's and release versions, so I can't rely on setting just one default.
I am testing this using gems manually installed in the test VM (not using rvm or bundler).
*** LOCAL GEMS ***

ast (2.3.0)
bigdecimal (1.2.0)
diff-lcs (1.3)
facter (2.4.6)
fast_gettext (1.1.0)
gettext (3.2.2)
gettext-setup (0.24)
hiera (3.3.1)
io-console (0.4.2)
json (1.7.7)
json_pure (1.8.6)
locale (2.1.2)
metaclass (0.0.4)
metadata-json-lint (1.1.0)
mocha (1.2.1)
parallel (1.11.2)
parser (<IP_ADDRESS>)
powerpack (0.1.1)
psych (2.0.0)
puppet (4.10.1)
puppet-lint (2.2.1)
puppet-syntax (2.4.0)
puppetlabs_spec_helper (2.1.2)
rainbow (2.2.2)
rake (12.0.0)
rdoc (4.0.0)
rspec (3.6.0)
rspec-core (3.6.0)
rspec-expectations (3.6.0)
rspec-mocks (3.6.0)
rspec-puppet (2.5.0)
rspec-support (3.6.0)
rubocop (0.49.0)
ruby-progressbar (1.8.1)
semantic_puppet (1.0.0)
spdx-licenses (1.1.0)
text (1.3.1)
unicode-display_width (1.2.1)

Is there anything else I can provide to help work out how to fix this?

I'm unable to reproduce this, would you be able to publish a repo or gist with the full reproducer?

I will run through the setup in another new VM and see if it still happens. Then I will be provide steps for what I've done.

I think this error is being caused by a hiera_hash call in our code and the line information might be misleading. I need to do more work on test setup and making the hiera lookups work correctly.
So will close this issue.


In Chromebooks, apps should start full screen

Currently, they start in a small window:

Closing as working as intended.


Mergify/bp/stable/0.14/pr 4403

Summary
Details and comments

This is an invalid change, the mergify/bp/stable/0.14/pr-4403 branch is an automatically, bot created, backport to the stable branch from master, we should not try to merge it into master. I'm going to close this


performance of role checking

Call this a premature concern if you want, but if I have 100s of roles representing granular permissions, I don't like the thought of the server having to do 50 string comparisons on average before every user operation.  Especially since my app will be running on a Raspberry Pi.
There's a very simple optimization: this package could just just cache maps with the (Flow type) structure {[userId: string]: {[group: string]: {[role: string]: true}}} for each user, and only recompute these when a user's roles is !== the last seen value.  Then the userIsInRole check, when the cache is up-to-date, would be (basically) _.every(roles, role => roleCache[_.isString(user) ? user : user._id][group][role]).
Would you be open to a PR for this?

Have you looked into v2.0 branch? I do not think there is this problem anymore.

I haven't looked into all the details of how you check for roles now, but I like the v2.0 schema a lot better.

I think I will close this. If you have concrete numbers that performance of v2.0 checking is slow, show them.
We cannot change much with v1.0 because this would break backwards compatibility.

Fine with me, thanks!


nms错误

我的代码每次运行到nms_wrapper.py文件中
    if dets_th.shape[0] == 0:
        inds = dets_th.new_zeros(0, dtype=torch.long)
    else:
        if dets_th.is_cuda:
            inds = nms_cuda.nms(dets_th, iou_thr)
        else:
            inds = nms_cpu.nms(dets_th, iou_thr)

都会崩溃：
Process finished with exit code 139 (interrupted by signal 11: SIGSEGV)

This might because the environment you are using now is different from that for compiling the code.
Directly Google/Baidu the bug message Process finished with exit code 139 (interrupted by signal 11: SIGSEGV) could help you, e.g., you could find this blog through Googling the message.

Please use English or English&Chinese for issues.


Encoded length larger than max_encoding_length when source length is divisible by 254

When source data doesn't contain any zero bytes and the data length is divisible by 254 then cobs.rs encoded data will be larger that the maximum size returned by max_encoding_length method.
use cobs::encode;

const SIZE: usize = 254;

fn main() {
    let mut dest = [0u8; SIZE * 2];
    let src = [1u8; SIZE];

    let size = encode(&src, &mut dest);
    let maxsize = cobs::max_encoding_length(SIZE);

    println!("maxsize: {}", maxsize);
    println!("size: {}", size);
}

prints:
maxsize: 255
size: 256

I tried this with C code
#include <iostream>
#include <cstring>
#include "cobs.h"

static constexpr size_t SIZE = 254;

int main(int, char**) {
    std::cout << "Hello, world!\n";
    uint8_t destination[SIZE * 2] = {};
    uint8_t src[SIZE];
    std::memset(src, 1, sizeof(src));
    size_t encodedLen =  cobsEncode(src, sizeof(src), destination);
    printf("Encoded length: %zu", encodedLen);
}

where the cobsEncode is the one from wikipedia.
The coded length is 255 here


[url_launcher] Endorse macos package

Description
Endorses url_launcher_macos by adding it as a platform in pubspec.yaml.
Also adds a macos folder to the example to support to support flutter run
Adds dummy macos podspec as a workaround for   https://github.com/flutter/flutter/issues/46618
Related Issues
Replace this paragraph with a list of issues related to this PR from the issue database. Indicate, which of these issues are resolved or fixed by this PR. Note that you'll have to prefix the issue numbers with flutter/flutter#.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See [Contributor Guide]).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the [Flutter Style Guide].
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I updated pubspec.yaml with an appropriate new version according to the [pub versioning philosophy].
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the [CLA].
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

@amirh ping!


test diff code

Codecov Report

Merging #5 (3a8166d) into main (208db33) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main        #5   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           12        12
  Lines          371       371
=========================================
  Hits           371       371

closing for now, see when it is implemented here


ceph: add support to read custom namespace

Earlier, we have the support to read custom namespace
but there was bug that it was reading from wrong config
file. This commit fixes that bug.
Closes: https://github.com/rook/kubectl-rook-ceph/issues/24
Signed-off-by: subhamkrai<EMAIL_ADDRESS>

As a separate PR it would be nice to run the CI in a different namespace than the default.

As a separate PR it would be nice to run the CI in a different namespace than the default, or potentially one action in the default namespace and another in a different namespace.

will add that.

As a separate PR it would be nice to run the CI in a different namespace than the default, or potentially one action in the default namespace and another in a different namespace.

will add that.

#28


[How to use] How to make the crop Circle? there are only square or rectangule options

Platforms
dart
Description
i want to crop a circular avatar, not a square or rectangule. how can i do that? (regarding the image editor)
My code
return ExtendedImage.memory(
imageData,
fit: BoxFit.contain,
mode: ExtendedImageMode.editor,
extendedImageEditorKey: UniqueKey(),
initEditorConfigHandler: (state) {
return EditorConfig(
maxScale: 8.0,
cropRectPadding: const EdgeInsets.all(20.0),
hitTestSize: 20.0,
cornerColor: Theme.of(context).colorScheme.secondary,
);
},
);
Try do it
irrelevant

https://github.com/fluttercandies/extended_image/blob/7b444abd813b4a5d256f4196045327dd48840754/example/lib/pages/complex/image_editor_demo.dart#L490


Remove unused variables and dead code

Minor clean up removing dead code and unused variables.

Landed, thank you!


plugins/slate: move serialize / unserialize to plugins

[ ] Move serialize/unserialize to slate plugins
[ ] Also refactor plugin system (don't use classes)

what is this about again?

I resolved #1

Plugin system is a bit improved (but not yet perfect), i'll close that here. Feel free to reopen


ValueError: source code string cannot contain null bytes

I've updated the REAMME so the command is now
echo OPENAI_API_KEY="API KEY" > keys.py

try deleting the keys.py file and trying this command


Product Description

I can't seem to find the product description with this new version.  With the previous version Nager we would do something like this:
foreach (var ed in item.EditorialReviews)
{
string descrip = ed.Content;
            }

But now this information doesn't seem to be available when doing a Multi Item Lookup.  AWS information on this data can be found here:
https://docs.aws.amazon.com/AWSECommerceService/latest/DG/RG_EditorialReview.html

https://webservices.amazon.com/paapi5/documentation/migration-guide/pa-api-40-to-50-mapping.html


Show invisibles breaks auto indent

When show invisibles is enabled, the auto indent doesn't handle decrementing indent properly. Seems to happen to me in all grammars. No errors in dev console.
OS X Yosemite 10.10.2 (14C109)
Atom Version 0.187.0 (0.187.0)

Glad i searched through the Github issues.
It wasn't so obvious to me that Show invisibles was causing this.
Thanks.
:+1:  for a fix experiencing the same problem

This appears to be fixed on the latest release.


Bad mappings in Dark Souls on Wine via hidraw

I'm trying to play Dark Souls 1 on Steam via Wine, and --hidraw --emulate-xboxdrv puts the buttons in pretty random places that don't exactly make the game playable. --emulate-xpad doesn't even work in the first place, because something silly is mapped to the right stick and the camera just spins. Here's what it's doing:
Good mappings:

L1, R2 - LB, RB
Lstick, Rstick - Lstick, Rstick

Bad mappings:

Circle - Y
Triangle - B
Cross - X
Square, L2, R2 - Nothing

Now, when I don't use ds4drv, the mappings are identical to Windows, so I can assume that the game is reading them the same way. I can elaborate on how it maps things there if that helps. I also don't know whether this is an issue in other games via Wine, but I do know that Souls 1 is known for funny behavior with gamepads.

This works for some reason. Left stick button and PS button might need to be swapped (souls doesn't use it), but at least I can play the game now.
create_mapping(
    "souls", "Make Dark Souls work with DS4",
    # Bus type, vendor, product, version
    0,          0,      0,       0,
    # Axes
    {
	"ABS_X":     "left_analog_x",
	"ABS_Y":     "left_analog_y",
	"ABS_RX":    "right_analog_x",
	"ABS_RY":    "right_analog_y",
    },
    # Axes settings
    {},
    #Buttons
    {
        "BTN_SELECT": "button_ps",
	"BTN_THUMBL": "button_options",
	"BTN_MODE":   "button_share",
	"BTN_X":      "button_cross",
	"BTN_Y":      "button_circle",
	"BTN_A":      "button_square",
	"BTN_B":      "button_triangle",
	"BTN_TL":     "button_l1",
	"BTN_TR":     "button_r1",
	"BTN_TL2":    "button_l2",
	"BTN_TR2":    "button_r2",
	"BTN_THUMBR": "button_l3",
	"BTN_START":  "button_r3"
    },
    # Hats
    {
	"ABS_HAT0X": ("dpad_left", "dpad_right"),
	"ABS_HAT0Y": ("dpad_up", "dpad_down")
    }
)

Hello,
Have you tried with Dark Souls II ? It doesn't seem to recognize the controller.

Has anyone ever tried Dark Souls using an actual Xbox 360 controller? Similar problems come up when using a real Xbox 360 controller while trying to play Dark Souls on Wine; similar problems also come up when trying to use a DS4 to play Dark Souls on Windows without using software to make a virtual Xbox 360 controller. This particular problem is not just an issue with ds4drv.
The spinning camera is due to ABS_Z being mapped to L2 and ABS_RZ being mapped to R2. Dark Souls in DirectInput mode apparently uses those axes as well as ABS_RX and ABS_RY for camera rotation. Dark Souls interprets the minimum trigger values as the max negative value for an axis as opposed to 0 so the neutral point ends up being when the triggers are at half pull; evtest seems to show that proper scaling is not done in ds4drv.
Buttons are swapped around but that problem exists when using a real Xbox 360 controller as well.


Clarify ModelCheckpoint behavior when self.monitor == None

📚 Documentation
In ModelCheckpoint, setting self.monitor to None (as for the built-in checkpoint callback in the Trainer) causes it to default to either val_loss or checkpoint_on, as per L459. This, in turn, overrides the behavior described in the docs, which is caused by L500.
Incriminated docs at L53:

monitor: quantity to monitor. By default it is None which saves a checkpoint only for the last epoch.

I think that this should be clarified to reflect the (pretty complex and a bit tricky, IMHO) behavior of ModelCheckpoint, perhaps by adding this to the doc entry:

monitor: quantity to monitor. By default it is None which saves a checkpoint only for the last epoch; if the user logs either val_loss or checkpoint_on, it will instead monitor their quantity.

I will happily submit a pull request once I hear from a maintainer! :)

if the user logs either val_loss or checkpoint_on, it will instead monitor their quantity.

This was kept for backwards compatibility. Before, you would monitor a quantity by setting it to those keys during training. Now, it is done via ModelCheckpoint's monitor. I guess we never deprecated it properly.
It is fine to update the docs with that piece of information. But it should also be properly deprecated and explicitly stated.

thanks for the clarification! Indeed, I saw a comment about it in the code.
This makes me wonder, however, about what's the desired future behavior once this functionality will be abandoned.
Will the built-in checkpoint store only the last weights, so that users must instantiate a separate checkpoint monitoring a custom variable (e.g. val_loss)? in this case, users can skip saving the latest weights by setting checkpoint_callback to False. I think this would be a sensible behavior, but I don't know what your future plans are :)

Yes, that is correct

alright, i'll prepare a PR with the chance mentioned earlier. Please let me know if it should contain more than that, and I'll be happy to help :)


The link of LRS3-TED doesn't work

The link of LRS3-TED doesn't work.
Can you provide a link to the new download please?

https://1drv.ms/u/c/ba3b81b8a2ef83ae/Ea6D76K4gTsggLrKAAAAAAABYnHnRW8PHOUqM7q6cz5Dwg?e=Fgheve

hi，could you please provide another download link

hi，Could you please provide another download link @ @JusperLee

This only has 20K videos whereas the paper states 150k videos, can you please share the details how can I access the comoplete dataset. Thank you so much for your help.

We do need the full data for research, many thanks in advance!

Thank you for your inquiry! Regarding the LRS3 dataset, we are unable to share the full dataset due to copyright issues. We suggest you refer to the relevant information in the paper or visit the official website of the dataset for more access instructions. If you have any other questions, please feel free to contact me!

https://1drv.ms/u/c/ba3b81b8a2ef83ae/Ea6D76K4gTsggLrKAAAAAAABYnHnRW8PHOUqM7q6cz5Dwg?e=Fgheve

Hello, it seems this link is no longer valid. Could you please provide an updated one? Thank you so much!


Need examples of influx pkg export

On our page for influx pkg export, we list the usage and all the parameters, but we don't provide any examples of the command being used. We should provide some examples so it's more clear how this command can be used.
Relevant URLs
https://v2.docs.influxdata.com/v2.0/reference/cli/influx/pkg/export/
What products and version are you using?

Actually, I just now realized that we have this page with influx pkg export example:
https://v2.docs.influxdata.com/v2.0/influxdb-templates/create/
We should ensure that these two pages link to each other:
https://v2.docs.influxdata.com/v2.0/influxdb-templates/create/
https://v2.docs.influxdata.com/v2.0/reference/cli/influx/pkg/export/
And that these two pages link to each other:
https://v2.docs.influxdata.com/v2.0/influxdb-templates/use/
https://v2.docs.influxdata.com/v2.0/reference/cli/influx/pkg/


”Sălcuța”, ”Stupini”, ”Zoreni” are listed in the drop-down menu

Description:
When user writes ”Sălcuța”, ”Stupini”, ”Zoreni” the locations appear in the drop-down menu next to the comune and the county witch is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Sălcuța"

Expected results
The location appears in the drop-down menu as "Sălcuța, Bistrița-Năsăud (Sânmihaiu de Câmpie)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Stupini"

Expected results
The location appears in the drop-down menu as "Stupini, Bistrița-Năsăud (Sânmihaiu de Câmpie)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Zoreni"

Expected results
The location appears in the drop-down menu as "Zoreni, Bistrița-Năsăud (Sânmihaiu de Câmpie)".


Add pry-byebug gem

For debugging purposes.

If you are going to say why the change is being made in the pull request, it would be good to have this information in the commit as well.

Fair enough - amended now.


Reference to German wordlist missing in help

If you look at the command line help, a reference to the included german word list is missing:
  -w WORDFILE, --wordfile WORDFILE
                        Specify that the file WORDFILE contains the list of
                        valid words from which to generate passphrases.
                        Provided wordfiles: eff-long (default), eff-short,
                        eff-special, legacy, spa-mich (Spanish), fin-kotus
                        (Finnish)

there are also issues with the documentation. the base list  provided is https://github.com/dassencio/langcmp/blob/master/wordlists/top10000de.txt. in the readme and documentation the top1000de.txt is linkes (missing one 0).
the correct list ist shipped though - with 9717 lines based on the top10000de.txt

Hey, thanks for letting me know - this is corrected in the newest release


Fix IDE0058

Fix unused expression errors - https://docs.microsoft.com/en-us/dotnet/fundamentals/code-analysis/style-rules/ide0058
I have mixed feelings about this one...none of these were actually issues and some of them are pretty weird to be returning values anyways (why does the query builder return itself when appending?)
But not using return values can sometimes indicate a bug (such as not updating a string when doing replace) so I think it's better to have this enabled and deal with a little bit of extra discard code.

Hmm, I'm not so sure about this one. Is there some finer-tuned control that we can use to still prevent some class of issues but not have to put _ =  everywhere?

I guess we could look into something like what Zi did in https://github.com/Azure/azure-functions-sql-extension/pull/100#pullrequestreview-769403712
But honestly I don't really think it's that huge of a deal. That suppressions file is kind of messy and this kind of stuff isn't going to cause any huge amounts of pain in the end.

I guess we could look into something like what Zi did in #100 (review)
But honestly I don't really think it's that huge of a deal. That suppressions file is kind of messy and this kind of stuff isn't going to cause any huge amounts of pain in the end.

I auto-generated the suppressions file from Visual Studio. The default is more granular and you need to suppress each occurrence, but I believe you can suppress it at a broader level, like everything under a namespace: https://stackoverflow.com/questions/11359652/suppressmessage-for-a-whole-namespace

@chlafreniere @lucyzhang929 What do you think about just not enabling this rule? I'm not sure it's worth the trouble to suppress it or add the extra code to avoid it - the impact is pretty easy to catch and either way we approach the fix (suppress or add discards) results in a bunch of extra stuff. And it's pretty likely more would keep needing to be suppressed as we keep adding stuff which is just going to get even more annoying.

@chlafreniere @lucyzhang929 What do you think about just not enabling this rule? I'm not sure it's worth the trouble to suppress it or add the extra code to avoid it - the impact is pretty easy to catch and either way we approach the fix (suppress or add discards) results in a bunch of extra stuff. And it's pretty likely more would keep needing to be suppressed as we keep adding stuff which is just going to get even more annoying.

I'm onboard. Let's not enable this one.


[FR] increase brightness

Proposal Summary
Add a way to increase brightness of images on the fly during visulaization
Motivation
I work woth a dataset that is a bit dark.
The neural nets needs dark images as input, so I can't preprocess them beforehand.
For human visualizations, it would be nice to be able to increase brightness on the fly.
What areas of FiftyOne does this feature affect?

[ x] App: FiftyOne application
[ ] Core: Core fiftyone Python library
[ ] Server: FiftyOne server

Hi @joihn. I already started implementing this as I think this a natural feature to have. It will need some optimizations before we release it. Stay tuned.

This (along with contrast adjustment) was requested by a Teams user; they use CVAT and CVAT now supports brightness/contrast adjustment

is the request for a permanent brightness adjustment or temporary in-app ?

A slider in each image controls. Possibly also a default for value for all images.

Hi Team, any update on this topic? Can we change brightness or contrast in app when we check the datasets? Thank you


AppLocker cmdlets only work with GP

AppLocker cmdlets do not have any knowledge of the MDM stack. Customers using the AppLocker CSP have run into issues when trying to use the cmdlets, and there isn't a clear warning for them.

Docs Build status updates of commit 9370304:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/applocker/AppLocker.md
:white_check_mark:Succeeded

docset/windows/applocker/Get-AppLockerPolicy.md
:white_check_mark:Succeeded

docset/windows/applocker/Set-AppLockerPolicy.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

@denisebmsft could you please merge this? Thanks!

@dariomws @get-itips @tseward
Peer review needed for this PR.
cc @JohanFreelancer9 for copy editing.
Thank you.

Also agreed with @get-itips

Docs Build status updates of commit 400f121:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/applocker/AppLocker.md
:white_check_mark:Succeeded

docset/windows/applocker/Get-AppLockerPolicy.md
:white_check_mark:Succeeded

docset/windows/applocker/Set-AppLockerPolicy.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 0daf51c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/applocker/AppLocker.md
:white_check_mark:Succeeded

docset/windows/applocker/Get-AppLockerPolicy.md
:white_check_mark:Succeeded

docset/windows/applocker/Set-AppLockerPolicy.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

@e0i are you able to merge these changes? I've removed the [!Note] tag. Thanks!

Docs Build status updates of commit 04ad0a8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/applocker/AppLocker.md
:white_check_mark:Succeeded

docset/windows/applocker/Get-AppLockerPolicy.md
:white_check_mark:Succeeded

docset/windows/applocker/Set-AppLockerPolicy.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 263e516:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/applocker/AppLocker.md
:white_check_mark:Succeeded

docset/windows/applocker/Get-AppLockerPolicy.md
:white_check_mark:Succeeded

docset/windows/applocker/Set-AppLockerPolicy.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


IGNITE-2766 Deterministically reopen cache after client reconnect.

We have dropped use of Ignite cache until this issue is resolved.

@chrisheaththomas can you please state what  problem you are facing? Because I could not reproduce IGNITE-2766.


chore: jsonfmt common-types

Ran jsonfmt -w "specification/common-types/**/*.json"

Can one of the admins verify this patch?

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go
Unable to detect any generation context from this PR.

Automation for azure-sdk-for-java
Unable to detect any generation context from this PR.

Automation for azure-sdk-for-ruby
Unable to detect any generation context from this PR.

Automation for azure-sdk-for-js


problem in some permissible_value?

@putmantime , I'm trying to acquaint myself with this repo in order to check out the linkml2sheets problems you mentioned on the Monarch LinkML Slack channel yesterday
After cloning and running poetry install, I did make all
poetry run gen-project \
  -d project src/linkml/include_linkml.yaml && mv project/*.py src/include_linkml/datamodel

ALL_SCHEMAS = ['src/linkml/include_linkml.yaml']
INFO:root:Generating: graphql
INFO:root: SCHEMA: src/linkml/include_linkml.yaml
INFO:root: PARENT=project/graphql
Traceback (most recent call last):
File "/Users/MAM/Library/Caches/pypoetry/virtualenvs/linkml-project-template-nsNn_beL-py3.9/bin/gen-project", line 8, in
sys.exit(cli())
File "/Users/MAM/Library/Caches/pypoetry/virtualenvs/linkml-project-template-nsNn_beL-py3.9/lib/python3.9/site-packages/click/core.py", line 1128, in call
return self.main(*args, **kwargs)
File "/Users/MAM/Library/Caches/pypoetry/virtualenvs/linkml-project-template-nsNn_beL-py3.9/lib/python3.9/site-packages/click/core.py", line 1053, in main
rv = self.invoke(ctx)
File "/Users/MAM/Library/Caches/pypoetry/virtualenvs/linkml-project-template-nsNn_beL-py3.9/lib/python3.9/site-packages/click/core.py", line 1395, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/Users/MAM/Library/Caches/pypoetry/virtualenvs/linkml-project-template-nsNn_beL-py3.9/lib/python3.9/site-packages/click/core.py", line 754, in invoke
return __callback(*args, **kwargs)
File "/Users/MAM/Library/Caches/pypoetry/virtualenvs/linkml-project-template-nsNn_beL-py3.9/lib/python3.9/site-packages/linkml/generators/projectgen.py", line 197, in cli
gen.generate(yamlfile, project_config)
File "/Users/MAM/Library/Caches/pypoetry/virtualenvs/linkml-project-template-nsNn_beL-py3.9/lib/python3.9/site-packages/linkml/generators/projectgen.py", line 112, in generate
gen = gen_cls(local_path, **all_gen_args)
File "/Users/MAM/Library/Caches/pypoetry/virtualenvs/linkml-project-template-nsNn_beL-py3.9/lib/python3.9/site-packages/linkml/generators/graphqlgen.py", line 18, in init
super().init(schema, **kwargs)
File "/Users/MAM/Library/Caches/pypoetry/virtualenvs/linkml-project-template-nsNn_beL-py3.9/lib/python3.9/site-packages/linkml/utils/generator.py", line 93, in init
loader = SchemaLoader(schema, self.base_dir, useuris=useuris, importmap=importmap, logger=self.logger,
File "/Users/MAM/Library/Caches/pypoetry/virtualenvs/linkml-project-template-nsNn_beL-py3.9/lib/python3.9/site-packages/linkml/utils/schemaloader.py", line 52, in init
self.schema = load_raw_schema(data, base_dir=base_dir, merge_modules=mergeimports,
File "/Users/MAM/Library/Caches/pypoetry/virtualenvs/linkml-project-template-nsNn_beL-py3.9/lib/python3.9/site-packages/linkml/utils/rawloader.py", line 69, in load_raw_schema
schema = yaml_loader.load(copy.deepcopy(data) if isinstance(data, dict) else data,
File "/Users/MAM/Library/Caches/pypoetry/virtualenvs/linkml-project-template-nsNn_beL-py3.9/lib/python3.9/site-packages/linkml_runtime/loaders/loader_root.py", line 85, in load
results = self.load_any(*args, **kwargs)
File "/Users/MAM/Library/Caches/pypoetry/virtualenvs/linkml-project-template-nsNn_beL-py3.9/lib/python3.9/site-packages/linkml_runtime/loaders/yaml_loader.py", line 25, in load_any
return self.load_source(source, loader, target_class, accept_header="text/yaml, application/yaml;q=0.9",
File "/Users/MAM/Library/Caches/pypoetry/virtualenvs/linkml-project-template-nsNn_beL-py3.9/lib/python3.9/site-packages/linkml_runtime/loaders/loader_root.py", line 66, in load_source
return target_class(**data_as_dict)
File "", line 58, in init
File "/Users/MAM/Library/Caches/pypoetry/virtualenvs/linkml-project-template-nsNn_beL-py3.9/lib/python3.9/site-packages/linkml_runtime/linkml_model/meta.py", line 447, in post_init
self._normalize_inlined_as_dict(slot_name="enums", slot_type=EnumDefinition, key_name="name", keyed=True)
File "/Users/MAM/Library/Caches/pypoetry/virtualenvs/linkml-project-template-nsNn_beL-py3.9/lib/python3.9/site-packages/linkml_runtime/utils/yamlutils.py", line 100, in _normalize_inlined_as_dict
self._normalize_inlined(slot_name, slot_type, key_name, keyed, False)
File "/Users/MAM/Library/Caches/pypoetry/virtualenvs/linkml-project-template-nsNn_beL-py3.9/lib/python3.9/site-packages/linkml_runtime/utils/yamlutils.py", line 200, in _normalize_inlined
form_1({k: v})
File "/Users/MAM/Library/Caches/pypoetry/virtualenvs/linkml-project-template-nsNn_beL-py3.9/lib/python3.9/site-packages/linkml_runtime/utils/yamlutils.py", line 153, in form_1
order_up(key, slot_type(**as_dict(raw_obj)))
File "", line 40, in init
File "/Users/MAM/Library/Caches/pypoetry/virtualenvs/linkml-project-template-nsNn_beL-py3.9/lib/python3.9/site-packages/linkml_runtime/linkml_model/meta.py", line 764, in post_init
self._normalize_inlined_as_dict(slot_name="permissible_values", slot_type=PermissibleValue, key_name="text", keyed=True)
File "/Users/MAM/Library/Caches/pypoetry/virtualenvs/linkml-project-template-nsNn_beL-py3.9/lib/python3.9/site-packages/linkml_runtime/utils/yamlutils.py", line 100, in _normalize_inlined_as_dict
self._normalize_inlined(slot_name, slot_type, key_name, keyed, False)
File "/Users/MAM/Library/Caches/pypoetry/virtualenvs/linkml-project-template-nsNn_beL-py3.9/lib/python3.9/site-packages/linkml_runtime/utils/yamlutils.py", line 200, in _normalize_inlined
form_1({k: v})
File "/Users/MAM/Library/Caches/pypoetry/virtualenvs/linkml-project-template-nsNn_beL-py3.9/lib/python3.9/site-packages/linkml_runtime/utils/yamlutils.py", line 153, in form_1
order_up(key, slot_type(**as_dict(raw_obj)))
File "/Users/MAM/Library/Caches/pypoetry/virtualenvs/linkml-project-template-nsNn_beL-py3.9/lib/python3.9/site-packages/linkml_runtime/utils/yamlutils.py", line 127, in order_up
raise ValueError(
ValueError: Slot: permissible_values - attribute text value (File "", line 29, col 15: list) does not match key (List)
make: *** [gen-project] Error 1

gen-linkml returns a similar error
gen-linkml src/linkml/include_linkml.yaml

ValueError: Slot: permissible_values - attribute text value (File "", line 29, col 15: list) does not match key (List)

matching the case of the text to the PV's key gets make all to complete
permissible_values:
  List:
    text: list

Thanks Mark, sorry for confusion, I had fixed that locally, thought it was in main but it wasn't.
Even with that fix made I am still getting
include-linkml git:(main) ✗ linkml2sheets -s src/linkml/include_linkml.yaml  src/data/sheets/*.tsv -d src/data/sheets --overwrite
Traceback (most recent call last):
  File "/Users/putmanti/Library/Caches/pypoetry/virtualenvs/linkml-poetry-gp-template-AkzsagWH-py3.9/bin/linkml2sheets", line 8, in <module>
    sys.exit(export_schema())
  File "/Users/putmanti/Library/Caches/pypoetry/virtualenvs/linkml-poetry-gp-template-AkzsagWH-py3.9/lib/python3.9/site-packages/click/core.py", line 1128, in __call__
    return self.main(*args, **kwargs)
  File "/Users/putmanti/Library/Caches/pypoetry/virtualenvs/linkml-poetry-gp-template-AkzsagWH-py3.9/lib/python3.9/site-packages/click/core.py", line 1053, in main
    rv = self.invoke(ctx)
  File "/Users/putmanti/Library/Caches/pypoetry/virtualenvs/linkml-poetry-gp-template-AkzsagWH-py3.9/lib/python3.9/site-packages/click/core.py", line 1395, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/putmanti/Library/Caches/pypoetry/virtualenvs/linkml-poetry-gp-template-AkzsagWH-py3.9/lib/python3.9/site-packages/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
  File "/Users/putmanti/Library/Caches/pypoetry/virtualenvs/linkml-poetry-gp-template-AkzsagWH-py3.9/lib/python3.9/site-packages/schemasheets/schema_exporter.py", line 297, in export_schema
    exporter.export(sv, specification=f, to_file=outpath)
  File "/Users/putmanti/Library/Caches/pypoetry/virtualenvs/linkml-poetry-gp-template-AkzsagWH-py3.9/lib/python3.9/site-packages/schemasheets/schema_exporter.py", line 90, in export
    writer.writerow(row)
  File<EMAIL_ADDRESS>line 154, in writerow
    return self.writer.writerow(self._dict_to_list(rowdict))
  File<EMAIL_ADDRESS>line 149, in _dict_to_list
    raise ValueError("dict contains fields not in fieldnames: "
ValueError: dict contains fields not in fieldnames: 'desc', 'slot'

desc and slot are column headers in a sheet tsv src/data/sheets/INCLUDE_Portal_V1_LinkML_Schema_Classes_Slots.tsv

note you don't need to include text at all, you can just say:
  permissible_values:
    List:
    ...


MS Office 2013 leads to segfault in marco compositor

On Startup marco segfaults:
[40156.404470] marco[6391]: segfault at 555f8970cd3e ip 0000555f868f7981 sp 00007fff196c1960 error 6 in marco[555f868de000+8f000]

Workaround: start up a console and restart marco

okay, it has to do with compositing. Viable workaround for now: disable compositing


Can't use extra tables on select while using the CTEBuilder

Hi!
Thanks for the latest addition of the CTEBuilder :clap:.
I'm having a problem when using it along with more tables, if I want to use the CTE table with another table, when using the CTE table, it wipes the other tables from the select. I don't know if this is a bug or my ignorance of not knowing how to do it, I tried searching for examples but didn't found one.
Let me add an example (groupwise-max) to explain better what I'm having trouble with (Live example):
package main

import (
	"fmt"

	"github.com/huandu/go-sqlbuilder"
)

type sqlService struct {
	ID      string `db:"s.id"`
	Name    string `db:"s.name"`
	EnvName string `db:"e.name"`
}

var serviceSQLBuilder = sqlbuilder.NewStruct(&sqlService{})

func main() {
	// Create virtual table that holds the services with the latest deployment.
	maxSB := sqlbuilder.NewSelectBuilder()
	maxSB = maxSB.Select(maxSB.As("MAX(d.updated_at)", "max_updated_at"), "d.service_id").From("deployment d")
	maxSB.GroupBy("d.service_id")
	cteb := sqlbuilder.With(sqlbuilder.CTETable("d2").As(maxSB))

	// Retrieve data from the virtual table and join with more data.
	sb := serviceSQLBuilder.SelectFrom("deployment d")
	sb.With(cteb).Where(
		sb.And(
			"d2.max_updated_at = d.updated_at",
			"d2.service_id = s.id",
		),
	)
	sb.Join("service s", "d.service_id = s.id")
	sb.Join("environment e", "s.env_id = e.id")

	fmt.Println(sb.String())
}

This gives the result query of (deployment d missing):
WITH d2 AS
  (SELECT MAX(d.updated_at) AS max_updated_at,
          d.service_id
   FROM deployment d
   GROUP BY d.service_id)
SELECT s.id,
       s.name,
       e.name
FROM d2
JOIN service s ON d.service_id = s.id
JOIN environment e ON s.env_id = e.id
WHERE (d2.max_updated_at = d.updated_at
       AND d2.service_id = s.id)

However what I would want based on the code should be this (note the deployment d table after the FROM d2):
WITH d2 AS
  (SELECT MAX(d.updated_at) AS max_updated_at,
          d.service_id
   FROM deployment d
   GROUP BY d.service_id)
SELECT s.id,
       s.name,
       e.name
FROM d2,
     deployment d
JOIN service s ON d.service_id = s.id
JOIN environment e ON s.env_id = e.id
WHERE (d2.max_updated_at = d.updated_at
       AND d2.service_id = s.id)

Many thanks!

It's a bug. SelectBuilder should not overwrite the FROM clause if it's set by From before With.

Released in tag v1.28.1.

That was fast! Many thanks for the fix :pray:


Update eslint

I feel that the best way to start moving towards v1.0 (#1666) is to first update eslint to latest.

[ ] update to eslint 4.x,
[ ] get rid of eslint-config-airbnb,
[ ] set CI builds fail on linting errors

isn't eslint-config-airbnb used as part of applying the rules through eslint?

@lirantal True, but it's geared towards React native and it's deprecated, too.

Actually if we want a predefined set of eslint rules, we should consider XO.

XO is very opinionated by a specific set of people. If we're at a cross-road to diverge how about standardjs ?


display newsletter subscribers count in the subscribe section

Addresses #27
A few notes:

the html files indentation and alignment is off, making changes a tad difficult
didn't add the count to the banners
didn't add the count to the ksuid page
made the text blue, but can change to red if its a requirement

checking 👀

yes the html is a bit overwhelming.. i am looking to split templates up and have more reusable and smaller files. will open an issue for later

i have just opened an issue to track the nested template https://github.com/golang-cafe/golang.cafe/issues/36

Thank you @dsantos!


Remove list comprehension in common.py and clarify comments

Per lab guidelines, list comprehensions are (sadly) discouraged. This PR removes one from common.py:sign_signable(). It also reduces the asymptotic runtime with respect to the number of keys signing a signable, though that doesn't much matter unless the number is huge. It also adds a log statement when we skip signing with a key because that key has already signed the signable.

Coverage decreased (-0.2%) to 85.87% when pulling c4ed0527057ec3565df4ba80d216fed0515ec55e on awwad:kill_list_comprehension into 821464acb89e33b9b65894e6da4b98272e74cca3 on uptane:develop.

Tests pass and the demo runs.


Update default channels with Tippin and Yalls?

Tried opening other predefined channels with Tippin and Yalls (well known LN services):

In fact I want to open a channel with Radar: https://github.com/LN-Zap/zap-desktop/issues/2271#issuecomment-495769664
But I'm getting this error: https://github.com/LN-Zap/zap-desktop/issues/723#issuecomment-495780286

This has been fixed in #2577 and will be included in 0.5.1


Minetweaker support for heating crucible

The fluid cooler has minetweaker support, which is great, but if the heating crucible were also supported then it would be possible to have new solid<->liquid round trip conversions. Thanks.

The problem is that the way the heating crucible is written, it cannot support more than 2 recipes. It was designed to only ever have 2 recipes, and optimized for that. While I could re-write the internal logic, that still leaves the rendering of the fluid/solid in it. Currently, that is done with JSON, something I can only get away with while there are only 2 recipes. This is very optimized compared to what I would have to do if I supported minetweaking recipes (A TESR most likely). Not to mention that adding a recipe would require specifying a texture for the contents, and I'm not sure if that's possible with minetweaker or not.

Noting that this has been programmed in as part of the alchemy update, and will be released along with the alchemy update.


Consuming DisclosureManager v3 across monorepo

Summary
This is the rest of the terra-framework changes necessary to consume the DisclosureManager context changes. The DisclosureManager changes are not included here, but its still a large number of changes.
The build will not succeed for this branch in isolation; once these changes have been reviewed, I'll pull them into the disclosure-manager-context-pr branch, where everything will build beautifully. You'll just have to trust me on that in the meantime.
I also removed the terra-app-delegate package here and marked it as deprecated in the README.

https://www.youtube.com/watch?v=pl3vxEudif8


Add fact command

This PR is for a re-implementation of the two (cat, java) fact commands, provided as sub-commands.

I would recommend using node-fetch over Axios as it replicates the native fetch API for node.js and is much cleaner, imo.

I tried<EMAIL_ADDRESS>but ran into issues related imports due to our use of commonjs for the TypeScript compiler's module system. I decided to use Axios as I didn't want to use the downgraded 2.x version.

node-fetch comes with discord.js just use import fetch from 'node-fetch'

Just realised I forgot to stage the changes for package.json in my commit, so axios isn't listed as a dependency.


New /hate command

After being petitioned numerous times, I finally gave in an created the /hate command. It makes angry villager thunderclouds appear on hated people.
Summary

command /hate and permission lovedup.hate.
command /unhateable and permission lovedup.hate.
Added dragon sounds for arch nemesis.

After numerious requests, /hate and /hateall has been revived.

Nicely done!


gem_package options per documentation throws ArgumentError

This recipe from https://docs.chef.io/resource_gem_package.html:
gem_package "bundler" do
  options(:prerelease => true, :format_executable => false)
end

with chef-client 12.0.3, test-kitchen w/ chefdk 0.4.0, throws this:
ArgumentError
       -------------
       options should be a string instead of a hash
       in gem_package[bundler] from /tmp/kitchen/cache/cookbooks/chef_gem/recipes/default.rb:10:in `from_file'

The problem I've hit is somewhat different than what I've described here. I'll open a new issue for that to keep things clear.
This behavior is correct in that you don't want to use the omnibus Ruby Gem API to install system gems, which is the purpose of `gem_package'  Sorry.


Make snapshot testing work for testing sandboxed macOS apps

Hi guys,
First of all, thanks for the great project (and the cool videos)! I'm trying to get snapshot testing to work with unit tests for a sandboxed macOS Cocoa application.
I'm running into a problem:
The unit tests are for testing my application target (not a framework) and hence run hosted inside my sandboxed app. Because of that, the snapshots directory in the project folder is not accessible to my unit tests and recording a snapshot fails with: Code=513 "You don’t have permission to save the file “TestOutput” in the folder “Snapshots”."
This problem does not exist when running unit tests for a framework target. Those are not sandboxed.
I considered a few things, but maybe you have a better idea:
Option 1: Move code to be tested into a framework target
While technically possible, not always practical…
Option 2: Don't run unit tests inside the hosting application
Tests don't run sandboxed this way, but we lose access to testing internal types and methods (@testable import requires unit tests to run inside a hosting application and the "Allow testing Host Application APIs" flag to be set.
Option 3: Copy snapshots to temporary directory before tests start, copy back once done
The idea:

Since sandboxed unit tests are allowed to read and write to the system temporary directory, copy the snapshots into a well-known location inside the temporary directory before the tests run.
Once the tests finish, copy the newly generated snapshots back from the temp directory into the project source directory.

Xcode allows to run run scripts pre and post test execution. I'm attaching a shell script that copies the snapshots around as a starting point. The script contains more details how to set this up.
tests-pre-post-action.sh.txt
To get this to work, the snapshot code needs one adjustment:
https://github.com/pointfreeco/swift-snapshot-testing/blob/fb70100e9f15e014bd0707575756f6812e2ec827/Sources/SnapshotTesting/AssertSnapshot.swift#L174
--
let envSnapshotsDir = ProcessInfo.processInfo.environment["USE_TEMP_SNAPSHOTS_DIR"] != nil
                  ? URL(fileURLWithPath: NSTemporaryDirectory(), isDirectory: true)
                    .appendingPathComponent("__Snapshots__")
                    .appendingPathComponent(fileName)
                  : nil

let snapshotDirectoryUrl = snapshotDirectory.map { URL(fileURLWithPath: $0, isDirectory: true) }
?? envSnapshotsDir
?? fileUrl
  .deletingLastPathComponent()
  .appendingPathComponent("__Snapshots__")
  .appendingPathComponent(fileName)

Option 4? Extend sandbox somehow
It would be great if we could simply pass the path to the snapshots directory as an argument to the executable running the unit tests. I'm not aware that extending the app sandbox this way is possible, but maybe something along those lines?

I ran into the same issue, is there a solution for this issue?


[Bug] "Cannot find namespace google" on version => 1.1.1

Description
Since updating to version 1.1.1 and 1.1.2 from 1.1.0, I am no longer able to build my project as the google namespace is not available.

Hard coding 1.1.0 and reinstalling works perfectly.

Including @types/google.maps with the updated library version also did not seem to work.
Steps to Reproduce
Set your dependency version to<EMAIL_ADDRESS>"^1.1.1" or<EMAIL_ADDRESS>"^1.1.2",
Define a polygon object - or anything else from google.maps
let polygon: google.maps.Polygon
Environment

Library version: => 1.1.1
Google maps version: weekly
Browser and Version: n/a
OS: n/a

Logs
No response

Hmm, there's gotta be more to this. Just created a fresh project to check, installed<EMAIL_ADDRESS>and it worked as expected.
Can you check that<EMAIL_ADDRESS>exists in your project when the error occurs?
Also, when is the error happening? There's something about npm run build, and I assume it's a next.js app. Could it be that you're tyring to use the google.maps namespace during prerendering and before the maps-api is actually loeded?

for reference, when I try to do this:
export default function Home() {
  const foo = new google.maps.Polygon();

  return (
    // ...
  );
}

I get an error message like this:
$ npm run build

> build
> next build

  ▲ Next.js 14.2.8

 ✓ Linting and checking validity of types
   Creating an optimized production build ...
 ✓ Compiled successfully
 ✓ Collecting page data
   Generating static pages (0/3)  [    ]ReferenceError: google is not defined
    at g (/Users/martins/src/ubilabs/oss/rgm-next-test/.next/server/pages/index.js:1:1038)
    at Wc (/Users/martins/src/ubilabs/oss/rgm-next-test/node_modules/react-dom/cjs/react-dom-server.browser.production.min.js:68:44)
    at Zc (/Users/martins/src/ubilabs/oss/rgm-next-test/node_modules/react-dom/cjs/react-dom-server.browser.production.min.js:70:253)
    at Z (/Users/martins/src/ubilabs/oss/rgm-next-test/node_modules/react-dom/cjs/react-dom-server.browser.production.min.js:76:89)
    at Xc (/Users/martins/src/ubilabs/oss/rgm-next-test/node_modules/react-dom/cjs/react-dom-server.browser.production.min.js:68:409)
    at Zc (/Users/martins/src/ubilabs/oss/rgm-next-test/node_modules/react-dom/cjs/react-dom-server.browser.production.min.js:70:210)
    at Z (/Users/martins/src/ubilabs/oss/rgm-next-test/node_modules/react-dom/cjs/react-dom-server.browser.production.min.js:76:89)
    at Zc (/Users/martins/src/ubilabs/oss/rgm-next-test/node_modules/react-dom/cjs/react-dom-server.browser.production.min.js:70:481)
    at Z (/Users/martins/src/ubilabs/oss/rgm-next-test/node_modules/react-dom/cjs/react-dom-server.browser.production.min.js:76:89)
    at $c (/Users/martins/src/ubilabs/oss/rgm-next-test/node_modules/react-dom/cjs/react-dom-server.browser.production.min.js:78:98)

Error occurred prerendering page "/". Read more: https://nextjs.org/docs/messages/prerender-error

ReferenceError: google is not defined
    at g (/Users/martins/src/ubilabs/oss/rgm-next-test/.next/server/pages/index.js:1:1038)
    at Wc (/Users/martins/src/ubilabs/oss/rgm-next-test/node_modules/react-dom/cjs/react-dom-server.browser.production.min.js:68:44)
    at Zc (/Users/martins/src/ubilabs/oss/rgm-next-test/node_modules/react-dom/cjs/react-dom-server.browser.production.min.js:70:253)
    at Z (/Users/martins/src/ubilabs/oss/rgm-next-test/node_modules/react-dom/cjs/react-dom-server.browser.production.min.js:76:89)
    at Xc (/Users/martins/src/ubilabs/oss/rgm-next-test/node_modules/react-dom/cjs/react-dom-server.browser.production.min.js:68:409)
    at Zc (/Users/martins/src/ubilabs/oss/rgm-next-test/node_modules/react-dom/cjs/react-dom-server.browser.production.min.js:70:210)
    at Z (/Users/martins/src/ubilabs/oss/rgm-next-test/node_modules/react-dom/cjs/react-dom-server.browser.production.min.js:76:89)
    at Zc (/Users/martins/src/ubilabs/oss/rgm-next-test/node_modules/react-dom/cjs/react-dom-server.browser.production.min.js:70:481)
    at Z (/Users/martins/src/ubilabs/oss/rgm-next-test/node_modules/react-dom/cjs/react-dom-server.browser.production.min.js:76:89)
    at $c (/Users/martins/src/ubilabs/oss/rgm-next-test/node_modules/react-dom/cjs/react-dom-server.browser.production.min.js:78:98)
 ✓ Generating static pages (3/3)

> Export encountered errors on following paths:
	/

The error isnt during build time, Ive had those before and found the solution - waiting for the google maps to actually load in the window.
All i did here was change 1.1.0 to 1.1.1 and run npm install, and I get this error.

You can see types/google.maps is available

I have just tried one of the examples provided using 1.1.1 and i am not seeing an issue.
Im very confused.

I see in the example project using 1.1.1, google maps type is
    "name"<EMAIL_ADDRESS>    "version": "3.58.0",

vs
    "version": "3.55.12",
    "description": "TypeScript definitions for google.maps",

Further debugging shows its an issue with my tsconfig.json, but only with the library version =>1.1.1
Further, further debugging shows its an issue with the types array in my tsconfig.json
I dont know why this would have only come about in the latest versions, but it seems that you need to add google.maps to the types array.
      "types": ["vitest/globals", "google.maps"],

Now here is something interesting: I had a look at a full diff of the package contents and noticed that for some reason the 1.1.0 version had a triple-slash directive referencing google.maps in every one of the .d.ts files.

Those are no longer there in the 1.1.2 version. Those aren’t in the source files, so they must have previously been added by the typescript compiler or microbundle/rollup.
This is also why adding it to the types in your tsconfig.json solves it for you. What it doesn't explain, though, is why this error doesn't happen with a fresh installation. Could it be that you have typeRoots specified in your tsconfig and that doesn't include the node_modules/@types directory?

This is my current tsconfig.
Ive also just noticed that simply removing the types array altogether also allows everything to work just fine.
{
  "compilerOptions": {
    "lib": ["dom", "dom.iterable", "esnext"],
    "module": "esnext",
    "moduleResolution": "bundler",
    "resolveJsonModule": true,
    "removeComments": true,
    "preserveConstEnums": true,
    "strict": true,
    "alwaysStrict": true,
    "strictNullChecks": true,
    "strictPropertyInitialization": true,
    "noUncheckedIndexedAccess": true,
    "noImplicitAny": true,
    "noImplicitReturns": true,
    "noImplicitThis": true,
    "exactOptionalPropertyTypes": true,
    "noUnusedLocals": true,
    "noUnusedParameters": true,
    "allowUnreachableCode": false,
    "noFallthroughCasesInSwitch": true,
    "target": "es2020",
    "useDefineForClassFields": true,
    "outDir": "out",
    "sourceMap": true,
    "esModuleInterop": true,
    "allowSyntheticDefaultImports": true,
    "allowJs": true,
    "checkJs": true,
    "skipLibCheck": true,
    "forceConsistentCasingInFileNames": true,
    "jsx": "preserve",
    "noEmit": true,
    "isolatedModules": true,
    "incremental": true,
    "types": ["vitest/globals", "google.maps"],
    "baseUrl": ".",
    "paths": {
      "@/*": ["src/*"],
      "@/public/*": ["./public/*"]
    },
    "plugins": [
      {
        "name": "next"
      }
    ]
  },
  "exclude": ["./out/**/*", "./node_modules/**/*", ".storybook/*.ts"],
  "include": [
    "git.properties",
    "next-env.d.ts",
    "**/*.ts",
    "**/*.tsx",
    "types.d.ts",
    ".next/types/**/*.ts",
    "**/*.mts"
  ]
}

It seems that this is more of a project configuration issue on my end, but whatever was generating that reference in the 1.1.0 version seems to make the library more robust. I think if possible, it would be good to bring that change to the later versions as well.

Seems to be one of the changes of typescript 5.5: https://devblogs.microsoft.com/typescript/announcing-typescript-5-5/#simplified-reference-directive-declaration-emit

I published 1.1.3, can you test this in your old setup to see if it is now working correctly?

That did it!
I added my types array back to the tsconfig and omitted google.maps, bumped to v1.1.3. Everything seems to be working fine now.
Thanks for getting this resolved so quickly.


invalid pointer error while using reshape

I tried to use reshape of a executor. If the executor is defined in cpu, it was fine. But when I tried to use it in gpu mode, I got this error:
Error in 'python' : munmap_chunk(): invalid pointer: 0x00000001042adb4
Aborted (core dumped)

any idea how this happened? Thanks a lot

Could you provide a script that reproduce this error? It's hard to figure out what exactly you have done.

ctx = mx.gpu(0)
data = mx.symbol.Variable("data")
net = mx.symbol.Convolution(data=data, kernel=(3,3), num_filter=64)
net = mx.symbol.Activation(data=net, act_type="relu")
net = mx.symbol.Convolution(data=net,kernel=(3,3), num_filter=64)
net = mx.symbol.Activation(data=net, act_type="relu")
net = mx.symbol.Pooling(data=net, pool_type="max", kernel=(2,2))
executor = net.simple_bind(ctx=ctx,data=(1,3,512,512),grad_req="null")
executor = executor.reshape(allow_up_sizing=True, data = (1,3,400,600))
executor.forward(is_train=False)

you can try this. The error is kind of unpredictable. Sometimes, it happens. Most of the time, I would get a segmentation fault error.  As long as I use this function, either I get a segmentation fault or a pointer error. Please check it out.

ctx = mx.gpu(0)
data = mx.symbol.Variable("data")
net = mx.symbol.Convolution(data=data, kernel=(3,3), num_filter=64)
net = mx.symbol.Activation(data=net, act_type="relu")
net = mx.symbol.Convolution(data=net,kernel=(3,3), num_filter=64)
net = mx.symbol.Activation(data=net, act_type="relu")
net = mx.symbol.Pooling(data=net, pool_type="max", kernel=(2,2))
executor = net.simple_bind(ctx=ctx,data=(1,3,512,512),grad_req="null")
executor = executor.reshape(allow_up_sizing=True, data = (1,3,400,600))
executor.forward(is_train=False)

you can try this. The error is kind of unpredictable. Sometimes, it happens. Most of the time, I would get a segmentation fault error.  As long as I use this function, either I get a segmentation fault or a pointer error. Please check it out.

@piiswrong

This is caused by destruction of the first executor. Not reshape. The crash point is in cudnnDestroyTensorDescriptor in cudnn_activation-inl.h
I'm looking into it.

@ascust Should be fixed once https://github.com/dmlc/mxnet/pull/1481 is merged.


Inference example

In the readme inference example, in the case we want to loop on more than one sequence can we recycle the inference processor. Is clear_memory enough to process a new video/sequence?

You should create a new inference core for each new video (practically free). clear_memory is for when the memory becomes "corrupted" and the user wants a clean slate while tracking the same set of objects.

ok thanks.


add permissions to semantic-release

:tada: This PR is included in version 0.0.13 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Canvas Background mode causes UI to resize when rendering/scaling_3d/scale is changed

Godot version
4.0beta6
System information
Windows 10, Vulkan
Issue description
Whenever you change rendering/scaling_3d/scale and reload the editor, UI will get resized as well when canvas background mode is used

Steps to reproduce
Create a WorldEnvironment node
Change background_mode to BG_CANVAS
Change rendering/scaling_3d/scale at settings
Reload the editor
Enjoy the show
Minimal reproduction project
LowRes.zip

Duplicate of https://github.com/godotengine/godot/issues/69122
Closes as the issue is best tracked in a single place


[bug]

Describe the bug
A clear and concise description of what the bug is.
using your package i can able to post a tweet successfully. thank you for that.  but i cannot able to reply a tweet using your package.
To Reproduce
Please indicate all steps that lead to this bug:

Request client setup (login method, OAuth2 scopes if applicable...)

const client = new TwitterApi({
appKey: config.twitter_config.api_key,
appSecret: config.twitter_config.api_secret,
accessToken: config.twitter_config.access_token,
accessSecret: config.twitter_config.access_secret,
});
using these  authentication. works for create a tweet.
** Endpoint used**

Endpoint used or code example of what's happening wrong
async function replyToTweet() {
try {
 	let tweet =  await twitterClient.v1.reply('Look at my video!',<PHONE_NUMBER>555717120);
 	  console.log('Reply sent successfully:', tweet.text);
 	} catch (error) {
 	  console.error('Error replying to tweet:', error.data);
 	}
   }

Error stack trace

Error stack trace, and if possible, error content (err.toJSON() when err is the caught error; take care of removing authentication HTTP headers)

data: {
title: 'Unsupported Authentication',
detail: 'Authenticating with OAuth 2.0 Application-Only is forbidden for this endpoint.  Supported authentication types are [OAuth 1.0a User Context, OAuth 2.0 User Context].',
type: 'https://api.twitter.com/2/problems/unsupported-authentication',
status: 403
}
Expected behavior
A clear and concise description of what you expected to happen.
*i want to post reply in tweet
Version

Node.js version - 14
Lib version -  "twitter-api-v2": "^1.14.2",
OS (especially if you use Windows)  - windows

Additional context
Add any other context about the problem here.

Applications can't create tweets, only users can.


Speed up the build and release tasks

by not skipping man:clean.
Doing man:clean on every build defeats the purpose of having the
txt2roff/ronn2roff tasks declared as file tasks, rendering the file-task
associated timestamp checking useless and it slows down rake build by
about 3.5s on my laptop.
This removes the man:clean dependencies.

The point of defining the tasks as file tasks is so that you can run rake man:build repeatedly. It has nothing to do with build/release. The reason that it is required to run man:clean before any release is that manpages can be added and removed between releases, and the only way to get a clean build of the manpages for a particular release is to run clean and then build.

Well, you still don't have to clobber all of your generated manpages and start from scratch, you just have to mirror deletes in the source directory, which takes abot 3--4 LOCs and it massively speeds up the whole thing:
https://github.com/bundler/bundler/compare/master...pjump:rakefile?expand=1

Can you explain why you want to optimize rake build so much? I'm willing to accept a pull request that speeds it up, I just haven't ever even felt like it was slow. It really only needs to be run at release-time...


Deprecate ZygoteRules, use ChainRulesCore

Zygote now uses ChainRules as backend for rules

from autodiff channel in slack:
rrule(typeof(possible), _args...) = false, _ -> (Zero(), DoesNotExist())


packages contain errors

go-callvis github.com/okex/okchain
-: no Go files in /Users/yangfan/go/src/github.com/okex/okchain
2020/05/19 10:56:21 packages contain errors
I'm sure this file exists in this path

me too
E:\projects\go_pro\src\github.com\ofabry\go-callvis>go run . E:\projects\go_pro\src\sdyxmall\business-gift -: go: directory ..\..\..\sdyxmall\business-gift outside available modules 2020/05/21 12:21:32 packages contain errors exit status 1

how can we resolve this error ? @ofabry


fix(reporter): Handle vulnerabilities being undefined

fix #3

:tada: This PR is included in version 1.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


.sql file doesn't open

If I assign dbeaver as the default application for a .sql file on MacOS or right click and open with dbeaver, it does not open the sql file.
I'd like it to open a new sql editor tab with the contents of the file.
Currently, I'm just opening the sql file up in another editor and then doing a copy / paste, but it would be nice if I could just double click an sql file and have it open in dbeaver.

Thank you for the idea.

Fixed.

verified


Update PPP

Codecov Report

Merging #225 (ab77325) into master (30aa897) will increase coverage by 0.75%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #225      +/-   ##
============================================
+ Coverage     77.85%   78.61%   +0.75%
- Complexity      699      729      +30
============================================
  Files            97       97
  Lines          2186     2301     +115
  Branches        269      297      +28
============================================
+ Hits           1702     1809     +107
- Misses          387      393       +6
- Partials         97       99       +2

Impacted Files
Coverage Δ

...ommands/modulelesson/ClearModuleLessonCommand.java
55.55% <0.00%> (-4.45%)
:arrow_down:

src/main/java/seedu/address/ui/PersonCard.java
0.00% <0.00%> (ø)

...ss/logic/commands/exceptions/CommandException.java
100.00% <0.00%> (ø)

...commands/modulelesson/FindModuleLessonCommand.java
100.00% <0.00%> (ø)

...commands/modulelesson/ListModuleLessonCommand.java
100.00% <0.00%> (ø)

...mmands/modulelesson/DeleteModuleLessonCommand.java
100.00% <0.00%> (ø)

...er/modulelesson/FindModuleLessonCommandParser.java
93.15% <0.00%> (+1.66%)
:arrow_up:

...ess/model/modulelesson/UniqueModuleLessonList.java
88.73% <0.00%> (+2.06%)
:arrow_up:

.../modulelesson/DeleteModuleLessonCommandParser.java
97.67% <0.00%> (+3.92%)
:arrow_up:

Continue to review full report at Codecov.

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


Update jest.config.json for windows compatibility

but this user changed more things on his PR
https://github.com/jaredpalmer/razzle/pull/1218#issue-391429543

Yes, but they are no longer needed after last pull by me,

GGGGGGGGGG 🚀


[libLAS - arm64] build failure

Host Environment

OS: MacOS - building for iOS
Compiler: AppleClang <IP_ADDRESS>00032

To Reproduce
Steps to reproduce the behavior:
sudo ./vcpkg install --triplet=arm64-osx liblas
After fixing the dependencies:

libffi fix - replace ARM64 with arm64 in line 14 in ~/vcpkg/ports/libffi/CMakeLists.txt
proj4 fix - build proj4 for x64 and copy ~/vcpkg/installed/arm64-osx/tools/sqlite3 from x64-osx

It came the time for libLas to compile and it fails with an error in ../incluide/liblas/detail/endian.hpp as arm64 does not have a definition to decide on LIBLAS_LITTLE_ENDIAN and LIBLAS_BYTE_ORDER.
Failure logs
The error is found in install-arm64-osx-dbg-out.log as follows:
vcpkg/buildtrees/liblas/src/libLAS-1-100581b933.clean/src/../include/liblas/detail/endian.hpp:96:3: error: The file liblas/detail/endian.hpp needs to be set up for your CPU type.
error The file liblas/detail/endian.hpp needs to be set up for your CPU type.
^
1 error generated.

Thanks for posting this issue. liblas:x64-osx can install succeed. arm64-osx is a community triplet, and there are no update about this issue for a long time. If there are new process or change of this issue, please reopen this issue.


Copy from docker buildprocessor just works™

What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
/area cmd
/area pkg
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
new: timeout flag
fix: docker copy outputs just the module

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign fntlnz
You can assign the PR to them by writing /assign @fntlnz in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leodido
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leodido]

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

LGTM label has been added.  Git tree hash: 5a12e4a8b05190b013a2628fa2851d177822fe55


Issue 35

#35

Coverage increased (+0.4%) to 89.764% when pulling 43f98c5171f3815378399f786c77090be8bf8f9e on issue-35 into b80da4f1478c3e49760b508b492386a5e23814fb on master.

Coverage increased (+0.4%) to 89.764% when pulling 364de9af158f911a68fdb3f49f62e4a7fa585935 on issue-35 into b80da4f1478c3e49760b508b492386a5e23814fb on master.


[Suggestion] Don't show the exact movie on multiple instances at the same time

I would love to have multiple display showing some posters one next to each other, just like at the cinema, but sometimes they show the exact movie at the same time and it ruins a bit the illusion.
It would be nice if it was possibile to add an option for showing different movies between the instances of the web browser.

Easiest way to do this is to create a second container for posterr on a different port, then call this on the second monitor. The container uses very minor resources, so you could run a bunch of them if you wanted to.


Support list() ctor for unicode types

Feature request
As title. This sort of thing should work:
list('abc')

Seems like this will work if unicode types become iterable https://github.com/numba/numba/issues/3836

Fixed by #3836


Rich text crashes when clicked multiple times in a row

Category
[ ] Enhancement
[x] Bug
[ ] Question
Version
Please specify what version of the library you are using: [3.5.0]
Expected / Desired Behavior / Question
Nothing happens.
Observed Behavior
The application crashes (react crashes), with message:
Uncaught Invariant Violation: Maximum update depth exceeded.
This can happen when a component repeatedly calls setState inside componentWillUpdate or componentDidUpdate.
React limits the number of nested updates to prevent infinite loops.

Crashes here, at the row 867, this.setState:
https://github.com/pnp/sp-dev-fx-controls-react/blob/2f2839739fbd6337fa171cab88afc67e1b434f53/src/controls/richText/RichText.tsx#L854-L873
Steps to Reproduce
Quickly (milliseconds) click in the RichText text control several times, like five or so (using left mouse button).
The rich text should not be empty (should contain some formatted text, such as a hyperlink)
It looks like the handler just enters the infinite loop, maybe a safeguard around it could solve it.

After investigating this a bit, there is an infinite loop (seems to be caused by fixing #666)
Setting the control value resets it's selection that is causing selection change event, then selection change event again resets the value with setState(), this causes the whole thing to loop.

@nbelyh many thanks for investigating it and providing more details. Would you be available to fix it and send a PR?

@joelfmrodrigues unfortunately I am not sure how to fix this properly... 😢  The issue looks a bit complicated for me. There were already several attempts to fix the core thing as far as I understand (linked issue), i.e. the <RichText value={....}> thing. There are even comments in the code related to it 😄  For myself, I just created a workaround of replacing the value={} with defaultValue={} thing and updating the text manually in my code in onChange event. But this won't work in general case, described in the linked issue.

I will look at it for sure.
Little business: find appropriate block of time :)


Add get releases and set which release to download

Hi, there
I'm combining release discovery from https://github.com/OvertureMaps/data/blob/main/utils/fetch-releases-from-s3.py into this python CLI, which enable user to check releases and download to latest release (provided the S3 tree not changing).
If its ok to merge this functionality then its good, if not feel free to reject this PR.
Many thanks for the whole effort.

Hi @andibuwono - Thanks for this!
I've been hesitant to have something like for this for a few reasons. One is that the mere presence of a release folder doesn't automatically indicate that the data is officially released. I think we've had a few instances where data was staged there but we found some last minute things and pulled it back. Nevertheless, I think it's worth it to have a way of listing all releases.
What if instead of a get_releases command, it was a new cli group called "releases" with a subcommand "list". So it would be called like overturemaps releases list. I could imagine us wanting more like overturemaps releases info 2024-08-20.0 or something like that. But just 'list' is a great start.
Allowing the version to be specified is a good idea - but one thing that needs to change is that the theme/types now become version dependent so we'd have to change that. Want to change this (or submit a new PR) with just the new releases subcommand? We can work out the details on specifying a specific version after.

Hi @jwass I'll update my PR, I think thank quite enough that we can check the releases and.update installed packages to get the new release uri

hi @jwass already revised to include only releases list command,
thanks


Not working on CodeChef

This is because codechef now started using https instead of http. Thus we should change pattern in native host manifest json.

Fixed.


Text colour on FAQ

please, can this not be blue? what do you think @av80r

@av80r changed my mind, it looks fine


Extra comma after "line"

Hello
I'm new to Gwen. Tried to use it to build some test cases.
I used this command to run the test: gwen -b -m 1_MJ-181.meta -f json -Dgwen.web.browser.headless=true --parallel -r target/reports features
and it will generate a json file.
Upon feeding the content of this json file into json parser, it shows that there is a comma at end of all lines "line": X
I believe these comma shouldn't be there.
Or am I doing something wrong?

"tags": [
{
"name": "@MJ-181",
"line": 1,
}
],
"elements": [
{
"keyword": "Scenario",
"id": "scenario;test-google-search-(gwen-web-demo);1",
"line": 6,
"name": "Test Google search (Gwen-web-demo)",
"description": "",
"tags": [
{
"name": "@MJ-182",
"line": 5,
},
{
"name": "@MJ-186",
"line": 5,
}
],
"type": "scenario",

Thanks for raising this @charleslcso. We've release a fix in gwen-web v3.53.7. Please try that and close issue when ready.

Nice! It is fixed! Closing issue now.


Add Jackson's AMOS publication

@the-one-and-only-jackson Can you please add your AMOS publication to the bib file by the end of the week?

Please also add a blurb to the news list about your presentation

Hi @the-one-and-only-jackson , it appears the AMOS publications are posted. Can you add a link to the bib file and news file by the end of this week?

The URLs for both AMOS papers have been added

Thanks @the-one-and-only-jackson ! In the future, please close issues once they are completed, thanks.


ElimLocalMultiStore: Reset structured successors for each function

Fixes crash reported in issue #824

@dneto0 I looked at the other definitions of ComputeStructuredSuccessors (I really need to consolidate these).
The reason the "multi-store" version failed is because it uses a pseudo-entry-block. The p-e-b was collecting successors across functions. Technically I could have just cleared the p-e-b's successors, but it was just easier to clear them all.
None of the other versions of CSS use a p-e-b. The reason is because "multi-store" needed all blocks in a function to be processed, even if they weren't reached. The other passes don't care if unreached blocks don't get processed.


Fix publish to allow for dev-dependencies to have circular references

Is your feature request related to a problem or challenge?
Currently we cannot have any dev-dependencies in the Cargo.toml files that would cause circular references because cargo publish in all it's wisdom doesn't exclude dev-dependencies when publishing to crates.io. This causes issues with tests where we either have to move the test to some 'common' location when the test wants to use a dependency that we otherwise do not want in the crate (for example, a dependency on datafusion-functions in the physical-expr crate), remove the test or somehow rewrite the test to not require the dependency. This is pretty limiting and really shouldn't be an issue.
I propose that the cargo publish commands that are used are tested with the --no-dev-deps flag from the hacks crate. If successful we could then once again allow the circular dev-dependencies which would ease testing.
Note that this change would not allow for dev-dependencies in rust docs -- it would likely work locally but I believe would likely still fail when published.
Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
No response

cc @andygrove
I am not 100% convinced that having the circular back references as dev-dependencies is a good idea (as it complicates / slows down cargo test -b <crate>
If we need to depend on more crates, pulling the tests into datafusion_core is probably the better approach

Sorry, but I'm not a fan of this approach. I think it is good to avoid circular dependencies of any kind. We should use rustdoc only to document the contents of a crate and not need to reference "parent" crates.
I think that the real issue here is that we don't have a good way yet to have end-to-end user documentation that includes Rust fragments that are tested for correctness. I would prefer that we find a solution to that problem.

Closing as won't do.


As a developer, I need to resize the graphics size

Where: In the search tool (tested), probably also in the Inspect and Compare tools
Issue: the legend in Y axis gets hidden by the switch button when the graphic is not high enough (see word "recor":

@avlisodraude: Tested in Master branch (since branch "#260" was already merged). I also tested in branch #166. The graphic seems to be OK when resizing, but you mentioned you had moved the label "number of records" to the bottom left, please check before sending pull request.

@avlisodraude I see this solved in branch 166, ready to be merged!


Fix multi-store URL path generation

Fix will correct behaviour on multi-store setups with same category structures

Merged pull request.


Include a setup() function for user configuration

Hi! I really like the colors from this plugin, but would like to tweak it a little bit according to my taste. I think this is something that other users might also want. I've modelled this functionality to work like https://github.com/ful1e5/onedark.nvim, which is a colorscheme I really liked to use.
most lua plugins have setup function, which takes a table with configuration as argument. This pattern is followed with this PR. Users can configure the same options as before, but now via the setup function. It is also possible to override colors defined in this theme as well as whole highlight groups.
I hope you like the changes I've made, and you are willing to include these into the repository.
Some more information:
User can configure the following settings:

transparent bg (default false)
italic_comments (default false)
disable_nvimtree_bg (default false)

And override the following:

colors (overriding the definitions in colors.lua)
groups (overriding the group highlights as defined in theme.lua)

Other things this PR/commit changes:

colors.lua now returns a table with the colors, instead of a function
generate() that returns this table.
utils.lua is removed, because it only had one function that was called
once (in init.lua:set().
Ran stylua

For backwards compatibility, it might be good to also check the vim.g.vscode_* variables. Will add support for it now.


img tag

While testing a migration from my Octopress 2 site, I noticed an error for the "img" tag that was a default option in Octopress 2 ( http://octopress.org/docs/plugins/image-tag/ ) was missing.  Should this be substituted with a jekyll plugin in my Gemfile or should we pull from  https://github.com/imathis/octopress/blob/master/plugins/image_tag.rb
With some plugins missing, it also seems odd that the _plugins folder is not created by default when doing a "octopress new".

I'm working on adding stand-alone plugin to replace what's in Octopress 2.0 (with tests!) but I haven't done them all yet. The reason there isn't a _plugins directory is that jekyll new doesn't add it and Octopress uses Jekyll's scaffolding but adds its own.

I've just released 1.0.0 of octopress-image-tag. Which you can find here: https://github.com/octopress/image-tag. It should be a drop in replacement for the plugin you're currently using.

I've added gem 'octopress-image-tag' to my Gemfile

source 'https://rubygems.org'

gem 'jekyll'

group :jekyll_plugins do
    gem 'octopress', '~> 3.0.0.rc'
    gem 'octopress-image-tag'
end

and bundle updated,
but I'm still getting an error.

Configuration file: /Users/e9t/dev/sites/lucypark.kr/source/_config.yml
            Source: /Users/e9t/dev/sites/lucypark.kr/source
       Destination: /Users/e9t/dev/sites/lucypark.kr/source/_site
      Generating...
  Liquid Exception: Unknown tag 'img' in _posts/2010-01-27-data-reduction-and-data-production.markdown
jekyll 2.5.3 | Error:  Unknown tag 'img'

Am I missing a step or doing something wrong?

You need to add it to the _config.yml file as well.
On Jul 25, 2015 11:45 PM, "Lucy Park"<EMAIL_ADDRESS>wrote:

I've added gem 'octopress-image-tag' to my Gemfile
source 'https://rubygems.org'
gem 'jekyll'
group :jekyll_plugins do
gem 'octopress', '~> 3.0.0.rc'
gem 'octopress-image-tag'
end
and bundle updated,
but I'm still getting an error.
Configuration file: /Users/e9t/dev/sites/lucypark.kr/source/_config.yml
Source: /Users/e9t/dev/sites/lucypark.kr/source
Destination: /Users/e9t/dev/sites/lucypark.kr/source/_site
Generating...
Liquid Exception: Unknown tag 'img' in _posts/2010-01-27-data-reduction-and-data-production.markdown
jekyll 2.5.3 | Error:  Unknown tag 'img'
Am I missing a step or doing something wrong?
—
Reply to this email directly or view it on GitHub
https://github.com/octopress/octopress/issues/70#issuecomment-124945378.

Ah, that fixes it. Thanks @busches!

Thanks for the assist @busches!


Dev mode

adding dev mode , dev cluster ci and fixing gak port mapping

@webcoderz should we be landing this PR as it's growing quite large?
as part of that, azure pipelines seems to be failing on it: https://graphistry.slack.com/archives/D02Q4SZ81N2/p1654800986419379

fix az pipeline stuff & merge?

https://github.com/graphistry/graphistry-helm/blob/7fb3d6e78ae4987a3b6e12cd2857d44dab20b505/acr-bootstrap/import-image-into-acr-from-dockerhub.sh#L71 yea here's where gak is imported , we started adding cuda tag on 2.39.13 should I add some logic in here to take non cuda tag into account? Also need to delete/comment out redis from delete acr containers script also

should I add some logic in here to take non cuda tag into account

streamlit container can assume cuda tag for now, let's get single node solid before worrying about multi-node (where streamlit might be cpu)


Parsing for Q# Attributes

Adds support for parsing the syntax of attributes. Includes some specific error handling for if the ending tag is missing on an attribute or the arguments are missing for an attribute, as well as context verification so that the attribute is only above an operation, function, or type definition.
Making this a draft pull request until a feature branch exists for attributes.
Targets #81

@jviszlai Thanks again for the contribution, and sorry for the delay in processing!
I don't know what the best way is to contact you, but if you reach out and contact me on my Microsoft address we've got a surprise for you. :) Purely optional of course.

@jviszlai Perfect, thanks!


Q: racer integration, find_definition usage

I recently started integrating racer for YouCompleteMe support, and this is the first of what I'm sure will be many questions :smile:. This question regards find_definition.
pub fn find_definition(src: &str, filepath: &path::Path, pos: usize, session: SessionRef) -> Option<Match>

One of the arguments is a source buffer. If I don’t write the buffer to disk first, I get a panic (full bt below).
thread 'engine::racer::tests::find_definition' panicked at 'index 0 and/or 89 in `` do not lie on character boundary', ../src/libcore/str/mod.rs:1441

However, writing the buffer to disk completely resolves that problem. It seems to me that, given a buffer, reading the initial file should be unnecessary. Is this a requirement? I would like to change the behavior to not do that if possible. I actually couldn’t find where it was doing that IO, either. Additionally, since this method is doing I/O, maybe it should return a Result instead of panicking while indexing a slice.
Backtrace
   6:        0x10c490097 - rust_begin_unwind
   7:        0x10c4b4060 - panicking::panic_fmt::h6c78ce0128588a957HK
   8:        0x10c4b7b60 - str::slice_error_fail::hae4b5de9c67475548OS
   9:        0x10c0f44b5 - str::traits::_<impl>::index::he8945502e3410953mBS
  10:        0x10c0f437e - string::_<impl>::index::h547be55ef8b5db17i3f
  11:        0x10c0f3b92 - core::_<impl>::deref::h6728862e5c44cb4atMa
  12:        0x10c1030aa - scopes::mask_comments::h758821d339eb152bJeb
  13:        0x10c10e321 - scopes::scope_start::h49adfd7fdda43b7544a
  14:        0x10c1c6f90 - nameres::search_local_scopes::h2aa638ee8218b5856xe
  15:        0x10c1bc129 - nameres::resolve_name::ha8fef8b2d1cc0115pPe
  16:        0x10c10912e - nameres::resolve_path::hd3f93ff20151730eeZe
  17:        0x10c10c842 - core::find_definition_::h8b9c5b4772ebed00tYa
  18:        0x10c10c16b - core::find_definition::h36205eee38345da54Xa
  19:        0x10c0dc75b - engine::racer::_<impl>::find_definition::h6f5d2878fe1b3851yea
  20:        0x10c0e34c0 - engine::racer::tests::find_definition::h607071473acdcea1yla
  21:        0x10c2469eb - boxed::_<impl>::call_box::call_box::h5504267803333294456
  22:        0x10c2490f0 - sys_common::unwind::try::try_fn::try_fn::h8503899523036047578
  23:        0x10c48fed8 - __rust_try
  24:        0x10c48de7e - sys_common::unwind::try::inner_try::h4c901f4c815f0df8y9r
  25:        0x10c2494ea - boxed::_<impl>::call_box::call_box::h16742266516396999763
  26:        0x10c4918ad - sys::thread::_<impl>::new::thread_start::h9bc185148c33bf72Kvw
  27:     0x7fff8add49b0 - _pthread_body
  28:     0x7fff8add492d - _pthread_start

@birkenfeld: @phildawes mentioned you might have some thoughts on this.

Hi @jwilm ,
The &str arg to find-definition is a bit historical and needs cleaning up, I didn't even realise it was still there!
Basically racer loads source from disk lazily and is expecting there to be a file there. Recently @birkenfield added a cache for source artifacts in the Session object.
The current racer plugins supply a filename and optionally a tmpfile to load the source from.
e.g. In the main.rs find_definition() function racer loads source from a tmpfile, using the session
    let session = core::Session::from_path(&fpath, &substitute_file);
    let src = session.load_file(&fpath);
    let pos = scopes::coords_to_point(&src, linenum, charnum);
    core::find_definition(&src, &fpath, pos, &session).map(|m| match_fn(m, &session));

I think the way to solve your problem is to load the source text into the session before calling find_definition. You'll probably want to add a new method to Session to load source text if that is how racerd is going to work.
Hope this helps, sorry this is a bit crufty.

Btw, I agree re: Result. When racer was first written Result wasn't as big a part of rust as it is now, and there was no From trait conversion easing Error propagation. Even rustc's libsyntax paniced on syntax errors until I posted rust-lang#23857 in february.
Having said that, panic backtraces are really handy, especially if you compile with debugging to get line numbers.

Good news, everyone!
I've made some serious progress on racerd and the YCM integration. Here's a quick completion demo.

Of course, this wouldn't be possible with all the work you've done here on racer; thank you!
There is one challenge I've run into that may not be a simple fix. One thing that I've done in racerd is keep a Session alive across http requests. Aside from saving tons of disk io, it measurably improved response time. This is great!. In my fork of racer, I've got some changes to the Session that allow me to update files in the FileCache without reading from disk. This is critical in YCM since each request contains all of the (potentially unsaved) buffers the user has open.
The problem here is that I cannot get a mutable reference to the IndexedSource object currently tracked in the FileCache's raw_map or masked_map. To get around this so far, I'm just asking the arena for a new object and updating the reference stored in those maps. Obviously this is really bad because the arena will just grow indefinitely.
Do you guys have any thoughts on how to make this work and keep the niceness of using an Arena in the FileCache? I was thinking the most straight forward way would be to stuff a RefCell into the IndexedSource. Aside from just making that work, I don't know what kind of consequences mutating the source String in IndexedSource would have. It seems that any Src objects it has handed out should become invalidated. Beyond that, I can't speculate about the consequences. @birkenfeld, I think you may have some insight here already since you wrote the latest Session implementation.
Thanks again!

Hi @jwilm,
This looks awesome!
Personally I'd be inclined to ditch the Arena for now and just try to find a way to do this using vanilla heap allocation. If it causes performance problems then we can re-visit.
(other people may disagree).

Well, the point of the Arena is to give all source references the lifetime of the session. Without it, you're relegated to using Rc in a lot of places, or having a crapton of lifetimes to care about.
In a long running process, I think you'll have to use a two-layered structure: an object (that we don't have currently) that is kept alive between requests, and an object (like the Session) that persists throughout one request.
One way I can think of right now is to have the arena append-or-invalidate-only during a request, and to clean it out after/before the next one, in the code that mutably owns the cache. This likely requires a specialized Arena like type that you'll have to write (not that difficult probably).

@birkenfeld is the idea with the two-layered object is that we know no references are handed out once the Session is destroyed? That's the issue I keep coming back to with a mutable file cache in the current setup.

Yep, that's the idea.

@birkenfeld
I started looking into separating out the FileCache and Session as a proof-of-concept before going and making an Arena with recyclable members, and I ran into some unexpected lifetime errors.
Here are the changes. The Session::from_path method now accepts a &FileCache in addition to the two paths. So, I updated the tests to have a cache in the stack and pass it into all of the from_path calls. So, I get errors about the session reference not living long enough and suggesting to use a let bindings. Ok, done. More lifetime errors; they are pretty ambiguous now. They all look like so:
tests/system.rs:1446:49: 1446:56 error: `session` does not live long enough
tests/system.rs:1446     let got = find_definition(src, &path, pos, &session).unwrap();
                                                                     ^~~~~~~
tests/system.rs:1444:40: 1450:2 note: reference must be valid for the block suffix following statement 4 at 1444:39...
tests/system.rs:1444     let cache = core::FileCache::new();
tests/system.rs:1445     let session = core::Session::from_path(&cache, &path, &path);
tests/system.rs:1446     let got = find_definition(src, &path, pos, &session).unwrap();
tests/system.rs:1447     fs::remove_file(&path).unwrap();
tests/system.rs:1448     assert_eq!("a", got.matchstr);
tests/system.rs:1449     assert_eq!(9, got.point);
                     ...
tests/system.rs:1445:66: 1450:2 note: ...but borrowed value is only valid for the block suffix following statement 5 at 1445:65
tests/system.rs:1445     let session = core::Session::from_path(&cache, &path, &path);
tests/system.rs:1446     let got = find_definition(src, &path, pos, &session).unwrap();
tests/system.rs:1447     fs::remove_file(&path).unwrap();
tests/system.rs:1448     assert_eq!("a", got.matchstr);
tests/system.rs:1449     assert_eq!(9, got.point);

The lifetime issues are surprising in the first place since the find_definition doesn't return a reference to anything. Normally I would go through a little more trial and error to figure out the problem, but updating all of the tests for each change is cumbersome even with vim macros. I appreciate any time you can spare to look at this.
Thanks!

The issue is that the SessionRef type is defined as &'s Session<'s>, so the session and the cache would have to have the same lifetime.
A (possible) fix is here: https://github.com/birkenfeld/racer/tree/wip-reusable-cache
(I would probably further rename the 's lifetime to 'c and 'r to 's.)

Actually, once you untie these two lifetimes, there is no reason for SessionRef to exist anymore, and it can just be &Session in function arguments and &'s Session<'c> in struct definitions.

ugh! I looked right as SessionRef and didn't think anything of it :flushed:
Thanks for helping with this @birkenfeld; you saved me a lot of time banging my head against the wall :grin:.

I have a prototype of reusable allocations implemented within the FileCache. When a cache entry is updated, a ref for the previous value is pushed onto a free list. When the Session object is dropped, it calls increment_generation on the FileCache, and the references in the free list are moved into an available list. In that process, the previous value is dropped.
This approach has a soundness issue; since the increment_generation is a public method if file cache, it is possible for it to be called while the Session is active and has a reference handed out for something in the free list.
The next step here is where I struggle a bit. We need an arena implementation which supports the free operation and whose API doesn't suffer from the same issue as above. I came across this proposal from a couple of years ago which suggests such an API. It relies on a smart pointer type instead of handing out raw references. Wouldn't that have a big impact on how arenas are used? Is that really any better than dealing with Rc<T>? This may just be a lack of thorough understand of the Deref trait on my part.
What do you guys think?

Closing this since all of the discussions have been resolved. Will open new issues as necessary.
Thanks again for all the help.