llvm 3.7

Package changes from 3.6:

CMake exports patch no longer necessary
Cosmetic purity patch fix
Build libc++ with private libc++abi headers visible from sources
Work around bugs in lldb's configure scripts

Please rename commit's name following our guide CONTRIBUTING.md
And I think this PR will cause lots of rebuilds so you need to make it into staging branch not master.

And I think this PR will cause lots of rebuilds so you need to make it into staging branch not master.

It will be better to leave switching the default llvmPackages to a little later (and staging branch).

Thanks. I believe LLVM 3.7 is needed to get Nix working on OS X 10.11 due to changes Apple made to nullable and generics in its system headers [1]. I was trying to get LLVM 3.7 working in my fork but didn't really know what I was going, so this will be very helpful.
[1] https://code.google.com/p/chromium/issues/detail?id=499448

It would be good to test this PR on darwin (I assume the author only tested Linux).

I'll test it on 10.11.

Tweaked commit name and set the default llvmPackages back to 3.6.
It's not obvious to me what the proper commit name format is for a change that adds a new version of an existing package without removing an old one or changing the default.

Does not build on OS X 10.11 GM and Xcode 7 GM.

Scanning dependencies of target cxxabi_shared
[ 94%] Linking CXX shared library ../lib/libc++abi.dylib
[ 94%] Built target cxxabi_shared
Scanning dependencies of target cxxabi_static
[100%] Linking CXX static library ../lib/libc++abi.a
/nix/store/9g7c4vsqfcmvvlng8vpfmh9myaygxgk6-CLTools_Executables/Library/Developer/CommandLineTools/usr/bin/ranlib: file: ../lib/libc++abi.a(cxa_unexpected.cpp.o) has no symbols
/nix/store/9g7c4vsqfcmvvlng8vpfmh9myaygxgk6-CLTools_Executables/Library/Developer/CommandLineTools/usr/bin/ranlib: file: ../lib/libc++abi.a(cxa_unexpected.cpp.o) has no symbols
[100%] Built target cxxabi_static
installing
error: install_name_tool: input file: lib/libc++abi.a is not a Mach-O file
builder for ‘/nix/store/3cp3qd5lf5n0qbqprim84nny2fzca9i1-libc++abi-3.7.0.drv’ failed with exit code 1

I had the same problem with my own fork. Anyway, I'm working on it.

OK, trivial fix. According to the man page, install_name_tool ignores the "-id" command when used on non-dylibs, but in OS X 10.11 this is no longer the case; it will give an error. Ralith, here is the patch to fix it:

