When building with ipa there is no xcarchive and the dsym file has a wrong name

xcclean
xcarchive(
  scheme: "Beta",
  archive_path: "build/Beta.xcarchive",
  output_style: :standard
)
xcexport
dsym_zip(
  archive_path: "build/Beta.xcarchive",
  dsym_path: "build/Beta.app.dSYM.zip"
)

resign(
  signing_identity: "iPhone Distribution: Roland leth (123456)",
  provisioning_profile: "fastlane/adhoc.mobileprovision"
)

hockey(
  api_token: "123456",
  notify: "1",
  ipa: "build/Beta.ipa",
  dsym: "build/Beta.app.dSYM.zip",
  tags: "alpha"
)

Properly works: .ipa, .xcarchive and dsym.zip are all present, named properly.
ipa(
  configuration: "Beta",
  scheme: "Beta",
  ipa: "Beta.ipa",
  destination: "build",
  clean: true,
  archive: true
)

resign(
  signing_identity: "iPhone Distribution: Roland leth (123456)",
  provisioning_profile: "fastlane/adhoc.mobileprovision"
)

hockey(
  api_token: "123456",
  notify: "1",
  ipa: "build/Beta.ipa",
  dsym: "build/Beta.app.dSYM.zip",
  tags: "alpha"
)

Does not work properly: there is no .xcarchive, the .ipa is named correctly, but the dsym.zip is named MyAppName.app.dSYM.zip. Then, of course, this error is raised: Symbols on path 'project-path/build/Beta.app.dSYM.zip' not found.

How is the dsym wrongly named? It's working fine for me. What's not as expected?
And yes, the ipa action doesn't generate a xcarchive. The ipa action uses shenzhen to build and sign your app.

For me the ipa is named Beta.ipa, while my dsym is named MyApp.app.dSYM.zip and that error is taken from terminal - it appears it's trying to read the name I gave it, Beta.

@rolandleth What is the Product Name in your build settings? Is that set to "MyApp"?

@rolandleth @KrauseFx I actually ran into this myself last night. I thought I was doing something stupid but maybe this is something that a newer version of Xcode caused?

Yes, it is. I didn't name them beta/debug/etc. But still, I name it beta inside the dsym command 😔

@rolandleth Can you drop a sample of what your dsym command looks like? I can take a look at this today (as I know I can duplicate the issue) and see if I can get a fix in for you? :wink:

Thanks @joshdholtz for fixing this :+1: :rocket:

@KrauseFx I haven't fixed it yet :wink: But I think I should be able to fix this :stuck_out_tongue_winking_eye:

So much wink today I guess - I got some good sleep :zzz: :rocket:

Hmm, doesn't the ipa command create the dsym? :S

It does 😊 I was using the IPA command myself but there is also a dsym command so wasn't sure if you were also using that. I am on this!
On Wed, May 13, 2015 at 7:34 AM, Roland Leth<EMAIL_ADDRESS>wrote:

Hmm, doesn't the ipa command create the dsym? :S
Reply to this email directly or view it on GitHub:
https://github.com/KrauseFx/fastlane/issues/277#issuecomment-101650577

Awesome!
While we're on this, I did manage to finish the process with xc commands, the build ended up on hockey, it said it's built with the proper identity/profile, but no one can download it. Any idea what's happening?

Hi @rolandleth! Are you still running into these issues? You've probably seen the new gym, are you using that one now? :)

Hey, yes I'm using gym and don't have any problems with it 😁 Sorry for not updating 😞

No problem, thanks for the response @rolandleth :+1:


adds bug fixes for laravel 5.2

Fixes typo from provide to provider and correct $config path in NorthstarServiceProvider.php.

👍!

oh wait @DFurnes found another error! Just fixed it - let me know if this is ok!

🚢


Declared License is missing

Type: Missing
Summary:
Declared License is missing
Details:
Declared license was missing.
Resolution:
Updated the declared license as per https://github.com/scikit-image/scikit-image/blob/v0.20.0/LICENSE.txt.
Affected definitions:

scikit-image 5e74a4a3a5149a8a14566b81a32bb15499aa3857

@ariel11 Do we consider all of the individual callouts as part of the overall license since this is for the GitHub repo or look at them as third party code?

@capfei - Well, I think when the license file makes a distinction between a default/overall license for most of the project and then licenses that apply only to a subset of files, we use that to distinguish declared vs discovered in a repo.  Here, I read the license file as saying the 'main' license for all the files, unless otherwise called out, is BSD-3-Clause.  And then the other licenses apply to a small set of files.  So my recommendation on this would be BSD-3-Clause for declared and the rest as discovered. Thoughts?


Client - index range error when entering blank input

Describe the bug
Opening ticket for tracking

When entering a blank input in the Hummingbot client, it triggers the following error:

2021-07-15 23:38:41,093 - 1 - hummingbot.client.hummingbot_application - ERROR - list index out of range
Traceback (most recent call last):
  File "/home/hummingbot/hummingbot/client/hummingbot_application.py", line 142, in _handle_command
    if command_split[0] == s['command']:
IndexError: list index out of range

although it doesn't cause any issue. This behavior can occur when using a docker build and source build.
Steps To Reproduce
Here are the steps to reproduce the issue (see attachments in the section below):

Open the Hummingbot client
Press enter multiple times without entering any input from the input pane.
...

Screenshots

Release version
0.41.0
Attachments
// Please attach your config file and log file to help us debug. It would be difficult for us to help you without those!
WARNING: Do NOT publish any exchange API keys or your wallet's private key. Whoever has access to them may steal your assets!
Optional: your discord username:
┆Issue is synchronized with this Clickup task by Unito

Hello, it's me
Change the line 140 in client/hummingbot_application.py from
            if shortcuts is not None:

in
            if shortcuts is not None and len(command_split):

m.c.

Fixed on staging (dev-1.2) and development branch


unable to run character-io and byte-io samples from composer

Both samples can be executed by running ballerina run from within the sample folder ex: ~/products/ballerina/build/23042018_2/ballerina-0.970.0-beta13-SNAPSHOT/bin/./ballerina run byte-i-o.bal
Response from command line
Start to copy files from ./files/ballerina.jpg to ./files/ballerinaCopy.jpg
File copy completed. The copied file could be located in ./files/ballerinaCopy.jpg

However, if we open the samples from the composer the samples cannot be run

Running byte-i-o.bal

ballerina: HTTP trace log enabled

error: ballerina.runtime:CallFailedException, message: call failed

	at ballerina-examples.byte-i-o:main(byte-i-o.bal:71)

caused by error, message: failed to open file: file not found: ./files/ballerina.jpg

	at ballerina.io:openFile(<native>)

Execution Ended.

Running character-i-o.bal

ballerina: HTTP trace log enabled

error: ballerina.runtime:CallFailedException, message: call failed

	at ballerina-examples.character-i-o:main(character-i-o.bal:69)

caused by error, message: failed to open file: file not found: ./files/sample.txt

	at ballerina.io:openFile(<native>)

Execution Ended

Steps
1.browse the ballerina-examples/examples folder
2.open the byte-io sample
3.click on run option on the composer

what is the way to run the 3 samples from the composer


Execution failed for task ':app:compileDebugJavaWithJavac'.

