Vallhorn motion sensor always reported as "clear"

Thanks for this integration. I installed this to incorporate a Vallhorn motion sensor to some automations, but unfortunately it's always reported as "clear" when e.g. the Google Home app correctly reports the activity.
I've dumped the JSON with and without motion being detected and I noticed that this sensor does not use the isOn attribute, but the isDetected one instead. Here's the dump while I'm standing in front of the sensor:
{
   "id":[redacted],
   "relationId":[redacted],
   "type":"sensor",
   "deviceType":"motionSensor",
   "createdAt":"2024-03-29T02:18:45.000Z",
   "isReachable":true,
   "lastSeen":"2024-03-30T12:13:56.000Z",
   "attributes":{
      "customName":[redacted],
      "firmwareVersion":"1.0.64",
      "hardwareVersion":"1",
      "manufacturer":"IKEA of Sweden",
      "model":"VALLHORN Wireless Motion Sensor",
      "productCode":"E2134",
      "serialNumber":[redacted],
      "batteryPercentage":87,
      "isOn":false,
      "isDetected":true,
      "motionDetectedDelay":20,
      "identifyPeriod":0,
      "identifyStarted":"2000-01-01T00:00:00.000Z",
      "permittingJoin":false,
      "sensorConfig":{
         "scheduleOn":false,
         "onDuration":600,
         "schedule":{
            "onCondition":{
               "time":"22:00"
            },
            "offCondition":{
               "time":"06:00"
            }
         }
      },
      "circadianPresets":[

      ]
   },
   "capabilities":{
      "canSend":[

      ],
      "canReceive":[
         "customName"
      ]
   },
   "room":{
      [redacted]
   },
   "deviceSet":[

   ],
   "remoteLinks":[

   ],
   "isHidden":false
}

Note the isDetected attribute set to true, which is false when the view is clear. Is there a quick way to patch my installation to use this attribute while a proper fix is implemented?

Answering my question, yes, there's a quick workaround. For reference:

I've added is_detected: bool here: https://github.com/sanjoyg/dirigera_platform/blob/3f3727a8d450363bea80f7460d1b11483263247d/custom_components/dirigera_platform/dirigera_lib_patch.py#L21-L23
I've changed this line to read the is_detected attribute instead: https://github.com/sanjoyg/dirigera_platform/blob/3f3727a8d450363bea80f7460d1b11483263247d/custom_components/dirigera_platform/binary_sensor.py#L104-L106

And now it works perfectly. This works for me because I'm planning to use Vallhorn sensors only, but a proper fix probably is bit more involved.

Just to clarify, so the value to read is is_on and not is_detected is that correct?

Try release <IP_ADDRESS> and let know. The issue should be fixed

I confirm it works perfectly, thanks!

Great, closing the issues


Lingering Floating Window rule

Hyprland Version
 System/Version info
Hyprland, built from branch HEAD at commit 03ebbe18ed8517ee22591eac82cd54322f42cb7d  (props: bump ver to 0.34.0).
Date: Mon Jan 1 12:03:15 2024
Tag: v0.34.0

Bug or Regression?
Bug
Description
I recently moved from i3 to Hyperland. I'm a heavy scratchpad user so i moved most of my "glance"-Apps into a pyprland skratchpad. Later on i decided that i want a specific "Messanger"-Workspace, so i removed all messenger related Skratchpads from pyprland and started them with the following script instead:
$ cat .config/hypr/scripts/focusOrStart.sh
#!/bin/sh

execCommand=$1
className=$2
workspaceOnStart=$3

running=$(hyprctl -j clients | jq -r ".[] | select(.class == \"${className}\") | .workspace.id")
echo $running

if [[ $running != "" ]]
then
	echo "focus"
	hyprctl dispatch focuswindow $className
else
	echo "start"
	hyprctl dispatch workspace $workspaceOnStart
	${execCommand} &
fi

Hotkey binding in hyperland.conf:
bind = , W, exec, .config/hypr/scripts/focusOrStart.sh "chromium --class=whatsApp --user-data-dir=$HOME/.cache/chromium/whatsApp-webApp --app=https://web.whatsapp.com/" "whatsApp" "9"

Now the Problem:
The apps always starts in a floating state and i have to manually make them tile. I don't have any window rule active for their class. Even if i completely disable pyprland the issue persists.
How to reproduce

make a app a pyprland skratchpad.
Remove the app from pyprland
Start the app normally

Crash reports, logs, images, videos
No response

Update:
i noticed the floating Behavior is cause by the --app command line switch of chromium

hey i'm getting the same issue with brave web apps ,
all window from --app open in floating windows , tiling manually is annoying !
i have created many .desktop file with line
Exec=brave --app=https://[any_site].com
which opens in floating windows always


APEXMALHAR-2324 Megh flume

Flume support from megh project. Preserved attribution for individual commits, filtered flume related commits only from the megh project. Applied formatting and licensing changes compatible with the Apex project as author Apex Dev<EMAIL_ADDRESS>@tweise @amolhkekre @gauravgopi123 @243826 @chandnisingh please see
megh: https://github.com/datatorrent/megh

@PramodSSImmaneni this PR contains a lot of noise when you look at the commit history. A lot of this can be compressed with rebase.

@tweise some of the newer build commits are easier to compress as they are clumped together, however, for others where there are consecutive commits from the same author it is possible to rebase them but it is a tedious process as the different author contributions are mixed together. How important is it to get it fully compressed. Would it be ok to clear out some of the recent build commit noise.

First pass should be interactive rebase that squashes subsequent commits of same author. That should trim it quite a bit. Recent commits that have to do with releases should all be squashed. Maybe we look at it again once these low hanging fruits are taken care of?

In a historic case like this, how important is it to preserve attribution ? Phrased differently: Have we ever had a case where this kind of distant attribution provided critically needed information ?

@tweise tried to squash successive commits from the same author but running into quite a few problems, since the commits are the result of multiple git filter-branch passes and there are resultant merge commits in there that throw rebase off. How about letting the commits be, many of them are old and if somebody is looking at all the commits in a log tool these will not add a lot of noise till they get to a while back. Also, the number of commits is around 350, while these are not a small number would only amount to 5% of total commits and will not add too much delay to fetch times.

@PramodSSImmaneni After looking at it a bit more I don't think that this commit history belongs into the Apex repository. I suggest that we squash and either attribute all files to Apex Dev or divide them between the main contributors. Most of the main code was written by Chetan, storage package by Gaurav, so it would be straightforward to commit the files like that.
I would also want to see the packages renamed and some testing done so that what goes into the repository is known to be working. What about the documentation?

A separate PR has been submitted with the requested changes
https://github.com/apache/apex-malhar/pull/598


H100 adaptability

int8 is not suitable for H100 device when using bitsandbytes.
So I add an judgment to automatically set the precision value.
Hoping you can receive! Thanks!!!

@yezhengmao1


Filter Config not quite working with Routes?

It feels like there is very little proper documentation on these filters.
They are a very good addon, but a little clarification would be good.
We have a filter, sessioncheckGlobal which deals with making sure that a user is logged in prior to accessing a webpage. The config for this filter is the following:
public $globals = [
		'before' => [
			'sessioncheckGlobal' => ['except' => ['maintenance', 'browseroutofdate', 'login', 'logout', 'website']],
		],
		'after'  => [
			'toolbar',
		],
	];

Where 'website' is in the exception list, it doesn't work.
Neither does '' (a blank element) or '/' (which stops the filter running at all).
So I guess the first question is, how do you make a filter not run on the main page where the main page is configured as the default controller?
In our case, the default controller is app/Controllers/Website/Website.php and the App is configured to route in this way:
$routes->get('/', 'Website/Website::index');
Second question, why does the following config make logincheck run on /admin/login ?
public $filters = [
		'loginfilter' => ['before' => ['login']],
	];

More routes if it helps:
$routes->get('/login', 'Dashboard/Login::index');
$routes->get('/admin/login', 'Admin/Login::index');
$routes->get('/admin/dashboard', 'Admin/Dashboard::index');

My understanding is that this filter should only run on website.com/login and not website.com/admin/login (if I wanted this behaviour, I should put "*/login" instead, right?)
Just in case it makes it any easier, our Controller structure:
Controllers
- Admin
      - Dashboard
      - Login
- Dashboard
      - Dashboard
      - Login
- Website
      - Website

Running CodeIgniter 4.0.0-beta.3
on PHP 7.3.5-1+ubuntu16.04.1+deb.sury.org+1
I know the way I've explained is very confusing but I've been sat looking at this all day and I'm going a bit crazy. Any better documentation or explanation would help, though I am not sure if its an issue with our configs or the actual framework.
Thanks

This is asking for help, not filing a bug. For help, please use our forums.
Luckily, one of your friends has already opened a thread there.

This is showing you incorrect behaviour of a feature - exactly what I consider to be a bug.

Right - but your initial post was "I'm not sure if this is a bug or if we're not doing it right." Which means you're asking for help. A bug report is "If I do X, then Y happens" and it should do "Z", with easily reproducible steps. Your post did reveal one bug with the login route which was fixed last night, but I was unable to replicate the primary thing you were asking for help on.
See the forum for more details.

@Pantoflarz Have you found a fix for this. I also checked the linked forum post didn't find a fix there either.


[Help]:

Contact Details (optional)
<EMAIL_ADDRESS>How can we help?
Hi Marcus
Two questions:-

Are you going to upgrade your node to the latest node-zwave.js ( 9.0)  and do we have to do anything when this is ready ?
We are having issues on some of our larger zwave networks where Node-zwave re-initializes the stick every time we do a deploy on Node-Red.  Is there something you can expose on the controller perhaps to prevent this ?

Any more details?
If this is specific to some hardware or specific software version, please explain here.
Version
No response
Node-RED Version
No response
What hardware are you using?
No response
Any code to add?
No response

Hi @ReziioAdmin

Are you going to upgrade your node to the latest node-zwave.js ( 9.0) and do we have to do anything when this is ready ?

I am indeed working on support for V9, I am part of the team Behind ZWave-JS, so naturally will follow major version updates.
this will come in the form of V7 of the Node RED Module.
It's actively being worked on, and is making progress.

do we have to do anything when this is ready

Other than upgrading within the Node RED UI - V7 contains some breaking changes (but you may not use these areas)
the following methods have their responses reviewed and updated

getValue
getValueMetadata

You can see the V7 (ZWJS V9) changelog here - which will get updated, as I progress.
https://github.com/zwave-js/node-red-contrib-zwave-js/blob/7.0.0/CHANGELOG.md

Node-zwave re-initializes the stick every time we do a deploy on Node-Red

This almost sounds like Node RED is set to full deploy - instead of just modified Nodes.
Node RED will restart any module when

The Node is Modifed
Set to Full Deploy

The Controller Node (the mothership if you will) will only restart when asked to by Node RED

Thank you.  I will watch progress on the changelog


