docs: Fix typos in git-guide.md.

Fixes minor grammatical errors as well as shifts the text to be in second person form (one sentence was in first person).
There exist similar first-person/second-person mixing in the document. Should they be updated to be consistent? Example:

One situation in which git rebase will fail and require you to intervene is
when your change, which git will try to re-apply on top of new commits from
which ever branch you are rebasing on top of, is to code that has been changed
by those new commits.
For example, while I'm working on a file, another contributor makes a change to
that file, submits a pull request and has their code merged into master.
Usually this is not a problem, but in this case the other contributor made a
change to a part of the file I also want to change. When I try to bring my
branch up to date with git fetch and then git rebase upstream/master, I see
the following:

I think that other case feels more reasonable to be written in first-person.


River all-outputs option feature?

Hello,
Is there a way in waybar to only show tags that are occupied in river window manager?
This was mentioned in #2101 but was never answered or resolved.
Sway has this:
    "all-outputs": false,

Thanks

Probably a duplicate of #3823 if so please close.


[Feature] Hide surface ores

Works similarly to vanilla, any blocks adjacent to blocks marked as air in ore structures will not be placed.

implemented with exposed in the ore template. true = show ores, false = hide ores. defaults to true.


[Bug] XmlConfiguration cannot be cast to OapConfiguration

Search before asking

[X] I had searched in the issues and found no similar issues.

Apache SkyWalking Component
OAP server (apache/skywalking)
What happened
skywalking-oap:8.7.0 when i start the oap server local, it comes to an error org.apache.logging.log4j.core.config.xml.XmlConfiguration cannot be cast to org.apache.skywalking.oap.server.core.logging.log4j.OapConfiguration
it seems like the OapConfigurationFactory not working, i read the apache logging docs and the org.apache.logging.log4j.core.config.plugins.util.PluginManager source code，and i do not find  PluginManager .addPackage() method is invoked in server-core/logging code
i do not quite understand the mechanisms，if i miss any setting?
What you expected to happen
none
How to reproduce
none
Anything else
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

This is not in the scope of SkyWalking.

skywalking-oap:8.7.0 when i start the oap server local, it comes to an error org.apache.logging.log4j.core.config.xml.XmlConfiguration cannot be cast to org.apache.skywalking.oap.server.core.logging.log4j.OapConfiguration

@kezhenxu94 But this description doesn't match the possible case, I am afraid. If something in default has cast exception, the OAP server can't start up.

skywalking-oap:8.7.0 when i start the oap server local, it comes to an error org.apache.logging.log4j.core.config.xml.XmlConfiguration cannot be cast to org.apache.skywalking.oap.server.core.logging.log4j.OapConfiguration

@kezhenxu94 But this description doesn't match the possible case, I am afraid. If something in default has cast exception, the OAP server can't start up.

Yes I agree absolutely that the default settings should be OK. That's why I ask @ayumiono to recheck and provide more information.


Implemented per-chromosome coverage plotting

Optional per-chromosome coverage plotting added.

This change is

Hello @spaul91! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file bin/coverage_report_single.py:

Line 32:1: E402 module level import not at top of file
Line 471:35: E261 at least two spaces before inline comment
Line 471:80: E501 line too long (82 > 79 characters)
Line 472:80: E501 line too long (82 > 79 characters)
Line 488:1: W293 blank line contains whitespace
Line 490:80: E501 line too long (84 > 79 characters)
Line 1145:80: E501 line too long (105 > 79 characters)
Line 1246:80: E501 line too long (111 > 79 characters)
Line 1540:80: E501 line too long (91 > 79 characters)
Line 1547:80: E501 line too long (102 > 79 characters)

version wants bumping to 1.3.0: https://github.com/eastgenomics/athena/blob/master/bin/version.py
fixes #53 that I forgot to do last time for 1.2.1

I'm including a fix to https://github.com/eastgenomics/athena/issues/56 in this PR, so please don't merge this until that's done. Thank you

I'm including a fix to #56 in this PR, so please don't merge this until that's done. Thank you

Have opened bugfix/issue-56 in an independent branch. Ignore my previous comment


Mat-menu dropdown positioned wrong on zoomed ipad

Bug, feature request, or proposal:
Bug

What is the expected behavior?
Menu opens and stays in the right position when zoomed.
What is the current behavior?
Open a menu while zoomed, or zoom with a menu open, and its will be positioned wrongly.
What are the steps to reproduce?
Use an iPad.
Go to angular.io Zoom in on the page. Open the basic menu. Or, open the basic menu then zoom the page.
What is the use-case or motivation for changing an existing behavior?
The dropdown should work on mobile devices while zoomed. A user might for instance zoom to make the text bigger, causing the menu to seemingly disappear from view.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular 7.0.3, Material 7.04, iOS 10 and 12, not tested on 11, Browser is Safari
Is there anything else we should know?
This problem does not occur with ngBootstrap's dropdown. So there are ways to implement drop down menus without this issue.
Zooming out with the Basic menu open (on Angular Material's example page for Mat-Menu), can result in items disappearing from the menu.

Duplicate of https://github.com/angular/material2/issues/10924.

Duplicate of #10924.

This issue isn't about the zoom css property, it's pinch to zoom. Something that happens automatically when selecting an input on an iOS.

The root of the issue is the same. The numbers provided by getBoundingClientRect are incorrect both when you use the zoom property and when you've zoomed in the page. There's a similar discussion on https://github.com/angular/material2/issues/13090 as well.

The root of the issue is the same. The numbers provided by getBoundingClientRect are incorrect both when you use the zoom property and when you've zoomed in the page. There's a similar discussion on #13090 as well.

I see. Thanks.
It is disappointing that this is a low priority issue. It makes a good number of controls problematic on iPad, as the iPad has a tendency to zoom automatically.


Security HTTP Headers

Implement Security HTTP Headers to prevent vulnerabilities.

[x] Content-Security-Policy
[x] X-Frame-Options
[x] X-Content-Type-Options
[x] Feature-Policy
[x] Referrer-Policy

Tool to use: https://securityheaders.com

Solved with Cloudflare Workers.
#199


Update CHANGELOG.md

added unreleased change to PfxImport

Pull Request (PR) description

This Pull Request (PR) fixes the following issues:

Task list:

[ ] Change details added to Unreleased section of CHANGELOG.md?
[ ] Added/updated documentation, comment-based help and descriptions in .schema.mof files where appropriate?
[ ] Examples appropriately updated?
[ ] New/changed code adheres to Style Guidelines?
[ ] Unit and (optional) Integration tests created/updated where possible?

This change is

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@ac69dc2). Click here to learn what that means.
The diff coverage is n/a.

@@         Coverage Diff          @@
##             dev   #135   +/-   ##
====================================
  Coverage       ?    94%
====================================
  Files          ?      5
  Lines          ?    519
  Branches       ?      2
====================================
  Hits           ?    491
  Misses         ?     26
  Partials       ?      2

@mattbowes Please see the comment https://github.com/PowerShell/CertificateDsc/pull/134#issuecomment-395687924. Please update the other PR with this change instead. Simply push more commits to your working branch in your fork an the original PR will update automatically. Then we can close this PR.

I'll close this one and we can just see if we can pick up and finish the other abandoned PR.


Add proper memory management to hnsw index and integrate with dense t…

…ensor attribute.
@toregge please review
@arnej27959 FYI

@toregge and @arnej27959 please have a look at the last commit.


chore(ownership): Change package ownership to @fca.gg

Updated package info to attribute ownership to the FCA organization. The package name and git repository URL has been updated.
Credentials need to be updated for the release CI/CD to function correctly with the new NPM ownership.

Please delete the branch upon merging


CUDA is required, but it's not in requirements.txt

You install torch without CUDA, and require it in generate.py
this repo won't run out of the box
(venv) PS C:\Users\entityinarray\Projects\open-oasis> python .\generate.py
Traceback (most recent call last):
  File "C:\Users\entityinarray\Projects\open-oasis\generate.py", line 13, in <module>
    assert torch.cuda.is_available()

https://github.com/etched-ai/open-oasis/blob/f4563c623715239a6b0a3080dcb2405767da0b87/requirements.txt#L24
https://github.com/etched-ai/open-oasis/blob/f4563c623715239a6b0a3080dcb2405767da0b87/generate.py#L13

Sorry about that! I've updated the installation instructions to explicitly require CUDA.
I think other backends should probably work as well, but for now we've only tested on CUDA. Let me know if the new instructions help?


broken link

The link to the Sample is broken   (archive.msdn.microsoft.com’s server IP address could not be found.)

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

ID: 278577af-89ab-c813-24e1-175a7b986b57
Version Independent ID: 45394d15-a85d-95c1-c662-86a00f7ed4b7
Content: IDataObject.GetData Method (System.Windows)
Content Source: xml/System.Windows/IDataObject.xml
Product: dotnet-api
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

Thanks for reporting the issue @tombrown571. This has now been fixed!


Not able to play imported models...

Great tutorial and video!
I've trained a model on GCP (Google Cloud Platform) and I've dowloaded a 1000 epochs checkpoint from GCP on my laptop in a demo-checkpoint-1000-epochs folder.
Then, I've tried to run the 1000 epochs model using the command: python3 -m trainer.task --render --restore --output-dir ./demo-checkpoint-1000-epochs replacing the 6000 epochs files by those ones: model.ckpt-1000.data-00000-of-00001, model.ckpt-1000.index, model.ckpt-1000.meta and I've got a ValueError: Can't load save_path when it is None.

Finally, just finishing writing this issue, I've found a solution, I've just to edit the checkpoint file to
model_checkpoint_path: "model.ckpt-1000"
all_model_checkpoint_paths: "model.ckpt-1000"
And effectively the model is weaker, loosing 20 / 2 against the cheating player.
I hope this can help someone else...
Thanks again folks!


Q: does the DOJ have jurisdiction or can help folks outside the US?

Description
Do we need to ask for country in the form for contact? Could the DOJ help someone who's experiencing an issue outside the US?
Examples:

Ability to vote while abroad
Discrimination while on activity duty outside the US?

