Detached process on Windows dies when parent process ends

In the following case a parent process creates a child process and exits. The child process is detached and supposed to complete some work. On Linux / OS X the code works fine while on Windows the child process dies. Only if the parent process uses process.wait it will work on Windows (obviously preventing the parent process to exit):
require 'childprocess'

class ProcessTest
  def self.start

    # Build the command line for the new process
    process = ChildProcess.build('ruby', "#{__FILE__}", 'run')

    # Set the working directory for the process
    process.cwd = Dir.pwd

    # Make sure process gets a new group
    process.leader = true

    # Make sure the process keeps running
    process.detach = true

    # Start the process
    process.start

    # Record the process pid
    File.open('test.pid', 'w') {|f| f << process.pid.to_s}

    #process.wait <= Needed on Windows otherwise child process dies when parent exits
  end

  def self.run
    i = 0
    while true do
      @log.write  "Inside the loop i = #{i}\n"
      i +=1
    end
  end

  if __FILE__ == $0
    if ARGV[0] == 'start'
      puts 'Parent process started'
      ProcessTest.start
      puts 'Parent process finished'
    elsif ARGV[0] == 'run'
      @log = File.open("test.log", 'w+')
      @log.write "Child process started\n"
      ProcessTest.run
      @log.write "Child process finished\n"
      @log.close unless @log.nil?
    else
      puts "Unexpected argument #{ARGV[0]}"
    end

  end
end

Run via ruby process_test.rb start.
Environment:

Microsoft Windows 10
ruby 2.0.0p647
childprocess 0.5.8

This could be related to the change made in #76 that we're looking to fix in #96.
Could you try to downgrade to childprocess 0.5.2, run the test again and report back? What version you're running can be checked using this code:
require 'childprocess/version'
p ChildProcess::VERSION

Could you try to downgrade to childprocess 0.5.2

AFAICT the issue still exists. Here is what I did - https://gist.github.com/hferentschik/338b8a35b13a9ba31dda
I am running bundle exec ruby ./childprocess-bug.rb start


ELEMENT_INDEX_TABLE_KEYS and ELEMENT_RESULT_NCOMP within class

As discussed in issue:
Definition of ELEMENT_INDEX_TABLE_KEYS and ELEMENT_RESULT_NCOMP

Nice work!  Looks like you figured the PR part.
Merging.  I'll generate a hot-fix for this so you can have a release.  Be on the lookout for ansys-mapdl-reader==0.51.3


#9: Allow custom tar archive path

Ok, finally had time to come back to this. @Justintime50 I think the existing tests (once modified) pretty much covers things, since test_formula.py just passes data directly to Formula.generate_formula_data(). We could add some sort of test in test_app.py that checks the case where TRUE is based for URL generation (which should auto-generate the path) and the case where a custom upstream URL is passed, is that something you want?

Yeah the problem I'm seeing is we currently check if the param is a string which is always true in this case. GitHub passes all the params of the action to the code as a string so I unfortunately have to convert true/false strings to True/False bools in Python to get around this problem. We may instead want to do one of the following:

Change the check to state if VAR not in set(True, False): so that it will still respect the current true/false logic
OR introduce 4 new params which can take in a custom path. If this option is used, you'd basically set which targets you want to True then you would specify their custom paths via these new params.

I'm not sure which of the two is better... Thoughts?
As for the tests, what I'm expecting is for a new test to be written that creates a new formula with the custom paths. The tests don't currently assert the output formula actually match expectations - it's meant more for visual inspection. Thinking through this though I'll probably adjust that here shortly to actually assert some data to protect against regressions. Then the current test output files I'd expect to remain the same since we don't want to disturb the current true/false default behavior.
TLDR: I'm expecting a new file to be generated in the test/formulas folder which will occur if you create a new test and use the _record_formula function - other formula tests can serve as a template on how to do this.

I added better information about tests to #23, hopefully this helps clarify what I meant above about tests ^.
I'd still like to see a test where we can prove this change doesn't break the current behavior as I feel the way it's currently written will have that effect.

I'm going to convert this to a draft until the above feedback has been addressed. Happy to take another look once that is done.

I'm going to close this PR for lack of activity, feel free to reopen if you're able to address the feedback I gave earlier. You will also want to rebase against the main branch since there were many additions there since this PR went up.
The biggest hurdle to this functionality I see is how can you define a custom URL template when pieces of the URL (such as name or version number) are dynamic on each release? It's much easier to define a well-defined standard template that all releases can follow (basically backwards which is how this is currently working) so that the user doesn't have to deal with that.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_8fh0g311eb/clone/recipe/meta.yaml:21: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_8fh0g311eb/clone/recipe/meta.yaml:32: missing_pip_check: For pypi packages, pip check should be present in the test commands
INFO: /tmp/abs_8fh0g311eb/clone/recipe/meta.yaml:43: invalid_url: https://googleapis.dev/python/iamcredentials/latest/index.html : URL domain redirect googleapis.dev ->  cloud.google.com
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_e4cghzcpdz/clone/recipe/meta.yaml:21: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_e4cghzcpdz/clone/recipe/meta.yaml:32: missing_pip_check: For pypi packages, pip check should be present in the test commands
Errors were found


Generate API and controller, and add basic Travis CI

Generated API and controller to watch SSP resource.
Added basic .travis.yml.

@omeryahud @ksimon1 @vatsalparekh

@omeryahud , can you enable Travis for this repo?

@omeryahud , can you enable Travis for this repo?

Done, according to documentation travis should be working once the .travis.yml file is merged.


Ldap requests that require signing

It does not appear that LDAP signing is supported for simple binds.  Can support for this be added?  This is commonly configured in more recent Active Directory installs as described here.  https://support.microsoft.com/en-us/help/4520412/2020-ldap-channel-binding-and-ldap-signing-requirements-for-windows
Here is the error received when connecting to AD servers with this requirement.
StrongAuthRequiredError: 00002028: LdapErr: DSID-0C090266, comment: The server requires binds to turn on integrity checking if SSL\TLS are not already active on the connection, data 0, v4563

Would you like to submit a pull request to add this feature? Please remember to add unit tests.

@jlieman do you have a workaround? I bumped into the same error. Thanks

@jlieman do you have a workaround? I bumped into the same error. Thanks

It would be really nice to have LDAP signing support since it is a recommended security update from Microsoft.

It would be really nice to have LDAP signing support since it is a recommended security update from Microsoft.

@kushwahav PRs are always welcome 😁👍

@kushwahav PRs are always welcome 😁👍

I wouldn't mind coding this, but I have no clue where to begin. I imagine there is some API on the LDAP server that needs to be used, but I can't find the docs for it. The above-linked KB doc deals with server configuration, not coding. Nor does it link to anything that would help a programmer, so far as I can tell. Thoughts?

https://ldapwiki.com/wiki/LDAP Signing via https://duckduckgo.com/?q=ldap+signing

👋
On February 22, 2023, we released version 3 of this library. As a result, we are closing this issue/pull request.
Please see issue #839 for more information, including how to proceed if you feel this closure is in error.


Minor fix: optimize var usage

thx @zuston !
TBH, this sample needs a quite big refactor/redisign, will do it probably soon.

Thanks for your great work on this project.
Got it. I'll close it. Right? @csviri

Thanks for your great work on this project.
Got it. I'll close it. Right? @csviri

thx! yes, pls

Merged it, for now, thx!


Search, content creation, ancestor child relationships, etc.

First off, thanks for starting this library. I need to create content, so I looked through the forks, and here was a fork with the functionality I'd like to use. I figured I'd bring it to your attention by way of a pull request in case you find it worth merging in. Thanks!

Thanks, looks interesting 👍
To be honest I don't actively use this library anymore myself (though I still use Confluence, so maybe I'll find an new use-case for it one day 😀).
I'll take a look at these new features and merge them when I find some time this week.

Thanks for the quick response. I'm primarily using the content creation and ancestry stuff right now. There's a few rough edges in the code, to be sure, but I have a future use case for the search too. If you look it over but decide you don't want to merge it wholesale, I might be able to part it out and clean it up a bit. Thanks!

Any plans to get it merged?
I would also like to have content creation functionality.

I didn't find the time yet to properly review and test this, I'm currently a bit busy with some other projects. As stated before, I don't maintain the project this library was made for anymore.
From first glance I already spotted some issues with this PR, meaning I won't merge it as-is:

I don't really like the prints to stdout (I understand it's useful while debugging, but I think it should be removed or wrapped in a logging library to prevent confusion when it is used in production).
I don't think returning a non-nill response when an error occurred is a good idea.

I'm planning to use this functionality for an app I'm working on, so I have an interest in getting this functionality in either here, or in my fork. I'll comment here when I have that done. It's a free-time project, so no timelines here. Thanks all!


feat(doc): revert "feat(doc): add --download-artifacts option to docker-installation.md"

Reverts autowarefoundation/autoware-documentation#556
ref. https://github.com/autowarefoundation/autoware/pull/4864

@oguzkaganozt Please review it.


Completely fails when using preprocessing

See this repo for a minimal reproduction, just clone, install, and run tests. Full details for recreating the repo from scratch are included in the README.
The gist of it is, when I use the preprocessing flag, I get the following output:
 FAIL  src/routes/index-dom.spec.js
  ● Test suite failed to run

    TypeError: The "id" argument must be of type string. Received an instance of URL

      at node_modules/.pnpm/svelte-jester@2.1.5_jest@27.4.5+svelte@3.44.3/node_modules/svelte-jester/dist/transformer.cjs:79:118

Test Suites: 1 failed, 1 total
Tests:       0 total
Snapshots:   0 total
Time:        1.534 s, estimated 2 s
Ran all test suites matching /src/i.
 ERROR  Test failed. See above for more details.

I think the problem is NODE_OPTIONS=--experimental-vm-modules when running jest, because the call of svelte-jester in jest.config.js uses the transformer.cjs which uses require and this fails.
My workaround in jest.config.js:
transform: {
  "^.+\\.svelte$": [
    "./node_modules/svelte-jester/dist/transformer.mjs",
    {
      "preprocess": true
    }
  ]
},

@binarious Works for me. Leaving this issue open for prosperity until this solution is either unnecessary or documented somewhere more accessible.

Should be fixed now 😄  Let us know if you run into any issues.

