ARROW-3847: [GLib] Remove unnecessary ''

Codecov Report

Merging #3007 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3007      +/-   ##
==========================================
- Coverage   86.65%   86.64%   -0.02%
==========================================
  Files         491      491
  Lines       69513    69513
==========================================
- Hits        60239    60229      -10
- Misses       9182     9188       +6
- Partials       92       96       +4

Impacted Files
Coverage Δ

go/arrow/math/int64_avx2_amd64.go
0% <0%> (-100%)
:arrow_down:

go/arrow/memory/memory_avx2_amd64.go
0% <0%> (-100%)
:arrow_down:

go/arrow/math/float64_avx2_amd64.go
0% <0%> (-100%)
:arrow_down:

go/arrow/math/uint64_avx2_amd64.go
0% <0%> (-100%)
:arrow_down:

go/arrow/memory/memory_amd64.go
28.57% <0%> (-14.29%)
:arrow_down:

go/arrow/math/math_amd64.go
31.57% <0%> (-5.27%)
:arrow_down:

go/arrow/math/float64_amd64.go
33.33% <0%> (ø)
:arrow_up:

go/arrow/math/int64_amd64.go
33.33% <0%> (ø)
:arrow_up:

go/arrow/math/uint64_amd64.go
33.33% <0%> (ø)
:arrow_up:

go/arrow/math/uint64_sse4_amd64.go
100% <0%> (+100%)
:arrow_up:

... and 3 more

Continue to review full report at Codecov.

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


Reduced cypress config numTestsKeptInMemory

Reduced cypress config numTestsKeptInMemory to 1 as an attempt to fix stability issues on CI

Since we've reverted Cypress, it looks like our stability issues are fixed, and this PR is not necessary anymore

Since we've reverted Cypress, it looks like our stability issues are fixed, and this PR is not necessary anymore


wip: per environment acme issuer

WIP: this updates the current implementation which creates a single Issuer for the environment (.Values.jxRequirements.ingress) to configuring an Issuer per environment (.Values.jxRequirements.environments).
Self-signed: if and .ingress.tls.enabled (not .ingress.tls.production) (not .ingress.domain)
LE Staging: if and .ingress.tls.enabled (.not .ingress.tls.production) .ingress.domain
LE Production: if and .ingress.tls.enabled .ingress.tls.production .ingress.domain
Note: still have to add a Certificate for the self-signed issuer and test it all out.

Closing as needs review.


Enable use as a library

It would be great to use rsync server as a library in other Go projects.

Is this done with https://github.com/gokrazy/rsync/pull/11, or is there anything else you’re planning on doing before this issue should be closed? :)

Great question!
How about docs, should we just link to the PR for the code example? I have nothing else in mind. (That reminds me, the code example is outdated w.r.t. the module slice. I'll update it when I get back home.)

Or maybe we add a minimal server example? Should only be a few lines of code :)

In documentation or as an example binary under cmd/?

Ideally as a testable example that shows up in godoc and is verified to compile using go test: https://go.dev/blog/examples :)

lol thank you, once again I'm learning from you, I've spat out Go code for years but never learned about "testable examples" :smile:


Local CUDA install directions with pip missing alternative to symlink

Description
This is just a documentation error/typo, so apologies in advance if I should report this elsewhere.
https://github.com/google/jax/blob/173a2701794571f3a102b0dd80bdedb4adbd8a5c/docs/installation.md?plain=1#L155-L161
The "either" here seems misplaced, or the doc is missing an alternative to creating a symbolic link. If there is a recommended alternative, I would love to hear it, since I cannot create a symbolic link there on the shared cluster I am building JAX on. We dynamically swap CUDA Toolkit versions based on the loaded Environment Modules on our machine.
What jax/jaxlib version are you using?
0.4.16
Which accelerator(s) are you using?
GPU
Additional system info
SUSE Linux Enterprise Server 15 SP3
NVIDIA GPU info
❯ nvidia-smi
Thu Sep 28 16:33:56 2023
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 470.103.04   Driver Version: 470.103.04   CUDA Version: 11.4     |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|                               |                      |               MIG M. |
|===============================+======================+======================|
|   0  NVIDIA A100 80G...  On   | 00000000:A3:00.0 Off |                   On |
| N/A   35C    P0    46W / 300W |      0MiB / 80994MiB |     N/A      Default |
|                               |                      |              Enabled |
+-------------------------------+----------------------+----------------------+

+-----------------------------------------------------------------------------+
| MIG devices:                                                                |
+------------------+----------------------+-----------+-----------------------+
| GPU  GI  CI  MIG |         Memory-Usage |        Vol|         Shared        |
|      ID  ID  Dev |           BAR1-Usage | SM     Unc| CE  ENC  DEC  OFA  JPG|
|                  |                      |        ECC|                       |
|==================+======================+===========+=======================|
|  No MIG devices found                                                       |
+-----------------------------------------------------------------------------+

+-----------------------------------------------------------------------------+
| Processes:                                                                  |
|  GPU   GI   CI        PID   Type   Process name                  GPU Memory |
|        ID   ID                                                   Usage      |
|=============================================================================|
|  No running processes found                                                 |
+-----------------------------------------------------------------------------+

I strongly recommend using the pip installation of CUDA, as we recommend in our installation docs.
It's 1000x easier to install that way (https://github.com/google/jax#installation).
If you are determined to use a local installation, then I think you just need to set
LD_LIBRARY_PATH to point to your cuda libraries (typically /usr/local/cuda/lib64, although you might need some additional directories for libcupti.so) and PATH to include your CUDA binaries (typically /usr/local/cuda/bin).

Indeed we set LD_LIBRARY_PATH accordingly, and it seems to be working. Should I add your note to the docs in a PR?

I updated the docs to just  mention LD_LIBRARY_PATH and PATH.


Removing/adding variables via In-client page designer

On the 'Cash Receipt Journal' (page 255) I wanted to remove the 'Account Name' and 'Bal. Account Name' field from the footer group and see if I could include them as columns (per line) instead. During the design (using the in-client page designer), I've removed the fields (~ variables) from the footer group, but afterwards I was no longer able to restore / add the deleted fields (variables: AccName, BalAccName) again.

A similar issue appears on the 'Sales Invoice Subform' when working with the Shortcut Dimension columns. Columns based on variables seem not to appear in the list of fields (I'm looking for 'Shortcut Dimension 7' that I first removed and tried to add again). Neither on the 'Dimension Name' , nor on the 'Shortcut Dimension' value.

We will look into it.


Handling of packs (Filter, Repositories, Local packs, etc.)

Introduce a section packs: in *.csolution.yml and *.cdefault.yml to:

reduce the scope of packs used in a project
specify pack versions
specify local paths to packs

packs:                                  # start section that specifics software packs
  - filter: AWS::                       # use packs from AWS
  - filter: NXP::*K32L*                 # use packs from NXP that relate to K32L series (would match K32L3A60_DFP + FRDM-K32L3A6_BSP)
  - filter: ARM::                       # use packs from Arm

  - pack: Keil::Arm_Compiler            # add always Keil::Arm_Compiler pack (bypasses filter)
  - pack<EMAIL_ADDRESS> # add Keil::MDK-Middleware pack at version 7.13.0
  - pack: NXP::K32L3A60_DFP             # add pack
    path: .\local\NXP\K32L3A60_DFP      # path to the pack (local copy, repo, etc.)

Thanks for the proposal
One question:  which entity is expected to copy the PACK to the specified path ?
Basically, if a csolution (and related project files) is shared from a user to another, can he expect that the file system project will be re-created in the same way ?

One question: which entity is expected to copy the PACK to the specified path ?
In the current specification the path: has no restrictions; if it is relative to the workspace, I assume it would be part of a repo; if it is an absolute path to some random directory, I would assume that the organization that uses it has some guidelines when such projects are shared.

Sounds good to have this flexibility of specifying a path to where a pack is installed.
Would be good to have this on the .cproject.yml level instead/as well.

Agreed.  Let's see if it is possible/easy to implement.  Might be challenging as you may change the packs based on what project is processed.

closing as I believe this is outdated.


rename variables starting with No to Exclude

-NoSystemDbs -> -ExcludeSystemDbs

will be hell D:

again, tackling this via code reviews. Don't think this should remain opened, but happy to leave it if @potatoqualitee or @wsmelton ping me here :P

It was primarily for the commands that had not been updated...fairly sure we got them all but not 100%.

part of ongoing effort to standardize things. Here too, given the sheer number of functions, either we make a whitelist and check in CI or we'll die trying.


Crash in transaction abort state

Just ran into this:
postgres=# INSERT INTO coordinates VALUES (1, 2);
ERROR:  paxos commands cannot run inside a transaction block
STATEMENT:  INSERT INTO coordinates VALUES (1, 2);
ERROR:  paxos commands cannot run inside a transaction block
postgres=# ROLLBACK;
TRAP: FailedAssertion("!(IsTransactionState())", File: "relcache.c", Line: 1737)
LOG:  server process (PID 11317) was terminated by signal 6: Aborted
DETAIL:  Failed process was running: ROLLBACK;
server closed the connection unexpectedly
	This probably means the server terminated abnormally
	before or while processing the request.

Leaves the following back-trace:
#0  0x0000003969632625 in raise (sig=6) at ../nptl/sysdeps/unix/sysv/linux/raise.c:64
#1  0x0000003969633e05 in abort () at abort.c:92
#2  0x000000000092d47e in ExceptionalCondition (conditionName=0xb31d96 "!(IsTransactionState())", errorType=0xb319e0 "FailedAssertion",
    fileName=0xb31944 "relcache.c", lineNumber=1737) at assert.c:54
#3  0x000000000091ae1e in RelationIdGetRelation (relationId=3079) at relcache.c:1737
#4  0x00000000004b678c in relation_open (relationId=3079, lockmode=1) at heapam.c:1063
#5  0x00000000004b6af8 in heap_open (relationId=3079, lockmode=1) at heapam.c:1238
#6  0x0000000000603c2d in get_extension_oid (extname=0x7fe999001b21 "pg_paxos", missing_ok=1 '\001') at extension.c:127
#7  0x00007fe998fec475 in IsPgPaxosActive () at src/pg_paxos.c:268
#8  0x00007fe998fed193 in PgPaxosProcessUtility (parsetree=0x23c2b80, queryString=0x23c2198 "ROLLBACK;", context=PROCESS_UTILITY_TOPLEVEL, params=0x0,
    dest=0x23c2f08, completionTag=0x7fff5a34d430 "") at src/pg_paxos.c:780
#9  0x00000000007ef9a2 in ProcessUtility (parsetree=0x23c2b80, queryString=0x23c2198 "ROLLBACK;", context=PROCESS_UTILITY_TOPLEVEL, params=0x0, dest=0x23c2f08,
    completionTag=0x7fff5a34d430 "") at utility.c:330
#10 0x00000000007eea8e in PortalRunUtility (portal=0x23f4998, utilityStmt=0x23c2b80, isTopLevel=1 '\001', dest=0x23c2f08, completionTag=0x7fff5a34d430 "")
    at pquery.c:1183
