Added mohr circle stress calc theory

@robbievanleeuwen quick basis of the Mohr's Circle theory - please edit and place in the theory as you prefer!

Codecov Report

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

@@          Coverage Diff          @@
##             v2     #160   +/-   ##
=====================================
  Coverage      ?   54.53%
=====================================
  Files         ?       20
  Lines         ?     3757
  Branches      ?        0
=====================================
  Hits          ?     2049
  Misses        ?     1708
  Partials      ?        0

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 83fdfbd...1dbcd74. Read the comment docs.


Register a birth. DNA test changes

Clarify that users are to pay any costs related to DNA testing
Add Somalia as the second country (Libya is the first) that may require DNA testing

Looks good to me.
It's great to see the switch from if country_of_birth == 'libya' to the much more descriptive if reg_data_query.may_require_dna_tests?(country_of_birth).


Cannot start IIS

I have installed docker:
Client:
Version:           18.06.0-ce
API version:       1.38
Go version:        go1.10.3
Git commit:        0ffa825
Built:             Wed Jul 18 19:05:28 2018
OS/Arch:           windows/amd64
Experimental:      false
Server:
Engine:
Version:          18.06.0-ce
API version:      1.38 (minimum version 1.24)
Go version:       go1.10.3
Git commit:       0ffa825
Built:            Wed Jul 18 19:23:19 2018
OS/Arch:          windows/amd64
Experimental:     false
And I have this simple Dockerfile
FROM microsoft/iis
RUN powershell -NoProfile -Command Remove-Item -Recurse C:\inetpub\wwwroot*
WORKDIR /inetpub/wwwroot
COPY Web/ .
I build the container with
docker build -t cpp-site .
Then I run it with
docker run -d -p 8000:8000 --name cpp-site cpp-site
I get the address of the running container and then go to my browser (Chrome) and go to the address http://xx.xx.xx.xx:8000 and I just get a timeout error. I looked at the logs (docker logs xxxx returns no output). I tried to exec into the running container to see if I could see anything amiss. Nothing.  Any hints as to how I would find what is tripping up my container?
Thank you.
Kevin

Hi Kevin,
I also run into this issue, and have some findings to share,
1st of all, I think this doc has mistake at this command line
docker run -d -p 8000:8000 --name my-running-site iis-site,
because in the container, there is no opening port 8000 at all, the only open port in it, is 80
#in the container
PS C:\inetpub\wwwroot> netstat -ano | findstr "80"
TCP    <IP_ADDRESS>:80             <IP_ADDRESS>:0              LISTENING       4
TCP    [::]:80                [::]:0                 LISTENING       4
2nd. To access the container you can do it in two ways,
*) as you've followed the README, you can access by the IP and port 80, in Chrome, don't use 8000.
or
*) run the command like below, to expose 80 to an unused port on local PC,e.g. 8011,
docker run -d -p 8011:80 --name my-running-site6 iis-jw-site01
then access in Chrome,  <IP_ADDRESS>:8011
Hope this help you too.

@KevinBurton As pointed out by @wcs9108 you're mapping to the incorrect port in the container. Trying mapping to port 80 inside the container instead as that the port the default site binding in IIS is listening on.

Thank you. This moved me forward.  Now the Chrome request is giving me "Sorry, an error occurred while processing you request."


Fixes #95

Note that this may also fix a potential bug when using a gpu. We were moving our model to gpu after creating the optimizer object instead of before.

Okay I’m fine reverting to what you had then, makes sense
On Tue, Nov 20, 2018 at 12:23 AM Braden Hancock<EMAIL_ADDRESS>wrote:

Treating "requires_grad=False" as "no grads and no initialization" may be
fine for now, but I can imagine cases where those will be different. I
agree silent failures are bad, but the issue before was that we ended up
having about a dozen modules in our whitelist of modules that didn't have
and didn't need any initialization (e.g., dropout, loss modules, every
model, many of our baselines, etc.) and that list was always growing. It
was going to be hard to maintain well and made it difficult for
user-provided modules to work the first time, since each of them would
raise the error until the user went and added a stub method (for
user-provided modules) or their name to the whitelist (for built-in modules
like CrossEntropyLoss). We could change it to a warning, but I'm afraid
that we'll quickly stop looking at those warnings when they print every
time for many modules. So I'd say if the user wants to plug into the
_reset_module() chain, they can, but they're not required to.
Separately, I don't think that we want to be resetting our model every
time we use train_model()--that would disallow a number of standard things
like pretraining/fine-tuning. I think if a user has some routine that will
be calling train_model() multiple times, they should be responsible for
deciding when/how to reset it.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/HazyResearch/metal/pull/96#issuecomment-440184524,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABgw_dMeJmABf9Dh4QebNkSvRsQ-9SP8ks5uw7wUgaJpZM4YpI7H
.


feat(challenger): refactor

The Challenger smart contract has been updated with some of the latest ideas and external feedback.
Some implementations that are TODO are marked as such

Can we close this @merklefruit?

yup


Error: Unable to 'pub upgrade' flutter tool.

Hi, I'm trying to install flutter via snapp_installer install and get the following error

********* Check Flutter Channel *********

Checking Flutter channel
Downloading Linux arm64 Dart SDK from Flutter engine c4cd48e186460b32d44585ce3c103271ab676355...
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100  212M  100  212M    0     0  5483k      0  0:00:39  0:00:39 --:--:-- 6186k
Building flutter tool...
/home/aka/sdk/flutter/bin/internal/shared.sh: line 23: /home/aka/sdk/flutter/bin/cache/dart-sdk/bin/dart: cannot execute: required file not found
Error: Unable to 'pub upgrade' flutter tool. Retrying in five seconds... (9 tries left)
/home/aka/sdk/flutter/bin/internal/shared.sh: line 23: /home/aka/sdk/flutter/bin/cache/dart-sdk/bin/dart: cannot execute: required file not found
Error: Unable to 'pub upgrade' flutter tool. Retrying in five seconds... (8 tries left)
/home/aka/sdk/flutter/bin/internal/shared.sh: line 23: /home/aka/sdk/flutter/bin/cache/dart-sdk/bin/dart: cannot execute: required file not found
Error: Unable to 'pub upgrade' flutter tool. Retrying in five seconds... (7 tries left)
/home/aka/sdk/flutter/bin/internal/shared.sh: line 23: /home/aka/sdk/flutter/bin/cache/dart-sdk/bin/dart: cannot execute: required file not found
Flutter Doctor:

************ Doctor summary: ************

Installation:
✓ | Linux Dependency packages | curl git unzip ...
* ✓ curl is installed
* ✓ git is installed
* ✓ unzip is installed
* ✓ xz-utils is installed
* ✓ zip is installed
* ✓ libglu1-mesa is installed
✓ | Flutter folder is located in /home/aka/sdk/flutter.
Flutter command not found globally. Checking common directories...
Flutter installation found at: /home/aka/sdk/flutter/bin/flutter
✓ | Flutter command is available in the PATH.
✓ | Flutter command is defined in .bashrc file.
✓ | Linux development packages | clang cmake ninja-build ...
* ✓ clang is installed
* ✓ cmake is installed
* ✓ ninja-build is installed
* ✓ pkg-config is installed
* ✓ libgtk-3-dev is installed
* ✓ liblzma-dev is installed
Kiosk Mode: unknown
✘ | Auto login is enabled in your device
✘ | Kiosk mode is enabled
✘ | Kiosk mode packages | xset xdotool ...
* ✘ x11-xserver-utils is missing
* ✘ xdotool is missing
System info:
RPI:
Raspberry PI 4B 8GB
Raspian 32Bit Headless
Host:
Windows 11
Flutter Version: 3.19.6
Let me know if you need more informations.
kind regards

Hi there, same problem here, how did you fix it?

Hi there, same problem here, how did you fix it? @RootKid87

@joaoparis You need to check the flutter version on your machine and your PI. If you your machine and PI have different flutter version try to upgrade with command: flutter upgrade

Hi there, same problem here, how did you fix it? @RootKid87

@joaoparis You need to check the flutter version on your machine and your PI. If you your machine and PI have different flutter version try to upgrade with command: flutter upgrade

thanks for the tip, but I'm still stuck. when running "snapp_installer install" this is my output:
No Flutter installations found.
Flutter command not found globally. Checking common directories...
No Flutter installations found.
******************************************
********** Installation Summary **********
******************************************

* Get Linux dependencies to be able to run the flutter: curl, git ,...
* Clone the flutter from Github: https://github.com/flutter/flutter
* move flutter to the /home/pi/sdk/flutter directory
* update the path to make the flutter command excutable
* Check and show the flutter path
* add dependencies for developing on linux
* check the flutter channel
* run flutter doctor
* run flutter precache command for linux

Version option not specified.
Installing the latest version of Flutter on the stable channel.

******************************************
******* Flutter Linux Dependencies *******
******************************************
Linux dependencies are already installed
    * ✓ curl is installed
    * ✓ git is installed
    * ✓ unzip is installed
    * ✓ xz-utils is installed
    * ✓ zip is installed
    * ✓ libglu1-mesa is installed

*****************************************
************* Clone Flutter *************
*****************************************
Clone the stable branch of the Flutter repository from GitHub

Cloning into 'flutter'...
remote: Enumerating objects: 521754, done.
remote: Counting objects: 100% (796/796), done.
remote: Compressing objects: 100% (553/553), done.
remote: Total 521754 (delta 434), reused 452 (delta 222), pack-reused 520958
Receiving objects: 100% (521754/521754), 135.83 MiB | 14.66 MiB/s, done.
Resolving deltas: 100% (408533/408533), done.
Updating files: 100% (8283/8283), done.
Flutter cloned successfully.
Flutter moved to the /home/pi/sdk directory

*****************************************
********** Add Flutter to PATH **********
*****************************************
adding flutter to PATH as env in ~/.bashrc.

******************************************
*********** Check Flutter Path ***********
******************************************

Flutter is located at:
/home/pi/sdk/flutter/bin/flutter

******************************************
***** Linux Development Dependencies *****
******************************************
Linux development dependencies are already installed
    * ✓ clang is installed
    * ✓ cmake is installed
    * ✓ ninja-build is installed
    * ✓ pkg-config is installed
    * ✓ libgtk-3-dev is installed
    * ✓ liblzma-dev is installed

*****************************************
********* Check Flutter Channel *********
*****************************************
Checking Flutter channel