Maybe that's another bug, but when I change ./node_modules/svelte-jester/dist/transformer.mjs back to svelte-jester I get the following error:

Cannot find module 'file:///home/myuser/projects/myvendor/myapp/svelte.config.js'
Require stack:
- /home/myuser/projects/myvendor/myapp/node_modules/.pnpm/svelte-jester@2.3.1_jest<EMAIL_ADDRESS>


Michael/dashboard issue fix 992

🔗 Issue
https://github.com/rubyforgood/pet-rescue/issues/992
✍ Description
I have edited the relevant foster and adoption application files under staff to have the same HTML classes which should solve the sizing issue. Below you are able to see the pages layered on top of each other as a demo.
📷 Screenshots/Demos

Thank you! Could you please provide a video or screenshots of the UI changes? Helps with review a lot!

Hi! Thank you for replying, the test cases should now be passing, here are some more screenshots comparing the before and after shots of the UI changes I made.
Before :

After:


Add getQuery function

[x] Changes have been thoroughly tested to not break existing functionality.
[x] New functionality has been documented or existing documentation has been updated to reflect changes.
[x] Visual changes are explained in the PR description using a screenshot/recording of before and after.

Pros:

I'm working on a big project where i've many auth models user, admin, patient, doctor, nurse and many more, instead of defining the inverse relation for each model this PR makes it seamless to work with relationships.

Doesn't override lines 314-316 which will break multi-tenancy.

Cons:

On edit page it breaks and need to specify qualified key name protected static ?string $recordRouteKeyName = 'analyses.id'; and this problem exists even when using with getEloquentQuery.

For hasMany relations
Before:
// Patient -> hasMany -> Prescription
public static function getEloquentQuery(): Builder
{
    return parent::getEloquentQuery()
      ->whereBelongsTo(auth('patient')->id());
}