#11 0x00000000007eec6a in PortalRunMulti (portal=0x23f4998, isTopLevel=1 '\001', dest=0x23c2f08, altdest=0x23c2f08, completionTag=0x7fff5a34d430 "")
    at pquery.c:1314
#12 0x00000000007ee1da in PortalRun (portal=0x23f4998, count=9223372036854775807, isTopLevel=1 '\001', dest=0x23c2f08, altdest=0x23c2f08,
    completionTag=0x7fff5a34d430 "") at pquery.c:812
#13 0x00000000007e83a4 in exec_simple_query (query_string=0x23c2198 "ROLLBACK;") at postgres.c:1104
#14 0x00000000007ec436 in PostgresMain (argc=1, argv=0x2352580, dbname=0x23523e0 "postgres", username=0x23523c0 "amit") at postgres.c:4030
#15 0x000000000076ac8c in BackendRun (port=0x2371c90) at postmaster.c:4239
#16 0x000000000076a3f7 in BackendStartup (port=0x2371c90) at postmaster.c:3913
#17 0x0000000000766ba8 in ServerLoop () at postmaster.c:1684
#18 0x000000000076626c in PostmasterMain (argc=3, argv=0x2351590) at postmaster.c:1292
#19 0x00000000006be4c6 in main (argc=3, argv=0x2351590) at main.c:228

Thanks for reporting! This is now fixed in the develop branch.
https://github.com/citusdata/pg_paxos/tree/develop

Great, thanks!


Update PPP for John

Pull Request Test Coverage Report for Build 611

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

Totals

Change from base Build 608:
0.0%

Covered Lines:
2219

Relevant Lines:
2766

💛  - Coveralls


is there a way to impliment full screen?

I don't think I need to say much.
I would like to be able to maximise a tab and have it fill the whole screen, as I am making a web browser.
thanks.

wrong repository sorry


Uncaught Error: Can't save destroyed buffer

Press cmd+alt+e on a function
Go to matching files
Open a file via the brackets icon in the code-peek panel
Close this file via tab bar
Click save in code-peek panel

Atom: 1.17.0-beta5 x64
Electron: 1.3.15
OS: Mac OS X 10.12.4
Thrown From: code-peek package 1.4.18
Stack Trace
Uncaught Error: Can't save destroyed buffer
At /Applications/Atom Beta.app/Contents/Resources/app/node_modules/text-buffer/lib/text-buffer.js:1114

Error: Can't save destroyed buffer
    at TextBuffer.module.exports.TextBuffer.saveAs (/Applications/Atom Beta.app/Contents/Resources/app/node_modules/text-buffer/lib/text-buffer.js:1114:1)
    at TextBuffer.module.exports.TextBuffer.save (/Applications/Atom Beta.app/Contents/Resources/app/node_modules/text-buffer/lib/text-buffer.js:1108:1)
    at TextEditor.module.exports.TextEditor.save (/Applications/Atom Beta.app/Contents/Resources/app/src/text-editor.js:925:1)
    at CodePeekView.module.exports.CodePeekView.saveChanges (/packages/code-peek/lib/code-peek-view.coffee:187:27)
    at Object.toggleCodePeekOff (/packages/code-peek/lib/code-peek.coffee:226:19)
    at Function.module.exports.Emitter.simpleDispatch (/Applications/Atom Beta.app/Contents/Resources/app/node_modules/event-kit/lib/emitter.js:25:1)
    at Emitter.module.exports.Emitter.emit (/Applications/Atom Beta.app/Contents/Resources/app/node_modules/event-kit/lib/emitter.js:141:1)
    at /packages/code-peek/lib/code-peek-view.coffee:67:16)
    at HTMLSpanElement.dispatch (/packages/atom-beautify/node_modules/jquery/dist/jquery.js:4435:9)
    at HTMLSpanElement.elemData.handle (/packages/atom-beautify/node_modules/jquery/dist/jquery.js:4121:28)

Hmm, unable to reproduce this on 1.16. This only occurs when you have multiple matches, right?

Also, did you make any changes to the code in the code peek panel or in the main editor when you opened it in a new tab?

Oh ok. I'm on 1.17-beta5.
I've screen captured it for you ;)

I see. Thanks for taking the time to make the gif. I'm very confused why your actions.js file opens immediately in a new tab when you peek the function in MenuView.js. That shouldn't happen until you click the blue code icon... So maybe something else is going on.
I still can't reproduce it on 1.16. Unless I can, I'm going to punt on it until 1.17 is officially released. Sorry for the inconvenience,


Directional Edges

Hello, I was trying to use your graph library to find the shortest way between multiple recipes.
It's for the following game:
https://neal.fun/infinite-craft/
The idea is that you can combine words to form other words. E.g.:
Fire + Water = Steam
And then:
Earth + Steam = Mud
I have created a Graph using your library which I'd like to use to find out what Path I should take.
The graph now uses intermediate nodes:
Earth => Earth+Steam => Mud
Steam => Earth+Steam
What I want is to have this directional though. Because you can not go from Mud back to Earth.
Is there a way to inform the Dijkstra shortest path finder to only use edges in a specific direction?
P.s. if you're interested in looking at my code / helping me solve it I can give you access. The repo is currently private.

Hello!
Each path finding algorithm have a PathType parameter.
You can see more detailed example here.
https://github.com/Kemsekov/GraphSharp.Samples/blob/main/samples%2FFindPathByMeetInTheMiddle%2FProgram.cs
In general you just invoke methods trough Do object.
var paths = G.Do.FindShortestPathsDijkstra(sourceNodeId,e=>1,pathType:PathType.OutEdges);

var pathToNode = paths.GetPath(targetNodeId);

sourceNode is start of the path
e=>1 is function to get weight of the edge
PathType.OutEdges is how algorithm build path, so with out edges it will build paths only by directed edges from previous node to next, so if a->b then this pair a->b can be in a path

I don't see the PathType parameter, but I do see the bool undirected. When checking out the code it seems to do the same.
Thanks for the explanation :).
I'm still playing around with the graph a bit on how I can somehow visualize it as the visualization I tried to copy over from one of the samples doens't really work for me.
It seems to put everything in the bottom left corner:

Yeah, I believe you just using old version of library.
I strongly recommend you to update to last nuget version.
I will not lie - I was developing this library for my daily use and it is not backwards compatible.
I will add readme to samples library with institutions how to run so you could play around with it and see how it works.
And about positioning;
Use Arrange method - it will try to assign 2d positions to nodes so distance between them will match weight of edges between them.
I don't have documentation for this project, but I have samples that are small enough and could be run to understand how to work with library.
I will update readme for samples RN


Support defining multiple optional fields with different types

This fixes #407 .
The fix avoids the hack to dynamically generate a Union type instance which will just modify the existing Union[int, str] instance and re-use it for any type annotated with Union.

I want to add that this is a super-annoying bug and makes it difficult to use odmantic 1.0. Additionally this bug might have very nasty side-effects as it modifies the built-in Union[int, str] type. Though it might not be used a lot, but in case it is used by some other module, it will have the ability to break it in very creative ways imho.

For people that cant wait for a released version, I use that to patch odmantic when building a docker image:
RUN patch -p1 < odmantic.patch
using a patch file
--- a/odmantic/model.py
+++ b/odmantic/model.py
@@ -194,11 +194,9 @@
         # generics is found
         # https://github.com/pydantic/pydantic/issues/8354
         if type_origin is Union:
-            new_root = Union[
-                int, str
-            ]  # We don't care about int,str since they will be replaced
-            setattr(new_root, "__args__", new_arg_types)
-            type_ = new_root  # type: ignore
+            # as new_arg_types is a tuple, we can directly create a matching Union instance,
+            # instead of hacking our way around it: https://stackoverflow.com/a/72884529/3784643
+            type_ = Union[new_arg_types]  # type: ignore
         else:
             type_ = GenericAlias(type_origin, new_arg_types)  # type: ignore
     return type_

If this patch works, this is amazing! I didn't find the workaround you use when I developed this feature.
I think this might also help with https://github.com/art049/odmantic/issues/379 as well.

Thank you so much @netomi !

ty for merging the fix and making a new release right away.


[install] does kube-ovn install script will install ovs in vm

Hello, I want to know if kube-ovn install script will install ovs in vm using apt-get or dnf? Thanks in advance.
:)

All the things about ovn/ovs is  build into image, which can be found in dist/image/Dockerfile.base
The install script for install ovn is  start-db.sh, script for ovs is start-ovs.sh. The script is in the same directory with install.sh.


Added JSON examples

Two new examples:

Send and receive JSON with runtime type checking
Send and receive statically typed JSON

I merged this manually, thanks! (And sorry for the delay).


fix: support muting bots in new TTS engine

Related to #1252
Implements bot message muting in the TTS engine within lib/tts_isolate.dart as per the issue requirements.

Adds a check to determine if a message originates from a bot using the isBotMuted flag and the author's bot status before vocalization.
Skips the vocalization process for messages identified as originating from bots, effectively muting them.

For more details, open the Copilot Workspace session.

Ok this looks good, just wanna go out and test it yet I can't get past text to speech disabled when trying to enable tts


Add cache_to option and do not pre-pull images specified via cache_from anymore

The way of using cache_from as described in #47 does not work for me. If I use an image in cache_from which was built via a normal docker build step, the build does not use any cached layers. Instead, everything is pulled and run again.
Reading the documentation, it seems when using cache_to, buildkit writes a different kind of manifest (the "cache manifest"). (See here for some context.) So in order to make caching work, we need to add this option.
With this PR, we can now use it like this:
steps:
  - name: build image
    image: thegeeklab/drone-docker-buildx
    settings:
      repo: my/image
      cache_from: my/image:cache
      cache_to: my/image:cache

With this change, I was able to re-run a build which initially took 1,5 hours in 26 seconds.
Also, buildkit seems to automatically pull all the images specified in the cache_from option. Thus the manual pre-pull of those images is not necessary. So I removed it.

Thanks for the effort! LGTM


Ability to modify custom attributes

I have a few custom attributes in my app to handle some colors...Is support for this possible?

Need info on this too

Me too

Is there a way to add more colors, when those below are not enough?

?colorPrimary
?colorPrimaryDark
?colorAccent
?backgroundColor
?backgroundColorDark
?backgroundColorLight
?menuIconColor
?subMenuIconColor

Me too, for example: I need to custom border color for dark/light theme. With new dark mode, I can do it with:
value/colors.xml
<color name="colorBorder">#EBEEF5</color>

value-night/colors.xml
<color name="colorBorder">#303133</color>

So I can use @color/colorBorder preference and it will switch automatically when user enter dark theme. How can I do with this library?


Makes PIX work with the latest zig version

Hello there, some small fixes to make samples compile and run when PIX is enabled.

Great, thanks!


Sorting, Filter, Search on SequenceField

