useOidcAccessToken always gets ACCESS_TOKEN_SECURED_BY_OIDC_SERVICE_WORKER_default

Issue and Steps to Reproduce
OidcTrustedDomains.js set as README:
const trustedDomains = {
  default:["https://demo.duendesoftware.com", "https://www.myapi.com/users"]
//  "https://demo.duendesoftware.com/connect/token", "https://demo.duendesoftware.com/connect/revocation", "https://demo.duendesoftware.com/connect/userinfo"
};

// Service worker will continue to give access token to the JavaScript client
// Ideal to hide refresh token from client JavaScript, but to retrieve access_token for some
// scenarios which require it. For example, to send it via websocket connection.
trustedDomains.config_show_access_token = { domains : ["https://demo.duendesoftware.com"], showAccessToken: true };

Versions
16.9.0
Screenshots
Expected
Actual
Additional Details

Installed packages:

After going through the source codes, I find out there are two places in the README that are misleading:

default config has always set showAccessToken=false. So to showAccessToken, we need to use a non-default one. See https://github.com/AxaFrance/react-oidc/blob/e3d208f86eba26765e064db675fed2cc4f15c846/packages/react/service_worker/OidcServiceWorker.ts#L48
That's why in README we have: trustedDomains.config_show_access_token . But the README doesn't mention that we need to use "config_show_access_token" as the configName from then on.

Solution:
OidcTrustedDomains.js
const trustedDomains = {
  config_show_access_token = { domains : ["https://demo.duendesoftware.com"], showAccessToken: true }
};

Then we should pass the configName in index.tsx:
        <OidcProvider
          configuration={configuration}
          configurationName='config_show_access_token'
        >
          <App />
        </OidcProvider>

And everywhere we are using it:
const oidcConfigName = 'config_show_access_token'
export function useAccessToken() {
  const token = useOidcAccessToken(oidcConfigName)
  return token
}
export function useOidc2() {
  const oidc = useOidc(oidcConfigName);
  return oidc
}
export function useOidcUser2() {
  const user = useOidcUser(oidcConfigName);
  return user
}

hi @silencej, thank you for your feedback.
I have to enhance the documentation.
The demo have a full sample of many configurations. I use it for the documentation:
https://black-rock-0dc6b0d03.1.azurestaticapps.net/multi-auth

Is the documentation already enhanced? I have also problems to get the token visible.

Even with his solution my auth seems to break.
OidcServiceWorker.ts:291 Uncaught TypeError: Cannot read properties of undefined (reading 'showAccessToken')
at $ (OidcServiceWorker.ts:291:111)
Maybe i should send the token to my backend and return it as a string, till there is a official solution for that.


ng serve --aot recompiles when source file is opened

🐞 Bug report
Command (mark with an x)
- [ ] new
- [ ] build
- [x] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Is this a regression?
Yes, no problem with Angular 7
Description
When I run ng serve --aot the application is recompiled everytime  I open a source file in an editor like notepad or vs code for the first time. A second opening of the same source file does not result in a recompile!
🔬 Minimal Reproduction
1.) ng new myApp
2.) ng serve --aot
3.) Open a HTML or TS file of the application
==> ng recompiles the app
ng serve without option --aot works as expected
🌍 Your Environment

Angular CLI: 8.3.5
Node: 10.16.3
OS: win32 x64
Angular:
...

Package                      Version
------------------------------------------------------
@angular-devkit/architect    0.803.5
@angular-devkit/core         8.3.5
@angular-devkit/schematics   8.3.5
@schematics/angular          8.3.5
@schematics/update           0.803.5
rxjs                         6.4.0

The CLI watches for file changes on disk by asking the OS when something changes.  If your IDE or OS (virus scanners on windows, for instance) are doing anything with that file on load then it could trigger a recompile.  I also cannot replicated this using VSCode or vi on MacOS.

Thanks @clydin, I have the issue since I have migrated my application from Angular 7 to Angular 8 a week ago. Therefore, I guess it has something to do with Angular 8.

I also tried this locally, and I was unable to replicate the issue.

Is this still reproducible?

