Build succeed with "hot reload", but fail when started from fresh "hugo server -D", resource became

What version of Hugo are you using (hugo version)?

$ hugo version
Hugo Static Site Generator v0.70.0-7F47B99E windows/amd64 BuildDate: 2020-05-06T11:17:50Z

Does this issue reproduce with the latest release?
yes
In a template, this builds fine, always :

	{{ $source := .Params.featured_image }}
	{{ printf "%#v" $source }}
	{{ $featuredImageResource := .Resources.GetMatch (print $source) }}
	{{ printf "%#v" $featuredImageResource }}

Then if I add one line, save my file, and this build fine in "hot reload" (watch), but fails if I build from a fresh "hugo server -D" command

	{{ $source := .Params.featured_image }}
	{{ printf "%#v" $source }}
	{{ $featuredImageResource := .Resources.GetMatch (print $source) }}
	{{ printf "%#v" $featuredImageResource }}
	{{ $test := $featuredImageResource.ResourceType }}

It fails on the last line, the resource "$featuredImageResource" became nil

Error: Error building site: failed to render pages: render of "taxonomyTerm" failed: execute of template failed: template: _default/list.html:16:3: executing "main" at <.Render>: error calling Render: failed to execute template ["post"] v: "C:\Users\Gilles\Projects\blog\themes\twenty-twenty-slim\layouts\_default\post.html:62:35": execute of template failed: template: _default/post.html:62:35: executing "_default/post.html" at <$featuredImageResource.ResourceType>: nil pointer evaluating resource.Resource.ResourceType

Thanks for getting in touch, but it feels like this is a question  that would be better suited to the Hugo Discussion Forum.  We prefer to use GitHub issues only for bugs and enhancements. Feel free to update this issue with a link to the re-posted question (so that other people can find it) or add some more details if you feel this is a genuine bug. You may also find the answers you search for in Hugo Documentation.

Ok, I have exactly this problem. Currently running
hugo version
Hugo Static Site Generator v0.75.1/extended darwin/amd64 BuildDate: unknown
on macos 10.15.7 catalina.
I have not found any forum post regarding this. Is it solved or is there any help available on this?


Edit workspace recommendations prompts for single folder

Testing #37186

start on an empty folder
open the extensions view
click on the edit icon in the 'workspace recommendations' section
prompt shows up with a single option
-> no prompt should be shown

The action being used here is Configure Recommended Extensions (Workspace Folder) which brings up the prompt even when there is just a single folder.
@sandy081 If you agree, then I can add a check on the number of workplace folders and skip this prompt if there is only one folder

Sure. But I use the workspace folder selection picker from @bpasero. Not sure if we would like to move that to the general picker?

We dont need to change the picker. Just skip calling the picker when there is a single root.
See https://github.com/Microsoft/vscode/commit/46280eecb11c796ea9e3f5056c25132addf8b022


Is there any way to add a user in multiple tenants with different levels of permissions in each tenant?

We have a private forum with multiple chat groups (tenants) and we have an intermediate-level moderator that has access to 5 or more chat groups, with different levels of permission. He has "modify and remove" permissions in 2 chat groups and is a normal user in other chat groups. How is it possible to implement it with this library?

Hi @SalehBagheri,
I'm pretty sure this library can't help you. I say that because a user can't be in control of other groups (tenants) and also be a normal user in another group.
That's because of the "a user can only have one tenant" limitation (see the end of the Roadmap. If that limitation was taken away it might be possible, but a) the user would need to log out and log into their other tenant and b) its quite hard to implement and no one has asked for it.


Update README.md

Describe at a high level the solution you're providing
A clear and concise description of what you changed.
Is this a patch, a minor version change, or a major version change
Patch, minor, or major.
Is this related to an open issue?
Please provide the link.
Additional context
Add any other technical detail or considerations here.

Thank you!


Feedback to Files App + Recommendation

Hi first of all i want to say that i really enjoy the app.
I just miss one feature:
A Rating-funtion with a 0to5 star appearance in an easy way.
I tested plenty of File-explorers and none of these (around 15to20) did not have any feature to set a rating for folders or data in these.
You had to go into properties always and had to set a rating in a very clumsy way.
So far so good.
Greetings

Please make a new issue following the issue templates.


James: unset JAVA_TOOL_OPTIONS env variable when running james-cli

GIVEN I run james-cli in kubernetes
THEN the pod crashes and reboots
Because the james-cli runs JVM it catches the environment variables set by James. As such it always pretouch 3GB of RAM. When added to memory consumed by James it exceeds the limit and pod get's OOM-killed.

Is the CI not picking up the newly opened PRs today?

Is the CI not picking up the newly opened PRs today?

It seems so...

Seems it did meanwhile :) Merging


Villager trades not working with vanilla non-stacking items.

Villager trades aren't working quite right since installing SimpleStack.
If I trade a villager for a vanilla non-stacking item, like an Enchanted Book of Mending, the villager will give me the item, but not take the Emeralds - breaking world economy.  I tested this with Suspicious Stew as well and it's the same effect where Villager does not take payment.
Vanilla stacking items. like paper or leather, seem to exchange just fine where the villager takes the Emeralds during trade.
Running PaperMC-204 on Minecraft 1.16.3. Survival mode.

Thank you for the bug report! The latest release should have fixed this bug. If you're still having issues with villager trading please add another comment explaining the issue and I will look into it.


:wrench: Setup environment

closes #1

Notes

Make consistent scss + tailwind code
Write good api layer in api service
Bro, what is that auth guard)

Important

Think about JWT implementation in Angular, rather than in backend. Maybe it would be faster and easier


Unauthenticated users not prompted to log in when clicking on Help tab

"Authentication Required to use Help Desk" is shown, but no indication is given regarding how to authenticate.
perhaps loading the "Help" tab as an unauthenticated user should load a login prompt either inline or with a modal or something.

done


Cedar/Proto_PPP: Fix EAP identifier duplication

Currently EAP identifier starts from zero and causes duplication between the first packet (usually EAP Identity Request) and the second (usually EAP TLS Start).
This change uses unified identifier NextId which is already used by LCP and MSCHAPv2.
Moreover, EAP Success and Failure packets should use the same identifiers as Response so incorrect comments regarding hacks were removed.
Fixes https://github.com/SoftEtherVPN/SoftEtherVPN/issues/1350
Changes proposed in this pull request:

Fix EAP identifier duplication and use unified identifier with LCP and MSCHAPv2

Thank you very much for your contribution!

@davidebeatrici @metalefty Could you please ping me next time when something is wrong with Proto_PPP? I accidentally stumbled upon this pull request, but the information here is really important for me (also because I actually wrote the code, so I feel like it would be nice to be notified of all related bugs).

Sure!


[Feat] - does a java version exist ?

does a java version exist ?

Hey @Dayvin30 welcome to the library.
The best place to ask questions would be in discussions
You can spot on the README:
Java Sample
This library works with Java and Kotlin code


Resolve NoMethodError on brew install

before:
 $ brew install homebrew/portable-ruby/portable-ruby