SequenceField values are stored as int so sorting, search & filter could be easily supported.
As of Flask-Admin 1.5.4, they aren't supported.
Getting errors:
raise Exception('Unsupported filter type %s' % n) Exception: Unsupported filter type _id
Exception: Can only search on text columns. Failed to setup search for "<mongoengine.fields.SequenceField object at 0x7f8b8988a080>"

from flask import Flask
from flask_mongoengine import MongoEngine
from flask_admin import Admin
from flask_admin.contrib.mongoengine import ModelView

app = Flask(__name__)
app.config['SECRET_KEY'] = 'your_secret_key'
app.config['MONGODB_SETTINGS'] = {
    'db': 'your_database_name',
    'host': 'localhost',
    'port': 27017,
}

db = MongoEngine(app)

# Define your MongoEngine Document with SequenceField
class Item(db.Document):
    seq = db.SequenceField()
    name = db.StringField(max_length=50)

# Custom ModelView with the SequenceFilter applied to the seq field
class ItemModelView(ModelView):
    column_filters = [SequenceFilter(column=Item.seq)]

# Add views to Flask-Admin
admin = Admin(app, name='Flask-Admin Example', template_mode='bootstrap3')
admin.add_view(ItemModelView(Item))

if __name__ == '__main__':
    app.run(debug=True)


Additional validations needed on the donation page

Bug entered by Valerie Campbell -<EMAIL_ADDRESS>Tested in Chrome OS Version 49.0.2623.112 (64-bit) on Windows
Issue:

Page accepts single letter for first name and last name, even after 'Review Donation and Continue' is clicked.

Donation amount, credit card, expiration date, phone number and CSC fields validate only after 'Review Donation and Continue' is clicked. These fields should not allow characters other than numbers.

If invalid phone number is entered, the Phone Number field shows red but the field is not listed in the error message text.

The Phone Number field does not accept international phone number format. Is that by design?
Attached screenshot covers 1-4 above.

City, State and Zip fields are listed in the error message as invalid when any one field is incorrect. (screenshot attached) Is that by design?

What donation page is this?

Thanks for the issue, this looks like bugs on the paypal site so out of scope for us


RegisterAsBrowser issue.

I found a lack of functional of your shell. And it's very important for me and may be for others. I use WebBrowser in my projects. It uses IWebBrowser2 interface which has RegisterAsBrowser property. This property is very important in case of interaction between top-level browser window and popups (especially popups that created by Javascript function showModalDialog). Setting RegisterAsBrowser = True doesn't work when i use CairoShell. And popup knows nothing about parent top-level browser window and therefore cannot pass necessary info to it. But when i use standard shell (explorer.exe) it works fine. An identical question was asked here, but it is unanswered yet and i think it wont be closed.

Try disabling the "Periodically re-establish notification area hook" option in Cairo Settings, and restart Cairo. This should allow the other apps to see and use explorer.exe functions while Cairo is running.

No, it's not about Notification Area (tray). But i did it and it didn't help. I think IShellWindows interface works in this scenario.

I believe the problem is that when Cairo runs as shell (no explorer.exe) some system functionality is lost due to some COM objects and interfaces that are not implemented in Cairo.
Sent from my iPhone
On Mar 20, 2018, at 4:02 AM, apzloy <notifications<EMAIL_ADDRESS>wrote:
No, it's not about Notification Area (tray). But i did it and it didn't help. I think IShellWindows https://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fmsdn.microsoft.com%2Fen-us%2Flibrary%2Fwindows%2Fdesktop%2Fcc836570(v%3Dvs.85).aspx&data=02|01||9ba32c4a308143e1206a08d58e38fb04|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636571297733607314&sdata=b22a87uD4TU5b8hQactGacwYLp%2FaeRN0gje24T%2B9bQg%3D&reserved=0 interface works in this scenario.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fcairoshell%2Fcairoshell%2Fissues%2F108%23issuecomment-374508073&data=02|01||9ba32c4a308143e1206a08d58e38fb04|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636571297733607314&sdata=zDlRAtcLGoUJb84P9qYbxBP%2BPxi7xgjDiCZF1z0xcZQ%3D&reserved=0, or mute the threadhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAGxXAcLbiyRl39_3_5hEeZlDS68neV02ks5tgLepgaJpZM4SwTvI&data=02|01||9ba32c4a308143e1206a08d58e38fb04|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636571297733607314&sdata=iB6KJWqPStO0QkUnNqGZSG82poCbO9ow%2F5tRJKN2mU8%3D&reserved=0.

Did you install Cairo with the option to replace Explorer? If not, Explorer is still in the background and can service those requests. The only thing I can think of that would prevent this from happening is Cairo's notification area support which intercepts messages to Explorer.

Why are you talking about notification area again? What a point of it?
When you create shell replacement (INSTEAD of explorer), you should create window with className "Shell_TrayWnd" for your notification area. Next step is to call SendNotifyMessageW(HWND_BROADCAST, RegisterWindowMessageW(L"TaskbarCreated"), 0, 0)
In this case you will be able to receive WM_COPYDATA message in your WndProc. Then you should handle COPYDATASTRUCT to determine if it is notification area data. If it is then you can manipulate this object as you wish. Add, show, remove, update notification icon etc... And that is all about notification area. And how does this relate to the described above problem?

You are correct, and Cairo follows these steps. However, many of the shell APIs send messages to the first window returned with the class Shell_TrayWnd. When Cairo's notification area is enabled, Cairo is the first window with class Shell_TrayWnd returned, meaning the message never makes it to Explorer. Hope this makes more sense. Maybe I can forward the requests to Explorer.

Yes. "first window returned with the class Shell_TrayWnd". You are right. But anyway i can't see relation between my problem and this thing we are talking to.

Do you have an example program I can use to reproduce the issue?

I made a small app that sets registerasbrowser on some explorer windows and the setting seemed to be sticking when querying them. Is this still happening on the latest builds?

Closing due to no response.


HUE-8671 [autocomplete] handle two escape character within quotes

Change looks good! :-) Could you remove the user-specific changes to the shell scripts and I'll merge it.
Thanks!

Added to master


Pass in a list of Publishers to the Scaffolder

Feature Suggestion

The Frontend will need some way of displaying a list of publishers that take a folder to create a repository out of.
These publishers should be listed on the scaffolder after initialisation
Could be displayed in the frontend in a drop down with more options if the publisher requires more input from the user.

Possible Implementation

Publisher has a publish function and getValues({ template })
/v1/publishers would list the available publishing choices with options

Context

Want to give users the ability to create their own publishers easily rather than just using github

Some work has been done here on this, but really need to look at this too. #2426

@muffix do you think you could pick up this too seen as you've done half of the hard part ;)

Probably want to look at enabling extra publishers to be in from the source template to though annotations.

Or maybe adding the concept of hooks to be defined in the source template which are executed around the stages.

Will close this in favor of the new approach in #2771


[OpenVR] Updated to version 2.5.1

[x] Changes comply with the maintainer guide.
[x] SHA512s are updated for each updated download.
[x] The "supports" clause reflects platforms that may be fixed by this new version.
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

@microsoft-github-policy-service agree

I will be converting your PR to draft status. The above suggested changes are only recommendations. If you are willing to adopt them, please click "ready for review" after making the modifications. If you do not wish to make any changes, please click "ready for review" directly. That way, I can be aware that you've responded since you can't modify the tags.

The new error is because you didn't execute the .\vcpkg.exe x-add-version openvr --overwrite-version command locally after committing the new changes. This caused the content of the openvr.json file to become incorrect.

However, my comment was just a suggestion, and whether to adopt it depends on you. So, since you've reverted your change, consider it as if you didn't adopt my suggestion.


Draft: 2743 windows elt working

Old MR from Gitlab https://gitlab.com/meltano/meltano/-/merge_requests/2639/diffs

I went ahead and rebased this onto Main -  Put the new PR here https://github.com/meltano/meltano/pull/6155


Add the ability to enable ATP for Azure Storage using azure-cli

Question:
The Docs article here describes how to enable Advanced Threat Protection for Azure Storage using a variety of methods, but there's no mention of how to use azure-cli to achieve this.
Desired Solution:
The ability to enable ATP for Azure storage using the azure-cli, ideally at the individual account level rather than for all storage accounts in a subscription.

Storage

Thanks for your feedback! Azure CLI supports it with az security atp storage -h, but it is not documented. @tamram could you help add Azure CLI part？

@cbellee - I've updated the doc with instructions for enabling Azure Defender for a storage account with Azure CLI or PowerShell.
@Juliehzl - FYI


Henson and intent builders create a web of related classes to big for 1st dex

On pre-lolipop devices, there is a limitation of 65k methods that can fit into the first dex.
Henson contributes to it: if a class A uses Henson to open an intent targeted to class B, then A.class will transitively depend on B.class. Placing A.class in the first dex would include all activity classes that can be reached by Henson in the first dex, making it too big.
The problem is that Henson will use an B$$IntentBuilder that knows about B.class. A workaround is to use reflection to load target activity classes and hence avoid to reference them indirectly from source classes.
Henson has to know about the intent builder types, so it can't use reflection to load them. We need the types there to be known at compile time and reach the DSL state machine. But IntentBuilders themselves only use the token / type litteral of the target class name. They would be a good candidate to use reflection and cut the branch of the tree of dependencies between classes.
Actually, by doing so, Henson would solve a larger problem in Android and mask reflection usage to reach other activities. There are other workarounds like using reflection in new Intent manual calls, or using implicit intents. But having Henson hiding reflection calls to load classes would just be so much more elegant than former workarounds..

Solved in 2.0.1


Build errors in CordovaPluginFileDemo

we need to setup the workspace's build location to be relative to the workspace.
In File -> Workspace Settings..., click Advanced... in Build tab to open the Build Location settings. Select Custom -> Relative to Workspace, and leave the default value as it is.
And we need to build for "iOS Device" to trigger Xcode to create the directory "Debug-iphoneos".

I'm considering to add search paths for both simulator and device build, or write a script to do the embedding work, which could make the setup less painful.

Unfortunately the above remedies about workspace settings and build for iOS device had already been applied and the error still occurs.

The directory third-party/cordova-ios contained no entries. I manually copied the files from my backup and the resultant build has no errors--only the 4 warnings I mentioned earler.

The third-party/cordova-ios is an submodule, and this file demo also depends on another submodule in third-party/Icenium/sample-file. so you need to run git submodule update --init --recursive in crosswalk-ios directory to retrieve source codes for those projects.

Interesting.  I didn't have to do that before!
The app actually does run--however there was a long timeout before it came up. The appstore image is a new message from that demo.

After fixing all the missing submodules, this is what I believe the copy bundle resources should be. I am going to start fresh as GitHub on my Mac desktop seems to have lost its brains.

I decided to start over since the Github desktop caused several of the problems. With a fresh clone, here are the steps to be followed to make the CordovaPluginFileDemo work properly which could be in the readme:

In Xcode. File-Workplace-Settings-Advanced. Build location-custom-relative to workspace.
In xcode project settings-general remove Launch screen file: main. (We want the plugin to display the launch screen)
Build phase-Copy bundle resources from the Icenium app-resources/ios directory add Default-568h.png, Default-Portrait.png, and Default-Landscape.png
Select scheme CordovaPluginMenu either from dropdown or product menu
Select destination iOS device either from dropdown or product menu
Build or run if device attached

