Closing Paragraph tag removed when no content provided

Related to issue #1
Encountered in v1.5.1 - When parsing a P block with no content, HAP will remove the closing P. For example:
Starting HTML:

test

HTML after HAP:

test

Using HtmlAgilityPack.HtmlDocument.DisableBehavaiorTagP = true; seem to resolve this issue. Should we expect to continue to use that option in future releases?

Hello @chrisnelsondotca ,
I'm currently looking at some issue similar to this one,
I hope to be able to provide more information by next Monday.
Best Regards,
Jonathan

Hello @chrisnelsondotca ,
We will try to work on all this kind of issue in September.
In a future release, you will not have to keep using this option. We will probably directly remove it once all this kind of issue is fixed.
We don't have yet a fixed date for it but I will try to keep you updated once we will start to work on this issue.
Best Regards,
Jonathan

Has the behaviour of this changed? Because DisableBehavaiorTagP = true still seems to remove closing P tags for me. I'm on version 1.7.1
HtmlDocument.DisableBehavaiorTagP = true;

var htmlDocument = new HtmlDocument();

const string testHtml = "
before
middle
after
";
htmlDocument.LoadHtml(testHtml);

var divNode = htmlDocument.DocumentNode.SelectSingleNode("/p/div");
var divParagraph = divNode.ParentNode;
divParagraph.InnerHtml = divParagraph.InnerHtml.Replace(divNode.OuterHtml, "
" + divNode.OuterHtml + "
");

Console.WriteLine(htmlDocument.DocumentNode.InnerHtml);

Expected result:

before

middle

after

Actual result:

before
middle

after

Now, this works in a browser, as the
 element closes the first
 element, but I'm writing something for Facebook Instant Articles, which is very strict, and doesn't want
 elements inside
 elements. Facebook being Facebook, it doesn't consider the behaviour of paragraph elements, and simply says the next element is a child.

Hello @Rene-Sackers ,
Thank you for reporting, we will look at it.
We added some methods that allow us more easily to handle this kind of scenario. So perhaps now we can do something about it.
Best Regards,
Jonathan


{{ array[beigin:end] }} not support in template

how can I do this in template

Please use https://discourse.gohugo.io/ for questions/troubleshooting. Also see  Hugo Documentation.


About the iou of rtmdet

May I ask if rtmdet is in image_ How to set IOU during demo.py prediction? There are always overlapping boxes

how did you solve it?

I solve it by add NMS in demo.py


Modify fsfreeze behavior with paused VMs

What this PR does / why we need it:
This Pull Request aims to modify fsfreeze behavior so, in case of using it with a paused VM, it just returns successfully instead of triggering an error.
Special notes for your reviewer:
Release note:

NONE

/test pull-kubevirt-e2e-k8s-1.23-sig-storage

@mhenriks what do you think about this PR?

When you say returning an error is where are we returning the error? in the freeze/unfreeze if?

When you say returning an error is where are we returning the error? in the freeze/unfreeze if?

virt-freezer returning non zero

/retest-required

Good question. And what do we do on VMExport? Is it allowed to export Paused VM?

I kind of wonder if returning an error is actually the right thing to do. I suspect it is possible that a paused vm has unflushed filesystem buffers in memory. And this is a different from having no guest agent at all where a user presumably understands that there are no guarantees. @brybacki what do you think?

Good question. And what do we do on VMExport? Is it allowed to export Paused VM?

I think the export is only doable in stopped VMs. In running/paused VMs, the vmexport object just remains in pending phase until the VM is stopped.


Question about bitsandbytes

Thanks for your great work!
I would like to ask about your modification in bitsandbytes library since we don't have cuda11 driver, so we are trying to transfer it into the new version of bitsandbytes to run it on cuda12. Thanks!

Thanks for your attention to our work. The main modification focus on custom_bitsandbytes/bitsandbytes/quantization_utils and custom_bitsandbytes/bitsandbytes/nn/modules.py as reference.

Thanks for your reply!
东
@.***

------------------ 原始邮件 ------------------
发件人:                                                                                "ChangyuanWang17/QVLM"                                                                                @.>;
发送时间: 2024年12月2日(星期一) 中午12:53
@.>;
<EMAIL_ADDRESS>主题: Re: [ChangyuanWang17/QVLM] Question about bitsandbytes (Issue #5)
Thanks for your attention to our work. The main modification focus on custom_bitsandbytes/bitsandbytes/quantization_utils and custom_bitsandbytes/bitsandbytes/nn/modules.py as reference.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Thanks for your reply, I would like to ask if there are any modifications in the source code of LLaVA? I need to use bitsandbytes==0.44.2.dev0 for compiling and it seems incompatible with llava 1.0.1.

No, we jest dirrectly use the source code of LLaVA and deploy bitsandbytes==0.41.0 in pyproject.toml
-----原始邮件-----
发件人:"Yongdong Luo" @.>
发送时间:2024-12-02 17:30:58 (星期一)
收件人: ChangyuanWang17/QVLM @.>
抄送: "Changyuan Wang" @.>, Comment @.>
主题: Re: [ChangyuanWang17/QVLM] Question about bitsandbytes (Issue #5)
Thanks for your reply, I would like to ask if there are any modifications in the source code of LLaVA? I need to use bitsandbytes==0.44.2.dev0 for compiling and it seems incompatible with llava 1.0.1.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

Thanks for your kindly sharing!

您好，看到这条issue想请问custom_bitsandbytes是只能在cuda11X上运行吗，我服务器上的cuda driver的版本是12.2，在尝试各种办法后仍遇到了CUDA Setup failed despite GPU being available的问题，不知道是不是和cuda版本有关，谢谢您 @ChangyuanWang17


Don't pull to same directory for push_to_hub in mixin

The current approach clones the repo to the same directory as where the files where saved, which leads to files unexpectedly appearing when pushing to hub. Also, if you then delete files cloned files locally, they are deleted in the remote repo.
This change is consistent with what is done in transformers implementation

which leads to files unexpectedly appearing when pushing to hub

Which files?

FWIW the transformers implementation might change because it seems weird to me to clone to a temp dir – we should just store everything into a git-versioned directory that's the user's current working copy. (like you would do with code)

Which files?

If you have a repo in the Hub, add a model card in the browser, and then you use push_to_hub, the existing implementation will clone the model card to your local directory, which is not what push_to_hub should do IMO. I would rather have an implementation closer to the transformers PushToHubMixin

FWIW the transformers implementation might change because it seems weird to me to clone to a temp dir – we should just store everything into a git-versioned directory that's the user's current working copy. (like you would do with code)

I think users should be able to do both

Simply push directly to the Hub without creating local files or directories
model.push_to_hub("my_repo_name")

Push to the Hub from an existing directory, which is closer to the existing implementation (which uses a staticmethod and really unrelated to the model class)
model.push_to_hub("my-local-directory-nam", "my_repo_name")

But imo, if what a user wants to do is 2, I don't see much of the value of the mixin. The push_to_hub is unrelated to the model object, could be used to upload any directory, and I would instead suggest to use Repository and its push_to_hub method which would already fit in "store everything into a user-accessible, git-versioned directory that's the user's current working copy"

If you have a repo in the Hub, add a model card in the browser, and then you use push_to_hub, the existing implementation will clone the model card to your local directory, which is not what push_to_hub should do IMO.

I think generally (ie. not talking specifically about the mixin here) this is consistent with git and actually a desirable behaviour (you need to pull before push)

Simply push directly to the Hub without creating local files or directories

Yes, why not (though it's obfuscating git a little bit and I'm not sure it's super useful – why wouldn't you want to also store  locally?). This should probably not be named push_to_hub though, but something else, as push_to_hub was the name of the method from Repository

But imo, if what a user wants to do is 2, I don't see much of the value of the mixin. The push_to_hub is unrelated to the model object, could be used to upload any directory

Yes, I kinda agree. I think the "main" API in the mixin was actually supposed to be model.save_pretrained(..., push_to_hub=True) and not this low-level one
maybe it's just a save_to_hub method that doesn't store anything locally (only in tmp) then. Again not sure this is super useful
Tagging @sgugger and @LysandreJik with whom we've had similar recent discussions

Closing this based on previous discussions and direction from transformers  😄


[data-import] Insertion stopped on MaintenanceWorkRule table

Hello,
I have a probleme to inset 12000 lines of datas in the MaintenanceWorkrule table.
I inserted successfully the 800 first lines but after this line , it stopped without mistake.

Is there a limit with inspector on this table?
Thanks for your answer.
best regards
Aurélien CHETANNEAU

Browser : chrome
Version [e.g. 1.20]

No error the process just stopped at nearly 800 lines. i tried with 5 other users and same bahaviour.

If you try to split the import in batches of less than 800 lines, have you the same error ?
I've seen some king of guidance for 2000 records in the help https://help.salesforce.com/s/articleView?id=service.fs_multiple_maintenance_schedule_guidelines.htm&type=5.
Just trying to see if the issue is on the total amount of records to create or the total existing number

Hello. I tried it insert 1 by 1 and it stopped on a line between 650 and 750.
2000 records is the number of workrule which are allowed by MaintenancePlan and we are less than this limit.

Could you try with dataloader to check if you are facing the same behaviour ? I'm afraid this is not related to Reloaded.
Are you in contact with any Technical Account Manager from Salesforce in your project ? He may help you by checking the internal logs of the org.


backtrace-sys still compiled when the backtrace feature is turned off

Apparently the backtrace-sys dependency is always compiled, regardless of whether the backtrace feature is present. This bit me because when compiling for the x86_64-unknown-linux-musl target and the musl C library isn't installed, backtrace-sys can't be compiled.
My question is, why is error-chain depending on backtrace-sys even when the backtrace feature isn't activated?

This is roughly what I was doing to replicate a failed travis build for my mdbook PR, azerupi/mdbook#361.
$ rustup target add x86_64-unknown-linux-musl
$ cargo build --target x86_64-unknown-linux-musl
   Compiling rustc-demangle v0.1.4
   Compiling backtrace-sys v0.1.11
error: failed to run custom build command for `backtrace-sys v0.1.11`
...
--- stderr
configure: error: in `/home/michael/Documents/forks/mdBook/target/x86_64-unknown-linux-musl/debug/build/backtrace-sys-9b035f64636a9441/out':
configure: error: C compiler cannot create executables

I then thought I could work around this by telling error-chain to not use the backtrace feature. With the backtrace feature removed this shouldn't try to compile anything related to backtraces.
// Cargo.toml
- error-chain = "0.10.0"
+ error-chain = { version = "0.10.0", features = [] }

Recompiling then gives you the same error.
The build succeeds if you install musl gcc wrapper (pacman -S musl), but that means you are still building the backtrace-sys crate even when the backtrace feature isn't enabled.

I think you have to add default-features = false.

I tried that too and it still compiles backtrace-sys regardless of whether I specify default-features = false and/or features = [].
error-chain declares backtrace as an optional dependency which is only meant to be compiled when the backtrace feature is enabled, do you know why cargo would still be compiling an optional dependency even when it is told not to?

Hum, did you try the dependency syntax from http://doc.crates.io/manifest.html (search dependencies.awesome)?

I ran into the same issue but as @Yamakaky suggested this fixed it:
[dependencies.error-chain]
error-chain = "0.10.0"
default-features = false


Why I cant find setMap() function for userProperty in Flutter

With Android Lib, I can set multiple properties for userProperty with setMap() function. But I cannot find this function in Flutter library. Is it have been removed? or Can I do it by other way

Any updates on this?


Changelogs not being updated to TestFlight

See https://github.com/fastlane/fastlane/issues/13229 .
Not clear if it's an issue with Fastlane or changelog but I have not changed fastlane or changelog practices or my code for those locally and this just started breaking all of a sudden.  Any ideas?
Thanks! :-)

Hey @joelgetaction ,
it's likely an issue with Fastlane itself not this plugin - it only manipulates content of your CHANGELOG.md not the actual upload to TestFlight 😉


Adds few other sites and a general url tracking cleaner

some of these rules are from letsblockit/filters and a few posts on Reddit, I don't know the exact source for each line :)

Please check these yourself and update me with your findings.

@ZivSimchoni there are a lot of generated id's in the filters, they will be hard to maintain and be very brittle.
Can you please improve those and submit one PR per website as they are easier and faster to review?
Thank you so much for your contributions 🚀


Custom images override default images

If you set a custom image for new theme A, it will override default images... but if you create a new theme B with another image, it will override the image of theme A...
We need theme dependent images.

Currently in the public folder we have the assets folder which contains these files:
public
|-- assets
     |-- logo_big.png
     |-- logo_login.png
     |-- logo_small.png

when uploading the files to the server, those files will get updated.
Now to get theme dependent images, we would have to add a directory for each theme which would contain the images. Eg:
public
|-- assets
    |-- basti
        |-- logo_big.png
        |-- logo_login.png
        |-- logo_small.png

So when uploading, the server needs to put the image files in the correct directory

When a new theme is created, the image-directory needs to be created with the images which are currently selected in the designer!

I have already changed the project structure. The last part of this ticket is depending on the server implementation and how I have to upload the images!

Add the folder structure in the archive and server will create the directory structure!

Ich tried that and it did not work.
I've tried the theme-file-name property with: basti.css and basti
the image is still just being saved in ui/assets/logo_big.png

Sure, it can't work with current VisionX, but will with a newer version.


i can't Run outline in debian 10

hello;
I have encountered this error when launching the software, but I will not fix this problem. I even solved the sandbox issues and interestingly the problem is different to the app client but the manager is working properly.
[9883:1113/083442.445757:FATAL:setuid_sandbox_host.cc(157)] The SUID sandbox helper binary was found, but is not configured correctly. Rather than run without sandboxing I'm aborting now. You need to make sure that /tmp/.mount_OutlinDERrHp/chrome-sandbox is owned by root and has mode 4755.
[1]    9883 trace trap  ./Outline-Client.AppImage

hello ;
It's a pity it's been a while since the issue was introduced, but no developer response.
Now is the time to submit my own answer.
Okay in many places you have been told to fix the not support sandbox you need to upgrade your google chrome to root access.
There is no need for this here.
Happens when you use Chrome/Chromium or libQt5WebEngineCore, or anything else based on Chromium Embedded in an application.
./Outline-Client.AppImage [[5667:1114/085635.801929:FATAL:atom_main_delegate.cc(207)] Running as root without --no-sandbox is not supported. See https://crbug.com/638180.
now insert command
./Outline-Client.AppImage --no-sandbox
open outline client
This will solve your problem.
best regard


The target node information is not available in update operation

I have a use case where I need the information of the target nodes in the EditScript. However, only the target node text is passed to the Update objects. For example the code at  code_diff/__init__.py:L200 is
if source_ast.type == target_ast.type and len(source_ast.children) == 0 and len(target_ast.children) == 0:
  # Both nodes are tokens of the same type
  # Only an update is required
  return EditScript([Update(source_ast, target_ast.text)])

I suggest that the last line to be replaced with:
return EditScript([Update(source_ast, target_ast)])

I am not a fan of referencing the target tree in the edit script. I would prefer if the edit script is only dependent on the tree for which it was computed for.
In addition, finding the subtree which is related to specific operation is not always possible (e.g. what is the related subtree for Move or Delete operations?)
If you are only interested in the update operation, you could compute the edit map (see code_diff/gumtree/init.py#L23 ). Whenever there are two nodes with different labels in the edit map, we would generate an update operation (+ some adjustments of the child nodes).


jsshell testing doesn't work on Windows

jsshell testing doesn't work on Windows:
d:\dart-repo\dart>python tools/test.py -mrelease -cdart2js -rjsshell --use-sdk --write-debug-log --write-test-outcome-log -t60 lib/mirrors/mirrors_reader_test
Test configuration: dart2js_jsshell_release_ia32
Process error:
  Command: \d:\dart-repo\dart\tools\testing\bin\jsshell.bat d:/dart-repo/dart/build/ReleaseIA32/generated_compilations/dart2js-sdk/tests_lib_mirrors_mirrors_reader_test/out.js
  Error: ProcessException: The system cannot find the file specified.
  Command: \d:\dart-repo\dart\tools\testing\bin\jsshell.bat d:/dart-repo/dart/build/ReleaseIA32/generated_compilations/dart2js-sdk/tests_lib_mirrors_mirrors_reader_test/out.js
Note that
  Command: \d:\dart-repo\dart\tools\testing\bin\jsshell.bat d:/dart-repo/dart/build/ReleaseIA32/generated_compilations/dart2js-sdk/tests_lib_mirrors_mirrors_reader_test/out.js
should have been
  Command: d:\dart-repo\dart\tools\testing\bin\jsshell.bat d:/dart-repo/dart/build/ReleaseIA32/generated_compilations/dart2js-sdk/tests_lib_mirrors_mirrors_reader_test/out.js
That is without the prefixed '/'.

jsshell is no longer a supported configuration.


sh: /Applications/Xcode: No such file or directory

I've installed the application with ruby 2.2 and rbenv and I get the following error:
➜  ~  xcode-install install 6.4
sh: /Applications/Xcode: No such file or directory
/usr/local/opt/rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/xcode-install-0.1.0/lib/xcode/install.rb:223:in `get_version': undefined method `split' for nil:NilClass (NoMethodError)
	from /usr/local/opt/rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/xcode-install-0.1.0/lib/xcode/install.rb:216:in `initialize'
	from /usr/local/opt/rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/xcode-install-0.1.0/lib/xcode/install.rb:92:in `new'
	from /usr/local/opt/rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/xcode-install-0.1.0/lib/xcode/install.rb:92:in `block in installed_versions'
	from /usr/local/opt/rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/xcode-install-0.1.0/lib/xcode/install.rb:92:in `map'
	from /usr/local/opt/rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/xcode-install-0.1.0/lib/xcode/install.rb:92:in `installed_versions'
	from /usr/local/opt/rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/xcode-install-0.1.0/lib/xcode/install.rb:88:in `installed?'
	from /usr/local/opt/rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/xcode-install-0.1.0/lib/xcode/install/install.rb:25:in `validate!'
	from /usr/local/opt/rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/claide-0.8.1/lib/claide/command.rb:311:in `run'
	from /usr/local/opt/rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/xcode-install-0.1.0/bin/xcode-install:12:in `<top (required)>'
	from /usr/local/opt/rbenv/versions/2.2.0/bin/xcode-install:23:in `load'
	from /usr/local/opt/rbenv/versions/2.2.0/bin/xcode-install:23:in `<main>'

➜  ~  xcode-install list
sh: /Applications/Xcode: No such file or directory
/usr/local/opt/rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/xcode-install-0.1.0/lib/xcode/install.rb:223:in `get_version': undefined method `split' for nil:NilClass (NoMethodError)
	from /usr/local/opt/rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/xcode-install-0.1.0/lib/xcode/install.rb:216:in `initialize'
	from /usr/local/opt/rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/xcode-install-0.1.0/lib/xcode/install.rb:92:in `new'
	from /usr/local/opt/rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/xcode-install-0.1.0/lib/xcode/install.rb:92:in `block in installed_versions'
	from /usr/local/opt/rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/xcode-install-0.1.0/lib/xcode/install.rb:92:in `map'
	from /usr/local/opt/rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/xcode-install-0.1.0/lib/xcode/install.rb:92:in `installed_versions'
	from /usr/local/opt/rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/xcode-install-0.1.0/lib/xcode/install.rb:189:in `list_versions'
	from /usr/local/opt/rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/xcode-install-0.1.0/lib/xcode/install.rb:127:in `list_current'
	from /usr/local/opt/rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/xcode-install-0.1.0/lib/xcode/install/list.rb:22:in `run'
	from /usr/local/opt/rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/claide-0.8.1/lib/claide/command.rb:312:in `run'
	from /usr/local/opt/rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/xcode-install-0.1.0/bin/xcode-install:12:in `<top (required)>'
	from /usr/local/opt/rbenv/versions/2.2.0/bin/xcode-install:23:in `load'
	from /usr/local/opt/rbenv/versions/2.2.0/bin/xcode-install:23:in `<main>'

➜  ~  ll /Applications/Xcode*
/Applications/Xcode 2.app:
total 0
drwxr-xr-x@ 16 romain  admin   544B 26 Feb 19:09 Contents

/Applications/Xcode-beta 4.app:
total 0
drwxr-xr-x@ 16 romain  admin   544B 20 Mar 01:57 Contents

/Applications/Xcode.app:
total 0
drwxr-xr-x  17 root  wheel   578B  8 Apr 22:09 Contents

➜  ~  ruby -v
ruby 2.2.0p0 (2014-12-25 revision 49005) [x86_64-darwin14]

Any idea? Thanks!

This looks like an error with spaces in the path to existing Xcodes, in your case "/Applications/Xcode 2.app". I'll whip up a fix, in the meantime, you should be able to use XcodeInstall by removing/replacing the space.

Indeed, that was it :)

@viteinfinite was so kind to fix this in #20 :tada:


Delete information about WAM

I checked with product group that LTSB or LTSC includes WAM module. So, we should delete "(Not included in Windows Server 2016 LTSC builds)" from following sentence.

Web Account Manager (WAM): WAM is the default token broker on Windows 10 or newer devices. WAM also provides a plugin framework that identity providers can build on and enable SSO to their applications relying on that identity provider. (Not included in Windows Server 2016 LTSC builds)

@Castermeek : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@MicrosoftGuyJFlo
Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Fix Code Sign iOS

I have a code sign problem on the v3.0.3, I can't use Mixpanel v3.0.3 as a carthage dependency because of that. Here is the fix.

Thanks @jdauphant for the PR! I'll merge it and test it out with the fix. Will probably send out an update this week or early next week. Thanks again!


Update crypto.ripemd160 method to use full algorithm name when alias not available

There is an issue with using the alias rmd160 with Electron, and the developers suggest to use the full algorithm name ripemd160 instead of fixing it in Electron itself.
https://github.com/electron/electron/issues/16195#issuecomment-477660937
The same fix is also applied to bitcoinjs.
https://github.com/bitcoinjs/bip32/pull/16

I forked this library at the very beginning of the BITBOX journey and we haven’t actively maintained it. Now it’s at the point that it’s very challenging to get the tests to run. So we try not to change anything so that we don’t break anything.
We’re taking the more conservative approach of creating new methods with the new functionality instead of updating existing methods.
Can you create a new method called ripemd160_electron and move your updated code there. Then we can talk about the BITBOX update once that is in place.

Unfortunately making the changes in the way that you have described would actually be more invasive than making the update in this PR. The reason is that this method is not being used directly by the desktop wallet, but it being used in different places in the SDK itself. So if I were to make a new method called ripemd160_electron, I would have to go in and update any and all upstream methods that call this method.

It's simply too risky to merge this PR and risk introducing a regression bug to BITBOX. Any PRs to this library need to use new function names so as to ensure backwards compatibility is preserved.


Add device creation to client

I have lots of devices that need to be created via API. Would it be possible to get a method on the data platform client for creating devices so I don't have to do it via requests?

@foxymiles sending to your queue


Update index.html - Initial Layout

Initial layout of the HTML file

Initial structure for #13 merged


CSV / PDF export: simplify exporterHeaderFilter, exporterFieldCallback

When exporting to CSV / PDF, the raw values are sent, unless exporterHeaderFilter and exporterFieldCallback are specified.
One could use already defined headerCellFilter / cellFilter, and provide both raw and filtered values to exporterFieldCallback. This would drastically ease the programmer's life, and still would be backwards-compatible, if the filtered value comes as the last argument to exporterFieldCallback.

On the second thought, if the cellFilter is applied in the custom template, where some additional data formatting is performed, this would only cause additional errors. So never mind ;-)


Transporter performance issues.

After a server has been running for a while, it's tick rate drops to ridiculously low rates. It is resolved by restarting the server. However, getting a jvisualvm sampler snapshot shows the following:

The full snapshot is here:
https://dl.dropboxusercontent.com/u/23843338/meklag.nps
Version <IP_ADDRESS>

It is possible under certain conditions for transporters to accept items without places to go, most likely there was some form of machine backlog that allowed for them to keep accepting items.


Add upload/download of registry tree

It would be very useful to be able to download an entire registry tree in some useful intermediate form (ideally XML or other human readable format) and then upload it.  This could be via explicit buttons and/or drag-and-drop.
I made an attempt to do this in python a couple years ago, and it didn't work very well, but I did write a labrad compatible XML flattener (that is probably not compatible with current pylabrad either, but might be useful to look at):
https://github.com/ejeffrey/Registry_Browser/blob/master/labrad_xml.py

Do you want to be able to drag in indivudual xml files or directories containing xml files?

I want a single file that represents either an entire registry directory, or a single key/value pair.

To be clear:  I am not looking for something that interacts with the registry server's internal database representation (the delphi text files or the scala sqllite database).  This is just an import/export tool as part of the scalabrad-web package that lets you copy e.g., the configuration of a single server or sample and email it to someone for use on another system.  I think it would be most useful if it was at least somewhat human readable, although a binary file or json would also work.

If you have access to both managers, there's a migration tool that will copy data; it would be easy to add source and destination paths to copy only a subtree. But a file format would be useful for sending to someone whose manager you can't directly connect to, as you suggest.

It is also way more useful if it is built into a common tool.  It should
also work if I have two windows open connected to scalabrad-web and I drag
and drop between them, whether they are connected to the same manager in
different folders or to different managers.
On Thu, Sep 24, 2015 at 1:34 PM, Matthew Neeley<EMAIL_ADDRESS>wrote:

If you have access to both managers, there's a migration tool that will
copy data; it would be easy to add source and destination paths to copy
only a subtree. But a file format would be useful for sending to someone
whose manager you can't directly connect to, as you suggest.
—
Reply to this email directly or view it on GitHub
https://github.com/labrad/scalabrad-web/issues/52#issuecomment-143043551
.

While I do think this is a useful feature, I'd like to voice my opinion, once again, that having a nice registry editor etc. is probably not the most important thing our one dedicated SWE could be doing with his time at the moment. It seems like this tool is far enough along that other developers can start to contribute these little goodies.

Dan, I filed this bug because I have spent several days setting up labrad
and all of the associated bits on our new lab network, and features like
this would have / will make my life a lot easier.  We have years of effort
into configuring our labrad system at UCSB and no good way to replicate
that piecemeal elsewhere, which I think is a critical bug.
On Thu, Sep 24, 2015 at 2:36 PM, Daniel Sank<EMAIL_ADDRESS>wrote:

While I do think this is a useful feature, I'd like to voice my opinion,
once again, that having a nice registry editor etc. is probably not the
most important thing our one dedicated SWE could be doing with his time
at the moment. It seems like this tool is far enough along that other
developers can start to contribute these little goodies.
—
Reply to this email directly or view it on GitHub
https://github.com/labrad/scalabrad-web/issues/52#issuecomment-143057275
.

I get that. Perhaps one of us can start contributing to this project now that the build system is reasonably stable!


ID-610 DRSHub gets signed URLs for GCS-backed DRS URIs

https://broadworkbench.atlassian.net/browse/ID-610
In order to retire Martha, DRSHub needs to replicate its functionality. DRSHub currently returns the Access URL defined in the DRS Spec, but won't actually return a signed URL for a GCS object using the fence account like Martha.
This PR replicates this functionality in DRSHub.

I've also updated the github actions checkout version because of the Node12 deprecation. Figured that I should just do it while I'm in the repo.


Update Laravel 10 compatibility

Update Laravel 10 compatibility

Thanks for your PR. I have already upgraded the version.


Merge Dev to Master

Forked PGoAPI version changes causes those on master not able to run bot.
Need to merge this asap.

You also need to change run.sh line 25
required=(cat requirements.txt | sed -e 's/.pgoapi$/pgoapi==1.2.0/' -e 's/[-_]//g' -e 's/==(.)/:\1/' | awk '{print tolower($0)}')
to
required=(cat requirements.txt | sed -e 's/.pgoapi$/pgoapi==1.2.1/' -e 's/[-_]//g' -e 's/==(.)/:\1/' | awk '{print tolower($0)}')

@TheCannings Done. And merged to Dev

Revising

Hi @BreezeRo Can join us in Discord?


Create <IP_ADDRESS>.yaml

Microsoft Reviewers: Open in CodeFlow

 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.:x: bergerpascal sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Ignore quotes and braces and case

Braces from SQL definition
Ignore braces in column names, as they are in the sql column from the sqlite_schema table:
CREATE TABLE "artists"
(
    [ArtistId] INTEGER PRIMARY KEY AUTOINCREMENT NOT NULL,
    [Name] TEXT(120)
);

Quotes in table names
Ignore quotes in the table names:
const query = generatedSchema.sql<{
  GenreId: number;
  Name: string | null
}>`SELECT * FROM "genres"`;

Without quotes, column selection fails:
const query = generatedSchema.sql<{

}>`SELECT * FROM genres`;

Matching case in column and table names
Ignore column and table case in the queries:
  const query = schema.sql<{
  Name: string | null
}>`SELECT Name FROM artists`;

Without matching case, column selection fails:
  const query = schema.sql<{/*
  Could not find selected column name in from clause
*/}>`SELECT name FROM artists`;

Missing Semicolons
Have a user friendly error message when defining schema and forgetting a semicolon or two:
CREATE TABLE "albums"
(
    AlbumId INTEGER PRIMARY KEY AUTOINCREMENT NOT NULL,
    Title TEXT(160)  NOT NULL,
    ArtistId INTEGER  NOT NULL,
    FOREIGN KEY (ArtistId) REFERENCES "artists" (ArtistId)
		ON DELETE NO ACTION ON UPDATE NO ACTION
)

CREATE TABLE sqlite_sequence(name,seq)

Quotes in table names should be resolved. Working on braces next.
I've also found that I get into a broken state pretty often where no types generate :\
Curious if you've seen that.


Local Rate Limit With Descriptors: Are tokens from default bucket consumed?

Title: Local Rate Limit With Descriptors: Are tokens from default bucket consumed?
Description:
The docs about Using rate limit descriptors for local rate limiting states that If there is no matching descriptor entries, the default token bucket is used:

The local descriptor’s token bucket settings are then used to decide if the request should be rate limited or not depending on whether the local descriptor’s entries match the route’s rate limit actions descriptor entries. If there is no matching descriptor entries, the default token bucket is used.

It gives me the idea that no token would be consumed from the default token bucket if there is a match between the request and the descriptors. However looking at this comment and evaluating the Local Rate Limiter implementation source code, it looks like tokens from global bucket are consumed for all requests, even when descriptors match.
To confirm it, I have done some tests, as shown below:

Global token bucket containing less tokens than the descriptor token bucket:

Envoy configuration:

Requests:

In this scenario, although we have configured a rate limit of 5 requests each 30 seconds to client application app1, it was able to execute only one, since the global bucket was configured with 1 token each 30 seconds.

Global token bucket containing more tokens than the descriptor token bucket:

Envoy configuration:

Requests:

In this scenario, the client application app1 was able to execute 2 requests in 30 seconds as allowed by the descriptor rate limit, since the global bucket was configured with 10 tokens each 30 seconds.
Is my understanding correct? If so, maybe the documentation could let this behavior clearer.
In my personal opinion, it makes sense always consume a token from the global bucket, since the main function of the Rate Limit is not allowing more requests than the maximum supported by the endpoint.
One more question. Considering a scenario where I need to guarantee throughput (requests per second) quotas per application, for example, application app1 must have the right to make up to 10 requests each second, application app2 must have the right to make up to 20 requests each second, and all other applications together, up to 100 requests each second. In this case, should I implement a custom local rate limit plugin or a custom global rate limit service?
Many thanks!
Diego

Giving a little bit more context.
Since global bucket tokens are burned before matching descriptors, it is possible that all tokens from global buckets are consumed even when requests are rejected by overrides. E.g:

In this case, the default bucket is configured to receive 5 tokens each 30 seconds and overrides allowing 2 requests each 30 seconds from app1 and app2.
If app1 sends 5 requests in 30 seconds the first 2 requests are be allowed and the last 3 rejected, as expected.
Nevertheless, since all tokens from the default bucket are burned, requests from app2 are rejected even when the configured limit is not reached.

@wbpcode @mattklein123

cc @diegomazine you are right. The logic of local limit is a little complex because the globle token and mutiple descriptor could be used. orz.

Since global bucket tokens are burned before matching descriptors, it is possible that all tokens from global buckets are consumed even when requests are rejected by overrides.

right. do you have any ideal to optimize it?

Hi, @wbpcode! Many thanks for your answer.
I think that we could update the method LocalRateLimiterImpl::requestAllowed to first match local descriptors (no changes on the sorting algorithm will be necessary), rejecting the request if there is no available token in the matching token buckets. Only then, the number of tokens in the default bucket is validated.
This way, no token from the default token bucket is burned if the request is reject because there is not enough tokens in the matching local descriptor’s token buckets.

Hi, @wbpcode! Many thanks for your answer.
I think that we could update the method LocalRateLimiterImpl::requestAllowed to first match local descriptors (no changes on the sorting algorithm will be necessary), rejecting the request if there is no available token in the matching token buckets. Only then, the number of tokens in the default bucket is validated.
This way, no token from the default token bucket is burned if the request is reject because there is not enough tokens in the matching local descriptor’s token buckets.

Then when the global token has a more strict limit, the tokens of local descriptors may be burned even if it's rejected by the global token. 🤔

I think I got your point. For example:

Global Rate Limit: 1 request per second
Descriptor A: 1 request per minute
Descriptor B: 1 request per minute

If both, descriptors A and B, send simultaneous requests, one of them won't get another chance within that minute. Have I got that right?
Considering that might be valid scenarios that requires burning global tokens before tokens of local descriptors and vice versa, maybe the Local Rate Limit plugin should implement both strategies. By default, global tokens will be burned first.
But if necessary, the plugin might be configured to utilize a greedy strategy that burns tokens of local descriptors first.

I think I got your point. For example:

Global Rate Limit: 1 request per second
Descriptor A: 1 request per minute
Descriptor B: 1 request per minute

If both, descriptors A and B, send simultaneous requests, one of them won't get another chance within that minute. Have I got it right?
Considering that might be valid scenarios that requires burning global tokens before tokens of local descriptors and vice versa, maybe the Local Rate Limit plugin should implement both strategies. By default, global tokens will be burned first. But, when necessary, the plugin might be configured to utilize a greedy strategy that burns tokens of local descriptors first.

@wbpcode i don't think I get your point, does it actually make sense to have a global bucket smaller than descriptors? It's actually recommended in the code that global bucket should be large because it is always consumed.

@wbpcode i don't think I get your point, does it actually make sense to have a global bucket smaller than descriptors? It's actually recommended in the code that global bucket should be large because it is always consumed.

Sounds reasonable to me. But it would be better to add a NOTE or warn to the end users' doc. I think it's a drawback of current design and implementation.
cc @diegomazine

Hi @wbpcode,
I have submitted the PR that updates the local rate limit filter.
Thanks a lot,
Diego


Merge add/delete buttons into one

Fixes https://github.com/BookMyComics-Developers/BookMyComics/issues/104.

LGTM on principle and base don diff. waiting CI


For B1 Logss

Node ID:051c72a467abdda8

confirmed:B1
node:051c72a467abdda8

4 May 2022
17.52 (UTC+3)
(Form Submitted)


[RuleDocGenerator] Fixes rectorphp/rector#4756 AbstractCodeSample parameter order

Thank you

"Check for Git Concficts Expected — Waiting for status to be reported" seems hanging...

@TomasVotruba github action shows "(waiting for status checks: {'Check for Git Concficts'})" which hanging, and still typo as well (already fixed in master), while this branch already rebased.

Close - re-open to check.

Now composer install error in php 7.3:
  - Installing tracy/tracy (v2.7.0): Extracting archive
Error: Failed to execute git checkout '1dc13433a50828c613c9ae50f339fd2309817aa7' -- && git reset --hard '1dc13433a50828c613c9ae50f339fd2309817aa7' --

fatal: reference is not a tree: 1dc13433a50828c613c9ae50f339fd2309817aa7

I see. Look like some composer bug.
The PHP 7.4 tests pass, so I guess the rest should be correct.

Thank you :+1:


nerves_system_br 0.13.5

Brings in

Fix deprecation warnings for Elixir 1.5
fwup 0.15.4 - Changed signing keys to be base64 encoded. Added commandline parameters for passing public and private keys via commandline arguments
Improved error messaging for when a toolchain is not found when requiring nerves_env.exs

👍
Tested on Linkit
Using https://s3.amazonaws.com/nerves/test/branch/nerves_system_linkit-system_br.fw


Add support for item_data type event

Describe the bug
When using Square Online, you can create an item that is for an Event. This item has metadata on it to differentiate it from other items in your catalog. Today, you can't ask the API for all items of type event. You must first ask for all items and then filter on event key under item_data. This is inconvenient especially as there are limits on how much the API can return over time.
Expected behavior
There should be a way to at least request all items with item_data.event as well as a way to paginate based on start_date. This data is already part of the payload today, ex:
        "event": {
          "uid": "*********",
          "start_at": "2023-07-01T20:00:00+00:00",
          "end_at": "2023-07-02T00:00:00+00:00",
          "event_location_time_zone": "America/Los_Angeles",
          "event_location_name": "EXAMPLE",
          "event_location_types": [
            "IN_PERSON"
          ],
          "address_id": "************",
          "all_day_event": false

To Reproduce
Go to square online and create a catalog item with type Event. Fill in the details. Then try and use the SDK to query for just that item(s). You cannot.
Screenshots
If applicable, add screenshots to help explain the bug.
Square SDK version
For example:  17.2.x
25.1.0
Additional context
Add any other context about the problem here.

Hi @Aghassi
I have reached out to our API team for further information. I'll report back here when I have an update for you. Thanks for bringing this to our attention!
In the future, we suggest that questions like this be asked on our Developer Forums as these forums are monitored by Square Employees and you will receive a response to questions within 48 hours.

Hey @Aghassi,
The team got back to me pretty quickly. This Improvement (along with achieving parity with Square Online for other types that are not supported in the catalog) is on the Roadmap for 2023, but unfortunately I don't have much more of a clear timeline than that. Rest assured this problem is known and will get addressed.
In the meantime I am going to close this ticket as it's not directly an issue with the nodejs SDK, and is instead an improvement to the API itself.
Thanks again for your feedback 😄
cheers!

Thanks for the update! Look forward to this improvement! I'll use the forum next time. I'm use to operating with open source work on GitHub which is why I filed her first. Thanks again!


Possible to merge some work with Upstream Webkit?

Webkit people have told me that they would be okay with accepting patching that are JSCOnly port, because then upstream can do things like it will be possible to have buildbot and run tests, to prevent regressions.

FWIW, I've just committed proper fix for issue corresponding to https://github.com/SoftwareMansion/jsc-android-buildscripts/blob/master/patches/intl/icu-disabled.patch (see https://trac.webkit.org/changeset/228136)


Fixing issue #2090 Searching night mode

Searching for images or categories in night mode works but the words doesn't show neither the recent researches.
Fixes #2090 Searching on Night Mode
What changes did you make and why?
Ux changes to be more clean on night mode
Tests performed (required)
Exploratory testing
Screenshots showing what changed (optional - for UI changes)

Codecov Report

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

@@          Coverage Diff           @@
##           master   #2095   +/-   ##
======================================
  Coverage    4.02%   4.02%
======================================
  Files         226     226
  Lines       11467   11467
  Branches     1064    1064
======================================
  Hits          461     461
  Misses      10972   10972
  Partials       34      34

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 677e009...2420127. Read the comment docs.

Thanks for your first time contribution @pereiraJr :) I have made some notes, please consider them. By the way it can be nice adding before after UI comparisons for such UI changes
Basically before:

And after:

There is a nice UI improvement. But you changed some of files you shouldn't. Whenever you fixed them we can review your PR again.

As final decision, I request changes @pereiraJr :)