==> Fetching dependencies for homebrew/portable-ruby/portable-ruby: homebrew/portable-ruby/portable-libyaml and homebrew/portable-ruby/portable-openssl
==> Fetching homebrew/portable-ruby/portable-libyaml
==> Downloading https://github.com/yaml/libyaml/releases/download/0.2.5/yaml-0.2.5.tar.gz
==> Downloading from https://objects.githubusercontent.com/github-production-release-asset-2e65be/2699072/83e7a200-a461-11ea-9884-f4fc06ba83bc?X-Amz-Algo
################################################################################################################################################## 100.0%
==> Fetching homebrew/portable-ruby/portable-openssl
==> Downloading https://curl.se/ca/cacert-2023-08-22.pem
################################################################################################################################################## 100.0%
==> Downloading https://www.openssl.org/source/openssl-1.1.1w.tar.gz
################################################################################################################################################## 100.0%
==> Fetching homebrew/portable-ruby/portable-ruby
==> Downloading https://cache.ruby-lang.org/pub/ruby/2.6/ruby-2.6.10.tar.xz
################################################################################################################################################## 100.0%
==> Installing portable-ruby from homebrew/portable-ruby
==> Installing dependencies for homebrew/portable-ruby/portable-ruby: homebrew/portable-ruby/portable-libyaml and homebrew/portable-ruby/portable-openssl
==> Installing homebrew/portable-ruby/portable-ruby dependency: homebrew/portable-ruby/portable-libyaml
Error: An exception occurred within a child process:
  NoMethodError: undefined method `humanize' for "big_sur":String

after:
 $ brew install homebrew/portable-ruby/portable-ruby
==> Downloading https://formulae.brew.sh/api/formula.jws.json

==> Fetching dependencies for homebrew/portable-ruby/portable-ruby: homebrew/portable-ruby/portable-libyaml and homebrew/portable-ruby/portable-openssl
==> Fetching homebrew/portable-ruby/portable-libyaml
==> Downloading https://github.com/yaml/libyaml/releases/download/0.2.5/yaml-0.2.5.tar.gz
Already downloaded: /Users/dug/Library/Caches/Homebrew/downloads/dc25902bf97807278d36e976dafb46da852c50025c8492c7d66eeee75a8aacf4--yaml-0.2.5.tar.gz
==> Fetching homebrew/portable-ruby/portable-openssl
==> Downloading https://curl.se/ca/cacert-2023-08-22.pem
Already downloaded: /Users/dug/Library/Caches/Homebrew/downloads/5189855c1bb1eb307e142401724978880a272b70d94939c3c91a51fc34568cc4--cacert-2023-08-22.pem
==> Downloading https://www.openssl.org/source/openssl-1.1.1w.tar.gz
Already downloaded: /Users/dug/Library/Caches/Homebrew/downloads/6300164b52319bb37dce9e31e2afc01b0daa2996027c1099d5ee5c520a06eb79--openssl-1.1.1w.tar.gz
==> Fetching homebrew/portable-ruby/portable-ruby
==> Downloading https://cache.ruby-lang.org/pub/ruby/2.6/ruby-2.6.10.tar.xz
Already downloaded: /Users/dug/Library/Caches/Homebrew/downloads/41a77b75a907d1f098a05fa8fbb8584d98ee230848263952c7078a0ea2fe5068--ruby-2.6.10.tar.xz
==> Installing portable-ruby from homebrew/portable-ruby
==> Installing dependencies for homebrew/portable-ruby/portable-ruby: homebrew/portable-ruby/portable-libyaml and homebrew/portable-ruby/portable-openssl
==> Installing homebrew/portable-ruby/portable-ruby dependency: homebrew/portable-ruby/portable-libyaml
Warning: You are building portable formula on 14.
As result, formula won't be able to work on older macOS versions.
It's recommended to build this formula on macOS Big Sur
(the oldest version that can run Homebrew).
==> ./configure --build=aarch64-apple-darwin23.0.0 --host=aarch64-apple-darwin20.1.0 --enable-static --disable-shared
==> make install
🍺  /opt/homebrew/Cellar/portable-libyaml/0.2.5: 8 files, 209.6KB, built in 17 seconds
==> Installing homebrew/portable-ruby/portable-ruby dependency: homebrew/portable-ruby/portable-openssl
Warning: You are building portable formula on 14.
As result, formula won't be able to work on older macOS versions.
It's recommended to build this formula on macOS Big Sur
(the oldest version that can run Homebrew).
==> perl ./Configure --prefix=/opt/homebrew/Cellar/portable-openssl/1.1.1w --openssldir=/opt/homebrew/Cellar/portable-openssl/1.1.1w/libexec/etc/openssl no-shared darwin64-arm64-cc enable-ec_nistp_64_gcc_128
==> make
==> make test
==> make install_sw
🍺  /opt/homebrew/Cellar/portable-openssl/1.1.1w: 119 files, 9.4MB, built in 1 minute 57 seconds
==> Installing homebrew/portable-ruby/portable-ruby
Warning: You are building portable formula on 14.
As result, formula won't be able to work on older macOS versions.
It's recommended to build this formula on macOS Big Sur
(the oldest version that can run Homebrew).
==> ./configure --build=aarch64-apple-darwin23.0.0 --host=aarch64-apple-darwin20.1.0 --enable-load-relative --with-static-linked-ext --with-out-ext=tk,sdbm,gdbm,dbm --without-gmp --enable-libedit --disable-insta
==> make RUN_OPTS=/private/tmp/portable-ruby-20230930-26428-o3vmog/ruby-2.6.10/tool/runruby.rb --extout=.ext
==> make install RUN_OPTS=/private/tmp/portable-ruby-20230930-26428-o3vmog/ruby-2.6.10/tool/runruby.rb --extout=.ext
==> Caveats
portable-ruby is keg-only, which means it was not symlinked into /opt/homebrew,
because portable formulae are keg-only.

If you need to have portable-ruby first in your PATH, run:
  echo 'export PATH="/opt/homebrew/opt/portable-ruby/bin:$PATH"' >> ~/.zshrc

For compilers to find portable-ruby you may need to set:
  export LDFLAGS="-L/opt/homebrew/opt/portable-ruby/lib"
  export CPPFLAGS="-I/opt/homebrew/opt/portable-ruby/include"

For pkg-config to find portable-ruby you may need to set:
  export PKG_CONFIG_PATH="/opt/homebrew/opt/portable-ruby/lib/pkgconfig"
==> Summary
🍺  /opt/homebrew/Cellar/portable-ruby/2.6.10_1: 1,393 files, 24.2MB, built in 1 minute 49 seconds

(I'm open to the suggestion that I have something misconfigured.)

Thanks @dduugg!


How to observe rerender event?

I notice that after about 10 seconds, the list of emails was rerendered and all the customized dom was disapeared. I tried to use gmail.observe poll and refresh but it didn't fired any event. Does anyone found a way to workaround?

New or old gmail? Any folder?  Just how do you reproduce this behaviour?
I'll need a lot more information to be able to offer advice for this one.

Opps, my mistake. It's new UI. My current workaround is use mutation-summary to detect custom dom disapear event and then rerender the UI.

Is there any gmail.js solution for this? If you watch the inspector on the email table on the new Gmail, you can see all email flashes and is re-rendered, any dom changes are gone.

Never tried to customize this page. If there's a way, it probably involves adding custom observers/event-handlers and combining it with the tools that GmailJS provides.

@josteink 👍 thanks for the thoughts. started using mutation observers, but it's not working reliably. perhaps something gmail does to prevent it.


CreateComponent bubbles to parent component

Feature
BC break - probably not
I added createComponent($name) method, which bubbles to parent if component is not found in current class.
This change is useful eg to include component into itemDetail and/or other templates. Templates belong to Datagrid, so there is current no way to use {control} macro in templates without overriding Datagrid.

Hi @peldax. Could you please add some tests? I don't know what side effects it could have.

Hi,
I could have back then, but its been more than 2 years and we no longer use this datagrid.
Feel free to close this PR.

What are you using instead of it? Just curious.

In projects with generated frontend we opt to use our private fork. Otherwise we have frontend/client in completely different technology.


[Bugfix][CI/Build] Fix testing for generated commit hash

@mgoin  can you take a look at https://github.com/vllm-project/vllm/pull/6420 ?

@DarkLight1337  I think #6420 should be merged, and this should be closed.

Closing as superseded by #6420.


Can't do adb backup on Kiwi Browser

Since I'm not root, I'm trying to use adb backup to backup app data. But when I do, all I'm left with is a 37 byte file.

This seems to be because allowBackup is set to false. Was this manually added in, or was it from the rebate?

It's following what is Chromium, this one is a bit tricky, I wonder if it could security issues, this is why allowBackup is not enabled, and it requires serious investigation before turning this flag on (as it could eventually otherwise allow all passwords to be "backuped" by third-party app)

However, to backup, you need to expressly give the app permission. There's no way to do it in the background. Or maybe make it a toggle?

Wait, really? Oh, that's going to be a problem (I was doing backups using adb for a while). Ok, yeah, you could try making a zip folder of all the pertinent folders (it should be found in the other backup thread).

How would you add a button in the settings to do an action?

        android:key="captions"
        android:title="@string/accessibility_captions_title"/>

This is a button in the "Setting" screen.
To add the translation string and the preference in the layout (xml): https://github.com/kiwibrowser/src.next#adding-a-new-setting
To bind an action on-click:
https://github.com/kiwibrowser/src.next/blob/kiwi/chrome/android/java/src/org/chromium/chrome/browser/accessibility/settings/AccessibilitySettings.java#L111
Btw @DUOLabs333 you have build access already (in your repo, you just have to do, Any branch: Build apk)
You can try with Accessibility Settings, as it's easy to experiment there, but it is possible to do the same with main_settings too

Even better than in Accessibility Settings:
To add in the Main Settings (home of the "Settings" screen), actually it's there:
https://github.com/kiwibrowser/src.next/blob/kiwi/chrome/android/java/src/org/chromium/chrome/browser/settings/MainSettings.java#L161
and
https://github.com/kiwibrowser/src.next/blob/kiwi/chrome/android/java/res/xml/main_preferences.xml

I can't figure out how to make a page appear when clicked on the "Export" button. Best I found was to open a system setting page.

getTabModel/getTabCreator
https://github.com/kiwibrowser/src.next/blob/258c95b5c85c8b806f33d0aaac5b241a2b9e975f/chrome/android/java/src/org/chromium/chrome/browser/ChromeTabbedActivity.java#L767
(if you mean an HTML page)

@kiwibrowser I saw your most recent commit, but it's still not very clear what to do. Where are the other setting screens located, so I could use that as a base?

Oh, I added it today actually (because I realized I forgot a file):
https://github.com/kiwibrowser/src.next/blob/9669e9f9b7d380f1bde9098fc8e1efb8749a197b/chrome/android/java/src/org/chromium/chrome/browser/settings/RadioButtonGroupTabSwitcherPreference.java + https://github.com/kiwibrowser/src.next/blob/4696cda1eb1df3805e6c7484604f16c301daac0e/chrome/android/java/res/xml/tabswitcher_preferences.xml + https://github.com/kiwibrowser/src.next/blob/kiwi/chrome/android/java/res/xml/main_preferences.xml#L107 + https://github.com/kiwibrowser/src.next/blob/3318188de825fbc79d1d17f7273bf168f649fb8e/chrome/android/java/src/org/chromium/chrome/browser/settings/TabSwitcherSettings.java
android:fragment="org.chromium.chrome.browser.settings.TabSwitcherSettings"
=> to open the fragment called org.chromium.chrome.browser.settings.TabSwitcherSettings
So I think this is what you are looking for ("opening a fragment", it's a PreferenceFragmentCompat)

I'm trying to merge it to my fork, but there are a bunch of merge conflicts, even though I only changed the main settings xml, and I have no idea why.

And even though you just added the files, "Tab Switcher" still works on my fork.

It's because with @d3ward  we fixed a lot and a lot and a lot of things in the last days and had to do a couple of force pushes.
To solve this:
git switch kiwi
git remote add upstream https://github.com/kiwibrowser/src.next.git
git fetch upstream
git pull upstream --rebase kiwi
(or if it doesn't work: git rebase -i upstream/kiwi)
git push origin --force

If you already have it in the workflow, you can use "Actions"
\_ Forked kiwi: Rebase on top of kiwibrowser:kiwi branch
and that will do it for you :)

Wow, you really have though of everything, huh?

Oh, I have to add it manually.

It will come once you are rebased / in sync with upstream.
It's a recent addition.
Sorry for all this mess, but I was testing (and still testing) the automatic rebase and this broke a bit the commit history.
But if you run the commands it should be fine.
You are among the pioneers :)

I tried to run it, failed at error: could not apply 3078529... Create rebase_kiwi_on_top_of_kiwi.yml (I had to add the file manually).

Ah, you created rebase_kiwi_on_top_of_kiwi.yml - so now it's conflict with upstream
You can just give me write access to the repo and I can solve it for you just by running the commands.

Invited you.

Voila, all fixed :)

Perfect

This sync system is a bit tricky and going to improve next week I think.
In summary:
[kiwibrowser:kiwi]
is sitting on top of
[kiwibrowser:chromium]
however we often change the commit where [kiwibrowser:kiwi] starts from (because we change [kiwibrowser:chromium] at every new version).
So as a result, all the commits in [kiwibrowser:kiwi] often get new IDs, and this easily confuses Git. As Git is not really a rebase-first platform, but rather a merge tool.
However, if we merge kiwi and chromium everything would be easier for forks, but to upgrade the chromium version would be very difficult, because kiwi and chromium code would be blended together.
Though maybe this will be a solution, who knows.
This is just to explain, why your fork couldn't find a why to attach to the main repository kiwibrowser:kiwi, and why this special action exists (to reattach/rebase on top of kiwibrowser:kiwi)

I understood there is a problem:

The build is broken (I fixed, I broke it yesterday accidentally)
You need to have a way to add a java file in the build

About 2) I think I can fix it in a few hours

That may be why my changes aren't compiling -- or that I messed something up :) Is it fixed in the main repository?

I'm trying to make a settings page for ExportImport.java (instead of having it split into two files like the Tab Switcher). I'm planning on getting the basics before customizing it, but it wouldn't compile. It may be because of an error (I'm more comfortable with Python), or something else:
ExportImport.txt

As surprising as it may seems, you are the first user to try to add a .java file :)
so I need to add the file where you can select what are the .java files that are packed and compiled in the APK.
Your changes weren't compiling because there was a bug in the deployment script (that was unrelated)

In a couple of hours I'll point you to the right file where you need to add your resources + java file :)
and then you'll be able to compile and it'll work

Error: Found files not listed in the sources list of the BUILD.gn target:
../../components/browser_ui/styles/android/java/res/drawable-hdpi/ic_devtools.png
../../components/browser_ui/styles/android/java/res/drawable-mdpi/ic_devtools.png
../../components/browser_ui/styles/android/java/res/drawable-xhdpi/ic_devtools.png
../../components/browser_ui/styles/android/java/res/drawable-xxhdpi/ic_devtools.png
../../components/browser_ui/styles/android/java/res/drawable-xxxhdpi/ic_devtools.png

I'm working on fixing this + adding the possibility to add java files and custom ressources

I fixed the build script
chrome/android/chrome_java_resources.gni -> to add a png file
chrome/android/chrome_java_sources.gni -> to add a java file
If you can try @DUOLabs333  :)

and you can just run build apk and it should be working fine now :)
before, you need to rebase on top of \_ Forked kiwi: Rebase on top of kiwibrowser:kiwi branch

Yeah, I tried about an hour ago and it failed (could you please stop rewriting history :)). I could give you access if needed.

Yes, let's do that :)

Added.

@DUOLabs333 finally fixed!
Now it's good and nice, and as I promised (just with a couple of hours late)
I fixed the build, plus I unified everything in 2 files that are easy to manipulate:
https://github.com/kiwibrowser/src.next/tree/kiwi/chrome/android
If you need to add a resource: chrome/android/chrome_java_resources.gni
If you need to add a new java class: chrome/android/chrome_java_sources.gni
If you want the latest changes of kiwibrowser:src.next, you can run the action \_ Forked kiwi: Rebase on top of kiwibrowser:kiwi branch and it will do everything for you.
Thank you very much (again!) for being one of the early user of this system and my apologies on blocking your workflow, your help is very appreciated.

Why does Build APK fail with
../../chrome/android/java/src/org/chromium/chrome/browser/settings/RadioButtonGroupExportImport.java:68: error: invalid method declaration; return type required
    public RadioButtonGroupExportImport(Context context, AttributeSet attrs) {
           ^

when RadioButtonGroupTabSwitcherPreference.java is allowed through, even though most of the code is copied from it?

This is because your class is named
public final class RadioButtonExportImport
and it should be
public final class RadioButtonGroupExportImport
(always the same as the filename and always the same as the constructor)
when you write public RadioButtonGroupExportImport the compiler thought you wanted to create a new method inside the class (it didn't know it was a constructor), so it asked you to add a return type to have like public some_type method_name.
However, this is just a confusing error message because the real issue is not with the constructor, but rather that the class name was not correct.

Ah, must have overlooked it -- thanks.

Well it took me time to realize too :D
As a side-note: It is a quite complicated task what you are trying to achieve but I think it's doable 100% from Android native if you are looking to take a snapshot of the profile.
You may find more satisfaction (in terms of people feeling grateful) by taking small issues as they are +/- instant gratification (and get familiar with the codebase)
During the next days I'm going to make importing new files from Chromium easier, and work on an interactive rebase solution (to be able to resolve conflicts either from Chromium, or with other Kiwi repos).
For now, if you want to fetch the latest kiwibrowser:src.next changes in your repo you can use "_ Fork: Rebase on top of kiwibrowser:src.next"

Profile = user profile, settings, passwords and extensions.
About protecting the export functionality, you can see in the Passwords settings screen (3 dots menu), when you press Export password, there is a very cool PIN protected export example (you can find the code and cs.chromium.org )

Ah, I understand. But for some reason, no matter what I do, I can't get my page to show up. Is there an option for error logs?

For now via:
adb logcat
(But this requires a computer connected to the phone or a rooted phone but maybe there is a better way?)

That could work, but a lot of it will have the noise of Android processes.

(But this requires a computer connected to the phone or a rooted phone but maybe there is a better way?)

If you're on android 11 you can use LADB (it's 3 dollars on playstore but you can compile the source if you want) and maybe Shizuku (you launch Termux with adb permissions or something, not really knowledgeable about it). They take advantage of the Wirelesss ADB feature and create a local adb server in the phone itself that connects to it.

I think there may be a possibility to create kiwi://adb or kiwi://logs
I'm looking into it

If you're on android 11 you can use LADB (it's 3 dollars on playstore but you can compile the source if you want) and maybe Shizuku (you launch Termux with adb permissions or something, not really knowledgeable about it). They take advantage of the Wirelesss ADB feature and create a local adb server in the phone itself that connects to it.

Update on this, I was looking through the LADB forks and someone already made an environment to build it through github actions here, so you can just fork it and do any commit and it'll auto build it for you. 😀

Very cool discovery @Delphox !


[QA] Dev Branch

[ ] Issue with GSI feature on CNN site
[x] Error: Uncaught (in promise) Error: No tab with id: 363275767.
[x] CDP switch and 0 cookies issue
[x] Error: Uncaught (in promise) TypeError: Cannot set properties of undefined (setting 'newUpdates')
[ ] Blank Table and blank screen issue
[x] Regression bug : Existing core functionality is breaking
[ ] PSAT CLI - Samesite does not show any value. It is empty.
[x] CLI: In the "Technologies" frame, the description is not clearly visible.
[x] CLI: If there is no description present for any technology then the proper message should be visible.
[x] CLI: The EXP date column shows wrong format for some cookies.
[x] Blocked cookie are not appearing Yellow BG color in CLI Dash board.
[ ] PSAT Extension - The "Unknown frames" is visible with URL rewrite.

Issue with GSI feature on CNN site
When I opened the CNN website with single tab mode, it kept processing but the GSI feature was not found. When I inspect the tool it threw the below errors on console.

This started working as expected when I opened one more new chrome profile.

Uncaught (in promise) Error: No tab with id: 848821757.
https://github.com/GoogleChromeLabs/ps-analysis-tool/assets/15173772/82eccbb4-2d13-40e1-afbf-0473ddce9cb6

CDP switch issue
Steps:

Open chrome://extension tab
Now, open 2-3 tabs with different URLs
Enable CDP and you must see the debug bar
Anlyse the different tabs in single tab environment and observe correct set of data should appear as CDP is enable. E.g. Blocked cookie, Yellow BG color etc
Now, Keep checking the chrome://extensions tab for any errors
Close each tab one by one and you will notice the error as mentioned here.
Now, repeat step 2 and observe that
- the debug bar is not reappearing
- No blocked cookies
- No other cookies getting loaded
- Cookies landing page shows 0 cookies
- CDP switch is still on but you won't get above thing

https://github.com/GoogleChromeLabs/ps-analysis-tool/assets/15173772/295b2d7c-6170-4302-a0e3-556a4d75e4e5

Error: Uncaught (in promise) TypeError: Cannot set properties of undefined (setting 'newUpdates')

Blank Table and blank screen issue
When I toggle the setting - multi tab and/or CDP; the blank table and blank screen issues are still visible. Especially with multi tab environment.

Multi tab -
https://github.com/GoogleChromeLabs/ps-analysis-tool/assets/15173772/a59a66b4-efcc-46e6-ab2c-bd7955b9d77a

Single Tab -
https://github.com/GoogleChromeLabs/ps-analysis-tool/assets/15173772/439347d9-f5b4-412a-a1d6-e9d19ab00945

Regression bug : Existing core functionality is breaking
As per the expected behaviour if the URLs in next to chrome://extensions page or if it is tab num - 2 then it should by default load the cookies. However, wit this branch this functionality is breaking.
Steps:

Checkout to develop branch > Pull the code > Run npm start
Open a new chrome profile and install the extension
Open a new tab with any URL
Observe that it loads cookies as soon as you hit the URL
Now, close the tab and again open the new tab with any URL
Observe that this time it does not load the cookie and shows "Analyse this tab" button instead

https://github.com/GoogleChromeLabs/ps-analysis-tool/assets/15173772/6b0d1142-d299-4136-acd5-5c7d63102011

Regression bug : Copy to clipboard is not working.
On the click of "Copy to clipboard," the content fails to copy. Furthermore, upon pressing Ctrl + V, the content does not appear.
copytoclipboard.webm

@kjnanda
The infinite loading issue was fixed in https://github.com/GoogleChromeLabs/ps-analysis-tool/pull/478 and QA tested by Pavan.
We cannot replicate the false positive issue on CNN after several attempts so we will handle in later.

CLI: If there is no description present for any technology then the proper message should be visible.

PSAT CLI - Samesite does not show any value. It is empty.

PSAT CLI
PSAT Extension

@ayushnirwal Blocked cookies are not appearing Yellow BG color in CLI Dash board.

https://github.com/GoogleChromeLabs/ps-analysis-tool/issues/471#issuecomment-1932016503

Hi @amovar18, It is still reproducible on the develop branch.
Check the below steps:

Go to bbc.com and open the cookies frame.
Now, rewrite the URL to example.com and check the unknown frames will be shown. It should not be displayed.
Now, refresh the page the page will be refreshed and unknown frame frame will be gone but cookies will be displayed.

screencast-bpconcjcammlapcogcnnelfmaeghhagj-2024.02.13-13_46_32.webm

Hi @amovar18, still able to replicate this issue.

@pavanpatil1 the issue is replicated is different than what you have shown, because the issue that was listed shows unknown frames in the sidebar which have been removed in 0.6.0.

The unknown frames have been removed and other issues have been resolved. Hence closing this issue


When using oneOf/dependencies default is not being populated

Prerequisites

[x] I have read the documentation;
[x] In the case of a bug report, I understand that providing a SSCCE example is tremendously useful to the maintainers.
[x] Ideally, I'm providing a sample JSFiddle or a shared playground link demonstrating the issue.

Description
Default value does not appear when adding a value, using a combo of array, oneOf, and dependencies:
https://mozilla-services.github.io/react-jsonschema-form/#eyJmb3JtRGF0YSI6eyJjb21wb25lbnRzIjpbeyJ0eXBlbmFtZSI6IkEifV19LCJzY2hlbWEiOnsicHJvcGVydGllcyI6eyJjb21wb25lbnRzIjp7Iml0ZW1zIjp7ImRlcGVuZGVuY2llcyI6eyJ0eXBlbmFtZSI6eyJvbmVPZiI6W3sicHJvcGVydGllcyI6eyJkYXRhIjp7InR5cGUiOiJvYmplY3QiLCJwcm9wZXJ0aWVzIjp7ImV4YW1wbGUiOnsidHlwZSI6InN0cmluZyIsImRlZmF1bHQiOiJ0ZXN0In19fSwidHlwZW5hbWUiOnsiZW51bSI6WyJBIl19fX0seyJwcm9wZXJ0aWVzIjp7ImRhdGEiOnsidHlwZSI6Im9iamVjdCIsInByb3BlcnRpZXMiOnsiZXhhbXBsZSI6eyJ0eXBlIjoic3RyaW5nIiwiZGVmYXVsdCI6InRlc3QifX19LCJ0eXBlbmFtZSI6eyJlbnVtIjpbIkIiXX19fV19fSwicHJvcGVydGllcyI6eyJ0eXBlbmFtZSI6eyJlbnVtIjpbIkEiLCJCIl0sInR5cGUiOiJzdHJpbmcifX0sInJlcXVpcmVkIjpbInR5cGVuYW1lIl0sInR5cGUiOiJvYmplY3QifSwidHlwZSI6ImFycmF5In19LCJ0eXBlIjoib2JqZWN0In0sInVpU2NoZW1hIjp7ImZpcnN0TmFtZSI6eyJ1aTphdXRvZm9jdXMiOnRydWUsInVpOmVtcHR5VmFsdWUiOiIifSwiYWdlIjp7InVpOndpZGdldCI6InVwZG93biIsInVpOnRpdGxlIjoiQWdlIG9mIHBlcnNvbiIsInVpOmRlc2NyaXB0aW9uIjoiKGVhcnRoaWFuIHllYXIpIn0sImJpbyI6eyJ1aTp3aWRnZXQiOiJ0ZXh0YXJlYSJ9LCJwYXNzd29yZCI6eyJ1aTp3aWRnZXQiOiJwYXNzd29yZCIsInVpOmhlbHAiOiJIaW50OiBNYWtlIGl0IHN0cm9uZyEifSwiZGF0ZSI6eyJ1aTp3aWRnZXQiOiJhbHQtZGF0ZXRpbWUifSwidGVsZXBob25lIjp7InVpOm9wdGlvbnMiOnsiaW5wdXRUeXBlIjoidGVsIn19fX0=
Steps to Reproduce

Click the above link in the playground.
Remove and add a component
Select type A or B
Observe that the default value of test for field example is not populated.

Expected behavior
Expect the default value to appear in the example field.
Actual behavior
The example field is always empty and no default value is populated.
Version
"react-jsonschema-form": "^1.7.0",

I would also just mention that I saw this PR was completed: https://github.com/mozilla-services/react-jsonschema-form/pull/1338 which is the reason I upgraded to 1.7.0, but this behavior is still occuring and occurs on the playground too (assuming the playground is running 1.7.0...)

This would be really nice to get resolved, it would greatly simplify our form process

This will actually be fixed by #1376 -- see the following test case: https://github.com/rjsf-team/react-jsonschema-form/pull/1376/files#diff-0de170772e45d2602c103dd916607787R862

This cannot be reproduced any more


Make CopyLogDataWorker conform to WITSML specification

When attempting to copy log data to a log on the Petrolink server, and the mnemonics do not exist on the target, the query that attempts to create the mnemonics in VerifyTargetHasRequiredLogCurveInfos fails with error -436 "When an update adds a new column-identifier to a systematically growing data-object, a structural-range must not also be specified. ".
The query should most likely be adjusted, so that the LogCurveInfos have their minIndex, maxIndex, minDateTimeIndex, and maxDateTimeIndex set to null.
Given that the LogCurveInfos come from the targetLog parameter, the LogCurveInfo changes should be performed on a clone of the object, so that we do not mutate targetLog. Possibly a clone operation should be implented for LogCurveInfo (or eventually for all WITSML objects).
Also, when the -436 error occurs in VerifyTargetHasRequiredLogCurveInfos, the job continues, disregarding the error (eventually leading to -452 because the LogCurveInfos are missing on the target). VerifyTargetHasRequiredLogCurveInfos should cause the whole job to fail on unsuccesful result.

I'm not able to reproduce this bug. It is either already fixed, or it was fixed when Petrolink updated a schema on their server.


Add tabulate to install_requires in setup.py.

To test create a new virtual environment and do a local install of Statick. Then run Statick.
python3 -m venv tmp_venv
. tmp_venv/bin/activate
pip install -e .
statick --help

The main branch will currently fail because tabulate is not found. This branch will show the help output.


the simulation crashes when i try to open it

i encounter a error saying java.lang.NoClassDefFoundError: processing/sound/SoundFile if i choose to move the file and i get a different error if i keep the file location in the same place

Look at the other issues there is a solution

Look at the other issues there is a solution

hey which ones should i look at and i am using windows

Make sure to install the Sound library from the Contribution Manager.

Steps are from my fork...

Go to the "Sketch" menu
Select the "Import Library..." menu
Select the "Manage Libraries..." menu
Search for "Sound" in the "Libraries" tab, and select the "Sound" library by "The Processing Foundation"
Press the Install button
Wait for it to download and install, and you're done!

Look at the other issues there is a solution

hey which ones should i look at and i am using windows

#5 mentions this too.

Make sure to install the Sound library from the Contribution Manager.

Steps are from my fork...

1. Go to the "**S**ketch" menu

2. Select the "**I**mport Library..." menu

3. Select the "**M**anage Libraries..." menu

4. Search for "Sound" in the "Libraries" tab, and select the "**Sound**" library by "The Processing Foundation"

5. Press the Install button

6. Wait for it to download and install, and you're done!

he did that though


Can I use gosdk to get the original pcm data？

Can I use gosdk to get the original pcm data？

Go SDK would not automatically decode for you. You'd want to hook it up to Opus decode.


Remove SelectorId

Now TcpStream, TcpListener and UdpSocket are just a file descriptor on
unix.
Closes #999.

So, SelectorId was added to enforce some level of platform consistency. I do think that it is not great to take up the struct space when not needed.
What if we did something like keep SelectorId in "debug" mode, but when compiled in release mode, it would be gone. We could use #[cfg(debug_assertions)] to guard, which seems appropriate as the check is basically a debug assertion.

@carllerche done that in #1004, closing this.


Add release notes for 1.12.6-cs7

ping @joaofnfernandes

The link checker is sad but this PR doesn't make it worse, so I'm merging.


Add test case for zero retro value

Signed-off-by: Ian Chen<EMAIL_ADDRESS>🦟 Bug fix
Summary
A follow up to #522. Retro values for objects that do not have anything set should now return zero. Added a check and remove todo comment.
Checklist

[x] Signed all commits for DCO
[x] Added tests
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

This pull request has been mentioned on Gazebo Community. There might be relevant details there:
https://community.gazebosim.org/t/new-ignition-releases-2022-03-01-citadel-edifice-fortress/1313/1


Adding extractor dependencies

Summary
Adds the extractor dependency functionality from the MEF
New behavior
Extractors can now be put in any order in the config file and will be sorted upon starting the client. Additionally an error will be thrown before extraction if any necessary extractors are missing. For the base ICARE use case this means the patient extractor will get sorted to be first and an error will be thrown if the patient extractor is missing since all other extractors depend on it.
Code changes

ICARECSVClient.js now calls the sortExtractors() function to check for missing dependencies and correctly order the extractors
ICARECSVClient.test.js updated to work with the above changes, which required adding a new fixture as well

Testing guidance

Make sure that all tests pass
Put the patient extractor at the bottom of your config and run the client (you can use --test-extraction to avoid posting data) and see that it still runs properly
Remove the patient extractor from your config and see that the proper error is thrown

This looks great. I can confirm that tests pass, that relocating the patient to the bottom of the config doesn't break anything, and that removing patient from the config throws a dependency-related error. Approving


Revision changes view is cutting off text

Environment

BookStack Version: v0.18.5
PHP Version: 7.1.12
MySQL Version: 5.7.20
app-editor: markdown
Other: local setup via valet

Current Behavior
The attached screenshot is from books/foo/page/bar/revisions and is only showing a few characters from each line.
Steps to Reproduce

Create a book -> chapter -> page
Add content and save
Make an edit
From the page go to More -> Revisions and click Changes for the edit

Potentially related

I think I created this blank page in wysiwyg mode, enabled markdown, and then added content and made edits... although I'm not certain nor know if this is related.

I think I created this blank page in WYSIWYG mode, enabled markdown, and then added content and made edits... although I'm not certain nor know if this is related.

That will cause unintended side-effects. Can you see if you can reproduce this on the demo website here - https://www.bookstackapp.com/#demo

tl;dr: Maybe this is a doc bug?
I am unable to reproduce using the demo.  There might be several requests here:

Improve revisioning across different page editors (likely something you're already considering)
Document the page editor choice in the installation docs
Document the known limitations in the install/admin guide e.g.,

It is an application-wide and not user-specific setting (this surprised me; I only now see this limitation referenced in the user guide but it's not a user setting)
Changing midstream could cause problems; another reason the installation guide should reference this important choice
When exactly is the app-editor default-setting used? Until a choice is saved via the GUI?

I'll play more with BookStack this upcoming weekend, thanks for creating this :)

@philip - Some of the limitations have been documented here - https://www.bookstackapp.com/docs/user/markdown-editor/

Thanks @Abijeet, that's a good start. Did you choose the markdown section because wysiwyg is the default? That makes sense, although the topic feels more general here as switching the editor (either way) is what causes the problem.
The installation guide feels like an important place for this information. A "Decide on an editor: wysiwyg or markdown." maybe along with a "Switching editors after adding content can lead to unexpected problems. For details, see X." or some such, where X explains the reasons and that it's a global setting.

I think the fundamental issue for this was found in #1553, due to the similarity in output. Therefore I'll close this off.


Add redirect callback url and display name for FOM in the infrastructure config file

Add the redirect callback and sign out urls to the config file for FOM client, so it won't disappear after redeployment.
Add attribute mapping for display name vs custom:idp_display_name

For Cognito "redirectSignin" Url setting for FOM (for admin frontend, only admin needs signin). Confirm these URL with Basil:
Dev (question):
https://fom-dev.nrs.gov.bc.ca (? or is it dynamic route based on PR number?)
or just use http://localhost:4200?
Test:
https://fom-test.nrs.gov.bc.ca
Prod:
https://fom.nrs.gov.bc.ca


Enums in C-binding are not extendable in a binary compatible and sane way

Required information
Operating system:
all
Compiler version:
all
Observed result or behaviour:
The enums in the C-binding have an Name_SUCCESS value as last element in the enums. This results in constant ABI breakings when new enum values are added or the Name_SUCCESS value will end somewhere in the middle of the enum
Expected result or behaviour:
Enum values can be appended without breaking ABI and without having the Name_SUCCESS somewhere in the middle of the enum
Conditions where it occurred / Performed steps:
C-Binding

@elfenpiff what do you think, shall we move all the Name_SUCCESS at the top as first enum value. This has the additional benefit of corresponding to the integer value of 0 which is often used to indicate success


"ERROR: File "output_path_not_set\IterationInfo.0.R0.txt" could not be opened" when the output path is not set

When a user does not specify the output path of elastix (command-line argument "-out"), messages appear, even in the library interface, saying things like:

ERROR: File "output_path_not_set\IterationInfo.0.R0.txt" could not be opened!

Are these messages still useful? If so, should they really be considered error messages, or are they merely informational?

Fixed by commit 99902e5da93abd1f5f6befcbd422e8e7c15cd40d "BUG: Library should write to output directory whenever it is specified" (pull request #899)


Improve qc defacing

Fix image orientation before generating png file.
Fix image size for uniformity.

@jcohenadad This is ready for review

I used it to generate the QC from here: https://github.com/spine-generic/data-multi-subject/pull/21


Docs not found

The "docs|stable" pill at the top of the README.md points to http://nonlinearsolve.sciml.ai/stable/, but this link does not exist, and gives a 404 error

Friendly reminder, still a problem

Hmm it probably needs a new minor for that to happen.

I think this is solved now?

Can confirm that the link works.


Added methods for downloading as png and jpg

Added option for download as JPG and PNG.
The implementations just uses the default options within Cytoscape.js.

@andersmandersen Going to go ahead and merge this. Thanks!


remove OptionalValueException?

Currently an OptionalValueException is raised when an optional relation attribute is accessed that has not received a value or has received an explicit null value. This is inconsistent with the behavior for nullable plain attributes and that for multi relations, which result in null and [] respectively. The goal of this ticket is to investigate the feasibility and discuss the value of dropping the OptionalValueException and returning null instead. When attempting to access an attribute/relation of the null relation, an OptionalValueException should still be raised.
As a next step, we could consider deprecating is defined in favor of != null / != [].

A non-exhaustive (exhaustive within the set of repos checked out on my system) overview of how this would affect other projects (assuming we keep the exception class around for backwards compatibility, and have it raise a deprecation warning):

changes Jinja's is defined behavior: null is considered defined, while the OptionalValueException was not. For models to be compatible with both old and newer versions of core, they would need something like x.y is defined and x.y is not null.
The lsm module catches the exception in some places. Mostly this is just to convert it to None (the change would be compatible with those) but in some cases the logic really branches on it.
Same for the yang module.
std uses it in compatible ways (except for the Jinja part)
openstack uses it in compatible ways
internal docs uses it in an example
pytest-inmanta uses it in a test module (module used to test pytest-inmanta, not something used by the actual pytest plugin)

So the main consequences are that lsm and yang need minor changes, as well as any module that uses is defined on optional relations. These changes could be made in backwards compatible ways so that we have the following compatibility constraints:

iso6 requires patched versions of lsm, yang and other relevant modules
iso5- is compatible with module versions before and after the patch

Planning meeting conclusion: delay this until a couple of months before the next major, migrate modules before that.

I will create tickets for this and put them in the appropriate epics.

inmanta/lsm#546
inmanta/infra-tickets#170
kept this one for the main change


Implement list.pop

Updated with following

inline listPop without Pop method on and wrap with mutex.Lock
using IndexInt now, let me know if that is not ok
This PR also included nil check on IndexInt in core.go becuase it's panic,
there is another one on str.find PR too, so probably may cause small merge conflict later on.

Updated to use IntType.Call.
One other issue about mutex.Lock
I got deadlock with this code
	l.mutex.Lock()
        if i >= len(l.elems) || i < 0 {
		return nil, f.RaiseType(IndexErrorType, "list index out of range")
	}
	item := l.elems[i]
	l.elems = append(l.elems[:i], l.elems[i+1:]...)
	l.mutex.Unlock()

But not with this
        if i >= len(l.elems) || i < 0 {
		return nil, f.RaiseType(IndexErrorType, "list index out of range")
	}
	l.mutex.Lock()
	item := l.elems[i]
	l.elems = append(l.elems[:i], l.elems[i+1:]...)
	l.mutex.Unlock()

Any ideas?

Updated with mutex changes, and test procedure.

Updated with those changes.


Speed up libcxx tests

Similar to #782, we should be able to massively speed up the build and run times of the full libcxx tests. Right now, these tests are disabled by default and only run at night because the build (on CI machines)  takes ~2hrs. @mikbras's idea goes like this:

Group them together
Includes all the C++ headers used by the tests first
Define a namespace for this test (main) to be compiled in.
By including the C++ headers first, putting the entire main source does not cause problems.
The problem is that the tests define many clashing globals.
So grouping them in namespaces seems to work.
We will have to work around the problem of putting the #includes's in the namespace too.

For libc tests, we worked around this by including the headers first and then including the main:
#include <vector>
namespace vector_test
{
    #include ".../vector_test.cpp"
}

Where vector_test.cpp includes <vector>. (The inclusion guards prevents inclusion of the headers within the namespace).

Optimizing the libcxx tests to all run as one large test is not a great idea. These are the following issues that I encountered when enabling libc tests to run on Windows where all the 240 tests are run one after the other.

If one test fails, the libc test fails and the other libc tests are not even tried. So, running each test separately is better.
Right now, cross compilation with clang cannot handle a list of more than 75 tests. This shortcoming may be fixed in the future but still something to note.


Docs.MS CI Updates for Azure.Storage.Common

CI Update for release of Azure.Storage.Common

Docs Build status updates of commit 6eba7eb:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

bundlepackages/azure-dotnet-preview.csv
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

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

File
Status
Preview URL
Details

bundlepackages/azure-dotnet-preview.csv
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


HorizontalSpacing acting up.

Hi!
It seems that increasing the horizontal spacing in your demo only adds spacing to the left size of the grid and not between items. I might be using the wrong method for this but setting the divider width works perfectly. Here's an example of what I mean.
Cheers!

Same problem for me .Except this, it's a good library

I should fix this. If you have time, please send a PR


How much data is retained and available to query?

Is there detail of the amount of data available to query or the time that data is retained?  For example, if I have a hub with millions of devices, and want to query a full list of devices and last connection -- should I be able to pull all this data?  Is there a limit to the amount of data a query can handle and/or how long it retains data for a device?

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

ID: c58bc573-ec3e-ed94-900d-1163a5700750
Version Independent ID: d83c0073-4a03-f46b-1855-4101b040d92c
Content: Understand the Azure IoT Hub query language
Content Source: articles/iot-hub/iot-hub-devguide-query-language.md
Service: iot-hub
GitHub Login: @kgremban
Microsoft Alias: kgremban

@breakpoint7
It would be great if you could add a link to the documentation you are following for these steps? This would help us redirect the issue to the appropriate team. Thanks!

I was referring to the IoT Query Language covered here: https://learn.microsoft.com/en-us/azure/iot-hub/iot-hub-devguide-query-language

@breakpoint7
Thanks for sharing the document. We will investigate and update as appropriate.

Thanks.
Where is the retention policy defined and configured for each IoT Hub tier?
Where are query units outlined or defined?  Is there any way to know how much you have available or have consumed?

Hello @breakpoint7

Where is the retention policy defined and configured for each IoT Hub tier?

The retention policy for each IoT Hub tier is defined and configured in the IoT Hub's settings.
IoT Hub allows data retention in the built-in Event Hubs for a maximum of seven days. You can set the retention time during creation of your IoT hub. Data retention time in IoT Hub depends on your IoT hub tier and unit type. In terms of size, the built-in Event Hubs can retain messages of the maximum message size up to at least 24 hours of quota.
The query units for Azure IoT Hub are outlined in the IoT Hub pricing details. Each tier of IoT Hub has a certain number of query units associated with it, and you can choose the tier that best suits your needs based on the expected number of query operations that you'll be performing.
To view your IoT Hub metrics in the Azure portal, follow these steps:

Log in to the Azure portal and navigate to your IoT Hub.
Click on "Metrics" in the left-hand menu.
Choose the time range for which you want to view metrics.
Select the "Query" metric from the dropdown list.
Review the graph and table to see how many query operations you've performed and how much of your allocated query units you've consumed.

For further discussions on How-To on the product, please visit the Microsoft QnA forum, we are more than happy to help.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.
Please also do visit Microsoft Q&A the home for technical questions and answers at Microsoft.


Make URL to download npm configurable

My Environment

[x] Windows 7 or below (not truly supported due to EOL)

[ ] Windows 8

[ ] Windows 8.1

[x] Windows 10

[ ] Windows 10 IoT Core

[ ] Windows Server 2012

[ ] Windows Server 2012 R2

[ ] Windows Server 2016

[ ] My Windows installation is non-English.

I have already...

[ ] read the README to be aware of npm gotchas & antivirus issues.
[ ] reviewed the wiki to make sure my issue hasn't already been resolved.
[ ] verified I'm using an account with administrative privileges.
[ ] searched the issues (open and closed) to make sure this isn't a duplicate.
[ ] made sure this isn't a question about how to use NVM for Windows, since gitter is used for questions and comments.

My issue is related to (check only those which apply):

[ ] settings.txt
[x] proxy support
[ ] 32 or 64 bit support

Expected Behavior
The ability to configure where npm is being downloaded from, in order to download from within the firewall, not needing to worry about proxy settings.
Actual Behavior
Fill me in...
Steps to reproduce the problem:
Fill me in...

Closing as duplicate of #83.


Make Emacs Mode Control+AE work for only Korean Input

I tried to make this work properly but I couldn't figure it out.
First I wanted to enable contrl+ae but just for Korean input. However, I couldn't find a similar example because it doesn't seem to do this in the readme under the Korean section. Or was I just missing something?
Second, I had tried the below but figured it doesn't recognize Korean characters as keys so this clearly didn't work :
 <item>
      <name>Control+AE to Command+Left/Right</name>
      <identifier>option.emacsmode_hanja_controlAE</identifier>
      <not>{{EMACS_MODE_IGNORE_APPS}}</not>
      <modifier_not>
        ModifierFlag::COMMAND_L, ModifierFlag::COMMAND_R,
        ModifierFlag::OPTION_L,  ModifierFlag::OPTION_R,
      </modifier_not>
      <autogen>
        __KeyToKey__
        KeyCode::ㅁ, MODIFIERFLAG_EITHER_LEFT_OR_RIGHT_CONTROL,
        KeyCode::CURSOR_LEFT, ModifierFlag::COMMAND_L,
      </autogen>
      <autogen>
        __KeyToKey__
        KeyCode::ㄷ, MODIFIERFLAG_EITHER_LEFT_OR_RIGHT_CONTROL,
        KeyCode::CURSOR_RIGHT, ModifierFlag::COMMAND_L,
      </autogen>
    </item>

Nevermind, I figured it out:
      <inputsource_only>KOREAN</inputsource_only>

I assume if you want Control+PNBF to  <not>{{EMACS_MODE_IGNORE_APPS}}</not> for all inputs but when <inputsource_only>KOREAN</inputsource_only> is on then it applies in all apps.
I have to make two seperate bindings in this case? Or is there a way to apply this within the same binding using some sort of if statement?


Update logic of 'hasError' in 'provisioning.cattle.io.cluster' model

Summary
Fixes #10043

Occurred changes and/or fixed issues

Update logic of hasError in provisioning.cattle.io.cluster model
Add unit test for cluster.status.conditions which affect the outcome of hasError model property

Technical notes summary

Areas or cases that should be tested

Check SURE details in #10043 and grab the HAR files
Use https://github.com/rancher/dashboard/blob/master/docs/developer/har-files.md#yarn-dev to run this branch
Verify that the cluster is available to "explore"

NOTE: I never tested it this way, so I am not 100% this will work. All of the investigation was based on the HAR files output and code checks in Dashboard which I've tried to sum up in the comments of #10043
Areas which could experience regressions

Screenshot/Video

@richard-cox me and @nwmac chose this course of action in a sync we had earlier in the day. @nwmac I think that I've captured the correct logic of what we discussed in the code change


Support APM agent using Kubernetes Operator

It would be great to see if New Relic could provide an option to deploy APM agents on Kubernetes platform (OpenShift, AKS, EKS, IKS, GKE) for the applications packaged and deployed as an Operator.

DD has it: https://www.datadoghq.com/blog/datadog-operator/


RAM usage.

On a run with the new set up, I am currently seeing about 32Gb of RAM usage vs (I think) about 5Gb max using the previous version. This could be a false impression as I have no log of RAM usage but we should keep an eye on this to make sure it doesn't increase.
R is pretty bad at letting go of RAM within a single session so I could imagine this being a factor.

Fixed when running each region and linking with bash. Suggest moving some of these control scripts into the public domain.


quickCreate? How to?

Description:
I've noticed HasQuickCreate trait in the last commits. That's pretty cool if you can supply with quick documentation how to use it?
https://github.com/z-song/laravel-admin/search?q=quickcreate&unscoped_q=quickcreate

This is a new feature in v1.7.3. Due to lack of time, updating English documents may be delayed.
Now I have added the documentation.  https://laravel-admin.org/docs/#/en/model-grid-quick-create
Also see the changelog here https://laravel-admin.org/docs/#/en/changelog/v1.7.3


feat: resize image height

PR Checklist
Please check if your PR fulfills the following requirements:

[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[x] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?
Issue Number: N/A
What is the new behavior?
From now on, users can resize image height. follow:
AS-IS
![[NewGeans.png|100x200]]

TO-BE
![image](/assets/img/yyyy-MM-dd-title/NewGeans.png) {: width="100" height="200" }

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

apply indent 2

Codecov Report
Patch coverage: 96.26% and no project coverage change.

Comparison is base (c1caa73) 97.27% compared to head (bb4b859) 97.27%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #80   +/-   ##
=======================================
  Coverage   97.27%   97.27%
=======================================
  Files           9        9
  Lines         147      147
  Branches       17       17
=======================================
  Hits          143      143
  Misses          4        4

Impacted Files
Coverage Δ

src/ObsidianRegex.ts
100.00% <ø> (ø)

src/jekyll/ResourceLinkConverter.ts
89.18% <87.87%> (ø)

src/core/ConverterChain.ts
100.00% <100.00%> (ø)

src/jekyll/CalloutConverter.ts
100.00% <100.00%> (ø)

src/jekyll/CommentsConverter.ts
100.00% <100.00%> (ø)

src/jekyll/EmbedsConverter.ts
100.00% <100.00%> (ø)

src/jekyll/FootnotesConverter.ts
100.00% <100.00%> (ø)

src/jekyll/FrontMatterConverter.ts
100.00% <100.00%> (ø)

src/jekyll/WikiLinkConverter.ts
100.00% <100.00%> (ø)

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

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


QC: test mapping files for expected databases

Based on the expected databases in a mapping file (which are different for GeneProtein/Metabolite/Etc.), there should be an output which databases should be present in the mapping files. @egonw @tabbassidaloii @DeniseSl22

expected databases in a mapping file

What do you mean with "expected" here? Expected by whom? Right now, the QC tool expects the same databases in the next release as the previous release. That is in the output.
You mean the BridgeDb Derby schema version, right?

When you compare with a previous database it already does report which databases are added or removed. That is separate from the differences.
What it does not do, is look at the attributes in both files.


Fix bug in sortData subRows

Fix sort subRows stop recursion when subRows length is one.
If row.subRows length is one, the subRows of row.subRows[0] will not sort in recursion

Why would you need to sort a single row?

Because I want to sort subRows data in recursion. If I have data[n].subRows[0].subRows[0].subRows[0] data set and data[0].subRows length is one, I can't sort the data in data[n].subRows[0] and subRows of the data in data[n].subRows[0]
You can find the case in https://codesandbox.io/embed/boring-sanne-d56h1?fontsize=14&hidenavigation=1&theme=dark

@tannerlinsley Can you take a look?

Ah. I see. Good find.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Support packageVerificationCode in SpdxDocumentFile

If a packageVerificationCode is given in a SpdxDocumentFile found by the Analyzer, that code should actually be verified to confirm that the package's expected files matchtes the actual files.

Closing, as this is not generally possible, because the SpdxDocumentFile might be stored separately from the source code / ORT might not have access to the source code.


xarray.concat docstring confuses PyCharm type detection

PyCharm inspects docstrings to figure out the return type of functions. In the case of xarray.concat(), it believes it is 'type'; this causes it to prominently highlight the user code as an error:

Do we know why it thinks it's type?

@max-sixty because of this https://github.com/pydata/xarray/blob/c33dab237a0aca316b7f6d4ba10857829a895a25/xarray/core/combine.py#L73

Interesting, we could definitely fix this.
Actually, I suspect the ideal situation would be to use type annotations -- then PyCharm should not need to inspect the docstrings at all.

@shoyer indeed. I'll start working on type annotations straight away.

#2877 doesn't fix it... More pulls will be needed


Add ability to group targets

Is your feature request related to a problem? Please describe.
I'm using Visual Studio and I would like to group targets by putting them into virtual folders, to avoid my workspace being a complete mess. You can look at what Premake 5 does here: https://github.com/premake/premake-core/wiki/group
Describe the solution you'd like
I think the way Premake handles this is not the best way to do.
I think it can be better handled by doing as target does:
group("name")
    target("first_target")
        ...
    target("second_target")
        ...
group_end()

Describe alternatives you've considered
Alternatively, it may be handled like this:
target("first_target")
    ...
target("second_target")
    ...
target_end()

group("name", "first_target", "second_target")
#or
group("name", {"first_target", "second_target"})

or maybe it can be handled the same way as Premake does, but I think the better solution here for users would be the first I propose.
Additional context
Here is a preview of virtual folders in Visual Studio, using an old version of NazaraEngine that I found on my hard drive.

Sorry for the duplicate of #1026.
I'm closing this issue and reporting my ideas to the other issue.


The Scrollable snapOffset() callback could allow snapping "back"

Currently, if a Scrollable's snapOffsetCallback() returns a value that effectively reverses the fling's direction, the fling simulation asserts.  For applications using the snapOffsetCallback() to scroll to page boundaries, it would be useful to allow the callback's value to reverse the fling's direction.
I/flutter : ══╡ EXCEPTION CAUGHT BY GESTURE LIBRARY ╞═══════════════════════════════════════════════════════════
I/flutter : The following assertion was thrown while routing a pointer event:
I/flutter : 'package:flutter/src/physics/friction_simulation.dart': Failed assertion: line 39 pos 12:
I/flutter : 'startVelocity == 0.0 || endVelocity == 0.0 || startVelocity.sign == endVelocity.sign' is not true.
I/flutter : When the exception was thrown, this was the stack:
I/flutter : #2      FrictionSimulation.FrictionSimulation.through (package:flutter/src/physics/friction_simulation.dart:39:12)
I/flutter : #3      _createSnapScrollSimulation (package:flutter/src/widgets/scroll_behavior.dart:16:14)
I/flutter : #4      OverscrollBehavior.createSnapScrollSimulation (package:flutter/src/widgets/scroll_behavior.dart:257:12)
I/flutter : #5      ScrollableState._createSnapSimulation (package:flutter/src/widgets/scrollable.dart:622:50)
I/flutter : #6      ScrollableState._startToEndAnimation (package:flutter/src/widgets/scrollable.dart:587:19)
I/flutter : #7      ScrollableState.fling (package:flutter/src/widgets/scrollable.dart:564:14)

@Hixie

This feature is gone now.


Panel dividers look misaligned in tmux

[x] I am on the latest Hyper.app version
[x] I have searched the issues of this repo and believe that this is not a duplicate

OS version and name: macOS Sierra
Hyper.app version: 0.8.3 (<IP_ADDRESS>3)
Link of a Gist with the contents of your .hyper.js: https://gist.github.com/rafaelrinaldi/d0db885568af90c57e90dae24a54fe5d
Relevent information from devtools (CMD+SHIFT+I on Mac OS, CTRL+SHIFT+I elsewhere): N/A
The issue is reproducible in vanilla Hyper.app

Issue
When in a tmux session, panel dividers look a bit off/misaligned.

Can you post your tmux version and config? I'm using tmux in Hyper without problems.

I can confirm the same behaviour. It's not a tmux issue, I would say, since the very same configuration in iTerm2 doesn't have the issue.
All the lines that contain special glyphs need more room in hyper. Not sure why that is.
I'm using it with the Patched Nerd Fonts.

This is still a bug in version 1.0.0


Travis CI invalid badges (rate limiting?)

It appears that the repo joyent/node on GitHub was moved and renamed. However, Travis CI does not know this. So, changing the URLs for these badges is needed to fix the examples.

Also, seeing this for HHVM badges and MyGet.

Not just the examples. Mine were failing so far but whilst I write this post they appear to be ok.

Maybe is something with the cache of shields. If they're projects with a lot of movement and they're building very consecutively maybe shields shows us every time a cached image of an on-going build.

Forget that. I've just seen how it changed from passing to unaccessible to passing again and I'm not currently running any build.

Same for me!

+1

+1

I am frequently getting inaccessible travis-ci badges. If I refresh my browser two or three times, the badge changes to success. If I leave and come back, the badge says inaccessible again. This is in contrast to the badge directly provided by travis-ci which always shows success.

I have added a log in prod to identify what causes that. The "normal" cause is that travis does not have that repo registered, which does not seem to be the case here. So far no luck.

The problem seemed to start, at least for me, within the past week. For an example repo, see here. If you keep refreshing, the travis-ci badge will randomly alternate between inaccessible and success.

Here is what I have: sometimes, Travis CI returns {"file":"not found"} when downloading their badge. Is that a rate-limiting thing?

Historically, we request badges instead of using the API because the straightforward API queries sometimes produce "pending", which is not a badge people ever want to see, and some other way to use the API to circumvent that had issues when dealing with branches. People wanted our badges to show the same information as the Travis CI badges (with the customizability that we offer).

Original switch from png header sniffing to straightforward Travis API: https://github.com/badges/shields/commit/ee4c9f885a07147b4b5e45b091f137fe3110af9d
Switch to builds.json to avoid "pending": https://github.com/badges/shields/commit/5050698c14f4530a559ad93ed769c25ae15571e1
Issues with that, and switch to SVG scraping: https://github.com/badges/shields/issues/288

I have gone full circle, and switched it to only perform a HEAD request, on the svg headers this time. Let me know if it changes anything.

Seeing a lot of invalid travis badges this morning.

Same here:

Shields invalid: https://img.shields.io/travis/Zenify/ModularPresenterMapping.svg
Travis valid: https://travis-ci.org/Zenify/ModularPresenterMapping.svg?branch=master

Same problem here!

It seems to me that the problem's gone. Is it a weekend's effect, I wonder.

Still seeing the Invalid Travis badges

Indeed, having the same problem. Shields.io:
 (https://img.shields.io/travis/netvl/xml-rs.svg?style=flat-square)
Travis:
 (https://travis-ci.org/netvl/xml-rs.svg?branch=master)
Occasionally the shields badge becomes green but usually it is invalid.

I currently see them better than before the change. It's true that now they appear as "invalid" sometimes but it's less often than when they said "inaccessible".

Also getting a bunch of Invalid badges across many repos.

:+1:

TeamCity badge is malformed.

If this problem recurs, feel free to open a new issue.


Can the API doc icon be an "icofont"?

E.g. so I would not have to copy-paste the SVG in https://github.com/http4s/http4s/pull/6771.

Apart from that, if you'd been on 0.19 already the problem would not even exist

Heh, right, sorry 😅

I also believe even with 0.18 using @:icon(api) in the template should work, I am not sure why this is not the case for the other SVG

Ah thanks, let me give that a try.

Thanks, the directive is very nice. And I promise we'll upgrade soon 🥺


[AutoRelease] t2-deviceupdate-2022-08-26-28507(Do not merge)

https://github.com/Azure/sdk-release-request/issues/3078
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=2430
BuildTargetingString
azure-mgmt-deviceupdate
Skip.CreateApiReview
true

issue link:https://github.com/Azure/sdk-release-request/issues/3078


Fix GetQuery compat with newer Trino

Required significant changes, but fixes CI/tests fully. Separated here for ease of review and release.

Closing as it was originally only created to show tests Can pass when env vars are setup.


Bugfixing mlflow registry linting issue

Describe changes
The model_extra can be None that's why mypy was giving us issues. It is also read-only so we have to modify the __pydantic_extra__ to get around it.
Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[x] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] I have based my new branch on develop and the open PR is targeting develop. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.
[x] If my changes require changes to the dashboard, these changes are communicated/requested.

Types of changes

[x] 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 change)
[ ] Other (add details above)

@coderabbitai summary


Upgrade react-native-device-info dependency

Summary
Affected Areas:

Permission dialogs to display the app name
App build number and version should be available: Easier to see it in the About screen or Report a Problem
Detect if the device is a phone or tablet

Upgrade changelog does not report any breaking changes.

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/mattermost-mobile/9270
https://circleci.com/gh/mattermost/mattermost-mobile/9268

Artifact links:
https://9270-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.apk
https://9268-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.ipa

@josephbaylon 👓 ??

@enahum Looks like that already exists in the current release builds (that's app info). I thought device info would be info about the actual device. If not, I'll go ahead approve the PR.

Yes, I made a mistake in the description, is about the app.
Thanks for double checking

@enahum
Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Fetching origin
+++ Updating remotes done...
!!! 'upstream/release-1.35' not found. The second argument should be something like upstream/release-0.21.
    (In particular, it needs to be a valid, existing remote branch that I can 'git checkout'.)

+++ Returning you to the master branch and cleaning up.

/cherry-pick release-1.35


rewrite api module to use provided traits from k8s_openapi

This has grown to be a semi-rewrite of Api...
Fixes #150, #158, #25, #96

[x] RawApi renamed to Resource (much better tests now)
[x] Api revamped
[x] openapi feature removed (now always depend on non-default-features version)
[x] runtime now use MetaContent trait for types with ObjectMeta- [ ] examples -> chore
[ ] examples + readme updated to showcase new api
[ ] complications (below) resolved

so glad I'm doing this before kubecon... this is such a huge change.
Notably, this makes our Api work with ANY kubernetes type exported by k8s_openapi, removes a bunch of duplicated structs from apimachinery so we can focus on what's unique to this library; the runtime.
A few cons:

openapi versions of apimachinery are quite Option heavy, leading to less ergonomic manipulation of their values (we can impl helpers later)
Object<P, U> is currently broken due to a trait restriction in https://github.com/Arnavion/k8s-openapi/issues/61

Tried to make some compatibility layer thing with:
#[cfg(feature = "openapi")]
impl<P, U> k8s_openapi::Metadata for Object<P, U>
where
    P: Clone,
    U: Clone,
{
    type Ty = ObjectMeta;
    fn metadata(&self) -> Option<&Self::Ty> {
        Some(&self.metadata)
    }
}

but that won't work because the Metadata trait seems to require the Resource trait implemented (and we need to do that separately.
It's also a bit awkward with the Metadata trait returning Option<MetadataType> (because it's catering to list type i guess?), even though in all normal object structs the Metadata is there. That's the one thing we have relied on, even for snowflakes (which are gone in this branch).

Currently fixes #158, fixes #150, fixes #25, fixes #96

It's also a bit awkward with the Metadata trait returning Option<MetadataType> (because it's catering to list type i guess?)

No. List<T> is also a resource with metadata, and thus also impls Metadata with Ty = ListMeta.
The reason that Metadata::metadata() returns an Option is that metadata is optional. Eg Pod::metadata and List::metadata. And that in turn is because the openapi spec says it's optional. (That said, I've always wondered exactly when the metadata can not be specified, since it's required for create / patch from the user, and always sent in responses from the API server. Is there a situation where the API server doesn't send it.)

The reason that Metadata::metadata() returns an Option is that metadata is optional. Eg Pod::metadata and List::metadata. And that in turn is because the openapi spec says it's optional. (That said, I've always wondered exactly when the metadata can not be specified, since it's required for create / patch from the user, and always sent in responses from the API server. Is there a situation where the API server doesn't send it.)

Yeah, that makes sense. Having investigated more, it looks like the kubernetes api is just super-conservative on optionality. Just like PodSpec on Pod. How can that ever be optional? The API definitely hard limits this on even input.
Interestingly, trying to create a Pod one without PodSpec will complain that spec.containers: Required value., which is the same error you get if you supply spec: {}. Maybe it's conflating optionality and defaulting, or maybe their internal data type they accept through the api, and the type they really must have before creating it differ.
At any rate, it's a shame because this causes so many guaranteed unwraps in controllers, and I really would just like to fail to even deserialize an object if it doesn't satisfy my assumptions about it.

No. List<T> is also a resource with metadata, and thus also impls Metadata with Ty = ListMeta.

(What I meant is that types with ListMeta has a much more restrictive offering in their metadata. I was speculating that maybe the optionality of metadata somehow was somehow related to the sparseness of metadata in LIST calls, but as you've pointed out, it's just plain marked optional even for types with Metadata<Ty=ObjectMeta>)

Interestingly, trying to create a Pod one without PodSpec will complain that spec.containers: Required value., which is the same error you get if you supply spec: {}.

The golang code defines Pod.spec as type PodSpec, so its zero value is an actual object with zeroed fields rather than nil. So both {} and { "spec": {} } will deserialize the same way - to an object with zeroed fields.

Have updated all examples, and fixed all tests to work with the current setup. Removed Object because there was no way to actually implement the required Resource trait except via codegen, so need to go down that route (which is fine, that's a nice pattern).
Have tried out the k8s-openapi-derive setup first. This gets most of the way there, and actually lets us have a custom thing that implements k8s_openapi::Resource. See crd_api example.
However, it's not super customizable atm and merging as it stands would (afaikt) make us drop support for manipulating status objects. Have raised https://github.com/Arnavion/k8s-openapi/issues/62 upstream. But might explore an additional approach.
@teozkr how do you feel about this pr?

Big fan of this in concept, but I'm a bit out of order right now so I haven't done a proper review yet (pizzas are bloody dangerous, as it turns out..).
That said, I don't think this resolves (my interpretation of) #150, although it did end up getting a bit derailed in this direction (sorry!). In particular, namespace is still implied (which bubbles up from Resource aka RawApi). It would also be great IMO if Api's K was given to the operations (so the path became Api::get::<K> rather than Api::<K>::get), so that a single Api instance could be shared. I'll sketch up a PR for that when I feel a bit better.

And for the record, :+1: from me on requiring k8s-openapi and dropping Object.

In particular, namespace is still implied (which bubbles up from Resource aka RawApi). It would also be great IMO if Api's K was given to the operations (so the path became Api::get::<K> rather than Api::<K>::get), so that a single Api instance could be shared. I'll sketch up a PR for that when I feel a bit better.

Yeah, I realised that this only addresses the "derailed part" of this issue, and didn't want to change that world in one go, as this PR is already quite big. But I agree that this could be quite nice. Although I am still not sure how connection pooling would be improved.
Note that the underlying connection is still shared

Right, forgot about the distrinction between Api (which is currently (at least) unique per K) and APIClient (which is shared and owns the connection pool). My bad.


For the R sample, where does the R runtime come from?

I'm trying to understand custom handlers, and how you write them in an arbitrary language.
R is a good example of this.
The R sample here though doesn't seem to tell me anything about where the R runtime actually comes from. The only Dockerfile provided uses a "node" base image.
Naively I was expecting there to be a Dockerfile for the R sample that specified some kind of base image that contained an R runtime environment? But I probably misunderstand how custom handlers, and custom image work.

I agree its not clear where the actual runtime comes from

As I understand it you need to install R on the dockerimage first. I'm not sure how old the dockerimage is, because by default it installed R 3.5 which is very old.


Update solana.tokenlist.json

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

add me

Yarn installed

Good
Kind regards,
Ayo Adesanya

On 3 Nov 2021, at 18:53, MetaBookCrypto @.***> wrote:

Yarn installed
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.


Issues

Hello,
Thank you for this repo. Decided to test it locally on my windows. There are issues.
For example:
#File
The private name '_tickerModeNotifier', defined by 'TickerProviderStateMixin', conflicts with the same name defined by 'SingleTickerProviderStateMixin'.

Is there any chance you could update this repo?

Hi - as this repo is only a snapshot of my submission to the Flutter Clock challenge from 2020, I will leave the code as is. If you want to make it work, you can use a version of Flutter from that time.


#3178: Fix for wormhole b0 reduce w

this works well on WHB0 on my testing after removing the transpose work-around. I'd like to add a few more commits into this PR before merging if that's OK since it makes a logical unit

Awesome I'm glad! Yes please go ahead and add the rest of your changes. Im sure there are more tests that are needed for this.


[FEATURE REQUEST] Support for regression based problems

Are you planning to add support for regression tasks ?
If so, I'd suggest replacing the exact equality condition of the original and perturbed input predictions in the stability metrics (ŷ_x == ŷ_x')  to something like |ŷ_x- ŷ_x'|< eps, where eps is a user-defined range.

For Ground_truth evaluations, one could use benchmark functions used in optimization like Ackley (or any other known mathematical functions) and fit several linear regression models for several particular ranges of inputs. Weights of linear regression models could be used as ground truth for faithfulness metrics.


Unable to see any Open-nti UI

Installed opennti as per instructions given:
git clone https://github.com/Juniper/open-nti.git
cd open-nti
sudo make start

Howver, we don't see any UI running on port 8080 as shown in the tutorial video at:
https://junipernetworks.sharepoint.com/portals/hub/_layouts/15/PointPublishing.aspx?app=video&p=p&chid=7a8d593a-2736-4165-b559-973736644cef&vid=7A3A554B-8F42-4B50-84DB-DC6475B11876
Grafana and influxdb ports are working fine.  What do we need to do to get the UI working ?

Hi @gaacad
The UI is grafana, so it does not matter if it's mapped to 8080 or 80 port, at the end it will be forwarded to 3000 port (grafana default).
I you want to have the grafana mapped to 8080 port the easiest way is to edit the opennti.params file
https://github.com/Juniper/open-nti/blob/master/open-nti.params#L47, and change the variable LOCAL_PORT_GRAFANA to 8080, at the end this value will be used in the container as follow
opennti:
  image: $MAIN_IMAGE_NAME:$IMAGE_TAG
  container_name: $MAIN_CONTAINER_NAME
# Uncomment environment to allow anonymous grafana access
# and disable user sign up
#  environment:
#   - "GF_USERS_ALLOW_SIGN_UP=false"
#   - "GF_AUTH_ANONYMOUS_ENABLED=true"
  volumes:
   - ./$LOCAL_DIR_DASHBOARD:/src/dashboards
   - ./$LOCAL_DIR_DATA:/opt/open-nti/data
   - /etc/localtime:/etc/localtime
  ports:
   - "$LOCAL_PORT_STATSD:8125/udp"
   - "$LOCAL_PORT_NGINX:80"
   - "$LOCAL_PORT_GRAFANA:3000" <<<<<
   - "$LOCAL_PORT_INFLUXDB:8083"
   - "$LOCAL_PORT_INFLUXDB_API:8086"

Regards


invalid syntax in sample - async_echo.py

i'm newbie to python enviroment
successfully run echo.py in sample but when i run async_echo.py than got error
python 2.7

File "async_echo.py", line 12
async def run():
^
SyntaxError: invalid syntax

Hi guys,
I have the same error, I tried to run with python 3.5.2. Someone found a solution about it?
webwhatsapi/async_driver.py", line 73
    yield from self.get_chat_from_id(chat_id)
               ^
SyntaxError: 'yield from' inside async function


The update-sift script breaks srch_strings

Summary: began with a fresh install of SIFT 3 (md5:  4cf6276a67f38980065f8563cb49fe46) and opened with VMware 10. Ran a test with srch_strings and it worked as expected.
Ran the update-sift script and (after succesful completion) then ran the same srch_strings command on the same target file and received the error message:
*** Error in `srch_strings': double free or corruption (fasttop): 0x00000000024d1250 ***
Aborted (core dumped)
Error is consistent and can be reproduced as described above.

