onIndexChange() delay

Current behavior
My view consists in two tabs, each with text input. The purpose is to search the values in each tab independently. For this, Search Button turns blue when there is some value in text input.
When I change a tab, the text input from the previous Tab must be cleaned, using onIndexChange function. However, there is a delay, so an index are not changed and consequently, a text input is not cleaned immediately, creating a bug.
Initial state:

TIN input change:

Tab change :

After few seconds:

Expected behavior
Expect to change index and edit (in this case, clean) values immediately, using onIndexChange().
Reproduction
https://snack.expo.dev/@ekaigna/-react-native-tab-view-bug-repro
Platform

[X] Android
[X] iOS
[X] Web
[ ] Windows
[ ] MacOS

Environment
{
"name": "pwaexpo",
"version": "1.0.0",
"main": "node_modules/expo/AppEntry.js",
"scripts": {
"start": "expo start",
"android": "expo start --android",
"ios": "expo start --ios",
"web": "expo start --web",
"eject": "expo eject",
"build-react": "webpack"
},
"dependencies": {
"@expo/webpack-config": "^0.16.15",
"@react-native-async-storage/async-storage": "~1.15.0",
"@react-navigation/native": "^6.0.6",
"@react-navigation/native-stack": "^6.2.5",
"@react-navigation/stack": "^6.0.11",
"babel-plugin-inline-import": "^3.0.0",
"expo": "^44.0.4",
"expo-app-loading": "~1.3.0",
"expo-blur": "~11.0.0",
"expo-device": "~4.1.0",
"expo-font": "~10.0.4",
"expo-linking": "~3.0.0",
"expo-secure-store": "~11.1.0",
"expo-status-bar": "~1.2.0",
"i18next": "^21.6.3",
"lottie-react-native": "^5.0.1",
"native": "^0.3.3",
"react": "^17.0.2",
"react-dom": "^17.0.2",
"react-i18next": "^11.15.1",
"react-native": "0.64.3",
"react-native-awesome-alerts": "^1.5.2",
"react-native-bouncy-checkbox": "^2.1.9",
"react-native-calendars": "^1.1274.0",
"react-native-drax": "^0.9.3",
"react-native-dropdown-picker": "^5.2.3",
"react-native-elements": "^3.4.2",
"react-native-gesture-handler": ">=1.8.0",
"react-native-masked-text": "^1.13.0",
"react-native-pager-view": "^5.4.9",
"react-native-raw-bottom-sheet": "^2.2.0",
"react-native-safe-area-context": "3.3.2",
"react-native-screens": "~3.10.1",
"react-native-svg": "^12.1.1",
"react-native-tab-view": "^3.1.1",
"react-native-toast-message": "^2.1.1",
"react-native-uuid": "^2.0.1",
"react-native-vector-icons": "^9.0.0",
"react-native-web": "^0.17.5",
"react-native-web-lottie": "^1.4.4",
"react-native-web-swiper": "^2.2.2",
"react-redux": "^7.2.6"
},
"devDependencies": {
"@babel/core": "^7.16.7"
},
"private": true
}

I have the same issue.
"react-native": "^0.66.4",
"react-native-web": "^0.17.5",
"react-native-tab-view": "3.1.1",

https://user-images.githubusercontent.com/18309525/156334205-8dccbcb7-5859-4c2d-9272-c95caf401990.mov