At this point the plugin demo will correctly generate the splash screen from the plugin and the file demo from the plugin.  It will also work on the simulator if you chose the destination device.


it seems that the model is overfitting

hello,thanks for sharing your code.
But when I run the code, I find that the model is overfitting. For example, in cifar10 dataset, the train acc can reach 98%+, but val acc only 90%+.So what can I do?

I find that so? Its the network' problem?

i wonder too, there is some much overfitting


⚠ Bofrost.lu has degraded performance

In d29c74b, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1933 ms

Resolved: Bofrost.lu performance has improved in 765164f after 7 minutes.


REQ ENHANCEMENT: Alarm states and Armed (or Disarmed) states to HA

Some additional states from the Concord to Home Assistant.   Also to the HA log.

Armed State  (or maybe 'disarmed' state?)  For what I'm thinking 'Armed (true, false)' would probably be enough.  Altho if it's being sent from the Concord to HA may as well report them all?

Alarm (and what kind?)   For my purposes I'm only interested in the Alarms that mean the sirens are blaring and lights are flashing and that the police or fire dept are on the way.  Gravy would be bringing back and reporting all the alarm states to HA.

Nice to have:

who (or what user code) armed or disarmed the Concord in the HA log.
Looking at the Automation Module Protocol it looks like all of these codes are available starting on page 14, with the actual alarm types starting on page 15.   What is unclear is which direction they are going.   Are these from the Concord going to HA?  Or does the Concord expect these to come from the peripheral (HA in this case)?
From a quick look at

concord4ws-py/concord4ws/types.py

it looks like a lot of these are already in the data on the -ws side.   I have not looked at the code on the -ha side yet.
I may take a look at this myself but it's been like 15 years since I did any real coding.  Everything since
has been in a vendor-proprietary "language" (and I use that term -very- loosely), or bash.  'Rust' to me is something
I take a grinder to.  So I don't have much hope for my own skills.  What can I say?      ¯_(ツ)_/¯
Other than what I reported earlier, this interface has been solid.  Thanks!

Posted in wrong place.  Apologies

Posted in wrong place.  Apologies


Scaffold for package example usages

Add some example usages in ES6 syntax.

Looks good.  What is the use of the Nodes attribute in the 'profiles' file?
Also consider calling neo.nodeWithBlock(blockIndex, 'latency', false) on the block related requests to get the fastest node that has been verified to have a certain block.  This will prevent the request hitting a node that does not have the block once we get dynamic seed registration up and running.  Eventually, we'll abstract all the RPC/local db calls and toggle based on optional parameters and the mode.
Other than that this is awesome!

Would be good to use async/await in these as there's so much promise action

What is the use of the Nodes attribute in the 'profiles' file?

You are right, the "nodes" are not provide any use, thus is now removed.

consider calling neo.nodeWithBlock(blockIndex, 'latency', false) on the block related requests to get the fastest node that has been verified to have a certain block.

@lllwvlvwlll, Can you provide code example?
My understanding is that you will need an active node in order to obtain current block height (which I assume is the value required for blockIndex?), which means you must have an local instance running for neo.nodeWithBlock(blockIndex, 'latency', false) to work, correct?

Would be good to use async/await in these as there's so much promise action

@notatestuser, async/await are already in-use in node-app.js. Or can you recommend further optimisation?
On a side note, I'm actually thinking of dumbing it down to Promise chain or even callback loops. Reason is, these examples are meant to be additional guidance for new developers of all level, and not everyone is comfortable (or has the environment setup) for ES2017 syntax.

Applying async/await in a few places could improve the readability a lot:
Before:
const getBlockCountDemo = function () {
  console.log('getBlockCountDemo:');
  return new Promise((resolve) => {
    neoNode.getBlockCount()
      .then(function (res) {
        console.log(res);
        // console.log(res.data.result);
        resolve();
      })
      .catch(function (err) {
        console.log('error:');
        console.log(err);
        resolve();
      });
  });
};

After:
async function getBlockCountDemo() {
  try {
    const blockCount = await neoNode.getBlockCount();
  } catch (err) {
    console.log('error:', err);
  }
  return blockCount;
}

Returns a promise as before.

@notatestuser, thanks for recommendations. examples updated accordingly.

neoNode = neoBlockchain.fastestNode();
But the response this method will provide will change over time depending on which node is fastest.  There is also the risk of being connected to a node which does not have the block you request because it isn't fully synced.
neoBlockchain.nodeWithBlock(10000, 'latency', true).getBlock(10000)
In the near future, all the methods will be abstracted and the core controller will automatically route to the best node (local or remote) depending on what the request was: neoBlockchain.getBlock(10000)

@lllwvlvwlll, after further evaluations, I've managed to obtain a valid node with:
neoBlockchain.nodeWithBlock(-1, 'latency', false)
That is because all nodes in blockchain.nodes have blockHeight; 0 and index: -1. For example:
{
    domain: 'http://test1.cityofzion.io',
    port: 8880,
    age: 0,
    active: true,
    latency: 0,
    blockHeight: 0,
    index: -1,
    connections: 0,
    pendingRequests: 1,
    getBalance: [Function],
    getBestBlockHash: [Function],
    getBlock: [Function],
    getBlockCount: [Function],
    getBlockHash: [Function],
    getConnectionCount: [Function],
    getRawMemPool: [Function],
    getRawTransaction: [Function],
    getTXOut: [Function],
    sendRawTransaction: [Function],
    sendToAddress: [Function],
    submitBlock: [Function],
    call: [Function],
    deferredUpdateLoop: [Function]
}

Is that the expected behaviour?
Bit of background on my setup:

I'm developing on macOS, NodeJS 8.4
No localhost mongo instance

@rockacola How are you running the code?
When you initialize a blockchain instance, it needs to get in contact with the nodes before they become 'available' for querying using that method.  If you are instantiating, then immediately calling that method, the nodes may not have become active yet.
try it with a 5 second setTimeout.  You can also try explicitly picking a node and making the call directly:
neoBlockchain.nodes[0]

@lllwvlvwlll, for testing/demonstration purpose, I can explicitly picking up a node directly with neoBlockchain.nodes[0].
Also, I'm restructure examples to make it a self documentation (oppose to just run through list of endpoints undocumented).
It may make sense to close/reject this PR until I'm ready with updates.


Forecasting Subquestions Task

Task: Generate Forecasting Subquestions
A forecasting question is a question about the future which can be answered with yes/no (“Will North Korea launch an attack on the United States?”) or a number (“What percentage of US small businesses will go bankrupt in the next year?”) or a date (“When will Nintendo launch its next videogame console?”).
A subquestion is another question that, if answered, would help you answer the original forecasting question.
The task is: given a forecasting question, generate related subquestions.
Authors: William Saunders (william@williamsaunders.net), Andreas Stuhlmüller<EMAIL_ADDRESS>@stuhlmueller)

Thank you for submitting your task! We are in the process of pairing submissions with task reviewers and will be in touch soon.

Thank you for this submission!

The task passes the code tests.
Is this really a “generation” task or is it a “discrimination” task? The submission describes itself as “given a forecasting question, generate related subquestions”. However, as I understand it the LLM taking this test is not invited to generate subquestions, but instead to pick one out of a multiple choice set of subquestions have have already been provided? Is the self-description accurate?
Another feature of the task is that it is not easy for a human. Of the ten multiple choice answers suggested for each question, none of them are obviously “wrong”, and you could make arguments in favor of each one. I would expect that human subjects would not have a sharply peaked distribution on which answers they picked. To give a more graded response, might it be interesting to add some more obviously “wrong” answers in there?
Here are some typos:
⁃	typo: “incentives to by electric vehicles”, by -> buy
⁃	typo: “ownership of electric * vehicles”
⁃	typo: “What is the percentage of new cars available for sale are electric”
⁃	typo? “geometric mean across of the probability” x2

Thanks for the comments.
I clarified in the task description that it is not a multiple choice task, instead we measure the average log probability of generating all possible options. The expectation is that language models that assign a higher log-probability of generation will be more likely to generate useful subquestions overall.

I just signed the CLA.

@googlebot I fixed it.

@googlebot I fixed it.

@Sohl-Dickstein Thanks for your comments. I've addressed all of them except I haven't created a multiple choice version of the task at this time. I think the CLA bot is complaining because I accepted one of your suggestions and that made you a co-author on the commit, could you check that a CLA has been signed for<EMAIL_ADDRESS>?

@googlebot I fixed it.

Hi @william-r-s - As a reminder you have until Friday, March 26th to revise your task to be accepted in the workshop. Please tag your reviewers when you have addressed their suggestions.
Reviewers:
@MrAdamBrown and @Sohl-Dickstein if the submitter has made revisions please be sure to notify me with a formal "accept" or "does not meet criteria" by Friday, March 26th.

@MrAdamBrown and @Sohl-Dickstein, I believe I've addressed your comments, though I haven't implemented a multiple choice version of the task.

@chiafullo Accept.
Thank you for the task! Also, don't miss the one additional nit change in my review, adding a space to the model query.

@chiafullo accept

@mathopus for meta review and merge (if acceptable)

@mathopus please review!

Sorry for the delay, was unexpectedly OOO. Everything looks good to me. Merging.