Thanks for the report @the-IMOL, when you say you started with a fresh SIFT 3, do you mean the VM image from the SANS Forensics website?

Yes the VM image (in OVF format).

I have the same issue using a fresh install of sift after updating  using update-sift. Running the VM on vmware 12 installed from the OVF

I'm seeing the same issue.   Followed through the Ubuntu error reporting prompt to submit a report upstream, but I'm not sure what happens with that.

Built a fresh Ubuntu 14.04 VM, then installed SIFT with wget.  After running update-sift script, have the same issue with search_strings

I have confirmed this problem. I am unsure as to what the cause is at this time.

As we are no longer supporting 14.04 going forward, 16.04 VM is about to be released, and the sift-saltstack is the now preferred method of install, we will re-open this issue if it is still happening there.


Shortest Distance in Weighted graph using BFS

Aim

I would like to add in Data Structure Folder.

Details

I would like to add solution of Shortest Distance in Weighted graph using BFS.

Programming language

[ ] C
[x] C++
[ ] Java
[ ] Python

@sameeksharl @Amit366 @Ayush7614 @ravikr126 Please Assign me this issue. I will solve it in C++.
LGM-SOC'21
Thank You

Can you assign me in java


Bugfix object leaks

File systems that are around for awhile seem to leak objects -- the amount of space used in terms of objects seems to keep growing.  If all files and directories are deleted, there can still be a lot of objects left over.
It turns out there were two bugs in the sortedmap package that were fixed in the following commits:
d64c276 -- when rebalanceHere() discards "rightSiblingNode" as part of merging two nodes, it fails to mark it dirty which means the underlying disk space is not reqcovered
a594a01 -- discardNode() stops walking the tree when it finds a node that is not loaded which means that tree.BPlusTreeCallbacks.DiscardNode() is not called to recover the underlying disk space
the rest of the commits simply add, remove, or update tracing of Swift objects and tree nodes.  They can be ignored (unless you're interested in that type of thing).

Apparently I haven't fixed all of the problems yet! I just deleted my "huge" (2 Gibyte) file system and there are lots of checkpoint objects still there. Investigating ...

I see that your latest commit did, at least, pass the "overlord that is travis" CI tests :-)


