Implement PlantedTank Sensor and MarineTank Sensor that extends Basic Tank Model in the Sensor Microservice

Issue: Implement PlantedTankService and MarineTankService
Description
We need to create two models, **PlantedTank Sensor ** and **MarineTank Sennsor **, that will manage the functionalities specific to planted and marine tanks, respectively. Both services should implement the BasicTank Model interface to ensure consistency and shared functionalities across different tank types.
Parameters

Basic Parameters:

Temperature
pH Levels
Light
Nitrate Sensor

Planted Tank Parameters:

CO2 Sensor

Marine Tank Parameters:

Salinity Sensor
Calcium Sensor
Alkalinity Sensor
Nitrite Sensor

Implement the PlantedTank class that:

Implements the BasicTank  Model interface.
Provides specific functionality for managing planted tanks, including:

Managing the CO2 sensor.

Implement the MarineTank class that:

Implements the BasicTank Model interface.
Provides specific functionality for managing marine tanks, including:

Managing the salinity, calcium, alkalinity, and nitrite sensors.

Test Cases

Create unit tests for PlantedTank that validate:

The ability to correctly add and retrieve CO2 sensor readings.
Proper monitoring of basic parameters (Temperature, pH Levels, Light, Nitrate).

Create unit tests for MarineTank that validate:

The ability to correctly add and retrieve salinity, calcium, alkalinity, and nitrite sensor readings.
Proper monitoring of basic parameters (Temperature, pH Levels, Light, Nitrate).

Acceptance Criteria

Both services should compile without errors.
Each service should have its own unique functionalities while adhering to the methods defined in the BasicTankService interface.
Unit tests should be created to validate the behavior of each service and should pass successfully.

Additional Notes

Ensure proper documentation for each method and class.
Follow the project's coding standards and conventions.

Hello @RomeoIndiaJulietUniform, I found this issue quite interesting will you please assign it to me?

Hii @LadislavTorma , thanks a lot for showing interest in the project. I'm happy to inform you that I have assigned this issue to you. You can take reference from the Actuator Service , the logic is quite similar.
Happy Coding 🙂!

Hi @RomeoIndiaJulietUniform ,
Today, I finally found some time to work on the issue assigned to me: Implement PlantedTankService and MarineTankService. However, as I reviewed the code, it looks like the issue has already been resolved as of September 22nd. Could you please confirm if I understood this correctly? Or are there specific aspects of the issue that still need to be addressed that I may have missed?
I just want to make sure I’m not duplicating work or misunderstanding the requirements.
Looking forward to your clarification!
Thanks,
@LadislavTorma