Still reproducible with current version :-(
Angular CLI: 8.3.20
Node: 12.13.0
OS: win32 x64
Angular: 8.2.14
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.803.20
@angular-devkit/build-angular     0.803.20
@angular-devkit/build-optimizer   0.803.20
@angular-devkit/build-webpack     0.803.20
@angular-devkit/core              8.3.20
@angular-devkit/schematics        8.3.20
@angular/cdk                      8.2.3
@angular/cli                      8.3.20
@angular/flex-layout              8.0.0-beta.27
@angular/material                 8.2.3
@ngtools/webpack                  8.3.20
@schematics/angular               8.3.20
@schematics/update                0.803.20
rxjs                              6.5.3
typescript                        3.5.3
webpack                           4.39.2


feat: deprecate php 7

This PR

deprecates PHP 7.4
implements full parity with php-sdk with integration test suite

Related Issues

Notes

Follow-up Tasks

How to test

Squashed and signed off all commits

Will require some more work on other packages before I can merge, but the core lib is updated now


Sort columns

When a visitor clicks on the headers:

Author: it sort notebooks by github_owner_login
Repo: it sort notebooks by github_repo_name
Filename: it sorts notebooks by github_filename

For example, we could use a URL parameter: ...?filter=author

Hey @hectorperez
Can i try this one?

Hi @FabriDamazio!
I'm just considering replacing the controller with LiveView.
If we do so, maybe it's worth to wait for this issue.
What do you think? Would you like us to use LiveView for notes.club?

Hi @FabriDamazio!
I'm just considering replacing the controller with LiveView. If we do so, maybe it's worth to wait for this issue.
What do you think? Would you like us to use LiveView for notes.club?

For me the application is pretty decent without LiveView but i think we can move to it.
I can try the replacing and then implement sorting if its ok to you

That's would be awesome, thanks!
I created an issue for LiveView: https://github.com/notesclub/notesclub/issues/59
Please leave a message there so I can assign you that issue too.

@FabriDamazio! That's would be awesome, thanks!
I created an issue for LiveView: https://github.com/notesclub/notesclub/issues/59
Please leave a message there so I can assign you that issue too.

We don't have columns any more.


fix: application name overflowing

Description

Set a max width for the application name
Make the value a constant as it is used in quite a few places

Fixes #8884
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Manually
Checklist:

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

Test coverage results :test_tube:
:green_circle: Total coverage has increased
// Code coverage diff between base branch:release and head branch: fix/application-name

Status
File
% Stmts
% Branch
% Funcs
% Lines

:green_circle:
total
54.83 (0.01)
36.55 (0.02)
33.46 (-0.02)
55.36 (0.01)

:green_circle:
app/client/src/components/ads/EditableText.tsx
79.41 (0.62)
31.25 (0)
75 (0)
74.07 (0.99)

:green_circle:
app/client/src/components/ads/EditableTextSubComponent.tsx
68.04 (0.33)
44.32 (0)
78.13 (0)
66.67 (0.38)

:green_circle:
app/client/src/components/propertyControls/MenuItemsControl.tsx
22.83 (-0.16)
26.32 (4.1)
7.69 (-1.4)
24.69 (-0.31)

:green_circle:
app/client/src/components/propertyControls/PrimaryColumnsControl.tsx
20.72 (0.35)
12.5 (2.82)
7.69 (-0.64)
22 (0.35)

:green_circle:
app/client/src/components/propertyControls/StyledControls.tsx
67.14 (1.39)
75 (-0.76)
43.9 (2.04)
67.14 (1.39)

:red_circle:
app/client/src/components/propertyControls/TabControl.tsx
40.4 (-1.09)
40 (4.29)
22.22 (-3.87)
42.7 (-1.35)

:green_circle:
app/client/src/reducers/uiReducers/applicationsReducer.tsx
10.47 (0.79)
0 (0)
6.98 (0.31)
10.84 (0.84)

:green_circle:
app/client/src/sagas/userSagas.tsx
16.44 (0.14)
1.45 (0)
5.26 (0.26)
19.68 (0.21)

:green_circle:
app/client/src/selectors/commentsSelectors.ts
85.25 (1.64)
64.71 (2.95)
73.33 (0)
90.59 (2.35)

/ok-to-test sha=de864f8

/ok-to-test sha=a520bd4

/ok-to-test sha=0cb9950

/ok-to-test sha=dfd013b


Remove Simulator Application from FBSimulatorControlConfiguration

This is effectively a Global requirement since it is derived from the xcode-select path. For this reason it makes more sense to be a global and not bloat the Configuration object.

lgtm


docs: Add custom prometheus metrics guide

As the example shows, it's not straight-forward how to use the middleware for serving custom metrics (along with the default ones). I haven't found any guide initially for doing such work. There is also an issue in the middleware's repository  asking for guidance about this problem.
It took me a while to figure everything out while digging around in the labstack/echo-contrib repo's code. With this addition to the documentation, I expect others will have an easier way to onboard this middleware to their apps.

thanks!


"No viable alternative at token end of file" doesn't say which file it happened in

compile-source:
[ceylon-compile] error: incorrect syntax: no viable alternative at token end of file
[ceylon-compile] ceylon compile: There was 1 error

Example for reproduce (\n just after {):
----------- begin of file--------------------
void f() {

----------- end of file--------------------

The cause is that position of EOF token could not be determined in LanguageCompiler.printError, line 734, so Position.NOPOS is used, which leads to use of BasicFormatKind.DEFAULT_NO_POS_FORMAT in BasicDiagnosticFormatter.formatDiagnostic (line 91).
I have a solution: https://github.com/MichaelNedzelsky/ceylon/commit/7c6e7fbff53261d1ee0098c4ee28080efa15e28b and a test https://github.com/MichaelNedzelsky/ceylon/commit/b5d2487514ae3b6382cd23b43bb717a122667b38
This solution is not very elegant, but it does the job.


update pr: for #771

What does this PR do?
This is an update PR as per the comments here:

Updates README
Adds test cases

Credits: original work done by @asnimansari

Thanks :smile:


[TimePicker] Add a Fade-in component for hours-minute change transition

[ ] PR has tests / docs demo, and is linted.
[ ] Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
[ ] Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

Added a Fade-In Component which internally uses ReactTransitionGroup Component to add a fade-in transition effect to the hours/ minute transition.

Closes https://github.com/callemall/material-ui/issues/1943

I haven't looked at the component to see if it's possible, but ideally, if we're going to fade, it should cross-fade between the clocks rather than the outgoing clocks disappearing before the new fades in.

@tintin1343 the transition component can be made to be simpler -- some of the complexity is left in there from the one you based it on (ExpandTransition) and is because of the difficulties in animating height.

@mbrookes I'm wondering if we should have a single transition component that has a couple of in/out options: fade, grow, etc.

@nathanmarks - it does seem there's quite a bit of duplicate code that could be rationalised.

@mbrookes although - the transition group child for expanding need to work slightly differently because of how we need to calculate/set the height in the transition lifecycle in separate ticks. Some other transitions (fade) are a lot simpler.

@nathanmarks @mbrookes : what do you guys suggest? I agree expand and slide transition works a bit differently. Hence I decided to keep it as a separate component.
@nathanmarks

some of the complexity is left in there from the one you based it on (ExpandTransition) and is because of the difficulties in animating height.

What complexities are you referring to?
@mbrookes : I can adjust the fading. Let me try and change that for you.

@tintin1343 there's some things that I have already refactored and improved a little in expandtransition (not yet merged) like the use of enterDelay.
Right now this transition component is a little messy in terms of how durations/delays are managed and can be improved for the purpose of fading components in/out. The same thing actually applies to ScaleIn and SlideIn, fortunately those APIs aren't public as they need a bit of work!!
Don't worry about that too much right now -- let's just try to get the UX right as @mbrookes suggested with the crossfade.

@nathanmarks : Alright. I will fix the effects for now.

@tintin1343 This is closer, however I think the clock hands still need some work. At the moment, when you switch from hours to minutes or vice versa, the alternate hand appears immediately, so you have two fully opacity hands at the same time, then the departing one fades out.
Instead, the departing hand should fade out as the entering hand fades in.

@mbrookes @tintin1343 where we at with this?

Not sure - I've pinged @tintin1343 in chat a couple of times, but no answer.

@mbrookes: I am right here. Just out of action for a couple weeks. I will get back to PRs and discussions very soon.
Don't close this. I will fix it.

@tintin1343 Any update? I'm wondering if it's worth continuing as the TimePicker might need a full re-write for the next branch.

I'm wondering if it's worth continuing as the TimePicker might need a full re-write for the next branch.

@tintin1343 Thanks for working on it! I'm closing this PR as I think it would be better to focus on the next branch for improvements like this one.


Add License

Hello, please add a license to your project, so people can decide under which conditions they can use your project and build on your code.

Done!


Use more meaningful ID for a site

Sites typically have IDs such as FoobarUS. The tool creates a site ID of 1 just now. Because this is also in the url of the site, it is a little off from reality.

You aren't going for random site names, aren't you? Would using the name for the ID work, like Site1?

I expected that the site name is used. Only make sure it is properly formatted aka most characters are not permitted. So maybe site name and id can be given in the config.


Enhancement: Add PHP 7.3 to Travis CI build matrix

This PR

[x] adds PHP 7.3 to the Travis CI build matrix

The Travis CI build is broken due to the php-cs-fixer dependencies problems.
Consider following error message:
friendsofphp/php-cs-fixer v2.13.0 requires php ^5.6 || >=7.0 <7.3

It seems that this php-cs-fixer version requires php < 7.3 and I think this PR should wait for this dependency until it supports the php-7.3.

@peter279k
Yes, I'll wait until then.


Correction for raw affiliation School of Community Health Portland State University

Correction needed for raw affiliation School of Community Health Portland State University
raw_affiliation_name: School of Community Health Portland State University
new_rors: 00yn2fy02
previous_rors:
works_examples: W1601052560
contact: 99beb50c2499850c00949e61f27ee6aa:ab2ef00f @ ourresearch.org

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


[WIP] scaffold service

WHY
WHAT
REF

Codecov Report

Merging #35 into master will increase coverage by 6.3%.
The diff coverage is 82.71%.

@@            Coverage Diff            @@
##           master      #35     +/-   ##
=========================================
+ Coverage   29.05%   35.36%   +6.3%
=========================================
  Files          35       35
  Lines        1053     1196    +143
=========================================
+ Hits          306      423    +117
- Misses        716      742     +26
  Partials       31       31

Impacted Files
Coverage Δ

pkg/grapicmd/cmd/generate.go
0% <0%> (ø)
:arrow_up:

pkg/grapicmd/internal/module/generator/service.go
83.33% <83.75%> (-4.43%)
:arrow_down:

Continue to review full report at Codecov.

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


_init function not invoke correctly

Godot version
4.0dev.calinou[9b8095ad0]
System information
Window 10
Issue description
Code like this:
class ValBase:
    func _init(val):
        prints("valbase init",val)

class ValA extends ValBase:
    var nn

class ValB extends ValBase:
    var nn

func _ready():
    var a = ValA.new("a")  # no print
    var b = ValB.new("b") # no print

I think it should print something(should invoke ValBase._init), but it not
sorry i can't upload a reproduction project, because my computer no network and i report this use my phone.
Steps to reproduce

create a project
create a scene and copy this code
run

Minimal reproduction project
No response

Duplicate of #42506
As a workaround you can do
class ValA extends ValBase:
	func _init(val):
		super._init(val)


⚠ Multimedia BRN has degraded performance

In a03ea7f, Multimedia BRN (https://mbrn.in/ping.html) experienced degraded performance:

HTTP code: 200
Response time: 28469 ms

Resolved: Multimedia BRN performance has improved in 519c854.


Kaw 7947 specification

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #KAW-7947
Test URLs:

Before: https://redesign-develop--edge-delivery-bimota--netcentric.hlx.page/drafts/tdziezyk/v2/specification
After: https://KAW-7947-specification--edge-delivery-bimota--netcentric.hlx.page/drafts/tdziezyk/v2/specification

The table int he test page is different form the design referring to the grey part and alignment. Was there an update?

 The table int he test page is different form the design referring to the grey part and alignment. Was there an update?

It looks like something is not loaded. I'll need to check it

There is more space compared to design between tables

Fixed

Padding between button and text for greater than > XS is 60px.Can you handle that

Fixed


Add social logins

Telerik Backend Services supports social logins so we should add the ability to sign in via Facebook and Twitter.

Closing in favor of #43.


How to create instance methods in a metaclass?

In regular Python3, these two classes work the same way:
class Conventional:

    def instance_method(self, arg):
        print (arg)

class MetaExample(type):

   def __new__(meta, name, bases, attributes):
        attributes['instance_method'] = lambda slf, b: print (b)
        return type.__new__(meta, name, bases, attributes)

class WithMeta(metaclass=MetaExample):

    pass

c = Conventional()
c.instance_method('con')

w = WithMeta()
w.instance_method('meta')

In the transcrypted version, however the metaclass created instance function isn't bound, so it doesn't get a self reference.  The first one prints "con" and "meta", but the second prints "con" and None because of the argument loss.
Is there a trick for adding instance methods via a metaclass?

I think this works:
class Conventional:

    def instance_method(self, arg):
        print (arg)

class MetaExample(type):

    def __new__(meta, name, bases, attributes):

        def inst(b):
            print(b)

        attributes['instance_method'] = inst

        result =  type.__new__(meta, name, bases, attributes)
        inst.bind(result)
        return result

class WithMeta(metaclass=MetaExample):

    pass

c = Conventional()
c.instance_method('con')

w = WithMeta()
w.instance_method('meta')

It's a little un-pythonic because the instance method has to reference this instead of self, but it appears to work


enableDraw cause TypeError

I installed the latest leaflet-geoman version by npm. When map.pm.enableDraw('Line'), It raised typeerror while the mouse over the map.
leaflet version: 1.6
error in Firefox:
TypeError: r is undefinedleaflet-geoman.min.js:1
    _calcLayerDistances leaflet-geoman.min.js:1
    _calcClosestLayer leaflet-geoman.min.js:1
    forEach self-hosted:235
    _calcClosestLayer leaflet-geoman.min.js:1
    _handleSnapping leaflet-geoman.min.js:1
    _syncHintMarker leaflet-geoman.min.js:1
    fire leaflet-src.js:593
    _fireDOMEvent leaflet-src.js:4452
    _handleDOMEvent leaflet-src.js:4409
    handler leaflet-src.js:2679

error in Chrome:
Uncaught TypeError: Cannot read property '0' of undefined
    at NewClass._calcLayerDistances (leaflet-geoman.min.js:1)
    at leaflet-geoman.min.js:1
    at Array.forEach (<anonymous>)
    at NewClass._calcClosestLayer (leaflet-geoman.min.js:1)
    at NewClass._handleSnapping (leaflet-geoman.min.js:1)
    at NewClass._syncHintMarker (leaflet-geoman.min.js:1)
    at NewClass.fire (leaflet-src.js:593)
    at NewClass._fireDOMEvent (leaflet-src.js:4452)
    at NewClass._handleDOMEvent (leaflet-src.js:4409)
    at HTMLDivElement.handler (leaflet-src.js:2679)

I can't reproduce the error. Can you please create a JSfiddle, showing the problem?

I have the same problem - as soon as i click on "draw polygon" button and hover on map i get bunch of errors of the same content: "Cannot read property '0' of undefined
at NewClass._calcLayerDistances (leaflet-geoman.min.js:1)"
Do I need initialize anything before using "toolbox"?

I do not have working JSfiddle, but if it changes something for you i cut some useless code for this problem and paste rest to JSfiddle. If you would like, please take a look: https://jsfiddle.net/hpbnruc2/4/

I'm just thinking... Is this error can appears because of using OpenStreetMaps instead of MapBox?

I wasn't able to check the error yet but I'm pretty sure your Tilelayer should not have an influence on Leaflet-Geoman. But I will check your error once I find the time - thanks for the reproduction link!

Ok, thank you for reply - Waiting for solution ;)

I'm having the same problem, holding alt seems to resolve it since it's an issue in the snapping mixin. The provided variable "closestSegment" is undefined which makes the [0] give a property/index error.
Lines in code where the issue lies:
https://github.com/geoman-io/leaflet-geoman/blob/41810b57bec8e60f66b3191e68f084555e55b804/src/js/Mixins/Snapping.js#L368-L373

Found out the issue, it was due to a polygon not having any coordinates attached to it a simple if around the L.polygon() function to check that the coordinates actually exist and that the array has a length makes everything work flawlessly.

@selenecodes it would be very helpful when you create a demo page or a gif, so we can see how to reproduce this error

Unfortunately I can't add a gif of it due to it being a customer's internal website.
The problem however isn't in this repo, it occurs when you add a feature to the map that doesn't have coordinates, causing the closestSegment in your code to break due to there not being any coordinates to read from.
The fix people should apply to their code
coordinates = []
if (coordinates.length) { // <- add this if statement to make sure that the polygon only gets created if there are actually coordinates to draw.
    new L.polygon(coordinates)
}


fix NSLeftTextAlignment in 10.13 is Deprecated

NSLeftTextAlignment in 10.13 is Deprecated,replace as NSTextAlignmentLeft

This issue has been fixed.


Change form

[FormMixin] Field components are always controled
Description
Field are always controlled. Like that, when we have a specific onChange for a field. We haven't to set an onChange for all field

This should fix #1347.

This PR need to be completed. Indeed when we cancel an edition. the field was undefined before it, will keep the value set during edition instead of become undefined.
Probrably use the definitionPath to get the entity and set all value undefined at null when getStateFromStore fix this issue.


New Features?

Is there anything anyone would like to see added to the application?

Closing as people are not interested


dropwizardifying of isochrone

Main work to get #1237 merged

Sorry, I'll probably update the target isochrone to master first so we have only the necessary changes here.

Ok, forget what I said. Somehow the files view lists changes against master not against isochrone_open_sourcen, see here for a better comparison @michaz https://github.com/graphhopper/graphhopper/compare/isochrone_dw


how to output cam map in infer_val.py

Hi,
Thanks for sharing your  work.
But, I have a problem. How to output and visualize cam map of each class in the infer_val.py.
Hope your reply.
Thanks.

Hi @liz6688,
CAMs are only produced for the baseline models, because our method does not have CAMs. It predicts the semantic maps directly. This is done automatically by infer_val.py (it saves the visualisation in the directory your specify).
Hope this answers your question.


Reconnecting won't work

Hello,
I use a looped os_timer_t to do my looped stuff such as reconnecting if .connect() returns false. I tried a lot of different things, the current code is
  if (!ESP_SmartHomeDevice::mqttClient.connected()) {
    if (millis() - ESP_SmartHomeDevice::lastMillis > TRY_RECONNECT_AFTER_MILLISECOND) {
      ESP_SmartHomeDevice::mqttClient.disconnect();
      if (ESP_SmartHomeDevice::mqttClient.connect(ESP_SmartHomeDevice::name)) {
        Serial.println("Now connected to mqtt server.");
      } else {
        Serial.println("Could not connect to mqtt server.");
      }
      ESP_SmartHomeDevice::lastMillis = millis();
      return;
    } else {
      return;
    }
  } else {
    ESP_SmartHomeDevice::mqttClient.loop();
  }

It always prints "Could not connect to mqtt server".
This function is static since its the parent class providing mqtt functionality for every child object.
Any idea why it won't work?
Complete project can be found here: https://github.com/nixmeer/ESP_SmartHomeDevice

What is the MQTT Server?
I have an Arduino MKR WiFi 1000/1010 talking to a Raspberry Pi with Mosquitto MQTT Server. I found that a 5-second delay between resets works well.


Windows Error on repo init

[new tag]         v0.4.9               -> v0.4.9
Traceback (most recent call last):
File "C:\Users\dpc\Desktop\winRepo.repo\repo/main.py", line 538, in
_Main(sys.argv[1:])
File "C:\Users\dpc\Desktop\winRepo.repo\repo/main.py", line 512, in _Main
result = repo._Run(argv) or 0
File "C:\Users\dpc\Desktop\winRepo.repo\repo/main.py", line 181, in _Run
portable.NoPager(cmd)
File "C:\Users\dpc\Desktop\winRepo.repo\repo\portable.py", line 277, in NoPag
er
RunWindowsShell(cmd)
File "C:\Users\dpc\Desktop\winRepo.repo\repo\portable.py", line 281, in RunWi
ndowsShell
redirect_all(executable)
File "C:\Users\dpc\Desktop\winRepo.repo\repo\portable.py", line 288, in redir
ect_all
p = subprocess.Popen([executable], stdin=subprocess.PIPE, stdout=old_sysout,
stderr=old_syserr)
File "C:\Python27\lib\subprocess.py", line 394, in init
errread, errwrite)
File "C:\Python27\lib\subprocess.py", line 644, in _execute_child
startupinfo)
WindowsError: [Error 2]

@AttITuder, have you seen discussion at #53, that you need to have cat and less available in PATH?

I have set the environment variables to follow the steps

@AttITuder, have you seen discussion at #53, that you need to have cat and less available in PATH?

I have set the environment variables to follow the steps


Fix: Discharge patient showing live

Proposed Changes

Fixes #9289
Change 1
Change 2
More?

@ohcnetwork/care-fe-code-reviewers
Screenshot

Merge Checklist

[x] Ensure that UI text is kept in I18n files.
[x] Prep screenshot or demo video for changelog entry, and attach it to issue.
[x] Request for Peer Reviews

Summary by CodeRabbit

New Features

Introduced a new section in the ConsultationCard to display patient status with a visual indicator.

Bug Fixes

Enhanced layout organization in the ConsultationCard for improved clarity.

Refactor

Removed the patient status display from the Demography component, adjusting the layout accordingly while maintaining core functionalities.

LGTM


Question about the scene query generation in BinsFormer method?

Hi,
Thank you for your excellent work.
I have two question.

How to generate the scene classification label?
How to generate the scene query?

Thanks a lot for your attention to our work. Here are my replies to your answers.

Generally, depth estimation datasets are organized by scene names. So for NYU depth, I classify scenes into about 19 classes, including bedroom, classroom, and so on.
That's easy to access, thanks to the extensibility of the attention mechanism. We add an extra query to the bins queries. So, as presented in the figures of our paper, the query set consists of a scene query and N bins queries.

Thank you for your reply!

Don't we need scene query and scene classification loss in the KITTI dataset?
Why are there no background Bin in the Binsformer method? There is a ground Bin in the CaDDN method.

I think we do not need and I did not implement scene query and cls loss for the KITTI dataset. But if there are reasonable classifications for such outdoor street datasets, it can be useful to add the classification part.

That's an interesting question. I list my understanding below.

First, let's see the depth estimation. Remember our goal is to predict pixel-wise depth, it is not necessary to divide pixels into fore- and background classes. (While there are several works indicating that a separated manner can benefit the depth estimation, it is hard to get accurate and fine-grained labels for this goal especially when we adopt LIDAR gt).
Now, let's consider Mono3D, more specifically, the CaDDN. The depth estimation is used to lift up the image features into space frustum (i.e., voxel). If we use regressithe on-based method here, the result features are too discriminative, leading to error cascade and performance degradation (proofed in their ablations). When adopting cls.-based method, the features are more smooth (imagining that the probability cannot be one-hot). And, here, is the point. Consider a pixel belonging to the background, and if we use the background bins, the lifted features are near zero (remember the dot operation). That's similar to the voxel-based point cloud 3D detectors (empty voxels). But if there is no background bin, the probability distribution can be close to the max depth, introducing futile noises and decreasing the detection performance (especially severe for distant objects).

Close for now.


#stash-pane should not be fixed, or the browser window will not have scrollbar to show full file list

#stash-pane should not be fixed, or the browser window will not have scrollbar to show full file list

#stash-pane should not be fixed, or the browser window will not have scrollbar to show full file list，and  #stash-pane #controls should set as follow，through the controls in this page may not work.
#stash-pane {
top: 0;
width: 60%;
height: 100%;
font-size: 16px;
}
#stash-pane #controls {
color: #2e2e2e;
right: 0;
top: 0;
z-index: 1;
margin: 5px 5px 0 0;
}
at the sametime，editpane controls overlaped by right presentation preview，should set as follow：
#edit-pane #controls {
color: #2e2e2e;
position: absolute;
right: 30px;
top: 0;
z-index: 1;
margin: 5px 5px 0 0;
}