Add inventory builder python script

Includes tox support for running unit tests.
Small note added to getting-started guide for using
inventory_builder.py
Also adds manual-only unit test.

tox test failed

Thanks @ant31. I fixed the gitlab-ci.yml path


feat: support installing helm chart

Signed-off-by: vgjm<EMAIL_ADDRESS>

@TomChv
Switched back to the first implementation. This one should be available and have a good readability.

Thank you, I'll play with your package and see what could be improved

LGTM!
Squash your commit to clean the history, make sure linter is happy and we can merge that one :D

Done : )


Add spaces layer

Adds the lot spaces layer and all its rendering logic
Bumps up the max zoom so that space icons can all be deconflicted

Closes #4

No plan/need to involve a tooltip or similar key for what the space icons mean?

No plan/need to involve a tooltip or similar key for what the space icons mean?

In this PR no.
Eventually this layer'll mimic the current parking map as in the spaces layer will be hidden by default and the filtering UI that shows them will include the icons + text labels (serving as a legend).
That's far down the issue chain though.


#132 Create Date scalar

Date scalar is string matching YYYY-MM-DD format, throws Error in case of format violation or invalid date (e.g. 2021-02-30).

Well done solution! Can we have some tests for src/schema/scalars/Date.ts?

Thank you. Added some tests.


Props to control start position and exit delay

Thank you for the awesome component.
I'd like to request two new props:

startPosition: A Number between 0 and 1 that controls the x-position the progress bar starts at.
exitDelay: A Number that controls the progress bar exit delay in milliseconds after navigation ends.

Hey new props are added 😊

Thank you for the fast push, @dhimankamal. The delay prop works; but on my end the progress bar now does not go all the way to the end (regardless if startPosition is set).

Hi @afif-nass, please share screen record video ?

@dhimankamal video is attached.
Component was used as follows:
<ProgressBar exitDelay={0} color={twConfig.theme.colors.primary} height="0.4rem" />
https://github.com/dhimankamal/svelte-progressbar/assets/47398601/13ae2373-8568-4a13-b8e5-0c62c6a23e76

Hi @afif-nass , this is also fix I publish new version. Check now

@dhimankamal works well when delay is not set. Same issue when delay is set. Try setting delay to 50 to reproduce the issue.

Hi @afif-nass , check now


text/template: template function index should permit integer values of float type

The text/template function index should permit integer values of float type as indexes, since there's no way to convert types in templates themselves.
I hit this particular problem when using unmarshaled JSON as the template data, similar to what's done in the play link below, where an integer in one part refers to an index of a list in another part, but when parsed, the integer is converted to float64.
Note that the function eq is also loose with types.
What did you do?
https://play.golang.org/p/_9K1wu6THk
What did you expect to see?
9

What did you see instead?
template: :1:2: executing "" at <index .a .b>: error calling index: cannot index slice/array with type float64

You should probably unmarshal into a concrete type, instead of unmarshalling into interface{}.
I don't think this is a change we want to make. I'm closing this, but I'll let @robpike or @adg reopen if they feel otherwise.

@bradfitz In my case, the JSON can be anything, hence why it unmarshals into interface{}. I suppose this might be an edge case, but it seems like we already went down this road with eq...

Happy to keep it closed.


Check, why ussability score is not added, then filter is checked

This is mobile information and now we only show desktop information.


Register ChequePreviewViewModel class and add business logic to the ViewModel

Create a class for ChequePreviewViewModel
Register ChequePreviewViewModelto the ViewModelLocationProvider at ChequeModule
Add business logic to the ChequePreviewViewModel
This is only use for Preview purposes.

Start after this work item is closed. #27


correcting volume changes not due to OBCS velocities

Hi all
OBCS_balance_flow corrects volume tendencies arising from obcs-imposed volume fluxes, but I don't think there is any way to correct nonzero budgets from other sources/sinks. Primary in my mind is, moving forward to synchronous ice sheet-ocean coupling, when grounded ice volume changes, sea level changes, which is amplified unrealistically in a small or regional domain. Im not sure if there are any other instances where this would be an issue ... sea ice import with realfreshwaterflux? I've heard anecdotally from a BAS scientist who was not using dynamic land ice this is possible, and can ask them for more details.
My former postdoc Kate Snow had an immediate solution: at each time step (if a flag is set), calculate the "mean" ocean volume defect (relative to a zero geopotential surface); convert this to a volume flux using an imposed time scale; and add it to the flux to be managed by obcs_balance_flow. We tested this in a number of idealised setups and it seemed to give good results. @jm-c @mjlosch is there a reason this could not be implemented, in some form?
Many thanks
Dan

Dan, I am not sure if understand the full context of your question, but there's a simple correction related to freshwater flux through the surface (model variable EmPmR). You can turn it on by defining
#define ALLOW_BALANCE_FLUXES
in CPP_OPTIONS.h and then there's a flag "balanceEmPmR" that needs to be set to true. With this flag, the global EmPmR is balanced to be exactly zero in each time step. This was meant to work for global configurations without open boundaries, but why not add this correction to any of the open boundaries? You'd just need to communicate the correction to pkg/obcs.

Hi Martin
Thanks for your response. I've had a look at balanceEmPmR. Two things on this:

it looks like there is no "communication" of this to pkg/obcs -- this is what I would like to do. My preference would be to add on to the variable "inFlow" (line 292 of pkg/obcs/obcs_balance_flow.F) and let the subroutine handle the rest, and i am hoping for feedback on such a change.

This would address excess precip, but I don't think I explained my specific issue well enough. In the movie below an ice sheet/shelf is thinning dynamically in response to melting of the ice shelf. There is displacement of water as well as the addition of fresh water and this will cause sea level to change; and with such a small domain, this will be accentuated. Communicating net EmP to obcs will not address this.

https://www.geos.ed.ac.uk/~dgoldber/darcy/Darcy2d.gif
Maybe there could be a general framework to communicate either excess volume OR precip imbalance to obcs_balance_flow?

Hi All
I am also trying to balance EmPmR with obcs; does anyone has an idea about it.

Hi
Check out
https://github.com/dngoldberg/MITgcm/blob/branch_horiz_remeshing/pkg/obcs/obcs_balance_flow.F
basically what is needed is a modification of the inFlow variable to your ends. To my knowledge this functionality is not implemented in the main branch.

Hi
excellent, thanks a lot; I am working on a high evaporative basin; will it be okay with shelfice options.

please understand this is just meant as a guide to show you what to do. the example i have pointed you is specific to shelfice. you will need to write your own version of lines 300-313 -- independent of shelfice -- to modify the "inFlow" variable to your needs. does that make sense?

yes; thanks

@dngoldberg I am sorry that no-one ever replied to your question. I think I now understand what you are doing: You assume that mean(eta)=0 is the "correct" mean sea level and add any change d(mean(eta))/dt to inFlow. In my initial reply I thought that any surface flux of freshwater (volume) could be used to do that correction so that mean(eta)=0 is not necessarily the result, but just a constant mean(eta) (which could be zero). So when there's is ice-shelf thinning due to melting, there should be a meltwater (freshwater) volume (or mass) flux that you could immediately use to compute the necessary compensation for inFlow. In the current main branch, this freshwater flux (variable shelfIceFreshWaterFlux) is added to the variable EmPmR: https://github.com/MITgcm/MITgcm/blob/f2a88c9ff84e58c10eb291fdd0b858dcaac37f8a/pkg/shelfice/shelfice_forcing_surf.F#L103-L104
So I think you could, instead of removing the mean from EmPmR as it is done with balanceEmPmR, one could pass this mean to obcs and use it to correct inFlow. Alternatively one could just do this for individual contributions to EmPmR (e.g. just shelfIceFreshWaterFlux), which I would not prefer.
Does that make sense? If so, we could try to work on something and create a pull request.

hi martin
i understand what you are saying and i believe this would be the answer if ice-shelf thinning were only due to melting. however, i am thinking of instances where there is dynamically active grounded ice in the domain. Consider an ice shelf fed by a glacier where, for some reason or another, experiences a large influx of ice from the glacier. With no melting taking place, this should displace ocean, with no array (such as shelficefreshwaterflux) to appeal to. This would matter very little to the global ocean; it matters a lot to an ocean in a small domain. If we assume that there is a "correct" sea level i.e. etan=0 (in reality of course, there is not) then i can see no other way to prevent sea level from diverging greatly from etan=0. does this make sense?
best
dan

But how do you account for the displacement then? By a change of etaH? I am not sure I understand how this happens.  Can you point me to relevant code? Can I find in the Branch remeshing PR #124?
For a free surface with "EXACT_CONSERV" turned on (i.e. also for non-linear fs) there is some variable detaHdt that records the change of etaH, that could be used, right?
A further complication with EmPmR is that S/R OBCS_CALC is called before anything else. One would have to move the call to S/R OBCS_BALANCE_FLOW out of OBCS_CALC and to the end of S/R DO_OCEANIC_PHYS to be consistent (otherwise one would work with field from the previous time step, and pickups would become an issue).
The free surface stuff on the other hand provide data from the previous time step, depending on which variant you use.
I would opt for a modification that is more general than just tailored towards a particular situation (ice shelves).

@mjlosch this is exactly what i am looking for; "instead of removing the mean from EmPmR as it is done with balanceEmPmR, one could pass this mean to obcs and use it to correct inFlow."

