Plot CMIP6 data in FOREST

Work by NCAR has made some of the CMIP6 data available through an intake catalogue, partly at a hackathon in October 2019
Hackathon: https://cmip6hack.github.io/#/
intake-esm: https://github.com/NCAR/intake-esm
This issue aims to implement a module to visualise remotely stored CMIP6 data. This can become a template for visualising remote datasets specified through an intake catalogue.

Development branch: https://github.com/informatics-lab/forest/tree/ems-intake

At 4033626 (folloowing the hackaton), the following functionalty is in place:

read metadata from intake catalogue
get selected data through intake and zarr.
plot data for one experiment/institution/member combo for air temperature.
tasks remaining:
selection of variable and updating plot
docstrings & code styling

@stevehadd great job! This looks very promising. I'm looking forward to the pull request

I found the issue described in #207 while testing this code. There is temporary fix in the code in the PR #195 .

The PR #195 has now been merged, so we can close this issue.


APT whitelist request for diffstat

The 'set(uid|euid|gid)' pattern only matched on text data (patches) used for unit testing diffstat. There are no occurrences in diffstat's source code.
This is thus a false positive, and the package should be accepted.

Hi,
I'd like to +1 that issue, the warning being a false positive, it would be awesome to have diffstat! :)
Thanks in advance.


Center community partners on view

This pull request fixes #177 where the component's content is off-center.
Centering
The changes revolve around a new single-file component, PartnersNew.vue. The layout and styling have been preserved from the original, although the new component uses scoped native CSS for now to prevent any CSS conflicts with styles that I'm not aware of.
Additionally, Home.vue has been modified slightly so that the Partners component does not rely on absolute positioning within the hero section. This helps to simplify the HTML layout.
Media queries have been added to the component style to allow the partners list to wrap in devices with narrower viewports.
The original functionality of the partners list items has been maintained, including filtering by software/partner as well as the hero banners changing on click.
@oeway and maintainers, I hope this helps improve the site layout a little bit.

Hi @junelsolis Thanks for looking into this!
I have just accepted the preview on netlify, so you can use the preview link too.
Here are my comments

Also noticed that the text are displayed now, but the font size is too small. To fit the style of the page, I would either hide the text (but show as hover text) or make it at least 1rem.
Maybe also reduce the white space a bit before and after the partners section.
It seems the "Explore the Zoo" button now breaks.
Before accepting the PR, I would rename PartnersNew.vue to Partners.vue since we don't need to keep both.
I do like that it's now centered on large screen, however, on small screens (width: 500-100 pixels) the icons are truncated:

@oeway could you review this?

The issue is not fixed completely, more work is needed, let's close this PR for now. But please feel free to reopen in the future.


Feature Request: Right-click duplicate

It would be awesome if right clicking on a resource on the navigation tree would include an option to duplicate the block, toolbox, or workspaces.

Dupe of #210


Fix https://github.com/forbole/cosmos-rust-wallet/issues/47

Goal
This is to fix the compiling failure issue reported here.
Extentions
There are some warnings. Anyone are free to submit fixes.

Hi @tlkahn thanks for the contribution.
I don't get why you are conditionaly compiling some functions with the with-bitcoin feature, on my machine just bumping the version of the bitcoin crate to 0.27.1 is enough to compile the crw-wallet crate.

Hi @manu0466, thanks so much for getting back to me. My apologies for submit a bad branch. Please kindly see this #64.
Other than bump up the version, there are two minor updates. It seems that bitcoin slightly changed their API on PrivateKey and PublicKey.

@tlkahn I had see, I have changed some part to fix the problems. Tomorrow I will merge the changes


Useful page but not easy to be found

Try to Google for "<Complier Include=", you will see that it's impossible to find this page which explains the meaning of this syntax.

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

ID: 2d4c1957-1da1-073e-e5d2-b88dbb6e4f0c
Version Independent ID: da079116-5827-d486-271b-9e2cb10bf063
Content: MSBuild Items - Visual Studio
Content Source: docs/msbuild/msbuild-items.md
Product: visual-studio-dev15
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

Thanks! I can make some SEO changes for this topic.


[2201.5.x] Add JSON parser support for exponential values

Purpose
$subject
Fixes #41334
Approach
Added logic for creating numerical Ballerina values if the string contains a valid exponential value.

If the string is a valid value within the range of the float type, the parser creates a Ballerina float value (uses the similar validations done in the parser)
Else, creates a new decimal value. (can throw runtime error for invalid decimal values)

Samples
In runtime API usage,
Object obj = JsonUtils.parse("0e-18"); // throws error earlier

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[x] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Shall we add some negative test cases with invalid hexadecimal strings and exponents?


[Bug]: ENOENT: no such file or directory, open '/Users/-/Prylar/Wizdo/TGG/Frontend/node_modules/react-photo-sphere-viewer/src/index.tsx'

What happened?
A bug happened!
What should have happened?
A bug happened!
Code
Compiled with warnings.
Failed to parse source map from '/Users/-/Prylar/Wizdo/TGG/Frontend/node_modules/react-photo-sphere-viewer/src/index.tsx' file: Error: ENOENT: no such file or directory, open '/Users/-/Prylar/Wizdo/TGG/Frontend/node_modules/react-photo-sphere-viewer/src/index.tsx'
Search for the keywords to learn more about each warning.
To ignore, add // eslint-disable-next-line to the line before.
WARNING in ./node_modules/react-photo-sphere-viewer/dist/index.js
Module Warning (from ./node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from '/Users/-/Prylar/Wizdo/TGG/Frontend/node_modules/react-photo-sphere-viewer/src/index.tsx' file: Error: ENOENT: no such file or directory, open '/Users/-/Prylar/Wizdo/TGG/Frontend/node_modules/react-photo-sphere-viewer/src/index.tsx'
webpack compiled with 1 warning
Sandbox Link
No response
Library Version
6.0.0
What operating system are you using?
macOS
What browser are you using?
Firefox
Logs
No response
Interest to fix the bug

[ ] I would like to fix this bug!

https://github.com/Elius94/react-photo-sphere-viewer/issues/48
Hi, check this. it seems the same :)


bug: upgraded Azure API version

Please ensure your pull request adheres to the following guidelines:

[x] For first time contributors, read Submitting a pull request
[x] All code is covered by unit and/or runtime tests where feasible.
[x] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[x] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[x] All commits are signed off. See the section Developer’s Certificate of Origin
[x] Provide a title or release-note blurb suitable for the release notes.
[x] Thanks for contributing!

Fixes: #23083

Upgraded Azure API version to 1st august 2021 in order to save it from retiring.
details here

Signed-off-by: Goverdhan Biradar<EMAIL_ADDRESS>

thanks, change makes sense. Can you squash the two commits together?

@goverdhan07 The bot doesn't recognize the current format for your sign-off line. I would suggest either using git commit -s and/or make sure that the email address is wrapped in angle brackets (eg <foo@bar.com>). Additionally, please rebase + squash the commits together. Cilium doesn't take merge commits into the tree.

Switching to draft given missing sign off from last week.

@goverdhan07 I would suggest using a rebase + squash approach and update the commit message to address the DCO requirement as the maintainer's little helper is suggesting above.

Closing in favor of https://github.com/cilium/cilium/pull/28311


My own buy/sell orders matching each other ?

Newly created asset , submitted sell order than buy order . My intention was to leave orders open and try something else but they match each other is that normal ?

If the order prices matched then yes that would be normal. Do you have some more details please?

thats been resolved by creating a spread as much as a satoshi .


Release notes for v0.5.1

Please add any relevant release notes and other config-related things that I missed.
This PR also fixes the layout of the tutorials cards homepage (the text currently needs to be a particular length or else the content will overflow the cards --- did no one notice this before?)

Does this need updates for #414?

Does this need updates for #414?

No, although it was a lot of code, it's not anything exciting for end-users


The count() function should return an integer and not a string

A simple cast fixes the problem.
Test added as well.

great :+1: thanks!

Thank you! :)


Unable to connect to site

Expected Behavior
Fetch and list all the hmangas in home page
Current Behavior
Bottom bar error shows "Unable to connect to site"
Context

Unable to access the contents and use the app
Your Environment

Version used: 3.0.1

Operating System version: 13

NHentai link (if relevant):

Yeah same here


support for PAT in hasura-auth

Motivation
Today when a user needs to interact with a hasura-auth/hasura backed service outside of a frontend application they have two options:

Share the admin secret
Implement something customized (like we did ourselves for the cli)