Should running tests with coverage fail if coveralls command fails?

I have run into issues today where my tests pass, but coveralls is down. Should this actually fail the build or is there a better way of handling an issue such as this?

That's an interesting question.
I'm not sure of the answer.  My general tendency is to favor failing if anything didn't go as expected, since otherwise you'll probably not notice, and then be surprised later.
You can always re-try the build later when coveralls comes back, right?  Maybe this could be a cli option or something?

Sure, retrying is no problem and I guess every developer will have his or her own preference on it? Was just curious. I may take a look into adding an option in the future, but see no reason for this to stay open. Thanks for your response!


InvalidDatabaseType: Exception thrown on connect() to secret_id / arn & friendly name

Describe the bug
A clear and concise description of what the bug is.
wr.postgrest.connect() fails to connect to the database, the error indicates it must be a PostgreSQL connection.
To Reproduce
Steps to reproduce the behavior. Also, add details about Python and Wrangler's version and how the library was installed.
con = wr.postgresql.connect(secret_id="arn:aws:secretsmanager:us-east-1:****************:secret:db/rds/postgres/awswrangler-OFWDkE")
with con.cursor() as cursor:
    cursor.execute("SELECT 1")
    print(cursor.fetchall())
con.close()

Throws
---------------------------------------------------------------------------
InvalidDatabaseType                       Traceback (most recent call last)
<ipython-input-8-75cb71999960> in <module>
      1 import awswrangler as wr
----> 2 con = wr.postgresql.connect(secret_id="arn:aws:secretsmanager:us-east-1:**************:secret:db/rds/postgres/awswrangler-OFWDkE")
      3 with con.cursor() as cursor:
      4     cursor.execute("SELECT 1")
      5     print(cursor.fetchall())

~/.pyenv/versions/anaconda3-2020.11/lib/python3.8/site-packages/awswrangler/postgresql.py in connect(connection, secret_id, catalog_id, dbname, boto3_session, ssl_context, timeout, tcp_keepalive)
    133     )
    134     if attrs.kind != "postgresql":
--> 135         raise exceptions.InvalidDatabaseType(
    136             f"Invalid connection type ({attrs.kind}. It must be a postgresql connection.)"
    137         )

InvalidDatabaseType: Invalid connection type (postgres. It must be a postgresql connection.)

P.S. Don't attach files. Please, prefer add code snippets directly in the message body.

awsrangler expects the engine attribute of the secret being set to postgresql for using wr.postgresql.connect(). In your case it seems to be postgres instead of postgresql, so for a quick fix you could just change this attribute.
However I can see that it would be fine to accept postgres aswell.

Alternatively you can also try if the fix works for you:
pip uninstall awswrangler -y
pip install git+https://github.com/maxispeicher/aws-data-wrangler.git@add-postgres-conn-kind

Covered in 2.7.0 release


Works with Panel (sort of)

I would like to document that Quak works with Panel. Might also test the widget in alternative ways that could provide input for further improvements.
I see some issues though that I could not figure out where where coming from or how to solve.
Max-Height of quak widget set to 274px
I can't figure out where this is coming from or change it. Setting widget.layout.height="100%", widget.layout.height="600px" or widget.layout.max_height="100%" does not seem to change anything.

Lots of errors in terminal while loading and interacting with the widget
Traceback (most recent call last):
  File "/home/jovyan/repos/private/quak/.venv/lib/python3.11/site-packages/quak/_widget.py", line 64, in _handle_custom_msg
    result = self._conn.query(sql).arrow()
             ^^^^^^^^^^^^^^^^^^^^^
duckdb.duckdb.ParserException: Parser Error: syntax error at or near "50000000000"
2024-08-03 18:19:22,213 Error processing query
Traceback (most recent call last):
  File "/home/jovyan/repos/private/quak/.venv/lib/python3.11/site-packages/quak/_widget.py", line 68, in _handle_custom_msg
    self._conn.execute(sql)