No clear document explains how to use pre-trained model

URL(s) with the issue:
Please provide a link to the documentation entry, for example:
https://www.tensorflow.org/versions/r1.14/api_docs/python/tf/keras/applications
Description of issue (what needs changing):
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 16.04 in Docker
TensorFlow installed from (source or binary): pip install
TensorFlow version (use command below): v2.0.0-rc2-26-g64c3d38
Python version: 3.5
CUDA/cuDNN version: 10.0 / 7
GPU model and memory: GTX 1080Ti / 11175MiB

Describe the current behavior
Hi authors and developers,
I noticed that tensorflow doesn't provide a clear document explain how to use pre-trained model.
So, I wrote a benchmark which showed the accuracy of pre-trained model with applying imageNet' validation set.
The following is the result:
[Testing][pixel vales are from (0,255)][model:ResNet50] - loss: 2.711 - accuracy: 0.457
[Testing][pixel vales are from (0,255)][model:DenseNet121] - loss: 39.000 - accuracy: 0.006
[Testing][pixel vales are from (0,255)][model:MobileNetV2] - loss: 9.979 - accuracy: 0.003

[Testing][pixel vales are from (0,1)][model:ResNet50] - loss: 8.535 - accuracy: 0.001
[Testing][pixel vales are from (0,1)][model:DenseNet121] - loss: 1.895 - acc: 0.599
[Testing][pixel vales are from (0,1)][model:MobileNetV2] - loss: 2.283 - accuracy: 0.523