Hi @william-r-s ,
I was just running this task using the colab
https://colab.research.google.com/github/google/BIG-bench/blob/main/notebooks/TaskTestingNotebook.ipynb
and I notice that the question and response strings look like b'Will a 100 trillion parameter deep learning model be trained before 2026?' (with an extraneous b and '. I think you may want to include a .decode("utf-8") somewhere in your code, to turn the raw strings you load from the file into Python strings. You could make this change in a followup PR.
Thanks again for the task!

@cdfreeman-google It looks like the task was modified after I added it in this commit https://github.com/google/BIG-bench/commit/4d7cdc5074e48ab51cdf350932f0bb024fe02b34 to read files in binary format. I'm not sure why this change was made, could you clarify what the edge case was and whether convertin the bytes to strings after loading would be compatible with this change?

@Sohl-Dickstein should be fixed by https://github.com/google/BIG-bench/pull/471


能告诉我一下这款App的应用场景吗？

Could you explain your issue in English, please?

做live coding的时候用，或者教学的时候之类的
For live coding use or making live stream let your audience know what have you typed.


Problem decoding Apple ][ floppys

Thank you for the help with c64 image decoding #159.  I was hoping that it would be easy to move on to Apple ][ discs, but I'm having issues there as well.
I've read the disc using the same method at the C64 images.  I use greaseweazel to read the disc--letting it read 82 tracks and both sides.  Then I convert the scp file to a flux file and feed it to the 'apple2' decoder.  Here's some of the results I'm seeing: (attached)
25_read.fluxengine.read.txt
26_read.fluxengine.read.txt

Well, that's a mess. I don't have any Apple II disks so I was working from Kyroflux dumps, which means I have no experience dumping them myself.
This looks like a 6-and-2 140kB disk, from AppleDOS 3.3 or later, so it should work.
Two things to suggest:

try fluxengine inspect -s yourdisk.flux:t=0:s=0 and see what the histogram looks like. This shows the width of pulse intervals on the disk. You should see two or three fairly sharp spikes. If they're rounded, that indicates a bad or poor disk.

try fluxengine read apple2 while varying the --bit-error-threshold value. Sometimes it helps to move it up to 0.5 or decrease it to 0.3 --- it's more of an art than a science. The other options very occasionally help too, but that's the main one.

It's possible there's something wrong with the alignment. It's a 40-track disk, so each track should be being read twice on a 80-track drive, and yet some tracks are showing nothing. The occasional data conflicts are particularly weird; this means the decode has seen the same sector more than once with different data. This can happen on disks which have been reformatted, but I doubt that's the case here...

Once I figure out what I'm dealing with, I can send you some .scp images if you think it will help. I've got penty of these discs to go through. ;)
I ran the inspect on the two disc captures.  They're attached.  One shows three nice peaks--narrow and well separated.  The other has poorer peaks, but they're still distinct.
25_read.fluxengine.inspect.txt
26_read.fluxengine.inspect.txt
You're missing a newline at the end of the analysis.  The command prompt appears on the same line as the clock data. Sorry, that should be a different bug report, but I just noticed it.  Hmm, different values are not helping.  I think I'm going to try to manually specify the clock value.  And that option isn't in the apple2 decoding method.  Okay, I'm at an impass.  I ran inspect on all even tracks and see it pretty reliably detecting 3.25 or 3.33us for the clock.  That doesn't explain things, though.

If it's any consolation HxC doesn't like them, either.


feat(share/pruner): disallow converting pruned node to archival

This PR add safety check, that prevents node runner from converting previously pruned node to archival, because it is not supported.

Codecov Report
Attention: Patch coverage is 66.66667% with 12 lines in your changes missing coverage. Please review.

Project coverage is 44.57%. Comparing base (2469e7a) to head (d64dea2).
Report is 102 commits behind head on main.

Files
Patch %
Lines

pruner/checkpoint.go
57.14%
11 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3467      +/-   ##
==========================================
- Coverage   44.83%   44.57%   -0.26%
==========================================
  Files         265      273       +8
  Lines       14620    15369     +749
==========================================
+ Hits         6555     6851     +296
- Misses       7313     7725     +412
- Partials      752      793      +41

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Malaysian language support.

Laravel Version: #.#.# 5.7.11
PHP Version: 7.2.10
Laravel-admin: #.#.# 1.6.5

Description:
I want to add a support for the Malaysian language.

Please feel free to send a pr.

Thanks for your reply.

Merged！


WIP: Magic spmi

still need to adapt at some places to the new NCL interface

He @bjoernbroetz
I just noticed you merged our magic-demo branch into the SMPI branch.
Unfortunately, the magic-demo branch is a temporary copy-paste of a number of recipes to get the demo to work. We have no intention of ever merging this back to anything. As a result this pull request is now un-mergeble.
Perhaps it would be best to revert this merge and/or branch of an old commit of this branch.

@nielsdrost Thanks! I took 5c12a64 back the branch needs to be cleaned up updated with for the new NCL-interface. Further discussion in #767


missing  tag

hi there.
I install hexo-optimize, and run hexo g.
Then I found that

the html file missed  tag.
<!DOCTYPE html> became <!doctypehtml>

may I ask if this is normal?

HTML minify is enabled by default. You can disable it by setting
filter_optimize:
  html:
    # minify all html files
    minify: false

in Hexo _config.yml

Yes I know. Do you mean this is normal compression?

Yes，it's expected behavior

thanks. it's awesome


Compare at price inconsistencies

Issue Description

Variant price does not save with an empty Compare at price. No error message prompted to the user.
Giving zero as Compare at price is not retained after save.
Place the cursor in the empty Compare at price field and click save. The save goes through.

Steps to Reproduce
1.Create a new variant. Under prices enter a value for price and leave Compare at price empty and click save. Save does not go through and no error message is shown.
2.Now set Compare at price to zero and save, save is successful. The Compare at price field does not retain the previous value. The same issue as repro step 1 persists.
3.Place the cursor in the Compare at price field and click save. Save goes through with no errors.

I don't know who fixed it, but entering a price without touching the Compare at price field now works. The price gets saved without any issue. However, entering 0 as a Compare at price still gets dismissed.
Do we want users to be able to save 0 as the Compare at price? I can make a PR for that.

I can confirm it's fixed.

However, entering 0 as a Compare at price still gets dismissed.
Do we want users to be able to save 0 as the Compare at price? I can make a PR for that.

Giving this some thought I believe 0 implies that no compareAtPrice should be set. So it makes sense not to retain 0. Users may assume that 0 gets displayed on storefront, which is not the case.
Once a compareAtPrice is set there is not intuitive way of clearing it. Just removing the value and saving retains the old value. The only way to clear it is to set the value to 0 (or negative) and save.
We can retain the 0 or not but in both cases I think there should be a message below the field which says "0 clears compareAtPrice" or something similar.
@delagroove @Akarshit @loan-laux

Yeah I guess "set 0 to disable" kind of message works. Even I have seen a lot of places where 0 means nothing.

Agree with @Akarshit, 0 would work in this case


fix(DayPickerRangeController): del hovered-span modifier for new end date

If not deleted, then when selecting a new range with an end date before the previous one, it will still keep the hovered-span modifier.

Steps to reproduce:

Select a range.
Select the same start date again.
Select an end date before the previous end date.
The end date keeps the CalendarDay--hovered-span when it shouldn't.

Fixes #520.

Coverage remained the same at 86.537% when pulling e0d65b8270ac750ce963c53f9b2120fc66db185a on relayrides:end_date_hovered_span_class into 380e6f96fac61732bc603d5336a0eff7436deb74 on airbnb:master.


Show function argument variables at call site

Within a function, incoming arguments are named as arg_1, arg_2...; however, in the caller, the argument variables to the callee are not documented. It would be nice if, beside a call instruction, something like this showed up: call sub_0xf00    {var_x, var_y, var_z} meaning that the local variable var_x in the current function is passed as the first argument to sub_0xf00.

This shows up in Medium Level IL now, but not yet in other views


[Event Change Request] table 36 "Sales Header" - OnBeforeUpdateSalesLineAmounts

Could you please change the event signature of the event OnBeforeUpdateSalesLineAmounts to include the parameter SalesHeader by reference?
[IntegrationEvent(false, false)]
//+Change
//-local procedure OnBeforeUpdateSalesLineAmounts(SalesHeader: Record "Sales Header"; xSalesHeader: Record "Sales Header"; CurrentFieldNo: Integer; var IsHandled: Boolean)
local procedure OnBeforeUpdateSalesLineAmounts(var SalesHeader: Record "Sales Header"; xSalesHeader: Record "Sales Header"; CurrentFieldNo: Integer; var IsHandled: Boolean)
//+Change
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 48695.


Quick Sort

Kindly review and add the pr

Can I know in which language you have written the code? @techsavvy0109

I have used Java
On Tue, Oct 18, 2022, 00:44 Shreya Christiana Malogi <
@.***> wrote:

Can I know in which language you have written the code? @techsavvy0109
https://github.com/techsavvy0109
—
Reply to this email directly, view it on GitHub
https://github.com/CodeMacrocosm/Algoathon-22/pull/118#issuecomment-1281350606,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3VGVZDG52VOFTK3IP56W23WDWQQDANCNFSM6AAAAAARHLOQJY
.
You are receiving this because you were mentioned.Message ID:
@.***>

@techsavvy0109 Okay.  Make sure that you first open the java folder and check if your algo is already present, if it's there, then try another algorithm. If it's not there, then create a file inside the java folder and give the filename as quicksort.java (not quicksort.txt), then write your code, commit changes. Also delete your old file which was not placed in the appropriate folder and had .txt extension. Then send a pull request.


Some elements cannot be targeted via CSS

I couldn't figure out a way to target the member element since it doesn't have a part — is there another way to do so? I would like to change the grid-gap value.
Elements that cannot be targeted:

summary and marker for the folding of the main "Metadata" title
.set on the table element
.member on the tr element

Hey @kepano thanks for bringing metatable into the minimal theme. I use your theme myself and I have to say I like your style way more than mine 😄
Let me look into making more parts available for deeper customisation 👍
Aside, the naked setting removes the shadow dom boundary so styling is completely free. Would this option be more accommodating to you?

The new version 0.13.0 adds parts for summary, marker, set and member. It also adds --metatable-member-gap


License as a service

No JIRA issue.
When applied it will

Expose the license creator as a RESTful service

Where should the reviewer @DANS-KNAW/easy start?
Perhaps here
How should this be manually tested?

Install it on deasy and check it with sudo service easy-license-creator status, etc.
curl -X POST http://localhost:8060/create?datasetId=easy-dataset:1 on deasy.

@DANS-KNAW/easy for review.


Inquiry Regarding Installation of eSIM Using activationCode

I am currently exploring the functionality of your repository and I am interested in installing an eSIM card using the activationCode provided by my eSIM provider.
I noticed that the current method in the repository does not have a property for activationCode. Could you please provide guidance on how I can achieve this alternatively? Are there any specific steps or modifications I can make to the existing code to accommodate the activationCode during the eSIM installation process?
Thanks!

the param confirmationCode is the activationCode

@faezmurshidi  Thanks. But I always get an error:
"EMBEDDED_SUBSCRIPTION_RESULT_ERROR - Can't add an Esim subscription"
Do you know what should be the problem?

@dzenankosuta have you checked with your RSP esim provider whether they have set up the configuration for your app - as carrier privileged. This include setting up your app package and sha signing key in the esim profile metadata.
alternatively, you might want to debug the device itself when doing the setup esim call - to get more logs.
adb logcat | grep "Euicc"


Error in useFieldset is not propagated

Hi. I'm not entirely sure what is happening. I might be wrong somewhere because i don't understand something but it also might be a bug.
I have a fieldlist and fieldset. In a fieldset, there's 3 inputs. Name, notes and templateId select. Name and templateId are required fields. However, when I submit the form, only name get's the error, not the templateId.
Here's sample repo
CodeSandbox
Try submit the form and you'll see that the last input does not have any error. The code is simplified a lot but it's just for a showcase

Hi @jansedlon,
It's because of the hidden input in your <Select /> component. Try changing it from <input type="hidden" {...} /> to <input hidden {...} /> and it will work. I understand this is confusing. Below you can find a detailed explanation. I will aso improve the integration docs as soon as I can.
What happened?
The custom select component renders an input with type hidden. Its value is captured by FormData correctly. The result of parse(formData) is correct as well with the expected error message. But the error of that particular field is not populated.
Why?
Conform relies on the constraint validation mechanism heavily. To populate the error, it set the error message on the input element, then trigger the invalid event and finally populate the error message.
But, this mechanism does not apply in some situations. For example, as input with type hidden was not meant for user input. As the value is not changable, there is no need for the browser to validate it. The invalid event will not be triggered and no error could be captured. This applys to both disabled or readOnly input as well (Except the hidden attribute 😅 ).
I am using a component library and it is using input with type hidden
Sometimes you just can't change the implementation. One solution is to bring your own hidden input:
import Select from 'some-component-library';