After
// Cleaner
public static function getQuery(): Builder {
    return auth('patient)->user()
      ->prescriptions()
      ->getQuery();
}

For hasManyThrough relations
Before:
// Patient -> hasMany -> Analyses -> through -> AnalysisRequest
public static function getEloquentQuery(): Builder
{
    return parent::getEloquentQuery()
      ->whereRelation('analysis_request', 'patient_id', auth('patient')->id());
}

After
// Cleaner
public static function getQuery(): Builder {
    return auth('patient')->user()
        ->analyses()
        ->getQuery();
}

Unfortunately I don't think there is a good name for this method, so I suggest you just add the tenancy scope to your override


Relation to peer-to-peer Colyseus?

Hi,
I've found this project by googling stuff related to this paper. I gave the source code a quick glance and it doesn't seem like this is an implementation of the network described in the paper. Is this an unfortunate name clash?

Hi @bgr, yes, looks like a name clash. This project does not handle peer-to-peer connections. Cheers!

Cool. You might want to add a note to the readme about that. Thanks.


Auto close popup when hit submit

Hi, When I add something I have to wait and then close the popup any way to fix this?

This would interest me too!

See #248. It appears to be related to the mail setup.


Issue 221 add metrics doc in json

@durack1 @gleckler1 @jservonnat this adds metrics info at bottom of files.
We can further tweaks the dictonaries keywords, etc.. but that's the idea. Please review and comment

@doutriaux1 is this all good to go?

@durack1 care to merge it now that it's rebased?


az storage account generate-sas doesnt create token for table while in powershell it does

az storage account generate-sas --account-name ${azure_billing_storage_account_name} --account-key $key1 --https-only --expiry 2018-12-31T23:59Z --permissions ralcup --resource-types sco --services qt

creates following sas:
"se=2018-12-31T23%3A59Z&sp=rlacup&spr=https&sv=2017-04-17**&ss=q**&srt=sco&sig=...."
no matter what I tries ss param does NOT include table.
I need a 1 SAS to publish to external service which should include queue and table, so creating sas on table is not a valid workaround for me!
In powershell same command works fine and create SAS with table and queue. The problem is that I use az-cli as part of linux playbook so I must have it
Please help, it currently blocks development
Thanks

Environment summary
Install Method (e.g. pip, interactive script, apt-get, Docker, MSI, edge build) / CLI version (az --version) / OS version / Shell Type (e.g. bash, cmd.exe, Bash on Windows)
azure-cli (2.0.26)
acr (2.0.20)
acs (2.0.25)
advisor (0.1.1)
appservice (0.1.25)
backup (1.0.6)
batch (3.1.9)
batchai (0.1.5)
billing (0.1.7)
cdn (0.0.12)
cloud (2.0.12)
cognitiveservices (0.1.10)
command-modules-nspkg (2.0.1)
component (2.0.8)
configure (2.0.13)
consumption (0.2.1)
container (0.1.16)
core (2.0.26)
cosmosdb (0.1.17)
dla (0.0.18)
dls (0.0.19)
eventgrid (0.1.9)
extension (0.0.8)
feedback (2.0.8)
find (0.2.8)
interactive (0.3.15)
iot (0.1.16)
keyvault (2.0.17)
lab (0.0.16)
monitor (0.1.1)
network (2.0.22)
nspkg (3.0.1)
profile (2.0.18)
rdbms (0.0.11)
redis (0.2.11)
reservations (0.1.1)
resource (2.0.22)
role (2.0.17)
servicefabric (0.0.9)
sql (2.0.20)
storage (2.0.24)
vm (2.0.25)
Python location '/opt/az/bin/python3'
Extensions directory '/home/ashapira/.azure/cliextensions'
Python (Linux) 3.6.1 (default, Feb  5 2018, 21:31:01)
[GCC 5.4.0 20160609]

@shapshapOne, Just tried:
az storage account generate-sas --permissions cdlruwap --account-name wilxremove --services qt --resource-types sco --expiry `date -d "30 minutes" '+%Y-%m-%dT%H:%MZ'`

and got the following:
"se=2018-08-27T18%3A04Z&sp=rwdlacup&sv=2018-03-28&ss=qt&srt=sco&sig=..."
ss=qt
Can you try this again with our latest version?

Closing due to inactivity


Add a day limit for the love command

Oh and after creating a new slot, update it to add the love point. Btw, return an error if the mentionned user is the author

Done, switched from SQLite to JSON cuz i dont like SQLite syntax lul


TypeError: Super expression must either be null or a function, not undefined

Can't figure this out.  I've done this many times, updated react projects from 15 to 16 including enzyme to the 16 adaptor and it's been fine.  But for this project it's complaining.  I updated react from 15 to 16.  When I try running my mocha test for the first time I get:
function _inherits(subClass, superClass) { if (typeof superClass !== "function" && superClass !== null) { throw new TypeError("Super expression must either be null or a function, not " + typeof superClass); } subClass.prototype = Object.create(superClass && superClass.prototype, { constructor: { value: subClass, enumerable: false, writable: true, configurable: true } }); if (superClass) Object.setPrototypeOf ? Object.setPrototypeOf(subClass, superClass) : subClass.__proto__ = superClass; } /* eslint no-use-before-define: 0 */
   at _inherits (/Users/daveschinkel/code/projects/consulting/diamond/fox/dcg-web-app/dcg-shared-react/node_modules/enzyme-adapter-react-16/build/ReactSixteenAdapter.js:45:113)
    at /Users/daveschinkel/code/projects/consulting/diamond/fox/dcg-web-app/dcg-shared-react/node_modules/enzyme-adapter-react-16/build/ReactSixteenAdapter.js:190:3
    at Object.<anonymous> (/Users/daveschinkel/code/projects/consulting/diamond/fox/dcg-web-app/dcg-shared-react/node_modules/enzyme-adapter-react-16/build/ReactSixteenAdapter.js:455:2)
    at Module._compile (module.js:569:30)

                                                                                ^

TypeError: Super expression must either be null or a function, not undefined
I've setup enzyme like I always do which has worked fine in other react projects:
import React from 'react'
import Enzyme from 'enzyme';
import chai, { expect } from 'chai';
import { shallow, mount } from 'enzyme';
import Adapter from 'enzyme-adapter-react-16';

Enzyme.configure({ adapter: new Adapter() });

package.json
{
  "name": "",
  "author": "",
  "version": "1.7.18",
  "description": "",
  "main": "lib/index.js",
  "files": [
    "lib/",
    "README.md"
  ],
  "engines": {
    "node": "6.x"
  },
  "directories": {},
  "repository": {
    "type": "git",
    "url": "xxxxxxxxxx"
  },
  "dependencies": {
    "axios": "^0.15",
    "babel-core": "^6.7.7",
    "babel-eslint": "^6.0.4",
    "babel-loader": "^6.2.4",
    "babel-plugin-webpack-loaders": "^0.7.1",
    "classnames": "^2.2.5",
    "cleave.js": "1.0.2",
    "hls.js": "^0.6.18",
    "lodash": "^4.16.1",
    "moment": "2.15.2",
    "postcss-import": "^9.1.0",
    "react": "^16.0.0",
    "react-dom": "^16.0.0",
    "react-cookie": "^1.0.4",
    "react-css-themr": "^1.3.0",
    "react-dotdotdot": "^1.0.17",
    "react-lazy-load": "^3.0.11",
    "react-markdown": "^2.4.4",
    "react-redux": "^4.4.5",
    "react-router": "^2.4.0",
    "react-router-redux": "^4.0.4",
    "react-slider": "^0.9.0",
    "react-timeout": "^1.0.1",
    "react-truncate": "^2.1.1",
    "redux": "^3.5.2",
    "scriptjs": "^2.5.8",
    "sinon": "^1.17.6",
    "validator": "^6.2.1",
    "webpack": "^2.3.2",
    "xml2js": "^0.4.17"
  },
  "devDependencies": {
    "babel-cli": "^6.26.0",
    "babel-core": "^6.14.0",
    "babel-loader": "^6.2.5",
    "babel-plugin-transform-assets": "^0.2.0",
    "babel-plugin-transform-runtime": "^6.23.0",
    "babel-preset-env": "1.5.1",
    "chai": "^4.1.2",
    "chai-enzyme": "^0.8.0",
    "css-modules-require-hook": "^4.0.6",
    "documentation": "^4.0.0-beta5",
    "enzyme": "^2.4.1",
    "enzyme-adapter-react-16": "^1.1.0",
    "eslint": "^3.18.0",
    "eslint-config-airbnb": "^14.1.0",
    "eslint-plugin-import": "^2.2.0",
    "eslint-plugin-jsx-a11y": "^4.0.0",
    "eslint-plugin-promise": "^3.5.0",
    "eslint-plugin-react": "^6.10.3",
    "expect": "^1.20.2",
    "expect-jsx": "^2.6.0",
    "jsdom": "^11.5.1",
    "mocha": "^4.0.1",
    "postcss": "^5.0.21",
    "postcss-modules-resolve-path": "^1.0.0",
    "postcss-use": "^2.2.0",
    "react-addons-test-utils": "^15.0.1",
    "react-styleguidist": "^2.3.1",
    "stylelint": "^6.7.0"
  }
}

This title is a duplicate of #1172. Is it perhaps the same issue?
Generally, this often occurs when there's a circular dependency somewhere; I'm not sure where there would be one here.

yea I took a look at that issue as well.  Let me investigate more.  thx

eh just needed to install "babel-core".  thought I did already but didn't.


build: require Go 1.19, gofmt comments

Hi @adamdecaf apologies for letting this sit idle, I did not see the Pull Request when it came in and just saw this as I was making some updates. Appreciate the contribution!


Fix clientID header

Codecov Report

Merging #41 (08edc88) into main (99d13d9) will increase coverage by 0.64%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #41      +/-   ##
==========================================
+ Coverage   78.82%   79.46%   +0.64%
==========================================
  Files           6        6
  Lines         255      263       +8
==========================================
+ Hits          201      209       +8
  Misses         38       38
  Partials       16       16

Impacted Files
Coverage Δ

internal/http/authenticate/authenticate.go
79.41% <100.00%> (+2.74%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Replace kafka-python with kafka-python-ng on Jupyter notebooks

Related to: https://issues.redhat.com/browse/RHOAIENG-4475
Description
Replace kafka-python with kafka-python-ng as it is the same api but from different working fork: https://github.com/wbarnha/kafka-python-ng
How Has This Been Tested?

Spin up a jupyter notebook generated from this PR
podman run -it -p 8888:8888 ghcr.io/atheo89/notebooks/workbench-images:jupyter-trustyai-ubi9-python-3.11-RHOAIENG-4475_dad8405dfc6b16ff6318147ce47ce656b479056e
Run the following mockup code to test kafka python api functionality

from kafka import KafkaProducer, KafkaConsumer
from unittest import mock
import json

# Mocking KafkaProducer to simulate sending a message
@mock.patch('kafka.producer.KafkaProducer.send')
@mock.patch('kafka.producer.KafkaProducer.__init__', return_value=None)
def test_producer(mock_producer_init, mock_send):
    # Create a mock producer instance
    producer = KafkaProducer(value_serializer=lambda v: json.dumps(v).encode('utf-8'))

    # Simulate sending a message
    test_message = {'key': 'value'}
    producer.send('test-topic', value=test_message)

    # Assert that the send method was called with the correct topic and value
    mock_send.assert_called_once_with('test-topic', value=test_message)
    print("Producer test passed: Message sent successfully.")

# Mocking KafkaConsumer to simulate receiving a message
@mock.patch('kafka.consumer.KafkaConsumer.__iter__', return_value=iter([mock.Mock(value=json.dumps({'key': 'value'}).encode('utf-8'))]))
@mock.patch('kafka.consumer.KafkaConsumer.__init__', return_value=None)
def test_consumer(mock_consumer_init, mock_consumer_iter):
    # Create a mock consumer instance
    consumer = KafkaConsumer(value_deserializer=lambda v: json.loads(v.decode('utf-8')))

    # Simulate consuming a message
    for message in consumer:
        print(f"Consumer test passed: Received message - {message.value}")
        break

if __name__ == '__main__':
    test_producer()
    test_consumer()

expected output
Producer test passed: Message sent successfully.
Consumer test passed: Received message - {'key': 'value'}

Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

/lgtm

/override ci/prow/rocm-notebooks-e2e-tests
/override ci/prow/notebooks-ubi9-e2e-tests
/override ci/prow/images


Incorrect Visual Studio 2017 generator

While looking through the code I observed that the visual_studio_generator function sets an incorrect generator for the VS "15.0" version(I was using CMake 3.8-rc and observed that it has a different generator).
Here is the list of generators that 3.8.0 reports on this Windows machine:
The following generators are available on this platform:
  Visual Studio 15 2017 [arch] = Generates Visual Studio 2017 project files.
                                 Optional [arch] can be "Win64" or "ARM".
  Visual Studio 14 2015 [arch] = Generates Visual Studio 2015 project files.
                                 Optional [arch] can be "Win64" or "ARM".
  Visual Studio 12 2013 [arch] = Generates Visual Studio 2013 project files.
                                 Optional [arch] can be "Win64" or "ARM".
  Visual Studio 11 2012 [arch] = Generates Visual Studio 2012 project files.
                                 Optional [arch] can be "Win64" or "ARM".
  Visual Studio 10 2010 [arch] = Generates Visual Studio 2010 project files.
                                 Optional [arch] can be "Win64" or "IA64".
  Visual Studio 9 2008 [arch]  = Generates Visual Studio 2008 project files.
                                 Optional [arch] can be "Win64" or "IA64".
  Visual Studio 8 2005 [arch]  = Generates Visual Studio 2005 project files.
                                 Optional [arch] can be "Win64".
  Visual Studio 7 .NET 2003    = Deprecated.  Generates Visual Studio .NET
                                 2003 project files.
  Borland Makefiles            = Generates Borland makefiles.
  NMake Makefiles              = Generates NMake makefiles.
  NMake Makefiles JOM          = Generates JOM makefiles.
  Green Hills MULTI            = Generates Green Hills MULTI files
                                 (experimental, work-in-progress).
  MSYS Makefiles               = Generates MSYS makefiles.
  MinGW Makefiles              = Generates a make file for use with
                                 mingw32-make.
  Unix Makefiles               = Generates standard UNIX makefiles.
  Ninja                        = Generates build.ninja files.
  Watcom WMake                 = Generates Watcom WMake makefiles.
  CodeBlocks - MinGW Makefiles = Generates CodeBlocks project files.
  CodeBlocks - NMake Makefiles = Generates CodeBlocks project files.
  CodeBlocks - NMake Makefiles JOM
                               = Generates CodeBlocks project files.
  CodeBlocks - Ninja           = Generates CodeBlocks project files.
  CodeBlocks - Unix Makefiles  = Generates CodeBlocks project files.
  CodeLite - MinGW Makefiles   = Generates CodeLite project files.
  CodeLite - NMake Makefiles   = Generates CodeLite project files.
  CodeLite - Ninja             = Generates CodeLite project files.
  CodeLite - Unix Makefiles    = Generates CodeLite project files.
  Sublime Text 2 - MinGW Makefiles
                               = Generates Sublime Text 2 project files.
  Sublime Text 2 - NMake Makefiles
                               = Generates Sublime Text 2 project files.
  Sublime Text 2 - Ninja       = Generates Sublime Text 2 project files.
  Sublime Text 2 - Unix Makefiles
                               = Generates Sublime Text 2 project files.
  Kate - MinGW Makefiles       = Generates Kate project files.
  Kate - NMake Makefiles       = Generates Kate project files.
  Kate - Ninja                 = Generates Kate project files.
  Kate - Unix Makefiles        = Generates Kate project files.
  Eclipse CDT4 - NMake Makefiles
                               = Generates Eclipse CDT 4.0 project files.
  Eclipse CDT4 - MinGW Makefiles
                               = Generates Eclipse CDT 4.0 project files.
  Eclipse CDT4 - Ninja         = Generates Eclipse CDT 4.0 project files.
  Eclipse CDT4 - Unix Makefiles= Generates Eclipse CDT 4.0 project files.

Thanks for the report! Is the fix here just to add the year?

Yes.
I have access to a Windows machine at work, I'll reconfirm today if this fixes the issue(or if it is necessary at all).

I see one more issue.
https://github.com/alexcrichton/cmake-rs/blob/master/src/lib.rs#L516 : it handles only traditional PC architectures and panics otherwise.
The CMake help indicates that you could pass "ARM" as an architecture.

I just tried it and seems like passing Visual Studio 15 to CMake is also ok(in messages, the generator is transformed into Visual Studio 15 2017).
The issue with ARM still remains.
I get an error that there is no install.vcxproj but that's probably because the project I am trying on, doesn't have an install target(I don't know what kind of file is generated for the install target on MSBuild).
At least for the default target, Makefiles have all and MSBuild has ALL_BUILD, so that's why I am not sure what kind of target would be generated for install.

Ah yeah I'm fine adding support for ARM as well, although we don't have an arm windows target upstream yet in rust-lang/rust so it wouldn't be very usable today.


.bower.json file created which is generating an error on bower update

D:\myapp >bower update {bower package}
bower EMALFORMED **   Failed to read D:myapp\bower_components\CodeMirror**.bower.json
Additional error details:
Unexpected token .
I don't know why .bower.json file is created although there is a valid bower.json file. the dot infront of bower.json file is throwin an error which says unexpected token.
I have checked the json format in bower.json as well as package.json. Is there anyone who's getting this same issue?

The .bower.json file isn't in the repository, and is not something CodeMirror is creating.


Log-Eintrag mit js-Controller 3.3.x

Nach Update des js-Controllers wird nachfolgendes im Log angezeigt.
proxmox.0 | 2021-04-30 21:33:23.351 | warn | (5832) State value to set for "proxmox.0.lxc_ioBroker-Rock.pid" has wrong type "string" but has to be "number"
-- | -- | -- | --
proxmox.0 | 2021-04-30 21:33:23.305 | warn | (5832) State value to set for "proxmox.0.lxc_MotionEye.pid" has wrong type "string" but has to be "number"
proxmox.0 | 2021-04-30 21:33:23.238 | warn | (5832) State value to set for "proxmox.0.lxc_Unifi-Controller.pid" has wrong type "string" but has to be "number"
proxmox.0 | 2021-04-30 21:33:23.211 | warn | (5832) State value to set for "proxmox.0.lxc_PIHole.pid" has wrong type "string" but has to be "number"
proxmox.0 | 2021-04-30 21:33:23.060 | warn | (5832) State value to set for "proxmox.0.node_pve.status" has wrong type "string" but has to be "boolean"
proxmox.0 | 2021-04-30 21:32:53.343 | warn | (5832) State value to set for "proxmox.0.lxc_ioBroker-Rock.pid" has wrong type "string" but has to be "number"
proxmox.0 | 2021-04-30 21:32:53.306 | warn | (5832) State value to set for "proxmox.0.lxc_MotionEye.pid" has wrong type "string" but has to be "number"
proxmox.0 | 2021-04-30 21:32:53.249 | warn | (5832) State value to set for "proxmox.0.lxc_Unifi-Controller.pid" has wrong type "string" but has to be "number"
proxmox.0 | 2021-04-30 21:32:53.212 | warn | (5832) State value to set for "proxmox.0.lxc_PIHole.pid" has wrong type "string" but has to be "number"
proxmox.0 | 2021-04-30 21:32:53.063 | warn | (5832) State value to set for "proxmox.0.node_pve.status" has wrong type "string" but has to be "boolean"
proxmox.0 | 2021-04-30 21:32:23.416 | warn | (5832) State value to set for "proxmox.0.lxc_PIHole.pid" has wrong type "string" but has to be "number"
proxmox.0 | 2021-04-30 21:32:23.319 | warn | (5832) State value to set for "proxmox.0.lxc_ioBroker-Rock.pid" has wrong type "string" but has to be "number"
proxmox.0 | 2021-04-30 21:32:23.270 | warn | (5832) State value to set for "proxmox.0.lxc_MotionEye.pid" has wrong type "string" but has to be "number"

Should be fixed with current version


Deprecate get_data and get_values methods in dataset

This PR deprecates the methods get_data and get_values from dataset in favor of get_parameter_data.
@Dominik-Vogel

@astafan8 do you know of a fast way of looking into docs for the usage? Maybe I can use grep..Not sure.

@astafan8 On the other hand, I think we should get rid off of them in docs after we retire.

do you know of a fast way of looking into docs for the usage? Maybe I can use grep..Not sure.

@GateBuilder just use "find everywhere" feature of your editor, both pycharm and vs code have it. grep is also OK, but i prefer pycharm's tool for convenience.
since we are deprecating these, we want to make sure that the docs don't mention/recommend its usage (also, as per our discussion offline).


Create agent for Ollama integration

Hi all! This PR adds integration with Ollama through langchain #375 . It also bumps the langchain version to make the integration possible.
Since llamacpp also has langchain integration, I based the Ollama agent on the llamacpp agent. There are a lot of commonalities between the two agents, so maybe a generic "langchain-based" agent could be useful to inherit from. I could try to take on that refactor if it is useful/helpful.
I've tested locally—connecting to an Ollama server on my M1 Mac using WhisperCPP and Azure. It works, but it's a bit clunky. I haven't written any unit tests.
This is a work in progress and I would love comments/feedback—thanks!

@j-sneh Can this version use any model with Ollama? I'd like to try with the new Llama 3


[orgmode]Error while highlighting: UnicodeEncodeError: 'ascii' codec can't encode characters

In my post.org file, there is a source block which have no-english character.
#+BEGIN_SRC shell
# 重进进入fastboot模式
adb reboot bootloader
# 查询bootloader状态（是否锁住）
fasboot oem get-bootinfo
#+END_SRC

So the parse error log:
*** Error while highlighting: UnicodeEncodeError: 'ascii' codec can't encode characters in position 24-29:
ordinal not in range(128) (file "/Users/myname/source/myblog.io/bin/../lib/python3.6/codecs.py", line 376, in write)
*** If this is a bug you want to report, please rerun with -v.

Are your locales set correctly (to something that supports UTF-8, that is installed on your system)?
Could you test the solutions here or here?

Add
(setq default-process-coding-system '(utf-8 . utf-8))
in my dot file, solved.
Thank you very much!


📦 Release

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
<EMAIL_ADDRESS>Patch Changes

maps: include type declarations ( #1362)

brand-icons: include type declarations ( #1362)

icon: include type declarations ( #1362)

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

:rocket: Welcome to Baloise ChatOps based GitOps Bot :tada:
- If you want to enable preview environments for this PullRequest, comment with `/preview`.
- If you want to disable the preview environment for this PR, comment with `/undeploy`.
- If you need help or want to see the docs, comment with `/help`.

Make sure to refresh this page after submitting a command.
Preview environments can be redeployed automatically if you enable the webhook for updates to the source branch. :new:

Have fun building awesome apps! :hammer:  :whale2:  :truck:  :ship:


Fix package vs artifact naming issues

/azp run js - template

@praveenkuttappan  I'm closing this PR but you can still use it for reference when you start working through these changes.


Improvement: No-Route Elements with HasErrorParameter are not detected

As in the docs, Components with HasErrorParameter should not have any Routes.
It would be really nice if they were detected in the Mock Setup.
For e.g.
@ParentLayout(MainLayout::class)
class AccessDeniedView : VerticalLayout(), HasErrorParameter {

I did not see the addErrorRoutes! and as I do not have any idea for the detection I guess I will live with that solution for today. Thanks for helping me out!

Actually I have find this handy library which performs all kinds of neat classpath scanning: https://github.com/classgraph/classgraph

Fixed in Karibu-Testing 1.0.2


pass error when rejecting promise

Now error can be used in catch block that handles the error

@redpandatronicsuk thanks for the PR. I see you are passing the error in the reject. If you take a look on the line before reject, you will see that I am using a function called handleError. This function actually extracts all the useful part of the error, format it, and then show it to the user. If we pass the error in reject as well, it is going to show a duplicate error. One formatted and the other non-formatted.

But I think your approach is better and is more general. Would you mind editing the PR and remove the handleError part as well as removing the dependency?


Use Github Actions for CI

This PR also fixes a couple of broken tests.
Releases to PyPI are not handled yet, but that can be a different PR I think.

Lgtm!

Lgtm!


Use custom ActivityBehaviorFactory for Runtime Bundle

Requires https://github.com/Activiti/Activiti/pull/1798

@daisuke-yoshimoto Thanks for the review. Seems your PR is mainly focused about signals. Can you create a separate PR for extensibility of behaviors? Maybe based on engine changes in this PR https://github.com/Activiti/Activiti/pull/1798

@daisuke-yoshimoto I saw you created some interceptors (CloudActivityBehaviorFactoryInterceptor), is that really needed? Can't users that want to override some behaviors simply extend CloudActivityBehaviorFactory instead?

@balsarori
I think that each project needs to be able to add ActivityBehavior independently.
If we directly write logic to CloudActivityBehaviorFactory, each project cannot add ActivityBehavior independently. It is a monolithic composition. So, I made CloudActivityBehaviorFactoryInterceptor for each project to add ActivityBehavior independently.

My main concern is that this default behaviour possibly shouldn't be added just because the starter-configure is added to a project. It should only be added if starter-configure is added and the RB starter that pulls in the dependencies that include the implementation of the behaviour.
I think a way to do this would be to create an interface with a name like MQServiceTaskCloudBehavior in configure and have MQServiceTaskBehavior implement that interface. Then configure could autowire the interface (with required="false") and it will only be non-null if the implementing component has been registered (i.e. all the RB depdendencies are available).
I am also seeing the appeal of what @daisuke-yoshimoto is doing. I think it would allow users of the starter to add it to their projects and still override whatever engine behaviours they want to by implementing their own interceptor. I am not sure the two approaches are in conflict though. If we do go with the interceptor approach when it is ready we could then switch to setting the MQServiceTaskBehavior in the interceptor.

@balsarori I'm currently working to provide a similar mechanism for non-cloud projects. The idea is to use a new default behaviour not relying on spring cloud.
I was wondering if we couldn't keep the behaviour name's expression at the engine side as it was before. Just rename it from MQServiceTaskBehavior to a more general DefaultServiceTaskBehavior. Then the bean implementation will be MQServiceTaskBehavior or the new default behaviour depending whether you are using the Activiti cloud or not. What do you think?

@erdemedeiros that can be an option. However that will mean that failure will only be detected at runtime not deploy/parse time.

@balsarori true, but I thought that was already the case for java delagates. Do we have any kind of validation at deploy/parse time for those?

@erdemedeiros yeah you are right. The only thing that is done in the engine for default service task behaviour is to log a warning (in 5.x and 6.x).

@balsarori in this case do you mind to close this PR without merging it?


Add CompatBounds and increase patch number

This closes the CompatBound PRs except for OptimizationMOI which ran into precompilation problems.
I tested the CompatBounds locally.
I also increased the version number and it would be good to get a new release, because the 0.1 release is not working for PyramidScheme.jl.
closes #21 #25 #24 #20

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 18.079%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
273

Relevant Lines:
1510

💛  - Coveralls


Updates README.rst with svg badge

Hi @sobolevn -- could you explain why this is better?
It looks better, just asking in case there is some other reason than looks. :)

svg looks way better on retina displays. no other reason.

Ah okay, thanks, looks good! :+1:


Breaking Changes by Docker GUI Contracts required on Jan 1

Version
NA
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
NA
Distro Version
NA
Other Software
Docker Desktop after license change which goes into effect Jan 1.
Repro Steps
Megacorp IT sent out a nastygram today warning that the Docker Desktop GUI can no longer be used without signing a contract and paying Docker for enterprise licencing.
Expected Behavior
A way to allow toggling of Docker container access to WSL2 instances without paying an enterprise license for the Docker GUI.
Actual Behavior
I need a third party contract for the GUI feature I need - the toggle that allows WSL2 instances to access Docker containers.
Diagnostic Logs
NA

CC @tyhicks  @lramos15 - the current best path for installing WSL2 on enterprise machines is to install the VSCode remote extension. Might be some cross cutting concerns with the vscode team.

@sickcodes I don't know the strace equivalent on Windows or else I would try to reproduce with a Powershell script. Customers paying Microsoft and Apple licenses should not also need a Docker license to connect to DockerOSX from WSL2. Step 4 is what needs to be traced https://docs.microsoft.com/en-us/windows/wsl/tutorials/wsl-containers

This is a Docker issue, not WSL actionable.

Your documentation breaks on January 1 and every WSL2 user will have to reach for their wallet. This means megacorp buying more M1 laptops and less Windows11 due to the extra expensing headache. I’ll try the Rancher workaround https://jason-umiker.medium.com/replacing-docker-desktop-with-wsl2-and-or-rancher-desktop-on-windows-5e036f9dc3a4

@chadbrewbaker are you able to run Docker instances directly inside of WSL using instructions like these: https://docs.docker.com/engine/install/ubuntu/

Crazy idea! What if the system distro will act a docker server 🤯

What if the system distro will act a docker server.

Good call, we will update our docs.


fix bug: when dataset.limit equals to remote result length,nothing shows in the menu

…hows in the menu

Duplicate of #1212

dist not rebuilded

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


Fix mismatch in documentation and examples

Some more mismatch in documentation and examples that I found left over from the previous documentation.

route() should be route

https://github.com/badges/shields/blob/6e793f1159f7f9689003431b944cc0e6763014f5/doc/TUTORIAL.md?plain=1#L140
https://github.com/badges/shields/blob/6e793f1159f7f9689003431b944cc0e6763014f5/doc/TUTORIAL.md?plain=1#L147

examples() is an array

https://github.com/badges/shields/blob/6e793f1159f7f9689003431b944cc0e6763014f5/doc/TUTORIAL.md?plain=1#L274

category() should be category

https://github.com/badges/shields/blob/6e793f1159f7f9689003431b944cc0e6763014f5/doc/TUTORIAL.md?plain=1#L296

schame was defined in step 3

https://github.com/badges/shields/blob/6e793f1159f7f9689003431b944cc0e6763014f5/doc/TUTORIAL.md?plain=1#L230

I'll raise a quick PR to fix these as well.


Upgrading to mysql 8.0.19 breaks migrations

Migrations worked fine on 8.0.17. I upgraded to 8.0.19 to support Catalina and now migrations no longer work.
pymysql==0.9.3
orator==0.9.9
orator migrate -c config/database.py -f -vvv

  [KeyError]
  'int unsigned'

Exception trace:
 /Users/gthomas1/.pyenv/versions/3.7.4/envs/autotb/lib/python3.7/site-packages/cleo/application.py in run() at line 94
   status_code = self.do_run(input_, output_)
 /Users/gthomas1/.pyenv/versions/3.7.4/envs/autotb/lib/python3.7/site-packages/cleo/application.py in do_run() at line 197
   status_code = command.run(input_, output_)
 /Users/gthomas1/.pyenv/versions/3.7.4/envs/autotb/lib/python3.7/site-packages/cleo/commands/command.py in run() at line 98
   return super(Command, self).run(i, o)
 /Users/gthomas1/.pyenv/versions/3.7.4/envs/autotb/lib/python3.7/site-packages/cleo/commands/base_command.py in run() at line 146
   status_code = self.execute(input_, output_)
 /Users/gthomas1/.pyenv/versions/3.7.4/envs/autotb/lib/python3.7/site-packages/orator/commands/command.py in execute() at line 39
   return self.handle()
 /Users/gthomas1/.pyenv/versions/3.7.4/envs/autotb/lib/python3.7/site-packages/orator/commands/migrations/migrate_command.py in handle() at line 41
   migrator.run(path, pretend)
 /Users/gthomas1/.pyenv/versions/3.7.4/envs/autotb/lib/python3.7/site-packages/orator/migrations/migrator.py in run() at line 51
   self.run_migration_list(path, migrations, pretend)
 /Users/gthomas1/.pyenv/versions/3.7.4/envs/autotb/lib/python3.7/site-packages/orator/migrations/migrator.py in run_migration_list() at line 69
   self._run_up(path, f, batch, pretend)
 /Users/gthomas1/.pyenv/versions/3.7.4/envs/autotb/lib/python3.7/site-packages/orator/migrations/migrator.py in _run_up() at line 88
   migration.up()
 /Users/gthomas1/intuit_repos/auto-threat-briefing/source/migrations/2018_10_24_170702_make_columns_nullable.py in up() at line 13
   table.string('desirable').nullable().change()
 /Users/gthomas1/.pyenv/versions/3.7.4/lib/python3.7/contextlib.py in __exit__() at line 119
   next(self.gen)
 /Users/gthomas1/.pyenv/versions/3.7.4/envs/autotb/lib/python3.7/site-packages/orator/schema/builder.py in table() at line 76
   self._build(blueprint)
 /Users/gthomas1/.pyenv/versions/3.7.4/envs/autotb/lib/python3.7/site-packages/orator/schema/builder.py in _build() at line 146
   blueprint.build(self._connection, self._grammar)
 /Users/gthomas1/.pyenv/versions/3.7.4/envs/autotb/lib/python3.7/site-packages/orator/schema/blueprint.py in build() at line 29
   for statement in self.to_sql(connection, grammar):
 /Users/gthomas1/.pyenv/versions/3.7.4/envs/autotb/lib/python3.7/site-packages/orator/schema/blueprint.py in to_sql() at line 52
   sql = getattr(grammar, method)(self, command, connection)
 /Users/gthomas1/.pyenv/versions/3.7.4/envs/autotb/lib/python3.7/site-packages/orator/schema/grammars/grammar.py in compile_change() at line 225
   table_diff = self._get_changed_diff(blueprint, schema)
 /Users/gthomas1/.pyenv/versions/3.7.4/envs/autotb/lib/python3.7/site-packages/orator/schema/grammars/grammar.py in _get_changed_diff() at line 250
   self.get_table_prefix() + blueprint.get_table()
 /Users/gthomas1/.pyenv/versions/3.7.4/envs/autotb/lib/python3.7/site-packages/orator/dbal/schema_manager.py in list_table_details() at line 49
   columns = self.list_table_columns(table_name)
 /Users/gthomas1/.pyenv/versions/3.7.4/envs/autotb/lib/python3.7/site-packages/orator/dbal/schema_manager.py in list_table_columns() at line 32
   return self._get_portable_table_columns_list(table, table_columns)
 /Users/gthomas1/.pyenv/versions/3.7.4/envs/autotb/lib/python3.7/site-packages/orator/dbal/schema_manager.py in _get_portable_table_columns_list() at line 65
   column = self._get_portable_table_column_definition(table_column)
 /Users/gthomas1/.pyenv/versions/3.7.4/envs/autotb/lib/python3.7/site-packages/orator/dbal/mysql_schema_manager.py in _get_portable_table_column_definition() at line 35
   type = self._platform.get_type_mapping(db_type)
 /Users/gthomas1/.pyenv/versions/3.7.4/envs/autotb/lib/python3.7/site-packages/orator/dbal/platforms/platform.py in get_type_mapping() at line 385
   return self.INTERNAL_TYPE_MAPPING[db_type]

Update:
Adding "int unsigned": "integer",
to
https://github.com/sdispater/orator/blob/0.9/orator/dbal/platforms/mysql57_platform.py
seems to have fixed my problem.

Update:
Adding "int unsigned": "integer",
to
https://github.com/sdispater/orator/blob/0.9/orator/dbal/platforms/mysql57_platform.py
seems to have fixed my problem.
Dunno what the devs want to do with this information lol.

Hello. After adding this line, do you meet any other problems with MySQL 8.0?


?_source= search doesn't respect multi-tenancy partition

NOTE: Before filing a ticket, please see the following URL:
https://github.com/hapifhir/hapi-fhir/wiki/Getting-Help
Describe the bug
Similar issue to https://github.com/hapifhir/org.hl7.fhir.core/issues/392
Data can leak from one partition to another when using the  _source search parameter.
To Reproduce
Steps to reproduce the behavior:

Start a server with partitioning enabled

docker run -p 8080:8080 -e hapi.fhir.partitioning.allow_references_across_partitions=false -e hapi.fhir.partitioning.partitioning_include_in_search_hashes=false hapiproject/hapi:latest

Create a partition aaa

curl -d '{"resourceType": "Parameters","parameter": [{"name": "id","valueInteger": 1},{"name": "name","valueCode": "aaa"}]}' -H "Content-Type: application/json" -X POST http://localhost:8080/fhir/default/\$partition-management-create-partition

Create a partition bbb

curl -d '{"resourceType": "Parameters","parameter": [{"name": "id","valueInteger": 2},{"name": "name","valueCode": "bbb"}]}' -H "Content-Type: application/json" -X POST http://localhost:8080/fhir/default/\$partition-management-create-partition

Create a patient in aaa partition

curl -d '{"resourceType": "Patient","name": [{ "family": "Aaronson" }], "meta": { "source": "http://acme.org/aaa" }}' -H "Content-Type: application/json" -X POST http://localhost:8080/fhir/aaa/Patient

Search for all patients in bbb partition

curl http://localhost:8080/fhir/bbb/Patient
# 0 results found

Search for source=http://acme.org/aaa in bbb partition

curl http://localhost:8080/fhir/bbb/Patient?_source=http%3A%2F%2Facme.org%2Faaa
# 1 result found from `aaa` partition

Expected behavior
No results should be found, as no patient matching that search exists in the tenant.
Environment (please complete the following information):

HAPI FHIR Version, docker hub tags:v5.4.1 and latest
OS: MacOS

Additional context
I've checked most of the search parameters for Patient, and _source is the only one not working as expected.

Same issue here! Any updates on this?


Add Scroll Area Component

Describe the feature in detail (code, mocks, or screenshots encouraged)
Supports custom cross-browser styling while maintaining the browser's native scroll behavior.
I'm new to Svelte and I have worked before with Radix-UI.
It would be awesome if this component was added to bits-ui.
What type of pull request would this be?
New Feature
Provide relevant links or additional information.
https://www.radix-ui.com/primitives

Will be added once this is approved and merged https://github.com/melt-ui/melt-ui/pull/908

I think this can be closed.


Update non-windows.md

LIve response is currently available for public preview customers
https://techcommunity.microsoft.com/t5/microsoft-defender-for-endpoint/announcing-live-response-for-macos-and-linux/ba-p/2864397

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/8155

@JohanFreelancer9 Please copy review

@mjcaparas Please merge this PR. Thanks


91 where is dropped from the query

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

Comparison is base (4a7df0e) 4.89% compared to head (76f154a) 5.31%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##             master     #94      +/-   ##
===========================================
+ Coverage      4.89%   5.31%   +0.42%
- Complexity       22      23       +1
===========================================
  Files            19      19
  Lines          1185    1185
  Branches         93      93
===========================================
+ Hits             58      63       +5
+ Misses         1121    1117       -4
+ Partials          6       5       -1

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


Change archive strategie for RKI

Pls, do not create seperate tables for each day... Use this: https://stackoverflow.com/questions/26568182/is-it-possible-to-insert-add-a-row-to-a-sqlite-db-table-using-dplyr-package and append current rki table into rkiarchive table...

wouldn't it take too long to filter and load the data for a specific publication date? like: tbl(conn, "rki_total") %>% filter(rkidate==wunschdate) %>% collect()
at the moment, even loading one day takes a few minutes without the filtering: tbl(conn, "rki_2020-11-30") %>% collect()

Not if we set an index for date

Can be done via pgadmin connection to DB, I can do it if needed, you just need to clean up. I think the appropriate field is Datenstand?


Fix not parsing body_only param for sections

Currently body_only is not filled into sections properly. This is an issue because if you will use configure-sections you will loss "Release summary" section. Because there is no way how to enable body_only param.

@ferrarimarco ping

Hi! Will have a look ASAP, thanks for your patience

ping, any updates on this?

@ferrarimarco ping

@olleolleolle can you have a look at this? You know the code base better :)
Thanks!

Thanks, @dusan-dragon!


No module named 'vtt_to_srt'

I have ran
pip install vtt_to_srt3

and it said Successfully installed vtt-to-srt3-<IP_ADDRESS>.
But can't import the module. Why?

i have fixed your issue and made a pull request, you can also just clone my fork https://github.com/neelu0/vtt-to-srt.py
and just do python setup.py install while being inside the project folder, make sure uninstall the older version first, by pip uninstall vtt_to_srt3 after installing my fork you can use the program from anywhere just do vtt_to_srt /path/to/file.srt from terminal

Thanks!

Very helpful change, but now the documentation should be updated to explain that running from the command line can now be done like this:
vtt_to_srt ...

(without the .py at the end)


New subjectAttribute

Hi @UlrikeS91 @lzehl,
Could we add a subject attribute for "injected" or something similar (e.g. "treated"). This is not for drug treatment, but mainly for an injection of a tracer for example. I know we could make these things clear when adding protocols (which I will add anyway), but I think it would be beneficial to also be able to "tag" subjects with an attribute that describes that state.
Any thoughts of this?

Thanks for the quick response. I agree and I would be happy with "treated" for now

Subject attribute "treated" has now been added. Thanks @UlrikeS91


Compiler: generated builder should be renamed in integrated in generated config impl

The refactoring in necessary to implement #132

done.


Update Tax.php

Correction of the Tax Claculation during the checkout process
Issue Reference

Description
During the chackout process, the tax amount is reset.

How To Test This?
The taxes must be configured as following:

@helgvor-stoll ,
After reviewing your concern and the pull request, I kindly request that you update the screenshots to display the English locale. Once these updates are made, I will be able to reproduce the issue more effectively on our end.

@amit-webkul
Sorry, I've updated the PR. It comes from the fact that when there is no state configured on the Tax Rates settings tab as following, then the field tax_rates.state is an empty string. We can replace:
! in_array($rate->state, ['', ''])
by:
! in_array(trim($rate->state), ['', ''])


Reusing functions within scripts

This is more of a question or feature request than an issue, but I have many long javascript functions within plotly panel scripts that are duplicated between panels, is there any way to easily reuse code and avoid duplication? Maybe hosting some javascript file that can be imported, or some way to add to the built in plotly plugin functions like matchTimezone?

You can use the global window variable and assign the functions as properties like window.your_fn = your_fn after you've defined it. Then it can be used once it is declared anywhere on the page.


Fix FileHandler flaky test

Temporary PR to diagnose the flaky test as I can't reproduce this locally.
Still haven't figured out the reason why it is failing.

Codecov Report

Merging #69 (ddcad6b) into main (16f49d3) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #69   +/-   ##
=======================================
  Coverage   85.55%   85.55%
=======================================
  Files          11       11
  Lines        2125     2125
=======================================
  Hits         1818     1818
  Misses        307      307

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

I think it's failing because the parametrised tests run at different time intervals.
Maybe using freeze gun would help because it could fix the date/time that the test runs https://pypi.org/project/freezegun/

It's strange that I see they used explicit sleep instead of changing the rollover time in CPython tests. Maybe I'm just missing something. I'll check again:
https://github.com/python/cpython/blob/49cceeb5c998a51bb12b7dbde17b53647bb52113/Lib/test/test_logging.py#L5700


mu: allow gtk-free builds

The postInstall step needs to be more careful to support successful
builds that do not produce msg2pdf and mug executables.
Motivation for this change
mu should not have gtk3 and webkit as hard dependencies. Perhaps defining a mu-no-gtk variant in all-packages.nix is warranted, but this change at least lets you put,
mu = pkgs.mu.override {
      gtk3 = null;
      webkitgtk24x = null;
    };

in your config.nix to get things working again if you are on darwin or just don't want those extra components.
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

I made the change requested by @the-kenny. This means there are cp failures included in the build output now, but the result is the same.

@acowley I removed the withMug flag because it was seemingly unnecessary; I wasn't aware that the webkit parts didn't build on mac.  As far as I am concerned, feel free to add back the withMug flag, just make sure to enable it by default on linux.

Even if everything built, it's a pretty big pair of dependencies to pull in if you don't need that functionality (I use mu almost exclusively through emacs, for example). I think the flag is the right thing, and I am fine with having it default to stdenv.isLinux if that seems reasonable.

I put the withMug flag back, but default it to stdenv.isLinux. Hopefully that does the right thing for most people.


Huggingface webcam option locally.

Hello, I've noticed in the huggingface demo, it asks for my webcam permission, and I have an option to select my webcam in the dropdown, however, I do not seem to have this option with the cloned repo, is there a way to add this feature locally?

I have the same question

This answer helped me:
https://stackoverflow.com/a/57376955

hi all, yes that happened to me as well, specially testing on Safari IOS, the way I solved it was to serve the server using a self-signed SSL
openssl req -newkey rsa:4096 -nodes -keyout key.pem -x509 -days 365 -out certificate.pem
uvicorn "app-img2img:app" --host <IP_ADDRESS> --port 7860 --reload --log-level info --ssl-certfile=certificate.pem --ssl-keyfile=key.pem

I'm encountering an issue when attempting to run the application locally on Firefox; the webcam doesn't appear in the selection options, indicating it's not recognized or initialized by the application.

Running this command:
uvicorn "app-img2img:app" --host <IP_ADDRESS> --port 7860 --reload

hi @Limbicnation , I just tested it locally on Firefox, and you might need to issue a self signed SSL to serve the app with HTTPs,
openssl req -newkey rsa:4096 -nodes -keyout key.pem -x509 -days 365 -out certificate.pem
uvicorn "app-img2img:app" --host <IP_ADDRESS> --port 7860 --reload --log-level info --ssl-certfile=certificate.pem --ssl-keyfile=key.pem

then go to https://localhost:7860

another solution here
https://github.com/radames/Real-Time-Latent-Consistency-Model/issues/17#issuecomment-1811957196


Additive glow is very strong.

Operating system or device, Godot version, GPU Model and driver (if graphics related):
Fedora 64 bits, Nvidia GTX prorietary drivers v. 384.90 ,  Godot a931f905f
Issue description:

The additive glow is very strong, even with default parameters, i dont remember it being so strong
here is a comparison between current master and alpha 1
MASTER

ALPHA 1

Steps to reproduce:
use additive glow
Link to minimal example project:

Not sure if something was fixed, but in general it is meant to be strong
On Oct 13, 2017 7:55 PM, "Daniel Ramirez"<EMAIL_ADDRESS>wrote:

Operating system or device, Godot version, GPU Model and driver (if
graphics related):
Fedora 64 bits, Nvidia GTX prorietary drivers v. 384.90 , Godot a931f90
https://github.com/godotengine/godot/commit/a931f905f577e6cf19adb3ea0cf4943e27c047bd
Issue description:
The additive glow is very strong, even with default parameters, i dont
remember it being so strong
here is a comparison between current master and alpha 1
MASTER
[image: screenshot from 2017-10-13 17-52-52]
https://user-images.githubusercontent.com/1103897/31569314-84439b6e-b03f-11e7-9167-e2af882c325b.png
ALPHA 1
[image: screenshot from 2017-10-13 17-53-36]
https://user-images.githubusercontent.com/1103897/31569316-8a911dca-b03f-11e7-90ca-904d8e4545c8.png
Steps to reproduce:
use additive glow
Link to minimal example project:
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/12082, or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z26Ht4fzuhNu_4R2luas94o85vUk6ks5sr-p0gaJpZM4P5JD0
.

@reduz but it is really strong, at the point of being unusable with most parameters, befora it was nice, here is another example
, with this parameters

it used to be more controlable, and switching between modes wasnt that drastic.

I did a fix recently because it was done in linear space, while it should have been done in SRGB. It's more correct now, and it's expected to be like that, Decreasing stength should help for this mode.


Eslint formatting

Adds eslint formatting and removed deprecated tslint
I left husky and lint-staged inside even though we do not really use them. If we want to, we should also add prettier and eslint-prettier...
Fixes #260

closed in favour of #275


feat: document nRF Cloud Cell Geolocation Service

See https://github.com/NordicSemiconductor/asset-tracker-cloud-docs/discussions/9

Sorry @peknis for re-request.

Sorry @peknis for re-request.

No worries. We were both quicker than a greased lightning.


glbic_2.29

hi this is my problem
/home/fhsbs/.virtualenvs/pythonProject2/bin/python /tmp/pycharm_project_762/depth_camera.py
Traceback (most recent call last):
File "/tmp/pycharm_project_762/depth_camera.py", line 4, in
import pyrealsense2 as rs
File "/home/fhsbs/.local/lib/python3.9/site-packages/pyrealsense2/init.py", line 2, in
from .pyrealsense2 import *
ImportError: /lib/aarch64-linux-gnu/libm.so.6: version `GLIBC_2.29' not found (required by /home/fhsbs/.local/lib/python3.9/site-packages/pyrealsense2/pyrealsense2.cpython-39-aarch64-linux-gnu.so)
Process finished with exit code 1
i try to run simple python code
python version 3.9.12
pyrealsense2  <IP_ADDRESS>17
i do the download from the clone (ask chat gpt for the download)
did someone can help me
(got ld515 cam)

Hi @naorwaiss  This error has not previously been reported in regard to the pyrealsense2 wrapper.
As aarch64 is referenced, it looks as though your computer / computing device is based on the Arm hardware architecture.  So that means that pyrealsense2 has to be built from source code, as the pip install pyrealsense2 package install method will not work on Arm.
Which computer / computing device are you using, please?
pyrealsense2 can be installed from source with CMake at the same time as building the librealsense SDK or afterwards.
https://github.com/IntelRealSense/librealsense/issues/6964#issuecomment-707501049 has a guide for installing librealsense and pyrealsense2 together on Nvidia Jetson (an Arm-based board) that you may be able to adapt for your own compuer if you are not using Jetson.
In the -DPYTHON_EXECUTABLE instruction, change python3.6 to python3.9.
Alternatively, the method for installing pyrealsense2 after librealsense has already been installed is described in the link below.
https://github.com/IntelRealSense/librealsense/blob/master/wrappers/python/readme.md#building-from-source

hi, i am using jetson nano

The https://github.com/IntelRealSense/librealsense/issues/6964#issuecomment-707501049 guide should be ideal for you then as it was written particularly for Jetson Nano (though it works with most Jetson board models).

hi... its didnt work..
i try use his version but didnt work
when i use pip instal pyrealsese the glib is back
when i didnt do the pip install and do the install manualy like what u send the python didnt recugnize the pyrealsense
thanks alot

I noe that you are using pycharm.  There is a past case at https://github.com/IntelRealSense/librealsense/issues/4089 where a Nano user with pycharm experienced the error ModuleNotFoundError: No module named 'pyrealsense2' when, like you, they simply ran the line the pyrealsense2 import instruction.  Another RealSense user in that case suggested at https://github.com/IntelRealSense/librealsense/issues/4089#issuecomment-496988560 to edit the bashrc file.

o try it also.
let me understand.. when i do the download of the realsense in the command i need to do also pip install librealsense2?
or when i do all the donlwad: cmake, SDK, etc at the end i need make path drom the python to the librealsense witouth the pip install?

pip install pyrealsense2 cannot be used on Jetson Nano as this installation method is not compatible with Arm devices such as Nvidia Jetson.  It should not be used at all on Nano.

ok have some step foword today
/home/fhsbs/.virtualenvs/pythonProject2/bin/python /tmp/pycharm_project_260/depth_camera.py
Traceback (most recent call last):
File "/tmp/pycharm_project_260/depth_camera.py", line 13, in
pipeline = rs.pipeline()
AttributeError: module 'pyrealsense2' has no attribute 'pipeline'
Process finished with exit code 1
now this problem is rising

Does the error still occur if you input the lines below into the terminal?
export PATH=$PATH:~/.local/bin
export PYTHONPATH = $PYTHONPATH:/usr/local/lib/python3.9/pyrealsense2

Instead of using import pyrealsense2 as rs, is the problem corrected if you instead use this:
import pyrealsense2.pyrealsense2 as rs

thanks bro its help i got camera and depth with the pyrealsense2

You are very welcome.  That's great to hear that you were successful!

Hi @naorwaiss  Do you require further assistance with this case, please?  Thanks!

Case closed due to no further comment received.


1.20.5 paper is out!

https://pastebin.com/t6H5MnRD  - 1.20.5 plugin error

1.20.5 has just come out, please allow us some time so we can update the plugin.
judging by the error provided, there will be another error beforehand, which I will assume to be PlaceholderAPI failing to load at startup
As DeluxeMenus depends on PlaceholderAPI, DeluxeMenus will fail to enable.
We are aware of the issue with PlaceholderAPI and there is already a a PR to fix this

1.20.6 is available.

1.20.5 has just come out, please allow us some time so we can update the plugin.
judging by the error provided, there will be another error beforehand, which I will assume to be PlaceholderAPI failing to load at startup As DeluxeMenus depends on PlaceholderAPI, DeluxeMenus will fail to enable. We are aware of the issue with PlaceholderAPI and there is already a a PR to fix this

The PR has been merged, will you be able to update it?

Dev builds are available for PlaceholderAPI with the fix
https://ci.extendedclip.com/job/PlaceholderAPI/193/
Let me know if this helps so we can close this issue

Dev builds are available for PlaceholderAPI with the fix https://ci.extendedclip.com/job/PlaceholderAPI/193/
Let me know if this helps so we can close this issue

After installing the PlaceholderAPI dev build and restarting the server, it seems functional! No errors in the console and the menus seem to work correctly in game.

head is not work?
Do you have this bug?

I am closing this issue since the original creator has not responded in 3 weeks. We also have a confirmation of this being fixed on the latest development versions of PlaceholderAPI
@addpromax either create a new issue or ask in Discord: https://helpch.at/discord. When you do, also share more info such as your menu, your server version, your DeluxeMenus version, etc.


Need proxy follows the system

I need a feature similar to PyCharm's that lets the proxy follow the system when I am using an external VPN or something
The proxy in VSCode will only go through what you set in the proxy setting, but I need to change the proxy server from time to time, it is complicated to change the setting whenever I change the proxy server

It should use the proxy configured in your OS if you don't set one in the VS Code user settings. Try reloading the VS Code window or just restart VS Code after changing the proxy in the OS. Does that work?

It should use the proxy configured in your OS if you don't set one in the VS Code user settings. Try reloading the VS Code window or just restart VS Code after changing the proxy in the OS. Does that work?

It should use the proxy configured in your OS if you don't set one in the VS Code user settings. Try reloading the VS Code window or just restart VS Code after changing the proxy in the OS. Does that work?

No, it does not work.
The "npm install" still fails due to the network issue while my outside proxy works fine on browsing websites

I don't know if there is a way to make npm use the proxy configured in the OS, you might have to configure it separately. VS Code doesn't change npm's configuriation.


Move to a shared   tag between different "Main" vs "Home" pages

The top right nav icons are not changing their styles when moving between home and main:


⚠ Bofrost.nl has degraded performance

In b1f48a2, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 920 ms

Resolved: Bofrost.nl performance has improved in 4227382.


(MAINT) Fix acceptance puppet_git load path

Prior to this, running the acceptance tests locally without specifying a specific load path would result in a require failure if attempting to test against puppet installed via git.
/cc @er0ck

For what it's worth, running locally I consistently get a failure that seems unrelated to the load path problem: see https://gist.github.com/rick/0c0bc2ec5c855cc5c5b3

hmmm... so i'm already hitting the limits of this solution.  eg: tests/foss_utils/clone_git_repo_on.rb would also need this patch. this leads me back to.. an options file.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/1357/
Test FAILed.

Yeah, I don't have a problem with getting the load path right for any tests that need it. In situations where there's something like a test_helper.rb or a bundler dep, or a Rakefile that are always run prior to tests, etc., lifting up is fine, but I'm used to seeing load path work happening in multiple files otherwise.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/1360/
Test FAILed.

Unrelated to this patch, but I would prefer if foss_utils lived in a slightly different directory structure:
acceptance/tests/dsl/foss_utils

good idea, alice.  hell, why not just jumble a whole slew of unrelated fixes in here :-)

Fine, fine.  I can file a ticket for that.  But I don't like you much for making me do that.

@anodelman i didnt' mean to be facetious.. i'm about to push to this branch ...
but shouldn't that be dsl/install_utils/foss_utils/ ?

Tests are passing for me locally on this branch now.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/1363/
Test PASSed.

:bow:


neighborhood scores?

Hi! The paper mentions that REMEDI was also evaluated on the "neighborhood" benchmark for CounterFact, but I seem to only see  "efficacy", "paraphrase", "generation", and "essence" as the supported benchmarks. How would one go about reproducing the neighborhood scores as in Table 3 of the paper?

That's right, there is no neighborhood score code in this repo.
For ROME/FT, we computed it using the ROME code: https://github.com/kmeng01/rome/blob/main/experiments/py/eval_utils_counterfact.py#L74
For REMEDI/prompting methods, we didn't explicitly compute it because they do not touch neighboring prompts :)
Hope this helps!


fix(computational_graph): Don't descend into quoted expressions

coverage: 87.314% (+0.2%) from 87.137% when pulling b7b9770cd3d0f91a7b7d137a91ba0850d4aba002 on dont_recurse_quoted_exprs into 75b36bf11daa5fd2e9bd11d9ee403a382e6b974c on main.

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.02 :tada:

Comparison is base (75b36bf) 87.41% compared to head (b7b9770) 87.43%.

: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.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #21      +/-   ##
==========================================
+ Coverage   87.41%   87.43%   +0.02%
==========================================
  Files          12       12
  Lines        1009     1011       +2
==========================================
+ Hits          882      884       +2
  Misses        127      127

Impacted Files
Coverage Δ

src/computational_graph.jl
85.36% <100.00%> (+0.14%)
:arrow_up:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Improvements for the mobile editor

Describe the Issue
### Tasks
- [ ] Replace the top breadcrumb on mobile with a small spacer and place the simple tunicated breadcrumb on the top of the title with foreground-quaternary color
- [ ] Remove portal button
- [ ] Turn the code button into a dropdown for files.

Screen shot

Additional context
Are there any usability implications that comes to mind?

I like these changes. Moving the breadcrumbs to the upper jaw and getting rid of the individual file buttons will definitely leave more room for the editor when zoomed in to 400% (which has been a concern of mine for quite a while).
Concerning the breadcrumbs, we need to make sure the color contrast with the background is at least 4.5:1.
The other issue here depends on what you mean by "turn the code button into a dropdown." The code button is already an active tab control, so programmatically, it can't have any other functional elements inside of it. Thus, any type of dropdown menu would have to be located within the editor tab panel in the DOM, but you could use CSS to move it up so it visually looks like it is inside of the code tab. The majority of the code button would remain the tab control and the toggle triangle would be the only active hit area to trigger the dropdown, in which case you might want to move it all the way to the right in the code tab. You might be able to get away with making the hit area for the menu toggle cover the entire code tab, thus causing the dropdown to open every time you mouse/touch click the code tab, but that might also be annoying.

I want to work on this issue . can you please assign it to me

Thanks for the feedback.
Would turning the tabs into a button group and turning one of the buttons into a dropdown help with resolving the accessibility issue?
example:
https://getbootstrap.com/docs/4.0/components/button-group/#nesting
or:
https://www.freecodecamp.org/learn/front-end-development-libraries/bootstrap/use-responsive-design-with-bootstrap-fluid-containers

Also, how about completely removing the nested tabs when there is only one file?
We could have the first line above the editor to be the name of the file. Once the user scroll, it would move with the editor and go out of the view.

Are these still relevant @ahmaxed @freeCodeCamp/mobile?


Set the read_version property if not already set

Fixes #107
It's not 100% clear to me why this property is special. It looks like the upstream change between 2.* and 3.* changed from the pulumi provider needing to set "latest_version" to and/or in addition to "read_version".
I've added a test for updating a akamai.Property to make sure we don't regress again.

Tested this on 4.1.0-alpha.1674599843 in npm. Fixed the issues I've been having 👏


tests: Fail fast on drivers' construction errors

Running tests locally with go test ./... was causing several panics due
to failing DNS queries for the locally set hostname ("lookup senart: no such host")
The assertion library being used wasn't failing fast in presence of such errors
being returned by driver constructors. This resulted in nil drivers
being accessed and hence causing panics.
This change set makes these constructors abort the tests immediately instead of
delaying the error reporting.

LGTM - thanks for the PR


Upgrade to version 5.3.1f1

Upgraded to 5.3.1, this consisted of:

Unity auto-generating new required files (such as ProjectSettings/ClusterInputManager.asset)
Fixing one script to use the new SceneManagement api instead of the old deprecated one.

Also I decided to finally upgrade the entire project to force serialization to text instead of 'mixed' to aid in merges (and make it consistent).  All of the binary files shown below have been touched because they are now being serialized as text.  In the future if they are modified, they will be merge-able instead of just a direct binary conflict.
@jo3w4rd
@protodeep
@codemercenary
@veronicaz41

Will be addressed in another PR


Require repo scope for PATs for private repos and basic authentication

The scoped token PR just checked all API routes but in fact, some web routes like LFS, git HTTP, container, and attachments supports basic auth. This PR added scoped token check for them.

This PR breaks my LFS client.
Reverting these 2 "lfs/*.go" files , then  my client works again.

gitea-app_1     | 2023/04/27 20:37:14 ...rvices/lfs/server.go:547:authenticate() [W] [644a6c7a-2] Authentication failure for provided token with Error: invalid token claim
gitea-app_1     | 2023/04/27 20:37:14 [644a6c7a-2] router: completed GET /org/repo.git/info/lfs/objects/xxxxxxxxxxxxxxxxxxxxxxxxxx for <IP_ADDRESS>:0, 401 Unauthorized in 5.9ms @ lfs/server.go:80(lfs.DownloadHandler)


Mark multiple habits at the same time

rich mark x
rich mark y
rich mark z

to
rich mark x y z
Sure, you could easily wrap rich in a shell script to get this behaviour, but I think it's better to implement this here. The thing rich is supposed to do is "track habits", which is kind of subjective and ambiguous. In short, how can you say that tracking habits ends here, and not later? I'm trying to use somewhat reasonable judgement here. Having this typing shortcut would help virtually every rich user, so it's not bloat

https://github.com/michaelskyba/rich/commit/cb6c527fdfe0a04e5ff7459a37a1a0fb1c35ee72


Some queries are returning very unusual responses

Given the following query:
curl -X 'POST' \
  'https://cqs.ci.transltr.io/query' \
  -H 'accept: application/json' \
  -H 'Content-Type: application/json' \
  -d '{
  "message": {
    "query_graph": {
      "nodes": {
        "sn": { "categories": ["biolink:ChemicalEntity"] },
        "on": { "categories": ["biolink:Disease"], "ids": ["MONDO:0007982"] }
      },
      "edges": {
        "t_edge": {
          "subject": "sn",
          "object": "on",
          "predicates": ["biolink:treats"],
          "knowledge_type": "inferred"
        }
      }
    }
  }
}' > cqs_response.json

It finishes very quickly, has a single knowledge_graph node, no knowledge_graph edges, and results are missing.

I ran this locally and there were no knowledge_graph.edges and one knowledge_graph.node in the response.  I suspected this was due to some merging problem, but that is exactly how the knowledge_graph looked from MoleProChembl.  All other KPs returned no knowledge_graph data.  There were no message.results.  I did, however, change the code to make message.results a Some(vec![]) instead of a None by default.

This is being sent to the WFR to target MolePro:
{"workflow":[{"id":"lookup","runner_parameters":{"allowlist":["infores:molepro"]}},{"id":"score"}],"message":{"query_graph":{"edges":{"e00":{"predicates":["biolink:in_clinical_trials_for"],"subject":"n00","object":"n01"}},"nodes":{"n00":{"categories":["biolink:ChemicalEntity"]},"n01":{"ids":["MONDO:0007982"],"categories":["biolink:DiseaseOrPhenotypicFeature"]}}}}}
And this is the Response per this issue:
{
"workflow": [
{
"id": "lookup",
"runner_parameters": {
"allowlist": [
"infores:molepro"
]
}
},
{
"id": "score",
"runner_parameters": {}
}
],
"message": {
"query_graph": {
"edges": {
"e00": {
"predicates": [
"biolink:in_clinical_trials_for"
],
"subject": "n00",
"object": "n01",
"attribute_constraints": [],
"qualifier_constraints": []
}
},
"nodes": {
"n00": {
"categories": [
"biolink:ChemicalEntity"
],
"set_interpretation": "BATCH",
"constraints": []
},
"n01": {
"ids": [
"MONDO:0007982"
],
"categories": [
"biolink:DiseaseOrPhenotypicFeature"
],
"set_interpretation": "BATCH",
"constraints": []
}
}
},
"knowledge_graph": {
"edges": {},
"nodes": {
"MONDO:0007982": {
"name": "metaphyseal chondrodysplasia, Jansen type",
"categories": [
"biolink:BiologicalEntity",
"biolink:Disease",
"biolink:DiseaseOrPhenotypicFeature",
"biolink:NamedThing",
"biolink:ThingWithTaxon"
],
"attributes": [
{
"attribute_type_id": "biolink:has_numeric_value",
"original_attribute_name": "information_content",
"value": 100.0,
"value_type_id": "EDAM:data_0006"
},
{
"attribute_type_id": "biolink:same_as",
"original_attribute_name": "equivalent_identifiers",
"value": [
"MONDO:0007982",
"DOID:0080020",
"OMIM:156400",
"orphanet:33067",
"UMLS:C0265295",
"MESH:C537564",
"NCIT:C131868",
"SNOMEDCT:24629003"
],
"value_type_id": "EDAM:data_0006"
}
],
"is_set": null
}
}
}
},
"logs": []
}

When I send this query directly to Molepro, it does only include the one knowledge graph node (which is a little unusual, but that node is the node that was asked for, so not really an issue), and they do return an empty results list. So the issue is looking like it's coming from either WFR or CQS. The fix by @jdr0887 mentioned above should take care of this in CQS, but I'll also look to see what's going on in WFR.

There is an issue/nuance in WFR where the empty results are being returned as None. This is valid TRAPI and not a validation issue (though just unexpected and we're going to change it to be an empty list), but the CQS should still return an empty results list regardless as well.

Closing since this is resolved.


Error: Can't find file DDNet_500Model.pt

Hi, I got an error during the training of your model "EfficientSOD". The error is "Can't find file DDNet_500Model.pt". How to resolve this issue? Thank you.

Hello, I assume this error would be generated from your TESTING code, where you must need the trained model (namely DDNet_500Mode.pt in general case) in the corresponding folder i.e., TrainedModels.
Once you've this trained model, the testing code will run smoothly.

Thanks for the reply, but this error is generated at the end of training.

Can you please post your training code here?
Make sure that you have "TrainedModels" folder in the same directory where your training.py code exists.

import torch
from torch import nn, cuda
from DataGenerator import DatasetLoader
from ModelNetworks import BaseNetwork_3
from torch.utils.data import Dataset, DataLoader
import torch.backends.cudnn as cudnn
import matplotlib.pyplot as plt

def count_parameters(model):
    return sum(p.numel() for p in model.parameters() if p.requires_grad)

def train(model, opt, crit, train_loader, epoch):
    model.train()
    for i, (X, Y) in enumerate(train_loader):
        X = X.to('cuda')
        Y = Y.to('cuda')

        output = model(X)

        loss = crit(output, Y) # this is for loss comparison

        opt.zero_grad()
        loss.backward()
        opt.step()

    return loss.item()

def main():

    print('__Number of CUDA Devices:', cuda.device_count(), ', active:', cuda.current_device())
    print ('Device name: .... ', cuda.get_device_name(cuda.current_device()), ', available >', cuda.is_available())

    model = BaseNetwork_3.DenseNetBackbone()

    cudnn.benchmark = True
    model.to('cuda')
    print(count_parameters(model))

    base_lr = 0.0001
    epochs = 100
    weight_decay = 1e-3
    k = 0
    total_loss = []

    optimizerr = torch.optim.Adam(model.parameters(), lr=base_lr, weight_decay=weight_decay, betas=(0.9, 0.95))
    criterion = nn.MSELoss().to('cuda')

    print('Model on GPU: ', next(model.parameters()).is_cuda)

    dataset_path = r'D:\My Research\Video Summarization\VS via Saliency\SIP'
    d_type = ['Train', 'Test']

    train_data = DatasetLoader(dataset_path, d_type[0])
    train_loader = DataLoader(train_data, batch_size=48, shuffle=True, num_workers=16, drop_last=True)

    test_data = DatasetLoader(dataset_path, d_type[1])
    for epoch in range(0, epochs):

        current_loss = train(model, optimizerr, criterion, train_loader, epoch)
        if epoch%4 == 0:
            print("Epoch: %d of: %d, loss: %f"%(epoch,epochs, current_loss))
        total_loss.append(current_loss)

    torch.save(model, 'TrainedModels\\DDNet_500Model.pt')
    torch.save(model.state_dict(), 'TrainedModels\\DDNet_500Weights.pt')

if __name__ == '__main__':
    main()