I've tried moving onIndexChangeRef.current(index) out of the animation with good results, but I don't think it's a good way...
diff --git a/src/PanResponderAdapter.tsx b/src/PanResponderAdapter.tsx
index 3d1964a..3c2b13e 100644
--- a/src/PanResponderAdapter.tsx
+++ b/src/PanResponderAdapter.tsx
@@ -81,6 +81,7 @@ export default function PanResponderAdapter<T extends Route>({

       const { timing, ...transitionConfig } = DefaultTransitionSpec;

+      onIndexChangeRef.current(index);
       Animated.parallel([
         timing(panX, {
           ...transitionConfig,
@@ -89,7 +90,6 @@ export default function PanResponderAdapter<T extends Route>({
         }),
       ]).start(({ finished }) => {
         if (finished) {
-          onIndexChangeRef.current(index);
           pendingIndexRef.current = undefined;
         }
       });

Hey @ekaigna, I can't reproduce you issue on iOS / Android. It looks like this is only present on Web. Where the delay is really noticeable. The fix that @Dylan0916 suggested looks good to me. @Dylan0916 can you make a PR adding this?

One way to address this would be to use renderTabBar and use the animated position to control the active tab instead of using the index

does this problem still exist?


[SQL][MINOR] Move general char/varchar test to the base test suite

What changes were proposed in this pull request?

This is a followup of https://github.com/apache/spark/pull/32501 . It moves a general char/varchar test from file source suite to the base char/varchar suite, so that it will be verified in all table formats, including v2.
Why are the changes needed?

improve test coverage
Does this PR introduce any user-facing change?

no
How was this patch tested?

N/A

cc @HyukjinKwon


[backport] Add minor version policy and feature backport policy

This is a backport of #3297

LGTM


"yarn format:fix" makes "yarn lint" an error

Describe the bug
Code before the yarn format:fix that passes the yarn lint
      console.log({ // eslint-disable-line no-console
        path: example,
        stdout,
      });

yarn format issues warnings for this
yarn format
Checking formatting...
[warn] tests/examples/examples.test.ts
[warn] Code style issues found in the above file. Run Prettier with --write to fix.

Then yarn format:fix change this code to
      console.log({
        // eslint-disable-line no-console
        path: example,
        stdout,
      });

yarn lint issues an error and a warning for this code
/Users/akihikokuroda/development/repositories/agenticflow/bee-agent-framework/tests/examples/examples.test.ts
  40:7  error    Unexpected console statement                                                   no-console
  41:9  warning  Unused eslint-disable directive (no problems were reported from 'no-console')

✖ 2 problems (1 error, 1 warning)
  0 errors and 1 warning potentially fixable with the `--fix` option.

To Reproduce
Refer to Describe the bug
Expected behavior
yarn lint and yarn format should not conflict each other.
Screenshots / Code snippets
Set-up:
Code level: commit b86629803bc1e3f65aaf1aefad2382053e3f27cd (origin/main, origin/HEAD, main)
Additional context

This issue relates to eslint-config-prettier.
However, the solution is easy. Use // eslint-disable-next-line no-console instead.
// eslint-disable-next-line no-console
console.log();

Works great!  Thanks!


Error Handling To Address Paper Check Crashes

Sometimes the program will crash when it cannot detect certain information about the paper version. While this doesn't address the root issue (why the program is not properly detecting information from the paper server), it does add error handling that prevents the program from crashing. This will, at the very least, allow users to be able to report where the issue is happening and prevent them from stopping use of the software because it crashes at times.
This also makes the paperCheckForUpdate function return false if an error is encountered. This can then be used if you wish to handle the error in some way in the future.

Made the requested changes. They have been pushed and added to the PR.

Hey @ZandercraftGames,
i merged it and released an update: v1.5.3
Thank you again so much for your PR! 😃
...and thanks for flying with pluGET! 🚀

Hey @ZandercraftGames,
i merged it and released an update: v1.5.3
Thank you again so much for your PR! 😃
...and thanks for flying with pluGET! 🚀

😄


AttributeError: Neither 'InstrumentedAttribute' object nor 'Comparator'

class Search(db.Model):
    id               = db.Column(db.Integer, primary_key=True)
    html             = db.Column(db.Text)

    def __init__(self, **kwargs):
        vars(self).update(kwargs)

    @classmethod
    def get_queue(self):
        urls = self.query.filter(self.html != None).limit(30).all()
        for url in urls:
            print url

AttributeError: Neither 'InstrumentedAttribute' object nor 'Comparator' object associated with Search.query has an attribute 'filter'

I am not sure if this is an issue, but I cannot find any information anywhere to help me. I have used Flask SQL Alchemy in other projects, what I am trying to do is really simple.....

Could you add more of the stack trace? Where exactly does this error occur?

I can't reproduce your error.  If this is an issue, it appears to be with SQLAlchemy, not Flask-SQLAlchemy.  Stack Overflow is the appropriate place to ask questions about your own code, and since you've already done that, I'm closing this unless new information demonstrates that this is a bug.

 I was using a reserved keyword.

Reserved keyword of SQL?

If anyone ever gets this again, the issue is that he was most likely using query or filter as fields in his database class.

This appears to happen to other fields to, i had
    id = Column(Integer, primary_key=True)
    hash= Column(String(64), nullable=False, index=True)

id and hash aren't reserved words but are global functions, and i got this error. However , just changing it to this seems to have fixed it:
    id_col = Column("id", Integer, primary_key=True)
    hash_col = Column("hash", String(64), nullable=False, index=True)

calling a column id - even though it's the name of a builtin - is perfectly fine and valid. if this causes a problem something else is w rong.

@ThiefMaster well you were fast, i commented and realized i was wrong so i deleted it, heh


_alter_table for sqlite3

Hey, I am wondering why there is no support for DROP and CHANGE in _alter_table for sqlite3. Is it just that nobody has implemented it yet or is it something more serious?
https://github.com/bcit-ci/CodeIgniter/blob/2ad24af488da7dc518684cc146914b2937749031/system/database/drivers/sqlite3/sqlite3_forge.php#L144-L157
I am asking because I would like to use that function both with mysqli (which is supported) and sqlite3 (which is currently not).

SQLite being "lite" simply doesn't support a lot of functionality that you'd otherwise find in a traditional SQL database. Such is the case here, as it simply doesn't support DROP and CHANGE column statements.
Docs: https://www.sqlite.org/lang_altertable.html
The commented-out SQL code seen in CI_DB_sqlite3_forge::_alter_table() is a potential work-around for the DROP functionality. In a way, that is indeed the case of just noone having implemented it yet, but that's because the implementation is much more complex than what other database platforms offer.


feature/cached-config-provider

Implemented CachedConfigProvider;
Implemented InMemoryDriver;

Probably should rename CachedConfigProvider to CachingConfigProvider.
Also thinking on adding NullCache.
Maybe an overkill with InMemoryDriverTest assertions.

I think the InvalidCacheKeyException could go to the Exceptions directory, other than that LGTM.


Gradio UI is not responsive after being packaged with Nuitka

Background: see here https://github.com/Nuitka/Nuitka/issues/2540
1.9rc7
Commercial: None
Python: 3.11.5 (main, Sep 11 2023, 08:31:25) [Clang 14.0.6 ]
Flavor: Anaconda Python
Executable: /Users/first_lastname/Projects/gradio_test/myenv/bin/python
OS: Darwin
Arch: arm64
Version C compiler: /usr/bin/clang (clang 14.0.3).

When running Nuitka 1.9rc7 from factory, I am able to package and run the example code below, but once I open the Gradio UI on the browser, the UI is not responsive. Upon inspecting the developer console on the browser, I see some JavaScript exceptions.
import gradio as gr

def greet(name):
    return "Hello " + name + "!"

demo = gr.Interface(fn=greet, inputs="text", outputs="text")

if __name__ == "__main__":
    demo.launch(show_api=False)

To debug (again assume nuitka 1.9rc7 from factory)
nuitka --static-libpython=no  --follow-imports --include-data-dir=./gradio_test/data=data --noinclude-setuptools-mode=nofollow --standalone --macos-create-app-bundle --clang gradio_test/main2.py

 ./main2.app/Contents/MacOS/main2

Open the browser and follow http://<IP_ADDRESS>:7860
In Chrome, go to View->Developer->Developer Tools
Select Console, reload the page, and try to press any button in the UI.
This is what I see

I am not super familiar with how Gradio works under the hood, but there is a JavaScript portion of the framework used to build the frontend. When installing gradio from source, the first step is to build the frontend (scripts/build_frontend.sh - which ultimately calls pnpm build).  I suspect Nuitka is not including some of the frontend files in the package.
One additional thing I noticed (this might be an unrelated issue), when loading the page, I see an error for a POST request. Upon inspecting the payload of that request, the payload has an empty "version" field. This field is not empty if launching the unpackaged script.

update here - I got it to work by adding the flag --import-data-package=gradio
this added the following files, and now the simple seem to be working fine

I guess, I only stumbled over "templates" during its startup, but seems _frontend_code surely is at least essential, and the version might be coming from package.json, just a hunch, need to check that. This is going to be 1.9 hotfix material though, since I largely want to get that out of the door ASAP now.

sounds good, thanks for looking into it. I can manually include the package data until then. Thanks!

I added it on the factory branch and it will be part of 1.9 release, which got delayed enough to finish this on the side with your input.

Thanks for your report, this is fixed on the factory branch, which is a development version under rapid development. You can try it out by going here: https://nuitka.net/doc/factory.html
Feedback if this is working is very welcome, just please do not share plans of doing it, but rather confirmations or denials of it working.

Part of the 1.9 release I just made.


hardcoded file extension

https://github.com/ksivamuthu/actions-setup-gh-cli/blob/52a7c6659db375e08e92b1bd0ff31be356f574f8/src/main.ts#L29
Hello there, I'm using this action in self-hosted runners and one of the usage is to run with macOS, but this line in the action hardcodes the format to be tar.gz, whereas, github.com/cli/cli 's releases page, the archive format is zip for macos, would you be open to a PR that'll take the zip format as an input and that will default to .tar.gz so that it's not a breaking change?

Merged #125  and released new version.


Fixed some text in README.md

PR Checklist:

[N/A] I have run npm test locally and all tests are passing.
[N/A] I have added/updated tests for any new behavior.

[N/A] If this is a significant change, an issue has already been created where the problem / solution was discussed: [N/A, or add link to issue here]

PR Description
This pull request fixes some text to be more consistent in README.md

@FredKSchott please review and merge if ok

@mikeal @nylen @simov @FredKSchott please have a look


[Bug]: Always Increase Version Code

Before you make the issue, confirm that

[X] You are running the master build from GitHub actions
[X] You have read the FAQ and the usage instructions
[X] You have searched pull requests and issues and have found none similar to your issue
[X] This problem comes from a failure in the tiling system and not a missing feature

Describe the bug
The version code is not increased with every (pre)release, this confuses plasma and makes it refuse to install the update because it thinks it's the same version. This forces the user to manually uninstall the previous version before installing the next one.
To reproduce
Download the latest beta and the previous beta.
Install the previous beta.
See that if you now attempt to install the current beta it fails.
Expected behavior
The new beta simply is installed
Log output
N/A

Screenshots and videos
No response
Additional context
No response

It is perhaps a bit too late for this, as the next planned release is v1.0 itself. The project follows semantic versioning for non-prerelease software, it's just been a while since thats been apparent.


The path information of the snapshot under one blob  isn't localized

Storage Explorer Version: 1.11.0
Build: 20191127.2
Branch: master
Language: Chinese(zh-CN) /Chinese(zh-TW)
Platform/OS: Windows 10/ Linux Ubuntu 18.04/macOS High Sierra
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Launch Storage Explorer.
Open 'Settings' -> Application (Regional Settings) -> Select 'Chinese (simplified)' -> Restart Storage Explorer.
Expand one storage account -> Blob Containers.
Create a new blob container -> Upload a blob to it -> Create snapshot for the blob.
Manage Snapshots for the blob -> Check the path information on editor.

Expect Experience:
The path information is localized.
Actual Experience:
The path information isn't localized.

More Info:
This issue also reproduces for one file under one file share.

@craxal Is this exposed to localization?

This issue also reproduces in Japanese.

Verified on build 20200319.10. The path information of the snapshot for one blob has been fixed. This issue still reproduces for the files under one file share.

@lukaschal This was moved to Translating a week ago. Have translators checked in an translations yet?

@craxal  Localized at 3/20/2020.
https://devdiv.visualstudio.com/DevDiv/_git/StorageExplorer/commit/dcf01f94fd9d179b4ca0525ec5b0ef420318672e?_a=compare&path=%2Floc%2Floc%2Fit%2Fsrc%2FStandalone%2Fresources%2FAzure.json.lcl
File: loc\en\src\Standalone\resources\Azure.json
ID: 0; "PrefixSnapshotsFor"
SRC: Snapshots for '%1$s'
TGT: Snapshot per '%1$s'

Verified this issue on build master/ 20200324.2.

The word Snapshots is not localized in the path information of the snapshot for one file share.

@v-xainya We likely haven't updated the Cloud Explorer reference in Storage Explorer for the above build. Verify on a newer build.

@craxal The above issue still reproduces on build 20200324.12.
And we will verify on a newer build. Thanks for your updates.

Moving to 1.14 for full verification then.

Verified this issue on build 20200627.2.
The word Snapshots is not localized in the path information of the snapshot for one file share. So we re-open it.

With the implementation of editable address bars, the current state of the code doesn't easily allow us to display "Snapshots for ...". Investigating potential solutions.

Verified this issue on build master/20200807.6. Fixed.


Rangefinder

I know that most phones can autofocus, and that a lot of cameras do this using an IR laser, so I was wondering if there was any way to get a useable distance output from that sensor...
If possible, I could see this being very useful...

My bad, I meant to put this under new feature suggestions... I'm new to GitHub


Warnings : sklearn/ensemble/_forest.py:396: FutureWarning: Criterion 'mse' was deprecated in v1.0 and will be removed in version 1.2.

I get lots of warnings like these from XT and RF models while running TabularPredictor().fit()
sklearn/ensemble/_forest.py:396: FutureWarning: Criterion 'mse' was deprecated in v1.0 and will be removed in version 1.2. Use criterion='squared_error' which is equivalent.
This is documented in https://scikit-learn.org/stable/modules/generated/sklearn.tree.DecisionTreeRegressor.html
I will submit a quick fix patch.

Scikit-learn 1.0 was only introduced in Sep 2021.  Should we enforce scikit-learn>1.0.0 in tabular./setup.py ?
Or maybe this is too early for this PR in case many people remain on old scikit-learn?  The prior version to 1.0.0 was 0.24 and it's not clear that "squared_error" was a supported alias at that point:
https://scikit-learn.org/0.24/modules/generated/sklearn.tree.DecisionTreeRegressor.html

Thanks for noticing and creating a fix! I believe the two remaining usages should be ok, and version should be capped correctly already. Will auto-close issue once PR is merged.


Add travis.yml and image deploy to bintray

deploy microkernel and ipxe build artifacts to bintray
Includes some ansible tweaks

@RackHD/corecommitters @RackHD/rackhd_dev

+1

:+1:

yay!

updated,  enabled file override in bintray.

Still :+1:


update code owners

Packages impacted by this PR
N/A
Issues associated with this PR
N/A
Describe the problem that is addressed by this PR
Added more code owners for Cosmos DB JS sdk
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
N/a
Are there test cases added in this PR? (If not, why?)
N/A
Provide a list of related PRs (if any)
N/A
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[ ] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

/check-enforcer override


elasticsearch setup snapshot and restore to S3

Configuration for an off-cluster storage (AWS S3) for Elasticsearch

Resolved  in #295


I can only pair one controller.

Hey, so I've had this stuff for over a year now, it never worked for me on Windows 7 but I've recently upgraded to Windows 10 so I decided to give it another shot. I can't even get as far as where I did in Windows 7. Only one controller paired. It took around 8 minutes of button mashing for it to finally pair, which seems a lot longer than what tutorials show. After mashing the other controller's button for 20 minutes straight and it still wouldn't pair; I can only assume that something is wrong. I really want this to work. I'm using the PS3 move controllers, 2 PSeyes which have the libusb drivers set up. I'm running Win10 and am using a DK2 if that has anything to do with it. Please help.
The controllers should work fine, as they once paired no problem back on my Win7 machine.
Thank you.

I''m having this same issue. DK2, 2 motion controllers. One pairs fine, the other won't pair at all. Using the 1903 build to get one of them to pair, the other one is showing up in my Bluetooth menu but won't pair. I've tried uninstalling the device through device manager (which did nothing) and through Bluetooth (which just tells me that 'remove failed' for some reason). Any help?

I manged to get the remove to succeed by turning the bluetooth off and back on. Try that @qwFaTE ?

Any particular time when to do that? I've tried restarting bluetooth before pairing and mid-pairing, but to no avail. When did you restart bluetooth? Also, how would I get the 1903 build? All of the releases are marked as #X.X.X in their version numbers. Thank you for the reply!

try #634

Thank you so much, @NikitaShokin ! Worked like a charm, much appreciated!


Redactor new link

Redactor (jQuery) http://redactorjs.com/  has a new link: https://imperavi.com/redactor/

Thanks! Fixed 😄


Reproduction: Question about Model Weights

Hi!
I just downloaded your model weights and I have a few questions. In the paper you train four different models:

Protmamba Foundation
Protmamba Foundation, FIM fine-tuned
Protmamba Foundation Long
Protmamba Foundation Long, FIM fine-tuned

My first question: Since there is only the Protmamba Foundation Long released, could you also consider releasing the weights for the remaining three models? At least the FiM tuned model would be great so I can reproduce the best Proteingym results from Figure 8.
Next I want to compare training curves. Does Figure 4 show the training of Foundation or Foundation Long?
Thanks again for your help!

Hi @smdrnks I confirm that Fig 4 show the training of Foundation Long. The training curve of Foundation is the same (since it's the same model trained for less steps) but it stops at $6\times 10^{10}$ steps (i.e. when we increase the context size from $2^{15}$ to $2^{16}$ tokens).


Homebrew support

For folks that have cocoapods installed via homebrew instead of as a ruby gem directly it'd be nice to be able to use this as well

@abejfehr did you ever figure out a way to use this plugin along with installing pod from homebrew?

@jzaefferer I actually did get it to work.
I don't remember exactly what I did, but I think I installed the gem via the homebrew ruby version that I was using like /opt/homebrew/opt/ruby/bin/gem install cocoapods-check and it made the pod check command available for me.
For me though, it saved essentially no time 😅 I think there are some very real performance issues associated with the design decision to use ruby as the language of choice for cocoapods and also some not efficient things that react native is doing with it.
Best of luck though if you do try it


MCscan Pipeline download results not working

Hi,
I am using the version for windows using the Docker image. After running MCscan Pipeline, it does not give back any results. The download button remains "empty". It seems to be running, though. It shows a page with a percentage of the analysis running.  (PS: I try it of files that I already used in jcvi as well with the example files).

Hi,
Thanks for using ShinySyn. Please be reminded that MCscan Pipeline takes FASTA and GFF3 files as input, and the download button will only be enabled after the pipeline is finished. ShinySyn supports the latest chrome, edge and firefox.
Bests,
Zhixia

Same error, hope it will be fixed soon.

We used the normal fa and gff3 downloaded from NCBI, when the progress bar seemed to be finished (100%), the download button was still grey, and couldn't be clicked.


Question on "mask"

Hi! Awesome work you've done here :)
What exactly does batch["mask"] refer to? I went through the create_dataset.py and noticed that you created the mask yourself if no masks are available. I'm quite new on scalar images / medical datasets, so wasn't sure what does that mean.
On the contrary, if I'd like to test this on other non-medical datasets, i.e. VisA, how should I modify these lines here? Realistically, my datamodule would contain batch["image"] as the input (healthy image) and also some ground-truth masks paired with "unhealthy" images for testing.

Hi. That one is just in case the mask is not given. It will create a mask on run-time. However, if you provide the mask, that condition will not be used.


Creation of Venv

We wish to have a venv created such that all of those wishing to develop/contribute can create a local & identical env

auto-sklearn can not be installed on arm64 and is therefore blocked untill it is removed, giving more evidence to #4

Now ready to be worked on with #4 effectively done, (changes in dev thanks to #8 )

Work for this completed and now in dev just needs to be merged into main


Vessel does not detect the right namespace when using containerd

By default Vessel uses namespace k8s.io.
While this is fine when k8s is deployed, it might lead to errors when issuing commands like Save if images where pulled from a different namespace.

Proposed solution is to scan all available namespace and try each one when issuing a command.


How to select dates of a week

Version 7.1.6
How can I select all the dates of a week in didSelectDate? or maybe somewhere?

calendarView.selectDates() function is used to select dates.


Missing lifting support for some instructions in iOS kernelcache

I'm aware that this is very much of a stretch-goal that I don't actually expect this to be fully done (and some are SVE ones so we don't have a good way to lift this), but I'm gonna post my list, so people have a better idea about what's missing in a typical iOS kernelcache.
I'll keep updating this list over time - so just see the edited comments (iPhone13,3_14.1_18A8395).
This list may be similar for the dyld_shared_cache and other binaries compiled by Apple's clang. Similar idea to issue Vector35/binaryninja-api#4002 but this is for instructions present in kernels as opposed to usermode code - some crossover may exist.

[ ] 2454 fcvtzu [0xfffffff007d66a0c, 0xfffffff007d66a70, 0xfffffff0082e9990, 0xfffffff008395b00, 0xfffffff0083eff84, ...]
[ ] 1634 fmla [0xfffffff009577c58, 0xfffffff009577c60, 0xfffffff009577cb8, 0xfffffff009577cc8, 0xfffffff009577d78, ...]
[ ] 1158 fmls [0xfffffff009577c54, 0xfffffff009577c5c, 0xfffffff009577cb0, 0xfffffff009577cc0, 0xfffffff009577d74, ...]
[ ] 286 ldset [0xfffffff007b3defc, 0xfffffff007b59e7c, 0xfffffff007b81ba4, 0xfffffff007b81e70, 0xfffffff007b821fc, 0xfffffff007b822c8, 0xfffffff007b82d24, 0xfffffff007b82e60, ...]
[ ] 254 ucvtf [0xfffffff0088da508, 0xfffffff008e9a200, 0xfffffff008e9a4a0, 0xfffffff008e9a6ec, 0xfffffff008e9a8f0, ...]
[ ] 253 ldclr [0xfffffff007b3e52c, 0xfffffff007b3e3a4, 0xfffffff007b3e440, 0xfffffff007b65b84, 0xfffffff007b65b9c, 0xfffffff007b65bb4, 0xfffffff007b65bcc, 0xfffffff007b65be4, ...]
[ ] 234 ldclrl [0xfffffff007b3e9f8, 0xfffffff007b3eb68, 0xfffffff007b3ef18, 0xfffffff007b3ef58, 0xfffffff007b3efc8, ...]
[ ] 150 fcvtzs [0xfffffff007bdb4e8, 0xfffffff007bdb6b0, 0xfffffff007bdbae0, 0xfffffff0082e8c88, 0xfffffff0082ed748, 0xfffffff008392c5c, 0xfffffff008395b9c, 0xfffffff008395bb4, ...]
[ ] 131 fdiv [0xfffffff00840269c, 0xfffffff008e9ad40, 0xfffffff008e9adf0, 0xfffffff008e9b204, 0xfffffff008e9b2b4, ...]
[ ] 113 scvtf [0xfffffff007bdb4e0, 0xfffffff007bdb560, 0xfffffff007bdb564, 0xfffffff007bdb618, 0xfffffff007bdb6a8, 0xfffffff007bdbad8, 0xfffffff007d669b4, 0xfffffff00800aa4c, ...]
[ ] 111 ld2 [0xfffffff007c64ebc, 0xfffffff007c64ec4, 0xfffffff007c64f10, 0xfffffff007c64f14, 0xfffffff007eb48dc, 0xfffffff007f8a8a0, 0xfffffff007f8a8a4, 0xfffffff0081b8f14, ...]
[ ] 83 fccmp [0xfffffff007bdb57c, 0xfffffff007bdb57c, 0xfffffff00841eb54, 0xfffffff0088c6624, 0xfffffff0088ce210, 0xfffffff0088ce294, 0xfffffff0088ce330, 0xfffffff0088ce3e0, ...]
[ ] 73 dup [0xfffffff007b88480, 0xfffffff007bad79c, 0xfffffff007bad7a4, 0xfffffff007c374e4, 0xfffffff007c5a2d8, 0xfffffff007c80d04, 0xfffffff007c811cc, 0xfffffff007de9a6c, ...]
[ ] 62 ld1r [0xfffffff007c50368, 0xfffffff007c50c1c, 0xfffffff007c5e8ec, 0xfffffff007c828f8, 0xfffffff0082941d8, 0xfffffff008295750, 0xfffffff0082ace00, 0xfffffff00833e5e4, ...]
[ ] 46 fminnm [0xfffffff008f37ec4, 0xfffffff008f420dc, 0xfffffff008f420f8, 0xfffffff008f42110, 0xfffffff008f42134, 0xfffffff008f421a8, 0xfffffff008f49350, 0xfffffff008f5a560, ...]
[ ] 38 tlbi [0xfffffff007b6e888, 0xfffffff007b6efd8, 0xfffffff007c84d64, 0xfffffff007c84e4c, 0xfffffff007c84e64, ...]
[ ] 37 fneg [0xfffffff008cf6954, 0xfffffff008f59a18, 0xfffffff008f59aa8, 0xfffffff008f64f38, 0xfffffff008f64f4c, 0xfffffff008f64f68, 0xfffffff008f67e70, 0xfffffff008f67ea8, ...]
[ ] 36 fmaxnm [0xfffffff008f7390c, 0xfffffff008f7c160, 0xfffffff00918f7f0, 0xfffffff00918f810, 0xfffffff00918fb58, 0xfffffff00918fb8c, 0xfffffff00918fb9c, 0xfffffff00918fa60, ...]
[ ] 35 caspal [0xfffffff009196660, 0xfffffff0091975e0, 0xfffffff009196fc4, 0xfffffff009196e40, 0xfffffff009196b90, 0xfffffff009197180, 0xfffffff00919807c, 0xfffffff0091985b8, ...]
[ ] 32 sxtl [0xfffffff007bcb3f8, 0xfffffff007bdcea0, 0xfffffff007c7a56c, 0xfffffff007d128a0, 0xfffffff007d12918, ...]
[ ] 25 ldseth [0xfffffff007bb95a8, 0xfffffff007bbbdf4, 0xfffffff007bbc7b8, 0xfffffff007d3ae30, 0xfffffff007d61bc4, 0xfffffff007d61c1c, 0xfffffff007f725fc, 0xfffffff007f74af8, ...]
[ ] 24 sys [0xfffffff007c84dfc, 0xfffffff007c84e6c, 0xfffffff0099d20e4, 0xfffffff0099d214c, 0xfffffff0099d25a4, ...]
[ ] 24 fcvtn [0xfffffff0083f01ac, 0xfffffff0083f01c4, 0xfffffff0083f81ec, 0xfffffff0083f8204, 0xfffffff008408fe4, ...]
[ ] 22 fmax [0xfffffff0084200d8, 0xfffffff008420114, 0xfffffff00842035c, 0xfffffff008420394, 0xfffffff0084205fc, 0xfffffff008420674, 0xfffffff00842f5dc, 0xfffffff0084344e8, ...]
[ ] 20 ldsetalh [0xfffffff007c89518, 0xfffffff007c89538, 0xfffffff007c89648, 0xfffffff007c89660, 0xfffffff007c8972c, ...]
[ ] 20 st2 [0xfffffff00958282c, 0xfffffff009582830, 0xfffffff0095a9fac, 0xfffffff0095a9fb4, 0xfffffff0095a9fb8, 0xfffffff0095a9dfc, 0xfffffff0095a9e08, 0xfffffff0095a9e18, ...]
[ ] 19 tbl [0xfffffff007c85c54, 0xfffffff007c85c68, 0xfffffff008c25018, 0xfffffff00946e270, 0xfffffff00946e274, 0xfffffff00946e278, 0xfffffff00946e27c, 0xfffffff0094aa8ac, ...]
[ ] 18 fcmeq [0xfffffff008eb7834, 0xfffffff008eb7b24, 0xfffffff008eb7f28, 0xfffffff008eb8218, 0xfffffff008eb862c, ...]
[ ] 14 ld4 [0xfffffff007b5f250, 0xfffffff007b5f254, 0xfffffff007b6ddec, 0xfffffff007b6ddf0, 0xfffffff009582850, ...]
[ ] 14 casp [0xfffffff007f721b4, 0xfffffff007f73a28, 0xfffffff007f73afc, 0xfffffff007f73e30, 0xfffffff007f74120, ...]
[ ] 13 ldclrh [0xfffffff007bbbe58, 0xfffffff007d3ae10, 0xfffffff007d626ec, 0xfffffff007d626f8, 0xfffffff007f9adf8, ...]
[ ] 13 ldclralh [0xfffffff007c865fc, 0xfffffff007c87f18, 0xfffffff007c89520, 0xfffffff007c89548, 0xfffffff007c895b0, ...]
[ ] 13 fcvtl [0xfffffff008f6a65c, 0xfffffff008f6a660, 0xfffffff008f6a8bc, 0xfffffff008f6a8d0, 0xfffffff008f6a8dc, ...]
[ ] 11 mova [0xfffffff008271d18, 0xfffffff0086708e0, 0xfffffff008c35ce0, 0xfffffff008dbb710, 0xfffffff008e99800, ...]
[ ] 11 fabs [0xfffffff0082c2094, 0xfffffff0082f722c, 0xfffffff0082f727c, 0xfffffff00841dacc, 0xfffffff0084205ec, ...]
[ ] 10 fmin [0xfffffff00841e1c4, 0xfffffff0084205f4, 0xfffffff008420670, 0xfffffff008434518, 0xfffffff008541558, ...]
[ ] 6 uaddl [0xfffffff00826fab8, 0xfffffff00826fb1c, 0xfffffff008dbb5d8, 0xfffffff008dbb63c, 0xfffffff0095b3598, ...]
[ ] 6 ngc [0xfffffff008312d50, 0xfffffff008312f2c, 0xfffffff008313040, 0xfffffff008313830, 0xfffffff008313a3c, ...]
[ ] 6 ld2r [0xfffffff00946df84, 0xfffffff00946dfc4, 0xfffffff00946e038, 0xfffffff00946e078, 0xfffffff00946e0f8, ...]
[ ] 5 ldsetal [0xfffffff007b6e414, 0xfffffff0094d1088, 0xfffffff0094d1090, 0xfffffff0099b3f7c, 0xfffffff0099e8c18]
[ ] 5 ldsetl [0xfffffff007b8b2ec, 0xfffffff007b8b370, 0xfffffff007b8b480, 0xfffffff007b8b5e8, 0xfffffff009947bd4]
[ ] 5 umull2 [0xfffffff00946dd3c, 0xfffffff00946dd54, 0xfffffff00946dd64, 0xfffffff00946dd7c, 0xfffffff00946ddac]
[ ] 4 at [0xfffffff00826daf0, 0xfffffff00826db28, 0xfffffff00826db58, 0xfffffff0099f29a8]
[ ] 4 raddhn [0xfffffff00946e0a8, 0xfffffff00946e0b0, 0xfffffff00946e18c, 0xfffffff00946e290]
[ ] 3 fnmul [0xfffffff008b565a8, 0xfffffff008b56908, 0xfffffff008cf6970]
[ ] 3 fcmgt [0xfffffff0091af36c, 0xfffffff0091b4d60, 0xfffffff009429fc4]
[ ] 3 fcmge [0xfffffff009443130, 0xfffffff0099235b8, 0xfffffff0099235dc]
[ ] 2 ldclral [0xfffffff007b6e348, 0xfffffff0099755f4]
[ ] 2 ldeor [0xfffffff007b8b0dc, 0xfffffff0080cadd4]
[ ] 2 ic [0xfffffff0082644fc, 0xfffffff00826453c]
[ ] 2 ngcs [0xfffffff008331964, 0xfffffff008337fac]
[ ] 2 smov [0xfffffff0084026b8, 0xfffffff0084026f8]
[ ] 2 faddp [0xfffffff008f7ed00, 0xfffffff00957518c]
[ ] 2 ldumax [0xfffffff009198924, 0xfffffff009198944]
[ ] 2 ldumaxb [0xfffffff0091acccc, 0xfffffff0091ad438]
[ ] 2 rshrn [0xfffffff00946df18, 0xfffffff00946e214]
[ ] 2 st4 [0xfffffff009582884, 0xfffffff009582888]
[ ] 1 hint [0xfffffff007b6e9a0]
[ ] 1 ldsetlh [0xfffffff007f96868]
[ ] 1 sxtl2 [0xfffffff0080526e8]
[ ] 1 ldclrb [0xfffffff0080caec0]
[ ] 1 ldsetb [0xfffffff0080caecc]
[ ] 1 ldeorb [0xfffffff0080caed8]
[ ] 1 ldeorh [0xfffffff0080caf20]
[ ] 1 sb [0xfffffff00826efd4]
[ ] 1 b.al [0xfffffff008341360]
[ ] 1 ldlar [0xfffffff008c04ee0]
[ ] 1 st1w [0xfffffff008c0a870]
[ ] 1 fabd [0xfffffff008c1cb38]
[ ] 1 sysl [0xfffffff008c1d074]
[ ] 1 ldsetalb [0xfffffff0094bee4c]

Converted the list to checkboxes so we can better track progress. Thanks for maintaining this! Hoping to make some progress on it now.

Thanks! I'm gonna update the table and include the version number each time I update this to make this easier to track.

Aww, then we can't see our progress. 😉 Yeah, that's totally fine. I do think even the current list we've gotten some on dev already which is why I converted with checkboxes. Just re-running and including the version is fine too and we can just ping you when to re-run it.

Aww, then we can't see our progress. 😉 Yeah, that's totally fine. I do think even the current list we've gotten some on dev already which is why I converted with checkboxes. Just re-running and including the version is fine too and we can just ping you when to re-run it.

fair enough, that saves me effort haha :)

Ok, as of
this is the current status of the instructions mentioned in this issue:
Some SIMD&FP instructions are a hybrid of intrinsics for vector variants and direct lifting for scalar variants.
System instructions and cache control or memory barrier instructions will remain unimplemented for now (but we could make up intrinsics for them, such as we have for rev and some others).
Load/Store instructions with specific acquire/release or other memory semantics are currently lifted as memory operations, but without any representation of the memory semantics. These could be represented similarly to the ILInstructionAttribute::SrcInstructionUsesPointerAuth attribute.
The LDn* and STn* instructions load and store structures from multiple vector registers, and would require some modifications to the intrinsic code generator.

x: already done

-: done in a stash

n: will not lift

/: can't figure out

i: intrinsic

[ ] [n] at: Address Translate (will not lift)

[x] b.al

[x] casp

[x] caspal

[x] dup

[x] [i] fabd

[x] fabs

[x] faddp

[x] fccmp

[x] [i] fcmeq

[x] [i] fcmge

[x] [i] fcmgt

[x] [i] fcvtl

[x] [i] fcvtn

[x] [i] fcvtzs

[x] [i] fcvtzu

[x] fdiv

[x] fmax

[x] [i] fmaxnm

[x] [i] fmin

[x] [i] fminnm

[x] [i] fmla

[x] [i] fmls

[x] fneg

[x] fnmul

[x] hint

[ ] [n] ic: Instruction Cache operation.

[ ] ld1r

[ ] ld2

[ ] ld2r

[ ] ld4

[ ] ldclr

[ ] ldclral

[ ] ldclralh

[ ] ldclrb

[ ] ldclrh

[ ] ldclrl

[ ] ldeor

[ ] ldeorb

[ ] ldeorh

[x] [i] ldlar

[ ] ldset

[ ] ldsetal

[ ] ldsetalb

[ ] ldsetalh

[ ] ldsetb

[ ] ldseth

[ ] ldsetl

[ ] ldsetlh

[ ] ldumax

[ ] ldumaxb

[ ] [n] mova: Not currently lifting SME instructions (no intrinsics available)

[x] ngc

[x] ngcs

[x] [i] raddhn

[x] [i] rshrn

[ ] [n] sb: "Speculation Barrier"

[x] scvtf

[x] smov

[ ] [n] st1w: Not currently lifting SVE instructions

[ ] st2

[ ] st4

[x] sxtl

[x] sxtl2

[ ] sys

[ ] sysl

[ ] tbl

[ ] [n] tlbi: TLB Invalidate operation (will not lift)

[x] [i] uaddl

[x] ucvtf

[x] [i] umull2

I updated the checklists above, and those changes are merged to dev with c3040ecfc43983af6f05da13cf2242d085b1e230, and should be reflected in builds >= 4.1.5669-dev.
I've made a best effort to lift all the instructions listed here that can be lifted, either directly or via intrinsics, with the possible exception of the ldset and other ld* atomic instructions.
(Note also that ld2 & st2 were already lifted to intrinsics, but I haven't really looked at them. All the (ld|st)[1-4] instructions could probably be handled the same way I handled tbl.)
Continuing to leave this issue open to track the remaining instructions listed.


Location based compass orientation is incorrect. It always says North is direction when starting up.

Bug. There are similar issue but all marked as closed, incorrectly.

What is the current behavior?
Currently on iOS (Safari and Chrome tested) the simple location examples will always point the north cube in the direction the phone is facing after any interaction or started.
If the current behavior is a bug, please provide the steps to reproduce.
Load the basic example
https://github.com/AR-js-org/AR.js/blob/master/aframe/examples/new-location-based/basic-js/index.html
On iOS at startup there will always be several permissions popups. After hitting them all you will see the Red cube (North) in the direction of whatever you started. If you tap the cube or any cube, you will get a popup and then the orientation will reset so that direction is North and the Red Cube is there.

It is always happening with the included example. I put a copy on website at www.pasotours.com/location2
Please mention other relevant information such as the browser version, Operating System and Device Name
iOS (Chrome and Safari) it works correctly on android.
What is the expected behavior?
If this is a feature request, what is motivation or use case for changing the behavior?

It looks to me like iOS no longer supports the compass deviceorientationabsolute event. It only supports the deviceorientation event and that seems to reset with the context in some way. In the basic example when you tap on a cube when it returns the device orientation resets with alpha = 0 regardless of the direction facing.
I believe on ios you now need to support the webkitcompassheading event to know which way the phone is facing absolutely.
https://developer.apple.com/documentation/webkitjs/deviceorientationevent/1804777-webkitcompassheading

It appears that this is know. There was a MR that was not merged that seems to help but enabling the webkitcompassheading. https://github.com/AR-js-org/AR.js/issues/466

We really need developers who have an iOS device, this will help progress the iOS issues and allow the merging of #466.
Also see #278 and #603.

For additional context: this appears to be a known problem in some Android devices, so much so that the DeviceOrientationControls object has been removed from the offical three.js repository.
See: https://github.com/mrdoob/three.js/issues/22613

Just an update on this: there is now a separate project for location-based AR within AR.js which is going to try to resolve, or find workarounds for at least (as they may be an inherent problem with the device), these kinds of bugs:
https://github.com/AR-js-org/locar.js


mep3_navigation neće da se build-uje u main brenču

Starting >>> mep3_navigation
--- stderr: mep3_navigation
CMake Error at CMakeLists.txt:23 (find_package):
By not providing "Findnav2_core.cmake" in CMAKE_MODULE_PATH this project
has asked CMake to find a package configuration file provided by
"nav2_core", but CMake did not find one.
Could not find a package configuration file provided by "nav2_core" with
any of the following names:
nav2_coreConfig.cmake
nav2_core-config.cmake

Add the installation prefix of "nav2_core" to CMAKE_PREFIX_PATH or set
"nav2_core_DIR" to a directory containing one of the above files.  If
"nav2_core" provides a separate development package or SDK, be sure it has
been installed.
make: *** [Makefile:326: cmake_check_build_system] Error 1
__

U pitanju je commit 1e4ca4ad3218528abc2a4589bdb2411fa7eeec11

Nisam video Issue ranije. Meni radi bez problema, pustano i na robotu.
Probao da reinstaliras nav2 pakete?
sudo apt install ros-galactic-navigation2
sudo apt install ros-galactic-nav2-core

Zatvaram issue, zastarelo.


Simpler use of Enum classes?

Hi.
Please, can somebody explain to me why the setup with all the enum classes is the way it is?
What are the advantages of having all the values stored as instances of the same class, with different IDs, over using plain and simple constants?
To see what I mean:
In SearchAdapter::getArticleNumberStatus() the value of $articleNumberSearchStatusEnum::IsArticleNumberResultFound() is returned, while on the counterpart of it (sSee index.php from the FACT-Finder Demo Project) we check against the same $articleNumberSearchStatusEnum::IsArticleNumberResultFound() when we want to know if anything was found.
Why don't we use a constant like ArticleNumberSearchStatus::ARTICLE_NUMBER_RESULT_FOUND and check against it? We could drop all the cumbersome code in these classes:

ArticleNumberSearchStatus
BreadCrumbType
FilterSelectionType
FilterType
SearchStatus

Thanks, Estigy

One reason for this design decision:
The library uses name spaces, thus the above const solution cannot be used the way you have described.
Furthermore the enum class describes a set of predefined values for a type much better.


Make using deno more egonomic

Currently if I want to use deno
import createPlugin from "https://raw.githubusercontent.com/extism/js-sdk/main/src/mod.ts";

I need to run like this deno run --import-map https://raw.githubusercontent.com/extism/js-sdk/main/deno.json deno_run.ts
You can see I had to specify the import map for the code to work
I think if this was published in deno.com/x it would have picked the deno.json file automatically.

Seems like even using deno.land have the same problem, I'll keep the issue open to improve the usage with deno , because being obligated to use that import map is not convenient at all

will look into this - thank you!


Fix DataLoader inspection and re-instantiation in Lite

What does this PR do?
Fixes #10329
A detailed explanation of why the bug occurs can be found in the linked issue.
A newly added test demonstrates the failure on master.
This fix should be reviewed carefully, at least by @carmocca and @tchaton .
Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
I made sure I had fun coding 🙃
Part of #1 (it's a lie, this is just here to avoid noisy GitHub bot)

@carmocca Do you think we should bring this to Lightning too ?

Is it the approach of how we handle dataloaders in general or the fix that I added?


Cannot set 0 outline (stroke thickness) width on Shapes -- the setStrokeStyle(0) function

I'm using the 0.8.2 version, but I've searched through the changelog and didn't see any changes that might deal with this problem.
UPDATE: In codepen linked below, I also used the 1.0.2 version of EaselJS.
The problem:
When I create any shape (Square in this case) and set its stroke thickness to 0, it still displays the stroke (outline) as it is 1px width. Of course, the workaround is not to set stroke at all, and it won't display any stroke at all, but the problem is that I had to initially set the stroke, so that I would have access to the "command" of it, for a later dynamic change of thickness, color, etc.
Here's the codepen - https://codepen.io/flackjap/pen/WzONgV?editors=1010
I tried using the endStroke() method, but it won't display any stroke on Shape if used, and it also messes up later with my command references for later use.
The workaround with not using the stroke at all (setStrokeStyle and beginStroke) seems ok, as I said, but since I need to change/add the stroke at some point later, I could eventually make it redraw the whole shape with setStrokeStyle and beginStroke if there wasn't any stroke, but also destroy the shape and redraw it again if user of my app decides to reduce thickness back to 0 again.
UPDATE2: There's another even simpler workaround. I found out that the subpixel thickness works by setting the alpha to existing 1px thickness. So, if I do setStrokeStyle(0.1) it will actually be as it's invisible, which completely fixes the problem. I will just set in my app that if the User sets thickness to 0, that it actually drops it to 0.1.

This uses CanvasRenderingContext2D.lineWidth under the hood, which can't be zero ( https://developer.mozilla.org/en-US/docs/Web/API/CanvasRenderingContext2D/lineWidth )

Oh, thanks @MannyC
Now I also see that the stroke is still visible with 0.1 value (tried editing the ctx.lineWidth on the page you linked), but the 0.01 or even 0.001 surely makes it invisible.

Oops, I should have said this first: you know you can just fill without stroking?

Yes, I know that :) and I explained in my initial description why I needed to set any stroke initially - because the User of my app will be able to change the stroke thickness at some point, so I thought that just setting the initial 0 thickness would work.


Remove usage of import_module in idom.run

Current Situation
There was a bug in idom.run because we failed to update an import after a name change.
Proposed Actions
We used import_module("idom.backend.default") presumably because it was syntactically convenient to write this. Instead we should just do an actual import:
if not implementation:  # pragma: no cover
    from idom.backend import default as implementation

Closing this in favor of #1113


cco:Agent is a subclass of bfo:'immaterial entity'

In the new release of the Agent Ontology, 'Agent' is a subclass of 'immaterial entity' and should not be.

I made a mistake


Update Pre/Post hooks example.

Updates our pre/post hook docs and extends it with an fsfreeze example.

This might also address #245

otherwise I think LGTM

Merging since there's an LGTM and the other comments appear to have been addressed.


Effects got promisified

@calebmer This PR is rough example how promisified effects may help us with server-side render. Further description in https://github.com/salsita/redux-side-effects/pull/9

I don't really like this approach, it breaks backwards compatibility and returns an unexpected object when coming from redux. I think the optimum solution would be to add a second dispatch-like function which returns the side effects.
And yeah, your right, non-promises will be automagically converted when using Promise.all 😊


Inline suggest preview should use replace range when available

It would be nice to have a setting to make the suggestion preview text replace the text to the right of the cursor, instead of the text to the right of the cursor moving onto the next line as I go though the suggestions.
https://user-images.githubusercontent.com/122867725/215074271-31dc0d40-578c-42ba-a4fd-35649d842d5f.mp4

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


[FEATURE] Add filter support for entity list operation

Describe the feature
We should add name filter support for entity list operations, like metalake, catalog, schema... list operations. So that we can support basic name filtering for these operations.
Motivation
With this, we can support basic name filtering for these list operations.
Describe the solution
No response
Additional context
No response

We need a new unified api and name filter support for all the entity list operations.


Bring back mono debug symbols

It was removed in the commit to re-create the nuget. This is useful for debugging/browsing into the library when I didn't compile it myself. Commit https://github.com/xyncro/chiron/commit/c3cc7e65a1533cd7001db276636241fc039897a9

Yup, this should definitely go back in, I'll fix that (or if you get there first, I'll accept a PR! ;) ) Only got removed as NuGet seemed to be being odd with file paths and path separators... (Grrrrr, NuGet)

6.3.0 enables SourceLink and includes the portable pdb files. That is the new format that works on mono too. This can be closed.


Firefox not running (headlessly or not) on docker with debian

What is your Test Scenario?

I'm using testcafe on a docker container which is based on debian. I use Xvfb to run a virtual screen in which i use testcafe.
What is the Current behavior?

Chrome has no problems at all, it runs with the --no-sandbox argument without problems. It even records and takes screenshots
Firefox on the other hand is not working properly. I tried running firefox on the Xvfb after using fluxbox on it. I tried even running firefox:headless without luck.
Tests just don't seem to open or run the web browser. I get to where every test should start and im getting the following error on each :
GeneralError: Unable to establish one or more of the specified browser connections. This can be caused by network issues or remote device failure.     at BrowserSet._waitConnectionsOpened (/var/lib/jenkins/workspace/XXXX/node_modules/testcafe/lib/runner/browser-set.js:63:30)     at /var/lib/jenkins/workspace/XXXX/node_modules/testcafe/lib/runner/browser-set.js:80:31     at Bootstrapper._getBrowserConnections (/var/lib/jenkins/workspace/XXXX/node_modules/testcafe/lib/runner/bootstrapper.js:63:16)     at Function.all (<anonymous>:null:null)     at Bootstrapper._bootstrapParallel (/var/lib/jenkins/workspace/XXXX/node_modules/testcafe/lib/runner/bootstrapper.js:168:39)     at Bootstrapper.createRunnableConfiguration (/var/lib/jenkins/workspace/XXXX/node_modules/testcafe/lib/runner/bootstrapper.js:184:69)  {   code: 'E1004',   data: [] }
What is the Expected behavior?

Firefox should run properly
Your Environment details:

testcafe version: 1.6.0
node.js version: 13.1.0
platform and version: Debian 10
other:

Hi @franciscolosardo
Could you please provide a docker image with which we can reproduce the problem?

Ill paste our dockerfile since it doesn't have any personal or business related info :
FROM debian:latest
ARG packageId

RUN echo "Installing Visual Interface (XVFB)"
RUN apt-get update > /dev/null && \
      apt-get -y install sudo > /dev/null

RUN apt-get -qq --assume-yes install procps > /dev/null
RUN apt-get -qq --assume-yes install wget > /dev/null
RUN apt-get -qq --assume-yes install xvfb xfonts-100dpi xfonts-75dpi xfonts-cyrillic xorg dbus-x11 > /dev/null
RUN apt-get -qq --assume-yes install curl > /dev/null
RUN apt-get -qq --assume-yes install fluxbox > /dev/null
RUN apt-get remove iceweasel
RUN apt-get --assume-yes install libgtk-3-0 libdbus-glib-1-2 libasound2 libgtk2.0-0

#RUN apk add --no-cache py-pip git openssh bash
RUN rm -rf node_modules
RUN rm -rf package-lock.json

RUN apt-get --assume-yes remove nodejs > /dev/null

RUN echo "Installing NODEJS"

RUN wget -qO- https://deb.nodesource.com/setup_13.x | bash -

RUN apt-get install -y nodejs build-essential  xvfb > /dev/null

RUN echo "Installing Chrome"

RUN wget https://dl.google.com/linux/direct/google-chrome-stable_current_amd64.deb > /dev/null
RUN apt-get -q --assume-yes install ./google-chrome-stable_current_amd64.deb > /dev/null

RUN apt-get --assume-yes install screen > /dev/null

RUN echo "Installing Firefox"

RUN apt-get --assume-yes install software-properties-common > /dev/null
RUN apt-get update > /dev/null
RUN apt-key adv --keyserver keyserver.ubuntu.com --recv-keys A6DCF7707EBC211F
RUN add-apt-repository "deb http://ppa.launchpad.net/ubuntu-mozilla-security/ppa/ubuntu bionic main"
RUN apt-get update > /dev/null
RUN apt-get --assume-yes install firefox > /dev/null

RUN echo "Cleaning NodeJS Cache"

RUN npm cache clean --force

RUN npm install -g testcafe  && \
 npm cache clean --force && \
 rm -rf /tmp/*

RUN apt-get --assume-yes install default-jdk > /dev/null

RUN npm install -g allure-commandline --save-dev

ARG CACHEBUST=1

RUN rm -rf screenshots
RUN rm -rf logs
RUN rm -rf videos
RUN mkdir screenshots
RUN chmod 777 screenshots
RUN rm -rf logs

RUN mkdir /tmp/.X11-unix
RUN chmod 1777 /tmp/.X11-unix
RUN chown root /tmp/.X11-unix/

I'll also add we tried with firefox-esr which is the usually distributed version of firefox on debian in docker

I'll close this issue in favor of https://github.com/DevExpress/testcafe/issues/4306
We will check this scenario after the https://github.com/DevExpress/testcafe/issues/4306 issue is fixed.

Just to make sure we both don't spend time testing concurrency, i want to insist that the problem isn't concurrency, as i tried to run just one instance of testcafe with just one test and that didn't work either.
Also, i created a virtual box machine with debian running and when trying the same commands we tried in the docker , the tests work fine.


hide the blue border

In the /definitions list (and elsewhere) there are cases where you click on an entry / element and its border is changed to a blue glow. See the screenshot below. I had just clicked on one of the entries in the list and the whole list starts to glow. This is distracting, exposes the component hierarchy (why not glow more? or less? of the DOM?), and does not guide the user.

Previously I have looked for ways to do this but for whatever reason never found success.

this is the focus outline and can be removed with outline: none on the grid control.
We should make sure this doesn't break navigating by keyboard when it is removed


Add support for mobile properties and screen calls

This is a follow-up from a PR on the old repo... reconfigured to use mobileTrackingId for calls from mobile libraries! See that PR for details on why this is important.
Functionally, firing:

... yields:

cc/ @amillet89 @ndhoule @segmentio/success

you are a badass
On Tue, Mar 31, 2015 at 12:53 PM, Chris Sperandio<EMAIL_ADDRESS>wrote:

This is a follow-up from a PR on the old repo
https://github.com/segmentio/integrations/pull/238... reconfigured to
use mobileTrackingId for calls from mobile libraries! See that PR for
details on why this is important.
Functionally, firing:
https://camo.githubusercontent.com/b5f6e9dfb4f535da6165bf4564ef489ef1d3b0df/68747470733a2f2f636c6f756475702e636f6d2f634f4b385f5a73684447752b
... yields:
https://camo.githubusercontent.com/892ff47433ad0100977e9e7e502518cf889a1931/68747470733a2f2f636c6f756475702e636f6d2f6358773763444a544c2d642b
cc/ @amillet89 https://github.com/amillet89 @ndhoule
https://github.com/ndhoule @segmentio/success
https://github.com/orgs/segmentio/teams/success
You can view, comment on, or merge this pull request online at:
https://github.com/segmentio/integration-google-analytics/pull/11
Commit Summary

add support for mobile properties and screen calls

File Changes

M lib/index.js
https://github.com/segmentio/integration-google-analytics/pull/11/files#diff-0
(13)
M lib/mapper.js
https://github.com/segmentio/integration-google-analytics/pull/11/files#diff-1
(52)
M lib/universal.js
https://github.com/segmentio/integration-google-analytics/pull/11/files#diff-2
(16)
A test/fixtures/screen-app.json
https://github.com/segmentio/integration-google-analytics/pull/11/files#diff-3
(30)
A test/fixtures/screen-basic.json
https://github.com/segmentio/integration-google-analytics/pull/11/files#diff-4
(20)
M test/index.js
https://github.com/segmentio/integration-google-analytics/pull/11/files#diff-5
(33)

Patch Links:

https://github.com/segmentio/integration-google-analytics/pull/11.patch
https://github.com/segmentio/integration-google-analytics/pull/11.diff
—
Reply to this email directly or view it on GitHub
https://github.com/segmentio/integration-google-analytics/pull/11.


[Bug] Could not connect because the maximum number of '1' dedicated administrator connections already exists

Environmental information

#### PowerShell version:

Name                           Value
----                           -----
PSVersion                      4.0
WSManStackVersion              3.0
SerializationVersion           <IP_ADDRESS>
CLRVersion                     4.0.30319.42000
BuildVersion                   6.3.9600.19170
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0}
PSRemotingProtocolVersion      2.2

#### dbatools Module version:

Name    : dbatools
Path    : C:\Windows\system32\WindowsPowerShell\v1.0\Modules\dbatools\dbatools.psd1
Version : 1.0.5

LCID             Name             DisplayName
----             ----             -----------
1046             pt-BR            Portuguese (Brazil)

SQL Server:

Microsoft SQL Server 2016 (SP2-CU12) (KB4536648) - 13.0.5698.0 (X64)   Feb 15 2020 01:47:30   Copyright (c) Microsoft Corporation  Enterprise Edition: Core-based Licensing (64-bit) on Windows Server 2012 R2 Datacenter 6.3 <X64> (Build 9600: ) (Hypervisor)

Report

When running Sync-DbaAvailabilityGroup -Primary $AGPrimary -AvailabilityGroup "$AGName" we get the following error:
Could not connect because the maximum number of '1' dedicated administrator connections already exists. Before a new connection can be made, the existing dedicated administrator connection must be dropped, either by logging off or ending the process. [CLIENT: <IP_ADDRESS>]

Host used

[X] powershell.exe
[ ] ISE
[ ] VS Code
[ ] Other (please specify)

Errors Received

Steps to Reproduce

Run Sync-DbaAvailabilityGroup
Check the error in Application events in Event Viewer

Expected Behavior
No errors.

Actual Behavior
Got two errors
Could not connect because the maximum number of '1' dedicated administrator connections already exists. Before a new connection can be made, the existing dedicated administrator connection must be dropped, either by logging off or ending the process. [CLIENT: <IP_ADDRESS>]

See #6372 for the root cause of this. Will close this here as a duplicate.


Updating common.xml message definitions to mavlink 2.0

This along with an update to the jMAVLib allows jMAVSim to receive mavlink 2.0 messages.
Note: For this to PR work, this pull request will need to be accepted and the resulting hash will need to be updated on this repo and committed.

@catch-twenty-two @clovett Are you interested to complete this?

Sure but what is to complete?  My pull request was merged, and this is just updating common.xml to the latest.  Do you need some testing perhaps?

@lovettchris It would be great to get jMAVSim working fully with MAVLink 2.0. I hacked it work, however that isn't a solution.
Do you know what else is missing to get it working ?

I see, well jMavSim had a version of mavlink 1 ported to Java, I guess it will need a full Java port of mavlink 2.  This would be some work then...

There is also the Java generator in pymavlink that got some mavlink 2 support now: https://github.com/mavlink/pymavlink/pull/3.
Not sure how much work it would be to port jMAVSim to use the generated Java mavlink implementation.


build.py -c closure: Show file name and line number of errors and warnings

Corresponds to the ticket http://trac.osgeo.org/openlayers/ticket/3519 with some adjustments.
The proposal is to change closure.py by the launcher jscompiler.py used in Closure Library. This launcher requires an array of filenames as input argument and returns the compressed code as a string. So it does not require the creation of temporary files.
NOTE: "jscompiler.py" is renamed as "tools/closure_library_jscompiler.py" and is at r1445
This pull request replaces the #49 that had many commits.
Tested on: Windows XP+python2.4, Vista+python2.7 and Ubuntu10+python2.6

I confirmed that building with and without Closure compiler works on OSX as well.  Also confirmed that line numbers are shown with failures.  Thanks for the work on this.

I'm really excited, my first proposals in Closure are from a year ago, I no longer thought that this was accepted.
Thank you very much.


Deprecated syntax

Current currying func declaration syntax will be deprecated -- https://github.com/apple/swift-evolution/blob/master/proposals/0002-remove-currying.md

So? Will you continue work on this lib? Or you will stop support it, when swift 3 will be released?

We already support the supported syntax for swift 3. We're not going anywhere.


Improve User Guidance for Linked Local Accounts

Describe the issue

The current hint provided under "Validate linked local accounts" under "Linked Accounts" within the "User Attributes" tab in an application in the console lacks clarity regarding its purpose.
Users may not readily understand that this option needs to be selected to link the local user account created in scenarios such as federated logins through JIT provisioning.

The hint could be enhanced to clearly convey that enabling this option is necessary to link the local user account created during federated logins via JIT user provisioning.

How to reproduce

Create a new connection (google) and add JIT user provisioning.
Create a new standard based application and enable app native authentication API.
Configure the application by giving the redirect URL, allowed origins, and enable code grant type.
Use the API based authentication flow to login using the added federated login through the redirect url.
Call scim2/me endpoint by setting the authorization header using the access token.

Environment

IS 7.0

Additional context

Failure to select this option results in the linked account not being associated with the one created through JIT user provisioning. Consequently, users will encounter difficulties accessing "scim2/me" endpoints, leading to a 404 error.

We can also change the default state of this check box to true, as most users will require to linked the account with the JIT provisioned user.


feat: cowswap tx parser

Description

[x] can parse cowswap txs
[x] uses dynamic dex name for order route from parsed tx
[x] dex names have been updated to user facing brand values, e.g. thor is now THORChain

TODO

[ ] requires unchained-client package bump to consume new parser from https://github.com/shapeshift/lib/pull/909
[ ] how to render fees @shapeshift/product

Notice

[x] Have you followed the guidelines in our Contributing guide?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[x] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #2218
Risk

0x trades are interpreted as cowswap trades
Testing

enable cowswap flag
do a cowswap trade (any erc20 -> erc20) will route via cowswap by default
ensure that trade renders as per screenshot

Screenshots (if applicable)
before

after

I tested this one out with the matching lib PR, unfortunately, that's not rendering as it should, I'm seeing the exact same compared to develop, the reason for that being this cheeky switch statement where we fall back to the default branch:
https://github.com/shapeshift/web/blob/5f41bc1a6aa777fe9d90d602dfa707eb0e441854/src/components/TransactionHistoryRows/TransactionRow.tsx#L57-L68
The reason for that is txDetails.type is Trade while the value of TradeType.Trade is trade, see:
https://github.com/shapeshift/lib/blob/7b4e7565f15707d494c2c7ddfd499113f53fc04b/packages/chain-adapters/src/types.ts#L120
Piggybacking to your comment on the lib PR, we have a duplicate TradeType enum on chain-adapters (that we use in that switch statement), and unchained-client (that we use in the parser as the trade type), both using a different casing.
We should consolidate them into a single type.
For the time being, using the chain-adapters version in the lib PR will fix the issue for Cowswap, but note that ZRX and Thor parsers are affected by the same issue so they should also use the version with the right casing.

Yes, the casing was changed in the lib PR, it works as it was, but I agree we should consolidate these types now that unchained-client is in lib. See: https://github.com/shapeshift/lib/pull/909/files#r930077009


way for bots to have cool down periods (dynamic tempature adjusts) & only direct reply setting

to add for for .env
#DISCORD | only respond when @ | cooldown on reply in MS
DISCORD_DIRECT_ONLY=true #true
DISCORD_COOLDOWN=0 #default 1 min (only applies when not direct @)

to add for messages.ts discord
async handleMessage(message: DiscordMessage) {
        if (
            message.interaction ||
            message.author.id ===
                this.client.user?.id /* || message.author?.bot*/
        )
            return;
    //only respond too direct @ or replys
    const isDirectMention = message.mentions.has(this.client.user?.id);
    const isReplyToBot = message.reference?.messageId && (
        await message.channel.messages.fetch(message.reference.messageId)
    ).author.id === this.client.user?.id;
    const isDirectInteraction = isDirectMention || isReplyToBot;
    if (settings.DISCORD_DIRECT_ONLY && !isDirectInteraction) {
        console.log("Direct-only mode: ignoring non-mention message");
        return;
    }
        if (isReplyToBot) {
        console.log("Reply to bot detected");
    }

    // Only apply cooldown check for non-direct mentions
    if (!isDirectInteraction) {
        const timeSinceLastResponse = Date.now() - this.lastResponseTime;

        if (timeSinceLastResponse < this.COOLDOWN_MS) {
            console.log(`Cooling down for non-direct messages. Time remaining: ${(this.COOLDOWN_MS - timeSinceLastResponse)/1000}s`);
            return;
        }

    } else {
        console.log("Direct mention detected - bypassing cooldown");
    }

.....
                await this.runtime.processActions(
                    memory,
                    responseMessages,
                    state,
                    callback
                );
                this.lastResponseTime = Date.now();

            }

Does something like this exist for X/twitter too? It would be cool to be able to set the agent's to be more "choose-y" and not just reply to everything that's specified all the time.


Led Upstream Sync

Use common types everywhere.
This commit removed LedAction and LedMap, instead
defined ActionSet and GroupMap and used it all the places.
Test:
Going on...
Change-Id: Id6473da0bc59b6a3a9f55b7f3cacb7f856dd89ae

Pushed new patch set - https://github.com/ibm-openbmc/phosphor-led-manager/pull/53


Sentinel1 timeseries - basic

Notebook is for example / test functionality.
Will look at timelapse later on.
I suppose you already tried to translate timeseries directely into timelapse clientside and not feasible?

Looks great! Many thanks for your contribution.


Web of Science logo

Have you checked that your feature request isn't already filed?

[X] I read through FAQ and searched through the past issues, none of which addressed my feature request.
[X] Yes, I have checked that this feature request isn't already filed.

Description & Motivation
Add Web of Science logo to social media
Pitch
No response
Alternatives
No response
Additional context
No response

Web of Science uses the clarivate logo, which is already in academicons. If you add this to the code, would you mind sending a PR?

Hi! I’d like to work on this issue and add the Web of Science logo to the social media links. Let me know if there are any additional requirements or suggestions. Thanks!

You can just follow the same solution we already have for other socials.


Node.js bindings

Create a new folder /bindings/js, which contains Node wrappers for Arbiter's C API.

Any guidance on how best to approach this? I've glanced at the Swift bindings, but would appreciate some advice on what to do first and how to progress.
I'd like to add Haskell bindings but think this is probably a good stepping stone for me to get used to the API with something I'm more familiar with.

I suppose I could try a similar progression as https://github.com/jspahrsummers/Arbiter/pull/35 unless there's a more logical sequence of steps.

Awesome, @paulyoung! There's no particular progression of steps, really—I just started with those headers that have the fewest cross-dependencies and built up from there. ☺


ele property should use querySelectorAll

ele property takes String values for use in querySelector, but should use querySelectorAll for this definition works:
VirtualSelect.init({ ele: 'select' });

(all select tags on page)
ref: https://github.com/sa-si-dev/virtual-select/blob/729a1aec380816f67a9aef96fc6a1161a65e2548/src/virtual-select.js#L2577

@bes-internal thanks for letting me know it. I will fix this issue in the future release.

@bes-internal this issue has been fixed in the release v1.0.23


nz-slider init value/set value not effect in component

Version
0.7.1
Environment
ubuntu 17.10, chrome 63.0.3239.108, ng-zorro 0.7.1
Reproduction link
https://stackblitz.com/edit/ng-zorro-antd-setup-slc7aw
Steps to reproduce
add nz-slider in form. for more details please check stacklitz link
What is expected?
nz-slider work in form, init value/set value effect in component
What is actually happening?
nz-slider not work in form, init value/set value not effect in component
Other?

@yinjs I've tested it as well, and, yes, you are right. An initial value is not rendered properly in the range component.
But the setting of new value works fine. In your example, it doesn't work because you have a typo in the name of init lifecycle method. Your example has ngInit instead of ngOnInit.
Here is StackBlitz that I've used to test:
https://stackblitz.com/edit/ng-zorro-antd-setup-s8g8m8


Can this module be used in HTTP?

@wevkovpower4 - This is for module issues mainly. Please refrain from asking general questions in this section. The module is HTTPS.


Pagination in Vanilla JS

Is your feature request related to a problem?
No, It's not a problem. I am trying to include a new feature.
Describe your feature
Pagination is the process of separating print or digital content into discrete pages.
For print documents and some online content, pagination also refers to the automated process of adding consecutive numbers to identify the sequential order of pages.
Online articles are sometimes paginated to break up content. There is some justification to organizing long articles into multiple pages to aid the reader.
Therefore, I would like to add a simple project called "Pagination" using HTML, CSS and vanilla JS.

Would be delighted if you consider my submission for this issue @Vishal-raj-1  @urvashi-code1255 @991rajat.
Thank You :)