function MyCustomSelect() {
  return (
    <>
      {/* This is the input that will be validated by Conform */}
      <input name="field-name" hidden />
      <Select
        {/* Don't pass any name to Select */}
      />
    </>
  );
}

This is basically the approach I demonstrated on the [integration] guide. The integration hopefully will be way simplier when the new  element is widely adopted in the future 😅

That's great explanation, thank you very much 🙏 And sorry for taking your time


apps: Remove all stuff related to CONFIG_xxx_CXXINITIALIZE

since it is moved to the central place in nuttx side instead
Signed-off-by: Xiang Xiao<EMAIL_ADDRESS>Change-Id: I544d6110f1ca6460f7c82f970870aa9b1e7ab3dd
Summary
depends on: https://github.com/apache/incubator-nuttx/pull/1341
Impact
Testing

This should NOT me merged apache/incubator-nuttx#1341 is stalled. It is the wrong approach .

Yes, we shouldn't merge this patch before apache/incubator-nuttx#1341. I provide this patch just show the current siutation as reference. Whether this is wrong approach is still in the discussion, please give you comment in apache/incubator-nuttx#1341 instead.

@xiaoxiang781216 I will go ahead and merge this now.  it still cannot pass the PR checks  because some defconfig files set appliation C++ settings that no longer exist, for example.  esp32-core/nsh
This will break all all PR checks until all of the relevant defconfig files are updated in the incubator_nuttx repository.

This will break all all PR checks until all of the relevant defconfig files are updated in the incubator_nuttx repository.

I will submit the the PR that fixes the PR checks in a few minutes.

Yes, here is the patch: https://github.com/apache/incubator-nuttx/pull/1347

@xiaoxiang781216 would you be kind enough to fill in the #Impact section and give porting instruction?

@davids5 the change is compatable as before since all config which utilize the global constructor should already define CONFIG_HAVE_CXXINITIALIZE. The user don't take any special action and the feature still work as it, the only difference is that some config doesn't need anymore(e.g. CONFIG_EXAMPLES_HELLOXX_CXXINITIALIZE or SYSTEM_NSH_CXXINITIALIZE), but there isn't any bad side effect if user forget to remove these config from their defconfig file.


Show result count in source filter

We could show the number of results on the checkbox list of sources. This is an enhancement that could help users understand how the information is distributed across sources.

This would require be adding *&aggs=source to every geneset search query.
https://mygeneset.info/v1/query?q=*&aggs=source.
On a side note, I don't know if the checkbox list is being dynamically generated on the frontend. If a new source is added, will it automatically show up here, and how would the layout accommodate it?

Not too difficult to add I think, but I'll mark as enhancement along with the other feedback from Casey. Once we get other user feedback, I can do these small enhancements in a batch.

On a side note, I don't know if the checkbox list is being dynamically generated on the frontend. If a new source is added, will it automatically show up here, and how would the layout accommodate it?

The checklist is being generated from the curated sources listed in /metadata, so if that updates, so will the checklist.

I think this is related - documentation for how to grab all wikipathways, for example:
https://mastodon.social/@rmflight/109387955847675692

Yeah, I think it would fit @cgreene . As someone who does a fair amount of gene annotation enrichments (GSEA, hypergeometric, etc), having a single source to grab collections of up-to-date genesets would be awesome.
It seems like it should be doable, especially based on the API docs, but I'm not quite seeing it at the moment.

@vincerubinetti as another UI idea for this feature, we could also create a biothings_client-based code dump of the genesets exporting from the UI as the batch download option. If we assume the need of this feature is only for the informaticians who have no problem running an exported Python script (also to modify to fit the specific need).


ENH: Add option to access index in point set registration