Suggestion: Filter out currently unobtainable

For those who like to complete and collect along the way instead of at the end of the game, it'd be great if the program could detect within the save file which spells you don't have (possibly through which quests you have or have not completed) and shorten the list by only displaying collectibles you can actually collect right now. If you need help identifying the 'requirement' for each collectible that shouldn't be hard to do.

Do you need help with this request?

Sorry, I had some higher priority things to fix first. This is next on my list, so hopefully I'll add it by next weekend? It isn't really all that difficult, it's just time consuming to go through everything


Removing Deps from Dep Management Tool

Removing two dependencies from bandep so they don't add further unwarranted dependencies (Gopkg.lock shows these 2 deps in turn depend on 2 more deps).
github.com/pkg/errors was replaced with it's equivalent internal calls since we are not introspecting the cause of the actual error.  Note the test TestCheckPkgBadFile passes before and after the removal of this dependency.
In order to remove github.com/alecthomas/kingpin, the assumption that the list of arguments were passed in as --ban <ban1> --ban <ban2> ... was changed to be more cohesive with golang's internal command line flag parser using a  comma separated list of dependencies like --ban <ban1>,<ban2>,....  Additionally, due to the dual arguments of help and version (to h and v respectively) a custom usage function was used to ensure no stuttering descriptions when parsing invalid arguments or displaying help.
Finally, once dependencies are removed, there is no need to use the dep dependency management tool, so removing Gopkg.lock and Gopkg.toml.
@caarlos0 hopefully this PR finds you well, and Merry Christmas 🎄