@shubhamjha25  Always get assigned first before making PR, It helps us to manage easily.

Thanks, I will keep this in mind for sure.


Picture

Good day
How do I display a picture or an icon on the notification?

https://developer.mozilla.org/en-US/docs/Web/API/notification


Add progress tracker toolcard

Summary

Added a new toolcard in the tools section showing progress for college, major, minor, and grad school requirements

This pull request is the first step towards finishing the tools page.
Remaining TODOs:

Find any outstandings bugs with the progress toolcard

Test Plan

Test that the progress bar moves according to the number of requirements fulfilled
Test that the text changes correctly with the number of requirements fulfilled and choice of major, minor, college, etc.
Test that the text does not show up when a minor or grad school were not chosen

we should probably handle decimals a little cleaner

Why did we move away from the design (shown below) for the righthand side of the tool card? Is this design out of date? I think it looks better to show what the user has left to do as opposed to a fraction.

I'd really prefer to reuse the functions in RequirementsHeader rather than trying to rework everything again. Redoing it will make it impossible to handle dangerous stuff in the future. (discussed w @zachary0kent offline)

This branch is terminated. We have pivoted to progress-tracker.


chore: unnamespace Matrix.dotProduct

Moves:

Matrix.dotProduct -> dotProduct
Matrix.dotProductᵣ -> dotProductᵣ
The notation ⬝ᵥ will no longer be scoped.
Lemmas about dotProduct that don't talk about matrices will be unnamespaced accordingly

