Add case-sensitivity note to HTTP verbs

Addition of note regarding case-sensitivity of HTTP verbs for clarity.

Why:
Closes #19420
What's being changed (if available, include any code snippets, screenshots, or gifs):
Per RFC 7231, section 4.1, HTTP methods are case sensitive. Using incorrect case results in a 403 Forbidden error. This simply clarifies that these verbs are case sensitive to avoid potential confusion.
Check off the following:

[x] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@ryanrhanson Thanks for opening this PR for your issue! I edited your post slightly just to link your PR to your issue. I'll get this triaged for review! ⚡


In the README article, the "Access it here" link isn't working

In the README article, the "Access it here" link isn't working, but the link referenced in the project (https://thiagodnf.github.io/ical-to-csv/) does work.
Could probably correct that link.  very small problem, love the project, thank you for the open source converter.

Thank you @seantapscott . I have just fixed it.


how do i download and install it

i downloaded the latest version but is it for 1.20.1? and I downloaded the one that doesnt say -sources. but it crashes on startup. Is this forge or fabric? https://gist.github.com/stunlocked1/e2f82949f6938ef11e9e1ca604fa41b0

I marked my mod as 1.20.1 compatible by mistake (Modrinth autoselected the version and I forgot to change 1.20.x to >=1.20.3 inside the mod json.
I am planning to make my mods cross-version but as of right now the mod isn't available for 1.20.1.


Refactoring of the datasets

I'm not sure if there is a practical reason for almost all datasets to individually define the attributes root, transform, and target_transform as well as the __repr__ method. Since I think there is no reason I've introduced a superclass VisionDataset to streamline the process. This achieves two things:

Since almost all classes have a root and target_transform attribute and all of them have a transform attribute, I think it is a good idea to move them to VisionDataset. The only two exceptions are: FakeData without root and PhotoTour without target_transform. Both cases are handled by VisionDataset if None is passed for these arguments.
I've implemted the __repr__ similar to torch.nn.Module. Thus, __repr__ always returns the name of the dataset as well as the number of samples within it. If the attributes root, transform, and target_transform are not None they are added. Additionally, the subclasses can override the extra_repr method to include information specific for that dataset.

Next to the refactoring with these changes the datasets CocoCaptions, Flickr*, Omniglot, SBU, and VOC* now also have a human-readable representation.

2. I'm unsure if I'd want to add the `transform` and `target_transform` to the base class. The reason is that for some tasks, we might want to apply the same random transform to both input and target. This is currently not well supported in torchvision anyway, but I'm planning a few things that would make it better.

Thoughts about 2. ?

I'm not sure if I understand you correctly, but wouldn't the following work in such a case?
class CrazyDataset(VisionDataset):
    def __init__(self, root, transform=None):
        super(VisionDataset, self).__init__(root, transform, transform)
        ...

Although I've never worked with it, isn't this somewhat similar to the PhotoTour dataset? It only accepts a transform but applies this to both returned objects.

@pmeier

Although I've never worked with it, isn't this somewhat similar to the PhotoTour dataset? It only accepts a transform but applies this to both returned objects.

Yes, you could handle those cases with something like
class CrazyDataset(VisionDataset):
    def __init__(self, root, transform=None):
        super(CrazyDataset, self).__init__(root, None, None)

but then, this means that VisionDataset is not generic enough to handle everything.
For example, in https://github.com/pytorch/vision/issues/230#issue-252705712 I mention one alternative, StandardTransform, which looks like
class StandardTransform(object):
    def __init__(self, transform, target_transform):
        self.transform = transform
        self.target_transform = target_transform

   def __call__(self, input, target):
        if self.transform:
            input = self.transform(input)
        if self.target_transform:
            target = self.target_transform(target)
        return input, target

and which could replace the transform and target_transform by a simple transforms.
Because I'm not 100% sure of such a thing, I'd prefer to have the base class be for now very dummy and basic, and then maybe add support for more things afterwards once we are sure about it. Thoughts?

Thanks for the additional information as I wasn't aware of that discussion.
Just to be clear: you want StandardTransform only as the fallback during the transition period, correct? Later on the user should define his own transform, which jointly handles the input as well as the target, correct?

Just to be clear: you want StandardTransform only as the fallback during the transition period, correct? Later on the user should define his own transform, which jointly handles the input as well as the target, correct?

@pmeier I'm not 100% sure about this either :-) My original idea was to have it as a fallback initially, but I still need to think a bit more about it. This is why I'd rather have VisionDataset for now be just a dummy class that takes only root as an argument.

@fmassa Alright. I will move transform and target_transform back to the subclasses. How do you want to handle the printing? I can think of two options:

Move the repr creation of transform and target_transform into extra_repr within every dataset:

# cityscapes
def extra_repr(self):
    lines = (self._format_transform_repr(self.transform, "Transforms: ") +
             self._format_transform_repr(self.target_transform, "Target transforms: ") +
             ["Split: {split}", "Mode: {mode}", "Type: {target_type}"])
    return '\n'.join(lines).format(**self.__dict__)

Leave the repr creation within VisionDataset and set transform and target_transform to None within the constructor in case they are not defined within the subclasses:

class VisionDataset(data.Dataset):
    _repr_indent = 4

    def __init__(self, root):
        if isinstance(root, torch._six.string_classes):
            root = os.path.expanduser(root)
        self.root = root
        self.transform = None
        self.target_transform = None

    def __repr__(self):
        ...
        if self.transform is not None:
            body += self._format_transform_repr(self.transform,
                                                "Transforms: ")
        if self.target_transform is not None:
            body += self._format_transform_repr(self.target_transform,
                                                "Target transforms: ")
        ...

I would prefer option 2. since it will keep the subclasses uncluttered.

@pmeier I'd go for option 2 as well, or by some intermediate solution like
if hasattr(self, "transform") and self.transform is not None:
    ...

so that we can avoid adding the self.transform = None to the base class

@fmassa It's good to have someone with experience for guidance ;)

Codecov Report

Merging #749 into master will increase coverage by 1.99%.
The diff coverage is 47.89%.

@@            Coverage Diff             @@
##           master     #749      +/-   ##
==========================================
+ Coverage   39.64%   41.63%   +1.99%
==========================================
  Files          29       30       +1
  Lines        2742     2678      -64
  Branches      430      438       +8
==========================================
+ Hits         1087     1115      +28
+ Misses       1581     1486      -95
- Partials       74       77       +3

Impacted Files
Coverage Δ

torchvision/datasets/lsun.py
21.11% <33.33%> (+1.5%)
:arrow_up:

torchvision/datasets/voc.py
20.38% <33.33%> (ø)
:arrow_up:

torchvision/datasets/cityscapes.py
19.44% <37.5%> (+2.16%)
:arrow_up:

torchvision/datasets/fakedata.py
25% <40%> (+0.75%)
:arrow_up:

torchvision/datasets/phototour.py
24% <42.85%> (+1.57%)
:arrow_up:

torchvision/datasets/coco.py
25.53% <42.85%> (+3.11%)
:arrow_up:

torchvision/datasets/vision.py
43.33% <43.33%> (ø)

torchvision/datasets/flickr.py
24.05% <50%> (+0.59%)
:arrow_up:

torchvision/datasets/omniglot.py
29.62% <50%> (ø)
:arrow_up:

torchvision/datasets/stl10.py
29.85% <50%> (+3.18%)
:arrow_up:

... and 8 more

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 cd12b9a...bffb68e. Read the comment docs.

@fmassa Yep you are right, Sorry for causing more work than this should have been.

Hi @pmeier
Lint is failing with
./torchvision/datasets/mnist.py:119:16: E126 continuation line over-indented for hanging indent

Could you address that?
Thanks!

@fmassa Can I do these checks locally? I seems counter-intuitive to me to first commit the changes and than "hope" that the commit passes all checks.

@pmeier yes, absolutely. pip install flake8 and then flake8 .


A sprinkling of new projects in the land of the free

excited to see how the CI works

Thanks! CI looks good 😄


How to get raw bytes of a nested message?

I am new to protobuf. Protobuf is a fantastic package. But, I could not find a method to fit my scenario. Any guidance is appreciated.
Consider this message:
syntax = "proto3";

package somepackage;

message parent {
    mainMsg data = 1;
    bytes signature = 2;
}

message mainMsg {
   int id = 1;
   int somedata = 2;
}

What I want to achieve is mainMsg should contain actual data required by the application. parent is sent to the application which includes mainMsg along with calculated signature of the mainMsg in bytes format so that receiving application can verify integrity of the mainMsg.
On receiver side I want to calculate signature from raw bytes of  mainMsg. Is there any way to get raw bytes of nested message apart from unmarshaling the parent and marshaling only mainMsg struct again to obtain bytes? I referred to package documentation but could not find any function which directly provides this.

There is no API to get the serialized bytes of a field given serialized bytes of a message.  To get serialized bytes of a message field from given serialized bytes of a message, unmarshal the message and marshal the message field as you've already pointed out.
However, for what you want to achieve, beware that generating a signature based off serialized bytes of a proto Message is not advisable -- https://developers.google.com/protocol-buffers/docs/encoding#implications.

Thank you very much for pointing out possible implications. I will look into it to make sure things work out perfectly.


Deploying additional partitions using the R3 infrastructure

Deployment of additional partitions needs to be supported. The first step and the intermediate implementation is going to include a way to store additional partition properties in the common Key Vault using prefixes for the new partitions. ConfigMap changes are out of scope of the initial implementation and will be done as the next step. The partition that's used now in a single-partition setup will not be removed or changed to make sure that the current setup works as expected.

This effort is now a project and there need to be subtasks or issues divided and separated out.
https://github.com/Azure/osdu-infrastructure/projects/4


Pattern32 java

Sorry @kamboj652 this issue was already assigned to @MrigankaGhosh and so I have merged his pr, it's a best practice to discuss and get assigned before making pr


Better exercise(s) to go with File Naming and Organization

The forensic exercise with this lesson is designed to motivate the lesson in advance of the slides. Based on feedback, the lesson would be more effective if we removed the pre-presentation exercise and added a new exercise that better demonstrated & used the best practices described in the slides.

Brainstorming possibilities:

Provide a script that does some analysis or makes a plot. The script has blank variables (directory_name, slug, extension, etc). The exercise is to organize / rename files in a directory so that the script works.
Provide two directories with the same project but organized in different ways and have the students write pseudocode to do something. Discuss with partner which structure made the analysis easier, provide other suggestions for even better organization.

Notes from group disscussion

Write shell scripts? The script would be giving them the code. Do we want to forcus on machine readable.
The two pieces to the module: Human readable and machine readable.
Automation section could go into machine sorting. While the organization section would focus more on R project set-up.
Give them links to other example projects, either in the repro or in Github. Then they set-up a project. This entire section should take about an hour if not less.

Proposed Acitivity from Karen:

no exercise in the begining
orgnaization and file naming (core concepts)
exercise (Human readablility activity)
slide deck optional detailed (more minor details)

Heads up for @jennybc that we have a proposal for re-organizing this lesson, including a big shuffling of the two existing slide decks from

organization and
file naming

to

core concepts on organizating and naming
more details and examples on organization and naming

All sounds good to me! I will keep my ears open for specific things I could help on, if you need it. Thanks for working on this!


fix: return back div wrapper that shifts layout

The problem

The fix

We desperately needs visual regression testing

@soleksy-splunk can you please review and merge if is looks ok? so we can rebase visual testing PR?

works and looks good now, just need a rebase


[SDA-7984] Add region when creating manual s3 bucket for oidc config

Related issue: https://issues.redhat.com/browse/SDA-7984
What
Adds regions param when user specifies a different region then default
Why
Manual command failed if region param is not specified

/lgtm


issues with cloning via ssh

it doesn't start properly, i get this error

and yes it is built with luajit

now i get this ._.

for every plugin btw

now i get this ._.

this seems like you're having issues establishing an ssg connection with github.
could you try cloning evergarden manually like this to see if this is the issue:
git clone git@github.com:comfysage/evergarden.git ~/evergarden

yeah it is i can't clone via ssh only https ;-;

take a look at this gist (https://gist.github.com/comfysage/83fa0d1bde3d75192d1df956bf368849) to setup ssh on your system

i set it up :D


Update cats-effect, cats-effect-laws to 3.3.2 in series/0.x

Updates

org.typelevel:cats-effect
org.typelevel:cats-effect-laws

from 2.5.4 to 3.3.2.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Applied Scalafix Migrations

org.typelevel:{cats-effect,cats-effect-laws}:3.0.0

github:typelevel/cats-effect/v3_0_0?sha=v3.0.0

org.typelevel:{cats-effect,cats-effect-laws}:3.3.0 (created no change)

github:typelevel/cats-effect/v3_3_0?sha=series/3.x

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel" } ]

labels: library-update, early-semver-major, semver-spec-major, scalafix-migrations, commit-count:n:2

Superseded by #254.

Superseded by #254.


[HUDI-2129] StreamerUtil.medianInstantTime should return a valid date…

… time string
Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
(For example: This pull request adds quick-start document.)
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Codecov Report

Merging #3221 (d4d7133) into master (2033d35) will increase coverage by 2.22%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3221      +/-   ##
============================================
+ Coverage     47.52%   49.74%   +2.22%
+ Complexity     5440      406    -5034
============================================
  Files           923       67     -856
  Lines         41036     2985   -38051
  Branches       4106      320    -3786
============================================
- Hits          19501     1485   -18016
+ Misses        19811     1365   -18446
+ Partials       1724      135    -1589

Flag
Coverage Δ

hudicli
?

hudiclient
?

hudicommon
?

hudiflink
?

hudihadoopmr
?

hudisparkdatasource
?

hudisync
?

huditimelineservice
?

hudiutilities
49.74% <ø> (-8.31%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...ies/exception/HoodieSnapshotExporterException.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../apache/hudi/utilities/HoodieSnapshotExporter.java
5.17% <0.00%> (-83.63%)
:arrow_down:

...hudi/utilities/schema/JdbcbasedSchemaProvider.java
0.00% <0.00%> (-72.23%)
:arrow_down:

...org/apache/hudi/utilities/HDFSParquetImporter.java
0.00% <0.00%> (-71.82%)
:arrow_down:

...he/hudi/utilities/transform/AWSDmsTransformer.java
0.00% <0.00%> (-66.67%)
:arrow_down:

...in/java/org/apache/hudi/utilities/UtilHelpers.java
40.69% <0.00%> (-23.84%)
:arrow_down:

...rg/apache/hudi/utilities/HoodieSnapshotCopier.java
0.00% <0.00%> (-13.80%)
:arrow_down:

.../action/clean/JavaScheduleCleanActionExecutor.java

...rg/apache/hudi/table/action/commit/BucketType.java

...ache/hudi/common/table/timeline/HoodieInstant.java

... and 852 more

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 2033d35...d4d7133. Read the comment docs.


Migrate DocService to more scalable frontend framework

.. so we can add more features and organize them easily. Vue.js or Angular?

http://vuematerial.io/ and https://material.angular.io/

Or React? :) www.material-ui.com/
Just to add the option, I guess there's never a right or wrong answer on which framework to pick (which makes it hard to pick...).

Haha, yeah. I have some experience with Angular.js, so I was rather inclined to Angular than to React, but you know, Vue.js is also hot these days. :-D I think it's just a matter of who's gonna do the job and he or she prefers which.


firefox 49.0 vertion not working

===== HTML
<button class="btn-copy" data-clipboard-text="text to copy" id="value-url">        Share </button>
===== JS
`var clipboard = new Clipboard('.btn-copy');
clipboard.on('success', function(e) {
console.log('Copied Success' + e);
$(e.trigger).html(' Share').addClass('share-cb');
});
clipboard.on('error', function(e) {
console.log('Copied Failed' + e);
});`
====
Here is console detail:
document.execCommand(‘cut’/‘copy’) was denied because it was not called from inside a short running user-generated event handler.

Just tested clipboardjs.com and all demos on Firefox 49 and everything is working fine. Please share a working example on JSFiddle or something like that so we can better debug it.

I'm seeing this with Firefox 49.0.2 Mac on clipboardjs.com (plus my own code, v 1.5.15).
Screenshot:

Everything seems to be working fine over here.

Huh, also works fine on another machine, same FF version, same OS. I reset FF completely and it does work on my machine too. That's puzzling because I had no extensions installed. Thanks for verifying!


Past events not rolling off THP: Michigan

Presumably connected to the issue that all these events have "Date Not Valid" on them?


[W5.7][W12-1]Lim Yu Xi

Added test to check method Utils.isAnyNull()

add sort feature to order name by alphabetical order


Request - Disable teleport to spawn if player is in creative mode over a void.

Currently players are teleported to their spawn point if there is no valid space within the knockback limit.
I would like players in creative mode to just be knocked back horizontally.
Well, I'd like to be able to turn it off per world or per player (i.e. permissions), but that would just be a bonus.

Hmm, fair enough; if a player is flying or possibly just in Creative mode, a solid block to stand on shouldn't be necessary. I'll add that to my "to-do" list.

Cool


Fix executing TappedCommand while scrolling

Closes https://github.com/enisn/UraniumUI/issues/353

Please fix it for IOS too


Refactor all classes by the name of Lightning to Simplix

Applied some of the feedback from this message in the discord server - https://discord.com/channels/752533664696369204/756216074042736701/924600893658251264

Thanks, merged!


[tw] Localize Monolithic Apps for Traditional Chinese

Localize Monolithic Apps for Traditional Chinese
https://glossary.cncf.io/monolithic-apps/

Get Merged! thank @johnlinp


Refactor of PR #11405

This is a port of PR #11405 code into the libnetwork codebase.
Hopefully this will resolve the following bug:
https://bugzilla.redhat.com/show_bug.cgi?id=1142357

@alecbenson Could you rebase against the latest fedora-1.7

Sure, I will do this as soon as I get home.
On Jun 17, 2015 5:22 PM, Daniel J Walsh<EMAIL_ADDRESS>wrote:
@alecbensonhttps://github.com/alecbenson Could you rebase against the latest fedora-1.7
�
Reply to this email directly or view it on GitHubhttps://github.com/rhatdan/docker/pull/79#issuecomment-112953221.

Rebased, this should merge cleanly now.

Good lets send this upstream.

Merged into rhel7 and fedora-1.7


mark as deprecated?

Probably should. What's the best way to do that? Big fat letters in the README?

Check this out: https://github.com/ConsenSys/eth-testrpc

nice - I added it to the repo description as well


Backup original boot files

While right now, SteamOS has an automatic file replacement function for the video and the css files, it's possible this will change in the future.
The original boot videos should be stored for restoration to original state when desired.

feature/backup_orig created for this.
Backup works presently, just deciding on user experience for the restore.
The difficulty is it is unknown what state the user's files are in at the present time. Are they original? Are they modified? I've made the decision to back up files using the md5 hash as part of their filename to eliminate duplicates while retaining idempotent behavior. Backup runs without the user requesting it do so.
However, how does the user know which one is "correct" to restore? I could (and do now) save the md5 hashes statically in the repository but that will quickly fall out of date and always lag behind SteamOS client releases. I believe I'll make it so the script makes a recommendation but still prompts the user on which is the right one to restore.
This structure also means duplicating all video files in /vids/ if a user uses randomizer.

With the new overrides dir, we no longer need to back up the original video. That should help keep the backup size small.
I also expect fewer versions of library.css and library.js to exist so it should be easier for a user to pick from a list.
Going to add the restore functionality as a pick list and call it good.


Question: Will this work for Android?

Hi!
Nice library. Will this work for Android?
Best regards
Daniel

I don't know. It's built for JDK 8. Did Android ever get CompletableFuture?

I don't know. It's built for JDK 8. Did Android ever get CompletableFuture?

Hi. I haven't tested it. Perhaps I will give it a try.


Cannot specify charset on 'Content-Type' header

To Reproduce
Querying a graphql endpoints that returns accents such as "Belvédère". Instead we get a string with "BelvÃ©dÃ¨re"
Expected behavior
Find the string "belvédère" in the jsonResponse Map after decoding (line 76 client.dart).
final Map<String, dynamic> jsonResponse = json.decode(response.body);

The header parameters are hardcoded and the 'Content-Type' is set to application/json.
In order to support the accents we should be able to specify the charset such as:
'Content-Type: application/json; charset=utf-8'
According to dart documentation:
/// The body of the response as a string. This is converted from [bodyBytes]
  /// using the `charset` parameter of the `Content-Type` header field, if
  /// available. If it's unavailable or if the encoding name is unknown,
  /// [latin1] is used by default, as per [RFC 2616][].
  ///
  /// [RFC 2616]: http://www.w3.org/Protocols/rfc2616/rfc2616-sec3.html
  String get body => _encodingForHeaders(headers).decode(bodyBytes);

@AlexandreRoba Thanks for reporting the issue. Will see if I can fix it today.

@AlexandreRoba

The header parameters are hardcoded and the 'Content-Type' is set to application/json.

Which version are you using?
In 1.0.0 we allow you to extend the headers your self:
HttpLink link = HttpLink(
  uri: 'https://api.github.com/graphql',
  headers: <String, String>{
    'Authorization': 'Bearer <YOUR_PERSONAL_ACCESS_TOKEN>',
    'Content-Type': 'application/json; charset=utf-8',
   },
);

Also reading trough the docs for the http package we use in the HttpLink. There they say the following about the body method on the the Response class.:

The body of the response as a string. This is converted from bodyBytes using the charset parameter of the Content-Type header field, if available. If it's unavailable or if the encoding name is unknown, latin1 is used by default, as per RFC 2616. [...]

So the Response.body should already be converted to the correct charset before we parse the the JSON to a map.

Hi @HofmannZ thanks for the quick response.
I'm currently using version 0.9.3.
When is the 1.0.0 expected to be releases?
Your are absolutely right about the decoding of the body. If the charset is specified it will be used to decode the body according to the documentation. This is why I pasted the last bit of code in my previous post :)
This seems to be possible with HttpLink. I will have a look at it once it is released.
Thanks for your help.

@AlexandreRoba You can already try 1.0.0 out out today. See the upgrade guide to for breaking changes.

Fixed with 1.0.0.


remove stream from stalled_by_transport list before destroy it

Can one of the admins verify this patch?

Can one of the admins verify this patch?


Update Remove-TeamUser.md

Needed to specify user role to remove.  Docs also says that -Role is only used to demote user, but -role removes the user from the team, as pr cmd description at the top.

Docs Build status updates of commit 423492a:
:clock10: Full build: the average full build time is 13 min(s) 58 sec(s), this is based on the last 413 build(s) for this repository.

Docs Build status updates of commit 423492a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/Remove-TeamUser.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@AlexanderHolmeset - thanks for taking the time to make the docs better and submit fixes. It is much appreciated!
@islubin - can you let me know if ok to merge?

Docs Build status updates of commit 8808b6d:
:clock10: Full build: the average full build time is 13 min(s) 57 sec(s), this is based on the last 419 build(s) for this repository.

Docs Build status updates of commit 8808b6d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/Remove-TeamUser.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@dariomws306  , @get-itips and @tseward Please review this PR and give your feedback. @kenwith looks forward to your comments.

Hello @AlexanderHolmeset how are you?
I tested this in my lab, and I could verify that if you ran
Remove-TeamUser -GroupId 31f1ff6c-d48c-4f8a-b2e1-abca7fd399df -User<EMAIL_ADDRESS>-Role Owner
it actually removes the ability for the user, to access the team, but, strange enough, if you ran
Get-TeamUser  -GroupId 31f1ff6c-d48c-4f8a-b2e1-abca7fd399df
you will see it still listed in the Powershell result, but as a member (as the current article says)

(You won't see it listed in the Teams UI though)
Maybe is some caché thing or the module is buggy, either way, it seems it is better to describe the current behavior of the cmdlet, as you suggest in this pr, so I'll accept it.

We should see if the PG considers this a bug prior to changing the article. As the module itself has [-Role] as optional, the PR would change the article and the product would no longer match the documentation.

@tseward you are right on that. thanks

There is a similar behaviour with Add-Teamuser (and different from Admin Center):
https://github.com/MicrosoftDocs/office-docs-powershell/issues/1665
I have done some testing and I found that the -Role paramater didn't exist in older version of Teams Powershell.
I am agree with Trevor, also I think it makes sense that this parameter is optional.

Here is a screenshot to better describe the error at my end.
The difrence between my and @get-itips error, is that with me the user is removed and not demoted.
So probably some kind of bug in the module.
Makes sense as you say to make the parameter -Role optional.
Maybe there should be a new cmdlet like this to demote/promote users:
Set-TeamUser -GroupID "XXXXXXX" -User<EMAIL_ADDRESS>-Role Owner

Here is a screenshot to better describe the error at my end.
The difrence between my and @get-itips error, is that with me the user is removed and not demoted.
So probably some kind of bug in the module.
Makes sense as you say to make the parameter -Role optional.
Maybe there should be a new cmdlet like this to demote/promote users:
Set-TeamUser -GroupID "XXXXXXX" -User<EMAIL_ADDRESS>-Role Owner

This only applies to the owner role..
When you add/remove the user as a member this error is not displayed.

Good catch Erwin.

@islubin can you comment on this one? Is this a bug and should be change the documentation?

Docs Build status updates of commit 475ef79:
:clock10: Full build: the average full build time is 13 min(s) 52 sec(s), this is based on the last 453 build(s) for this repository.

Docs Build status updates of commit 475ef79:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/Remove-TeamUser.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 30ee8e5:
:clock10: Full build: the average full build time is 13 min(s) 14 sec(s), this is based on the last 529 build(s) for this repository.

Docs Build status updates of commit 30ee8e5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/Remove-TeamUser.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Add Jamie Stark ( @nomorephones ) who took over Teams PowerShell from @islubin to review and sign off.

Docs Build status updates of commit 56ca189:
:clock10: Full build: the average full build time is 12 min(s) 13 sec(s), this is based on the last 579 build(s) for this repository.

Docs Build status updates of commit 56ca189:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/Remove-TeamUser.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Any updates on this? The PowerShell to remove users still does not work.

@nomorephones any update?

Tested this again with module version 1.0.1
Removed user that was owner on a group using the -Owner parameter, and it was demoted to member. This was reflected right away both in UI and Powershell Get-TeamUser cmdlet.
Removed user that was owner on a group NOT using the -Owner parameter, and it was removed from the channel/group. This was NOT reflected right away in Powershell Get-TeamUser cmdlet, and I am still waiting to see this reflected.

Docs Build status updates of commit 0fb3f8d:
:clock10: Full build: the average full build time is 11 min(s) 58 sec(s), this is based on the last 744 build(s) for this repository.

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

File
Status
Preview URL
Details

teams/teams-ps/teams/Remove-TeamUser.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Also tested with 1.0.1, my comments:

Same as Andres using -Role parameter (if I use user UPN or UserId)
Without -Role parameter:

If I use UserId same as Andres, user is not removed from the Team, not demoted to member and
no output
If I use UPN same as Erwin, I get the same error

Cmdlet documentation says that we should use UPN.

@tseward what do you think of our conclusions ?

@get-itips we're still on hold for this one.

@AlexanderHolmeset Can you test again with module version 1.0.1 ? Thanks

@get-itips have the same result as you now

Hello @nomorephones
Issue creator now experiences the same as we all do.
Based on that, and the current behavior of the cmdlet, do you think is necessary to update the documentation?
We really appreciate your time to review this and give us your thoughts.
Thank you

Docs Build status updates of commit 8dbc93a:
:clock10: Full build: the average full build time is 11 min(s) 56 sec(s), this is based on the last 786 build(s) for this repository.

Docs Build status updates of commit 8dbc93a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/Remove-TeamUser.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 41215d8:
:clock10: Full build: the average full build time is 11 min(s) 53 sec(s), this is based on the last 871 build(s) for this repository.

Merging this based on feedback from the product team and they have also added an internal work item to address the inconsistencies brought up above.

Docs Build status updates of commit 41215d8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/Remove-TeamUser.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

This is still an issue, I think. Using v1.0.21
I have a Team with 1 existing owner. I add a new Owner:
Add-TeamUser -GroupId 557acb93-6e5d-4031-8def-xxxxxxxxx -User<EMAIL_ADDRESS>-Role Owner
and I can see Sally as an owner of the group. All good.
Then, if I attempt to demote her to Member as per docs:
Remove-TeamUser -GroupId 557acb93-6e5d-4031-8def-xxxxxxxxx -User<EMAIL_ADDRESS>-Role Owner
Sally is removed from the Team altogether.
:-(


Add a separate README.md for runtime

Because it deserves it.

@alanjds I am not sure the description is right. About system level in particular. It might be that runtime provides shim for Python abstractions over Go, which handles all interfacing with system level. There is also lib dir that needs to be explained. What is the difference in runtime/ and lib/ shims?

runtime/ is Go code. lib/ is Python code.
Both composes the Grumpy-made stdlib.
When an import is issued, stuff on the 3rd-party folder is tried later if not found from this 2 folders you cited.

Thanks @abitrolly. Sorry for the long delay in merging it.


Implement codeLens or coc-references-used

When using references, I get results in Telescope that aren't very useful, such as definitions and imports. It would be useful to be able to use <plug>(coc-codelens-action) or <plug>(coc-references-used) to show more meaningful references (actually used).

As far I know, https://github.com/fannheyward/telescope-coc.nvim/issues/16 implement <plug>(coc-references-used)


"Incorrect use of <<" warning is shown in checksum code.

The code below causes warnings for almost all operators in the code. I believe the warnings are false positives.
function adler32(input) {
    var adlerModulo = 65521, index, a = 1, b = 0;
    for (index = 0; index < input.length; index++) {
        a = (a + input.charCodeAt(index)) % adlerModulo;
        b = (b + a) % adlerModulo;
    }
    return (b << 16) | (a & 0xFFFF);
};

/*jshint strict: false, bitwise: false */

function adler32(input) {
    var adlerModulo = 65521, index, a = 1, b = 0;
    for (index = 0; index < input.length; index++) {
        a = (a + input.charCodeAt(index)) % adlerModulo;
        b = (b + a) % adlerModulo;
    }
    return (b << 16) | (a & 0xFFFF);
}

Works for me.
I removed the semicolon after the function block since that's not necessary.

I still get a Warning	1	JS Hint: Unexpected use of '++'. with those options set.  The whole code is in a function scope. The extra semicolon comes from the original code which was in namespace.adler32 = function (input) {  format.

Ah ok. You can remove the ++ warning with the plusplus option. I believe it's plusplus: false

Hmm changing index++ to index += 1 seems to have fixed it. Thanks!

As a side note, "bitwise:false" alone fixes the issue.

Yeah, or you can do that.


Add option to extend maintainFocus to allow focus outside of a11ydialog markup

Had the same problem as in #66
I'm using popper.js for dropdowns and tooltips. In my case, dialogs are scrollable and sometimes it's better the dropdowns overflow the dialog element.
Thought might be useful for others.

Hello.
I’m just coming back to this pull-request. While this is nice addition, I don’t think it’s worth integrating in the library which aims at remaining small and focused on the main cases.
Thank you for your work nevertheless, and sorry for not merging it. ✨

😿
i'll try extending the _maintainFocus method to suit my needs

@jurajk Don’t take it personally of course: the PR was very good and clean. I just think a fork is better for this feature. :)

No worries :)


add provide dashboard

In this PR I add the provide dashboard to the Interactive apps, including an description how to use it and example questions which could be addressed with the dashboard.

After the review of @lilianschuster, I think we can merge and deploy the first version of our provide dashboard site!


zset zrangebyscore

Bug Report
In some environments zrangebyscore(zSetKey, range).get(WAIT_TIME, TimeUnit.SECONDS) is not responding
Current Behavior
In the development environment and test environment, the program is normal. But there is no response in the production environment
code(java)
        StatefulRedisClusterConnection<String, String> connection = null;
        try {
            connection = lettucePool.acquire().get(WAIT_TIME, TimeUnit.SECONDS);
            RedisAdvancedClusterAsyncCommands<String, String> command = connection.async();

            // no response
            List<String> expireKeyList = command.zrangebyscore(zSetKey, range).get(WAIT_TIME, TimeUnit.SECONDS);

            return doDeleteKey(command, zSetKey, expireKeyList);
        } catch (Exception e) {
            log.error(e.getMessage(), e);
            return false;
        } finally {
            if (connection != null) {
                IndicatorUtil.releaseConnection(lettucePool, connection);
            }
        }

What do you mean with no response? Care to add more context what happens (timeout, …) here?
Please also enable debug logging for the io.lettuce.core.protocol logger to see what's happening on the transport.

No response means the program has been stuck here.

This can be caused by various things:

Connection to Redis is disconnected (connection.isOpen() returns false)
The connection is blocked by a previous command
The connection is in a transactional state (connection.isMulti() returns true)

This can be caused by various things:

Connection to Redis is disconnected (connection.isOpen() returns false)
The connection is blocked by a previous command
The connection is in a transactional state (connection.isMulti() returns true)

Thank you very much for your reply.
I don’t think it’s the case, In order to solve the production problem immediately, I have replaced the implementation of jedisClusterClient implementation, and the problem is solved.

Thanks for your feedback. Closing this one as solved.

我平时这样使用StatefulRedisClusterConnection demo

I usually use the StatefulRedisClusterConnection demo like this.
 RedisClusterClient redisClusterClient;

    public void test() {
        StatefulRedisClusterConnection<String, String> connection = LettuceUtil.getConnection(redisClusterClient); //get a connection
        RedisAdvancedClusterAsyncCommands<String, String> command = connection.async();
        command.setAutoFlushCommands(false);
        List<RedisFuture> futureList = new ArrayList<>();
        futureList.add(command.get("aaa"));
        futureList.add(command.get("bbb"));
        futureList.add(command.get("cc"));
        futureList.add(command.get("ddd"));
        // ....Some command .....
        command.flushCommands();
        LettuceUtil.execute(futureList.toArray(new RedisFuture[futureList.size()]));//call LettuceFutures.awaitAll

        // other command
        List<RedisFuture> futureList1 = new ArrayList<>();
        futureList1.add(command.get("aaa"));
        futureList1.add(command.get("bbb"));
        futureList1.add(command.get("cc"));
        futureList1.add(command.get("ddd"));
        // ....Some other command .....
        command.flushCommands();
        LettuceUtil.execute(futureList1.toArray(new RedisFuture[futureList1.size()]));  //call LettuceFutures.awaitAll

        LettuceUtil.releaseConnection(connection);//call connection.close

    }

Is this used correctly?
@mp911de

@yinbingqiu Please do not resurrect closed issues for other purposes. Please post your question to Stackoverflow.


simple script to modify and clean the SVG background colors

Fixes https://github.com/resolvetosavelives/healthicons/issues/156


uv self update does not work on windows

platform：windows 10
uv version：0.1.23
I have a standalone installation scripts, so there shouldn't be this warning, right?
If there is no latest version, should the latest version be prompted?

Can confirm, the self updater doesn't work on windows:
> powershell -c "irm https://astral.sh/uv/install.ps1 | iex"
Downloading uv 0.1.23 (x86_64-pc-windows-msvc)
Installing to C:\Users\Konstantin\.cargo\bin
  uv.exe
Everything's installed!
>  uv self update
warning: Self-update is only available for `uv` binaries installed via the standalone installation scripts.

If you installed `uv` with `pip`, `brew`, or another package manager, update `uv` with `pip install --upgrade`, `brew upgrade`, or similar.
> where.exe uv
C:\Users\Konstantin\.cargo\bin\uv.exe


[POC] Markdown editor / viewer

Based on the previous research work done by @Nasdan on markdown, create a set of POC''s (in a different public repository) that will cover all the scenarios and challenges we have to face in LeanMood.
Based on @Nasdan research:

mark-down-it is the library to choose (allow adding custom tokens).

mark-down-react library support plugins:

https://github.com/alexkuz/markdown-react-js
All this work will be documented in a wiki as well.

Duplicated #37


New docker image and dockerfile

New docker image configurable with environments and NGINX as frontend

Nella descrizione di questa pull request è necessario elencare le modifiche e le funzionalità da questa introdotte

Hey guys, I'm writing in english because we have some collegues of the R&S world that's interested in this project, so let's speak english :-)
I have enabled the GH container deployment for each new release.
I've seen that in the current docker image we stuck the metadata in the image, I would like instead to have the metadata in a docker volume (see docker-compose.yml)
can we have this improvement in this PR?


Node Selector Feature

Is it possible to define a certain worker node for each lab node in the topology file, meaning one can choose which worker node should the lab node be deployed on, instead of relying on the kubernetes default scheduler?

This is not something we have explored. Although I could see that being very useful for cases when you have a topo with some nodes requiring nested virt support and only some worker nodes that supported nested virt for example.
This may not be too difficult to add using the existing labels field: https://github.com/openconfig/kne/blob/b5d78250ff387e7d472a830b0818272d5e338b96/proto/topo.proto#L70 which get applied to the pod: https://github.com/openconfig/kne/blob/b5d78250ff387e7d472a830b0818272d5e338b96/topo/node/node.go#L453
Contributions are welcome.


python3Packages.identify: 2.4.11 -> 2.4.12

Description of changes
Update to latest upstream release 2.4.12

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 164547 run on x86_64-linux 1

3 packages built:

pre-commit
python310Packages.identify
python39Packages.identify


Warnings when rendering on a server not running x server

🐛 Bug Report

(This can wait until after xmas! 🎅)
Data renders incorrectly on a jupyter notebook when run from a headless server.
Warnings are displayed that PyVista may segfault, and data appears to render incorrectly in threejs viewer.
This system does not appear to be running an xserver.
PyVista will likely segfault when rendering.

Try starting a virtual frame buffer with xvfb, or using
  ``pyvista.start_xvfb()``

  warnings.warn('\n'

I cannot use the provided suggested code as I do not have sudo rights to install additional packages
Input:
pyvista.start_xvfb()

Output:
---------------------------------------------------------------------------
OSError                                   Traceback (most recent call last)
/var/tmp/ipykernel_96063/1929267992.py in <module>
----> 1 pyvista.start_xvfb()

~/.conda/envs/geovista-dev/lib/python3.10/site-packages/pyvista/utilities/xvfb.py in start_xvfb(wait, window_size)
     45
     46     if os.system('which Xvfb > /dev/null'):
---> 47         raise OSError(XVFB_INSTALL_NOTES)
     48
     49     # use current default window size

OSError: Please install Xvfb with:

Debian
$ sudo apt install libgl1-mesa-glx xvfb

CentOS / RHL
$ sudo yum install libgl1-mesa-glx xvfb

How To Reproduce
Steps to reproduce the behaviour:

Run ipython notebook server on a headless linux machine
Run the attached notebook Geovista-Render.zip.

The screenshots below are what is shown in the three.js panel in the browser window in the notebook.
Screenshots

https://user-images.githubusercontent.com/22805/147116429-c1a96b04-e254-4801-acb4-c53057a7774a.mp4

I'm not sure if the rendering inconsistency is my fault? I am now running in a x window terminal which stops the warning from being shown in the notebook. However the strange rendering is still occurring.
I am running the example from examples/example_from_2d__orca.py pretty much as-is. The only addition is:
import pyvista
pyvista.global_theme.jupyter_backend = 'pythreejs'

to support output in the web browser.

OK, based on sprint demo 24 I found with this change it renders correctly. not sure why or what the numbers in the Sphere mean though!
#plotter.add_base_layer(color="grey")
base_layer = pv.Sphere(radius=0.99, theta_resolution=360, phi_resolution=180)
plotter.add_mesh(base_layer)

So I think the incorrect rendering is not related to the xserver warning, there are two different things going on here.
But the xserver warning is still a thing. If I don't set
import pyvista
pyvista.global_theme.jupyter_backend = 'pythreejs'

then the process segfaults.

@jamesp This may be a pyvista issue rather then a geovista one

This may also be due to the base layer being too close to the polydata surface to the specific render that you've chosen.

@jamesp I've pushed a tweak (https://github.com/bjlittle/geovista/commit/5ecd6afb7999e1bbdbed0dcf62a2b5b7021ed87a) that should resolve the rendering issue that you're seeing with  pythreejs.
When rendering in the browser you should always configure the rendering backend with either the jupyter_backend kwarg to the plotter show method, or a one-time configuration with the pyvista.set_jupyter_backend("pythreejs") function.
Does this resolve your issue?

If it's a common issue that users might hit when plotting in a browser and they're using a geovista geoplotter, then we could easily ensure that the pyvista backend is set to a sensible default, if it hasn't already been done by overriding the show method

Also, note that the pyvista.global_theme.jupyter_backend defaults to ipyvtklink.
If you're running on a headless server, then you need to ensure that ipyvtklink is pre-installed e.g., conda install -c conda-forge ipyvtklink for it to work... this might be the original source of your issues, see the pyvista docs,

You've only circumvented this by explicitly using pythreejs, which I'm guessing is installed implicitly as part of the geovista requirements?, whereas ipyvtklink (which is more stable and feature complete over pythreejs btw) isn't

Note that, there is also the geovista convenience function goevista.common.set_jupyter_backend which will set the rendering backend to by default to pythreejs (or whatever backend you provide) given that there is an active IPython kernel.


[C++][Parquet] SIMD-accelerated dictionary coding

Reporter: Daniel Lemire
Related issues:

[C++] Hardware optimizations for dictionary / RLE encoding/decoding (is duplicated by)

Note: This issue was originally created as PARQUET-688. Please see the migration documentation for further details.

Daniel Lemire:
Let us close this one then.

Deepak Majeti / @majetideepak:
[~lemire], @wesm opened a JIRA for this recently. https://issues.apache.org/jira/browse/PARQUET-684

Daniel Lemire:
Parquet could benefit from SIMD accelerated processing on recent x64 hardware.
Further reading :
https://github.com/lemire/dictionary
http://lemire.me/blog/2016/08/25/faster-dictionary-decoding-with-simd-instructions/


[SPARK-26324][DOCS] Add Spark docs for Running in Mesos with SSL

What changes were proposed in this pull request?
Added docs for running spark jobs with Mesos on SSL

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

@srowen @felixcheung  pr updated

Test build #4482 has started for PR 23342 at commit fbc2e70.

@srowen Why did you not merge it ?

It is merged to master. I missed running the tests so those are running now, but it's a doc change and shouldn't be a problem

I saw it. I was just confused about the git symbol. Thx

Test build #4482 has finished for PR 23342 at commit fbc2e70.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

That error can be ignored; this didn't cause functional changes in Python


Make Linter load faster since all clients need to load this class.

Defers the loading of lodash and XRegExp until they are used.
Replaced _.memoize() with custom logic to pave the way to removing
the lodash dependency altogether in Linter.
The only other top-level require() calls that could be eliminated
are for log and warn.
One thing that I found a little odd is that, assuming @executablePath
never changes over the lifetime of the Linter once it is set,
@_statCache will only ever have one entry in it.

One thing that I found a little odd is that, assuming @executablePath
never changes over the lifetime of the Linter once it is set,
@_statCache will only ever have one entry in it.

Yeah, I think that's what this TODO is referring to. I don't actually know if @executablePath is invariant though.


Make rustup update when specific version not present.

This PR causes us to update to the most recent release of rustup, as a result.

This change is

r? @metajack
cc @SimonSapin

This current solution will cause new machines to fail to bootstrap if rustup publishes a newer version than 1.7.0.
I would want to have versioned-locked download URLs for reproducibility.
The long term goal here is versioned uploads of rustup itself, published by the rustup.rs repo (either S3 or GH releases).
In the meantime, we can manually mirror selected versions to a servo-controlled S3 bucket (we have a couple of them).
I would prefer to download rustup itself directly if it is a static binary on all platforms (similar to sccache), but if that is not possible it would also be acceptable to have versioned URLs for the rustup installer.

I don’t think we should be in the business of hosting rustup binaries. We can parse the output of rustup --version to extract a tuple of integers, and check for a minimum version instead of an exact one.

Something like…
-    - unless: rustup --version | grep '{{ rustup.version }}'
+    - unless: rustup --version |  python -c 'import sys, re; [int(i) for i in re.match("rustup (\d+).(\d+).(\d+) ", sys.stdin.read()).groups()] >= [1, 7, 0] or sys.exit(1)'

:umbrella: The latest upstream changes (presumably #767) made this pull request unmergeable. Please resolve the merge conflicts.


how do I uninstall?

Please follow the guide below

Issues submitted without this template format will be ignored.
You will be asked some questions and requested to provide some information, please read them carefully and answer completely.
Put an x into all the boxes [ ] relevant to your issue (like so [x]).
Use the Preview tab to see how your issue will actually look like.

Before submitting an issue, make sure you have:

[x] Updated to the latest version
[x] Read the README and docs
[x] Searched the bugtracker for similar issues including closed ones

Purpose of your issue?

[ ] Bug report (encountered problems/errors)
[ ] Feature request (request for a new functionality)
[x] Question
[ ] Other

The following sections requests more details for particular types of issues, you can remove any section (the contents between the triple ---) not applicable to your issue.

For a bug report, you must include the Python version used, code that will reproduce the error, and the error log/traceback.
Paste the output of python -V here:
Code:
# Example code that will produce the error reported
from instabot import Bot

bot = Bot()
bot.login(username="user", password="pass")
followers = bot.get_user_followers(bot.user_id)

Error/Debug Log:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ZeroDivisionError: integer division or modulo by zero

Describe your issue
Explanation of your issue goes here. Please make sure the description is worded well enough to be understood with as much context and examples as possible.

I have no need for it anymore, how do I either uninstall or stop it?

This is not an issue of the maintainer. You should uninstall using pip uninstall instabot command.

And close the issue please.


(IAC-333) Adding in hsts-max-age for security compliance.

Not quite sure why main and staging are so off such that the merge has conflicts. Will work through these. Will be testing on Wed.

The diff between main and staging is from https://github.com/sassoftware/viya4-deployment/pull/202/files

Validated that the config map of the ingress controller now sets the value to approximately 2 years.
data:
  allow-snippet-annotations: 'true'
  hsts-max-age: '63072000'
  use-forwarded-headers: 'true'

Testing now shows with TLS active that the hsts-max-age is now part of the header response:

Looking to have this one closed. Not sure what/how the merge branch was changed from staging to main. Guessing since the last release the staging branch was deleted. This might cause issues in the future for work that has been targeted for merge into staging but did not make the cut for that release.


re-add deploy job

re-adds the deploy job. Run when tests pass.

yes the change in .github/actions/compose/action.yml is unrelated.

Going to test this proper by merging to main here. It worked on my fork™
But i'm off for next 3 days and I think it would be nice to have this working again...


vscodium: 1.59.1 -> 1.60.0

Motivation for this change
Upgrades vscodium from 1.59.1 to 1.60.0
This PR was automatically generated by nixpkgs-upkeep.

CI workflow that created this PR.
Internal tag: 59ba93b5f1ba31e392f0a3894b2e266d.

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 137328 run on x86_64-darwin 1

1 package built:

vscodium

Thank you!


nanopolish installation, fatal error: too many errors emited

Hi,
I'm trying to install nanopolish to ultimately learn how to use minimap2 (weird route, I know but i'm having trouble starting minimap2 and no one around me knows anything about data analysis, so i'm on my own following every tutorial that I can find), following this:

git clone --recursive https://github.com/jts/nanopolish.git
cd nanopolish
make

but I keep getting this error:
fatal error: too many errors emitted, stopping now [-ferror-limit=]
20 errors generated.
make[2]: *** [cache.o] Error 1
make[1]: *** [all-recursive] Error 1
make: *** [lib/libhdf5.a] Error 2
How can I fix that?
attached are part of the terminal codes

Hi,
Nanopolish is complicated to install on MacOS so I suggest you use bioconda to install it. In any case, minimap2 is much easier to compile than nanopolish so I think focusing there is your best bet.
Jared

Thanks for the advice! I'll keep working on minimap2.

From: Jared Simpson @.***>
Sent: Saturday, April 3, 2021 07:33
To: jts/nanopolish
Cc: Ruiz Manzano, Ana; Author
Subject: Re: [jts/nanopolish] nanopolish installation, fatal error: too many errors emited (#894)

External Email - Caution *

Hi,
Nanopolish is complicated to install on MacOS so I suggest you use bioconda to install it. In any case, minimap2 is much easier to compile than nanopolish so I think focusing there is your best bet.
Jared
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/jts/nanopolish/issues/894#issuecomment-812859434, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARPWHH2BUNKBT4G72EIFR2TTG4DK7ANCNFSM42JWQVMQ.


Updated dependencies to remove security vulnerabilities

Great. Thank you!

I saw the Travis build failed. Do you know why? Are you still going to merge it?


Make removeLabels applies to deployments..spec.template.metadata.labels

Hi,
Currently the removeLabels only applies to the manifest.metadata.
helm.transformer = map (
  lib.kube.removeLabels [
    "app.kubernetes.io/version"
    "helm.sh/chart"
    "app.kubernetes.io/managed-by"
  ]
);

It will be nice it also removes  applications.<name>.resources.deployments.<name>.spec.template.metadata.labels.

Sounds like a good idea. However, we will need to collect all such cases in different resource types as I'd want the function to work the same accross the board.
Quickly I can think of:

Deployment
ReplicaSet
DaemonSet
Job
StatefulSet

But CronJob breaks the pattern and has .spec.jobTemplate.metadata.labels and .spec.jobTemplate.spec.template.metadata.labels.

I updated the function to do this. Please let me know if something is missing.

I updated the function to do this. Please let me know if something is missing.

Thanks for adding this. It works as expected.


toast in wrong place

@exezbcz  i think we can get rid of this toast completely . wdyt?

yeah, I will make issue for modals on mobile tomorrow or later today. It should also solve the overflowing etc.
we can get rid of it on mobile devices yes


API instalation bug

Describe the bug
CI rrun
ERROR  - ModuleNotFoundError: No module named 'httpx'
Expected behavior
No errors
Additional context
Maybe add httpx to requirements?

bug reported to `starlette' - https://github.com/encode/starlette/issues/1945

Ive added httpx to requirements


Sticky job state gets overridden by template state

When the template is active and the job inactive, the job gets re-activated. This PR will fix this.

I squashed and merged your commits into 566691916eb775120cadb6d6c0aea88940a58ff8. Jenkins-autojobs 0.14.3 is now on pypi with your fix. Thanks.


csproj parsing error

on any csproj file modification by extension the xml became invalid if contain &amp.
from
<Content Include="help\Content\Boards & Widgets.htm" />
to
<Content Include="help\Content\Boards & Widgets.htm" />
and error
error: Can not create file: Error: Invalid character in entity name
Line: 155
Column: 44
Char:

Special characters as '&', '>' or '>' in file names are not allowed in Visual Studio Enterprise, Professional or Community:

Anyway I have added a fix in the XML parser to save some special characters encoded:
& -> &
< -> <
> -> >
" -> "
' -> '

You will find it in the next version.

Yeah, not sure if this is from legacy files.
Unfortunately the fix doesn't work.

Before create a new file

After Create a new file

Basically we dont want to replace & by &

Ok, I see you reverted the change.

#166


Artisans to produce Services

For many countries in the world it is true that tourism makes up a very large part of their national economy, but I don't it's portraited well enough in the mod. How do you accumulate the income of tourism in the mod? Tourism is important for many countries, notably in the mediterreanen (Asia Minor, Northern Africa). But due to the recent evcents (terrorism, arab spring) tourism has declined, which causes unemployment rates and debts. So integrating it would greatly benefit the mod, I think.

As part of issue #851 I am playing around with the thought to create services (maybe several types of them), as a goods. This goods would be produced by artisans (currently called entrepreneurs - to be renamed to ??).  As industrial productivity increases (particularly in the late 2000th and 2100 century), the number of workers need to produce industrial (factory) goods,  wheres services productivity changes little (i.e. artisans producing various types of services). This mod the "de-industrialization" of highly developed nations (i.e. the industry - by technology and invention - is extremely efficient and does not require lot's of workers).
Tourism is the trade of service goods.

Well, entrepreneurs fits just fine. I wonder, will services also need ressources or actually be ressources and will being an industrialized country be a worse choice than rather being one that mainly focuses on services ?

Services
Transportation Services
Advanced Services
They would be separated by the goods that they require. Services are consumed by pops.
Maybe some aspects of the financial industry could be considered part of "Advanced Services".

From http://www.trademap.org/Index.aspx services
These are the services to be added (according to BPM6.
travel_services (service code 6 Travel)
transport (service code 3 Transport)
advanced_services (service code 10 - Other business services)
financial_services (service code SOX Commercial services)
services (other services)
Financial_services exists already and are produced in "bank factories". The other will be produced by artisans only.

Will close this issue but, we need a lot of balancing of the economy before release. It is pretty unbalanced at the moment.


OpenAPI in minimal web APIs .NET 7 /1

Fixes #25672
Internal review URL
Public review  of the doc build:

download the HTML files
unzip
Review OpenAPI and the two following sections
For links outside the doc, remove the review. from URL.

please review  @zuckerthoben  @RicoSuter  @martincostello  @hwoodiwiss @feO2x  @commonsensesoftware

cc @captainsafia

@brunolins16  please review

Per dotnet/aspnetcore#41426, I believe that Stream should be added to all relevant sections and examples for Special Types. A Stream parameter in a Minimal API will be bound to HttpContext.Request.Body. This will new in 7.0

Per https://github.com/dotnet/aspnetcore/issues/41426, I believe that Stream should be added to all relevant sections and examples for Special Types. A Stream parameter in a Minimal API will be bound to HttpContext.Request.Body. This will new in 7.0

Can you file a new issue for this?

@captainsafia sure. what will I call it? I was reading everything in totality, but then I just realized this was specifically regarding OpenAPI.  If @Rick-Anderson is already aware of the forthcoming support for Stream and will add it later, then there's no sense in creating busy work. If that wasn't known or needs an issue track adding new content, I'm happy to create a new issue for it.

Per dotnet/aspnetcore#41426, I believe that Stream should be added to all relevant sections and examples for Special Types. A Stream parameter in a Minimal API will be bound to HttpContext.Request.Body. This will new in 7.0

Can you file a new issue for this?

See #25918

Sorry for being late to the party. I think all of the docs are fine. The only thing I would mention is that you can use a dedicated method (not an anonymous method/lambda) and XML comments on it (as shown here for Swashbuckle. This might also be interesting to people that have an existing MVC code base: they can move to Minimal APIs in a way where they do not have to rewrite everything from scratch but simply register/map things differently (and maybe change from constructor injection to method injection where needed). The existing XML comments for Swashbuckle can stay in place.

@captainsafia please review


Update AriaStreamer class

Update the AriaStreamer class for -

[x] Supporting live-stream with aria
[ ] Supporting  passing different "class-objects/classes" that it can focus on, e.g., markers, objects, receptacles, human-actions etc

TODO:
@jimmytyyang  @zephirefaith @rutadesai @KavitShah1998 should discuss the requirements of what this class should support.

@KavitShah1998 @jimmytyyang Can we close this as immediate needs are satisfied? We do not even know the path to "how" or "from where" we will get receptacle or human-action specific data yet.


Some modules in JS output don't seem to be correctly transpiled

For example, the name.js file in enzyme-matchers/lib/utils is converted to
'use strict';

Object.defineProperty(exports, "__esModule", {
  value: true
});
exports.default = getNameFromArbitraryWrapper;

var _instance = require('./instance');

var _instance2 = _interopRequireDefault(_instance);

function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }

/**
 * Gets the name of the node or component for the SINGLE item
 */
function getNameFromRoot(root) {
  // shallow
  if (root.unrendered) {
    var type = root.unrendered.type;

    return type.name || type;
  }

  var inst = (0, _instance2.default)(root);
  if (inst) {
    return inst._tag;
  }

  // direct node
  if (typeof root.type === 'string') {
    return root.type;
  }

  return typeof root.name === 'function' ? root.name() : '(anonymous)';
}

/**
 * Can take any sort of wrapper. A single node, a component,
 * multiple nodes, multiple components.
 *
 * examples of outputs:
 * - "Fixture"
 * - "input"
 * - "(anonymous)"
 * - "Fixture, 2 "span" nodes found"
 * - "Fixture, 2 mixed nodes found"
 */
/**
 * @function name
 * @returns string
 *
 *
 */

function getNameFromArbitraryWrapper(wrapper) {
  var nodeCount = wrapper.nodes.length;

  switch (nodeCount) {
    case 0:
      {
        return '[empty set]';
      }
    case 1:
      {
        return getNameFromRoot(wrapper);
      }
    default:
      {
        var nodeTypeMap = {};

        // determine if we have a mixed list of nodes or not
        wrapper.nodes.forEach(function (node) {
          var name = getNameFromRoot(node);
          nodeTypeMap[name] = (nodeTypeMap[name] || 0) + 1;
        });

        var nodeTypeList = Object.keys(nodeTypeMap);

        var nodeTypes = nodeTypeList.length === 1 ? nodeTypeList[0] : 'mixed';

        var root = getNameFromRoot(wrapper.root);

        return `${root}, ${nodeCount} ${nodeTypes} nodes found`;
      }
  }
}
module.exports = exports['default'];

It's not a native es6 module any more so it's treated by webpack as a regular js file but notice the return statement has back-ticks which breaks our phantomjs testing setup.

Is this a new issue in 3.5.0 or was it around in 3.4.0?

Hi @blainekasten I think that's a new issue. We are using 3.5.1 atm. It's brought in by jasmine-enzyme implicitly (we are using 2.8.1).

@blainekasten sry I copied the wrong version number. For jasmine-enzyme we are using 3.0.1

Ran into this as well, I think it's because the target for babel-preset-env is set to node@4, which supports backticks.
Since this library is being consumed by jasminze-enzyme, it seems reasonable to have no target in babel-preset-env, which defaults to es2015-es${currentRelease}

@ek5000 Yeah, good call! Would you be willing to put up a PR for that and we can check it out?

Will be releasing as soon as I can get #130 landed


Typos in Usage documentation

This issue was discovered during the September 2022 Joint OGC  ISO Code Sprint.
The issue refers to text at https://github.com/geopython/OWSLib/blob/master/docs/en/usage.rst
The text "OGC API - Features 1.0 - Part 1: Core" should be "OGC API - Features - Part 1: Core 1.0"
The text "OGC API - Coverages 1.0 - Part 1: Core" should be "OGC API - Coverages - Part 1: Core 1.0"
The text "OGC API - Processes 1.0 - Part 1: Core" should be "OGC API - Processes - Part 1: Core 1.0"
The text "OGC API - Records 1.0 - Part 1: Core" should be "OGC API - Records - Part 1: Core 1.0"

Thank you @ghobona it has been fixed


Multiple Agent

Hi,

Micro-XRCE-DDS Agent 1.0.1 (feature/serialdebug)
Micro-XRCE-Client 1.0.1
ROS2 Crystal

When I try to run the agent using serial first and then udp, the following error occurs.

It simply says ERROR.
Conversely, if I run udp first and then serial, I can run both, but no Topic will be generated.

Hi @OpusK,
I fixed this issue in feature/serialdebug. The problem was that Fast RTPS made use of the port 7400, and this is the discovery port established by the XRCE. Now both Agent and Fast RTPS can shared this port. I hope it helps you!

Hi, @julianbermudez
Thanks for your support!
Resolved!


A test edit

This works! The link-checker fails but the preview still deploys, so #30 is solved.

The modifications in #59 seem to work great.
Closing this and #30.


ClickablePointAsync throws error "Node is either not visible or not an HTMLElement"

Version 8.0.0
ClickablePointAsync fails at !quads.Any(),
I debugged during the run and found out that the values of the BoxModelPoints are reset to 0 directly after ApplyOffsetsToQuad.
Before it is called this was one of my BoxModelPoints:
{X = 75.5, Y = 191.0}
offsetX = 0
offsetY = 0
after ApplyOffsetsToQuad, my point is:
{X = 0, Y = 0}

Do we have any code example to test?

var url = "http://<IP_ADDRESS>:9123";
using var browser = await Puppeteer.ConnectAsync( new ConnectOptions { BrowserURL = url, SlowMo = 0 });
var pages = await browser.PagesAsync();
var pageOne = pages[0];
await pageOne.GoToAsync("https://en.wikipedia.org/wiki/Main_Page");

...

var s = (await pageOne.XPathAsync("//span[text()=\"From today's featured article\"]")).First();
var point = await s.ClickablePointAsync();
pageOne.Mouse.ClickAsync(point.X, point.Y, new ClickOptions());

I connect to a CefSharp client inside a WPF application with the port 9123.
The main problem I have is that this doesn't work consistently for me. It failed consistently yesterday, but doesn't fail today, despite no code being altered in between.


UTF-8 encoding on labels

Best,
At the moment I have a dataset with lots of terms words with special characters like ä ö etc. But it seems the characters are not encoded and therefore, I got ?? signs in SPARQL results instead of the proper character for example.

I've had this with multiple input files, XML input is encoded in UTF-8, output strings get encoded to something that is definitely not UTF-8 anymore. German umlauts is an easy testcase.

Interesting. Is it possible to provide a test case?

I can cook something up. I use it via https://github.com/netage/carml-cli by @nicky508. Not sure on what side the UTF gets lost.
Will update here

Cool thanks!

OK this is weird, I created the following input XML file and carml-config: https://gist.github.com/ktk/2a5e81ca079c74612b9b096e7720f51b
I use carml-cli from @nicky508 to convert it:
java -jar cli-0.0.6-SNAPSHOT-jar-with-dependencies.jar -i utf-test.xml -m utf-test.ttl -of ttl -o utf-out.ttl
or
java -jar cli-0.0.6-SNAPSHOT-jar-with-dependencies.jar -i utf-test.xml -m utf-test.ttl -of nt -o utf-out.nt

Result on my mac (BSD subsystem):
<http://example.org/1> a <http://example.org/UTF>;
  <http://example.org/english> "This is english text";
  <http://example.org/french> "Un texte en français";
  <http://example.org/german> "Das wäre ein Text in Deutsch";
  <http://example.org/japanese> "スプーンはありません" .

-> looks perfectly fine.
if I do that on Linux I get:
<http://example.org/1> a <http://example.org/UTF>;
  <http://example.org/english> "This is english text";
  <http://example.org/french> "Un texte en fran��ais";
  <http://example.org/german> "Das w��re ein Text in Deutsch";
  <http://example.org/japanese> "������������������������������" .

Which looks like double-encoding to me.
I used one of our docker-containers for the linux-environment: https://hub.docker.com/r/zazukoians/node-java-jena/, Java 8 based, same version I have on my mac.
Any ideas how this is possible?

Might be a problem with reading the input file. If carml-cli doesn't specify the input encoding, then it is probably defaulting to standard encoding of the OS, which is usually a bad idea.
I could check this later, when I'm behind a pc.

On my windows pc it looks fine as well. That is probably the issue. Take a look at it tommorow

I checked the standard encoding in a docker container from the zazukoians/node-java image. This is ANSI_X3.4-1968 aka US-ASCII. So I believe carml-cli is reading the input file as ASCII. That would indeed explain it.

Tnx @pmaria. What would be the proper way to handle that, take it as utf-8 by default or read out the encoding in the xml itself

If you know your input files encoding upfront, you can either default to an encoding when creating the inputstream, or make it parameterizable via de CLI.
If you don't know the encoding upfront it is more complicated. There is no exact way of determining the encoding of a file. There are utils like Apache Tika's CharsetDetector which you could use. But it is advisable, if feasible, to check the resulting data for Unicode replacement characters, if by chance a wrong encoding was determined.

Thanks @pmaria, I've set my docker image to UTF now with this commit. That fixed it!
We can close this issue. BTW are you interested in shipping @nicky508 CLI interface with carml or should we keep that separately?

Great!

We can close this issue. BTW are you interested in shipping @nicky508 CLI interface with carml or should we keep that separately?

We could look and see if we can combine it with https://github.com/pmaria/carml-jar. Most notably the FnO function loading. However, that project wat thrown together pretty quickly and also needs some work. I'll chat with @nicky508 to see what we can do.
Closing this issue.


qt58: enable darwin compatibility

Motivation for this change
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] macOS
[x] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@orivej Thx again for your time and thorough review. Fyi, this patch is a port of the similar patch for qt-5.6 (7a9400d, intial by @acowley) and porting it by me for qt-5.7 (#23602, #23764). However, atm i ported it to qt-5.7, 5.8 got pushed into staging :)

fyi, qt 5.8 does not build with system-libpng anymore, further info QT-Wiki

I can't test on macOS myself, but this since this is just a copy of the Qt 5.7 Darwin patches, it looks good enough!

@ttuegel i am building this for x86_64-linux and x86_64-darwin on my company's hydra and use it currently myself.


Expose functions for parsing statements

This is tightly focused on the needs for executing statements in the REPL.
There will probably also be the need to expose functions for definitons like imports but I think that will be a second PR

Updated so that statements is the only public function. block is only useful when parsing declarations is_whitespace makes sense to be private when passing in strings.
statements takes in a string and was already documented.
I've updated all the tests expression_something to use statements because there is no point duplicating tests and this makes those cases that are not testing the module much simpler

Closing due to inactivity, please feel free to reopen

For context. progress on my project was blocked by inactivity on this PR https://github.com/DanielleMaywood/glexer/pull/17
I would need the glexer PR first before returning to this project.


[Missing Feature] Add objetive map and name when cycle

http://imgur.com/u4Qieea
I was on my iPad when screenshot taken, but i think you understand me

Why do people keep making pull requests from this branch to suggest features.


Deployment issue

Describe the bug
Trying to test the product and getting an error everytime it hits the build stage npx vite build
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-dhduwk
Steps to reproduce

Click deploy

Expected behavior
I expect it to be deployed with a preview link
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 130
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 114569206,
  "usedJSHeapSize": 105440166,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = daniellannon
Chat ID = 7835e9d32c27

Additional context
Error:
An error occured while streaming this message.

Hi @daniellannon,
We are aware of this issue and are tracking it here: #1022.


Runtime error when start rippled on Ubuntu

Fresh install as described here with default config.
$ /opt/ripple/bin/rippled --net --conf /etc/opt/ripple/rippled.cfg
Loading: "/etc/opt/ripple/rippled.cfg"
Watchdog: Launching child 1
terminate called after throwing an instance of 'std::runtime_error'
  what():  locale::facet::_S_create_c_locale name not valid
Watchdog: Launching child 2
terminate called after throwing an instance of 'std::runtime_error'
  what():  locale::facet::_S_create_c_locale name not valid
Watchdog: Launching child 3
terminate called after throwing an instance of 'std::runtime_error'
  what():  locale::facet::_S_create_c_locale name not valid
Watchdog: Launching child 4
terminate called after throwing an instance of 'std::runtime_error'
  what():  locale::facet::_S_create_c_locale name not valid
Watchdog: Launching child 5
terminate called after throwing an instance of 'std::runtime_error'
  what():  locale::facet::_S_create_c_locale name not valid

What version of ubuntu? Googling the error, it seems that it might be a system locale issue. Perhaps trying to update your system locale?

Deal with it as described here https://github.com/potree/PotreeConverter/issues/281#issuecomment-375997005


Is it possible to check that 'last timestamp' is the actual time from the corresponding GPS data?

Hey Gary - Congrats on the release of icloud3 v3. I have updated already.
I am having an ongoing battle with speed calcs in the last km from home related to the high frequency GPS data. I am working on a couple of solutions but still struggling to find one that works!
As an alternative to dropping some of the data for speed calculations, I wanted to explore if there was a way to improve the speed calculation accuracy. My GPS accuracy is high (4 or 7m) so I'm thinking that speed calcs should be quite accurate too.
Is there a way you can help me test/confirm that the last timestamp attribute is the actual/exact time relating to the GPS data?
The reason I ask is Jeff (composite author) is using GPSlogger with a high frequency GPS update rate and speed calcs are good.

If you set the log level to rawdata on the Event Log > Actions > xx or Configure Settings > Page 2 > Format Parameters screen, the actual data from iCloud is displayed in the iCloud-0.log file. The timestamp value and lat/long values are displayed. You could compare that value with the one in the devtrkr entity’s attribute for locations from famshr.

OK thanks Gary. Congrats again on the release of v3.
I think I have cracked the solution from the other side. Getting good GPS data at 2min intervals when away from home using an automation/script and a template sensor triggered by the script.
The drive home tonight will be an good test as this is when I get the high frequency GPS data affecting speed calcs...

Working! Below is a chart with raw icloud and a composite that has combines HA app with corrected icloud3 data.
Thanks for all the help with this over the last week or so - much appreciated

Glad you got it working. I assume they they get their bicycles out if there is a 25% over spike.
I assume the spike in the top image is the short interval iCloud3 data.

The operation of iCloud3 is handled by icloud3_main.
Every 5-secs, see if anything needs to be done [ _polling_loop_5_sec_device]

Check is made to see if the mobile app issued an update trigger or the location changed. [_main_5sec_loop_update_tracked_devices_mobapp]

If so, validate the data (gps accuracy, not old, not already processed, etc). [_validate_new_mobapp_data]

If useable, update the device [process_updated_location_data]

Determine the interval to the next location update, Zone distance, stationary zone update, etc and update the sensors and device tracker entity. [determine_interval.determine_interval]

Check the timer (next update Time, enter stationary zone, old location update, enter zone delay, etc) to see if an iCloud l location is needed [_main_5sec_loop_update_tracked_devices_icloud]

If so, get the location data from iCloud for all devices in the famshr account and check the data for all devices to see if it is useable [_validate_new_icloud_data]

If useable, update the device [process_updated_location_data]

Determine the interval to the next location update, Zone distance, stationary zone update, etc and update the sensors and device tracker entity.  [determine_interval.determine_interval]

A lot of other things go on to handle other things dealing with bad Location data, Stationary zones, devices that are monitored, zone handling, etc. but that it’s the basic logic. Once the data is useable, it is always handled the same for all devices and all data sources.
Gps accuracy is different for each device, even sitting side-by-side. It depends on the cell signal, satellite signal, internal components, etc. Google can explain it in more detail.
Hope this helps. Dig through the code to see how things are handled if you are so inclined.

Thanks Gary.
I have been eyeballing the log and found something strange with these two items. They are in the log one after the other (2mins apart) but the GPS data is only 5 secs apart.
03-26 17:02:02 [pyicloud_ic3:1506] ICLOUD > ──────── FAMSHR DATA - <MAGDA'S IPHONE/MAGDA_IPHONE> ────────
{'▶ITEMS◀ (items)': {'id': 'AdpQUk+gea...', 'modelDisplayName': 'iPhone', 'lostModeCapable': True, 'name': "Magda's iPhone", 'deviceClass': 'iPhone', 'deviceStatus': '200', 'rawDeviceModel': 'iPhone14,2', 'batteryLevel': 0.4399999976158142, 'deviceDisplayName': 'iPhone 13 Pro', 'prsId': 'MTQ1MDE3NTcyNg~~', 'batteryStatus': 'not charging', 'deviceModel': 'iphone13Pro-1-1-0', 'data_source': 'FamShr'}, '▶LOCATION◀ (location)': {'isOld': False, 'isInaccurate': False, 'altitude': 0.0, 'latitude': -33.86737200527077, 'horizontalAccuracy': 13.59509039048701, 'timeStamp':<PHONE_NUMBER>789, 'verticalAccuracy': 0.0, 'longitude':<PHONE_NUMBER>4724404, 'timestamp':<PHONE_NUMBER>, 'location_time': '5:02:01p'}}

03-26 17:04:00 [pyicloud_ic3:1506] GETLOC ⡇ ──────── FAMSHR DATA - <MAGDA'S IPHONE/MAGDA_IPHONE> ────────
{'▶ITEMS◀ (items)': {'id': 'AdpQUk+gea...', 'modelDisplayName': 'iPhone', 'lostModeCapable': True, 'name': "Magda's iPhone", 'deviceClass': 'iPhone', 'deviceStatus': '200', 'rawDeviceModel': 'iPhone14,2', 'batteryLevel': 0.4399999976158142, 'deviceDisplayName': 'iPhone 13 Pro', 'prsId': 'MTQ1MDE3NTcyNg~~', 'batteryStatus': 'not charging', 'deviceModel': 'iphone13Pro-1-1-0', 'data_source': 'FamShr'}, '▶LOCATION◀ (location)': {'isOld': False, 'isInaccurate': False, 'altitude': 0.0, 'latitude': -33.86806048337412, 'horizontalAccuracy': 35.0, 'timeStamp':<PHONE_NUMBER>349, 'verticalAccuracy': 0.0, 'longitude':<PHONE_NUMBER>5677794, 'timestamp':<PHONE_NUMBER>, 'location_time': '5:02:05p'}}

Not strange at all. iCloud does this all the time. I guess it will return the data it already has if there are delays contacting the phone for its location.

It’s that these two gps data points create speed delta of 100km/h which was not remotely possible at that time

Sorry to be a pain. Just trying to work out what’s happening so I can see if I can use the data for speed tracking.
I suspect this might also be related to the speed jitter when close to home and there is high frequency gps data being requested.

For your use case, you have to analyze the data you are getting and filter out what you should not use. You can ignore a result if it doesn’t make sense (spike, x% change in short period of time, etc) because you can not control the data you are getting.

That is a good idea. I have been thinking the same but I need to catch this before I push the data into the composite integration but I need the composite integration to do the speed calculation that catches the error.
I will find a way but good to see if we can fix it at source. Then it will ‘just work’ (it is an Apple thing after all) haha

I am working on a slightly different approach with my speed sensor. I will only use GPS data if it is 1min plus old. This should help with random blips caused when the data is too close together. Also eliminates speed data being used during the high frequency period when getting close to home.
I will keep you posted!
And thanks again for all your help with this.


About GPU memory

Hi,
I try to test features on cifarfs by this command:
"""
$ python main.py --dataset-path "" --dataset cifarfs --model resnet12 --test-features "[/AS1.pt11, /AS2.pt11, /AS3.pt11]" --preprocessing ME --n-shots 1
"""
and it output "RuntimeError: CUDA out of memory".
I am using 3080 GPU with 8GB memory.
I also tried colab and got the same problem.
May I know how to fix it?
Thanks

Hi,
Sorry for the late response.
It is probably due to small GPU memory. We have added a new version with a parameter --batch-fswhich controls the batch size during few shot evaluation. Can you try to run the script with the new parameter :
$ python main.py --dataset-path "" --dataset cifarfs --model resnet12 --test-features "[/AS1.pt11, /AS2.pt11, /AS3.pt11]" --preprocessing ME --n-shots 1 --batch-fs 20
Default value is 100. If it's still not working, can you try to reduce it even more?
Best,

It works!
Thank you very much.


Update aggregates.md

Fixes https://github.com/duckdb/duckdb/issues/14144 (see discussion there for proof that this formula is now correct).

Thanks!


Extend AccountId to two Felts and refactor creation process

What has changed
AccountId previously fit into a felt or ~64 bits. Now its representation is two felts:
pub struct AccountId {
    first_felt: Felt,
    second_felt: Felt,
}

In Rust, the first and second felts are always called as such, typically accessed with id.first_felt() and id.second_felt().
In MASM, the first felt is the hi felt and the second felt is the lo felt and its typical stack representation is [account_id_hi, account_id_lo].
The layout of an ID is:
1st felt: [zero bit | random (55 bits) | storage mode (2 bits) | type (2 bits) | version (4 bits)]
2nd felt: [block_epoch (16 bits) | random (40 bits) | 8 zero bits]

See the AccountId documentation for details on the layout.
There is a type AccountIdPrefix representing the validated first felt of an AccountId which is primarily used for non-fungible assets and for asset deserialization. Ideally we would make that private, but for now it must be public for the constructors of non fungible assets.
Notable Changes
Here is a quick overview of what changes with this PR. Layouts are in "memory order", that is, stack order would be the reverse of that.

Fungible Asset representation:

previously: [amount, 0, 0, faucet_id]
now:        [amount, 0, faucet_id_lo, faucet_id_hi]

Non-Fungible Asset representation:

previously: [hash_0, faucet_id, hash_2, hash_3]
now:        [hash_0, faucet_id_hi, hash_2, hash_3]

NoteMetadata

previously: [tag, sender_acct_id, encoded_type_and_ex_hint, aux]
now:        [sender_hi, sender_lo_type_and_hint_tag, note_tag_hint_payload, aux]
The NoteMetadata AfterBlock can only take values less than u32::MAX so that when encoded together with the note tag in a felt (i.e. note_tag_hint_payload above) felt validity is guaranteed.

The serialization of AccountIdPrefix is compatible with AccountId to allow for deserialization of a prefix from a serialized AccountId. This is used for the next point.
AccountIdPrefix serialization is such that the first byte of the serialized layout contains the metadata. This is used in the asset deserialization to read the type and based on that deserialize the bytes as a fungible or non-fungible asset.
The NoteTag::from_account_id effectively takes the most significant bits of id_first_felt << 1. This means the high zero bit is ignored with the rationale being that it would not add any value for matching an account ID against a note tag, since all ID's high bits are zero. Let me know if this doesn't make sense.
The previous account creation tests were replaced because they now require a MockChain due to the dependence of the account ID on an anchor block.
The layout of account IDs in various keys, stacks and hashes across Rust and Masm is not entirely consistent I think because it's not always quite clear to me whether a layout needs to be reversed or not. What is also unclear to me is in what order IDs should be layed out within layouts like [account_nonce, 0, account_id_hi, account_id_lo]. So anything related to that would be particularly helpful to have reviewed.
I snuck in some small improvements to the debugging experience but I'd like to revisit this topic in a future PR.

This PR does not change anything about the account tree, e.g. the accounts: SimpleSmt<ACCOUNT_TREE_DEPTH> field in MockChain.
I think a rough sensible review order would be AccountId + AccountIdPrefix followed by the changes to assets, all in Rust.
Then moving over to MASM.
Open Questions

Should we enforce MIN_ACCOUNT_ONES in both first and second felt? For now only the first felt is enforced with the only rationale being that the first felt is used as the key in the asset SMT for fungible assets. But if we treat this as an implementation detail then it might make sense to also enforce something similar on the second felt?
Do we need a serialization format for NoteMetadata that is different from the Word encoding? We had a different one before this PR, but now the serialization format and word encoding are identical.

Follow-Up Tasks
There are still some TODOs in the code and those could mostly be addressed in a follow-up PR so this PR could would not necessarily be blocked by that (checked boxes are included in this PR).

[ ] Introduce AccountIdError. There are now new error conditions for IDs and a separate error might be cleaner.

With the introduction of this error it might be possible to make AccountId::new_dummy a const fn and then we could possibly replace the somewhat duplicate functionality of miden_objects::testing::account_id::account_id.

[x] Quite a bit of Rust documentation needs to be updated.
[ ] Implement Display for AccountType and use it in error messages.
[ ] Remove unnecessary code in generate_account_seed.
[ ] Remove build.rs constants patching for POW which we no longer have.
[ ] Move type, storage mode, version out into their own modules to reduce size of account_id.rs.
[ ] Move newly introduced duplicate procedures into the new util library (after #1002).
[ ] Go through error messages and double check if they are up to date (e.g. ones previously mentioning "account id" should now likely mention first or second felt for accuracy).
[ ] validate_fungible_asset_origin validates the account ID and then calls validate_fungible_asset which validates it again check if we can remove some redundancy here.
[x] Deal with NoteExecutionHint::AfterBlock being constructable by users but it now cannot contain u32::MAX and we might want to prevent that.
[x] Update and test build_swap_tag.
[ ] Add lots of new tests for changed things: version and epoch in ID, test new and changed MASM procedures more thoroughly.
[ ] Think about account ID encoding for network identifier and error detection (e.g. bech32, base58, etc.).

Why it changed
See #140, although I'd like to ideally add a summary here.
closes #140

To answer the open questions:

Should we enforce MIN_ACCOUNT_ONES in both first and second felt? For now only the first felt is enforced with the only rationale being that the first felt is used as the key in the asset SMT for fungible assets. But if we treat this as an implementation detail then it might make sense to also enforce something similar on the second felt?

As mentioned in one of the inline comments, I'm actually not sure if we need this constraint at all. The original motivation was to avoid IDs which consist of all zeros, but with 128-bit IDs I think finding such an ID is pretty unlikely.

Do we need a serialization format for NoteMetadata that is different from the Word encoding? We had a different one before this PR, but now the serialization format and word encoding are identical.

I don't think we need to do anything different here. I'd still implement Serializable and Deserializable for NoteMetadata - but these could rely on word serialization.

I think I've addressed all the minor changes as well as the removal of the top zero bit and changing the non-fungible asset layout to avoid having to set the fungible bit.
Other outstanding issues have been added to the follow-up tasks in the PR description or need feedback in the comments. I think they could pretty much all be resolved after this PR as well if we like to merge it, so I don't think the PR needs to necessarily be blocked by these.