Good idea! Thanks!!

Oh, before I forget, Merry Christmas 🎄


[WIP] tools: Add API to evict owner in etcd

What problem does this PR solve?
Add API to evict invalid owner in etcd manually.

[x] Show all information like ETCDCTL_API=3 etcdctl --endpoints={PD address} get /tidb/ddl  --prefix
[x] delete owner campaign like  ETCDCTL_API=3 ./etcdctl --endpoints={PD address} del /tidb/ddl/fg/owner/Lease_ID
[x] delete schema version like ETCDCTL_API=3 ./etcdctl --endpoints={PD address} del /tidb/ddl/all_schema_versions/DDL_ID
[ ] Add Unit Tests

What is changed and how it works?
Add new function in pd-ctl.
Check List
Related changes

Need to update the documentation

 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.

Codecov Report

Merging #1434 into master will decrease coverage by 0.45%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #1434      +/-   ##
==========================================
- Coverage   67.67%   67.22%   -0.46%
==========================================
  Files         158      159       +1
  Lines       15076    15190     +114
==========================================
+ Hits        10203    10211       +8
- Misses       3955     4054      +99
- Partials      918      925       +7

Impacted Files
Coverage Δ

tools/pd-ctl/pdctl/command/etcd.go
0% <0%> (ø)