[Testing][pixel vales are normalized from (-1,1)][model:ResNet50] - loss: 8.313 - acc: 0.001
[Testing][pixel vales are normalized from (-1,1)][model:DenseNet121] - loss: 1.896 - acc: 0.599
[Testing][pixel vales are normalized from (-1,1)][model:MobileNetV2] - loss: 2.287 - acc: 0.524

First, we can see the accuracy is not comparable with the original result(Top-1 accuracy is 70% up).
I thought that this issue is I'm not sure which crop and pad method is applied in the original result.
Therefore, I defined a custom function CenterCrop to fit the model's input size.
But, we can skip this issues there.
What I want to mention is normalization issue.
If I don't apply any normalization(run_aug=1 in code), pixel's values are defined in (0, 255).
All models' accuracy are near 0.001, except for resNet50 which achieves a meaningful accuracy.
If I do normalization(run_aug=2 in code), pixel's values are defined in (0, 1).
This time, DenseNet121 and MobileNetV2 have a meaningful accuracy.
If I do standard normalization(run_aug=3 in code), pixel's values are defined in (-1, 1).
The results are similar to previous case. But I'm sure why those two cases have same accuracy.
Those behavior let me confused.
Before applying pre-trained model, I have to which normalization method should be applied.
After reading the source code, I found that those applications are import from keras_application in tensorflow.
keras-applications
weight download