Discussion:
https://leanprover.zulipchat.com/#narrow/channel/287929-mathlib4/topic/Matrix.2EdotProduct.20rename.3F

Unfortunately
@[deprecated (since := "2024-12-12")] alias Matrix.dotProduct := dotProduct

messes up with all files that open the Matrix namespace.
Can I simply remove it and leave no deprecation warning?

@madvorak, does deprecating the lemmas work, even if it doesn't for the main definition?

Hadn't seen your comment above. Something you could do is make the deprecated lemmas protected (though that of course makes the deprecations much less useful).

bors r+


CRUD endpoints for Tags

[ ] fetch
[ ] create
[ ] update
[ ] replace
[ ] destroy

this is actually unnecessary as a Tag will never be created unless attached to a PullQuote


Querier error messages: level=error ts=2021-01-20T23:36:23 caller=frontend_processor.go:61 msg="error processing requests" address=172.20.x.x:9095 err="rpc error: code = Unavailable desc = transport is closing"

Doesn't matter if I point the querier at frontend or frontend headless...
level=error ts=2021-01-20T23:36:23.059978229Z caller=frontend_processor.go:61 msg="error processing requests" address=172.20.19
4.35:9095 err="rpc error: code = Unavailable desc = transport is closing"

https://github.com/cortexproject/cortex/issues/3606#issuecomment-745179803