tools/pd-ctl/pdctl/ctl.go
5% <0%> (-0.13%)
:arrow_down:

server/etcd_kv.go
63.63% <0%> (-10.91%)
:arrow_down:

server/core/kv.go
76.47% <0%> (-1.48%)
:arrow_down:

server/leader.go
81.97% <0%> (-1.29%)
:arrow_down:

server/option.go
90.41% <0%> (-0.84%)
:arrow_down:

server/cluster.go
80.23% <0%> (-0.59%)
:arrow_down:

server/server.go
77.61% <0%> (-0.5%)
:arrow_down:

server/grpc_service.go
53.65% <0%> (+0.48%)
:arrow_up:

... and 3 more

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 376c933...9a6053c. Read the comment docs.

This PR is not recommand to file in PD repo.
I am going to close this PR.
Sorry to bother everyone here.


Make codecov a little less scolding

I want codecov for advice, not for another hoop all commits must jump through.

Did it work? It still seems unhappy...

Hmm.  It stopped the comment, but not the Red X of Shame.


[TEST]

Signed-off-by: KMAnju-2021<EMAIL_ADDRESS>

Codecov Report

Merging #3385 (e2cfe17) into main (24f85ce) will decrease coverage by 19.75%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #3385       +/-   ##
===========================================
- Coverage   62.11%   42.36%   -19.76%
===========================================
  Files         269      201       -68
  Lines       26852    24277     -2575
===========================================
- Hits        16680    10285     -6395
- Misses       8334    12931     +4597
+ Partials     1838     1061      -777

Flag
Coverage Δ

kind-e2e-tests
?

unit-tests
42.36% <ø> (+0.02%)
:arrow_up:

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


Add specification

This PR adds a loose specification document to; define the expression grammar, and highlight behavior and differences from the Java implementation.
I want to revise the implementation and copy-paste and modify the tests from the Java repo to comply with this spec.
The Java implementation has some ambiguities for parsing expressions and different choices in handling non-existing keys and indexes in the input.

bors r+


Update rating design on mobile

Give more space between rating stars icons on the Mobile view.
See image below

Where is the PR on this ?
@Asam237  Did you resolve this one?