Downloading Linux arm64 Dart SDK from Flutter engine edd8546116457bdf1c5bdfb13ecb9463d2bb5ed4...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  220M  100  220M    0     0  31.0M      0  0:00:07  0:00:07 --:--:-- 37.5M
Building flutter tool...
/home/pi/sdk/flutter/bin/internal/shared.sh: line 23: /home/pi/sdk/flutter/bin/cache/dart-sdk/bin/dart: No such file or directory
Error: Unable to 'pub upgrade' flutter tool. Retrying in five seconds... (9 tries left)
/home/pi/sdk/flutter/bin/internal/shared.sh: line 23: /home/pi/sdk/flutter/bin/cache/dart-sdk/bin/dart: No such file or directory```

@joaoparis try to delete the sdk folder sudo rm -r sdk. And retry to install by running sudo snapp_installer install

@RootKid87 thanks for the suggestion. I partially did what you suggested, aside from "sudo snapp_installer install", that retruned: "sudo: snapp_installer: command not found" (that's normal right?)
I ran it without being the super user, and still got the same result.
Moreover, I noticed #10 was adding flutter_pi to the script. I used to have flutter_pi installed on this PI, which was what I was using to run my Flutter App on this machine. Could it be that some configuration is still there and it's causing the issue? I have removed flutter_pi the best way I know how: rm the folder and removing it from the PATH.

@joaoparis Ok, unfortunately I don't know how your config looks like. You have a 64Bit raspian on your PI?

@RootKid87  i think this is what your are asking:

I don't have access to the PI physically right now :( but I'll try to do that on Monday

Let me know if it worked. Have a nice weekend.

Will do! Meanwhile, I'm going to ping @payam-zahedi maybe he can help ;)
Have a good weekend!!

Hey @joaoparis
I'm not sure about your need in here?
What exactly is your problem?
you want to install flutter on raspberry? Or you want to install flutter-pi?

If you want to have a specific version of flutter, just use this command.
snapp_installer install -v 3.22.0 -f


Redirect (Turbolinks-Location) example

Could anyone show me a Turbolinks-Location example using only Javascript?
I've tried something like this but it's not working:
document.addEventListener("turbolinks:request-end", function(event) {
  var xhr = event.data.xhr;
  xhr.setRequestHeader('Turbolinks-Location', './build');
});

Turbolinks.visit('./build/permalink', {action: 'restore'});

The visit command works fine, keeps the scroll position and calls the controller (it's a php application), but the url should go back to '/build' instead of '/build/permalink'.
Thanks for now.

You need to add the Turbolinks-Location to the server response, not the request.

How to set headers on server side:
Rails
In rails this is done for redirect_to automatically by the Turbolinks gem.
PHP
In PHP you can do something like this: header('Turbolinks-Location: /redirected_path');
node.js
in node.js this works: res.header('Turbolinks-Location', '/redirected_path');

Closing as resolved by https://github.com/turbolinks/turbolinks/issues/200#issuecomment-364039452. Thanks!


McLaren Commit #5:

Adding mdf_start_time to metadata.
Aimed at solving issue 76 --> https://github.com/microsoft/adx-automotive-demos/issues/76

This solution enables calculating the offset of the recording - enabling usage of timeseries on the recording as part of a postprocessing activity.


feat(Theming): Adds theming support with documentation.

Please verify the following:

[x] yarn test jest tests pass with new tests, if relevant
[x] yarn lint eslint checks pass with new code, if relevant
[x] yarn format:check prettier checks pass with new code, if relevant
[x] README.md (or relevant documentation) has been updated with your changes

Describe your PR
This PR adds theming support to the ignite boilerplate as well as relevant documentation on how to use the theming system.
Note that this PR is onto the v10 branch and not main.

Before we merge this, I want to make sure we get some real-world experience. @markrickert you're starting a new project soon, so this will be a good opportunity for that.

@jamonholmgren Absolutely. I'll keep this branch up to date with main and use this as the starting point for my next project.

Rebased on the v10 branch and fixed a few issues caused by the Switch/Toggle rewrite

TODO: need to check over the showroom issue: https://infiniteredcommunity.slack.com/archives/C41PK1LAY/p1716828905367989?thread_ts=1716662953.807429&cid=C41PK1LAY

This is ready for another review.

Looking into root cause of the screen jumping in this video (running on Android device), as I don't think it's themeing, but it does point out to me that I think we should be setting the actual app background as a dark background color so we don't get any white strips accidentally. Since we have expo-system-ui, I think https://docs.expo.dev/versions/latest/sdk/system-ui/#systemuisetbackgroundcolorasynccolor should do it?
https://github.com/user-attachments/assets/f463b03b-aaa4-421b-9083-bf91d132666c

@markrickert I do think we should address https://github.com/infinitered/ignite/pull/2636#issuecomment-2261960598. It not only makes the demo look better, but there are likely to be other use cases where somebody needs to set a theme on non-React components (i.e., imperatively on a native component).
I took a stab at it:
https://github.com/user-attachments/assets/35e79a49-9813-41df-861f-ad9876ea19c7
Here's my diff:
diff --git a/app/utils/useAppTheme.ts b/app/utils/useAppTheme.ts
index 8d03a0c..712bf81 100644
--- a/app/utils/useAppTheme.ts
+++ b/app/utils/useAppTheme.ts
@@ -1,4 +1,4 @@
-import { createContext, useCallback, useContext, useMemo, useState } from "react"
+import { createContext, useCallback, useContext, useEffect, useMemo, useState } from "react"
 import { StyleProp, useColorScheme } from "react-native"
 import { DarkTheme, DefaultTheme, useTheme as useNavTheme } from "@react-navigation/native"
 import {
@@ -9,6 +9,7 @@ import {
   lightTheme,
   darkTheme,
 } from "app/theme"
+import * as SystemUI from 'expo-system-ui';

 type ThemeContextType = {
   themeScheme: ThemeContexts
@@ -23,6 +24,13 @@ export const ThemeContext = createContext<ThemeContextType>({
   },
 })

+const themeContextToTheme = (themeContext: ThemeContexts): Theme => (
+    themeContext === "dark" ? darkTheme : lightTheme)
+
+const setImperativeThemeing = (theme: Theme) => {
+  SystemUI.setBackgroundColorAsync(theme.colors.background)
+}
+
 export const useThemeProvider = (initialTheme: ThemeContexts = undefined) => {
   const colorScheme = useColorScheme()
   const [overrideTheme, setTheme] = useState<ThemeContexts>(initialTheme)
@@ -34,6 +42,10 @@ export const useThemeProvider = (initialTheme: ThemeContexts = undefined) => {
   const themeScheme = overrideTheme || colorScheme || "light"
   const navigationTheme = themeScheme === "dark" ? DarkTheme : DefaultTheme

+  useEffect(() => {
+    setImperativeThemeing(themeContextToTheme(themeScheme))
+  }, [overrideTheme])
+
   return {
     themeScheme,
     navigationTheme,
@@ -78,7 +90,7 @@ export const useAppTheme = (): UseAppThemeValue => {
   )

   const themeVariant: Theme = useMemo(
-    () => (themeContext === "dark" ? darkTheme : lightTheme),
+    () => themeContextToTheme(themeContext),
     [themeContext],
   )

My only concern here is that we haven't documented using useThemeProvider, which is what would run this. Is that meant to be a required part of the theming system?

Pushed commit with changes to set native background color after discussion with @markrickert https://github.com/infinitered/ignite/pull/2636/commits/4fb4d6127d63aba9790fa396fbc3011a38898dfe

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


When meltano.yml is updated programatically, order of object and set values changes

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2085
Originally created by @DouweM on 2020-06-08 16:59:34

As seen in https://gitlab.com/meltano/demo-project/-/commit/166866c84ae8105e3300dbbeae98c3265ff7d3d3

View 2 previous comments from the original issue on GitLab


add METHOD convert_to_table.

METHOD convert_to_table.

For excel upload, This is counterpart to BIND_TABLE
This method has 2 exporting parameters.
ER_DATA is a reference to internal table of string columns. intend to get raw data without conversion loss.
ET_DATA is an internal table supplied. There may be losses in the conversion(MOVE-CORRESPONDING).

Request from: https://github.com/boy0korea/ABAP2XLSX_HELPER/issues/1#issue-1144673227

Before I begin the review, one quick question: have you noticed that now the demos and the main library are split into two distinct repositories? We would need a demo for this new method and that demo goes into the new "demos" repository.

@boy0korea do you have time for the demo? No pressure, just want to know if I should take care of that.

1 issue from linter: is https://github.com/abap2xlsx/abap2xlsx/pull/1056 waiting to be merged

@boy0korea could you please look at the linter messages?
Also, what about the demo program?

@boy0korea could you please look at the linter messages? Also, what about the demo program?

demo: https://github.com/abap2xlsx/demos/pull/23

1 issue from linter: is #1056 waiting to be merged

How do we merge this? It's in another branch which requires this to be merged first which requires the linter to be happy. I'd simply fix the issue in the PR branch and be on my way.


Typo in error message: The Transaction has wrong format or unexpcted...

Environment: Windows, Linux
How to produce:

Run a pchain command:

pchain.exe submit tx --to-address null --value 0 --tip 0 --gas-limit 10000000 --gas-price 1 --data "hello_contract.wasm" --nonce 1 --keypair-name keypair.json

Error mesage:

The Transaction has wrong format or unexpcted problem encountered. The transaction is not sent.

Typo: unexpcted

Thanks for your reporting.
The latest version of pchain_client binary can be downloaded, includuing the mentioned fixes.
(as of 21-Nov)
Windows
https://cms.parallelchain.io/parallelchain-client_v0.3.1_win.zip
Linux
https://cms.parallelchain.io/parallelchain-client_v0.3.1_linux.tar.xz
Mac
https://cms.parallelchain.io/parallelchain-client_v0.3.1_mac.tar.xz


Typescript syntax highlight breaks on function calls with multiline generics

VSCode Version: 1.36.1
OS Version: macOS 10.14.5

Steps to Reproduce:

create a new typescript file with the provided code content// Function declaration syntax highlight works as expected
const fn = <
  ThisVeryLongTypeParameterNameTellsALotOfHowTheParamIsUsedInThisFunction,
  B
>(
  a: ThisVeryLongTypeParameterNameTellsALotOfHowTheParamIsUsedInThisFunction,
) => {
  return {} as B;
};

type ATypeWithAnIncrediclyLongYetStrangelyDescriptiveName = {};
type AnOtherTypeWithAVeryLongName = {};

// Function call syntax hightlight breaks
fn<
  ATypeWithAnIncrediclyLongYetStrangelyDescriptiveName,
  AnOtherTypeWithAVeryLongName
>({});

Review the broken syntax highlight at the function call:

Does this issue occur when all extensions are disabled?: Yes

This is working as per limitation of tmLanguage grammar since it doesnot allow multiline lookup. At fn we don't know if its a less than expression or call and hence the coloring.


⚠ Secret Website 3 has degraded performance

In be4a5b5, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2483 ms

Resolved: Secret Website 3 performance has improved in 7913ead.


Bot probably shouldn't ask regression-author to set the severity field on regressions, particularly if they're not full-time MoCo folks

Quoting https://bugzilla.mozilla.org/show_bug.cgi?id=1807769#c3 :

:mrobinson, since you are the author of the regressor, bug 1791759, could you take a look? Also, could you set the severity field?

mrobinson is a community member (working at Igalia on some gecko features), and probably isn't familiar with our severity levels or triage process. We probably shouldn't be requesting that such folks set severity, since it's implicitly asking that they do one of the following (neither of which we really want to request for non-full-time folks):
(1) we're asking them to learn the details of the various rating-levels & triage process (which isn't super discoverable & is asking a lot of them)
...or:
(2) we're asking them to just make a gut assessment, e.g. "I dunno, on a scale of 1-5 this seems like a 4, so let's call this S4"
Also, it takes the bug out of the triage queue for the relevant component, which means the triagers in that component may not notice the bug (it won't be in the untriaged-bugs list).  This may or may not be OK, depending on how diligent the needinfo'd person is on following up on the bug.
So: the bot probably shouldn't be flagging arbitrary authors-of-regressing-bugs to set severity levels.  I'd suggest either removing that request, or that we only include it after doing some sort of "is this person likely to already understand our triage/severity-rating process" check (to the extent that we can implement such a check).

Yeah, editbugs isn't a good signal here -- it's a very low bar.  We often just hand that out when folks have helped out on a bug or two (since they need it if they e.g. want to post a reduced testcase on a bug that they didn't themselves file).
I think editbugs might technically give users the ability to adjust Severity/Priority fields (and most other fields), but it's not at all a signal that they understand (or should concern themselves with understanding) our triage process.

We could rely on the people.json then, and only do this if the user is in people.json.


Release zip is outdated

The brew cask install is outdated due to using an old release zip. Please generate a new build as there's a year's worth of updates not included.

@Mortennn

@robmonte

@gingerbeardman
Why tag me back?


Add Media struct and UploadMediaFromMedia method

https://docs.joinmastodon.org/methods/statuses/media/
Fixes #129.

Codecov Report

Merging #130 into master will decrease coverage by 0.75%.
The diff coverage is 37.77%.

@@            Coverage Diff             @@
##           master     #130      +/-   ##
==========================================
- Coverage   80.10%   79.35%   -0.76%
==========================================
  Files          29       29
  Lines        1503     1511       +8
==========================================
- Hits         1204     1199       -5
- Misses        205      219      +14
+ Partials       94       93       -1

Impacted Files
Coverage Δ

status.go
74.48% <31.70%> (-11.14%)
:arrow_down:

mastodon.go
74.00% <100.00%> (+3.77%)
: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 3e91c76...8345c00. Read the comment docs.

Thank you


[Feature][Branch] Select -> "Title" section

Description

props of "Title" section.

title
titleTextStyle

(Please do not merge this until I get Todo list done :)
Related Issues
#60
Tests
Will do it...
Todo

[ ] 1. Get reviews from team members
[ ] 2. Fix the code
[ ] 3. Write testing code

Checklist
Before you create this PR confirms that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the [CLA].
[x] Run yarn test or yarn test -u if you need to update snapshot.
[x] Run yarn lint
[x] I am willing to follow-up on review comments in a timely manner.

I can be off-topic but looks like @smallbee3  has merged the branch instead of rebasing which causes bringing all the prev commits to history.

@hyochan Your opinion is correct.
We pulled each other's sources together for rapid development. But We are wondering if it was the right way to manage history. The sample you provided seems to be the direction we're going to take, I'll spread it to our team and go in the direction you suggest.

It seems my new pull request is ok except that it lacks testing code. Right?
I started my commits from upstream/feat/select as what geoseong did with his branch.
Welcome any comments :)


Allows definition of custom HDF proxy through SWIG and small bug fixes

What does this implement/fix? Explain your changes.
Allows definition of custom HDF proxy through SWIG
Also:

add custom HDF proxy example in C#
fix an issue that occurs when receiving a PutDataArrays in the ETP server example
fix an issue that occurs when providing a nullptr proxy parameter in WellboreTrajectoryRepresentation::setGeometry

Does this close any currently open issues?
No.
Any relevant logs, error output, etc?
No.
Any other comments?
No.
Where has this been tested?
Operating System: Windows 10 64bits
Platform: Visual Studio Community 2019

Thank you for the review as well as for the encouraging comment.


Refactor structure of OASIS service

To be discussed with @wangyoucao577
// this two paragraphs will be placed in doc :)
OASIS stands for Optimized Charge Station Selection Service, which mainly supports generating optimal charge station candidates for given query based on user's vehical information.  It expects routing engine(like OSRM) to generate electronic spcific cost and only focus on how to choose most optimal charge station combination which balances charging cost(charging time, payment, etc).
For example, OASIS might suggest user spend 1 hour to charge at station A for x amount of energy then reach destination with 2 hours in total, or charge at station B and station C for 30 minutes each and reach destination with duration of 2 hours 10 minutes.
Structure layer

layer 1: handles oasis request and generate oasis response

target is handling logic related with http, rgc related framework
call NewSelector().GenerateSolution()

layer 2:

Selector is the layer contains logic for how to select charge stations, such as
* whether need charge or not
* is destination reachable by single charge
* multiple charge station solution finding, which could be search along route or charge station based routing, implemented by other packages
solution and GenerateSolution abstract the interface for how to generate multiple charge station solution
stationgraph implements the GenerateSolution interface, and this package represents algorithm, graph in this diagram
chargingstrategy abstract logic of charge which supports calculation in stationgraph

layer 3: chargestation abstract logic related with how to handle chargestations

The major abstraction is value object of Place and operations(usecases) on that object
topological_querier_interface
spatial_querier_interface
place_algorithm
more details could go to below

File structure

├── api
│   └── oasis
│       └── request.go
│       └── response.go
│
├── cmd
│   └── oasis
│   └── place-connectivity-gen
│
├── service/oasis
│   │
│   ├── userinterface
│   │   ├── http
│   │          └── handler.go
│   │
│   ├── solution
│   │          └── selector
│   │               └── current files in selectionstrategy
│   │          ├── domain
│   │          │   ├── entity
│   │          │   │      └── solution.go
│   │          │   ├── usecases
│   │          │          └── solution_interface.go
│   │          └── stationgraph
│   │          └── chargingstrategy
│   │          └── chargestation
│   │               ├── domain
│   │               │   ├── entity
│   │               │         └── place.go
│   │               │   ├── usecases
│   │               │         └── topological_querier_interface.go
│   │               │         └── spatial_querier_interface.go
│   │               │         └── location_querier_interface.go  // may be put into spatial_querier_interface.go
│   │               │         └── place_algorithm.go
│   │               │
│   │               │
│   │               ├── repository   // communication with other services
│   │               │      └── connector_interface.go
│   │               │      └── osrmconnector
│   │               │      └── searchconnector
│   │               │
│   │               ├── spatialquerier
│   │               │      └── current files in spatialindexer, implements spatial querier interface
│   │               │
│   │               ├── topoquerier
│   │               │      └── current files in stationconnquerier, implements topological_querier_interface
│   │               │
│   │               ├── placealgorithm
│   │               │      └── current files in stationfinder except querier_impl_by_stationfinder
│   │               │
│   │               ├── topobuilder
│   │               │      └── current files in connectivitymap
│   │               │

place.go
// Place records place(location, point of interest) related information such as
// ID and location
// Place represents charge stations for most of times for OASIS service, but it
// could also represent for a user select location such as original location or
// destination location.
type Place struct {
   ID       PlaceID
   Location *nav.Location
}

// PlaceID defines ID for given place(location, point of interest)
// The data used for pre-processing must contain valid PlaceID, which means it
// either a int64 directly or be processed as int64
type PlaceID int64

topological_querier_interface.go
// TopoQuerier used to return topological information for places
type TopoQuerier interface {

   // GetNearByPlaces finds near by stations by given placeID and return them in recorded sequence
   // Returns nil if given placeID is not found or no connectivity
   GetNearByPlaces(placeID common.PlaceID) []*common.RankedPlaceInfo

   // LocationQuerier returns *nav.location for given placeID
   LocationQuerier
}

spatial_querier_interface.go
// SpatialQuerier answers special query
type SpatialQuerier interface {

   // GetNearByPlaceIDs returns a group of places near to given center location
   GetNearByPlaceIDs(center nav.Location, radius float64, limitCount int) []*common.PlaceInfo
}

location_querier_interface.go
// LocationQuerier returns *nav.location for given placeID
type LocationQuerier interface {

   // GetLocation returns *nav.Location for given placeID
   // Returns nil if given placeID is not found
   GetLocation(placeID PlaceID) *nav.Location
}

place_algorithm.go
// Algorithm contains algorithm implemented based on NearbyStationsIterator
type Algorithm interface {
   // CalculateWeightBetweenNeighbors accepts locations array, which will search for nearby
   // charge stations and then calculate weight between stations, the result is used to
   // construct graph.
   CalculateWeightBetweenNeighbors(locations []*nav.Location,
      oc *osrmconnector.OSRMConnector,
      finder StationFinder) chan stationfindertype.WeightBetweenNeighbors

   // FindOverlap finds overlap charge stations from given two iterators
   FindOverlap(iterF stationfindertype.NearbyStationsIterator,
      iterS stationfindertype.NearbyStationsIterator) []*stationfindertype.ChargeStationInfo
}

solution_interface.go
// SolutionGenerator generates OASIS solutions
type SolutionGenerator() interface {
   // GenerateOptimumSolutions creates optimal charge solutions
   GenerateOptimumSolutions() []*solution.Solution
}

handler
Handler accepts oasis.Request and put oasis.Response to certain channel.
to do:

move all http related logic into handler, such as http.ResponseWriter.WriterHeader from GenerateSolutions4MultipleCharge
move all logic from handler such as osrmhelper.RequestRoute4InputOrigDest, selectionstrategy.HasEnoughEnergy to package selectionstrategy

selectionstrategy(selector)
It accepts `oasis.Request` and generates `oasis.Response`.

It will generate internal `solution` format following such logic:
- detect wether destination is reachable or not
- detect whether charge is needed or not
- detect whether destination is reachable via single charge, find best solution
- best solution for multiple charge
   + SearchAlongRoute
   + ChargeStationBasedStrategy

to do:

error code definition
mock and testing format for multiple charge

Package stationgraph
Ideally, stationgraph should able to let people easy to implment graph algorithm based on graph interface
https://user-images.githubusercontent.com/16873751/81879163-d60e6500-953e-11ea-9d49-f9ccae39cccc.png
algorithm
      // relax
      node := g.Node(currID)
      for _, targetID := range g.AdjacentNodes(currID) {
         if g.Edge(currID, targetID) == nil {
            glog.Errorf("No connectivity between %+v and %+v which is unexpected, check your logic.\n", currID, targetID)
         }

         len := g.Edge(currID, targetID).Distance
         t := g.Edge(currID, targetID).Duration

         // todo
         // !g.Node(currID).tooNearToCurrentNode(g.Node(targetID), len)
         if g.Node(currID).reachableByDistance(len) {
            chargeTimeNeeded := g.Node(targetID).calcChargeTime(node, len, g.ChargeStrategy())
            if m.add(targetID, currID, len, chargeTimeNeeded+t) {
               g.Node(targetID).updateArrivalEnergy(node, len)
               g.Node(targetID).updateChargingTime(chargeTimeNeeded)
            }
         }
      }

Dijkstra should only depend on Graph
Node should be a value object, move its functionality to other places, such as chargestrategy, only keep setter and getter there
chargestrategy should provide functionality to

isTargetDistanceReachable()
calculateArrivalEnergy
calculateChargingTime

station_graph

Current station_graph_test.go is more like integration test, and which expose the implemenation detail of a connectivity interface, which should be replaced to mock: when(input)->Then(returns) for interfaces

graph_mock

change name from mockGraph to fakeGraph

stationfinder

put CalcWeightBetweenChargeStationsPair as internal function

package chargingstrategy

change null_charge_strategy to fake_charge_strategy
change fake_charge_strategy to simple_charge_strategy

Looks really good!
Some questions for discussing:

cmd/oasis->cmd/oasisd
cmd/place-connectivity-gen->cmd/place-connectivity-builder
put handler.go in service/oasis directly
put solution.go and solution_interface.go in service/oasis/solution directly (what I'm learning from recently ranking service api/osrm refactoring is that don't over splitting packages)
what's the relationship between selector and chargestrategy?
why name repository for the connectors?
Maybe it's time to have a doc along with the(https://github.com/Telenav/osrm-backend/blob/master/integration/doc/oasis/api.md) that descibes algorithms of oasis to help to understand the code, ha!

Looks really good!

[Perry] Thanks for your feedback, more strength to move on.

Some questions for discussing:

cmd/oasis->cmd/oasisd

[Perry] Also think about to make this change.

cmd/place-connectivity-gen->cmd/place-connectivity-builder

[Perry] agree

put handler.go in service/oasis directly
put solution.go and solution_interface.go in service/oasis/solution directly (what I'm learning from recently ranking service api/osrm refactoring is that don't over splitting packages)

[Perry] yes, don't over splitting package

what's the relationship between selector and chargestrategy?

[Perry] selector implements high level functions, such as detecting whether no charge is needed, or just one charge could reach destination, or multiple charges
chargestrategy is a package which abstracts time spend in charge stations and new energy got.
Let me think more about names and how to make there relationship more clear.

why name repository for the connectors?

[Perry] Learn from here and here, seems people like to put logic which communicate with external components like message bus, database into a folder called repository

Maybe it's time to have a doc along with the(https://github.com/Telenav/osrm-backend/blob/master/integration/doc/oasis/api.md) that descibes algorithms of oasis to help to understand the code, ha!

[Perry] Totally agree.

Discussion with @wangyoucao577 on 06012020.


rcljava_examples build failure

When attempting to follow the build instructions, ament bails with an error when building rcljava_examples:
* Where:
Build file '/home/mallan/ros/ros2_java_ws/build_isolated/rcljava_examples/build.gradle' line: 2

* What went wrong:
A problem occurred evaluating root project 'rcljava_examples'.
> Failed to apply plugin [id 'org.ros2.tools.gradle']
   > Could not create an instance of type org.ros2.tools.gradle.JavaAmentPluginExtension_Decorated.

the root exception is
Caused by: org.gradle.api.internal.MissingMethodException: Could not find method findProperty() for arguments [ament.source_space] on root project 'rcljava_examples'.

The full error is attached below - this is on Ubuntu 16.04. Let me know if you need more info or if I can help.
examples_build_failure.txt

Thanks for filing this issue. What version of Gradle do you have? The ros2_java Gradle plugin needs 3.2 or higher. If you're using the package that comes with 16.04, you can install a more recent version of Gradle via the following PPA:
sudo add-apt-repository ppa:cwchien/gradle
sudo apt install -y gradle

let me know if that helps.

Ah, thanks for the quick reply - yes, Gradle version was the issue. I had the 16.04 default (2.1), after updating the build succeeded.  It'd be helpful to have that in the install instructions (or is it there and I missed it?)  Cheers

Awesome! I kept meaning to update the instructions (I had left #35 open to remind myself). I've finally made a quick edit with the instructions for installing Gradle on Ubuntu https://github.com/esteve/ros2_java/commit/053c8b4329364dbdf5331d570cbb9cdd86f009c0


fix(server-routes): Filter by Collection

🐛 The bug
the current filter by colelction in server-routes put only the routes that are in a folder into a collection, example:

vscode

devtools

🛠 To reproduce
n/a
🌈 Expected behavior
Is it a good idea to make it like this:

ℹ Additional context
No response

Sounds good. I think we should have a button to toggle between tree/list views

Sounds good to me too

Sure!


[5.8] fix MorphTo Relation ignores parent $timestamp when touching

Fixed bug with touching model which has $timestamps set to false resulting in Illuminate\Database\QueryException: SQLSTATE[HY000]: General error: 1 no such column: updated_at
for source issue and more details see #28638

Tests seem to be failing and there are some styling issues. Feel free to resubmit with the fixes.


feat: support other package managers

🚥 Resolves #894

🧰 Changes
Added a detection logic for deno, bun, yarn & pnpm before running the install command.
🧬 QA & Testing
Running this project in the above environments to try if the package gets installed correctly.

Per @erunion's comment, could you use preferred-pm instead? I'd feel much better about deferring to a maintained library with coverage for this logic as opposed to maintaining it ourselves. Thanks @DaniAkash!

Sure! I'll add the change 👍🏽

has this landed on main?
I am getting:
✖ Installing required packages
Command failed with exit code 1: npm install --save /Users/joscha/dev/.../.api/apis/affinity
npm error Cannot read properties of null (reading 'matches')

npm error A complete log of this run can be found in: /Users/joscha/.npm/_logs/2024-08-28T10_53_59_578Z-debug-0.log

when running api install with deno 1.45.5 and api 6.1.2.
I can't find any mention of #894 in the changelog on https://github.com/readmeio/api/releases, however 21230f8 I can see it is part of v7.0.0-beta.8

Hey @joscha — this change is exclusive to the v7 channel (tracked via main). We have no plans to backport this to the v6 channel (tracked via v6). With the exception of critical security fixes, all current and upcoming feature work will be on the v7 channel.

this change is exclusive to the v7 channel
Understood. It doesn't seem to be working as expected on 7.0.0-beta.8 for me as far as I can tell. See second part of the log above.

@joscha mind filing a new issue for this? Thanks!

@joscha mind filing a new issue for this? Thanks!

Sorry, after there was no answer and I also read somewhere else in this repo that it doesn't support distribution, I moved to https://github.com/OpenAPITools/openapi-generator.


Can ink() method use the value to be redacted?

Just found your package, great work by the way
Ive got a use case where I'd like to partially redact a value.
<EMAIL_ADDRESS>=><EMAIL_ADDRESS>
Just wondering if this was possible ?

Sorry looking in more depth I don't think this is doable as, should have read the source in more depth before opening.

Currently the ink method only does a full redaction. It would be possible if the class was modified to accept a closure. The closure would be executed for every redaction, though. I'd gladly accept a PR of this feature.

    /**
     * Set the value to replace redacted key values with, this can be a string or the return value of a closure.
     *
     * @param  mixed $ink What should replace the redacted data
     * @return object \Mtownsend\ArrayRedactor\ArrayRedactor
     */
    public function ink($ink = "[REDACTED")
    {
        $this->ink = is_callable($ink) ? $ink() : $ink;
        return $this;
    }

Does this look OK ? Not made a submission before but this looks like it could add something to an already useful package.


useEnsAddress on L2s

Describe the bug
ChainDoesNotSupportContract: Chain "OP Mainnet" does not support contract "ensUniversalResolver". This could be due to any of the following: - The chain does not have the contract "ensUniversalResolver" configured. Version<EMAIL_ADDRESS>at
Link to Minimal Reproducible Example
No response
Steps To Reproduce
Config:
export const config = createConfig({
  chains: [mainnet, polygon, arbitrum, optimism, sepolia, base],
  multiInjectedProviderDiscovery: false,
  transports: {
    [mainnet.id]: http(),
    [polygon.id]: http(),
    [arbitrum.id]: http(),
    [optimism.id]: http(),
    [sepolia.id]: http(),
    [base.id]: http(),
  },
})

Call
const ensAddress = useEnsAddress({
  name: normalize('sergiy.eth'),
  chainId: 10 //optimism.id
})

Wagmi Version
2.5.7
Viem Version
2.7.6
TypeScript Version
5
Check existing issues

[X] I checked there isn't already an issue for the bug I encountered.

Anything else?
No response

Error message explains the issue. ENS contracts are not deployed.

Error message explains the issue. ENS contracts are not deployed.

So this is at the chain level? Is there a way to view which chains have the contract deployed? Latest version of wagmi seems to only allow 1 as the chainId parameter, so it's mainnet only?

https://docs.ens.domains/web/multichain


Put rpc gas cap in EthApi in a new type

Describe the feature
the configurable rpc gas cap accepts an u64
https://github.com/paradigmxyz/reth/blob/eb32fd3c6d2feec5e845981ecbe91139656e6bff/crates/rpc/rpc/src/eth/api/mod.rs#L79-L86
which should default to 50M: see https://github.com/paradigmxyz/reth/pull/3812
to make this easier we should put this in a new type so you can invoke it via Default::default()
TODO
add new type wrapper for gas cap in rpc module
change new to accept impl Into<GasCap>
Additional context
No response

Hey, can I work on this?

yes!


The URL given in help does not match the repository URL

https://github.com/metasys-server/powershell-metasysrestclient/blob/ec498315e7e809f85403def5aa76bc5463456120/MetasysRestClient/Invoke-MetasysMethod.psm1#L110
I tried using this for more information and got a 404. I believe powershell-metasysrestapi should instead be powershell-metasysrestclient. I ended up finding the right repo through a search which matched this line of code.

sorry I missed the notifications on all my feature requests.


Clarify docs for 'username' parameter in user.show

The docs are unclear about how to retrieve user information for the currently authenticated user via GET '/user' vs GET /users/:username
Useful for identifying users via Github OAuth

Cool, thanks!


Update vendordeps to use current 2024 libraries and Updates Names for REVLib

This updates _vendordep_s to use the currently available set of 2024 libraries.

I extended the PR to also update to flags that have changed name for REVLib.  This enables the code to execute in simulation.


Enhancement: Fix the entire process of Assembling Collator

Assembling collator is executed every session.
Merge desired and proposed candidate.  Take all the desired candidate first the remaining slot will be given to the sorted proposed candidate with bond amount.
Remove the proposed candidate with zero bond amount.
Remove the existing collator if the address is not found in merged desired and proposed candidate.

This is done: https://github.com/Xode-DAO/xode-blockchain/commit/18f42767c70fb16d98d60341dba5157a198e3edf


Dartium 1.18.1 with package:js resolves to error (dartium, js-interop)

When we try to use package:js in dartium 1.18.1 it resolves to error with static method.
Steps to reproduce

clone https://github.com/jakuub/js-interop-dartium-exception.git
run pub get and pub serve
open http://localhost:8080/auth0.html in dartium (version 1.17 or 1.18.1)

Throwned error
Exception: No static method 'Auth0Settings.' declared in class 'Auth0Settings'.

NoSuchMethodError: method not found: 'Auth0Settings.'
Receiver: Type: class 'Auth0Settings'
Arguments: [...]

Environment
OS: Ubuntu linux and Arch linux (Manjaro)
browser: dartium 1.18.1 (and tried also with 1.17.0) downloaded from https://www.dartlang.org/install/archive
uname -a: Linux masina 4.4.15-3-MANJARO #1 SMP PREEMPT Mon Jul 11 19:04:26 UTC 2016 x86_64 GNU/Linux
I will be very happy if I am only doing somethin wrong, but I can't find out what and it works on Dartium 1.13.0

You need to add
@JS()
to the top of each library that uses JS interop.
That requirement was added to dartium in 1.18 to speedup dartium startup for large applications.

Thank you very much, I didn't notice it in package:js documentation, maybe because it works before, but now I can see, it's there.
Thank you very much, it fixes our problem.


Use correct dashboard definition URL

Applied. Thanks a lot, happy to see this working finally :) Added one more Trello card though as some of the metrics are still empty (not too important for our purposes, though).


@everyone bypass by notSoBot

-t test {node:console.log(String.fromCharCode(0x202E) + "@e"+String.fromCharCode(0x202E)+"veryone"+String.fromCharCode(0x202E))}
bypass discord @everyone restrictions

Can reproduce

happened on my server as well


Update airframe-http, airspec to 21.7.0

Updates

org.wvlet.airframe:airframe-http
org.wvlet.airframe:airspec

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

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

labels: library-update, semver-minor

Superseded by #243.


Type Blob can't edit directly with SQL

Hello, i found your bundle after long time search, it's the only one do ip filters ?
I installed your bundle, and it works very fine but how it's possible to edit ip blacklisted without any PHP execution ?
I got phpmyadmin, i seen the type of ip is Blob, and i can't edit it directly with the panel. phpmyadmin ask me to upload binarie file. I can create this file with text editor like notepadd ?
Or it's possible to add the possibility to choose the type of column IP/IPRange on databases with config file ?
Thanks for help.

Hi,
You can not edit blacklisted IP addresses because because they are stored in binary.
It is the quicklyest way to list and compare IP addresses.
IMO, the best way to edit those addresses is to create a script or console command.

I updated the way I store IP addresses in database.
The IPs stored are now in text, but you will see the hex representation (<IP_ADDRESS> =><PHONE_NUMBER>00000000000000C0A80101).
You can try using the new v2.0.x branch:
composer require "spomky-labs/ip-filter-bundle" "2.0.*@dev"

Be carefull, I changed the intefaces, managers, entities and configuration of the bundle (should be better for developpers now).


Add JKSEntryPKeyCopy.

Subj.

Codecov Report

Merging #11 (3302f4f) into main (9ee5fdb) will decrease coverage by 0.32%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main      #11      +/-   ##
==========================================
- Coverage   65.60%   65.28%   -0.33%
==========================================
  Files          19       19
  Lines        2774     2780       +6
==========================================
- Hits         1820     1815       -5
- Misses        954      965      +11

Impacted Files
Coverage Δ

keylib/jks.c
77.10% <0.00%> (-0.62%)
:arrow_down:

dstulib/compress.c
66.40% <0.00%> (-2.40%)
:arrow_down:

engine/sign.c
70.83% <0.00%> (-1.71%)
:arrow_down:

engine/ameth.c
49.87% <0.00%> (-0.26%)
:arrow_down:

dstulib/key.c
51.02% <0.00%> (-0.18%)
:arrow_down:

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 9ee5fdb...3302f4f. Read the comment docs.


WIP: Started tests on headders and img title and txt

Adds tests related to accessibility.
Link descriptions:

Images need to have both a title and alt-txt ![ A picture of a whale ]( /img/whale.png 'Whale' )
Links need to have a title  ![ Wikipedia article about whales ]( https://whales.com )

Headers

Only one lvl 1 allowed per file
No jumps in depth allowed, e.g. 1->3

Unclear
Should link and images be handled in the same way, i.e both tile and alt-txt required ?

Out of curiosity, what's the status of this PR? Would be nice to have it in production!


Disabling all USB Device behavior except for chosen HID class

I'm trying to define an hid device using ConfigFs.  The problem I run into is that when I try to set up the UDC attribute of the device that I have defined, I get a device busy error.
I'm running BBB 4.4.9-ti-r25.
In looking at the output of lsusb, I see a lot of modules that depend on libcomposite.  So, what I would like to do is to remove all of them, e.g., usb_f_acm, g_serial, u_serial, ...  Basically, I only want libcomposite to be loaded.
So how do I tell the boot process not to load these beasties?
-David Glaser

They get loaded by an early ran boot script under: /opt/scripts/boot/
Remove this block:
https://github.com/RobertCNelson/boot-scripts/blob/master/boot/generic-startup.sh#L72-L75
Regards,

The fixed worked like a charm.
I'm up and running with my hid function.
Thanks

Worked like a charm
Thanks
On 10/26/2016 08:03 PM, Robert Nelson wrote:

They get loaded by an early ran boot script under: /opt/scripts/boot/
Remove this block:
https://github.com/RobertCNelson/boot-scripts/blob/master/boot/generic-startup.sh#L72-L75
Regards,
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/RobertCNelson/omap-image-builder/issues/90#issuecomment-256511416,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABTURYKxbjlFiloknk625hDX7QCp_Bteks5q3-pBgaJpZM4KhvpC.


no tag/GitHub release for 0.2.3

I found a pypi release for version 0.2.3 but no corresponding git tag/GitHub release. Would you mind adding that?
(I'm about to submit a package for showcert to Fedora and at first I was confused which version is the latest.)

Thank you! Added tag for latest release 0.2.3.

Great, thank you :-)


Unable to Test otlp_grpc_exporter_cw_amp_xray_ecs in ECS Environment

I am trying to run the test otlp_grpc_exporter_cw_amp_xray_ecs, but have run into multiple errors. The first issue is an issue with setting up the s3 buckets.
Steps to reproduce this error:

run cd terraform/setup && terraform init && terraform apply -auto-approve from here. The error is

Error: Value for unconfigurable attribute

with aws_s3_bucket.example,
on main.tf line 1, in resource "aws_s3_bucket" "example":
1: resource "aws_s3_bucket" "example" {

Can't configure a value for "acl": its value will be decided automatically based on the result of applying this configuration.

This is due to the terraform version 4 upgrade and new resources need to be made for acl, server_side_encryption_configuration , and versioning.
After this first issue is resolved, setup and imagebuild work fine, though imagebuild gives an error but still builds and pushes all images to the ECR except the jaeger-zipkinsample-app. It also gives a local-exec provisioner error`, that seems to be the cause of the errors that follow as well.
Steps to reproduce this error:

run cd terraform/setup && terraform init && terraform apply -auto-approve
run cd terraform/imagebuild && terraform init && terraform apply -auto-approve

│ Error: local-exec provisioner error
│
│   with null_resource.build_and_push_sample_apps[0],
│   on main.tf line 58, in resource "null_resource" "build_and_push_sample_apps":
│   58:   provisioner "local-exec" {
│
│ Error running command 'docker build -t
│<PHONE_NUMBER>82.dkr.ecr.us-west-2.amazonaws.com/otel-test/sample-apps:jaeger-zipkin-sample-app-latest
│ ../../sample-apps/jaeger-zipkin-sample-app/': exit status 1. Output: #1 [internal] load build definition
│ from Dockerfile
...
...

Then, when running the test, the test setup eventually hangs on retrying to create a validator.

This will last until you Ctrl+C, at which point you get this error:
│ Error: local-exec provisioner error
│
│   with module.validator[0].null_resource.validator,
│   on ../validation/main.tf line 84, in resource "null_resource" "validator":
│   84:   provisioner "local-exec" {
│
│ Error running command 'docker-compose -f validator_docker_compose.yml down
│ docker-compose -f validator_docker_compose.yml build
│ docker-compose -f validator_docker_compose.yml up --abort-on-container-exit
│ ': signal: killed. Output: Container ecs_validator_1  Stopping

This error seems familiar, do you have docker running on the setup?. I'd try to run docker in background before running the test case. :)

This error seems familiar, do you have docker running on the setup?. I'd try to run docker in background before running the test case. :)

Unfortunately running the test case with docker running in the background still gives me the same error.

Closing this issue since we can run the tests in ECS environment.


Add trailing / to transformIgnorePatterns

Fix a small documentation issue with the transformIgnorePatterns suggested fix for performing proper transformations on the mocks for Jest compatibility. Need to add a trailing / to the regex for the transform ignore, otherwise it will match all packages in node-modules that start with the string react-native!

Hey @genuinezack , thanks for your contribution!
This looks legit to me, although, I'll ask for a second opinion :) cc @thymikee

@genuinezack this behavior is expected, as the vast majority of react-native-* packages are not compiled to JSC-compatible code, but only expose sources. It's a shame, but this is the reality we live in currently. I don't think we should merge this.

I disagree that this is expected/desirable, @thymikee, for three reasons:

The line before this snippet in the documentation says this line will transform "the whole scope of react-native-community and react-native". I understand (or misunderstand) that to mean those scopes specifically and not all scopes that happen to start with those strings.
Casting a wide net like this is likely to have false positives and negatives, meaning developers will have to edit this line anyways. They'll have an easier time reasoning about which packages are included if this line matches specific packages not string prefixes.
Finally, Jest's recommended pattern for this line does include a trailing slash for packages. Since it's their tool, I'd argue we should agree with them.

That being said, it is a single character in a legacy documentation so, as I noted, no concern to my life if you leave it as-is, I'll still use the package and appreciate you for making and maintaining it.

Casting a wide net like this is likely to have false positives and negatives, meaning developers will have to edit this line anyways.

Never happened to me in 2 years

They'll have an easier time reasoning about which packages are included if this line matches specific packages not string prefixes.

transformIgnorePatterns is one of the worst APIs Jest provides and believe me it causes constant confusion, especially for people starting with React Native.
Fun fact – Metro (the React Native bundler) transpiles all of your dependencies already with the same Babel preset.
I get your point though, it's completely correct. I'm just too tired answering the same questions (I'm a Jest maintainer btw). What would be a nice compromise I guess, is leaving this regex as you like, explain what it does, and how to make it more permissive. It's not quite the scope of this package, but maybe it helps someone :)

@thymikee - I'm only 9 months into the world of react-native, but we ran into this recently with cavy and @segment/analytics-react-native, both of which needed to be added to this and is why this particular regex is top of mind for me 😅 But, relatedly, 100% agree with your comment that it is a source of constant confusion!
I updated my change with your suggested clarification and will leave the compromise in your court to approve/deny.
(Oh also thanks for maintaining Jest too! I have no idea how y'all open source maintainers deal with hearing the same issues/suggestions/requests all day long, so sorry to add to the din.)

Right, this should resolve any doubts about this regexp :) I'm going to look into android failing test later today and merge it.

Can't figure out why Android e2e fails at deps installations, so, for now, I'll go ahead and just merge this.


Documentation for tools that are being handed over

The documentation has been written. Mark will review it with Jared and Berber and make any updates if necessary.

See https://github.com/cdisc-org/DDF-RA/issues/131#issuecomment-1688726902

@drewcdisc I have invited you to Admin the neo4j-field/usdm-dictionary-generator Git repo.  I tried to transfer ownership to cdisc-org but I got an error that I don't have permission to create a repo there.

I checked the documentation. It is not extensive but I believe enough to work with from a technical perspective.


feat(#291): Make file paths relative to root for easier opening inside vscode

Linking issue: https://github.com/pahen/madge/issues/291

Closing due to inactivity. I don't see the point of getting only first 4 characters.


Different fontSize in simulator than physical device

We're having some issues related to font sizing, because it's rendered differently in iOS simulator than in the device.
I created a small app to reproduce it (https://rnplay.org/apps/VP0enw), including only a Text component with fontSize = 40.
Left side is how  the app looks in iPhone 6 (iOS 9.1), and at the right is the same app running in  simulator (iPhone 6, iOS 9.1). (updated with the correct image. Thanks @ide )

I ran the same app on android simulator / android device, and apparently returns the same result without issues, but in our real-app it returns some different sizes too (I didn't found a pattern for that issue on android).
I was reading about Pixel Ratio, but I don't think that this is the issue, because simulator and physical device represent the same device (iPhone 6 in this case).
Any suggestion about what is happening?

You might not be running the same code -- your screenshot on the right is missing a colon after fontSize. I'm not sure what else could be the cause here.

I just updated image with two correct snapshops. I was doing several tests to reproduce the issue, and I choose bad when I compose image with comparation, thanks @ide for your advise.
Anyway, it's easy to reproduce the issue render a simple Text component as follows:
<View style={{ justifyContent: 'center', flex: 1}}>
   <Text style={{ alignSelf: 'center', fontSize: 40, backgroundColor: 'red' }}>fontSize: 40</Text>
 </View>

Finally I found the source of my problems: Dynamic Type.
It's a feature introduced on iOS7 which allows to user to change size of font "multiplier" depending his preferences. Here is a good article talking about how to implement it.
I usually designed UI with storyboard (before react-native), so I've never had this "problem", but some similar happens on Android, so I suppose that there are the same feature on android too.
Please, feel free to close this issue :)

Good to know that was the issue. Thanks for following up!

Hello,
I'm working for android UI. My UI showing correct in emulator but its not showing correctly in real device. I think this is pixel ratio issue, Currently I'm using 5.5" Android device.  Please help!

Hi,
Place the EditText of both fields in LInearLayouts in vertical orientation
that's it and place the button below the Linear layout. The UI will display
correct. Make the RelativeLayout your parent in xml.
Hope this helps you, as I have done this a long time ago.
Regards,
Siddharth Chugh.
On Sat, Sep 17, 2016 at 11:20 PM, Mohammed HUSAIN<EMAIL_ADDRESS>wrote:

Hello,
I'm working for android UI. My UI showing correct in emulator but its not
showing correctly in real device. I think this is pixel ratio issue,
Currently I'm using 5.5" Android device. Please help!
[image: image]
https://cloud.githubusercontent.com/assets/12196308/18609975/ba06fe7c-7d2c-11e6-9d48-70bd6d87fe89.png
[image: image]
https://cloud.githubusercontent.com/assets/12196308/18609979/f318dc62-7d2c-11e6-880d-f4e620f73ced.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/facebook/react-native/issues/3892#issuecomment-247793239,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKk1gA0oL23Kyv4-jKqqi0s245RYip31ks5qrChfgaJpZM4Gb7mB
.

Hi Siddharth,
In which file change linear layout to relative layout and  how. Plz provide steps

Hello,
Please reply if anyone have solution.

Did you try Pixel Ratio ?

Hi,
I have done by using following code
import React from "react-native";
import {
Dimensions,
Platform
} from 'react-native';
let { height, width }   = Dimensions.get('window');
const deviceType 			= Platform.OS === 'ios' ? true : false ;
if( width > height ) {
height = width;
width = height;
}
// Calculating ratio from iPhone breakpoints
const ratioX = deviceType ? ( width < 550 ? ( width < 430 ? ( width < 375 ? (width < 320 ? 0.75 : 0.875) : 1 ) : 1.125 ) : 1.25 ) : 1;
const ratioY = deviceType ? ( height < 800 ? ( height < 640 ? ( height < 568 ? (height < 480 ? 0.75 : 0.875) : 1 ) : 1.125 ) : 1.25 ) : 1;
// We set our base font size value
const base_unit = 16;
// We're simulating EM by changing font size according to Ratio
const unit = base_unit * ratioX;
// We add an em() shortcut function
function em(value) {
return unit * value;
}
// Then we set our styles with the help of the em() function
export default Style = {
// FONT
FONT_SIZE				: em(1),
FONT_SIZE_SMALLER		: em(0.75),
FONT_SIZE_SMALL		: em(0.875),
FONT_SIZE_TITLE		: em(1.25),
};

@uc-hus Hi , can you please show an example of how to use this ?


Fixes typo that caused exception when handling errors in electron-service

Not much to say, fixed a typo.

Ups. Thanks for this fix!


OwnedFeeCollector for Arbitrum perps IFeeCollector - audit modifications

…0) in constructor and slight gas improvement in claimFees removing modifier

Changes in this PR:

according to the audit notes using this error pattern instead of the modifiers is slightly more gas efficient and we already use the error pattern throughout so I changed it although its not that important imo

audit suggestion to add the ability to change the fee share. at first I thought it was better immutable but this is actually important since we are changing the fee share for the first month of arbitrum perps. Kaleb agreed we should be able to change it in the contract rather than deploy a new one.

ability to change the fee share controller address with this new functionality of changing the fee share. think the pDAO controls the fee share and the TC claims the fees

adds zero address check in the controller

further changes after review by Leo: pdao is the owner. tc is the fee share recipient and can claim fees. pdao handles all other admin and can also claim fees to the fee share recipient or change the recipient address

thanks fred, will take this over from here 🫡


selinux::module does not support subdirectories

As the title says, you cannot use puppet-managed subdirectories in the source parameter of selinux::module. I have a "site" module in which I keep all my local stuff, and I planned to keep my selinux modules like this: puppet:///modules/site/selinux/<mod_name>/…
However, this is not possible using this module, since the regex rejects that path.

We're having exactly the same issue, using a 'site' module for custom files and therefor sourcedir validation fails. e.g. 'source => 'puppet:///modules/siteA/selinux/hostA/'


fix CORS permissions and allow credentials

In https://github.com/cargo-lambda/cargo-lambda/pull/311/files a permissive CORS layer was introduced which causes fetches to fail in browsers:

Access to fetch at 'http://localhost:9000/lambda-url/api/graphql' from origin 'http://localhost:3000' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: The value of the 'Access-Control-Allow-Origin' header in the response must not be the wildcard '*' when the request's credentials mode is 'include'.

This PR injects all valid localhost TCP ports and also allows the usage of credentials for local development (note that allow_methods and allow_headers cannot use the wildcard '*' when allowing credentials)

I have just run into this issue too.

The solution is to configure CORS in the handler which requires that the framework does not implement it.
#[tokio::main]
async fn main() -> Result<(), Error> {
    env_logger::init();

    info!("Starting up...");

    tracing_subscriber::fmt()
        .with_max_level(tracing::Level::INFO)
        .with_target(false)
        .without_time()
        .init();

    // this could be coming from env or config

    let origins: [HeaderValue; 1] = [
        "http://localhost:3000".parse().unwrap()
    ];

    let cors_layer = CorsLayer::new()
        .allow_methods(vec![Method::GET, Method::POST])
        .allow_origin(origins)
        .allow_credentials(true)
        .allow_headers([ACCEPT, CONTENT_TYPE]);

    let handler = ServiceBuilder::new()
        .layer(cors_layer)
        //.layer(CompressionLayer::new())
        .service(service_fn(router::function_handler));

    run(handler).await
}

Would that work?

I think the changes here look fine. Do you mind running make fmt to format the changes?

Running that gives me no diff, it seems the workflow is having an issue?

Hope you are feeling better @calavera 🙏🏻


Function Routes Defined in file not loaded

In my working serverless project deployed already to AWS we have our functions defined in seperate yml files. This doesn't seem to work with serverless-offline
functions:
  devices: ${file(resources/functions/devices.yml)}
  load: ${file(resources/functions/load.yml)}
  registration: ${file(resources/functions/registration.yml)}
  getMessages: ${file(resources/functions/getMessages.yml)}

handler: functions/registration.handler
events:
  - http:
      path: /${self:custom.version}/registration
      method: get
      cors: true

I start serverless with sls offline --providedRuntime 11.10.1
I keep getting error

{"statusCode":404,"error":"Serverless-offline: route not found.","currentRoute":"get - /v1/registration","existingRoutes":[]}

Your Environment Information ---------------------------
Operating System:          darwin
Node Version:              11.10.1
Framework Version:         1.51.0
Plugin Version:            1.3.10
SDK Version:               2.1.0

Might be better to post it over there instead?
https://github.com/dherault/serverless-offline

Whoops totally missed I wasn't in the project's github, thanks!


Lodash tree-shaking

I’ve pin-pointed in a project that lodash is not tree-shaken within the tailwind code base. As such, I’m not able to shake it myself as tailwind pulls the _ object in, making my attempts redundant.
Given that tailwind doesn’t use much of lodash, I think it would be a good decision to ‘shake things up’ 😉.

Aside…
I’m new to all of this (world of webpack and modules), but a colleague has also told me about the modularised version of lodash. Can’t see how it’s better than shaking, but perhaps also worth consideration…

@adamwathan I think my issue here is related, so I'll comment here.
I noticed when I want to import some variables from tailwind into my app:
import {screens} from '../../../../tailwind.config'
I get lodash imported into my webpack bundle,  which probs means it's importing some other tailwind assets.
Is there a different way I should referencing these vars, or maybe a different webpack config?
I think this maybe be causing the issue here https://github.com/tailwindcss/discuss/issues/50#issuecomment-441997940 as well

@knynkwl Is it still an issue if you use resolveConfig? Released in v1.0.0-beta.6 #877
It uses a few lodash functions but they are directly imported.

@tlgreg I'm still getting lodash in my bundle.
I updated to ^1.0.0-beta.8
I removed all code in my app.js except:
import resolveConfig from 'tailwindcss/resolveConfig'
import tailwindConfig from '../../tailwind.config.js'

const config = resolveConfig(tailwindConfig)

This is what my bundle report shows:

Are we sure that's pulling in all of lodash? resolveConfig only pulls in a few lodash methods:
https://github.com/tailwindcss/tailwindcss/blob/next/src/util/resolveConfig.js

@adamwathan I from what I am seeing it's pulling in more than just those 5.
Here is my app and bundle: https://gist.github.com/knynkwl/c3100b64236e27290dd6b47327e2235e

Doing import tailwind from '../../tailwind.config.js' seems to not let lodash in, but I get:
WARNING in ./src/js/app.js 3:12-20
    "export 'default' (imported as 'tailwind') was not found in '../../tailwind.config.js'

But ideally we shouldn't have any lodash in the bundle, only the config file, right?

In a perfect world yes but lodash is required to resolve the contents of your config file — we use a bunch of its functions to do the actual merging logic.
You could generate a flattened/static version of your config as part of your build process and import that so you don't need the resolveConfig function, that way you wouldn't need lodash to create the final config on demand.

Checked your bundle, it's not pulling in all of lodash, only the parts we use. You can see it doesn't contain functions like partition, intersection, zipObject, etc. etc. I'll see about re-implementing the parts of lodash we use in a minimal way to avoid the dependency altogether but for now parts of lodash are necessary to actually resolve the final config object.

This actually seemed to solve the problem. No Lodash anymore ;)
import { theme } from '../../../tailwind.config';
const {screens } = theme;


[VOTE] Adopt the New RFC Process

Dear Community:
This is a formal vote to adopt the new RFC process after being discussed in the community for a while. Please see the proposed process here https://github.com/apache/tvm-rfcs
This is a formal voting thread for the community to adopt the new RFC process.
Please vote
Vote:
+1
0 - I don't feel strongly about it, but don't object
-1 - because..
The VOTE will open for a week.
This thread is mirrored to dev@, please vote by replying to this thread

+1

+1

+1
One small question: Under "Tracking Issue", the tracking issue is closed "when the RFC is either completed or abandoned."  Is "abandoned" the same as "postponed" in this context, or is there a distinction between them?

+1
Agree with @Lunderberg on the tracking issue

+1

@Lunderberg @junrushao1994 good catch, would changing "abandoned" to "postponed" resolve the ambiguity sufficiently?

Thanks @hogepodge! Yeah that sounds better to me

+1 Thanks @hogepodge for your hard work on this!

+1

@Lunderberg @junrushao1994 good catch, would changing "abandoned" to "postponed" resolve the ambiguity sufficiently?

Thank you!  That resolves my question.

+1

+1

+1

+1

+1

+1

+1

Ziheng

On Fri, May 14, 2021 at 01:50 Yuchen Jin @.***>
wrote:

+1
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/apache/tvm/issues/7991#issuecomment-840724342

Hi,
+1
Kind Regards,
Furkan KAMACI
On Thu, May 13, 2021 at 8:58 PM Ziheng Jiang @.***> wrote:

+1

Ziheng

On Fri, May 14, 2021 at 01:50 Yuchen Jin @.***
.invalid>
wrote:

+1
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/apache/tvm/issues/7991#issuecomment-840724342

+1

+1

Thanks everyone for participating so far, this vote is extended for another week in case there are more participations

+1

Thanks everyone for participating, the results are:
+1
Chris, Matt, Lunderberg, Tristan, Lily, Josh, haichen, Zach, Xiyou, Thierry, Andrew, leandro, Yuchen, ziheng, Furkan, liangfu, chenfan, altan
+0 None
-0 None
The vote has passed


No validation are implemented when entering data from the UI

The scenario that made me see the bug that should not be possible.

As we can see in the picture, there is no syrup farm. What happens is that the maple farm id is undefined, and the API returns a status 400 with the error

id
[ "The value 'undefined' is not valid." ]

To close this issue, this must be done:

When there is an error, display information at the appropriate place in the form
Don't display a page that cannot be used without a syrup farm selected.

Adding a note without specifying a date works. Still need a test on this use case too.

There is a failure in the automation:

The offset sign is now fixed

With the last commit, we can see how some validation messages are mapped to the HTML form. The solution is not elegant, but for now, it allows us to continue the development and display of errors in the forms.
This issue will be closed when validations are implemented in the two Alerte forms.

Gonna close this issue even if there is no validation in the alert form. The user experience is ok. Add, modify and delete accessible


OAuth proxy origin is not correct when run inside Home Assistant

Filing this after conversation today, for tracking

Fixed.


Suggested edits for Azure article automation-create-runbook-from-samples.md

This pull request was automatically generated by AzureDocs EasyEdit.

Hi @AzureDocsBot, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

Please ignore - testing for HackADoc.


Subcircuit layout upgrade #48

Fixes #36
Describe the changes you have made in this pr -
implements the the surface led (circular can be changed to rectangular..)
label is not visible yet and the layout size some time unstable ...
need more thorough work  and guidance from the core team ...
check before merging because all leds will be surface leds for now needed to work on that also
I request to be assigned on this issue for GSOC 2019
Screenshots of the changes (If any) -

This looks like a good start and you are right about more thorough work and guidance from the core team. This is quite a critical module and we would want it to be done only during the GSoC period. You should try and tackle easier issues like #155 for now.

As previously discussed, this is a big project, and we would love that you tackle this as a GSoC project. You are of course open to experiments and write your findings in the proposal. We can then discuss which is the best approach based on your thoughts.


Check Event Handling on Nomad Agent Simultaneous Restart

In #612 we noticed that on a simultaneous restart of all Nomad agents, some jobs are completely gone despite its restart and rescheduling policies.

Check if Poseidon handles such cases right

nomadEventDump-RestartTogether-withoutAlert.txt

It appears that Poseidon handles these cases correctly:
For runner jobs, Poseidon removes these jobs from the idle/used runners. After some time, the Prewarming Pool Alert might recreate the runner jobs.
For the environment jobs, Poseidon logs Environment stopped unexpectedly. If we have gained some more insights into this behavior in the future, we might introduce some recovery techniques once an unexpectedly stopped environment has been detected.
Of course, it would be the best that no runners become dead on restarts. However, in #673 we already identified a local and an upstream issue that should be fixed to improve this scenario.


chore: repository governance

Repository governance update
This PR was automatically created by the AVM Team hive-mind using the grept governance tool.
We have detected that some files need updating to meet the AVM governance standards.
Please review and merge with alacrity.
Grept config source: git::https://github.com/Azure/Azure-Verified-Modules-Grept.git//terraform
Thanks! The AVM team :heart:

@jensheerin have reviewed and approved for you to merge but would recommend looking into the failing tests before merging


Hexo setup

[x] Template repo
[ ] Installation
[ ] Configuration and themes
[ ] Plugins
[ ] GitHub actions
[ ] Gitlab pages

https://ntumitolab.github.io/site-hexo-next/2020/07/hexo-setup/


bevy_reflect: Ignored field order

Objective
Fixes #5101
Solution
Make it so that placing #[reflect(ignore)] or #[reflect(skip_serializing)] on a field that is not at the end of the type definition results in a compile error.
Rationale
Invalid ordering is a common and annoying footgun that can be very difficult to debug— or at least very frustrating. Ignoring a field in the middle of your struct can throw off the index you get back from Struct::index_of and friends. This is made even worse for things like tuple structs, where doing so breaks both FromReflect and serialization.
It should not be so easy to completely break reflection when using the derive macro. Thus, I think it warrants an error so we prevent users from entering this state in the first place.
Alternatives
One alternative, would be to output a compiler warning. Unfortunately, that API is still unstable.
We could also print out a warning at runtime whenever we rely on ordering. However, as discussed in this comment, we run the risk of some things not being caught (namely by third-party crates). Doing this is also just more cumbersome and prone to error.
Lastly, we could potentially store some sort of mapping that gives us the correct declaration index for a given index. This might work, but makes things like FromReflect a bit more difficult to implement. It also means we need to add a public API to access this information, which may be confusing/unintuitive. That being said, this is probably the most viable alternative out of the three.

Changelog

#[reflect(ignore)] and #[reflect(skip_serializing)] attributes now result in a compile error if placed in an invalid order

Migration Guide
Fields marked with #[reflect(ignore)] must now be placed at the end of their respective type definition. This applies to all structs, tuple structs, and enum variants.
// OLD:
#[derive(Reflect)]
struct MyStruct {
  #[reflect(ignore)] // <- ERROR!
  _ignore_me: i32,
  foo: i32,
  bar: i32,
}

// NEW:
#[derive(Reflect)]
struct MyStruct {
  foo: i32,
  bar: i32,
  #[reflect(ignore)] // <- OK!
  _ignore_me: i32,
}

Additionally, #[reflect(skip_serializing)] must also be placed at the end of the type definition. However, it must also come before #[reflect(ignore)].
 Moving to draft until that PR is merged.

This should be good to go now!

Closing in favor of #7575


Customizable keybindings

It would be really nice if it was possible to customize keybindings for spawning an editor, for scrolling, etc.
It could be implemented the way Alacritty does it:
  - { key: Insert, mods: Shift, action: PasteSelection   }

Alternatively, I'd love to have vim-like <S-Insert> descriptions of keys, which are pretty compact and readable (though I might be biased here).

It's not very vim-like, but I'd love to be able to press q to quit. It would match the behavior of other lightweight "document viewers" that I use (for images, PDFs, etc.)

Is this an actual bug, or was it removed but the readme wasn't updated?

I've missed it in the README, I'll give it a refresh once I'm done with the changes.

Could you try it out and let me know if it makes sense to you?

It works well!
Some quick notes/ideas:

It should be possible to omit modifiers entirely. Right now this causes a parsing error.

key_name values should probably be normalized, so that e.g. both "escape" and "Escape" work (right now only the second does).

In fact, it would be nicer to just have a single attribute named key like in Alacritty's config, instead of having to choose between key_name/key_char. Though I don't know how tricky it would be for the code to guess which one the user meant.

And, since you're imitating the format of Alacritty's config anyway, why not reuse it's nocemclature directly? So mods instead of modifiers, etc. Unless you feel it's not clear or explicit enough.

All of these are only minor quibbles or "would-be-nice-to-have" features, it still does the job well enough :+1:

It should be possible to omit modifiers entirely. Right now this causes a parsing error.

Yeah, that's true, fixed.

And, since you're imitating the format of Alacritty's config anyway, why not reuse it's nocemclature directly? So mods instead of modifiers, etc. Unless you feel it's not clear or explicit enough.

"Mods" sounds good enough to me :+1:

key_name values should probably be normalized, so that e.g. both "escape" and "Escape" work (right now only the second does).

The problem with this is that there doesn't seem to be a consistent normalization. If it was just about lowercasing, that'd be fine, but "Escape" works where "Minus" doesn't -- it's "minus". Really, the core problem both with this and the common "Key" prop is that I'm just delegating to GDK trying to do as little special work as possible 😅. Not just because of the extra work, but also because I don't know about subtleties that GDK handles, like = is actually shift + +, for example, but is that going to be the same on non-qwerty keyboards? I feel like I'd rather just give whatever comes in to GDK and tackle problems as they come in, which is why I opted to do it both ways.
Using name universally would work for the alphabetic keys, I think, but it wouldn't work for +, -, I don't know what else. For now, I'd rather not touch this part. Maybe I'll come back to it later.

I've merged the branch and cut a new release, I added your PR as well (sorry again for missing it). With this, I'll close this issue for the moment. Feel free to reopen if you find any bugs or want to discuss changes, or you could open a new one.


Deep import of pdfmake

Do NOT ignore this template or your issue will have a very high chance to be closed without comment.

Describe the bug
I am trying to use pdfmake with Vue 3 and vite. I get the following errors:

[vite] Avoid deep import "pdfmake/build/pdfmake" (imported by /src/App.vue)
because "pdfmake" has been pre-optimized by vite into a single file.
Prefer importing directly from the module entry:
import { ... } from "pdfmake"
If the dependency requires deep import to function properly,
add the deep path to optimizeDeps.include in vite.config.js.
[vite] Avoid deep import "pdfmake/build/vfs_fonts" (imported by /src/App.vue)
because "pdfmake" has been pre-optimized by vite into a single file.
Prefer importing directly from the module entry:
import { ... } from "pdfmake"
If the dependency requires deep import to function properly,
add the deep path to optimizeDeps.include in vite.config.js.

I am importing it like this
import * as pdfMake from "pdfmake/build/pdfmake"; import * as pdfFonts from 'pdfmake/build/vfs_fonts'; (<any>pdfMake).vfs = pdfFonts.pdfMake.vfs;
and my vite.config.ts file looks like this:
optimizeDeps: {   include: ['pdfmake/build/pdfmake', 'pdfmake/build/vfs_fonts'] }
Reproduction
Please provide a link to a repo that can reproduce the problem you ran into.
A reproduction is required unless you are absolutely sure that the the problem is obvious and the information you provided is enough for us to understand what the problem is. If a report has only vague description (e.g. just a generic error message) and has no reproduction, it will be closed immediately.
System Info

required vite version:"^1.0.0-rc.8"
required Operating System: Windows
required Node version: 10.16.1
Optional:

npm/yarn version
Installed vue version (from yarn.lock or package-lock.json) "^3.0.2",
Installed @vue/compiler-sfc version : "^3.0.2",

Logs (Optional if provided reproduction)

Run vite or vite build with the --debug flag.
Provide the error log here.

The pdfmake/build/vfs_fonts is not a part of pdfmake. I think we should provider an way to dismiss this.

As of latest release, this just works without need for config:
import pdfMake from 'pdfmake/build/pdfmake'
import pdfFonts from 'pdfmake/build/vfs_fonts'

pdfMake.vfs = pdfFonts.pdfMake.vfs

As of latest release, this just works without need for config:
import pdfMake from 'pdfmake/build/pdfmake'
import pdfFonts from 'pdfmake/build/vfs_fonts'

pdfMake.vfs = pdfFonts.pdfMake.vfs

Unfortunately is not working with

    "vite": "^2.3.8",
    "vite-plugin-vue2": "^1.7.2",

When building I get:
'pdfMake' is not exported by 'node_modules/pdfmake/build/vfs_fonts.js'
Illegal reassignment to import 'pdfMake'

Would you open a new issue with a reproduction against 2.4 @fessacchiotto ?


"model" property returning empty string

The "model" property was returning an empty string due to deviceFamily using platformVal (ex. "x86_64") rather than hardwareModelVal (ex. "MacBookPro10,1").
Small change allows model to return a usable string.

Great thanks, I'll merge it in.


error with http package to get data from api (flutter web)

Launching lib\main.dart on Chrome in debug mode...
Throwing away cached asset graph because the build phases have changed. This most commonly would happen as a result of adding a new dependency or updating your dependencies.
Error creating http|lib/http.ddc.dill
Error creating kernel summary for module:http|lib/http.ddc.dill

Response:--dart-sdk-summary=file:///C:/flutter/flutter/bin/cache/flutter_web_sdk/kernel/flutter_ddc_sdk.dill --output=C:\Users\Mahmoud salah eldin\AppData\Local\Temp\scratch_spacead5b8b75-d8d1-11e9-bbda-645106afd5e9\packages/http/http.ddc.dill --packages-file=file:///C:/Users/MAHMOU~1/AppData/Local/Temp/kernel_builder_bae193c0-d8d1-11e9-bbda-645106afd5e9/.packages --multi-root-scheme=org-dartlang-app --exclude-non-sources --summary-only --target=ddc --libraries-file=file:///C:/flutter/flutter/bin/cache/flutter_web_sdk/libraries.json --input-summary=org-dartlang-app:///packages/path/path.ddc.dill --input-summary=org-dartlang-app:///packages/http_parser/http_parser.ddc.dill --input-summary=org-dartlang-app:///packages/async/async.ddc.dill --input-summary=org-dartlang-app:///packages/pedantic/pedantic.ddc.dill --input-summary=org-dartlang-app:///packages/collection/collection.ddc.dill --input-summary=org-dartlang-app:///packages/string_scanner/src/eager_span_scanner.ddc.dill --input-summary=org-dartlang-app:///packages/typed_data/typed_data.ddc.dill --input-summary=org-dartlang-app:///packages/source_span/source_span.ddc.dill --input-summary=org-dartlang-app:///packages/charcode/ascii.ddc.dill --input-summary=org-dartlang-app:///packages/collection/src/algorithms.ddc.dill --input-summary=org-dartlang-app:///packages/collection/src/utils.ddc.dill --input-summary=org-dartlang-app:///packages/collection/src/iterable_zip.ddc.dill --input-summary=org-dartlang-app:///packages/collection/src/comparators.ddc.dill --input-summary=org-dartlang-app:///packages/collection/src/priority_queue.ddc.dill --input-summary=org-dartlang-app:///packages/collection/src/canonicalized_map.ddc.dill --input-summary=org-dartlang-app:///packages/meta/meta.ddc.dill --input-summary=org-dartlang-app:///packages/charcode/charcode.ddc.dill --input-summary=org-dartlang-app:///packages/typed_data/typed_buffers.ddc.dill --input-summary=org-dartlang-app:///packages/term_glyph/src/generated/ascii_glyph_set.ddc.dill --input-summary=org-dartlang-app:///packages/charcode/html_entity.ddc.dill --source=package:http/src/multipart_file_io.dart --source=package:http/src/multipart_file.dart --source=package:http/src/boundary_characters.dart --source=package:http/src/streamed_request.dart --source=package:http/src/multipart_request.dart --source=package:http/http.dart
org-dartlang-app:///packages/http/src/multipart_file.dart:14:8: Error: Error when reading 'org-dartlang-app:///packages/http/src/multipart_file_stub.dart': File not found
import 'multipart_file_stub.dart'
       ^

AssetNotFoundException: http|lib/http.ddc.dill

BuildStepCompletedException: Attempt to use a BuildStep after is has completed
BuildStepCompletedException: Attempt to use a BuildStep after is has completed
BuildStepCompletedException: Attempt to use a BuildStep after is has completed
BuildStepCompletedException: Attempt to use a BuildStep after is has completed

BuildStepCompletedException: Attempt to use a BuildStep after is has completed

BuildStepCompletedException: Attempt to use a BuildStep after is has completed

BuildStepCompletedException: Attempt to use a BuildStep after is has completed

BuildStepCompletedException: Attempt to use a BuildStep after is has completed

BuildStepCompletedException: Attempt to use a BuildStep after is has completed

BuildStepCompletedException: Attempt to use a BuildStep after is has completed

BuildStepCompletedException: Attempt to use a BuildStep after is has completed
BuildStepCompletedException: Attempt to use a BuildStep after is has completed

BuildStepCompletedException: Attempt to use a BuildStep after is has completed

BuildStepCompletedException: Attempt to use a BuildStep after is has completed

BuildStepCompletedException: Attempt to use a BuildStep after is has completed

BuildStepCompletedException: Attempt to use a BuildStep after is has completed

BuildStepCompletedException: Attempt to use a BuildStep after is has completed

BuildStepCompletedException: Attempt to use a BuildStep after is has completed

BuildStepCompletedException: Attempt to use a BuildStep after is has completed

BuildStepCompletedException: Attempt to use a BuildStep after is has completed

BuildStepCompletedException: Attempt to use a BuildStep after is has completed
BuildStepCompletedException: Attempt to use a BuildStep after is has completed

BuildStepCompletedException: Attempt to use a BuildStep after is has completed

BuildStepCompletedException: Attempt to use a BuildStep after is has completed
BuildStepCompletedException: Attempt to use a BuildStep after is has completed

BuildStepCompletedException: Attempt to use a BuildStep after is has completed

BuildStepCompletedException: Attempt to use a BuildStep after is has completed
BuildStepCompletedException: Attempt to use a BuildStep after is has completed
Exited (sigint)

@mahmoudsalah37 Please provide your flutter doctor -v output.
Please try this https://github.com/flutter/flutter/issues/34858#issuecomment-530595459.
Also, the fix is already merged to master channel. Report if you still encounter it on master.

[√] Flutter (Channel unknown, v1.10.2-pre.39, on Microsoft Windows [Version 10.0.17134.1006], locale en-US)
• Flutter version 1.10.2-pre.39 at C:\flutter\flutter
• Framework revision 7affc0f4c6 (2 days ago), 2019-09-14 17:51:12 -0700
• Engine revision 7ea9884ab0
• Dart version 2.5.0 (build 2.5.0-dev.4.0 be66176534)
[!] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
• Android SDK at C:\flutter\sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-28, build-tools 28.0.3
• ANDROID_HOME = C:\flutter\sdk
• Java binary at: C:\flutter\android-studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b01)
X Android license status unknown.
Try re-installing or updating your Android SDK Manager.
See https://developer.android.com/studio/#downloads or visit https://flutter.dev/setup/#android-setup for detailed instructions.
[√] Chrome - develop for the web
• Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe
[√] Android Studio (version 3.3)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin version 34.0.1
• Dart plugin version 182.5215
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
[√] Android Studio (version 3.5)
• Android Studio at C:\flutter\android-studio
• Flutter plugin version 34.0.4
• Dart plugin version 191.6183.88
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b01)
[√] VS Code (version 1.38.1)
• VS Code at C:\Users\Mahmoud salah eldin\AppData\Local\Programs\Microsoft VS Code
• Flutter extension version 3.4.1
[√] Connected device (2 available)
• Chrome • chrome • web-javascript • Google Chrome 76.0.3809.132
• Server • web    • web-javascript • Flutter Tools
! Doctor found issues in 1 category.

@mahmoudsalah37 Please provide your flutter doctor -v output.
Please try this #34858 (comment).
Also, the fix is already merged to master channel. Report if you still encounter it on master.

thanks


No version for 2.1 solidus?

Hi! I currently have only one client funding development of this gem, and we're currently migrating them from Solidus 1.3 to 2.0.
I'd be very pleased to have someone test solidus_reserved_stock against Solidus 2.1 and file issues. A cursory review of the changes in 2.1 suggests that there shouldn't be too many changes required to have the gem working, but you never know.
I've spun off a v2.0 branch so we can take pull requests for Solidus 2.1 on master, if any are needed.


Update upstream/FernFlower

This cannot be done until the next Minecraft version, as it will alter the decompiled code.

Ah Ok
I guess we can leave this open until that time period

You could, but the lack of altered patches would suggest to me that this isn't a complete merge - and it would likely be easier for lex to merge it himself.

https://github.com/MinecraftForge/ForgeFlower/commit/5f139bc1b8e5b91f10f8c166f53703081448d0ea