(regardless, we can update the form since it's a less common path but want to check this detail_
Definition of done

[x] Ask
[ ] Update form
[ ]

Attachments
title

Daniel:
Yes,

you may have expats who are US citizens who have returned to their country...
human trafficking
very rare


Lost Data (For Me - Orders)

Api Problem
We have encountered a problem where our order data has lost. This is on me orders query. How we could fix this issue?
Apparently we updated some modifications of the api by Saleor.
Thanks!

Please provide more details or steps to reproduce the problem, it's hard to tell what could be the issue from the brief description.


Stimulus Reflex with Rspec not working

Hi. I have added stimulus reflex to a form and its working fine. But my rspec tests are failing because Form submit button is disabled even when form is valid. Does stimulus reflex works with rspec? I didn't find any documentation related to rspec.
Versions
StimulusReflex

Gem: stimulus_reflex (2.2.3)
Node package:"stimulus_reflex": "^3.2.3"

External tools

Ruby: [ 2.5]
Rails: [5.2]
Node: [ 12.4.0]

Browser

Browser [chrome]
Version [83]

The issue is fixed. Stimulus reflex works fine with Rspec. I had to print test logs to figure out the issue
tail -f log/test.log
There was exception being thrown which was causing action cable broadcast to fail.


zf 6.2 - meteor package adjustments

Meteor package changes:

removed problematic Motion-UI dependency (there is also PR for Motion-UI Meteor package: https://github.com/zurb/motion-ui/pull/73 )
added missing .scss files

There is needed meteor publish asap, because otherwise it will not work well.
The new version for Meteor is 6.2.0_1 which is a proper versioning for Meteor packages. It is still foundation 6.2.0 but with fixes for Meteor.

Thanks!

Published Meteor package: 26698d8aed59787705ca4246bfd3e9e12e7b852b


Discourage use of global class names with an escape hatch

This is a feature request.
I have just discovered a code base that abuses global class names all over the place, like this:

const StyledDiv = styled.div`
  display: flex;

  .customer-name {
    margin-top: 1em;
  }
`

<StyledDiv>
  <div className="customer-name">
    {name}
  </div>
</StyledDiv>

I propose that styled-components makes it impossible to do this by default. It could offer a configuration flag to turn this behavior on for legacy code bases already relying on it.
For the case where one truly needs to use global class names, styled-components could follow the css-modules approach and introduce a :global pseudo-class:
const StyledDiv = styled.div`
  display: flex;

  :global(.btn .btn-primary) {
    margin-top: 1em;
  }
`

Closing since I don't think this is implementable, or has been presented as particularly desireable, since we never got similar issues again. Sorry! Happy to reopen if things change.


Portainer agent configuration in swarm

I'm trying to deploy the Portainer agent in a stack (run on docker swarm) using the classical compose yaml file.
The agent seems to start properly, but when I try to connect manually from a Portainer instance running as docker container outside the stack, I get an error and I don't see any containers, volumes or networks information.
In Portainer docker logs I see these errors every time I refresh the dashboard:
2018/06/20 14:37:20 Response: map[err:Get https://<IP_ADDRESS>:9001/containers/json?all=1: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)]
2018/06/20 14:37:20 http: proxy error: Invalid Docker response
2018/06/20 14:37:20 Response: map[err:Get https://<IP_ADDRESS>:9001/networks: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)]
2018/06/20 14:37:20 http: proxy error: Invalid Docker response
docker run command to start Portainer
docker run -ti --name portainer -v portainer_volume:/data -p 9000:9000 portainer/portainer
agent URLs manually configured
tasks.agent:9001
or
tasks.kafka_agent:9001
or
<IP_ADDRESS>:9001 (the IP of the docker host)
docker-compose.yml
_version: '3.6'
networks:
  kafka_n:
    driver: overlay
services:
  zookeeper:
    image: wurstmeister/zookeeper
    hostname: zookeeper
    deploy:
      mode: replicated
      replicas: 1
      restart_policy:
        condition: any
        delay: 1s
    volumes:
      - ~/.dockerhost/data/zookeeper/data:/opt/zookeeper-3.4.9/data
    ports:
      - 2181:2181
      - 2888:2888
      - 3888:3888
    networks:
      - kafka_n
  kafka:
    image: wurstmeister/kafka
    hostname: kafka
    deploy:
      mode: replicated
      replicas: 1
      restart_policy:
        condition: any
        delay: 3s
    environment:
      KAFKA_ADVERTISED_HOST_NAME: <IP_ADDRESS>
      KAFKA_ZOOKEEPER_CONNECT: zookeeper:2181
    volumes:
      - ~/.dockerhost/data/kafka:/kafka
    ports:
      - 9092:9092
      - 7203:7203
    networks:
      - kafka_n
  agent:
    image: portainer/agent
    deploy:
      mode: global
    environment:
      # AGENT_CLUSTER_ADDR: tasks.agent
      AGENT_CLUSTER_ADDR: tasks.kafka_agent
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
    ports:
      - 9001:9001
    networks:
      - kafka_n_

I'm sure doing something wrong, but what?

I think that you'll want to use the following service definition for the agent:
services:
  agent:
    image: portainer/agent
    environment:
      AGENT_CLUSTER_ADDR: tasks.agent
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
    ports:
      - target: 9001
        published: 9001
        protocol: tcp
        mode: host
    networks:
      - portainer_agent
    deploy:
      mode: global

And then use the IP address of any node in your cluster as the agent URL (<IP_ADDRESS>:9001 ?).
More info in our docs here: http://portainer.readthedocs.io/en/stable/agent.html#connecting-an-existing-portainer-instance-to-an-agent
We also recommend to put the agent inside its own overlay network.
Closing the issue, feel free to re-open/comment if needed.

With the configuration you wrote everything works perfectly :) Thanks a lot!!
My error was the network: instead of connecting to the kafka network 'kafka_n', I have to crate a different one (e.g. 'portainer_agent').
Now I have a couple of other questions:

Do I have to deploy portainer as service on the same swarm and connected to the same portainer-agent network in order to let everything work properly, right?
Is it not possible somehow to run a portainer instance outside any swarm and let it connect to one or more agents deployed in one or more swarms?

Thanks a lot for your time

Hi @bygui86
There are two ways to connect Portainer to an agent:

You deploy Portainer and the agent in the same overlay network and you connect Portainer to the agent via tasks.<agent_service_name>:<agent_port>

This is our recommended setup, we provide a stack file for this available via http://portainer.readthedocs.io/en/stable/deployment.html#inside-a-swarm-cluster

You deploy the agent separately. As the Portainer instance might not be in the same overlay network, you need to expose the agent port on each host (see the configuration example that I gave you above) and you connect Portainer to the agent via <node_ip>:<agent_port>

So to answer your questions:

no, but it's recommended if you have only one Swarm cluster to manage

it's definitely possible to run a Portainer instance somewhere (for example, on your laptop) connected to multiple Swarm environments via the agent

Great. Thank you very much!
And compliments! Portainer is a really good project!


Fix pipeline alert tabulator to use resource_arn

Currently it appears to assume channel_arn is the attribute.
We want to change it to use the partition key from the Events table.

Updated alert tabulator and preprocessing of data to use resource_arn from table partition key.


Props changes doesnt reflect to children component when used with connect

Props is not sync with the connect and the component.
When i toggle a state change props changes inside the connect

But the changes doesnt reflect to the passed component

See. editVisible is true inside the connect but false in the child component

Fixed on #49, breaking change, will be up on next version.

Any ETA on the next version?

I'll publish it soon I need to take time to rewrite flowtypes before publishing the 4.0.0

Is this test covering this issue ? https://github.com/didierfranc/react-waterfall/blob/master/src/test/index.test.js#L30-L40


MySQL Exceptions taking preference during testing

When testing you will find many tests ailing with an error like the below:
_mysql_exceptions.IntegrityError: (1452, 'Cannot add or update a child row: a foreign key constraint fails (`test_my_project`.`users_historicaluser`, CONSTRAINT `users_historicaluser_history_user_id_7f91d047_fk_users_user_id` FOREIGN KEY (`history_user_id`) REFERENCES `users_user` (`id`))')

After removing the history references you can uncover the real error:
"the 'fields' attribute is prohibited." % self.__class__.__name__
django.core.exceptions.ImproperlyConfigured: Using ModelFormMixin (base class of EntryUpdateView) without the 'fields' attribute is prohibited.

It is a shame that the library is hiding errors during testing, so had to remove the package.
Would be nice if it could be turned off during testing,.

When you generate a test user, do you save it to the database? Or do you generate mock 'users' that are not present in the database throughout your tests to save on database calls? The foreign key constraint is indicating that a history record was saved with a user_id that is not present in your users_user table.
It looks to me like django-simple-history is accidentally surfacing a situation where the environment set up for that test does not match something you would find in production.
Maybe temporarily you could remove the middleware that allows for user tracking, and see if that removes the errors. If it does, you may need to save the users you're using to make test requests to the database before tests with django-simple-history will work.

I am saving to the database:
def setUp(self):
        self.user = get_user_model().objects.create_user(
<EMAIL_ADDRESS>            password='1234pass',
            company_id=1
        )

Yes, it looks like the situation is accidently surfacing. An interesting thing though is the error only pops up when the a test fails. If all is fine, then all tests pass without issue.
It is making fixing tests quite hard, so had to remove the package.

This does not look like an issue that needs to remain open as the user abandoned use of the project.  The root cause was not ascertained and no further action had been taken.  Closing.


New version: Facebook.Messenger version 176.0.442527159

- [X] Have you signed the Contributor License Agreement?

[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.4 schema?

Resolves #95099

Microsoft Reviewers: Open in CodeFlow

@stephengillie / @denelon - Can we get a manual merge to resolve the linked issue since MSFTbot is taking a vacation?


i18n(es): add directives-reference

What kind of changes does this PR include?

Translated content

Description

Adds reference/directives-reference.md in Spanish

LGTM!

Thanks @kevinzunigacuellar, @agustinmulet. Amazing work as always ❤!


Suggestion: mention motor driver specs in spec sheet

It would be convenient if the "BeagleBone_Blue_ShortSpec.pdf" and the website could mention the specs of the motor drivers, i.e., 1.2A continuous, 3.2A peak, 2.5-13.5V; and/or the part number TB6612FNG. Thank-you.

I don't get what you did there. The .docx file you added just says the same as the pdf, "4 DC motor drivers". It doesn't mention the power (current & voltage) specs.

@ElhemEnohpi
This was super helpful, thanks, LMNOP! I found a PDF for the specs for the TB6612FNG H-Bridge which lists the following

Power supply voltage; VM = 15  V(Max)
Motor supply (2.5 V~13.5 V)
Output current; IOUT=1.2 A(ave) / 3.2 A (peak) per channel
Output low ON resistor; 0.5Ω (upper+lower Typ. @VM ≥ 5 V)

Also suggest adding to the spec (for noobs like me) noting that there are two of them on the board, each driving two of the DC motor drivers (M1,M2) and (M3,M4).

@jadonk I'm a bit confused, your commit of BeagleBone_Blue_ShortSpec.docx says "Add motor driver strength", but the document still doesn't mention the motor driver strength. Maybe you put up the wrong version?

I would add Output Currents mentioned by AJRepo are 'per-channel' there are four channels on the board, two per chip.


UI: Connector list improvements

Screenshot of current list:

Suggested improvements

Pull the "Create a Connector" button to the left - directly to the right of Connectors title (with appropriate spacing)
In ConnectorListItem, pull the connector name to the left - closer to the icon
Improve the 'tasks' display (currently it's just text).  Rene suggested colored dots possibly with popover info.  I am unclear what the task statuses mean atm - discuss with Rene / Gunnar.  There can be multiple task states simultaneously

PR #155 and PR #156 merged


ja: Make docs/tasks/run-application/run-replicated-stateful-application.md follow v1.18 of the original text

This is a Feature Request

What would you like to be added
Update docs/tasks/run-application/run-replicated-stateful-application.md to follow v1.18 of the original (English) text.
Why is this needed
content/ja/docs/tasks/run-application/run-replicated-stateful-application.md is outdated.
Comments
/language ja
/good-first-issue
File to update
https://github.com/kubernetes/website/blob/dev-1.18-ja.1/content/ja/docs/tasks/run-application/run-replicated-stateful-application.md
Original
https://github.com/kubernetes/website/blob/fb6364d/content/en/docs/tasks/run-application/run-replicated-stateful-application.md
diff between translated and v1.18
https://gist.github.com/fc6029105fab4da99bd4d554777d8328
Note
Currently, we use dev-1.18-ja.1 branch. Please open a PR targeting the branch.
ref
https://kubernetes.io/ja/docs/contribute/localization/
If you have a question, feel free to ask us at slack.k8s.io #kubernetes-docs-ja channel

/assign

/close


fix: MD007/ul-indent

Unordered list indentation.
Trimed leading/trailing spaces
Spaces around headings

There were more files flagged, so I didn't re-enable the rule yet


write register code 10

hello, my Sofar Solar HYD 6000 inverter may have certain registers written with "write register" code 10. I don't see it in pysolarmanv5 but does it exist?
Thanks

Code 10 in hex is 16. See the documentation here.


registry: add default value for pg_version if not found in manifest

The publish endpoint would occasionally fail with Failed to decompress Trunk archive (PLAT-112). It appears this is because of how we fetch the Postgres version a package was built for: checking for pg_version within manifest.json.
Older versions of Trunk CLI did not append this info which caused the endpoint to fail. The new behavior is to assume the Postgres version is 15 since, if the user's Trunk CLI was old enough to not append this info, it'd also not be enough to build the extension for any other version other than 15.

Looks good to me!


[Shadowrun5th] add the data-i18n tags

Hello Jens/@darkxyro,
Can you please add the data-i18n tags so that your superb character sheet can be translated in other languages?
Shadowrun is really appreciated in other countries and languages and no other shadowrun sheet is available in other languages.
It will be very appreciated.
Thanks by advance.

@dyvim you could yourself get this started, there exist instructions and examples on how to do this, no programming knowledge is needed,  just follow some simple rules in inserting the tags. https://wiki.roll20.net/Character_Sheet_i18n

Yes I know that and I know the rules but I don't want to start this without the creator being noticed

Fyi,
I put in the pull request for the Shadowrun 5e sheet I've been working on. It has the translation keys to be multilingual. Downside is its a whole new sheet so you may not want to start over. The translation keys might help you get started on adding ones to the Advanced sheet.

Cassie

I'll check that (and probably translate your new sheet ASAP). Thx

That would be very helpful! I could then adjust the CSS to be more accommodating for multilingual.

@dyvim
The new Shadowrun 5th Edition sheet is now live and includes the translations files.

@dyvim
Are we good to close this request? Do you still want to translate the Advanced sheet? It appears to have not been updated in 6 months.
Thanks,
Cassie

Yes you can close the request. It seems I will have to do it myself if I want it to be done.


@stop does not stop baritone

Bug Description
@stop does not stop baritone
Steps to Reproduce (as best as you can)
Execute a task, then run @stop
Expected Behavior
Baritone and altoclef stops
Actual Behavior
Baritone doesn't stop
Crashlogs (if applicable)
None

This is a legitimate bug.
There are a lot of tasks to execute, is there a specific command that you ran? What part of the command were you on?
We could fix this by simply running onLostControl on every single baritone process to make sure it's all stopped, but that would only be a duct tape solution, it would not fix the bug. This means a task isn't stopping baritone in its onStop method when it should, and it would be great to figure out which task that is.

I "think" I found the issue you encounteded: In GetToBlockTask. I should call super.onStop there where the baritone goal is stopped, but I don't.
But to deal with the problem as a whole I will add the duct tape patch (stop all processes on @stop) but also print a warning if baritone doesn't stop, so it can be made as an issue.


The portal doesn't always warn for unsaved data in Application Settings

I was working with Application Settings and found that it doesn't persist them until you click Save (see my other issue in the docs section https://github.com/MicrosoftDocs/azure-docs/issues/57341).
Although when clicking on Refresh (once I saw that toolbar on top :-) ) it shows this warning, which is a good thing:

But if you click away on any other link, in the left sidebar, for example, it just goes away without warning and without saving.
It should either warn the user that items aren't saved or save automatically when an item is added.
Thank you

Thanks for this! We have made the necessary changes on our end which should be deployed in a couple of weeks.


[defectdojo] Adding SSL verification management

Either:

REQUESTS_CA_BUNDLE points to the CA bundle
config.defectDojo.ssl is:

False: verification is optional
True: verification is mandatory (default behavior)
path to CA bundle: verification is mandatory, and this bundle is used

I suggest adding tests. Also, the existing test failed.

fixed & added pytests
let's discuss separately the best solution for the username/password. I can think of 3 ways :

increase the pylint argument limit (locally on this function or globally) and split into 2 arguments ?
assume the function will always be called correctly and simply remove the try/except ?
leave it as it is, possibly rewording the message ?

leave it as it is, possibly rewording the message ?

+1.  in fact, I like the change of combining password into one. Just wanted to point out that the failing test code needs to be updated according to the change.


WIP: Implement IterativePhaseEstimation

Add IPhaseEstimation IterativePhaseEstimation iterative phase estimation algorithm.
This is the plain-vanilla iterative phase estimation algorithm. It estimates the eigenphase of a pair (unitary, state).

This PR is blocked (or I intend so) on #6238
After #6238 is merged, I'll push appropriate commits here.


fix: check that service-worker.js exists before attempting to move

Attempting to move service-worker.js when it's already been moved fails the build.
If existsSync is not the way to go, happy to be corrected.

Feel free to ignore the actual contents of the PR
Please treat it more like a suggestion - my reason for opening this was that building from master failed in Heroku,
And to me it makes sense that once the file was moved it doesn't need to be moved again.

Oops, missed this. Yes, we can do this for sure!


Publish angular-promise-buttons to npm

I found that you have package.json but it looks like your package wasn't published to npm.
As an npm user I want to install angular-promise-buttons package with
npm install angular-promise-buttons --save

Here you can get more information about the publishing process:
https://docs.npmjs.com/getting-started/publishing-npm-packages
Also this screencasts might be useful https://egghead.io/series/how-to-write-an-open-source-javascript-library (check videos about npm)

Here you go:
https://www.npmjs.com/package/angular-promise-buttons

Thanks!


UTF-8 fixes for ASN and DCH names which come from external data sources.

Fix for https://github.com/fireeye/GeoLogonalyzer/issues/2.
Valid ASN and DCH names from some countries may contain non-ASCII, UTF-8 characters. This fix improves how they are handled.

Had the same issue and it is fixed
Guys you are awesome

Thanks for the suggested fix! I've updated Version 1.01 to support Unicode ASN data. Please let me know if you have any additional issues.


Add Modules from the PowerShell/Modules Repo to the tracked modules list

PR Summary
Modules from the PowerShell/Modules Repo to the tracked modules list:

Microsoft.PowerShell.SecretManagement
Microsoft.PowerShell.RemotingTools
Microsoft.PowerShell.TextUtility

PR Context
Since we have released a new module, I am adding it to the list of modules we are interested in tracking.
PR Checklist

[X] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[X] Summarized changes
[X] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[X] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[X] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[ ] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[X] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[X] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

Is it for 7.0-Servicing?

@JamesWTruher @SteveL-MSFT Can you approve the change?

@SydneyhSmith Please rebase to pass CIs.

We should take this for 7.1 to track SecretManagement usage and it's also low risk.

Re-opening to restart CI

@SydneyhSmith Should we also add Microsoft.PowerShell.SecretStore


OSError should be an Error or Exception, or not escape the SDK

In the Flutter CLI, we recently landed a change to enable the analyzer lint avoid_catches_without_on_clauses here https://github.com/flutter/flutter/pull/52021.
In particular, our goal was to avoid swallowing Dart SDK Error exception types, StateError, ArgumentError, AssertionError, etc. because they certainly indicate a language or API usage mistake. So we changed several instances of:
try {
  ...
} catch (err) {
  ...
}

to
try {
  ...
} on Exception catch (err) {
  ...
}

However OSError is not an Exception, and despite not indicating a mistake like an Error, slipped through our clutches.
OSError: OS Error: Operation not supported on socket, errno = 102
  | at _NativeSocket.nativeSetRawOption | (socket_patch.dart:1251 )
  | at _NativeSocket.setRawOption | (socket_patch.dart:1182 )
  | at _RawDatagramSocket.setRawOption | (socket_patch.dart:2071 )
  | at MDnsClient.start | (multicast_dns.dart:137 )

There are a few options:

Make OSError implement Exception. Since Exception is an empty interface, this should be non-breaking.
Make OSError implement Error. This could make sense even though it won't help me solve my problem. It might also be breaking.
Don't let OSError escape the Dart SDK. Always wrap it in something else first, like FileSystemException, SocketException, ProcessException, etc.

@a-siva @rmacnak-google @ZichangG

I prefer option 3 over the other. Exceptions or errors need to be informative. OSError only gets errorCode and doesn't contain any context information on where this happened. This is less helpful than receiving with FileSystemException, SocketException and ProcessException.
However, this will lead to a breaking change in setRawOption and getRawOption. Both expect OSError to be thrown.
I also looked at implementation. Places that throw OSError are mostly in socket and process.
@lrhn Any opinion?

A component of option (3) would also be ensuring that the issue is not reintroduced.

One option to avoid breakage is to figure out which exception setRawOption/getRawOption should be throwing, then throw an implementation of that exception which also implements OSError. Anyone already catching OSError will keep working, and they can migrate to the proper exception.
If OSError isn't intended to be exposed, then maybe do a breaking change to remove it (move it to the appropriate patch file or make it private or something similar). That will definitely make everybody not see it. (Can't stop you throwing it, but that's what unit tests are for).
For the record, I also recommend against doing on Exception. Exceptions are intended to be captured and handled, and you can't properly handle it if you don't know which exception it is. Not catching a documented exception is, arguably, as big an error as doing something which throws an Error. I guess I'm saying: don't feel too good about on Exception catch (e) over catch (e). If you really are blindly catching all uncaught throws, because you are framework code wrapping user code, then treat uncaught exceptions the same way you would uncaught errors. Nothing wrong with that—if you are framework code. Users should catch the exceptions.
(Also, not all thrown objects need to implement Exception or Error. You should treat all non-Errorsas exception objects, so what you want might betry { ... } on Error { rethrow; } catch (e) { everything else }`. Might be overkill in practice, though.)

@lrhn In the flutter tool case, we want Errors to go to crash logging, because it means that we messed up. Exceptions are nearly always from dart:io and indicate a transient or end-user error. When we know how to advise the end-user, we do so. It's all case-by-case. Explicitly catching every possible exception in every catch, as well as the pattern you recommend (try {} on Error {rethrow;}...) are both too verbose, and not enforced by any lint. on Exception is short and enforced by a lint.

Exceptions are intended to be captured and handled, and you can't properly handle it if you don't know which exception it is.

That an exception is an Exception and not an Error is often sufficient information for the handling we need to do. TBH, I do feel pretty good about it, because I'm finding lots of bugs in our code, and it's a simple rule for folks to follow going forward.
Dart exceptions can be any object, but it would be nice for the SDK to follow some convention. At least in dart:io it looks like OSError is the only exception not following the apparent convention.

@zanderso
Flutter analyzer complained with this change. Perhaps this file needs to be updated accordingly.
packages/flutter_tools/lib/src/build_system/targets/icon_tree_shaker.dart:191:7

Log: https://logs.chromium.org/logs/dart/buildbucket/cr-buildbucket.appspot.com/8884898989594943616/+/steps/flutter_analyze/0/stdout?format=raw


Is it possible to generate a project that contains multiple c files and can be compiled directly?

Just like a real engineer, you just enter a code function into it, and it will return an correctly project code, instead of code snippets. This will be exciting.

Hello @mpu2018 ,
We will eventually integrate OneCompiler
It looks to already support multiple files, so it might be a possibility.
Best Regards,
Jon


Make output of extracted files optional in ExtractFiles task

Environment
Azure Pipelines, Linux agent
Issue Description
Using the Extract Files task to extract a fairly large .tar.gz file (approx 140MB and 70,000 files) takes about 1:40 to complete.  I tried the exact same tar command used by the task in a custom Command Line task, except that I ommitted the v flag so that the files names were not output during the extraction.  The command completed in 0:20.
Given the huge speed gain here, it would be nice to have an option for whether the task should output the extracted files, which controls use of the v flag or the equivalent for other archive types.

This issue is stale because it has been open for a year with no activity. Remove the stale label or comment on the issue otherwise this will be closed in 5 days


Implicit Type Not Passing Through Generic With String Map

TypeScript Version:  3.5.3

Search Terms: implicit type passing string map generic
Code
// A *self-contained* demonstration of the problem follows...
// Test this by running `tsc` on the command-line, rather than through another build tool such as Gulp, Webpack, etc.
export interface ICrxModelEvent<T> {
    'change': ValueCallback<T>;
    'changeInternal': ValueCallback<T>;
    'changeExternal': ValueCallback<T>;
}

type ValueCallback<T> = (value: T) => void;
type ValidationCallback = (validationState: CrxValidationState) => void;

export abstract class CrxModel<T> {
    addEvent<K extends keyof ICrxModelEvent<T>>(type: K, callback: ICrxModelEvent<T>[K]) {

    }
}

export class CrxNumber extends CrxModel<number> { }

and then to test it...
let num = new CrxNumber();
num.addEvent('change', v => {});

Expected behavior:
param v is implicitly typed as number.
Actual behavior:
param v has implicit type of any, (which, as a side-note, in my case give a no implicit any error)
(and when I mouseover the addEvent call, it has correct implicit types listed for both 'change' and number)
Note that if I specify the type explicitly, it works, and v is correctly types as number:
num.addEvent<'change'>('change', v => {});

Additional Info:
The idea is basically an extension of lib.dom.d.ts:
addEventListener<K extends keyof HTMLElementEventMap>(type: K, listener: (this: HTMLInputElement, ev: HTMLElementEventMap[K]) => any, options?: boolean | AddEventListenerOptions): void;

The main difference being that I'm adding in an extra generic type onto the callback.

I am unable to reproduce your issue in either typescript 3.5.3 called from a terminal or in the playground. The type of parameter v is definitely inferred as number (otherwise the ASSERT in my example wouldn't produce an error).
Can you reproduce your issue in the playground and provide a link for us? Or can it only be reproduced within a specific environment?

There's no definition for CrxValidationState here and trying to simplify the repro doesn't display the problem described. Can you reduce this to something that can be demonstrated in the Playground? Thanks

@webstrand thank you for putting it in the playground! I have just opened your playground, and as you can see - I have hovered my mouse over v (without changing your example at all) and it is inferred as any.

@RyanCavanaugh ah sorry, CrxValidationState wasn't supposed to be there, I'll delete that out of the original post.

Sorry, I was reading the wrong part like a derp. I'll have a play and then get back to you :)

Sounds good, please unassign if you narrow it down to something, or close if it turned out to be not-a-bug 🙂

Sure thing - of note, I've pasted the playground example into my environment and it's correctly assigning number... so there must be another factor in my real code - an extra complexity...

Ok, so I've managed to get a cut-down version that has the problem on my machine...
https://www.typescriptlang.org/play/#code/PTAEEFQKgZwUwDYDMC0BjA9gOwC4EMBLLOAEylBLgFtsYcAnPHA7UDJUHACzlAAd6GAEYJqoJBgQIMAdxgA6RQCgQoACpw6nLgRighAT1D0ArlixEA5qAAGOGGhtss23pipU8WEigRE4ADTGTDz02l7agiaWXKBeGNxwYUImBAgknBiSoDAmaLF4egDiJgh8QQDqcEJ8eGgA1kFwOGjySkpEOElIdbwAkgDC9AAeALIYlAgAogBucLgAPGoAfKAA3kqgW6AA5PlelnA7AFygAGp4CCZwA5cIQnX1S8sA3Jvbe1wHcH24SViXE7nS7XW5SB4NZ5vba7fZYQ5TYZdegAhBAi5XG53CFPFbQj4zS4EEhMI6nDHEpgsLBg+6PF6gVTuKjzHDaXSgPzENgmNleDIENkyDD0ep6TD0ehwNA4BAGJQAX3aOAMfF4GNB2Mez1AAF5QAAKQmY05qACUetWMwwxLeKrVwL8JOY2FpOL1huNlJdWAAyvguqchFlRF4LbqrTaSG8lHghHRGDLQGgEIU9ENhgNsAxsmtQErY-GGHU2Sm06AM+NJjq4Ej5iR0yMs7hBAh1u8tngSCRZqyFgBpUC1rrePT1OAGdigQYjKuIXuLFbLA32uCnftBNBahqnGdjCbzuaL5YAbX7AF0LRsO-nFe0yzBG8MAHImKhCJJDuujiuzg8IBYsDfD96FWPMC1ENkgKoD1iBkX8X2ApIDTNN5oPkLseyPHADU+b4diCGZLXWfNUKUIA
However, in the Playground it works correctly! Ahhh! What's the difference? Same piece of code.
Playground is running 3.5.1 and I'm running 3.5.3, but surely that's not the problem.
When I comment out the 'validate' line in the ICrxModelEvent the problem goes away, but when it's included, the problem is present. But... not in the playground :(

Oh noes, how embarrassment...
So this problem first came up when I was running Angular 7, which uses Typescript 3.2.4. It presented both in Visual Studio 2017's intellisense, and in the Typescript compiler.
Yesterday I upgraded to Angular 8, which uses Typescript 3.5.3, and checked the problem. Intellisense still reported the problem, so off to github I went. Sadly I forgot that Visual Studio 2017 doesn't use the project's version of Typescript, but it's own weird internal version.
So, in short, this problem was fixed somewhere between 3.2.4 and 3.5.1, and is now only presenting in Visual Studio 2017. Sorry! Looks like I just need to figure out how to upgrade TS in VS 2017.
I shall close the case :)
Thank you for your help everyone!


Errors in adding a MongoDB connection

I have a MongoDB on my local machine without authentication. qstudio is requiring it

Hi @jrca025
I tried quite a few different combinations but none worked. Even once connected i was getting:
Command failed with error 59 sqlGetResultSchema
Looking at the driver page https://github.com/mongodb/mongo-jdbc-driver it says:
The MongoDB Atlas SQL JDBC Driver provides SQL connectivity to [MongoDB Atlas](https://www.mongodb.com/atlas) for client applications developed in Java.
See the [Atlas SQL Documentation](https://www.mongodb.com/docs/datalake/admin/query-with-sql/) for more information.

MongoDB Atlas SQL JDBC Driver - I guess they only want to provide JDBC access to Atlas users :(
I had a look at alternatives but they don't seem great either. Unsure what to do.
Is there another database tool you have seen solve this well?

I'll tag on here to say that a similar issue persists with using the Mongo Driver for Azure Cosmos DB. Using either a connection string (in the jdbc format) or manually entering everything times out for both options.


get_all_enterprise() fails

from attackcti import attack_client
lift = attack_client()
all_enterprise = lift.get_all_enterprise()
Traceback (most recent call last):
File "", line 1, in
File "\attack_api.py", line 203, in get_all_enterprise
enterprise_stix_objects[key] = self.parse_stix_objects(enterprise_stix_objects[key], key)
File "\attack_api.py", line 100, in parse_stix_objects
'created_by_ref': software['created_by_ref'],
File "*********\base.py", line 178, in getitem
return self._inner[key]
KeyError: 'created_by_ref'

Thank you very much @mvelazc0 . I fixed it with Version 1.7 . When the parsers were being applied to Software STIX objects, it was not taking in consideration that some fields do not have values anymore. ATT&CK updated their content and there were several changes that broker some of the parsers. It was an easy fix. Thank you very much. You can uninstall it and install version 1.7.

Test:
>>>
>>> from attackcti import attack_client
>>>
>>> lift = attack_client()
>>>
>>> all_enterprise = lift.get_all_enterprise()
>>>
>>> all_enterprise['techniques'][0]
{'type': 'attack-pattern', 'id': 'attack-pattern--65917ae0-b854-4139-83fe-bf2441cf0196', 'created_by_ref': 'identity--c78cb6e5-0c4b-4611-8297-d1b8b55e40b5', 'created': '2018-10-17 00:14:20.652000+00:00', 'modified': '2018-10-31 13:45:13.024000+00:00', 'object_marking_refs': ['marking-definition--fa42a846-8d90-4e51-bc29-71d5b4802168'], 'url': 'https://attack.mitre.org/techniques/T1222', 'matrix': 'mitre-attack', 'technique': 'File Permissions Modification', 'technique_description': "File permissions are commonly managed by discretionary access control lists (DACLs) specified by the file owner. File DACL implementation may vary by platform, but generally explicitly designate which users/groups can perform which actions (ex: read, write, execute, etc.). (Citation: Microsoft DACL May 2018) (Citation: Microsoft File Rights May 2018) (Citation: Unix File Permissions)\n\nAdversaries may modify file permissions/attributes to evade intended DACLs. (Citation: Hybrid Analysis Icacls1 June 2018) (Citation: Hybrid Analysis Icacls2 May 2018) Modifications may include changing specific access rights, which may require taking ownership of a file and/or elevated permissions such as Administrator/root depending on the file's existing permissions to enable malicious activity such as modifying, replacing, or deleting specific files. Specific file modifications may be a required step for many techniques, such as establishing Persistence via [Accessibility Features](https://attack.mitre.org/techniques/T1015), [Logon Scripts](https://attack.mitre.org/techniques/T1037), or tainting/hijacking other instrumental binary/configuration files.", 'tactic': ['defense-evasion'], 'technique_id': 'T1222', 'platform': ['Linux', 'Windows', 'macOS'], 'data_sources': ['File monitoring', 'Process monitoring', 'Process command-line parameters', 'Windows event logs'], 'defense_bypassed': ['File system access controls'], 'permissions_required': ['User', 'Administrator', 'SYSTEM', 'root'], 'effective_permissions': None, 'system_requirements': None, 'network_requirements': None, 'remote_support': None, 'contributors': ['Jan Miller, CrowdStrike'], 'technique_references': ['https://attack.mitre.org/techniques/T1222', 'https://docs.microsoft.com/windows/desktop/secauthz/dacls-and-aces', 'https://docs.microsoft.com/windows/desktop/fileio/file-security-and-access-rights', 'https://www.tutorialspoint.com/unix/unix-file-permission.htm', 'https://www.hybrid-analysis.com/sample/ef0d2628823e8e0a0de3b08b8eacaf41cf284c086a948bdfd67f4e4373c14e4d?environmentId=100', 'https://www.hybrid-analysis.com/sample/22dab012c3e20e3d9291bce14a2bfc448036d3b966c6e78167f4626f5f9e38d6?environmentId=110', 'https://docs.microsoft.com/windows-server/administration/windows-commands/icacls', 'https://docs.microsoft.com/windows-server/administration/windows-commands/attrib', 'https://linux.die.net/man/1/chmod', 'https://linux.die.net/man/1/chown', 'https://www.eventtracker.com/tech-articles/monitoring-file-permission-changes-windows-security-log/', 'https://docs.microsoft.com/windows-server/administration/windows-commands/takeown', 'https://docs.microsoft.com/powershell/module/microsoft.powershell.security/set-acl'], 'detectable_by_common_defenses': None, 'detectable_explanation': None, 'difficulty_for_adversary': None, 'difficulty_explanation': None, 'tactic_type': None}
>>>


Use profiles with VSCode server

The --profile option is not available when running the VSCode server. Using it gives a: Ignoring option 'profile': not supported for server.
However, if you launch the VSCode server and connect a VSCode client, the client itself can make use of a profile (through the UI).
Would it be feasible for the VSCode server to accept the --profile option, so that connected clients automatically make use of the provided profile?  Or is there a current workaround to get this auto-loaded-profile behaviour?

Profiles are controlled always by the client and hence this request cannot be supported on the server.

Profiles are controlled always by the client and hence this request cannot be supported on the server.

@sandy081 is there any workaround to have the client automatically load a desired profile located at a pre-determined path? Perhaps some URL parameter that is given when accessing the client URL. Or maybe by placing the profile in the default profile location?


[SPARK-28772][MLLIB] Update breeze to 1.0

What changes were proposed in this pull request?
Update breeze dependency to 1.0.
Why are the changes needed?
Breeze 1.0 supports Scala 2.13 and has a few bug fixes.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existing tests.

Test build #111095 has started for PR 25874 at commit be03b1f.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/16179/
Test PASSed.

Test build #111095 has finished for PR 25874 at commit be03b1f.

This patch fails build dependency tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/111095/
Test FAILed.

Test build #111100 has started for PR 25874 at commit 1851e0d.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/16183/
Test PASSed.

Test build #111100 has finished for PR 25874 at commit 1851e0d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/111100/
Test PASSed.

Merged to master. Thank you, @srowen .


Rename test classes to **/*Tests.java per JUnit 5 convention

If implemented, this would have to be an option. Some test classes call one another so a mass rename could easily break things.

Personally I do not need this feature at the moment. Happy to review a PR.

Project/repository will be archived (see #59).


[BUG]Error in reading json and csv files

Describe the bug
I find a chart with QA conversational retrieval chain cannot read csv and json files correctly.
To Reproduce
Steps to reproduce the behavior:

I use chatgpt to create 10 random transactions
put them into a csv file and json file
put it into a chart and ask 'how many transactions are there?'
It tells me 4

Thank you for developing FlowiseAI! It is really a useful tool and I hope the problem can be fixed soon.

@Studying-365 Thank you for informing. I will check it. can you share any screenshots?

@Studying-365 Thank you for informing. I will check it. can you share any screenshots?

Thank you for your reply! Please find the attached screenshot. I also tried other CSV files, and the error persists.

You will need to increase the number of K i.e. Context. The default value of context is 4 hence LLM is responding with 4.

@Studying-365 You will need to increase the number of K i.e. Context. The default value of context is 4 hence LLM is responding with 4.

Thank you so much for helping me solve the problem!


 Is it maybe DNS-Resolver specific?

Looks like it is resolving now, but there was definitely an issue - the first resolver attempt was google's DNS and the second is my local unbound with no upstream servers. Closing as non issue now, cheers!


Add HolyC

The compiler can be easily installed from https://holyc-lang.com/install

Add holy C essential


up should be stabilizing until we figure out what path we want to go with bindle options

Merged the CLI stability table but realized we should have the status of spin up be Stabilizing until we reconcile @itowlson's question about the bindle options.

Sorry this clashes with the heavy changes made this morning Re: table formatting |
Will ask this in the Spin channel and then see if we need to update this or not.


ID Issue

// Start NFC emulator with AID, cardUid, and optional AES key:
await NfcEmulator.startNfcEmulator("666B65630001", "cd22c716", "79e64d05ed6475d3acf405d6a9cd506b");
When i replace the UID with the card id that i read from the reader
The emulated card is completely different .
Any way of solving it
Thank you

Hi @heeshala
The NFC Emulator was written mainly for the Cidron reader (the AID is in the example):
http://www.civintec.com/info_22.aspx?itemid=35
their reader is using peer-to-peer mode to exchange data, which includes the encrypted card UID, the reader requires UID to be AES encrypted.
You can't emulate the UID you selected, the Android phone will always generates a random UID (that is the case for most Android phone).  The only way to send over a UID is via P2P mode, like the example.

Hi @heeshala
The NFC Emulator was written mainly for the Cidron reader (the AID is in the example):
http://www.civintec.com/info_22.aspx?itemid=35
their reader is using peer-to-peer mode to exchange data, which includes the encrypted card UID, the reader requires UID to be AES encrypted.
You can't emulate the UID you selected, the Android phone will always generates a random UID (that is the case for most Android phone). The only way to send over a UID is via P2P mode, like the example.

Thank you for the reply
I am on a project where i have 13.5MHz NFC cards which are readable by mobile phones as well they act as physical passes i only get the hexadecimal id from the card the thing is that i want to be able to store the card by shared preferences in the app and use the app instead of the physical card to be read by the reader. Just like a wallet. Is there any possibility for it. Its for aa project and its a great help

@heeshala  there is no problem for mobile app to read NFC card, even for iPhone.  But for emulate a physical card, there is another story, iPhone is complete out of the question, Android will generate random UID for secure reason.   If your physical
card reader work in P2P mode or it can other part of data instead of solely the UID for authentication, then that is possible that you can use your phone to act like a card.

How can I decode and get a value that determines that random id created belongs to which user?

Hi @heeshala The NFC Emulator was written mainly for the Cidron reader (the AID is in the example): http://www.civintec.com/info_22.aspx?itemid=35
their reader is using the peer-to-peer mode to exchange data, which includes the encrypted card UID, the reader requires UID to be AES encrypted.
You can't emulate the UID you selected, the Android phone will always generate a random UID (that is the case for most Android phones). The only way to send over a UID is via P2P modes, like in the example.

But how can the reader know the exact device/user that has tapped? Is there anything that can uniquely define the tapped device?

How can I decode and get a value that determines the random id created belongs to which user?

The reader is using peer-to-peer mode to exchange data, the phone can transmit the encrypted AES encrypted UID (in the payload) to identity a user.

Hi @heeshala The NFC Emulator was written mainly for the Cidron reader (the AID is in the example): http://www.civintec.com/info_22.aspx?itemid=35
their reader is using the peer-to-peer mode to exchange data, which includes the encrypted card UID, the reader requires UID to be AES encrypted.
You can't emulate the UID you selected, the Android phone will always generate a random UID (that is the case for most Android phones). The only way to send over a UID is via P2P modes, like in the example.

But how can the reader know the exact device/user that has tapped? Is there anything that can uniquely define the tapped device?

The reader can read the encrypted AES encrypted UID.  It support OSDP protocol.

So when the emulated UID is decrypted using AES, we can get the same exact UID we set during startNfcEmulator(). Because I am using ESP32 where I can encrypt/decrypt AES, PN532 as an NFC device allows peer-to-peer communication.

Yes, it is possible to use ESP32 and PN532 to read the decrypted emulated UID.  You can even customise the exchanging data format in processCommandApdu() method.  This package is mainly made for the CIDRON reader.


chore: Simplify CI, allow merging of releases to main

Issue #
Fixes #505
Fixes #620
Description of changes
Note: While in review, the following features are disabled/modified to simplify review & testing:

Release and CI workflows are set to trigger on this branch only so they can be tested & demonstrated.
The full release/ branch has not yet been merged back to this PR because it would severely clutter the diff below.
The release CI workflow does not yet commit changes back to its branch, for the same reason.
Release builds run with dependencies pointing to main instead of the last published version, else it would fail due to Endpoints 2.0 being merged since the last release.

All of the above will be re-enabled before this PR merges.
Also note: Seeing sporadic instances of this bug when running xcodebuild on Xcode 14.  Building with Xcode 13 instead of 14 until this issue is fixed.
Simplifies the Package.swift and the rest of the project so that main can contain the latest release, eliminates several dependencies from CI builds, and enables all CI tasks to be run from the desktop (or any other machine).
Specific changes:

Package.swift improvements:

The script that builds the Package.swift for this repo has been improved significantly, and is now the reference source of this package's manifest at all stages of the development cycle.
Package.swift can be generated to point to its dependencies using a published version, a branch, or a path.  This eliminates almost all need for special configuration on CI before dependency resolution, and allows devs to point to a local copy if desired.
Permits the latest release to be safely merged back to main since CI will no longer rely on special logic in Package.swift.
Eliminates the need for all hard-coded installation paths and CI-specific logic in Package.swift, allowing the generated Package.swift to be free of special configuration in all phases of the development cycle.
Developers can develop on the package either by using the "package edit mode" features provided by Xcode and Swift Package Manager, or the Package.swift can be generated with hard-coded local paths.
The separate codegen/Package.swift for protocol tests has been eliminated.  Instead, the main Package.swift may be optionally generated with protocol tests included.  (This is done automatically on CI before performing protocol tests.)

CI simplification:

Eliminates the dependency on CRT's master build script.
Eliminates the dependency on CRT's Swift build container.
Linux builds are performed on "stock" Apple Swift docker containers; no container library / maintenance is required.
Eliminates the need for AWS credentials in the build process.
iOS & Mac platform builds are now performed using xcodebuild for an actual platform-native build.
Build steps have been extracted into scripts, allowing actions performed on CI to also be easily performed on development machines or as upstream/downstream jobs by other projects.
Preserves the ability for CI to automatically check out & build with the same branch name on smithy-swift and aws-crt-swift as the current branch for this repo.  Developer may also specify smithy-swift and aws-crt-swift branch names for development (not for releases) in the versionDependencies.plist file.

Other:

The release CI action now builds the entire SDK and runs AWSClientRuntime tests before succeeding.  (This takes about 1hr 10mins on a Linux runner / Docker container, so I'm open to just running tests, or moving it to a macOS runner which has more CPU/RAM)
Moves instructions that were in Package.swift to the project's main README.md.
Moves README content that is relevant to integrators (i.e. just using the SDK & not code-generating) to the top of the file.
A couple .gitignore adds.

New/existing dependencies impact assessment, if applicable
No new dependencies were added to this change.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Thanks for putting this up, nice work.

Things that we need to do fix main and make the experience better

Rename release directory to Source
Create Tests directory at root that contains the test packages which includes protocol and service tests

While we do this, individual smithy-codegen projects in build directory must stay workable.

I am closing this PR but keeping the branch.  Rather than make all this change to the project & CI at once, we will do it on a more gradual basis.


Openhab2 ?

Hello,
great Job.
Is it planned to work with openhab2.
Here a sample output from http://<IP_ADDRESS>:8080/rest/sitemaps/homekit?type=json
{"name":"homekit","label":"homekit","link":"http://<IP_ADDRESS>:8080/rest/sitemaps/homekit","homepage":{"id":"homekit","title":"homekit","link":"http://<IP_ADDRESS>:8080/rest/sitemaps/homekit/homekit","leaf":false,"widgets":[{"widgetId":"homekit_0","type":"Slider","label":"Stehlampe","icon":"slider","mappings":[],"switchSupport":false,"sendFrequency":0,"item":{"link":"http://<IP_ADDRESS>:8080/rest/items/DIMMER_EG_WZ_STEHLAMPE","state":"0","type":"DimmerItem","name":"DIMMER_EG_WZ_STEHLAMPE","label":"Stehlampe","category":"slider","tags":[],"groupNames":["gEG_WZ","gBELEUCHTUNG"]},"widgets":[]}]}}

not sure if this is necessary as there is a homekit binding for openhab2

In the next weeks i will install openHAB 2 and then i try to integrate with this plugin.

As far as I know, openHAB 2 dropped WebSockets support, which this is using to get status updates from the openHAB server.  Some re-architecture to use long polling http requests will be needed.

Tom - Any updates on OpenHab2 support ? HomeKit doesn't support all devices, so it would be great if you could support OpenHab2

I'm also planning to use homebridge with OpenHab2 as the integrated OpenHab2 binding supports only limited types of devices. Any updates on this? Thanks

Are there any plans to support OH2?

I have just released a new repository with support for openHAB2.


Fix tests

@chopan123 please can i be assigned to this

heyyyy @chopan123 can i work on this? i will review the broken test and fix it, let know

hi

@chopan123 can I work on this?

Hello


Fixed parsing of quoted CSV without trailing NL

Thank you for your continued efforts.
Carl A. Wright


How getting all the sandbox output into html?

Hello,
I do not find (or  I do not understand) how showing in html all the output given in sandbox, including the result of write predicate .
For example with fCube prover that is at the moment running online via SWI here https://www.vidal-rosset.net/fCube/ , with  query decide((p=>p)).  I get in Tau Prolog sandbox the following correct output:

true

Input Formula:
F (p => p)
Sign Permanence tried,input formula equivalent to:
F (p => p);
F->,Main SWFF: F (p => p)
T p;
F 1;
1 search result = provable (fCube-4.1)

parsing query: decide((p=>p)) ok!

My problem is that, if I follow for fCube the example of program given by Phil Zucker here https://www.philipzucker.com/javascript-automated-proving/ , with this line of code:
function fillbox(e){     console.log(e);  document.getElementById("querybox").value = provable((${htmlDecode(e.innerHTML)} ), Proof).; }
I do not succed to get as output in my html page anything else than the first line of the output in the sandbox, that is "true", and all the yellow part of the sandbox that gives the output of write predicate does not appear.
I am sorry to ask this question at this place, but I find on the web neither Tau Prolog forum nor mailing list to get some help.
Best wishes,
Jo.

Try the return button.

Try the return button.

?

Try the return button. After the query.
If you didn't edit the query, and press return again, it will show more solutions:

Press the return button on your keyboard. After the query. If you didn't edit the query, and press return again, it will show more solutions:

I get "false", that's it. My problem is getting into a html prover all the solution that is given in sandbox, including the output of write predicate. I do not know the javascript code to get it...

In Tau Prolog, you can open a regular HTML element as a stream. I think the best option is to write the output directly from Prolog.

:- use_module(library(dom)).

output :-
    get_by_id(writeme, WriteMe),
    open(WriteMe, write, Stream),
    write(Stream, hello),
    write(Stream, world).

In Tau Prolog, you can open a regular HTML element as a stream. I think the best option is to write the output directly from Prolog.

:- use_module(library(dom)).

output :-
    get_by_id(writeme, WriteMe),
    open(WriteMe, write, Stream),
    write(Stream, hello),
    write(Stream, world).

Many thanks. So, I do not know if it is correct, but to get all the written output,  I have only addedd
get_by_id(writeme, WriteMe),     open(WriteMe, write, Stream),  in the body of the clauses for decide/1 (i.e. intDecide/3) , without forgetting :- use_module(library(dom)). at the beginning of the prolog program. But I get this error:

uncaught exception: error(existence_error(procedure,get_by_id/2),intDecide/3)

I add that Tau Prolog is called by:


fix build and exports

Made a few fixes to the build and exports:

I have copied the tsconfig.json settings in the repo you linked. Those are scattered across a few files so I put them all in one tsconfig.json.
I fixed the build settings so it generates both types and compiled files
I added the "exports" field in package.json in addition to main/types, which should be more futureproof
I switched from using .npmignore to allowlisting what files to pack using files in package.json
Made tests run correctly

In JS libraries is now good practice to export both ESM and CJS files, but you need a bundler for that and I also don't see that happening in the original package. So this should be good enough.
I've run this through both https://publint.dev/ and https://arethetypeswrong.github.io/ and got the all clear. You never know with JS libraries, but, hey, gotta start somewhere. I also don't know if there's anything specific to how AtProto folks do things that I may be missing.
One thing: the codegen script doesn't seem like it'd do anything? I'd remove that, but I leave it up to you :)

looks good to me! Just pull from my branch once more, I pushed a change to my branch before I saw this.
I'm going to remove the code gen script, I have to add instructions in my PR or a follow up on how to generate the types from our other Lexicon repo

I've run this through both https://publint.dev/ and https://arethetypeswrong.github.io/ and got the all clear.
Didn't know about this, it's super helpful!

....and done :D


[2.0]XML schema validation seems too strict

BC Break Report

Q
A

BC Break
yes

Version
2.0.1

--

Summary
After updating version 2.0. Getting exception for below line. It is working on previous version.
XML Schema validation throw an exception about this line
<field field-name="created" type="date" name="created">         <gedmo:timestampable on="create"/>   </field>
Previous behavior
There wasn't any error about that line in previous versions. We just return.
 if (@$document->schemaValidate(__DIR__ . '/../../../../../../doctrine-mongo-mapping.xsd')) {                 return;    }
Current behavior
In current version throwing exception.
if (! $document->schemaValidate(__DIR__ . '/../../../../../../doctrine-mongo-mapping.xsd')) {                                 throw MappingException::xmlMappingFileInvalid($filename, $this->formatErrors(libxml_get_errors()));  }
I don't know if it's possible, but maybe the validation should be less strict same like in ORM.

Refer: https://github.com/doctrine/mongodb-odm/issues/2095

Well, there's still no resolution in https://github.com/doctrine-extensions/DoctrineExtensions/issues/2055 for which we hoped. I believe the only tangible thing we can do is to let users opt-out from the XSD validation, but that's obviously a footgun.

Yeah. Can we do that if it is possible?

Sure, are you up for the implementation?

@malarzm  Can you check PR and let us know if we need to change anything.
https://github.com/doctrine/mongodb-odm/pull/2535/files

I wonder if that could be fixed with a composer script that generate a custom xsd depending on the required packages ?


Small UI Module Housecleaning

This is the version getting sent to QA
Just deleting some unused files and fixing an error in the main UI Module Scene caused by duplicate hand names.

Heyyyyyyy let's get this in; there aren't any code changes.  I just changed a tickbox in the example scene and deleted some unused assets.


Multiple stack exchange sites

Hey! first of all, really cool concept. I was wondering if you could extend this to wrap all stack exchange sites. For example it would be cool to be able to search among:

stackoverflow
serverfault
gamedev
blender
etc

If you go to network profile you can see all the stack sites a user has accounts in.
Or directly in the bookmarks tab you can see the bookmarks from all the sites combined.

I think this could add a lot of value

That's a good suggestion. When I have the time, I'll take a look into that.

I've added a dropdown to select the site to search. I'm considering adding an 'all sites' option in the future but I'm not sure as that would mostly be more involved.


Add 'renameFile' command to services

Here's how I think it would work:

First, the user renames a file normally using the file explorer in their editor.
Tsserver will pick up on this change and update the project (which will break a lot of imports to the old location).
After the rename, the editor will offer the user to update references to the file.
If the user says yes, it will then ask tsserver to get code edits, passing in the old  and new paths to the file.
Tsserver will walk all imports, see which ones are unresolved now and could have resolved to the old path (but don't since that's been moved now), and update them to the new path instead, and send these edits to the editor.
The editor will apply the edits to all files. (Hopefully with undo functionality if the user decides against renaming the file.)

This is a bit tricky on the tsserver side since we need to be able to figure out what unresolved imports could have resolved to. This meant I had to leave moduleResolutionCache around so we could look at the failed lookup locations for failed imports.
Fully expecting to need a lot of revision on this. CC @mjbvz for protocol review.

//cc @mjbvz

We also need to consider /// <reference path="..." /> comments. bonus point for tsconfig.json "files" entries.

Ok, I think this flow makes sense and the api looks good.
We may also need an api to check if any renames are necessary so we can avoid prompting the user in cases with loose files

@andy-ms the soruceFile for the tsconfig.json is stored in options.configFile.

FWIW: most IDEs I’ve used, asked me – before the file was actually moved: “Do you also want to update references?”
Naively, it seems like that could simplify the algorithm.

Is this supposed to work for renaming folders as well? It doesn't seem to in VS Code 1.24 / TypeScript 2.9.1.

Folder rename should be working in<EMAIL_ADDRESS>and next VSCode release.

Filed https://github.com/Microsoft/TypeScript/issues/24904 to track folder renames.


Allow facts be reloaded dynamically during a deploy

Is your feature request related to a problem? Please describe
When trying to deploy a server I need to wait until a container has finished what it is doing (in this case pushing container images to a registry). I know when this is done by checking if the container exits in a running state. I can only check this one time at the beginning of a deploy using:
host.get_fact(Command,
              'sudo runuser -l -s /bin/sh tinkerbell -c "podman ps --noheading -f name=images-to-local-registry"',
)

Which, maybe I am doing this wrong, but I also cannot inject variables, or environment variables into this fact, (so I cannot switch user or sudo in the fact, so I sudo in the command). That is an aside I guess. The main issue is that I want to run this in a loop so that I can tell when container no longer exists. It appears that this is not possible, at least, I have not been able to see how. Possibly this is not a good idea for some reason, or, maybe this should be written into an operation. Either way, I think that you should be able to dynamically reload facts/specific facts at any point during a deploy.
Describe the solution you'd like
Allow facts, or at least one particular fact, to be reloaded anytime during a deploy.  When trying to assess the state of something, say the status of a systemd unit, I want to be able re-assess the state of the systemd unit  in a loop so that I can make sure that an action has completed properly before continuing on with the deploy.
cc: @jmpolom @storrgie

Fantastic idea! This could be a simple host.reload_fact(...) that works like get_fact but ignores any cached value.

This has been released in v2.1.


wip: docs(falco): auditLog commands

Signed-off-by: Leonardo Grasso<EMAIL_ADDRESS>/kind documentation

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

Needs approval from an approver in each of these files:

falco/OWNERS

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

LGTM label has been added.  Git tree hash: 4bc4e59defca2030802e49a2c1c5aae743a20c08

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

Needs approval from an approver in each of these files:

falco/OWNERS [fntlnz]

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


[Question] Android build failure

i want to test this library i get error on the build time
FAILURE: Build failed with an exception.

* Where:
Build file '/Users/xxx/.pub-cache/hosted/pub.dev/media_kit_libs_android_video-1.0.6/android/build.gradle' line: 146

* What went wrong:
A problem occurred evaluating project ':media_kit_libs_android_video'.
> media_kit: x86_64 JAR verification failed.

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 951ms
Exception: Gradle task assembleDebug failed with exit code 1

It's likely that network connection had a fault in the middle of download.
You can do one of the following:

rm -r build\media_kit_libs_android_video
flutter clean

Please inform us if your issue gets resolved. You may see our CI runs to see successful builds.

I hope it worked.

thank you
my issue has been fixed


Implement send and receive function #32

Created with new branch after discussion with @Kratika0907 , have tried to not keep lots of changes in the main index page as per discussion and UI changes and warnings/error messages are added.
Still working on submit handler

@Kratika0907 @ankushdharkar , I have merged Kratika's code and have created this PR....The file changes includes both PR

@sumitd94  I have resolved all the requested changes

@Harshi7016  Please resolve the conflicts and we can close this. Sorry for the delay.


Add OTP 2 checkbox in deployment viewer

Checklist

[x] Appropriate branch selected (all PRs must first be merged to dev before they can be merged to master)
[x] Any modified or new methods or classes have helpful JSDoc and code is thoroughly commented
[x] The description lists all applicable issues this PR seeks to resolve
[x] The description lists any configuration setting(s) that differ from the default settings
[ ] All tests and CI builds passing

Description
This adds a checkbox to the DeploymentViewer component so that a user can select whether a jar file is an OTP 2.x jar. This is the quick and dirty way of implementing this and I'm open to other implementation ideas. This works in conjunction with https://github.com/ibi-group/datatools-server/pull/363.

Assigning back to myself to make deployments be able to include extra files (otp-config.json and walk-config.json).

Ready for a review again with custom file support for deployments.

Marking as Blocked until #641 is merged.

I added some more validation checks for custom files.

Assigning back for one last review.


Added Images to Physics Content

Related Issue
#12

Info about Issue or bug
There were no images present with the physics content

Fixes: #12
Describe the changes you've made
Added the icon images to all the physics content
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.

Screenshots

Original
Updated

Hey @Amit366  please check this out and do let me know if any changes are required

@Swastik1710 its nice but can u add any other image for thermodynamics. Its becoming similar to nuclear.

Done!
Please have a look @Amit366

@Swastik1710 send the pic.

Here it is.

@Swastik1710 why did u change statistical mechanics?? And for thermodynamics instead of calculations can u place some image like heating of rod or kinetic theory of gas but don't place calculations.

Ohh my bad.
I'll get back to you.

Is this one fine ?

Nice!!

I've pushed the code.
Please check it out .

@Swastik1710 yes it will merged today itself.

Hey @Amit366, I didn't get any google form link for this PR as I got for the PR #26.
Please have a look.

@Swastik1710 u can fill that google form only. But send separately for both. Today only I automated it with GitHub.

@all-contributors please add @Swastik1710  for contribution

@all-contributors please add @Swastik1710  for code


Issues found through editing Note Permission

I was trying to use LDAP authorization manner to login and looks like the Reader could edit the entire Note Permission which supposed to be read only.
For example I am UserA, I am the owner and writer of the Notebook, and set UserB as the reader. However, once UserB logged in, s/he could remove the owner (UserA) and becomes as the Owner of the Notebook.
Would you please help resolve?
Thank you!

Hi - if you have an issue, please report to our mailing list: https://zeppelin.apache.org/docs/0.7.3/development/howtocontribute.html#stay-involved

hmm? can you close this?


Typo for Enabled attribute (part of Updates element)

As bold line below:
Enabled attribute (part of Updates element)
Optional. The default value is TRUE if not specified.
If set to TRUE, Office will check for updates. If set to FALSE, Office won't check for updates, but the user can check for updates by going to File > Account > Update Options > Update Now.
Allowed values:
Enabled="TRUE"
Enabled="FALSE"

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

ID: 95a0b1ff-0498-505a-6024-b813980d310f
Version Independent ID: b8057930-b4bd-1304-17ce-5561f837e901
Content: Configuration options for the Office Deployment Tool
Content Source: DeployOffice/office-deployment-tool-configuration-options.md
Service: o365-proplus-itpro
GitHub Login: @jwhit-MSFT
Microsoft Alias: jwhit

@AneurinHsu . I could not able to find  any typo error in the following paragraphs
Enabled attribute (part of Updates element)
Optional. The default value is TRUE if not specified.
If set to TRUE, Office will check for updates. If set to FALSE, Office won't check for updates, but the user can check for updates by going to File > Account > Update Options > Update Now.
Allowed values:
Enabled="TRUE"
Enabled="FALSE"

Ah, I think I misunderstood it. Thanks!
VARADHARAJAN K<EMAIL_ADDRESS>於 2020年5月26日 週二 12:33 寫道：

@AneurinHsu https://github.com/AneurinHsu . I could not able to find
any typo error in the following paragraphs
Enabled attribute (part of Updates element)
Optional. The default value is TRUE if not specified.
If set to TRUE, Office will check for updates. If set to FALSE, Office
won't check for updates, but the user can check for updates by going to
File > Account > Update Options > Update Now.
Allowed values:
Enabled="TRUE"
Enabled="FALSE"
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/OfficeDocs-DeployOffice/issues/795#issuecomment-633802385,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APWUB6PS22BONIHIERE6L2DRTNBAJANCNFSM4NJFIOAQ
.


Does cutoff_len truncate long documents or splits based on the cutoff_len?

Reminder

[X] I have read the README and searched the existing issues.

Reproduction
Hello, I am tokenizing a large data of large documents to further pretrain Mistral. I am using a cutoff_len of 2048 (because it fits well the memory and the average seq length). Otherwise, I have many documents in the data that are more than 30k-40k tokens. I want to know whether these documents are split into slices of 2048 or they are truncated.
Is there a way to consider the whole document instead of this truncation?
python src/train_bash.py
--stage pt
--do_train
--model_name_or_path mistral7B
--dataset mydataset
--finetuning_type lora
--lora_target q_proj,v_proj
--output_dir demo_model
--overwrite_cache
--overwrite_output_dir
--cutoff_len 2048
--preprocessing_num_workers 16
--cache_dir cache_dir
--tokenized_path tokenized_data
Expected behavior
No response
System Info
No response
Others
No response

Interested. I'm in a similar scenario

During pre-training, we group the sequences and then split them into constant-length chunks using the cutoff_len parameter, just the same as the this document described: https://huggingface.co/learn/nlp-course/chapter7/6#preparing-the-dataset

During pre-training, we group the sequences and then split them into constant-length chunks using the cutoff_len parameter, just the same as this document described: https://huggingface.co/learn/nlp-course/chapter7/6#preparing-the-dataset

Thanks, can you please refer to this in the code? As Huggingface tokenizers use return_overflowing_tokens parameter to split input into chunks.
Couldn't find this in the code.

https://github.com/hiyouga/LLaMA-Factory/blob/b87f8f15194923392d1a1c3d3c617b463cdd8379/src/llmtuner/data/preprocess.py#L33-L41


Check .is_empty() for all occurences of insert_many

When the iterator passed to insert_many is empty, MongoDB will through an error. Alternatively, we could also match on the error returned by insert_many.

Another option which I explored in #566 is to use a chunks iter over the inserts. This will accomplish two things:

We will not try to insert an empty array.
We will not try to insert too many records at once.

Yeah, that's even better

This is solved indirectly by batching.


implement refresh button

fixes #153

Enable refresh button on player profiles
DotaCoach button not enabled yet.  @kennsippell can you set up your side so that a player link does not require a match ID to be passed?  The current component does not contain match ID data and I would prefer not to have to send it down the component tree.  Currently we send over player account ID and MMR.

Reviewers:
@albertcui
@mike-robertson
@kennsippell

Updating today or tomorrow. Sorry for the slow reply.


PyPI package has a problem

This is ukr-itn 0.1.3

Fixed in latest version 0.1.5
Thansk for poiting it out!


Templates: Ensure all resources have baseColor property

Feature Description

Not all templates' resources have a baseColor set.
Let's add it everywhere for consistency.
We should also add a test to cover that.
Here's a WIP branch: https://github.com/google/web-stories-wp/tree/add/templates-baseColor
Alternatives Considered

Additional Context

BlurHash is mostly useful for the media library, not the canvas or output. So not really relevant for templates.
Let's focus on base color for now. We could always add BlurHash later in a separate ticket if we have more use cases for it.

How did you add these base colors, manually?

Yeah I did those ones manually using some online tool, since in some templates there was just 1 missing blurhash so it was quick.
A quick search suggests that something like convert imagename.jpg -scale 1x1\! -format '%f\n%c\n' -colors 5 -define histogram:unique-colors=true histogram:info: or magick imagename.jpg -colors 1 -unique-colors txt: might work to get the dominant color via CLI.

Could we add a migration?

This is a one time thing.
A migration for adding basecolor would also run when opening an old story in the editor, making it unusable until the migration has finished. Not great.


Improve 'About' Page for Curriculum

** Describe the problem**
The About Page should provide all necessary information for users, but currently, it's a bit crowded. In general, it should be quick and easy to find & navigate to important info. This is a placeholder fix until we have UX help
Proposed Solutions
Curriculum Page

[ ] Add the curriculum menu on the left-hand side

About Page

[ ] Move the 3-colored part to be horizontal, not vertical.
[ ] Leave the 'Welcome to Launchpad', 'Expectations', and 'Principles sections as is
[ ] Make the 3 part say 'Cover the Prerequsites', 'Learn the Curriculum' and 'Explore Dev Tools', make it more obvious these are links
[ ] Embed just one video underneath (Palo Alto or Paris) under 'Colo Week Experience' title

Hi @walkerlj0
I decided to do some few things around this issue, you can check out if you prefer it this way
https://launchpad-psi.vercel.app/about/


Apply 'no-loc' entries across the doc set

Fixes #18076

I was able to get away with a simple RegEx-based find-'n-replace.
There were existing entries for "Identity" and "Let's Encrypt," so I preserved those in the new list.
I hope 🙏 that "Identity" in quotes will only address the capitalized word.
From this point forward, a plain old find-'n-replace will be a snap to maintain the list.

Really nice work!

Luckily, it only took a couple of minutes. The RegEx find-'n-replace is powerful ... if one can figure out the crazy RegEx syntax! 😵😄

@guardrex , great you even caught oddities like the UID reference.  Well done.

@scottaddie I'm going to go ahead and get this in ... It's creating 💥 merge conflicts 💥 all over the place! :smile: Message me if I need to make changes.

@JasonCard, this larger fix across the en-us doc set was originally triggered by this specific request for a correction tot he Polish version:  Loc Issue: 18076.   We have some product unit PM's and customers that would like to be notified when that Polish version is updated.  What would be the timeline for this fix to appear in the localized version of the docs, specifically for that document in Polish?   Thanks again for your guidance.

@wadepickett Once the changes are merged to the live branch, our automation platform should detect the changes and machine translated content should be updated and published within 1-2 days.

Cool!  Thanks Jason!


D3 new testing

Product Version(s):

Issue:

Link to docs preview:

SME review:

[ ] An SME has approved this change.

DOCS review:

[ ] A member of the docs team has approved this change.

Additional information:

Merge checklist:

[ ] Deploy preview inspected wherever changes were made
[ ] Build completed successfully
[ ] Test have passed

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.3 out of 4 committers have signed the CLA.:white_check_mark: ctauchen:white_check_mark: vara2504:white_check_mark: ronanc-tigera:x: dependabot[bot]You have signed the CLA already but the status is still pending? Let us recheck it.


While setting up defaults for systemIntegrations in setupSystemIntDefaults function, also set up defaults for intercom

https://github.com/Shippable/pm/issues/9729

verified on rc


bug: vdi-wrapper script does not close sometimes..

hey, not sure why.. i will try to attach to the process and see what's happening.. not sure if it is centos only.  vdi-wrapper doesn't close sometimes.  doesn't happen always... after running server for a day or so a few come up.
heres a screenshot.

this is hard to reproduce... i killed off the vms before tracing the problem.  i'll open it when i make it happen again


quickstart gives no hint that "application.py" is magically considered to be the default for FLASK_APP

The quickstart gives no hint that application.py is magically considered to be the default for FLASK_APP.
I'm not the first to point this sort of thing out - see #16676 where you just blew off the question. You should indicate what is magic peculiar to Azure, and where to look in the docs to do more customization.

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

ID: 0077a811-b0b7-93b1-80b0-a4d45615ef4b
Version Independent ID: cc95198d-5560-2bf1-09ae-b8d210e45f2d
Content: Create Python app on Linux - Azure App Service
Content Source: articles/app-service/containers/quickstart-python.md
Service: app-service-web
GitHub Login: @cephalin
Microsoft Alias: cephalin

@nealmcb, Thanks for bringing this to our attention and your feedback on this item. I have assigned the issue to the content author and your feedback has been shared. The author will review further and update the document as appropriate.

Just copied and pasted from the configure python article
 for easy reference (I had been bugged by this for a while and believed others may experience the same):

For Flask, App Service looks for a file named application.py or app.py and starts Gunicorn as follows:

@yychenca, Thank you for highlighting the statement from the doc here, it would benefit the users with a similar question.


[APK Management Server to Agent Client Communication] Implement GRPC server in management server

Description:
A GRPC server implementation is needed in the management server to receive CP agent grpc calls.

PR: https://github.com/wso2/apk/pull/115


Fixed plugin to resolve paths

Codecov Report

Merging #5 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master     #5   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           2      2
  Lines          47     50    +3
  Branches        4      6    +2
=====================================
+ Hits           47     50    +3

Impacted Files
Coverage Δ

src/plugin.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Updated examples to be compatible with self-hosted

Uses the new asRange calling convention from https://github.com/floooh/sokol/pull/710
Change Mat4 to extern struct as arrays in packed structs are no longer supported

All the examples (except for instancing.zig) run for me on self-hosted (Windows, x86_64) in both opengl and directx. I did notice the point size seems different between dx and gl for sgl-points, but this seems true with -fstage1 as well.
The instancing.zig example crashes on self-hosted with a stack overflow, at the start of frame(). Debugging this, it seems that it tries to pass ~40M to ___chkstk_ms, which comes from lib/compiler_rt/stack_probe.zig. I recall seeing some activity about this so this may just be a self-hosted bug. I only have access to Windows, so it may not affect other platforms. This may also be related to the issue with structs being copied on to the stack (https://github.com/ziglang/zig/issues/12638) but I'm not familiar with how the stack probe is supposed to work, so I can't say without more investigating.

My own zig implementation of the sokol instancing example is is actually what triggered this issue: https://github.com/ziglang/zig/issues/12568
If you do a similar workaround to the one I posted there you can probably get the instancing to work on stage2 too.

For reference, this is what my update loop looks like now on my side (I don't have the data wrapped in a state struct and just use the sokol c code directly, so maybe it won't work here):
    {
        var i: u32 = 0;
        // update particle positions
        while (i < cur_num_particles) : (i += 1) {
            const particle_pos =  &pos[i];
            const particle_vel =  &vel[i];

            particle_vel.y -= frame_time;
            particle_pos.* = particle_pos.add(particle_vel.scl(frame_time));
            // bounce back from 'ground'
            if (particle_pos.y < -2.0) {
                particle_vel.y = -1.8;
                particle_vel.y = -particle_vel.y;
                particle_vel.x *= 0.8;
                particle_vel.y *= 0.8;
                particle_vel.z *= 0.8;
            }
        }
    }

Ah, thanks! Indeed that workaround works here as well, just added the change. Cheers!

Ok, I have merged this PR manually into a new sokol-zig branch 0.10.0:
https://github.com/floooh/sokol-zig/tree/zig-0.10.0
...once Zig 0.10.0 is officially released, I'll merge that back into master, and (hopefully), this PR should then automatically resolve too.

...and it's in master now.

awesome!