hi @mjlosch in response to how this is done -- please take a look at
https://github.com/dngoldberg/MITgcm/blob/branch_remeshing/pkg/shelfice/shelfice_thermodynamics.F
https://github.com/dngoldberg/MITgcm/blob/branch_remeshing/pkg/shelfice/shelfice_step_icemass.F
line 688. The load anomaly changes with shelficemass - and (if streamice is enabled) shelficemass is updated from streamice thickness each time step; and streamice thickness is updated via horizontal ice velocities as well as thermodynamically. Therefore, shelficeloadanomaly is not necessarily represented by the time-integrated melt rate in a given cell. I hope that makes more sense.
with response to consistency, i thought we had done this correctly but i will need to examine this (note the change to obcs_balance_flow i mention above is not part of PR #124)
the detahdt suggestion you make, however, might be reasonable (though more difficult); though i point out that the correction in obcs_balance_flow i mention above need not be tailored to ice shelves in any way.

hi @mjlosch and @dngoldberg : I hope it won't distract your discussion; I have some doubt- that you may be able to help me; I am very new to MITgcm; the EmPmR (tmpVal)  in https://github.com/dngoldberg/MITgcm/blob/branch_horiz_remeshing/model/src/remove_mean.F is converted to m/s by dividing with rhoconstfresh; before adding this term to inFlow of obcs_calc.f. what are the changes required;

@salimvkd I think that your questions are very valid, but they should be asked at the<EMAIL_ADDRESS>mailing list which is meant for this type of discussion. This here is meant for "issues", like bug-reports and code enhancements.
I am not 100% sure, but I would multiply EmPmR (or the mean of it) with mass2rUnit (which is 1/rhoConst in most cases) to get to m/s, at least that's what's done in the code all over the place. Note the sign convention: EmPmR is positive upward, so a positive value means an outflow, hence inFlow - mean(EmPmR) should be the correct sign.
I will answer the next question(s) only if posted at<EMAIL_ADDRESS>

@dngoldberg but then it sounds to me, as if the change in EtaH, ie. dEtaHdt, is the variable to use because it should include all effects that change the surface height, both freshwater fluxes from melting, evaporation, precipitation, runoff, and add ice mass (or any other mass). Can you try if using the area-weighted sum of dEtaHdt instead of yourETA/deltaT gives similar/same results? Or maybe provide a simple test where I can play with this?

@salimvkd I have tried to implement a freshwater flux correction based on the freshwater flux of the previous timestep (-PmEpR) on my fork.
Get it like this: git clone -b obcs_balance_surface https://github.com/mjlosch/MITgcm.git. Use OBCSbalanceSurf=.TRUE. to turn on the code. This will only work approximately, and will break restarts, because PmEpR is not available at the first time step of a restart. The alternative is to reorganise the calling sequence in do_oceanic_phys (see above) and call obcs_balance_flow at the very end after EmPmR is computed for the current time step. This gives almost perfect balance.

@dngoldberg after playing a little with this I have these observations:

Your method in https://github.com/dngoldberg/MITgcm/blob/branch_horiz_remeshing/pkg/obcs/obcs_balance_flow.F is very effective, but does not correct anything in the first time step (when eta = 0). I didn't test this, but I think this will happen for each restart (when mean(eta) will be nearly zero again).
using dEtaHdt instead does not work very well, because the correction only resets the sea level change of the previous time step; when you have a (nearly) constant sea level rise, than this compensates the sea level rise of the current time step, so that in the next time step the correction for the following time step will be very small. In this way you only correct the sea level rise every other time step.
for freshwater flux using EmPmR*mass2rUnit AT THE END of do_oceanic_phys gives nearly perfect balance.
In my simple test with a changing shelficeMass (I modified the isomip experiment to have a rapidly growing ice shelf), I realised that here the correction needs to take into account only the ice free part. The mean sea level does not change, because we do not really add mass, but just depress the surface underneath the ice shelf, which is hydrostatically balanced by a sea level rise in the ice free part of the domain. A little bit of thinking would have given the same without any experiments. In this context it could make sense to convert the new ice mass of this time step into an outflow.

My conclusion from this is, that one should move the obcs_balance_flow call to the end of do_oceanic_phys. There you'd have to account for freshwater (EmPmR) and icemass (shelfIceMassDynTendency) separately.
What do you think? I will add this option for freshwater flux into the branch obcs_balance_surface on my fork in a second commit for you to have a look.

hi @mjlosch many thanks for looking at this and thinking about it so carefully. your method sounds good but unf i cannot write a more detailed response until mid-next week -- will look into this very carefully then.

@salimvkd I have tried to implement a freshwater flux correction based on the freshwater flux of the previous timestep (-PmEpR) on my fork.
Get it like this: git clone -b obcs_balance_surface https://github.com/mjlosch/MITgcm.git. Use OBCSbalanceSurf=.TRUE. to turn on the code. This will only work approximately, and will break restarts, because PmEpR is not available at the first time step of a restart. The alternative is to reorganise the calling sequence in do_oceanic_phys (see above) and call obcs_balance_flow at the very end after EmPmR is computed for the current time step. This gives almost perfect balance.

Thanks a lot; I will try this;

Dan, I have pushed more code to my fork showing how I would include the extra mass implied by a moving (not melting) ice shelf. So far this does not balance perfectly, but I also find very strong non-linear effects, because my mass increase (by a prescribed shelficeMassDynTendency in a modified isomip.icefront experiment) is probably too strong and abrupt. By using the forcing (shelficeMassDynTendency) rather than the effect on a dynamic variable (mean etaN), one can make sure that there will be no stability issues stemming from this balancing code. All of this is not quite cleaned up, and I need to sort out a few things, but maybe this can be the basis for a new PR.

and I also meant to say that it would be nice @jm-c could have a look at this, too.

@dngoldberg @jm-c  I think that with the most recent commits (May08) I have now a working version of what I have outlined above. I think it is in a state where I could create a pull request (I would include a new test isomip.obcs to test the principle of this). Do you want to comment on this beforehand or should I just go ahead and create the PR?

Hi @mjlosch @jm-c  could you direct me where on your fork to look? A few things:

Your prev post says that you will be "correcting" obcs via shelficeMassDynTendency. I agree this could work when this is the one and only factor influencing ice mass beyond shifwflux, but  --

if ALLOW_STREAMICE is defined, then shelficemass is set directly from H_streamice. This removes the convenience of a rate (such as shelficeMassDynTendency) to use directly. I suppose this could be corrected by adding to pkg/streamice an array that takes dynamic thinning into account, but this seems extremely complicated. Far more so than using some sort of global average of ice shelf-free etaH.

Looking back at your post, your comments around using etaH directly are (a) it will not have an effect in the first time step, (b) it might not "work" upon pickup, and (c) the calculation should only take into account the ice-free part. See below --

a) obcs_balance_flow is called from do_oceanic_phys, so if etaH is nonzero initially, i don't see why this would be the case?
b) similarly, I am not clear why this poses an issue for the pickup?
c) the S/R i pointed you to https://github.com/dngoldberg/MITgcm/blob/branch_horiz_remeshing/pkg/obcs/obcs_balance_flow.F does exactly this, by invoking another S/R which averages eta over ice shelf-free ocean (which there is a mask for). So again Im not clear on why this would be problematic? Clearly for e.g. ISOMIP where there is no ice-free ocean, then it simply would not be used...?

Hi @dngoldberg , this is how to get the branch from my fork:
git clone -b obcs_balance_surface https://github.com/mjlosch/MITgcm.git
(or you can just go to my fork https://github.com/mjlosch/MITgcm.git and select the branch obcs_balance_surface, I guess). Note that the branch does not include any test experiment yet. If you want to see my test, then I will add it later today.
etaH will be zero in a simulation starting from rest. In the first time step, when obcs_balance_flow is called, there will be no correction (as etaH is zero) and this introduces a first lag. I haven't thought about pickups yet, so you are probably right and there would be no issue.
But the main reason for NOT using etaH is that it is a dynamic variable and this may introduce numerical stability issues because the correction of the outflow directly feeds back onto mean(etaH); I guess in most cases it won't be a problem, but I would like to avoid even this small potential for problems. Instead, I think that using the forcing that leads to a defined change in mean eta (i.e. the freshwater flux EmPmR or the change of ice mass) is much safer because there will be no feedback with these variables, because the new etaH will not affect the next EmPmR (and I am assuming it won't affect the ice mass, either).
I don't know exactly what to do when there is no "shelficeMassDynTendency". In my branch I suggest this:
https://github.com/mjlosch/MITgcm/blob/14e01419276fd09778dba002eb3c026f94bc682f/pkg/shelfice/shelfice_step_icemass.F#L151
C--   this assumes that the change in H_streamice include the melting flux
CML             shelfIceNetMassFluxTile(bi,bj) =
CML     &              shelfIceNetMassFluxTile(bi,bj)
CML     &            +(H_streamice(i,j,bi,bj)-H_streamice_prev(i,j,bi,bj))
CML     &            /deltaT * streamice_density * mass2rUnit
CML     &            * _rA(i,j,bi,bj) * maskInC(i,j,bi,bj)

but that's really just a rough guess. We would have to come up with something useful together. That's why I didn't create a PR yet, so that you may add stuff to my branch (assuming that you can create a PR for my branch, but do I know that?).

I just added a new verification/isomip.obcs to my branch, that can also be used for long tests. Currently the initial ice mass is not hydrostatically balanced, I am afraid, but otherwise it works.

Hey there iceshelfers. I have been looking at the code implemented in #251 and I'm a little bit confused. I'm curious about two things, specifically:

Since the melt rate in the variable SHELFICEFreshwaterFlux is negative if melting (adding volume to the ocean) and positive if freezing (taking volume from ocean), why is this being added to the inFlow? The specific line is:

https://github.com/MITgcm/MITgcm/blob/04904a066e85341574a85aef72438b5a212dc4f3/pkg/obcs/obcs_balance_flow.F#L344
I would expect this to be subtracted, so that, say in the case of net melting inFlow is greater than before.

In the case of using a realFreshwaterFlux with ice shelf melting (with or without BoundaryLayer), so that the freshwater flux gets added to EmPmR, wouldn't we want to skip over this line? That is, SHELFICEFreshwaterFlux goes into EmPmR in shelfice_forcing_surf, then EmPmR is subtracted from inFlow in obcs_balance_flow. Should we add an if statement such as:

SHI_useRealFWFlux = useRealFreshwaterFlux .AND.
     &  ( .NOT.SHELFICEBoundaryLayer .OR. SHI_withBL_realFWFlux )

       IF ( useShelfIce .AND. .NOT.SHI_useRealFWFlux ) THEN
        CALL SHELFICE_NETMASSFLUX_SURF(
     O                        shelfIceNetMassFlux,
     I                        myTime, myIter, myThid )

...
<other stuff>
...
IF ( useShelfIce .AND. .NOT.SHI_useRealFWFlux)
     &        inFlow = inFlow - shelfIceNetMassFlux*mass2rUnit

I have implemented this in my domain, which is an ocean only regional domain with no prescribed ice mass tendencies, and it seems to do the correction properly. However, I'm not sure if I'm missing something here.

Ok I see I'm mistaken in case 1 above, where SHI_useRealFWFlux=False, then the iceshelf flux shouldn't enter into any of this.
But now, I'm confused why the lines to compute the net mass flux from the ice shelf are even added to obcs_balance_flow in the first place, when the iceshelf flux is already added to EmPmR (before this routine). In this case, the shelficeNetMassFlux seems to be designed to cancel the shelfice portion that gets added to EmPmR in shelfice_forcing_surf. Why is this cancellation necessary? The iceshelf is adding volume to the domain, don't we want to take that into account in the inFlow variable? The comments in shelfice_step_icemass.F seem to suggest that this cancellation is necessary:
https://github.com/MITgcm/MITgcm/blob/04904a066e85341574a85aef72438b5a212dc4f3/pkg/shelfice/shelfice_step_icemass.F#L185-L189
Can anyone explain that to me?

hi tim i believe @mjlosch wrote this modification to obcs_balance_flow. I was arguing for something a bit simpler where "defects" in domain average sea level would lead to adjustments in inflow/outflow with some prescribed relaxation timescale, but concerns were raised regarding such an approach. (i was afraid that the implementation you are looking at might not work with grounded ice going afloat, but this needs to be checked.)

hi @timothyas the ice shelf has two contributions:

the mass flux due to advancing ice; this ice mass will, by Archimedes's principle, displace ocean mass and raise the sea level in a closed domain (in the world ocean this would be part of the eustatic sea level change); is the domain is small, the sea level rise will be excessive, that's why I added the option to let this excess flux go through the open boundaries. In shelfice_netmassflux_surf, this is only implemented for a prescribed mass flux in shelfIceMassDynTendency. With ALLOW_STREAMICE, this is only sketched.
the mass flux due to melting shelfIceFreshWaterFlux (positive upward). Any melting reduces (freezing increases) the mass flux due to advancing ice, that's why it needs to be added (beware of the sign) to shelficeMassDynTendency, so that shelficeNetMassFlux contains the mass flux due to an advancing ice shelf minus the melting flux. The melting flux is added to EmPmR.

There are two cases: 1. Iceshelves are static and there's no mass flux due to advancing ice shelves and this routine is never called; then the melting flux contribution is counted through EmPmR. 2. Iceshelves advance and the ice shelves displace ocean water of their mass minus their melt flux, because that's already taken care of through EmPmR. In this way the handling of the EmPmR part is independent of whether there advancing iceshelves or not.
Does that help? Maybe the comments in the code are misleading and should be changed?

Hi Tim,
I don't remember the exact details, but I remember that I mentioned to Martin that the
pieces of code for OBCS-balance with shelfice was doing something meaningful  but the comments
were not right. And this is assuming Dan's SHI_useRealFWFlux is turned off (=False),
since it was added after Martin's changes and did not try to make it consistent (or compatible)
with OBCS-balance & shelfice.
Jean-Michel
On Fri, Jun 05, 2020 at 03:13:35PM -0700, Timothy Smith wrote:

Ok I see I'm mistaken in case 1 above, where SHI_useRealFWFlux=False, then the iceshelf flux shouldn't enter into any of this.
But now, I'm confused why the lines to compute the net mass flux from the ice shelf are even added to obcs_balance_flow in the first place, when the iceshelf flux is already added to EmPmR (before this routine). In this case, the shelficeNetMassFlux seems to be designed to cancel the shelfice portion that gets added to EmPmR in shelfice_forcing_surf. Why is this cancellation necessary? The iceshelf is adding volume to the domain, don't we want to take that into account in the inFlow variable? The comments in shelfice_step_icemass.F seem to suggest that this cancellation is necessary:
https://github.com/MITgcm/MITgcm/blob/04904a066e85341574a85aef72438b5a212dc4f3/pkg/shelfice/shelfice_step_icemass.F#L185-L189
Can anyone explain that to me?
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/MITgcm/MITgcm/issues/145#issuecomment-639859708

Oh jeez, now I'm seeing it. Thanks for the thorough explanation @mjlosch!
I am working with a static iceshelf, but somehow convinced myself that SHELFICEMassStepping needed to be turned on ... but in the case of a static iceshelf this is not the case and this routine just never gets called... I guess I let the code trip me up or something - thanks for the explanation.
And thanks @jm-c. Actually, I think the code still works just fine with SHI_useRealFWFlux, since all this flag does is clarifies the logic through which the meltrate gets added to EmPmR. Once it's added to EmPmR, all of the balancing code is the same.
Hmm. Everything I was confused about in this thread seem somewhat obvious now. The comments are clear to me now, but I could suggest a small change to the comments:
 C--   Freshwater flux by thermodynamic processes. In the case of
 C     melting this will be negative (positive upward) so it will reduce
 C     the volume of ice mass added to the domain. The same melt (or freezing)
 C     volume has already been added to the domain through EmPmR.

I guess this just removes the mention of cancellation, but maybe this is a somewhat insignificant change. Perhaps more useful - with this new-found understanding I could add some quick lines to the documentation. I would add a paragraph or two explaining:

the flags and options to get virtual -vs- real freshwater flux with an iceshelf, given the boundary layer treatment
the options and cases described above which would lead one to use the OBCSBalanceSurf, and link to that section of the documentation.

What do you think? It is only unclear if it should go in the "shelfice" section or the "shelfice remeshing" section.


glTF2 use alpha, alpha to coverage and depth prepass

0bdbce672
For https://github.com/KhronosGroup/glTF-Sample-Models/tree/master/2.0/AlphaBlendModeTest cases.

Fixes the second case of blending from gltf2 test case.

Fixes hair for the the blend case.
Not fixed in the PR: Alpha being order independent transparency.

Need to evaluate again after https://github.com/godotengine/godot/pull/55084 and https://github.com/godotengine/godot/pull/55085

Closing for now due to need to reevaluate.


ops not propagating from pylink to charybdis with juno in the middle

does this work

does this work?

how about this

?

OK so sometimes convert_cmode_string() is putting an extra space in between the +modes and the TS6 UID targets. It seems to have fixed itself, so I am not sure what causes it. It will probably show up again in the future. The result is that charybdis ignores the parameter when there's an extra space.

Fixed at ec011a91453741d7f59ff7e27242c2b89dc5906f


ignore *.VC.db files

Reasons for making this change:
In addition to *.VC.opendb, *.VC.db file should be regenerated by Visual Studio automatically. So, it makes sense to ignore *.VC.db file too as it's been ignored in https://github.com/github/gitignore/blob/master/VisualStudio.gitignore
Links to documentation supporting these rule changes:
https://github.com/github/gitignore/blob/master/VisualStudio.gitignore
If this is a new template:

Link to application or project’s homepage: TODO

In addition to *.VC.opendb, *.VC.db file should be regenerated by Visual Studio automatically. So, it makes sense to ignore *.VC.db file too as it's been ignored in https://github.com/github/gitignore/blob/master/VisualStudio.gitignore

@NuLL3rr0r thanks!


Control panel

ping @ONE001, @eugulixes,
There is an strange behavior: when I click on the full screen button, the command rsz,24x80 appear in the console.

@dshil, it's ok. rsz used to be a command. So, this is the main reason why control-panel is experimental.

@eugulixes, ok, could we merge this PR with such behavior?

@dshil, yes.


[stable/atlantis] Allow creation of config.yaml

What this PR does / why we need it:
This allows the creation of a config.yaml for users to fill with whatever arguments they feel necessary
Special notes for your reviewer:
I only bumped the patch value but I did rename the ConfigMap so I'm not certain if that would require a minor version bump instead.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @callmeradical

Rebased

/unassign @callmeradical

/assign @lkysow