I'm getting this error when I execute react-native run-android.
:app:compileDebugJavaWithJavac/home/esganzerla/helloredux/android/app/src/main/java/com/helloredux/MainActivity.java:19: error: cannot find symbol
	protected void onCreate(Bundle savedInstanceState) {
	                        ^
  symbol:   class Bundle
  location: class MainActivity
/home/esganzerla/helloredux/android/app/src/main/java/com/helloredux/MainActivity.java:24: error: package ReactInstanceManager does not exist
		getReactNativeHost().getReactInstanceManager().addReactInstanceEventListener(new ReactInstanceManager.ReactInstanceEventListener() {
		                                                                              ^
2 errors
 FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:compileDebugJavaWithJavac'.
> Compilation failed; see the compiler error output for details.

MainActivity.java
package com.helloredux;

import com.facebook.react.ReactActivity;
import com.mehcode.reactnative.splashscreen.SplashScreen;
import com.mehcode.reactnative.splashscreen.SplashScreenPackage;

public class MainActivity extends ReactActivity {

    /**
     * Returns the name of the main component registered from JavaScript.
     * This is used to schedule rendering of the component.
     */
    @Override
    protected String getMainComponentName() {
        return "helloredux";
    }

	@Override
	protected void onCreate(Bundle savedInstanceState) {
		// Show the js-controlled splash screen
		SplashScreen.show(this);

		// After react is initialized; set our background color (override splash screen theme)
		getReactNativeHost().getReactInstanceManager().addReactInstanceEventListener(new ReactInstanceManager.ReactInstanceEventListener() {
			@Override
			public void onReactContextInitialized(ReactContext context) {

				getWindow().getDecorView().setBackgroundColor(Color.WHITE);
			}
		});

		super.onCreate(savedInstanceState);
	}
}

you probably forgot an import

I did the react-native link rn-splash-screen and then followed the instructions as shown in the android.md...
Can you tell me if I should import anything other than import com.mehcode.reactnative.splashscreen.SplashScreen;?
I'm using RN 0.35 as well.
Thanks!

I've just checked the example for android MainActivity.java and there are a few more imports than the standart file I had.
I'll try to add those imports tomorrow and update the instructions, if needed.

Yep. It did work!
Thanks @vonovak


Abstract object/array type out so that arbitrary types can be used

As suggested by @palacaze in #161, this allows using an unordered_map
as well as anything that obeys the template<K,V> type for the object
type or template for the array type.

This patch replaces #165 based on the comments that were received there.  It does nothing about the allocators, as the suggestions in #161 will be able to done after this.  This DOES however make the allocator changes easier, so I'll put together a pull request based on this in the future.

This patch replaces #165 based on the comments that were received there.  It does nothing about the allocators, as the suggestions in #161 will be able to done after this.  This DOES however make the allocator changes easier, so I'll put together a pull request based on this in the future.

Thanks a lot for the PR. Can you please make the changes also to file json.hpp.re2c - file json.hpp is generated out of that file.

@erichkeane Sorry for not coming back to this PR earlier. There were quite some fixes on the MSVC side, so I would like to ask you to resubmit the PR to work on the latest version.

@erichkeane Sorry for not coming back to this PR earlier. There were quite some fixes on the MSVC side, so I would like to ask you to resubmit the PR to work on the latest version.

Sure! I can rebase these.

Closed here - overworked PR at #209.


pasting with thousand seperator

I have a sum formula in the last cell of a column, which is adding up the numbers above. When I paste or enter numbers into the cells above the formula without thousand separators, the sum is correct. If I paste or enter numbers with thousand separator the formula is not calculated correctly but shown as concatenated string.

Javascript must consider your input as Number, if for any reason consider a string the result would be concatenation. Sorry, I am not sure we can do something about that, one possible is MASKING the input to remove any character.
http://localhost:8002/jexcel/examples/currency-and-masking-numbers


minor version upgrades for OpenTracing API and Hazelcast

tests passed with ./mvnw test.
also tested using Hazelcast 3.12.1 but 3.12 introduced breaking changes.

Pull Request Test Coverage Report for Build 30

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 17.297%

Totals

Change from base Build 29:
0.0%

Covered Lines:
352

Relevant Lines:
2035

💛  - Coveralls


Strict mode runtime error without warning

With the following tsconfig.json:
{
	"compilerOptions": {
		"target": "es5",
		"module": "amd"
	}
}

The following file:
if( true ) {
	function foo() {}
}

export = foo;

Compiles with no errors or warnings, and emits:
define(["require", "exports"], function (require, exports) {
    "use strict";
    if (true) {
        function foo() {
        }
    }
    return foo;
});

This, however, results in a runtime error in Firefox, because in strict mode, function declarations may not appear within if blocks.
I believe this should cause a compiler error, or at very least, a warning.
This caused an issue on our production website today.  When we updated our TypeScript compiler, our code, which was working without "use strict", stopped working (without any warning or build issues).  I'm currently trying to make sure nothing else broke because of "use strict", but the lack of warnings makes finding these issues difficult.

FWIW this was listed as a breaking change in 1.8 here.

Yes, but that link also states:

This shouldn't have any visible changes in most code as TS considers most strict mode errors as errors at compile time, but it means that some things which used to silently fail at runtime in your TS code, like assigning to NaN, will now loudly fail

In this case, it's silently breaking code.  I'm not saying this code should work, I'm just saying it should be an error or warning.

The compiler can not statically catch all the strict mode issues. there are ones that are dynamic.
the issue you listed of inner function declarations should be caught by the compiler, so this is a bug.
for the breaking change in 1.8, consider using --noImplicitUseStrict to revert back to pre-1.8 behaviour

@mhegazy , this is not only a breaking change, but one that happens at runtime, which is totally unexpected and hard to debug, since it's an error generated by transpiled code, hidden behind source maps.
The legacy app that I'm working on also had problems after upgrading to 1.8, because of global variables:
import 'something';

declare var someLegacyGlobal;

someLegacyGlobal = true; // <-- fails at runtime, no error at compile time

@use-strict, to be fair, in this instance you told the compiler via your declare statement that someLegacyGlobal was already declared.  It could not, therefore, have warned you about it.

@errorx666 , I see your point, but the problem is still that the compiler allows something which would result in a runtime error. The "noImplicitUseStrict" compiler flag is an OK workaround, it's just very hidden. For instance, using "Symbol" in ES5 target results in a compiler error and I would expect this case to be no different.
Because the issue itself is in conflict with the ES6 spec, I see no way to fix it other than make it an explicit compile time error and disallow the "declare" keyword inside ES6 modules. However, this would make globals unusable and require yet another compiler flag for "compatibility" that would relax this check (e.g. nonStrictModules).
I would have expected to be able to use window.someGlobal as a workaround, but typescript currently complains there is no someGlobal variable in the Window interface. Not sure about this behavior.

I would have expected to be able to use window.someGlobal as a workaround, but typescript currently complains there is no someGlobal variable in the Window interface. Not sure about this behavior.

Two options:

window[ 'someGlobal' ]
2:

declare global {
  interface Window {
    someGlobal: any;
  }
}

Would not be type-safe. Also looks like Window no longer has an index signature.
That's the augmentation feature from 1.8 I assume. I didn't know it was possible to do it with global scope as well. Seems to work OK. I assume this affects the whole project not just the current file.

As noted earlier, --noImplicitUseStrict should revert to the pre-1.8 behavior and not inject a "use strict" in your modules.

(from #7339)
This is an issue, even when you don't have modularlized code.  For example, even when you provide the "use strict"; prolog, TypeScript does not properly realise scoping of the function:
'use strict';

if (true) {
  function bar() {
    console.log('bar');
  }
}

bar();

In ES5 strict mode, it disallows block scoped functions.  In ES6 strict mode, it allows block scoped function, but in both cases, bar() is not available and TypeScript should disallow this.  Essentially it always considers that function scoping is non-strict mode.

(and also to note, the --noImplicitUseStrict is not a valid workaround for the above, because it is explicit strict usage)

Thank you @sheetalkamat for fixing this! 👍


Pisces主题一级标题后无空白行 || In theme Pisces, there is no blank line after headline 1.

I agree and want to create new issue

[x] Yes, I was on Hexo Docs page, especially on Templates, Variables, Helpers and Troubleshooting.
[x] Yes, I was on NexT Documentation Site.
[x] And yes, I already searched for current issues and this is not help to me.

Expected behavior
一级标题之后有空行
there is  blank line after h1
Actual behavior
一级标题之后没有空行，其他级别标题后有空行
there is no  blank line after h1
Steps to reproduce the behavior

N/A
N/A
N/A

Link to demo site with this issue: https://pobenliu.github.io/2018/03/19/how-to-read/
Link(s) to source code or any usefull link(s): N/A

Hexo Information

hexo: 3.6.0
hexo-cli: 1.1.0
os: Darwin 16.7.0 darwin x64
http_parser: 2.7.0
node: 9.8.0
v8: 6.2.414.46-node.21
uv: 1.19.2
zlib: 1.2.11
ares: 1.13.0
modules: 59
nghttp2: 1.29.0
napi: 2
openssl: 1.0.2n
icu: 60.2
unicode: 10.0
cldr: 32.0.1
tz: 2017c

NexT Information
NexT Version:

[x] Latest Master branch.
[] Latest Release version.
[] Old version -

NexT Scheme:

[] All schemes
[] Muse
[] Mist
[x] Pisces
[] Gemini

Other Information
Browser: chrome
Syetem: macOS
Screenshots:

似乎找到问题的原因了。
路径./next/source/css/_common/scaffolding/base.styl
文件中第28行有如下代码h2, h3, h4, h5, h6 { margin: 20px 0 15px; }
将其修改为h1, h2, h3, h4, h5, h6 { margin: 20px 0 15px; }即可，源文件中没有给h1加margin。
修改后的示意图：

NexT is rebased into organization repo.
If you want new feature, fix, or support, create new in NexT v6.x repo (desirable in English).
There is instructions on English or Chinese how to update from v5.1.x to v 6.x
You also may read this for details.


fix(client.scalar.com): localStorage isn’t enabled

Alright this works... is it the best? Probably not.
@hanspagel its just a static site right? So there's no process.env. I just hard coded it into a script tag 🤷🏾

@hanspagel we can make it a different config later, its fine for now being a define : )


Updates CRS, fixes some tests

Hello,
I updated the CRS and did some work to make the handler of the CRS unit test more similar to httpbin. Overall:

CRS and ignored rules (because of rules renumbering) are updated to the latest CRS commit
added base64 feature in the handler
added urldecoding feature in the handler for x-www-form-urlencoded requests. 954120-1 and 954120-2 are the only two requests that results to be sent as x-www-form-urlencoded and were failing. Currently it is the only workaround I found.
some more tests are passing
two tests are failing and require attention:

920250-1 and related ones: didn't debug much but seems that @validateUtf8Encoding evaluation is not called. These are the only tests that are using this operator
941180-7: false positive, this rule should not be triggered because REQUEST_FILENAME should have been removed by ctl:ruleRemoveTargetById. It is a new "logic" used by CRS for performance reasons: if the element seems "safe enough", some further rules are not evaluated on that element. Some more info: https://github.com/coreruleset/coreruleset/commit/2422173a3ca76f7d074febb86d546436b7a7b7d4

I also tried yet another approach about the handler trying to rely on go-httpbin as a library instead of mimicking it.
This would be the draft implementation: same tests are passing. Still, I found that the execution is more error-prone (mainly timeouts) and networking complexity is increased.
Taking into account the overall limited CRS usage of httpbin endpoints used, I feel that it still doesn't worth it.
Open to any feedback! 🙏

Codecov Report
Base: 78.08% // Head: 78.34% // Increases project coverage by +0.26% :tada:

Coverage data is based on head (a1edbc9) compared to base (80c5f90).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##           v3/dev     #526      +/-   ##
==========================================
+ Coverage   78.08%   78.34%   +0.26%
==========================================
  Files         145      145
  Lines        6876     6876
==========================================
+ Hits         5369     5387      +18
+ Misses       1270     1257      -13
+ Partials      237      232       -5

Flag
Coverage Δ

default
73.48% <ø> (+0.13%)
:arrow_up:

examples
27.19% <ø> (ø)

ftw
56.33% <ø> (+0.89%)
:arrow_up:

tinygo
71.53% <ø> (+0.23%)
:arrow_up:

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

Impacted Files
Coverage Δ

internal/corazawaf/transaction.go
79.94% <0.00%> (+0.75%)
:arrow_up:

actions/ctl.go
72.50% <0.00%> (+1.00%)
:arrow_up:

bodyprocessors/multipart.go
82.27% <0.00%> (+3.79%)
:arrow_up:

transformations/base64decode.go
95.06% <0.00%> (+4.93%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

For the 920250-1, in the CRS tests we set up the docker container with the specific config of validating utf8 encoding now, so this might be pointing to a real bug?
941180-7 also might be related to a bug.

920250-1: it was trivial, back in the day, it has the rule has been disabled because of too much noise (when the utf8 check was broken).
941180-7: If my debug flow is right, it was indeed a bug when parsing ctl. The problem rises when the target has just the collection name without any key nor a trailing : at the end (e.g. ctl:ruleRemoveTargetById=2;REQUEST_FILENAME:" is working fine). I ended up populating the collection name and not the key, which may actually be empty

If it is okay I would merge this one and address in another PR Rag's suggested clean up and some ctl unit tests refactoring

Go for it
On Wed, 14 Dec 2022, 16:46 Matteo Pace, @.***> wrote:

If it is okay I would merge this one and address in another PR Rag's
suggested clean up and some ctl unit tests refactoring
—
Reply to this email directly, view it on GitHub
https://github.com/corazawaf/coraza/pull/526#issuecomment-1351667545,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXOYARVULUHUX3AU3EIQGTWNHTWHANCNFSM6AAAAAASXOMOJI
.
You are receiving this because your review was requested.Message ID:
@.***>

It would be cool to upgrade coraza-proxy-wasm and decrease the ignored tests there.


Add usage example to README

Just a quick example/documentation of how the querystring should be formatted and what options are available.

Solved in Pull Request #3


User input in custom action script

Rasa Core version:
0.9.0a3
Python version:
3.6.5
Operating system (windows, osx, ...):
Windows 10
Issue:
I am finding difficulties in making a custom action where I can break free from normal user-rasa processing-bot flow and can ask for certain inputs directly.
I mean I don't want another intent to ask a follow up question (for example, Booking time), I should be able to directly input it in my custom action script.
There is formfilling option but it would be great if I can take over the flow and communicate with the user directly.
For example, I want a password or any information of that kind, I need to use
getpass.getpass()

for getting it but there is no way to do that using formfilling
Please help me out if I am missing something.
If it can't be done, I'll be happy to work on this enhancement!
Content of domain file (if used & relevant):

I am looking for something similar. In my use case, I have an additional REST service (let's call it SymptomsService) which provides questions to be asked from the user. It is sort of like a service within service (SymptomsService within Rasa service).
Explanation: Using Rasa's conversation flow, when I reach a certain point in conversation, I want a method to take over and handle the Q&A with the user, something like:
* greet
- utter_greet
- utter_how_may_i_help
* i_am_sick
- action_get_symptoms

This action_get_symptoms action is dependent on the additional REST service, SymptomsService. I cannot implement it directly in Rasa conversation flow since it is already built, a flow of multiple questions has been formed, and only needs to be presented to the user, and their answers need to be returned to the SymptomsService so that a symptoms statement may be generated.
The reason I cannot use Rasa's FormAction class is that this Q&A flow is context dependent itself. Question 2 coming from SymptomsServer might be different according to the input provided as answer of Question 1, and FormAction's required fields don't work in that way.
As an example, the flow of this Q&A with SymptomsService might go like:
Q1: Enter any symptoms that you are feeling:
Answer: Flu and fever
Q2: Have you tried any medication?
Answer: Yes (or No)
Q3: (If previous answer was Yes) Enter the medicine name.
(If previous answer was No, service will skip this question and move on to Q4).
Q4: ...

This is essentially like filling out a form, and has been converted to conversational format in my separate SymptomsService, I just need to integrate it within Rasa flow.
I would appreciate any feedback on this.

Exactly! This would be a much needed enhancement. Sometimes, it is required to communicate with the user beyond the RASA flow and currently there is no way to do that.

This is a very good point and a good enhancement. I'll think about it as well :+1:

couldn't you do this by sending a ConversationPaused and then ConversationResumed event?

@amn41 Could you please elaborate this, perhaps using a simple example? The functionality of these events isn't quite clear to me looking at the docs only.

sure - if your action returns a ConversationPaused event, Rasa Core will not take any action when it receives new messages, until a ConversationResumed event is triggered. This allows you to take over the handling elsewhere without Rasa Core interfering

@amn41 a basic example for this would help. Like for confirming a name using fuzzy logic in the middle of an action or getting a simple yes/no from the user.

there is a new version of the forms coming in v0.12 which should do what you need. that will come with examples too :)

Hey, @amn41  Is the issue solved in the new version?

yep! check out this post https://blog.rasa.com/building-contextual-assistants-with-rasa-formaction/

@amn41 can this issue be closed then?

yes


Update OWNERS file

Add coreydaley as an approver

/assign @adambkaplan

/label docs-approved
no doc

/label qe-approved
no qe

/label px-approved
no px

/cherry-pick release-4.8


Add public methods to TypeScript definitions

We're using the library in a TypeScript project but some of the Notification methods we're using (length, compile) aren't defined forcing us to call them with Notification as any.

Coverage remained the same at 100.0% when pulling e5976ec84b22c0f5af9b10f546daae7448778967 on spasiu:patch-1 into 38a357ed0c153aad09c2857e48a710527e685bfc on node-apn:master.


Update PPP for Cheryl Toh

resolves #106

Pull Request Test Coverage Report for Build 193

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 90.973%

Totals

Change from base Build 188:
0.0%

Covered Lines:
2076

Relevant Lines:
2282

💛  - Coveralls


Docs: C++ program linking stdlib statically

I have a project that creates multiple cdylib using corrosion, but those resulting cdylibs are imported using dlopen instead of using CMake's target_link_libraries. As a result, when trying to dlopen the cdylib, the libstd-d5189b81a4fa4d36.so cannot be found. Because of having multiple cdylibs, statically linking stdlib to every cdylib is not preferable.
I couldn't find out what arguments to pass to the cmake executable definition to get it to statically link to the rust stdlib. Is there anything that corrosion provides at this time to facilitate this?
Thanks.

but those resulting cdylibs are imported using dlopen instead of using CMake's target_link_libraries.

Are you sure that dlopening is the real cause for this issue?

As a result, when trying to dlopen the cdylib, the libstd-d5189b81a4fa4d36.so cannot be found.

Are you running the executable on the same machine that built the project? Does the libstd.so get found if you manually export the environment variable LD_LIBRARY_PATH=$HOME/.rustup/toolchains/<your_toolchain>/lib before running your executable? (please note that you need to replace <your_toolchain> with the full name of the Rust toolchain you built your project with - just look into the folder to see what options are there).
If that works, then you'll probably want to set the rpath by passing -rpath=something to the linker - and if you are shipping your project to other machines, you would also need to package the stdlib .so.

Because of having multiple cdylibs, statically linking stdlib to every cdylib is not preferable.

I couldn't find out what arguments to pass to the cmake executable definition to get it to statically link to the rust stdlib. Is there anything that corrosion provides at this time to facilitate this?

These two statements conflict a bit, so I'm not quite sure what you want. I believe the standard behavior is to statically link the rust stdlib - you may want to check if you are somewhere setting the -C prefer-dynamic rustflag.

Ack, I'm living in la la land again. My imagined approach is not going to work.
What I want to achieve is having smaller library output (hence the dynamically linking to stdlib) but I don't control the environments of where these libraries will be deployed. As the rust toolchain is not installed there, I'll have to supply libstd myself. Yet another problem is that some of the libraries are compiled with different versions of rust. So if I wanted to go down the dynamic linking path, I would have to supply a copy of libstd for each library.
Maybe static linking is the easiest after all.
Sorry for taking up your valuable time.


Are multiple scripts needed in chapter 2?

Hi @aitoehigie was going through the second chapter and noticed multiple single lined scripts. What is the rationale behind this? Can you not have a single script with all the lines?

hello @sizhky
I split the scripts into one-liners because they are commands that are to
be run at different points and in the book, some explanations come in
between them so I couldn't lump them into one single script.
Your thoughts?
On Sun, Apr 25, 2021 at 6:57 AM sizhky @.***> wrote:

Hi @aitoehigie https://github.com/aitoehigie was going through the
second chapter and noticed multiple single lined scripts. What is the
rationale behind this? Can you not have a single script with all the lines?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PacktPublishing/FastAPI-By-Example/issues/1, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABPDWCI5S54YRH2QO7YEIDTKOVNTANCNFSM43Q5JB6Q
.


Fixed back to top via JSInterop

The back to top did appear to be broken. It was earlier dependent on header #anchor. the changes proposed let it trigger independently and that too via JS

@JSM33T I'm closing this as i didn't hear back from you and as part of house keeping.. Feel free to open back once you solve the above issues.

@JSM33T I'm closing this as i didn't hear back from you and as part of house keeping.. Feel free to open back once you solve the above issues.

This pr had the fixed one. I will send an updated PR later!! Thankyou


Add KernelCI favicon

Closed because already exists on https://dashboard.kernelci.org/tree

@MarceloRobert sorry but where do you see the favicon?


Add cancel button to video uploader and also cancel upload when leaving route

This adds an abortController to the uploader, which can be called by pressing a cancel button when a video is being uploaded. It is also called when a user leaves or reloads the upload route. The controller then aborts the current XMLHTTPRequest and rejects the promise it was being called from.
Closes: #354

Some notes:

this is still missing a way to cleanup the state on the opencast side
when an upload is cancelled, this will throw an error by rejecting the respective promise. It then informs the user that their upload was cancelled, similar to the other error messages that can occur for uploads, but without the added "Failed to upload video" message (see second screenshot)
maybe there should be an additional button which links back to or rather rerenders the select files dialog, or perhaps it should even automatically do that. Right now, you would need to take a detour via the homepage to upload another file, as clicking the upload video option from the user dropdown menu just shows the same error message
it might be a little confusing or irritating for the user to still see the video title and description input fields below the error message (I believe this and the previous point should also apply for other error messages)

I agree with Julian that you probably want to just add another upload state. Then it should be possible to do sth like this: controller.signal.addEventListener("abort", () => setState("cancelled"))

Yep, I agree with your points on the button position.
The other bug you found is interesting. Maybe we just want an "upload more stuff" button once an upload is done. But yeah, maybe the menu should also reset the state. :shrug: Certainly doesn't need to be done in this PR, bu then we should create a separate issue.


[Bug] (Docs): Missing terminology in docs for https://remix.run/docs/en/v1/guides/accessibility#links

Which Remix packages are impacted?

[X] remix (Remix core)
[ ] create-remix
[ ] @remix-run/architect
[ ] @remix-run/cloudflare-workers
[ ] @remix-run/dev
[ ] @remix-run/express
[ ] @remix-run/netlify
[ ] @remix-run/node
[ ] @remix-run/react
[ ] @remix-run/serve
[ ] @remix-run/server-runtime
[ ] @remix-run/vercel

What version of Remix are you using?
n/a
Steps to Reproduce
The second paragraph in this links section on https://remix.run/docs/en/v1/guides/accessibility#links appears to be missing a term.
"Remix also provides the which behaves the same as <Link>".
I think there is supposed to be a word after "the" in that sentence, but I'm not sure what that term is. I'm guessing it is whatever the other component that Remix provides that behaves the same as  <Link>.
Expected Behavior
Docs include the name of additional component provided by Remix referenced in the accessibility links section.
Actual Behavior
Docs do not include the name of additional component provided by Remix referenced in the accessibility links section.

It was NavLink. Documentation is now fixed.


Overriding Endpoints in ServiceManifest.xml

when passing ports in application manifest, change will not be reflected in service manifest ,however the port that will be used will be the one from application manifest ?
could you please confirm ?
application manifest
<Parameter Name="Port" DefaultValue="10023" / / this one will be used ?
service manifest

[Enter feedback here]

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

ID: 06afb00a-c2b7-964a-60cd-0c94df9e912b
Version Independent ID: 8450fbfe-fcec-363e-fad0-0e66acfc579d
Content: Describing Azure Service Fabric apps and services - Azure Service Fabric
Content Source: articles/service-fabric/service-fabric-application-and-service-manifests.md
Service: service-fabric
GitHub Login: @erikadoyle
Microsoft Alias: edoyle

@Hannahayek, Thanks for the question! We are taking a look into this and will get back to you soon.

@Hannahayek Apologies for all the delay in response and all the inconvenience caused because of the issue.
When an endpoint resource is defined in the service manifest, Service Fabric assigns ports from the reserved application port range when a port isn't specified explicitly.
Port 14023 has been given in example for  TCP endpoint traffic in Service Manifest example as mentioned here and Port 14023 is mainly used for TCP/UDP communication
To override Service Endpoints in Service Manifest.xml file you can either explicitly define ports as per your requirement otherwise it will take one mentioned in Application Manifest.xml file.
You can indeed provide the port number as mentioned by you in the query or any other port as per your requirement.
Hope the provided information is helpful.
We will now close this issue. If there are further questions regarding this, please tag me in a comment. I will reopen it and we will continue the discussion.


Please remove android:label attribute from AndroidManifest

Thanks for you awesome work.
Can you remove the attribute android:label from your AndroidManifest, in fact you don't need one. When you specify this attribute it's hard to apps change their android:label in build time, as you can note in the error above:
Error:Execution failed for task ':app:processProdDebugManifest'.

Manifest merger failed : Attribute application@label value=(YVD) from AndroidManifest.xml:21:9-44
is also present at [com.github.jlmd:AnimatedCircleLoadingView:1.1.5] AndroidManifest.xml:13:9-41 value=(@string/app_name).
Suggestion: add 'tools:replace="android:label"' to  element at AndroidManifest.xml:18:5-143:19 to override.

There is not android:label attribute in the AndroidManifest from the library. Maybe you are using the sample app as dependency instead of the library.


Assigned myself but next step hasn't displayed

I completed the first step but the next assignment hasn't displayed. I waited and refreshed page but no new assignment has been given.

Issue resolved.


Use Throw helpers for throwing exceptions

use the ArgumentNullException.ThrowIfNull throw helper where possible to improve performance. Available with .NET 6
see: https://devblogs.microsoft.com/dotnet/performance_improvements_in_net_7/#exceptions
source: https://github.com/dotnet/runtime/blob/main/src/libraries/System.Private.CoreLib/src/System/ArgumentNullException.cs#L67-L69
https://andrewlock.net/exploring-dotnet-6-part-11-callerargumentexpression-and-throw-helpers/#throw-helpers-in-c-
https://github.com/search?q=repo%3AHavunen%2FSystemTextJsonPatch throw new Argument&type=code

Hi, thanks for sharing. We shiuld implement this IMO

This is now implemented in master branch and will be available in the next release. The change improved overall performance about 5%.


PLACEHOLDER - News from GH Universe

URL
https://github.blog/2022-11-09-everything-new-from-github-universe-2022/#invest-in-community
When was this post released
09 Nov 2022
Summary
GitHub Universe happened this week. During the keynote several new features were announced:

CoPilot for Business
Hey Github
GitHub Fund
GitHub Accelerator
GIthub Actions Importer

Code of Conduct

[ ] I would like my name mentioned on the podcast
[X] I agree to follow this project's Code of Conduct

I think for the show we can just focus on the "Hey Github". Would make for a great accessibility topic.


move test fixtures into conftest.py from fixtures.py

as described in this thread (https://github.com/astral-sh/ruff/issues/4046), ruff (and other linters) complain about imports that are made in order to provide fixtures to a pytest test. for example, we do things like:
from fixtures import _039a6_dotnetfile_extractor

def test_dotnet(_039a6_dotnetfile_extractor):
    ...

ruff complains with:
tests/test_main.py:xxx:yy: F811 Redefinition of unused `_039a6_dotnetfile_extractor` from line xxx

as suggested in the linked issue, we should move our fixtures into a file named conftest.py, which pytest will use to autodiscovery fixture names.
after we make this change, we should also update any relevant linter configuration that ignores "F401" or "F811".

@williballenthin So basically, what I understood from your explanation is that we need to copy the function with the decorator<EMAIL_ADDRESS>from the "fixtures.py" file and move it to the "conftest.py" file. After that, we have to modify the linter's configuration file to ignore the "F401" or "F811" errors.
If I am right can I try to fix this ?

not quite: please rename the file "fixtures.py" to "conftest.py". then it should be possible to remove many of the imports that currently reference "fixtures".
i'm not exactly sure precisely the changes to be name, but the above is approximately correct. i'm happy to lend a hand if you hit and problems. please feel welcome to take this issue!
however, you may want to wait until Monday when I can finish and merge the PR that improves the code linters. these will make it more obvious when this issue is fixed.

Oh yes just let me know when you make a successful PR so I can start working!!
Will discuss with you more about the changes after monday

ok, so #1591 has been merged, which updates our linter config to be easier to use and more thorough. notably, it includes checks that help demonstrate the issue here. that will make it easier to fix.
the goal is to remove these lines:
https://github.com/mandiant/capa/blob/506d677684692e4bc195b5806d2ff0bbc0dcb020/.github/ruff.toml#L46-L61
so i'd suggest removing those lines and then invoking the linter and trying to get it to pass. you can invoke the ruff linter like this:
$ pre-commit run --all-files ruff

more background on the issue:
we use pytest to invoke our tests. some of our tests use "fixtures", that is, data loaded from the system to help enable a test to work. for example, a PE file that we analyze and assert matches a rule.
a pytest test case looks like this:
def test_foo():
    assert 1 == 1

when there's a fixture, its passed as an argument to the test function:
def test_foo_with_fixture(foo):
    assert foo == 1

pytest inspects the name of the argument and tries to find a variable in the current scope with that same name and uses this as the fixture.
today we keep our fixtures in the file fixtures.py and import them into the test files like from fixtures import foo which brings the symbol into scope. however, because the thing "foo" isn't directly referenced by the code (only indirectly via the function argument name), linters raise warnings. this is what we want to fix.
from the above linked github issues, pytest recommends putting test fixtures into a file named conftest.py and it will auto-load fixtures from here. then we don't have to have the explicit imports from fixtures.py and the linters won't complain. this is your job @AmanSal1 :-)
in the past, we liked the explicit imports because it makes more clear where the test data is coming from. but today, i think pytest is well known enough that we can expect devs to understand the use of conftest.py.
make sense? any questions?
im happy to help if you need a hand - just let me know.

based on the following article i think we can keep the fixtures.py file and add a conftest.py file with the contents from fixtures import *. i like the balance of this solution in that the fixtures are clearly in the fixtures.py file and magic is minimized.
https://www.revsys.com/tidbits/pytest-fixtures-are-magic/


Ensure we always use modified lines

Codecov Report

Merging #648 (576e5ca) into main (287e264) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #648      +/-   ##
==========================================
- Coverage   93.55%   93.55%   -0.01%
==========================================
  Files          85       85
  Lines        7992     7991       -1
==========================================
- Hits         7477     7476       -1
  Misses        394      394
  Partials      121      121

Impacted Files
Coverage Δ

internal/reporter/bitbucket.go
92.50% <100.00%> (-0.05%)
:arrow_down:


Add support to pass ssl certificate as a string, not a file path

Supersedes #73, applied review comments.
This is required to use this driver in https://github.com/starburstdata/grafana-trino/ and get that plugin certified by Grafana. Grafana now does not allow plugins to make any file system calls, so we need a way to pass a certificate as a value, instead of saving it to a temp file.

@losipiuk PTAL


Not Getting profit, just inverse..

Hello friend. I am not getting profit.
I am buying and sell on wrong .
My market is ADA/EUR
Actually i Am selling when Low ADA and buy ADA when High.
Should be suppose SELL ADA when High And buy ADA when low?!
{
    "binance": {
        "api_url" : "https://api.binance.com",
        "api_key_file" : "binance.key",
        "config": {
            "base_currency": "ADA",
            "quote_currency": "EUR",
            "live": 1,
            "sellatloss": 1,
            "selllowerpcnt": -1,
            "sellatresistance": 1,
            "disablebullonly": 1,
            "disablebuymacd": 0,
            "disablebuyobv": 0,
            "disablebuyema": 1,
            "disablebuynearhigh": 1,
            "disablebuyelderray": 1,
            "disableprofitbankreversal": 0,
            "disablefailsafefibonaccilow": 0,
            "disabletelegram": 0,
            "autorestart": 1,
            "verbose": 1,
            "graphs": 1
        }
}

Can give me some help?
Thank you

This is what I run for ADA on Coinbase with USD.  I lose 2% at the most but typically profit.
{
"coinbasepro": {
"api_url": "https://api.pro.coinbase.com",
"config": {
"base_currency": "ADA",
"quote_currency": "USD",
"live": 1,
"verbose": 1,
"sellatloss": 1,
"selllowerpcnt": -2,
"disablebullonly": 1,
"disablebuyobv": 1,
"disablebuyelderray": 1,
"autorestart": 1
},
"api_key_file": "coinbasepro.key"
},
"telegram": {
"token": "
"client_id":
}
}

@hellresistor Is this fixed? Can we close it?

Actually, we agreed that support questions and questions on how to configure the software are best addressed in the
telegram channel
Closing as this is not an issue with the software.


Fix issue #58 and issue #42: crashes on Windows 11 24H2 and wrong results on processes with spaces in name

Solves issue #58 : On windows systems ps-tree uses WMIC to get the list of child processes. WMIC is deprecated since 2016. This PR replaces WMIC with powershell (as recommended by Microsoft),
I encountered this problem, when using the start-server-and-run package (see https://github.com/bahmutov/start-server-and-test/issues/384).
Solves issue #42 - ps-treee returns wrong results when process name contains spaces (primarily on windows systems). Issue is caused by splitting a row into columns on spaces. Fixed by getting the column widths, when extracting the headers (all columns have fixed width).

Running build/tests on GitHub CI just so progress can be made on this PR ... 🤞

Any chance this can get looked at? I'm starting to see failures on new regular Windows installs from ps-tree, so it's going to start affecting significantly more people soon.

@nelsonic

Could you please add a windows-specific test that will run on GitHub Actions CI to confirm this us working and avoid future regressions when something changes.

https://github.com/indexzero/ps-tree/pull/62/checks already shows successful tests on Windows
The branch BePo65:pu/issue58 needs to be rebased on the master branch, or the master branch needs to be merged into the branch for this PR.
Before any more work is done on this repo there needs to be confirmation from the owner @indexzero that there will be a possibility to release a new version, otherwise there is no real benefit from any fixes.
I checked that this branch runs successfully on windows-2025. That should be added to the list of tested operating systems in https://github.com/indexzero/ps-tree/blob/master/.github/workflows/ci-test.yml


Figure out how to handle strings stored in DB

For the moment, I've dumped the existing English strings into repo files, but while this makes it easy to import them into Weblate, we don't yet have a strategy for getting them back out and to a state where we can use them. We do need things things to be primarily in the DB, rather than in the code, so that non-dev admins can easily add or edit them.

Going to go with Laravel-Translatable along with some actions to read in/dump out strings for models.

Done and deployed! See #43 for the majority of the changes.


Update to group component to properly handle zone changes in tracked devices

Description:
If you have a group that has device trackers that move between zones, sometimes the group incorrectly reports it as home, when all devices are away in other zones.
Related issue (if applicable): fixes #
#1588 and #2157
Checklist:
If the code does not interact with devices:

[ x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ x] Tests have been added to verify that the new code works.

Change looks good. Can be merged when linting passes 🐬

Looks good! :dolphin:


document module types

Per #4076, we kinda need a way to document everything exported from the $app/* modules. This WIP gets us part of the way there, but it means /docs/types and /docs/modules duplicate each other, so we need to figure out the right way to handle that.

alright, i think this is ready

merging so we can make progress on #4021 etc


Don't log on default log level when an error occurs in optimistic locking

This PR stops the pollution of cert-manager-controller logs from multiple controllers attempting to update the same object version. Instead, we log to debug, however we still re-queue the object so has not functional change to any controller.
E0323 18:09:56.487221       1 controller.go:158] cert-manager/controller/CertificateReadiness "msg"="re-queuing item due to error processing" "error"="Operation cannot be fulfilled on certificates.cert-manager.io \"osm-ca\": the object has been modified; please apply your changes to the latest version and try again" "key"="default/osm-ca"

/assign @irbekrm @jakexks @maelvls @wallrj
/milestone v1.3
/cleanup
fixes #3667
NONE

/kind cleanup

If this is our own behavior causing this, it'd make sense to try and address the root cause of the problem rather than just hiding the messages IMO

Adding a hold to this as not everyone agreed with the approach, but feel free to unhold.
/hold
/lgtm

To me it seems like at the moment this is the better easy fix.

Personally I'm not sure I agree, but I'll leave it up to the group :) I think this does hide transient errors somewhat, however in cases where users are running into genuine failures, this will make it a lot harder to actually dig in/work out what's going wrong.
I'd strongly advocate for actually fixing the underlying issue - if we are going to merge this change, can we make it a priority to come up with a strategy that avoids these messages in the first place?
Given that each one of these controllers actually only manages a very specific subset of the Certificate status, some careful programming and a switch to either managed fields based server-side apply (or perhaps more simply right now, a patch operation) would help a lot.
As an alternative, instead of hiding these messages, could we add some additional text to the error when it is logged for the Certificate controllers cases? e.g. "Failed to update status, this is likely to not be an error and will be corrected on the next sync iteration: Operation could not be fulfilled..." (potentially with a link to the GitHub issue tracking these errors too, so the user has more information they can follow along with?)

Given that each one of these controllers actually only manages a very specific subset of the Certificate status, some careful programming and a switch to either managed fields based server-side apply (or perhaps more simply right now, a patch operation) would help a lot.

At the moment there are conditions that are managed by multiple controllers (i.e Issuing condition would be applied by certificate-trigger controller, removed by certificate-issuing controller)
We could prioritize re-designing it so that each condition is only managed by a single controller and that would allow patching the status updates?

Perhaps leaving these two specific cases to continue using an Update operation, and have the others use a Patch, would improve things? :)

I wonder if it'd be confusing if some were patched some not, but I guess we can live with that?
I will create an issue for this.

I am also in favor of not hiding these messages under the -v=4 level since the optimistic locking retry messages are useful to understand why a logline appears twice in a row; it is also useful to us to spot unexpected data races.
-E0326 controller.go:158] "msg"="re-queuing item due to error processing" "error"="Operation cannot be fulfilled on certificates.cert-manager.io \"test5\": the object has been modified; please apply your changes to the latest version and try again" "key"="default/test5"
+I0326 controller.go:158] "msg"="re-queuing item due to optimistic locking" "key"="default/test5"

A switch to either managed fields based server-side apply would help a lot.

Good point, I never thought about managed fields + server-side apply. I guess we can investigate this idea.
Regarding the PATCH idea, I tried to think of any unintended effects but could not find one; I guess we could go with this.

The v1.3 is now frozen; if that's OK, I will postpone this issue to v1.4 (planned for Friday 11 June 2021).
/milestone v1.4

/close

/close

Huh? Why?

/close cancel
As discussed, I am changing this PR to instead Info log that the resource will be re-queued due to optimistic locking, rather than suppressing the log at debug.

/hold cancel


AltheaC_Win_Blog_20_05_25

Docs Build status updates of commit bc07125:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

documentation/blog/MS-WININTBLOGLP/1f17c33c-5abf-464f-80c4-cd97c5434a4a.md
:white_check_mark:Succeeded

documentation/blog/MS-WININTBLOGLP/6183f838-c21e-4c44-a5bb-1a84e1864c2e.md
:white_check_mark:Succeeded

documentation/blog/MS-WININTBLOGLP/MS-WININTBLOGLP_files/image008.png
:white_check_mark:Succeeded

documentation/blog/MS-WININTBLOGLP/MS-WININTBLOGLP_files/image010.png
:white_check_mark:Succeeded

documentation/blog/MS-WININTBLOGLP/MS-WININTBLOGLP_files/image012.png
:white_check_mark:Succeeded

documentation/blog/MS-WININTBLOGLP/toc.yml
:white_check_mark:Succeeded

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 in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Implement starknet.go support for write endpoints

Implement autogenerated starknet.go code for the following RPC endpoints

starknet_addInvokeTransaction
starknet_addDeployAccountTransaction

Additional Context:

https://github.com/NethermindEth/rpc-request-builder/issues/1
https://github.com/NethermindEth/rpc-request-builder/blob/main/src/api/rpcspec/methods.ts

@jelilat can I work on this issue?

Hey @jelilat, can i take this up?

hey @jelilat assign this to me

Hey @g4titan, i am on it. Will create a PR in a while.

okay


Container fails to start with the dynamic nfs provisioner

Describe the bug: A clear and concise description of what the bug is.
Any container fails to start, when using the nfs provisioner
Expected behaviour: A concise description of what you expected to happen
Container starts and nfs storage is provisioned
Steps to reproduce the bug:

mkdir -p /mnt/data
sudo chown -R admin:admin /mnt/data
kubectl apply -f https://gist.github.com/tampler/44baacbd7606561a11cf949fa462cedd
kubectl get pvc -> find the pvc created: ebs-rwx-pvc                           Bound    pvc-505263e6-25d8-4b86-9362-b95ea79fa7fc   10Gi       RWX            openebs-rwx        32m
kubectl get pv -> find the pv created: pvc-505263e6-25d8-4b86-9362-b95ea79fa7fc   10Gi       RWX            Delete           Bound    grant/ebs-rwx-pvc                           openebs-rwx
kubectl get po | grep busybox ---> Find the following: busybox-76cbb94474-bq4dr                    0/2     Init:0/1   0          24m
kubectl describe deploy busybox. Find the following output:

Mounts:
      /mnt/otp-shared-data from nfs-volume (rw)
  Volumes:
   nfs-volume:
    Type:       PersistentVolumeClaim (a reference to a PersistentVolumeClaim in the same namespace)
    ClaimName:  ebs-rwx-pvc
    ReadOnly:   false
Conditions:
  Type           Status  Reason
  ----           ------  ------
  Available      False   MinimumReplicasUnavailable
  Progressing    False   ProgressDeadlineExceeded
OldReplicaSets:  <none>
NewReplicaSet:   busybox-76cbb94474 (1/1 replicas created)
Events:
  Type    Reason             Age   From                   Message
  ----    ------             ----  ----                   -------
  Normal  ScalingReplicaSet  26m   deployment-controller  Scaled up replica set busybox-76cbb94474 to 1

kubectl describe rs busybox. Find the following log:

Environment:  <none>
    Mounts:
      /mnt/otp-shared-data from nfs-volume (rw)
  Volumes:
   nfs-volume:
    Type:       PersistentVolumeClaim (a reference to a PersistentVolumeClaim in the same namespace)
    ClaimName:  ebs-rwx-pvc
    ReadOnly:   false
Events:
  Type    Reason            Age   From                   Message
  ----    ------            ----  ----                   -------
  Normal  SuccessfulCreate  27m   replicaset-controller  Created pod: busybox-76cbb94474-bq4dr

So, as we see. Container is created and the nfs volume is provisioned. However, the container fails to boot
The output of the following commands will help us better understand what's going on:

kubectl get pods -n <openebs_namespace> --show-labels
kubectl logs <upgrade_job_pod> -n <openebs_namespace>

Anything else we need to know?:
Add any other context about the problem here.
Environment details:

OpenEBS version (use kubectl get po -n openebs --show-labels):

NAME                                                            READY   STATUS    RESTARTS   AGE   LABELS
nfs-pvc-505263e6-25d8-4b86-9362-b95ea79fa7fc-6b6f5df896-7spqf   0/1     Pending   0          48m   openebs.io/nfs-server=nfs-pvc-505263e6-25d8-4b86-9362-b95ea79fa7fc,pod-template-hash=6b6f5df896
openebs-localpv-provisioner-66b6b855fc-km4nb                    1/1     Running   4          18d   name=openebs-localpv-provisioner,openebs.io/component-name=openebs-localpv-provisioner,openebs.io/version=2.6.0,pod-template-hash=66b6b855fc
openebs-ndm-6xzkm                                               1/1     Running   8          18d   controller-revision-hash=7994455ddf,name=openebs-ndm,openebs.io/component-name=ndm,openebs.io/version=2.6.0,pod-template-generation=1
openebs-ndm-lv7kb                                               1/1     Running   0          17d   controller-revision-hash=7994455ddf,name=openebs-ndm,openebs.io/component-name=ndm,openebs.io/version=2.6.0,pod-template-generation=1
openebs-ndm-operator-6bd5949967-kx6wd                           1/1     Running   4          18d   name=openebs-ndm-operator,openebs.io/component-name=ndm-operator,openebs.io/version=2.6.0,pod-template-hash=6bd5949967
openebs-nfs-provisioner-7b4c9b87d9-2b9lb                        1/1     Running   0          24h   name=openebs-nfs-provisioner,openebs.io/component-name=openebs-nfs-provisioner,openebs.io/version=dev,pod-template-hash=7b4c9b87d9

Kubernetes version (use kubectl version):
Client Version: v1.20.4
Server Version: v1.20.4
Cloud provider or hardware configuration:
OS (e.g: cat /etc/os-release):
Ubuntu Server 20.04.2
kernel (e.g: uname -a):
Linux grant-02 5.4.0-66-generic
others:
kubectl describe deploy nfs-pvc -n openebs yields the following:

Pod Template:
  Labels:  openebs.io/nfs-server=nfs-pvc-505263e6-25d8-4b86-9362-b95ea79fa7fc
  Containers:
   nfs-server:
    Image:       itsthenetwork/nfs-server-alpine
    Ports:       2049/TCP, 111/TCP
    Host Ports:  0/TCP, 0/TCP
    Environment:
      SHARED_DIRECTORY:  /nfsshare
    Mounts:
      /nfsshare from exports-dir (rw)
  Volumes:
   exports-dir:
    Type:       PersistentVolumeClaim (a reference to a PersistentVolumeClaim in the same namespace)
    ClaimName:  nfs-pvc-505263e6-25d8-4b86-9362-b95ea79fa7fc
    ReadOnly:   false
Conditions:
  Type           Status  Reason
  ----           ------  ------
  Available      False   MinimumReplicasUnavailable
  Progressing    False   ProgressDeadlineExceeded
OldReplicaSets:  <none>
NewReplicaSet:   nfs-pvc-505263e6-25d8-4b86-9362-b95ea79fa7fc-6b6f5df896 (1/1 replicas created)
Events:
  Type    Reason             Age   From                   Message
  ----    ------             ----  ----                   -------
  Normal  ScalingReplicaSet  51m   deployment-controller  Scaled up replica set nfs-pvc-505263e6-25d8-4b86-9362-b95ea79fa7fc-6b6f5df896 to 1

I was able to detect the issues. This happens due to the lack of NFS pre-configuration itself. I need to create a /nfsshare folder and setup/validate a NFS service properly before apply.
This is a big issues with documentation, since no info is available with pre-requesties

@tampler Thanks for posting this issue as well as providing the various troubleshooting outputs.
What was the default storageClass in the cluster when you tried to create the NFS provisioner? If you run kubectl get sc, it should tell you which storage class is the default. If you had only one storage class at that time then it would necessarily be the default, also.

After poking for a week with this issue, I've switched to this project and was happily able to dynamically provision NFS for my services. Now having this:
NAME               PROVISIONER                                     RECLAIMPOLICY   VOLUMEBINDINGMODE      ALLOWVOLUMEEXPANSION   AGE
nfs-client         cluster.local/nfs-subdir-external-provisioner   Delete          Immediate              true                   4d

However, if you want me to debug this, I'll try with your project and supply a feedback. If anyone wants to do that and write some documentation


Adding a lecture schedule for instructor, rooms, and groups

@hamedshehab @mohaltheeb

Yah good job 👏🏻
But I wonder why you still working here...?!!  And where are our experience certificates...?!!
@mohaltheeb


WIP: fix tests for starship

Fixing the starship tests

Hey @Anmol1696,  fyi wasmd v0.42, v0.43 and v0.44 are retracted, so you should use either v0.41 or v0.45

Hey @Anmol1696, fyi wasmd v0.42, v0.43 and v0.44 are retracted, so you should use either v0.41 or v0.45

Shouldn't this information be announced or showcased clearly? By example, in the wasmd releases page?

Shouldn't this information be announced or showcased clearly? By example, in the wasmd releases page?

Yes, I will try to document it better. Thank you for the suggestion


Add compatibility for Reunion

Adding compatibility for Reunion mod https://steamcommunity.com/sharedfiles/filedetails/?id=1985186461
Just patching the random calls.
Was able to get through 3 reunion events in regular MP. Also attached validation below
Before
https://user-images.githubusercontent.com/33187058/105952341-119e8000-6037-11eb-971e-02eff955d1f5.mp4
After
https://user-images.githubusercontent.com/33187058/105952356-17946100-6037-11eb-9e19-20ad088e17e5.mp4

Looks fine, thanks.

Looks fine, thanks.


Split appmaps in navigator by (sub)project

Currently all appmaps are lumped into one single list in the sidebar; this can be especially inconvenient for monorepos and make it more difficult to find a specific map. They should be split by appmap.yml files found.

A user also suggested that they be split out by test folder. Wdyt?
…
On Mon, May 22, 2023 at 8:50 AM Rafał Rzepecki @.> wrote: — Reply to this email directly, view it on GitHub <#693>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAVC67R2MLIBXNEVVHZDJ3XHNOK7ANCNFSM6AAAAAAYKMZCXI . You are receiving this because you are subscribed to this thread.Message ID: @.>

Yes, I think it's a good idea to make the tests hierarchy reflect the filesystem hierarchy.


Protocol version in $_SERVER isn't parsed

$_SERVER['PROTOCOL_VERSION'] is not parsed in ServerRequestFactory and hence we don't set the protocolVersion property to the correct value.

Here you go @akrabat! #122 :)

Fixed with #122


Change context provider to only allow one child

We are having an issue writing Rhodium tests due to RoactGamepad.Focusable.Frame always being called Focusable and not the actual given name. For example, if you have two frames created like this:
return Roact.createElement("Frame", {
	Size = UDim2.new(1, 0, 1, 0),
}, {
	LeftFrame = Roact.createEelement(RoactGamepad.Focusable.Frame),

	RightFrame = Roact.createEelement(RoactGamepad.Focusable.Frame),
})

Both these frames will be called Focusable instead of LeftFrame and RightFrame which can make traversing the tree to rhodium test things a problem.
Focusable creates the frame in the following way:
return Roact.createElement(FocusContext.Provider, {
	value = self.focusNode,
}, {
	Focusable = Roact.createElement(innerComponent, innerProps)
})

I can not think of a way to fix this in the RoactGamepad library without Roact changes and it seems like it is important that components like this can be created without causing the naming of created components to be unexpected.
There are two different ways I see we could fix this in Roact, either switching to only allowing Providers to have oneChild or allowing a render function to be passed to Providers instead of children like is done with Consumers. In this PR I decided to present the oneChild solution since this seems like a smaller API change but I am open to either solution or other suggestions.
Checklist before submitting:

[ ] Added entry to CHANGELOG.md
[ ] Added/updated relevant tests
[x] Added/updated documentation

Coverage remained the same at 94.188% when pulling fd50d86ddd28628c7385be566e8cea5af9d3c13f on ConorGriffin37:changeProviderToAllowNamePreservation into 60197ac8789840956c364edcf464c460d14cd092 on Roblox:master.

This would be a breaking change. I don't think that we want to revert to only allowing one child for context providers, as this would impact many use-cases beyond the naming issues we're hitting in Roact Gamepad.
Many libraries, like Roact-Rodux, depend on this functionality and don't have naming issues as they're not higher-order components (HOC).
We have a couple solutions we should consider.
Looser Coupling
In the immediate term, can we stop depending on the exact names of more children? Tightly coupling an integration test to an exact instance tree makes those tests brittle.
Rhodium has XPath queries to help traverse chunks of the tree with more unknown variables. If those queries fall short, we should be able to expand them.
Expose self.key on components
We may be able to expose a component's key. If Roact Gamepad had access to a value like self.key, it could use that key as the key into the provider it creates.
React does not expose this information, but keys in React are not as meaningful as in Roact.
Change fragment key generation
When a component uses fragments, Roact currently uses the given keys as-is. We could instead generate new keys by concatenating the parent key with the key from the fragment container.
For this example, your components might end up with names like LeftFrame-Focusable instead of just Focusable.
If there is other code at Roblox that is tightly coupled to the names of constructed instances, we should make sure we won't break that code.

Changing this to use oneChild wouldn't stop people from passing a fragment as that one child and having multiple children but I guess this is a pretty big breaking change to make.
I think perhaps exposing self.key would be the best solution, this might also be useful for debugging purposes. I'm not familiar with the the XPath queries beyond name but fixing it like this would still mean that many objects in the tree would be named Focusable which impacts development in other ways such as making it harder to find an object in the tree with explorer search to inspect it's properties for example.


Rename s3 bucket

What's changing?
s3 bucket is no longer in use, but we need to change it to a fake name
How's this tested?
Related to: https://github.com/github/valet/issues/7306

This is a sample fake pipeline so I am unsure how they would be able to get our deployment info, but I changed it again. Please let me know if it needs additional tweaking.


Allow disabling "spellcheck" attribute

The spellcheck attribute on span elements is not supported by AMP.
It would be great to have an option to turn this attribute off for comments.
This block in prism-core.js is responsible:
if (env.type == 'comment') {
    env.attributes['spellcheck'] = 'true';
}

I would be happy to submit a pull request for this.

This can be done with a plugin. I'll let someone comment on which hook.

Thanks @LeaVerou. I solved it :smile:
prism.hooks.add('wrap', function(env) {
    if (env.type == 'comment') {
		delete env.attributes.spellcheck;
	}
});


Remove izip_longest import where unneeded.

It is not used by the CompetitionRanking or TieRanking leaderboards,
and causes issues with Python 3 when trying to use these.

@mikeszahaj I pushed 3.6.1 to PyPI with the fix. Thanks.

Awesome, thanks for getting this in so quickly.


Harvest Seal

The Harvest Seal doesn't seem to be working.
I have placed the Harvest Seal down, and my Golem goes around and attempts to farm to his hearts content, however he never really "harvest" the crops, and doesn't deposit them in a chest despite there being a store seal.
https://youtu.be/NpioEmxgm2U
This was attempted on a server with the following Modpack:
Renaissance

This is obviously a mod interaction of some kind. You might want to report this to the modpack owners.

try in the wild instead of inside a protected area, golem might be acting under a non-trusted fake player


[Feature][server] Add Kibana service

Tell us what feature you want?
elasticsearch's web operator need kibana-web

Can you contribute it? The community needs everyone to develop together。

ok,have a try


[BUG] Failed to launch FiftyOne app with GraphQL API Error

Describe the problem
Failed to launch FiftyOne app with version 1.0.1.
Code to reproduce issue
The issue can be reproduced by following the official FiftyOne example:

https://github.com/voxel51/fiftyone-examples/blob/master/examples/Video Labels.ipynb

Minimal code to reproduce (adapted from above notebook):
pip uninstall fiftyone fiftyone-brain fiftyone-db # remove existing fiftyone
pip install fiftyone==1.0.1

!pip install kaggle
!kaggle datasets download -d risangbaskoro/wlasl-processed
!mkdir wlasl-processed
!unzip wlasl-processed.zip -d wlasl-processed

main_path = './wlasl-processed/'
mp4_dir = main_path + "videos"

import fiftyone as fo

dataset = fo.Dataset.from_dir(dataset_dir=mp4_dir,dataset_type=fo.types.VideoDirectory)
session = fo.launch_app(dataset)

GraphQL API Error
localhost:38725: [Errno 111] Connection refused (configured timeouts: socketTimeoutMS: 20000.0ms, connectTimeoutMS: 20000.0ms), Timeout: 30s, Topology Description: <TopologyDescription id: 674ddd61f39e616c7b09ed31, topology_type: Unknown, servers: [<ServerDescription ('localhost', 38725) server_type: Unknown, rtt: None, error=AutoReconnect('localhost:38725: [Errno 111] Connection refused (configured timeouts: socketTimeoutMS: 20000.0ms, connectTimeoutMS: 20000.0ms)')>]>

File "/weka/home-meng/miniconda3/envs/py310/lib/python3.10/site-packages/graphql/execution/execute.py", line 1038, in await_result
2    return build_response(await result, errors)  # type: ignore
3  File "/weka/home-meng/miniconda3/envs/py310/lib/python3.10/site-packages/graphql/execution/execute.py", line 453, in get_results
4    await gather(*(results[field] for field in awaitable_fields)),
5  File "/weka/home-meng/miniconda3/envs/py310/lib/python3.10/site-packages/graphql/execution/execute.py", line 537, in await_result
6    self.handle_field_error(error, return_type)
7  File "/weka/home-meng/miniconda3/envs/py310/lib/python3.10/site-packages/graphql/execution/execute.py", line 571, in handle_field_error
8    raise error
9  File "/weka/home-meng/miniconda3/envs/py310/lib/python3.10/site-packages/graphql/execution/execute.py", line 530, in await_result
10    return_type, field_nodes, info, path, await result
11  File "/weka/home-meng/miniconda3/envs/py310/lib/python3.10/site-packages/strawberry/schema/schema_converter.py", line 750, in _async_resolver
12    return await await_maybe(
13  File "/weka/home-meng/miniconda3/envs/py310/lib/python3.10/site-packages/strawberry/utils/await_maybe.py", line 12, in await_maybe
14    return await value
15  File "/weka/home-meng/miniconda3/envs/py310/lib/python3.10/site-packages/fiftyone/server/query.py", line 435, in estimated_dataset_count
16    return await info.context.db.datasets.estimated_document_count()
17  File "/weka/home-meng/miniconda3/envs/py310/lib/python3.10/concurrent/futures/thread.py", line 58, in run
18    result = self.fn(*self.args, **self.kwargs)
19  File "/weka/home-meng/miniconda3/envs/py310/lib/python3.10/site-packages/pymongo/collection.py", line 1810, in estimated_document_count
20    return self._retryable_non_cursor_read(_cmd, None, operation=_Op.COUNT)
21  File "/weka/home-meng/miniconda3/envs/py310/lib/python3.10/site-packages/pymongo/collection.py", line 1910, in _retryable_non_cursor_read
22    return client._retryable_read(func, self._read_preference_for(s), s, operation)
23  File "/weka/home-meng/miniconda3/envs/py310/lib/python3.10/site-packages/pymongo/mongo_client.py", line 1540, in _retryable_read
24    return self._retry_internal(
25  File "/weka/home-meng/miniconda3/envs/py310/lib/python3.10/site-packages/pymongo/_csot.py", line 108, in csot_wrapper
26    return func(self, *args, **kwargs)
27  File "/weka/home-meng/miniconda3/envs/py310/lib/python3.10/site-packages/pymongo/mongo_client.py", line 1507, in _retry_internal
28    ).run()
29  File "/weka/home-meng/miniconda3/envs/py310/lib/python3.10/site-packages/pymongo/mongo_client.py", line 2353, in run
30    return self._read() if self._is_read else self._write()
31  File "/weka/home-meng/miniconda3/envs/py310/lib/python3.10/site-packages/pymongo/mongo_client.py", line 2483, in _read
32    self._server = self._get_server()
33  File "/weka/home-meng/miniconda3/envs/py310/lib/python3.10/site-packages/pymongo/mongo_client.py", line 2439, in _get_server
34    return self._client._select_server(
35  File "/weka/home-meng/miniconda3/envs/py310/lib/python3.10/site-packages/pymongo/mongo_client.py", line 1322, in _select_server
36    server = topology.select_server(
37  File "/weka/home-meng/miniconda3/envs/py310/lib/python3.10/site-packages/pymongo/topology.py", line 368, in select_server
38    server = self._select_server(
39  File "/weka/home-meng/miniconda3/envs/py310/lib/python3.10/site-packages/pymongo/topology.py", line 346, in _select_server
40    servers = self.select_servers(
41  File "/weka/home-meng/miniconda3/envs/py310/lib/python3.10/site-packages/pymongo/topology.py", line 253, in select_servers
42    server_descriptions = self._select_servers_loop(
43  File "/weka/home-meng/miniconda3/envs/py310/lib/python3.10/site-packages/pymongo/topology.py", line 303, in _select_servers_loop
44    raise ServerSelectionTimeoutError(

Trace
1AppError@http://localhost:5151/assets/index-o2FK2t37.js:593:54430
2GraphQLError@http://localhost:5151/assets/index-o2FK2t37.js:593:54578
3@http://localhost:5151/assets/index-o2FK2t37.js:625:15981

System information

OS Platform and Distribution (e.g., Linux Ubuntu 22.04): Linux Ubuntu 20.04.6
Python version (python --version): Python 3.10.14
FiftyOne version (fiftyone --version): FiftyOne v1.0.1, Voxel51, Inc.
FiftyOne installed from (pip or source): pip

Other info/logs
I can open the FiftyOne app with 1.0.2 version but encountered video related issue

#5199

and was recommended to use 1.0.1 by @benjaminpkane .
I followed the uninstall guide here to remove 1.0.2 version and reinstall 1.0.1 version. Please kindly let me know how we can solve this, thanks!
pip uninstall fiftyone fiftyone-brain fiftyone-db
pip install fiftyone==1.0.1

Willingness to contribute

[x] No. I cannot contribute a bug fix at this time

Ubuntu 20.04 is quite old, it may be a fiftyone-db issue. Using your own MongoDB connection may be more reliable


Use cluster volumes to share jenkins workspace

We are running several jenkins instances in one k8s cluster. The jenkins pods need to share the workspace with the build pods. As this is a multinode cluster it is not feasible to use the hostDir volumes mechanism in the current codebase for this purpose.
All of our jenkins instances have their workspace located on a cluster volume. To share that cluster volume with the build pods, it was necessary to add the ability to mount k8s volume claims in the pod and remove the implicit hostDir mount in KubernetesFacade.
It would be great if you can take a look, I am open to any suggestion. :)

@k9ert: I will have a look shortly!
The thing is that I am not 100% sure how we should proceed. Most of this functionality has move the kubernetes-plugin, which lifts the hostPath requirement by adding the build container inside the slave pod.
So I was in between deprecating kubernetes.pod() stuff or aligning them with the kubernetes-plugin.
Anyway, I'll have a look and let you know...

@cluk33: I cherry-picked and merged your commits as part of an other pull request and did some minor improvements and compatibility related changes.
Thanks.


Export tags?

I have a bunch of repos that are all checked out to a git tagged release. When I export my workspace, the versions are all recorded as git hashes. I didn't see a way to export the tag instead. I'd like the tagged release so I can more easily understand how packages are being updated. Is this possible?

Until now that was not possible. Please try the patch from #100 which should reference the tag name if there is exactly one tag referencing the current hash.


Feature/jax builder class

What?
General builder class for Jax systems.
Why?
For building systems in a highly flexible way using callback system components.
How?
Create a builder class with general-purpose builder methods and hooks.
closes #426 #443 #440

Closing this PR since it is being subsumed here: #457.


New Pixiv Profile Design

Is your feature request related to a problem? Please describe.
It doesn't work with the new Profile Design they introduced, which doesn't even show the number of bookmarks now and the tag selection has no english translation,
so this tool is kinda important to have here.
Describe the solution you'd like
Having it work just like before. Probably not a simple fix tho.
Describe alternatives you've considered
Another way would be with the pixiv search engine to somehow enable the search of all the works from a specific artist.
Additional context

Hello, what the version of Patchouli you use? What is your OS and browser?
It works in v4.2.0-alpha.20 in both my latest Chrome and Firefox and both my Windows and Linux.
I released the alpha version and explained the reason in README
You can also check the README of v4.2.0

Ok I checked again, and it was just my mistake, apparently a new browser extension I installed interfered... everything works fine now. So yea, thanks anyway


[3.x] Drop mouse focus and over when gui input is globally disabled

Version of #59099 for 3.x.
Since some porjects may be relying on the former behavior, this is opt-in via a new project setting, disabled by default, but enabled for new projects, since it's the new standard behavior (and the only one in 4.0).

Thanks!


Question: Force update instnacedUniforms

HI, @agargaro
Here is my samlpe code.  i wana update instanced unifrom perioldly. It seems not working. Is there any explicit update function need to call?

const PeriodUpdateInstances = ()=>{
  const  renderer  = useThree((state) => state.gl)
  const objs = InstancedMesh2(
      renderer, 1000, new PlaneGeometry(1, 1), new RttMaterial(1000)
  )

  objs.createInstances((obj, index)=>{
    obj.setUniform('vTexture', index);
  })
  objs.computeBVH()

  useEffect(()=>{
    const update = ()=>{
      for (let i = 0; i < 1000; i++) {
        objs.instances[i].setUniform('vTexture', Math.random());
      }
    }

    // do updated every 30 seconds
    const intervalId = setInterval(() => {
      update()
    }, 30000)
    return () => clearInterval(intervalId)

  })

  return <primitive object={objs}  />

}

Hi!
I should add more documentation about this 🤔
To use setUniform you must use a DataTexture.
There are utility methods to create and use them and you have to modify the shader (you can use Material.OnBeforeCompile).
Is RttMaterial a your custom material?
Example that use setUniform: https://stackblitz.com/edit/three-ezinstancedmesh2-custom-material?file=src%2FtileMaterial.ts,src%2Fmain.ts
Later I will try to write a simpler example and create utility methods also to patch existing materials like yours.
Ps. I see that you don't set the object positions and create a BVH. This might slow down.

I have prepared an example where you can set the opacity for each instance:
https://stackblitz.com/edit/three-ezinstancedmesh2-opacity-per-instance?file=src%2Fmain.ts
In your case you should do something similar.
Lacking utility methods it can be quite complex.
I will try to improve these things in future versions.

@agargaro   Thank you very much for your detailed answers. I think I might not have explained my question clearly.
I have already read the Minecraft demo[tileMaterial].
What I actually want is to regularly update the textures that are transmitted to the shader for storing uniforms.
Just like the opacity per instanced demo  you mentioned. I want to modify the opacity values at regular intervals.
I see that you have alreday written a comment in this eample
obj.setUniform('opacityTexture', id * 0.1 + 0.1); // can automatic create data texture TODO

What I want to know is how to actively trigger an update of the datetexture once. some  function lile forceUpdateDataTexture()

I have prepared an example where you can set the opacity for each instance:

const intervalId = setInterval(() => {
  console.log('update opacityTexture');
  for (const oneIns in trees.instances) {
    obj.setUniform('opacityTexture', Math.random()); // can automatic create data texture TODO
  }
}, 5000);

Just like in your example, I want to update the opacity value every 5 minutes.  It don'st work
const intervalId = setInterval(() => {
  const size = Math.ceil(Math.sqrt(count));
  const array = new Float32Array(size * size);
  array.fill(0.5)
  for (let i=0; i<count; i++) {
    array[i] = Math.random();
  }
  console.log('update opacityTexture', array);
  const newData =  new DataTexture(array, size, size, RedFormat, FloatType);
  console.log('update opacityTexture', newData);
  (trees.material as unknown as ShaderMaterial).uniforms.opacityTexture.value = newData;
}, 5000);

Create a new texture isn't a good approach.
Yesterday I updated the library (0.2.8) and created a new example:
https://stackblitz.com/edit/three-ezinstancedmesh2-custom-material-zuvyuc?file=src%2FtileMaterial.ts,src%2Fmain.ts

Create a new texture isn't a good approach.
Yesterday I updated the library (0.2.8) and created a new example:
https://stackblitz.com/edit/three-ezinstancedmesh2-custom-material-zuvyuc?file=src%2FtileMaterial.ts,src%2Fmain.ts
Let me know if it works by updating the library and following the example 😄

It' work,ing now  awsome~

@OhBonsai
I am closing this issue. If you still need support open another bug or join us on our discord server (you can find it in the readme) 😄


Fix test case issue with including " and \n and other special characters.

Currently this is still quite hacky, we should do something more robust and actually validate the json at compile time in BAML rather than what we are doing today which is evaluating it at runtime.

Are there any regressions with enum input or output, or single arg strings or booleans?


public folder and favicon clean up

Prerequisites

[ x ] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in QA
Followed all applicable steps in the TROUBLESHOOTING guide
Checked for relevant discussions on Discord: https://discord.gg/pPERUuv
Checked that your issue isn't already filed: https://github.com/COVID-19-electronic-health-system/Corona-tracker/issues

Description
client/public folder has some artifacts from create-react-app inside (logo files) that can be deleted and the favicon can be made a bit sharper by having the background transparent (just the globe with the plus showing)
Steps to Reproduce

start app in web browser
look at the browser tab for favicon

Expected behavior:
transparent favicon background
Actual behavior:
globe icon has a purple background (could look nicer)
Reproduces how often:
100%

I'd like to do this!


Add sample data

Hello,
When I ran in your repo few days ago, I wanted to try the tool but realized there was no sample data anywhere that could help me to test your tool.
Since I had no data to test the tool on either, I couldn't test.
Maybe it would be a good idea to add sample data for people that just want to have a look on your project ? :smile:

Hi, and thanks for the issue! :smile:
Could you elaborate on what sort of sample data you're looking for? As a hex editor, heh can inspect the bytes of any file that you have permission can read and write to - images, text files, binaries/executables, you name it.
As a part of https://github.com/ndd7xv/heh/issues/4, I'll add examples of how to run the command, especially when there are more options. For now, it's only heh path/to/file - so, for example, if you clone this repo, you could run cargo r -- demo.png. If you have heh installed already, you could run heh <path_to_repo>/demo.png to inspect the bytes of the image used in the README.
I'm not entirely sure if I should add sample data, since a user should be able to specify nearly any file on their system (as well as any file in this repo) and inspect its bytes.

hello @ndd7xv ,
My apologies, I believe it was some sort of Hex dump reader, but that was the usage I had in mind, not the primary one ahah.
Thanks for that precision. :smile:


Unify date names

There are 3 dates of interest for coupled simulations:

start_date  (referred to as date0 in the coupler, can be renamed to start_date like in atmos)= date when the simulation begins (used to calculate date  and to prescribe the initial boundary conditions from files; unused in Insolation.jl)
date or current_datetime = p.simulation.start_date + Dates.Second(round(Int, integratort) = current sim date
DateTime(epoch_string, dateformat"y-m-dTHH:MM:SS.s")   (referred to as date0 in the Insolation and ClimaAtmos)= defines a reference date for the beginning of an astronomical epoch for which we know orbital parameters (like position of Earth relative to its perihelion). As far as I can see, this is used to calculate Δt_years which then returns the orbital parameters expected for that date . The default epoch (for our current runs) is J2000 from CLIMAParameters, defined as January 1, 2000 at 12hr UTC. See section 3.2.2 and table 3.1 here. So epoch is just a reference date that is associated with a particular orbital parameters and the Insolation equations then calculate the expected insolation based on date . So the epoch date is different from the start_date. Unless we're running paleo climate simulations (for 1000s years), we should keep this epoch date as the default. That said, it would be more future proof to ensure we're setting the same epoch date from the coupler, and document what all the different dates are.

NB:
There are some dates relevant locally to each package as well:

in the ClimaCoupler.BCReader, we also use date1 to signify the first day of month for time interpolation.
in ClimaCoupler.TimeManager, we have ref_date which tracks the next date a callback will be invoked.

@juliasloan25 @sriharshakandala @szy21 please do check and chime in.

@kmdeck tagging for visibility as we may want to rename our dates in ClimaLSM to unify with the other packages.
ClimaLSM uses:

date_ref, a DateTime before or at the start of the simulation. ClimaLSM keeps track of time using the number of seconds since simulation start, so this reference date allows us to map our simulation time to an actual DateTime. This allows us to coordinate reading of multiple input data files that vary over time.
date0, the date of the start of the simulation, derived as date0 = date_ref + Dates.Second(t_start), where t is the number of seconds elapsed since date_ref at simulation start. Used when reading in data from files to ensure we use data from the right point in time.

We definitely need to address this naming convention! 😆 Thanks for being proactive about this, @juliasloan25 !

epoch_string should be a parameter in the end (see https://github.com/CliMA/Insolation.jl/issues/41, epoch in CLIMAParameters is actually not used in Insolation now). We can rename date0 in Insolation if needed.
For information about when the simulation starts and the current simulation time, it makes sense to me to use start_date (or start_datetime) and current_datetime, but I don't have a strong preference for the naming.


Add support to ProxyResultSet in order to optimize the findColumn loo…

…kup on Oracle/SQLServer. Postgres has had a similar optimization for many years
First off, thanks for great lib. My performance testing showed a 200x improvement vs a "safe" tomcat pool setup for the get/close connection case.
I hope it's not presumptuous to toss this your way without opening an issue, but I figure code is easier to discuss. I'm moving to use HikariCP on a system that supports SQLServer and Oracle.
Both of which have an inefficient findColumn method that I've had to work around for years and has caused real world perf issues. Postgres has had an optimized method for 10+ years:
https://github.com/pgjdbc/pgjdbc/blob/265f22b28fd7b5511e30a28ad959d5645e9722cd/pgjdbc/src/main/java/org/postgresql/jdbc/PgResultSet.java#L2624
This ancient artifact discusses the issue and workarounds that were taken:
https://hibernate.atlassian.net/browse/HB-1072
I'm curious if you'd be amenable to including this in HikaricCp. I understand why the answer would be no, as maybe it should be the job of the driver not to be silly with this lookup, but this will likely give some further bragging rights on Oracle/MSSQL :)
If so, happy to write test cases and make any modifications.
Things I weren't sure about is if it really made sense to force the optimizeFindColumn boolean down everywhere, but seemed to be the least icky pattern available. Also, made this configurable (or will make it) as it could change some behavior in nuanced ways and there is no reason to ever enable for postgres.
Thanks!!!

You are correct that I cannot integrate this into HikariCP on principle.  However, at least in the case of SQL Server, the mssql-jdbc driver is now open source (here on GitHub) and I have contributed many performance related fixes in the past 2-3 years.


Hondsrug example doesn't work with the newer Modflow binaries (> 6.3)

In GitLab by @JoerivanEngelen on Jul 7, 2023, 09:59
Apparently something changed in Modflow 6 which causes that it throws an error upon reading the Hondsrug example.
This doesn't happen with Modflow 6.3, but happens with Modflow 6.4. We should investigate the root of this problem.

In GitLab by @JoerivanEngelen on Jul 7, 2023, 10:03
marked this issue as related to #457

In GitLab by @HendrikKok on Jul 12, 2023, 10:07
Hondsrug example works with mf6.4.0 and mf6.4.2, but not with mf6.4.1. This version crashes with a floating invalid and a stack-trace to the npf-module (load file).


Return "sensors -j" to null

Modified original sensors -j call to output all to /dev/null. This prevents temp errors when something cannot be read. i.e. when Wi-Fi is disabled, this will cause an error to log to journal that "temp1_input" cannot be read for device.

Thanks !

Oh god, I should have tried that before merging :facepalm:
Of course it should be sensors -j 2> /dev/null otherwise there's nothing to parse

That would be why I commented. =)


DP override example corrected.

Fix for https://github.com/kumahq/kuma/issues/1005

Thank you for fixing it!


Removing unnecessary code to narrow down chat issues

I'm just going to give the heads up that this will disable the autoscroll service -- I'll approve it but I'll make sure to review it later.


Errors in browser builds after recent update

I'm using this in a webpack app, and updating from 1.1.0 to 1.3 started causing lots of errors like

Module not found: Error: Can't resolve '@babel/runtime/helpers/get' in 'C:\data\temp\repro-yaml-1.3\node_modules\yaml\browser\dist\schema'
@ ./node_modules/yaml/browser/dist/schema/Map.js 21:35-72
@ ./node_modules/yaml/browser/dist/Anchors.js
@ ./node_modules/yaml/browser/dist/Document.js
@ ./node_modules/yaml/browser/dist/index.js
@ ./src.js

would it be possible to make this an optional dependency? or is there another solution for this problem?
for project using node, it really makes no sense to draw in babel as dependency. maybe it should be a devDependency, since you only need it when building with webpack, right?

Having @babel/runtime as a dev dependency doesn't help, as those aren't included in installs. It would be nice if optional dependencies worked, but they're unlikely to really work like you'd expect (see e.g. npm/npm#14185). So we're left with it as a direct dependency, which I figured is the least wrong right option atm. And yes, for most users this unfortunately means that you get about 624KB of unused code sitting in your node_modules directory.
For context, @babel/runtime is currently (1.4) a dependency of yaml, but is only used when run/compiled in an environment like Webpack which reads the "browser" field of package.json. Here's what the readme mentions about it:

It runs on Node.js 6 and later with no external dependencies, and in browsers from IE 11 upwards (Note: @babel/runtime is used only by the "browser" entry point).

The logic that I went through to get here is that first of all it makes sense to use @babel/runtime in browsers builds, due to the measurable file size savings that it enables; these savings can't be re-captured during the application's minifications, but must be a part of the yaml library's own transpilation. This effectively does mean that we need to have @babel/runtime as a dependency, even if it's not used in Node.js environments.


Travis issues

It looks like travis isn't working for the romero-gateway any more:
...
travis_fold:end:before_install
Could not locate requirements.txt. Override the install: key in your .travis.yml to install dependencies.
travis_time:start:0a43edd8
$ .omero/lib-docker
.omero/lib-docker: line 6: e.: command not found

travis_time:end:0a43edd8:start=1522664111923424591,finish=1522664111930001373,duration=6576782

The command ".omero/lib-docker" exited with 127.
...

Could that be related to https://github.com/openmicroscopy/omero-test-infra/pull/3 , especially fdbd628abd4c9c

Fixed by https://github.com/openmicroscopy/omero-test-infra/pull/15


Allow user to specify smaller traffic graphs on dashboard widget

A dashboard should usually aim to be visible to the eye, not needing scrolling etc too much. But even with extra columns, the traffic graphs are quite large, so this adds a "chartsize" option to the widget to set a smaller size.
It works well for me, with one CSS related caveat which I can't fix, and need someone who knows the widget system better to put it right ( @phil-davis ? :) )
The sole issue is the scaling and positioning of the graphs. At the moment I can't figure how to do this "natively", by scaling the svg's themselves, or their viewport, or a better way. So I'm solving it using CSS zoom which is well supported on all browsers for many years. It gives almost the right results but also a lot of trailing white space, which I can't figure how to remove.
But as the code's essentially there with the exception of CSS/appearance, can I ask someone who knows the codebase and widgets better than me, to figure out the required final change to make this good, and I'll push it to my repo when that's done. Everything else is good. Makes the dashboard a lot easier to get an overview of the system's activity.

I'm not a fan of adding options when they aren't absolutely necessary, as they tend to add a ton of technical debt within themselves and especially when compounded on existing/future options.
I agree that traffic graphs should be smaller, but think we should just make them smaller across the board. A simple CSS change to pfSense.css from a height of 250px to 150px:
.traffic-widget-chart, .traffic-widget-chart svg {
    padding: 0;
    height: 150px;
}

produces this:

what do you think?

I would NOT want min any smaller than currently are !


mask entire screen

Is there a way that you know of, to tell p-loading to mask the entire screen?   I am attaching to  (the highest element) but if the content doesn't fill the screen it looks kind of strange.  See this screenshot
Also, thanks for doing this plugin, it's pretty sweet!

Hi Midget,
I'm sorry for the late reply, I was off for 2 weeks.
Regarding your question, definitely there is a way.
The width and height of the mask created by P-loading  is taking the 100% of size of the height and width of its parent/selected element $('#this_is_the_selected_element').ploading({action: 'show'}).
So, if you want to display the mask in the entire screen then, you should select an element which has width and height as 100%. In many cases people tend to give these properties to the body element (the HTML element should have them too) `html,body { height: 100%; margin: 0px; padding: 0px; } (it can be different), and that's it.
Important: be sure that element where you want to show the mask doesn't have any padding, as the mask will respect it.
Please let me know if this fix your problem.
If you want to learn more about using 100% of height in CSS, you can check this article: http://www.tutorialrepublic.com/faq/how-to-set-a-div-height-to-100-percent-using-css.php
Cheers.

Closing this as it's not an issue and already suggest a way how to implement the mentioned feature.