I didn't test other models, such as ResNet50V2, InceptionV3 and Xception because their input size are 299 instead of 244 and this is a time consuming task.
However, anyone can modify the test case and do the benchmark.

Because of licence issue for ImageNet, I can't provide imagenet in public.
But the following is the minimal test case:
# pip install tensorflow-gpu==1.14.0
# pip pandas
#%%
import time
import numpy as np
import pandas as pd
import tensorflow as tf

from glob import glob

#%%
# input image dimensions
img_h = 224
img_w = 224
channels = 3

# information for dataset
dataset_path = "dataset-imagenet/"
num_classes = 1000
num_testing = 50000

#%%
class DataGenerator:

    def __init__(self, dataframe, batch_size, run_aug = True):

        self.total_len  = len(dataframe.index)
        self.batch_size = batch_size
        self.run_aug = run_aug
        self.dataframe  = dataframe
        self.on_epoch_end()

    def __build_pipeline(self, file_path, labelY):

        # mapping function in tf
        def preprocess_fn(file_path, labelY):

            def fn_x(img_array):

                img_array = img_array.numpy()

                if self.run_aug == 1:
                    # image's range is [0,255]
                    image = img_array

                if self.run_aug >= 2:
                    # image's range is [0,1]
                    image = img_array / 255.0

                if self.run_aug == 3:
                    # std normalization
                    image[0,:,:] -= 0.485
                    image[1,:,:] -= 0.456
                    image[2,:,:] -= 0.406
                    image[0,:,:] /= 0.229
                    image[1,:,:] /= 0.224
                    image[2,:,:] /= 0.225

                return image

            def fn_y(label):
                return tf.keras.utils.to_categorical(label , num_classes)

            # read image from files
            image = tf.io.read_file(file_path)
            image = tf.image.decode_image(image, channels=channels)
            aug_size = 256
            imageX = tf.compat.v1.image.resize_image_with_pad(image, aug_size, aug_size)
            imageX = tf.image.resize_with_crop_or_pad(image, img_h, img_w)

            # do normalizarion
            [imageX] = tf.py_function(fn_x, [imageX], [tf.float32])
            imageX.set_shape([img_h, img_w, channels])
            imageX = tf.image.random_flip_left_right(imageX)

            [labelY] = tf.py_function(fn_y, [labelY], [tf.float32])
            labelY.set_shape([num_classes])

            return imageX, labelY

        dataset = tf.data.Dataset.from_tensor_slices( (file_path, labelY) )
        dataset = dataset.shuffle(batch_size * 8)
        dataset = dataset.repeat()
        dataset = dataset.map(preprocess_fn, num_parallel_calls=tf.data.experimental.AUTOTUNE)
        dataset = dataset.batch(self.batch_size)
        dataset = dataset.prefetch(tf.data.experimental.AUTOTUNE)

        self.dataset   = dataset

    def  __len__(self):

        return self.total_len // self.batch_size

    def on_epoch_end(self):

        cleanX = np.array(self.dataframe["File"])
        totalY = np.array(self.dataframe["One-hot"])

        # run permutation
        rand_idx = np.random.permutation(self.total_len)
        cleanX = cleanX[rand_idx]
        totalY = totalY[rand_idx]

        self.__build_pipeline(cleanX, totalY)