https://github.com/cortexproject/cortex/issues/3606#issuecomment-745179803


PR template: Fix broken link to contributing.md

The link is broken on actual PR pages.
Not using relative link, as it would be github-specific anyway, and the relative link would be broken on some other pages.
Pull request title

[ ] I read contributing guideline
[x] I didn't find a similar pull request already open.
[x] My PR is related to Spleeter only, not a derivative product (such as Webapplication, or GUI provided by others)

Description
I didn't read the contribution guidelines.
How this patch was tested
You tested it, right?
I clicked some links.

@ttepatti feel free to improve on grammar in the templates and other docs in the repo.

Thanks for the fix


Players without a team cannot .pause

By design players must join a team with .team a/b
Bit it's not needed that every player joins a team, it's enough that one player per team (leader/captain) joins a team.
If the game is then in progress others players cannot .pause.
This might lead to confusion/trouble during the match if some players get dropped from the game.
Possible ideas:

Enforce that every player must join a team before starting the match
Allow some commands to be executed without team assignment
Guess the team of the player (If T player writes something, look all T players if one has a team assigned) [not sure this is possible]

Pause command can now be executed without a team assignment: 3f6c909832162297a1f08827ac9a84e1b0b96d3e


FIX: sample(frac=...) on SQL.

Have to add a from_obj clause inside the sub-select.
@sandhujasmine please try this against your datasets.