duckdb.duckdb.BinderException: Binder Error: column "bronze" must appear in the GROUP BY clause or must be part of an aggregate function.
Either add it to the GROUP BY list, or use "ANY_VALUE(bronze)" if the exact value of "bronze" is not important.
LINE 4: ... TABLE IF NOT EXISTS cube_index_5407d6bc AS WITH "counts" AS (SELECT CASE
                                ...
                                                  ^
2024-08-03 18:19:22,332 Error processing query
Traceback (most recent call last):
  File "/home/jovyan/repos/private/quak/.venv/lib/python3.11/site-packages/quak/_widget.py", line 68, in _handle_custom_msg
    self._conn.execute(sql)
duckdb.duckdb.BinderException: Binder Error: column "bronze" must appear in the GROUP BY clause or must be part of an aggregate function.
Either add it to the GROUP BY list, or use "ANY_VALUE(bronze)" if the exact value of "bronze" is not important.
LINE 4: ... TABLE IF NOT EXISTS cube_index_3d676da9 AS WITH "counts" AS (SELECT CASE
                                ...
                                                  ^
2024-08-03 18:19:22,448 Error processing query
Traceback (most recent call last):
  File "/home/jovyan/repos/private/quak/.venv/lib/python3.11/site-packages/quak/_widget.py", line 68, in _handle_custom_msg
    self._conn.execute(sql)
duckdb.duckdb.BinderException: Binder Error: column "bronze" must appear in the GROUP BY clause or must be part of an aggregate function.
Either add it to the GROUP BY list, or use "ANY_VALUE(bronze)" if the exact value of "bronze" is not important.
LINE 4: ... TABLE IF NOT EXISTS cube_index_cc74a395 AS WITH "counts" AS (SELECT CASE

Dark Theme?
I would be nice with a Dark Theme well. Panel supports dark themes really work and I mostly use them in my work.
Panel Native Widget?
With Panel 1.5.0 (not yet released as of 2024-08-03) you also build AnyWidgetComponents with Panel. I.e. reuse the javascript and build the Python model using Python.
I think this repository and widget could be a test of AnyWidget as a standard supported by libraries outside Jupyter like Marimo and Panel. So if you are interested I could contribute a Panel version of the quak Widget. Let me know if you are interested. Thx.

Max-Height of quak widget set to 274px

Yes, the height is currently hardcoded to show ~10 rows. We can add an API for this.

Lots of errors in terminal while loading and interacting with the widget

Duplicate of #21

Dark Theme?

Theming is a bag of worms. Right now I don't have the capacity to take this on as I'm wrapping up my PhD. It would be an API that I'd be open to and willing to support through CSS variables in the future."

Panel Native Widget?

I don't have the capacity to take this on and encourage someone from the Panel community who is familiar to take it on.

Thanks for your feedback. Makes sense.
I've tried to minimize and make neutral. Let me know what you think. Thanks.
For context my contribution was probably inspired by what I have contributed in other places like https://github.com/whitphx/transformers.js.py#panel and https://developmentseed.org/lonboard/latest/ecosystem/panel/.

I persisted the original tutorial here https://discourse.holoviz.org/t/works-with-quak/7537.


persisting, orphaned errors with browser back button

Versions

vee-validate: 2.1.3
vue: 2.5.16

Describe the bug
If cursor is focused in an input with a vee-validate error, the error is not cleared if the browser back button is clicked. At this point, the error is orphaned and cannot be cleared except through a hard refresh.
To reproduce
Steps to reproduce the behavior:

go to a form with inputs
enter an invalid value in an input
make sure the cursor is focused in the input
click the browser back button
v-validate errors.items will still contain the focused input's error message from the previous page, even though that component has been destroyed

Expected behavior
Whether leaving programatically or by using the browser back functionality, errors should be cleared when a component is destroyed.
Desktop (please complete the following information):

Windows 10
Browser Chrome

Sorry for taking so long to address this.
It depends on how are you "leaving the page" if you are using Vue-Router then it should clear properly since the destroyed hooks are being called.
If you are using something like turbolink or pjax other means of achieving navigation then Its outside of this plugin's scope to detect that. You could call $validator.reset before leaving the page and I would imagine it solve your issue. I would be more helpful if you have an example or a code sample I can take a look at.


Experiment with prompt in background

Given that a response from an AI service can take some time, it may be useful to always be able to accept more user input whilst the service is processing. This branch experiments with this approach.

This didn't feel stable. In particular I lost the rich (colour) rendering in the terminal. I've since added a pipeline absctraction in the CLI which means I can bring in something like this in an experimental manner (without risking the simple case).  For now closing this and taking lessons from this experiment.


Implement grammar flash cards

Generate sentence with missing word according to fluentforever technique

What role does the mnemonic play here?

Some examples from the book:


Apple Silicon (M1) Compatibility

As of now, the course application doesn't start on a recent Mac with the M1 processor.
The main reason is the missing ARM support for the several Docker images that we're using:

LocalStack
Keycloak
sqs-alpine

(also with the Rosetta 2 emulation, the linux/amd64 images don't start properly)
Working images:

PostgreSQL ✅

I'll use this issue to further track the compatibility.

With a recent Docker, MacOS, and LocalStack version, the SQS emulation of LocalStack seems to work on an Apple M1 (ARM64) laptop.
For Keycloak, there are various community ARM64 Docker images available: https://github.com/richardjkendall/keycloak-arm and https://hub.docker.com/r/mihaibob/keycloak


Update shared-channels.md

Adding note that sharing files of shared channels externally who are not members of the channel is not supported per https://docs.microsoft.com/en-us/sharepoint/teams-connected-sites.

Docs Build status updates of commit cb8d897:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Teams/shared-channels.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@MikePlumleyMSFT PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

Docs Build status updates of commit 9d23c05:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Teams/shared-channels.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


preset record selector from dependant application

There is a preset record selector in Forms builder -> Form settings -> States -> Conditions. But the selector offers only preset records from the current Application, not from the Build-on Applications.  (bug or feature?)
It would be great if such preset record selector will be available also in Frontend/Backend functions. (feature request)

I´ve had a look. We can easily enable the preset selector for build-on apps - but I ran into an issue during implementation. We do dependency checking between apps so that you cannot easily break your application. The problem is that filters (form state conditions are just limited versions of filters) are used everywhere und can in some contexts use sub-queries which can also use sub-queries and so on. I will need to implement recursive checks to make sure that you cannot remove an application that any of your preset-filters still rely on.
It should be rather straightforward, adding preset record placeholders in functions, when I can solve the issue of the recursive checks.

We´ve implemented the required checks - in the next release, you will be able to access preset records from built-on applications in form state conditions and filters.
Inside front- & backend functions, you´ll also be able to retrieve the current record ID for a preset with a new instance function, called get_preset_record_id().


Composite primary key - export/import problem

Hi, nice package. Found some problems regarding validation of the json which can be exported/imported. Assume following json table:
{
  "name": "foo_bars_bar",
  "schema": [
    {
      "column": "\"fooId\"",
      "value": "integer NOT NULL"
    },
    {
      "column": "\"barId\"",
      "value": "integer NOT NULL"
    },
    {
      "constraint": "\"FK_d84ae99b331c8b558039a2d1f78\"",
      "value": "FOREIGN KEY (\"fooId\") REFERENCES \"foo\" (\"id\") ON DELETE CASCADE ON UPDATE CASCADE"
    },
    {
      "constraint": "\"FK_8577beb4178aeb72187702c53ab\"",
      "value": "FOREIGN KEY (\"barId\") REFERENCES \"bar\" (\"id\") ON DELETE NO ACTION ON UPDATE NO ACTION"
    },
    {
      "column": "PRIMARY",
      "value": "KEY (\"fooId\", \"barId\")"
    }
  ],
  "indexes": [
    {
      "name": "IDX_d84ae99b331c8b558039a2d1f7",
      "value": "\"fooId\""
    },
    {
      "name": "IDX_8577beb4178aeb72187702c53a",
      "value": "\"barId\""
    }
  ],
  "values": []
}

The primary "column" is counted as column and resulting in invalid json without any modifications applied in export/import process.
https://github.com/jepiqueau/jeep-sqlite/blob/master/src/utils/utils-json.ts#L98

@ebess can you try as a constraint
{
"constraint":""PK_fooId_barId"",
"value":"PRIMARY KEY ("foodId","\barId"")"
}
This should work

@jepiqueau ok i see but i just exported + try to reimported without any adjustments. so, the exportToJson method has to be corrupted exporting the composite key as a column to json.

@ebess may be i missed that one in the export i will see. Does the import works as i suggested. It is the right way to define it as a constraint if yes you may look in the export an propose a PR

Yap, that seems to solve the problem but tbh I don't know exactly how to start using the repo locally for developing. But I will take a look

@ebess there are plenty of app starter on https://github.com/jepiqueau

@ebess This has been fixed in 1.5.6


Jupyter notebook dark mode doesn't work on first load

Acknowledge the following

[x] I carefully read and followed the Getting Started guide.
[x] I read through FAQ and searched through the past issues, none of which addressed my issue.
[x] The issue I am raising is a potential bug in al-folio and not just a usage question.  [For usage questions, please post in the Discussions instead of raising an issue.]

Describe the bug
When first opening the blog post about jupyter notebooks, activating the dark mode doesn't affect the notebook. After a reload it works as expected.
To Reproduce
Steps to reproduce the behavior:

Go to https://alshedivat.github.io/al-folio/blog/2023/jupyter-notebook/
Change theme to dark and back
Reload the page
Change theme to dark and back again

Same issue:
I try the start jekyll with this command:
bundle exec jekyll serve --lsi
I get this error:
....
Jekyll Feed: Generating feed for posts
Populating LSI...
Rebuilding index...

 Jekyll Diagrams: Command Not Found: mmdc
Conversion error: JekyllJupyterNotebook::Converter encountered an error while converting 'assets/jupyter/blog.ipynb':
                  No such file or directory - jupyter
                  ------------------------------------------------
    Jekyll 4.3.2   Please append `--trace` to the `serve` command
                   for any additional information or backtrace.
                  ------------------------------------------------
/Users/redabenh/.rvm/gems/ruby-3.1.3/gems/jekyll-jupyter-notebook-0.0.5/lib/jekyll-jupyter-notebook/converter.rb:50:in `spawn': No such file or directory - jupyter (Errno::ENOENT)
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/jekyll-jupyter-notebook-0.0.5/lib/jekyll-jupyter-notebook/converter.rb:50:in `block in convert_notebook'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/jekyll-jupyter-notebook-0.0.5/lib/jekyll-jupyter-notebook/converter.rb:49:in `pipe'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/jekyll-jupyter-notebook-0.0.5/lib/jekyll-jupyter-notebook/converter.rb:49:in `convert_notebook'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/jekyll-jupyter-notebook-0.0.5/lib/jekyll-jupyter-notebook/converter.rb:28:in `convert'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/jekyll-4.3.2/lib/jekyll/renderer.rb:105:in `block in convert'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/jekyll-4.3.2/lib/jekyll/renderer.rb:104:in `each'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/jekyll-4.3.2/lib/jekyll/renderer.rb:104:in `reduce'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/jekyll-4.3.2/lib/jekyll/renderer.rb:104:in `convert'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/jekyll-4.3.2/lib/jekyll/renderer.rb:84:in `render_document'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/jekyll-4.3.2/lib/jekyll/renderer.rb:63:in `run'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/jekyll-4.3.2/lib/jekyll/site.rb:572:in `render_regenerated'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/jekyll-4.3.2/lib/jekyll/site.rb:564:in `block in render_pages'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/jekyll-4.3.2/lib/jekyll/site.rb:563:in `each'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/jekyll-4.3.2/lib/jekyll/site.rb:563:in `render_pages'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/jekyll-4.3.2/lib/jekyll/site.rb:211:in `render'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/jekyll-4.3.2/lib/jekyll/site.rb:80:in `process'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/jekyll-4.3.2/lib/jekyll/command.rb:28:in `process_site'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/jekyll-4.3.2/lib/jekyll/commands/build.rb:65:in `build'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/jekyll-4.3.2/lib/jekyll/commands/build.rb:36:in `process'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/jekyll-4.3.2/lib/jekyll/command.rb:91:in `block in process_with_graceful_fail'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/jekyll-4.3.2/lib/jekyll/command.rb:91:in `each'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/jekyll-4.3.2/lib/jekyll/command.rb:91:in `process_with_graceful_fail'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/jekyll-4.3.2/lib/jekyll/commands/serve.rb:86:in `block (2 levels) in init_with_program'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/mercenary-0.4.0/lib/mercenary/command.rb:221:in `block in execute'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/mercenary-0.4.0/lib/mercenary/command.rb:221:in `each'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/mercenary-0.4.0/lib/mercenary/command.rb:221:in `execute'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/mercenary-0.4.0/lib/mercenary/program.rb:44:in `go'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/mercenary-0.4.0/lib/mercenary.rb:21:in `program'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/gems/jekyll-4.3.2/exe/jekyll:15:in `<top (required)>'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/bin/jekyll:25:in `load'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/bin/jekyll:25:in `<main>'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/bin/ruby_executable_hooks:22:in `eval'
      from /Users/redabenh/.rvm/gems/ruby-3.1.3/bin/ruby_executable_hooks:22:in `<main>'

There is any workarround ?
i'm on Mac OS

I finally build jekyll on docker
i think you need to complet the getting started guide> local-setup-standard to include jupyter installation steps

Actually the error you reported is not related to this. As you already figured it out, you were just missing jupyter installed on your machine. We have a PR #1523 to make Docker the recommended way of using this, but I agree that jupyter installation is missing in README. I will add it now.

I couldn't reproduce this bug anymore. Maybe it is solved now.

I'm getting this but after I push my code to Github Pages in the build step.
  Jekyll Diagrams: Command Not Found: mmdc
  Conversion error: JekyllJupyterNotebook::Converter encountered an error while converting 'assets/jupyter/blog.ipynb':
                    No such file or directory - jupyter
                    ------------------------------------------------
      Jekyll 4.3.2   Please append `--trace` to the `build` command
                     for any additional information or backtrace.

Is there something I'm missing?

Update on my issue: I was able to fix this by adding the missing Mermaid CLI and Jupyter to the jekyll.yml file:
      - name: Install Node.js for Mermaid CLI
        uses: actions/setup-node@v3
        with:
          node-version: "14"

      - name: Install Mermaid CLI
        run: npm install -g @mermaid-js/mermaid-cli

      - name: Install Python for Jupyter
        uses: actions/setup-python@v4
        with:
          python-version: "3.9"

      - name: Install Jupyter
        run: pip install jupyter

wontfix

Update on my issue: I was able to fix this by adding the missing Mermaid CLI and Jupyter to the jekyll.yml file:
      - name: Install Node.js for Mermaid CLI
        uses: actions/setup-node@v3
        with:
          node-version: "14"

      - name: Install Mermaid CLI
        run: npm install -g @mermaid-js/mermaid-cli

      - name: Install Python for Jupyter
        uses: actions/setup-python@v4
        with:
          python-version: "3.9"

      - name: Install Jupyter
        run: pip install jupyter

Can't fix after adding those lines to yml :(

Where is the jekyll.yml file you are using? Is it for hosting in another provider? Also what is your error @roilhi?
This issue specifically is related to jupyter dark mode, but a lot of people used it to report a different bug.

Where is the jekyll.yml file you are using? Is it for hosting in another provider? Also what is your error @roilhi?
This issue specifically is related to jupyter dark mode, but a lot of people used it to report a different bug.

Hello @george-gca, my jekyll.yml file is located at .github/workflows/ directory. I'm using github pages. Everything is fine at local by running the docker compose file. I have the same issue as @lesterbotello, github pages is not building the Jekyll file because of the jupyter notebook issue.

I believe you are using a very old version of the template, since currently there is no file named jekyll.yml in .github/workflows/.
You probably have to update your code to get these changes, since the current building script does install jupyter.

@george-gca Thanks. I don't think is an old version, since I have the same deploy.yml file that you show. I will try to update.

Is that action that is currently running to deploy your site?

Yes, it is. I've deleted the jekyll.yml file. It´s working but the theme is not renderizing.

What do you mean?

I believe you are using a very old version of the template, since currently there is no file named jekyll.yml in .github/workflows/.
You probably have to update your code to get these changes, since the current building script does install jupyter.

If it does install jupyter then what is the cause of error, as I am facing the same error

@george-gca I am facing this error:  Conversion error: JekyllJupyterNotebook::Converter encountered an error while converting 'assets/jupyter/blog.ipynb':
No such file or directory - jupyter
------------------------------------------------
Jekyll 4.3.3   Please append --trace to the build command
for any additional information or backtrace.
------------------------------------------------
What could be the problem?

A lot of comments here deviated from the original issue. The issue here is about dark mode not working on the first load. Other issues must be discussed in their respective issues (or discussions) on the repository.
@fareedqk check the comment above about related problems. If it doesn't solve your problem, please open a new issue for this matter.

Can confirm the error below still exists even after jupyter installation.
This error doesn't pop in some environments / machines, not clear yet why.
  Conversion error: JekyllJupyterNotebook::Converter encountered an error while converting 'assets/jupyter/blog.ipynb':
                    No such file or directory - jupyter
                    ------------------------------------------------
      Jekyll 4.3.3   Please append `--trace` to the `serve` command
                     for any additional information or backtrace.
                    ------------------------------------------------

Conversion error: JekyllJupyterNotebook::Converter encountered an error while converting 'assets/jupyter/blog.ipynb':

This is the solution that worked for me. Thanks.
The old workflow is no longer supported it seems.

So is there a solution? I copied the most recent version of alshedivat.github.io today (June 2024), followed the instructions and hit this error. If the solution is to "build Jekyll on Docker" then please explain what that means and how to do it.

If your error is not related to jupyter notebook dark mode not working, please open a new issue or Q&A to keep this focused on the error in question.

The problem is related to your system path. In my case, I installed Jupyter through pip, and the installer put the exec on ~/.local/bin/. This folder wasn't on my path, so Jekyll did not find it when it required it. I just put this folder in the path and solved the problem.

Yeah, I know, but idk how to fix this yet. If someone can find a solution for this and is willing to send a PR, I am happy to review it.

   let jupyterTheme = determineComputedTheme();

   $(".jupyter-notebook-iframe-container iframe").each(function () {
-    $(this).contents().find("head").append(cssLink);
+    var iframe = $(this);
+    iframe.contents().find("head").append(cssLink);

-    if (jupyterTheme == "dark") {
-      $(this).bind("load", function () {
-        $(this).contents().find("body").attr({
-          "data-jp-theme-light": "false",
-          "data-jp-theme-name": "JupyterLab Dark",
-        });
+    function changeToJpDarkTheme() {
+      iframe.contents().find("body").attr({
+        "data-jp-theme-light": "false",
+        "data-jp-theme-name": "JupyterLab Dark",
       });
+    };
+
+    if (jupyterTheme == "dark") {
+      if (iframe[0].contentWindow.document.readyState === "complete") {
+        changeToJpDarkTheme();
+      }
+      iframe.bind("load", changeToJpDarkTheme);
     }
   });

Maybe something like this 🤔 I'm not really familiar with these though...

I solved mine by install jupyter in python and adding the python path to PATH.

Does it really work? Because it doesn't make sense. We install what is needed to convert jupyter to html in the GitHub actions and convert it using the plug-in.

a complete jekyll.yml that should work for pages deployment, following the suggested fixes above:
# This workflow uses actions that are not certified by GitHub.
# They are provided by a third-party and are governed by
# separate terms of service, privacy policy, and support
# documentation.

# Sample workflow for building and deploying a Jekyll site to GitHub Pages
name: Deploy Jekyll site to Pages

on:
  # Runs on pushes targeting the default branch
  push:
    branches: ["main"]

  # Allows you to run this workflow manually from the Actions tab
  workflow_dispatch:

# Sets permissions of the GITHUB_TOKEN to allow deployment to GitHub Pages
permissions:
  contents: read
  pages: write
  id-token: write

# Allow only one concurrent deployment, skipping runs queued between the run in-progress and latest queued.
# However, do NOT cancel in-progress runs as we want to allow these production deployments to complete.
concurrency:
  group: "pages"
  cancel-in-progress: false

jobs:
  # Build job
  build:
    runs-on: ubuntu-latest
    steps:
      - name: Checkout
        uses: actions/checkout@v4
      - name: Setup Ruby
        uses: ruby/setup-ruby@8575951200e472d5f2d95c625da0c7bec8217c42 # v1.161.0
        with:
          ruby-version: '3.1' # Not needed with a .ruby-version file
          bundler-cache: true # runs 'bundle install' and caches installed gems automatically
          cache-version: 0 # Increment this number if you need to re-download cached gems
      - name: Install Node.js for Mermaid CLI
        uses: actions/setup-node@v3
        with:
          node-version: "14"
      - name: Install Mermaid CLI
        run: npm install -g @mermaid-js/mermaid-cli
      - name: Install Python for Jupyter
        uses: actions/setup-python@v4
        with:
          python-version: "3.9"
      - name: Install Jupyter
        run: pip install jupyter
      - name: Setup Pages
        id: pages
        uses: actions/configure-pages@v5
      - name: Build with Jekyll
        # Outputs to the './_site' directory by default
        run: bundle exec jekyll build --baseurl "${{ steps.pages.outputs.base_path }}"
        env:
          JEKYLL_ENV: production
      - name: Upload artifact
        # Automatically uploads an artifact from the './_site' directory by default
        uses: actions/upload-pages-artifact@v3

  # Deployment job
  deploy:
    environment:
      name: github-pages
      url: ${{ steps.deployment.outputs.page_url }}
    runs-on: ubuntu-latest
    needs: build
    steps:
      - name: Deploy to GitHub Pages
        id: deployment
        uses: actions/deploy-pages@v4

@upunaprosk have you tested it?


Merge src/transport and src/messaging in github

Problem
Based on the latest messaging layer diagram:
Messaging layer is responsible for message framing & routing: With an interaction encrypted and signed, the Message Layer constructs the payload format with required and optional header fields; which specify properties of the message as well some routing information. In current CHIP codebase, ExchangeManager and SecureSessionMgr fall under this layer.
Transport layer is responsible for IP framing & transport management: After the final payload has been constructed, it is sent to the underlying transport protocol for IP management of the data. In current CHIP codebase, Inet falls under this layer.
Proposed Solution
Merge the components under src/transport into src/messaging in github

Per the architecture diagram, these are separate items. We shouldn't be merging base IP transport...

@woody-apple your diagram show the SecureSessionMgr, SecureSession.... are under messaging.
It seems all components under following following directory should be merged to src/messaging.
https://github.com/project-chip/connectedhomeip/tree/master/src/transport

Happy to sync up offline, but the TT diagram has a transport box which contains the base transport (TCP/UDP).
There are many things to clean up here for sure, much of what's in transport needs to move around. BLE should move to rendezvous, rendezvous* should move to /rendezvous/ and secure session should move to likely into lib/message.
We also need to be more explicit about the API that's allowed outside the messaging box (the Secure Channel API). @pan-apple can you help write up some documentation on that, and help us clean up headers here so it's clear what's private to the message implementation, vs usable outside of Secure Channel?

I think the transport box in TT diagram should also contains Inet which defines the TCP/UDP endpoints. I agree we should clean them up later with more specific tasks.
Thanks for clarification.


Remove proxy for asynchronous event handler

Right now it need to register normal handler first. Then invoke RegisterBackgroundTask to register asynchronous handler. It will be better if we can directly register the asynchronous handler directly without registering the normal one.

Switch to use this will be better: https://www.nuget.org/packages/AsyncEvent


Ghost comment approach for virtual fields (e.g. relations)

Problem
The problem for this is the barebone model is being bloated by virtual fields from relation that do not actually map to the database in reality
Suggested solution
Maybe we can have at least clear separation of field that are persisted and that are virtual maybe via syntax highlights or like ghost comments

Can you elaborate a bit? What do you refer to by "virtual fields" exactly?
Can you show an example of a schema that has this problem?
(I assume you refer to the "virtual" back relations that describe the return route for a relation - but that assumption might be wrong)

(I assume you refer to the "virtual" back relations that describe the return route for a relation - but that assumption might be wrong)

Yes this is write the back relations...
This is some model
model Account {
  id          String @default(cuid())
  anonymousId String @default(cuid())

  email       String?
  phoneNumber String?
  password    String?

  username String
  name     String
  website  String?
  bio      String?
  coverUrl String?
  photoUrl String?

  category   Category? @relation(fields: [categoryId], references: [id])
  categoryId String?

  content   UserContent? @relation(fields: [contentId], references: [id])
  contentId String?

  location   Location? @relation(fields: [locationId], references: [id])
  locationId String?

  type   AccountType
  role   UserRole
  gender Gender

  birthday  DateTime?
  createdAt DateTime  @default(now())
  updatedAt DateTime  @updatedAt

  followees             UserFollower[]                  @relation("followees")
  followers             UserFollower[]                  @relation("followers")
  notifications         UserNotification[]

  memberships GroupMember[]
  contents    GroupContent[]
  reactions   GroupContentReaction[]
  votes       GroupPollVote[]
  answers     GroupQuestionAnswer[]
  purchases   GroupExamPurchase[]
  scores      GroupExamScore[]
  examAnswers GroupExamModuleQuestionAnswer[]

  GroupMember GroupMember[] @relation("AccountToGroupAnonymousMember")
  @@id([id])
  @@unique([anonymousId])
  @@unique([username])
}

This is some kind of Ghost Comment (see trailing comments)
Atleast its easy to identify which fields will be persisted and which will be not on the database

Im not literally talking about making them comments. But at least a clear separation of not persisted fields and not

I think this is mostly a request for our language server and we can transfer it there.


Components non-direct children of Teleport do not trigger vnode-unmounted

Version
3.0.11
Reproduction link
Vue SFC Playground
Steps to reproduce
Place a component with a onUnmounted hook and / or vnode-unmounted event inside a node inside a teleport component.
What is expected?
When the component is unmounted, hook and / or event must be triggered
What is actually happening?
Nothings

I'm not sure this is a bug or by design: only direct-children (e.g. the div that is wrapping Comp) trigger the hook

duplicate of #3623

duplicate of #3623


Don't call xrandr on OS X

Usual OS X instalation doesn't contain the xrandr utility. One can bring
it only by installing XQuartz and even after it the utility doesn't work
well with the native windows.
Use more precise checkout (os.name == "posix" in OS X as it is
posix-compliant) to identify the Linux system and avoid redundant call.

Hi, this is a cosmetic fix for renderer on OS X.
Please pay attention that it wasn't tested on Linux, although I am sure it will work :)

Merged in 2401650


Launcher backend leaks file descriptors

From @jarifibrahim on January 9, 2019 13:33
Issue Overview
The launcher-backend (https://github.com/fabric8-launcher/launcher-backend) stores all the boosters locally and the /reindex API is used to update the local cache.
If multiple requests (more than ~10 requests) are sent to launcher-backend (doesn't need to be sent concurrently), it crashes with Too many files open exception. With every request sent to /reindex endpoint, the number of open files keeps on increasing.
Use the following command to see the number of open files
ls -l /proc/{launcher_backend_process_id}/fd | wc -l

This causes two issues

The /booster-catalog endpoint returns empty (or partial response). The empty response issue https://github.com/openshiftio/openshift.io/issues/4670 (which was a SEV-1) was also caused because there were too many open files. To reproduce this issue, see Part A of steps to reproduce

Backend crashes with an exception.

2019-01-09 18:19:25,903 INFO  [NativeGitBoosterCatalogPathProvider] (EE-ManagedExecutorService-default-Thread-6) Created /tmp/booster-catalog1252048683612888507
2019-01-09 18:19:25,904 INFO  [NativeGitBoosterCatalogPathProvider] (EE-ManagedExecutorService-default-Thread-6) Executing: git clone https://github.com/fabric8-launcher/launcher-booster-catalog.git --branch master --recursive --depth=1 --quiet -c advice.detachedHead=false /tmp/booster-catalog1252048683612888507
2019-01-09 18:19:25,904 SEVERE [AbstractBoosterCatalogService] (EE-ManagedExecutorService-default-Thread-6) Error while indexing: java.io.IOException: Cannot run program "git": error=24, Too many open files
	at java.lang.ProcessBuilder.start(ProcessBuilder.java:1048)
	at io.fabric8.launcher.booster.catalog.spi.NativeGitBoosterCatalogPathProvider.cloneRepository(NativeGitBoosterCatalogPathProvider.kt:63)
	at io.fabric8.launcher.booster.catalog.spi.NativeGitBoosterCatalogPathProvider.createCatalogPath(NativeGitBoosterCatalogPathProvider.kt:35)
	at io.fabric8.launcher.booster.catalog.AbstractBoosterCatalogService.doIndex(AbstractBoosterCatalogService.kt:214)
	at io.fabric8.launcher.booster.catalog.AbstractBoosterCatalogService.access$doIndex(AbstractBoosterCatalogService.kt:43)
	at io.fabric8.launcher.booster.catalog.AbstractBoosterCatalogService$index$1.get(AbstractBoosterCatalogService.kt:110)
	at io.fabric8.launcher.booster.catalog.AbstractBoosterCatalogService$index$1.get(AbstractBoosterCatalogService.kt:43)
	at java.util.concurrent.CompletableFuture$AsyncSupply.run(CompletableFuture.java:1590)
	at org.jboss.as.ee.concurrent.ControlPointUtils$ControlledRunnable.run(ControlPointUtils.java:105)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at org.glassfish.enterprise.concurrent.internal.ManagedFutureTask.run(ManagedFutureTask.java:141)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)
	at org.glassfish.enterprise.concurrent.ManagedThreadFactoryImpl$ManagedThread.run(ManagedThreadFactoryImpl.java:250)
	at org.jboss.as.ee.concurrent.service.ElytronManagedThreadFactory$ElytronManagedThread.run(ElytronManagedThreadFactory.java:78)
Caused by: java.io.IOException: error=24, Too many open files
	at java.lang.UNIXProcess.forkAndExec(Native Method)
	at java.lang.UNIXProcess.<init>(UNIXProcess.java:247)
	at java.lang.ProcessImpl.start(ProcessImpl.java:134)
	at java.lang.ProcessBuilder.start(ProcessBuilder.java:1029)
	... 16 more

The only way to resolve this is to kill the process and start a new one. To reproduce this, see Part B of the steps to reproduce section
Expected Behaviour
Launcher-backend shouldn't crash with Too many files open exception. It should gracefully close all the open files.
Current Behaviour
Launcher-backend doesn't close the file descriptors and crashes.
Steps To Reproduce

The empty/partial response issue
Start launcher backend with
LAUNCHER_BACKEND_CATALOG_REINDEX_TOKEN="abc" java -jar -Xdebug -Xrunjdwp:transport=dt_socket,server=y,suspend=n,address=5005 web/target/launcher-backend-thorntail.jar

and run https://gist.github.com/jarifibrahim/ebb1b9e6e8e2d9a3eb4f2a27d7e4c940
NOTE - This issue isn't always reproducible. You might have to try multiple times to reproduce the issue

Server Crash
Start launcher backend with
LAUNCHER_BACKEND_CATALOG_REINDEX_TOKEN="abc" java -jar -Xdebug -Xrunjdwp:transport=dt_socket,server=y,suspend=n,address=5005 web/target/launcher-backend-thorntail.jar

and run https://gist.github.com/jarifibrahim/76def0939fb82b01897df2e3355f6f46

Copied from original issue: openshiftio/openshift.io#4697

This could be because the BCS isn't properly closing Files streams: https://github.com/fabric8-launcher/launcher-booster-catalog-service/blob/master/src/main/kotlin/io/fabric8/launcher/booster/catalog/AbstractBoosterCatalogService.kt#L263

May have been solved with https://github.com/fabric8-launcher/launcher-backend/commit/5f316c7ab5b07794583b7675a9343e2170ce4ea2. Closing now, reopen if the problem persists


Retry on 403 "Forbidden" errors

Yes, the standard says the client should not retry on 403.
(see eg. https://httpstatuses.com/403)
But that doesn't take into account servers that give us unwarranted
403s.
See https://github.com/broadinstitute/gatk/issues/3735 for context.

@droazen please take a look.


Frontend for metrics of microservice in K8S

Overview
Completed frontend metric page to fetch, parse and render metrics data for microservice.
Issue Type

[ ] Bug
[x] Feature
[ ] Tech Debt

Description
A metric tab is now available on the Galvr homepage. Within it, a dropdown menu is provided to select microservice, and the corresponding metrics will be rendered upon selection.
Steps to Reproduce Bug / Validate Feature / Confirm Tech Debt Fix

Rebuild the docker images for metricEndpoint (backend) and galvr-nextjs (frontend).
(Optional) if you are using OTEL demo app, re-deploy the OTEL demo via HELM per the readme in demo-otel directory.
Deploy the updated metricEndpoint (backend) and galvr-nextjs (frontend) to your cluster.
See the frontend at http://localhost:30003/.

Screenshots & Videos

the graphs look great!


Move getDefaultFile to trait

See: https://github.com/qossmic/deptrac/issues/689

Use getDefaultFile in DebugCommands.
DebugLayer layer is now optional, if no layer is provided all layers will be printed

This is just me, but I am not a fan of traits and rather saw a common parent class. Otherwise, I like it. :)

This is only temporary anyway, as we will remove support for the old filename at some point anyway. So, I see no reason to split hairs over trait vs. abstract class just for this. Thanks @gennadigennadigennadi


movefocus feature implementation

Hi there, I want to implement some of the cross monitor support to hycov:movefocus as found in the original function
I could add it here
And it could perhaps be an additional argument or dispatcher if not always desired
I am going to sleep now, but if you are awake, may I have your opinion on this addition
Many thanks for the project

This does not seem to have much to do with the functionality of this plugin, but rather an additional focus switch scheduling, and does not conflict with the current plugin, I think you can write a separate plugin to implement it

@DreamMaoMao
The reason i mention is that binding hycov:movefocus to hjkl for example, does not only switch windows in the overview, but also everywhere else,
so one must replace movefocus with hycov:movefocus to avoid double movement.
That is ok, but then the feature where one can move between monitors using movefocus is lost as it is not implemented in hycov:movefocus which is what i proposed to implement.
It can be done with submaps but it is fiddly, i am using submaps for now

I wanted to get your thoughts, but this is not the territory of another plugin

I don't quite understand, bind hycov:movefocus move window focus
And binding movefoucs mobile display focus why does it conflict?
Don't they bind to different  key?

oh oh, I think I know what you mean

yeah so adding move montior support to hycov:movefocus if not in the overview would allow it to behave like movefocus
i was going to try and do this in a submap though, maybe easier, like
bind= SUPER, h, movefocus
...
bind= SUPER, space, hycov:enteroverview
bind= SUPER, space, submap, overview
submap= overview
bind= SUPER, h, hycov:movefocus
...
bind= SUPER, space, hycov:;leaveoverview
bind= SUPER, space, submap, reset
submap= reset

have dont in:
https://github.com/DreamMaoMao/hycov/commit/2f17ef51566f7be4912ce4104445651d3a83dbc5

Oh shit, i did mean id do it, but thanks!


Updated resources to build with c++ 14 instead of 11

LGTM

LGTM

I think you should add what we discussed with a CMake configure-time check for the required features.

Closing.  C++14 may not be needed for this.


Feature request initialHeight or minHeight

I'm setting the initialHeight or minHeight with minRows={2} but it's not exactly. There is a possibility to add an initial height.

You can provide style={{ height: 10 }} for this purpose

I added style={{ height: 48 }}  but the value is replaced by style="height: 32px !important;". Also, I'm using TS so I tried to add minHeight but it is not allowed.

Could you describe your use case? It sounds like you want a different height than computed through rows - this ain't possible and I'm not sure why you'd like it.

The idea that the min-height could be 48px. But with minRows={2} is height: 56px !important;. So I don't have way to set that value as min-height

source here

I guess we could remove this restriction on style.minHeight:
https://github.com/Andarist/react-textarea-autosize/blob/58c9807645dceb9034b733d9cb63aa66df2364b0/src/index.tsx#L47-L51
Feel free to prepare a PR.

as a workaround;
textarea{     min-height: 0!important; }

Have thte same problem
          <TextareaAutosize
            className="focus:pointer-events-auto border-1 rounded-sm border-[#434343] bg-[#141414] text-white hover:border-primary-accent"
            style={{ width: 365,height: 200 }}
            value={newComment}
            placeholder={"Comment"}
            onChange={(e) => setNewComment(e.target.value)}
          ></TextareaAutosize>

Only width works. The height value is replaced with "height: xxx px !important"

I created a component to work with auto-size. Maybe you can use the same library that I used to fix all your issues.
autosize


Cloudant/CouchDB package should be available in open source

Follow the instructions for vagrant in README
I see that cloudant and alarms package are missing. Why?
$ wsk package list /whisk.system
packages
/whisk.system/slack                                               shared
/whisk.system/util                                                shared
/whisk.system/github                                              shared
/whisk.system/samples                                             shared
/whisk.system/system                                              shared
/whisk.system/watson                                              shared
/whisk.system/weather                                             shared
~/dev/whisk/demos/cleansetup
👻  $ wsk package get --summary /whisk.system/alarms
error: The supplied authentication is not authorized to access this resource (code 448)
~/dev/whisk/demos/cleansetup
👻  $ wsk package get --summary /whisk.system/cloudant
error: The supplied authentication is not authorized to access this resource (code 451)
~/dev/whisk/demos/cleansetup
👻  $ wsk package get --summary /whisk.system/alarms
error: The supplied authentication is not authorized to access this resource (code 453)

Also in the error message what does (code 448) (code 451) and (code 453) ?
If its an error code I would assume I see the same error code for the same command 448 !== 453

They are not in openwhisk.

I think both will be useful to move to open.

alarms seems like a basic core component as a scheduler
cloudant, it seems to be focus on IBM Cloudant, but someone can take it as a base/example and create a package for CouchDB or another data store.

What do you think?

Yes. It's in plan.

OK, I will hunt down those issues numbers and move to backlog

Found 1/2 #390 for alarm

Leaving this issue to track only cloudant/couchdb package, alarm is being tracked in another issue #390

opened a corresponding "documentation" issue #514, as packages.md is not usable by openwhisk users since it assumes cloudant is avail. as a pre-installed openwhisk package. We may wish to fix the packages.md if this sits too long on the backlog...

Cloudant package now in https://github.com/openwhisk/openwhisk-package-cloudant repo.

Thanks @rabbah


Fix confusing math

Changes
Fixed confusing math

Implementation details
Changed "... in the set $(i,j) = E$" to "... in the set $E$"
because $(i, j)$ is not $E$ itself but an element of it.

How to read this PR

Screenshots

Added @frankharkins to review the change 🙏


Add benchmarkSight to Foundry VTT V9.220

I'm closing this, since this is really just a test method and will likely be removed soon.


Add new sliders and style for them

Change the sliders and rework the style:

Appropriate small design and slim
Add rounded numbers instead of fixed ones
Improve handling

Still no workaround for making the steps of the sliders dependent on the data.

Rework the quarter or time slider to a custom one.


[AutoPR resourcehealth/resource-manager] [Microsoft.ResourceHealth] Add metadata api

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5834

This PR has been merged into https://github.com/Azure/azure-sdk-for-js/pull/2726


Support CORS settings for application

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
There is a new feature that has been implemented in PingOne which our organisation has been requesting. It is related to CORS settings on Application level.
In the PingOne console, it is possible to set specific CORS settings per application as visible from this screenshot:

These settings should be configurable via terraform, otherwise they would be overwritten with every new deployment via terraform.
All features that are supported via the PingOne console for CORS should be supported via terraform (also wildcards for example).
New or Affected Resource(s)

pingone_application (introduce a new nested schema named for example "cors" which would contain everything needed?)

Potential Terraform Configuration

# Copy-paste your PingOne related Terraform configurations here - for large Terraform configs,
# please use a service like Dropbox and share a link to the ZIP file. For
# security, you can also encrypt the files using our GPG public key.

# Remember to replace any account/customer sensitive information in the configuration before submitting the issue

References

#0000

For those watching this issue and waiting for this feature, I've released v0.24.0 today that includes these CORS settings.  OIDC app doc link / SAML app doc link


Something went wrong when browsing to Personal info in the MyAccount

Describe the issue:

Unable to reproduce this issue in m-16


refectoring

refactoring startBlockElement .
refactoring if condition in handleStartTag and handleEndTag.

refactoring

I want to keep this code as much in line with the original HtmlToSpannedConverter from AOSP to keep the possibility of merging changes upstream. I nevertheless very much appreciate the work you've done here.


parsl 2024.12.09 breaks ComputeManager

Describe the bug

To Reproduce

$ micromamba create --name qcfractal qcfractalcompute -c conda-forge -yq && micromamba activate qcfractal
$ python -c "from qcfractalcompute.compute_manager import ComputeManager"
Traceback (most recent call last):
  File "<string>", line 1, in <module>
    from qcfractalcompute.compute_manager import ComputeManager
  File "/Users/mattthompson/micromamba/envs/qcfractal/lib/python3.13/site-packages/qcfractalcompute/__init__.py", line 9, in <module>
    from .compute_manager import ComputeManager
  File "/Users/mattthompson/micromamba/envs/qcfractal/lib/python3.13/site-packages/qcfractalcompute/compute_manager.py", line 37, in <module>
    from .executors import build_executor
  File "/Users/mattthompson/micromamba/envs/qcfractal/lib/python3.13/site-packages/qcfractalcompute/executors.py", line 7, in <module>
    from parsl.channels import LocalChannel
ModuleNotFoundError: No module named 'parsl.channels'

This error does not show up after downgrading Parsl:
$ micromamba install "parsl <2024.12.09" -c conda-forge -yq
$ python -c "from qcfractalcompute.compute_manager import ComputeManager"
$

Expected behavior

Not this error
Additional context

https://github.com/Parsl/parsl/pull/3716

Already fixed in PR #863 :)
There will be a new release either later this week or early next week

Nice! ... I need to find better search terms


run

你好，想请教一下这个问题，谢谢

Are you asking what these do, or are you asking because these lines caused an error? If so, what was the error message?


Redirect http to https

The problem
Currently everything works as expected. My tunnel is established and works very well. But I have seen, that the tunnel is forwarding http (port 80), as well as https (port 443) to my homeassistant instance in my network. I would have expected an http to https rewrite, as the homeassistant locally only listens on port 443.
Is there maybe anything I am missing in my cloudflared configuration or in Cloudflare?
What version of Cloudflared has the issue?
4.2.12
What was the last working version of Cloudflared?
No response
What type of installation are you running?
Home Assistant OS
Add-on YAML Configuration
No response
Anything in the logs that might be useful for us?
No response
Steps to reproduce the issue
Setup the cloudflared addon as per documentation. The domain that is being used, is a subdomain (ha.xxx.de). No additional configuration is made.
Additional information
No response

You can enable Always use https by following this tutorial (which I have).

Perfect! Many thanks! Exactly what I was looking for.


Error: Range Not Satisfiable

Hi, I have been getting this error recently:
Error: Range Not Satisfiable
at new Error (native)
at Error.AppInsightsAsyncCorrelatedErrorWrapper
Any idea what could be wrong?
Thanks!

Do you have more of the stack trace? Seeing Error.AppInsightsAsyncCorrelatedErrorWrapper at the top isn't necessarily a sign that the error comes from this SDK, it's just because we wrap around exceptions to be able to collect them with the asynchronous context (to correlate with the correct incoming request).

@OsvaldoRosado thanks for prompt response, I attach more detailed stack trace, please note I removed some info (like folder names).
2017-06-09 18:35:03.507] [ERROR] console - Error: Range Not Satisfiable
at new Error (native)
at Error.AppInsightsAsyncCorrelatedErrorWrapper (/node_modules/applicationinsights/out/AutoCollection/CorrelationContextManager.js:141:18) []
at SendStream.error (/node_modules/send/lib/send.js:145:16) []
at SendStream.send (/node_modules/send/lib/send.js:370:19) []
at /node_modules/send/lib/send.js:322:10 []
at Zone.runTask (/node_modules/zone.js/dist/zone-node.js:151:47) [ => ]
at ZoneTask.invoke (/node_modules/zone.js/dist/zone-node.js:345:33) []
at data.args.(anonymous function) (node_modules/zone.js/dist/zone-node.js:1239:25) []
at FSReqWrap.oncomplete (fs.js:123:15) []

Looks like you might be running into a problem described about send.js here: https://stackoverflow.com/a/38499519 rather than with the AI SDK.
Thanks!

Closing as this doesn't look related to this SDK. Please feel free to re-open if this doesn't appear to be the case!


12.29.2020 action items

Additional context

update version number to v1.2.0
push version to CBP for posting on their server
create a release called "CBP 2019 Trends"
rename the existing release "CBP July 2020 Release" as "CBP 2018 Trends"

Renamed old release.
Some clean up from CMD check and lintr check.

Done.
Did some minor clean up (CMD check).  Added GitHub Actions for CMD check and pkgdown (adds new branch and autobuilds the weblink after each submittal).  Added some basic badges to the README so shows up on the website and on the main GitHub page.

Done.
Did some minor clean up (CMD check).  Added GitHub Actions for CMD check and pkgdown (adds new branch and autobuilds the weblink after each submittal).  Added some basic badges to the README so shows up on the website and on the main GitHub page.

Commit v1.2.0
https://github.com/tetratech/baytrendsmap/commit/cbe77d181b488cfa60e9b98f4f30a3b24fe99f7d
Release
https://github.com/tetratech/baytrendsmap/releases/tag/v1.2

Commit v1.2.0
https://github.com/tetratech/baytrendsmap/commit/cbe77d181b488cfa60e9b98f4f30a3b24fe99f7d
Release
https://github.com/tetratech/baytrendsmap/releases/tag/v1.2

Added new release with the additional color palette options.
https://github.com/tetratech/baytrendsmap/releases/tag/v1.2.1

Added new release with the additional color palette options.
https://github.com/tetratech/baytrendsmap/releases/tag/v1.2.1

concur


Re-instate TARGET_OS=linux in configure.ac

It is required for out of tree builds.

The result from the buildserver;
Bitcoin_Linux-static-64_186.txt
I'll close this because it doesn't build. If you fix this feel free to reopen the pull request.


Failed to find attribute guest

ENVIRONMENT

Operating System: Windows 7 64bits
Python version: 3.6.4
VirtualBox version: 5.2.6 r120293 (Qt5.6.2)
VirtualBox SDK version: 6.0.2-128162
Location where VirtualBox SDK is installed:
pyvbox version: 1.3.2
[x] Happens in latest master branch?

SUMMARY
Can't find attribute guest in virtualbox.library_ext.console.IConsole object, it was working before, now it ism't working anymore
STEPS TO REPRODUCE
Follow examples (with necessary changes)
EXPECTED RESULTS
Run normally
ACTUAL RESULTS
Traceback (most recent call last):
  File "<pyshell#8>", line 1, in <module>
    gs = session.console.guest.create_session('runner', '')
  File "C:\Users\User\AppData\Local\Programs\Python\Python36-32\lib\site-packages\virtualbox\library.py", line 14536, in guest
    ret = self._get_attr("guest")
  File "C:\Users\User\AppData\Local\Programs\Python\Python36-32\lib\site-packages\virtualbox\library_base.py", line 165, in _get_attr
    attr = self._search_attr(name, prefix='get')
  File "C:\Users\User\AppData\Local\Programs\Python\Python36-32\lib\site-packages\virtualbox\library_base.py", line 161, in _search_attr
    raise AttributeError("Failed to find attribute %s in %s" % (name, self))
AttributeError: Failed to find attribute guest in <virtualbox.library_ext.console.IConsole object at 0x03C2B690>

I no longer have time to maintain this library, so am closing this issue.


splunkclient-go: Do not use incorrect build tags

Overview
go build (and friends) include build tags for current and prior versions of go (i.e., go 1.21 has tags go1.21, go1.20, go1.19, ...). The existing go:build directive in splunkclient-go checks the for absence of tags go1.1 -> go1.16, which will always fail.
This change modifies the directive to indicate go 1.17 as the min version.
Also fixed lint and test failures. For the latter, removed attributed that were tested for but not returned from ClientRequest for semconv 1.17.
Testing
Before
❯ cd splunk-otel-go/instrumentation/k8s.io/client-go/splunkclient-go
❯ go test ./... -v
?   	github.com/signalfx/splunk-otel-go/instrumentation/k8s.io/client-go/splunkclient-go	[no test files]
?   	github.com/signalfx/splunk-otel-go/instrumentation/k8s.io/client-go/splunkclient-go/internal/config	[no test files]
?   	github.com/signalfx/splunk-otel-go/instrumentation/k8s.io/client-go/splunkclient-go/option	[no test files]
?   	github.com/signalfx/splunk-otel-go/instrumentation/k8s.io/client-go/splunkclient-go/transport	[no test files]

After
❯ go test ./... -v
?   	github.com/signalfx/splunk-otel-go/instrumentation/k8s.io/client-go/splunkclient-go	[no test files]
?   	github.com/signalfx/splunk-otel-go/instrumentation/k8s.io/client-go/splunkclient-go/internal/config	[no test files]
?   	github.com/signalfx/splunk-otel-go/instrumentation/k8s.io/client-go/splunkclient-go/option	[no test files]
=== RUN   TestNewWrapperFuncNilRoundTripper
--- PASS: TestNewWrapperFuncNilRoundTripper (0.00s)
=== RUN   TestRequestToSpanNameUnrecognized
--- PASS: TestRequestToSpanNameUnrecognized (0.00s)
=== RUN   TestRequestPathToSpanName
--- PASS: TestRequestPathToSpanName (0.00s)
=== RUN   TestRequestMethodToSpanName
--- PASS: TestRequestMethodToSpanName (0.00s)
=== RUN   TestWrappedBodyRead
--- PASS: TestWrappedBodyRead (0.00s)
=== RUN   TestWrappedBodyReadEOFError
--- PASS: TestWrappedBodyReadEOFError (0.00s)
=== RUN   TestWrappedBodyReadError
--- PASS: TestWrappedBodyReadError (0.00s)
=== RUN   TestWrappedBodyClose
--- PASS: TestWrappedBodyClose (0.00s)
=== RUN   TestWrappedBodyCloseError
--- PASS: TestWrappedBodyCloseError (0.00s)
=== RUN   TestWrappedRoundTripperError
--- PASS: TestWrappedRoundTripperError (0.00s)
PASS
ok  	github.com/signalfx/splunk-otel-go/instrumentation/k8s.io/client-go/splunkclient-go/transport	(cached)

fixes https://github.com/signalfx/splunk-otel-go/issues/2890

I have read the CLA Document and I hereby sign the CLA

recheck

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

Comparison is base (29298b1) 78.34% compared to head (d6de4fc) 80.04%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2891      +/-   ##
==========================================
+ Coverage   78.34%   80.04%   +1.70%
==========================================
  Files          85       88       +3
  Lines        3293     2897     -396
==========================================
- Hits         2580     2319     -261
+ Misses        639      502     -137
- Partials       74       76       +2

Flag
Coverage Δ

Linux
80.04% <100.00%> (+0.59%)
:arrow_up:

Windows
?

macOS
?

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

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

@andrewwdye thank you for the contribution! Please sign your commits. We required signed commits for the project to ensure authenticity.

I rebased with my commits signed (and @pellared 's commits signed as co-authored). I assume there is another step here to co-sign but was unsure


refactor: replaced atproto to xrpc (#574)

1. Description

1.1. Checklist

[ ] The title of my PR starts with a Conventional Commit prefix (fix:, feat:, docs: etc).
[ ] I have read the Contributor Guide and followed the process outlined for submitting PRs.
[ ] I have updated/added tests for ALL new/updated/fixed functionality.
[ ] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
[ ] I have updated/added relevant examples in examples.

1.2. Breaking Change

[ ] Yes, this is a breaking change.
[ ] No, this is not a breaking change.

1.3. Related Issues

Codecov Report

Merging #585 (01328b5) into main (00488f3) will increase coverage by 8.91%.
The diff coverage is 87.50%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #585      +/-   ##
==========================================
+ Coverage   69.46%   78.38%   +8.91%
==========================================
  Files         213       33     -180
  Lines        2250      606    -1644
==========================================
- Hits         1563      475    -1088
+ Misses        687      131     -556

Impacted Files
Coverage Δ

packages/bluesky_text/lib/src/entities/entity.dart
77.27% <75.00%> (-3.68%)
:arrow_down:

packages/bluesky_text/lib/src/api/find_did.dart
100.00% <100.00%> (ø)

... and 180 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Supports configuring the log level without exiting the program

In a production environment, the log level is usually set to a lower level. However, we also need to see more detailed log information when debugging a issue, but don't want to quit the program in order to protect the current context.
So I think we should support configuring the log level without exiting cri-containerd.

@miaoyq How does other system usually support this? Add an API, or use signal, or something?

@Random-liu Use signal like dumpstack, docker support reload config file via SIGHUB signal that contains log level setting.

@Random-Liu We should support setting log level at least.

interesting.. not against dynamic log level.. but it's pretty easy to cycle cri-containerd