#%%
def build_clf(model_name):

    if model_name == "ResNet50":
        clf_model = tf.keras.applications.ResNet50(include_top=True, pooling='max', weights='imagenet')

    if model_name == "DenseNet121":
        clf_model = tf.keras.applications.DenseNet121(include_top=True, pooling='max', weights='imagenet')

    if model_name == "MobileNetV2":
        clf_model = tf.keras.applications.MobileNetV2(include_top=True, pooling='max', weights='imagenet')

    if model_name == "InceptionV3":
        clf_model = tf.keras.applications.InceptionV3(include_top=True, weights='imagenet')

    clf_model.compile(loss='categorical_crossentropy', optimizer='adam', metrics=['accuracy'])

    return clf_model

#%%
def list_testing_data(classes, file_path, onehot_map):

    try:
        testing_data = pd.read_pickle('imagenet_test_list.pkl')
        print('[Successful] Testing_data loaded from pickle ...')
    except:
        testing_image_info = []
        for iter_class in classes:
            files = glob(os.path.join(file_path, iter_class, '*.JPEG'))
            for iter_img in files:
                data_info = [iter_img, iter_class]
                testing_image_info.append(data_info)

        testing_data = pd.DataFrame(testing_image_info, columns=['File', 'Class'])
        testing_data["One-hot"] = testing_data["Class"].replace(onehot_map, inplace=False)

        testing_data.to_pickle('imagenet_test_list.pkl')

    assert(testing_data.shape[0] == num_testing, "[Fatal] Mismatched total length of testing data")
    return testing_data

#%%
if __name__ == '__main__':

    # set GPU
    import os
    if os.environ.get("CUDA_VISIBLE_DEVICES") is None:
        os.environ["CUDA_VISIBLE_DEVICES"] = "0"

    # Hyperparameters
    batch_size = 100
    epochs = 5

    # load one-hot labels
    file_path = dataset_path + 'val'
    classes = os.listdir(file_path)
    list_class = sorted( list( set(classes) ) )
    onehot_map = dict( zip( list_class, list(range(0, num_classes)) ))

    # load list of validation data, those data should be considered as testing data
    testing_data = list_testing_data(classes, file_path, onehot_map)

    # build data generator
    gen_type1 = DataGenerator(testing_data, batch_size, run_aug=1)
    gen_type2 = DataGenerator(testing_data, batch_size, run_aug=2)
    gen_type3 = DataGenerator(testing_data, batch_size, run_aug=3)
    gen_list = [gen_type1, gen_type2, gen_type3]

    # build model
    model_list = ["ResNet50", "DenseNet121", "MobileNetV2"]

    # print result for type1
    test_gen = gen_type1
    for model_name in model_list:
        model = build_clf(model_name)
        meta_string = '[Testing][pixel vales are from (0,255)][model:{:s}] '.format(model_name)
        prefix_string = ''
        output = model.evaluate(test_gen.dataset, steps = test_gen.__len__())
        for ii in range( len( model.metrics_names) ):
            meta_string = meta_string + '- {:s}{:s}: {:.3f} '.format(prefix_string, model.metrics_names[ii], output[ii])

        print(meta_string)

    # print result for type2
    test_gen = gen_type2
    for model_name in model_list:
        model = build_clf(model_name)
        meta_string = '[Testing][pixel vales are from (0,1)][model:{:s}] '.format(model_name)
        prefix_string = ''
        output = model.evaluate(test_gen.dataset, steps = test_gen.__len__())
        for ii in range( len( model.metrics_names) ):
            meta_string = meta_string + '- {:s}{:s}: {:.3f} '.format(prefix_string, model.metrics_names[ii], output[ii])

        print(meta_string)

    # print result for type3
    test_gen = gen_type3
    for model_name in model_list:
        model = build_clf(model_name)
        meta_string = '[Testing][pixel vales are normalized from (-1,1)][model:{:s}] '.format(model_name)
        prefix_string = ''
        output = model.evaluate(test_gen.dataset, steps = test_gen.__len__())
        for ii in range( len( model.metrics_names) ):
            meta_string = meta_string + '- {:s}{:s}: {:.3f} '.format(prefix_string, model.metrics_names[ii], output[ii])

        print(meta_string)

@CNOCycle,
Could you please take a look at this link and check if it helps. Thanks!

@amahendrakar
Thank for your reply. I'm not want to do transfer learning. What I want to do is reproduce the result of pre-trained model.
Those pre-trained model's weight are from imagenet dataset, so those models' top-1 accuracy on imagenet should be 70% up.
Also, from result which I posted, the behavior of those models are not unified.
So, my suggestion is that tf should mentioned input image should do normalization manually or not before feeding input image into pre-trained model.

Oh, there is still no clear document in tensorflow=2.0.0
https://www.tensorflow.org/api_docs/python/tf/keras/applications

All pre-trained modes using TF mode expect the input channels to be in range of [-1,1]
https://github.com/tensorflow/tensorflow/blob/23c3bdaacdc27bb82dfd1772efefad687508923a/tensorflow/python/keras/applications/imagenet_utils.py#L36