diff --git a/pkgs/development/compilers/llvm/3.7/libc++abi.nix b/pkgs/development/compilers/llvm/3.7/libc++abi.nix
index cbccfd4..a1b300f 100644
--- a/pkgs/development/compilers/llvm/3.7/libc++abi.nix
+++ b/pkgs/development/compilers/llvm/3.7/libc++abi.nix
@@ -18,7 +18,7 @@ stdenv.mkDerivation {

   installPhase = if stdenv.isDarwin
     then ''
-      for file in lib/*; do
+      for file in lib/*.dylib; do
         # this should be done in CMake, but having trouble figuring out
         # the magic combination of necessary CMake variables
         # if you fancy a try, take a look at

LLVM 3.7 is now building on my OS X 10.11 system. It will take awhile but I'll let it run and report back.

With my small patch applied to efab778, LLVM 3.7 builds on OS X 10.11.
I still cannot get a working compiler in the Nix environment when bootstrapping with Xcode 7's command-line tools, but that is a separate issue, one related to the options passed to stdenv's ld and similar to what was seen previously in #2208, but that problem occurs whether you use LLVM 3.6 or LLVM 3.7. (I will file a separate issue once I've gotten to the bottom of it.) Therefore, I believe that it's unrelated to this pull request and I'm reasonably confident that, once this other problem is resolved, the LLVM 3.7 built by this pull request will work correctly.
(In fact, by opening a nix-shell and hacking the NIX_CFLAGS_COMPILE variable, I can produce a working "Hello, world" executable with clang 3.7.)

Your patch has been applied, thanks!

As for mesa, the latest stable release (10.6.7) can't yet deal with 3.7 renaming R600 to AMDGPU. It's likely 11.0.1 should be fine with that; I expect them releasing it in about two weeks.


ZCoin client shows the wrong transfer values

Zcoin client is not showing the correct transfer values. This was transfers to myself to fund a zcoin master node.

The ZCoin core client shows the correct values

Hi, I'm not quite sure I understand. The transaction amounts shown in QT look like there is one transaction of approximately 10 XZC, and three transactions of 479 satoshi. In order to fund a Znode, a single transaction of 1000 XZC (100,000,000,000 satoshi) is required. To confirm, the issue is that you made 3 transactions with a value of 479 satoshi, and these are not properly displayed in the Electron wallet?

Hi, I'm not quite sure I understand. The transaction amounts shown in QT look like there is one transaction of approximately 10 XZC, and three transactions of 479 satoshi. In order to fund a Znode, a single transaction of 1000 XZC (100,000,000,000 satoshi) is required. To confirm, the issue is that you made 3 transactions with a value of 479 satoshi, and these are not properly displayed in the Electron wallet?

Sorry, I was not clear. It is not a transaction to fund a ZCoin, it is a transaction to register a ZCoin masternode on the blockchain using https://github.com/zcoinofficial/znode-tool-evo
I followed this guide https://github.com/zcoinofficial/zcoin/wiki/Migrating-your-Legacy-Znodes-to-the-new-Deterministic-Znodes-system

@camakira This looks to be related to the deterministic znode stuff.


Fix/assert storage async

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

21 of 23 (91.3%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-2.5%) to 95.614%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/plugin.ts
21
23
91.3%

Totals

Change from base Build<PHONE_NUMBER>:
-2.5%

Covered Lines:
82

Relevant Lines:
86

💛  - Coveralls


icon not display well in macOS

os: mac
lsd --version: lsd 0.20.1
echo $TERM: xterm-256color
echo $LS_COLORS:

Expected behavior
when run lsd -alh should show the icons
Actual behavior
like that（something like a framed question mark):
 $ lsd -alh
drwxr-xr-x chao wheel 416 B  Wed Aug 18 16:50:11 2021  .
drwxr-xr-x chao wheel 576 B  Wed Aug 18 16:22:06 2021  ..
.rw-r--r-- chao wheel  11 KB Wed Aug 18 16:22:06 2021  app.rs
.rw-r--r-- chao wheel 7.4 KB Wed Aug 18 16:22:06 2021  color.rs
.rw-r--r-- chao wheel  10 KB Wed Aug 18 16:22:06 2021  config_file.rs
.rw-r--r-- chao wheel 4.7 KB Wed Aug 18 16:22:06 2021  core.rs

I also get this.

Just make sure you have proper fonts installed. Check to the linked issue.
Ref: https://github.com/Peltoche/lsd/issues/199#issuecomment-494218334


Final Review

In this project, I create a budget app that allows a user to keep track of his/her expenses.
Accomplishments:

Splash screen

A simple page with the name of your app (yes, you need to choose one), and links to the sign up and log in pages.

Sign up and log in pages

The user should be able to register in the app with full name, email and password (all mandatory).
The user can log into the app using email and password.
If the user is not logged in, they can't access pages that require the user to be logged in (all the pages described below).

Home page (categories page)

When the user logs in, they are presented with the categories page.
For each category, the user can see their name, icon and the total amount of all the transactions that belongs to that category.
When the user clicks (or taps) on a category item, the application navigates to the transactions page for that category.
There is a button "add a new category" at the bottom that brings the user to the page to create a new category.

Transactions page

For a given category, the list of transactions is presented, ordered by the most recent.
At the top of the page the user could see the total amount for the category (sum of all of the amounts of the transactions in that category).
There is a button "add a new transaction" at the bottom that brings the user to the page to create a new transaction.
When the user clicks on the "Back" button (<), the user navigates to the home page.

"Add a new category" page

The user fills out a form to create a new category, indicating their name and icon (both mandatory).
The user clicks (or taps) the "Save" button to create the new category, and is taken to the home page on success.
When the user clicks on the "Back" button (<), the user navigates to the home page.
"Add a new transaction" page

The user fills out a form to create a new transaction with:

name (mandatory)
amount (mandatory)
categories (mandatory at least one)
The user click (or taps) the "Save" button to create the new transaction, and is taken to the transactions page for that category.
When the user clicks on the "Back" button (<), the user navigates to the transactions page for that category.

Testing requirements

Create unit and integration tests for all the most important components of your RoR application.

Deploy app on render

Hi @Fene-87,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that is invalid because you did not follow the correct Git-flow workflow, thus creating the PR from a descriptive feature branch like budget-app into the dev branch. At the moment, what we have now is GitHub-workflow
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

Hey @addod19 what do you mean? I dont understand you. I have created pull request from dev into main

The PR is from the dev branch @addod19. budget_app is the name of my application not a feature branch.

Hey @addod19 what do you mean? I dont understand you. I have created pull request from dev into main

@Fene-87, yes I know and can see you have created the PR from dev into main, and that is called the GitHub-flow, what you are suppose to do is to follow the Git-flow which is creating the PR from a feature branch, you can name it whatever you want but it should be descriptive into the dev branch. I hope this is clearer

The PR is from the dev branch @addod19. budget_app is the name of my application not a feature branch.

@Fene-87 , it is just an example I gave. You can ignore that name and use whatever name you please

Thank you so much for your time and feedback @Kamba56 I appreciate.


CLI add SSL binding command does not pass right payload

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Describe the bug
command: **

az webapp config ssl bind --certificate-thumbprint thumbprint --ssl-type SNI --name sitename --resource-group resourcegroupname --slot slotname --debug --verbose

**
ICM: 181031414
From kusto logs in the icm, the cli command does not include properties necessary to create SSL binding.
$PropertiesObject = @{     SslState   = "SniEnabled"     thumbprint = $thumbprint }
To Reproduce
Create a webapp, SSL binding, add custom hostname, create SSL binding using the command.
Expected behavior
Running the commands returns cert not found
ERROR: cli.azure.cli.core.util : Certificate for thumbprint 'thumbprint' not found.

webapp

ping @ElleTojaroon


Dns benchmark - Adding numpy installation to run script

Adding numpy installation to run script, due to numpy being required by py/run_perf.py. This command installs numpy or does nothing is numpy s already available.

/assign @wojtek-t

/lgtm
/approve

@wojtek-t For some reason lgmt and approve were not triggered.

/lgtm
/approve


Sharding DDL statement for multiple table's connection leak problem

As cached connection map is stored as multiple connection, but single table's DDL statements doesn't reuse the connection and caused the leak connection problem.

fixed at 2.0.0.M3


App logout

As a DAO  user,  I  want  to disconnect from my wallet so that I leave in a secure state.

I think this is the easier part... An app-interface part... One interaction with Clarity smartcontract and one with app display... After logout the member can also sees the general info of the defined DAO or its proposals

this will use stacks connect


[BEAM-11643] Add BigDecimal support for SpannerIO

This cherry-picks https://github.com/apache/beam/pull/13762 to the 2.28.0 release branch.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Java PreCommit

Run Java PreCommit


Have top right button click be recognized in presentational component using react native router flux

My app scenes use a container component which calls a presentational component.
I can create the top right button within the container component like this:
static right = () => {
  return (
    <TouchableOpacity onPress={() => buttonPressed()}>
      <Text style={styles.rightButtonText}>Done</Text>
    </TouchableOpacity>
  );
};

Now I am trying to have the top right button click be recognized by the presentational component (so it can save data).
I can communicate between the two by sending a function as a prop from the container component to the presentational component. And when that is called from the presentational component, it is called in the container component.
But I have found no way to make the static right function in the container component hear the click in the presentational component (to then take the appropriate action).
I tried these two solutions along with several others (including Action.refresh) but none work. I thought of using state, but I cannot access state or this. if I am using a static function. But removing static makes the button disappear altogether.
Thanks in advance for any help. I am not sure if you prefer handling such discussions here or stackoverflow.

Someone provided a helpful solution on StackOverflow. Here is is in case it can help someone:
https://stackoverflow.com/a/53455766/4407512


Head Ache Killer for spark serialization of closures

Added object capable of logging closure cleaning stuff and to ensure a function or an object is serializable

cc @vidma @maasg

can you add more details when is this useful?
see my comment.

This is helpful to check if a function is serializable, if not, the errors shows what is tried to be serialized and very helpful to debug not serializable exceptions.
But also, since we are updating the logger we can have insight on how the cleaner is behaving and thus we can know if the closure isn't taking much more than required (and thus increasing the size of the serialization of the closure)

@vidma I refactored to avoid the runtime to access the object. The reflection to access the private logger is kept... no other ways to access them :-)


Merge 6 -> main

➡ Forward port
Port ign-sensors6 to main
Branch comparison: https://github.com/ignitionrobotics/ign-sensors/compare/main...ign-sensors6
Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)

The Focal test failures seem to be new

I think we need this fix: https://github.com/ignitionrobotics/ign-common/pull/338

I think we need this fix: ignitionrobotics/ign-common#338

I'll trigger new focal nightlies

I think we need this fix: ignitionrobotics/ign-common#338

I'll trigger new focal nightlies

queued:  https://build.osrfoundation.org/view/ign-garden/job/ign-common5-debbuilder/305

I think we need this fix: ignitionrobotics/ign-common#338

I'll trigger new focal nightlies

queued:  https://build.osrfoundation.org/view/ign-garden/job/ign-common5-debbuilder/305

nightly rebuilt, retrying in jenkins:  https://build.osrfoundation.org/job/ignition_sensors-ci-pr_any-ubuntu_auto-amd64/954/

nightly rebuilt, retrying in jenkins:  https://build.osrfoundation.org/job/ignition_sensors-ci-pr_any-ubuntu_auto-amd64/954/

fix confirmed; I'm rerunning the other Focal CI jobs


MRG: Minor fixes to testing framework

This PR:

Gets rid of some nose cruft
Moves to standard pytest slow-test markers
Fixes our decorators for sklearn that had incorrect versions
Makes AppVeyor a little bit nicer

Closes #4518.

@agramfort CIs look like they're headed toward happiness

Codecov Report

Merging #4541 into master will increase coverage by 3.67%.
The diff coverage is 95.95%.

@@            Coverage Diff             @@
##           master    #4541      +/-   ##
==========================================
+ Coverage   83.33%   87.01%   +3.67%
==========================================
  Files         349      349
  Lines       65438    65467      +29
  Branches    10160    10160
==========================================
+ Hits        54530    56963    +2433
+ Misses       8029     5623    -2406
- Partials     2879     2881       +2

besides my concern of keeping some nose in travis LGTM
thx heaps @larsoner
@massich wanna have a look?

No we still do a lot of nose assert imports. It will take a separate PR
with thousands of changed lines to fix that

how about having our assert_* functions in fixes.py so we don't need to change every single line for now?

Only hundreds of lines then :)
Probably better for a follow up PR though

yes ok.

LGTM

how about having our assert_* functions in fixes.py so we don't need to change every single line for now?

I would do that as intermediate step if we really need to remove the nose dependency. But as end goal I would change all assert_* to pytest like asserts.

@larsoner you need to rebase

@agramfort done

is the Makefile and the make test target already switched to pytest ?
if so +1 for MRG

Yep, @massich did that in a previous PR and I tweaked the values here based on the attribute/mark switch
Just pushed a commit to fix a rebase error, will merge once the CIs come back happy
Then we can figure out #4523 :) Maybe the first thing to do is just put the existing Travis 2.7 config in environment.yml. That way we'll have something that works on CIs, even if it doesn't work well for users. Then we can iterate on just modifying that file (and/or adding a separate 3.5/6 one) without needing to mess very much with Travis or AppVeyor.

Going to have to rebase again. @agramfort okay to merge once CIs are happy?

yes ok to merge when Cis are green


Is there a separate reason to set max_chars to 20?

First of all, thank you for releasing this model as an open source.
I have a question while using it, so I'd appreciate it if you could answer.
Is there a separate reason to set max_chars to 20?
I mean I was wondering if it's because the model is only trained under 20 chars or if there's any other reason.

Well there is no specific reason, only because based on the statistic of our training dataset AnyWord-3M, 99.6% of lines are less than 20 chars(refer to Table 6 in the paper). We set max_chars=20 to cut off some long sentences.


Add property injection example to DI application

DependencyInjectionConsoleApplication is not using the PropertyrInjection.

I hope you will go ahead with this.


Setup: Stop bitcoind after test

Stop bictoind either directly after testrun or at least before provision.
Goal is to safe traffic and speed on cheap routers for the rest of setups.
Should start then normal after final setup reboot.
Also if user chooses "COPY LAN" Blockchain - dont ask and just delete if its just some megabytes of sync data from testrun.

Added code changes. Testing needed with new sd card build.
Also recheck if bitcoind is running during provision of recover/update - just in case.


Update readme to provide a simple example of how to dispatch multiple actions without notifying subscribers until all complete (without react or or other dependencies)

It seems like this might be what I'm looking for, but can't fully understand it with the advanced/indirect examples.

We for example batch using request animation frame and React's batchedUpdates. Below is a simple example using just request animation frame. This means that when mouse / touch events trigger actions (which are triggered more often than animation frames), you can make sure the subscribed (connected / smart) components only get notified every animation frame (and only when there where actions called of course).
import raf from 'raf';
import { createStore } from 'redux';
import { batchedSubscribe } from 'redux-batched-subscribe';

let rafID;
function rafUpdateBatcher(notify) {
  if (rafID) return; // prevent multiple request animation frame callbacks
  rafID = raf(() => {
    rafID = null;
    notify();
  });
}

// Note: passing batchedSubscribe as the last argument to createStore requires<EMAIL_ADDRESS>const store = createStore(reducer, intialState, batchedSubscribe(rafUpdateBatcher));

@peteruithoven can you show the Complete example，i still cannot understand what the notify mean?  thank you very much. :)

oh~ it seem like control the subscription asynchronously

Testing my PR (https://github.com/tappleby/redux-batched-subscribe/pull/17) I created the following simple example, would that clarify things?
import { createStore, compose, applyMiddleware } from 'redux';
import raf from 'raf';
import { batchedSubscribe } from 'redux-batched-subscribe';

// Actions
export const ADD = 'ADD';
export function add(text) {
  return { type: ADD, text };
}

// Reducers
const initialState = [];
function reducer(state = initialState, action) {
  console.log('reduce: ', action.type);
  switch(action.type) {
    case ADD:
      return [...state, action.text];
    default:
      return state;
  }
}

// Batcher
let rafID;
export default function rafUpdateBatcher(notify, action, getState) {
  if (rafID) return;
  rafID = raf(() => {
    rafID = null;
    notify();
  });
}

// Store
const store = createStore(
  reducer,
  initialState,
  // disable this line to see difference
  batchedSubscribe(rafUpdateBatcher)
);

// Experiment
store.subscribe(function() {
  const state = store.getState();
  console.log('notified: ', state);
});

store.dispatch(add('a'));
store.dispatch(add('b'));
store.dispatch(add('c'));

This RAF example looks handy but feels like more of an advanced implementation. I think it would be hard to follow in a readme.
debouncing is a fairly common pattern for throttling how often a function gets called. Perhaps more of a description here would help out?


Issue with connecting to server

I'm having an issue with my android device trying to connect to a server. It immediately get's disconnected with the following error:
Client Recv: failed to connect to ip=XX.XX.XXX.XXX port=30000 reason=System.Net.Sockets.SocketException (0x80004005): Connection refused System.Net.Sockets.Socket.Connect (System.Net.EndPoint remoteEP) [0x00000] in <00000000000000000000000000000000>:0  System.Net.Sockets.TcpClient.Connect (System.String hostname, System.Int32 port) [0x00000] in <00000000000000000000000000000000>:0  Telepathy.Client.ReceiveThreadFunction (System.String ip, System.Int32 port) [0x00000] in <00000000000000000000000000000000>:0  System.Threading.ThreadStart.Invoke () [0x00000] in <00000000000000000000000000000000>:0  System.Threading.ContextCallback.Invoke (System.Object state) [0x00000] in <00000000000000000000000000000000>:0  System.Threading.ExecutionContext.RunInternal (System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, System.Object state, System.Boolean preserveSyncCtx) [0x00000] in <00000000000000000000000000000000>:0  System.Threading.ThreadSta
The odd thing is that other clients are able to connect (The Unity Editor Client in particular). The server runs in a docker container and has remapped ports. I'm not sure if that makes a difference though.

I believe the issue was I wasn't setting the server to listen in time before the connections arrived.


Enterprise policy line updated

Enterprise policy line - "MV2 extensions will continue to be supported through enterprise policy at least until Chromium MV2 support timeline (which is currently January 2024)." updated.

Learn Build status updates of commit 8af2685:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: CannotMergeCommit] Cannot merge commit 8af268572bba860cd0cba51ecc59b955be755509 in branch v-bhuvana/msedgeaddons of repository https://github.com/MicrosoftDocs/edge-developer into branch main (commit 4171871a16ec0d286dbcdcb58312c5fe229cae7b). Please follow this documentation: https://help.github.com/articles/resolving-a-merge-conflict-using-the-command-line/ to use git.exe to resolve you content conflicts locally and then push to remote.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit ce9662f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/extensions-chromium/developer-guide/manifest-v3.md
:white_check_mark:Succeeded
View

microsoft-edge/extensions-chromium/publish/contact-extensions-team.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


refactor(intercptor): use rxjs instead of promises

Super quick update to the code to change the interceptor from running toPromise and the relative code afterwards to using the .pipe method and tap operator. Everything runs as expected.
Fix: #3

Merged into 1.0.4 release.  Thanks @jmcdo29 !


"LOGIN TO OUR PLATFORM" Text Improvement

The current design of the this text is nice but this section can be improved by reducing the text from "LOGIN TO OUR PLATFORM" to "LOGIN". By reducing the text in this way, we can make this section look even better.
Current Text

Expected Updated Text

Hello iamRabia-N, thanks for opening a issue, your contribution is valuable to us. The maintainers will review this issue and provide feedback as soon as possible.

Not required


⚠ Fortnite has degraded performance

In f8d6735, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1639 ms

Resolved: Fortnite performance has improved in f2c8664.


Incorrect type inference when using spread operator with index signature and adding another prop

Bug Report

🔎 Search Terms

typescript spread wrong type inference
🕗 Version & Regression Information

This is the behavior in every version I tried (ex: 3.3.3-4.7.4)

⏯ Playground Link

Playground link with relevant code
💻 Code

interface A {
    title: string
    [year: number]: number
}
let a: A = {title: 'A', 2020: 123};

let b = { ...a}
let c = { ...a, label: 'revenue'}

🙁 Actual behavior
// d.ts inference
declare let b: {
    [x: number]: number;
    title: string;
};
declare let c: {
    label: string;
    title: string;
};

🙂 Expected behavior
// d.ts inference
declare let b: {
    [x: number]: number;
    title: string;
};
declare let c: {
    [x: number]: number; // <--- this is missing
    label: string;
    title: string;
};

It seems that when I add an extra property to the spread object, it loses the index signature properties from the spread source.
But, if I do not add any extra property it does infer correctly the type...

Duplicate of #27273. Used search terms: spread index in:title


hash_condition_match returns nil when subject is ActiveRecord Collection Proxy

When the subject is an ActiveRecord_Associations_CollectionProxy(ActiveRecord::Relation), the inner_foo value is true, but the outer_foo value is nil as seen below;
This is because it calls this method instead of array's any?:
https://api.rubyonrails.org/classes/ActiveRecord/Associations/CollectionProxy.html#method-i-any-3F

System configuration
Rails version:
5.2.0
Ruby version:
2.5.1
CanCanCan version
2.3.0

I'm still struggling to isolate exactly why any? is returning nil, but while debugging it, got a casual segfault from the gem which makes no sense to me so far. Putting here incase I manage to debug it in the next couple days.
/Users/ubuntu/source/cancancan/lib/cancan/conditions_matcher.rb:75: [BUG] Segmentation fault at 0x0000000000000004
ruby 2.5.1p57 (2018-03-29 revision 63029) [x86_64-darwin16]

-- Crash Report log information --------------------------------------------
   See Crash Report log file under the one of following:
     * ~/Library/Logs/DiagnosticReports
     * /Library/Logs/DiagnosticReports
   for more details.
Don't forget to include the above Crash Report log file in bug reports.

-- Control frame information -----------------------------------------------
c:0133 p:0014 s:0920 E:0009f8 BLOCK  /Users/ubuntu/source/cancancan/lib/cancan/conditions_matcher.rb:75 [FINISH]
c:0132 p:---- s:0916 e:000915 IFUNC
c:0131 p:---- s:0913 e:000912 CFUNC  :each
c:0130 p:0018 s:0909 E:0018e0 METHOD /Users/ubuntu/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/activerecord-5.2.0/lib/active_record/relation/delegation.rb:41 [FINISH]
c:0129 p:---- s:0901 E:000070 CFUNC  :any?
c:0128 p:0012 s:0897 E:0016a0 METHOD /Users/ubuntu/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/activerecord-5.2.0/lib/active_record/relation.rb:228
... etc

can you provide a gist to reproduce your issue? Is not clear to me, sorry.

Here is a reproduction of the issue: https://github.com/laurawatson/can_can_check.
In this reproduction, the issue occurs when you first login as an adviser user and try to access the foo show page. It will redirect you to the home page despite the adviser user having access to foo. If you then head back to the foo show page you will be allowed in (more info in the readme).

@coorasse We're currently effectively having to inline our ActiveRecord conditions to avoid this bug. We suspect it's some unusual interaction between ActiveRecord lazy loading and this gem but I have been unable to diagnose it further than that any? method mentioned previously.
Would you be so kind as to confirm if you see the same issue as us in the example repo linked by @laurawatson above? If it is at least a confirmed bug then we may be able to put some time aside to help fix it, until then all we can do is assume it's our mistake and not use this gem.

I confirm that I see the same behaviour.

changing the rule
can :read, Foo, user_roles: { user: user } (wrong)
into
can :read, Foo, user_roles: { user_id: user.id } (correct)
fixes the issue.
But I still don't understand why.

And the user can also see http://cancan_error.localhost:3000/bars/1 which should be protected. By using the id instead of the association name user (which is not supported) fixes it

It seems like I'm not able to reproduce the issues outlined by @laurawatson.
Manually checking the Foos that the user has access to I was able to record the same behaviour for
can :read, Foo, user_roles: { user: user }
and
can :read, Foo, user_roles: { user_id: user.id }
In addition to that, by using either authorize! or load_and_authorize_resource in controllers/bars_controller.rb
I'm not able to access bars/1 with neither {user : user} nor {user_id : user.id }
I guess I must have missed something ^^
Would anyone be so kind as to point me into the correct direction ?

@BookOfGreg Interesting thing about the line of code you referenced ^^
By changing:
    attribute.any? { |element| matches_conditions_hash?(element, value) }

to:
    match_results = attribute.map { |element| matches_conditions_hash?(element, value) }
    match_results.any?

It works :D


V2 ETA?

Great project. When will you approximately release the v2 dataset?

Hi @xuebinqin! The v2 demo results seems incredibly awesome!! Could you please tell us when will it be released??
Thanks.

Thanks for your interest. Due to the heavy workload of the annotation, we
can not give an exact ETA. It may take another few months.
On Mon, Oct 10, 2022 at 2:08 AM ਰਾਹੁਲ ਭੇਲੇ @.***> wrote:

Hi @xuebinqin https://github.com/xuebinqin! The v2 demo results seems
incredibly awesome!! Could you please tell us when will it be released??
Thanks.
—
Reply to this email directly, view it on GitHub
https://github.com/xuebinqin/DIS/issues/44#issuecomment-1273009300, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADSGORI2IKWGIGEVQXKAM3LWCPMI3ANCNFSM6AAAAAAQMIOG5U
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Xuebin Qin
PhD
Department of Computing Science
University of Alberta, Edmonton, AB, Canada
Homepage: https://xuebinqin.github.io/

Thanks for a quick reply!
Maybe you can release the model trained on a subset of whole V2 dataset for now. I understand that's not a correct way of presenting research results but for production that result you demoed seems already a lot better than U2Net.
Again thanks for every other model you've released so far!

I'm also looking forward to the updated dataset - and as a side note, I've implemented DIS  into a game test (AI Paintball) that creates all the artwork "live" and it worked great.  ( Video here )

Thanks for your patience and interest. Our DIS V2.0 Dataset is almost
there. But we need to finish the new paper for releasing that. It may take
a while.
On Fri, Nov 11, 2022 at 1:10 PM Seth Robinson @.***>
wrote:

I'm also looking forward to the updated dataset - and as a side note, I've
implemented DIS into a game test (AI Paintball) that creates all the
artwork "live" and it worked great. ( Video here
https://www.youtube.com/watch?v=FoYY_90KlyE )
—
Reply to this email directly, view it on GitHub
https://github.com/xuebinqin/DIS/issues/44#issuecomment-1312194132, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADSGORKBLZENKOXVIFPTZHTWH2Y3ZANCNFSM6AAAAAAQMIOG5U
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Xuebin Qin
PhD
Department of Computing Science
University of Alberta, Edmonton, AB, Canada
Homepage: https://xuebinqin.github.io/

Interesting video @SethRobinson. So you basically segment an object and draw paint splash on it, right?

Interesting video @SethRobinson. So you basically segment an object and draw paint splash on it, right?

Yes  - I'd say the key thing is that I modified the text to image stable diffusion server its using while the game is playing to have a "remove background" feature using DIS, so it returns the created image with an alpha mask cutout so the images are ready to be used as game sprites on the fly.

Wow! That's really an interesting use of Stable Diffusion but doesn't Stable Diffusion takes like a lot of seconds to generate just one paint splash image? I wonder how did you make it real time?

Wow! That's really an interesting use of Stable Diffusion but doesn't Stable Diffusion takes like a lot of seconds to generate just one paint splash image? I wonder how did you make it real time?

It takes around 1.5 seconds to generate a single text to image on one nvidia 3090 graphics card - in my video, I'm using three Nvidia cards at once to run three servers (all on the same machine) so I can generate nearly two images a second.
Obviously this isn't something most people would have so this is all in the "just playing around with concepts" phase.  :)

I have only 1 P100 in cloud with free time limits. Lol.

Thanks for your patience and interest. Our DIS V2.0 Dataset is almost there. But we need to finish the new paper for releasing that. It may take a while.
…
It's been six months, maybe it's time?

Thanks for your patience and interest. Our DIS V2.0 Dataset is almost there. But we need to finish the new paper for releasing that. It may take a while.
…
On Fri, Nov 11, 2022 at 1:10 PM Seth Robinson @.> wrote: I'm also looking forward to the updated dataset - and as a side note, I've implemented DIS into a game test (AI Paintball) that creates all the artwork "live" and it worked great. ( Video here https://www.youtube.com/watch?v=FoYY_90KlyE ) — Reply to this email directly, view it on GitHub <#44 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADSGORKBLZENKOXVIFPTZHTWH2Y3ZANCNFSM6AAAAAAQMIOG5U . You are receiving this because you were mentioned.Message ID: @.>
-- Xuebin Qin PhD Department of Computing Science University of Alberta, Edmonton, AB, Canada Homepage: https://xuebinqin.github.io/

Any update?

I think you should create a patreon so we can support your efforts; I hope that might speed things up :)

Yeah. I hope you didn't abandon the project. @xuebinqin (When any project delays, TimberTron comes to my mind.)

Thank you for your interest and support. We have had the v2 dataset ready
and started the v2 model as well. The delay are mainly caused by the
following three factors: (1) I changed my job and I can only work on this
project in my spare time; (2) So that I need to re-configure all my
hardwares and softwares from scratch; (3) More importantly, some of the V2
data samples are extremely complicated and these samples took a huge amount
of time for labeling and polishing, hope you can understand the
difficulties and challenges. Sorry for the delay, we are trying our best to
release the new version as soon as possible.
On Mon, Mar 20, 2023 at 8:57 PM RahulBhalley @.***>
wrote:

Yeah. I hope you didn't abandon the project. @xuebinqin
https://github.com/xuebinqin (When any project delays, TimberTron comes
to my mind.)
—
Reply to this email directly, view it on GitHub
https://github.com/xuebinqin/DIS/issues/44#issuecomment-1477246748, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADSGORLOCFKC3Z2452NNXF3W5ERLNANCNFSM6AAAAAAQMIOG5U
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Xuebin Qin
PhD
Department of Computing Science
University of Alberta, Edmonton, AB, Canada
Homepage: https://xuebinqin.github.io/

any news?

@xuebinqin, Is there a  tool you could recommend to help us segment our data?  Thanks!

Hi @xuebinqin, could you please open-source your code and dataset (also the incomplete segmentations) so that we can help you do it??? We can also help train the model if you don't have time to re-configure your hardware. 🙏

@xuebinqin will you release V2 dataset?


Make SurfingContext class public

Access to the class is needed in order to write custom/experimental JsonParserAdapter implementations, for example when implementing JsonParserAdapter.parse() and JsonParserAdapter.create*Parser(), which accept the constructed SurfingContext.
The SurfingContext constructor can meanwhile remain package-private and be constructed only via JsonSurfer

Thanks! I'm a little bit busy recently. I will check later.


Extends - parameters no returned from setup

Version
3.0.0
Reproduction link
https://codesandbox.io/s/vue-3-start-forked-rrcp8?file=/src/Bar.js
Steps to reproduce
The Setup function is not inherited
What is expected?
Parameters must be passed
What is actually happening?
Returns undefined

See https://github.com/vuejs/vue-next/issues/1866#issuecomment-674881439

I see, thank you


Working replays marked as corrupt

Every replay from the 14th October 2015 until 6th October this year has been put into a folder marked corrupt in my replay folder. The games I played after installing this client are not marked as corrupt. I didn't actually use this client to play those games however.

Please attach one

5110133-JaggedAppliance.zip
Here's 3 to be sure ;)

Have you tried the latest client version? It doesn't happen for me

I can't reproduce it with the latest version, closing.

yay


6.0.2 causes import failure from #minurl

See https://github.com/QwikDev/qwik/actions/runs/10038434388/job/27740676939#step:10:478 (should be available for 30 days I think)
looks like something doesn't understand the import map?

Sorry wrong repo :facepalm:


Generate deployment scripts for Hardhat

Description
As a developer, I would like to have deploy scripts generated for my project.
The deployment is an important part of the development lifecycle, hence it would improve the dev UX if the generator could prepare deployment scripts.
Acceptance criteria

A new confirm choice is proposed to the user to choose whether or not to generate the deployment scripts
Deployment scripts are generated and working with Hardhat

Closed by #37


upgrading dependencies to jakarta and removing thorntail

Still working to pass tck that are failing after upgrading jakarta dependencies and removing thorntail


Update Storage-file Readme

Purpose: update readme based on Preview3.
Total changes: 1

Wrong class imported, should be "SharedKeyCredential", "FileServiceClient" in order to use them in the example

Origin

Proposal

close for not valid anymore.


Share Button Social Functionality Issues

If I don't close the social share popup, it persists on other pages. It should close if you leave the page. Example of current issue:

https://github.com/ucd-library/dams/assets/17831969/3e7f1d9a-7b40-43dd-9b23-b8d1ee3020ca

When on the page, it isn't super intuitive to close the button since you have to click the button itself to do it. A click anywhere outside of the overlay would be better.

The Pinterest button doesn't work. Nothing happens when it is clicked.

When you share an item to Twitter it says "undefined" as the title of the link. Can we set it to pull the item title for this any other social that allows us to define it?

Twitter rebranded, so we need to change the Twitter icon out for the new X Logo. icon: https://fontawesome.com/icons/x-twitter?f=brands&s=solid

As long as they are still using "twitter.com" as primary URL I feel we should maintain Twitter in the name. When it fully shifts over, we can drop it from the label.

@wrenaria
I'll create a PR into sandbox for this. I can't test Pinterest completely because when trying to add it to the board it seems to fail, I think because the url is pointing at my local machine. We'll have to finish testing once it's deployed to sandbox. But at least the errors it was throwing previously are addressed, as well as the other items.


Proposal: change custom dispatcher signature

As rightly noted by @vslinko, it's better to provide getState instead of initialState.
I think I was wrong now when I said this. In fact, I find it now easier to implement composable dispatchers only if each “parent” has the ability to “intercept” both getState and setState, and if the child can “query” the latest state.
I'm going to keep this open for a while, as I'm still not 100% sure which approach is more correct, but I'm leaning towards (getState, setState) => action => () now as the signature for the dispatcher.

We also need to rethink where the state "lives." In the current version, there's no way for the Redux instance to push a new state atom to the dispatcher — for instance, a call to Redux::setState() does not result in the dispatcher's internal state being updated, leading to a divergence.

The thing is, I'm not sure we want to expose setState at all. If you need it you may as well create a new Redux instance. And you can always provide the initial state for a Redux instance when creating it.
I want the state flow from dispatcher to Redux and never the other way around. This makes it much easier to create custom dispatchers IMO.

I thought about it a bit more, and the only change I'll probably do here is make setState return the state that was set. This satisfies my use case.

So the signature stays the same (with initialState) and only setState behaves differently?

Yep.

:+1:


[owncloud/ nextcloud] Add "max-version" tag

Hello together,
to support the Nextcloud wich develops much faster, rainloop should add a max-version tag to the
info.xml
Nextcloud requires that tag in a few versions
so the dependencies should look like that:
<dependencies>
    <php min-version="5.3"/>
    <owncloud min-version="6" max-version="9.2" />
</dependencies>

additional you clould add an nextcloud element
<nextcloud min-version="6" max-version="11" />

Hello the Rainloop team,
Could we find a way to mutualize our work on the ownCloud and Nextcloud packages? I do maintain this repo https://github.com/pierre-alain-b/rainloop-nextcloud/ with a (not totally up to date) version of Rainloop for use in Nextcloud store.
For example, could you warn me everytime you build a new version of the ownCloud package? I could then use the same codebase to have Rainloop updated in the Nextcloud app store as well?
Let me know your thoughts on this.

Check out #479 and subscribe to that :-D

Thank you!

@ervee In fact #479 is not active anymore... too bad.

@RainLoop, this issue can be closed as the rainloop version is maintained with the correct tag in https://github.com/pierre-alain-b/rainloop-nextcloud


Illegal Reflective Access in Java 10

When using [com.google.cloud/google-cloud "0.47.0-alpha"] in Clojure [org.clojure/clojure "1.9.0"] using openjdk version "10.0.2" 2018-07-17, when my application starts, I receive the following warnings from the JVM:
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by io.grpc.netty.shaded.io.netty.util.internal.ReflectionUtil (file:/usr/local/bin/lrm-logging-service.jar) to constructor java.nio.DirectByteBuffer(long,int)
WARNING: Please consider reporting this to the maintainers of io.grpc.netty.shaded.io.netty.util.internal.ReflectionUtil
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

The project producing this error is using the following class imports from the Google Cloud library:
com.google.cloud.datastore: Key KeyFactory Datastore DatastoreOptions
com.google.cloud.logging: Logging Logging$WriteOption LoggingOptions LogEntry Payload Payload$JsonPayload
com.google.cloud: MonitoredResource

That's a really old version. Can you try in 0.62.0-alpha and see if it still happens?

The latest version in Maven Central is 0.42.0-alpha which is why I'm using it. I'm using the artifact link described in the README. Is this not the correct one to be using?
https://search.maven.org/search?q=g:com.google.cloud a:google-cloud

I also tried using these two dependencies instead of the 0.42.0-alpha for the "all-in-one" artifact with the same result:
[com.google.cloud/google-cloud-datastore "1.45.0"]
[com.google.cloud/google-cloud-logging "1.45.0"]

Off the top of my head, I'm not sure what the com.google.cloud:google-cloud artifact is. Wherever it comes from, it hasn't been updated since May. If 1.45.0 is failing, we probably need to take a look at this. I'm not sure what our official stance on Java 9 and 10 support is.
It looks like the issue comes from deep inside Netty:
WARNING: Please consider reporting this to the maintainers of io.grpc.netty.shaded.io.netty.util.internal.ReflectionUtil
so that's where the initial fix needs to go followed by an upgrade to our dependency here.


docs: amundsendatabuilder has been deprecated

Summary of Changes
The https://github.com/amundsen-io/amundsendatabuilder repository has been deprecated. Point to databuilder subdirectory instead.
Tests

Documentation

CheckList
Make sure you have checked all steps below to ensure a timely review.

[x] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[x] PR includes a summary of changes.
[x] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[x] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

@Fokko good to see you here, but there is a previous change (https://github.com/amundsen-io/amundsen/pull/1106) which did sth similar.

@feng-tao Great to be here! I've missed that PR, thanks for pointing it out, closing this one.


Typo when ding Lottery token transfer

When transferring the ticket there's a typo in the Window

Will fix. (Y)

Done.


unix: keep track of bound sockets sent via spawn

We use the UV_HANDLE_BOUND flag to mark a socket as bound to a
port. We need to do this for sockets that are sent from another
process as well as sockets that created by the process itself.
First check if the port number is non-zero. If yes then mark
it as bound.

I have added a regression test which would fail on z/OS before the fix in src/unix/tcp.c.
This is because the socket that is passed down to child loses it's UV_HANDLE_BOUND flag. So on z/OS, the
child process attempts to bind it again.

Fixed all nits.
I can certainly put the new code only for z/OS (which seems to be the only platform that needs to track whether sockets are bound or not :-))
How do you suggest doing this?
I can simply use #ifdef directives.
But if we want to avoid that, I could create a separate maybe_new_socket_zos(...) function.

But if we want to avoid that, I could create a separate maybe_new_socket_zos(...) function.

This makes sense to me. Let's see what other @libuv/collaborators think of it.

No opinion on the change itself, just noting that this needs a rebase.

rebased

@santigimeno just wondering if this still needs more time for comment from other collaborators or it can move forward  ?

I think separating the code into a new function is a good idea but don't want to block this any longer and it can be done in another PR. I'll land this later today.

@santigimeno thanks !

Sorry, I forgot about this. One last CI run: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/409/

There is relevant windows failure.

Talked with @jBarz to help him get access to Windows machines in order to debug.

Added a commit to fix the test case for windows. Can we try the CI run again?

CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/537/

Unfortunately CI link is stale.  New CI run: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/552/
and run on zos: https://ci.nodejs.org/view/libuv/job/libuv-test-commit-zos/211/

The new CI run failed for the following reason. Not sure why.

ERROR: Couldn't find any revision to build. Verify the repository and branch configuration for this job.
Notifying upstream projects of job completion

However, z/OS shows a clean success

@jBarz I set up the parameters incorrectly.  New run: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/553/

Saw this failure on windows
not ok 247 - threadpool_cancel_fs
# exit code 3
# Output from process `threadpool_cancel_fs`:
# A

Run on clean branch to see if that error occurs there or not https://ci.nodejs.org/job/libuv-test-commit-windows/643/

And this failure on linux
https://ci.nodejs.org/job/libuv-test-commit-linux/581/nodes=ubuntu1404-64/console
ok 101 - getaddrinfo_fail
not ok 102 - getaddrinfo_fail_sync
# timeout
# Output from process `getaddrinfo_fail_sync`: (no output)

Earlier instance of failure on linux  getaddrinfo_fail
https://ci.nodejs.org/job/libuv-test-commit-linux/573/nodes=centos6-64/console
so most likely not related.

Windows failure seems persistent.

The threadpool_cancel_fs test has been flaky for a while

Just rebased so hopefully another CI run might work

CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/554/

Different windows failure on different platform this time:
not ok 34 - fs_copyfile
# exit code 3
# Output from process `fs_copyfile`: (no output)

Same failure seen in last run was also present in a past run not related to this PR. https://ci.nodejs.org/job/libuv-test-commit-windows/635/

Opened https://github.com/libuv/libuv/issues/1618 for windows failure which I don't think is related to this PR.  So I think latest CI run for this PR was good.

Yep, fs_copyfile is definitely unrelated.

@cjihrig do you think this is ready to go ?

ping @cjihrig

@mhdawson I had only requested a test. No blocking objections from me.

Yesterday a launched a CI job for this and it came out fine: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/581/. Landing right now

Landed in https://github.com/libuv/libuv/commit/c5dd2d4218a79f2da0b365794139ad0f6e8cb60d removing a couple of unused variables that were causing warnings in the tests. Thanks and sorry for the delay on merging this.


Add live view for grapher directories

There are still some ui rough edges here, but this adds a basic live view that opens new plots in a given directory.

Sorry I haven't had a chance to look at this yet... I'll try tomorrow.

Just changed to this branch, when I do gulp compile-ts I get a compiler error:
app/scripts/app.ts(277,21): error TS1109: Expression expected.

Looking at the code, it looks fine? Do I have to upgrade my typescript first?

Well, that's broken. Line 277 should say 'await' instead of 'return'. I'll push a fix.

fixed.

Very cool. I'm able to run it now. Live update works, looks nice!
I get a gob warnings about async from the compiler. That's expected, right?
I'll continue to work through this.

There shouldn't be any warnings about async. Do you have typescript 1.7? Might just want to npm install and jspm install to be sure. (Also, you can do npm install -f typescript to update the global typescript package, though I don't think that's strictly necessary.)

$ tsc -v
message TS6029: Version 1.7.5

Stray UI/UX nits: The ? delimeter between the current folder and 'live' is confusing. It looks kind of like a formatting error. I would suggest distinguishing the live button from the path text by aligning it to the right hand of the nav bar (next to the logout) and making it a distinct button with an icon or icon.
Also as it stands I can't navigate to the live view when I have a graph presently open.

@joshmutus, I cleaned up the dir view/live view links to try to make it more clear that these are alternate views of the current directory. Let me know what you think. It's a bit awkward because the underline means a link to the other location, but it looks like an emphasis which might indicate the current location. There are various ways this could be addressed, but I think to really make this work nice would require restructuring element classes so that, for example, we could put toggle buttons in; right now I've just hacked the breadcrumbs class in an admittedly janky way.

A couple questions, but everything works. LGTM


[BACKLOG-5671] Re-architect the Job/Transformation Run Options dialogs code

@mdamour1976 Please Sir!...  Small refactoring issue that was causing a NPE during a Transformation test...


MyGroups: not all groups show up

Hi
I am using the module on the profile page and not all of the groups that I belong to show up. Most do. Is there criteria that is used to determine what groups are selected? Also, is there is easy way to show the page link instead just a picture?
I am not a developer but can figure some things out :)
Thank you
carl
ORIGINAL: https://dnnmygroups.codeplex.com/workitem/13397

UPDATE
I found out what the problem is. If a group role is defined as "BOTH" group and normal, all groups do not show up. I it is changed to just group role then they show up ok

It should work with that condition.  So, I'll get that updated soon.


Build the image with code commit

Need to find a way to pass the commit into the final binary. FYI @yanmxa


Extended Price Calculation

Hi,
When upgrading a customer from NAV2016 to BC17.1 we've migrated the old price structure to the new Price Lists (v16).
When implementing NAV2016 there was a customer specific request to develop a functionality to avoid creating prices/cost for every combination of Resource/Group and Work Type Code, since there is a global cost/price for every Work Types and a specific cost/price for a specific Work Type.
That customization was made in codeunit 220 Resource-Find Cost, adding the following code:

That way the user does not have to set up the price for every combination of Resource/Group and Work Type Code. He just needs to specify the global price (Work Type Code = '') and the exceptions for a specific Work Type.
Is there a way to apply this to the new functionality of price calculation v16? Or is the user always obliged to set up the prices for all possible combinations?

Will be added two new events:

OnAfterPutRelatedAssetsToList(PriceAsset, var PriceAssetList); to codeunit 7043 "Price Asset - Resource"
OnAfterPutRelatedAssetsToList(PriceAsset, var PriceAssetList); to codeunit 7044 "Price Asset - Resource Group"

These events return PriceAssetList that should be modified to add more assets at different levels with different parameters.
In your case you would need to add another price asset for the resource group and the blank work type code to the list below the asset for the resource group and the filled work type code that is added by "Price Asset - Resource" codeunit.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Availability update: We will publish a fix for this issue in the next update for release 18.


feat(verbose): print the exit map

Print the exit map when the verbose mode is enabled
I know there's a shortest way to do this(we can just make build_exit_map public and call it from the main function), but I took this way because I thought that we maybe wanted to track other things inside the clank instance, like the process_query method maybe?

Closes: #102

@0x61nas This looks awesome. TY.

I thought that we maybe wanted to track other things inside the clank instance, like the process_query method maybe?

That would be cool. Would you like to implement it?

Yes, of course + I have 4 PRs for hacktoberfest to make XD


error when redirecting to a root route '/'

atm if u tried to redirect to the root route ex.home => / you will get an error cuz now the url will be
something.dev//

Great catch. Thanks!

Is it solved yet?


header self-sufficience test

self sufficient header is a header which requires no other headers to be included before the header.
the example of an introduced issue because of introduced header non-self-sufficiency is multiprecision/roots.hpp in 1.70
an automated test of boost headers self-sufficiency would scan all boost headers and compile its inclusion lines. if compilation successful then the header test passed well.

@ohhmm : Could you reopen this issue against boost.multiprecision?


Allow users to compare different counties (easily)

This was a suggestion provided through user feedback, documented, I believe, by @philmcguinn here: https://docs.google.com/document/d/1A76kDOGFsNkRkDeKfCq1AJv3svGBjn4erFeSl-y2Diw/edit#
I happened upon it by chance and was very glad to have read it.  It provides a great case study of somebody who has made great use of our tools.
One of his feature requests / pain points:

Ability to choose more than one area to do side-by-side comparisons.  I usually just screen grab and build my own comparison charts.  For example, I hand-built a chart showing the deaths/100K for all the counties in PA - very informative.

I think this is a low hanging fruit, because (it appears) we already have most of the code written: for our "Top 10" tab: https://covidcast.cmu.edu/?sensor=doctor-visits-smoothed_adj_cli&level=county&date=20201107&signalType=value&encoding=color&mode=top10
It would seem that we could reuse the "Top 10" tab but simply allow the user to search for and add an arbitrary number of counties.  After all, in the current functionality, a user may search for a location, which appends that location after the Top 10.  But searching for another location replaces the previously-searched-for one.

Hey Addison! Glad that was useful, though the file is only under my name because I copied/pasted an email from back in August. Given the timing, his needs might not be the same. When I interviewed Bill again recently, viewable here, he seemed to have good luck comparing counties using the functionality on the main map that was added since the prior interview.
That said, I like the overall idea of making a more customizable compare mode on a dedicated page, since it's so information dense on the mapping page. I could see "Top 10" being one option on a tab dedicated to comparison of counties across a particular factor, instead of having a dedicated Top 10 tab.

Cool!  The leads can prioritize this however they like - it just seemed like a logical feature to build out :)

For example, I hand-built a chart showing the deaths/100K for all the counties in PA - very informative.

@huisaddison, can you attach the chart that you built here?  That would be a good illustration of what you'd like to see.

@tildechris I quoted that from an interview -- but it looks like compare mode has already been built!  :) Closing the issue.


Add unit tests for SlurmCommandGenStrategy

Summary
Add unit tests for SlurmCommandGenStrategy
Test Plan
CI passes

Please find the updated code, @amaslenn .


Associated models not automatically joined when sorting in list view

This is similar to issue #1543.
class Material < ActiveRecord::Base
  belongs_to :affiliate, inverse_of: :materials

  rails_admin do
    list do
      sort_by :updated_at

      field :description
      field :affiliate
      field :updated_at
    end
  end
end

Then if I go to http://localhost:3000/admin/material?sort=affiliate, I get the error message:
ActiveRecord::StatementInvalid at /material
PG::UndefinedTable: ERROR:  missing FROM-clause entry for table "affiliates"
LINE 1: SELECT  "materials".* FROM "materials"   ORDER BY affiliates...
                                                      ^
: SELECT  "materials".* FROM "materials"   ORDER BY affiliates.name asc LIMIT 20 OFFSET 0

https://github.com/sferik/rails_admin/issues/1543#issuecomment-496809733


[Docs] Represent Meshery's broader management capabilities

Current Behavior
Meshery manages all kinds of cloud native infrastructure. Branding it as a service mesh manager only misrepresents the full-breadth of its capabilities.
Desired Behavior
Replace most, but not all, instances of "service mesh" with "cloud native". Examples: "service mesh manager", "service mesh management plane", "multi-mesh manager", "multi-mesh management plane"... these should be replaced with "cloud native manager".

Contributor Guides and Resources

📚 Instructions for contributing to documentation

Meshery documentation site and source

🛠 Meshery Build & Release Strategy
🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

Sounds great.


Attaching SBOMs "--sbom" flag uploads literal filepath if file does not exist

If I run something like the following:
cosign attach sbom "${REMOTE_IMAGE}" --sbom=./sbom.txt

and the file sbom.txt does not yet exist, the blob uploaded contains the literal string contents ./sbom.txt.
Ability to upload raw string should be supported (and is via STDIN/--sbom=-), but I would expect an error if I'm attempting to load a file and it doesn't exist.
Source seems to be https://github.com/sigstore/cosign/blob/main/cmd/cosign/cli/attach/sbom.go#L63

@jdolitsky This behavior also happens to --signature= parameter for the attach signature command. I am not sure if it should be limited to stdin and files only.

I am not sure if it should be limited to stdin and files only

imho it should, how would you distinguish between the case where one wants to pass a literal "abcd" and a file called abcd. It only opens a new attack vector as it is implemented right now. This overloaded wannabe smart functionality doesn't do what it is expected if the file is not there.


select_subclasses not work with multi model inheritance

select_subclasses can't cast to model that was inherited from few models.
Code example:
class E(models.Model):
    objects = InheritanceManager()

    name = models.CharField(max_length=255)

class A(models.Model):
    namea = models.CharField(max_length=255)

class B(E, A):
    nameb = models.CharField(max_length=255)

B.objects.create(name='be', namea='ba', nameb='bb')
print E.objects.all().select_subclasses()
>>> []

By any change, is there any update on this issue?


RaFD dataset training

When training RaFD, you said the input domain is fixed as the "neutral" expression, the target domain varies among the seven remaining expressions.
But I have not found that the "neutral" is assigned in the training process.
out_src, out_cls = self.D(real_x) (in solver.py line 267 )
The real_x is read through torchvision.ImageFolder.
Could you help me? Thank you very much.

@wmonica That restriction is only for comparing between StarGAN and other cross-domain models such as CycleGAN and DIAT. In StarGAN, input domain can be one of eight expressions.


Short solution needed: "How to set password for Redis" (redis)

Please help us write most modern and shortest code solution for this issue:
How to set password for Redis (technology: redis)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments to make solution explained.
Link to this issue in comments of pull request.

How to set password for Redis


lightspeed auth: forceNewSession by default

Looks like a bug, we should re-open this issue if we think this will resolve the problem I am facing.

I think your local cache is corrupted for some reason. I'm not sure if we can recover from this state and if this is a consequence of some problem with the extension itself.


Support Pass Private Registry Configuration From CLI

--registry /etc/autok3s/registries.yaml

# registries.yaml
mirrors:
  docker.io:
    endpoint:
      - "https://mycustomreg.com:5000"
configs:
  "mycustomreg:5000":
    auth:
      username: xxxxxx # this is the registry username
      password: xxxxxx # this is the registry password
    tls:
      cert_file: # path to the cert file used in the registry
      key_file:  # path to the key file used in the registry
      ca_file:   # path to the ca file used in the registry

https://github.com/cnrancher/autok3s/issues/207

Failed to start cluster with --registry
Version: {"gitVersion":"v0.3.0-rc2","gitCommit":"3d01ce9dcb3c5cd6594b0dc6aa1dcfe7bb78b29f","gitTreeState":"","buildDate":"2020-12-25T02:01:30Z","goVersion":"go1.13.12","compiler":"gc","platform":"darwin/amd64"}
 ksd@Ksd  /tmp  autok3s -d create -p alibaba --name myk3s3 --master 1 --registry "/tmp/registry.yaml"

INFO[0000] [alibaba] executing create logic...
INFO[0000] [alibaba] generate default key-pair
DEBU[0005] [alibaba] 1 masters and 0 workers will be added in region cn-hangzhou
DEBU[0006] [alibaba] config default security group for vpc-bp1pmxlidz9jwc5fbl5f3 in region cn-hangzhou
DEBU[0006] [alibaba] get portRange [8999 8999] for security group sg-bp1ir35kqtnc3h25k6b0
DEBU[0006] [alibaba] get portRange [22 22] for security group sg-bp1ir35kqtnc3h25k6b0
DEBU[0006] [alibaba] get portRange [8472 8472] for security group sg-bp1ir35kqtnc3h25k6b0
DEBU[0006] [alibaba] get portRange [6443 6443] for security group sg-bp1ir35kqtnc3h25k6b0
DEBU[0006] [alibaba] get portRange [10250 10250] for security group sg-bp1ir35kqtnc3h25k6b0
INFO[0006] [alibaba] launching instance with auto-generated password...
DEBU[0006] [alibaba] prepare for 1 of master instances
DEBU[0008] [alibaba] 1 of master instances created successfully
DEBU[0008] [alibaba] waiting for the instances [i-bp1fxu3qio2csr8quq2i] to be in `Running` status...
DEBU[0038] [alibaba] instances [i-bp1fxu3qio2csr8quq2i] are in `Running` status
DEBU[0040] [alibaba] tagging eip(s): [eip-bp148pn8enz3u2qac5ixu]
DEBU[0040] [alibaba] successfully tagged eip(s): [eip-bp148pn8enz3u2qac5ixu]
DEBU[0040] [alibaba] prepare for associating 1 eip(s) for instance(s)
DEBU[0041] [alibaba] associated 1 eip(s) for instance(s) successfully
DEBU[0041] [alibaba] waiting eip(s) to be in `InUse` status...
DEBU[0041] [alibaba] eip(s) [eip-bp148pn8enz3u2qac5ixu: <IP_ADDRESS>] is in `Associating` status
DEBU[0071] [alibaba] eip(s) [eip-bp148pn8enz3u2qac5ixu: <IP_ADDRESS>] is in `InUse` status
DEBU[0071] [alibaba] eip(s) are in `InUse` status
DEBU[0071] [alibaba] Waiting for upload keypair...
DEBU[0072] [alibaba] upload the public key with command: mkdir -p ~/.ssh; echo 'ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAABAQDDRmHot8Jn2vS9ZeT7XbAI5otpU+CfCWYCbD8Vg4cx5UJP2ZZB7WI8Ab0gyDWn5QtHGkYhPGipO9fVsJDe18tn04ILiyc27LPSiutUKM744x0ji2vjRjVayDd48g40E6VRSZY6YL5ux2GZilx2qJLAwYpAxbM5HYu3ohrpPQyiEeiCIU/PJQjFF1i6jZuDTbP4cP2TwFKLwCou1pDMhB368OZ2TiChXphCunivmenVhj8k0HSnR4mNa9m5jInqQcIvFBlkK/pxtKfuS0MKYtCWsLODQFoAcUNTOWRxUSJY1mxa1xkzPgC6yeGLZM4jZssuNzx73Nu5+CHEZTZIwyBr' > ~/.ssh/authorized_keys
DEBU[0072] [alibaba] upload keypair with output:
INFO[0072] [alibaba] executing init k3s cluster logic...
INFO[0072] [alibaba] creating k3s master-1...
ERRO[0072] registry map is nil
INFO[0072] [alibaba] executing rollback logic...
DEBU[0072] [alibaba] instances [i-bp1fxu3qio2csr8quq2i] will be rollback
DEBU[0072] [alibaba] unassociating eip address for 1 master(s)
DEBU[0072] [alibaba] successfully unassociated eip address for master(s)
DEBU[0072] [alibaba] waiting eip(s) to be in `Available` status...
DEBU[0072] [alibaba] eip(s) [eip-bp148pn8enz3u2qac5ixu: <IP_ADDRESS>] is in `Unassociating` status
DEBU[0103] [alibaba] eip(s) [eip-bp148pn8enz3u2qac5ixu: <IP_ADDRESS>] is in `Available` status
DEBU[0103] [alibaba] eip(s) are in `Available` status
DEBU[0103] [alibaba] releasing eip: eip-bp148pn8enz3u2qac5ixu
DEBU[0103] [alibaba] successfully released eip: eip-bp148pn8enz3u2qac5ixu
INFO[0104] [alibaba] successfully executed rollback logic
 ksd@Ksd  /tmp  cat /tmp/registry.yaml
mirrors:
  "docker.io":
    endpoint:
      - "https://fogjl973.mirror.aliyuncs.com"
      - "https://registry-1.docker.io"
 ksd@Ksd  /tmp 

Test passed


fix(CodeQL): remove conditional for restore step

Description
Since I feared this yesterday already I monitored the CodeQL Action, and low and behold the conditional NuGet restore step caused issues.
Here is me fixing this with this PR to run that step regardless of cache-hit=true, sorry for the inconvenience.
Changes

change to run NuGet restore even on cache hit

Issue

n/a

@h1dden-da3m0n Is it possible to disable CodeQL for forked repos?

hi @tsvietOK IIRC yes, and I understand why u are asking :wink: (let me check and add this to this PR)

@tsvietOK Done :tada:

@h1dden-da3m0n Thank you for the fix!

@yaichenbaum I just cleaned up my own caused mess :wink:


Wrong year in license version

Description:
Second line of file LICENSE should be reverted back to Version 2.0, January 2004. It looks like typo or unintended change.
Suggested Labels:
Priority/Low, Typo
Affected Versions:
eeab96f25a4e6f6ff26f907f52fc282607e04bba-latest (24188b4f603775932320f10d15507bb26687e74c)

@vlna  Thank you for reporting this.

2016 is the correct year.

Hi @pahans, year 2016 in LICENSE is wrong. The year is related to release of license itself and not related to your code.

@vlna Yes, you are correct.  Reopened the issue.


added detail screen

I have added a detail screen with clean code.

Nice PR, but I created another one based on the fixes I have done. #5


chore: adding resource links to readme

SUMMARY

It was not easy to find certain things, so I've added them to the readme:

the In the Wild page, and other markdown files on the repo that have no links to them
Instructions to actually join the dev mailing list
The superset wiki
The database rolodex doc
The SIP board
The community calendar

Maybe this'll boost some visibility/engagement?
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

n/a
TESTING INSTRUCTIONS

read, and click links ;)
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

nice additions, thanks Evan!


Test across operating systems

Add system-dependent installation instructions

System-dependent installation instructions (specifically, MacOSX) have been added to the wiki.


[cti] Add --validator-tag

This command allows to override validator tag(similar to how to --cluster-test-tag allows to override cluster test tag).
Use case here is running cti with --pr and have it new cluster test image, but pick up some different(existing) validator image as a base

@bors-libra delegate+

:v: @andll can now approve this pull request

@bors-libra r=snakech

:pushpin: Commit 3b336ed has been approved by snakech

@bors-libra r=snakech

:bulb: This pull request was already approved, no need to approve it again.

There's another pull request that is currently being tested, blocking this pull request: #4672

:pushpin: Commit 3b336ed has been approved by snakech

:hourglass: Testing commit 3b336ed1bd53cec9e992ece3101165075aed379b with merge a0b0f7abd5ebaa009d65d87976757d8977c61559...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: snakech
Pushing a0b0f7abd5ebaa009d65d87976757d8977c61559 to master...


Unable to interact with modded blocks

Describe the bug
Does not interact with any modded blocks
To Reproduce
Steps to reproduce the behavior:

Open game
Put the hopper into or out of a block such as a sophisticated storage barrel
Put item into the hopper
Nothing moves

Expected behavior
Items moving from the hopper into the storage as a vanilla hopper does
Screenshots
If applicable, add screenshots to help explain your problem.
Environment

Minecraft version: 1.21.1
Mod version: [NeoForge] 1.21.x-2.1.0 [https://www.curseforge.com/minecraft/mc-mods/hopper-xtreme/files/5757579]
OS: windows 11 pro

Additional context
Tried some other blocks too and pulling out of but nothing modded interacted with it

Sorry for my delayed response on this. I will look into this and see what I can figure out. Thanks for the report!

No worries don't stress about taking your time :) I had already moved onto another solution installed in my modpack but figured I still mention it as you had a cool mod :)

I just tried with both Minekea (another of my mods) and Sophisticated Storage. Minekea's modded containers worked fine, but the barrels from SS didn't. It seems like the other dev might be doing their own kind of inventory management, but I am not sure. I definitely noticed that even some vanilla behavior (middle-click, middle-click with drag, etc) doesn't work in the SS barrel, so my gut tells me the issue is on their end.
I'll leave this open for now, in case there are other mods besides Sophisticated Storage that you were having issues with, but in that case at least, it seems to be a compat issue on their side.

Same problem. 1.21.1, hopper xtreme 2.2.1, neoforge
hopper xtreme absolutely doesn't work with sophisticated storage or sophisticated backpacks. Vanilla hoppers work fine, pipes from 3 different mods all work, hopper xtreme does not work. I'm not sure it's a sophisticated $thing issue?

Same problem. 1.21.1, hopper xtreme 2.2.1, neoforge hopper xtreme absolutely doesn't work with sophisticated storage or sophisticated backpacks. Vanilla hoppers work fine, pipes from 3 different mods all work, hopper xtreme does not work. I'm not sure it's a sophisticated $thing issue?

Which mods' pipes work with the sophisticated stuff? It's possible they have some special handling for sophisticated storage, so if I can see what someone else is doing I may be able to replicate it.
For my part, I am just checking if the block implements InventoryProvider or if the block entity implements Inventory. I'd be happy to add support if I can, but sophisticated storage doesn't use either of those interfaces as far as I have been able to determine.


golang PURLs contain URL-encoded Go module names

As per https://owasp.slack.com/archives/C6R3R32H4/p1708437612711809, it seems that cdxgen is URL-encoding Go module names in the purl field of the BOM.
For example:
{
  "group": "",
  "name": "github.com/apache/thrift",
  "version": "v0.0.0-20190309152529-a9b748bb0e02",
  "hashes": [
    {
      "alg": "SHA-256",
      "content": "729d92b96331944670dabfa23f618d09d222e02d6a994cdd6454956fe6da7304"
    }
  ],
  "purl"<EMAIL_ADDRESS>  "type": "library",
  "bom-ref"<EMAIL_ADDRESS>}

While this is not technically wrong (one could argue github.com/apache/thrift is the package name, in which case it should be URL-encoded to differentiate it from the namespace), it causes friction with other tools (Dependency-Track, OSV, possibly more) that do not expect it.
For example, querying OSV with the URL-encoded version yields no results, but it does for the non-encoded version:
$ curl --silent -d \
  '{"package": {"purl"<EMAIL_ADDRESS>\
  "https://api.osv.dev/v1/query" | jq '. | length'
0

$ curl --silent -d \
  '{"package": {"purl"<EMAIL_ADDRESS>\
  "https://api.osv.dev/v1/query" | jq '. | length'
1

@nscuro, Thank you for filing this issue. I was doing some research. Go doesn't have a concept of namespace at all. A module name (which translates to an SBOM component) is always in the full name form, so the purl generated has the URL encoded names which match your observation.
I feel the best way to resolve this is to get something official from package URL authors. Since cdxgen uses packageurl-js library, any package manager type-specific workarounds must be performed officially by these libraries rather than the individual tools.
This way, other tools, such as depscan (which uses package-python), would work without any custom hacks.
cc: @pombredanne

Related: https://github.com/package-url/packageurl-python/blob/ef3747a2e6c136f3fbe1742c81d6da295cee4546/CHANGELOG.rst#0130-2023-12-08
Unlike packageurl-python, cdxgen simply cannot put things under namespace without breaking a lot of downstream tools :(

Looks like this problem exists for npm as well since our move to Arborist.

Resolved in 10.3.0


x/mobile/app: trying to close the closed channel

There is a race condition I came across on android.go. It's reproducible with the program below.
import "golang.org/x/mobile/app"

func main() {
	app.Run(app.Callbacks{})
}

Launch the app.
Use back button to stop it.
Relaunch the app.

The following panic happens.
I/GoLog   (12527): app.Run called
I/Go      (12527): app.Run
E/Go      (12527): panic: close of closed channel
E/Go      (12527): goroutine 12 [running, locked to thread]:
E/Go      (12527): golang.org/x/mobile/app.run(0xf43bae30, 0xf43bae34, 0xf43bae2c, 0xf43bae38)
E/Go      (12527): 	/Users/jbd/src/golang.org/x/mobile/app/android.go:281 +0x144
E/Go      (12527): golang.org/x/mobile/app.Run(0xf43bae30, 0xf43bae34, 0xf43bae2c, 0xf43bae38)
E/Go      (12527): 	/Users/jbd/src/golang.org/x/mobile/app/app.go:20 +0x3c
E/Go      (12527): main.main()
E/Go      (12527): 	/Users/jbd/src/github.com/rakyll/gopher/main.go:77 +0x50
E/Go      (12527): golang.org/x/mobile/app/internal/callfn.CallFn(0xf420488c)
E/Go      (12527): 	/Users/jbd/src/golang.org/x/mobile/app/internal/callfn/callfn_arm.s:10 +0x20
E/Go      (12527): created by golang.org/x/mobile/app.callMain
E/Go      (12527): 	/Users/jbd/src/golang.org/x/mobile/app/android.go:79 +0x16c
E/HTC Acoustic(  202): set_rt5506_amp:mode = 0 device = 0x4
D/HTC Acoustic(  202): query headset om 3
E/Go      (12527): goroutine 17 [syscall, locked to thread]:
D/HTC Acoustic(  202): premode = 7, rt5506mode=7

/cc @crawshaw @hyangah

I rewrote a lot of the state handling this morning to deal with this and several other bugs and limitations. I'll pull it apart and send out the CLs over the next couple days.


adds support for client hints width/viewport-width.

LGTM except for a couple of things:

Please, use version/2.1 branch as a base. I switched back from fasthttp to the built-in net/http, and your code became incompatible.
It's better to disable client hints by default since it's a dangerous thing as I said before. Let's add a config value IMGPROXY_ENABLE_CLIENT_HINTS.

Thanks for the feedback, I missed the fasthttp switch, I've opened a new PR from version/2.1 branch with the env variable too.
https://github.com/DarthSim/imgproxy/pull/98


[GHSA-jrcf-4jp8-m28v] miow invalidly assumes the memory layout of std::net::SocketAddr

Updates

Affected products

I saw the comment with the link to the pull request for the backported fix. The fix will go into the references section in addition to the vulnerable version range being updated to show that 0.2.2 is patched.


Memory usage keeps growing (1.0.32)

Memory usage of MetaTrader 4 grows rapidly when using MtApi.
To reproduce start a backtest in MT4 with the strategy tester (one day is enough), run app below (which does nothing but connect), watch the memory usage of the MetaTrader process in the task manager quickly grow to several GB's.
using MtApi;
using System;

namespace MT4ClientConsole
{
   class Program
   {
      static void Main(string[] args)
      {
         MtApiClient client = new MtApiClient();
         client.BeginConnect("<IP_ADDRESS>", 8222);
         Console.ReadKey();
      }
   }
}

Yes, I aslo found this problem in last versions of MtApi (MT4 and MT5).
I will try to fix it in version 1.0.33

Thank you, really interesting project you have here.

I have found memory leaks in log4net libraries (logging).


Nothing happend

Hi
I am trying to use your ROS package for Anafi 4k.
All instalaltions are good.
When I run :
roslaunch olympe_bridge anafi.launch
I get :
SUMMARY
PARAMETERS

/anafi/drone_serial: I delete it
/anafi/ip: <IP_ADDRESS>
/anafi/model: 4k
/anafi/name: anafi
/anafi/rest_api_version: 1
/anafi/skycontroller_enabled: True
/anafi/wifi_key: I delete it
/rosdistro: noetic
/rosversion: 1.16.0

NODES
/anafi/
anafi (olympe_bridge/anafi)
set_anafi_log_level (rosservice/rosservice)
auto-starting new master
process[master]: started with pid [12635]
ROS_MASTER_URI=http://localhost:11311
setting /run_id to f6afa2f8-fa15-11ed-a6bf-598011ea070e
process[rosout-1]: started with pid [12660]
started core service [/rosout]
process[anafi/set_anafi_log_level-2]: started with pid [12667]
process[anafi/anafi-3]: started with pid [12668]
[INFO] [1684920778.876837] [/anafi/anafi]: Anafi_bridge is running...
[INFO] [1684920778.931761] [/anafi/anafi]: Connecting through SkyController
[INFO] [1684920778.984018] [/anafi/anafi]: Connecting to I delete it with I delete it
[anafi/set_anafi_log_level-2] process has finished cleanly
log file: /home/zoe/.ros/log/f6afa2f8-fa15-11ed-a6bf-598011ea070e/anafi-set_anafi_log_level-2*.log
after that nothing happened and rostopic echo /anafi/battery/voltage for example  give nothing too.
Did I miss somthing ? I tried with skycontroller on and tried off. same thing.
THanks by advance

It looks like the skycontroller does not connect to the drone. Try to connect to the default paired Anafi. If I got it correctly, you are trying to connect to Anafi with a specific SN and WIFI key. Instead, I recommend connecting to the default paired Anafi. So, just leave SN and WIFI key in the launch file blank as in the original file.

Check the reply to this issue: https://github.com/andriyukr/anafi_ros/issues/3#issuecomment-1560812643

THanks, now it's working.
I didnt undurnsdunfd before that  Skycontroller sould be connected to PC with usb cable.
2.Skycontroller-PC connection problem: check whether skycontroller is properly connected to your PC. In the terminal, write ifconfig and check whether there is a USB device with <IP_ADDRESS> IP address. If yes, then the connection is fine.


tests (ticdc): skip one unstable http_api integration test case

What problem does this PR solve?

Issue Number: ref #2958
What is changed and how it works?
Check List
Tests

No code

Code changes
Side effects
Related changes
Release note
None

/run-integration-tests

/run-integration-tests

/merge

/run-kafka-integration-tests


Final steps migration to CNCF infrastructure

This issue tracks the progress of the migration to CNCF infrastructure.

[x] moved the GCP cert-manager-release project to the CNCF GCP organisation
[x] clean-room deploy of cert-manager testing infrastructure in new GCP projects
[x] bug and stability fixes
[x] via CNCF service desk: point registar to new CloudDNS instances on GCP
[x] install triage-party tool
[x] migrate Netlify
[x] tear-down old testing infrastructure (https://github.com/jetstack/terraform-jetstack-build-infra/pull/36)

Can we close this ticket? Or do we still need to do something? I think we have decided we want to continue using http://triage.infra.cert-manager.io/

The migration has been completed (ofc. further improvements are possible and desirable, but will be tracked separately). 🥳

Hey, about the migration of the Kubernetes clusters, is the documentation up to date regarding the Kubernetes name and GCP project name, e.g., in https://github.com/cert-manager/testing/tree/master/prow?


Seems not work with shield

when having shield installed, I got following exception:
[2016-01-26 10:24:05,236][ERROR][org.codelibs.elasticsearch.reindex.service.ReindexingService] [Autolycus] Failed to reindex tweets2.
ElasticsearchSecurityException[action [indices:data/read/search] is unauthorized for user [__es_system_user]]
Even though I provided -u in the command line. I guess "Reindex" is a two-step operation, -u only used for reading part. The writing part does not get the -u option

I used the java lib commons.httpclient send the reindex request,
add the anthorization info like this
client.getState().setCredentials(AuthScope.ANY, new UsernamePasswordCredentials(username, password));
the reindex plugin received the request, but the reindex process is not correctly created


Train, Validation and Test split overlap

I was reviewing the files which were split by labelme2yolo and noticed that the function create splits with overlap.
In my case I have 1231 files, and I pass val_size=0.15 and test_size=0.1 and as the result of this it was split:
train=922
validation=185
test=124
Issues:
The amount of files by split seems fine.

But when I move all split files (images or annotations) into a single folder file manager says the file exists and I skipped the same files. This resulted in total of 1077 files. This means 154 files were not utilized.
Verified duplicates with a python script:

Overlap between Train and Val: 141 files
Overlap between Train and Test: 0 files
Overlap between Val and Test: 13 files

Currently I'm trying to find out why it's not right, but could you please also check this.
Also this issue was reported in the original branch. https://github.com/rooneysh/Labelme2YOLO/issues/5

this part of the code is not correct I believe:
train_idxs, test_idxs = train_test_split(
    range(tmp_train_len), test_size=test_size / (1 - val_size))

I think it should be:
train_idxs, test_idxs = train_test_split(
    train_idxs, test_size=test_size / (1 - val_size))

as we want to split already created train indexes further for new train and test. Will test and post the result here.

@GreatV I just tested and seems like it works correctly now. I commited the updated code to the repo.

Verified duplicates with a python script:

Overlap between Train and Val: 0 files
Overlap between Train and Test: 0 files
Overlap between Val and Test: 0 files

Amount of combined split files are now matching the total amount.

@Arshadoid Thanks for the feedback, you can submit a pr to fix the issue.

@Arshadoid Thanks for the feedback, you can submit a pr to fix the issue.

Thank, will do now. This will be my first contribution :)

Issues was fixed.


Init TracerProvider ourselves to set immutable Resource attribute

Description
We are getting ready to upstream this AwsLambdaInstrumentor to the OpenTelemetry Python Contrib repo. In doing so, there is a caveat because one of the Resource attributes cannot be set until the Lambda function is actually invoked and the lambda_context is available.
This PR tries to solve that. Instead of letting the opentelemetry_configurator entry point configure the TracerProvider. We take the relevant methods and try to initialize the TraceProvider ourselves. We must, because we can't initialize again after it's done. It is immutable (and so is its Resource).
There are some other miscellaneous formatting changes and doc changes. If it's too busy I can remove these and focus on this solution first!

@wangzlei @anuraaga Still thinking this through and will have to test it. But would appreciate your initial thoughts? Especially because there are a lot of changes needed just to get 1 attribute on the spans.


Use local duplicate of AuctionState struct

Create a duplicate of AuctionState struct in preparation for it being removed from casper-types.

Closes https://github.com/casper-network/casper-sidecar/issues/365


Can't resolve 'Dappy' component

Hey folks - thanks for open sourcing this!
I'm noticing some components (e.g. DappyModule) import Dappy from the /Flock/ directory but it doesn't seem to be present. When I try to run yarn start on the project, it yields these errors:
 ERROR #98123  WEBPACK

Generating development JavaScript bundle failed

Can't resolve '../Flock/Dappy' in '/Users/samrichards/code/example-dapp-high-priests/src/components/Redeem'

File: src/components/Redeem/RedeemDappy.js

 ERROR #98123  WEBPACK

Generating development JavaScript bundle failed

Can't resolve '../Flock/Feather' in '/Users/samrichards/code/example-dapp-high-priests/src/components/Redeem'

File: src/components/Redeem/RedeemDappy.js

Am I missing something? Or is there a missing component that hasn't been committed?
Thanks! Keep up the great work 😄

Hey @samajammin! Had the same issue a couple days ago, went ahead and dropped a line to @pi0neerpat, who very helpfully explained RedeemDappy contains the Dappy checkout widget (the yellow pane one on the far right you can find in any flock page).
Since it is not open sourced, (Dappy is a service in itself) it was omitted on purpose by the devs, which makes perfect sense.
Removing the code in question clears the errors and (of course) removes the widget from the build!

This is correct. Also, some new work and (lots of) clean up has been made since the hackathon. I moved to a private repo to make my workflow easier, but I'm happy to make a PR here if y'all are interested

Thanks, @pi0neerpat, I would be interested! Or I'd be curious if you recommend any Gatsby starter dapp templates I could check out. Cheers.

@samajammin and @blsnttt I've updated the repo with my latest changes:

Lots of clean up and organization
Stripped out Dappy so you don't have to
Added better loading using react-loading-skeleton
Update readme

Thanks for you patience. Let me know if yall need anything else of have questions!


Add a bar chart to page 1 and make footnotes page links open in new tab

Pull request overview
Resolves #55, resolves #31.
Pull request checklist
Please check if your PR fulfills the following:

[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[ ] Tests have been run locally and are passing (shinytest2::test_app())
[x] Code is styled according to tidyverse styling (checked locally with styler::style_dir() and lintr::lint_dir())

What is the new behaviour?
Have added a bar chart to the front page that shows regions with an interactive tool tip.

Updated the footnotes links to use the dfeshiny::external_link() function.
Anything else
Tests are failing though expecting #70 to resolve this separately.
Took a look at the cookies / analytics setup but have got a bug to fix in dfeshiny first so left them for now, I'll pick them up at a later date.

This is looking better with a chart here, would it be possible to have it so the bars are selective?


[ENQUIRY] why decompressing is using ByteBuffer.capacity instead of ByteBuffer.limit as its boundary

Hi, I'm using version 1.2.0, which is the newest version available as of writing this,
And I found no open or closed issue regarding this question.
com/github/luben/zstd/Zstd.java around line 695:
    long size = decompressDirectByteBuffer(dstBuf, //decompress into dstBuf
    dstBuf.position(), //write decompressed data at offset position()
 ***dstBuf.capacity() - dstBuf.position(), //write no more than capacity() - position()
    srcBuf, //read compressed data from srcBuf
    srcBuf.position(), //read starting at offset position()
    srcBuf.limit() - srcBuf.position()); //read no more than limit() - position()

and later in line 704: dstBuf.position(dstBuf.position() + (int)size);
also in corresponding JNI native code, it seems that the dstBuf's limit is not respected.
this will not only cause runtime exception as shown below:
Exception in thread "main" java.lang.IllegalArgumentException
    at java.nio.Buffer.position(Buffer.java:244)
    at com.github.luben.zstd.Zstd.compress(Zstd.java:354)
    ...

but it will also cause data corruption in violation to ByteBuffer's contract.
So my question here: is this a design decision? if so, why?

Looks like an error

@luben OK, hope it get fixed soon

The list commit (I just pushed it) makes the fix. I will push later today to sonatype/maven.

@revintec , just pushed 1.2.0-2 with the changes


Continue create workflow post gitops failure

Issue #, if available:
Description of changes:
Continue create workflow post gitops failure
Testing (if applicable):
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/approve


Increase ssh tunnel's kChunkSize for faster reads

This greatly reduces the hangs we are sometimes seeing on capture
stop. The real cause is the fact that the tunnel is updated on the ui
thread, which shouldn't be the case as it can being drastically delayed
by a long render frame.
See b/172686811

Reviewed in #1434, as discussed, pushing to release branch.


playlist file not created

Hi, in the instructions it doesnt say where the file you create will be saved?
when I run the script it works, i can see its finding my playlists but then it exits and I cant find the file it was supposedly creating
also when i run it in "IDLE" i get this:
Enter a file name (e.g. playlists.txt): c:\temp\tg.txt
Traceback (most recent call last):
File "C:\temp\spotify-backup.py", line 170, in
main()
File "C:\temp\spotify-backup.py", line 141, in main
log('Logged in as {display_name} ({id})'.format(**me))
File "C:\temp\spotify-backup.py", line 115, in log
sys.stdout.buffer.write('[{}] {}\n'.format(time.strftime('%I:%M:%S'), str).encode(sys.stdout.encoding, errors='replace'))
AttributeError: 'PseudoOutputFile' object has no attribute 'buffer'

The file gets saved in the working directory. Often that's where the python file is located but it can end up anywhere else depending on how you run the file.

This was an issue with logging, fixed.


Support running tests ('behave features/') on OS X.

Right now, 'behave features/' command fails on OS X with boot2docker. This is because docker-py client has to be created differently on that OS, and additionally, os.ttyname will not work.
Since dockerpty itself works on OS X (with slight changes to test code snippet, as shown below), it would be nice if tests worked too.
Great project!
Code snippet:
import docker
import dockerpty
from docker.utils import kwargs_from_env

kwargs = kwargs_from_env()
kwargs['tls'].assert_hostname = False

client = docker.AutoVersionClient(**kwargs)
container = client.create_container(
    image='busybox:latest',
    stdin_open=True,
    tty=True,
    command='/bin/sh',
)

dockerpty.start(client, container)

If tty isn't used I suggest you remove the call to ttyname instead of setting tty to something bogus. Other than that: :+1:

Thanks! Sorry it took so long to get to, but I've done an updated cherry-pick in #55.


common: spec out Scene Clipper

Scene Clipper is an unusual feature
that is too unstable and ambiguous in ThorVG.
Users can achieve the same functionality
with multiple composed shapes instead of scene clipping.
bin size: -2.5kb
issues:

https://github.com/thorvg/thorvg/issues/1548
https://github.com/thorvg/thorvg/issues/1549
https://github.com/thorvg/thorvg/issues/1573

Invalid case:
shape->composite(std::move(scene), tvg::CompositeMethod::ClipPath);
Still valid case:
scene->composite(std::move(shape), tvg::CompositeMethod::ClipPath);
I`m right?

I`m right?

@SergeyLebedkin yes. :-)