@kwmsmith - It doesn't appear to be subsampling now. See below.
In [7]: odo(compute(sub_parts.nrows, prsx), int)
Out[7]: 40040

In [8]: odo(compute(sub_parts.sample(frac=0.75).nrows, prsx), int)
Out[8]: 40040

Also added a failing test in blaze.

Closing, as this is subsumed by #1452.


fix and remove hardcoded wait after base instance creation

Currently, a "wait" of 60 seconds has been added after base instance creation, so that the instance can download required data before ami creation takes place.
We need to replace this hard-coded wait with a mechanism that makes sure that all required files have been successfully downloaded and it is now safe to start ami creation.
Possible solutions:
1- ssh into the instance after its created and wait till the files don't appear?  How many files will we check?

Fixed


Missing tag for v0.14

v0.14 was released quite a while ago, but the source code for it was never tagged. Could you still add that tag? It makes packaging the software slightly nicer.

Sorry about that. I haven't had a chance to fix the binary packaging. It was originally using travis-ci, and I need to convert it over to github actions


Add new game mode (speed shoot)

(From Target-1 Akiba)
Adds new game mode "Speed Shoot" as follows:

Game start
Turn ON the LED of all targets
LED will be turned off if an user shoots the target
Game will end when the user finishes shooting all targets

