Exceptions thrown after starting a trip can't be caught

When starting a trip with a geofence tag that doesn't exist, the app crashes and we are unable to catch the exception.
See the following stack trace:
  at RadarIO.Xamarin.RadarSDKImpl+<>c__DisplayClass43_0.<StartTrip>b__0 (iOSBinding.RadarStatus status, iOSBinding.RadarTrip trip, iOSBinding.RadarEvent[] events) [0x00000] in /Users/cameron/Desktop/work/RadarIO.Xamarin/Library/RadarIO.Xamarin.iOS/RadarSDKImpl.cs:122
  at ObjCRuntime.Trampolines+SDRadarTripCompletionHandler.Invoke (System.IntPtr block, System.IntPtr arg0, System.IntPtr arg1, System.IntPtr arg2) [0x00016] in /Users/cameron/Desktop/work/RadarIO.Xamarin/Bindings/RadarIO.Xamarin.iOS.Binding/obj/Release/iOS/ObjCRuntime/Trampolines.g.cs:429
  at (wrapper native-to-managed) ObjCRuntime.Trampolines+SDRadarTripCompletionHandler.Invoke(intptr,intptr,intptr,intptr)
  at (wrapper managed-to-native) UIKit.UIApplication.UIApplicationMain(int,string[],intptr,intptr)
  at UIKit.UIApplication.Main (System.String[] args, System.Type principalClass, System.Type delegateClass) [0x0003b] in /Users/builder/azdo/_work/1/s/xamarin-macios/src/UIKit/UIApplication.cs:83
  at WB.Mobile.CX.Client.iOS.Application.Main (System.String[] args) [0x00001] in /Users/wy/Work/wbdigital/src/WB.Mobile.CX/WB.Mobile.CX.Client.iOS/Main.cs:12 ```

Reproduction code:
            try
            {
                Radar.StartTrip(new RadarTripOptions
                {
                    ExternalId = "999",
                    DestinationGeofenceTag = "tag",
                    DestinationGeofenceExternalId = "id",
                    Mode = RadarRouteMode.Car,
                    Metadata = new JSONObject
                        {
                            { "bing", "bong" },
                            { "ding", "dong" },
                            { "int", 10 },
                            { "bool", true }
                        }
                });
                Radar.StartTracking(Radar.TrackingOptionsContinuous);
            }
            catch (Exception ex)
            {
                Console.WriteLine("Exception: {0}", ex.Message);
            }


[AutoPR marketplaceordering/resource-manager] Add a directive for node and TS to delete format: "date-time".

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/2550

Codecov Report

Merging #2066 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2066   +/-   ##
=======================================
  Coverage   53.06%   53.06%
=======================================
  Files        4774     4774
  Lines      119483   119483
=======================================
  Hits        63406    63406
  Misses      56077    56077

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 5ecbcc0...29a3c2e. Read the comment docs.

NodeJS only PR


chore: update generate pdf scripts

First-time contributors' checklist

[x] I've signed Contributor License Agreement that's required for repo owners to accept my contribution.

What is changed, added, or deleted? (Required)
Since the structure of TOC.md has been changed, the parse method for generating PDF is failed. This PR is to fix the parse method.
And this issue caused by https://github.com/pingcap/docs-dm/pull/831

Which DM version(s) do your changes apply to? (Required)

[x] master (the latest development version)
[x] v5.3 (TiDB DM 5.3 versions)
[x] v2.0 (TiDB DM 2.0 versions)
[x] v1.0 (TiDB DM 1.0 versions)

What is the related PR or file link(s)?

This PR is translated from:
Other reference link(s):

Do your changes match any of the following descriptions?

[ ] Delete files
[ ] Change aliases
[ ] Need modification after applied to another branch
[ ] Might cause conflicts after applied to another branch

/merge

/remove-status LGT1
/status LGT2

/merge


Norwegian survey should not be used

The Norwegian bottom trawl survey that has been used so far in FishGlob has many errors and should not be used in any analysis until it has been replaced by the correct dataset.

@tomjwebb and @edwardlavender big warning here, we were recently notified by IMR about this, and we will replace the Norwegian survey dataset in the coming week(s)

Thanks for the heads up!

that might disappear with the new dataset because it's quite a different one

@LaurenePecuchet are we all good with the code or are there things I need to double check besides loading the data?

Dataset was replaced by the Barents Sea ecosystem survey available here for year 2021 http://metadata.nmdc.no/metadata-api/landingpage/3b4b5ee0f92a647642ffffdec39c8814 and see reports:

https://www.hi.no/en/hi/nettrapporter/imrpinro/2011/atlas_august_2011_press_quality_til_web
https://www.hi.no/en/hi/nettrapporter/rapport-fra-havforskningen-en-2021-15


Update test_type_string.py

This change is to make this repo compatible with Azure synapse varchar datatype
resolves #
This is a:

[ ] documentation update
[ ] bug fix with no breaking changes
[ ] new functionality
[ ] a breaking change

All pull requests from community contributors should target the main branch (default).
Description & motivation

Checklist

[ ] This code is associated with an Issue which has been triaged and accepted for development.
[ ] I have verified that these changes work locally on the following warehouses (Note: it's okay if you do not have access to all warehouses, this helps us understand what has been covered)

[ ] BigQuery
[ ] Postgres
[ ] Redshift
[ ] Snowflake

[ ] I followed guidelines to ensure that my changes will work on "non-core" adapters by:

[ ] dispatching any new macro(s) so non-core adapters can also use them (e.g. the star() source)
[ ] using the limit_zero() macro in place of the literal string: limit 0
[ ] using dbt.type_* macros instead of explicit datatypes (e.g. dbt.type_timestamp() instead of TIMESTAMP

[ ] I have updated the README.md (if applicable)
[ ] I have added tests & descriptions to my models (and macros if applicable)
[ ] I have added an entry to CHANGELOG.md

Hi @aashish72it, I'm going to close this for the same reason as https://github.com/dbt-labs/dbt-utils/pull/654#issuecomment-1236866173, and encourage you to open issues on the https://github.com/dbt-msft/dbt-synapse repo directly.


Bevy does not start on WASM because of the rand crates.

Bevy version
"0.4.0"
Operating system & version
Windows 10 and building for WASM.
What you did
I created a basic project to play with Seed (a Rust wasm framework) and Bevy to see how the 2 could interoperate. As I ran into other issues with Bevy I tried to setup a project for reproducing the bug. When I removed the Seed dependency another bug showed up (this one).
What you expected to happen
In this repository I set up the project is working fine as it is. But it contains Seed as a dependency that isn't needed. I expected to run just the same when removing the Seed dependency. Instead I got a crash with this error:
package.js:367 hello world
package_bg.wasm:0x13306b4 Uncaught (in promise) RuntimeError: unreachable
    at rust_start_panic (http://<IP_ADDRESS>:8000/static/package_bg.wasm:wasm-function[96256]:0x13306b4)
    at rust_panic (http://<IP_ADDRESS>:8000/static/package_bg.wasm:wasm-function[88735]:0x12fb685)
    at std::panicking::rust_panic_with_hook::hc5713da015ebaa19 (http://<IP_ADDRESS>:8000/static/package_bg.wasm:wasm-function[19582]:0xc80a7a)
    at std::panicking::begin_panic_handler::{{closure}}::hc5eba7f0030e8f4f (http://<IP_ADDRESS>:8000/static/package_bg.wasm:wasm-function[84877]:0x12d12fc)
    at std::sys_common::backtrace::rust_end_short_backtrace::he811f0bd07938b42 (http://<IP_ADDRESS>:8000/static/package_bg.wasm:wasm-function[85867]:0x12dd102)
    at rust_begin_unwind (http://<IP_ADDRESS>:8000/static/package_bg.wasm:wasm-function[76786]:0x1266ab9)
    at std::panicking::begin_panic_fmt::h308260c2e5b91752 (http://<IP_ADDRESS>:8000/static/package_bg.wasm:wasm-function[84876]:0x12d12c3)
    at rand::rngs::thread::THREAD_RNG_KEY::init::{{closure}}::h0d5a160d272afa5c (http://<IP_ADDRESS>:8000/static/package_bg.wasm:wasm-function[22564]:0xd1ee8e)
    at core::result::Result<T,E>::unwrap_or_else::h0acb8f77032d9da3 (http://<IP_ADDRESS>:8000/static/package_bg.wasm:wasm-function[5362]:0x7a5162)
    at rand::rngs::thread::THREAD_RNG_KEY::init::hb4d7b1557044de65 (http://<IP_ADDRESS>:8000/static/package_bg.wasm:wasm-function[9958]:0x9c051e)

To sum up, if you use the repo linked earlier, it will run fine as it is. When you edit the Cargo.toml to remove the Seed dependency it still build fine but crash at runtime while it shouldn't
Additional information
The call stack and some messages here and here on the Bevy discord seems to indicate that is a buggy version of the rand crate (or one of its dependencies). This would explain why it works with Seed. Since it uses rand it might trigger another version to be used which doesn't have this bug.
What could improve the situation is to pin exact version of the random crates that are known to work with WASM
PS: Thanks to Alice and DJMcNab on the Discord server for helping newcomers.

:wave: welcome!
The one difference between rand in bevy and in seed is a feature:
https://github.com/seed-rs/seed/blob/2fc928781c6abf4af47fd617be7ab263d25c84d8/Cargo.toml#L32
As feature are additive, enabling it for rand through seed also enables it in bevy dependencies.
Could you try to not import seed, but import rand with the same feature? It would not be a real fix, but would help to identify the cause.
rand = { version = "0.8.0", features = ["small_rng"] }

Thanks for feedback. I tried the rand trick but it didn't work. I run a diff between the output of cargo tree for the 2 versions and one interesting thing is that rand_core and rand_pcr show up in the version with Seed but not with the one. I tried to add many dependencies but I wonder if the linker does not remove them somehow because I did not find a single configuration that works by adding dependencies.

Swapping out the seed dependency for uuid = {version="=0.8.1"} makes things work. I'm not sure why this is necessary exactly, but https://github.com/mrk-its/bevy_webgl2_app_template also does this.

Here's a more complete stack trace in case it's useful
package_bg.wasm:0x133dcdd Uncaught (in promise) RuntimeError: unreachable
    at __rust_start_panic (http://localhost:9000/static/package_bg.wasm:wasm-function[96258]:0x133dcdd)
    at rust_panic (http://localhost:9000/static/package_bg.wasm:wasm-function[88826]:0x13096a6)
    at std::panicking::rust_panic_with_hook::h123718ba3bf480af (http://localhost:9000/static/package_bg.wasm:wasm-function[19192]:0xc70187)
    at std::panicking::begin_panic_handler::{{closure}}::hf393a82e58397bd2 (http://localhost:9000/static/package_bg.wasm:wasm-function[30862]:0xe7f73b)
    at std::sys_common::backtrace::__rust_end_short_backtrace::h2ff2cfc953878925 (http://localhost:9000/static/package_bg.wasm:wasm-function[85847]:0x12e9e85)
    at rust_begin_unwind (http://localhost:9000/static/package_bg.wasm:wasm-function[77271]:0x127a28e)
    at std::panicking::begin_panic_fmt::h96153d9dc0e882ed (http://localhost:9000/static/package_bg.wasm:wasm-function[84857]:0x12de077)
    at rand::rngs::thread::THREAD_RNG_KEY::__init::{{closure}}::h8ae4330291bfd8f8 (http://localhost:9000/static/package_bg.wasm:wasm-function[22415]:0xd1bf6a)
    at core::result::Result<T,E>::unwrap_or_else::h633a94d6b0dafd1c (http://localhost:9000/static/package_bg.wasm:wasm-function[5509]:0x7c3c1e)
    at rand::rngs::thread::THREAD_RNG_KEY::__init::h5b5b278c06a41cb3 (http://localhost:9000/static/package_bg.wasm:wasm-function[9984]:0x9cda71)
    at core::ops::function::FnOnce::call_once::ha51a275f0968dcf3 (http://localhost:9000/static/package_bg.wasm:wasm-function[78877]:0x12901cd)
    at std::thread::local::lazy::LazyKeyInner<T>::initialize::h0288a6bbce1ccea8 (http://localhost:9000/static/package_bg.wasm:wasm-function[15958]:0xbaaded)
    at std::thread::local::statik::Key<T>::get::h7d594a2cc1db358b (http://localhost:9000/static/package_bg.wasm:wasm-function[25295]:0xda45c1)
    at rand::rngs::thread::THREAD_RNG_KEY::__getit::h8a02932e64c3636a (http://localhost:9000/static/package_bg.wasm:wasm-function[95021]:0x133980e)
    at std::thread::local::LocalKey<T>::try_with::h2e6a6b9b9ee15f76 (http://localhost:9000/static/package_bg.wasm:wasm-function[19692]:0xc8bde0)
    at std::thread::local::LocalKey<T>::with::hb2a688111ead90af (http://localhost:9000/static/package_bg.wasm:wasm-function[36450]:0xf34793)
    at rand::rngs::thread::thread_rng::h7315cb71fd2fc4e5 (http://localhost:9000/static/package_bg.wasm:wasm-function[41818]:0xfcb679)
    at rand::random::h6c400ee9fc5b67e0 (http://localhost:9000/static/package_bg.wasm:wasm-function[58430]:0x1144724)
    at bevy_ecs::system::system::SystemId::new::h6e9dc5eb5c413125 (http://localhost:9000/static/package_bg.wasm:wasm-function[73622]:0x12457fe)
    at <bevy_ecs::schedule::stage::RunOnce as core::default::Default>::default::hb788ef4d44199b5b (http://localhost:9000/static/package_bg.wasm:wasm-function[28038]:0xe16f85)
    at bevy_app::app_builder::AppBuilder::add_default_stages::hc6079b3d539ebee0 (http://localhost:9000/static/package_bg.wasm:wasm-function[2675]:0x5bac41)
    at <bevy_app::app_builder::AppBuilder as core::default::Default>::default::hdc7bc2b4c376e2ad (http://localhost:9000/static/package_bg.wasm:wasm-function[25915]:0xdbfaeb)
    at bevy_app::app::App::build::hd77e41c462b67f70 (http://localhost:9000/static/package_bg.wasm:wasm-function[95960]:0x133d713)
    at bevy_wasm::start::h49c318fd183fea71 (http://localhost:9000/static/package_bg.wasm:wasm-function[689]:0x2f48db)
    at start (http://localhost:9000/static/package_bg.wasm:wasm-function[83331]:0x12cb254)
    at init (http://localhost:9000/static/package.js:1108:10)

@rparrett  Are you sure about the uuid thing? I tried adding it and it still crash on my side.

No, I may have goofed, or maybe there's something intermittent going on. I can't seem to even reproduce the panic anymore.
By the way, did you mean to link to your other bevy-repro repo in the issue text? If not, I'm even more lost.

I am so sorry, I indeed linked the wrong repository. I now edited the first message to avoid confusion.

I just went through the procedure again, adding cargo clean into the mix and being extra careful.
Adding uuid = {version="=0.8.1"} does seem to help. Bevy 0.4 otherwise specifies
bevy_scene, bevy_reflect, and bevy_utils otherwise specify uuid = "0.8" which picks up 0.8.2
From uuid 0.8.2's changelog:

Use getrandom instead of rand (#447)

So I think there could be something to this.

@rparrett Thanks for giving this another go. I will do another round later today because I want to make sure we can reproduce it and pin down to a particular crate.
Regarding the (maybe) intermittent nature of the bug: If this the case (I will triple check this), doesn't it look a bug in cargo and/or rustc? Because, to me it means that the same input files (cargo.toml and some rust source code) would behave differently at runtime depending on earlier commands which isn't something I want.

So toggling on and off uuid = {version="=0.8.1"}
and building with
 rm -f -r target && rm -f Cargo.lock && cargo clean && wasm-pack -v build --target web --out-name package --out-dir out/static --dev

I was able to confirm that indeed this make the program run normally or crash depending if uuid in version 0.8.1 is used.
Thanks for the help @rparrett. At this point we have at least pinpointed the problem and found a workaround.

This is really unfortunate, but I don't think there's anything we can do at this point. Going to close as unactionable :(
This is another argument in favor of #3992.

Reopening... We should just remove rand, as @DJMcNab points out.

Closed, as #3992 is merged. If you have encountered a new related problem, please open a new issue :)


output images are black

Unfortunately this code doesn't work for me. All output images are black, as seen in tensorboard. What could be the problem?
I'm running train.py with the default settings. After adding some debug printing i see that all the output images have NaN values. The input images are fine. I'll dig into this deeper.

My jax and flax are more modern than yours:
jax  0.4.13
flax 0.7.2
In order to run the code without errors i had to to replace
ema_params = state.params.copy(add_or_replace={})
by
ema_params = state.params.copy()
No idea if that led to the nan issues in the outputs.

Update:
i upgraded my jax to 0,4,30 and flax to 0.8.5 and the problem disappeared. I do see non-black putput iages in tensorboard now.

With jax 0.4.30 the trainings were still quite unstable, but after upgrading to jax 0.4.31 the output looks good :)


Spyder crashes at "Open file"

Description of your problem
Whenever I try to change my working directory or open a file Spyder crashes
What steps will reproduce the problem?

Open Spyder from Windows start menu
Open file (Ctrl+O)

What is the expected output? What do you see instead?
Instead of a dialog box I get an pop up screen with "Python has stopped working"
Please provide any additional information below
I am pretty new to Anaconda/Spyder, just installed this version, but can't get it to work. Hope this is the right place to ask this question, I am sorry if it is already mentioned before...
Versions and main components

Spyder Version: 3.2.4
Python Version: 3.6.3
Qt Version: 5.6.2
PyQt Version: 5.6.0
Operating system: Windows 7

Dependencies
IPython >=4.0    :  6.1.0 (OK)
cython >=0.21    :  0.26.1 (OK)
jedi >=0.9.0     :  0.10.2 (OK)
nbconvert >=4.0  :  5.3.1 (OK)
numpy >=1.7      :  1.13.3 (OK)
pandas >=0.13.1  :  0.20.3 (OK)
pycodestyle >=2.3:  2.3.1 (OK)
pyflakes >=0.6.0 :  1.6.0 (OK)
pygments >=2.0   :  2.2.0 (OK)
pylint >=0.25    :  1.7.4 (OK)
qtconsole >=4.2.0:  4.3.1 (OK)
rope >=0.9.4     :  0.10.5 (OK)
sphinx >=0.6.6   :  1.6.3 (OK)
sympy >=0.7.3    :  1.1.1 (OK)

Thanks for reporting. This appears to be a duplicate of #3903 , which should be fixed as far as it can be on Spyder's end. The problem is due to an interaction between windows and other Qt5 installs/applications on the system; one particular culprit was Dell Backup and Recovery. In any case, you can take at look at the other issue for some suggested workarounds, and this issue can be closed.


The 32bit app is downloading the wrong zip file

Describe the bug
When checking for updates in the About dialog, the app always downloads the 64bit version zip file even if the running app is 32bit. The app doesn't check if the running process is 64/32 bit and/or takes the wrong download link.
To Reproduce
Steps to reproduce the behavior:

Start the 32bit version
Go to About dialog
Click on 'Check for update'
Check the filename of the downloading zip

Expected behavior
If the 32bit version is running, I expect the 32bit zip file to be downloaded.
Desktop (please complete the following information):

TumblThree version: <IP_ADDRESS>
OS: Windows 10

Who is able to find the problem? Can you solve it?

This one is for first-timers-only. That means that we will only accept a PR for this one from someone who's never contributed to open source before. This one is particularly easy (but don't make that statement make you feel bad if you have a hard time with it, there's more to contributing to open source than changing lines of code, especially if it's your first time). We walk you through it if you need help. :-) Here are the steps to get a PR merged here (see also our wiki).

[x] Create a fork of the repository
[x] Clone it to your local disk and create a new branch (e.g. 100-download-correct-zip-file)
[x] You should inspect around line 41 in ApplicationUpdateService.cs in folder Services of the project TumblThree.Applications.
[x] You need to find out if the currently running app/process is running under 64bit (Is64BitProcess) Environment or not
[x] You need to find out which descendant child node "browser_download_url" contains "x64-App" or "x86-App" in its url and assign that one to the variable "downloadLink"
[x] Commit your changes mentioning "issue #100" in the comment
[x] Push your changes to your fork, create a PR, get merged, celebrate ??

We are happy to support you if you need help.

This one is for first-timers-only. That means that we will only accept a PR for this one from someone who's never contributed to open source before. This one is particularly easy (but don't make that statement make you feel bad if you have a hard time with it, there's more to contributing to open source than changing lines of code, especially if it's your first time). We walk you through it if you need help. :-) Here are the steps to get a PR merged here (see also our wiki).

[x] Create a fork of the repository
[x] Clone it to your local disk and create a new branch (e.g. 100-download-correct-zip-file)
[x] You should inspect around line 41 in ApplicationUpdateService.cs in folder Services of the project TumblThree.Applications.
[x] You need to find out if the currently running app/process is running under 64bit (Is64BitProcess) Environment or not
[x] You need to find out which descendant child node "browser_download_url" contains "x64-App" or "x86-App" in its url and assign that one to the variable "downloadLink"
[x] Commit your changes mentioning "issue #100" in the comment
[x] Push your changes to your fork, create a PR, get merged, celebrate ??

We are happy to support you if you need help.

Could I please take this!  First timer here :-)

Thanks for your interest. I'll assign that one to you.
Please ask if you have a question or get stuck.

If this one gets free, I am ready to give it a go. First time contributer for open source

Thanks for your interest. Keep an eye on the issue, or maybe try to solve one of the other open issues.

@thomas694  I was able to add the Is64bit.  However, I feel like I'm over complicating the next step.  Looking at this page https://api.github.com/repos/tumblthreeapp/tumblthree/releases/latest I can see the browser_download_url is different for each version, but I'm not seeing any other node that would make it easy to know which one to assign.  Should I be doing a search on the name to find one with 64 or 86?  Or is there something easier I'm missing?
Sorry if this is a dumb question.  Total newbie here.

We are currently using this pattern for naming the release files: {ProductName}-{VersionString}-{Bitness}-{PackageType}.zip
Yes, as this release json is done by GitHub and there's no better or unique field, you just have to search the browser_download_url fields for a string part "x64-App" or "x86-App" (ideally case-insensitive). In the case the code shouldn't find an entry, it should use the first entry like it's done currently.
Better to ask an easy question than spending too long searching for a way too complicated answer.

@khintze1 If you have any other question, please feel free to ask me.

@thomas694 I was thinking of doing something like this
http://csharp.net-informations.com/xml/search-in-xml-file.htm
is that the right path?

@khintze1 I think you lost a bit your way, that's not really the right page you found.
You can work with the XElement object which now contains a xml representation of the json string we downloaded before. And you have to find a way to filter the nodes by element name (browser_download_url).
While working with the XElement object, you can either use LINQ methods or do it old-fashioned.

So I think I pushed it to the branch if you want to take a look?

You're going into the right direction...
In the else-branch you have overlooked to put curly braces. Without them, it won't do what you're expecting, even you've indented your code. In the following I pretended they are there.
Besides if the code is like here, you can put every line which is in both branches of the if statement in front of the if. It prevents code duplication and makes it more readable.
Does your code compile? Have you tested your code, either by letting the app download the json or using a temporary debug construct like the following?
string str = @"{copy the json text here}";
StringReader sr = new StringReader(str);
XElement root = XElement.Load(sr);

I think you could find a method (inside XElement) that returns more than one element (with the searched element name) at once, so that you can apply Where to it and check if one of the element's Value Contains the searched string. You also don't have to care on which level of the xml structure the browser_download_url nodes are, as they are "unique" enough.
If I expressed myself unclear or you have any question, let me know.

OK round two.  Thanks so much for all of your help!!  I really appreciate it.  Let me know if that was more what you were looking for.  Which file should I try compiling to see if it works?

Yes, you fixed the first two points and now it looks better than before.
Just build the whole solution in Visual Studio. If there are errors in your code, it will show you in the errors window.
If the solution is built without errors, you can put a breakpoint on e.g. line 36 and run the app in debug mode. Check for an update version in the about dialog and the debugger stops on that line. Now you can inspect the content of the variables by hoovering over them or using the QuickWatch window. QuickWatch can also be used with partial marked lines, e.g. root.Element("assets") or root.Element("assets").Element("item").Element("browser_download_url").
Does your code work?

I think I'm really close now.  This one compiles.  The only issue I'm having is I can't get it to write the value back to downloadlink variable.  It currently fails because it's not set to an instance of an object.

Well, you're much closer, but still something missing.
Did you set breakpoints and look what the values are which have been returned? I assume not.
Where doesn't find any nodes, so the foreach loop can't iterate any elements in query and assign a value to downloadLink. That's why you see a 'not set to an instance of an object' failure.
Which elements does Elements() return? "Returns a collection of the child elements of this element or document, in document order". Where are the browser_download_url nodes compared to the root node? Does the root (XElement class) have another method (also look on its overloads) that suits better (returning every node below root at once)?
Later when you find the correct nodes, you have to improve your filter. Are you looking for anything containing "x64-App" or a browser_download_url node which contains "x64-App"? That's a difference. Look on the release file again, whether it's json or xml doesn't matter for that, the levels are the same.
At the end, if you still keep that line, you should use downloadLink = element.Value;. The output is different.

Alright, I think I have the right nodes?  Does the elements return just the values in the browser_download_url?  So what is there works for 64 bit, but not for 32 bit in testing.  In the 32 bit it goes back to the null error.  Is it just a fluke that the 64 bit one is working?

Your variable elements returns nodes. But for assignment to downloadLink you need the node's value, the url string. You can test/simulate both cases, either (temporarily) negate your if condition or set the breakpoint line in the else block when the debugger stops on your breakpoint at the if statement. For both cases the downloadLink variable should be set correctly and the downloaded zip file should be the x86 or x64 one.
But your 32 bit code is not giving you the right thing, because you used root.Element("assets").Element("item").Element("browser_download_url") again like before (see above). That's returning always the first item on every level, in our release json that's the "x64-App" one.
To solve it, you can e.g. use a for-loop on every level to go through all the nodes and search the right one, or you just find another method which does that for you. Please take a look at MS Docs - XElement Class. Which method can give back all child nodes regardless of the level and at the same time filter for the desired node name ("XName", here "browser_download_url")?

Ok so I finally got it :-) I needed to go through the descendants not elements :-)  In the debug one I have tested that it works for 64 and 32 bit and gets the correct download link.

That's great. :-) Your answer is working (and accepted).
But we should polish your code a bit. You like?
Actually you used the chained method Where (your point at the end of previous line) and not the Linq Where, so actually it means, take every element of IEnumerable<XElement> and return them as IEnumerable<XElement>:
IEnumerable<XElement> elements =
                          from el in {IEnumerable<XElement>}
                          select el;

As Descendants() and Where() already return IEnumerable<XElement> that step is unnecessary, so just use the inner one (*1):
IEnumerable<XElement> elements = root.Descendants("browser_download_url").Where(s => s.Value.Contains("x64-App"));
// Linq where would be:  where el.Value.Contains("x64-App")

Your for-loop is ok, but C# has a new "null-conditional operator", so you could shorten it:
downloadLink = elements.FirstOrDefault()?.Value;

At the end you could even combine all in one line:
downloadLink = root.Descendants("browser_download_url").Where(s => s.Value.Contains("x64-App")).FirstOrDefault()?.Value;

After your if-else statement, it would be good, if the first found entry would be assigned (like in the old code). Just in case the deployment is changed in the future without updating this method, it would still give back the first download link (*2):
if (string.IsNullOrEmpty(downloadLink)) downloadLink = root.Element("assets").Element("item").Element("browser_download_url").Value;

Please adapt at least *1 and *2. Then create your pull request.

So in the end, I shortened it to this:
//IEnumerable elements = root.Descendants("browser_download_url").Where(s => s.Value.Contains("x86-App"));
downloadLink = root.Descendants("browser_download_url").Where(s => s.Value.Contains("x86-App")).FirstOrDefault()?.Value;
I don't even need the for loop or the first line do I?
I also added the ending part.

Yes, you can shorten it to one line. One line for each part of your if-else statement (x86/x64).

Thanks for your first contribution and PR to open source, and especially to the TumblThree project.
Your PR has been merged and you've been added to our list of contributors.
You are welcome to solve some more issues (not first-timers-only).


bump CUDA.jl to 4.0

Codecov Report

Merging #248 (ce0f4b7) into master (4f3eee1) will increase coverage by 2.01%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##           master     #248      +/-   ##
==========================================
+ Coverage   74.17%   76.18%   +2.01%
==========================================
  Files          38       38
  Lines        3880     3574     -306
==========================================
- Hits         2878     2723     -155
+ Misses       1002      851     -151

see 30 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Explorer Selector Not Appearing - Not Expected Window

Hi Thio, long time fan!
This is an incredible tool (ExplorerDialogPathSelector) 40b274d however the script is not opening the dialog window:
Environment

Windows 11 23H1
Windows 11 23H2
Using ExplorerDialogPathSelector v1.1.0 40b274d
Debug Mode
Standard Windows File Explorer (not the other restricted types)

Issue

Hotkey pressed
Window ID: 723778
Class: CabinetWClass
Window class does not match expected: CabinetWClass

I assume the ID is ephemeral.
What I've Tried

Restarting explorer.exe
Restarting machine
Using different version of Windows 11
Pausing/Unpausing Script
Suspend/Unsuspend Hotkey
Changed Hotkey
Used both the raw AHK and EXE variants of the script

Do you mean it doesn't show up in a regular file explorer window? Or it doesn't show up in "Save As" and "Open" type windows? It's not supposed to show up in the regular file explorer, only in the Save As and Open type ones

Ah you're totally right!
It does show up in those explorer windows - I was just a bit eager and thought it would appear in the regular File Explorer box for quick switching. My bad.


feat(Algebra/Category/ModuleCat/{Presheaf, Sheaf}): forgetful functor from sheaves of modules over sheaf of ring R to sheaves of R(X)-module when X is initial

Let $R : C^{op} \to Ring$ be a sheaf of rings and $M$ a sheaf of $R$-modules. Then $M$ is naturally a sheaf of $R(\top)$-modules where $\top$ is an initial object.
In the case of a sheafed space $X$, for any $x \in X$ and $M$ a sheaf of $\mathcal{O} _X$-modules, then $M$ is also a sheaf of $\Gamma(\mathcal{O} _X, X)$-modules; thus we can take stalks to naturally have that $M_x$ is a $\Gamma(\mathcal{O} _X, X)$-module. Thus we can use this construction to define $M_x$ as a module over $\mathcal{O} _{X, x}$.
Co-authored-by: Sophie Morel<EMAIL_ADDRESS>

[x] depends on: #14283

(I have removed Kevin's comment in the PR message and fixed the LaTeX. You need spaces before underscores so that Github markdown does not register it as italics)

bors d+

bors r+


CircleCI flutter orb unexpected argument: version

Orb version:
1.0.1

What happened:
I've got the error:
Error calling workflow: 'distribute'Error calling job: 'distribute'Error calling command: 'flutter/install_sdk_and_pub'Unexpected argument(s): version

Expected behavior:
It should download and install Flutter version passed as argument version.

Additional Information:
Yaml file:
version: '2.1'
orbs:
  flutter<EMAIL_ADDRESS>jobs:
  distribute:
    macos:
      xcode: 12.5.1
    steps:
      - checkout
      - flutter/install_sdk_and_pub:
          version: 2.8.1
          app-dir: ./app
      - flutter/install_ios_pod:
          app-dir: ./app
      - flutter/install_ios_gem:
          app-dir: ./app
      - run:
          command: bundle exec fastlane ios distribute
          working_directory: ios
workflows:
  distribute:
    jobs:
      - distribute

Using flutter_version instead of version fixed the error:
          flutter_version: 2.8.1

Please update sample docmentation.

Using flutter_version instead of version fixed the error:
- flutter/install_sdk_and_pub:
          flutter_version: 2.8.1

Please update sample documentation.

Using flutter_version instead of version fixed the error:
          flutter_version: 2.8.1

Please update sample docmentation.

Using flutter_version instead of version fixed the error:
- flutter/install_sdk_and_pub:
          flutter_version: 2.8.1

Please update sample documentation.

I believe this parameter should also be called version. It's unintuitive as all the other jobs that allow for setting the SDK version use that naming convention, so it makes more sense to install_sdk_and_pub maintain this consistency.


Remove DefaultAssemblyPartDiscoveryProvider.GetAssemblyName

We have a copy of the GetAssemblyName method from the CLI because it is currently static. This can be removed once once https://github.com/dotnet/cli/pull/2117  lands.

https://github.com/dotnet/cli/pull/2117 is merged so hopefully we can get this into RC2 soon.

@pranavkm was this done?


G> Prezi "Decentralized ..."

Goal: create Prezi for 17 Feb, Governance Meeting
"Decentralized setting of budgets and rewards for solving self defined issues on Github" is the key for building a 10.000+ swarmwise organisation that helps RChain getting on the rails.
Send email addresses to<EMAIL_ADDRESS>for collaboration on Prezi
This is the very very beginning of a draft

What is RAM or SMART?

You can help me creating this presentation by logging in to https://prezi.com
as<EMAIL_ADDRESS>pw: Decentralize

@lapin7 requirements for the presentation? Where can I find the topics ? Or I have to come up with one by myself.

@zsluedem To be clear.
The subject is:
"Decentralized setting of budgets and rewards for solving self defined issues on Github" is the key for building a 10.000+ swarmwise organisation that helps RChain getting on the rails.
I want to cover the spreadsheet system
Use #251 "M> Translate video Exclusive chat Lawrence Lerner & Nash Foster" as an example of great collaboration.
Use #261 "O>	slashing conditions in the RAM budgeting and reward system?" as input
Use #260 "O> Usable Web app for RAM distributed budgeting and reward system" as our future system
Set up a form for making decisions about:

high level budget for decentralized work (10,000,000 RHOC)
goal: 10,000 RAM's before 10-10-2018
approval of the whole system
.... what else

I added a few topics including objectives and oversight and then leah and I did an overall edit for clearity and language.
It may be better to set up loomio for voting rather than a form @lapin7 ?

@jimscarver The slide title is spelled "Oversite" but it should be "Oversight"

@lapin7 delivered the presentation at the RChain Governance Forum 2018


get businesses from database

delete dummy data on state in App.js  and can now directly get data from the database
Description
Please include a summary of the change and a link to which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
We can now directly get businesses from the database. I deleted the dummy data on state in App.js.
We've not agreed on criteria on Featured Businesses and Popular Businesses, what we have right now is based on  boolean and number of stars, respectively.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] My code has been reviewed by at least one peer
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Nice work.  Merging PR.  for future PR's please document how the code was tested, even if it was just checking functionality.


reviewManager doesn't work

hey, hi. When I click on Rate app and then dismiss window (without raiting), I can never open rate app view again. What's problem? If you need a video, I can screen it

The new "In app review" API from google isn't fully stable, i faced many issues with it (sometimes it won't even show at first try).
Guess i will remove it in the next update.

absolutely rught. this api is garbage


isFederated flag is not set for authorized user with JWT Bearer Grant when the token info is taken from the database

This issue is observed in identity-inbound-auth-oauth-5.6.63.
When an access token is generated by passing a saml bearer, the user is handled as a federated user and therefore, the isFederated flag for the user is set to true. When the token is unavailable in the cache (e.g. after the cache timeout) the latest access token information is taken from the database and the properties are set to the authorized user and added to the cache. Setting the isFederated flag is missed here and therefore during the API invocation, in scenarios where the user store is accessed, this will fail to identify the user in the userstore. An exception similar to the below posted lines will be thrown.
org.wso2.carbon.user.core.UserStoreException: Invalid Domain Name
at org.wso2.carbon.user.core.common.AbstractUserStoreManager.callSecure(AbstractUserStoreManager.java:170)
at org.wso2.carbon.user.core.common.AbstractUserStoreManager.isExistingUser(AbstractUserStoreManager.java:3646)

This has been fixed in  identity-inbound-auth-oauth-6.0.53 (for wso2is-5.7.0 ) by the improvements done for #4189. Same has to be done for v5.6.63.

Fixed along with https://github.com/wso2/product-is/issues/4189


SpriteKit port

Hi there,
I’ve been using this repo for a while now and am really liking it. You did a really good job with it! I ported it to SpriteKit over the last week, and was about to PR it when I had some thoughts.
The most important change is that it’s written in SpriteKit, which means that users would have to import that framework as well, adding to their app’s binary size. SpriteKit will be way more “expensive”, storage-wise, than this framework, which might be problematic especially given cellular download limits.
Users of a mature framework might not be okay with having to add SpriteKit in order to continue to use new versions, which is why I’m posting this as an issue instead of directly PRing.
I’m happy to open a PR so that the existing implementation can be replaced with my SpriteKit port, or I can upload it as a new project (crediting you as the original creator, of course), depending on whichever way you’d like to go with this.
Please let me know how you’d like to proceed. Thanks for making this!

Hey!
This is another user of this library and I just want to say if you could make this happen I would be so happy! With this SpriteKit change it can be used on Apple Watch which would be sick🤘🏼🤘🏼 Let me know if I can help !

Hey guys,
The initial idea for this control was being able to add it as easy as possible, so SpriteKit would bring in a lot of overhead that I didn't want. While working on it I considered OpenGL ES implementation, too, but didn't go for it in favor of the usage simplicity. I wanted it to be a simple drop-in view, didn't even support Cocoapods or Carthage at first 😄
On the other hand, providing different implementations for the end users to be able to choose from would also be great. So what I'd suggest is adding the SpriteKit implementation to the existing and listing it as a subspec. Not sure if it's possible to exclude SK framework from the original spec though, but I think we'll figure it out 😀
@adilanchian did you try the lib on Apple Watch? There were some issues with it I guess, maybe it's possible to work around them with UIKit only?

It really doesn’t make sense to me to have both tied together, which is why I didn’t even bring that up. While the output is the same, they are two different projects in terms of the code.
Having both be just different implementations of the same spec needlessly limits the scope of both. You could do so much more if both were not tied to each other and you didn’t have to ensure the any behaviour had to be doable in both to be added.
I can totally respect your desire for the usage simplicity of your implementation, so I think the best way forward is to open source my port as a separate project.

Okay, agreed. Let it be a separate port.

Hey @maxkonovalov,
I have not tried this on watchOS yet. I’m currently in the process of localizing Cycles for a couple different languages haha. I’ll open an issue for this when I start that process if someone doesn’t try it out before me. Thanks my dude! Love seeing all the support on this library. It’s so dope 🤘🏼

Sorry @adilanchian, missed your post somehow! I haven't used WatchKit at all (don't even have an Apple Watch), so I might need some help once it's out, if you've got the time to spare. Should be open in a couple of days, I'll let you know when it's ready

Don’t worry about it! I started my swift journey with Apple Watch so I can definitely make time if you want to try and tag team it? My experience with SpriteKit is very basic :). Thanks!

And it's finally ready: https://github.com/HarshilShah/ActivityRings
Lemme know what you folks think!

Hey @HarshilShah!
Really nice implementation! What I like is being able to calculate the gradient using a shader.
What about performance though, does it work well for interactive animations?

Thanks!
Still a bit of testing to do, but yeah, the init phase could do with a bit of work
Sent with GitHawk


Bump version and fix types

I want to merge this change because it applies changes made in https://github.com/mirumee/macaw-ui/pull/21
PR intended to be tested with API branch: master
Screenshots

Pull Request Checklist

[ ] This code contains UI changes
[ ] All visible strings are translated with proper context including data-formatting
[ ] Attributes [data-test-id] are added for new elements
[ ] Changes are mentioned in the changelog
[ ] The changes are tested in different browsers and in light/dark mode

Test environment config

API_URI=https://qa.staging.saleor.cloud/graphql/

Task linked: SALEOR-3287 Use Macaw-UI in both Saleor Cloud and Saleor Dashboard

Task linked: SALEOR-3578 Improve typing and component API


[CI:BUILD] Packit: trigger builds on commit to main branch

This commit lets packit trigger builds on
rhcontainerbot/podman-next copr after a commit to the main branch instead of the current github webhook trigger.
The builds triggered via packit also provide more information in their version-release:
Current webhook triggered build:
101:0.0.git.914.c2222792-1
Packit triggered build:
101:1.6.0~dev-1.20230318180556434499.pr589.60.g82922eb
The packit triggered build correctly shows the PR#, commit id, timestamp as well as the upstream version.
Similar PR for aardvark-dns:
https://github.com/containers/aardvark-dns/pull/304

@Luap99 @flouthoc @mheon @baude PTAL

lgtm


Bug on new version 0.4.1 with Drift Search

Do you need to file an issue?

[x] I have searched the existing issues and this bug is not already filed.
[x] My model is hosted on OpenAI or Azure. If not, please look at the "model providers" issue and don't file a new one here.
[x] I believe this is a legitimate bug, not just a question. If this is a question, please use the Discussions area.

Describe the bug
Hello, I am having a problem with GraphRag version 0.4.1. This problem I did not encounter in the older version 0.3.6.
I use this version because of the Drift Search function, but it seems that this function has a problem, it has an error when calling Local Search "Query column vector must be a vector. Got list<item: double>.",
I have referred to issue 1335, to solve this bug, however I found that when calling Local Search, there is no result returned because querying entities on lanceDB returns empty, even when I perform Local Search separately, I also have the same problem and have not found a solution.
Please let me know how to solve the above problem or is there another version without the bug. Thank you very much!
Steps to reproduce
Install GraphRag version 0.4.1
Indexing 1-2 documents
Testing Drift or Local Search
See error
Temporarily ignore the error that does not support Json, this can be fixed with prompt
Expected Behavior
No response
GraphRAG Config Used
API_VERSION = 2024-08-01-preview
API_BASE = https://fpga.openai.azure.com/
LLMS_TYPE = azure_openai_chat
LLM_DEPLOYMENT_NAME = gpt-35-turbo
LLM_MODEL_NAME = gpt-35-turbo
EMBEDDNIG_DEPLOYMENT_NAME = text-embedding-3-small
EMBEDDING_MODEL = text-embedding-3-small
EMBEDDING_TYPE = azure_openai_embedding
JSON_SUPPORT = false

Logs and screenshots

Additional Information

GraphRAG Version: 0.4.1
Operating System: Window
Python Version: 3.10.15
Related Issues: 1335

We fixed a couple of bugs related to drift search in the latest release (v0.5.0). Please upgrade to the latest release and let us know if you're still experiencing the same issue.
Thanks

We fixed a couple of bugs related to drift search in the latest release (v0.5.0). Please upgrade to the latest release and let us know if you're still experiencing the same issue.
Thanks

Hi, I have updated to the latest version (0.5.0) and tried Drift Search again.

First it still has the error LLM does not support Json format, because I use GPT 3.5 model, I fixed it by adjusting the prompt and processing the result from LLM to return to the correct Json format, then I passed this part.

The next error is similar to what I encountered above, and it seems that it does not work perfectly.

Is it possible that the ipynb example is outdated compared to the new version? Please let me know how to handle it or a new ipynb example that can run with this version. Thank you very much


Use a released version of sassc

The change we were fetching via git has been released:
https://github.com/bolandrm/sassc-rails/pull/17
/cc @boffbowsh

:+1:
Love the side effect of coding in the open of us polluting that PR with our own commit comments.


Update pox-pydoc

POX includes its own version of pydoc, which is surely way out of date now.  Especially as part of the Python 3 port, it should be updated.

Just a note that, IIRC, the main POX-specific feature it has is that it can list events that a given class raises.
Instead of upgrading it, another possibility is just removing it.  The main pox documentation is now generated by Sphinx.  I don't think we currently are, but we could start using Sphinx to autogenerate reference documentation too.  In which case, we might want to extend Sphinx to list events...


Make index.html static, send defaults over WS

Currently index.html is rendered on the server with handlebars, which is rather unnecessary, this also leaks defaults from config in private mode.
Making index.html static would also completely remove client dependency on the server.

Worth doing with #1271 somewhat around the corner?

Both of them are kinda tied to each other.


Glossary references not rendering

Glossary references appearing as plain text instead of links (e.g. Formal Languages chapter)

Glossary links now work using the following notation:
[finite state automaton](appendices/glossary.html#finite-state-automaton)


2.x Check coding style

There should be at least 3 basic checks

syntax
coding style
static analysis

Currently we are several thousands violations away from WPCS.
Shouldn't we follow a modern coding standard?
https://github.com/Automattic/phpcs-neutron-ruleset ?

Currently we are several thousands violations away from WPCS.

You are totally correct @szepeviktor. This is one of our final steps before we release 2.0. We've been holding off so that we don't hit a bunch of merge conflicts between 2.x and 1.x. To help get us there though, is there a current preferred tool for running/applying/checking? I notice you had composer lint in the Travis file on this PR

Yes, "composer lint" equals "phpcs" (in composer.json)
The WordPress way is wpcs or neutron,
the OOP way is PSR-12 possibly with a more modern tool: https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/4502

@szepeviktor Cool! That’s the missing step to automate coding standards! I’m no Travis expert, so I’m very happy that you provide us with the proper way to incorporate this 👍.
We added composer lint in #1933 and defined what Coding Standards to use in #1603. The Coding Standards are documented in https://github.com/timber/timber/blob/master/CONTRIBUTING.md#coding-standards, but as @jarednova mentioned, we will only apply them as on the last tasks for Timber 2.0.
We’ll see whether we will make use Automattic/phpcs-neutron-ruleset, depending on how well we can apply the standards we already defined. I’m a little wary about adding yet another dependency. Especially one that we don’t have much control over. Plus, the Neutron standards seem to have a lot of standards which would mean so much work for us just to create compatibility. But we could surely poke around and see what they have in there to make our lives easier 😉.
But Tools like VariableAnalysis and ImportDetection definitely look interesting!
For automation, phpcs recommends to use phpcbf.

Just adding the blocked label for now. We can pick this up once we get the remaining Factories stuff in. From what I can tell, this looks like a clear/simple implementation so we only run once w Travis

Codecov Report

Merging #2166 into 2.x will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                2.x    #2166   +/-   ##
=========================================
  Coverage     96.04%   96.04%
  Complexity     1530     1530
=========================================
  Files            50       50
  Lines          3893     3893
=========================================
  Hits           3739     3739
  Misses          154      154

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 03bf4bf...226d9fb. Read the comment docs.

Very BIG 🍏 success.

 With #2592 I believe we can!
The specific code with this PR might be stale (with its Travis file) — but all the same ideas can be brought over to GH Actions. Any ✋ ? I think we should follow "le mieux est l'ennemi du bien" 🇫🇷 and start one level/piece at a time so the PR(s) stay managable and mergable.

The specific code with this PR might be stale (with its Travis file)

ALL my projects run on Travis CI.
Travis CI Travis CI Travis CI
Travis CI Travis CI Travis CI
Travis CI Travis CI Travis CI

With #2592 being merged quite some time ago, we can close this one.
As always, big thanks to @szepeviktor for showing us how it can be done and holding us accountable for clean code and a modern setup 🙌.


Improve configuration handling in the invitation event handler

Purpose

$subject

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6106320441

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6106320441
Status: success


Cross platform controller support for Unity XR and WebVR.

This PR adds ability to map create input gestures and map them to the appropriate Unity XR (Using Unity Input Manager) or Web gamepad API input systems.
This enables us to easily work between the Unity and WebVR using a single control scheme.
This includes changes from https://github.com/mozilla/unity-webvr-export/pull/295 which has now been closed in favor of this PR.

Will merge this in the interest of moving fast, @cvan will follow up with another review.


scrfd 10g model (attached) is unable to detect any faces in sample images

I've modified the code in app_scrfd.cpp to load my own scrfd 10g (which can be downloaded here) instead of the default one. The only other thing I've changed is the batch size, from 6 to 1, here.
When I then build the trt model and run inference using the example code, it is unable to find any faces in any of the sample images in workspace/inference/.
If I then load the model directly with onnxruntime C++ and run inference, I can find faces in all of those same images.
Can you please download and test the scrfd model and tell me what is missing there.

Here are the images when I run them with the same model onnxruntime c++. Faces detected in all. Therefore I'm wondering if there could be something in the model that is not supported by your implementation code. Is there any chance you can please download and run the model.

Or is there something special I'm supposed to do when exporting from pytorch to onnxruntime. And if so, could you share what that is with me please.

I was able to solve the issue by following the instructions here: https://github.com/namdvt/SCRFD_FaceDetection_TensorRT#export-scrfd-onnx
To summarize, my model had the following output dimensions:

While the model needs to have these output dimensions:

Using the code above, I was able to export the model from pytorch to onnx with the correct output dimensions.


responseXML implementation using jsdom (htmlparser)

Hello,
Is is possible to include my implementation of responseXML into your
mainstream git ?
Related to
https://github.com/driverdan/node-XMLHttpRequest/issues/8
https://github.com/driverdan/node-XMLHttpRequest/issues/13
I implemented responseXML in my fork of node-XMLHttpRequest
using "jsdom" (using "htmlparser"). The benefit is that you can parse
non XML url like any HTML website and get a DOM.
I implemented it within an exception handler so node-XMLHttpRequest
should continue to work even if jsdom has not been installed. Of
course without those modules responseXML cannot be used but
responseText will continue to work.
The implementation :
https://github.com/lordbaco/node-XMLHttpRequest/commit/1303a2596ad7fad504a1e1d4c696ee76b78c6d9a
I am not sure my implementation will work with sync but async is working.
To use this feature you need to :
npm install jsdom
git clone<EMAIL_ADDRESS>then use it in your code
var sys = require('util');
var XMLHttpRequest = require("./node-XMLHttpRequest/XMLHttpRequest").XMLHttpRequest;
var url = 'https://github.com';
var xhr = new XMLHttpRequest();
xhr.onreadystatechange = function() {
if (this.readyState == 4) {
var document = this.responseXML;
//sys.puts("XML:\n" + sys.inspect(document.innerHTML, false, null));
var nodes = document.body.getElementsByTagName('*');
for (var i=0; i<nodes.length; i++) {
// ...
sys.print(nodes[i].nodeName.toLowerCase() + "\n");
}
}
};
xhr.open("GET", url);
xhr.send();
Best Regards,
Guy (Lordbaco)

Sorry about the late response. Can you issue a pull request? That way I can implement it with proper attribution and can modify a few items before accepting.


Enter password twice to ensure it is correct

When signing up, the new person should enter their password twice to ensure it was entered correctly.

Fixed in bdcd70f


CS is low during 80 clock ticks init

In tf_card.c in disk_initialize around line 285 CS is taken LOW before sending the 80 dummy clocks. Everything I've read says CS should be high during init. I commented out CS_LOW().

@Iangitpers Thank you for your remark. I deleted CS_LOW() since it seems quite right from several information I've checked.
	FCLK_SLOW();
	for (n = 10; n; n--) xchg_spi(0xFF);	/* Send 80 dummy clocks */


[Refactor] add connector metadata request to simplify interfaces

Why I'm doing:
What I'm doing:
Add ConnectorMetadataRquest to simplify interfaces. So I can add more variables without rewriting interfaces.
Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[ ] Enhancement
[x] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.4
[ ] 3.3
[ ] 3.2
[ ] 3.1
[ ] 3.0

@mergify backport branch-3.4


Implement Node/Module Deployment

Given an IP address or a list of addresses and needed modules, deploy modules on these nodes and mount a communication channel

[x] Define how nodes are identified (ids, ips, etc.)
[x] Define how modules are identified (a module is unique for a node, but there can be multiple same modules across many nodes)
[x] Define configuration file format
[x] Implement nodes and module identification management
[x] Define how nodes are deployed and modules integrated (supposely pre-defined Docker containers deployed over ssh)
[ ] Implement node and module deployement
[ ] Define how communication is maintained with node (supposely over RabbitMQ over two channels : 1 signaling and 1 data channel)
[ ] Implement node communication

[x ] Nodes are identified per ip:port
[x] Configuration : .toml for internal project, .yml for Ansible
[x] Deployment : Ansible spins up boxes (1 master, n nodes), clones project, deploys PKI, installs docker, composes, and run.
[ ] Deployment implementation : WIP. See netm4ul-ansible
[x] Communication : http requests, probably over Mutual TLS, on 1-to-1 between server and nodes.
[x] Communication is implemented.


Problems with nexus parser

We have installed a nomad oasis in our lab environment and are trying to upload nexus format data.
The upload works and the parser recognises the data files, we are able to view the data in the oasis. However none of the metadata included in the nexus file is incorporated in a searchable way. I included some screenshots of a simple nxs file we created from the nexus homepage to include an instrument and chemical formula.
However the two metadata entries are not incorporated. Are we doing something wrong on our side or is there something missing in the parser/normalisers?

Hi @KGoetz91!
In the default search interface, only the properties under results will be searchable. Since the nexus parser is probably not filling up anything in results, you will also not be able to search anything.
There are two approaches to solving this:

You could define a completely new app that loads the interesting bits from your nexus data into the search interface. The default app is not configured to automatically read anything from nexus application definitions (or from any custom Python schemas), because it would get awfully crowded otherwise.
You could introduce an additional normalizer or we could modify the nexus parser to fill the results section with the data. This way the entry would become more cross-compatible with other types of data, but there are constraints set by the rather rigid structure of the results section.

I think option 1 will be an easier. Maybe you could provide me with an example of the nexus files that you are interested in, and I could draft you an initial app to get started? I could then also add a page about this procedure in our online docs.
@domna: Do you have any comments on this? I guess it would quite a hard task to map each application definition to the results section consistently in the nexus parser.

Hi @KGoetz91,
I agree with @lauri-codes's view that creating an app for nexus would be the way to make the data searchable. However, we will provide a general NeXus search app eventually and currently also support extracting the chemical formula into the results section (and hence make your data searchable in the periodic table viewer). We may also further extend which informations we compile into the results section. If there is something you urgently need we can try to find the correct place to put it into the results section - but as Lauri mentioned the results are a rigid structure and we may not find an appropriate place.
Additionally, I see that your data is not even parsed into the nexus metainfo (which is not searchable but browsable). This is a prerequisite to make any data searchable (even for an app). I think this is due to the fact that your file is not following any application definition (it would need to have at least an /entry/definition pointing to a valid application definition, e.g., NXoptical_spectroscopy - only then nomad is able to attach it to the correct metainfo). You could try to add the term NXoptical_spectroscopy into /entry/definition (as from the data it looks like that its an optical spectroscopy experiment). The metainfo will only parse the documented fields according to this application definition.


The Managed Identity Video By Varun

[Enter feedback here]
The video demo was on AzManaged Identity. I loved this demo.
However, am high on my vodka still I can figure out that Varun was repeatedly saying that:
A) he will create an Identity that will destroy with the deletion of the resources linked.
And in the contrary he did demo on "User Managaed Identity" instead of "System Managed Identity"
I hope you understand what I say!
Kudos! and Luv You: Microsoft
Reg,
Soumen Dey
https://www.linkedin.com/in/ta-soumen/

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

ID: 72861cc1-7081-cf6e-b7f9-e700a16faa9a
Version Independent ID: c870872d-0559-6b77-280f-f478e8c84776
Content: Managed identities for Azure resources - Microsoft Entra ID
Content Source: docs/identity/managed-identities-azure-resources/overview.md
Service: active-directory
Sub-service: msi
GitHub Login: @barclayn
Microsoft Alias: barclayn

@Rimbik
Thanks for your feedback! We will investigate and update as appropriate.

@Rimbik
Thank you for bringing this to our attention.
I've delegated this to content author @barclayn, who will review it and offer their insightful opinions.
@barclayn
Could you please review add comments on this, update as appropriate.

@Rimbik thank you for contacting us. #please-close


Fixed 5.2 artifact inventory UI change bug

Version 5.2 update added a sort by obtained to the top of the artifact inventory which shifted the items down a bit, additionally added a favorite icon that moved the lock icon a bit to the left.

increased size of top fill region
reduced item count per page for artifacts
moved lock bitmap to new location
reduced frequency of page scrolling backtracking

additional note: the lock bitmap location is shared with weapons, but since the weapon lock was already bugged before any changes, Im gonna leave it at that.

2024-11-22 22:31:10.867|FATAL|Program|Main application crashed|System.TypeInitializationException: The type initializer for 'InventoryKamera.GenshinProcesor' threw an exception. ---> System.IO.FileNotFoundException: Data file does not exist for Characters.
   at InventoryKamera.DatabaseManager.GetList(ListType list) in C:\Users\Admin\Downloads\Inventory_Kamera\InventoryKamera\data\DatabaseManager.cs:line 175
   at InventoryKamera.DatabaseManager.LoadCharacters() in C:\Users\Admin\Downloads\Inventory_Kamera\InventoryKamera\data\DatabaseManager.cs:line 125
   at InventoryKamera.GenshinProcesor.ReloadData() in C:\Users\Admin\Downloads\Inventory_Kamera\InventoryKamera\scraping\GenshinProcesor.cs:line 107
   at InventoryKamera.GenshinProcesor..cctor() in C:\Users\Admin\Downloads\Inventory_Kamera\InventoryKamera\scraping\GenshinProcesor.cs:line 91
   --- End of inner exception stack trace ---
   at InventoryKamera.MainForm.ValidateCustomName(Object sender, EventArgs e) in C:\Users\Admin\Downloads\Inventory_Kamera\InventoryKamera\ui\main\MainForm.cs:line 180
   at System.Windows.Forms.Control.OnParentChanged(EventArgs e)
   at System.Windows.Forms.Control.AssignParent(Control value)
   at System.Windows.Forms.Control.ControlCollection.Add(Control value)
   at System.Windows.Forms.Form.ControlCollection.Add(Control value)
   at InventoryKamera.MainForm.InitializeComponent() in C:\Users\Admin\Downloads\Inventory_Kamera\InventoryKamera\ui\main\MainForm.Designer.cs:line 1066
   at InventoryKamera.MainForm..ctor() in C:\Users\Admin\Downloads\Inventory_Kamera\InventoryKamera\ui\main\MainForm.cs:line 37
   at InventoryKamera.Program.Main() in C:\Users\Admin\Downloads\Inventory_Kamera\InventoryKamera\Program.cs:line 25
2024-11-22 22:31:13.140|INFO|Program|Application closed

Hello,
You noted "moved lock bitmap to new location".
Isn't it ignored during scanning?

The changes look good to me. I ran a full scan and it ran about as I remembered it to. @aisalexa666 lock status isn't ignored but @taiwenlee is correct in that it doesn't read weapon lock statuses correctly, which isn't the end of the world.


[C4GT] Design and develop login and signup

Description
[Provide a brief description of the feature, including why it is needed and what it will accomplish. You can skip any of Goals, Expected Outcome, Implementation Details, Mockups / Wireframes if they are irrelevant. Please note that this section of the ticket is suggestive & you can structure it as per your prerogative.]
Goals

[ ] Design and develop Signup and login UI and API integration

Expected Outcome
UI development and API integration
Acceptance Criteria

[ ] User should be able to see signup option
[ ] User should be able signup by filling all mandatory fields.
[ ] After successful of signup user can login with the same credentials

Implementation Details
Mockups / Wireframes

[Please note that the below section of the ticket has to be in the format as mentioned as it is key to enabling proper listing of the project. Please only choose the options mentioned under the headings wherever applicable.]
Product Name
Unnati
Project Name
Unnati PWA
Organization Name:
Shiskhalokam
Domain
Education
Tech Skills Needed:
HTML, CSS, javascript, Angular and Ionic
Mentor(s)
@vishwanath1004
Complexity
Medium
Category
UI/UX/Design and Integrations
Sub Category

I would like to work on this

I would Like to work on this, #17

I would like to contribute !

I would like to contribute.
On Tue, Jul 25, 2023, 17:23 vishwanath1004 @.***> wrote:

Description
[Provide a brief description of the feature, including why it is needed
and what it will accomplish. You can skip any of Goals, Expected Outcome,
Implementation Details, Mockups / Wireframes if they are irrelevant. Please
note that this section of the ticket is suggestive & you can structure it
as per your prerogative.]
Goals

Design and develop Signup and login UI and API integration

Expected Outcome
UI development and API integration
Acceptance Criteria

User should be able to see signup option
User should be able signup by filling all mandatory fields.
After successful of signup user can login with the same credentials

Implementation Details Mockups / Wireframes
[Please note that the below section of the ticket has to be in the
format as mentioned as it is key to enabling proper listing of the project.
Please only choose the options mentioned under the headings wherever
applicable.]
Product Name
Unnati
Project Name
Unnati PWA
Organization Name:
Shiskhalokam
Domain
Education
Tech Skills Needed:
HTML, CSS, javascript, Angular and Ionic
Mentor(s)
@vishwanath1004 https://github.com/vishwanath1004
Complexity
Medium
Category
UI/UX/Design and Integrations
Sub Category
—
Reply to this email directly, view it on GitHub
https://github.com/Code4GovTech/C4GT/issues/17, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS3K62CMP35NQHJ5A2C6TMLXR6XTTANCNFSM6AAAAAA2W6NI4I
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I would like to contribute

Please assign this to m6

I would love to contribute in this project


On deploy_challenge "IndexError: list index out of range"

When I try to renew a certificate, the following happens:
Processing vpn.staging-wanderio.com

Checking domain name(s) of existing cert... unchanged.
Checking expire date of existing cert...
Valid till Jan 10 15:19:00 2017 GMT (Less than 30 days). Renewing!
Signing domains...
Generating private key...
Generating signing request...
Requesting challenge for vpn.staging-wanderio.com...
CloudFlare hook executing: deploy_challenge
Traceback (most recent call last):
File "cloudflare_hook/hook.py", line 196, in
main(sys.argv[1:])
File "cloudflare_hook/hook.py", line 192, in main
opsargv[0]
File "cloudflare_hook/hook.py", line 165, in create_all_txt_records
create_txt_record(args[i:i+X])
File "cloudflare_hook/hook.py", line 103, in create_txt_record
zone_id = _get_zone_id(domain)
File "cloudflare_hook/hook.py", line 81, in _get_zone_id
return r.json()['result'][0]['id']
IndexError: list index out of range

This has worked in the past, but all of a sudden in broke (a change in CF API?).
CF_EMAIL and CF_KEY are exported in the environment in which the script runs

Ok, after manually replaying the request with PostMan I got it: the domain was actually not in CloudFlare, but still on Route53.
Maybe an error message would help.

Agreed, I'll push an update in the coming days.

Yup, encountered the same bug, was scratching my head for a better while.

  File "hooks/cloudflare/hook.py", line 84, in _get_zone_id
    return r.json()['result'][0]['id']
IndexError: list index out of range

Same. Resulting json from CF is:
  "messages": [],
  "result_info": {
    "page": 1,
    "count": 0,
    "per_page": 20,
    "total_pages": 0,
    "total_count": 0
  },
  "errors": [],
  "success": true,
  "result": []
}

Is this still an unaddressed issue @kappataumu ?

Ran into this issue on Ubuntu 18.04.
Installing python 2 requirements resolved it for me even though the system uses python2 and 3 (and I think requirements.txt downgraded two python packages).

Appears to be related to #53

Looks like cloudflare deleted my site :(

Because the get_tld changed:
tld = get_tld('http://' + domain)
res = get_tld('http://' + domain, as_object=True)
sub_domain, tld = res.subdomain, res.fld

I have the same problem and I confirm that what @kuleyang says is correct. It fixes the problem.
Thanks!

Try https://github.com/walcony/letsencrypt-cloudflare-hook to see if that has resolved an issue.  That fork is actively maintained and PRs are welcome.

Try https://github.com/walcony/letsencrypt-cloudflare-hook to see if that has resolved an issue. That fork is actively maintained and PRs are welcome.

this other fork works for me


#31: Check pending requests completion before polling sockets.

The ClusterConnection::pending_requests is the queue of requests which are not yet responded as result of ClusterConnection::read calls.
Occasionally it might have multiple entires in different states like:

Unfulfilled request (requested from node 0);
Fulfilled request (responded by node 1);
Fulfilled request (responded by node 2);

Currently calls to ClusterConnection::read will be "head of line" blocked until request on top of the queue is fulfilled. So fulfilling request on top of the queue might unblock multiple queued requests. The response to completed requests on top of the queue should not be blocked with socket read operation.

@mstyura does it definitely fix #28 ?

Yes, this PR fixes the issue #28. I can no longer reproduce it in actual app nor running test hundreds of times via echo {1..1000} | xargs -n1 -- cargo test --lib -- tests::cluster::commands_to_different_node.
An additional conclusion drawn from the fix is that the current implementation is subject to head-of-line blocking when the cluster connection receives commands routed to different nodes. The responses to the client app are delivered strictly in the order they are handled by the cluster connection. This might cause performance degradation when some nodes are significantly slower than others. While it appears this issue could potentially be addressed, it would certainly require a considerable amount of effort to be rectified properly.


format :json is not scoped to one API, it affects other APIs

We have two endpoints, one accepts application/json content-type, the other accepts text/plain. They live together. However, when format :json is included in one of our API's the other one is effected as well.
Consider the example below:
class API < Grape::API
  format :json
  get '/v1/users' do
    {'api': 'api'}
  end
end

class BAPI < Grape::API
  post '/v1/users' do
    {'bapi': 'bapi'}
  end
end

application = Rack::Builder.new do
  map '/' do
    run Rack::Cascade.new [API, BAPI]
  end
end

In this case, POST /v1/users returns {"error":"The requested content-type 'text/plain' is not supported."}. When format :json is removed, it returns {"bapi": "bapi"} as expected.

Something is fishy. Try to build a spec in Grape for this?

@dblock the behavior above is expected because Rack::Cascadecatches only the responses with 404/405 status code but the grape middleware formatter returns the response with 406 status code(when explicit json format is provided but the request has a different type). so Rack::Cascade.new([API, BAPI], 404..406) might help.
But I have a different problem here :tired_face:

When format :json is removed, it returns {"bapi": "bapi"} as expected.

I cannot repeat this on master branch.
@hantuzun which grape version is used?

Thanks!
@dblock; I experienced that with grape 0.10.1.
I achieved what I wanted by swapping the endpoints in cascade constructor array like: run Rack::Cascade.new [BAPI, API].
What's the best practice for serving two sets of methods that accepts different content-types?

0.10.0 is really old, we're on 0.18.0, just want to make sure we're not fixing something that's fixed long time ago

Oh, I don't know why bundler installed that. Thanks for the notice!
I'll try to reproduce the error on the latest release.


Generic Types

This PR adds generic types to all hooks except useFormConnect.

Codecov Report

Merging #41 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #41      +/-   ##
==========================================
+ Coverage    99.5%   99.51%   +<.01%
==========================================
  Files          14       14
  Lines         204      207       +3
  Branches       40       41       +1
==========================================
+ Hits          203      206       +3
  Partials        1        1

Impacted Files
Coverage Δ

src/useField.ts
100% <100%> (ø)
:arrow_up:

src/useFieldArray.ts
100% <100%> (ø)
:arrow_up:

src/Form.tsx
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Go SDK Modules: Fix conflicts with core types

While we namespace objects in graphql and in client codegen, in the go sdk codegen is all in the same package so if you want to name a type "Container" your code won't compile, even though it's gonna be namespaced later anyways.
This may also cause problems in the introduction of a new core type could create conflicts with existing modules and thus break them.
In terms of fixes:

We should skip namespacing if an object already has the namespace applied. Eg if a module named cool has a type called CoolContainer, we don't need to renamespace and call it CoolCoolContainer

That's probably worth doing no matter what and partially solves the problem, but it doesn't fix the issue of new core types breaking existing modules.
To solve that, we may need to just move the client codegen to a subpackage rather than be in the same. This is fraught as we need to ensure that always importing the correct codegen package is as seamless as possible.
Another option is to make core as versioned and modular as all other modules, but that's a fairly large project (though one we want in the long term anyways)

I think this is probably worth closing out?
We've had https://github.com/dagger/dagger/pull/6258 done for a while now.
In terms of introducing new core types, the solution will be more module-compat work to version the types that are added.


How to get the traceId in logback's LogstashTcpSocketAppender

Question
how can I get the traceId in logback's LogstashTcpSocketAppender please

There are a lot of appenders there. We have given your the typical examples, you should be able to do a new one, and contribute to the community.

Added.

thank you!


Taskbar not auto hiding.

I have the taskbar set to auto hide in the window setting, and it does not auto hide. After a fresh computer restart the task bar does hide.

This seemed to be a problem with the Window's store version of Slack. Once that was uninstalled the taskbar returned to the expected behavior.


es6 rewrite

you forgot to gitignore the lib

src is going to git, lib to npm

Just realized another problem, test are not gonna work via gh-pages as they reference from node_modules.
The same issue has been solved in jss-examples by copying jss from node_modules, but in this case it would be dirty as its not just examples.
Any suggestions?

Also I think we need remove jss from devDependencies.

This is not a solution, but what about removing the online test running completely. When I need to run the tests, I want to run npm run test locally. Also with some new tests for server side rendering, you will have to run them on node anyway.

yeah we def need to change testing, it needs to run in node and browser and we need a CI, so yes, for now lets remove it from running over gh-pages.

OK.
jss cannot be removed from devDependencies, because it is being used in "jss-*/test/index.html"

true, latest npm will not install peerDependencies

Yes, but I think it must be left as is, to hint the user about the compatible version of jss (while installing a plugin).

Another thing wich is a matter of preference, is the removal of bower support and xpkg. This will simplify the packaging.

I would love to remove bower.json but can we do that? People will complain, lots of them are using bower .... whats your suggestion?

I wish I had stats of bower usage, but my feeling is that it is fading out. I personally use npm only, because anything more than one package manager is overkill for me.

Also I think that bower users can install directly from github (I'm not sure)

lets check that out, I am for removing xpkg and bower too.

Bower users can install from github and also
from http://bower.io/docs/creating-packages/

While Bower does not directly use main files, they are listed with the commands bower list --json and bower list --paths, so they can be used by build tools.

I think this mean that without a bower.json they can install form github and then their build tool, will work correctly because it will read from package.json instead of bower.json (and maybe its browser field instead of main). I'm not sure.

You can point the browser field in package.json to "./dist/jss-px.js" and that's what their build tool will include.

great, lets do this!

With this change, npm users will also get the file from dist in their bundles. This will probably mess with their source maps, because their build tool will have too import the created source map of the dist file, to the source map of the bundle.

we need to make npm users use lib ....

Yes thats the correct link, I'm just not sure if bower users will have a problem with that. Probably they won't, because their build tool will read the current main field in package.json and will bundle all required files as normally. The problem is that I don't use bower and I cannot be sure, because maybe there is a slightly different workflow there.

Just took a look at popular packages here https://github.com/rackt they all don't use bower and don't specify anything special for it

I can't take that decision. I'm changing now the tests to be able to run npm run test instead of open test/index.html. I don't have an open command. Just xdg-open ;-) I'will use https://www.npmjs.com/package/opener

lets make tests refactoring a separate task, we need to rethink it and make a good solution in one go.

I think this is very small step and can be done now. It is for cross-compatibility. And we agreed to remove the online tests. So nothing changes. Only two lines in the readmes.

Do you agree?

so it will just open those tests in the browser and thats it?

Yes only that

We can do that temporarily .... but real thing would be more complicated

OK, but I think the command would be still useful, mabe with a sub name and also have one command for testing in node and one for testing with phantom(?)

or jsdom

jsdom is not an option at least for those which need real browsers ... karma with real browsers is a better fit.

yeah there are better solutions

so are we removing bower.json and xpkg?

Also I think we should add test and dist folder to .npmignore.  Npm users use tools, they don't need dist.

Ok take a look at what I did in the main repo: https://github.com/jsstyles/jss/commit/6b8fbd840e13a3736bc50642e16055b0511fcac6
This is how we get bower support without anything redudant and without xpkg.

Why did you remove the test directory? How can I run the tests in node now. I wan't to run them without forking the project somewhere else, but directly from node_modules if needed. Also the dist directory can be useful in some cases in development mode. I can tell my builder to alias the import to the dist/dist.min file (in development) and also to not parse it at all. This will speed up my build times. I do that sometimes for big libraries. Maybe ti can be useful in other ways too. It's only some kbs.

Normally when you install some module to use it as a dependency you are not running tests. If you want to change the package itself you clone it anyways. Tests are on git, just not in the npm package now.  Post of libs do it same way.

Not sure reagrding  dist though ... didn't use it for years, wondering who does ... in development to speed up builds webpack for e.g. has an incremental build feature which allows to build changed parts.

Whatever you want, this is my preference. Many libs keep test dir in npm too. I actually do the dist aliasing above incremental builds with webpack. Not for this library but it doesn't matter.

I'm saying that the only reason to remove these dirs is some kbs

yeah but also purity)

So are you ok if I do the same changes in all plugins?

https://github.com/rackt/redux/issues/579

I agree with http://gofore.com/ohjelmistokehitys/stop-using-bower/
I think we need to remove bower.json completely. Because I don't run into issues with plugins and different jss versions in peerDependencies

It makes it all more complex than I want to maintain. I think bower users should get somewhere a build with all plugins installed.

Idea: jss-dist repository where we create different builds for all possible use cases including bower support (even if I dislike bower). This way we can remove all dist folders from all other repositories.

Professional software (in my eyes) will use npm and browserify/webpack anyways. This jss-dist thing would be for semi professional and/or small websites who might consider bower or whatever else.

Also jss is a tool which solves high scale problems for web applications, its not something people who build single page landing pages will want to use in the first place. So we are targeting more professional devs with sophisticated apps.

I think bower users won't have a problem if you keep the dist dir at least in github. The point of not having build products in the git repository is valid, but I believe it can be done if there will be a problem with many PRs and diffs.
If it was my project I would remove bower.json, xpkg and have a dist dir at least in the git repo.
I think jss-dist would be a very boring repo to maintain, without a good reason.

I have already heard that 5 lines of plugins setup  is too much for some people, so jss-dist would make it for them.
Reasons jss-dist or jss-full or something:

quick setup, all plugins already installed
different builds
support for bower, npm and whatever else

Also

wdyt?

The only problem I see is that we need to release new jss-dist on every plugin or core change. Not sure how much of maintenance overhead it is.

I also think you should promote the plugin concept by not creating this monolith build that you are saying.

 But we might have different targeting groups.
Also we can first drop the dist builds completely and then if we really see complaints jss-full can be introduced. Maybe we are not gonna have those complaints because our users are smarter and use better tools.

I'm -1 for removing dist completely. Keep it at least in git. At least for now.

So you can remove the bower.json from the last patch and decide if you want the test and dist dirs in npm.

The prob is we don't know if someone needs dist or bower.json until we remove it)))

+1 on complete bower.json removal

let me tweet about it, lets see what comes out

ok no problem

Also we can provide dist versions via some cdn by pushing it before publishing.

This way bower user can use it from there, redux does it this way

I will do whatever is needed tomorrow. Just tell me what you have decided. My opinion is above. Maybe you can do it in jss repo and show me the patch (like before).

Please don't push any changes to the plugin repos or jss-examples until we decide about those PRs, because there are a lot of changes. I think tomorrow they will be all ready.

ack

Ok, take a look at my last commit, gitignore and npmignore files.
dist is now part of npm package, but not part of the repo.
cdnjs will take it from dist automatically

https://github.com/jsstyles/jss/commit/c59598d977fa43a6aae2eacac4e5442b8a1c03ba

pushed latest updates

merged!

Nice! Hoped that logos would show up in the original repos. Do you have any idea? They are at the first line of each readme and in examples->index.html.

The urls are ok and they are visible in local md preview. But not in github pages. The url is from the jss group page.

I see why, its because of http ... needs to be https, just fixed that in jss-px

Oh yeah I see the console error now. Can you make the changes?

That's one of the reasons that I proposed before to change all urls to https. But I left it for another day!

I am on it

All right

You can also change the repo's description to match the description in package.json and the first h2 in the readme.

yep will do it in all repos

the last two are already matched

In examples->index.html, a "repo" link could be added next to each "source" link.

Or better next to the plugin title

true, however examples page is very minimalistic ... at some point it should become a layout, will note it in a separate issue.

also the links should be on every example page because people might link them directly

Yes, I wanted to navigate from one plugin to another and couldn't. Only the main repo has the links

Should be probably more like a tutorial, you start with first example and go to next and next until you finished.

Examples could also contain some explanation.

It can be done in many intresting ways. Maybe another day some simple links could be added at least for now.

The next real change that I would like to add or use is a true separation of the parsing and the dom manipulation logic. A non breaking approach could be, two new repos jss-core and jss-dom that will be imported in the main jss repo. Then users could import only what they needed. I use only the parsing parts of jss for example, but I have to import everything. A breaking approach would be to keep parsing logic in the jss repo and move dom manipulation to a new jss-dom project.

Ideally yes, however need to see if the benefit is big enough, the lib is still tiny ...

There is no limit to tinyness

It will help also in the progress of the project, because a separation like this will help to add features and controlling the size a little better.

I think jss core right now is mostly dom independent. Its a very few places where it uses dom. Basically for attach/detach and for linking if linked option is true. In total dom related code is about 3 LOC

Right now top prio is better documentation, people are still confused.

Also more examples.

Yes that's more important

Also top prio is to find and demonstrate a way of using frameworks like bootstrap with jss. I think there is a good way to do so by converting it to jss via cli.

Something like this https://www.npmjs.com/package/jss-css could help maybe.

Have not used it

But I think something builded from scratch with js styles in mind, would be by far better than a fragile conversion

https://github.com/jsstyles/jss/issues/83#issuecomment-139899152

jss2jss converter needs to handle all cases like pseudo selectors and comma separated selectors and siblings and some other stuff... its already using a css parser, it just needs to be smarter and translate things closer to how jss plugins can use it.

I didn't know about it. Personally I like to avoid a conversion like this and trying to write styles from scratch, to see the real benefits. I also don't use global selectos, but they are nice to have as an option. Finally I also use normal styles in parallel for more static things.

Yes this is the best case scenario, but I understand people who want to use feature rich libs ...
Also I see a way how to avoid global selectors but still be able to use something like bootstrap. In this example you don't render bootstrap styles with global selectors, you just use those selectors to access rule props.
import bootstap from 'bootstrap'
export default {
    myButton: {
      ...bootstrap['.btn'],
      color: 'red'
    }
}

True for utilities and more but there are global parts too in bootstrap.

These global parts are where I use normal css (with pre/post/no-processor), eg. resets, grids etc.

I think those global parts should not be global in jss. Think of a component which can be used inside of any other page or app. You can't load there your reset) because it will potentially conflict

I agree but some things like grids and resets are real globals, in a context of an app.

There some good and specific css libs like normalize, which I don't see why to rewrite it in js.

The bad thing about it is that you make your component depend on some globally defined stuff. If you take your component and try to use somewhere else - it won't work as expected.
Never say never. At some point you want to use some components written for one app in another app.
Normalization libs are fine. There is just no cost at using them from jss instead of via global css.

Also if you write lib components like UI lib, which should be used in different applications with potentially different default css expectations - its much more clean to not to touch anything globally.

I try to have my eyes and ears open and I'm experimenting with js-styles the last few months.


⚠ Bofrost.be has degraded performance

In 64241e8, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1665 ms

Resolved: Bofrost.be performance has improved in d8aa268.


Define a model

It would be great if there was an easy interface to define a model without having to create a custom struct and wrap it in LogDensityModel. Its signature would be LogDensityModel(logpdf, D) and it would be something like
function LogDensityModel(logpdf, D)
    struct LogTargetDensity
        dim::Int
    end
    LogDensityProblems.logdensity(p::LogTargetDensity, θ) = logpdf(θ)
    LogDensityProblems.dimension(p::LogTargetDensity) = p.dim
    LogDensityProblems.capabilities(::Type{LogTargetDensity}) = LogDensityProblems.LogDensityOrder{0}()

    model = AdvancedHMC.LogDensityModel(LogDensityProblemsAD.ADgradient(Val(:ForwardDiff), LogTargetDensity(D)))
end

With such interface the lengthy example from the README would become
D = 3; initial_θ = rand(D)
model = LogDensityModel(x -> -sum(abs2, x)/2, D)
n_samples, n_adapts, δ = 1_000, 2_000, 0.8
sampler = NUTS(δ)
samples = AbstractMCMC.sample(
      model,
      sampler,
      n_adapts + n_samples;
      nadapts = n_adapts,
      initial_params = initial_θ,
  )

We should probably do something like:
struct LogDensityFunction{F}
    f::F
    dim::Int
end

LogDensityProblems.logdensity(ldf::LogDensityFunction, x) = ldf.f(x)
LogDensityProblems.dimension(ldf::LogDensityFunction) = ldf.dim
LogDensityProblems.capabilities(::Type{<:LogDensityFunction}) = LogDensityProblems.LogDensityOrder{0}()

which is going to be type-stable.
But IMO this is not something we should put in AdvancedHMC.jl :confused: Probably would be better suited to go in LogDensityProblems.jl itself or maaaybe AbstractMCMC.jl or something like this. @devmotion thoughts?


How to get the currentPage and set a Page?

Could you please explain me how can I get the currentPage and set a Page?
Thank you.

For now this feature is not implemented. But this can be marked as a feature request.Am 24.03.2019 18:49 schrieb D-Apps notifications@github.com:Could you please explain me how can I get the currentPage and set a Page?
Thank you.
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.


[snippets] prefix not matching forward slashes

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

VS Code Version: 1.70.2
OS Version: Windows 10 Pro

Steps to Reproduce:

Define a snippet with a prefix that begins with a forward slash
Attempt to use that snippet by typing the prefix, it will never trigger.

This is a resubmit of https://github.com/microsoft/vscode/issues/5996
I have a coding standard with a block of documentation that begins
//:Doc Purpose
And I'd like to trigger the full snippet by typing //: but this doesn't work even if editor.quickSuggestions comments:"on" is set.

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.70.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

I updated to the latest version and the issue persists.

Using the snippet below and JavaScript I see this working
	"159486": {
		"prefix": "//:",
		"isFileTemplate": true,
		"body": [
			"// command comment"
		]
	},

https://user-images.githubusercontent.com/1794099/207086237-8af560e1-c9d2-4330-984c-fbe3c65ca447.mov
@askeeve Please share you snippet and repro steps

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Implement getMatchedAppIds

Adds support for the app_control channel's getMatchedAppIds method.
Should be rebased after https://github.com/flutter-tizen/engine/pull/184 is merged.

Rebasing done.


build: add a BOM to the project

No versions updated, they will be done in a new release
This is just for a patch release

This is just for a patch release

@timyates we cannot release a new module in a patch release. This should be in a minor release of rx-java3. We can do that tomorrow.

@sdelamo Cool, thanks 👍
Question for tomorrow, should I have updated
githubCoreBranch=3.2.x

In gradle.properties?
See you tomorrow

@sdelamo Cool, thanks 👍
Question for tomorrow, should I have updated
githubCoreBranch=3.2.x

In gradle.properties?
See you tomorrow

We will need to merge these PRs:
https://github.com/micronaut-projects/micronaut-rxjava3/milestone/10


Add phase to Content Item

In order to mark an item as either Alpha or Beta a phase attribute
has been added. This should only be alpha, beta or nil and can be
used on the Frontend to alter functionality or visual elements based on
its' state.

@alext Sorted both of your comments.


solution: urls fixed in README

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.