Goal
The goal is to be able to generate tokens to authenticate a backend service or tool as a user. For instance, a cli tool, or some backend service that needs access to hasura.
Potential implementation
A potential implementation could to simply rely on the existing refresh-tokens functionality. In this scenario, a tool like the cli could:

Authenticate the user as normally done
Have the user request a PAT
Store it

At this point the CLI could just reuse that refresh token and act on behalf of the user.
A similar workflow could be used to authenticate backend services:

An admin could create a "service account"
Give it a role that has permissions to do what the backend service needs to do
Create a PAT/refresh_token
Configure it in the backend service

Requirements
For this to work I guess we need:

A mechanism to generate refresh tokens with a longer timespan, i.e., 1 year, 2 years, 99 years....
A mechanism to list them and invalidate them

Security implications
The benefit of this mechanism is that the PAT needs to be authenticated every time to generate an access token so it can be revoked anytime.

@szilarddoro Is it possible to list and invalidate the tokens as well? I can't find any documentation about that.


Run tests with Java 21

Once starter is released for 4.2.0 and we can generate Java 21 projects, this can be merged
Currently it is set up to use 4.2.0-SNAPSHOT as the version target.
Excluded lambda and function guides which only work on 17.
There is one odd change that was required.  When running Kapt projects, we needed to kill the Kotlin Compile Daemon as it holds on to the old properties it was started with, and doesn't notice that it needs to update to include the --add-opens for Kapt to work.

We can't --stop the gradle daemons (as this would kill the build itself)
We cannot use --no-daemon for all the builds as it takes AGES to run all the tests
We cannot use --no-daemon on just the kapt builds, as the Gradle daemon finds the old Kotlin daemon.

So I went with find the kotlin daemon and kill it before and after Kapt builds

@sdelamo should we switch to just testing 21 on a nightly basis?

@sdelamo should we switch to just testing 21 on a nightly basis?

yup.
great job making the java 21 transition


Add MLContextMLogregTest

Tried out by removing the comments of the Lines and made changes to the code.

Currently getting the following error

Debugging the code to get rid of the error.

Issue Solved.

Need to make a small change in the DML script.
Name of the Output variable of DMLscript and java output variable was different, thus was getting an error.

Thank you, @omkarghugarkar007 . I will merge this after a deep pass.

As for the next task, have a look at https://github.com/j143/systemds/commits/GLM_MLContext

We will work on GLM
This script is a bit tricky and long (try your best!)


fix dtmutil relevant ci lint error

Codecov Report

Merging #164 (a222fae) into main (de1355d) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #164   +/-   ##
=======================================
  Coverage   96.74%   96.75%
=======================================
  Files          44       44
  Lines        2153     2155    +2
=======================================
+ Hits         2083     2085    +2
  Misses         38       38
  Partials       32       32

Impacted Files
Coverage Δ

dtmsvr/storage/sql/sql.go
100.00% <100.00%> (ø)

dtmutil/db.go
96.36% <100.00%> (ø)

dtmutil/utils.go
95.58% <100.00%> (ø)

dtmsvr/config/config_utils.go
96.36% <0.00%> (+0.13%)
: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 de1355d...a222fae. Read the comment docs.

已更改


feat: support async plugins

Description
Based on discussion: https://github.com/vitejs/vite/discussions/8509
Support config.plugins: (Plugin | Plugin[] | Promise<Plugin | Plugin[]>)[]
Additional context
Turns out deep flattening an array of promises only take 3 lines 😄
Side note:
@poyoho it looks like config.worker.plugins isn't flatten and being handled with falsy, apply prop etc like:
https://github.com/vitejs/vite/blob/c5c424a93cf97e254dae46a35ad6de46ec350a16/packages/vite/src/node/config.ts#L333-L343
Is it intentional?

What is the purpose of this pull request?

[ ] Bug fix
[x] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.

Marked as draft. There’s one more case in asyncFlatten that I’ll fix tomorrow.

I think config.worker.plugins needs to flatten and handled apply prop too. 😁

We talked about the PR in today's meeting and we decided to move forward. @bluwy once you resolve the conflicts you can merge it 👍🏼

Love it! 🔥


Create alpiz82.yml

Hey @alpiz82 so sorry I didnt see this sooner I somehow overlooked this, not sure why this pull request got stuck open, but it looks like you were able to get your badge so I assume everything is fine, but do reply and let me know if you need any assist with anything. Thanks!

No worries thanks ...what badge are you referring to?
On Thu, Jul 8, 2021 at 10:24 PM Arthur Fewell @.***>
wrote:

Closed #146
https://github.com/ModernAppsNinja/modernappsskills101_ms4043/pull/146.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ModernAppsNinja/modernappsskills101_ms4043/pull/146#event-4998253249,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQVEEJ2LNAU5RNWFD56YSFTTWZ2WVANCNFSM47VLGGZQ
.


Generic Soufflé detector

It is possible to add a "generic" Soufflé detector to enable users to create a new detector without writing extra TypeScript code.
The idea is to create a library of constraint generators which provides API to create pre-defined Soufflé facts. For example, such a library might contains constraint generators from the readOnlyVariables detector that adds facts varUse, varAssign and varDecl to a any detector if requested.
Having this kind of a library, the user can implement a new detector in two steps:

Define a small configuration in TypeScript that loads the required constraint generators.
Add a custom Soufflé rule using the bindings library.

After that, they can simply run a new detector without any interaction with IR in their code.
Here is an example:
export class TestDetector extends CustomSouffleDetector {
  // Load the required constraint generators
  constructor() {
    this.loadConstraints("varDecl", "varUse", "varAssign");
  }

  // Add custom Soufflé rules that uses the facts generated by constraints
  addCustomRules(program: SouffleProgram) {
     // readOnly(var, func) :-
     //   varDecl(var, func),
     //   varAssign(var, func),
     //   !varUse(var, func).
     let rule = SouffleRule("readOnly", "var", "func");
     rule.addCondition("varDecl", "var", "func");
     rule.addCondition("varAssign", "var", "func");
     rule.addCondition("!varUse", "var", "func");
     program.addRule(rule)
  }
}

This could be implemented later when developing new detectors. Actually, it will be a kind of refactoring performed when adding the same functionality to new detectors.


chore(package.json): add type module

Related Issues
vite 4 required type module

Description

Hi @NaotoshiFujita 👋 Without this change, I am currently blocked from compiling my project with Nuxt 3. Is there some blocker for the merge, or some other consideration? Thanks for your great work on this lib!


Make storage version able.

For #4, when storage structure was changed, the stored data may cannot be used. So make them version able. @mrpachara

commit cc15e97 add this feature.


Time Zone Support

It would be great if there was time zone support. E.g, I could input "9 am - 3 pm Eastern Time". Helps with situations where I have data from different time zones.

^^^


Separate poller token pools for tasklists with isolation

What changed?
Separate poller token pools for tasklists with isolation

Why?
To allow concurrent forwarding of pollers from different isolation groups. This is to help increase concurrency because pollers from isolation groups are isolated.

How did you test it?
new unit tests and manual test in staging2

Potential risks

Release notes

Documentation Changes

Pull Request Test Coverage Report for Build 01887e63-0ed6-494c-9357-817207827e5a

38 of 38 (100.0%)  changed or added relevant lines in 2 files are covered.
41 unchanged lines in 10 files lost coverage.
Overall coverage increased (+0.05%) to 57.311%

Files with Coverage Reduction
New Missed Lines
%

common/persistence/sql/common.go
2
50.65%

common/persistence/sql/sqlplugin/postgres/db.go
2
85.0%

service/history/execution/mutable_state_util.go
2
35.95%

service/history/handler.go
2
47.15%

service/history/task/transfer_standby_task_executor.go
2
87.34%

common/cache/lru.go
3
90.73%

common/types/history.go
4
45.35%

service/history/historyEngine.go
4
68.45%

service/history/task/fetcher.go
4
91.24%

service/history/execution/mutable_state_task_refresher.go
16
62.03%

Totals

Change from base Build 01887d1f-620f-4f23-8407-5eaf5c239cf4:
0.05%

Covered Lines:
87098

Relevant Lines:
151974

💛  - Coveralls


ResNet18/34

I tried to use ResNet18/34 instead of ResNet50, and can't it rises this error. It works fine for ResNet50, but not for 18/34. Any suggestions how to solve this error please. RuntimeError: Given groups=1, weight of size [256, 256, 1, 1], expected input[5, 64, 56, 56] to have 256 channels, but got 64 channels instead

