Research input/output prior art

One goal of this project is to improve the interface to models and other data sources.
These were mentioned in #17:

xmodelform - particularly form field factories
django-betterforms - particularly MultiForm and MultiModelForm

Two  more things for inspiration:
https://github.com/Mechrophile/schemagic -- more schemas.
https://github.com/TriOptima/tri.form -- more forms, worth checking out.

In schematics
http://schematics.readthedocs.io/en/latest/usage/importing.html
http://schematics.readthedocs.io/en/latest/usage/exporting.html

Another snippet found when closing old prs:
+
+class MagicApp(AppConfig):
+    name = 'dal_select2'
+
+    def ready(self):
+        MagicFormMetaclass.register_formfield_for_modelfield(
+            model_field=models.ForeignKey,
+            form_field=Select2ModelChoiceField,
+        )
+
+        MagicFormMetaclass.register_formfield_for_modelfield(
+            model_field=models.OneToOneField,
+            form_field=Select2ModelChoiceField,
+        )
+
+        MagicFormMetaclass.register_formfield_for_modelfield(
+            model_field=models.ManyToManyField,
+            form_field=Select2ModelMultipleChoiceField,
+        )

https://github.com/yourlabs/django-autocomplete-light/pull/579/files
Maybe working maybe not working example :joy:


Hello. Does the prefetch address generated by Pythia sequentially access L1D, L2, LLC, and DRAM? I mean, does the prefetch address generated by Pythia access the cache and main memory in a traditional manner?

Hello. Does the prefetch address generated by Pythia sequentially access L1D, L2, LLC, and DRAM? I mean, does the prefetch address generated by Pythia access the cache and main memory in a traditional manner? @rahulbera

Yes.


chore: Fix repository links

Description
Addressing #115 and adding tree/main/ to the repository URLs
Type of Change

[ ] ✨ New feature (non-breaking change which adds functionality)
[ ] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 Code refactor
[ ] ✅ Build configuration change
[x] 📝 Documentation
[ ] 🗑 Chore

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (520910f) 75.06% compared to head (e46eca9) 75.06%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #116   +/-   ##
=======================================
  Coverage   75.06%   75.06%
=======================================
  Files          34       34
  Lines        1115     1115
=======================================
  Hits          837      837
  Misses        278      278

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


map returns copy

addresses #15

I'm not quite sure what childrenWithProperty() does, and why we need it. Could you elaborate?

I see. I don't think it's a good idea to add that export to children.js, since I don't think it's more broadly useful. Even if we did need it, it's probably more suited to a private function in the map module. That said, I'm not quite sure we need it, each invocation of _map() includes a pointer to the parent AND the property, and that's where we set parent[property] as well, so I'd just clone node if there was no result, and set parent[property] anyway.

Quasi-related: I wonder if transform() might be a better name than map(). I chose map() as it relates to an existing concept authors may be familiar with, but if it doesn't have enough similarities that could hinder, rather than improve usability.

Quasi-related: I wonder if transform() might be a better name than map(). I chose map() as it relates to an existing concept authors may be familiar with, but if it doesn't have enough similarities that could hinder, rather than improve usability.

I definitely support this, to me, when I think of map, I think of something linear, not a complex tree structure like we have

Let's merge this, the other issue is entirely orthogonal!


Support Meting API

Reference: https://github.com/metowolf/MetingJS
cc @metowolf

Make self-hosted Meting API: https://github.com/metowolf/MetingJS/issues/1#issuecomment-349229512

It will be great if you can give an exact example how to use the Meting API.

Support MetingJS in version 3.0.0-beta.5
Please report us if you find any problem.


can't update role

Hello ,
when i try to update a role i get this error
Unknown permission
C:\xampp\htdocs\qcm\vendor\laravel\framework\src\Illuminate\Foundation\Application.php#1043
any help

Hello 👋🏻
Are you doing anything different from the installation instructions? Can you please provide more details?


Test snippets factory

I want to test this out a bit before merging. Are the 5000 dependencies required?


Voice

Will it be possible to add voice commands to change the URLs between the webviews? or to control the screens

Hi, I don't know if I can add voice commands, but I'll try to work on it.

You amazing man


Implement typing Literal support

After #1640 stops the crashes, we'll have to add real Literal support. This involves:

Adding a new class PythonLiteral and adding it to the typing module.
Modify the SpecificTypeConstructor delegate type to accept IMembers instead of IPythonTypes, since we now need to be able to have non-types.
Potentially undo some of #1631, since None would be a valid non-type and could be passed through, allowing NoneType to return with None as its sole instance.

Related to #535, but this is a more specific case.

This is probably going to require some hefty changes to the TypeAnnotation stuff, since right now if a constant string is seen, we assume that it's supposed to be another type and parse it, but if it's a literal, we don't want to do that. (Maybe we shouldn't be doing that at all past a top level annotation being a string; need to read the spec.)
That and TypeAnnotation also relies on annotations be solely constructed with IPythonType.

FWIW the new LS we released today has full typing support: https://devblogs.microsoft.com/python/announcing-pylance-fast-feature-rich-language-support-for-python-in-visual-studio-code/


protoc-gen-go: fix generation of proxy getters for distant types

Skip generating getters for fields that are not defined in the same file
as the publicly imported message.
This is the wrong way to fix this problem. It is, however, the expedient
one to hold us over until (soon, I hope) we completely redo public
imports as type aliases and make all of this moot.
The scenario:
a.proto publicly imports b.proto
b.proto has a message M
M has a field F of type T
We generate forwarding methods for public imports.
// T is defined in the same Go package as a.proto.
func (m *M) GetF() T { ... }
Depending on what package T is defined in, we might need to qualify its
name (foo.T), and we might need to add an import for that package.
That's not what we used to do, however: Instead, we'd only generate
the GetF forwarder if T is defined in b.proto.
Commit 9d4962b4 made it so that we'd generate the forwarder if it T is
defined in the same Go package as b.proto, which seems safe and
reasonable. It turns out that something elsewhere in the generator is
getting confused by this, however. Rather than figuring out what--which
would also change a lot of generated code by adding forwarding methods
that didn't used to be there--this just reverts back to the behavior of
looking at files rather than packages.
All of this goes away completely once we start using type aliases,
because we don't need any forwarding methods at all at that point.

Is this up for review or you still working on it?

Up for review. Sorry, I thought I'd already added you to it.


[BUG] 404 missing url api/plugin

When you want to print labels, note the red message in the background:

Do you have plugins enabled on the server?

It is coming from demo.inventree.org, possibly plugins not enabled.
But it would be nice to avoid this red message even if no plugins enabled.

Fix incoming at https://github.com/inventree/InvenTree/pull/2785

Thank you!


feat(SPV-897): introduce correct path and command for js regression tests

Pull Request Checklist

[x] 📖 I created my PR using provided  : CODE_STANDARDS
[x] 📖 I have read the short Code of Conduct: CODE_OF_CONDUCT
[x] 🏠 I tested my changes locally.
[ ] ✅ I have provided tests for my changes.
[x] 📝 I have used conventional commits.
[ ] 📗 I have updated any related documentation.
[ ] 💾 PR was issued based on the Github or Jira issue.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 46.87%. Comparing base (24e726e) to head (82f1bf0).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #680      +/-   ##
==========================================
- Coverage   46.89%   46.87%   -0.02%
==========================================
  Files         271      271
  Lines       12831    12831
==========================================
- Hits         6017     6015       -2
- Misses       6220     6222       +2
  Partials      594      594

Flag
Coverage Δ

unittests
46.87% <ø> (-0.02%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file with indirect coverage changes

Continue to review full report in Codecov by Sentry.

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


fix unnecessary boxing/unboxing

Most of the changes make the code just more readable and the compiler does auto(un)boxing where necessary.
Replacing Boolean.valueOf() and Float.valueOf() on the other hand completely avoids a boxing/unboxing cycle because in both cases the result is stored in a primitive.

Please close this one (I can close it through the commit message but forgot, twice)


refactoring util away

Hello @amirziai! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file flatten_json/__init__.py:

Line 194:80: E501 line too long (98 > 79 characters)


docs: update book to include 'forge inspect --pretty  abi' example

This change provides an example using forge inspect --pretty <contract> abi included in https://github.com/foundry-rs/foundry/pull/5306

@trevorgjohnson the correct sintax is forge inspect <contract> abi --pretty instead of forge inspect --pretty <contract> abi


[Request] Tftpd64 / Tftpd32

Tftpd64 is a free, opensource IPv6 ready application which includes DHCP, TFTP, DNS, SNTP and Syslog servers as well as a TFTP client.
http://tftpd32.jounin.net/tftpd32_download.html

If these are cli apps, perhaps they should live in the main bucket?

This is a GUI app.


fix(oembed): relax regex for detecting XML responses

both text/xml and application/xml are valid content types for OEmbed-XML responses
fix #48
/cc @royfielding

Coverage remained the same at 98.452% when pulling fb17df164d9fe38ed688367a1860b66439839db3 on trieloff:soundcloud-oembed into a08dc9ac016910dda1c2c330aac7f1182c016bf6 on jacktuck:master.

Coverage remained the same at 98.452% when pulling fb17df164d9fe38ed688367a1860b66439839db3 on trieloff:soundcloud-oembed into a08dc9ac016910dda1c2c330aac7f1182c016bf6 on jacktuck:master.

Thanks for this. Will merge and release later :)

Hi @jacktuck I hope you had a good weekend. Let me know if there is anything else that needs to be done with this PR.

Should be all good. Have done a release. I really need to add some CI to this project 😆

Let me know if you'd like a PR for a CircleCI + Semantic Release set-up.


Node.js bindings: Add JavaScript bindings for UART

This patch provides native and highlevel JavaScript bindings
for UART. Also, it adds enum-string translation functions to
soletta.
Signed-off-by: Sudarsana Nagineni<EMAIL_ADDRESS>Signed-off-by: Srinivasa Ragavan<EMAIL_ADDRESS>

@barbieri @gabrielschulhof please review

@glima and @ibriano can take a look.

Replaced by  #1804


[zh] Resync page architecture/leases.md

Resync page architecture/leases.md

Looks like we hit a conflict?

Looks like we hit a conflict?

a～oh…

A dup of #39428.
Before you open a PR, please search for the PR list to check any possible conflicts.


[CELEBORN-823][DOC] Add Celeborn architecture document

What changes were proposed in this pull request?
As title.
Why are the changes needed?
As title.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
No.

cc @pan3793

is svg editable? usually we should provide the editable source file of image too

ping @pan3793 @AngersZhuuuu comments addressed, please take a look again

is svg editable? usually we should provide the editable source file of image too

I'm not sure about this, I just followed the docs/assets/logo/celeborn-*.svg files. They are relative small.