This changes adds the class PointSetToPointSetMetricWithIndexv4 to the Metricsv4 module.
This class contains the previous implementation of everything in PointSetToPointSetMetricv4 but changes the
pure virtual functions such that the index in addition to the point is passed to the functions.
This avoids using the fixed points locator to find the index in subclasses that need access to it (see for example https://github.com/samuelgerber/ITKThinShellDemons/).
The pure virtual functions GetLocalNeighborhoodXXX are changed to pass the PointIdentiferType index of the point as well.
The PointSetToPointSetMetricv4 inherits from PointSetToPointSetMetricWithIndexv4 and passes the functions above to the original ones, such as to remain unchanged for classes inheriting from PointSetToPointSetMetricv4.
Thus future point set to point set metrics have the choice of which method to implement by choosing to inherit from
PointSetToPointSetMetricv4 or PointSetToPointSetMetricWithIndexv4.
PR Checklist

[x] No API changes were made (or the changes have been approved)
[] No major design changes were made (or the changes have been approved)
[x] Added test (or behavior not changed)
[x] Updated API documentation (or API not changed)
[x] Added license to new files (if any)
[x] Added Python wrapping to new files (if any) as described in ITK Software Guide Section 9.5
[x] Added ITK examples for all new major features (if any)

This functionality could also be added by adding those methods to PointSetToPointSetMetricv4.h and have them by default call the versions without index.
A superclass seems like a cleaner solution though and does not require subclass requiring access to the index to override both options.

Maybe also rebase on current master, to resolve merge conflicts.

LGTM
@samuelgerber thanks for pushing this upstream.
@ntustison does this look good to you?

I assume this is ready for merging?

I'd like to have @ntustison , or someone else familiar with the v4 framework, input on this before merging.
This functionality is not absolutely necessary but helpful for subcalsses in two instances for me so far. It could also be added in different ways.

Hey all, thanks for the consideration.  It looks fine to me but it would seem as though y'all are stretching the capabilities more than we have so I would tend to go with your judgement with respect to this pull request.  Thanks for all the work.


The output with 1.1.0 is basically containing only one fixture, the last one, and only odd teams. Teams within a fixture are fine.

Great explanation.
I tried using ```` and vector instead of [] in vain.
@ sounds like a good idea then.
Thanks for the quick reply and also for your great lib!

Anyone using @ for deref in a faconne range would have to switch to deref, right?  I don't know if there are many/any users doing that, but it is something that could also break <1.1.0 code.
One idea might be something like ~@ and have it work like splicing unquote in a macro - so faconne would provide the apply.
But could someone be using backtick quoting forms in faconne expressions?  I guess this would break them.  I would expect that to be less common, but idk.
Idk.

How about metadata?
E.g. ^{:faconne/expand true} [a b]

I really like that... I rarely/never use metadata and have no idea if that use is idiomatic, but I'm into it.

I think metadata is a good tool for the job: we are tagging elements of a data structure so it can be handled differently by the parser. It's comparable to what the metadata doc says in its intro:

It is used to convey information to the compiler about types, but can also be used by application developers for many purposes, annotating data sources, policy etc.
In our case faconne is the compiler.

It could also help for literal symbols: ^:faconne/literal my-symbol
It would not work for keywords unfortunately as they can't have metadata.
Maybe literal keywords just need to be quoted since macro expansion preserves the quoting for the macro to pick up on:
':k
=> :k
(let [k ':k] (type k))
=> clojure.lang.Keyword
(macroexpand '(let [k ':k] (type k)))
=> (let* [k (quote :k)] (type k))

Thanks @fmjrey for the suggestion, I think metadata is the perfect fit. It lets us easily and idiomatically specify this type of operation, and lets us support other types of operations with a uniform syntax.
I also like changing (:literal :k) to ':k and (:literal symbol) to (^:literal symbol), although I'll continue to support the former version. I'll work on getting those into a later release, because this bug needs to be fixed quickly.
I believe we can use unqualified keywords for the metadata (the advantage being that no one has to know how to spell faconne), because we're attaching them to structures/symbols whose lifetime is exclusively the transform. They're written as arguments to the transform, and since the transform is a macro, these arguments need to specified inline, not def'd anywhere else. So, those symbols/structures are used only in the arguments to the transform.
I was playing with the fix yesterday, and I believe it's fairly straightforward. I'll work on getting this released within the next several days.
Thanks again for your help!

Faconne 1.1.1 fixes this.

I can confirm 1.1.1 now works for me, no need to change any code.
Thanks for the prompt fixing @disalvjn


Redis Backend Occasionally Results In Blocked Task

I'm already using Redis as the broker. I would also like to use it as the back-end. But when I switched, occasionally a task will be blocked on the tornado side waiting on a result. But logs from the celery side shows that the task has completed successfully. My application is also using redis module (not tornado-redis). After some digging, this situation seems to occur if the application is accessing redis (issuing redis commands on other coroutines) while tornado-celery is yielding/waiting.
These logs, taken from redis-cli, which I've marked as good seq and bad seq on the right-hand side shows that in the bad seq scenario, there are other redis commands (marked as other redis cmds) being issued by my app which interleave with the messages from the celery task.
1460564262.493698 [1 <IP_ADDRESS>:48027] "PUBLISH" "ctl.status" "{\"data\": {\"id\": \"aff9e1c0-48d2-4979-b1d6-2a83f8a6dcf6\"}, \"event_type\": \"task_started\"}"
1460564262.494833 [0 <IP_ADDRESS>:40544] "SUBSCRIBE" "celery-task-meta-93d78830-f1c7-4669-8a88-9813e6048820" <----- GOOD SEQ
1460564262.497228 [1 <IP_ADDRESS>:43483] "HKEYS" "ctl.resource.group"
1460564262.497747 [0 <IP_ADDRESS>:41200] "MULTI"
1460564262.497759 [0 <IP_ADDRESS>:41200] "SETEX" "celery-task-meta-93d78830-f1c7-4669-8a88-9813e6048820" "3600" "{\"status\": \"SUCCESS\", \"traceback\": null, \"result\": {\"ids\": []}, \"ch$ldren\": []}"
1460564262.497787 [0 <IP_ADDRESS>:41200] "PUBLISH" "celery-task-meta-93d78830-f1c7-4669-8a88-9813e6048820" "{\"status\": \"SUCCESS\", \"traceback\": null, \"result\": {\"ids\": []}, \"childre$\": []}"
1460564262.497815 [0 <IP_ADDRESS>:41200] "EXEC"
1460564262.500741 [0 <IP_ADDRESS>:40544] "UNSUBSCRIBE" "celery-task-meta-93d78830-f1c7-4669-8a88-9813e6048820" <----- GOOD SEQ
1460564262.501175 [1 <IP_ADDRESS>:48027] "HSET" "ctl.ids.result" "aff9e1c0-48d2-4979-b1d6-2a83f8a6dcf6" "{\"id\": \"aff9e1c0-48d2-4979-b1d6-2a83f8a6dcf6\", \"timestamp\":<PHONE_NUMBER>.493039, \"state\": \"SUCCESS\", \"result\": {\"ids\": []}, \"elapsed\": 0.008044004440307617}"
1460564262.501390 [1 <IP_ADDRESS>:48027] "PUBLISH" "ctl.status" "{\"data\": {\"id\": \"aff9e1c0-48d2-4979-b1d6-2a83f8a6dcf6\"}, \"event_type\": \"task_completed\"}"
1460564262.507267 [1 <IP_ADDRESS>:48027] "HSET" "ctl.ids.result" "1e623404-3121-45a7-9c60-724e620d069c" "{\"timestamp\":<PHONE_NUMBER>.507146, \"state\": \"PENDING\", \"id\": \"1e623404-3121-45a$-9c60-724e620d069c\", \"elapsed\":<PHONE_NUMBER>.507149}"
1460564262.507585 [1 <IP_ADDRESS>:48027] "PUBLISH" "ctl.status" "{\"data\": {\"id\": \"1e623404-3121-45a7-9c60-724e620d069c\"}, \"event_type\": \"task_created\"}"
1460564262.507820 [1 <IP_ADDRESS>:48027] "PUBLISH" "ctl.status" "{\"data\": {\"id\": \"1e623404-3121-45a7-9c60-724e620d069c\"}, \"event_type\": \"task_started\"}"
1460564262.509007 [0 <IP_ADDRESS>:40544] "SUBSCRIBE" "celery-task-meta-1cbfd370-29a7-4ade-b417-03f5bca62ab7" <----- BAD SEQ
1460564262.511634 [1 <IP_ADDRESS>:43483] "HKEYS" "ctl.resource.appliance"
1460564262.512055 [0 <IP_ADDRESS>:41200] "MULTI"
1460564262.512071 [0 <IP_ADDRESS>:41200] "SETEX" "celery-task-meta-1cbfd370-29a7-4ade-b417-03f5bca62ab7" "3600" "{\"status\": \"SUCCESS\", \"traceback\": null, \"result\": {\"ids\": [\"146596$8-5f98-4949-84cb-11c95aee92e7\"]}, \"children\": []}"
1460564262.512109 [0 <IP_ADDRESS>:41200] "PUBLISH" "celery-task-meta-1cbfd370-29a7-4ade-b417-03f5bca62ab7" "{\"status\": \"SUCCESS\", \"traceback\": null, \"result\": {\"ids\": [\"146596c8-5f$8-4949-84cb-11c95aee92e7\"]}, \"children\": []}"
1460564262.512148 [0 <IP_ADDRESS>:41200] "EXEC"
1460564263.202898 [1 <IP_ADDRESS>:48027] "HSET" "ctl.ids.feed" "cfd3c05d-ff1e-4b43-925d-f741e213dc6a" "{\"timestamp\":<PHONE_NUMBER>.202797, \"id\": \"cfd3c05d-ff1e-4b43-925d-f741e213dc6a\", \"c$annel\": \"ctl.status\"}" <----- OTHER REDIS CMDS
1460564263.207252 [1 <IP_ADDRESS>:48027] "HSET" "ctl.ids.feed" "cfd3c05d-ff1e-4b43-925d-f741e213dc6a" "{\"timestamp\":<PHONE_NUMBER>.207148, \"id\": \"cfd3c05d-ff1e-4b43-925d-f741e213dc6a\", \"c$annel\": \"ctl.status\"}"
1460564263.211087 [1 <IP_ADDRESS>:48027] "HSET" "ctl.ids.feed" "cfd3c05d-ff1e-4b43-925d-f741e213dc6a" "{\"timestamp\":<PHONE_NUMBER>.210988, \"id\": \"cfd3c05d-ff1e-4b43-925d-f741e213dc6a\", \"c$annel\": \"ctl.status\"}"
1460564263.214782 [1 <IP_ADDRESS>:48027] "HSET" "ctl.ids.feed" "cfd3c05d-ff1e-4b43-925d-f741e213dc6a" "{\"timestamp\":<PHONE_NUMBER>.214685, \"id\": \"cfd3c05d-ff1e-4b43-925d-f741e213dc6a\", \"c$annel\": \"ctl.status\"}"
1460564263.218434 [1 <IP_ADDRESS>:48027] "HSET" "ctl.ids.feed" "cfd3c05d-ff1e-4b43-925d-f741e213dc6a" "{\"timestamp\":<PHONE_NUMBER>.21834, \"id\": \"cfd3c05d-ff1e-4b43-925d-f741e213dc6a\", \"ch$nnel\": \"ctl.status\"}"
1460564263.221910 [1 <IP_ADDRESS>:48027] "HSET" "ctl.ids.feed" "cfd3c05d-ff1e-4b43-925d-f741e213dc6a" "{\"timestamp\":<PHONE_NUMBER>.221821, \"id\": \"cfd3c05d-ff1e-4b43-925d-f741e213dc6a\", \"c$annel\": \"ctl.status\"}"
1460564266.127377 [0 <IP_ADDRESS>:40544] "UNSUBSCRIBE" "celery-task-meta-1cbfd370-29a7-4ade-b417-03f5bca62ab7" <----- BAD SEQ

I should further note that tornado is reporting a GeneratorExit exception at the location where I am yielding to the task.
I don't know if the issue is with tornado-celery or one of the modules it is using (e.g., pika, etc.)

pip list
anyjson (0.3.3)
backports.ssl-match-hostname (<IP_ADDRESS>)
billiard (<IP_ADDRESS>)
celery (3.1.18)
certifi (2015.4.28)
kombu (3.0.26)
pika (0.9.14)
pip (7.0.3)
pytz (2015.4)
redis (2.10.3)
setuptools (17.0)
tornado (4.2)
tornado-celery (0.3.5)

i have same problem, and the pika is 0.9.14

This is due to a race condition. When I have some cycles I'll document the conditions.


Make sure editor font size is large enough on mobile

I'm not sure how to detect a mobile device reliably.


Option is lowercased upon decorating function

pip list
Package        Version

...
click          6.7
...

@click.command()
@click.option("--someOption", help="Help text")
def test_fun(someOption):
    print("Test print!")

if __name__ == "__main__":
    test_fun()

This will give an error:
Traceback (most recent call last):
  File "AutoGenerateVoicePromptPackage.py", line 9, in <module>
    test_fun()
  File "C:\Python27\lib\site-packages\click\core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "C:\Python27\lib\site-packages\click\core.py", line 697, in main
    rv = self.invoke(ctx)
  File "C:\Python27\lib\site-packages\click\core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "C:\Python27\lib\site-packages\click\core.py", line 535, in invoke
    return callback(*args, **kwargs)
TypeError: test_fun() got an unexpected keyword argument 'someoption'

As you can see, it expects the casing to be "someoption" - and not "someOption" as I specified it.
I'm running this on Windows 10 with Python 2.7.14 - could that be a factor?

I think this is expected behavior see this line in core.py
What specifically are you trying to accomplish?

This is wholly undocumented. I see the line of code and that is how I figured out it wasn't me that was causing the problem - but it is no documented.
Yes, all the examples use lowercase, but nowhere I can find does it say only lowercase is allowed (or that it is converted into lowercase).

https://github.com/pallets/click/pull/1055

Resolved in #1055


Exception thrown at 0x18E789F8 (vgui2.dll) in csgo.exe: 0xC0000005: Access violation reading location 0x77C38AAC.

I know one of yall mfs got the commit with the fix.  I fixed and now it's just crashing $$$

nullptr at line 43

Ok don't be mean :(

dude, we can't help you if we don't see any code

Please I will help later ahhhh


kubectl incorrect path bug fix, closes #28

Fixed the incorrect path bug detailed here: https://github.com/Azure/k8s-bake/issues/28.
The kubectl tool name was not being appended when the version was already cached and didn't need to be downloaded.

@OliverMKing Thanks for the PR 👏 !! We made some changes to the way integration tests run from forked repos. Can you pull the upstream changes in your branch so that the checks become green?

@OliverMKing Thanks for the PR 👏 !! We made some changes to the way integration tests run from forked repos. Can you pull the upstream changes in your branch so that the checks become green?

Done! Should be ready for a merge now. Thanks for your help 😄

@OliverMKing I will cherry pick this to the releases branch and updated the v1 version of this action tomorrow. Post that the fix should be available in the v1 version


music, camera: volume & zoom slider tooltips

This PR adds the following tooltips to the option sliders for the music and camera plugins:

This PR also adds transparency to the music slider buttons when dragging, which mimics the behaviour of the zoom slider in the official client.
Edit: This feature is no longer a part of this PR.
Closes #11056 (minus the request for a tooltip for the brightness slider, which only has 4 states anyway).

I've managed to make the Music plugin's custom sliders work well with the suggested setOnMouseRepeatListener()-based approach, but I can't quite get the vanilla client's zoom slider to work the same way.
With the vanilla zoom slider, the script set using setOnMouseRepeatListener() stops being run when dragging the handle, even when the it is set on another widget such as its track instead. I can't quite figure out why this is happening (Jagex does something special here?), so for now I might settle with the overlay-based approach for the Camera plugin.
Any ideas?


Do not check that the file got smaller

I know the purpose of this plugin is to optimize images, and that usually means making them smaller.
But some plugins, in particular SVGO, sometimes make the image larger on purpose. In particular, some of its plugins add data to the SVG. For example, the prefixIds plugin prefixes class names and IDs in the SVG so that many SVGs with similar IDs on their elements can be used inline in a single document without conflicting with each other. This necessarily adds something to the filesize.
Often the amount added is less than the amount taken away by other optimizations, and so everything seems fine. But sometimes the input SVG was already sufficiently optimized that more is being added than taken away.
In these cases, imagemin-webpack-plugin was discarding the optimizations, and the original image was left alone. This can lead to all sorts of bugs (IDs colliding, elements within the SVG not being addressable since you expected them to be prefixed, accessibility attributes not having been added to the SVG element, wrong display size due to viewBox not being added, ...), and these are very frustrating and difficult to trace.
imagemin-webpack-plugin shouldn't care about whether the image got smaller. That corcern lies solely with the individual optimization tools like SVGO, Optipng, etc. Instead of testing for a smaller file in imagemin-webpack-plugin, it's better to file bugs against any of those tools which sometimes output larger files even when undesirable.

I thought the bug was with SVGO (see https://github.com/svg/svgo/issues/991) when I first came across this issue a year ago. Only today did I finally trace it to imagemin-webpack-plugin, after @missmatsuko was facing the same issue.
I propose removing this check, as implemented in this patch.

Thanks for the PR!
I'd be willing to have this as an option (maybe even as the default option), but not as a built-in choice.
This is pretty vital behaviour for some, especially when used with plugins that convert images from one format to another.
Monday morning I have some time scheduled to touch a few things here, so I'll be able to take a better look at this then.

Disagree. I wouldn't expect a webpack plugin for imagemin to change the
core functionality of imagemin. If you want it as an option, you should
make a PR to imagemin itself, or in an imagemin plugin.
On Sat., Jul. 27, 2019, 7:25 a.m. Gregory Benner<EMAIL_ADDRESS>wrote:

Thanks for the PR!
I'd be willing to have this as an option (maybe even as the default
option), but not as a built-in choice.
This is pretty vital behaviour for some, especially when used with plugins
that convert images from one format to another.
Monday morning I have some time scheduled to touch a few things here, so
I'll be able to take a better look at this then.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Klathmon/imagemin-webpack-plugin/pull/89?email_source=notifications&email_token=AEIOIJTS236VXYFJOIDKEFDQBRLENA5CNFSM4IHGX2IKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD26MM3A#issuecomment-515688044,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEIOIJVXCGO7ZGBQL57ETPLQBRLENANCNFSM4IHGX2IA
.

How about as a compromise I bump the version of the plugin to 3, change the default settings so that the plugin will no longer check final size, and add an option to re-enable that functionality for those who want it?
I'm gonna close this here and move the discussion to #90.

Sounds good to me. Thanks.


feat: port Analysis.BoxIntegral.Box.Basic

Remaining errors:

[ ] Lots of @[norm_cast] errors.
[ ] le_TFAE uses unknown tactics (tfae_have, tfae_finish)
[ ] coe_inf

bors r+


🌱 Disable dependabot for hack/* modules

What this PR does / why we need it:
It's not necessary to update dependency for internal tools so frequently. So, it's better to disable dependabot for them, or otherwise it creates to many PRs.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

It's not necessary to update dependency for internal tools so frequently. So, it's better to disable dependabot for them, or otherwise it creates to many PRs.

If it is just too frequent but we still need updates, we could increase the interval of updates done by the bot and keep it running, otherwise, let's remove it completely

can we do monthly updates? For test package also maybe

/approve


Expand setup configuration

Would it be possible to expand the configuration page to allow us to disable certain aspects of the plugin. For example, the Temperature Sensor must not be available on my Kia Stinger. I have the Kia Connect Ultimate and the only temperature reading I ever see is 32 degrees and it never changes.
Great work on the Plugin by the way. This is really cool.

Just to elaborate, homebridge is receiving a NaN (number) from my Stinger for the Temperature reading.

Agreed - would also be good to extend the remote start time to 10 minutes from 5 minutes. It looks like there's a setting in client.ts that would allow this extension. I also note that the Home Assistant plugin lets you choose 10 minutes. I like the ease of this plugin vs the Home Assistant one, so I haven't fully moved over yet.

I had another idea for the plugin. Would it be possible to take the battery status percentage and create something like a light sensor so we could use the percentage in an automation to warn us when the battery drops below a certain percentage. I already had to change my car battery after one year (strange) so I’ve been watching this data. It’s just buried in the engine accessory settings

Hey @johnsills1 and @asabatino88 sorry for the delay. I think these would be great additions to the plugin. I think all we would need to do is add some fields to the config.schema.json file so they can be filled in by the user upon installation, then we should use those as parameters throughout.
We should take the opportunity to add some tests here. I might work on this shortly!

@johnsills1 let me know what you think of #11

Looks good to me. I’m not a code writer, but I love beta testing, haha
John Sills
On Tue, Mar 12, 2024 at 7:57 PM Trevor Hutto @.***>
wrote:

@johnsills1 https://github.com/johnsills1 let me know what you think of
#11 https://github.com/huttotw/homebridge-kia-connect/pull/11
—
Reply to this email directly, view it on GitHub
https://github.com/huttotw/homebridge-kia-connect/issues/8#issuecomment-1992893933,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABOLG7ZRZJBX5YWHRM67KD3YX6QAFAVCNFSM6AAAAABDRWEAEWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOJSHA4TGOJTGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

For what it's worth, I'm also experiencing the NaN on the external temperature sensor (which means it just defaults to 32 degrees all the time). I tried to comment out all of the external temperature sensor code in my forked code base, but wasn't able to stop it from showing up as an accessory in Homebridge. I'm not a dev but have done some basic coding over the years, so I was likely just missing something when playing around with it.

How are the updates coming? I am hoping we can expose the battery as a light brightness sensor so we can utilize it in automations without jumping through hoops with the Eve, Controller for HomeKit and Home apps. Thanks

Have you had time to work on this?


Include Dockerfile.dev and docs to run neuroglancer dev locally

@kabilar -- in this PR -- just extracting form https://github.com/lincbrain/neuroglancer/pull/2 the Dockerfile.dev aspects that I'd like to preserve for now

@kabilar -- thanks for the review here -- I updated the Dockerfile.dev and merged


Cant start live-share, refresh loop

Version: 1.45.1 (user setup)
Commit: 5763d909d5f12fe19f215cbfdd29a91c0fa9208a
Date: 2020-05-14T08:27:35.169Z
Electron: 7.2.4
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.19041
Steps to Reproduce / Scenario:

clean install visual studio code

install live share extension

sort of refresh loop/flickering left bottom says "Extension 'live share' added 1 folder to workspace"

removing extension and starting visual code, it visual studio works fine again.
(%USER%/.vscode/extensions)

go back to 2 (when installing live share extension again.)

If i open a file, it doesn't start in the loop, if i try to join a liveshare link, it starts.
If i only start visual studio without any files, it immediately

Thanks for reaching out. I tried to repro the scenario you described but couldn't reproduce it. Your logs may provide us with helpful information. You can access them via the Live Share: Export Logs command from the command palette (ctrl + shift + p) and attach them to this issue.

Here is the log file.
LiveshareLog.zip

Hey @Rikkamerbeek are you still seeing this issue?

Closing this as resolved, since we believe we've fixed this issue and we haven't heard it reported it in a while. Please re-open this if you're still encountering a problem here. Thanks!


Some scope watch may caused angular dirty check goes wrong.

In Line 481:
 _bindEvents: function () {
                    var that = this;
                    this._scope.$watch('[ngc.alignMiddle, ngc.offsetTop, ngc.offsetBottom]', function () {
                        that.setDialogCenter('bindEvents');
                    });

due to angular offical document (https://docs.angularjs.org/error/$rootScope/infdig ),those code may caused dirty check error.
I modified to this.
_bindEvents: function () {
          var that = this;
          var boxSets = [this._scope.ngc.alignMiddle, this._scope.ngc.offsetTop, this._scope.ngc.offsetBottom];
          this._scope.$watch(boxSets, function () {
            that.setDialogCenter('bindEvents');
          });

Thanks for this, will make the changes soon

same error here, woodu's solution works

from v1.1.0
ngConfirm wont use scope for internal operations,
this caused problems in memory leaks and circular references.
previous code is completely removed.


[Docs]: Split Ubuntu pages in separate files

Type
Bug report
What do you want to report?
The update process for the images generates markdown documentation, one per OS version. So it's easier to maintain these pages as separate.
The proposal is to separate OS Ubuntu page into these pages:

ubuntu-18.04-image.md
ubuntu-20.04-image.md
ubuntu-22.04-image.md
ubuntu-22.04-arm-image.md
ubuntu-24.04-image.md

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Pushed to prod


test,win: fix compilation with shared lib

Tests were failing to link because of undefined snprintf symbol with
VS < 2015 and using shared library.
snprintf is implemented in libuv in src/win/snprintf.c when compiling
with VS < 2015, so this commit add src/win/snprintf.c to test sources
to make snprintf available in tests.
Fixes: #676

LGTM

Landed in ad2cc8f, thanks!


cannot find -lddsc&ddscxx

my system is ubuntu20.04
I have installed the unitreeSDK2 on my computer
the path is /usr/local/
but there is an error about cannot find -lddsc&ddscxx occurred

@Sher1ockFan I'm experiencing the same issue. Did you find a solution?

Same issue here, the -lddsc error could be solved by apt install cyclonedds-dev. However the -lddscxx still remains unfound. So I'm suspecting the problem is somehow related to building procedure of cyclonedds or the environment variable.

I don't know if the maintainer of the repository made a deliberate mistake or is not familiar with modern cmake.The solutions to these errors are:

Thanks so much for the solution! It worked flawlessly.

@Sher1ockFan I'm experiencing the same issue. Did you find a solution?

I found the installation path was wrong
It was /usr/local/lib/x86_64/ but it should be /usr/local/
So you can move libddsc.so and libddscxx.so
/ to the correct path
sudo mv /usr/local/lib/x86_64/* /usr/local/lib/

@Sher1ockFan I'm experiencing the same issue. Did you find a solution?

I found the installation path was wrong
It was /usr/local/lib/x86_64/ but it should be /usr/local/lib
So you can move libddsc.so and libddscxx.so to the correct path
sudo mv /usr/local/lib/x86_64/* /usr/local/lib/

It looks like a bug in the unitree_sdk2 cmake install script, there is a missing /, which causes the prefix. You can add this / and it will install to /usr/local.

编译unitree_mujoco时出现，cannot find -lddsc&ddscxx #19
#查找libddscxx.so
locate libddscxx.so
#将libddscxx.so的位置链接到/usr/local/lib/libddscxx.so
sudo ln -s /home/liu/unitree_sdk2-main/thirdparty/lib/x86_64/libddscxx.so /usr/local/lib/libddscxx.so
locate libddsc.so
sudo ln -s /home/liu/unitree_sdk2-main/thirdparty/lib/x86_64/libddsc.so /usr/local/lib/libddsc.so

my system is ubuntu20.04 I have installed the unitreeSDK2 on my computer the path is /usr/local/ but there is an error about cannot find -lddsc&ddscxx occurred

The previous version of unitree_sdk2 was installed using install.sh. This method is no longer used in the new version. See https://github.com/unitreerobotics/unitree_sdk2. The installation paths of these two versions are inconsistent, resulting in compilation errors.
We have fixed this issue in the latest submission.


Move to slf4j with logback

Move logging to slf4j, any exceptions should be sent by email.
Use markers/appenders for serious issues
http://logback.qos.ch/manual/appenders.html

fixed with a2081577f3922d1b1b3cc0fd9ce09da0d2fa73fb


Firebase messaging get token returns null (firebase_messaging 5.0.2)

After updating our flutter application from Android to Android X we are having some issues with firebase messaging (firebase_messaging 5.0.2). When calling get token in our flutter application we are getting null on all Android version below or equal to 4.4.
We are currently using flutter version 1.5.4-hotfix.2.
Could someone please help with this issue?
Best regards

Hi @GervaisYO
re you still experiencing this issue with the latest version of Flutter?
If so please reply attaching your flutter doctor -v

Hi @GervaisYO
for what I can see the issue is related to a FlutterFire plugin,
please open the issue in the dedicated github.
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
thank you


Bump golangci-lint to v1.50.1

Signed-off-by: Luca Comellini<EMAIL_ADDRESS>
Bumps golangci-lint from v1.49.0 to v1.50.1

Thanks!


feat(turborepo): Watch mode

Description
Implements an MVP version of watch mode that takes the package changes stream and re-runs package tasks as needed.
Testing Instructions

Closes TURBO-2519

#7613  👈
nicholasyang/package-watching

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NicholasLYang and the rest of your teammates on  Graphite

How do we use it? :)