Hi
Because my code is not implemented for ResNet18 or ResNet34, it might cause errors.
I'm not sure solving the current error can make the code works for ResNet 18 / 34.
Anyway, I think your error made by this line
https://github.com/clovaai/overhaul-distillation/blob/76344a84a7ce23c894f41a2e05b866c9b73fd85a/ImageNet/models/ResNet.py#L176
I never consider ResNet with basic blocks. It returns channel size of bottleneck block.
So, you have to put a channel setting of BasicBlock ResNet such as ResNet18 & 34.
For example
    def get_channel_num(self):
        if isinstance(self.layer1[0], Bottleneck):
            return [256, 512, 1024, 2048]
        elif isinstance(self.layer1[0], BasicBlock):
            return [64, 128, 256, 512]
        else:
            raise NotImplementedError

I hope it helps you to run the code for your setting.

Thank you so much, this actually solved the issue.


[DOC] cucim.skimage.measure.regionprops wrong naming of intensity_{min|mean|max} properties

Report incorrect documentation
Location of incorrect documentation
https://docs.rapids.ai/api/cucim/stable/api.html#cucim.skimage.measure.regionprops
Describe the problems or issues found in the documentation
the properties intensity_min, intensity_mean and intensity_max do not exist, but exist as min_intensity, mean_intensity and max_intensity
Steps taken to verify documentation is incorrect
Example code using only the alternative naming works
Suggested fix for documentation
It would be better to stick to the intensity_{min|mean|max} naming convention, because that is consistent with the skimage regionprops at https://scikit-image.org/docs/stable/api/skimage.measure.html#skimage.measure.regionprops. That implies a bug fix rather than a change in documentation.
https://github.com/rapidsai/cucim/issues/430#tasklist-block-deb0c5f0-0f47-4780-ad28-b235b8c13056

Oops, I got things mixed up. In fact, there is no issue with the naming, you can even use both conventions. The original issue was with a CPU only skimage script. So, no action needed.

sorry for the confusion


Trusting System.currentTimeMillis()

We are currently using System.currentTimeMillis() for the values first_visit and previous_visit.
If the user changes the system time, gets a NTP update or wtf else, this value can jump leading to freak values in the Piwik Dashboard.
Basically this is an issue because the user has to send these values and we have to trust them.
I currently see no better solution for it than just letting the freak values drown in the data pool as they probably won't tip the scale.

@dotsbb Can you add the tag "help wanted"?

This can be tackled in two different ways, each has its own merits.

When an user opens your end application, make a call to your own server for knowing the current time or use an API from https://timezonedb.com/api . If the time set in the device is wrong, show a pop up saying the phone time is set wrongly and ask the user to update it. Whatsapp follows this method and unless the user updates their time, the user is not allowed to use the app further.

The second approach is to use the time API and fetch the actual time instead of using System.currentTimeMillis() to get the current time. But the issue over here is you are making REST calls each time you want to fetch the current time. One quick hack to avoid several REST calls in the app is to fetch the currentTime from a server or https://timezonedb.com/api on the loading of the app, note down the offset from the server time and the local time. Then using the offset, update your System.currentTimeMillis()

You can use any of the two approaches above to populate clean data in your dashboard. Without making any changes in the piwik servers, the above two methods can be implemented. But if piwik decides to provide an option to take the timestamp from the server instead of getting from the mobile, you no longer need the above options. You just should take care of the timezone of the users & the servers, everything is clean


Query parameters are longer being sent

What happens?
It seem that there was a regression in one of the recent scalar versions. The web client currently does not send any query parameters, it's reproducible with an Example Galaxy API if we try to make a Get All Planets request with limit and offset configured.

What did you expect to happen?
I expect the API client to send query parameters if they were configured and enabled.
OpenAPI Document
https://sandbox.scalar.com/files/u7-Dw/openapi.yaml

Thanks for the report, fix on the way!


Error in fastlane init

Hey I'm still having the issue, in fast lane init
Here is my output
Guest:testApplication Guest$ fastlane init
[10:06:28]: Detected iOS/Mac project in current directory...
[10:06:28]: This setup will help you get up and running in no time.
[10:06:28]: fastlane will check what tools you're already using and set up
[10:06:28]: the tool automatically for you. Have fun!
[10:06:28]: Created new folder './fastlane'.
[10:06:28]: $ xcodebuild clean -showBuildSettings -project ./testApplication.xcodeproj
Your Apple ID (e.g. fastlane@krausefx.com): *****.com
[10:06:47]: Verifying if app is available on the Apple Developer Portal and iTunes Connect...
[10:06:47]: Starting login with user<EMAIL_ADDRESS>+----------------+-----------------------------------------------------------------------+
| Detected Values |
+----------------+-----------------------------------------------------------------------+
| Apple ID |<EMAIL_ADDRESS>|
| App Name | testApplication |
| App Identifier | ****.testApplication |
| Project | /Users/Guest/Desktop/testApplication/testApplication.xcodeproj |
+----------------+-----------------------------------------------------------------------+
[10:06:57]: This app identifier doesn't exist on iTunes Connect yet, it will be created for you
[10:06:57]: This app identifier doesn't exist on the Apple Developer Portal yet, it will be created for you
Please confirm the above values (y/n)
n
App Identifier (com.krausefx.app): com.testApplication
[10:07:33]: Created new file './fastlane/Appfile'. Edit it to manage your preferred app metadata information.
Would you like to create your app on iTunes Connect and the Developer
Portal? (y/n)
n
Optional: The scheme name of your app (If you don't need one, just hit
Enter): testApplication
[10:07:42]: 'snapshot' not enabled.
[10:07:42]: 'cocoapods' not enabled.
[10:07:42]: 'carthage' not enabled.
[10:07:42]: Created new file './fastlane/Fastfile'. Edit it to manage your own deployment lanes.
[10:07:42]: fastlane will send the number of errors for each action to
[10:07:42]: https://github.com/fastlane/enhancer to detect integration issues
[10:07:42]: No sensitive/private information will be uploaded
[10:07:42]: Successfully finished setting up fastlane

Hi @Sea-Eagle I don't see any problems in your output. Can you let me know what you think the problem is? Thanks!

[10:07:42]: 'snapshot' not enabled.
[10:07:42]: 'cocoapods' not enabled.
[10:07:42]: 'carthage' not enabled.

@Sea-Eagle So sorry for the delay in getting back to you. Are you using snapshot cocoapods or carthage on this project? What happens if you run snapshot init? Definitely want to get to the bottom of this!


Consider making a standalone version

This component is actually heavily depending on npm and latinize, onorm and object-assign.
Because in the current project I'm not using node, I had to edit the source to fit my needs. The thing is, I believe that what is being achieved with latinize and onorm should be an opt-in behavior instead default. That is part of the be preprocess step in order to make sure the filename is correct.
What object-assign achieves can also be done using $.extend which is generally better supported.
Thoughts?