@LadislavTorma thanks for the question,  I think there is a issue with the clarity , I have reformated the issue section for better understanding.
So, the project needs seperate sensor service for marine tank and planted tank as biological paramteres are different, you can refer the Actuator Microservice it already implements seperate Actuator while having a parent Basic Aquarium model (https://github.com/RomeoIndiaJulietUniform/AquaManage_Backend/tree/main/ActuatorService/src/main/java/com/DragonFish/aquaManage/ActuatorService/Models)
For more understanding you can refer this Excali Draw link (https://t.co/NTjHOfdP3j) . Pls feel free to ask if you have any more doubt , I am happy to help !!!


Won't connect.  Then crashes.

On Windows 7 Ultimate x64.  Tried yakyak 1.3.0 and 1.3.1
When launching app for the first time, it successfully takes my credentials and MFA, but then proceeds to a blank screen where it attempts to connect unsuccessfully, over and over:

If I close the app and then relaunch the exe, I get a javascript exception:

I'm using a google apps for work account.
If I delete the yakyak folder in %APPDATA%, I can start over but it always does the same thing.
Is there a log somewhere I can possibly see a little more info about the exception?

Now thinking this is a proxy issue after reading issue #255.
We point to a proxy PAC file (proxy auto-config script) - so I'm not sure how to apply that to the two environment variables referenced in issue #255.  I know I can get the actual proxy addresses out of the PAC file, but they do change periodically.  Would be nice if the app could just use the system proxy configuration.

@KennethScott
As a workaround to set proxy on launch from Windows (and still launch with double-click from an icon), create a shortcut and specify as the target:
C:\Windows\System32\cmd.exe /c "SET HTTP_PROXY=http://<proxy-host>:<port> && SET HTTPS_PROXY=http://<proxy-host>:<port> && START /D ^"\YakYak^" yakyak.exe"

If your proxy requires a username/password, I'm not sure how to set it like this.
With this, I am able to run YakYak under Windows behind a proxy. Alternatively, something like Proxifier could likely be used to force the connection through the proxy.

Same problem for me, but on a Mac.  Is there a workaround for the Mac?

@datanotion
Untested, but a likely fix: this answer on StackOverflow or one of the other ones on the page cover setting environment variables for apps launched through Spotlight.

@joshstrom - bummer, didn't seem to have any effect.  Thank you for the idea though!

I am also having this issue. I was able to install and use it just fine until about a week ago.

@catskull On december 1st google changed the protocol. You need to update to 1.4.0 or 1.4.1 to get it back working

@averissimo Thanks! I install with brew cask, should that have the latest version?

@catskull you can use brew cask commands to see which version it is going to install (seems 1.4.0).
I don't remember who is maintaining the cask package :)

Okay, looks like cask installs 1.2.5 for me. I'll just download the release!

thank you


Fix for issue #1191

Modified cosmology.py so that it complains when mu-sigma scale-dependent parameters are non-zero but transfer function isn't the isitgr one

@sankarshana16 are you able to address these?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

7 of 9 (77.78%)  changed or added relevant lines in 1 file are covered.
4 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.02%) to 97.389%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pyccl/cosmology.py
7
9
77.78%

Files with Coverage Reduction
New Missed Lines
%

pyccl/cells.py
1
97.96%

pyccl/_nonlimber_FKEM.py
3
96.15%

Totals

Change from base Build<PHONE_NUMBER>8:
-0.02%

Covered Lines:
6527

Relevant Lines:
6702

💛  - Coveralls


Datepicker and daterange picker : add "close" button

Add a "X" button near the input that clear the field and emits a "null" / empty value

@SebConejo logic is done, we need to style it now


feat: #1501 - refactored user preferences files into a dedicated folders

What

In #1797 @M123-dev noted that we should put user preferences files in a dedicated folder.
It was not appropriate to do so while developing - too many files, possible confusion during review between actual changes and just moved files
Now it's appropriate: the only change is the creation of the folder and the moving of files

Part of

https://github.com/openfoodfacts/smooth-app/issues/1501

Codecov Report

Merging #1812 (48854bd) into develop (2ea0da3) will decrease coverage by 2.11%.
The diff coverage is 2.36%.

@@            Coverage Diff             @@
##           develop   #1812      +/-   ##
==========================================
- Coverage     8.86%   6.74%   -2.12%
==========================================
  Files          161     169       +8
  Lines         6623    7630    +1007
==========================================
- Hits           587     515      -72
- Misses        6036    7115    +1079

Impacted Files
Coverage Δ

...h_app/lib/cards/category_cards/abstract_cache.dart
0.00% <0.00%> (ø)

...p/lib/cards/category_cards/asset_cache_helper.dart
0.00% <0.00%> (ø)

...p/lib/cards/category_cards/raster_async_asset.dart
0.00% <0.00%> (ø)

...oth_app/lib/cards/category_cards/raster_cache.dart
0.00% <0.00%> (ø)

..._app/lib/cards/category_cards/svg_async_asset.dart
0.00% <0.00%> (ø)

...smooth_app/lib/cards/category_cards/svg_cache.dart
0.00% <0.00%> (ø)

...th_app/lib/cards/data_cards/image_upload_card.dart
0.00% <0.00%> (ø)

...es/smooth_app/lib/cards/data_cards/score_card.dart
4.54% <0.00%> (-0.22%)
:arrow_down:

...t_cards/knowledge_panels/knowledge_panel_card.dart
0.00% <0.00%> (ø)

...t_cards/knowledge_panels/knowledge_panel_page.dart
0.00% <0.00%> (ø)

... and 101 more

Continue to review full report at Codecov.

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

Thanks @monsieurtanuki


⚠ Bofrost.at has degraded performance

In a4956d9, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1267 ms

Resolved: Bofrost.at performance has improved in 18ef274 after 6 minutes.


Example?

Would love to see an example of extracting the raw midi data from the message in C++/winrt. Is it really going to be necessary to create a DataReader to interpret the DataBuffer? Seems expensive for real-time.

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

ID: f7e67c6d-029e-9d8a-dc51-03e67e2292cc
Version Independent ID: 7e9bd3b0-a180-7e09-c3b6-82bfb0fa3de7
Content: IMidiMessage.RawData Property (Windows.Devices.Midi) - Windows UWP applications
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.devices.midi.imidimessage.yml&version=GBlive&_a=contents
Product: uwp
Technology: winrt-reference
Git Login: stevewhims
Microsoft Alias: stwhi

Hi, @jevansAN, I'd recommend making this an ask of the MIDI sample. It sounds like the ask is for a C++/WinRT version of the existing C# sample. You can file an issue against the samples repo if you follow that link.
-Steve

I'll go ahead and close this issue now since there's no further action we can take here. Please reopen if I made a mistake in closing. ~Steve


docker:build to use a temporary directory inside the ${basedir}/target to avoid multi-project build issues

I've just hit an issue where a second docker image in a build got jars from a previous docker build :)
It seems like the temporary folder docker:build is using doesn't take into account the current ${basedir} from the project module.
It'd be good to make the default use ${basedir}/target/docker/... or something?

I think we need to add ${basedir}/ to these properties
https://github.com/jstrachan/docker-maven-plugin/blob/master/src/main/java/org/jolokia/docker/maven/BuildMojo.java#L50-50
https://github.com/rhuss/docker-maven-plugin/blob/master/src/main/java/org/jolokia/docker/maven/BuildMojo.java#L45

i thought $basedir was always assumed if the path is relative

It doesn't seem to be in a multi-maven build; I can see the docker plugin using the exact output folder for every module ;)

yeah - i just noticed that myself but adding in $basedir where you suggested doesn't work but i have another idea.

ok - i have a fix for this. let me put together a pull request.

great thanks!
FWIW as a workaround I added this to my pom.xml:
        <docker.target.dir>${basedir}/target/docker-plugin</docker.target.dir>

https://github.com/fabric8io/quickstarts/blob/master/pom.xml#L79
which seems to be a work around

fixed - if you want to build a local copy, feel free to pull from my branch.

thanks @jgangemi

fixed in 0.11.2


elasticdump not importing from v1.00 on debian wheezy

Hi,
I played around with the elasticdump. I can make a full backup from the elastic-server using multielasticdump (all versions).
If I try to restore it, I only can use elasticdump up to 0.18.0 - if I try any version including/behind 1.0.0, it fails.
I tried it with nodejs 0.12, 4.x and 5.x - nothing worked :-<
elasticdump --output=http://<IP_ADDRESS>:9200/testindex --input=/not/for/public/testindex.json --type=data --debug=true
This is the output I receive.
Thu, 03 Mar 2016 18:01:56 GMT | starting dump
Thu, 03 Mar 2016 18:01:59 GMT | got 1 objects from source file (offset: 0)
[debug] | thisUrl: http://<IP_ADDRESS>:9200/testindex/undefined/undefined, elem._source: undefined
Thu, 03 Mar 2016 18:01:59 GMT | sent 1 objects to destination elasticsearch, wrote 0
Thu, 03 Mar 2016 18:01:59 GMT | got 0 objects from source file (offset: 1)
Thu, 03 Mar 2016 18:01:59 GMT | Total Writes: 0
Thu, 03 Mar 2016 18:01:59 GMT | dump complete
Any help would be nice.
Best
Ronny

Version Mismatch Sorry


Add strain

@BNAndras The stub and the example are swapped.

Oh jeez. This is why I shouldn’t port an exercise right before bed.


Customizable columns support

Currently we can see Time/Level/Message. Do you think we can make it customizable? The default config would be as follows:
[
    {
        "title": "Time",
        "ref": ".time"
    },
    {
        "title": "Level",
        "ref": ".level"
    },
    {
        "title": "Message",
        "ref": ".msg"
    }
]

Custom config:
[
    {
        "title": "Time",
        "ref": ".time"
    },
    {
        "title": "Level",
        "ref": ".level"
    },
    {
        "title": "Custom Field",
        "ref": ".meta.customField"
    },
    {
        "title": "Message",
        "ref": ".msg"
    }
]

WDYT?
P.S. I really like your viewer. It's simple, but so much usable in my case.

Hi, it is a good idea.
Config (by priority):

$PWD/.json-log-config.json
$HOME/.json-log-config.json
Default values.

And make the format a little more customizable (ref is a slice and kind for colorizing and for additional processing):
{
    "fields": [
        {
            "title": "Time",
            "ref": [".time", ".ts"],
            "kind": "time"
        },
        {
            "title": "Level",
            "ref": [".level"],
            "kind": "level"
        },
        {
            "title": "Custom Field",
            "ref": [".meta.customField"],
            "kind": "any"
        },
        {
            "title": "Message",
            "ref": [".msg", ".err", ".error"],
            "kind": "message"
        }
    ]
}

Yeah, I like it. Just maybe not sure how multiple values of ref are meant to be processed.
P.S. I'd maybe rename the config to just .jlv.json for simplicity. I feel your tool has a potential and deserves a good short name :)

@denisvmedia
This feature should be available in<EMAIL_ADDRESS>Details: https://github.com/hedhyw/json-log-viewer#customization


HARP-7109: Street outlines are visible even if outline width is 0

Codecov Report

Merging #921 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #921      +/-   ##
==========================================
- Coverage   47.62%   47.62%   -0.01%
==========================================
  Files         224      224
  Lines       21558    21565       +7
  Branches     4903     4907       +4
==========================================
+ Hits        10268    10271       +3
- Misses      11290    11294       +4

Impacted Files
Coverage Δ

...e/harp-mapview/lib/geometry/TileGeometryCreator.ts
14.48% <0%> (-0.17%)
:arrow_down:

@here/harp-test-utils/lib/TestUtils.ts
81.17% <0%> (+3.52%)
:arrow_up:

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 3a74c90...69e1755. Read the comment docs.

Rebuild please

@heremaps-bot retest this please

@heremaps-bot retest this please

@heremaps-bot retest this please


need "return_direct" function

The "return_direct" function working in python langchain tools, is accomplished in langchaingo? If sure, where is it? If not, how can I use the similar function?

nobody has similar problem？

@tmc


Capture stderr in case the workspace hasn't been created yet

Merging to get a fix in for the job. Please comment if you have issues with this.


1.4.13: Requirement for 'Hoverable' is unclear

The requirements for the 'hoverable' part of 1.4.13 Content on Hover and Focus say to:

Manage focus and move the mouse pointer directly from the trigger onto the new content.

I don't understand what 'manage focus' means in the context of making an element hoverable.
It sounds like "manage focus" refers to managing keyboard focus.
Note: I'm reporting all these issues because I believe that this doc will be very useful to many people once it's finalised.

I agree, when we discussed this in the team we couldn't make sense of it either.
Suggestion for writing it differently:

When hovering over the trigger, move the mouse pointer over the new content. That content should stay visible.

Sounds good to me.


Server: Monitor value change

Hi!
I need to monitor a node on a client. But there is no value displayed in the client monitoring window.

How to set up monitoring so that new values are displayed correctly?
Need this function to display updated values? How to use it (I didn't succeed)?

Thank you in advance

Tried monitoring in UaExpert:

In that version of UaExpert, what does it look like if you subscribe to a publishing node in the sample test server?
Also, could you take a Wireshark capture of network traffic with security set to None?

Several tests:

Sample test server monitoring in UaExpert:

It`s working, but some error.
In Matricon Opc Ua Explorer:

When you add a variable to monitoring, an error is generated and the connection is disabled.
In Prosys Opc Ua Client:

And I will do Wireshark Capture soon.

Wireshark Capture in multiple programs: https://drive.google.com/drive/folders/1t0SpjmXAlYrrWpVx9pHYESY1yooEb1R-?usp=sharing

Can you upload the 3 clients?

The clients: https://drive.google.com/drive/folders/1p7z_6tT8pFVGvst7kMaDOnXu_RhMpVnB?usp=sharing

For uaexpert, looks like PublishRequest includes an array of acknowledged publishes (subscription id, sequence number) and expects the next PublishResponse to include a list of status code updates on those acknowledges if I understand correctly. Those errors show up sporadically due to timing differences between uaexpert acknowledging a publish and the next response containing a useless "good" for that.
Not sure how fixing that would work because publishes are done asynchronously for performance reasons (either on timer, immediately when queued or triggers), but it's safe to disregard that warning I believe.
Matrikon is calling SetMonitoringMode, which isn't currently supported. I'll add support for that.
Looks like Prosys isn't sending any publish requests, will look for a workaround.

Matrikon fixed: https://github.com/nauful/LibUA/commit/4b6df3f06ce5f46ef57c1ccda01444491b055c75

I've added more UA address space variables that Prosys UA client tries to read when connecting, but it still won't send a PublishRequest.
Previous issue here: https://github.com/nauful/LibUA/issues/25
Closing for now, until I find more information about Prosys client. Other two clients are working now.

Thank you for the help!


[CFF2] Support ‘maxstack’

A late patch from @readroberts.

I see it's mentioned in CFF2 spec https://www.microsoft.com/typography/otspec/cff2charstr.htm, but don't see any reference to it in the CFF 1.0 spec (Tech Note #5176).

I should have raised this way before, but I truly don't see why we need a maxstack operator.  I'm sure all implementations could have just used the max maxstack and be fine. Bummer

I should have raised this way before, but I truly don't see why we need a maxstack operator. I'm sure all implementations could have just used the max maxstack and be fine.

Yes, but it's in the official spec by now so I believe we should support it until/unless is officially deprecated.

I plan to work in this in about two weeks, when I will also remove the global data references.

Thanks @readroberts.
Read, Miguel, you must have been using this updated cffLib for a while internally; do you believe this is stable enough to cut a new fonttools release?
I'd like to push out some varLib bugfixes today so that fontmake use of them. However, I fear that after merging cff2 branch we may have introduced other regressions in the CFF 1.0 part of the code which we might not be aware of.
I guess it's a bit too late to revert the cff2 merge; so I'm thinking of tagging 3.9.0 and wish that users will spot and report back any issues they may find.

@anthrotype it's as stable as it can be for the time being, until @readroberts has time to clean up the code. So I'd say go ahead with a new release. Would be great if the #881 changes were included as well, but it sounds like they will have to wait until they're validated.


Update azure-monitor-agent-mma-removal-tool.md

If we use the command in "Step 4 Uninstall inventory", it will fail with an error.

@ynakajima9 Thank you for your dedication to our documentation!
#sign-off


Update python version on boot.py

Update python version on boot.py
Update location of the base.bit

Codecov Report

Merging #1281 (250e27b) into image_v2.7 (d1cade7) will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           image_v2.7    #1281   +/-   ##
===========================================
  Coverage       46.59%   46.59%
===========================================
  Files              28       28
  Lines            4945     4945
  Branches          824      824
===========================================
  Hits             2304     2304
  Misses           2584     2584
  Partials           57       57

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 d1cade7...250e27b. Read the comment docs.


Storing activation after every minibatch

Hello,
Sorry to bother you again. Now I now how to manipulate the weight, I am wondering if there is a way to store the activation (i.e., output of a convolution layer) to a text file after a mini-batch?
I can use the tensorbard summary function, but only provides histogram, rms, etc but I would prefer to have access to all the values for post-processing.
My initial thinking is to modify the Conv2D code so create an additional tf.Variable to store the output of the tf.nn.conv2d, then I can access it after every minibatch through tf.global_variables().  However this doesn't seen to work because tf.Variable requires all dimension to be known.
Thanks for the help.

class MyCallback(Callback):
    def _setup_graph(self):
		t = self.graph.get_tensor_by_name('conv0/output:0')
        self._fetches = tf.train.SessionRunArgs(fetches=[t])

    def _before_run(self, _):
        return self._fetches

    def _after_run(self, _, rv):
        t = rv.results
        np.save('output-{}.txt'.format(self.global_step), t)

This is probably a common need (I'm using it recently as well), just added a callback DumpTensor: http://tensorpack.readthedocs.io/en/latest/modules/callbacks.html#tensorpack.callbacks.DumpTensor

Is there a way to dump the gradient tensors for a batch of inputs?

Create that tensor in the graph with tf.gradients and use the above callback.

Could you give an example? That would be helpful.
Say, in the tensorpack imagenet-resnet.py example, what should I do to dump the two gradients (wrt input and wrt weights) for CONV2 layer?

Also, I was using DumpTensor in my callbacks.
I put DumpTensor('conv0/output:0'), in callbacks but I get error:
KeyError: "The name 'c:0' refers to a Tensor which does not exist. The operation, 'c', does not exist in the graph."
What should I put for the name of a tensor? I was trying this on mnist-convnet.py example.
Thanks

DumpTensor expects a list of names:
Parameters:	names (list[str]) – names of tensors

This diff can make mnist-visualizations.py dump the gradients:
diff --git i/examples/mnist-visualizations.py w/examples/mnist-visualizations.py
index b9fc60c..1ff6803 100755
--- i/examples/mnist-visualizations.py
+++ w/examples/mnist-visualizations.py
@@ -112,6 +112,12 @@ class Model(ModelDesc):
                               regularize_cost('fc.*/W', tf.nn.l2_loss),
                               name='regularize_loss')
         self.cost = tf.add_n([wd_cost, cost], name='total_cost')
+
+        c1_input = p0
+        c1_W = c1.variables.W
+        grads = tf.gradients(self.cost, [c1_input, c1_W])
+        print(grads)
+
         summary.add_moving_summary(cost, wd_cost, self.cost, accuracy)

         summary.add_param_summary(('.*/W', ['histogram', 'rms']))
@@ -144,6 +150,8 @@ def get_config():
             ModelSaver(),
             InferenceRunner(
                 dataset_test, [ScalarStats('cross_entropy_loss'), ClassificationError('incorrect')]),
+            DumpTensor(['gradients/conv1/Conv2D_grad/Conv2DBackpropInput:0',
+                'gradients/conv1/Conv2D_grad/Conv2DBackpropFilter:0'])
         ],
         steps_per_epoch=dataset_train.size(),
         max_epoch=100,

Thanks. I had somehow missed the mnist-vis example. Its a nice example to access the internal variables. I also saw you updated the docs to mention these. Thanks again.

One (stupid) question:
How did you figure out the string for the gradient tensor? For example this string -- gradients/conv1/Conv2D_grad/Conv2DBackpropInput
I see such string in the events file in the log directory but for multi-gpu training they have "towerXX" in front of them. Is there a way to print the aggregate gradients?
Here is what I have in the callbacks and I am getting errors:
DumpTensor(['gradients/group2/block2/conv2/Conv2D_grad/Conv2DBackpropInput:0', 'gradients/group2/block2/conv2/Conv2D_grad/Conv2DBackpropFilter:0']),
DumpTensor(['group2/block2/conv2/output:0']),

KeyError: "The name 'gradients/group2/block2/conv2/Conv2D_grad/Conv2DBackpropInput:0' refers to a Tensor which does not exist. The operation, 'gradients/group2/block2/conv2/Conv2D_grad/Conv2DBackpropInput', does not exist in the graph."
The error is produced even when I run with a very small batch size on 1 gpu.

I used print(grads) in the above snippet to see the name. I agree this is annoying.
For multigpu training it'll be hard to access the aggregate gradients, because gradients are internal to the trainers. One feasible way is to write a GradientProcessor that does nothing but print the tensor.


Allow parenthesis usage in #if expressions in shader code

See https://forum.babylonjs.com/t/bug-when-parsing-some-if-in-shaders/9329
Parenthesis were not handled in #if expression parsing.
In the new implemention, the expression is first converted to postfix notation to easily generate the tree of ShaderDefineExpression instances corresponding to the input expression.
Note I first replace the defined(...) strings to defined[...] before converting to postfix as the converting code handles parenthesis as special characters (which is the point of this PR after all).

Do you want me to perform the Stack to _Stack changes (in another PR as this one is merged)?

Yes please :)


Leading type cast leads clangd to crash at code completion

I found two problems, when having a leading type cast leads clangd to crash at code completion.
Problem 1
void bar(){}
void foo() {
  (void)(bar)(^);
}

This could be fixed here: https://github.com/llvm/llvm-project/blob/main/clang/lib/Sema/SemaCodeComplete.cpp#L5605 by adding Fn->getType().isNull() but I think this is not a correct way to fix this.
Problem 2
Similar to the previous one. (this one could be reproduced only with C code)
struct Struct {
  int I;
};
int main() {
  struct Struct *S;
  return (int)(S)->^I;
}

This could be fixes with the similar check here: https://github.com/llvm/llvm-project/blob/main/clang/lib/Sema/SemaCodeComplete.cpp#L5168
Seems these problems are related, because removing leading type cast helps. Can't figure out how to solve this in the correct way =(

sent out https://reviews.llvm.org/D95935 for problem 1.

added the fix for second one into that patch too, since they are related.


Rework answer-api to be 2-staged

The sister PR on bloop/answer-api is: https://github.com/BloopAI/answer-api/pull/8

we no longer require it to respond in JSON, bloop/answer-api has been updated accordingly.


typo fixed

capitalized "V" in Version

@kuehndaniel Can we also update the COO info on the About Carbon page with this PR?

@alisonjoseph Any idea why the netlify build failed/if it matters?

Weighing in - The log says it was timeout, re-running to see if there is an actual problem.


Was wondering if you guys could point me in the right direction - really appreciate it.

Your demo app works fine but my pretty basic app throws an error in Android only that I cant seem to figure out.
its only when I have the relayer in the app.js
<magic.Relayer/>
My app is pretty basic - I am hoping I am just missing something simple.
import React, { useState } from 'react'
import { SafeAreaProvider } from 'react-native-safe-area-context'
import { useUser } from 'stores/user'
import * as SecureStore from 'expo-secure-store'
import { Magic } from '@magic-sdk/react-native-expo'
import { Navigation } from 'navigation'

const magic = new Magic('omitted-for-issue-discussion')

export default function App() {
  const [loading, setLoading] = useState(true)

  const setToken = useUser(state => state.setDidToken)
  const getUser = useUser(state => state.getUser)

  React.useEffect(() => {
    async function loadResourcesAndDataAsync() {
      let token = await SecureStore.getItemAsync("alphaToken")

      if (token) {
        setToken(token)
        try {
          await getUser()
        } catch (e) {
          //token expired
        }
      }
      setLoading(false)
    }
    loadResourcesAndDataAsync()
  }, [])

  if (loading) {
    return null
  } else {
    return (
      <SafeAreaProvider>
        <magic.Relayer />
        <Navigation magic={magic} />
      </SafeAreaProvider>
    )
  }
}

"dependencies": {
    "@magic-sdk/react-native-expo": "^14.3.0",
    "@react-navigation/drawer": "^6.5.8",
    "@react-navigation/native": "^6.1.3",
    "@react-navigation/native-stack": "^6.9.9",
    "expo": "~47.0.12",
    "expo-secure-store": "~12.0.0",
    "expo-status-bar": "~1.4.2",
    "nativewind": "^2.0.11",
    "react": "18.1.0",
    "react-native": "0.70.5",
    "react-native-gesture-handler": "^2.9.0",
    "react-native-reanimated": "~2.12.0",
    "react-native-safe-area-context": "^4.5.0",
    "react-native-screens": "~3.18.0",
    "react-native-webview": "^8.0.0",
    "unstated-next": "^1.1.0",
    "zustand": "^4.3.3"
  },
  "devDependencies": {
    "@babel/core": "^7.12.9",
    "tailwindcss": "^3.2.4"
  },

The error I am getting is:
Invariant Violation: Failed to call into JavaScript module method .onMessage(). Module has not been registered as callable. Registered callable JavaScript modules (n = 12): Systrace, JSTimers, HeapCapture, SamplingProfiler, RCTLog, RCTDeviceEventEmitter, RCTNativeAppEventEmitter, GlobalPerformanceLogger, JSDevSupportModule, HMRClient, RCTEventEmitter, AppRegistry.

Hey @lommaj thanks for point this out, we'll dig into it and see if there's anything we can do to prevent this on our side.
In the meantime, we've found this article about the error from Expo to be immensely helpful in trying to track down the source of the Invariant Violation errors.
Let us know if you learn anything new that'll help us find the fix.

I just want to say thanks for pointing me to the docs. I have resolved the issue - seems like the out of the box react native starter has mis-matched dependencies and once I adjusted them to match the versions in your project and pegged them to that version - it worked fine.
I really appreciate your time and I apologize for not trying that first.
thanks for the great work!!  This example project really helped me get magic implemented!


The system pod will not be deployed on the new added system node pool automatically

From the document, it states System node pools serve the primary purpose of hosting critical system pods such as CoreDNS and tunnelfront, but actually after adding a new system nodepool, the tunnelfront pod will still running on the existing nodepool even we changed the existing one to user mode.
Additionally, there is not much difference between system nodepool and user nodepool, all applications could run on them. I believe we have a plan for this in the future. But now, could you please correct the document to make all things clear?

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

ID: 225b99ee-4c4d-54c2-cafa-ddcd286438da
Version Independent ID: ca9410a8-8ddb-5528-a0ac-0963458f190e
Content: Use system node pools in Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/use-system-pools.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

@gsxmax
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@mlearned
Can you please check and add your comments on this doc update request as applicable.

Thank you @gsxmax for the feedback.
The article has been updated to address your feedback and the changes should be available soon.


Proposal: Add a #AVA_DESIGN_TIME preprocessor to help to write the designer specific codes

Is your feature request related to a problem? Please describe.
Currently I/m using MSDI as my Dependency Injection container. However, because I injected all the required components in the VM's and UserControl's constructor. This will cause the AvaloniaUI's preview to stop working:
Describe the solution you'd like
Introduce a preprocessor will easily solve this issue:

#if AVA_DESIGN_TIME
public RandomUserControl() // introduce an empty ctor for the previewer
{
    InitializeComponent();

    ...

    DataContext = new A();
}
#else
public RandomUserControl(A a, B b, C c)
{
    InitializeComponent();

    ...

    DataContext = a;
}
#endif

Describe alternatives you've considered
No response
Additional context
No response

Designer doesn't have a control over compilation process. It uses precompiled binaries.
So, this feature is a bit problematic to implement.

Designer doesn't have a control over compilation process. It uses precompiled binaries.
So, this feature is a bit problematic to implement.

Any possible ways to provide an empty ctor for the previewer in the design time? Currently all of my preview windows are dead because ctor with paras.

@laolarou726 we technically can reuse Design.PreviewWith property.
Something like:
public static class DesignData
{
    public static MyWindow Window => new MyWindow(DesignParam1, DesignParam2);
}

<Window x:Class="MyWindow"
        Design.PreviewWith="{x:Static DesignData.MyWindow}">
    <Button />
</Window>

Where previewer will take Design.PreviewWith instance instead of creating object itself.
The problem is, though, with current designer/previewwith code  it will likely go into a recursion.

I think parameterless constructor is allowed to be marked as internal
We have if (Design.IsDesingMode) which can be used to have a code path to cover that
a. Maybe above can be combined with # IF DEBUG, so the code is not in release mode

Internal ctor should be supported even now.

Thanks! I'll try internal ctor with #if DEBUG check first

@maxkatz6 @timunie  Looks like the previewer still not working for the internal ctor:
System.Xaml.XamlException: “Unable to find public constructor for type XXXViewModel()

yeah but for your ViewModel, check error message carefully!

@laolarou726 can you confirm the suggested solution works for you?

@laolarou726 can you confirm the suggested solution works for you?

Yes. It does work on some of my classes. But not all of them, I'm still trying to figure out why.


CLDC-1249 supported housing scheme

Pairing with Kat to finish the remaining issues:

Resetting location when a scheme is changed
Derive the location when a single one exists for a scheme


Updated tutorial-custom-images.md

Added a warning stating that the deprovisioning and capture process will lock down the original VM and becomes unusable.

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

@msLinuxNinja Thank you for the contribution! I'm pulling this over into the private repo to make some edits to the note.
#please-close https://github.com/MicrosoftDocs/azure-docs-pr/pull/84249


Update Operator docs to reflect latest Istio config

Docs only update
Update the wasme Istio operator docs to reflect the changes in the config field of the proto

/skip-changelog docs-only update.


Force local runs of devicelab tests to use en_US locale

Devicelab tests fail locally if the terminal locale isn't set to en_US (export LANG=en_US.UTF-8).
Updated instructions in https://github.com/flutter/flutter/wiki/Running-and-writing-tests#running-device-lab-tests-locally but it would be better if the tests handled forcing the locale.
See https://github.com/flutter/flutter/pull/51126#issuecomment-612061218.

Also, I didn't know about that wiki page, there are also instructions to running these in the README: https://github.com/flutter/flutter/tree/master/dev/devicelab.

Triage: there is a todo to add a health check.

I was suggesting the test runner itself add it instead of the CI environment needing to know anything about the required locale.
Something like
 environment: <String, String>{
  'LANG': 'en_US.UTF-8',
   if (deviceId != null)
     DeviceIdEnvName: deviceId,
 },

https://github.com/flutter/flutter/blob/78cbfbff4267643bb1913bc820f5ce8a3e591b40/dev/devicelab/lib/framework/runner.dart#L46-L49


Chain hypermedia calls.

I'm just wanting to see if you are interested in functionality like this. Basically, whereas before octokat.js assumed that ${key}_url should be converted into a function that returns a GET fetch promise, this branch converts ${key}_url into a Chainer object. Documentation and tests will have to be rewritten so this is not ready to merge. Just looking for some input.

This seems to be breaking nextPage() and friends. I think it might be possible to add a function for hypermedia methods without args. Alternatively, it might make sense to change the API so that you call:
js .nextPage().fetch()
Although that seems a little bit inconvenient.

My original reason for not adding was brevity but I think the chaining (as you described) makes for a more consistent API:
"Async calls are always verbs .fetch(), .read(), .create(), .update(), etc"
I'll try to update the tests and then merge these in to a new minor version since these will introduce breaking changes.

Actually, what about:

.nextPage() works as before (and deprecate)
.nextPage.fetch() or .nextPage.read() or others work as you described

That way there are no breaking changes?


Firefox style search with * and ^ in the address box?

To take most advantage of Fauxbar I enabled both "Show results from history" and "Show results from bookmarks" but quite often I'd like it to search bookmarks only since there are usually too many entries in the browsing history. So can we make it behave like Firefox's address bar?

If the query starts with *<space> then search bookmarks only
If the query starts with ^<space> then search history only

Found the secret is:fav in code.

Yeah, adding is:fav to the query is the only way to restrict the search to bookmarks at the moment.
Would be keen to make * and   behave like Firefox as options, sure.
Definitely room for improvement within Fauxbar's Options to explain such things better.

Hi ChrisNZL,
The following change works fine for me. Could you help integrate it into the repo and build a new release?
diff --git a/Fauxbar/js/fauxbar-1.js b/Fauxbar/js/fauxbar-1.js
index 73115a1..7c4fc84 100755
--- a/Fauxbar/js/fauxbar-1.js
+++ b/Fauxbar/js/fauxbar-1.js
@@ -2555,7 +2555,7 @@ function getResults(noQuery) {

                    for (var w in words) {
                        if (words[w] != "") {
-                           if (words[w].toLowerCase() == 'is:fav') {
+                           if (words[w] == '*' || words[w].toLowerCase() == 'is:fav') {
                                modifiers += ' AND type = 2 ';
                            }
                            else {

Marking this issue for the upcoming 1.9 version.
If you create a pull request, I can link it properly...
But if not, your change above looks fine; I'll integrate it in due course.
(Lately, have been slowly tackling the Manifest V3 refactor, so not really keen to push a new version live until that's done. But can apply your change as I go through the list)

I'd not bother making a pull request. Just integrate it. We really need this feature.


Inconsistencies in implementation and api docs for arrays of resource names

Revised summary
The service response with JSON arrays of object names is inconsistent as far as whether or not the service prefix (e.g. /hatrac) is present on each name, and this behavior differs from the REST-API documentation in this repo.  We suspect that there are programmatic clients working with these implementation quirks now, so we need to think about the right fix and transition strategy.
Original issue description
The chunked job retrieval for a URL mentioned here returns a list of paths, but they don't have SERVER_NAME i.e. hatrac prepended to it.
REQUEST
https://dev.isrd.isi.edu/hatrac/5678/17255efa2361c4d3799ad5937909fa8a;upload
RESPONSE
["/5678/17255efa2361c4d3799ad5937909fa8a;upload/PFQ2HYG5MXG356W6ZLK36NGQCE"]

The actual response should have hatrac prepended
["/hatrac/5678/17255efa2361c4d3799ad5937909fa8a;upload/PFQ2HYG5MXG356W6ZLK36NGQCE"]

Hmm, this is actually behaving consistently with the namespace listing API. Both return lists of hatrac resource paths as a JSON array and the listed paths are always paths missing the /hatrac prefix.  It seems strange to change this for one API and not both, but I suspect we'll have a backwards compatibility problem with existing clients if we change the namespace listing response.
Would you be able to negotiate and consume text/uri-list content type if I added that as an alternative response format?  This could return proper URIs like /hatrac/5678/17255efa2361c4d3799ad5937909fa8a;upload/PFQ2HYG5MXG356W6ZLK36NGQCE while leaving the JSON response unchanged for legacy clients...
I had intended to support this content negotiation but never got around to it since nobody seemed to need it. You'd just need to send an Accept: text/uri-list header in your request.

The new text/uri-list support is now pushed to master.  It is available for both namespace and job listings but they still default to the existing JSON response format if the client does not explicitly request text/uri-list.

@hongsudt I am not sure who should review this...

@karlcz I am ok sending the header Accept: text/uri-list

It turns out the service implementation is more inconsistent than I realized!  The returned JSON result for namespaces actually includes the service prefix, contradicting the api docs.
I'd like to make the following cases all consistent:

Content-Type
Namespace Listing
Version Listing
Upload Job Listing

json
["/hatrac/N/C1", ...]
["/hatrac/N/O:V1" ...]
/hatrac/N/O;upload/J1" ...]

uri-list
/hatrac/N/C1 ...
/hatrac/N/Oj:V1 ...
/hatrac/N/O;upload/J1 ...

This will mean changing the JSON response for versions and upload jobs to include the service prefix and to update the API docs.  For programmatic clients, the best way to implement a transition might be to start negotiating the text/uri-list which should always be correct (if available) and tolerating a JSON response to this request meaning that the server is running older code that hasn't been patched yet.
@mikedarcy @svoinea  @robes @hongsudt

The final round of code and doc fixes has been pushed to master.  All of the URI lists now include the service prefix both in application/json and  text/uri-list responses.

Maybe @chiragsanghvi  @mikedarcy and @svoinea can test their respective hatrac client codes against this version to ensure interop as a review task?

@karlcz This works

I have tested on rbk-dev the Namespace Listing for the json content type, and it works as described above.


Pygame running speed of website(pygbag) is much slower than Local-EXE

I made a simple games by pygame, and it is running good in Local-Exe files.
after use Pygbag to build it to a web-games,  the running speed is running very very slow in localhost:8000,  and also in itch.io.
May I have any comments to fix the issue, thank you very much

usually the first thing to check is your display size : 1024x600 is ok enough including mobile.
incremental display also saves a lot of processing ( ie pygame.display.update() instead of flip() )
next to check is game logic, especially assets/surface handling.


Broken on 8.1

Hello!
Is library currently not supported? It looks like it doesn't work all good on 8.1 (didn't check other os versions). Also "how to use" is misleading.

You have to use '.showOverlaySetting(true)', otherwise it won't prompt you to enable setting (and without it it doesn't show
You have to add android.permission.SYSTEM_ALERT_WINDOW to your manifest, otherwise you won't be able to enable setting
Misleading guide: onTerminate won't be called ever, so Takt won't be finished until system kills your process, so you need to move Takt.finish() to some other place.

Also it may be only my problem, but it somehow duplicates one over other, even when launched only once, so it's essentially impossible to see current fps :(

Working on android 4.4, but yes, broken on android 8.


Fixing biochemical examples

Codecov Report

Merging #42 (c15c683) into master (dc4c2e6) will decrease coverage by 4.86%.
The diff coverage is 87.94%.

@@            Coverage Diff             @@
##           master      #42      +/-   ##
==========================================
- Coverage   98.84%   93.97%   -4.87%
==========================================
  Files          10       11       +1
  Lines         433      465      +32
==========================================
+ Hits          428      437       +9
- Misses          5       28      +23

Impacted Files
Coverage Δ

src/libraries/libraryfunctions.jl
33.33% <ø> (ø)

src/libraries/standard.jl
50.00% <50.00%> (ø)

src/equations.jl
81.44% <69.23%> (-17.30%)
:arrow_down:

src/basetypes/BondGraph.jl
86.95% <78.94%> (-13.05%)
:arrow_down:

src/basetypes/AbstractNode.jl
100.00% <100.00%> (ø)

src/catalyst.jl
100.00% <100.00%> (ø)

src/construction.jl
100.00% <100.00%> (ø)

src/libraries/biochemical.jl
100.00% <100.00%> (ø)

src/plotrecipes.jl
95.65% <100.00%> (+0.19%)
:arrow_up:

... and 4 more

Continue to review full report at Codecov.

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


Move a direct lake semantic model to a different workspace

We have 10 direct lake semantic models in a premium capacity workspace, P1, in the South Central US region.
We need to move these models to a Fabric capacity workspace in the East US region.
Is it possible to use Semantic Link to copy or move the direct lake models to a different workspace?

We were able to use a deployment pipeline to deploy the model in premium capacity to a workspace in fabric capacity.

You can use this function. There is actually a bug in the function at the moment but I will be releasing a new version this week which has the fix.


Remove Dec 2020 global notifications

PR Overview
This PR removes the global notification, which was most recently set up in Dec 2020 to inform users of a resolved problem.

This PR also adds instructions on how to reactivate the Global Notifications should warnings/updates/etc be necessary in the future.

I have absolutely no idea why I didn't merge this last month.


Update cri plugin to v1.0.2.

Update cri plugin to v1.0.2 for containerd 1.1.1 release.
Release note: https://github.com/containerd/cri/releases/tag/v1.0.2
Signed-off-by: Lantao Liu<EMAIL_ADDRESS>

LGTM

Codecov Report

Merging #2376 into release/1.1 will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           release/1.1    #2376   +/-   ##
============================================
  Coverage        50.12%   50.12%
============================================
  Files               75       75
  Lines             7280     7280
============================================
  Hits              3649     3649
  Misses            2957     2957
  Partials           674      674

Flag
Coverage Δ

#linux
50.12% <ø> (ø)
:arrow_up:

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 2326b62...d05461d. Read the comment docs.

LGTM


Uncaught ReferenceError: definition is not defined

Hello,
I'm getting the error
JSON.sortify.js:1 Uncaught ReferenceError: definition is not defined
on latest version. I hadn't the same problem on earlier version.
Do I need to do anything different (I seems to comply with the docs, though) or something went wrong with the build?
Thank you!
Andrea

Thank you! I did a half-assed job on renaming a variable, just before releasing.
Note to myself: Don’t.
This is a perfect example why you need tests. -> GH-3

@AnimaLupi
Please try version 2.0.1, everything should be fine now.

Error is gone, thank you!


Caja does not make use of SMB2 FSCTL_SRV_COPYCHUNK

Copying or moving files within a share that holds two mounts of different devices sends the data to client and client has to send it back to server.
FSCTL_SRV_COPYCHUNK should avoid the indirection to client and directly move it from one device to the other on server.
Tested on Ubuntu Mate 18.04 with Caja 1.20.2. Server has samba 4.1+ installed.

I'm sorry, but you've found the wrong project named Caja. This project is about JavaScript sandboxing and has nothing to do with the desktop file manager.


Create new api_key fails

When I try to add new api_key i get error:

"SQLSTATE[22P02]: Invalid text representation: 7 ERROR:  invalid input syntax for integer: "8MARLdT6gVayOaSQ8YI9Bn5nRdSfqljbnD4iVBTUFPjWXDXvGj" (SQL: select * from "api_keys" where "api_keys"."id" = 8MARLdT6gVayOaSQ8YI9Bn5nRdSfqljbnD4iVBTUFPjWXDXvGj limit 1)"

I'm using postgresql

Hi,
Unfortunately I've never used postgresql in the past and can't really help you with this issue.
As screeenly uses Laravel under the hood I can only point you to the database docs.
I can also point you to the code which executes above query:
https://github.com/stefanzweifel/screeenly/blob/2187309ecf80e358de3043875c642e10e5d4c94f/modules/Screeenly/Models/ApiKey.php#L36-L45
And this is the database migration which creates the database table:
https://github.com/stefanzweifel/screeenly/blob/2187309ecf80e358de3043875c642e10e5d4c94f/database/migrations/2015_07_08_211146_create_api_keys_table.php#L15-L23

What is really weird is the WHERE clause. That should be api_keys.key and not api_keys.id.
select *
from "api_keys"
where "api_keys"."id" = 8MARLdT6gVayOaSQ8YI9Bn5nRdSfqljbnD4iVBTUFPjWXDXvGj
limit 1

This should be the query
select *
from "api_keys"
where "api_keys"."key" = 8MARLdT6gVayOaSQ8YI9Bn5nRdSfqljbnD4iVBTUFPjWXDXvGj
limit 1

I've updated the code a bit in https://github.com/stefanzweifel/screeenly/commit/2187309ecf80e358de3043875c642e10e5d4c94f. Maybe you can update your copy and try it again?

Thanks it works now


Explicitly convert undefined to null for res.json

It's too bad I can't tell if there are existing users out there who are expecting res.json() to work like res.type('json').end() :(

It's too bad I can't tell if there are existing users out there who are expecting res.json() to work like res.type('json').end() :(

It should, imo.

3.20.0, the last 3.x that will ever have an enhancement like this, will be released 2/18. This is just too risky to put into 3.x and is just going to be bumped from there. I think it makes sense in the end, so can easily be added for 5.0, but even the 4.x inclusion is iffy because of how it changes the body contents.

From reading the discussion, it sounds like this got decided as an addition into Express 5, but from what I can tell it hasn't made it in yet. Just to check, was the change to be made the one currently in this pull request, or was it one of the other discussed alternatives (ie throwing or returning an empty object)?
cc/  @dougwilson @Fishrock123 @xaka

Hm, I'm not sure why @dougwilson got unassigned from this, I guess it's actually some automatic thing and not my doing. I don't have any permissions to do anything like that.


api-docs: Remove incorrect error response from /api/subscribe.

Removes a response example in the POST users/me/subscriptions documentation that was listed in zerver/openapi/zulip.yaml as a 400 error response. It is actually a variation on the success response for this endpoint.
The current rendering of our API documentation is not set up to support "anyOf" which would allow for validating examples that match multiple response schemas. Adding that support can be a follow-up issue to fixing this error.
The updated documentation for the endpoint's response does clearly document that the unauthorized list of stream names that weren't subscribed is only present if "authorization_errors_fatal": false. And the unique error for when that parameter is true is still documented.
Screenshots and screen captures:

Subscribe to a stream - response documentation
Current documentation

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Merged, thanks @laurynmm!


Include re-exports in pyi file

The recent addition of the __init__.pyi file means VSCode's Pylance looks to that file for static analysis, rather than the .py file.
This PR copies re-exports to the .pyi file, allowing VSCode to resolve re-exported paths such as this:

from authzed.api.v1 import CheckPermissionRequest

# Without this PR, Pylance only resolves this path
# from authzed.api.v1.permission_service_pb2 import CheckPermissionRequest

Any thoughts on this one? It's pretty clunky to have to write imports like this to get type hints:
from authzed.api.v1.core_pb2 import RelationshipUpdate, SubjectReference
from authzed.api.v1.experimental_service_pb2_grpc import ExperimentalServiceStub
from authzed.api.v1.permission_service_pb2 import (CheckBulkPermissionsRequest, CheckBulkPermissionsRequestItem,
                                                   CheckBulkPermissionsResponse, CheckPermissionRequest,
                                                   CheckPermissionResponse, Consistency, DeleteRelationshipsRequest,
                                                   DeleteRelationshipsResponse, WriteRelationshipsRequest,
                                                   WriteRelationshipsResponse)
from authzed.api.v1.permission_service_pb2_grpc import PermissionsServiceStub
from authzed.api.v1.schema_service_pb2_grpc import SchemaServiceStub
from authzed.api.v1.watch_service_pb2_grpc import WatchServiceStub

@mateenkasim I'm a new maintainer and I'm also not super familiar with python library conventions around typing and interfaces. Is this a common pattern? It also looks like this is manually extending something that was manually created - does that match your understanding?
Do the import paths for the annotations match the import paths for the actual objects? Do they need to?
I think I'm generally good with this approach, but I want to make sure I understand the how and the why of it.

Sure, I'll put my understanding. The interfaces, or stubs, are meant to be a stripped-down version of the code itself, where the module path and code structure is the same, but implementation is missing. This gives something clean off of which you (or static-analyzers, like type-checkers) can read the type information. Sometimes types are enhanced in the stub file compared to the code file itself, since that's where analyzers are looking anyway. Example from an established project:

Django file for QuerySets: https://github.com/django/django/blob/main/django/db/models/query.py
Django stub file for QuerySets: https://github.com/typeddjango/django-stubs/blob/master/django-stubs/db/models/query.pyi

This can also hide information that you didn't intend to be publicly used, like imports of 3rd-party libraries that you need to use but don't want to re-export. In that case, you just don't put the import in the stub file.
In this case, I haven't checked the motivation or details of why __init__.pyi exists in the first place (maybe I should?). The Client/AsyncClient stuff isn't mine, so I didn't touch it. I do know it appeared here.

https://github.com/authzed/authzed-py/pull/149
https://discord.com/channels/844600078504951838/844600078948630559/1222604495867875369

All I know is that, if there is going to be a stub file in the first place, then it will need all the type/import information that we wanted publicly available from the implementation file.

Okay, that makes sense. I had a look too at what's autogenerated vs manually generated, and it looks like the current python protoc tooling doesn't create a __init__.py file anyway, so that was something that we built by hand. It looks like all you're doing in the pyi file is bringing it in line with what's in __init__.py and I'm here for it. I'm gonna go ahead and accept and merge this.
I'll talk with the team on Monday about getting a release cut.

One last question: if the .pyi file didn't exist and we just typed the __init__.py file, would it autodiscover? I suppose we can do that in future work if desird.


Added relative link to license.txt

[clabot:check]

Great idea @Minimalistic, I went ahead and merged this onto master.


NLog 4.0

http://nlog-project.org/2015/06/09/nlog-4-has-been-released.html

Hello Kori, thanks for the PR.
Checking the NLog source code, I see that they created an ILogger interface finally. It makes this library completely redundant.
If you still need to support the new version with NLog.Interface for some reason I'll merge the PR and create the package. Otherwise I'll put a note that NLog 4.0 makes this library obselete.

@kfrancis ping.

It's alright, I also saw the new interface available in 4.0.0 - you can just ignore this then, it's not really necessary for 4+


[2201.8.7-stage] Introduce transactionAutoCommitTimeout and transactionCleanupTimeout configuration values for transactions

Purpose
Introduces new configurations to prevent premature commits of long-running transactions before reaching the commit phase.
Related #42922
Approach
This PR introduces two new configurations values for transactions, transactionAutoCommitTimeout and transactionCleanupTimeout.
Samples
You can change the Config.toml with these two new configs as below.
[ballerina.lang.transaction]
transactionAutoCommitTimeout=120
transactionCleanupTimeout=600

Remarks
ballerinai-transaction PR: https://github.com/ballerina-platform/module-ballerinai-transaction/pull/550
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

@dsplayerX Can we change [ballerina.lang.transaction] to something else ? using lang seems this is something related to lang.transaction module, but this is not.

@gimantha If we were to put the configurables in module-ballerinai-transaction. How do we let the user pass the configurable values via the Config.toml?

@gimantha I checked and it works. We can directly pass the configurables to ballerinai-transaction like below.
[ballerinai.transaction]
transactionAutoCommitTimeout=150
transactionCleanupTimeout=600

Would that be okay?

@gimantha I checked and it works. We can directly pass the configurables to ballerinai-transaction like below.
[ballerinai.transaction]
transactionAutoCommitTimeout=150
transactionCleanupTimeout=600

Would that be okay?

My concern is whether it is correct to keep those configurables within the Ballerinai module. Normally, we don't expose ballerinai, do we? (This approach seems to expose the Ballerinai module to the user).

@gimantha I checked and it works. We can directly pass the configurables to ballerinai-transaction like below.
[ballerinai.transaction]
transactionAutoCommitTimeout=150
transactionCleanupTimeout=600

Would that be okay?

My concern is whether it is correct to keep those configurables within the Ballerinai module. Normally, we don't expose ballerinai, do we? (This approach seems expose the Ballerinai module to the user).

we shouldnt expose ballerinai. Can we not use a meaningful property for this?

There are a few minor suggestions(PR for mater) that we can address with a different PR.

Addressed in https://github.com/ballerina-platform/ballerina-lang/pull/42964.


Not able to build new iOS project

Describe the bug
After I integrate the amplify-tools.sh script "${PODS_ROOT}/AmplifyTools/amplify-tools.sh" into my project I cant build it anymore. It generates all the associated files and folders but that's it.

A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

created the Xcode Project "Todo"
installed pods, see pod file:

Got no errors while doing so:

There are a ton of warnings only shown really shortly, after that I just see "Build Failed" and cant make the app run. It is also not generating the model files when I set "modelgen=true".

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
macOS Version:

** Xcode Version:**

Cocoapod Version:
1.9.3
Node Version:
12.14.0
Amplify CLI Version:
4.21.0

We made a change to amplify-app and the structure of the generated files which may have fixed this. The warnings you are describing seem related to amplify-ios, I would suggest creating an issue in their repo if you want to pursue this further.

Okay - will do so.
Thanks
Daniel


Refactor move/zoom events

Refactoring to make movement-related events in Leaflet less confusing, more consistent across different types of movements/animations and fitting the new paradigm with fractional zoom levels and flyTo animations better.

[x] remove viewreset event and depend solely on zoom event in layers instead
[x] refactor confusing Map _resetView code into small separate methods
[x] refactor Map.TouchZoom code to rely on zoom events during pinch-zoom
[x] decide how to reset view when doing long-distance panning
[x] simplify transformation math to enable transforming vector layers from any state
[x] make Renderer rely on zoom events to scale renderer
[x] get rid of the ambiguity of relying on zoomanim in places when there's no actual animation
[x] stop vector reprojection/clipping/simplification on every frame #2982
[x] bring viewreset event back but in a different meaning (not zoom but full reset)
[x] fix tests

Closes #3216, closes #3134. Related: #2982

@IvanSanchez just rebased on master.

We need to make sure this doesn't introduce a regression of #3272

Yes. Also, pretty weird that the only failing test here is something about popups.

OK fiddling with this, I think we should bring back viewreset, just change the semantics so that it's only expected when the view is completely reset, not on zoom.
Now the vector layers are a special case in that they need expensive reprojection/clipping/simplification process, which we can't do on each frame because that would be too slow. So we have two options:

only reproject on moveend (e.g. after flyTo animation ended)
reproject on any move but heavily throttled (e.g. not more often than once per 100ms)

The first option can cause unwanted appearance in some cases, while the second can cause hiccups during a smooth flyTo animation, and are generally less reliable and can cause subtle/hard to track bugs (we've been burned by this in GridLayer refactor).
I think we should go with the first for now and see how it goes.

Got unstuck and made tons of progress here today! Vectors are working with flyTo, all the logic is much cleaner, all events should work as expected. Now we need to fix viewreset and tests, then test it thoroughly on stuff and it'll be a huge relief when this is merged.

\o/
I'll do some testing asap to join the party :)

Where is this Like button, when we need it so much?

👏🏼😄

@mourner, when you say "Vectors are working with flyTo", does this mean that they now render/ reproject as the flyTo animation is happening? If so, is there a parameter/ option that needs to be used?

@joshdickerson92 they now render as the flyTo animation is happening, and don't reproject every frame but CSS-transformed instead for performance. Do you have any issues with this?

yes, as per this Stack Exchange question the vectors layers don't appear to be rendering during the animation. To be more specific I am using a marker.cluster vector layer with this setup.

@joshdickerson92 please set up a minimal JSFiddle test case and open an issue.

It appears that my issue is actually when using a GeoJSON as this JSFiddle works as expected when purely using markers, however this JSFiddle uses a GeoJSON and you can clearly see the issue.

@joshdickerson92 hmm, your both JSFiddles are identical, so I don't see the issue.

@mourner Whoops, should have updated the latter one now. If not then here is the link

Ah, I see what you mean. The vectors are CSS-transformed, so zooming in they look pixelated and things like circle size doesn't change. I think we could update the behavior so that reprojection happens at least once in a while (e.g. every 200ms)

Maybe we could reuse the logic from the tile loading algorithm: update things once per zoom level change.

@IvanSanchez that is how I would perhaps expect it to work. @mourner can you see any quick work around to this in the time being?


how to specify oss trigger

trying to specify an OSS trigger but getting 400 error:
✔ Checking Service, Function, Triggers (2.4s)
✖ Creating Service, Function, Triggers (1.19s)
✖ Creating Service, Function, Triggers (0.8s)
✖ Creating Service, Function, Triggers (0.41s)
✖ Project fc-sxfi-api-msgbox-dev-sxfi-api-msgbox-dev-app failed to execute:
ERROR:
POST /services/sxfi-api-msgbox-dev/functions/sxfi-api-msgbox-dev-app/triggers failed with 400. requestid: da3d9d99-5b45-4a58-8dda-1c172e31f116, message: invalid OSS trigger config.
edition: 1.0.0
name: msgbox
access: default
vars:
  region: ap-southeast-1
services:
  fc-msgbox-dev-msgbox-dev-app:
    component: devsapp/fc
    props:
      region: ${vars.region}
      service:
        name: msgbox-dev
        description: msgbox-dev service
        role:
          name: msgbox-devDefaultRole
          policies:
            - AliyunLogFullAccess
        internetAccess: true
        logConfig: auto
      function:
        name: msgbox-dev-app
        description: msgbox-dev app
        handler: index.handler
        timeout: 30
        memorySize: 1024
        runtime: nodejs12
        environmentVariables:
          NODE_ENV: development
        instanceConcurrency: 5
        codeUri: ./
      triggers:
        - name: ossTrigger
          type: oss
          config:
            bucketName: msgbox-sg-dev
            events:
              - oss: oss:ObjectCreated:PutObject
            filter:
              Key:
                Prefix: /
                Suffix: ''

would like to find out what is wrong as there isn't much doc on how to specify it properly.

https://github.com/devsapp/fc/blob/main/examples/non-http-trigger/s.yaml#L37-L47
oss 触发器的示例配置，看起来是 events 配置错误，多了一个 oss:  可以尝试一下这样配置 - oss:ObjectCreated:PutObject

Thanks for the sample.
removing the extra oss: works. Thanks.


The method 'signInWithGoogle' isn't defined for the class 'FirebaseAuth.

Steps to Reproduce

...After firebase_auth and google_sign_in update, and AndroidX migrate am facing an issue  with this code, it was working fine doe
...

if (await auth.currentUser() == null) {
    GoogleSignInAuthentication credentials =
        await googleSignIn.currentUser.authentication;
    await auth.signInWithGoogle(
        idToken: credentials.idToken, accessToken: credentials.accessToken);
  }
}

...

Logs

  +50 ms] executing: [/flutter/] git rev-parse --abbrev-ref --symbolic @{u}
[  +67 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic @{u}
[        ] origin/master
[        ] executing: [/flutter/] git rev-parse --abbrev-ref HEAD
[  +15 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[        ] master
[        ] executing: [/flutter/] git ls-remote --get-url origin
[  +14 ms] Exit code 0 from: git ls-remote --get-url origin
[   +6 ms] https://github.com/flutter/flutter.git
[   +1 ms] executing: [/flutter/] git log -n 1 --pretty=format:%H
[  +34 ms] Exit code 0 from: git log -n 1 --pretty=format:%H
[        ] 04546cc69c627d8223b67097f957207a2904ac26
[        ] executing: [/flutter/] git log -n 1 --pretty=format:%ar
[  +31 ms] Exit code 0 from: git log -n 1 --pretty=format:%ar
[        ] 11 hours ago
[   +1 ms] executing: [/flutter/] git describe --match v*.*.* --first-parent --long --tags
[ +156 ms] Exit code 0 from: git describe --match v*.*.* --first-parent --long --tags
[   +5 ms] v1.1.9-194-g04546cc69
[ +576 ms] executing: /Users/test/Library/Android/sdk/platform-tools/adb devices -l
[   +9 ms] Exit code 0 from: /Users/test/Library/Android/sdk/platform-tools/adb devices -l
[        ] List of devices attached
           emulator-5554          device product:sdk_gphone_x86 model:Android_SDK_built_for_x86 device:generic_x86 transport_id:2
           emulator-5556          device product:sdk_gphone_x86 model:Android_SDK_built_for_x86 device:generic_x86 transport_id:3
[  +31 ms] executing: idevice_id -h
[  +60 ms] /usr/bin/xcrun simctl list --json devices
[ +304 ms] More than one device connected; please specify a device with the '-d <deviceId>' flag, or use '-d all' to act on all devices.
[  +12 ms] /Users/test/Library/Android/sdk/platform-tools/adb -s emulator-5554 shell getprop
[  +57 ms] ro.hardware = ranchu
[   +3 ms] /Users/test/Library/Android/sdk/platform-tools/adb -s emulator-5556 shell getprop
[  +51 ms] ro.hardware = ranchu
[   +3 ms] Android SDK built for x86 • emulator-5554 • android-x86 • Android 9 (API 28) (emulator)
[   +2 ms] Android SDK built for x86 • emulator-5556 • android-x86 • Android 9 (API 28) (emulator)
[  +19 ms] "flutter run" took 1,039ms.
[        ] "flutter run" took 1,039ms.

#0      throwToolExit (package:flutter_tools/src/base/common.dart:26:3)
#1      RunCommand.validateCommand (package:flutter_tools/src/commands/run.dart:235:7)
<asynchronous suspension>
#2      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:531:11)
<asynchronous suspension>
#3      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:488:33)
<asynchronous suspension>
#4      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:142:29)
<asynchronous suspension>
#5      _rootRun (dart:async/zone.dart:1124:13)
#6      _CustomZone.run (dart:async/zone.dart:1021:19)
#7      _runZoned (dart:async/zone.dart:1516:10)
#8      runZoned (dart:async/zone.dart:1463:12)
#9      AppContext.run (package:flutter_tools/src/base/context.dart:141:18)
<asynchronous suspension>
#10     FlutterCommand.run (package:flutter_tools/src/runner/flutter_command.dart:479:20)
#11     CommandRunner.runCommand (package:args/command_runner.dart:196:27)
<asynchronous suspension>
#12     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:396:21)
<asynchronous suspension>
#13     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:142:29)
<asynchronous suspension>
#14     _rootRun (dart:async/zone.dart:1124:13)
#15     _CustomZone.run (dart:async/zone.dart:1021:19)
#16     _runZoned (dart:async/zone.dart:1516:10)
#17     runZoned (dart:async/zone.dart:1463:12)
#18     AppContext.run (package:flutter_tools/src/base/context.dart:141:18)
<asynchronous suspension>
#19     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:356:19)
<asynchronous suspension>
#20     CommandRunner.run.<anonymous closure> (package:args/command_runner.dart:111:29)
#21     new Future.sync (dart:async/future.dart:224:31)
#22     CommandRunner.run (package:args/command_runner.dart:111:11)
#23     FlutterCommandRunner.run (package:flutter_tools/src/runner/flutter_command_runner.dart:242:18)
#24     run.<anonymous closure> (package:flutter_tools/runner.dart:60:20)
<asynchronous suspension>
#25     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:142:29)
<asynchronous suspension>
#26     _rootRun (dart:async/zone.dart:1124:13)
#27     _CustomZone.run (dart:async/zone.dart:1021:19)
#28     _runZoned (dart:async/zone.dart:1516:10)
#29     runZoned (dart:async/zone.dart:1463:12)
#30     AppContext.run (package:flutter_tools/src/base/context.dart:141:18)
<asynchronous suspension>
#31     runInContext (package:flutter_tools/src/context_runner.dart:49:24)
<asynchronous suspension>
#32     run (package:flutter_tools/runner.dart:51:10)
#33     main (package:flutter_tools/executable.dart:51:9)
<asynchronous suspension>
#34     main (file:///flutter/packages/flutter_tools/bin/flutter_tools.dart:8:3)
#35     _startIsolate.<anonymous closure> (dart:isolate/runtime/libisolate_patch.dart:298:32)
#36     _RawReceivePortImpl._handleMessage (dart:isolate/runtime/libisolate_patch.dart:171:12)

Analyzing ...

   info • This class (or a class which this class inherits from) is marked as '@immutable', but one or more of its instance fields are not final: ActivityFeedItem.mediaPreview,
          ActivityFeedItem.actionText • lib/activity_feed.dart:65:7 • must_be_immutable
   info • Unused import: 'dart:async' • lib/edit_profile_page.dart:2:8 • unused_import
   info • This class (or a class which this class inherits from) is marked as '@immutable', but one or more of its instance fields are not final: EditProfilePage.nameController,
          EditProfilePage.bioController • lib/edit_profile_page.dart:7:7 • must_be_immutable
   info • Name non-constant identifiers using lowerCamelCase • lib/edit_profile_page.dart:11:35 • non_constant_identifier_names
   info • Unused import: 'package:async/async.dart' • lib/feed.dart:4:8 • unused_import
  error • The method 'signInWithGoogle' isn't defined for the class 'FirebaseAuth' • lib/main.dart:46:16 • undefined_method
  error • The method 'signInWithGoogle' isn't defined for the class 'FirebaseAuth' • lib/main.dart:90:16 • undefined_method

[✓] Flutter (Channel master, v1.1.10-pre.194, on Mac OS X 10.14.4 18E174f, locale en)
    • Flutter version 1.1.10-pre.194 at /flutter
    • Framework revision 04546cc69c (11 hours ago), 2019-01-25 23:37:19 -0800
    • Engine revision 36acd02c94
    • Dart version 2.1.1 (build 2.1.1-dev.3.2 f4afaee422)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/test/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 10.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.1, Build version 10B61
    • ios-deploy 1.9.4
    • CocoaPods version 1.6.0.beta.1

[✓] Android Studio (version 3.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 32.0.1
    • Dart plugin version 182.5124
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)

[✓] VS Code (version 1.30.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 2.21.1

[✓] Connected device (2 available)
    • Android SDK built for x86 • emulator-5554 • android-x86 • Android 9 (API 28) (emulator)
    • Android SDK built for x86 • emulator-5556 • android-x86 • Android 9 (API 28) (emulator)

• No issues found!

Why is this coed giving an error??
FirebaseUser firebaseUser = await firebaseAuth.signInWith(
accessToken: googleAuth.accessToken,
idToken: googleAuth.idToken,
);
imports:
import 'dart:async';
import 'package:cloud_firestore/cloud_firestore.dart';
import 'package:firebase_auth/firebase_auth.dart';
import 'package:flutter/material.dart';
import 'package:google_sign_in/google_sign_in.dart';
import 'package:shared_preferences/shared_preferences.dart';
Pubspec:-
firebase_core: ^0.4.0+3
firebase_auth: ^0.11.1+5
cloud_firestore: ^0.12.1
firebase_storage: ^3.0.0
google_sign_in: ^4.0.1+3

@ebastuart  - your url is bad. you have (url) not (https://github.com/flutter/plugins/blob/master/packages/firebase_auth/example/lib/main.dart)

final FirebaseUser user = await _auth.signInWithCredential(credential);

i am getting this error ............
A value of type 'AuthResult' can't be assigned to a variable of type 'FirebaseUser'.
Try changing the type of the variable, or casting the right-hand type to 'FirebaseUser'.

I added cast now its working

final FirebaseUser user = (await _auth.signInWithCredential(credential)).user;
use this line instead of that It'll work just fine .

@pojoba02
This issue has been moved to https://github.com/FirebaseExtended/flutterfire/issues/484. Any further collaboration will be done there.

All Possible working solution [Solved] Undefined class ‘FirebaseUser’


Correction for raw affiliation Université Lyon1, ESPE, PRALIJE - UMR 5317 – IHRIM

Correction needed for raw affiliation Université Lyon1, ESPE, PRALIJE - UMR 5317 – IHRIM
raw_affiliation_name: Université Lyon1, ESPE, PRALIJE - UMR 5317 – IHRIM
new_rors: 029brtt94;02wmc6m46
previous_rors: 029brtt94
works_examples: W2982087517
contact: 8df0957a43f8d421d9bdefb17c9dfa95:10ee49d187a2e728afe9ed0ef3f7 @ ens-lyon.fr

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


Installation with pip

Hello
Is there any plans to standardize installation process with pip?
P.S. Thank you for such a convenient library!

Hello.
I've made it available on PyPi here
Now you can just run pip install pyxtension and it should work. Let me know if it doesn't.

Please, don't hesitate do add suggestions for new functionalities, or contribute to this repository.

Thanks a lot!


[improvement suggestion] Failed run-dependencies shoudn't block dependent packages

When run-dependency fails, it should produce a warning, and the dependent package should still be produced, instead of skipping it.
This will allow poudriere build to proceed as far as possible, because run-dependency isn't a blocking factor for build.

I don't like this, your original bug was about a broken package because its run dependency was missing.  I would rather not make it a feature to produce broken packages.

Closing for now.


fix: refactor file watchers (chokidar/linux workaround)

Types of changes

[ ] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

When the previous pr (#4717) got merged pi0 commented 'For experiment' (i think we discussed it on discord as well), but since that code actually got released in 2.4 its time to refactor/cleanup it a bit so (mainly) the files watcher also benefit from the workaround.
I'd prefer to move the createFileWatcher method from the Builder class to some util file (and pass this.options.watchers as an argument), but I was not sure what the best place would be. A util file in the builder package or a filewatcher file in util package? Or just keep it as a class member?
(I will fix the tests once the above has been discussed)
Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[ ] I have added tests to cover my changes (if not applicable, please state why)
[ ] All new and existing tests are passing.

@pimlie Would you please fix the CI?


Finish all in-progress documentation

There are several pieces of documentation that are in a half-done state. This issue is a reminder to go through all docs an look for TODOs and empty code blocks before the release.

FYI my plan is to just comment those out for this release -- everything in that page is just a nice bonus.

Not sure if that can be closed, but I removed it from the 0.21 milestone.


Fix Emission VCs not being considered Root

Fix PURCHASE-103
This is far from an elegant solution. Another option is amending the Emission View Controllers to have the isRootNavViewController property (which is currently considered false, which caused the bug).  That would mean an Emission release though, and I opted in favour for this, for the time being.

Definitely 👍

I would have expected that to work through this class extension, e.g. https://github.com/artsy/eigen/blob/41a0e35850ff4792ae3a0d7c700642749649ef2f/Artsy/App/ARAppDelegate%2BEmission.m#L238-L241. Maybe it did work for home but we just need more extensions for the other now RN root VCs?

Ha, it's already there.
I tested it with Home, Inbox, and Favorites. All are being pushed, rather than "tabbed" if you will, and skip that if block.
I can step through and test a bit more though! 👍

@alloy I might have been using the wrong route for home, but yes this works! Thanks 👍
I did keep the animated set to NO, because if we're swapping root, it should never be animated from what I can think of. Without changing that, it animates a "push" currently.

Makes sense 👍


Speeds up phase of registration by reducing runtime duplicated logic

Container build dynamic method for every registration from the very beginnings, including object graph analysis and emmiting CIL code. It requires refactoring, which allows to save much CPU time, which will cause the system to reuse already constructed and analyzed object graphs and already emitted CIL code.

Need to transform CilEmitter to GilEmissionRecorder and unbind it from ILGenerator.

Hm, no. Need to reuse already built code, not record method of building.


Unsupported activation: relu in function 'cv::dnn::darknet::ReadDarknetFromCfgStream'

modelConfiguration = "C:/Users/olivia lai/Downloads/poseEstimation/OpenPose-Multi-Person/pose/darknet/openpose.cfg"
modelWeights = "C:/Users/olivia lai/Downloads/poseEstimation/OpenPose-Multi-Person/pose/darknet/openpose.weight"
darknet = cv2.dnn.readNetFromDarknet(modelConfiguration, modelWeights)
And then I got this error

error                                     Traceback (most recent call last)
 in ()
----> 1 darknet = cv2.dnn.readNetFromDarknet(modelConfiguration, modelWeights)

error: OpenCV(4.0.0) C:\projects\opencv-python\opencv\modules\dnn\src\darknet\darknet_io.cpp:552: error: (-212:Parsing error) Unsupported activation: relu in function 'cv::dnn::darknet::ReadDarknetFromCfgStream'
Please help. Thanks so much for your work =D

I think OpenCV prompted this error because relu could not be used as an activation function in convolutional layers???


Show zoom level as percentage

zoom level should be shown as a percentage, as opposed to an integer.

Fixed in 315fc1f1d55be0688cd72ece9507d456d385a694


The shaded version of lettuce breaks EPollProvider

Pretty minor issue with EPollProvider
epollEventLoopGroupClass = getClass("io.netty.channel.epoll.EpollEventLoopGroup");
When maven shades this line, it appends com.lambdaworks to the front. The problem is, the epoll library is not included(perfectly fine, it's optional). So the class in the classpath is
io.netty.channel.epoll.EpollEventLoopGroup
while the EPollProvider is looking for
com.lambdaworks.io.netty.channel.epoll.EpollEventLoopGroup
I tried to add a filter for
io.netty.channel.epoll.* io.netty.channel.unix.*
to the maven shade relocate. It didn't change much. The string the code was looking for became
com/lambdaworks/io.netty.channel.epoll.EpollEventLoopGroup
The workaround is to simply not use the shaded version of lettuce. Not sure how to make shade-plugin do the right thing.

Thanks for the bug report. I think the bugfix is quite simple as the code needs to check both class names (io.netty.… and com.lambdaworks.io.netty.…).

I think the shade plugin is altering that string. The source I have for Epoll doesn't reference com.lambdaworks at all, but the arg to getClass leads with it.

My bad. I just started reproducing the issue, then I recognized the optional package is not shaded at all.

The fix is to remove the com.lambdaworks prefix in getClass. I'll include the fix for 3.4.2, 3.4.x, 4.1.2 and 4.x. The only other option seems to be writing an own Shader.


Update accounts list with new accounts.

Not the most efficient way to implement this but it works.
Fixes #50.

And besides updating the accounts list pane, this will also update all comboboxes that let you select an account.

tOK
Will write a separate issue to add double click protection to the add account dialog.


[DIR-1534] merge cycle 4 back to main

Description
These two hotfixes have been merged into cycle4 and need to be merged into main as well

https://github.com/direktiv/direktiv/pull/1482
https://github.com/direktiv/direktiv/pull/1472

⚠ After merging, we need to recover the branch to still have this available as a release targetbranch ⚠
Checklist

[x] Documentation updated if required
[x] Test coverage is appropriate

Checklist Internal

[x] Linear issue linked (e.g. [DIR-XXXX] pull request title)
[x] Has the PR been labeled

Closed in favor of https://github.com/direktiv/direktiv/pull/1531


Cranelift: Generate load/store using AMode::RegScaled on aarch64

Feature
Currently, on aarch64 backend, the following piece of CLIF instructions...
; Equivalent to: int64_t *v9; int64_t v10; v4 = v9[v10];
v1 = iconst.i64 3
v2 = ishl.i64 v10, v1  ; v1 = 3
v3 = iadd v9, v2
v4 = load.i64 v3

... will generate the assembly like below:
adrp    x4, 0x780000
ldr     x4, [x4]
lsl     x5, x3, #3
ldr     x4, [x4, x5]

However, the assembly can be converted into more efficient one like this:
adrp    x4, 0x780000
ldr     x4, [x4]
ldr     x4, [x4, x3, lsl #3]

Benefit
The shorter instruction sequence will help improve the performance.
In fact, this problem was found when I was diffing the assembly generated by cranelift and llvm, where llvm was around 20% faster than cranelift in my case.
Implementation
I've walked through the cranelift codebase and figured out that such addressing mode seems to be represented as AMode::RegScaled, but not sure how I can teach the code generator to use RegScaled for ldr.
Editing isle rules or something like that?

This would be great to have!
In fact, we even have a TODO in the code already (but unfortunately it looks like we didn't file an issue at the time, sorry!)
As can be seen at that link, we actually never translated the lower_address implementation to ISLE, so it's still done with manual pattern-matching in Rust. The ideal longer-term solution would be to rework it into ISLE, as we have on x64, which would make additional pattern-matching like this easier. I think we'd prefer that, but it's also a bit more work.
It might be possible to shoehorn it into the manual Rust code above, but it's a little tricky: the code works by collecting addends32 and addends64, with the semantics that the address is the sum of all of those (with 32-bit values zero-extended). We'd have to have a separate shifted (with a type something like Option<(Reg, u8)> to keep a value-in-register and a scale), collect it with the addends, and incorporate it if possible. But that's rapidly getting more complex than the equivalent in ISLE, so I think it's probably not the best approach.

Thank you for your quick reply.
I agree that it gets more complicated if we change the current lower_address implementation, but translating lower_address entirely to ISLE is also a hard work.
I'll try to translate a part of lower_address into ISLE, and hopefully make a PR.


Move Language/dialect fields

The Language/Dialect fields are now in a bit of an odd place. Where could we better move them?
Consideration: when there are data from multiple sign languages in a single database, we will need a filter to select which language's data to show. When we think of a good way to do that, displaying 'language' as a field (in the detail view) is a bit redundant. The dialect field could still stay where it is, or be displayed somewhere in connection with the language filter.

Would be good to combine this with #123.


i18n(de): update translation reference/configuration

Description
Translation of reference/configuration.mdx into German language.
Participating in Hacktoberfest!

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

de
reference/configuration.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

Link fixed in #2458

Link fixed in #2464


RSS Feed to Excel Mapper is needed

I want to demonstrate the "Mapping Excel to CSV" feature. In order for demo, I need some URL that

can respond *.xlsx file
the *.xlsx file is updated fairly frequently; more than several times a day, but not required to be so dynamic (second-by-second)
the information contained should be meaningful and interesting for human readers; samples of "A", "B", "C" are too boring, not inspiring.

I started with つみたてNISAの対象商品 as the firsut fixture, as I am familiar with it. But it has it's own limitation:

It is in Japanese; people who do not read Japanese would never understand it
It is owned by a Authority which is too high. I would be punished if I make a rude use of it.

So I need a better fixture.

Amazon provides a RSS feed.

 RSS Feed | Amazon.com, Inc

I would be able to develop a Mapper which converts RSS Feed XML into Excel xlsx.
The Excel of "Amazon Press Room News Releases" would be a good fixture for the demo.

"Map Reduce framework for Web Resources"

Amazon stoped providing RSS feed at that URL.


Update main.py so add() returns id of new memory along with message in returned dict.

changed
add():  ..........
return {"message": "ok"}
to
add(): .........
return {"message": "ok", "id":function_result}
so that the memory id of the newly created memory is returned in the dict. This is to allow external code to access memory ids when required without having an extra search step for each added memory.
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactor (does not change functionality, e.g. code style improvements, linting)
[ ] Documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Please delete options that are not relevant.

[ ] Unit Test
[ ] Test Script (please provide)

Checklist:

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

Maintainer Checklist

[ ] closes #xxxx (Replace xxxx with the GitHub issue number)
[ ] Made sure Checks passed

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Not required


Allow .cjs extension for the configuration file

What is your Scenario?
I work with a Node.js project with "type": "module" defined in package.json
So .js files are loaded as ES modules, while testcafe expects a module.exports statement (CommonJS shape) in .testcaferc.js
thus testcafe is not able to use that file.
If I set the .cjs extension to .testcaferc ("Regardless of the value of the "type" field, .mjs files are always treated as ES modules and .cjs files are always treated as CommonJS") testcafe won't recognize it.
What are you suggesting?
Some popular tools (eslint, prettier, mocha) allow .cjs extension to their configuration file.
It would be nice if testcafe allow the same.
What alternatives have you considered?
.json extension is the only alternative but it do not allow to create dynamic configuration files.
Additional context
No response

This is exactly what I came to inquire about.
I find myself in a situation where I want to use global before/after hooks which, according to the docs, are configured through the JS version of the config file. When I converted the existing JSON config to the JS format, I started getting the error An error has occurred while reading the "path/to/.testcaferc.js" configuration file. I had to poke around in configuration-base.js to see that the error was reported as so:
Error [ERR_REQUIRE_ESM]: require() of ES Module /path/to/.testcaferc.js from /path/to/node_modules/testcafe/lib/configuration/configuration-base.js not supported. .testcaferc.js is treated as an ES module file as it is a .js file whose nearest parent package.json contains "type": "module" which declares all .js files in that package scope as ES modules. Instead rename .testcaferc.js to end in .cjs, change the requiring code to use dynamic import() which is available in all CommonJS modules, or change "type": "module" to "type": "commonjs" in /home/jordan/repos/rp-web/uitests/package.json to treat all .js files as CommonJS (using .mjs for all ES modules instead).
I can't change the require() in configuration-base.js, and I have way too many existing import statements to just remove the type: "modules in my package.json. When I try changing the extension of my config file to .cjs, it gets loaded as a JSON file since the extension isn't recognized anymore. This leads to parsing errors, as the JS version is obviously not valid JSON.
If I may be so bold as to offer a suggestion, if the function that identifies the config as JS vs JSON were to also allow for the .cjs extension, this appears to work perfectly without requiring me to greatly refactor or otherwise change global settings in my code. I gave it a quick-and-dirty test drive locally, and it does seem to work fine, but that's obviously a long way from being properly tested.
If that approach sounds reasonable, I can open a PR.

Hi folks,
Thanks for pointing this out to us. It's a good enhancement of the Configuration subsystem.

Hello, a PR has been submitted: https://github.com/DevExpress/testcafe/pull/7614