Proxy not working as well

Hi.
I use MTProto proxy. The problem I have is that I can't change proxies frequently. If a proxy does not respond and I want to change it, I must restart my application once.(or require reopen client)
what is the problem?
I use AddProxy Method
also I use DisableProxy method to disable my proxy

You can use enableProxy and disableProxy whenever you want. The proxy settings are applied immediately, there is no need fot application restart.

Before the user logs in, I create a client called `pingClient 'to receive the status of the Internet network.  But the problem I have is that it doesn't show the status of the network properly.  When does the network status change properly and at the right time?  What is the problem?

@BehzadArabi Why do you think that there is a problem?

Because the user cannot know the exact status of the network.When will the network change momentarily?

All status changes are for informational purposes only to be shown to the user. They can be received any time.


Update Israeli phone prefixes

According to: https://en.wikipedia.org/wiki/Telephone_numbers_in_Israel#Cellular_and_mobile_devices_area_code_05
Additions:

051
079

Removal:

057

Coverage increased (+0.0005%) to 99.369% when pulling 92cab12cad3f396cc2c219330016c92e626e2510 on sergeydgr8:fix_israeli_numbers into 41da50a39fee9944f34ed508a7962a66fdc4d1ab on floere:master.

Coverage increased (+0.0005%) to 99.369% when pulling 92cab12cad3f396cc2c219330016c92e626e2510 on sergeydgr8:fix_israeli_numbers into 41da50a39fee9944f34ed508a7962a66fdc4d1ab on floere:master.

Coverage increased (+0.0005%) to 99.369% when pulling 92cab12cad3f396cc2c219330016c92e626e2510 on sergeydgr8:fix_israeli_numbers into 41da50a39fee9944f34ed508a7962a66fdc4d1ab on floere:master.

Hi @sergeydgr8 – thanks so much for this neatly documented and put together PR! 🙇 Released in 2.17.3 🚀


Improve SimpleCSSValuesImpl

Currently SimpleCSSValuesImpl gets style values like so:

returns the value from the underlying style if it exists.
Walks up the parent tree and returns the parent value if it's inherited.*
I uses a lookup for either css class names or Class names to get assigned style info
The Class names are saved in compiled MXML and added to the SimpleCSSValuesImpl by a call to ValuesManager.valuesImpl.init(this) in any class which implements IMXMLDocument
If the object is a class, it checks the qualified name and uses that to look up the styles.
When it doesn't find the styles, it will walk up the super-tree and does the same for each of the super-classes.

This has multiple issues:

Getting styles can be fairly expensive. As mentioned by @estanglerbm https://lists.apache.org/thread/qb1x76ohszmllkflzrq3mg70gy20y6tv The style lookups can be made much more efficient.
Requiring lookups based on qualified class names requires keeping these names around in minified code for every class in the application.
There's no way to cache known values to make lookups easier. (related to 1)
The code is pretty convoluted and can be made cleaner.

Additonally there's some issues identified here:
https://github.com/apache/royale-asjs/issues/641#issuecomment-573836029
An example of a compiled MXML lookup is this:
0,
1,
"org.apache.royale.core.Application",
function() {this["MozUserSelect"] = "none";
this["MsUserSelect"] = "none";
this["WebkitUserSelect"] = "none";
this["margin"] = 0.0;
this["padding"] = 0.0;
this["userSelect"] = "none"},
0,
1,
"org.apache.royale.html.Container",
function() {this["alignItems"] = "flex-start";
this["iBeadLayout"] = org.apache.royale.html.beads.layouts.BasicLayout;
this["iBeadView"] = org.apache.royale.html.beads.ContainerView;
this["iViewport"] = org.apache.royale.html.supportClasses.OverflowViewport;
this["iViewportModel"] = org.apache.royale.html.beads.models.ViewportModel},

Here's a potential improvement:

Instead of using "org.apache.royale.html.Container", the value can be written unquoted to refer to the class directly like we do for bead references. This will enable to compiler to minimize the references normally.
This means that you can't add the class names to a hash in the valuesImpl.
Instead, we can add to IStyleableObject setStyle() and getStyle() That would put the logic of saving the style information and retrieving it in the class itself.
setStyle() and getStyle() would be instance methods, but it would save the information to the class so it's written once.
Additonally, getStyle can cache info it gets from the super-class so subsequent lookups would not need to walk up the hierarchy again. **
We might have additional methods for saveCacheValue and getCacheValue for optimizing lookups when walking up parents if it's known that it's safe to do so. These values would need to be saved in instances for it to work. The right balance of memory usage and speed optimization would need to be found for this.

It looks like there's an inefficiency in step 2 where it will walk up the tree multiple times if the value is not set anywhere up the tree. That's because both getValue and getInheritedValue is called recursively in getInheritedValue. The current parent should probably be saved as an instance variable to prevent having to walk up the tree many times.

need to figure out if caching the super-values might be an issue for modules.

After thinking about this some more:

It seems like instead of adding to IStyleableObject, we should add a new interface (ICSSStyleable) which inherits from IStyleableObject.
I'm not sure if UIBase should implement ICSSStyleable or there should be a subclass of that which does.
There should probably be 5 distinct methods in ICSSStyleable.

saveStyle (or similar) would save a default for the class.
findStyle would get a style from the defaults walking up the inheritance if necessary. The results should be cached so the walk would only happen once per style per class.
findInheritedStyle would call find style up the parent chain until it finds a style. The parent chain results would not be cached by default. We'd need to figure out whether caching is ever a good idea.
setStyle would apply a style to an instance. In the case of HTML, this should apply the change to the element's inline css.
getStyle would get the directly applied style of the instance (if any).

In MX/Spark setStyle and getStyle already have defined meanings. We'd need to figure out how/if that fits with the above. mx.core.UIComponent might need a different implementation. This might be a reason to make the implementation a subclass of UIBase.
For many current uses of ValuesManager.valuesImpl.getValue(), there is no need to walk up the parent chain (such as finding default beads). For such cases simply calling findStyle() would be enough.
It's worth researching whether we can/should have an option to avoid style lookups for cases where it's not needed (such as HTML elements where it's known that styling is applied by HTML CSS). This might provide significant performance improvements.
It's worth researching how expensive it currently is to get an element's parent. It currently uses the DOM for HTML elements to get the DOM element's parent. The rule of thumb is that DOM operations are much more expensive than native JS. It might be worthwhile looking into whether keeping track of the parent is something that can improve performance. If yes, this would need to be done carefully to make sure that parent references don't get out of sync.

I'd love to hear others' thoughts on all of this.

I haven't measured specifically, but a single DOM parentNode is pretty quick (it's not an expensive traversal and doesn't modify the DOM).
I think the recursion (classes + containers) of each getStyle() call is more the issue, in general, since getStyle() is called so much.

I'd recommend modifying a version of SimpleCSSValuesImpl so that it can log all calls to getStyle so we truly understand why it is being called.  If 99% is for bead lookup, then optimize bead lookup.  There shouldn't be many, if any, calls to getStyle in JS.  Also, the parent shouldn't be involved if the style is not inheriting.


Add font-size in card styles

it would be nice when i can set the font size in the card via styles

Thank you for the idea :heart: This will be a part of v2.7.0
type: custom:meteoalarm-card
entities:
  - entity: sensor.32_weather_alert
integration: meteofrance
scaling_mode: disabled
card_mod:
  style: |
    ha-card {
      --headline-font-size: 12px;
    }


Bump used RPC to version 0.7.0

Closes #1857
Introduced changes

Bumped used RPC version to 0.7.0-rc2

Checklist

[x] Linked relevant issue
[x] Updated relevant documentation
[x] Added relevant tests
[x] Performed self-review of the code
[ ] Added changes to CHANGELOG.md

Blocked by https://github.com/foundry-rs/starknet-foundry/pull/1851. Changes made in https://github.com/foundry-rs/starknet-foundry/pull/1851 are required for this to pass.


go/types, types2: typeparams: sanitize Scopes, etc

Reminder issue: unexpanded *instance types, which are meant to exist only within the type checking pass, are currently leaking in a number of places, most notably within Scopes.
It might be possible to avoid sanitization entirely, by packing the *instance data into *Named, and nilling out upon expansion, similar to what is done in https://golang.org/cl/318849.
CC @griesemer

This would be addressed by https://golang.org/cl/328153.


Enable calling context on Option by adding an OptionExt trait.

I am not sure I like the idea of treating Option<T> as Result<T, E> as this is not the purpose of it. Would you mind to explain your motivation behind it?

Good, I understand your motivation. My question now is if it makes sense to use Option<T> if None is an error. Instead, it seems it should use Result<T, failure::Error> and then return the Error when None would be used.
When I see Option<T> it communicates to me that None is valid. If it is not, I think we should not use Option<T>. Maybe I am missing something?

When it comes to https://doc.rust-lang.org/std/path/struct.Path.html#method.to_str , please add the Rust development team :)
When it comes to my usecase, yes, Option is exactly what I need. There is either a session or there is no session. If a user tries to execute an action, which requires an existing session, I need to report a human-readable error.

Ok, I'll change that to ok_or_context

Is there anything that prevents this from being merged?

I'm not entirely opposed but also not fully convinced yet that this is a good idea. Does anyone else have opinions on this?

I'd be very happy with it, I've run into the same verbosity issue many times, and the ability to quickly and conveniently say "This why None is an error" is highly necessary.

I'm in favor of this change, or something that looks a lot like this. This is something that I've run into often.

Just adding my experience that yes, this comes up a fair amount in practice when you're calling a function for which None is a valid outcome for the function but is an error for you, the caller. Also, aside from the awkward naming I don't actually see why FailureExt<T, ()> couldn't be implemented for Option<T>.
The alternative is the slightly verbose maybe_foo().ok_or(()).context("whatever")?, relying on () being a valid argument for Context.

Is there anything that prevents this change from being merged?

Since there is no response from the failure maintainers, I created a crate which provides ok_or_context for Option (and, as an opt-in feature, context for futures::Future)
https://github.com/marmistrz/failure_ext
https://crates.io/crates/failure_ext


Tables in markdown preview not rendered correctly, no bord for table

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

VS Code Version: 1.63
OS Version: win10

Steps to Reproduce:
1.Create a new file in VSCode and paste above markdown in it and save the file with a name with extension .md.
2.Right click on the file tab en choose Open Preview.

Table example

Column 1
Column 2
Column 3

Row_1_1
Row_1_2
Row_1_3

Row_2_1
Row_2_2
Row_2_3

Row_3_1
Row_3_2
Row_3_3

but preview as   ##following:
#85183


Fixed minor typos

Mega typo fix PR, review pleasee @rthadur @dynamicwebpaige ?????

@kyscg please rebase your branch

Where should I rebase it to? I'm new to this, sorry if I'm holding up stuff

Where should I rebase it to? I'm new to this, sorry if I'm holding up stuff

rebase to master , i mean resolve conflicts.

Hi, why isn't this being merged, should I make anything better? @rthadur @chsigg

@rthadur Gentle ping to pull, all changes have been approved

It's not being merged because tests fail (the exception check is case sensitive). I will fix it and submit.


Fix work_history back link

Fix work_history back link in 'edit_has_work_history' by adding logic to check if the application_form is empty

 Work history back link broken


[P0] Error at the end of deployment script running on Windows / AKS EE

Successfully installed MEC-Accelerator!
Alerts-UI and Control-Plane-UI services deployed in:
error: error parsing jsonpath | Control Plane web app URL (control-plane-ui-service pod) | http://{.status.loadBalancer.ingress[].ip}:{.spec.ports[].port} |{\n}, unrecognized character in action: U+005C ''
error: error parsing jsonpath | Alerts Dashboard web app (alerts-ui pod)                 | http://{.status.loadBalancer.ingress[].ip}:{.spec.ports[].port} |{\n}, unrecognized character in action: U+005C ''

Fixed with this PR
https://github.com/Azure/mec-app-solution-accelerator/pull/167
and now finishes like this:

close when verified from your side, Victor.


tests: Reduce context timeout to avoid false positives

context timeout tests had long timeouts leading
to false positives in some cases when functional
tests are run against faster disks.
Reduce them such that the possibility of this
would never occur in future usage.
Fixes #880

function test check on Travis failed for the following
{"alert":"","args":{"bucketName":"minio-go-testxns0okpetpou4rs6k","ctx":{"Context":0},"objectName":"0n1i3humrguu2dnrzrtgg0vaf49g3z"},"duration":156,"function":"GetObjectWithContext(ctx, bucketName, objectName)","message":"GetObjectWithContext failed due to non-cancellation upon short timeout","name":"minio-go","status":"fail"}

The PR is updated and fixed the context usage properly.

Any other concerns other than the typo ?

fixed comments PTAL @kannappanr @poornas


[FEATURE REQ] Add Windows and Mac OS to GitHub workflow

Is your feature request related to a problem? Please describe.
The tests GitHub workflow currently only tests on ubuntu-latest. This can lead to some platform specific bugs being ignored (such as this. It could have been prevented by running the tests on Windows automatically.
Describe the solution you'd like
Refer this page of the GitHub actions docs and add the required platforms (Windows and Mac OS).
Additional context
The [CI] tag has to be added to the commit.

This has been worked on in the feature-cli-platforms branch.


unix,win: add uv_udp_try_send2

Add a version of uv_udp_try_send that can send multiple datagrams.
Uses sendmmsg(2) on platforms that support it (Linux, FreeBSD, macOS),
falls back to a regular sendmsg(2) loop elsewhere.
This work was sponsored by ISC, the Internet Systems Consortium.

The first commit is a NFC that is prep work for the second commit. Same line count, more functionality, improved legibility (IMO)
I moved uv__udp_sendmsg to the bottom of udp.c because the diff is crazy hard to read otherwise.
I don't love the name uv_udp_try_send2 but I guess it's convention by now. Can I see hands for uv_udp_try_sendv? uv_udp_try_send_many? Other suggestions?

don't love the name uv_udp_try_send2 but I guess it's convention by now. Can I see hands for uv_udp_try_sendv? uv_udp_try_send_many? Other suggestions?

I like uv_udp_try_send2 and uv_udp_try_sendv. The former is a more used convention, and I don't recall us having any instances of the latter, even if APIs like uv_write take multiple buffers, so there is that...

Okay, uv_udp_try_send2 it is then. Any objections to me merging this?


sort paths to enable iac scan all caching

When receiving tar files in GIM, we first hash the tarfile bytes content to generate a key used for caching.
The goal is to not scan the same tar file twice.
However, the order of bytes are important and if we do not generate the tarfile using always the same path order, the tarfile will have different bytes order (event if the uncompressed directory is the same)

Codecov Report

Merging #626 (230f25b) into main (35c6e2b) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #626      +/-   ##
==========================================
- Coverage   93.98%   93.98%   -0.01%
==========================================
  Files         103      103
  Lines        4792     4805      +13
==========================================
+ Hits         4504     4516      +12
- Misses        288      289       +1

Flag
Coverage Δ

unittests
93.98% <100.00%> (-0.01%)
:arrow_down:

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

Impacted Files
Coverage Δ

ggshield/cmd/iac/scan/scan.py
100.00% <100.00%> (ø)

... and 6 files with indirect coverage changes


How to set value of Name tag in DacMetadata.xml file ?

How to set value of Name tag in DacMetadata.xml file in DACPAC file ? I managed to set name of the output .dacpac when it is built by setting the property <AssemblyName>, but value in DacMetadata.xml is still the project name.
I tried setting properties:
Name, OutputAssembly, TargetName, SqlTargetName, RootNamespace, DacpacName
but none of this affected the value in DacMetadata.xml.
Is there a way to set Name via project file ?
My environment SDK: MSBuild.Sdk.SqlProj/2.6.0

I did some digging and it looks like we are using the built-in MSBuild property called MSBuildProjectName as the name of the package. This should be the name of the .csproj file without the extension. So renaming your project file should result in a different Name tag in the DacMetadata.xml.

Is there a reason you would want these to be different? At least now there is a bit of traceability between the project and the resulting .dacpac. Adding a separate property for this would be trivial, but it is an additional things to maintain so I'm just wondering what the use case is.

@AlexETF Any comments?

Closing as no response with additional details explaining the connection between project name, .dacpac name and name tag was provided.
@AlexETF FYI


Add open_recent option to VS Code desktop

Together with https://github.com/coder/vscode-coder/pull/272 closes https://github.com/coder/vscode-coder/issues/241
This also (accidentally) fixes what I would imagine is a bug where if there is no folder, it would exclude the URL.

@code-asher this will only work when using the VS Code desktop module. Do we need to do something to also support this with the Coder built-in VSCode desktop button?

Do we need to do something to also support this with the Coder built-in VSCode desktop button?

I asked the same question here: https://github.com/coder/vscode-coder/issues/241#issuecomment-2113600314
I am not sure if we have any way to customize the built-in button so I think we would have to either make this a default or add options for this, but I feel like we should encourage folks to use the module anyway.

We decided to make openRecent the default for the built-in app.  I think that makes sense, since you cannot configure folder for that one anyway (it always uses the agent directory, I believe).


Add keyboard shortcuts for queue up/down

I don't have the tooling on my system to be able to compile this code to test it, but I think this should work (following the example of delete keyboard shortcuts).
The reason I chose Alt-Up/Down instead of Ctrl-Up/Down is because Ctrl-Up/Down seems to already be reserved by wxWidgets for moving the selection "outline" without changing the selection. There is a separate issue that you can't then use space bar (or some other key) to select the current outlined item, but that's something I'd want to be able to compile and test before opening a PR.
The use case for this keyboard shortcut is having many torrents in the list and wanting to easily push one or two up or down the queue without needing to do a right click and hover into a second menu each time.

Hi! I like this :smiley: The pipeline will give you artifacts you can download and try to make sure the changes work as you expect. The code looks looks correct at a glance, though!

I don't see anything obvious with the code. I can clone the code and run it this weekend and see if I find anything :+1:

Hey! Sorry for the late reply - I've ran the code and can reproduce the issue with Alt-Up but I can't figure out why it's not working... I'll see if I can dig some more but at least we both have the same issue :sweat_smile:

Unfortunately I don't have time to continue with the implementation on this feature, primarily because I don't have an easy way to compile and test locally. Closing the PR, but if anyone else wants to get this working, feel free to use the branch as inspiration.


Error when downloading images by IDs

I try to download image by ID and get the next error:
Image ID (50197884): time data '5 4 2015, 22:17' does not match format '%Y-%m-%d %H:%M'
Dumping html to: Error medium page for image 50197884.html
PixivDownloader2 version 20150427
https://nandaka.wordpress.com/tag/pixiv-downloader/
Trying dowload by memder ID give ne the same result.

what is your settings for language in the pixiv website? currently only english and japanese are supported.

Language was set to Russian.
Now I set it to English. Also I remove cookie string from ini file. Program started working. Thanks!
But when I'm downloading files by member ID, I encounter an another error. A part of files downloaded normally but on some files I got the error.
Problem occurs on different files and not every time, so I think it may be caused by pixiv webserver.
#9
Processing Image Id: 47327604
Image ID (47327604): 'NoneType' object has no attribute 'findAll'
Dumping html to: Error medium page for image 47327604.html
Page  2
Member Url: http://www.pixiv.net/member_illust.php?id=143026&p=2
Traceback (most recent call last):
File "PixivUtil2.py", line 272, in process_member
File "PixivModel.pyc", line 48, in init
File "PixivModel.pyc", line 96, in ParseImages
AttributeError: 'NoneType' object has no attribute 'findAll'
Error at process_member(): (<type 'exceptions.AttributeError'>, AttributeError("
'NoneType' object has no attribute 'findAll'",), <traceback object at 0x02D49120

)
Dumping html to: Error page for member 143026.html
Traceback (most recent call last):
File "PixivUtil2.py", line 1758, in main
File "PixivUtil2.py", line 1543, in main_loop
File "PixivUtil2.py", line 1211, in menu_download_by_member_id
File "PixivUtil2.py", line 272, in process_member
File "PixivModel.pyc", line 48, in init
File "PixivModel.pyc", line 96, in ParseImages
AttributeError: 'NoneType' object has no attribute 'findAll'
press enter to exit.

hmm, try to delete the cookie value from config.ini and set keepsignedin = 1?

I take several tests with this option and I don't encounter this error so far.
If this (or other) error occurs again, I let you know.
Thanks for quick response.


Allow overrides for default mechanisms and morphology

Like with the YAML configuration, there should be a way to progressively override the default mechanism and morphology without duplication all of the configuration files. For example, a user might change the cat.mod mechanism in their code base, while still falling back on the default mechanism in all other cases.
Proposed implementation:

Copy defaults into tmp directory within venv, merge in user files to potentially override defaults
Compile dll and return location of tmp directory to be used instead of user directory
Compute a directory hash of user directory (likely using a suitable subprocess call) to avoid unnecessary recompilation

Generally speaking, a set of cell morphologies will be associated with particular synaptic distributions and connectivity, so it makes little sense to me to have a method to change only the morphologies for a particular model. Are you thinking of the case of simulating a reduced model that inherits its synapses and connectivity from a full morphology?


base model training without se

Hi, I am currently training the base model without se-module. The reg_loss decreased from 1.031 to 0.065 while the am loss seems stay at 16.287. Isn't training without Se-module just by commenting out the following part:
if i % 2 == 1:
net = se_module(net)
net = net + shortcut
shortcut = net
print('| shortcut')
at the face_resnet.py ? Would you please help me? Thank you.

I have tried to train the base model with Se-module from the beginning this morning,  which is your original code without your pre-trained model, but still I have got the same result. The reg_loss has reduced dramatically from 1.068 while the (am) loss flatuated around 16.3
Is ti normal?

@hustzeyu hi, did you solve the problem? I have exactly the same loss behaviour


LockAtLeastFor not respected?

Describe the bug

Which version do you use: 5.13.0
Which Lock Provider: shedlock-provider-jdbc-micronaut
ShedLock configuration: see code below
ShedLock logs: see below

Expected behavior
Not sure if this is a bug or me mis-understanding.
Given the code below, I was expecting when 2 processing running in parallel, the task would only be executed once due to the LockAtLeastFor set to 1 second.
        val txManager = DataSourceTransactionManager(dataSource)
        val lockProvider = MicronautJdbcLockProvider(txManager)
        val executor = DefaultLockingTaskExecutor(lockProvider)
        val lockConfiguration = LockConfiguration(
            Instant.now(), "foo", Duration.ofSeconds(1), Duration.ofSeconds(1))

        val f = File("/tmp/log2.txt")

        val uuid = UUID.randomUUID().toString()

        val executorService = Executors.newScheduledThreadPool(1)
        val command = Runnable {
            executor.executeWithLock(
                Runnable {
                    "[${Instant.now()}|$uuid] Executed!".also {
                        f.appendText("$it\n")
                        println(it)
                    }
                },
                lockConfiguration)
        }
        executorService.scheduleAtFixedRate(command, 0, 1, TimeUnit.SECONDS)

Actual behavior
Each process is executing the same task every 1 second:
[2024-07-16T11:55:07.206796Z|531c4486-4a99-44cd-8ed7-0e99221b53af] Executed!
[2024-07-16T11:55:07.232144Z|650af8e8-68ca-4e6a-99c7-4a4a25b8ccdf] Executed!
[2024-07-16T11:55:08.211072Z|531c4486-4a99-44cd-8ed7-0e99221b53af] Executed!
[2024-07-16T11:55:08.233553Z|650af8e8-68ca-4e6a-99c7-4a4a25b8ccdf] Executed!
[2024-07-16T11:55:09.204585Z|531c4486-4a99-44cd-8ed7-0e99221b53af] Executed!
[2024-07-16T11:55:09.233596Z|650af8e8-68ca-4e6a-99c7-4a4a25b8ccdf] Executed!
[2024-07-16T11:55:10.217054Z|531c4486-4a99-44cd-8ed7-0e99221b53af] Executed!
[2024-07-16T11:55:10.236675Z|650af8e8-68ca-4e6a-99c7-4a4a25b8ccdf] Executed!
[2024-07-16T11:55:11.210968Z|531c4486-4a99-44cd-8ed7-0e99221b53af] Executed!
[2024-07-16T11:55:11.236006Z|650af8e8-68ca-4e6a-99c7-4a4a25b8ccdf] Executed!
[2024-07-16T11:55:12.209571Z|531c4486-4a99-44cd-8ed7-0e99221b53af] Executed!
[2024-07-16T11:55:12.234226Z|650af8e8-68ca-4e6a-99c7-4a4a25b8ccdf] Executed!
[2024-07-16T11:55:13.208047Z|531c4486-4a99-44cd-8ed7-0e99221b53af] Executed!
[2024-07-16T11:55:13.231663Z|650af8e8-68ca-4e6a-99c7-4a4a25b8ccdf] Executed!

I did not have time to check, but I think you have to create a new LockConfiguration instance every time you execute the method

Thanks @lukas-krecan
Change the code as per below seems to largely fix this (see logs).
There still seems to be a bit of a discrepancy in time but I'll do some more testing tomorrow.
val command = Runnable {
            executor.executeWithLock(
                Runnable {
                    "[${Instant.now()}|$uuid] Executed!".also {
                        f.appendText("$it\n")
                        println(it)
                    }
                },
                LockConfiguration(Instant.now(), "foo", Duration.ofSeconds(1), Duration.ofSeconds(1)))
        }
        executorService.scheduleAtFixedRate(command, 0, 1, TimeUnit.SECONDS)

[2024-07-17T17:50:26.383018Z|8d8a743c-69c6-48dc-b13e-628a53167b1b] Executed!
[2024-07-17T17:50:27.385348Z|8d8a743c-69c6-48dc-b13e-628a53167b1b] Executed!
[2024-07-17T17:50:28.385021Z|8d8a743c-69c6-48dc-b13e-628a53167b1b] Executed!
[2024-07-17T17:50:29.385061Z|8d8a743c-69c6-48dc-b13e-628a53167b1b] Executed!
[2024-07-17T17:50:30.412951Z|8d8a743c-69c6-48dc-b13e-628a53167b1b] Executed!
[2024-07-17T17:50:31.385306Z|8d8a743c-69c6-48dc-b13e-628a53167b1b] Executed!
[2024-07-17T17:50:32.385326Z|8d8a743c-69c6-48dc-b13e-628a53167b1b] Executed!
[2024-07-17T17:50:33.727623Z|091bc8b9-fa1f-406a-a77c-cb6e0fd2f4a1] Executed!
[2024-07-17T17:50:35.072395Z|be73f7e0-a5bb-47bb-8aeb-dbca30d4003d] Executed!
[2024-07-17T17:50:36.385838Z|8d8a743c-69c6-48dc-b13e-628a53167b1b] Executed!
[2024-07-17T17:50:37.385476Z|8d8a743c-69c6-48dc-b13e-628a53167b1b] Executed!
[2024-07-17T17:50:38.385668Z|8d8a743c-69c6-48dc-b13e-628a53167b1b] Executed!
[2024-07-17T17:50:39.386702Z|8d8a743c-69c6-48dc-b13e-628a53167b1b] Executed!
[2024-07-17T17:50:40.386945Z|8d8a743c-69c6-48dc-b13e-628a53167b1b] Executed!
[2024-07-17T17:50:41.722913Z|091bc8b9-fa1f-406a-a77c-cb6e0fd2f4a1] Executed!


Add another R example notebook

Add necessary R libraries for interacting with lakeFS
Sample R/lakeFS notebook

Should we have the common code for calling the API out as a package on CRAN? (Are there sufficiently many R users to make it worth generating and exporting some R API code?)

I did try this, and failed. Notes here: https://github.com/treeverse/lakeFS/issues/6177


Fix #8834: exclude paramters and variables from flag checks

Fixes #8834

@RyanCavanaugh can you take a look.

:+1:


Improve the CircleCI Config

Speed up the entire process, this allows us to be able to utilize the bundle cache. Moving the test time down from around 5 minutes to 2 minutes.
Remove the rake dev:prime call. We don't have a dev:prime currently.

LGTM


Villager eyebrows not working

I did it as it says on the page.
Am i doing something wrong ?

Are you using the correct pack
Is the Expressive.zip not Eyeless.zip

Yes i am using Expreesive

did you upload that modified skin.png to your account?

no i used essential


Unit2unit

Codecov Report
Attention: Patch coverage is 0% with 26 lines in your changes are missing coverage. Please review.

Project coverage is 19.06%. Comparing base (04883f0) to head (f58115a).

Files
Patch %
Lines

espnet2/tasks/s2st.py
0.00%
14 Missing :warning:

espnet2/s2st/espnet_model.py
0.00%
7 Missing :warning:

espnet2/mt/frontend/embedding.py
0.00%
5 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           master      #21       +/-   ##
===========================================
- Coverage   76.57%   19.06%   -57.52%
===========================================
  Files         720      719        -1
  Lines       66695    66677       -18
===========================================
- Hits        51072    12709    -38363
- Misses      15623    53968    +38345

Flag
Coverage Δ

test_configuration_espnet2
?

test_integration_espnet1
?

test_integration_espnet2
?

test_python_espnet1
19.06% <0.00%> (-0.01%)
:arrow_down:

test_python_espnet2
?

test_utils
?

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

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


Compare versions.

Working on the comparison.
Currently, it involved changing version from a tuple to a full class.

Abandoning this dev


Add missing SI units to SensorTypes

Adds comments to each SensorType with its corresponding SI unit.
Resolves https://github.com/adafruit/Wippersnapper_Protobuf/issues/96

@lorennorman  Added value formatting


NEW Allow database read-only replicas

Issue https://github.com/silverstripe/silverstripe-framework/issues/11352

There's also some failing CI - does some of that rely on other PRs?

I think the unit test failures are related to the branch having /5/ in it when it's a CMS 6 PR (originally it pointed to CMS 6) - our CI looks at the branch pattern to get the version of installer
Issue links to a recipe sink build so we can rely on that


try to proper redirect download

trying to fix the https://github.com/apache/netbeans/issues/7224 serverside

Well, it'll work - does redirect twice. /download => /front/main/download => /front/main/download/
I'm not sure there's an easy way around that without doing something more complex with rewrite rules?
Should consider whether the other links should work correctly without a trailing slash too - eg. https://netbeans.apache.org/blogs

more changes for blog and issue from ide branding  resources

I will let that in. url may be complex but better than 404


Add React 0.14 Example

With React 0.14, the dangerouslySetInnerHTML is unnecessary. The result is a bit cleaner.

Great, thanks!

@joearasin Could you please send a new PR addressing our suggestions? :)


There is no "Debugging Information" item under Properties > Build > Advanced > Debugging Information in Visual Studio 2022.

There is no "Debugging Information" as referenced in:

To configure this in Visual Studio, right-click the project, then choose Properties > Build > Advanced > Debugging Information.

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

ID: 2ecc1202-a940-caa0-968f-9abfe2679d8b
Version Independent ID: 20ed4d8d-35ff-f1fe-74e4-ce2c27bb72c8
Content: Debug .NET Core on Linux - Visual Studio (Windows)
Content Source: docs/debugger/remote-debugging-dotnet-core-linux-with-ssh.md
Product: visual-studio-windows
Technology: vs-ide-debug
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

Also, searching the properties for "Debugging Information" returns no results.

@CLIRResearch Thanks! Yes, it looks like these options were moved to the General section in the properties in the VS 2022 update for .NET 6, I believe. I'll update the article.

Fix should be live by EOD today, 2/13/23


wsgi remote_addr can be user spoofed

Summary
The REMOTE_ADDR wsgi environment value can be easily spoofed by a user. Zappa takes the first value from the X-Forwarded-For header. This should be changed to the second to last one.
https://github.com/Miserlou/Zappa/blob/36ff4941f6ad37276944f16c12e101e461f9767e/zappa/wsgi.py#L71-L73
I will open a pull request with a fix.
Explanation
AWS adds ips to the back of the existing X-Forwarded-For header. The last one is the ip of the cloudfront proxy server while the second to last is the real client ip. Everything before that can be user supplied.
Currently if a client already sends a X-Forwarded-For header that value will then be used by Zappa as the remote_addr instead of the client ip.
See http://docs.aws.amazon.com/elasticloadbalancing/latest/classic/x-forwarded-headers.html for more information

This was merged, so this can now be closed?

Yep


Cannot upload firmware

I've been trying to upload a new firmware with latest version. I think the function has get broken. When starting upload in verbose mode the software starts initialisation block and then transfers the first block successfully. The second block also starts uploading, but then breaks. You cannot get any command working without resetting the device.
Here a brief copy of the communication:
Communicating with /dev/ttyUSB0
TX  9 bytes [7e 09 04 00 2f b2 f3 cc 7f]
RX  9 bytes [7e 89 00 04 00 03 66 0f 7f]
Download progress: 2% Communicating with /dev/ttyUSB0
TX 1036 bytes [7e 0a f0 1f 00 20 a1  ....
....b9 fe e7 94 78 00 2c ae c2 7f]
RX  6 bytes [7e 8a 00 f4 53 7f]
Download progress: 4% Communicating with /dev/ttyUSB0
TX 1035 bytes [7e 0a 5e d0 4c 1e...
....f1 01 0f 0d d0 21 89 68 1e 71 18 7f]
Error: timeout talking to device /dev/ttyUSB0.
From this point on there is no responce from opendps and you cannot communicate any longer.
You need to power-cycle the device to restart. After that it runs for other commands but upload fails again.

Can you try adding -b 115200 as a flag to dpsctl when you upload the firmware?

Same behavior and same error! I changed Baudrate in MAKEFILE (opendps, dpsboot) to 115200 and also added -b 115200 Flag.
I know that this upload worked in one or two former versions very well because I used it very often when writing a class similar to DPSCTL in Visual Studio (.NET).

Have tested. Does not work either . Same behavior.
Am 18. Oktober 2019 12:03:09 schrieb Xenoamor<EMAIL_ADDRESS>
Can you try adding -b 115200 as a flag to dpsctl when you upload the firmware?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

What DPS model is this? Also how are you connecting to the serial port, are you using any interface board between it and the PC, some of these models come with opto isolators boards

Same issue here, DPS5015 rev 2.3, checked two usb to serial boards, one from DPS kit, second ftdi232, commands are working without errors, but I can't upload new firmware. Maybe one month ago it was working.
opto isolators board:
`dpsctl.py -d /dev/ttyUSB0 -v -U opendps.bin
Communicating with /dev/ttyUSB0
TX  9 bytes [7e 09 04 00 a2 a4 ec ff 7f]
RX  9 bytes [7e 89 00 04 00 03 66 0f 7f]
Download progress: 1% Communicating with /dev/ttyUSB0
TX 1035 bytes [7e 0a f0 1f 00 20 e9 6f 00 08 e7 6f 00 08 e5 6f 00 08 e5 6f 00 08 e5 6f 00 08 e5 6f ............. 00 00 b9 fe e7 94 78 00 2c e6 1d 7f]
Error: timeout talking to device /dev/ttyUSB0.`
ftdi232 chip board:
`dpsctl.py -d /dev/ttyUSB0 -v -U opendps.bin
Communicating with /dev/ttyUSB0
TX  9 bytes [7e 09 04 00 a2 a4 ec ff 7f]
RX  9 bytes [7e 89 00 04 00 03 66 0f 7f]
Download progress: 1% Communicating with /dev/ttyUSB0
TX 1035 bytes [7e 0a f0 1f 00 20 e9 6f 00 08 e7 6f 00 08 e5 6f 00...............................84 00 00 b9 fe e7 94 78 00 2c e6 1d 7f]
Error: timeout talking to device /dev/ttyUSB0.`

Have been trying to test even deeper into the problem. Unfortunately I only have basic skills in C-programming. I put some debug-outputs to different points in the communication in dpsboot.c I found that communication works just before flashing the first block of new data. So it might be possible that flashing new firmware is the problem behind it.
Also, an older version works very well. I don’t know the reason but for me it seems to be a problem after doing some housekeeping with the code. Older version, that I mentioned, still uses the PACK- / UNPACK-macros. This version are running. Same as przemu stated.
Von: przemu<EMAIL_ADDRESS>Gesendet: Samstag, 19. Oktober 2019 13:16
An: kanflo/opendps<EMAIL_ADDRESS>Cc: hseibel12<EMAIL_ADDRESS>State change<EMAIL_ADDRESS>Betreff: Re: [kanflo/opendps] Cannot upload firmware (#191)
Same issue here, DPS5015 rev 2.3, checked two usb to serial boards, one from DPS kit, second ftdi232, commands are working without errors, but I can't upload new firmware. Maybe one month ago it was working.
opto isolators board:
`dpsctl.py -d /dev/ttyUSB0 -v -U opendps.bin
Communicating with /dev/ttyUSB0
TX 9 bytes [7e 09 04 00 a2 a4 ec ff 7f]
RX 9 bytes [7e 89 00 04 00 03 66 0f 7f]
Download progress: 1% Communicating with /dev/ttyUSB0
TX 1035 bytes [7e 0a f0 1f 00 20 e9 6f 00 08 e7 6f 00 08 e5 6f 00 08 e5 6f 00 08 e5 6f 00 08 e5 6f ............. 00 00 b9 fe e7 94 78 00 2c e6 1d 7f]
Error: timeout talking to device /dev/ttyUSB0.`
ftdi232 chip board:
`dpsctl.py -d /dev/ttyUSB0 -v -U opendps.bin
Communicating with /dev/ttyUSB0
TX 9 bytes [7e 09 04 00 a2 a4 ec ff 7f]
RX 9 bytes [7e 89 00 04 00 03 66 0f 7f]
Download progress: 1% Communicating with /dev/ttyUSB0
TX 1035 bytes [7e 0a f0 1f 00 20 e9 6f 00 08 e7 6f 00 08 e5 6f 00...............................84 00 00 b9 fe e7 94 78 00 2c e6 1d 7f]
Error: timeout talking to device /dev/ttyUSB0.`
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/kanflo/opendps/issues/191?email_source=notifications&email_token=AMPLUQUP2YRD5JUJ3DZSWSTQPLT6JA5CNFSM4JCDQRFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBXMMJQ#issuecomment-544130598 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AMPLUQXFUVX6HX46V7TXYATQPLT6JANCNFSM4JCDQRFA .  https://github.com/notifications/beacon/AMPLUQUWBACQOQ574BSWBQDQPLT6JA5CNFSM4JCDQRFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBXMMJQ.gif

Possible regression caused by #176
I'll have a look into it

Hmmm.. I'm unable to reproduce this with c548a92ee5ef0424ced7c079b52a8e0ce1cbd180

I am having a look into it right now. Will try all combinations of boot loader/app pre/post #176.

Bummer, reopening...

Weeeeell, this was a rather disappointing Sunday outing for me. I am using macOS which means a world of pain nowadays when it comes to serial devices. FTDI support is wonky so I am using a Black Magic Probe. My Mojave machine crashes when running dpsctl and the older High Sierra machine lies to me. PySerial write(...) claims to have written data that neither the STM32 receive nor are are visible on my oscilloscope. Yes, I brought my trusty old Rigol out to debug this.
I am taking a brake for coffee now and see if I can find a proper Linux machine.

Hi Johan,
I am using VirtualBox & Linux. Works well on a Windows machine. FTDI
adapter are quite well supported .

You can use the "Windows Subsystem for Linux" on Windows 10 and avoid the virtual machine altogether. Saves having the slow down caused by virtualisation

I got it going, sort of. I loose a bunch of frames from time to time which reminds me to stabilise the UART upgrade protocol, will open separate issue.

Interesting, any idea why you're loosing frames? Is the RX buffer too small?

Using a logic analyser I see that the data Pyserial claims have been written never reaches the TX line. TX stops moving in the middle of a frame and is silent for a few retries (due to RX timeout) and then suddenly starts moving again. I blame Apple for this one. On the flip side I got the perfect opportunity to stabilise the UART protocol...

I am using another debugging method and it worked quite well for me: I have collected all the strings which dpsctl is going to report in verbose mode. Then I changed to HTERM (the Hammer-Terminal) where I could insert the hex-string in sending dialogue to be transmitted and then were looking for the answer. I found that the first block of firmware data was sent (and recieved) successfully. I could verify that the decice is still responding by sending another commend i.e. the query. When tryiing to send the next step the error occurs. So it breaks behind the first data block. Should we reduced the length of the sending block to, let's say 512 bytes?

256 bytes works for me. I think I saw a number of buffer overruns in the code but need to look more carefully in the coming week.

Additionally, If I send a 512 byte frame I end up getting a SIGSEGV in memcpy(frame->buffer, data, status) called from uframe_extract_payload(...) on the reception of the 512 byte frame. Not that strange given the following struct:
#define MAX_FRAME_LENGTH (128)
typedef struct
{
    uint8_t buffer[MAX_FRAME_LENGTH];
    uint32_t length;
    uint16_t crc;
    uint32_t unpack_pos;
} frame_t;

So, anything above 128 byte chunks might cause a crash.
Additionally, sending a 512 byte frame bricks my boot loader :-/  When writing the past_upgrade_started unit, the byte at  0x08000007 changes from 0x08 to 0x00.


IMDB classification

I am trying to use the imdb model(predicting_movie_reviews_with_bert_on_tf_hub.ipynb) on my own data set. I have two labels for my dataset. I want the prediction probabilities for both the labels.
I see that in the imdb notebook output the probabilities do not add up to 1. Since we are using softmax as our last layer I guess the probabilities should add up to 1. Also the value are negative so obviously they don't look like probabilities to me.
Please help me in interpreting the probability output and also how can I get a probability value as output.

Sorry I got the answer, I missed that you are using log_softmax. I will close the issue.


"Slatina" is listed in the drop-down menu

Description:
"Slatina" is listed in the drop-down menu alongside its corresponding county according to the law
Precondition:
The website is up an running.

Step 1
Type "Slatina" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Slatina, SUCEAVA".


What's the syntax for filtering queries?

Hi,
I'm massively struggling to work out the syntax for filtering a result set with the in-built filters.  Are you able to provide a sample query for returning filtered nodes from Umbraco?
For example, with a custom node called book in Umbraco and bookDirectory being the parent folder of books, what should the query be?  I've tried lots of combinations, but not getting anything valid!
{
  umbraco {
    content {
      byType {
        bookDirectory {
          pageTitle
          bookStartsWithA: _children(
            filter: { _name_starts_with: "A" }
            orderBy: _name_ASC
          ) {
            items {
              ... on BookPublishedContent {
                _name
                author
              }
            }
          }
        }
      }
    }
  }
}

Hi @dchallener
Looking at the query I'm assuming you are using the latest version from the develop branch.
Filtering isn't implemented yet as I wanted to rethink how they should work. I don't think the current implementation in the Umbraco 7 version is very useful (it was only included as a POC) since you can only filter on fields that are "known" (ordering has the same problem).
What I'm thinking of is making the "field" part a string you pass in, it might look something like this:
{
  umbraco {
    content {
      byType {
        bookDirectory {
          pageTitle
          bookStartsWithA: _children(
            filter: [{ field: "_name", op: STARTS_WITH, value: "A" }
            orderBy: "_name"
            sortOrder: ASC
          ) {
            items {
              ... on BookPublishedContent {
                _name
                author
              }
            }
          }
        }
      }
    }
  }
}

This would not only make the backing code more simple (I think) but also allow you to filter on fields like author.
Only drawback is that it's not strongly typed so you might end up with spelling mistakes, but I think the flexibility of being able to pass in every field outweighs that

Hi
Just wanted to let you know that the project is moving to the Umbraco Community GitHub organisation, so we are closing all existing issues.
If you think your issue is still relevant, please feel free to reopen it.
/Rasmus

Hi
Just wanted to let you know that the project is moving to the Umbraco Community GitHub organisation, so we are closing all existing issues.
If you think your issue is still relevant, please feel free to reopen it.
/Rasmus


Buddy Feature

Buddy Feature has now been launched will this be added to Bot ? as walking with buddy rewards candy so would be great to use in this bot

Yes.

Any progress update on this?

apparantly this does work already if ur buddy is set in official app you will collect the candys in Bot and they will be added


Added MovieInfoApp

Developer Checklist

[x] Followed guidelines mentioned in the readme file.
[x] Followed directory structure. (e.g. ProjectName/{USERNAME}/...yourfiles)
[x] Starred ⭐ the Repo (Optional)

Summary
A simple JavaScript web application to search for and display movie details.
Screenshot

GIF

@PBJI ,please review my request and merge it.


make elasticsearch config path configurable

Got the following exception today while attempting to run a diag in a docker container:
java.io.FileNotFoundException: Source '/usr/share/elasticsearch/config' does not exist
They have their config in a custom directory
Keep in mind that this could also be a docker problem only

Docker problem more than likely.  Check the nodes.json settings.  If there's a config directory set it will look for the config file there. So the question would be what it will get when it tries to access that directory.
Note, per the instructions you can run it with
--type remote
to bypass copying and scrubbing the config and the logs.
Also curious where you are running the diagnostic from?  That may have an impact as well.

Saw your comment too late.
This makes sense. I will try to verify with the customer


Monaco 2.10.0 - incorrect order deploying network zones

During deployment Monaco tries to deploy management:network-zone before builtin:networkzones
it fails on first run with following message:
Failed to send HTTP request: (HTTP 400)!     Response was: Not allowed because network zones are disabled
it is successful on rerun
How to reproduce

new environment with network zones disabled
management:network-zone set to enable in /builtinnetworkzones/config.yaml+ json
a network zone described network-zone/config.yaml + json

Expected behavior
monaco turns on network zones setting and sets them up
Monaco 2.10.0

Hi @evaldasc Thanks, for reaching out.
You are right, this kind of "static" dependency between that setting and the network zone config is not taken into consideration by monaco. If the ordering is relevant to you you would need to add a "fake" dependency between these two config entities. However, I agree that it would be a better UX if monaco would just take care of that, by knowing that the network zone setting must come first.


Example of how to initialize from PrincipalId/ClientId

I'm migrating code from the deprecated Fluent API to the ResourceManager API and the previous code used Microsoft.Azure.Management.Network.Models.ManagedServiceIdentityUserAssignedIdentitiesValue(principalid, clientid) to initialize a UserAssignedIdentity from principalid and clientid. I don't see how to do the same thing in the ResourceManager API.

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

ID: e796ef8c-b218-1cd0-c3af-8d56a57a8a2c
Version Independent ID: 18fdaa1d-8364-819c-5858-0cfa7b6e483a
Content: UserAssignedIdentity Class (Azure.ResourceManager.Models) - Azure for .NET Developers
Content Source: xml/Azure.ResourceManager.Models/UserAssignedIdentity.xml
Service: azure
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

Looks like the principalId and clientId aren't required.


[AES] Journal log pruning failure

Start / End / Elapsed:     20170314 18:20:57.863 / 20170314 18:20:57.920 / 00:00:00.057
18:20:57.864     INFO     Executing command 'journalctl --vacuum-size=1M'.
18:20:57.920     INFO     Command exited with return code 0.
18:20:57.920     INFO     ${output} =
18:20:57.920     INFO     ${stderr} =
18:20:57.920     INFO     ${rc} = 0

The output is empty and it should have returned the "Vacuuming done, freed 0B of archived journals on disk."
https://gerrit.openbmc-project.xyz/#/c/3022/

Code is merged


Nix configuration

What does this mean out of curiosity?

We meant using nix in one of the examples.
It was proposed by @rizerkrof. I suggest to close this issue because no work has been done on it the past weeks, and I'm not willing to work on this.
If someone wants to show its nix skills, then why not! Feel free to re-open this issue :)


Timespec to datetimespec conversion utility

This is an item of project: https://github.com/apache/incubator-pinot/issues/2756
We plan to internally start treating timeFieldSpec as dateTimeFieldSpec. This PR adds a utility function which converts a timeFieldSpec to an equivalent dateTimeFieldSpec.
Note that dateTimeFieldSpec doesn't have the concept of incoming/outgoing, and hence we add a transform function to convey the conversion.

Codecov Report

Merging #5324 into master will decrease coverage by 9.34%.
The diff coverage is 63.63%.

@@            Coverage Diff             @@
##           master    #5324      +/-   ##
==========================================
- Coverage   66.23%   56.88%   -9.35%
==========================================
  Files        1066     1073       +7
  Lines       54552    54665     +113
  Branches     8128     8154      +26
==========================================
- Hits        36132    31097    -5035
- Misses      15774    21124    +5350
+ Partials     2646     2444     -202

Impacted Files
Coverage Δ

...e/pinot/broker/api/resources/PinotBrokerDebug.java
0.00% <ø> (-76.67%)
:arrow_down:

.../BrokerResourceOnlineOfflineStateModelFactory.java
41.86% <ø> (-13.96%)
:arrow_down:

...quota/HelixExternalViewBasedQueryQuotaManager.java
64.84% <ø> (-3.04%)
:arrow_down:

...roker/requesthandler/BaseBrokerRequestHandler.java
20.10% <ø> (-59.25%)
:arrow_down:

...rg/apache/pinot/broker/routing/RoutingManager.java
66.53% <ø> (-14.62%)
:arrow_down:

...ting/instanceselector/InstanceSelectorFactory.java
71.42% <ø> (ø)

...er/routing/segmentpruner/SegmentPrunerFactory.java
83.33% <ø> (ø)

...outing/segmentselector/SegmentSelectorFactory.java
60.00% <ø> (ø)

...oker/routing/timeboundary/TimeBoundaryManager.java
89.74% <ø> (+2.24%)
:arrow_up:

...mmon/assignment/InstanceAssignmentConfigUtils.java
67.50% <ø> (-5.00%)
:arrow_down:

... and 526 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 6a1565c...c0fb8a2. Read the comment docs.

Is the new util class to be used inside the Schema class (and everyone else outside sees only DateTimeFieldSpec)?

Is the new util class to be used inside the Schema class (and everyone else outside sees only DateTimeFieldSpec)?

That's right. Only Schema class should know about this converter. Maybe it can even go into the Schema class

Is the new util class to be used inside the Schema class (and everyone else outside sees only DateTimeFieldSpec)?

That's right. Only Schema class should know about this converter. Maybe it can even go into the Schema class

Yes, as methods in Schema class?

Is the new util class to be used inside the Schema class (and everyone else outside sees only DateTimeFieldSpec)?

That's right. Only Schema class should know about this converter. Maybe it can even go into the Schema class

Yes, as methods in Schema class?

Done.


Is there any way to change autoSelectField label

First of all I would like to thank you for providing us a great  library.
Is there any possibility to change the autoselectfield  label ------ from some custom text. (If I choose autoSelectField false)

Not currently, unfortunately, but I'll put a version out shortly that will add that option. Perhaps even later today.
Thanks for the suggestion. I'm mad that I didn't think of that when I added the autoSelectField prop!

@syam5492009, I just released v4.2.5 which adds a couple of new options to the translations prop. To change the label for the placeholder field, set the fields.placeholderLabel property in the translations prop, like this:
import { QueryBuilder } from 'react-querybuilder';
import { fields } from './fields';

const App = () => {
  return (
    <QueryBuilder
      fields={fields}
      autoSelectField={false}
      translations={{ fields: { placeholderLabel: 'Please select a field' } }}
    />
  );
}

Documentation for the autoselectfield prop has also been updated -- see here.

Thanks for the update.
On Fri, 13 May 2022, 12:40 am Jake Boone, @.***> wrote:

@syam5492009 https://github.com/syam5492009, I just released v4.2.5
https://github.com/react-querybuilder/react-querybuilder/releases/tag/v4.2.5
which adds a couple of new options to the translations prop. To change
the label for the placeholder field, set the fields.placeholderLabel
property in the translations prop, like this:
import { QueryBuilder } from 'react-querybuilder';import { fields } from './fields';
const App = () => {
return (
<QueryBuilder
fields={fields}
autoSelectField={false}
translations={{ fields: { placeholderLabel: 'Please select a field' } }}
/>
);}
Documentation for the autoselectfield prop has also been updated -- see
here
https://react-querybuilder.js.org/docs/api/querybuilder#autoselectfield.
—
Reply to this email directly, view it on GitHub
https://github.com/react-querybuilder/react-querybuilder/issues/300#issuecomment-1125333538,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD2NE6CTTFIDQFC3CJK23CLVJVJTVANCNFSM5VYLFR7A
.
You are receiving this because you were mentioned.Message ID:
@.***>


Add detail to some FilterErrors

This commit:

adds some detail about the nature of the error to a few sites that
raise Scimitar:FilterErrors. This is limited to errors that arise
due to invalid input

Why?

This should allow for easier debugging when these errors are raised

Very useful. Thanks!

Available now in v2.1 (https://rubygems.org/gems/scimitar).


State machine redesign

TODO
Add wiki pages for design.
Impacted tests need to be updated at some point.
This merge is still ready for review in the meantime


An Exception occurred while running blox strap

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] I have thoroughly looked through the available Wiki articles and could not find a solution to my problem.
[X] I am using the latest version of Bloxstrap.
[ ] I did not answer truthfully to all the above checkboxes.

What problem did you encounter?
I keep getting an error when i try to open a game through the website on roblox. everytime i try to run it, this pops up:
System.Net.Http.HttpRequestException: Response status code does not indicate success: 403 (Forbidden).

Bloxstrap Log
Bloxstrap_20241003T182258Z.log

READ OTHER POSTS.

READ OTHER POSTS.

I DID....  You would have read the wiki and the solution would help you. But you just don't read it for some reason.


Provide a way of checking meta info of page object

While working on assert lib for page object I faced and issue described here.
There is no way to check if property is a property or a function without calling it, for example:
create({
  clickOnMe: clickable('a'),
  href: attribute('href', a),
});

So to check if property is a function I need to call it like this:
typeof page.clickOnMe === 'function' // true
typeof page.href=== 'function' // false

The problem with this is that we actually execute getter and if it raises an error, our check will fail.
typeof page.href === 'function' // Element not found!

Do you think it's possible to store it somehow?

Yes, there is such a limitation currently. I believe a good way to do it would be go play around  getPageObjectDefinition(. But:

that's private for now
even with that you don't have a way to distinct between props VS actions.

As a short term solution, we could mark all the current built-in action descriptors by smth like isAction: true. But that doesn't feel super optimal to me, cause on the assertions end you would need to use private getPageObjectDefinition( and private isAction, which can easily breake because of hard dependencies on private APIs.
I think we should solve it, when introducing a public way to declare a custom action. I expect it to be done in v2.
Meanwhile, I think your current approach to create assertions for actions is a good(in terms of reliability) solution for now.


API available

FYI, Fuji released a SDK in April 2021
https://fujifilm-x.com/global/special/camera-control-sdk/

hmph


Proposal to give much more customization to tremor from recharts props

Introduction
Hey guys, I have been experimenting with an idea I had to solve an issue I told you in videomeet (#128). My issue was that i wanted to be able to show all the x axis labels for a bar chart that could have up to 20 bars. By default, recharts hides some bar labels to avoid overlap between them, and that behaivour is controlled by the interval prop. So, I just had to play a bit with the angle, interval and a few other props to get the result I wanted that you can see in the storybook example:

The idea
The interesting part is how easy it was to get into there, and the potential it has to customize all the charts. First I went to the BaseChartProps and imported the XAxisProps from recharts. I just had to add that as new prop for the base type:
// Other imports
import type { XAxisProps } from "recharts"

interface BaseChartProps {
    // ...
    xAxisProps: XAxisProps
}

export default BaseChartProps;

Now in the BarChart component I just have to get it and pass the props to the XAxis component from recharts:
const BarChart = ({
    // rest of props
    xAxisProps = {},
  }: BarChartProps) => {
    return (
      // ...
      <XAxis hide={!showXAxis} dataKey={dataKey} /*...rest of props*/  {...xAxisProps} />
      //
    );
  };

So in the storybook example I just have to add this params to get the result I want:

This prop extensions can be encapsulated in a single type like ExtendedProps that groups all rechart components that are used across charts (YAxis, CartesianGrid, Legend, Bar, etc) and thats the one added to BaseChartsProps.
What do you think guys? I think its a really nice and clean way to give tremor users all the power of recharts just by forwarding some props when tremor defaults doesnt fit within a specific use case. If you think its a good idea I could try to implement it in a PR (I am having some problems with code formatting btw #186 )

Hi @angelhodar, long time no see! ;)
Many thanks for this input and thinking ahead. Our philosophy is to offer more default than customization. This means that the axis labels would be rotated automatically if a certain threshold is met / if labels get too dense. Those are the things we are currently trying to tackle behind the scenes.
However, I agree with your point in some ways. There will be always edge cases in which developers want to override / extend our props. We will definitely think of this in more detail.
Many thanks for the discussion and happy hacking :)

Hi @christopherkindl, I know that this implementation would interfere a bit with the main goal of tremor, but I just see it as a complement to the well-defined set of defaults that tremor provides. In my case I have had to install recharts to achieve that specific chart functionality, when in reality it is something that (with a short overview to the code) tremor could implement easily with a cheap cost in time and complexity.

However, I agree with your point in some ways. There will be always edge cases in which developers want to override / extend our props. We will definitely think of this in more detail.

I just ran into a situation where I wanted to change the animation speed, so having the ability to override the recharts defaults would have helped in this situation too.

hey @all thanks for all the feedback! I think it makes sense to implement/override certain features from recharts, however, there are some points we need to consider which I'd like to share for transparency:

Adding too many properties bears the risk of breaking the opinionated design system and effectively becoming another low-level chart library.
We are considering migrating off recharts at some point as it has certain limitations and behaviours we would like to implement differently under the hood. Moreover, the package currently introduces "high severity" security vulnerabilities for which we don't know when or if they will be fixed. Bearing that in mind, we need to carefully consider which recharts-specific properties we are adding as they are potentially not be available in other libraries or have different naming.
We are planning to substantially alter the current tremor Chart API, involving the prop naming and component base structure, within a tremor v2 release next year (breaking change). This means that we again need to be picky about which props we add now, in order to avoid too many breaking changes in the v2.

I hope that makes sense. This does not mean that we will be rejecting the suggested features, just that we have to think a bit more about the implementation.

Just my two cents: I think it's very wise to keep the defaults (both in terms of design and API) very opinionated and simplistic. This was one the main things that attracted me to the project, since it's a huge time-saver, and is particularly important when it comes to the documentation and type hints provided by the library.
Of course, the end user will always need customizations, and it seems like some sort of plugin approach would be ideal to handle these cases, in a way that doesn't pollute the global namespace. I'm guessing you may be alluding to something similar with the extensibility changes coming in v2, but just wanted to provide an end user perspective :)


All products showing even if home category not in the association

Howdy all,
I was testing this out today, and it seems that even if I don't have any products associated with the home category, all products loaded so far, still show up in the featured products on the home page.
I changed the category name in the database directly to read shop instead of home, however I did not change the ID at all, it still remains as ID of 2.
What could be the issue?
[Update]
If I select any other category, it will honour that category, however if I select 2 (Home) then it will select the first set of products loads, or random if selected, from any category, it does not explicitly select from category 2
So I am thinking something else is wrong here, maybe a database issue? however I have also tested this on a brand new install of 1.5.1 and I have the same issue

I managed to fix this, however I am unsure if this fix is correct as I have no reference for it.
In the database on the categories table then is a column called nLeft and nRight, with values of 2 and 168 respectively. I changed the 168 to a 2 and it started to works as expected.
@Traumflug can you confirm if this is an acceptable fix for this?

In the database on the categories table then is a column called nLeft and nRight, with values of 2 and 168 respectively. I changed the 168 to a 2 and it started to works as expected.

This is most likely not the solution, and the issue will probably be reintroduced when you resave the home category.
Columns nleft and nright continains ids of first and last children

When I read the original issue, I don't think this is a bug, but rather a misunderstanding how the module works. When you select home category, it will display any products associated with any category that is children of Home category. The product doesn't have to be associated with Home category explicitlly.

Brilliant, I have done this and it works perfectly, thanks so much. Its great that in general products form sub categories will show, but just the home category its disabled


bug(satellite): potential race condition in _performSnapshot()

_performSnapshot() must be executed serially. All calls to are protected with a mutex and we have an assertion on the function to ensure we enforce serial execution. However, sometimes we'd still see the assertion fail.
We are not setting properly the performingSnapshot variable when exceptions occur in _performSnapshot(). Although this probably constitutes a bug, it happens. The exceptions being thrown might also be swallowed sometimes, which made more difficult to track the error (and potentially still hiding other error).

I apologize to the reviewers for the horrid diff :D

@balegas Not bad at all to review when hiding whitespace diffs :wink:


HTML&CSS - Do I follow JavaScript best practices?

Hey there :wave:
Some recommendations for your CSS code.

You recently understood better how to correctly use variables in CSS, so, you will see that from line 22 to 26, you have the variables declared for this project, but not correctly. You must use the :root pseudo class to encapsulate them.

Already fixed with the :root pseudo class
:root {
  --text: #414856;
  --check: #4f29f0;
  --disabled: #c3c8de;
  --background: #fff;
  --font: 'Montserrat', sans-serif;
}


fix failing lints in aar_init_script.gradle

android studio is marking those three imports as unused imports.
import org.gradle.api.Project
import org.gradle.api.artifacts.Configuration
import org.gradle.api.publish.maven.MavenPublication

the lint :

and this method could be converted to static :
String getFlutterRoot(Project project) {
    if (!project.hasProperty("flutter-root")) {
        throw new GradleException("The `-Pflutter-root` flag must be specified.")
    }
    return project.property("flutter-root")
}

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[x] I followed the breaking change policy and added Data Driven Fixes where supported.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

test-exempt: code refactor with no semantic change


Crash when opening settings.

Microsoft PowerToys version
0.76
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
General, Settings
Steps to reproduce
PowerToysReport_2023-12-07-01-05-32.zip
When I try to open settings a black window pops up for few seconds then closes. It started to happen after I downloaded version 0.76 from github. When I downgraded back to 0.75.1 it came back to working normally.
✔ Expected Behavior
No response
❌ Actual Behavior
No response
Other Software
No response

@stefansjfw

would you be willing to test unofficial, custom build, to check if that one works for you? Trying to localize the issue if possible :)

I think I could try

Thanks. You can download the installer in assets here: https://github.com/stefansjfw/PowerToys/releases/tag/v0.0.1
First, make sure you uninstalled the current version of PowerToys. After that, you'll be able to install this one. Remember that this is an unofficial build and you won't receive automatic updates for new versions, so you'll have to manually remove it later on.
Let me know if Settings app works for you with this version. Thanks in advance!

PowerToysReport_2023-12-10-01-23-09.zip
Still not working. Here is the bug report if needed.

Hm. Good to know, thank you. I wonder if WinAppSDK version is related to this. Could you try this one https://github.com/stefansjfw/PowerToys/releases/tag/v0.0.1 ?
Thanks for taking your time to help with this

Tried it. Still the same.
PowerToysReport_2023-12-10-15-37-04.zip
Thank you for taking your time to help me btw.

Ok. I'll keep digging. :) could you try installing .NET 8 Runtime? Does that change anything?

Not fixed in 0.76.2

Fixed in v0.77.0


Improve Memory Team with Diary Bot

Diary Bot Assigned
The Memory Team needs you, Diary Bot! How do we do that? Crafty stitching.

[x] Ensure that Diary Bots have access to vectorstore (for uploading other inputs)
[x] Ensure Diary Bots have all the tools available to them
[x] Ensure that Memory Team description includes optional Diary Bot
[x] Return of +bot dropdown button
[x] functions.mjs - discovery: createBot() utilizes teams, but not sure if leveragable
[x] Functionality for adding optional bot to team

[ ] custom bot coming soon

[ ] Test Diary Bot addition
[ ] Test Journaler addition
[ ] retireBot
[ ] migrateChat/Upgrade

Poor Splotch is to be dispatched to test for a new diary bot :(
Also deleted journaler and then spawned two new ones successfully!


Create msi package

Task
Create msi package for painless windows installation with system service integration
Acceptance criterea
Generated msi package should be easily install-able on Windows

MSI package is built. Problem is that bee cannot start as a Windows service, it fails with
Error 1053: The service did not respond to the start or control request in a timely fashion
It looks like that solution is to change how bee is being started, either this svc or this kardianos/service

Blocked by: #935


Time synchronisation

Tracking issue for server<->client time synchronisation

This is needed to properly fix animation timing when start time comes from the network: https://github.com/AmbientRun/Ambient/issues/586#issuecomment-1700793303

Diffs are also not synced with time: https://github.com/AmbientRun/Ambient/issues/664

Same for reliable messages: https://github.com/AmbientRun/Ambient/issues/665

Same for reliable messages: #665

I think this is a separate issue? For this, I think the fix would just be to send them over one unistream pipe so that QUIC/WT do ordering for us

Same for reliable messages: #665

I think this is a separate issue? For this, I think the fix would just be to send them over one unistream pipe so that QUIC/WT do ordering for us

Good point. It would definitely make it simpler.


Feature to use a regular Twitter/X account

Describe the problem
It looks like Twitter/X is not allowing the creation of temporary guest accounts anymore.
Is this new policy permanent? Who knows!
Will there be another solution to be able to access Twitter/X in a privacy oriented manner? Good question!
Describe the solution
By implementing the possibility to log with a regular account, it makes Squawker usable for the future.
There are ways to create regular accounts with no phone number and a temporary email address that is not linked to the user.
Without being completely anonymous, it might be a good enough compromise for the people who still want to use Squawker.
And who knows maybe in the future someone will find a solution to access Twitter/X without a regular account. For now it gives time for Squawker to continue to be functional.
Please comment or vote to let me know if you are interested in such a feature.

Absolutely!

Hey bro, that is so cool! I'm looking forward to it.

Thanks for the ongoing hardwork on the app, it must be a bit of a nightmare trying to keep up with the whims of Elon Musk!
I'd definitely be interested in such a feature if it would enable Squawker to continue to be used.
"There are ways to create regular accounts with no phone number and a temporary email address". Excuse my lack of Twitter /technical knowledge, I have no idea how to create an account without a phone number (I use an email alias service so that shouldn't be an issue). If anyone has a moment to explain how, I would be grateful.

as much as i want twitter to go die already and we can all finally switch to mastodon, which will never happen ofc (a man can dream) i support your idea of keeping squawker alive with your idea of making it usable with regular accounts. so thanks for your contineous support and development.
squawker really was a blessing and hopefully still will be in the future.

I for sure would like this. My main issue with twitter has always been it being super slow personally

Having to create a throwaway account would be a totally fine compromise, I think. This has been the best way to keep track of folks & industries without using Twitter in the traditional way. Jumping through a simple hoop is okay in my books.

This would be great, just not having to use the standard x app with all the pollution in my feed

This would be amazing, there are a lack of 3rd party twitter clients now.

A million times yes!
Forced back to the X app for the first time in years and every 5th post is an ad. How do people live like this?! It's unusable.
Merci d'avance!

I would really like this a well since all links go via twitters tracked services (that I've blocked which mean I can click none).

The version v3.7.0 allows you to use a throwaway regular Twitter/X account.
To add the information of an existing regular Twitter/X account, go to Settings / Account, tap on the plus icon.
Only the username and password are mandatory. When saved an authentication token is created. It is automatically renewed every 30 days.
There can be many existing regular Twitter/X accounts (since each account is as rated limited as guest accounts were).
To delete a regular Twitter/X account information (and its authenticated token), swipe to the left and tap on delete.
The project's wiki describe how to create a (regular) Twitter/X account.

The version v3.7.0 allows you to use a throwaway regular Twitter/X account.

Anxiously awaiting when it will be on creativaxion :-)

@fjelsten It's done.

thank you so much. it works! what a boss you are.

@fjelsten It's done.

Works nicely. Excellent work. Thank you very much.
Are there any refresh and issues or limits when you use a real account? Do I need to create several to bypass limitations.

Can't log in with my account, it sent me an email telling me I need to answer some security questions.

ObviousIy, I can't complete them withing Squawker.
(Also, the Squawker message I got was quite user-friendly, I believe it's just the raw JSON or whatever language.)

@mayhair Could you check if version v3.7.2 fixes this issue?

Thank you for the latest versions and the wiki etc, appreciated (and apologies, in a previous reply I was under the mistaken impression you neeeded a phone number to create an account. If it's not bad enough you've got to contend with Twitter, you've also got to deal with the likes of ignorant me!).

@0xc1c4da Can you provide a screenshot please?

@mayhair Could you check if version v3.7.2 fixes this issue?

Seems alright now, thanks.

@0xc1c4da Did you do an import data?


Routers and Selectors Draft

Let's this discussion be just an issue https://github.com/nevalang/neva/issues/802


Error with adding Elgiganten links after update and new feature request

Thank you so much for the new update! But now I'm having a problem with adding Elgiganten link from .dk and .se but just commented out the .dk link in domains.py so it works for me now :)
    def _get_json_api_data(self) -> dict:
        id_number = self._get_product_id()

        # API link to get price and currency
#        if "elgiganten.dk" in self.url:
 #           api_link = f"https://www.elgiganten.dk/cxorchestrator/dk/api?appMode=b2c&user=anonymous&operationName=getProductWithDynamicDetails&variables=%7B%22articleNumber%22%3A%22{id_number}%22%2C%22withCustomerSpecificPrices%22%3Afalse%7D&extensions=%7B%22persistedQuery%22%3A%7B%22version%22%3A1%2C%22sha256Hash%22%3A%229bfbc062032a2a6b924883b81508af5c77bbfc5f66cc41c7ffd7d519885ac5e4%22%7D%7D"  # noqa E501
  #      elif "elgiganten.se" in self.url:
        api_link = f"https://www.elgiganten.se/cxorchestrator/se/api?getProductWithDynamicDetails&appMode=b2c&user=anonymous&operationName=getProductWithDynamicDetails&variables=%7B%22articleNumber%22%3A%22{id_number}%22%2C%22withCustomerSpecificPrices%22%3Afalse%7D&extensions=%7B%22persistedQuery%22%3A%7B%22version%22%3A1%2C%22sha256Hash%22%3A%22229bbb14ee6f93449967eb326f5bfb87619a37e7ee6c4555b94496313c139ee1%22%7D%7D"  # noqa E501
#        else:
#            raise WebsiteVersionNotSupported(get_website_name(self.url, keep_tld=True))
        response = request_url(api_link)
        return response.json()

Also changed short_url to .se instead of the .dk
    def get_short_url(self) -> str:
        id = self._get_product_id()
        return f"https://www.elgiganten.se/product/{id}"

One more question, is it possible to fetch the outlet prices from like https://www.elgiganten.dk/product/mobil-tablet-smartwatch/mobiltelefon/google-pixel-7-pro-smartphone-12128-gb-obsidian/525075 or add a parameter to the database from like https://www.elgiganten.dk/product/outlet/google-pixel-7-pro-smartphone-12128-gb-obsidian/582585 to see when it is back in the outlet stock? Maybe hard to code it.. :)

I can see both problems. Both problems should be fixed in #233 and #234.
I don't think I will prioritize getting the outlet price for now. If I get the time and find a way to implement a way to look when a product is back in stock, I might add the functionality to do so.


Change main branch origin point

Lemmy 2023 introduced improvements to the codebase. We should branch off from there instead of Place 2023

Completed by https://github.com/r-ainbowroad/2024-minimap/commit/91a95604d6c584e11fd148ac4678d09e9aa9e0c9


openebs helm charts needs to support CAS Template related CRDs

Error while using openebs helm charts with volumepolicy
volumepolicies.openebs.io "openebs-policy-taints-0.6.0" is forbidden: User "system:serviceaccount:openebs:openebs-maya-operator" cannot get volumepolicies.openebs.io at the cluster scope

https://github.com/openebs/maya/pull/509 : adds required CRDs as part of openebs installation.


[monorepo-builder] MBConfig::packageDirectoriesExcludes input validation makes exclusion impossible

Problem
MBConfig::packageDirectoriesExcludes performs an Assert::allFileExists($packageDirectories);. This means that paths must either be absolute or relative to the current repository root.
However, Symfony's ExcludeDirectoryFilterIterator will turn any non-simple (i.e. more than one path segment) path into a matcher. The actual matching then happens against a path that is relative to the directories provided to packageDirectories. That's usually already a step removed from the repository root.
This means that the input that is valid for packageDirectoriesExcludes will never match in ExcludeDirectoryFilterIterator.
An example folder structure
  tools/
    oste/
      data/
        composer.json
      composer.json

Desired: Get all packages in tools/*/ but exclude specifically tools/oste/data/composer.json.
Config that passes input validation but doesn't match anything:
  $mbConfig->packageDirectories([
    __DIR__ . '/tools',
  ]);

  $mbConfig->packageDirectoriesExcludes([
    'tools/oste/data',
  ]);

When debugging the matching in ExcludeDirectoryFitlerIterator you will see that oste and oste/data are matched against the exclude input which is tools/oste/data which never matches.
Config that would match the exclude but doesn't pass input validation:
  $mbConfig->packageDirectories([
    __DIR__ . '/tools',
  ]);

  $mbConfig->packageDirectoriesExcludes([
    'oste/data',
  ]);

This doesn't pass input validation because MBConfig::packageDirectoriesExcludes tries to check that oste/data exists (which it doesn't because it lives in tools/oste/data).
Proposed solution
Either modify MBConfig::packageDirectoriesExcludes to remove the folder existence validation. This allows excluding subdirectory patterns in included directories. A downside is that I may want to include directoreis A/ and B/ and exclude B/Foo but not A/Foo which would then not be possible (since the iterator matches only on subpaths).
An alternative is to make sure that ExcludeDirectoryFilterIterator actually gets the path relative to the project root to match on, rather than relative to the include directory. That would allow configuring an exclusion of B/Foo specifically.
Workaround
The deprecated $mbConfig->parameters()->set(Option::PACKAGE_DIRECTORIES_EXCLUDES, []);  can be used to work around the faulty input validation in MBConfig.

Thanks for sharing the issue in detail.
Could you kick off with a failing PR for your situation? It would be easier to iterate from there.

@TomasVotruba Here you go: https://github.com/symplify/symplify/pull/4390 :)


Add networking logic to perform HTTP calls to API

Technical requirement in order for the SDK to perform requests to the CIO API.
Why are we doing this for our customers?

Today, our customers need to write custom code in their apps to perform requests to the CIO API to add data to their workspace. This task will setup the technical code in order for us to perform the API calls on your behalf.
How do we test that this change is successful?

The end goal is to perform a successful HTTP request to the CIO API in an Android app via the SDK. This includes:

Authentication header
Path is dynamic

How are we building this?

Not 100% sure. The end goal is keeping the SDK slim so not using retrofit or volley is a great start.
My current idea is to use kotlin coroutines. Google gave a nice code snippet in it's docs:
sealed class Result<out R> {
    data class Success<out T>(val data: T) : Result<T>()
    data class Error(val exception: Exception) : Result<Nothing>()
}

class LoginRepository(private val responseParser: LoginResponseParser) {
    private const val loginUrl = "https://example.com/login"

    // Function that makes the network request, blocking the current thread
    fun makeLoginRequest(
        jsonBody: String
    ): Result<LoginResponse> {
        val url = URL(loginUrl)
        (url.openConnection() as? HttpURLConnection)?.run {
            requestMethod = "POST"
            setRequestProperty("Content-Type", "application/json; utf-8")
            setRequestProperty("Accept", "application/json")
            doOutput = true
            outputStream.write(jsonBody.toByteArray())
            return Result.Success(responseParser.parse(inputStream))
        }
        return Result.Error(Exception("Cannot open HttpURLConnection"))
    }
}

[ ] Decide how we will do HTTP networking calls. Implement that code in the SDK.

[ ] Update logger to log the request and response of the request.

[ ] Create a network retry system. Wendy does not handle this. When our network operation runs, we want to retry failed network operations a few times before we tell Wendy that the task failed and will need to be put back on the queue.

This retry system will be...
HTTP request fails ---> retry 300 milliseconds later ---> retry 600 milliseconds later --> retry 1 second later ---> failed.
That's 2 seconds in total. Let's give that time a try.

[ ] Add User-Agent header for all requests. Value set to Customer.io Android SDK vX.X.X, Y Android app vX.X.X  where Y is the name of the customer's Android app. Thinking you can specify an optional app identifier in the configuration?

From the iOS SDK, here is some help for you on this task:

the headers to set
Currently, we are only handling 401 error codes
You will want to create a way to cancel HTTP requests we can use later on.

I updated the ticket to remove some features like retry. Don't worry about retry, we will plan that out at a later time.


Tests: ENS not working

Testing both my local bot and qa bot for setting wallet using ENS.

This test is happening at the latest commit https://github.com/ubiquity/ubiquibot/commit/48e006da17c46cc172263fe9a3924cd90a0f3bcb at the development branch

/wallet vitalik.eth

/wallet 0x1F9FCc0E84dc2094499b848AE37Cd6a51db03Ff2


Cannot use issue-colorpicker in GreaseMonkey "GM_addStyle is not defined"

Tittle pretty much covers it. Can't use GM_addStyle in greasemonkey 4 as it's been deprecated...

Hi @douglasg14b!
I may look into this (see https://github.com/Mottie/GitHub-userscripts/issues/29#issuecomment-354628733), but I've been leaning toward not supporting GM4. The GM_ commands in GM4 are not deprecated, support was completely dropped. Yes, I understand that a major version bump implies that there are breaking changes, but to implement that change without warning or discussion and then insist that the polyfill not have a version number makes the whole situation even more ridiculous.
Try Tampermonkey.

Yeah.... that's not nice of them at all.
A lot of my personal user-scripts completely broke when I updated to GS 4 in my browser. And I've all but abandoned my library of them because of it.
I didn't even realize I could get TamperMonkey for Firefox :o Could have sworn it used to just be GM.


Fix from date to date reversal

For the Committer

[ ] I made sure that the site looks great and functions perfectly live
[ ] I ran splell check
[ ] I'm ready for these changes to be made live

For the Reviewer
General Review

[ ] All changes/updates requested were intentionally changed or added (no extraneous file or partial updates)
[ ] The live rendered page behaves and looks like intended.
[ ] All links work as intended. (click every link to make sure that it takes the user to the expected place).
[ ] The markdown was rendered to HTML as intended. (Tables look right, code samples are gated properly).
[ ] Any style or CSS changes are tested on multiple different pages to ensure nothing unexpected broke or changed.
[ ] I proof-read the live site (gitbook serve) and there are no oblivious splelling or grammer misteaks.

Gitbook Specific

[ ] Any new documentation pages have been added to the SUMMARY.md file so they are rendered to HTML.
[ ] Any links to other pages within this documentation use relative links ([read more about ...](guides/voice/voicemail.md)).
[ ] Gitbook can install and build the documentation (gitbook install & gitbook build)

Code Specific

[ ] All code has been tested against the live API.
[ ] All code can be run without throwing errors or new warnings.
[ ] All code is formatted correctly and consistently (spaces, tabs).
[ ] All code is legible and idiomatic to the language.
[ ] All code uses sensical method, function, and variable names.
[ ] All code samples use the appropriate syntax highlighting.
[ ] Any examples re-use same phone numbers, uuids, variable names, etc... throughout (IE don't have something like {from: '+19191231234', to: '+19191231234'} unless it's meant to show the same phone number calling/texting itself ).

TL;DR

[ ] I did these things
[ ] I'm ready for these changes to be made live

 Usabilla Feedback Item - Hey guys! These definitions for toDate and fromDate are backwards and need to be switched. Thanks! :)

Preview Changes at:
http://bw-apdocs-Fix-fromDate-toDate-reversal.s3-website-us-east-1.amazonaws.com/


Add host allow/deny config?

Hi, thanks for the very useful image!
Netatalk offers config options for allowing/denying specific hosts or whole subnets, which sometimes is very handy:
http://netatalk.sourceforge.net/3.0/htmldocs/afp.conf.5.html
hosts allow = IP host address/IP netmask bits [ ... ] (V)
Only listed hosts and networks are allowed, all others are rejected. The network address may be specified either in dotted-decimal format for IPv4 or in hexadecimal format for IPv6.

Example: hosts allow = <IP_ADDRESS>/16 <IP_ADDRESS> 2001:0db8:1234::/48

hosts deny = IP host address/IP netmask bits [ ... ] (V)
Listed hosts and nets are rejected, all others are allowed.

Example: hosts deny = 192.168.100/24 <IP_ADDRESS> 2001:db8::1428:57ab

It would be nice to expose this via a parameter as well!
PS: This is a volume-level parameter, so it would need to be injected into the [<<volume-name>>] section in add-account.

I think at this point of granularity in config, the easiest way is just copy afp.conf from the container to your computer, set the config as you need it and then mount it in your continer.
Thanks!


tag exclusion not working

according to the flickr docs, the tags should be able to do "-people" and exclude that tag. however when i have :tags => '-people' as part of the search arguments, i get a result set of photos who have that tag associated with it.

I can reproduce what you are describing. It seems to be a bug from flickr, see http://www.flickr.com/groups/api/discuss/72157625456418732
As described on this page, try to add the parameter :tag_mode => "bool" and be sure to add a non exclusive tag in the tags list.

will try, thank you for the help!


image: remove bounds check in "At" and "Set" methods, with re slice

What version of Go are you using (go version)?
1.6

What operating system and processor architecture are you using (go env)?
linux amd64

I just saw this https://go-review.googlesource.com/#/c/20654/
Could we rewrite
func (p *RGBA64) RGBA64At(x, y int) color.RGBA64 {
	if !(Point{x, y}.In(p.Rect)) {
		return color.RGBA64{}
	}
	i := p.PixOffset(x, y)
	return color.RGBA64{
		uint16(p.Pix[i+0])<<8 | uint16(p.Pix[i+1]),
		uint16(p.Pix[i+2])<<8 | uint16(p.Pix[i+3]),
		uint16(p.Pix[i+4])<<8 | uint16(p.Pix[i+5]),
		uint16(p.Pix[i+6])<<8 | uint16(p.Pix[i+7]),
	}
}

to
func (p *RGBA64) RGBA64At(x, y int) color.RGBA64 {
	if !(Point{x, y}.In(p.Rect)) {
		return color.RGBA64{}
	}
	i := p.PixOffset(x, y)
	s := p.Pix[i:i+8]
	return color.RGBA64{
		uint16(s[0])<<8 | uint16(s[1]),
		uint16(s[2])<<8 | uint16(s[3]),
		uint16(s[4])<<8 | uint16(s[5]),
		uint16(s[6])<<8 | uint16(s[7]),
	}
}

I did some benchmarks on my package https://github.com/pierrre/imageserver/tree/master/image/internal (that borrows code from image).
And I see interesting results.

CC @nigeltao

Yeah, I'd be willing to accept a CL, if the results are sufficiently interesting.

How to check if bounds check are really removed?
Should I read assembly code with go build -gcflags="-S"?
What do they look like?

On Mon, Mar 21, 2016 at 1:15 AM, Pierre Durand<EMAIL_ADDRESS>wrote:

How to check if bounds check are really removed?
Should I read assembly code with go build -gcflags="-S"?
What do they look like?

Yes, that will work.  They are just a compare and branch where the
destination is a call to panicindex (or panicslice, for slice bounds
checks).
We could use a -d=bounds to print all the places where a bounds check
happens.  We have -d=nil and -d=wb for nil checks and write barriers,
respectively.  It might be kind of tricky to implement though.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/golang/go/issues/14884#issuecomment-199167924

FWIW, the correct line to use is s := p.Pix[i:i+8:len(p.Pix)].


Fix market errors

For some reason message.deletable and message.deleted no longer reliably work
also removed check for if a user is in the server because it was creating false negatives causing the market list to be empty
added a check for when the market list is empty too

fixes #593
refs #588


Unable to resolve module react-native-vector-icons/Feather

The development server returned response error code: 500
URL: http: //<IP_ADDRESS>:8081/index.android.delta?platform=android&dev=true&minify=false
Body: {
"originModulePath": "G:\ReactNative\pApp\node_modules\react-native-elements\src\helpers\getIconType.js",
"targetModuleName": "react-native-vector-icons/Feather",
"message": "Unable to resolve module react-native-vector-icons/Feather from G:\\ReactNative\\pApp\\node_modules\\react-native-elements\\src\\helpers\\getIconType.js: Module does not exist in the module map\n\nThis might be related to https://github.com/facebook/react-native/issues/4968\nTo resolve try the following:\n  1. Clear watchman watches: watchman watch-del-all.\n  2. Delete the node_modules folder: rm -rf node_modules && npm install.\n  3. Reset Metro Bundler cache: rm -rf /tmp/metro-bundler-cache-* or npm start -- --reset-cache.  4. Remove haste cache: rm -rf /tmp/haste-map-react-native-packager-*.",
"errors": [
{
"description": "Unable to resolve module react-native-vector-icons/Feather from G:\\ReactNative\\pApp\\node_modules\\react-native-elements\\src\\helpers\\getIconType.js: Module does not exist in the module map\n\nThis might be related to https://github.com/facebook/react-native/issues/4968\nTo resolve try the following:\n  1. Clear watchman watches: watchman watch-del-all.\n  2. Delete the node_modules folder: rm -rf node_modules && npm install.\n  3. Reset Metro Bundler cache: rm -rf /tmp/metro-bundler-cache-* or npm start -- --reset-cache.  4. Remove haste cache: rm -rf /tmp/haste-map-react-native-packager-*."
}
],
"name": "Error",
"stack": "Error: Unable to resolve module react-native-vector-icons/Feather from G:\\ReactNative\\pApp\\node_modules\\react-native-elements\\src\\helpers\\getIconType.js: Module does not exist in the module map\n\nThis might be related to https://github.com/facebook/react-native/issues/4968\nTo resolve try the following:\n  1. Clear watchman watches: watchman watch-del-all.\n  2. Delete the node_modules folder: rm -rf node_modules && npm install.\n  3. Reset Metro Bundler cache: rm -rf /tmp/metro-bundler-cache-* or npm start -- --reset-cache.  4. Remove haste cache: rm -rf /tmp/haste-map-react-native-packager-*.\n    at ModuleResolver.resolveDependency (G:\ReactNative\pApp\node_modules\metro\src\node-haste\DependencyGraph\ModuleResolution.js:158:1476)\n    at ResolutionRequest.resolveDependency (G:\ReactNative\pApp\node_modules\metro\src\node-haste\DependencyGraph\ResolutionRequest.js:92:16)\n    at DependencyGraph.resolveDependency (G:\ReactNative\pApp\node_modules\metro\src\node-haste\DependencyGraph.js:271:4465)\n    at dependencies.map.relativePath (G:\ReactNative\pApp\node_modules\metro\src\DeltaBundler\traverseDependencies.js:379:19)\n    at Array.map ()\n    at resolveDependencies (G:\ReactNative\pApp\node_modules\metro\src\DeltaBundler\traverseDependencies.js:378:16)\n    at G:\ReactNative\pApp\node_modules\metro\src\DeltaBundler\traverseDependencies.js:203:33\n    at Generator.next ()\n    at step (G:\ReactNative\pApp\node_modules\metro\src\DeltaBundler\traverseDependencies.js:296:307)\n    at G:\ReactNative\pApp\node_modules\metro\src\DeltaBundler\traverseDependencies.js:296:467"
}
processBundleResult
BundleDownloader.java: 231
access$100
BundleDownloader.java: 37
onResponse
BundleDownloader.java: 204
execute
RealCall.java: 135
run
NamedRunnable.java: 32
runWorker
ThreadPoolExecutor.java: 1133
run
ThreadPoolExecutor.java: 607
run
Thread.java: 761

I have same problem.
Is there any workaround in this time?

same problem

I think we need to update version v.4.4.0(minimum) to use Feather Icons

same problem

@gperfran @eltonxue please install latest version or min v.4.4.0 to support Feather Icons

i have above problem

I have the same issue when I uncomment the line: import { List, ListItem } from "react-native-elements";
I get the following error:
Loading dependency graph, done.
error: bundling failed: Error: Unable to resolve module react-native-vector-icons/Feather from C:\Ofir\react-native-facebook\Projects\test6Project\node_modules\react-native-elements\src\helpers\getIconType.js: Module react-native-vector-icons/Feather does not exist in the Haste module map

Same problem with latest version

remove the libraries
"react-native-linear-gradient": "^2.5.6",
"react-native-vector-icons": "^4.5.0",

and then Reload (cmd+R) the app


Broken contributing link in architecture docs

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/9ddffee2bb4ca204a91cd0f204792e3ebef3b31a/