Just released v4.0.0, which removed unorm and latinize.  Made the scrub function customizable, with a simple default.
Will leave object-assign alone for now, since $.extend assumes jquery (unless I'm missing something), which I wouldn't want to assume exists or depend on.  The library is tiny at least.
Thanks


ibzmq-dev vs libzmq3-dev

Is there a reason to choose one over the other?

Some users have reported libzmq3-dev fixed issues in Hydrogen. See issues #73 and #88 for details.

That seems to be the consensus. Updated readme to reflect that.


How to use jQuery in 1.0.0-beta.11-webpack.2

OS?  Windows 7, 8 or 10. Linux (which distribution). Mac OSX (Yosemite? El Capitan?)

Mac OSX EI Capitan

Versions. Please run ng --version. If there's nothing outputted, please run
in a Terminal: node --version and paste the result here:

angular-cli: 1.0.0-beta.11-webpack.2
node: 6.3.0
os: darwin x64

Repro steps. Was this an app that wasn't created using the CLI? What change did you
do on your code? etc.

npm install jquery --save
typings install dt~jquery --global --save
import * as jQuery from 'jquery';
It's no longer working since I updated to beta.11-webpack.2...

The log given by the failure. Normally this include a stack trace and some
more information.

error: Cannot find module 'jquery'

Mention any other details that might be useful.

The angular-cli@webpack is how to merge the external libs and typings to a file.

Thanks! We'll be in touch soon.

the new version of the cli uses a newer version of typescript. so rather then having a separate typings folder you just install typings via npm. if you take a look in the generated package.json file you will see:
"devDependencies": {
    "@types/jasmine": "^2.2.30",
    "@types/protractor": "^1.5.16",

so to install typings for jquery. npm i --save-dev @types/jquery

I'm experiencing the same issue.
I've installed jQuery + @types but still receiving the error "Cannot find name 'JQueryStatic'."

@pchristou Okay, I know.
The jquery.d.ts is include these declares.
declare module "jquery" {
    export = $;
}
declare var jQuery: JQueryStatic;
declare var $: JQueryStatic;

So we just need to run
tsd install jquery --save
or
typings install dt~jquery --global --save
The jQuery interface could be work in typescript.

@myAoyue I've tried the steps detailed here but still no luck: http://stackoverflow.com/questions/38855891/angular-cli-webpack-how-to-add-or-bundle-external-js-files

@pchristou sometimes vscode will get confused with its error highlighting so while the IDE says its an error it compiles just fine.

the steps in the stack overflow answer should be correct.

@pchristou I think we probably didn't import jQuery lib successfully ..
I got stuck in Cannot find module 'jquery' ~
import * as moment from 'moment';

Hi,
I got this to work using:
npm install --save {@types/},jquery

then in the code:
import * as jQuery from 'jquery';

P.S
i'm using<EMAIL_ADDRESS>but i don't see a reason<EMAIL_ADDRESS>will break it.

I got the solution :


Cannot read property 'kind' of undefined in angular 8.2.14

I updated my dependencies yesterday, and when building --prod, it throws the following error:
ERROR in ./node_modules/ngx-country-picker/fesm2015/ngx-country-picker.js
Module build failed (from ./node_modules/@angular-devkit/build-optimizer/src/build-optimizer/webpack-loader.js):
TypeError: Cannot read property 'kind' of undefined
    at isAngularDecoratorMetadataExpression (/Users/manolo/Documents/Dev/work/rec/rec-admin-ngcli/rec-admin/node_modules/@angular-devkit/build-optimizer/src/transforms/scrub-file.js:265:35)

This started happening with Angular v8.2.14 and ngx-country-picker v2.2.0
This did not happen with Angular v8.1.3
Going back to ngx-country-picker v2.1.0 fixes the issue, though...

i have the same issue and I do not understand the solution.
I am using angular 7 and typescript 3.1.3
the problem only persists when building with --prod flag
Any advice for me?

Have you tried updating typescript? I currently use<EMAIL_ADDRESS>if error persists I will recommend you try and update Angular CLI, and Angular, If this is something you can do.

I am using other components which seen to need the old typescript version. I tried to update but then ran into other dependency issues. The bug you posted previously is talking about 3.7.3, though. I don't think it can be related then, if you use 3.4.5.
If I try to use 3.4.5, I get:
<EMAIL_ADDRESS>requires<EMAIL_ADDRESS><3.3' but 3.4.5 was found instead.
Using this version can result in undefined behaviour and difficult to debug problems.

Please run the following command to install a compatible version of TypeScript.

    npm install<EMAIL_ADDRESS><3.3"

To disable this warning run "ng config cli.warnings.typescriptMismatch false".

If I try the highest possible version, 3.2.4, of typescript. It still fails.  I took this unmaintained thing over and I am new to angular, so no idea at the moment what I can do, except sticking to country picker 2.1.0 :(

Ohh sorry I forgot about that, I got into these errors too, I will look up what I ended up doing

I took this unmaintained thing over and I am new to angular, so no idea at the moment what I can do, except sticking to country picker 2.1.0 :(
Relatable, I have been in those situations more times than I like to admit :P

I will try gathering all the info I can on how I fixed it though

I ended up downgrading back to 2.1.0 as well, it seems.

@nombrekeff Can you re-open the Ticket please?
@guillaume-ro-fr can we have your attention please?

Sure!

Although I'm not certain it's this project's fault, I think it has more to do with angular and typescript.


SQL Support

How can convert json to sql ?

Hi, I have the same issue here. Is there any way I can do it?


Make docs home page the Pixie Overview (left nav always open)

This eliminates the main landing page and means that the Left Nav is always open.


Xbox not showing up on home app, but showing up and working on homebridge accessories.

I enabled it and got all the settings right (I think), it works turning off and on on homebridge, but doesn't show up in the app.

Are You add it to the HomeKit app?

Hi there, same issue here, i see it on Homebridge, but can't reach it in Apple Home

You need add it to the HomeKit, does not appear automatically. Please read readme.


feat: Update image tag management mechanisms

Ⅰ. Describe what this PR did

Use tag values in values.yaml as an override.
Use .Chart.AppVersion as the default image tag since we always build images for all components when releasing a new version.

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #495 (85a9802) into main (02b98dc) will decrease coverage by 0.05%.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #495      +/-   ##
==========================================
- Coverage   40.36%   40.31%   -0.05%
==========================================
  Files          45       45
  Lines        6186     6186
==========================================
- Hits         2497     2494       -3
- Misses       3517     3519       +2
- Partials      172      173       +1

see 1 file with indirect coverage changes


GPU Driven Rendering

for mesh and shadow pipeline
reduce drawcall
reduce state change(bind vertex,index buffer,bind descriptorset, pipelinestate)


Added Persian (fa) Locale

I added a new strings.xml file to support the Persian language.

There are multiple other pull request like this, you could've checked theme before making a pull request. #739 #642.

Thanks for the contribution! There are a couple more strings missing:

a11y_plant_item_image
a11y_back

CLA still failing (see https://github.com/android/sunflower/pull/751/checks?check_run_id=19988830749). Alternatively, you can open a new PR with a commit signed by an email address you used to sign the CLA.


Android OutOfMemoryError occurs when readTiff for 400MB tif file

Please fill out as much known and relevant information as possible.
Version Information:

TIFF Java Version: 2.0.2
TIFF Java Source: Central Repository
IDE Name & Version: Android Studio 3.2.1
Maven Version: (mvn -version) : gradle:3.2.1
Java Version: (java -version) : Android-26
Platform & OS: Android
Other Relevant Libraries:

Expected Results:

What did you expect to happen?
Open the big tif file : TiffReader.readTiff(tiffFile)

Observed Results:

What happened instead?
AndroidRuntime: FATAL EXCEPTION: Thread-19
java.lang.OutOfMemoryError: Failed to allocate a 536870928 byte allocation with 25165824 free bytes and 234MB until OOM, target footprint 316450016, growth limit 536870912

How often does this occur? Always

Output:

Any logs, errors, or output messages?
java.lang.OutOfMemoryError: Failed to allocate a 536870928 byte allocation with 25165824 free bytes and 234MB until OOM, target footprint 316450016, growth limit 536870912
at java.util.Arrays.copyOf(Arrays.java:3161)
at java.io.ByteArrayOutputStream.grow(ByteArrayOutputStream.java:118)
at java.io.ByteArrayOutputStream.ensureCapacity(ByteArrayOutputStream.java:93)
at java.io.ByteArrayOutputStream.write(ByteArrayOutputStream.java:153)
at mil.nga.tiff.io.IOUtils.copyStream(IOUtils.java:104)
at mil.nga.tiff.io.IOUtils.streamBytes(IOUtils.java:83)
at mil.nga.tiff.io.IOUtils.fileBytes(IOUtils.java:67)
at mil.nga.tiff.TiffReader.readTiff(TiffReader.java:51)
at mil.nga.tiff.TiffReader.readTiff(TiffReader.java:35)
at com.mvsoftech.mvgis.forestServices.DemFinder$2.run(DemFinder.java:49)
at java.lang.Thread.run(Thread.java:919)

Steps to Reproduce:

just run app in mobile

Relevant Code:
// Code to reproduce the problem?
TIFFImage tiffImage = TiffReader.readTiff(tiffFile);

Test Files:

Links to any files needed for testing?

Additional Information:

Any additional configuration, data, or information that might help with the issue?
Thank you If you can fix it

Can you provide file(s) that demonstrate the problem? Even the results of a gdalinfo dump may help.
Also, what do you have set for your Dalvik heap size to?

This implementation loads the entire tiff file into memory for raster random read access.  Your reported error occurred attempting to copy the file into memory.  Allocation of ~537mb with a growth limit of ~537mb.
Try increasing the heap size.


Random order of XML attributes when saving

If I open a syncdata file and save it, the attributes in the XML row, value and interpolation are shuffled each time, and git thinks all keys in the file have been rewritten. This makes merging files a headache. Apparently, this is caused by QDomElement.setAttribute (here) storing the keys in a hashmap, which shuffles the keys on purpose (see here). This could be fixed by seeding the hash as suggested in the SO thread.

Hmm, interesting. I haven't seen this behavior myself, so I wonder if this is dependent on Qt versions... I also wasn't aware that QtXML is deprecated, which I think is a strong argument to rewrite the code to use  QXmlStreamReader/QXmlStreamWriter instead... I think the old Qt4 support can be binned anyway.

This behavior started in Qt5.6. The automated windows build (which is what I use) use Qt5.5. This explains why I haven't seen this behavior.
Thanks for reporting. I think a quick fix should be easy to do, like you suggest.

Hmm, I just tried to reproduce this on Qt 5.11 on Ubuntu, and I seem to get consistent behavior...
@mgunyho: What environment do you trigger this under?

OK, never mind. I managed to trigger it after some more tries. I had to restart the editor in between, I guess the hash stays constant.

@mgunyho: OK, could you check if #74 fixes the issue for you?

Seems to work, thanks!


Pixart Support

I absolutely love your work and everything you do for the community! Do you have any plans to add support for Picsart to this project?

PixArt alpha uses the SDv1 VAE (latent_src: v1) and PixArt sigma uses the SDXL VAE (latent_src: xl) so it's already supported without any extra changes.
Feel free to reopen if that's not what you meant lol


Protect identify and alias from being fired where they shouldn't be.

:+1:

Marking this ready to :ship: after testing solution with Catherine today.

:+1:

Marking ready to :ship:. Deploying right before the other PRs.


[WIP] Add a bookmark tab

This is a spike to explore a "bookmark tab" feature, as part of an exercise.

Add a button/link under an post item to bookmark it (store it in localStorage)
Add a tab to display the bookmarks
Allow to remove a bookmark from a bookmarked item
Add a second button to hide news: useful to what you already read

(1) + (3) I added a button/link under a post item to bookmark or remove it as a bookmarked item.

The logic implemented in this PR is handled by the NewsDetail component which now has got a boolean 'bookmarked' state.
Bookmarks are stored in localStorage : the key is the user Id and the key's value is an array of all bookmarked NewsDetail props. If the user is not logged in the bookmark is store as a value of "guest" key in localStorage.
Every time a user bookmark (resp. unbookmark) a post item, the NewsDetail props are saved to (resp. removed from) localStorage using the saveStateToLocalStorage  method.
I added a hydrateStateWithLocalStorage method in order to keep hydrated NewsDetail components even if the user changes/refresh page.
In order to retrieve the logged in user id (which is actually username) information from the database I used the existing meQuery graphql query and  added it to the NewsFeed props.

Other remarks

Ask current user to log in on "bookmark" click.
Right now a bookmarks update is made everytime the user bookmark/unbookmark. One could just update the bookmark state on every click and call the  saveStateToLocalStorage  when the componentWillUnmount
I did store the entire NewsDetail props for post Item that were bookmarks. I did not store only the post Item id because I thought it would be easier then to retrieve this information later for the bookmarks tab.

(2) Add a tab to display the bookmarks

It is still to be done
I just added a link in the navbar routing to an empty NewsFeed page and created a bookmarks.js page
I now would have to retrieve my localStorage information along with the user Id and pass it along to my NewsFeed component.
For that matter I would probably have to do a graphql query on localStorage post item id's to retrieve all the props needed for the NewsFeed component, and to ask the user to log in to access the bookmarks tab.

** (4) Add a second button to hide news: useful to what you already read**

From what I understand this 'hide' feature is already implemented inside the original repo.
I might have missed the point here though.

You are right about number 4, thought it was missing good catch


Removed rc1 qualifier from front and backend plugin versions.

Description
Removed rc1 qualifier from front and backend plugin versions.
Issues Resolved
https://github.com/opensearch-project/notifications/issues/438
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #445 (39d021a) into main (24c2d0f) will decrease coverage by 9.50%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #445      +/-   ##
============================================
- Coverage     70.71%   61.21%   -9.51%
  Complexity      108      108
============================================
  Files           124       73      -51
  Lines          3923     2444    -1479
  Branches        626      261     -365
============================================
- Hits           2774     1496    -1278
+ Misses          979      781     -198
+ Partials        170      167       -3

Flag
Coverage Δ

dashboards-notifications
?

opensearch-notifications
61.21% <ø> (ø)

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

Impacted Files
Coverage Δ

.../Channels/components/modals/DeleteChannelModal.tsx

...otifications/public/components/Modal/ModalRoot.tsx

...ages/CreateChannel/components/ChannelNamePanel.tsx

...Channels/components/details/ChannelDetailItems.tsx

...ic/pages/Emails/components/tables/SendersTable.tsx

...pages/Emails/components/tables/SESSendersTable.tsx

...s-notifications/public/components/coreServices.tsx

...ifications/public/pages/Emails/CreateSESSender.tsx

...es/Channels/components/modals/DetailsListModal.tsx

...notifications/public/pages/Emails/CreateSender.tsx

... and 41 more

Continue to review full report at Codecov.

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

@AWSHurneyt Looks like the backport failed, would you mind backporting this change to 2.0 as well?

@AWSHurneyt Looks like the backport failed, would you mind backporting this change to 2.0 as well?

Will do.


Best practice for using AutoML?

Saw the sample of using PipelineData to drive the pipeline: https://github.com/Azure/MachineLearningNotebooks/tree/master/how-to-use-azureml/machine-learning-pipelines/nyc-taxi-data-regression-model-building
My question is : what's  the proper way to add in 'Evaluate' and 'Register' steps after AutoMLStep?
Thanks in advance!

Here comes the answer: https://github.com/Azure/MachineLearningNotebooks/tree/master/how-to-use-azureml/automated-machine-learning/continuous-retraining


132 Pattern algorithm in C++

Hey @SjxSubham
Added 132 Pattern algorithm

Thank U for your contribution
Star the repo... Happy HAcktober

Sorry this kind of Solution I can't merge... Go through the repo description Properly and add Only LeetCode Problems...


microROS agent- client serial communication is not working in zephyr RTOS

RTOS: Zephyr
MCU : olimex STM32E407
Tried the microROS zephyr RTOS int32_publisher program on olimex STM32E407 board. Tried agent client communication through usb-serial and its working fine. Tired the same with serial communication but the agent client communication is not establishing.

Have you configured the serial port that you want to use here?

Ok thanks @pablogs9


Links to markdown prompt a download

See here http://redux.js.org/docs/advanced/index.html
This link does not render, but prompts to download the markdown file.
http://redux.js.org/docs/advanced/UsageWithReactRouter.md
Once you eventually track it down here in the repo (or download and open it), you discover it's coming soon. Would be cool if there was an easy way to flag that directly in the docs.

Yeah. The real fix would be to write that document.
I’m waiting for some React Router 3.0 changes to get to it.
I’ll probably close because this is going to be irrelevant soon.


Make Jason's responsibilities/how he can help more clear

As requested by the Code Graph PMs.

So glad this is helpful! Merging.


Feature defer pushnotification permissions ios

Some users were having issues with the fact that users were being asked for permission to user push notifications immediately on launch. This PR gives the option to opt out of auto-registration and start it manually using a JS call.
This PR contains:

[x] Changelog
[x] Documentation
[ ] Tests

 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.1 out of 2 committers have signed the CLA.:white_check_mark: Duckers:x: cbaggersYou have signed the CLA already but the status is still pending? Let us recheck it.


Gstreamer pipeline

I just wanted to ask what gstreamer pipeline is used to transmit the video and where this pipeline is defined on the AirPi and GroundPi.
Thank you in advance

No gstreamer used. We use the Pis video api directly through raspivid and the "hello_video.bin" player implementation.


Implemented Hailey's Comments

Added defaults to the RD file
Updated the tabbing for else statement

Thanks for your patients with my lack of paying attention last time :)!


[app] Initialise AngularCLI project

This PR initialises an Angular CLI Project and the following changes have been made to the boilerplate code:

Remove boilerplate code
Install @angular/pwa support
Refresh PWA Cache on update
Comment out failing dummy test
Add @angular/material
Add source code headers
Add WebComponent support and Lit
Add Light and Dark theme functionality

Screenshots:

One more comment.
This PR is already large. Can we focus on one or two goals only? I see there are many goals here:
Remove boilerplate code
Install @angular/pwa support
Refresh PWA Cache on update --> move to a following PR
Comment out failing dummy test
Add @angular/material
Add source code headers
Add WebComponent support and Lit --> move to a following PR
Add Light and Dark theme functionality --> move to a following PR

I see this file is empty: diagnostics-app/src/assets/.gitkeep, What is this for? Can you remove  it, if not going to be used?

The assets directory will have all the assets that will be passed on to the build folder as it is. Currently it is empty but will be populated with all the assets (icons, fonts, graphics) that we'll use. Hence we keep the folder with .gitkeep which is just an empty file.

What are the .png and icons? Are these Angular-logo? We need to use icons with appropriate copyrights. Let me know if you need help with this.

They came along with the boilerplate code. Removed them for now. 👍🏻

This PR is already large. Can we focus on one or two goals only?

@MahmoudAGawad Most of the files were auto-generated and only a few files were added by me here. I will make sure to make small PRs in future.

This PR is already large. Can we focus on one or two goals only?

@MahmoudAGawad Most of the files were auto-generated and only a few files were added by me here. I will make sure to make small PRs in future.

But I see changes that can really be moved to next PRs, code logic like:

Refresh PWA Cache on update
Add WebComponent support and Lit
Add Light and Dark theme functionality

If it doesn't break the any part of the app, let's move them, what do you think?

This PR is already large. Can we focus on one or two goals only?

@MahmoudAGawad Most of the files were auto-generated and only a few files were added by me here. I will make sure to make small PRs in future.

But I see changes that can really be moved to next PRs, code logic like:

Refresh PWA Cache on update
Add WebComponent support and Lit
Add Light and Dark theme functionality

If it doesn't break the any part of the app, let's move them, what do you think?

Okay 👍🏻 I'll try to separate them into three PRs.

@MahmoudAGawad The goals listed in the PR description were squashed into a single commit before raising this PR. I've separated the other logic into their separate PRs.

In general looks good, but please double check dependencies, looks like there are some of them we don't need (but I might be wrong about some deps).


[opendistro-1.11][backport] Do not trim SAML roles

(cherry picked from commit 2ecb088a1ac58bd1ac4a0c1100492bc1566b9be7)
opensearch-security pull request intake form
Please provide as much details as possible to get feedback/acceptance on your PR quickly

Category: (Enhancement, New feature, Bug fix, Test fix, Refactoring, Maintenance, Documentation)
Maintenance

Github Issue # or road-map entry, if available:

Description of changes:
Backport for #1207

Why these changes are required?

What is the old behavior before changes and new behavior after changes? (Please add any example/logs/screen-shot if available)

Testing done: (Please provide details of testing done: Unit testing, integration testing and manual testing)

TO-DOs, if any: (Please describe pending items and provide Github issues# for each of them)

Is it backport from main branch? (If yes, please add backport PR # and commits #)

By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or
(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or
(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.
(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

:exclamation: No coverage uploaded for pull request base (opendistro-1.11@5f51815). Click here to learn what that means.
The diff coverage is n/a.

@@                Coverage Diff                 @@
##             opendistro-1.11    #1243   +/-   ##
==================================================
  Coverage                   ?   63.96%
  Complexity                 ?     3167
==================================================
  Files                      ?      239
  Lines                      ?    16902
  Branches                   ?     3032
==================================================
  Hits                       ?    10811
  Misses                     ?     4544
  Partials                   ?     1547

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 5f51815...7bc1d53. Read the comment docs.


Chrome driver should support --no-sandbox

If running this with chrome as root (e.g. inside a docker container as part of a continuous integration setup), it's necessary to pass in the --no-sandbox option to it.  Otherwise you get an error: Chrome failed to start: exited abnormally.  See https://github.com/SeleniumHQ/selenium/issues/4961 .
Therefore, this package should allow as an option (not by default) to give chrome the --no-sandbox option.  Alternatively, there could be a generic way for the user to specify additional custom arguments to pass to chrome.
Thanks!

Sounds good, I'll put one together.


Message.messageThreadId for bots

It is always 0. Is it unavailable for bots?

Yes, bots don't support interaction with message threads.
But they know replyToMessageId and can track message threads using it.


Patch to compile on Debian Linux 10

I tried out building revision https://github.com/mbaze/bzpack/commit/9294688b98ce243fec49e4188c042509ba836bf7 on a Debian Linux 10 server with clang 7.0.1 and gcc 8.3.0 and had to modify a few bits to make everything compile. I uploaded everything to the server and here's just the patch. I didn't test decompression of the resulting file yet but both compilers produce the same output file and its size seems plausible: the 92 KiB of an 86-DOS application binary compressed to 68 KiB, using the default compression scheme.
diff --git a/src/BitStream.h b/src/BitStream.h
index 014c78c..1d03310 100644
--- a/src/BitStream.h
+++ b/src/BitStream.h
@@ -5,6 +5,8 @@
 #define BIT_STREAM_H

 #include <vector>
+#include <algorithm>
+#include <cstdint>

 class BitStream
 {
diff --git a/src/Main.cpp b/src/Main.cpp
index 776c84f..7515b0c 100644
--- a/src/Main.cpp
+++ b/src/Main.cpp
@@ -4,6 +4,7 @@
 #include <memory>
 #include <string>
 #include <stdio.h>
+#include <algorithm>
 #include "Compressor.h"

 const char* pInputFileError = "Error: Cannot open input file.\n";
@@ -72,7 +73,7 @@ int main(int argCount, char** args)
     }

     FILE* pInputFile;
-    if (fopen_s(&pInputFile, args[1], "rb"))
+    if ( (pInputFile = fopen(args[1], "rb")) == NULL )
     {
         printf(pInputFileError);
         return 0;
@@ -89,7 +90,7 @@ int main(int argCount, char** args)
     rewind(pInputFile);

     std::unique_ptr<uint8_t[]> spInputStream = std::make_unique<uint8_t[]>(inputFileSize);
-    size_t bytesRead = fread_s(spInputStream.get(), inputFileSize, 1, inputFileSize, pInputFile);
+    size_t bytesRead = fread(spInputStream.get(), 1, inputFileSize, pInputFile);
     fclose(pInputFile);

     if (bytesRead != inputFileSize)
@@ -105,7 +106,7 @@ int main(int argCount, char** args)
     }

     FILE* pOutputFile;
-    if (fopen_s(&pOutputFile, args[2], "wb"))
+    if ( (pOutputFile = fopen(args[2], "wb")) == NULL )
     {
         printf(pOutputFileError);
         return 0;
diff --git a/src/UniversalCodes.h b/src/UniversalCodes.h
index e40934f..c69cc41 100644
--- a/src/UniversalCodes.h
+++ b/src/UniversalCodes.h
@@ -5,6 +5,8 @@
 #define UNIVERSAL_CODES_H

 #include "BitStream.h"
+#include <cassert>
+#define _ASSERT assert

 uint32_t GetElias1Cost(uint32_t value);
 void EncodeElias1(BitStream& stream, uint32_t value);

Thank you, I will try to incorporate this, these are sensible changes to make.

Like you can see in the commands I used I had to pass the switch -Wno-format-security to clang, otherwise it complained in the Main.cpp source file that you used printf with a string constant instead of a string literal. Doesn't really harm anything, just another thing to note.


官网demo缺少依赖

One-line summary [问题简述]
http://echarts.baidu.com/examples/editor.html?c=graph-webkit-dep
从上述页面中下载的Demo缺少jQuery依赖
Version & Environment [版本及环境]

ECharts version [ECharts 版本]:
Browser version [浏览器类型和版本]:
OS Version [操作系统类型和版本]:

Expected behaviour [期望结果]
ECharts option [ECharts配置项]

option = {

}

Other comments [其他信息]

👍  同样遇到该问题， 手动引入jquery之后解决，对新手稍微有点不友好 :) 。

Guys, could you please write in English?
I've recently started watching in project, and I believe that it has no chance of graduating from the Apache Incubator if people need use a translation service for everything.
Thank you!

@SchnWalter
Yeah, you know this project was developed by chinese company named baidu, so I use chinese, but i was glad to translate for you.
This issue is about echarts' official website.
In the demo on url http://echarts.baidu.com/examples/editor.html?c=graph-webkit-dep
click the DOWNLOAD button will download a sample file with bug which lack the import of jquery.

Yes, I know and I'm thankful for that. But now that the library is part of the Apache Incubator, which is great, it would be nice to gradually transition to English where it may be relevant or helpful for other people. It's nice to be able to search the web for an issue and find a discussion around it. Most people will search in English.
P.S. Thanks for all your work!


Remove bogus Foundation.framework embedding

Embedding Foundation.framework into ObjCAA target doesn't make any sense, because this framework is part of iOS.
I've got following error while uploading my app to App Store Connect with embedded Foundation:
ERROR ITMS-90206: "Invalid Bundle. The bundle at 'MeteorActive.app/Frameworks/ObjCAA.framework' contains disallowed file 'Frameworks'."

I fail to make a CocoaPod release. Running pod lib lint allow-warnings SwiftAA I get
    - ERROR | xcodebuild:  /Users/onekiloparsec/code/SwiftAA/Sources/SwiftAA/Earth.swift:55:117: error: extra argument in call
    - ERROR | xcodebuild:  /Users/onekiloparsec/code/SwiftAA/Sources/SwiftAA/Earth.swift:55:98: error: cannot convert value of type 'KPCAAPlanetStrict' to expected argument type 'Bool'

WTF

@alex-vasenin any idea ?

@alex-vasenin any idea ?

Hi Cédric! Have no idea, but I can check once I’ll back from vacation on 20th.

@onekiloparsec It seems like SwiftAA.podspec for 2.3.1 still has ObjCAA v.2.3.0 dependency, so pod lib lint builds ObjCAA 2.3.0 and tries to feed it to newer SwiftAA 2.3.1 which fails due to mismatched method signatures. Don't forget to to add 2.3.1 tag!

Thanks. ObjCAA and SwiftAA 2.3.1 released.


Fix OSX compilation of trianglell example

OSX doesn't support our d3d12 or vulkan backend, so in this case the example will just do nothing.

Thanks!
@homu r+

:pushpin: Commit b6f8b30 has been approved by kvark

:hourglass: Testing commit b6f8b30 with merge d154741...

:sunny: Test successful - status


/miniplaceholders doesn't display placeholder count

It should display placeholder count for players with permission, and plugin version for players without permission. But PermissionChecker requires TriState.TRUE, and players may have a permission enabled by default. Server operators have all default: op permissions implicitly with NOT_SET, maybe there are other such cases. So even if I /op myself, I still get MiniPlaceholders' version in /miniplaceholders

This problem seems to be specific to Paper that does not return the proper Tristate. Still, it would be ideal to use a permissions plugin like LuckPerms to manage server permissions instead of a buggy system like Minecraft vanilla's "op"


[BUG] mars tiling odps source reader hang

Describe the bug
mars tiling odps source reader hang:

To Reproduce
To help us reproducing this bug, please provide information below:

Your Python version: 3.7.9

The version of Mars you use: 0.9.0 / 9cfcc946c789776121b8b4da87329166e527d154

Versions of crucial packages, such as numpy, scipy and pandas: numpy1.19.5, pandas1.3.5

Full stack of the error.

Minimized code to reproduce the error.

df = o.to_mars_dataframe(input_table_name, columns=["id","a","b"], chunk_size=400).head(8000)
df = df.astype({'id': 'string','a':'float','b':'float'}).execute()
print(df.shape)

Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.

Not a Mars issue. Move to pyodps repo.


Need to allow users to use helper functions for the recursive challenges

Currently we recently added 3 recursion challenges to the curriculum.  Each challenge has a test that checks to see if the user used recursion to solve the challenge.  We check this by first converting the calling function to a string and then using regex to match the name of the calling function somewhere inside it (after removing comments).  This works well unless the user implements a helper function that actually makes recursive calls.
The Use Recursion to Create a Countdown challenge starts with the following seed code:
//Only change code below this line
function countdown(n){
  return;
}
console.log(countdown(5)); // [5, 4, 3, 2, 1]

The following is a perfectly valid solution, but our tests expect to see countdown called inside the countdown function instead of counter.
function countdown(n) {
  function count(arr, n) {
    if (n >= 1) {
      arr.push(n);
      return count(arr, n - 1);
    }
    return arr;
  }
  return count([], n);
}

What is the best way to validate recursion is used for these challenges while allowing the user to create a helper function for the recursive calls?

To keep things simple, I suppose we could change the test text to something like:

You should use recursion to solve this problem by calling countdown within the countdown function.

We could use a regex along the lines of
(?:const|var|let)\s(\w+)\s=\s\(\w+\)\s=>\s{\s\1\(\w+\)\s}|function\s(\w+)\(\w+\)\s{\s*\2\(\w+\)\s*}

it's likely to be bug prone, though, so we'd need to be careful.
Another option is to rely on the test that checks if they're using various looping functions.  I doubt many people will come up with non-recursive non-looping solutions.

Another option is to rely on the test that checks if they're using various looping functions. I doubt many people will come up with non-recursive non-looping solutions.

I like this option.  Does anyone want to work on achieving this with a new test or modification of an existing test?

I can take a look at adding this 🙂

Heya @RandellDawson - so poking around in the file, we already have a test that checks for the looping functions:
testString: assert(!removeJSComments(code).match(/for|while|forEach|map|filter|reduce/g));

Should I work with the regex proposed above to test for recursion? Or would it be better to remove the recursion test?

Thank you for reporting this issue.
This is a standard message notifying you that the issue has become stale, so we are going to close it.
If you think we're wrong in closing this issue, please request for it to be reopened. Thank you and happy coding.


Support IAM Roles for Service Accounts in EKS

Hello, I'm using your plugin on EKS.
AWS has released the new feature for EKS. When I was migrating to this, I found something that didn't work with your plugin.
This feature uses OIDC JSON web token so Aws::AssumeRoleCredentials could not get IAM Roles...
So I modified your plugin to use Aws::AssumeRoleWebIdentityCredentials if web token exists.

@cosmo0920 Thanks for the review. I will write an example configuration in the README.

I wrote an example configuration in the README.
Please review again 😄


Flextype website: wrong dates on changelog (0.9.16 & 0.9.15)

On the website. the changelog shows incorrect dates for 0.9.16 & 0.9.15
It shows them as having been updated in 2020 (instead of 2021).
Since earlier changes are "below the fold" this may make newcomers think flextype is not being actively updated.  I was almost discouraged myself -- and then (after scrolling down) a slight bit confused that dates seemed so out of order.

@FBachofner Thanks! I will fix this ASAP

Fixed

I am just astonished (and pleased) you are so responsive!  Good for you.
Going to play with Flextype this upcoming weekend!


⚠ Processors has degraded performance

In ca487ca, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 89 ms

Resolved: Processors performance has improved in 049fd67.


support Scala 2.13.0-M2

zinc compiler-bridge need sbt-util and sbt-io 2.13.0-M2 ?
https://github.com/sbt/zinc/pull/445

To validate this pull request against other sbt modules, please visit this link.

https://github.com/sbt/zinc/pull/453


Compelling Use Cases for User-Focused Client Constraining Access Control

We've heard multiple use cases for User-Focused Client Constraining Access Control (ie Client Constraining Access Control that is not controlled by the resource controller, but by the app user). I'd like to use this issue to collect them.

from gitter per @jaxoncreed 's request:

i thought that one of the fundamental principles of Solid was that the user should be able to use whatever app she wants to access whatever data. it's not reasonable for a resource controller/owner to have to maintain a whitelist of all the different apps all the different consuming users want to use. i thought we-all had decided that a long time ago. however, the minutes from monday's meeting sound like people had gone back to the controller/owner explicitly specifying what apps users can use. that or "user uses a proxy". i'm very -1 on both of those antipatterns.

note that the meeting minutes from 2020-04-20 is in a file named 2019-04-20, so if that gets fixed the link above will break. :)

One compelling use case revolves around multiple communities. These communities are relatively tight-knit and each member trusts the other members. In order to avoid the friction of asking to use an application, the access control in the community accepts any application that one of its members uses.
A concrete example of this could be an open-source project where all members of the project trust each-other.

A concrete example of this could be an open-source project where all members of the project trust each-other.

I think we can take @solid as an example. While CG has more open entry, github org has higher requirements for membership. If we consider @solid members as invited experts, who in addition only have access to relevant repositories, I think we don't we need to restrict which applications they can use to participate in issues, project boards, chatrooms (gitter) etc.
W3C Working Groups which have even higher level of entry possibly can leave it up to experts participating in them, which applications/clients they want to use.

This is a highly perplexing issue, and conversation to be having.
What you're asking is the exact logical equivalent of Apple asking "Let's come up for some compelling use cases for allowing users to pick the app they're accessing our music with."
That's not a metaphor, that's not an exaggeration, it is the exact functional equivalence / definition.
User-Focused Client Constraining access control == an "ok" (not perfect) protection from phishing, and is a genuine innovation Solid can bring to the world. Oh, and, most importantly, is technically possible.
Resource Controller Focused Client Constraining access control == is literally DRM. Not technically possible for the vast majority of Solid apps. And does not bring any benefit or added security.

I think appending Linked Data Notifications makes interesting example:
Alice doesn't want to restrict which applications other users can use to append Linked Data Notifications to her inbox. Other users want to restrict which applications can append LDN to Alice's (and other) inboxes

Currently we have following use cases documented:

Chat with friends
Discussion boards
Photos viewer
Photos organizer
Photos processing

All of those use cases illustrate scenario where Alice wants to constrain client when accessing data which she doesn't have resource controller access rights.


[MRG] Do not try to build the image with root as the primary user.

Fixes #267 and #395. Solution suggested in https://github.com/jupyter/repo2docker/issues/267#issuecomment-478691267

Did I draw a short straw and ubuntu removed artful within past few days/hours?

I think it is more likely this was a temporary connection thing as some of the tests passed that use the same docker file 🤞.

Though there is no good reason to use artful here, so we should switch them to bionic which we use for the rest of repo2docker.

I think it is more likely this was a temporary connection thing as some of the tests passed that use the same docker file .

I can reproduce that locally: docker run --rm -ti ubuntu:artful apt-get update -qqy. I think it's gone, gone. I will bump to bionic.

lol. It looks like the first test "passes" because we expect it to fail and it does, just not because of overallocation of RAM but because there is a problem with building the image. This means your suggestion of "artful support stopped" is probably right. I'll make a PR to switch the tests to bionic.

lol. It looks like the first test "passes" because we expect it to fail and it does, just not because of overallocation of RAM but because there is a problem with building the image. This means your suggestion of "artful support stopped" is probably right. I'll make a PR to switch the tests to bionic.

OK, will wait for you PR then.

https://github.com/jupyter/repo2docker/pull/677 for updating the base image.

#677 is merged. If we rebase this PR we should be good to go.
This PR is a great example of why it is so hard to estimate how long it will take to do something. "Change up a few lines to give a better error message, child's play!" ... except then we go down the rabbit hole of Ubuntu releases and how the Docker API works :)

#677 is merged. If we rebase this PR we should be good to go.

Already did that :)

Ah yes, I should make use of GitHub snitching on the git push -f crowd. It used to be that well kept secret between me and github when I did that ... and now they tell everyone! :)


errors on run...

i'm receiving some errors, and the instructions are a little unclear (although I think I've done things correctly?)
JINKITOSXLT02:~ bjozsa$ /Users/bjozsa/Library/Developer/Xcode/DerivedData/Kube-Solo-frcfrvqaeruemvcvxjgrykpibbnn/Build/Products/Debug/Kube-Solo.app/Contents/Resources/up.command; exit;
cat: /Users/bjozsa/kube-solo/.env/resouces_path: No such file or directory
cp: /bin/xhyve: No such file or directory
chmod: /Users/bjozsa/kube-solo/bin/xhyve: No such file or directory
/Users/bjozsa/Library/Developer/Xcode/DerivedData/Kube-Solo-frcfrvqaeruemvcvxjgrykpibbnn/Build/Products/Debug/Kube-Solo.app/Contents/Resources/up.command: line 18: /bin/webserver: No such file or directory
File with saved password is not found:

Your Mac user's password will be saved to '~/kube-solo/.env/password' file
and later one will be used for 'sudo' command to start VM !!!
This is not the password for the VM access via ssh or console !!!
Please type your Mac user's password followed by [ENTER]:
/Users/bjozsa/Library/Developer/Xcode/DerivedData/Kube-Solo-frcfrvqaeruemvcvxjgrykpibbnn/Build/Products/Debug/Kube-Solo.app/Contents/Resources/functions.sh: line 278: /Users/bjozsa/kube-solo/.env/password: No such file or directory
chmod: /Users/bjozsa/kube-solo/.env/password: No such file or directory

ls: /Users/bjozsa/kube-solo/imgs/.*.vmlinuz: No such file or directory
Couldn't find anything to load locally ( channel).
Fetching lastest  channel ISO ...

/Users/bjozsa/Library/Developer/Xcode/DerivedData/Kube-Solo-frcfrvqaeruemvcvxjgrykpibbnn/Build/Products/Debug/Kube-Solo.app/Contents/Resources/functions.sh: line 219: /bin/coreos-xhyve-fetch: No such file or directory

Starting VM ...
/Users/bjozsa/Library/Developer/Xcode/DerivedData/Kube-Solo-frcfrvqaeruemvcvxjgrykpibbnn/Build/Products/Debug/Kube-Solo.app/Contents/Resources/up.command: line 43: /bin/dtach: No such file or directory
You can connect to VM console from menu 'Attach to VM's console'
When you done with console just close it's window/tab with CMD+W
Waiting for VM to boot up...
\

what am I missing?
just to show that xhyve is installed...
JINKITOSXLT02:src bjozsa$ xhyve -h
Usage: xhyve [-behuwxACHPWY] [-c vcpus] [-g <gdb port>] [-l <lpc>]
             [-m mem] [-p vcpu:hostcpu] [-s <pci>] [-U uuid] -f <fw>
       -A: create ACPI tables
       -c: # cpus (default 1)
       -C: include guest memory in core file
       -e: exit on unhandled I/O access
       -f: firmware
       -g: gdb port
       -h: help
       -H: vmexit from the guest on hlt
       -l: LPC device configuration
       -m: memory size in MB
       -p: pin 'vcpu' to 'hostcpu'
       -P: vmexit from the guest on pause
       -s: <slot,driver,configinfo> PCI slot config
       -u: RTC keeps UTC time
       -U: uuid
       -v: show build version
       -w: ignore unimplemented MSRs
       -W: force virtio to use single-vector MSI
       -x: local apic is in x2APIC mode
       -Y: disable MPtable generation

@v1k0d3n  I need more details the steps you did.
App uses xhyve which comes with it.
What do you mean about unclear instructions?

btw I will a new version soon, which does not save Mac user's password to the disk but saves to Keychain instead

yes please! :+1: i saw mr. burns request, and i completely agree. outside of my company I am ok, but this is a sticking point for running on corp MBP. thanks for the addition!
so I used the included project to compile the application and grabbed the final application artifact (.app file), ran it and used the menu bar "Setup" and "Up" menu items. Is there more to it?

Just under "Setup" you need to type your Mac user password, maybe that step was not right?
as "sudo" needs to be used twice in the App, get macaddress to uuid and start xhyve

deleted the /Users//kube-solo directory and ran Setup again, and things skipped along happily. thanks for the awesome project! excited to see where this ends up going and having more features/options added in.

resolved. closing issue.


feat: Implementation of the Sign In page, with AuthPagesSignInForm, v…

…alidateSignInForm and addition of authentication for login in auth.ts

This PR is related to issue #4


MLSA under review

What product(s) do you have a question about?
not product
What is your question?
Hello!!
I have applied to the MSLA program in July8,2024 , currently my application is under review. 2.5 weeks has passed and I was wondering how much does the review duration usually is?
I'm just too excited to start my journey if I got accepted!
I would love to hear from your experience as a MSLA too!
Thank you!
Additional context

What is your current role?
Student
What technical topic is this question related to?
Other

Hi @NS-MALIK
Join our discord Server to be assisted


Added Keen IO integration example

Added Keen IO integration example, as well as updated text on where to find integration information.

Thanks!


[Tracking] Development standards

Tracking decisions and documentation around development standards not otherwise covered by automated tooling

An initial pass on this early on with Thomas was recorded here (internal document): https://docs.google.com/document/d/1KCbeg6eoINxk6RFE1RdUGwtNj9_5xSs1DD78PJSZxM4/edit and further discussion with Thomas and Tamas was had at our October on-site. Upshot:

Using conventional commits via PR titles (picked up in main history when squash-committing)
Required code reviews via PRs
Squash commit to make history more meaningful (can always reference original history in PR if needed)
Use a single long-lived main branch

Protected branch, disabling force push, requiring approved PRs (approvals dismissed with additional changes) with passing CI
When features need testing before launch, either test on feature branches or use feature flags (in many cases, the latter should probably be preferred)
Our process for cutting releases is still somewhat up in the air. The current thought is that releases will be made by merging release PRs, though we haven't fully ruled out other options for push-button or merge-to-branch releases.
Our branch protection rules should allow contributors with write access to our repositories to merge their own PRs as long as they have the necessary approvals. Note this means eschewing our original plan to require up-to-date PRs before merging (as this would mean other PRs would need to be re-approved every time a PR is merged - this should likely be fine, and is common practice for many teams/repositories)

Branch names should follow the format <type>/<kebab-case-short-description>
We should have automated tests. We are not aiming for 100% code coverage, but we want enough tests to verify that we can update dependencies and feel reasonably secure that nothing breaks.

As far as process, at this point we plan on sticking with our once-a-week sync via video, plus we've set up daily standups via Slack.
https://github.com/eternagame/.github/ has also been set up to hold our PR template (eternagame/.github#4), issue templates (eternagame/.github#8), contributing guidelines (eternagame/.github#5), and security policy (eternagame/.github#9).
Any additional standards will be worked out as we identify gaps.

A note re: required approvals: We're setting up CODEOWNERS files. By default, this will have everything owned by the maintainers group (currently Rhiju, Thomas, and myself). We may opt to expand ownership of certain sections of the codebase as appropriate.
Additionally, we'll ensure things are set up such that only one of the individuals in the maintainers group can actually trigger a release.