Implemented by commit 818bf63 (client) and commit https://github.com/katsuster/target-6-server/commit/d61d751a9af75e8ce5444bdb856726323bc939c1 (server).


undefined method `win_friendly_path' for #<#Class:0x0000000004220908:0x0000000007eb5cd8>

Cookbook version
5.1.3
Chef-client version
14.5.33
Platform Details
System Info:
chef_version=14.5.33
platform=windows
platform_version=10.0.14393
ruby=ruby 2.5.1p57 (2018-03-29 revision 63029) [x64-mingw32]
program_name=C:/opscode/chef/bin/chef-client
executable=C:/opscode/chef/bin/chef-client
Scenario:
trying to acl_add an existing certificate.
Steps to Reproduce:
windows_certificate 'C:\temp\star_bowl_com.pfx' do
pfx_password vault['star_bowl_com']
private_key_acl ['NT AUTHORITY\NETWORK SERVICE']
action [:create, :acl_add]
end
The create step is skipping due to 'up to date' which is correct
the acl_add step is failing due to 'undefined method `win_friendly_path' for #<#Class:0x0000000004220908:0x0000000007eb5cd8>'
Expected Result:
acl_add should also skip due to being 'up to date'
Actual Result:
chef-client  fails

https://github.com/chef-cookbooks/windows/commit/35baf2085fb6a5f2e7977d5bbf203f14860c6a56 breaks windows_certificate resource.

This was fixed by https://github.com/chef-cookbooks/windows/pull/589 and that was released in 5.1.6


Fix / password button alignment and infinite scroller

Description
We've completed some rework, resulting in the following 2 fixes:
Changes

fix(project): alignment of password toggle button

Fixes the alignment op the password toggle button. We adjust the width and height for the controls in the Input component if its either a div (e.g. prevalent in the credit card form) or button (e.g. prevalent in the shelf)

chore(project): default infinitescroll to not use window

This change will make the infinite scroller listen to browser events initiated from the parentNode instead of the window. This will make the infinite scroller more adaptable when used in different layouts.

Steps completed:

According to our definition of done, I have completed the following steps:

[x] Acceptance criteria met
[ ] Unit tests added
[ ] Docs updated (including config and env variables)
[ ] Translations added
[x] UX tested
[x] Browsers / platforms tested
[x] Rebased & ready to merge without conflicts
[x] Reviewed own code

Submitted to OS: https://github.com/jwplayer/ott-web-app/pull/526


ensure kubelet --node-ip is set correctly when joining a cluster

Summary

References #2402
Closes #3184
Changes

Make sure to set the --node-ip kubelet flag properly when joining a cluster where the flag is set.

Testing

Install MicroK8s on a node, then configure --node-ip kubelet argument
2a. Install MicroK8s on a second node, then join as a control plane or worker-only node
2b. Install MicroK8s on a second node, disable HA-cluster, join node.

In all cases, the --node-ip of the joining node should be updated and set to the correct IP address.
Possible Regressions

This only affects clusters where the --node-ip argument has been explicitly set by the administrator. It does not have implications by default.
Checklist

[X] Read the contributions page.
[X] Submitted the CLA form, if you are a first time contributor.
[X] The introduced changes are covered by unit and/or integration tests.

Notes

This needs to be backported to all supported tracks

Although this fixes the issue with the mismatching node-ip, the ingress controller will not work on the newly joined worker node and will be in a Pending state.
This means that ingress is accessible from the controller node public ip, but not from the worker nodes, FWIW.
To replicate this, assuming that I have:

kube-controller at <IP_ADDRESS>
worker-node at <IP_ADDRESS>

kube-controller
# Enable the needed addons
microk8s enable \
    hostpath-storage dns rbac \
    metrics-server dashboard ingress
microk8s enable registry --size 40Gi

# Update the `--node-ip` so that future workers use the correct interface
sudo echo --node-ip=<IP_ADDRESS> >> /var/snap/microk8s/current/args/kubelet
sudo snap restart microk8s

# Verify that the internal IP is being used with
microk8s.kubectl get nodes -o wide

# Prepare the join command
microk8s add-node

worker-node
microk8s join <IP_ADDRESS>:25000/some-very-long-string/shorter-string --worker

It's also worth mentioning that microk8s disable ingress will get stuck forever at this point. Also force terminating the controller ingress pod on worker-node causes it to be stuck in a terminating state.

Hi @usersina, which MicroK8s version are you using? I am not able to replicate the issue you are describing following these commands.
A pod that is stuck in Pending and then terminating is most likely due to a changed kubelet IP address, so the --node-ip must be playing into this, but I'm not sure how. Can you create an issue with more details? Thanks!

@neoaggelos, there is an open issue for it already where I documented what worked for me in addition to the versions I've used.


Fixes #32 - make path to cookieFile if it's not present

TL;DR, this ensures ~/jira.d is present, with 0755 perms.
If ~/jira.d isn't present, we can't write to the cookieFile, which
breaks CmdLogin. This is particularly an issue when using /etc/go-jira.yml
to get an entire team using go-jira easily :)
This fixes this by ensuring the cookieFile dir is present before
writing to it.

Hey @coryb -- this fixes a blocker for me announcing go-jira and go-jira-ui to the wider engineering team here, could you merge and cut a new release?
I guess the release should be 0.1.0 due to the change to the CmdWatch interface aye?