Hmm okay now I see (after reading this PR https://github.com/helm/charts/pull/14503) that this is required for Slack. Is that why you need this too?

Yes, this PR was started so we could configure Slack.


Remove setting edit_by when submitting references

Context
This was introduced in https://github.com/DFE-Digital/apply-for-postgraduate-teacher-training/commit/e129c5be147f4bfe158546cf5c166034ced8edd6, but I think it's not needed anymore. The edit_by date is already set when the application is submitted:
https://github.com/DFE-Digital/apply-for-postgraduate-teacher-training/blob/b607578b5536756e5a2e3b7da2a931a64dbacdd3/app/services/submit_application.rb#L13-L14
Changes proposed in this pull request
Remove the code.
Guidance to review
Is this right? @malcolmbaig to confirm.
Link to Trello card
https://trello.com/c/6aEZkNZi
Things to check

[x] This code doesn't rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] New environment variables have been added to the Azure config

I think I just understood the original intention. The edit_by isn't actually set to now if the references haven't come in for this application (this can happen when the candidate swaps a reference). So this code intends to prevent editing in this case. I think that still holds, even without this code, if we check for the form state when determing whether the user can still edit.

This looks sound to me 👍 . Now that we're:

enforcing the fact that apply_again applications can't be edited after submission
setting the edit_by for apply_again applications to now at point of submission

the code in SubmitReference doesn't need to know anything about the apply_again or edit_by concepts.


Updated the Bio and Location in Windows Client - Shows up on Live Website, but does not show up in Windows Application

Using Windows 10 Pro, Build 14393. Using Keybase Alpha regular for Windows, downloaded just minutes ago from the website.
Had no bio and location beforehand, so input in the application - again, it shows on the website and not in the application.
See pictures for proof:
On the website: https://i.imgur.com/jwBDcUD.jpg
On the windows app: https://i.imgur.com/faLtGxr.jpg
Feel free to ask more questions at keybase.io/icewobs or here on my GitHub.
Cool. Thanks.

CC @zanderz

It will be there next time the GUI is restarted.

I would love to read the Atlassian bug report, but it says that I don't have permission nor an account to see it. So RIP in user bug-checking I guess.
Also, I guess further improvements are being planned then? (ref. https://i.imgur.com/CPZLwbl.jpg)


SLN file searching problems for Unity3D projects

The SLN search stops at the first SLN file. That causes an issue with Unity3D. Unity gerates 2 SLN files "FolderName.sln" and "FolderName-csharp.sln". When it tries the former the assemblies are not loaded. When I delete the former and omnisharp finds the latter , everything works as charm.

What about Sublime Text? I have an issue and I'm afraid it has to do with this. Basically Unity creates the two .sln files cited above, however Omnisharp can't detect the 2008 version of the .sln, only the 2015 version generated by Unity through Visual Studio. Is there anything I can do?


#with in combination with ractive.set broken

Here is the fiddle:
http://jsfiddle.net/7tpzoo08/
Half-broken in latest
Doubly-broken in edge

Note to self (or anyone else bug hunting): Making the references within the with relative makes the example work, so it's hopefully a simple ref resolution bug.

I went ahead an looked into it, and this is another artifact of twoway binding. The binding on the select forces a value to be resolved so that the view and model match. Since there is no scoping on the references and with scoping isn't exclusive, they end up being resolved at the root of the model. Making the refs relative is a workaround ({{.choice}} and {{#if .choice === 'B'}} instead of {{choice}} and {{#if choice === 'B'}}).
The only way that I can conjure to solve this would be to add some sort of ambiguity tracking for references that could resolve in more than one context. That might end up being more surprising than the current behavior, though.

We could print a warning in this situation - if we get to this line, we could say 'Two-way binding warning: val reference is ambiguous. Consider initialising your data so that references can be resolved immediately, or using a restricted reference (http://docs.ractivejs.org/latest/references#restricted-references)'. Does that sound like a plan? I get headaches just thinking about how unpredictable things would become if we tried to track the ambiguity, or how we'd deal with view -> model changes on unresolved references...

@Rich-Harris
Hi,
From a user view point, there is nothing ambiguous about it. It's variable "x" within scope of "y". ...what should be ambiguous about that? Also, if you use another syntax, it works flawlessly:
http://jsfiddle.net/7tpzoo08/1/
It is broken using {{#with example}}... but works fine with {{#example}}...
...which leads me to think the issue is related to {{#with}}

It is variable x within the scope of y, but since there is no y, the scope stack pops and becomes root when it is forced by the twoway binding.
The plain mustache syntax does not really have an analogue in handlebars syntax. They have subtly different meanings (unless I have a misunderstanding of how they're supposed to work, which is a distinct possibility).
@Rich-Harris if that line is reached, perhaps along with the warning, the binding could also subscribe to its exact keypath and rebind if it changes? Or is that completely bonkers?

@evs-chris in the provided example, there is a y.x

@evs-chris Oh, you are right, at the time of creation it is indeed ambiguous. Indeed, a warning wouldn't harm.

I remember we discussed before about whether ambiguous references, when forced, should bind to root of instance or to current context. But I don't remember why exactly. Should ambiguous reference be bound to current context instead?

It is broken using {{#with example}}... but works fine with {{#example}}...

Yeah, that seems totally off. AFAIK they should resolve to the same thing.

Now that a syntax has been introduced to reference root scope ( {{~...}}) in  #889, I definitely vote for local scope preference.

@dagnelies It would seem to make sense that {{#with foo}}{{bar}}{{/with}} means foo.bar, not bar. But maybe there are counter examples.

I've just submitted a PR (#1765) that prints a warning in these situations. Local scope preference might make sense (though on the flip side, you can always do <input value='{{./foo}}'> to enforce local scope), but I'm not sure if the case is strong enough to change the current behaviour and potentially breaking existing code. Do you reckon we can get away with just warning/nagging about uninitialised data?

Do you reckon we can get away with just warning/nagging about uninitialised data?

@Rich-Harris I wouldn't change it with 0.7, maybe with a more compelling wiz-bang future release. :)

Will close this as I've just merged #1692 - running the fiddle above now causes this to be printed to the console:

Discussion about whether to default to local context is over on #1776.


Reduce two 'chdir' methods to one using 'rethrows'.

Also used ‘defer’ to simplify restoring the original working directory.

Nice @kareman, thanks for the pull request. This is much better.

Merged, the Circle CI failures are unrelated to these changes.

@kylef Wow that's got to be some kind of record for fastest merging of a pull request :)

Oh, that's a nice idea. Totally forgot about defer when I implemented that. :+1:


New operation for cloud-volume: safe-delete

Safe-delete operation for cloud-volume is similar to the existing delete operation.
The difference is that safe-delete isn't supposed to actually delete the volume. Instead it puts the volume on a watch list.
After a per-determined period (safe delete wait timeout) the volume is deleted if there was no IO operations during the period on the volume.

Hi @gregoryb-at-ibm
Is there a term that would be better suited here than safe delete? like stale, probation, mark for delete, super secret probation?
just reading the code did not give me a good idea of the difference between safe delete and regular delete.

Hi @kbrock

reading the code did not give me a good idea of the difference between safe delete and regular delete.

Would you like a more detailed explanation about what "safe-delete" is and how it's different from "delete"? Or were you just mentioning the similarity on MIQ side?
The code in MIQ is very similar between delete and safe-delete because they do essentially the same thing as most other MIQ operations -- call an endpoint on the provider. The difference is in the endpoint they call.
The delete operation calls the 'delete' endpoint on the provider. While the safe-delete operation calls the 'safe-delete' endpoint on the provider.

Is there a term that would be better suited here than safe delete

Is there a name that would intuitively confer the meaning of the operation without the need for further elucidation?
The term "safe delete" clearly indicates that the end is deletion. The "safe" part is reminiscent of safely-removing a USB drive from your laptop. And it indicates that the operation is similar to a regular deletion, but safer. Which is ultimately what the user should care about.
Some of the terms you had suggested, such as "stale", and "probation" might be used to craft a good name. But for now it eludes me.
@erantzabari @ereztheodorou , could you comment on the alternative names suggested?

Hi team, let me elaborate a bit more from another angle.
We can refer to the two action mentioned above as follow:
Force Delete - AKA Delete - This operation enforcing the volume won't be available immediately and its content will be destroyed. This operation has consequences and if it is done by mistake, it cannot be rollback and important data may be lost.
Which bring us to the next best practice action which storage administrators usually do:
Safe Delete - AKA Protected Delete - This operation perform a sequence of actions which protect the user from possible mistake. Those actions can be done by the manager or by the storage itself (a flag can be marked on the volume to be protected). The majority of storages and best practices of this function are waiting reasonable time to abort the action in case of mistake and second to verify that no use is done toward the volume.
Conclude that, we can either approach with one Delete action with selection whether Force/Protected or we can create two different action. AutoSDE team assume the later will be the right approach.
Either way, the user should be able to activate each one the two options.
Do you think "Protected Delete" is more suitable?

MacOS Activity Monitor has a Quit button.  When you click on it, it asks Quit or Force Quit (explaining the differences).  Wondering if we should consider a similar approach.  In the ManageIQ case, there is a 3rd option - delete from our database, but do NOT take any real action.

I agree this feels more like a delete option (:force => true/false) rather than a new operation

OK. 10x.
So we will go with the approach of delete operation with a flag Force/Not-Force (with explanation of the meaning).

On VMs we have a similar concept with Terminate/Delete and Pause/Stop - they are separate actions, but, in hindsight, I feel like that would have been better as an option to a delete or pause operation, respectively.  Wonder if we can change those existing operations to work that way.

Hi @agrare and @Fryguy. What do you think about Himdel's recommendation?

Yeah, separate buttons would be simpler, and would also be consistent with how terminate/stop is now, so it's be ok with that.
My concern above is more around documenting the difference.  Its hard to explain to someone why there are two different buttons and why they are different, whereas a dialog with options under a single button makes the distinction clearer.  I'll hands-down take user experience over us having to maintain a dialog any day.

Closed in favor of https://github.com/ManageIQ/manageiq/issues/21055


build: Update Vulkan-Headers and VUL to 1.3.294

CI CrashDiagnosticLayer build # 231 running.

CI CrashDiagnosticLayer build # 231 passed.


Reach out to ("silent") users of ORT to ask for permission to use their logos

We should include some nice company logos to ORT's README.md to show off who's already using ORT, also to make us more attractive to more users.

This has been partly addressed with the introduction of ADOPTERS.md where contributors can add their own logos. We should still encourage contributors to actually do so, e.g. by adding a note to the pull request template, and leading by example for the companies the TSC members are working for.


chore(main): release audit 1.27.0

:robot: I have created a release beep boop
1.27.0 (2023-03-07)
Features

generate ids for Sandworm issues (ccaf8ea)

Bug Fixes

better error for no lockfile found (4c430d9)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/sandworm-hq/sandworm-audit/releases/tag/audit-v1.27.0 :sunflower:


Update Footer

Description 📜
Added Footer Template according to the design suggested. Please review my PR and revert in case any changes need to be made. Any Suggestions and Feedbacks are Welcomed.
I did not use any framework, the code is pure HTML and CSS.
Fixes #4

This would be great if you provide any of this   📝

[x] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[x] Footer

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm Hacktoberfest contributor

Screenshots / Gif (Compulsory) 📸

@Ayan-thecodeking Please review my PR and revert in case any changes need to be made. I created the footer element in accordance to design given by you without using any frameworks, just pure HTML,CSS.
I have also attached the screenshot of the footer element.

@Ayan-thecodeking Please review my PR and revert in case any changes need to be made. I created the footer element in accordance to design given by you without using any frameworks, just pure HTML,CSS. I have also attached the screenshot of the footer element.

Hey @pranshukas pls add screen recording ... showing responsiveness of atleast footer ....or if you want you can try for whole website

@Ayan-thecodeking
How can I make commits now? It's showing resolve conflicts. You merged a PR that had some lines changes conflicting with mine. Should I create new PR or I there's another way ?

@Ayan-thecodeking Please check I have added the screen recording of my work.
https://user-images.githubusercontent.com/58984074/135708709-aec37751-578c-42b0-8568-31eb78327798.mp4

ok cool @pranshukas just resolve the conflicts ...
or if you will get any issues in resolving then do create another pr and close this


option to prevent word break on new line

Hi, thanks for a great plugin!
I'm using it in an element that has a fixed width, so I'm getting word breaks
to new line... is there a way to prevent this already?
or.. please add this feature :)

If I'm understanding this correctly, you want  VueTyper to not line-wrap and instead overflow the container,
If so, just put VueTyper inside your fixed-width element, and then set VueTyper to position: absolute.

 I actually solved this by using 3 VueTypers, one for each line for my purpose :) but it would be nice as a built in functionality..

Ah, your original post said you wanted to prevent word breaks, which was doubly confusing because word-break in CSS would let you break-by-letter, which I believe is the current VueTyper behavior.
Line wrapping on normal text can behave many ways depending on the CSS word-break or overflow-wrap. Ideally there would be a way to simulate that on VueTyper.
I likely won't have free time in the near future to look into this, so a PR would be welcome.

I created a new feature request with more details. Closing this one.


Droppable and Draggable at the same time

Hi all,
I've a quite simple problematic but I don't know which is the best solution to achieve it !
Actually i've an object that is draggable but also droppable, I've implemented draggable successfuly, but still impossible to wrap this to be droppable

                 <Stack>                     {item.name}                 </Stack>

Refer to how the useSortable hook work, which combines the useDraggable and useDroppable hooks.
The trick is to combine both the useDraggable and useDroppable refs https://github.com/clauderic/dnd-kit/blob/master/packages/sortable/src/hooks/useSortable.ts#L79


Error message in dashboard: "Something went wrong: Unauthenticated"

Hi,
I tried to set up the checkout and checkout-app according to the deployment manual in README. However, I can't load or save settings in checkout-app (in dashboard).

The error message like this is quite useless. I have tried to reinstall the app, generate fresh authToken, redeploy both of vercel apps, to no avail.
What is actually wrong, and what can I do to fix it? What is unauthenticated?
Thank you!
EDIT:
I can see multiple requests in browser console:
GET https://saleor-checkout-app.vercel.app/api/payment-provider-settings
HTTP/401 Unauthenticated
{"error":{"message":"Unauthenticated"}}

POST https://saleor-checkout-app.vercel.app/api/set-payment-provider-settings
HTTP/401 Unauthenticated
{"error":{"message":"Unauthenticated"}}

For anybody else struggling with this, I managed to get this working by removing the issuer check, and replacing all the usages of the issuer (which is a domain name) with the domain extracted from the existing NEXT_PUBLIC_SALEOR_API_URL env var. This should be safe so long as the jwksUri is also directed at the domain that your api is hosted on.
The files which need changing are all in saleor-app-checkout:

auth.ts
payment-provider-settings.ts
set-payment-provider-settings.ts

For what it's worth, while having the issuer on the token would be ideal, it'd probably make more sense for payment-provider-settings.ts and set-payment-provider-settings.ts to use the apiUrl env var instead of reconstructing it from the token issuer in any case.

It's a Saleor Core bug. It'll be fixed in the core.
Internal link for reference: https://app.clickup.com/t/2549495/SALEOR-7262

Any updates, ETA when this will get fixed?

Closed via:
https://github.com/saleor/saleor/pull/10845
https://github.com/saleor/react-storefront/commit/4a93b7e4bc33f9d2a4f5ccd4e4c54b18560a910d


Need a way to force players to not stay in the planet for 10+ minutes

My idea is that if it's been more than 10 minutes since the last FTL, start spawning wildlife if they're planetside or begin putting ships out. A bad workaround would have the FSC just glass the point the players are at but that's boring as hell

This is bad and should only be for planets

Okay we have naturally spawning fauna we're all good


Ensure CMAKE_INSTALL_LIBDIR is set consistently

There's another include(GNUInstallDirs) in ROCMCreatePackage.cmake, so we were getting different behaviour depending on the order of includes.

No, but it's been about a week and the change has almost managed to percolate through the math-ci. I think we can merge.


Download Glitch

Steps to reproduce
click on download but it doesnt download
Expected behavior
it suppose to download the chapter clicked on
Actual behavior
doesnt download
Mangayomi version
0.2.4
Device
kali linux
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have updated all installed extensions.

same issue, shell output on linux is:
'/home/nix/Applications/Mangayomi-v0.2.4-linux.AppImage'
package:media_kit_libs_linux registered.
[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: LateInitializationError: Field '_storage@1813152722' has already been initialized.
#0      LateError._throwFieldAlreadyInitialized (dart:_internal-patch/internal_patch.dart:184)
#1      BaseDownloader._storage= (package:mangayomi/services/background_downloader/src/base_downloader.dart:47)
#2      new BaseDownloader.instance (package:mangayomi/services/background_downloader/src/base_downloader.dart:107)
#3      new FileDownloader._internal (package:mangayomi/services/background_downloader/src/file_downloader.dart:45)
#4      new FileDownloader (package:mangayomi/services/background_downloader/src/file_downloader.dart:39)
#5      downloadChapter (package:mangayomi/modules/manga/download/providers/download_provider.dart:237)

#6      FutureHandlerProviderElementMixin.handleFuture.. (package:riverpod/src/async_notifier/base.dart:348)

Fixed!


QuasiDistribution/ProbDistribution key change methods behave differently to Counts

Environment

Qiskit Terra version: 0.22
Python version: 3.9
Operating system: MacOS

What is happening?
The binary_probabilities hex_probabilities methods of QuasiDistribution and ProbDistribution return a regular dictionary instance rather than the same original such as is done with Counts when converting key types.
How can we reproduce the issue?
Call quasi_distribution.hex_probabilities() will return a regular dict for example, while calling counts.hex_outcomes() returns a Counts object.
What should happen?
These classes should behave like Counts in that changing the type of the keys should not change the type of object so that key types (int, bin, hex) can be switched back and forth as needed.
Any suggestions?
I think it would make a lot of sense to have a base dict subclass that is just a keyed default dict that can handle the conversion of keys between integer, bitstring and hextring (like counts currently does) for any value types, and then Counts, ProbDistribution, QuasiDistribution could all inherit from this.

I tried comparison and saw Counts.hex_outcomes() returns a regular dict too.
from qiskit.result import Counts, QuasiDistribution

c = Counts({'00': 0.5, '11':0.5})
q = QuasiDistribution({'00': 0.5, '11':0.5})

ch = c.hex_outcomes()
qh = q.hex_probabilities()

print(type(c), type(ch), ch)
print(type(q), type(qh), qh)

output
<class 'qiskit.result.counts.Counts'> <class 'dict'> {'0x0': 0.5, '0x3': 0.5}
<class 'qiskit.result.distributions.quasi.QuasiDistribution'> <class 'dict'> {'0x0': 0.5, '0x3': 0.5}

I'm likely to miss your point. Could you give me an example code?

This isn't really relevant anymore with SamplerV2


add script for htlc && update sample files

Codecov Report

Merging #201 into release-rc2 will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@                Coverage Diff                @@
##             release-rc2     #201      +/-   ##
=================================================
- Coverage          28.78%   28.76%   -0.03%
  Complexity           329      329
=================================================
  Files                110      110
  Lines               3540     3543       +3
  Branches             249      251       +2
=================================================
  Hits                1019     1019
- Misses              2441     2444       +3
  Partials              80       80

Impacted Files
Coverage Δ
Complexity Δ

...om/webank/wecross/config/AccountManagerConfig.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...ain/java/com/webank/wecross/utils/ConfigUtils.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)


soda_logs used before being declared causes UnboundLocalError

          @AlessandroLollo The above issue has been fixed with this PR https://github.com/sodadata/prefect-soda-core/pull/21

But there is this new error
Encountered exception during execution:
Traceback (most recent call last):
  File "/usr/local/lib/python3.11/site-packages/prefect/engine.py", line 1730, in orchestrate_task_run
    result = await call.aresult()
             ^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/prefect/_internal/concurrency/calls.py", line 292, in aresult
    return await asyncio.wrap_future(self.future)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/prefect/_internal/concurrency/calls.py", line 346, in _run_async
    result = await coro
             ^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/prefect_soda_core/tasks.py", line 148, in soda_scan_execute
    return soda_logs
           ^^^^^^^^^
UnboundLocalError: cannot access local variable 'soda_logs' where it is not associated with a value

Finished in state Failed("Task run encountered an exception UnboundLocalError: cannot access local variable 'soda_logs' where it is not associated with a value")

Originally posted by @DVD-99 in https://github.com/sodadata/prefect-soda-core/issues/18#issuecomment-1686674965

@AlessandroLollo  Here is my code
'''
from prefect import flow, get_run_logger
from prefect_soda_core.soda_configuration import SodaConfiguration
from prefect_soda_core.sodacl_check import SodaCLCheck
from prefect_soda_core.tasks import soda_scan_execute
soda_configuration_block = SodaConfiguration(
configuration_yaml_path="/app/proto.yml"
)
soda_run_public = soda_scan_execute(
data_source_name="landing_public",
configuration=soda_configuration_block,
checks=SodaCLCheck(sodacl_yaml_path="/app/public_checks.yml"),
variables={},
verbose=True,
return_state=True
)
logger.info(soda_run_public)
if soda_run_public.is_failed():
pass
if name == "main":
# Deployment Setup
run_soda_scan()
'''
Make sure to set the atleast one check in public_checks.yml to failure.
Traceback
'''
File "/app/main.py", line 58, in
run_soda_scan()
File "/usr/local/lib/python3.8/site-packages/prefect/flows.py", line 560, in call
return enter_flow_run_engine_from_flow_call(
File "/usr/local/lib/python3.8/site-packages/prefect/engine.py", line 272, in enter_flow_run_engine_from_flow_call
retval = from_sync.wait_for_call_in_loop_thread(
File "/usr/local/lib/python3.8/site-packages/prefect/_internal/concurrency/api.py", line 243, in wait_for_call_in_loop_thread
return call.result()
File "/usr/local/lib/python3.8/site-packages/prefect/_internal/concurrency/calls.py", line 283, in result
return self.future.result(timeout=timeout)
File "/usr/local/lib/python3.8/site-packages/prefect/_internal/concurrency/calls.py", line 169, in result
return self.__get_result()
File "/usr/local/lib/python3.8/concurrent/futures/_base.py", line 389, in __get_result
raise self._exception
File "/usr/local/lib/python3.8/site-packages/prefect/_internal/concurrency/calls.py", line 346, in _run_async
result = await coro
File "/usr/local/lib/python3.8/site-packages/prefect/client/utilities.py", line 51, in with_injected_client
return await fn(*args, **kwargs)
File "/usr/local/lib/python3.8/site-packages/prefect/engine.py", line 375, in create_then_begin_flow_run
return await state.result(fetch=True)
File "/usr/local/lib/python3.8/site-packages/prefect/states.py", line 91, in _get_state_result
raise await get_state_exception(state)
File "/usr/local/lib/python3.8/site-packages/prefect/engine.py", line 1730, in orchestrate_task_run
result = await call.aresult()
File "/usr/local/lib/python3.8/site-packages/prefect/_internal/concurrency/calls.py", line 292, in aresult
return await asyncio.wrap_future(self.future)
File "/usr/local/lib/python3.8/site-packages/prefect/_internal/concurrency/calls.py", line 346, in _run_async
result = await coro
File "/usr/local/lib/python3.8/site-packages/prefect_soda_core/tasks.py", line 148, in soda_scan_execute
return soda_logs
UnboundLocalError: local variable 'soda_logs' referenced before assignment
'''

Hey @DVD-99 👋
What's the return_state parameter in soda_scan_execute?

Hey @fakhavan 👋
Would you be willing to provide some help on this issue?

Hey @DVD-99 👋
I should have fixed the issue you're facing in this PR
Before I merge the PR into main, can you give it a try?
Thanks!

It solved the issue for me @AlessandroLollo, thanks

Great @PaoloLeonard 🚀

Fixed in v0.1.7


What to expect?

For me, using any model all I get is hallucination.

Hello @ErfolgreichCharismatisch, we have just released version 5, which basically rewrites everything. There are also stepwise performance gains with v5.
If your issue persists, please reopen a new issue using paper-qa>=5. Also, you will need to include more specific details so we can reproduce


Makes the rules attribute return an empty list if it is not set

Issues:
Fixes #1170
Problem:
If there are no rules for a Virtual, the API will return no attribute.
This causes problems when wanting to loop through that.
Analysis:
So instead, the SDK will return just an empty list so that you dont need to catch
AttributeError exceptions
Tests:
functional

this is a breaking change so will need a rev of the major version of the SDK

as discussed this might break backwards compatibility, so it will be a warrant for a major release, merging


Praj/minus opscode expander

Opscode Expander was a Ruby service responsible for expanding node
objects into the schema expected by our search index.
This commit removes the service as we now do this expansion directly
inside Erchef. The new indexing pipeline allows us to remove the
operational complexity of RabbitMQ for our users. The new indexing
pipeline is not as battle-tested as expanded, but we hope that over
time it can reach the levels of reliability we had with Expander.
Expander was an excellent service. It performed well in our Hosted
Chef environment and for our many users. It had remarkably few bugs in
its history and is a good example of the skill of its authors.
https://buildkite.com/chef/chef-chef-server-master-omnibus-adhoc/builds/1653
https://buildkite.com/chef/chef-chef-server-master-integration-test/builds/354

We'll be sad to see this service go.  Many thanks to @danielsdeleo and its other authors who created such a reliable service.


Dec. 10 Review Ticket by Alara Ipek

Video: Hexademical and Letter Game Video
Issues that helped:
Issue for review by Andrew: https://github.com/tuckergol/binarylogicrepo/issues/2
Issue to update what work was still in order: https://github.com/tuckergol/binarylogicrepo/issues/3
Key commits:
Redone Hexadecimal and Binary Teacing Game: Key Commit #1
Comments to Letter Game: Key Commit #2
Matching the style to the rest of the games: Key Commit #3
Instructions for Letter Game: [Key Commit #4] (https://github.com/tuckergol/binarylogicrepo/commit/05223ab423b2e13fc1d5fbf234620d3435a3b741)
Insight to Github analytics of work done by team:
Analytics

[x] Individual should show that they were key contributor and example to team.  This includes their participation in ideas, plans, creating individual issues, providing code commits to project, crossover grading participation, being on task and positive example in the classroom.
[x] Video in CPT caption style , includes Web demo of key contribution to project, 1 minute
[x] Issue(s) that show plans/progress to team objectives
[x] Highlights of key commit(s) in Issues, summarizes code contributions
[x] Review GitHub analytics for key commits in each week during the project, shows consistent participation for 3 weeks
[x] Retrospective: If I had more time, I'd create a better more organized page for teaching hexadecimal and binary, and possibly making it mandatory to play before moving onto the other games. Therefore, I'd improve on UI design. Moreover, I'd like to implement a better score tracking system in the future, since we ran out of time to complete that.

Full code:
layout: default
permalink: /letter
title: Letter Game

Binary to Letter Game

Individual Review "Gurshawn" grading "Alara"
0.8/0.9
Individual(0.9)
Video in CPT caption style , includes Web demo of key contribution to project, 1 minute(0.7)
Issue(s) that show plans/progress to team objectives(0.8)
Highlights of key commit(s) in Issues, summarizes code contributions(0.8)
Review GitHub analytics(0.8)
Retrospective(0.8)
4.8/6 = 0.8
Good game that teaches the user what binary and hexadecimal values are. Explains the way someone can calculate the values based on a decimal value and allows the user to check if they are right or wrong. Could improve on the SASS(styling/appearance) of the page


Notification Settings is blank

Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

Describe the bug*
The notifications settings pane doesn't show any content

Steps to Reproduce

Click username
Click settings
Click Notifications (on the left)

Expected behavior
I assume there should be a set of notification settings?
Deployment Method

[X] Docker
[ ] Bare metal

Version Information
Version Information:
InvenTree-Version: 0.8.4
Django Version: 3.2.15
Commit Hash: ca1fbf9
Commit Date: 2022-10-20
Database: postgresql
Debug-Mode: False
Deployed using Docker: True
Relevant log output
No response

@mrkylegp did you recently update InvenTree from an older version?
https://docs.inventree.org/en/latest/faq/#feature-x-is-not-visible-after-update

@SchrodingersGat Yes I did update today. I tried the cache clear and also tried visiting my inventree instance from a computer which I have never browsed on before. Same issue.

@SchrodingersGat I do not have a plugins menu on the left like your image

I enabled plugins in my .env and rebooted the docker instance. Also enabled CoreNotificationsPlugin and enabled email notifications. There is now the email setting under the notifications menu.
Is CoreNotificationsPlugin the same as the Inventree Notifications plugin? If so, why is the label different?

Anyway, I enabled plugins in my .env and rebooted the docker instance. Also enabled CoreNotificationsPlugin and enabled email notifications. There is now the email setting under the notifications menu.

Ah, this makes sense. I should have asked if the instance had plugins enabled.

The documentation I saw didn't outline that I needed to enable plugins and enable an additional email plugin for that to work. Maybe I missed something?

This has been addressed in the upcoming 0.9.0 release.

It seems CoreNotificationsPlugin the same as the Inventree Notifications plugin. How come the labelling is different on mine?

The screenshot is from the demo server which is running latest code, not stable version.


Use single HashedWheelTimer to prevent thread exhaustion

HashedWheelTimer creates a new thread per instance so there should only be one of them

Thanks. Looks good to me.


Suggestion: Wiki on how to use this.

As title says.

There're some code examples and explanations on https://www.curseforge.com/minecraft/mc-mods/opticaljs , they should be able to serve as a temporary wiki. I will add some more documentations to cover more edge cases if time allowed.


Artifactory bundles a PostgreSQL driver, downloading from Maven.org in an issue

https://github.com/jfrog/JFrog-Cloud-Installers/blob/e0f14ca449060f192fecbd7d8400e76978942189/Ansible/ansible_collections/jfrog/platform/roles/artifactory/tasks/install.yml#L171
The tarball includes a driver. When installing in an airgapped site whitelisting 'maven.org' is like opening Pandora's box (for shops not using Java).

@bbaassssiiee is there any possible workaround  you can suggest ?

@bbaassssiiee FYI , artifactory binary internally bundles  PostgreSQL driver by default , you can disable this task if its a air gapped environment

It should be the other way around because the download is currently conditional on a value defined in the defaults. I.e. this is always true now:
  when:
    - postgres_driver_download_url is defined

A block with a Boolean download_postgres_driver would do.

yes , makes sense , I'll try to provide a solution in upcoming patch

@bbaassssiiee  you can set postgres_driver_download to false for airgap environments

Thanks!!


So I was wondering, if you, the people beside me using Benotes, have any thoughts on the subject in order for me to get (an)other perspective(s).

So I was wondering, if you, the people beside me using Benotes, have any thoughts on the subject in order for me to get (an)other perspective(s).
Originally posted by @fr0tt in https://github.com/fr0tt/benotes/discussions/20

Hi,
was there a reason why you opened this issue ?


Setting playback rate on ToneBufferSource

Hi, this is less a bug and more an observation. If you create a ToneBufferSource node, then set its playbackRate value, and then fire start(), there is an audible delay before the playbackRate gets updated, i.e. you can hear that the starting rate is 1 and then it shifts to whatever you set it to. It's most obvious when you pass currentTime to start(), less so if it's triggered via a transport scheduled event.
const player = new Tone.ToneBufferSource({ url: myBuffer });
player.playbackRate.value = 2;
player.start(Tone.context.currentTime);

Setting playbackRate in the constructor solves this, and indeed feels neater. But it throws you a bit if you're expecting it to behave like the native BufferSourceNode.

All values internally are scheduled with a "lookAhead" applied to Tone.context.currentTime. internal player.playbackRate.value = 2 is just doing player.playbackRate.setValueAtTime(2, Tone.now()). Since Tone.now() === Tone.context.currentTime + Tone.context.lookAhead, the value is going to be after the currentTime. Which is why changes after the source is started.
To fix this, you could use Tone.now() in start to schedule it at the same time, or you could schedule the value explicitly so that it's changed at the same time as the player is started by doing this: player.playbackRate.setValueAtTime(2, Tone.context.currentTime)
Let me know if that makes sense!

Ahh ok. Yes that makes sense. Thanks for explaining.


Mailing

Kivanc WIP.

We need a table to store different email types and user subscriptions to each of those.
Think about unsubscribe links (that do not require login).
Also think about emails that users can't unsubscribe from (terms update? security notice? etc). We can use a separate column for this information.
Also think about using this table to multiple ways of communications (for future use). SMS, Push, On Site notifications..
Obviously, add some logging and monitoring. For a quick solution, perhaps do send emails with errors.

new-assignment

An email to be sent after new assignments are sent out. If we end up implementing a $user->assign($repo), this can be triggered from there.
subject: "Your $month assignment is ready!"

new-assignee

Like new-assignment, but for repo owners.
subject: "Your repository/ies is/are assigned to someone!"
Consider sending "one" email if a repo owner received multiple assignees in a month, instead of sending 4 separate emails with same subject and body.

open-assignment-reminder

Open assignment reminder to be sent to all users who has an open assignment. Should be sent 7*24 hours before UTC 00:00 of next month.
Consider adding a link to /login?next=skip-confirm (which will require some session magic).
subject: "One week reminder: Your assignment on PullRequest.Club"
Needs a cronjob

assignment-done-contributor

Transactional email sent to contributor confirming they have completed their assignment.

assignment-skip-contributor

Transactional email sent to contributor confirming they have skipped their assignment.

assignment-done-maintainer

Email sent to repo maintainer letting them know one of their repositories have received a PR.

assignment-skip-maintainer

Email sent to repo maintainer letting them know one of their repositories have received a skip.

First things first: Let's get sendgrid connected, and two emails: new-assignment and new-assignee.


Step 6: Add clang cl pipeline for Windows

Commit Message: Step 6: Add clang cl pipeline for Windows
Additional Description:
Now that we have regenerated envoy build tools to update bazel and llvm, and picked up the llvm 11 tooling for windows on envoyproxy/master, finally introduce a clang-cl pipeline based on clang-cl
Co-authored-by: Sunjay Bhatia<EMAIL_ADDRESS>Risk Level: low (new facility)
Testing: local
Docs Changes: TBD
Release Notes: TBD
Platform Specific Features: Windows clang-cl compilation
Replaces https://github.com/envoyproxy/envoy/pull/14135
Fixes #11974

@lizan , @sunjayBhatia this looks ready for final review.

@lizan , @sunjayBhatia this looks ready for final review.

Once again, @sunjayBhatia and @lizan, I think all the concerns have been asked and answered, but let me know if I can add any more polish to this submission before it's merged. We will be iterating over .pdb output (just as on msvc-cl) and the build output (right now, a 33% larger binary than the equivalent MS tooling, speed delta TBD.)

Once again, @sunjayBhatia and @lizan, I think all the concerns have been asked and answered, but let me know if I can add any more polish to this submission before it's merged. We will be iterating over .pdb output (just as on msvc-cl) and the build output (right now, a 33% larger binary than the equivalent MS tooling, speed delta TBD.)

@htuch as this is your ask, and it's been reviewed and lizan's comments addressed, if you would like to move ahead please go ahead and merge this.

@htuch as this is your ask, and it's been reviewed and lizan's comments addressed, if you would like to move ahead please go ahead and merge this.

Amazing, thank you!

Amazing, thank you!


iOS notifications stop working

Home Assistant release with the issue:
0.80.2
Last working Home Assistant release (if known):
0.80.1
Operating environment (Hass.io/Docker/Windows/etc.):
Component/platform:
iOS notifications
Description of problem:
iOS notifications stop working after updating to 0.80.2
Traceback (if applicable):
2018-10-18 23:03:02 ERROR (SyncWorker_13) [homeassistant.components.notify.ios] Internal server error
2018-10-18 23:03:02 ERROR (SyncWorker_11) [homeassistant.components.notify.ios] Internal server error
2018-10-18 23:03:03 ERROR (SyncWorker_12) [homeassistant.components.notify.ios] Internal server error

Additional information:

Looks like an issue with the iOS push server. Happens on 0.80.1 as well.

Same Issue here.

Ditto. Running: 0.80.2

Dammit I wish i read this first lol. Spent hours trying to fix it thinking it was on my end. Same problem here no notifications for ios.

I just tried to send via the /dev-service page and its working so whatever wasn't working seems to be fixed as of 9:14pm PST
2018-10-18 21:12:57 INFO (SyncWorker_2) [homeassistant.components.notify.ios] iOS push notification rate limits for None: 3 sent, 150 allowed, 0 errors, resets in 19:47:02

Can confirm it's working for me now.


Special-purpose agents that are called as functions from other workers

We have found at least one case where one agent changes the behavior of how another runs before triggering it. Those options are currently stored as shared state between agents, but could better expressed as function arguments to cut down on the number of "global" variables visible by all agents (it makes me nervous and leads to large init functions).

[ ] Add a "queue_decorator" agent
[ ] Add an example that illustrates how to use it

After more working with Colmena, I don't think this one is needed. I've ended up using this "call another agent" pattern very often in a way that isn't better covered by other patterns.


Automated continuous deployment

Discussion from 2021-09-23 dev team meeting:

Jamie: Can we fully automate the release generation in a CI (continuous integration) pipeline?
Boris: A release could be triggered by a release tag. Worth experimenting on next version.
Boris: Hiccups with onto_tool
Rebecca: We didn't open the files to see what was in them
Jamie: Sanity check can be part of build or CI pipeline.
Boris: If we add CD (continuous deployment) that eliminates the gap between ontologists and developers.
We already have CI running on gist. For CD, creating a tag will automatically trigger CD.

Is anyone willing to take this on? @Jamie-SA and devs?

@Jamie-SA, @rjyounes @bpelakh
Is this issue still ongoing?

It does not appear that we have any continuous deployment happening right now.
A 'release' would require the SA server which is hosting our releases to expose a service where GitHub Workflows could push an artifact from a build. The redirect we have at w3id.org is already version-neutral.
Given the infrequent nature of our releases, I would be inclined to mark this as 'nice to have', or perhaps just close it.


fetch_tools: 0.1.1-0 in 'indigo/distribution.yaml' [bloom]

Increasing version of package(s) in repository fetch_tools to 0.1.1-0:

upstream repository: https://github.com/fetchrobotics/fetch_tools.git
release repository: https://github.com/fetchrobotics-gbp/fetch_tools-release.git
distro file: indigo/distribution.yaml
bloom version: 0.5.20
previous version for package: null

No source entry?

Oops. I'll add it next release.


Allow to use existing uv_loop_t instance

Applications might already have a libuv eventloop for its own business.  It would be nice to be able to schedule uWebSockets tasks on the existing instance of uv_loop_t.

Yep. There is a uv_default_loop() function that returns the default loop. This one is used by default. So Hub::getLoop() will return the same uv_default_loop() by dfault. Otherwise it shold be very easy to pass a loop to the constructor og Hub (with a tiny modification).


Thought on Finnhub API ?

Hi guys,
Just wondering if we can create a new Java wrapper for Finnhub stock API as well : https://finnhub.io/ . They provide 60 free calls/minute with intraday data going back for many years. Websocket streaming is also available for free. I think this is a way better alternative for alpha vantage.

Everyone needs to be aware that alpha332 is a shill account for an employee/owner of Finnhub - Spencer Sands
Finnhub has been shill spamming various forums including Reddit, Quora, Github, EliteTrader, Medium and other venues. Their posts have been moderated/deleted in many places.
They have been verified as blatantly copying/redistributing data without license from places include Investors Exchange, Toronto Stock Exchange and Australian Securities Exchange, and have been sent legal cease and desist notices.
They have created multiple shill accounts and even pretend to have user conversations. Their Github repositories are littered with unanswered/unsolved issues.
Other Github projects have refused to deal with this sort of nonsense.
Avoid this one!
Examples of more of their shilling/spam here:
https://github.com/public-apis/public-apis/issues/1137
https://github.com/piquette/finance-go/issues/14
https://github.com/iexg/IEX-API/issues/1410
https://www.reddit.com/r/algotrading/comments/in5u6x/beware_medium_guide_stock_market_apis_for_2020/


New method in DeferredManager that accepts iterable

New method in DeferredManager that accepts iterable

@aalmiray i'm moving this into 2.x. but, this will won't gain much from the type safety that was added recently.  However, I also noticed that we no longer are able to accept dm.when(promises[]) due to the signature change. Having a signature that takes in iterable might offer some compatibility for such a use case.

Moved to #121. PTAL.


Gain a basic understanding of docker, how it works, integration with Windows Server, and how to get it set up on Windows Server 2016.

How Docker for Windows works: https://docs.docker.com/docker-for-windows/
Docker for Windows Tutorial: https://blog.docker.com/2016/09/build-your-first-docker-windows-server-container/
Windows containers and Docker: https://www.simple-talk.com/cloud/platform-as-a-service/windows-containers-and-docker/
Seems like old Docker only ran on Linux, so Linux VM had to be installed on Windows machine, and then controlled through Windows. Now, Docker can run natively on Windows using native Hyper-V virtualization. Docker engine is what runs the containers, and the containers are what store individual applications for process separation.

At the time of this writing, the VM method is the sanctioned method for running Docker on Mac machines. I only mention this for the future benefit of people wanting to port Snap to Mac (if the Docker plugin doesn't already work on Mac).

In order to get Docker set up on Windows 2016 VM:
Start at https://docs.docker.com/docker-for-windows/#what-to-know-before-you-install -> Click on: Setup - Windows Server 2016 (Lab) -> Follow the instructions there (Install "Windows-native Docker Engine" instead of "Docker for Windows")
After this, I ran into error when trying to do 'docker version' command:
docker: error during connect: Post http://%2F%2F.%2Fpipe%2Fdocker_engine/v1.25/containers/create: open //./pipe/docker_engine: The system cannot find the file specified..
Apparently, the Docker Engine should start automatically after installation, but it doesn't (https://github.com/Microsoft/Virtualization-Documentation/issues/350). Use 'Start-Service docker' in Powershell to start it manually.

Continuing on with the Get Started with Windows for Docker, tried to run 'docker run hello-world', but got the error:
image operating system "linux" cannot be used on this platform.
According to https://docs.docker.com/docker-for-windows/troubleshoot/, if we install the native Windows binary (which allows us to develop and run Windows containers without Docker for Windows), we cannot develop or run Linux containers. That is why this error occurs. Something to look out for when first starting with Docker for Windows Server 2016.

From https://github.com/docker/labs/blob/master/windows/windows-containers/README.md, after setup, follow Getting Started with Windows Containers for specific instructions on how to use this native Docker binary.

Wait. Can Docker on Windows only run Windows containers?

That's what it seems like, at least for the Docker version made for Windows Server. At https://docs.docker.com/docker-for-windows/, it says that,
"The current version of Docker for Windows runs on 64bit Windows 10 Pro, Enterprise and Education (1511 November update, Build 10586 or later). In the future we will support more versions of Windows 10."
...which I guess means it won't run on Windows Server 2016? But then if you go to "https://docs.docker.com/docker-for-windows/#about-windows-containers-and-windows-server-2016 -> Click on: Switch between Windows and Linux containers", it seems like we should be able to switch between the two (with Docker for Windows v. 1.13.x+).

I'll also note here that following the instructions at https://github.com/docker/labs/blob/master/windows/windows-containers/Setup-Server2016.md to download the Docker Engine for Windows Server 2016 gave me version 1.12.2 of Docker, so not sure if that's making it so I can only do Windows containers or if that's just how it's supposed to be on the Windows Server. I'll ask Taylor about it.


v-model is extremely slow in big components

Version
2.6.10
Reproduction link
https://codepen.io/mikilll94/pen/RwbYrpz
Steps to reproduce
Just run the code sample and type something in the input. As you can see there is a clear lag when you are typing.
What is expected?
Everything should run smoothly.
What is actually happening?
There is a clear lag when you are typing something in the input.

It needs to re render the whole thing when you type. You can avoid this by wrapping either in a component: the input or the big list of divs

@posva Hello , I'm facing a similar issue - though much got solved when using lazy, but still facing it on custom components . Are there any relevant docs/articles/patterns you'd recommend to follow when creating data heavy components?
Thanks

Facing same issue for a chat app :(
Anyone could solve?

@hkmsadek hey , components is the way to go

It needs to re render the whole thing when you type. You can avoid this by wrapping either in a component: the input or the big list of divs

Could you give sample of code? I am new to it, please 🙏🏻


add support for structs

Looks like the generator does not support struct (and, maybe, record struct):
    public readonly partial struct MyStruct
    {
        [Zomp.SyncMethodGenerator.CreateSyncVersion]
        public async Task MethodAsync() => throw new NotImplementedException();
    }

    public partial class MyClass
    {
        [Zomp.SyncMethodGenerator.CreateSyncVersion]
        public async Task MethodAsync() => throw new NotImplementedException();
    }

there is no sync sibling for MyStruct.

Fixed in 1.4.1


Azure functions v3 project fails to run after running convert-from-nuget

Description
I'm not sure that this is the right place to report this issue, but it seems to be Paket related.
Basic azure functions project generated from template fails to run after it is converted to Paket
Repro steps
I have created a repository with a sample project
https://github.com/landy/PaketAzureFunctions
run func host start from Company.FunctionApp directory
I have Azure Functions Core Tools 3.0.1740 installed (https://github.com/Azure/azure-functions-core-tools)
Expected behavior
Application should run normally and handle http request
Actual behavior
application starts but emits an error:
Microsoft.Azure.WebJobs.Host: Error indexing method 'TestHttpTrigger'. Microsoft.Azure.WebJobs.Host: Cannot bind parameter 'log' to type ILogger. Make sure the parameter Type is supported by the binding. If you're using binding extensions (e.g. Azure Storage, ServiceBus, Timers, etc.) make sure you've called the registration method for the extension(s) in your startup code (e.g. builder.AddAzureStorage(), builder.AddServiceBus(), builder.AddTimers(), etc.). [12/06/2019 13:57:52] Error indexing method 'TestHttpTrigger' [12/06/2019 13:57:52] Microsoft.Azure.WebJobs.Host: Error indexing method 'TestHttpTrigger'. Microsoft.Azure.WebJobs.Host: Cannot bind parameter 'log' to type ILogger. Make sure the parameter Type is supported by the binding. If you're using binding extensions (e.g. Azure Storage, ServiceBus, Timers, etc.) make sure you've called the registration method for the extension(s) in your startup code (e.g. builder.AddAzureStorage(), builder.AddServiceBus(), builder.AddTimers(), etc.).
Known workarounds
None

Azure functions doesn't work with 3.1 yet. This is a typical AzureFunctions host issue.
You need to downgrade all libs to 3.0 in order to run the ~3 runtime on functions or even to 2.2 if you target~2 runtime.
As a future way to mitigate we proposed: https://github.com/fsprojects/Paket/issues/3744


Set downloaded torrent on pause without deleting from the list of the torrents

Is there way to set the torrent on pause by Id and resume it again?
Other clients allow to set torrent on pause for  completed  and for uncompleted torrents.

Not at the moment. You will have to remove the .torrent from the watched directory, but that also removes its entry. Support for pausing torrents will require some work. Essentially you have to be able to stop most of the supervision tree in and around a torrent to make it pause.