@ymodak
Thank your reply.  I found that a clear comment have been added in the master branch.
But, I still doubt whether this description is correct or not.
Because in my test case, ResNet50 model's accuracy is near 0.001 if we normalize the value range to [-1, 1].
In the tutorial(https://www.tensorflow.org/tutorials/generative/adversarial_fgsm),
# Helper function to preprocess the image so that it can be inputted in MobileNetV2
def preprocess(image):
  image = tf.cast(image, tf.float32)
  image = image/255
  image = tf.image.resize(image, (224, 224))
  image = image[None, ...]
  return image

It only rescales data to range [0, 1] instead of [-1, 1].

For anyone else finding this: TensorFlow requires images to be in range [0,1), see https://www.tensorflow.org/api_docs/python/tf/image/convert_image_dtype

Images that are represented using floating point values are expected to have values in the range [0,1). Image data stored in integer data types are expected to have values in the range [0,MAX], where MAX is the largest positive representable number for the data type.

All pre-trained modes using TF mode expect the input channels to be in range of [-1,1]
https://github.com/tensorflow/tensorflow/blob/23c3bdaacdc27bb82dfd1772efefad687508923a/tensorflow/python/keras/applications/imagenet_utils.py#L36

I think this is the right approach, i.e., using keras-application preprocess_input to get meaningful results, regardless of convert_image_dtype. This should have already been tested.

Closing this issue now. Thanks!


change RTD links from astrodb scripts to astrodb utils

sent email!

Pretty sure this is done. https://astrodb-utils.readthedocs.io/en/latest/


WIP: DPL-249

Closes https://github.com/sanger/traction-ui/issues/699
Changes proposed in this pull request:

The ability to enter a Cost Code on importing tubes and plate from Sequencescape
If no cost code is provided, the default ToL cost code is used

5th April: Emailed Craig explaining this story only covers the importing from Sequencescape, and another story will be added to handle the importing from Samples Extraction once requirements have been clarified.


A Dockerfile for sql-client image so that the Flink version can be chosen

What is currently missing?

In the example, there's 3 containers for jobmanager, taskmanager and sql-client. While I can set jobmanager and taskmanager Flink's version manually, I can't do the same with sql-client image ftisiot/flink_sql_cli:1.13.2. That's because ftisiot/flink_sql_cli:1.13.1 image simply does not exist.
How could this be improved?

Providing a Dockerfile for ftisiot/flink_sql_cli:1.13.2 would be very useful. It will enable the opportunity to set base image or JAR links to Maven repos manually if needed.

Sorry, I must be blind. There's a directory 'sql-client' that contains exactly what I wanted. Idk how I overlooked it :(


feat(motion_velocity_smoother): add guard before interpolation

Description
Add guards before the linear interpolation.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

Merge


fix compiler warning: comparison of integer expressions of different … (IDFGH-3767)

Fix compiler warnings in 'components/soc/esp32/include/soc/cpu.h'

unsigned and signed comparison

 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.SiLeader seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for your contribution.

Hi @SiLeader,
Thanks for sending this patch. The same warning was reported in #6120 and a fix merged to that branch in ec2817a28553fd13421f045782b26b5f54cce94a for v4.1 (not published to GitHub yet), but we missed that the same fix was needed in release/v4.0 branch. Will apply this patch ASAP.

Hi @SiLeader,
Thanks for sending this patch. The same warning was reported in #6120 and a fix merged to that branch in ec2817a28553fd13421f045782b26b5f54cce94a for v4.1 (not published to GitHub yet), but we missed that the same fix was needed in release/v4.0 branch. Will apply this patch ASAP.

Changes merged with https://github.com/espressif/esp-idf/commit/d7c8dd270f4dff3774730dad0b6df8e4ad32ffe5, thanks for your contribution again.


Add export for props interface

Adds export for props interface so there will be much easier to add types for custom conponents based on framework7 react components.
Full description here - https://github.com/framework7io/framework7/issues/4024

Merged, thank you!


Rails error when opening a post with no user information

Clicking the single result here results in a Rails error:

ActionController::UrlGenerationError in Posts#show
Showing /var/railsapps/metasmoke/releases/20170320150615/app/views/posts/show.html.erb where line #65 raised:
No route matches {:action=>"show", :controller=>"stack_exchange_users", :id=>nil}, missing required keys: [:id]

c3dc554ac5d86626f330ac58e9e2fa1f4ce54981


Travis CI: Fix Ruby/Rubygem version SNAFU.

Pin Ruby to version 2.3.8 and Rubygem to 2.7.10. The Rubygem version provided by TravisCI failed to install bundles.
I have literally zero experience with Ruby and have no idea what I'm doing.

Out of date. We are know using a different version.


Trouble installing package

So I'm trying to import it but with no success:

the go.mod:

if I run go mod tidy:
go: finding module for package github.com/chunqian/go-raylib
github.com/jeedee/kobu-server imports
        github.com/chunqian/go-raylib: module github.com/chunqian/go-raylib@latest found (v0.0.0-20210519095144-df36dae46c76), but does not contain package github.com/chunqian/go-raylib

@nonunknown
import
import (
    rl "github.com/chunqian/go-raylib/raylib"
)

not
import (
    rl "github.com/chunqian/go-raylib"
)

ok I'll try and tell you

yeah it worked, but it has another problem: #16


Update README examples to use Postman-echo api

Running express is a bit heavy to test this library.  Postman-echo has a great set of api's that can be used.
https://docs.postman-echo.com

nice idea, coming soon 👍

Done!


Verbiage updates to the front page README

I'd like to propose these edits to the English language README.adoc

@cslauritsen Thank you so much for the PR. Much appreciated!

My pleasure!


Partial Tailwind styles overhaul; replace interactive Spline model with static image; simplify responsive layout

Lots of changes in this PR. The idea is to compromise on some features and preferences in favor of a reliable responsive layout and core functionality. Layout is a minimal Flex layout with minimal breakpoint rules. The interactive 3D Spline model has been temporarily replaced with a static image. Some files have been renamed in an effort to take a step toward better file/folder structure.

Fixes #398
Type of change

[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 not work as expected)
[ ] 📝 Documentation update
[ ] Other 🗒 **__**

How Has This Been Tested?

I'd like as many eyes on this as possible. Please pull the changes from feature/styling and run locally. Look for any issues cropping up on uncommon screen sizes, etc.
Checklist:

[x] I have performed a self-review of my own code.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[ ] Any dependent changes have been merged and published in downstream modules.
[ ] I have reviewed the Security Best Practices Document.
[ ] I have followed the Naming Conventions Guide for my changes.
[x] I have ensured that my pull request title is descriptive.

Additional Information:

Can comment out the first e2e test to avoid test failure for now since we have commented out the header from the Root Layout.


Gopacket fails to properly decode SNAP layers

Gopacket fails to decode SNAP layers when using a parser structured like the one in parser.go  It tries to parse two LLC layers in a row and then tries to parse an unknown layer type, failing.
The reason that this happens is that the CanDecode() method of the SNAP struct in llc.go mistakenly returns LayerTypeLLC.  It should be returning LayerTypeSNAP- almost all of the CanDecode() methods I've found return their own layer type (e.g. LLC.CanDecode() returns LayerTypeLLC, Dot11MgmtProbeReq.CanDecode() returns LayerTypeDot11MgmtProbeReq, etc.)  As-is, gopacket treats the bytes following the LLC layer as a second LLC layer.  When the LLC.NextLayerType() (llc.go lines 61-69) function runs, it doesn't find a correct value:
func (l *LLC) NextLayerType() gopacket.LayerType {
	switch {
	case l.DSAP == 0xAA && l.SSAP == 0xAA:
		return LayerTypeSNAP
	case l.DSAP == 0x42 && l.SSAP == 0x42:
		return LayerTypeSTP
	}
	return gopacket.LayerTypeZero // Not implemented
}

Since gopacket is now trying to treat a section of bytes as an LLC when they aren't in fact an LLC, l.DSAP and l.SSAP refer to arbitrary bytes, which won't have one of those special values mentioned.  NextLayerType() then returns LayerTypeZero, which errors as an unknown layer type, replicating the above symptoms.
The incorrect code, on lines 97-100 of llc.go:
// CanDecode returns the set of layer types that this DecodingLayer can decode.
func (s *SNAP) CanDecode() gopacket.LayerClass {
	return LayerTypeLLC
}

What it should be:
// CanDecode returns the set of layer types that this DecodingLayer can decode.
func (s *SNAP) CanDecode() gopacket.LayerClass {
	return LayerTypeSNAP
}

Whops that was a copy & paste error on my side.
Can you move this from #542 into it's own pull request, so we can merge it?

@notti yeah I'll do so soon.  The problem is that my master branch on my fork from gopacket is pretty dirty from the changes I've made in #542, and I can't figure out how to get a clean fork from the current HEAD that gopacket is at.  Do you have any suggestions?

Move your changes to a branch (it's always easier in github to work on branches and make a pull request from there).
Then you can also rebase your branch to enjoy the current master changes.

@notti will do, thanks.  I realized a while ago to always make changes on branches, but to be honest I was originally intending that fork to be very short so I got sloppy.  Whoops.
If I already committed some changes to master, is there any way to go back and peel them off into a branch of their own?  I really appreciate your help on this, by the way- I'm much less skilled with Git than I'd like to be.

Hmm I think adding a new branch and then moving master back with git reset should work. Commits can be moved aroud /edited with interactive rebase (https://git-scm.com/book/en/v2/Git-Tools-Rewriting-History)
Although if you do this, your pull request (it was from master) will be affected (you would need to create a new one).
Maybe you could also rebase your master onto the current gopacket master, and move the one commit to it's own branch.


Nested Arrows clobber internal dependent variables

Recursive application of functions with dependent variables may clobber variable names before they should unify under proper order. This only affects recursive application of the same function candidates, i.e. "-(-(1))" so is only medium priority.
To reproduce the bug, uncomment the related code in tests/open_issues.rs and run "cargo test".

fixed by unify V2


Offshoot ideas

Random, potentially malformed ideas for add-ons.

--explain flag for errors
When a type error occurs, use the (currently non-existent) machinery to draw out an abstract design and show why the circuit diagram is invalid.
CC @sgpthomas

C2Fuse
Write a compiler that converts a random C/C++ program into a Fuse program. Having feature parity with Vivado HLS (i.e. C2Fuse fails iff HLS fails) would be really cool.

""Debug Symbols"" for EDA Failure Diagnosis
We'd like to be able to diagnose failures that happen later in the EDA toolchain—i.e., synthesis, technology mapping, and place-and-route—by exposing them in the source code. For example, if your design fails because the target FPGA doesn't have enough LUTs, we want to "blame" the consumption on some source-level construct in our language. This would require embedding (into the generated RTL) the analog of debug symbols in CPU object code to perform the reverse mapping to our source code.

Hardware Externs
Take hardware modules, either written directly in Verilog or in some other "generator" language like Chisel, and expose them as extern-like functions to Fuse programs. Seen this way, Fuse is a composition language for hand-coded RTL components. In the limit, we would replace built-in primitives like + with externs.

Fully serializable able IR/AST
Tools like Jason’s automatic loop optimiser might want to use fuse instead of vivado HLS to write transformations and verify that they don’t violate the safety guarantees that Fuse provides. Having a common representation that be used by external tools and can be easily parsed back for typechecking would be useful for quickly integrating them with Fuse.

Position Paper: Demonstrate common SW compiler optimizations don’t make sense for HW
Most HLS tool from the past decade rely on LLVM/GCC toolchain to get SW optimizations “for free”. However some optimizations like loop order changes directly affect constructs like unrolled loops, pipelines and reduction trees.
This would involve implementing SW optimization passes and systematically showing that they generate bad hardware.

HLS for Digital Signal Processing
According to this overview doc by texas instruments, DSPs have similar restrictions to FPGA programs (statically known loop bounds/estimates for instruction pipelining etc.)
Might be worth targeting DSPs in the future.

Bindings in other languages
From talking to people in Oculus it seems having a whole new language is a non starter for any kind of adoption. One way to sidestep is problem in having bindings in C++ and Python that generate a Fuse AST and pass it onto the compiler.
This seems to be the strategy taken by Halide (by extension, TVM), TACO, etc.

^^ It occurs to me that "c2fuse" (https://github.com/cucapra/seashell/issues/160#issuecomment-477346046) could be construed as a different solution to the same problem of basic language familiarity.


Shift creating shadow table

After completion of migration, the shift is creating a shadow table like below
+---------------------------+
| Tables_in_test            |
+---------------------------+
|<PHONE_NUMBER>5700887_sbtest1 |
|<PHONE_NUMBER>0253090_sbtest5 |
|<PHONE_NUMBER>0952087_sbtest5 |
|<PHONE_NUMBER>2052953_sbtest5 |
|<PHONE_NUMBER>2924812_sbtest5 |
|<PHONE_NUMBER>2924891_sbtest5 |
|<PHONE_NUMBER>2924892_sbtest5 |
|<PHONE_NUMBER>3204700_sbtest5 |
|<PHONE_NUMBER>3431865_sbtest5 |
|<PHONE_NUMBER>1503073_sbtest5 |
| sbtest1                   |
| sbtest2                   |
| sbtest3                   |
| sbtest4                   |
| sbtest5                   |
+---------------------------+```

Do we have any parameters to stop this ?

Currently, no, although I think this was an oversight because we definitely should.

This will be an overhead for big tables. We can skip the rename migration part and make it to drop the old table.
Also wanted to know, do we have any scheduled migration kind of process ?

Just to provide some background on why it is the way it is - we define the "pending_drops_db" configuration option on the runner to be a special db, and we have a cron job that drops tables that are older than 7 days from that db. I agree, though, that we should provide a way to just drop the table immediately after the migration finishes.
You cannot schedule things to run at a specific time in the future, if that's what you're asking.

@Dhanasekar93 @michaelfinch How about adding a parameter like enable_trash? I would like to create PR about it.

That sounds good, I'll definitely accept that PR. Please update the docs as well

Alternatively, if pending_drops_db is empty in the config, we could just make it automatically drop the old table.

@michaelfinch I have created a pull request about it~  #55

If enable_trash is false it is dropping the shadow table

Isn't that the behavior we want? From the README: enable_trash: boolean. if true, the runner move the original table into the pending_drops_db with a timestamp prefix name after non-shortrun alter/drop table.
So if enable_trash is false, it should drop the shadow table. If it is true, it should move the shadow table into the pending_drops db.


ARROW-3696: [C++] Add feather::TableWriter::Write(table)

Codecov Report

Merging #2898 into master will increase coverage by 1.09%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2898      +/-   ##
==========================================
+ Coverage   87.36%   88.46%   +1.09%
==========================================
  Files         417      348      -69
  Lines       65116    59638    -5478
==========================================
- Hits        56890    52757    -4133
+ Misses       8132     6881    -1251
+ Partials       94        0      -94

Impacted Files
Coverage Δ

cpp/src/arrow/ipc/feather.cc
93.68% <100%> (+0.16%)
:arrow_up:

cpp/src/arrow/ipc/feather-test.cc
96.41% <100%> (+2.16%)
:arrow_up:

rust/src/record_batch.rs

go/arrow/array/table.go

rust/src/array.rs

go/arrow/math/uint64_amd64.go

go/arrow/internal/testing/tools/bool.go

go/arrow/array/bufferbuilder.go

go/arrow/internal/bitutil/bitutil.go

... and 62 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 010fad2...21699b5. Read the comment docs.


Allow sorting on related attributes of has_one relationships.

Current master has a bug when sorting on related attributes, if the relationship is a has_one/foreign_key_on: :related. The following code will produce this error
Mysql2::Error: Unknown column 'containers.container_id' in 'on clause': SELECT  `containers`.* FROM `containers` LEFT JOIN sensors AS sensor_sorting ON sensor_sorting.id = containers.container_id

class Container < ApplicationRecord
  has_one :inventory_sensor, required: false
end

class Sensor < ApplicationRecord
  belongs_to :container, required: false
end

class ContainerResource < BaseResource
  has_one :sensor, foreign_key_on: :related

  def self.sortable_fields(context)
    super(context) + [:"sensor.reading"]
  end
end

_build_joins() was looking at the wrong table for the related foreign_key. The join needs to be built differently depending on where the foreign key will live. The current tests don't catch this because they seem to all use a self-referential has_one relationship, so the foreign_key is there.
So I just added a conditional that alters the query based on whether or not it is a has_one relationship, and updated the test. I have integration testing around this in my project, but I was having difficulty dealing with the jsonapi-resources test suite, so I just fixed the one test I broke, but it would be great to have some integration testing around this.
Feel free to make any necessary adjustments or provide me with any feedback.
Thanks for the awesome gem.

@Ross-Hunter Thanks! Merged via #1106


Added p’th Smallest/Largest Element in Unsorted Array in cpp

Related Issues

Added p'th Smallest/Largest Element in Unsorted Array in cpp

Closes: #[issue number that will be closed through this PR]
Describe the changes you've made
Given an array and a number p where p is smaller than size of array, we need to find the p’th smallest element in the given array.
It is given that all array elements are distinct. I have added the solution in c++.
Type of change

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

How Has This Been Tested?
Yes.
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Checklist:

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

Screenshots

Original
Updated

original screenshot
updated screenshot

I have contributed to issue #265. Please take a look.


Full Simple Example in NextJS 13 throws an error

mantine-react-table version
6.05
react & react-dom versions
v18
Describe the bug and the steps to reproduce it
Trying to learn the project using Mantine with NextJS 13 using the app directory structure ( I understand this is not fully supported yet, but it works without react-table): https://www.mantine-react-table.com/docs/getting-started/usage#full-simple-example
Minimal, Reproducible Example - (Optional, but Recommended)
My package.json
{
  "name": "ia-mantine",
  "version": "0.1.0",
  "private": true,
  "scripts": {
    "dev": "next dev",
    "build": "next build",
    "start": "next start",
    "lint": "next lint"
  },
  "dependencies": {
    "@emotion/react": "^11.10.6",
    "@emotion/server": "^11.10.0",
    "@mantine/core": "^6.0.5",
    "@mantine/hooks": "^6.0.5",
    "@mantine/next": "^6.0.5",
    "@types/node": "18.15.11",
    "@types/react": "18.0.31",
    "@types/react-dom": "18.0.11",
    "eslint": "8.37.0",
    "eslint-config-next": "13.2.4",
    "mantine-react-table": "^1.0.0-beta.0",
    "next": "13.2.4",
    "react": "18.2.0",
    "react-dom": "18.2.0",
    "typescript": "5.0.2"
  }
}

my test-component.tsx:
'use client';

import React, { useMemo } from 'react';
import { MantineReactTable } from 'mantine-react-table';
import type { MRT_ColumnDef } from 'mantine-react-table'; // If using TypeScript (optional, but recommended)

//If using TypeScript, define the shape of your data (optional, but recommended)
interface Person {
  name: string;
  age: number;
}

//mock data - strongly typed if you are using TypeScript (optional, but recommended)
const data: Person[] = [
  {
    name: 'John',
    age: 30,
  },
  {
    name: 'Sara',
    age: 25,
  },
];

export function Test() {
  //column definitions - strongly typed if you are using TypeScript (optional, but recommended)
  const columns = useMemo<MRT_ColumnDef<Person>[]>(
    () => [
      {
        accessorKey: 'name', //simple recommended way to define a column
        header: 'Name',
        mantineTableHeadCellProps: { sx: { color: 'green' } }, //custom props
      },
      {
        accessorFn: (originalRow) => originalRow.age, //alternate way
        id: 'age', //id required if you use accessorFn instead of accessorKey
        header: 'Age',
        Header: <i style={{ color: 'red' }}>Age</i>, //optional custom markup
      },
    ],
    [],
  );

  return (
    <MantineReactTable
      columns={columns}
      data={data}
      enableRowSelection //enable some features
      enableColumnOrdering
      enableGlobalFilter={false} //turn off a feature
    />
  );
}

my page.tsx:
import { Test } from './test-component';

export default function Page() {
  return <Test />;
}

my layout.tsx:
import RootStyleRegistry from './emotion';

export default function RootLayout({ children }: { children: React.ReactNode }) {
  return (
    <html lang="en-US">
      <head />
      <body>
        <RootStyleRegistry>{children}</RootStyleRegistry>
      </body>
    </html>
  );
}

emotion.tsx:
'use client';

import { CacheProvider } from '@emotion/react';
import { useEmotionCache, MantineProvider } from '@mantine/core';
import { useServerInsertedHTML } from 'next/navigation';

export default function RootStyleRegistry({ children }: { children: React.ReactNode }) {
  const cache = useEmotionCache();
  cache.compat = true;

  useServerInsertedHTML(() => (
    <style
      data-emotion={`${cache.key} ${Object.keys(cache.inserted).join(' ')}`}
      dangerouslySetInnerHTML={{
        __html: Object.values(cache.inserted).join(' '),
      }}
    />
  ));

  return (
    <CacheProvider value={cache}>
      <MantineProvider withGlobalStyles withNormalizeCSS>
        {children}
      </MantineProvider>
    </CacheProvider>
  );
}

error:
Warning: Can't perform a React state update on a component that hasn't mounted yet. This indicates that you have a side-effect in your render function that asynchronously later calls tries to update the component. Move this work to useEffect instead.
MRT_TableRoot@webpack-internal:///(app-client)/./node_modules/mantine-react-table/dist/esm/mantine-react-table.esm.js:2685:75
MantineReactTable@webpack-internal:///(app-client)/./node_modules/mantine-react-table/dist/esm/mantine-react-table.esm.js:2954:1249
Test@webpack-internal:///(app-client)/./src/app/test-component.tsx:27:67
ScrollAndFocusHandler@webpack-internal:///(app-client)/./node_modules/next/dist/client/components/layout-router.js:153:1
InnerLayoutRouter@webpack-internal:///(app-client)/./node_modules/next/dist/client/components/layout-router.js:196:44
RedirectErrorBoundary@webpack-internal:///(app-client)/./node_modules/next/dist/client/components/layout-router.js:361:9
RedirectBoundary@webpack-internal:///(app-client)/./node_modules/next/dist/client/components/layout-router.js:368:25
NotFoundErrorBoundary@webpack-internal:///(app-client)/./node_modules/next/dist/client/components/layout-router.js:397:9
NotFoundBoundary@webpack-internal:///(app-client)/./node_modules/next/dist/client/components/layout-router.js:404:53
LoadingBoundary@webpack-internal:///(app-client)/./node_modules/next/dist/client/components/layout-router.js:317:64
ErrorBoundary@webpack-internal:///(app-client)/./node_modules/next/dist/client/components/error-boundary.js:72:56
RenderFromTemplateContext@webpack-internal:///(app-client)/./node_modules/next/dist/client/components/render-from-template-context.js:12:34
OuterLayoutRouter@webpack-internal:///(app-client)/./node_modules/next/dist/client/components/layout-router.js:23:177
ThemeProvider@webpack-internal:///(app-client)/./node_modules/@emotion/react/dist/emotion-element-6a883da9.browser.esm.js:96:64
MantineProvider@webpack-internal:///(app-client)/./node_modules/@mantine/styles/esm/theme/MantineProvider.js:78:25
RootStyleRegistry@webpack-internal:///(app-client)/./src/app/emotion.tsx:17:25
body
html
ReactDevOverlay@webpack-internal:///(app-client)/./node_modules/next/dist/client/components/react-dev-overlay/internal/ReactDevOverlay.js:61:9
HotReload@webpack-internal:///(app-client)/./node_modules/next/dist/client/components/react-dev-overlay/hot-reloader-client.js:20:39
Router@webpack-internal:///(app-client)/./node_modules/next/dist/client/components/app-router.js:48:89
ErrorBoundaryHandler@webpack-internal:///(app-client)/./node_modules/next/dist/client/components/error-boundary.js:59:9
ErrorBoundary@webpack-internal:///(app-client)/./node_modules/next/dist/client/components/error-boundary.js:72:56
AppRouter@webpack-internal:///(app-client)/./node_modules/next/dist/client/components/app-router.js:24:39
ServerRoot@webpack-internal:///(app-client)/./node_modules/next/dist/client/app-index.js:147:25
RSCComponent
Root@webpack-internal:///(app-client)/./node_modules/next/dist/client/app-index.js:164:25 [hydration-error-info.js:27:13](webpack://_N_E/node_modules/next/dist/client/components/react-dev-overlay/internal/helpers/hydration-error-info.js?32aa)
    error hydration-error-info.js:27
    printWarning react-dom.development.js:94
    error react-dom.development.js:68
    warnAboutUpdateOnNotYetMountedFiberInDEV react-dom.development.js:32016
    detectUpdateOnUnmountedFiber react-dom.development.js:14949
    getRootForUpdatedFiber react-dom.development.js:14936
    enqueueConcurrentHookUpdate react-dom.development.js:14814
    dispatchSetState react-dom.development.js:18894
    onStateChange index.mjs:63
    setState index.mjs:2665
    makeStateUpdater index.mjs:25
    setPagination index.mjs:1541
    setPageIndex index.mjs:1548
    resetPageIndex index.mjs:1560
    _autoResetPageIndex index.mjs:1531
    _queue index.mjs:2646
    (Async: promise callback)
    _queue index.mjs:2644
    _autoResetPageIndex index.mjs:1530
    onChange index.mjs:3213
    memo index.mjs:65
    getFilteredRowModel index.mjs:1191
    getPreGroupedRowModel index.mjs:1393
    getGroupedRowModel index.mjs:3394
    memo index.mjs:56
    getGroupedRowModel index.mjs:1401
    getPreSortedRowModel index.mjs:2479
    getSortedRowModel index.mjs:3309
    memo index.mjs:56
    getSortedRowModel index.mjs:2487
    getPreExpandedRowModel index.mjs:859
    getExpandedRowModel index.mjs:3536
    memo index.mjs:56
    getExpandedRowModel index.mjs:867
    getPrePaginationRowModel index.mjs:1624
    getPaginationRowModel index.mjs:3571
    memo index.mjs:56
    getPaginationRowModel index.mjs:1632
    getRowModel index.mjs:2681
    MRT_Table mantine-react-table.esm.js:2446
    renderWithHooks react-dom.development.js:17498
    mountIndeterminateComponent react-dom.development.js:22553
    beginWork$1 react-dom.development.js:24128
    beginWork react-dom.development.js:32040
    performUnitOfWork react-dom.development.js:30938
    workLoopConcurrent react-dom.development.js:30924
    renderRootConcurrent react-dom.development.js:30880
    performConcurrentWorkOnRoot react-dom.development.js:29836
    workLoop index.js:10
    flushWork index.js:10
    performWorkUntilDeadline index.js:10
    (Async: EventHandlerNonNull)
    853 index.js:10
    853 index.js:10
    __nccwpck_require__ index.js:19
    833 index.js:19
    __nccwpck_require__ index.js:19
    <anonymous> index.js:19
    <anonymous> index.js:19
    NextJS 4
    <anonymous> react-dom.development.js:27
    <anonymous> react-dom.development.js:34563
    NextJS 4
    <anonymous> index.js:37
    NextJS 4
    <anonymous> client.js:3
    NextJS 4
    <anonymous> app-index.js:10
    NextJS 4
    <anonymous> app-next-dev.js:5
    appBootstrap app-bootstrap.js:52
    loadScriptsInSequence app-bootstrap.js:18
    appBootstrap app-bootstrap.js:51
    <anonymous> app-next-dev.js:4
    NextJS 7

Screenshots or Videos (Optional)
No response
Do you intend to try to help solve this bug with your own PR?
None
Terms

[X] I understand that if my bug cannot be reliably reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

Interesting. My docs site itself is in Next 13 and I have not seen this. I'll look into it a bit more

Wait this is just a warning

yes, sorry, it's a warning.

There is a solution discussed in the MUI MRT project https://github.com/KevinVandy/material-react-table/issues/200


fix debug-panel z-index

how
debugPanel 用的 Popover 默认 z-index 是 1030，Modal 默认 z-index 是 1000，因此改为 1040
before

after

 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.


Changing fiscall address message and help

Changed fiscal address title to be clearer
Added help on admin address page

Plase @tonipinel take a look at the markup. I added the help as an info message but you surely can improve this ;) Remove the WiP when this is finished.

For me it's ok! Really a good job, the only thing is the text but we need to check all so... Merge as you want!

Harmless changes, I'll merge this tomorrow if no issue arises.


raidboss: Another Aloalo Island Normal & Savage

Sorry no Oopsy 😭
English only but, able to play on Japanese client
SImple triggers + timeline

* Ketuduke Fluke Gale 2: incorrect call for location ("Go to 1/2 area")

I make option for strat selection. But I had no problems 😫

* Statice Pinwheeling Dartboard 2: incorrect call for color tile to stand on/tile swaps (it appears that the `Bull's-eye 2` trigger isn't firing correctly and it is using the `Bull's-eye 1` trigger instead)

Also I had no problems. Please let me know the strat you played. It seems to be tested in diffent strat. I think.

Some other suggestions:

I'll fix it as long as I can 😭
Thank you for testing!

It wasn't a strat difference issue, the triggers weren't firing properly in-game or when tested in the emulator.  The Fluke Gale trigger appeared to be using stale data when the mechanic repeated (it was correct on the first use), and the Bull's Eye 2 trigger wasn't firing at all.

There's some good work here, I wouldn't abandon it completely.  At the very least, parts of it could be merged with what quisquous is working on in #5889 once they're done with their blind prog.  It's just a bit awkward to collaborate while the project owner is avoiding spoilers.

Yeah, I know it's incredibly awkward, thank you for your patience.
If xiashtra has read this through entirely and thinks this looks good and could be merged, I'd be happy to merge it without looking at it if you (either of you, both of you) would be willing to fix any bugs or complaints if they come up before I'm done with my very slow to schedule blind prog.


python310Packages.pyezviz: <IP_ADDRESS> -> <IP_ADDRESS>

Description of changes
https://github.com/BaQs/pyEzviz/releases/tag/<IP_ADDRESS>

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 170691 run on x86_64-linux 1

3 packages built:

home-assistant-component-tests.ezviz
python310Packages.pyezviz
python39Packages.pyezviz

Result of nixpkgs-review pr 170691 run on x86_64-linux 1

3 packages built:

home-assistant-component-tests.ezviz
python310Packages.pyezviz
python39Packages.pyezviz


x64 for Linux

Hello!
I was just wondering if the build for x64 for Linux could be tested, as we don't have a Linux device to use atm.
And also were there any updates on build for Arm for Linux
Thanks!

Attempted it through WSL but seemed to get a default response (expecting an int and string but I get back 0 and empty string). Not sure if this is because I'm running through WSL or not, wondering if anyone else had a similar issue

False alarm sorry, had a git confusion resolved and it seems to work!
Closing issue and thanks for the great package!


Add import/export feature

With this patch, it is possible to export and import a Khepri store.
khepri:export/{2,3,4} uses a callback module and private data to export the content of the store. It is possible to specify a path pattern to filter what should be exported.
khepri:import/{2,3} uses a callback module and private data — of course the same used for export — to import what was previously exported. This function doesn't remove existing tree nodes. It is up to the caller to take care of this. There is no support for filtering on import.
The callback module API is the same as Mnesia's "Backup and restore" API. The callback module itself doesn't know what is inside the list of "backup items" passed to it, it's completely opaque.
The implementation uses khepri_machine state machine commands as the export format: each tree node matching the export path pattern is converted to a put command. Tree nodes which don't have any payload or "keep while" conditions are skipped (i.e. not exported).
During import, the commands are processed one by one. In a future patch, we could process them in batches to speed things up.
Fixes #153.

This work in unfinished at this stage:

It lacks type spec and Dialyzer is probably unhappy for now.
It lacks documentation.
Perhaps we want to support batch processing in this pull request as well.

Perhaps we want to support batch processing in this pull request as well

I'm not sure what batch processing will look like. This PR looks well-contained though with just the import/export changes so I think this should be split to a separate PR unless its implementation and import/export are very similar or inform each other.

I think this API looks good +1
I was thinking that the khepri_export_erlang importer/exporter might need to use something like term_to_binary/1/binary_to_term/1 for encoding/decoding because of functions in stored procedure but pretty-printing ends up working well because of the functions are just records with the beam code. Could we add a stored procedure case to test/export.erl so this is tested too?

Thank you! Yes, I will add testcases with extracted functions. Some of them could be actual function references. We should probably extract it at the time of export.

Perhaps we want to support batch processing in this pull request as well

I'm not sure what batch processing will look like. This PR looks well-contained though with just the import/export changes so I think this should be split to a separate PR unless its implementation and import/export are very similar or inform each other.

You are right, let's prepare another patch for batching. Import/export can take advantage of it but doesn't require it.