Okay, I will bump the version and get a release out.  Thanks
-Cory

Created the new release:
https://github.com/Netflix-Skunkworks/go-jira/releases/tag/v0.1.0
With the vX.Y.Z tags you should be able to use gopkg.in instead of github for your import if you want to be more insulated from go-jira changes, so I think this will work:
gopkg.in/Netflix-Skunkworks/go-jira.v0.1
-Cory


Bug@Ionic 2 : nav.pop not work with ActionSheet and Alert

both ActionSheet and Alert handler not work with this.nav.pop()
constructor(nav: NavController) {
  this.nav = nav;
}

presentActionSheet() {
  let actionSheet = ActionSheet.create({
  title: 'Leave this page',
  buttons: [
    {
       text: 'Destructive',
       style: 'destructive',
       handler: () => {
           this.nav.pop() //pop not work here!
        }
   },
 {
     text: 'Cancel',
     style: 'cancel',
     handler: () => {
      console.log('Cancel clicked');
   }
  }
 ]
  });

  this.nav.present(actionSheet);
}

i have the same case that it can't use nav.push in actionsheet.
My solution is that use settimout to push in the button handler.

+1
The same issue here.

+1
Same issue, this.nav.pop() doesn't work inside of an ActionSheet.

One of the solution:
handler: () => {           this.nav.rootNav.pop().then(() => {                 this.nav.pop();             });               }               }

@kitkimwong Perfect that works great!
Plus, has anyone noticed that the ActionSheet with material design doesn't have colors? e.g. if you set the style to destructive it's still black not red like in iOS or in their docs!

this.nav.push() doesn't work as well within Alert handlers

Hello everyone! Thanks for opening an issue with us! The answer for this is to use promises. Our navigation system returns promises for all the nav methods (push(), pop(), present(), dismiss() etc etc) so the proper way to use nav.push() with an alert would be like this:
this.alert.dismiss().then(() => {
    this.nav.push(myAwesomePage);
})

You can also check out this link and this link to read more about using promises with navigation in Ionic 2.

Also, @callodacity material design does not color "destructive" actions red, so thats why the destructive attribute does not color the buttons in the actionsheet red. Hope that clears it up for you!

@jgw96 Thank you very much - I don't have an Android Device so I just assumed! :)

@callodacity No problem! Thanks for using Ionic!

@jgw96 I've tried with then promise but still no luck :(
loading.dismiss().then(() => {
    this.nav.pop();
});

but this is work
loading.dismiss().then(() => {
    setTimeout(() => this.nav.pop(), 500);
});

Hi Is there any progress on the issue? I am working on 2.0.0-beta.10 and it seems it still not rectified.
Am i right?

@tejit2004 this should be fixed in next beta! we refactored the whole overlay system (alert, action sheet, modals) so it is not longer coupled with the navigation stack.
Please reopen this issue once the beta11 is released and you consider this issue was not fixed :)

Thanks, that will be handy.
Is there any timeframe when beta11 will be released? as I am ready with my app to release on the stores, this bug prevents me. Is nightly code available which i can use?
Anyway, Thanks and it's relieving that these are out of the navigation stacks.

@tejit2004 There is a bleeding edge nightly of Ionic 2 available, which you can install with npm install ionic-angular@nightly but i would not recommend using it in a production app as it is a nightly and is not considered stable for the next release yet. You can be assured that we are working hard on getting the next release out, but we do not have a solid timeline on when it will be released. Thanks for using Ionic!

Am I correct that this issue is also to do with Alert buttons not working after opening Alert in an ActionSheet in Beta 10? I managed to fix this issue by asynchronously dismissing the ActionSheet before creating the Alert, which seems to be the solution in this issue.

@kz with the overlay component refactor that is coming in beta.11 you will not have to always dismiss the actionsheet before opening an alert. Hope that answers you question!

Still having issues on pop (tested with loadingController). Had to had a timeout value, that is not ideal.

Here is a snippet of my code. I am stuck at this for 2 days, my alert never gets dismissed :(
` let alert = this.alertCtrl.create({
title:'Oops',
message: error.message,
buttons: [
{
text: "Ok",
          handler:()=>{

            console.log('handler called');

            alert.dismiss().catch((err)=>{
              console.log(err);

            }).then(()=>{

            })

             return false;

          }
        }
      ]
    });

    alert.present().catch((err)=>{
     console.log(err);

    });

  });`

BTW guys, you should probably reopen this. When I import AlertController into my constructor for a component, the NavController.pop breaks. I had to resort to this:
this.nav.pop().then(()=>{this.nav.pop()}) to pop my page off.
I did not even place my this.nav.pop() inside the AlertController. I simply imported it, place it in constructor, and none of my other NavController actions work.

I resolved this issue by copying a class member reference to my NavController to the local function scope.  snippet is below. local variable needed for some of the promise/callback magic that happens with javascript.
openContactForm(recType) {
let newRecFields = this.modalCtrl.create(AddContact, {recType});
let localNav = this.navCtrl;
newRecFields.onDidDismiss(data => {
// Save the new record
if(Object.keys(data).length > 0) {
Meteor.call('axPutContact', this.id, data, function(error, result) {
if(typeof error !== 'undefined') {
alert(Posting error '${error}'.);
} else {
localNav.pop();
}
});
}
});
newRecFields.present();
}


kodi-retroarch-advanced-launchers: take lib arg

This package was changed to refer to lib instead of stdenv.lib, but
lib was not actually in scope. I don't know what this package is about
or how evaluation got to that point, but this is clearly more correct.
Motivation for this change
nix-env -qasP errored out with a non-empty retroarch attrset in my nixpkgs config
Things done
[x] nix-env -qasP doesn't error out anymore. i didn't actually build anything

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

LGTM. I assume this was missed in the treewide stdenv.lib -> lib PR for the same reason described here.

Yeah. I'm mostly confused because I didn't understand the mechanism that prevents it from being evaluated and why it then did get evaluated for me.

Thanks!


Cattle upgrade and related files

@sangeethah : Cattle upgrade files. Please review. Tests have passed.

@soumyalj , LGTM.
Can you add a flag to make these upgrade tests be skipped unless parameter "UPRGADE_TESTING" is available?

@sangeethah : Added the flag UPGRADE_TESTING.

LGTM


Add notification email script to the freezing code.

Need to split up the freezing, so that for each dataset added there is an email that goes out through the SMTP server.

Note that this required the use of the sendmailR package, which also required the addition of rJava.
rJava requires the use of the Java-sdk.  Instructions here: https://www.r-bloggers.com/installing-rjava-on-ubuntu/
One point was that the package itself needed to be installed by the superuser, otherwise installation fails.

Hmm, installed it with sudo, but cannot run it in R without sudo. . .
Trying this: https://anhqle.github.io/rstudio-cannot-load-rjava-in-ubuntu/

Seems that the mail program for centOS is going to provide the most useful interaction.  Will set this up as a bash script from my account.


PROBLEM ON LINUX DEBIAN

If I run locally it works on Windows but just simple example with text hello world and it crashes always on document.Save
Unhandled exception. System.ArgumentNullException: Value cannot be null. (Parameter 'key')

@BorisDmv
We apologize for the delayed response. Would you kindly try now with the latest version of Aspose.PDF.Drawing package instead of Aspose.PDF. You can simply install it through NuGet package manager and run the project. In case issue still persists, please let us know.

Since there have been no updates or additional questions for more than half a year, the issue is considered resolved.


Template Adoption with TM 2016 Tool

Like to understand if there are particular steps to adopt this template for the new and old TM files. If I try to open these new tb7 files in TM 2016 as a template or just pick 'Azure Cloud Service.tb7', it throws deserialization error. VSCode hasn't pointed out any XML document errors either.

The TB7 files are templates for the Threat Modeling Tool, which allow you to edit the Stencils, Threats and Threat Properties to be used when creating a new TM7 file, which contains the threat model itself.
Because the TB7 file used to create a TM7 file is used to populate parts of that TM7 file, there's no way to take a new/modified TB7 file and apply it to an existing TM7 file. I'm able to download and then open these template files in the 2016 and Preview versions of the Threat Modelling Tool. I can even create a new Model with the stencils showing for the new Template, indicating that it's been successfully loaded.

This is likely because you have saved a HTML file from GIT repo instead of .tb7


Caffe->Pytorch load model eval() failed

Platform (like ubuntu 16.04/win10): ubuntu 16.04
Python version: python 3.6
Source framework with version (like Tensorflow 1.4.1 with GPU): caffe GPU
Destination framework with version (like CNTK 2.3 with GPU): pytorch 1.0.0
Pre-trained model path (webpath or webdisk path):  resnet50 with private dataset
Running scripts:
File "train.py", line 44, in init
the_model.eval()
File "/home/wuziheng/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/nn/modules/module.py", line 1000, in eval
return self.train(False)
File "/home/wuziheng/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/nn/modules/module.py", line 988, in train
for module in self.children():
File "/home/wuziheng/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/nn/modules/module.py", line 888, in children
for name, module in self.named_children():
File "/home/wuziheng/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/nn/modules/module.py", line 906, in named_children
for name, module in self._modules.items():
File "/home/wuziheng/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/nn/modules/module.py", line 535, in getattr
type(self).name, name))
AttributeError: 'KitModel' object has no attribute '_modules'
according to https://github.com/Microsoft/MMdnn/blob/master/mmdnn/conversion/pytorch/README.md ,I transfer a caffemodel to pytorch model, which use python2, and furthur we need use this model in python3.6 pytorch, i have already fixed torch.load with encoding, and faided at the_model.eval() step. help!

Hi @wuziheng , Maybe you can downgrade pytorch to 0.4.0 to have a try. Thanks!

Hello,
First python2 is (or is about to be) deprecated so I really would not advise to use it anymore.
Regarding your issue, the stack trace says that during your call to the eval() function, pytorch is unable to load the children modules of your network because it is missing the attribute _modules. So you should look in the python file generated by MMdnn. Make sure that the definition and initialization of the KitModel class inherits from the module class:
class KitModel(nn.Module):
    def __init__(self, weight_file):
        super(KitModel, self).__init__()

Both of the above should solve it. If not try a more simple model such as VGG and see if it works.

Side note: MMdnn and caffe work fine with python3. You may need to recompile caffe for python make pycaffe.


Add ability to set frontend name for Docker provider

Signed-off-by: Stuart Clark<EMAIL_ADDRESS>

Hi @stuart-c, thanks for your contribution.
There is already a proposal to refactor the labels https://github.com/containous/traefik/issues/444#issuecomment-228050001.
It would also solve naming frontends.
WDYT?

The proposal on #444 does look like a better general update. Is anyone actually working on that? If no is it worthwhile merging this change in the meantime?

@stuart-c We are not currently working on #444 but this will be done soon. I prefer not to refactor labels 2 times in 1 month or we gonna lose everybody ;)
Sorry for that, but I'm closing this one. I hope you will understand and not take it personally.
Thanks anyway for your contribution :)


[Update] Added repo url for github source

Codecov Report

Merging #63 into master will decrease coverage by 0.56%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master      #63      +/-   ##
==========================================
- Coverage   73.86%   73.30%   -0.57%
==========================================
  Files           2        2
  Lines         199      206       +7
==========================================
+ Hits          147      151       +4
- Misses         52       55       +3

Flag
Coverage Δ

#pytests
73.30% <66.66%> (-0.57%)
:arrow_down:

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

Impacted Files
Coverage Δ

quantecon_book_theme/__init__.py
69.61% <66.66%> (-0.51%)
:arrow_down:

Continue to review full report at Codecov.

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

Tested the links in the chapters, looks good. Merging this one.