... They are relative small.

@waitinfuture I'm not concern the size, I mean we should provide the "editable source file" of image in case we want to modify it in the future. how do you generate the svg?

... They are relative small.

@waitinfuture I'm not concern the size, I mean we should provide the "editable source file" of image in case we want to modify it in the future. how do you generate the svg?

I replaces the files with editable svg format, drawn in https://app.diagrams.net/

Thanks, merging to main.


Use node >=6.11.2

Coverage decreased (-0.05%) to 90.712% when pulling eb652d6238bb8f60542a9c3c3d148038da1a65f6 on SAP:bump_node into 358b6d3cf235793178228c67559c3ad3d857b2bd on cloudfoundry-incubator:master.

Coverage remained the same at 90.719% when pulling 9109b794b37e27a2282a56f5ad33361d418fc7da on SAP:bump_node into b6ea0e9d686fcf23b47db6fd2b54719684842b70 on cloudfoundry-incubator:master.

Coverage decreased (-0.1%) to 90.616% when pulling 9b5c0abf30c31903a5fa2122048ec7978cf8660c on SAP:bump_node into b6ea0e9d686fcf23b47db6fd2b54719684842b70 on cloudfoundry-incubator:master.

We already have PRs for 6.11.2 and 8.0.0. Closing this one


Rename "Atom Helper" on Mac

I followed the instructions at https://github.com/atom/atom-shell/blob/master/docs/tutorial/application-distribution.md:
"You can rename Atom.app to whatever you want, and you also have to rename the CFBundleDisplayName, CFBundleIdentifier and CFBundleName fields in following manifest files if they have these keys:
Atom.app/Contents/Info.plist
Atom.app/Contents/Frameworks/Atom Helper.app/Contents/Info.plist"
I did exactly that and it mostly works:
When running the default_app, in Activity Monitor, the parent process is now renamed correctly, and one of the Helper's is also renamed correctly (the one with an icon), but the Helper with no icon is still called "Atom Helper" in Activity Monitor.
How can I get this last remaining "Atom Helper" (the Helper without the icon) to be renamed?
I would prefer to not have to rebuild Atom on all platforms.
Also, I noticed this commit in iojs: https://github.com/iojs/io.js/commit/3d46fefe0cb5abf45bb6a21e5d0a377c58f0f896 ("win,node-gyp: allow node.exe/iojs.exe to be renamed"). Will this allow the .exe to be renamed without breaking native node modules?
Finally, in the documentation regarding renaming the .exe, it's not clear whether native node modules refers to core modules shipped with Node (e.g. tls) or third party modules which have native bindings?

Currently Atom Helper is hard-coded, so the only way to change it is to rebuild with your own branding. I think we should probably search for Helper app instead of hard-coding the path when initializing.
I'm merging this to #686.

Also, I noticed this commit in iojs: iojs/io.js@3d46fef ("win,node-gyp: allow node.exe/iojs.exe to be renamed"). Will this allow the .exe to be renamed without breaking native node modules?

Yeah.

Finally, in the documentation regarding renaming the .exe, it's not clear whether native node modules refers to core modules shipped with Node (e.g. tls) or third party modules which have native bindings?

It refers the third party modules with native bindings.

Thanks, it would be great if Atom-Shell could search for a "... Helper" app in Frameworks.

I would prefer to not have to rebuild Atom on all platforms.

fwiw, this is exactly what we do at Slack. It's not that bad!

Do you do this using VMs? Could you give some tips as to how you automate it?

@jorangreef https://github.com/paulcbetts/grunt-build-atom-shell will do everything for you, including fix up native modules


Can't run with a static vm with a disk unless you specify disk_size

I got this error:
2016-04-12 10:07:11,731 ee17f302 MainThread fio(1/1) ERROR    Error during benchmark fio
Traceback (most recent call last):
  File "/usr/local/google/home/ehankland/PerfKitBenchmarker/perfkitbenchmarker/pkb.py", line 381, in RunBenchmark
    detailed_timer)
  File "/usr/local/google/home/ehankland/PerfKitBenchmarker/perfkitbenchmarker/pkb.py", line 269, in DoRunPhase
    collector.AddSamples(samples, name, spec)
  File "/usr/local/google/home/ehankland/PerfKitBenchmarker/perfkitbenchmarker/publisher.py", line 598, in AddSamples
    sample['metadata'], benchmark_spec)
  File "/usr/local/google/home/ehankland/PerfKitBenchmarker/perfkitbenchmarker/publisher.py", line 168, in AddMetadata
    data_disk.disk_size * data_disk.num_striped_disks)
TypeError: unsupported operand type(s) for *: 'NoneType' and 'int'

Looks like the problem is this line

@noahl This looks related to your disk refactor - mind taking a look?

@ehankland yep, I've got it. Looks like with the old metadata keys, we just ignore or use None as the size if we don't know it, but the computation assumes that we actually have that information.

This should be fixed with the merge of #946 . Closing now, but please reopen if you see more problems.


[sorting] should "foo" comes first before "foo - bar"?

For example, in file folders.postman_collection.json I changed auth to Auth - auth and here's the result:
$ npx @matthewlein/postman-collection-sorter folders.postman_collection.json
npx: installed 2 in 2.312s
Sorting collection file:  folders.postman_collection.json
Sorting Successful!
Sorted collection saved to: folders.postman_collection.sorted.json
$ jq -C '.item[].name' < folders.postman_collection.sorted.json
"Animals"
"Auth - auth"
"Auth"
"bikes"
"Bikes"
"Fruit"
"users"
$ printf '%s\n' Auth 'Auth - auth' | sort
Auth
Auth - auth
$

You're right, there was a small syntax error in the sorting. Should be fixed now

Verified and it works fine now.


Update patient phone constraint and add profile command in summary

Summary

Resolves #238 by changing the constraint details in the UG

Resolves #241 by adding profile command into summary of UG

Checks

[X] Link PR to issue
[X] Add Reviewer
[ ] Pass CI

Codecov Report

Merging #246 (2468364) into master (a488e21) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #246   +/-   ##
=========================================
  Coverage     73.35%   73.35%
  Complexity     1063     1063
=========================================
  Files           146      146
  Lines          3442     3442
  Branches        402      402
=========================================
  Hits           2525     2525
  Misses          771      771
  Partials        146      146

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 a488e21...2468364. Read the comment docs.


[WFRP4e Core] Ranged/Size Modifiers Working Incorrectly

A few weird things going on here with range modifiers and size considerations:

Range modifiers are calculating incorrectly given target size. Range modifiers for size seem to be only adding +10 per size level larger the target is than the attacker.

Example: a clan rat with a bow shooting a basilisk should get a +40 modifier (shooting at an Enormous target per the chart on CRB p.161) plus possibly another +10 if the attacker is smaller than the target (CRP p. 341, smaller attacker bonus). Instead the clan rat is incorrectly getting a +20 bonus (assume this is happening because the basilisk is 2 sizes larger than the clan rat, so the ruleset is giving a +10 per size larger bonus to the attempt).

Using the above example to highlight a 2nd issue: the calculated modifier is not being taken into account to determine if the roll was successful or not. The behavior I'm seeing is as follows: If the BS roll isn't successful using the attacker's unmodified BS value then it's considered a miss, even though with modifiers it may have been a hit. If the roll is successful using the attacker's unmodified BS value then it's considered a hit (as it should) and the modifier is correctly used to calculate SL.

Example: clan rat has an unmodified BS of 30. If it targets a basilisk and the roll is 31 or higher (an unmodified miss) the test logs as a miss in chat regardless of appropriate modifiers, with no way to rectify, and chat reports the test results "vs 30". If the BS roll was 30 or lower it will then log a hit, but also apply the modifier (currently incorrectly calculated per issue #1) which results in a correct SL calculation. Basically any BS modifiers are not taken into account to determine whether the BS check was initially a hit or not.From<EMAIL_ADDRESS>3.3.3wfrp4e-core: 1.2.0wfrp4e-starter-set: 1.3.0

Hi,
The above problem persists. However I think that the 2nd calculation above is not correct.
The WFRP Rulebook states that if the Size modifier allows you to hit when the test would have otherwise failed then you succeed but only with +0 SL.
Example: clan rat has an unmodified BS of 30. Disregarding other modifiers to the test, if it targets a basilisk (+40 due to size) and the roll is between 31 - 70 the test should log as a success with a SL of +0.

Also just to add to the above Size should not add any modifiers in close combat.
It is supposed to be only for ranged combat.

This will be resolved in 3.5.0 where I plan on changing it to a bonus in the dialog instead of after the test, and Close/ranged is for the player to decide.

@ArisKrik See my comment on your other issue. There is a +10 bonus to attacking larger creatures.

@ArisKrik As I am unaware of any other way to contact you, please see my comment on the unrelated issue #387.
If I could have a discord tag or email to better contact you directly, I don't see you on either discords if you have a different name there.


Remove properties on diagnostics logs

When turning on diagnostics logs, and send them to Azure Monitor logs, the request and response body are also being logged, which is an issue with privacy of user data. is there a way to prevent it from being logged, while still getting the reset of the properties?

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

ID: ecc5e84d-603c-6f88-a941-df33aad0e1d8
Version Independent ID: ae6976bb-aadd-4d40-bb62-57589265f4ff
Content: Monitor published APIs in Azure API Management
Content Source: articles/api-management/api-management-howto-use-azure-monitor.md
Service: api-management
GitHub Login: @vladvino
Microsoft Alias: apimpm

@amir-microsoft , Thanks for raising the issue. We will investigate and update you.

@amir-microsoft, may I know Api Management instance name in the subj, so I can look into it? Thanks!

Thank you for getting back to me @promoisha
This is more of a general product question, and I rather not share our instance details on a public thread,
If you still need it, maybe we can continue over email?
Thanks!

@amir-microsoft sure, my alias is glfeokti

@amir-microsoft Just following if you still have any questions .

Thanks @DixitArora-MSFT and @promoisha,
seems like the issue is resolved by setting the Azure monitor logging to zaro bytes of the body resolved the issue

@amir-microsoft Thanks for the update. For now we will proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Make Screen Resoluction and FPS configurable

It's now hardcoded in RoomClient.js, is it possiable to make to configurable?
Its will be very good if I can set it in settings (in web page).

@YiPrograms
is it possiable to set in frontend?

Fixed in d1ffe6cd439266e973fe82580ed882b864507a25


Support interactiveApi with the full-screen Scaling interactive [#181103123]

This change upgrades the lara client client to the current version and passes the full initInteractive message passed by AP to the fullscreen interactive mixed in with the current initMessage the fullscreen interactive creates.  The interactiveApi in CFM needs the info in the full message.
It was decided in this PR to only address the fullscreen interactive.  This same init message passing may need to be done for the other interactives that use the iframe-runtime container.

@scytacki I added a couple of commits since your approval:

https://github.com/concord-consortium/question-interactives/pull/156/commits/3788936d7059217ae3832210cd6e9623ef44f2db : this removes the escaping of the url in the iframe runtime
https://github.com/concord-consortium/question-interactives/pull/156/commits/4b31f0ff881f5986018ece63be1454458bf81ac4 : this fixes a line to set the initMessage that got dropped in the previous commit


ENV escaping behavior is confusing

(This came up when I tried to construct dockerfile programmatically.)
It looks like the parsing of ENV lines is a rough hack.  Specifically, it seems that:

If the line ends, no error is thrown if a variable is mid-value: ENV VAR="1234 results in VAR being set to 1234.

Backslashes are implemented as half-quote-chars, applicable only for double quotes and dollar signs?  What I see is that

ENV VAR="12\"34" works as expected,
ENV VAR="12\34" has a backslash between the 2 and the 3,
ENV VAR="12\\34" has two backslashes.

As a result, you cannot use ENV to have a backslash before a $variable or at the end of the string.  For dockerfile-generation tools, the implication is usually more severe, they should just avoid backslashes completely (or become very complicated).

In addition, it looks like there is no way to have a newline in the string value?

Is there a way to get some consistent quoting scheme?  (I expected some json-like syntax, but looks like there isn't one?)
Given all of these, it would be nice if the parsing got cleaned up to make backslashes be proper escape characters similarly to how they're treated in (double-quoted) shell strings, which would imply throwing an error for \x with an unknown x, which in turn would make it possible to parse \n as a newline.
But I'm sure that you'd worry about backward compatibility so the minimum that could be done with this is clarify these limitations in the reference page.

We should have a JSON form.
On 27 Mar 2017 15:09, "Eli Barzilay"<EMAIL_ADDRESS>wrote:

(This came up when I tried to construct dockerfile programmatically.)
It looks like the parsing of ENV lines is a rough hack. Specifically, it
seems that:

If the line ends, no error is thrown if a variable is mid-value: ENV
VAR="1234 results in VAR being set to 1234.
Backslashes are implemented as half-quote-chars, applicable only for
double quotes and dollar signs? What I see is that

ENV VAR="12"34" works as expected,

ENV VAR="12\34" has a backslash between the 2 and the 3,
ENV VAR="12\34" has two backslashes.

As a result, you cannot use ENV to have a backslash before a $variable
or at the end of the string. For dockerfile-generation tools, the
implication is usually more severe, they should just avoid backslashes
completely (or become very complicated).
In addition, it looks like there is no way to have a newline in the
string value?
Is there a way to get some consistent quoting scheme? (I expected some
json-like syntax, but looks like there isn't one?)
Given all of these, it would be nice if the parsing got cleaned up to make
backslashes be proper escape characters similarly to how they're treated in
(double-quoted) shell strings, which would imply throwing an error for \x
with an unknown x, which in turn would make it possible to parse \n as a
newline.
But I'm sure that you'd worry about backward compatibility so the minimum
that could be done with this is clarify these limitations in the reference
page.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/docker/docker/issues/32140, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdcPEaLudYJCgtVKEYwg7KEFXwRuZtKks5rp8MngaJpZM4MqUej
.

/cc @tonistiigi @tiborvass @duglin @simonferquel

Tried to add a JSON form some time ago, but it was rejected since we didn't want to have more json in the Dockerfile.

it looks like there is no way to have a newline in the string value?

I think you can use single quotes.

Tried to add a JSON form some time ago, but it was rejected

Maybe we can try again? I would LGTM that PR.

@elibarzilay I renamed the issue to be more clear about the actual issue, which is on ENV 👼

@dnephin: I tried single quotes -- looks like they're more predictable, but still no newlines.  Using
FROM ubuntu
ENV FOO='x
y'

I get Error response from daemon: Unknown instruction: Y'.  I even tried the bash-like $'x\ny' thing, and that didn't work also (with more questionable behavior: no complaint about the $).
Is there something else I'm missing?
@vdemeester: well, "confusing" sounds fine -- but in general I was referring to the fact that such parsing results are something you get from a naive "regexp-based parsing"...

@elibarzilay it's not regex-based parsing. The implementation is in builder/dockerfile/shell_parser.go.
When this was written there may not have been any good options for shell parsing libraries. Now there are a few, and we even use a couple already. It might be a good idea to use a more complete library, which could fix some of these issues. Although I suspect we'll have to fork them to implement the escapeToken hack that was added for windows Dockerfiles.

The env var processing was based on what we see from the shell. For example:
$ foo="12\34" env | grep foo
foo=12\34

is what I see from bash and sh.
And for \n:
$ foo="12\n34" env | grep foo
foo=12\n34

so I'm not sure there's an issue.

I take that back, there might be an issue on double-\ :
$ foo="12\\34" env | grep foo
foo=12\34

I can take a look at that after CNCFcon this week.

@duglin  I think the "newline in single quotes" example in https://github.com/docker/docker/issues/32140#issuecomment-289818734 is relevant as well?
Do you think it's worth maintaining our own implementation of this?

@duglin: yes, backslash in a double-quoted string is left alone if it's followed by something that it doesn't know about.  The relevant part in the man page is:

The backslash retains its special meaning only when followed by one of the following characters: $, `, ", \, or <newline>.

I'd like to address these issues slowly to ensure we don't go too far too fast.  So, PR #32328 address part of this - the double-backslash in double-quotes and will now generate an error is the trailing " or ' is missing.

Newlines are just a pain and may never work correctly until we don't need to worry about backwards compatibility. The biggest issue I see with them is that we try to parse things on a line-by-line basis, and we don't really know what type of Dockerfile command we have until later. This means that the logic to split things up based on quotes isn't done until after we have what we think is "a line".  The biggest exception to that is the continuation char at the end - which, obviously, will try to wrap lines. But even then its not great since \\ at the end should not be treated as a \ but it is and fixing that involves a more char-by-char analysis of the line.
@elibarzilay where in the man pages does it talk about \x expansion? I see it for echo and printf but its not clear to me those apply to env vars.

I believe the behaviour has changed between recent versions.
Previously this was supported:

ENV "phpopts_suhosin.get.max_value_length=2048"

Now in docker 17.06.0-ce and presumably other versions this raises a parsing error and it is impossible to set environment variables that contain a period.

@macropin If you wrap the key in quotes then it seems to work.
FROM scratch
ENV "phpopts_suhosin.get.max_value_length"=2048

$ docker build .

Sending build context to Docker daemon  1.693MB
Step 1/2 : FROM scratch
 --->
Step 2/2 : ENV "phpopts_suhosin.get.max_value_length"=2048
 ---> Running in a8db2ccd3d35
Removing intermediate container a8db2ccd3d35
 ---> eb8b6069ff42
Successfully built eb8b6069ff42

$ docker inspect eb8b6069ff42 --format='{{.ContainerConfig.Env}}'

[PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin phpopts_suhosin.get.max_value_length=2048]

Actually, looks like you don't even need quotes.
$ cat Dockerfile
FROM scratch
ENV phpopts_suhosin.get.max_value_length=2048

$ docker build .
Sending build context to Docker daemon  1.693MB
Step 1/2 : FROM scratch
 --->
Step 2/2 : ENV phpopts_suhosin.get.max_value_length=2048
 ---> Running in adeb8478b6d4
Removing intermediate container adeb8478b6d4
 ---> 0d328c2faa23
Successfully built 0d328c2faa23

YEah I've got
ENV REACT_APP_API=http://abcdefg.com/api
its not working... why dont they just make it JSON!


P2p

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@be6da7c). Click here to learn what that means.
The diff coverage is 25.6%.

@@            Coverage Diff            @@
##             master       #1   +/-   ##
=========================================
  Coverage          ?   29.67%
=========================================
  Files             ?        8
  Lines             ?      765
  Branches          ?        0
=========================================
  Hits              ?      227
  Misses            ?      476
  Partials          ?       62

Impacted Files
Coverage Δ

p2p.go
25.6% <25.6%> (ø)

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 be6da7c...12d71ef. Read the comment docs.


Use Packet Terraform Provider

Description
Packet has a provider on Terraform we can use to manage instances. Where we used the Packet library before, we can now use the Terraform provider. Using it will increase task reproducibility as well as our overall confidence in the underlying infrastructure used to run tasks.
My thought on how the complete setup should look like is:

[x] Having a "main" Terraform directory hosting the required configuration to run a single-node on Packet. #77
[x] For each task we run, copy that "main" Terraform directory to the task's build directory. #93
[x] Proceed to initialize Terraform in the task's build directory. #93
[x] Plan and Apply the configuration. #83
[x] Later on, destroy the plan. #99 #101

Because these steps are too numerous and broad, this issue will focus on the first step. But will be used as a reference for future issues.

All related issues and pull requests closed, closing this issue.


pkg/ansible/controller; Support for hybrid user-driven status

Description of the change:
Adds support for hybrid managed status between AO and user's Ansible. Get's latest version of resource to ensure that we are updating the latest version of the resource.
Motivation for the change:
Hybrid use case panics when user updates status from Ansible task.

@dymurray Before we merge this we need to make sure that we add some migration steps to some sort of doc to describe what it looks like if you're using a CRD without sub-resources and what to do to fix it.

Just correcting Shawn's comment, this requires v1.11 of Kubernetes by default:
https://github.com/kubernetes/kubernetes/pull/63598

fixes: https://github.com/operator-framework/operator-sdk/issues/809 as we're turning on the status subresource by default which is beta in k8s 1.11 per kubernetes/kubernetes#63598
Although I would like to see a manual confirmation of the status subresource being on by default (and not behind a feature gate) on a k8s 1.11 cluster.

@dymurray Have you looked into the Helm error in this build at all? I'm seeing the same error trying to add a status subresource to a CRD.
https://travis-ci.org/operator-framework/operator-sdk/builds/464000402?utm_source=github_status&utm_medium=notification

I think it's a bug fixed in 1.11.3 here, but I'm not positive. With 1.11.2 the helm e2e test works if I change the operator-sdk new command to --kind=Memcached. With 1.11.3, the helm e2e test works as is.

@joelanford I haven't dug into it just yet but your link looks promising. I will report back if I find anything. It only began happening when I added the status writer to the helm controller.

/hold cancel

@dymurray Can you please update the PR title and description to more clearly say that we're enabling the status subresource by default.

Helm CI tests are related to the bug Joe mentioned above. I ran the tests with kubectl version 1.11.3 and the error went away.

I have removed and client library bump from this PR because of #807

Thanks @hasbro17, added the note to the changelog.

My bad, I forgot to squash before merging.


adding fields on jobs offers

Issue https://github.com/kdti/kdti-server/issues/52

Does it close any open issues?

Great!!


Landmark Positions

Looks like you guys published the paper.
What did you use for ground truth annotation?

Hi sorry for the late reponse. Please refer to this issue: https://github.com/XunshanMan/Object-oriented-SLAM/issues/2


allow publishing to own infrastructure

I don't want to use your https://*.flowerapp.sites
I want to push it to my own s3/vercel/githubpages/ssh/whateverhostingbbqomg

@airtonix thanks for the feedback and that's already possible and we can make that clearer on the front page that "self-publishing" is possible as well.

FIXED. @airtonix this is resolved with the release of self publishing tutorial and functionality https://flowershow.app/docs/publish-tutorial


Proxy

Add proxy param to login session

@ob1vanq are you planning the complete this PR? if you don't have to time, I can do it.

I would really appreciate if the suitable proxy support would be added, Thanks :+1:

i use
export HTTPS_PROXY="https://<IP_ADDRESS>:8080"
i get
Session file does not exist yet - Logging in.
Traceback (most recent call last):
  File "/usr/local/bin/instaloader", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.9/dist-packages/instaloader/__main__.py", line 471, in main
    _main(loader,
  File "/usr/local/lib/python3.9/dist-packages/instaloader/__main__.py", line 110, in _main
    instaloader.login(username, password)
  File "/usr/local/lib/python3.9/dist-packages/instaloader/instaloader.py", line 634, in login
    self.context.login(user, passwd)
  File "/usr/local/lib/python3.9/dist-packages/instaloader/instaloadercontext.py", line 214, in login
    csrf_json = self.get_json('accounts/login/', {}, session=session)
  File "/usr/local/lib/python3.9/dist-packages/instaloader/instaloadercontext.py", line 325, in get_json
    resp = sess.get('https://{0}/{1}'.format(host, path), params=params, allow_redirects=False)
  File "/usr/lib/python3/dist-packages/requests/sessions.py", line 555, in get
    return self.request('GET', url, **kwargs)
  File "/usr/lib/python3/dist-packages/requests/sessions.py", line 542, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/lib/python3/dist-packages/requests/sessions.py", line 655, in send
    r = adapter.send(request, **kwargs)
  File "/usr/lib/python3/dist-packages/requests/adapters.py", line 439, in send
    resp = conn.urlopen(
  File "/usr/lib/python3/dist-packages/urllib3/connectionpool.py", line 696, in urlopen
    self._prepare_proxy(conn)
  File "/usr/lib/python3/dist-packages/urllib3/connectionpool.py", line 966, in _prepare_proxy
    conn.connect()
  File "/usr/lib/python3/dist-packages/urllib3/connection.py", line 359, in connect
    conn = self._connect_tls_proxy(hostname, conn)
  File "/usr/lib/python3/dist-packages/urllib3/connection.py", line 500, in _connect_tls_proxy
    return ssl_wrap_socket(
  File "/usr/lib/python3/dist-packages/urllib3/util/ssl_.py", line 453, in ssl_wrap_socket
    ssl_sock = _ssl_wrap_socket_impl(sock, context, tls_in_tls)
  File "/usr/lib/python3/dist-packages/urllib3/util/ssl_.py", line 495, in _ssl_wrap_socket_impl
    return ssl_context.wrap_socket(sock)
  File "/usr/lib/python3.9/ssl.py", line 500, in wrap_socket
    return self.sslsocket_class._create(
  File "/usr/lib/python3.9/ssl.py", line 997, in _create
    raise ValueError("check_hostname requires server_hostname")
ValueError: check_hostname requires server_hostname


Support loading symbols at configurable base addresses

TODO
The last feature that needs to be added to this branch is that we need to remove the decompilation pane when we are outside the memory range of the manually set mem location.
This is a problem because how do we know if someone is at the end of a memory section?
We could:

create another command called --base-addr-end and change the old one to --base-addr-start but that sounds like a really shitty thing to make the user do
we could try and search for the end given we know the name of the current memory region... this scares me though. perhaps we should just go with the first approach.


nrfx uart/uarte driver: allow non-standard baudrates?

Is your enhancement proposal related to a problem? Please describe.
I have to use non-standard baudrates like 10400. This is not supported by the nrfx uart/uarte drivers.
Describe the solution you'd like
The driver implements setting the baudrate based on the values given in the datasheet for the nRF52840 SOC. But in https://devzone.nordicsemi.com/f/nordic-q-a/391/uart-baudrate-register-values a simplified formula to calculate the baudrate register value for non-standard baudrates is given. The values calculated by the formula are not exact and can differ from the values given in the datasheet (e.g. baud 14400: datasheet value = 0x0x003AF000, formula value = 0x003B0000). The function baudrate_set of the driver could calculcate the baudrate based on the formula only if a non-standard baudrate is given (i.e. in the default case, instead of returning -EINVAL). The cases for the baudrates 300 and 600 then could be removed because they are not supported by the Nordic HAL and the formula calculates the currently implemented values correctly for these baudrates.

You can try to add your own baudrate by inserting new parameters in the folowing files:
zephyr/drivers/serial/uart_nrfx_uart.c
zephyr/drivers/serial/uart_nrfx_uarte.c
Into the function «static int baudrate_set(const struct device *dev, uint32_t baudrate)»:
case 10400:
	/* value not supported by Nordic HAL */
	nrf_baudrate = 0x002AA000;
	break;

P.S.:  Actual baudrate will be 10406.494140625

The manual contains inaccurate values of the BAUDRATE register, because there is no formula for calculating the baudrate directly in the manual itself. Apparently the employees made mistakes when calculating manually. See for yourself what's going on in the «nRF51 Series Reference Manual Version 3.0.1»:
Value Id    Value       Description
Baud1200    0x0004F000  1200 baud (actual rate: 1205)
Baud2400    0x0009D000  2400 baud (actual rate: 2396)
Baud4800    0x0013B000  4800 baud (actual rate: 4808)
Baud9600    0x00275000  9600 baud (actual rate: 9598)
Baud14400   0x003B0000  14400 baud (actual rate: 14414)
Baud19200   0x004EA000  19200 baud (actual rate: 19208)
Baud28800   0x0075F000  28800 baud (actual rate: 28829)
Baud31250   0x00800000  31250 baud
Baud38400   0x009D5000  38400 baud (actual rate: 38462)
Baud56000   0x00E50000  56000 baud (actual rate: 55944)
Baud57600   0x00EBF000  57600 baud (actual rate: 57602)
Baud76800   0x013A9000  76800 baud (actual rate: 76797)
Baud115200  0x01D7E000  115200 baud (actual rate: 115204)
Baud230400  0x03AFB000  230400 baud (actual rate: 230393)
Baud250000  0x04000000  250000 baud
Baud460800  0x075F7000  460800 baud
Baud921600  0x0EBEDFA4  921600 baud (actual rate: 921585)
Baud1M      0x10000000  1Mega baud

I think the correct formula is:
baudrate_reg_val = (baudrate * 2^32 / 16000000) & 0xFFFFF000
Then check actual baudrate:
baudrate = baudrate_reg_val * 16000000 / 2^32
Add 0x1000 to baudrate_reg_val and check actual baudrate again. Use the closest value.


CocoaPods support?

It's probably simpler and easier to just drop the 2 Swift files into your project.

Yep, plus I'm not a big fan of Cocoapods :) Too much work to support the functionality - my opinion of course. If somebody else wants to do it, totally up to them.


Update VDK deps + Clean up the code

MSE is not working that well...
Need time to investigate

MSE is not working that well... Need time to investigate

Well, I've forgot to replace FormValue: https://github.com/LdDl/video-server/pull/25/commits/02f76eb9f98e87269c12c2ec947342c3faf4f7dc
Defer call has been reduced too (

Also it could fix #20


Houdini service missing GUI libs

From the SideFX Forums:

At the moment I get this error if I try and run houdini with your container. Hython and hscript work fine. I'm running on Ubuntu 22.04. (For my purposes I'll mostly be running headless tasks.)

root@539d5b0c2631:/opt/hfs19.5# houdini
/opt/hfs19.5/bin/houdini-bin: error while loading shared libraries: libsmime3.so: cannot open shared object file: No such file or directory

Steps to recreate issue:

Launch houdini with base hbuild install

Required fix:

Install libs libglu1 libsm6 bc wget libnss3 libxcomposite1 libxrender1 libxrandr2 libfontconfig1 libxcursor1 libxi6 libxtst6 libxkbcommon0 libxss1 libpci3 libasound2 to all existing Docker images with hbuild

Addressed in #8


Bug trying to select a file in CentOS stream 9

There is an issue when trying to select a file during setup. It is not always possible to select the file by clicking the 'Open' button. It depends on the path where the file is located.
For example, the following paths:
/home/centos/licenses --> does not work
/home/centos/Downloads --> works
/home/centos/Downloads/PFSlicenses --> does not work
/home/centos/Downloads/licenses --> does not work
/home/centos/Desktop/licenses --> works
/home/centos/Desktop/licenses/PFSlicenses --> does not work
Permissions and files accessibility have been checked.
It's worth noting that the issue only occurs with the 'Open' button when clicking it or selecting it with the keyboard. If the user double-clicks the file they want to select, it works correctly and they can proceed with the setup.
To Reproduce
Steps to reproduce the behavior:

Open a file selector dialog
Select a file
Press "Open" (depends on the path it works or not)

Expected behavior
Select the file when press "Open" button
Screenshots

Desktop (please complete the following information):

OS: CentOS stream 9
(tested in AlmaLinux and Ubuntu and works well)

Thanks for the report.
Would you be able to test this using the latest v2 package to see if we fixed it there already?

I preferred to use the latest stable version, but I'll try v2 to see if it fixes it.
I'll keep you updated.
Thanks.

Could you at least pull v2 down and run UICatalog on CentOS and tell us what happens?

Hello tig,
I tested UICatalog on CentOS Stream 9, especially the file selection which was failing in the previous version. The result is that directory navigation works well, file selection also works, but when trying to make any customization to the dialogs, it breaks. As an example, I'm attaching an image showing the result of configuring the text of the "Ok" button.

I have also tested it on Alma Linux and similar issues occur.


Retrain with new images

How can I correctly upload images to retrain the model? The new mask images are binary 1 bit images, and they are all the same dimensions as the original training images, but when I try to use my new training/validation in the training demo, I get an index error saying: IndexError: index 2 is out of bounds for dimension 0 with size 2.
I have followed the structure for the image file paths and names, so I don't know why I am getting this error.

It's hard to diagnose this without the data you're using, but dimension 0 should be the channel dimension. Could you please check, how many dimensions your images have? They might be binary, but still have 3 dimensions (i.e. be RGB images).

Do you mean whether they are 1 bit or 8 bit or 24 bit?

Thanks for posting the stack. I think that helped a lot...
Forget what I said about the image format.
--> 139 confusion_matrix[label_gt, label_pred] += 1
141 return confusion_matrix

IndexError: index 2 is out of bounds for dimension 0 with size 2

This is during the evaluation, where the code tries to compute the confusion matrix, for the different classes. The error says that the confusion matrix was initialized with 2 classes. This is the default value. The first class is the background class (0) and the second class is the particle class. However, your ground truth data has more than two classes. The number of classes in the ground truth data, is defined as the number of subfolders (e.g. "particle"). Do you happen to have more than one subfolder in your "train2" and/or "valid2" folder?

While we're at it: Unfortunately, I cannot provide too much support for paddle any longer, since the respective project ended. Fortunately, there has been a lot of progress in the meantime, with regard to the usability of Mask R-CNN for custom applications. Therefore, I'd recommend making a switch to the mmdetection framework, which has a large community and therefore a much more detailed documentation.

This is a good place to get started: https://mmdetection.readthedocs.io/en/latest/user_guides/train.html#train-with-customized-datasets
I understand, if you don't want to make the switch right away, since you might already be close to getting paddle to run properly. So I can still try to help you, to sort this issue out. However, if you encounter further problems, I'd definitely advise making the switch.

Thanks for the help, we were able to get it working.

Great! Then please give a short explanation, what caused the problem, in case that others have the same issue. :+1:


msw: Version bump in mockServiceWorker.js 1.2.2 -> 1.2.3

msw was bumped from 1.2.2 to 1.2.3 in https://github.com/RedHatInsights/image-builder-frontend/pull/1264
This changes the version in mockServiceWorker.js as done by npm ci.

Nice! Just double checking, are you generating the new mockServiceWorker.js file programmatically? For these minor releases it is unlikely for anything to change other than the version but if there is a release with breaking changes it's possible the diff might be bigger.

Yup, the file gets automatically updated on the first clean install after msw was bumped by dependabot.

Oh it's automatic? I didn't realize that, that's great!


added success_codes[0,3010] to windows_package resources

3010 is returned as errorlevel in win2008 R2, which means SUCCESS but reboot required.
For some unknown reason, in Windows 2008 R2, the system returns this message even if /norestart options is given.
You can see https://msdn.microsoft.com/en-us/library/windows/desktop/aa376931(v=vs.85).aspx for more info about 3010 result code.

@patarra This is a fork of https://github.com/chef-cookbooks/ms_dotnet45.  You should make the cookbook at https://github.com/chef-cookbooks/ms_dotnet45 and I'll get it merged in.

Thanks @tas50 , I will do it.

I actually made the change in the upstream and released it already so you're good to go.

Awesome!! Many thanks !!


Downsides to forced parameterized queries?

The "why" for "parameterized queries" is spelled out well enough, but what about the "why not"? When should I not do this? What are the downsides of it? Should I just force parameterized queries on all my new DBs?

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

ID: 100228be-3a7f-ca84-3fe4-f3e537b27030
Version Independent ID: 73602e3b-9ecb-05cf-ec63-e89ca1f78218
Content: Performance recommendations - Azure SQL Database
Content Source: articles/sql-database/sql-database-advisor.md
Service: sql-database
Sub-service: monitor
GitHub Login: @danimir
Microsoft Alias: danil

@MisinformedDNA  This question is out of the scope of this article. There are plenty resources on DB administration on the internet describing this concept. For example, see this Microsoft blog on how and why to use parametrized queries. You would use them to avoid SQL injection attacks. The reason not to use them is if you don't care about SQL injections. Thanks.

@MisinformedDNA Hope the above response was helpful.
We will proceed to close this issue now.

@MisinformedDNA  - There are at least two reasons to use parameterized queries. The one was already mentioned and linked (SQLI prevention), but the other reason is that parameterized queries perform equally to stored procedures in terms of being able to cache the execution plans and reuse them. This is why its a performance recommendation in the Azure portal and not a security item.
The downside is that all code that accesses the DB needs to be parameterized or its not going to be executed. This can be a good reason to not enable it (yet), but it depends on how much code you have that isnt using stored procs or paramterized statement queries and how much work it would be to parameterize it. This is also going to include any ad-hoc queries you need to run from SSMS or AzDS for support/troubleshooting. It's do-able but its really not convenient.
@danimir - having a "con" list when listing "pro's" is not really out of scope. I dont get why so many of the docs that are put out like this have to be biased to the point of propaganda or why there would be any resistance at all to including it. Its just another tool for the toolbox, and one that better have warning stickers on it or an MSDS sheet (the "cons" list).

@StingyJack Thank you for the reply.
Here is how I understand things at this point:
From a client perspective (i.e. code): parameterized queries are good for security and performance and are (generally) the preferred/ideal method and this should be out of scope (though this is where @danimir's reply focused).
From a database perspective: "Forced parameterization" does not force developers to write parameterized queries (nor force anything else on the developer). It, instead, changes how queries are processed by providing hints to the execution plan.
Finally, here is good guidance on why not to use forced parameterization (emphasis mine):

Forced parameterization, in effect, changes the literal constants in a query to parameters when compiling a query. Therefore, the Query Optimizer might choose suboptimal plans for queries. In particular, the Query Optimizer is less likely to match the query to an indexed view or an index on a computed column. It may also choose suboptimal plans for queries posed on partitioned tables and distributed partitioned views. Forced parameterization should not be used for environments that rely heavily on indexed views and indexes on computed columns. Generally, the PARAMETERIZATION FORCED option should only be used by experienced database administrators after determining that doing this does not adversely affect performance.

@MisinformedDNA  I read the docs page linked to this post twice now and I don't get that explanation from it. I do see the one you linked way over in a super long doc I probably would never have found. Thanks!
@danimir that section should be linked to but this tuning recommendation.
I'll be putting in a ticket to have the name and/or description in the portal updated. I know at least a dozen other people that took the same meaning I did from it.


python310Packages.sqlalchemy-continuum: 1.3.14 -> 1.4.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.sqlalchemy-continuum is: Versioning and auditing extension for SQLAlchemy
meta.homepage for python310Packages.sqlalchemy-continuum is: https://github.com/kvesteri/sqlalchemy-continuum/
meta.changelog for python310Packages.sqlalchemy-continuum is: https://github.com/kvesteri/sqlalchemy-continuum/blob/1.4.0/CHANGES.rst
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.4.0 with grep in /nix/store/5iy870d363c3mj89rrfglwphkbh8ny4z-python3.10-sqlalchemy-continuum-1.4.0
found 1.4.0 in filename of file in /nix/store/5iy870d363c3mj89rrfglwphkbh8ny4z-python3.10-sqlalchemy-continuum-1.4.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
python310Packages.sqlalchemy-continuum
python311Packages.sqlalchemy-continuum

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/5iy870d363c3mj89rrfglwphkbh8ny4z-python3.10-sqlalchemy-continuum-1.4.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.sqlalchemy-continuum https://github.com/r-ryantm/nixpkgs/archive/424844677b2c83d91aebcff5264dd75918f0d8be.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/5iy870d363c3mj89rrfglwphkbh8ny4z-python3.10-sqlalchemy-continuum-1.4.0
ls -la /nix/store/5iy870d363c3mj89rrfglwphkbh8ny4z-python3.10-sqlalchemy-continuum-1.4.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages marked as broken and skipped:

python310Packages.sqlalchemy-continuum
python311Packages.sqlalchemy-continuum

Maintainer pings

Replaced by https://github.com/NixOS/nixpkgs/pull/243766.


Create  component

Having a safe way to add emojis to your email would be a great addition to React Email.
From Discord:

"What is the recommended approach for working with emojis? Specifically, I want to insert Github emojis into my HTML (and later export these templates). Should I use SVGs or is there a better way?"

Notes:

Most email clients don't support SVG
Should we render a PNG or wrap a HTML entity? http://www.amp-what.com/unicode/search/emoticon

Useful resources:

https://dreamyguy.github.io/react-emojis
https://github.com/twitter/twemoji

When working with emojis in email templates, it's important to keep in mind that not all email clients support all types of emojis. While some email clients may support SVGs, others may not, so it's generally a good idea to use a fallback method as well.
One common approach is to use PNGs to display the emojis. This can be done by downloading the PNGs from a library like Twemoji, which provides a set of emojis that can be used for free. Once you have the PNGs, you can include them in your email templates using standard HTML  tags.
Another approach is to use HTML entities to display the emojis. HTML entities are special codes that can be used to represent characters that can't be displayed directly in HTML. There are a number of HTML entities that can be used to display emojis, and you can find a list of them on sites like amp-what.com. Once you know the code for the emoji you want to use, you can include it in your HTML using the appropriate entity.
When it comes to React Email, you may want to consider using a library like react-emojis to simplify the process of adding emojis to your templates. This library provides a set of React components that can be used to display emojis using either PNGs or HTML entities, depending on the browser's capabilities.
In summary, the best approach for working with emojis in email templates will depend on your specific needs and the capabilities of your target email clients. Using PNGs or HTML entities are both valid options, and you may want to consider using a library like react-emojis to simplify the process.

Chat GPT detected 😅

Should we render a PNG or wrap a HTML entity? http://www.amp-what.com/unicode/search/emoticon

I'd lean on wrapping an HTML entity. Doing PNGs implies choosing a specific emoji set that may or may not match the user's system. This is exaggerated by the fact that the body might include an emoji used in the headline where it would use the system's style.
On the other side, PNGs add big maintenance costs but allow the library to ship new emojis even before some operating systems. (eg: WhatsApp takes this approach so Android users don't have to deal with "���" emojis when chatting with iPhone users)

last month I created an example using emojis, I just copied and pasted from the original email
https://demo.react.email/preview/codepen-challengers


manage-tools should have a search

manage-tools search crypto for example should list the crypto tools and descriptions.
the stuff is all in the .md file, but it would be nice to be able to search from the script itself.

That is a cool idea! We could even parse the md itself.
On Sat, Oct 24, 2015 at 9:21 PM, Jeffrey Crowell<EMAIL_ADDRESS>wrote:

manage-tools search crypto for example should list the crypto tools and
descriptions.
the stuff is all in the .md file, but it would be nice to be able to
search from the script itself.
—
Reply to this email directly or view it on GitHub
https://github.com/zardus/ctf-tools/issues/29.


Make ThenWithCtx and Map implement IterParser when relevant

I can drop the MaybeMut changes if you want - I think it looks a little nicer, but it's not a functional change


Unpack and concat wrongly transformed into reshape in tflite converter

1. System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 20.04
TensorFlow installation (pip package or built from source): pip
TensorFlow library (version, if pip package or github SHA, if built from source): 2.14.0-dev20230602

2. Code
This is the minimized code to reproduce the issue:
import tensorflow as tf
import numpy as np
x1 = tf.constant([1., 2., 3., 4.], shape=[2, 2, 1])

class Model(tf.keras.Model):

  def __init__(self):
    super(Model, self).__init__()

  @tf.function(input_signature=[tf.TensorSpec(x1.shape, x1.dtype)])
  def call(self, x):
    unpack_op = tf.raw_ops.Unpack(value=x,num=2,axis=0)
    return tf.concat(unpack_op, -1)
m = Model()
m(x1)
print('Keras mode output: ', m(x1).numpy())

converter = tf.lite.TFLiteConverter.from_keras_model(m)
tflite_model = converter.convert()

def _evaluateTFLiteModel(tflite_model, input_data):
    interpreter = tf.lite.Interpreter(model_content=tflite_model)
    interpreter.allocate_tensors()
    input_details = interpreter.get_input_details()
    output_details = interpreter.get_output_details()

    for i in range(len(input_data)):
        interpreter.set_tensor(input_details[i]['index'], input_data[i])
    interpreter.invoke()
    output_data = [interpreter.get_tensor(output_details[i]['index'])
                   for i in range(len(output_details))]
    return output_data
print('Lite mode output: ', _evaluateTFLiteModel(tflite_model,[x1])[0])
tf.lite.experimental.Analyzer.analyze(model_content=tflite_model) #Output IR

3. Failure after conversion
Output:
Keras mode output:  [[1. 3.]
 [2. 4.]]
Lite mode output:  [[1. 2.]
 [3. 4.]]

Lite IR:
Subgraph#0 main(T#0) -> [T#2]
  Op#0 RESHAPE(T#0, T#1[2, 2]) -> [T#2]

Tensors of Subgraph#0
  T#0(serving_default_args_0:0) shape:[2, 2, 1], type:FLOAT32
  T#1(concat) shape:[2], type:INT32 RO 8 bytes, buffer: 2, data:[2, 2]
  T#2(PartitionedCall:0) shape:[2, 2], type:FLOAT32

Model produces wrong results:

Using tf.lite.experimental.Analyzer.analyze, we can see the entire calculation is wrongly transformed into a single RESHAPE operator.

I am able to replicate on tf-nightly consistently, here is the gist
unsure if the reshape is incorrect, but output should be the same after conversion.

Hi @YaoJiayi, similarly this is also handled better with AI-Edge-Torch.
PT/TFLite Program:
import ai_edge_torch
import torch

class Model(torch.nn.Module):
    def __init__(self):
        super(Model, self).__init__()

    def forward(self, x):
        unpack_op = torch.chunk(x, 2, dim=0)
        return torch.cat(unpack_op, -1)

sample_input = (torch.tensor([1., 2., 3., 4.], dtype=torch.float32).reshape(2, 2, 1),)
m = Model()
print('PyTorch model output: ', m(*sample_input).numpy())

edge_model = ai_edge_torch.convert(m.eval(), sample_input)

actual_value = edge_model(*sample_input)
print('TFLite model output: ', actual_value)

output:
python convert.py
2024-06-13 18:54:18.859326: I external/local_xla/xla/tsl/cuda/cudart_stub.cc:32] Could not find cuda drivers on your machine, GPU will not be used.
2024-06-13 18:54:18.862719: I external/local_xla/xla/tsl/cuda/cudart_stub.cc:32] Could not find cuda drivers on your machine, GPU will not be used.
2024-06-13 18:54:18.902421: I tensorflow/core/platform/cpu_feature_guard.cc:210] This TensorFlow binary is optimized to use available CPU instructions in performance-critical operations.
To enable the following instructions: AVX2 FMA, in other operations, rebuild TensorFlow with the appropriate compiler flags.
2024-06-13 18:54:19.943877: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Could not find TensorRT
WARNING:root:PJRT is now the default runtime. For more information, see https://github.com/pytorch/xla/blob/master/docs/pjrt.md
WARNING:root:Defaulting to PJRT_DEVICE=CPU
WARNING: All log messages before absl::InitializeLog() is called are written to STDERR
I0000 00:00:1718304862.252478 1629230 cpu_client.cc:424] TfrtCpuClient created.
PyTorch model output:  [[[1. 3.]
  [2. 4.]]]
WARNING: All log messages before absl::InitializeLog() is called are written to STDERR
W0000 00:00:1718304863.973475 1629230 tf_tfl_flatbuffer_helpers.cc:392] Ignored output_format.
W0000 00:00:1718304863.973507 1629230 tf_tfl_flatbuffer_helpers.cc:395] Ignored drop_control_dependency.
2024-06-13 18:54:23.974569: I tensorflow/cc/saved_model/reader.cc:83] Reading SavedModel from: /tmp/tmpemo0od2g
2024-06-13 18:54:23.974761: I tensorflow/cc/saved_model/reader.cc:52] Reading meta graph with tags { serve }
2024-06-13 18:54:23.974782: I tensorflow/cc/saved_model/reader.cc:147] Reading SavedModel debug info (if present) from: /tmp/tmpemo0od2g
2024-06-13 18:54:23.980117: I tensorflow/compiler/mlir/mlir_graph_optimization_pass.cc:388] MLIR V1 optimization pass is not enabled
2024-06-13 18:54:23.980301: I tensorflow/cc/saved_model/loader.cc:236] Restoring SavedModel bundle.
2024-06-13 18:54:23.987552: I tensorflow/cc/saved_model/loader.cc:220] Running initialization op on SavedModel bundle at path: /tmp/tmpemo0od2g
2024-06-13 18:54:23.990757: I tensorflow/cc/saved_model/loader.cc:462] SavedModel load for tags { serve }; Status: success: OK. Took 16191 microseconds.
2024-06-13 18:54:23.996306: I tensorflow/compiler/mlir/tensorflow/utils/dump_mlir_util.cc:268] disabling MLIR crash reproducer, set env var `MLIR_CRASH_REPRODUCER_DIRECTORY` to enable.
2024-06-13 18:54:24.015047: I tensorflow/compiler/mlir/lite/flatbuffer_export.cc:3531] Estimated count of arithmetic ops: 0  ops, equivalently 0  MACs
INFO: Created TensorFlow Lite XNNPACK delegate for CPU.
TFLite model output:  [[[1. 3.]
  [2. 4.]]]
I0000 00:00:1718304864.969025 1629230 cpu_client.cc:427] TfrtCpuClient destroyed.

Let us know if this resolves your issue.


mcontrolcenter: init at 0.4.1

MControlCenter provides a relatively full-featured control panel for MSI laptops, written in Qt. It allows users to control MSI laptop power plans, fan speed, battery charging limits, keyboard backlight settings, USB power sharing and more.
The application comes with strong desktop integration: shortcuts, taskbar item, etc. etc. -- screenshots here
The package comes with a patch that corrects MControlCenter's path to the modprobe binary, allowing it to automatically load the ec_sys kernel module (included with NixOS by default) with the right settings when launched.
This package has been previous requested by the NixOS community (https://github.com/NixOS/nixpkgs/issues/244881). Another pull request (https://github.com/NixOS/nixpkgs/pull/251816) was submitted for this same package. The pull request has been reviewed but I contacted the author personally and he is not interested in completing it any more. For this reason I'm opening a new pull request incorporating all the changes suggested in the review.
This NixOS package has been tested on a NixOS Plasma5 and GNOME 45 desktops.
fixes:  #244881

Description of changes
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3847

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review-december/1711/21

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

1 package built:

mcontrolcenter

I guess you could move this to pkgs/by-name. I think using libsForQt5.callPackage is not needed, and you can maybe directly extract packages from libsForQt5. Could you try ?

I applied all the suggested changes and tested the package on x86_64-linux, everything should work as intended. Here is a list of what I did:

move to pkgs/by-name
avoid using libsForQt5.callPackage by extracting packages from libsForQt5
getting hash from finalAttrs.version
remove unused cmake flags
patching with substituteInPlace
add kmod to build inputs

@nim65s con you confirm the pool request looks fine to you now?

Discussions on matrix raised that maybe this .service file (and maybe the desktop file btw) should be proposed upstream. Maybe you can contribute a PR there with those files ?
Also, using pkgs.writeText instead of those echo would be a bit cleaner.

Built and tested, launches fine on linux-x86-64 and wayland once the ec-sys kernel module is actually loaded.
However, most options are pretty much locked to something with no way to edit them, with the exception of fan curves. Whether that is due to my extensive customization on tlp, acpi and similar i have not yet fully tested.

Built and tested, launches fine on linux-x86-64 (msi GF75 thin) once the ec-sys kernel module is actually loaded. However, most options are pretty much locked to something with no way to edit them, with the exception of fan curves. Whether that is due to my extensive customization on tlp, acpi and similar i have not yet fully tested.

It seams to be an upstream issue, someone else reported the same problems for the same model which apparently is not fully supported.
For me the battery options are working correctly, it might be a problem related to tlp in your case (as you suggested).

Ah upstream lists my precise model (GF75 thin 10SDR) as fully unsupported currently, soo i guess thats my hardware just not being supported then

Discussions on matrix raised that maybe this .service file (and maybe the desktop file btw) should be proposed upstream. Maybe you can contribute a PR there with those files ?
Also, using pkgs.writeText instead of those echo would be a bit cleaner.

In an effort to learn more about nix, i tried doing just that; turns out getting the $out path for mcontrolcenter into writeText is either nearly or completely impossible, at which point you'd have to use substituteInPlace during installPhase anyways. Hence there is no reason why you shouldn't just take the upstream dbus service config and fix that using substituteInPlace instead.

I added the suggested changes:

squashed everything into two commits
using upstream service file
using substituteInPlace to patch the Exec path
using --replace-fail in every other occurrence of substituteInPlace

@Aleksanaa, can I ask you if you approve the current state of this pr? After I applied the last changes.


App download count

Are any numbers available for installed App count yet for both IOS and Android?

330 total installs of Android. Last count, iOS was much higher. We're way over 1,000 for sure. @austin9350 has the exact number

Current count is 741

:tada: 1k+ downloads in < 1mo is pretty impressive! Especially in the 'offseason'. Exciting stuff.


Inline scripts

Description
Inject scripts into script tags
Steps to reproduce
all the scripts generated after build should be inserted into script tag
Paste the information here as shown by gridsome info

Hi @vishnucr and what is your problem?

@hjvedvik Not enough information has been provided. Issue can be closed.


Add links for 0.15.0 and 0.16.0

Was reviewing the changelogs that are aggregated here: https://docs.us.lifeomic.com/release-notes/phc-sdk-py/
Will add version links for the diffs.

No worries 👍


PR for #21

Adds pause functionality and update first example to demo it

thanks for pointing that out.
Adjusted.

We better name it “play” * as it will continue paused tween
On Wed, Aug 8, 2018 at 5:03 PM Hirako<EMAIL_ADDRESS>wrote:

@hirako2000 commented on this pull request.
In types/between-tests.ts
https://github.com/sasha240100/between.js/pull/31#discussion_r208592788:

@@ -22,7 +22,7 @@ between.on('complete', (value) => { value });
// Test pause control
between.pause();
between.start();

I called it start because the emitted event that was already present is
called start. It's for consistency.
But up to you, I can rename if you think it's better.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sasha240100/between.js/pull/31#discussion_r208592788,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHTX1cnErHxn5ZG32l44A1-XjLGvmmrWks5uOu-egaJpZM4Vx0Rk
.

Renamed


Audit backend interface for cross platform compatibility

I haven't had the time to prototype enough loadbalancer backends (eg: envoy, h2o, cloudprovider) with the generic_controller's interface. Currently it's pretty minimalistic, and captured in this commit: https://github.com/kubernetes/ingress/pull/15/commits/f908fb94bc073b88798f68b9600748957dc188ae
Tl;dr is an interface:
type Controller interface {
	Test(file string) *exec.Cmd
	OnUpdate(*api.ConfigMap, Configuration) ([]byte, error)
	BackendDefaults() defaults.Backend
	Info() *BackendInfo
}

and the Configuration type
type Configuration struct {
	HealthzURL         string
	Backends           []*Backend
	Servers            []*Server
	TCPEndpoints       []*Location
	UPDEndpoints       []*Location
	PassthroughBackend []*SSLPassthroughBackend
}

We need to attack this from 2 angles:

Are the methods in this interface enough to give the backends the flexibility they need?
Is the information passed through OnUpdate adequate

Right now, every onUpdate effectively gets:
Configuration
  Server
    virtual server -> location -> Backends ([]endpoitns + ssl) -> Endpoints (address:port, health check info)
                ...more locations
    ...more virtual servers
  ...more Servers

@bprashanth  in the location you also get information about the annotations (for instance ExternalAuth)

Closing. Interface already improved and use by third party controllers


Add documentation link for NuGet Tool Installer, other minor string fixes

@emanuelquintero can you have a look at this and make sure it looks solid before I merge?


UpdateSubscriptionAsync method invoked despite no changes in subscription configuration

Contact Details
No response
Version
8.x
On which operating system(s) are you experiencing the issue?
Windows
Using which broker(s) did you encounter the issue?
Azure Service Bus
What are the steps required to reproduce the issue?
1. Create new .net application
2. Create a message class
3. Create a consumer class implementing IConsumer<T>
4. Configure MT using UsingAzureServiceBus and specifying connection string using Host method
5. Run the application once and close it.
6. Run the application again.

What is the expected behavior?

_administrationClient.UpdateSubscriptionAsync shall not be called if there is no change in the configuration
_administrationClient.UpdateSubscriptionAsync shall be called with targetForwardTo value

What actually happened?
Hi,
To give you some background - recently, while scaling pods on AKS, we have begun to encounter a significant number of errors:
(I used X characters to hide some data.)

Azure.Messaging.ServiceBus.ServiceBusException: The service was unable to process the request; please retry the operation. For more information on exception types and proper exception handling, please refer to http://go.microsoft.com/fwlink/?LinkId=761101. TrackingId:XXXXXXXXXXXXXXXXXXXXXXXXXXX, SystemTracker:XXXXXXXXXXXXXXXXXXXXXXXXXXX:Topic:XXXXXXXXXXXXXXXXXXXXXXXXXXX, Timestamp:2024-03-22T04:17:11
Status: 500 (Internal Server Error)
Headers:
Transfer-Encoding: chunked
ETag:<PHONE_NUMBER>06970000
Server: Microsoft-HTTPAPI/2.0
Strict-Transport-Security: REDACTED
Date: Fri, 22 Mar 2024 04:17:11 GMT
Content-Type: application/xml; charset=utf-8
--- End of inner exception stack trace ---
at Azure.Messaging.ServiceBus.Administration.HttpRequestAndResponse.ThrowIfRequestFailedAsync(Request request, Response response)
at Azure.Messaging.ServiceBus.Administration.HttpRequestAndResponse.SendHttpRequestAsync(Request request, CancellationToken cancellationToken)
at Azure.Messaging.ServiceBus.Administration.HttpRequestAndResponse.PutEntityAsync(String entityPath, String requestBody, Boolean isUpdate, String forwardTo, String fwdDeadLetterTo, CancellationToken cancellationToken)
at Azure.Messaging.ServiceBus.Administration.ServiceBusAdministrationClient.UpdateSubscriptionAsync(SubscriptionProperties subscription, CancellationToken cancellationToken)

We have created an issue with Microsoft support, but at the same time, we decided to decompile and debug the code on our side. The first thing we wonder about is why the UpdateSubscriptionAsync method is called even though no changes have been made to the MassTransit configuration.
We have discovered two interesting points and have questions regarding them.

The code shown above originates from the ServiceBusConnectionContext class (this is the only place where the invocation of the ServiceBusAdministrationClient.UpdateSubscriptionAsync method can be found).

The NormalizeForwardTo method will never normalize the ForwardTo because Endpoint.ToString() begins with 'sb://', but the subscription's ForwardTo from subscriptionProperties starts with 'http://' although, when verifying this in the Azure portal, it only displays the name of the queue - not the fully qualified name. Due to this, the following if statement always executes and calls the UpdateSubscriptionAsync.

Within the if statement, the UpdateSubscriptionAsync method is called using the same subscriptionProperties that were retrieved from the ServiceBusAdministrationClient, not the targetForwardTo. The targetForwardTo value is only utilized to check the if condition. It is not used elsewhere to update the subscription.

I've reported this as a bug because subscriptions being updated every time the application starts with the same configuration appears unexpected. However, if we've misunderstood the code, I apologize for raising this issue. (This is probably not the root cause of our 500 errors returned from the Service Bus, though.)
Related log output, including any exceptions
No response
Link to repository that demonstrates/reproduces the issue
No response

Are you planning to submit a PR? If not, issues require a complete standalone repository to reproduce the issue. If no repository can be provided, you'll need to go through support.

I am sorry,
From what I understand I shall create discussion first.
I am closing the issue.


[2.5 backport] Can't upgrade an app launched in Ember 2.5.x

https://github.com/rancher/dashboard/issues/4708

Verified on 2.5.13-rc2
Upgrades completed via Ember UI without issue on 2.5.13-rc2


Master/Child example returns init() when back to background

Hi @neokree,
First of all, congrats for your library, it is very useful.
I have a problem in my app and this also appears in your Master/Child example, the steps are,
1- Select Section 1 or Section 2
2- Click Button Open Child
3- Go to Background (change another app)
4- Back from background and the app restart the fragment's title to "Section 1" but the content remains second fragment.
Any idea?

Any idea @neokree?

Sorry, I have some trouble with my computer and in this week I didn't have the time to do this.
It will be fixed

Thaks for your answer @neokree  , this bug is fixed in 1.3.3 release?
Thanks for your effort.

no, I'm working on it

Hi @neokree, any advance?
Thanks.


Makes the Entries respect INotifiyCollectionChanged fixes #30

This will let you use an ObservableCollection and get automatic updates!

This fixes #30 , doing it in the title didn't link them.   Also I checked the build log for the failed check. The CI isn't restoring packages so it didn't really check the build.

Also along these lines, I have a Fork where Xamarin.Forms is a lot cleaner with thebindings.
            <StackLayout VerticalOptions="FillAndExpand">                 <microcharts:ChartView MinimumHeightRequest="200" HeightRequest="200"  >                     <microcharts:ChartView.Chart>                         <microcharts:LineChart  Entries="{Binding Entries}"/>                     </microcharts:ChartView.Chart>                 </microcharts:ChartView>             </StackLayout>
If this is something that interests you let me know. I can work with you to get it into a proper pull request. If not I may just hardfork and make this work for what I need.

I think it would be a very interesting contribution

👍 to getting this in.

This library is amazing, but I wanted it to be a bit more forms friendly, as well as properly chart across two axis. Since there has been no response I did a hard fork for anyone interested: https://github.com/clancey/simplecharts


Revert "[Doc Feature][Edit Teams app manifest for VS][2510690] (#6376)"

This reverts commit 028944a4002a7392d601417c838917752f4798b1.

Docs Build status updates of commit 253b7a2:
:warning: Validation status: warnings

File
Status
Preview URL
Details

[msteams-platform/toolkit/Deploy Teams app to the cloud using Visual Studio.md](https://github.com/MicrosoftDocs/msteams-docs/blob/revert-6376-preview-custamize-AAD-VS/msteams-platform/toolkit/Deploy Teams app to the cloud using Visual Studio.md)
:warning:Warning
View
Details

[msteams-platform/toolkit/Provision cloud resources using Visual Studio.md](https://github.com/MicrosoftDocs/msteams-docs/blob/revert-6376-preview-custamize-AAD-VS/msteams-platform/toolkit/Provision cloud resources using Visual Studio.md)
:warning:Warning
View
Details

msteams-platform/toolkit/VS-TeamsFx-preview-and-customize-app-manifest.md
:bulb:Suggestion

Details

msteams-platform/assets/images/Tools-and-SDK-revamp/edit-manifest-for-visual-studio/vs-hover.png
:white_check_mark:Succeeded

msteams-platform/assets/images/Tools-and-SDK-revamp/edit-manifest-for-visual-studio/vs-hover-key-placeholder.png
:white_check_mark:Succeeded

msteams-platform/assets/images/Tools-and-SDK-revamp/edit-manifest-for-visual-studio/vs-hover-placeholder.png
:white_check_mark:Succeeded

msteams-platform/assets/images/Tools-and-SDK-revamp/edit-manifest-for-visual-studio/vs-hover-placeholder1.png
:white_check_mark:Succeeded

msteams-platform/assets/images/Tools-and-SDK-revamp/edit-manifest-for-visual-studio/vs-hover-undefined1.png
:white_check_mark:Succeeded

msteams-platform/assets/images/Tools-and-SDK-revamp/edit-manifest-for-visual-studio/vs-open-manifest.png
:white_check_mark:Succeeded

msteams-platform/assets/images/Tools-and-SDK-revamp/edit-manifest-for-visual-studio/vs-overwrite.png
:white_check_mark:Succeeded

msteams-platform/assets/images/Tools-and-SDK-revamp/edit-manifest-for-visual-studio/vs-prepare-dependencies.png
:white_check_mark:Succeeded

msteams-platform/assets/images/Tools-and-SDK-revamp/edit-manifest-for-visual-studio/vs-preview.png
:white_check_mark:Succeeded

msteams-platform/assets/images/Tools-and-SDK-revamp/edit-manifest-for-visual-studio/vs-preview1.png
:white_check_mark:Succeeded

msteams-platform/assets/images/Tools-and-SDK-revamp/edit-manifest-for-visual-studio/vs-preview-manifest1.png
:white_check_mark:Succeeded

msteams-platform/assets/images/Tools-and-SDK-revamp/edit-manifest-for-visual-studio/vs-provision.png
:white_check_mark:Succeeded

msteams-platform/assets/images/Tools-and-SDK-revamp/edit-manifest-for-visual-studio/vs-solution-explorer.png
:white_check_mark:Succeeded

msteams-platform/assets/images/Tools-and-SDK-revamp/edit-manifest-for-visual-studio/vs-solution-explorer1.png
:white_check_mark:Succeeded

msteams-platform/assets/images/Tools-and-SDK-revamp/edit-manifest-for-visual-studio/vs-update-manifest.png
:white_check_mark:Succeeded

msteams-platform/assets/images/Tools-and-SDK-revamp/edit-manifest-for-visual-studio/vs-update-manifest1.png
:white_check_mark:Succeeded

msteams-platform/assets/images/Tools-and-SDK-revamp/edit-manifest-for-visual-studio/vs-zip.png
:white_check_mark:Succeeded

msteams-platform/assets/images/Tools-and-SDK-revamp/edit-manifest-for-visual-studio/vs-zip1.png
:white_check_mark:Succeeded

msteams-platform/assets/images/Tools-and-SDK-revamp/edit-manifest-for-visual-studio/vs-zip-app-package.png
:white_check_mark:Succeeded

msteams-platform/assets/images/Tools-and-SDK-revamp/images for provision cloud resources in TTK VS/activate-teams-toolkit.png
:white_check_mark:Succeeded

msteams-platform/assets/images/Tools-and-SDK-revamp/images for provision cloud resources in TTK VS/add-capabilities.png
:white_check_mark:Succeeded

msteams-platform/assets/images/Tools-and-SDK-revamp/images for provision cloud resources in TTK VS/add-cloud-resources.png
:white_check_mark:Succeeded

This comment lists only the first 25 files in the pull request.

[msteams-platform/toolkit/Deploy Teams app to the cloud using Visual Studio.md](https://github.com/MicrosoftDocs/msteams-docs/blob/revert-6376-preview-custamize-AAD-VS/msteams-platform/toolkit/Deploy Teams app to the cloud using Visual Studio.md)

Line 83, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'create-new-teams-app-for-Visual-Studio.md'.
Line 84, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'provision-cloud-resources-using-Visual-Studio.md'.
Line 85, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'VS-TeamsFx-preview-and-customize-app-manifest.md'.
Line 86, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'debug-teams-app-visual-studio.md'.

[msteams-platform/toolkit/Provision cloud resources using Visual Studio.md](https://github.com/MicrosoftDocs/msteams-docs/blob/revert-6376-preview-custamize-AAD-VS/msteams-platform/toolkit/Provision cloud resources using Visual Studio.md)

Line 282, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'deploy-Teams-app-to-the-cloud-using-Visual-Studio.md'.
Line 283, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'VS-TeamsFx-preview-and-customize-app-manifest.md'.
Line 284, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'debug-teams-app-visual-studio.md'.

msteams-platform/toolkit/VS-TeamsFx-preview-and-customize-app-manifest.md

[Suggestion: file-not-redirected - See documentation] File msteams-platform/toolkit/VS-TeamsFx-preview-and-customize-app-manifest.md was deleted without redirection. To avoid broken links, add a redirection.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


added-graph strongly connected components kosaraju-algorithm

I have added the graph strongly connected components kosaraju algorithm cpp code.

I will request (if you can) you to please add other languages also to implement this algorithm.


Support mapping to multiple IRIs

In some cases, a property appearing in the data file should be linked to more than one IRI, however, at the moment, the mapping file doesn't allow it.

This has been solved by #70


perf(txpool): filter by sender id

https://github.com/paradigmxyz/reth/pull/1976 followup that only collects transactions sent by the address get_transactions_by_sender instead of: clone all + filter + collect

Codecov Report

Merging #2080 (8599f82) into main (481f60a) will decrease coverage by 56.65%.
The diff coverage is 0.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff             @@
##             main    #2080       +/-   ##
===========================================
- Coverage   72.07%   15.42%   -56.65%
===========================================
  Files         435      435
  Lines       53251    53249        -2
===========================================
- Hits        38380     8214    -30166
- Misses      14871    45035    +30164

Flag
Coverage Δ

integration-tests
15.42% <0.00%> (-3.90%)
:arrow_down:

unit-tests
?

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

Impacted Files
Coverage Δ

crates/transaction-pool/src/identifier.rs
1.49% <ø> (-89.56%)
:arrow_down:

crates/transaction-pool/src/lib.rs
9.23% <0.00%> (-20.40%)
:arrow_down:

crates/transaction-pool/src/pool/mod.rs
7.02% <0.00%> (-36.65%)
:arrow_down:

crates/transaction-pool/src/pool/txpool.rs
3.68% <0.00%> (-59.31%)
:arrow_down:

... and 319 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Add block quotes for where we are referencing upstream documentation

Each of the components is pulling content  from the upstream docs, these should be blockquoted.

@agracey Can you please provide some information on proceeding with this issue?

For example, the text Rancher is a powerful open-source Kubernetes management platform [...] should be a block quote since we are pulling it from the Rancher documentation itself.


bug: [Android] onValueChanged does not trigger or send wrong values sometimes

Hello,
I recently tried using your library, and it works great on iOS. However, I've encountered a couple of issues on Android. Specifically, the onValueChanged callback sometimes doesn’t trigger, or it returns an incorrect value. I’ve recorded a video demonstrating these issues and also created a simple Snack example for easy testing.
Issues demonstrated in the video:

Around the 3-second mark, I select the value 4, but the onValueChanged callback receives the value 3 instead.
Around the 7-second mark, I change the value from 1 to 2, but the callback is not triggered at all (you can confirm this by checking the logs—onValueChanged is not called).
The above issues continue to occur intermittently with different values, indicating that the problem is not limited to a specific value.

I'm using a OnePlus 7T.
I’d appreciate any guidance or fixes you could provide for this behavior.
Thank you in advance!
Snack : https://snack.expo.dev/@eeynard_allocab/wheelpicker-android-issue
https://github.com/user-attachments/assets/e318d089-dea4-4ccc-9000-0725aa94580d

@eeynard Thank you for your issue!
Please answer a few questions:

Is the new or old architecture being used?
What is the version of React Native?
Build Debug or Release?

I have not been able to reproduce the problem, but I have suggestions that this may be due to the braking of the JS stream.
I see that you have a new reference to data and onValueChanged for each render, try to memorize them and see if it will work better or not.
At the moment, I understand that the conditions for calling onValueChanged are not implemented in the best way and it needs to be improved.

In the near future, I will try to improve the onValueChanged call and remove the 300ms delay in calling the function.
To determine when scrolling stops, use the onScrollBeginDrag, onScrollEndDrag, and onMomentumScrollEnd events.

I accidentally forgot to save my code, but I did test it using a static dataset (you can see it in the updated snack). The issue still persists.

Is the new or old architecture being used?

old

What is the version of React Native?

Expo Go 52 => RN 0.76. And on my app Expo 51 => RN 0.74.5

Build Debug or Release?

I believe Expo Go runs in release mode, while my app was tested in debug mode.
Thanks for the feedback! I’ll look into finding a workaround in the meantime.

So, just to add more information, I'm stumbling in the same thing. My problem is the sometimes it doesn't change at all or changes by itself.
This is what I'm getting from scrolling:

One down: most of the time it won't register the change
Two down: works fine all the time
Three down: Goes one up (turns to two down)
Four down: Sometimes turns to three down (goes one up)
Scrooling up: Always works

But, btw, great wheel, it was by far the best I've tested.
https://github.com/user-attachments/assets/2e28cd1b-48c7-4cdb-b196-031ba8f9d761

@thacio Thank you for your additional information!
@thacio @eeynard Can you check in version 1.3.1-beta.0 how onValueChanged works now?

Noticed a small change on one down, the rest were similar. I tested these ones :

One down: skips the odd indexes
Two down: works fine all the time
Three down: Goes one up (turns to two down)
Scrooling up: Always works
https://github.com/user-attachments/assets/5ff45587-0535-4f07-850b-f03e21980cff

@thacio Thank you very much for helping in this difficult task!)
I have released version 1.3.1-beta.1, please see if the problem has been fixed? In this version, I fixed the index definition and removed unnecessary calls when synchronizing scrolling. This should improve onValueChanged work.

@thacio @eeynard Could you please provide feedback on version 1.3.1-beta.1?

All right, congrats! Working flawlessly! Thanks, I tried reproducing in a simple example but it didn't work. Everything worked now in the same setup.

@thacio Thank you so much!)

Released 1.3.2


No pagination in community resources

All the community resources are displayed in one really long list.  There is no pagination or "infinite scroll, load more only as needed" approach.
001.zip

@ndebuhr This is a duplicate and also doesn't provide the design information needed. Please identify the duplicate and consolidate any additional information you want into it.

Automated close - duplicate


Getting { Error: 'response code 403' } just after one request from server.

This is too vague. Reopen after you have tested against multiple urls/ sites and can reliably reproduce and demonstrate.


Allow result["output"] to support JSON objects

Currently result["output"] has to be of type string.
If it's not a string this will appear
2015/10/23 01:02:40 json: cannot unmarshal object into Go value of type string

Need to make sure output doesn't contain unicode.  Refer to Curl module for proper example


SuccinctRoles.is_delegated_role() add a test case

Description of the changes being introduced by the pull request:
Add a test case when there is a bin name with the desired prefix, but
which cannot be cast to a hexadecimal number.
Signed-off-by: Martin Vrachev<EMAIL_ADDRESS>Please verify and check that the pull request fulfills the following
requirements:

[ ] The code follows the Code Style Guidelines
[ ] Tests have been added for the bug fix or new feature
[ ] Docs have been added for the bug fix or new feature

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 98.129%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
1303

Relevant Lines:
1319

💛  - Coveralls

Looks good.
Looking at these two cases:
    "bin-s",
    "bin-0t",

is the difference that first one fails because of length already and second one fails at hex parsing stage?

Yes, exactly that is the difference.


No df_spx.csv in repo files

There is no df_spx.csv and no reference to public download resources where file can be found.

Thanks for letting me know! I added the csv file to the repository and referenced it in the documentation. It's from yahoo finance. Commit: https://github.com/m-dadej/MarSwitching.jl/commit/7e0af868baf30c36dcce3dd18c33f150f877e4f7